Warning msg: The process cannot access the file because it is being used by another process - when killing TagUI prematurely with Ctrl+C

@kensoh, as mentioned in #417, I pressed Ctrl-C a lot during testing and debugging of TagUI scripts.
In Windows 10, when I kill a TagUI script using Ctrl-C and immediately re-run the script, a lot of times I will get this warning message:

The process cannot access the file because it is being used by another process.

Although the error message appears, the script actually can still run properly.  However, in the windows task bar at the bottom, I will see additional command windows that are leftover of the previous session (which are not closed properly).
I would like to write some scripts (using DOS) to delete away these "leftover" processes and command windows (as a result of pressing Ctrl-C). Want to do this because users are always worried when they see these error messages.
I noticed that when we run a tagui script, there is always a second command window that will also appear - that contains debugging as well as Sikuli info.
May I know in the error message:

The process cannot access the file because it is being used by another process

exactly which file is that?
Or if you have any suggestion how to kill those additional command windows which are leftover from the previous session?
p.s. to replicate this error message on Windows 10, simply run a TagUI script with some LIVE command in it. When the script pauses, kill the script using Ctrl-C. Then re-run the script again. Do this a couple of times and you will see the same error message, as well as more and more command window in the task bar.

Hi CK, thanks for raising this and the detailed replication steps!
This has been resolved in https://github.com/kelaberetiv/TagUI/issues/417 - where I created the scripts to kill dead processes (shared with you over email this solution on 2 May). Thereafter, the 2 scripts are committed in the cutting edge release.
For which file, it depends on which integrations are being called (eg SikuliX or just Chrome). There may also be other leftover files from breaking the script halfway during execution. TagUI has no access to clear as control is lost when Ctrl+C is used to prematurely terminate it.
macOS / Linux
https://github.com/kelaberetiv/TagUI/blob/master/src/end_processes
Windows
https://github.com/kelaberetiv/TagUI/blob/master/src/end_processes.cmd

Adding on, besides trying to clean the dead processes due to Ctrl+C, what are the use cases for users having to break TagUI prematurely? Maybe there is some other ways to handle such use cases instead.

This has been resolved in #417 - where I created the scripts to kill dead processes (shared with you over email this solution on 2 May). Thereafter, the 2 scripts are committed in the cutting edge release.

I raised this issue again because the methods in #417 didn't work for me. I still continue to get the errors

The process cannot access the file because it is being used by another process.

Adding on, besides trying to clean the dead processes due to Ctrl+C, what are the use cases for users having to break TagUI prematurely? Maybe there is some other ways to handle such use cases instead.

Such errors not only occur when users press Ctrl+C, it also occurs when there are errors in the TagUI script and the program stops with errors. The effect is the same as Ctrl-C, and when you re-run the script, you will see the same error:

The process cannot access the file because it is being used by another process

When developing a robot, it's very common to make mistakes and such errors are very common.

I see, for normal TagUI script error, can you give a replication scenario of error in TagUI script and this happens? It may be caused by another root cause that is not yet known.
Because the design is such that if errors happens in TagUI script, control will still be passed back to tagui.cmd batch file to do the cleaning up of background processes. But for Ctrl+C, control is lost immediately without being returned to tagui.cmd to do clean-up.

For Ctrl+C, I tried 10x with default PhantomJS mode without issues. For chrome option, I sometimes can get the process cannot access the file message. I notice when that happens when the Chrome integration process is still running and not closed by TagUI as Ctrl+C is used to kill it.
When I see the Chrome integration process command window still open after Ctrl+C, and I run end_processes.cmd, this window will close and when I run TagUI again there is no error message.
I can't replicate the case where running end_processes.cmd don't shut off that Chrome integration command window. Can you try restarting computer and each time after you Ctrl+C, run the end_processes.cmd? The best guess I have is maybe there are multiple dead processes in memory which the end_processes.cmd somehow only triggered closure of 1 but not all the dead processes killed before.

Adding more technical notes for internal reference
After checking through TagUI's engines for handling integration processes (Chrome, Python, R, SikuliX), I found that only one of them has a potentially blocking code that could be the only point of weakness where the end_processes script can't kill. That is in tagui_chrome.php.
In that engine, when TagUI script has an action to interact with Chrome, TagUI sends an instruction to Chrome and then waits until Chrome responds back to acknowledge. This is a blocking transaction within the integration engine without timeout and is by design.
As it does not make sense to continue sending a next instruction when Chrome has not responded to the first one. In normal UI automation, sequential execution is very important and each step often has dependency on the success of the previous action.
while ($intent_result_string == "") {try {$intent_result_string = trim($client->receive());} catch (Exception $e) {}}

I couldn't imagine a scenario where Chrome does not respond. Otherwise, a breaker can be inserted above to check for the 'finish' signal from TagUI before looping. But doing so requires adding a sleep in order not to rapidly read from disk and consume even more CPU power.
However, that will result in slower performance for normal transactions between TagUI and Chrome. For eg, a flow with 10 actions may now take 10 seconds longer to complete, due to the additional overhead. Thus, without concrete evidence of benefit of inserting this breaker it would not be worth considering tackling from this angle.

Though above finding and scenario is not replicable from my side, I think there should be a more robust end_process / end_processes.cmd that can kill any lingering processes from TagUI and friends. Will see if I can come up with something and update here again!

Fyi CK, made a commit with following comments. Have a try! They are pretty aggressive and kill all running processes of TagUI and integration friends (SikuliX, Python, R, Chrome) and browser.
Scripts now available in cutting edge release - https://github.com/kelaberetiv/TagUI#set-up

Making a commit to improve on the existing script to kill TagUI processes.
For eg if Ctrl+C is used to kill TagUI prematurely, TagUI main process would lose the control to do clean-up of integrations (SikuliX, R, Python, Chrome) and Chrome browser.
This shell script (for all the 3 OSes) will kill those processes to free up memory and allows TagUI to start running from a clean state.
These actions are not done by default when TagUI is launched, as it is expected that TagUI exits cleanly. Otherwise cleaning up all processes by default will hide potential issues should they arise and they will end up not getting reported.

Below are contents of the scripts for Windows and macOS/Linux respectively -
end_processes.cmd
@echo off
rem # TO MANUALLY KILL TAGUI PROCESSES, EG IF CTRL+C WAS USED TO QUIT TAGUI ~ TEBEL.ORG #

rem aggressive failsafe for tagui and friends that have not exited gracefully
rem works by scanning processes for tagui keywords and killing them one by one

rem set path to unix utilities for windows command prompt
if exist "%~dp0unx\gawk.exe" set "path=%~dp0unx;%path%"

:repeat_kill_php
for /f "tokens=* usebackq" %%p in (`wmic process where "caption like '%%php.exe%%' and commandline like '%%tagui_chrome.php%%'" get processid 2^>nul ^| cut -d" " -f 1 ^| sort -nur ^| head -n 1`) do set php_process_id=%%p
if not "%php_process_id%"=="" (
    taskkill /PID %php_process_id% /T /F > nul 2>&1
    goto repeat_kill_php
)

:repeat_kill_chrome
for /f "tokens=* usebackq" %%p in (`wmic process where "caption like '%%chrome.exe%%' and commandline like '%%tagui_user_profile --remote-debugging-port=9222%%'" get processid 2^>nul ^| cut -d" " -f 1 ^| sort -nur ^| head -n 1`) do set chrome_process_id=%%p
if not "%chrome_process_id%"=="" (
    taskkill /PID %chrome_process_id% /T /F > nul 2>&1
    goto repeat_kill_chrome
)

:repeat_kill_sikuli
for /f "tokens=* usebackq" %%p in (`wmic process where "commandline like '%%tagui.sikuli%%'" get processid 2^>nul ^| cut -d" " -f 1 ^| sort -nur ^| head -n 1`) do set sikuli_process_id=%%p
if not "%sikuli_process_id%"=="" (
    taskkill /PID %sikuli_process_id% /T /F > nul 2>&1
    goto repeat_kill_sikuli
)

:repeat_kill_python
for /f "tokens=* usebackq" %%p in (`wmic process where "commandline like '%%tagui_py.py%%'" get processid 2^>nul ^| cut -d" " -f 1 ^| sort -nur ^| head -n 1`) do set python_process_id=%%p
if not "%python_process_id%"=="" (
    taskkill /PID %python_process_id% /T /F > nul 2>&1
    goto repeat_kill_python
)

:repeat_kill_r
for /f "tokens=* usebackq" %%p in (`wmic process where "commandline like '%%tagui_r.R%%'" get processid 2^>nul ^| cut -d" " -f 1 ^| sort -nur ^| head -n 1`) do set r_process_id=%%p
if not "%r_process_id%"=="" (
    taskkill /PID %r_process_id% /T /F > nul 2>&1
    goto repeat_kill_r
)

:repeat_kill_tagui
for /f "tokens=* usebackq" %%p in (`wmic process where "executablepath like '%%\\tagui\\src\\%%' and not caption like '%%cut.exe%%' and not caption like '%%sort.exe%%' and not caption like '%%head.exe%%'" get processid 2^>nul ^| cut -d" " -f 1 ^| sort -nur ^| head -n 1`) do set tagui_process_id=%%p
if not "%tagui_process_id%"=="" (
    taskkill /PID %tagui_process_id% /T /F > nul 2>&1
    goto repeat_kill_tagui
)

end_processes
#!/usr/bin/env bash
# TO MANUALLY KILL TAGUI PROCESSES, EG IF CTRL+C WAS USED TO QUIT TAGUI ~ TEBEL.ORG #

# aggressive failsafe for tagui and friends that have not exited gracefully
# works by scanning processes for tagui keywords and killing them one by one

while true; do
    php_process_id="$(ps -x | grep tagui_chrome\.php | grep -v 'grep tagui_chrome\.php' | sed -e 's/^[ ]*//' | cut -d' ' -f 1 | sort -nur | head -n 1)"
    if [ -n "$php_process_id" ]; then
        kill $php_process_id > /dev/null 2>&1
    else
        break
    fi
done

while true; do
    chrome_process_id="$(ps -x | grep remote-debugging-port=9222 | grep tagui_user_profile | sed -e 's/^[ ]*//' | cut -d' ' -f 1 | sort -nur | head -n 1)"
    if [ -n "$chrome_process_id" ]; then
        kill $chrome_process_id > /dev/null 2>&1
    else
        break
    fi
done

while true; do
    sikuli_process_id="$(ps -x | grep tagui\.sikuli | grep -v 'grep tagui\.sikuli'| sed -e 's/^[ ]*//' | cut -d' ' -f 1 | sort -nur | head -n 1)"
    if [ -n "$sikuli_process_id" ]; then
        kill $sikuli_process_id > /dev/null 2>&1
    else
        break
    fi
done

while true; do
    python_process_id="$(ps -x | grep tagui_py\.py | grep -v 'grep tagui_py\.py'| sed -e 's/^[ ]*//' | cut -d' ' -f 1 | sort -nur | head -n 1)"
    if [ -n "$python_process_id" ]; then
        kill $python_process_id > /dev/null 2>&1
    else
        break
    fi
done

while true; do
    r_process_id="$(ps -x | grep tagui_r\.R | grep -v 'grep tagui_r\.R'| sed -e 's/^[ ]*//' | cut -d' ' -f 1 | sort -nur | head -n 1)"
    if [ -n "$r_process_id" ]; then
        kill $r_process_id > /dev/null 2>&1
    else
        break
    fi
done

while true; do
    tagui_process_id="$(ps -x | grep /tagui/src | grep -v 'grep /tagui/src'| sed -e 's/^[ ]*//' | cut -d' ' -f 1 | sort -nur | head -n 1)"
    if [ -n "$tagui_process_id" ]; then
        kill $tagui_process_id > /dev/null 2>&1
    else
        break
    fi
done

Updated with a commit to filter away WMIC process on Windows to prevent hanging. Works now! 😓

Committed further improvements to macOS and Linux end processes script above -

Hopefully this aggressive kill script works for you now, CK. In addition, do let me know if you have replication steps where TagUI has below error message (due to dead processes), for situations where TagUI exits normally or run into errors and exit. Expected behaviour would be clean-up of background processes even if that happens, unless is killing by Ctrl+C or closing command prompt window.
The process cannot access the file because it is being used by another process

Thanks a lot, @kensoh!
Have just downloaded your latest release.
Will give it a try with my scripts and get back to you if I still bump into any errors.

@kensoh, have pressed at least a hundred Ctrl-C's over the past 2 hours.  Looks like you have fixed the thing!
Now I always see only one command window. There are no more leftover command prompts from the previous session like what I always have last time when I press Ctrl-C.
Thanks a lot, Ken. You're amazing!

Thanks for your confirmation CK!!
I've iterated a few times over the last few days to improve on this kill processes script, but this sort of script to work across all OSes on an intimate level may encounter edge cases along the way. Closing the issue for now, but I'll look out for feedback from you and other users if there are new environmental setups and situations where this script can be improved further.
The reason I'm spending more time than I should on this is because I see the lack of ability to kill somehow dead TagUI processes as a roadblock for the tool to scale across enterprise. People will need to flush the environment now and then, and having this script allows fast flushing of background processes without restarting computer (and only takes 0.25 seconds on my 2014 laptop).

(above iteration to remove warning message for end_processes script and tagui script on Linux OS)


collapse card when sliding

In GitLab by @lakhoune on Apr 11, 2020, 15:50
When sliding to a new card in the carousel, close the old one

In GitLab by @lakhoune on May 7, 2020, 11:27
use collapsable item from mdbootstrap

In GitLab by @lakhoune on May 9, 2020, 21:55
closed

In GitLab by @VerwaltungHilton on May 15, 2020, 22:57
Do not forget to mark the social service as verified                  here


IP Version filtering

Particularly useful if you are using proxy_bind (with which mixed stack resolver responses will result in connection failures).

Found a problem, it doesnt use all possible servers. Fixing.

Associated fix also attached for upstream peer name not being set (resulting in trucated error log lines).

Updated to master so that tests can pass

I'm fine with extracting addr family and comparing with that throught the logic. I will update the pr.

Note: Have not performed manual testing.
Can squash if this implementation is preferred.


SQL: optimize database command

We need a command to optimize the database on various aspects. The first is transforming edges from regular to lightweight if possible.

Documented on: https://github.com/orientechnologies/orientdb-docs/wiki/SQL-Optimize-Database


feat: create sidebar menu

Add sidebar menu in layout

Closing this request as the assignee created another branch with the requested changes.


Update container schema type to include project id

https://github.com/rancher/rancher/issues/12656

@mrajashree
Thanks for the fix.
The type should be /v3/projects/${project_id}/schemas/container instead of /v3/project/${project_id}/schemas/container.  Then our ember-api-store UI module can read it.
But it looks like there are too many types we need to change not just for container.
Close this PR. I think it's better to rewrite the type in frontend for now.
Thanks again!


apply for showcase

Hey there,
Cool flow made by cool people, love it!
So inspired by xyflow and unreal engine, we made a visual programming language: Hongtu or 鸿图 in Chinese. Hopefully, it can be added to xyflow showcase
https://console.kaihong.com/hongtu/
We took the hard work to make a full blow programming language, features:

Javascript like data types
All control flows plus Parrallel & Converge
Variables and functions
Event driven
Node copy&paste
Undo redo
Visually backend service development and serverless deployment
Built-in support for LLM (based on ChatGLM) with Intention-Entity prompt
More intuitive and extensible Node-resizer that can wrap any custom node (try select mulitple nodes and add commet)
Animated edge like Postman flow
Built-in BI charts
Full life-cycle IOT development
Export flow to javascript
Light & dark mode

Thanks in advance!
-Lance Mao

@lancemao we're missing some information that we would need to be able to consider it for the showcase, can you please add the information available our new issue template: https://github.com/xyflow/web/issues/new?assignees=&labels=showcase&projects=&template=submit-showcase.yaml
You can either do this by changing the description of this issue, or submitting a new issue using the linked template

created new using the template. so close this one


Support latest CDH v5.x and CDH v6.x (6.2.0 currently)

This PR is based on https://github.com/cloudera/cloudera-playbook/pull/23 that includes work from @maciejkowalczyk9ld
The version to deploy is set through one single variable (with current default to latest 6.2.x):
#cdh_version: 5.16.1
cdh_version: 6.2.0

I tested deployment of both versions (on AWS, with some other modifications, that I want to contribute separately)
Another nice thing: all other related variables (like the the repo_url, the parcel_repo, the scm_prepare_database script) are set through a helper dict configs_by_version (also to avoid any if/else logic)

Msg to the repo maintainers/committers ( @roczei ?):

Any chance of getting contributions merged in reasonable time?

I'm (currently) very motivated to contribute (and there is lot), and even taking extra time to make nice atomic commits/PRs, but if I see no action here I'll not bother anymore creating upstream PRs but only manage a nice+improved fork 👍

I highly suggest adding integration tests, ideally using Ansible Molecule (I might have some time in 2w), that could then be run for free on Travis 🎱

alternatively there's some existing work with vagrant (I found in a fork): https://github.com/cloudera/cloudera-playbook/compare/master...cgsyam:test_br

Hi @maciejkowalczyk9ld,
You need to first sign and return an ICLA and CCLA before we can accept and redistribute your contribution. Once these are submitted you are free to start contributing to cloudera-playbook. Submit these to<EMAIL_ADDRESS>Here you can find the pdf files:
https://github.com/cloudera/cloudera-playbook/tree/master/icla
Please update this issue (26) if you receive the confirmation from our CLA team.
Thank you!

Hi @roczei

Did you see my signed ICLA/CCLA were ack'ed by Cloudera (Casey V.),?
In case @maciejkowalczyk9ld is not interested in contributing, I can rewrite the initial code I based my PR on.. just need to know!  (ps If I had known earlier about the need to sign docs, I'ld have re-written directly my own code)
I'm happy to fix conflicts (due to changes in master) in these days, if my PRs really get merged

I suggest getting it done better earlier than later, not that your (master) branch deviates too much 👍
In general it would be great to know  for each PR/contribution if it will get accepted (of course after testing etc)
If all PRs acceptable, it would be easier for me todo in 1 PR (because I've done the conflict resolution already between PRs, when I ff-merged them in our fork)

Hi Laurent (@lhoss),
Yes, I saw that you have signed these docs. Thank you very much for your efforts and dedication what you provide for improving the cloudera-playbook community!
I would like to clarify two things:

The cloudera-playbook git repository is not officially supported by Cloudera, but the authors recommend using it.

My support regarding the cloudera-playbook is a best effort work. I do not have dedicated time for this. I have added updates to this repository because I am working on a new Cloudera engineering blog article and there are some files  here what I referenced from the blog. It will be published here: https://blog.cloudera.com This will cover two topics:

a) Dynamic Inventory Script for Cloudera Manager: https://github.com/cloudera/cloudera-playbook#dynamic-inventory-script-for-cloudera-manager
b) SSSD setup with Ansible (applicable for RHEL 7 / CentOS 7): https://github.com/cloudera/cloudera-playbook#sssd-setup-with-ansible-applicable-for-rhel-7--centos-7
I would like to help you, therefore I have contacted somebody inside Cloudera who may help by this work: accepting  and reviewing the pull requests, answering your questions, etc. Hopefully he will contact you soon here.
Best regards,
  Gabor

would like to help you, therefore I have contacted somebody inside Cloudera who may help by this work: accepting and reviewing the pull requests,

Thanks Gabor!
The one to help, it's not by chance @alexandruanghel  (the maintainer of  https://github.com/hortonworks/ansible-hortonworks, where I did many contribs also)? Actually, as you don't have much time for this ansible, he could be a great help (from inside Cloudera).
Another solution could be to give me committer permissions (that I can earn' by first providing working (molecule)tests .. ).
ps: I'd recommend to do any new features, in feature branches first.. and only merge to master when the feature is done (and tested, etc)

Hi @lhoss
Finally we have have found the owner for the cloudera-playbook repository. @dbeech is a Clouderan as well. He will help you by accepting and reviewing the pull requests, answering the questions, etc.
Best regards,
   Gabor

Merging into cdh6_support branch to rebase onto recent changes - will fix outstanding issues from there.


Add feature in interactive mode issue #40 show all vocabularies

I add the feature that mentioned on issue #40.
The feature is "list" in interactive mode.
Type list that will show all of vocabularys in db.

This PR has been merged and will be released in the next release. I've made some updates to this feature in #44 which include:

Extending the command parsing logic to support translation of sentences starting with the word "list".
Add docs

Thanks for your contributions!

thx. btw are u implementing "no cmd" for mulit-options? If u r not doing it, you may assign to me.

thx. btw are u implementing "no cmd" for mulit-options? If u r not doing it, you may assign to me.

Sure, I'm not working on this feature right now. I've created an issue that describes the user story in https://github.com/piglei/ai-vocabulary-builder/issues/47 , feel free to add your comments.


My community contribution: ASCII art from image

Video: https://youtu.be/fJMaec6zi9k
This command line tool made by me can create ASCII art from given images (square images) and can render and save the ASCII art as .png image as well. More information in the GitHub README.md.

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

What do I need to change someone please do help me


Fixes for mobile and interaction in typeahead mode.

My previous pull request broke interaction when in the typeahead mode.

The "blur" event fires before the "tap" event, so tap selections within the dropdown were being clobbered by the _onInputBlur() handler.
(Somehow I had blocked out the painful memories of running into this before and working around it with that separate on-down handler.)
Added an _ignoreBlur Boolean to track whether the blur should be ignored.
Added _onInputFocus() and _onItemDown() handlers to coordinate the state of this new flag.
(It would have been nice to have been able to simply use the FocusEvent::relatedTarget in _onInputBlur() to filter out blurs triggered by focus change to the element's iron-dropdown items - alas, that property is experimental and not available in Firefox.)

I also missed making the "click" -> "tap" change for the second template in the iron-selector element.

I have now applied the same changes to the second template as the first.

Switched out the "click" event type for "tap" in _boundOnCaptureClick(), _onCaptureClick(), etc. to ensure these also fire on mobile devices and updates the internal bookkeeping properties.

Found an issue with my changes - _onCaptureClick isn't currently firing.  Hold off on this for now.

Fixed a bug where the typeahead mode was not resetting the filtered options when it should:

Scenario:

Enter an partial match into a radium-combo in typeahead mode.

Tap somewhere outside the dropdown menu.

The partial match value in the input should be cleared.

Focus the typeahead input again.

Expected: The options should no longer be filtered by the previous cleared input.
Observed: The options were still filtered by the previous cleared input.

Added CSS to prevent rapid subsequent toggling of the dropdown between open & closed from selecting the text in the input field when disabled (i.e. not in typeahead mode).
Replaced previous dropdown positioning workaround with the no-overlap attribute that was subsequently documented.
Added on-touchend handlers to prevent ghost clicks from bleeding through the dropdown to input fields, etc. below.
Fixed the direction icon toggling binding for _dropdownIcon() so the icon changes to match the dropdown state.
Refactored _onCaptureTap so that we're not fighting against iron-dropdown's capture method.

Added no-cancel-on-outside-click to iron-dropdown instance.
Our version blurs the input and hides the dropdown if the "tap" occurs outside our element's DOM tree.
Changed the lifetime of that listener to be as long as this element is attached rather that just when the dropdown is open.

Otherwise, the input would only be notified to blur if the dropdown was open.
Necessary for mobile devices where tapping outside an input isn't guaranteed to blur it.

Added _onInputFocus() handler because software keyboards on mobile devices include alternate ways to focus an item beyond taps (form navigation buttons at the top of the software keyboard) - wrong dropdown ended up open.

Sorry for all the changes, but the end result is much improved usability on mobile touch devices!
Tested this using the demo and our internal app - this PR is ready for review now.


Connection problem

Hi,
When I run the example file with my IP and port of my solaredge main convert (have in total 3 + battery) I get this error directly:
pymodbus.exceptions.ConnectionException: Modbus Error: [Connection] ModbusTcpClient(<IP_ADDRESS>:502): Connection unexpectedly closed 0.000037 seconds into read of 8 bytes without response from unit before it closed connection
I know for sure that I can connect to this IP and port, becasue with NodeRed I can read the modbus via the modbus module there. Any thoughts?

Is the inverter actually listening on port 502 and not 1502?

yes
From: niels @.>
Date: Friday, 30 September 2022 at 10:58
To: nmakel/solaredge_modbus @.>
Cc: Remco Ploeg @.>, State change @.>
Subject: Re: [nmakel/solaredge_modbus] Connection problem (Issue #56)
Is the inverter actually listening on port 502 and not 1502?
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnmakel%2Fsolaredge_modbus%2Fissues%2F56%23issuecomment-1263303450&data=05|01||dfd3701a340a4c6f18bf08daa2c1e9d3|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638001250973950030|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=yWDcLNPk6O5GLLHXvOjSJ%2FeBZPnGEO1eVdaRc%2FncCHY%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAL3SSDPDOYFPYWRL2AZK7B3WA2TSPANCNFSM6AAAAAAQZSC72Y&data=05|01||dfd3701a340a4c6f18bf08daa2c1e9d3|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638001250973950030|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GlH1Tq5RE5AVkNVEwxckuFE7rAm5CQEj8tvPxw0qL%2FQ%3D&reserved=0.
You are receiving this because you modified the open/close state.Message ID: @.***>


storage: fix vmware volume cleanup failure

Description
Fixes #7311
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Tested the fix with a <IP_ADDRESS> patch. DeleteCommand send to correct host in the source cluster
@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5682

@blueorangutan test centos7 vmware-67u3

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

Trillian test result (tid-6265)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40762 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7312-t6265-vmware-67u3.zip
Smoke tests completed. 99 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_04_deploy_vm_with_extraconfig_throws_exception_vmware
Error
1.17
test_deploy_vm_extra_config_data.py

test_05_deploy_vm_with_extraconfig_vmware
Error
1.14
test_deploy_vm_extra_config_data.py

test_01_deploy_vm_on_specific_host
Error
4.31
test_vm_deployment_planner.py

test_02_deploy_vm_on_specific_cluster
Error
0.12
test_vm_deployment_planner.py

test_03_deploy_vm_on_specific_pod
Error
0.15
test_vm_deployment_planner.py

test_04_deploy_vm_on_host_override_pod_and_cluster
Error
1.17
test_vm_deployment_planner.py

test_05_deploy_vm_on_cluster_override_pod
Error
1.14
test_vm_deployment_planner.py

@vladimirpetrov probably update the wrong PR. Re-requesting your review

@shwstppr is this PR still relevant for 4.17 or can be closed?

@nvazquez cc @vladimirpetrov yes, we can close this. As per my update on this tagged issue, the problem is fixed with https://github.com/apache/cloudstack/pull/5575. ACS marks volume as detached once migration is complete and then deletion of source volume file is handled by an appropriate host.


SSE emit payload attributes before each slot

SSE payload attributes before each slot if enabled via a cli option --emitPayloadAttributes
ref:

https://github.com/ethereum/beacon-APIs/pull/305

Closes #5251

:tada: This PR is included in v1.7.0 :tada:


[BUG - Release] UE4SS Causes Palworld to crash

Branch or Release
The Tag for the Release you are using, or the commit you used if you built from source yourself.
v3.0.0
Game and Engine Version
Game where the bug occurred, and the UE engine version of the game.
Palworld. Unknown UE Engine version
Describe the bug
A clear and concise description of what the bug is.
When launching the game, game immediately crashes with the following errors. I re-installed the mods, but still didn't work... I provided the crashes.zip file.
nhandled Exception: EXCEPTION_ACCESS_VIOLATION reading address 0x000001b336ced058
UE4SS
UE4SS
UE4SS
UE4SS
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
UE4SS
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
UE4SS
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
UE4SS
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
Palworld_WinGDK_Shipping
kernel32
ntdll
Mods directory
A zip of your Mods folder if you've installed any mods that don't come with UE4SS by default
vuxReduceFoodSpoilRate120 - https://www.nexusmods.com/palworld/mods/232

https://www.nexusmods.com/palworld/mods/11?tab=posts&BH=0

Mods.zip
To Reproduce
Steps to reproduce the behavior:

Launch game
Attempt UHT Dump
See error in log

Expected behavior
A clear and concise description of what you expected to happen.
The game should launch
Screenshots and UE4SS Log
If applicable, add screenshots to help explain your problem, and upload the log.
Crashes.zip
Desktop (please complete the following information):

OS: [e.g. Win10, Win11]

Additional context
Add any other context about the problem here.

I also have this crash.
It is persistent once it happens and only uninstalling UE4SS resolves it. I can sometimes get some mods to work and at one time it was stable with 4 mods, but after one crash it crashes until everything is uninstalled.
Tested on reinstalled fresh palworld as well.
the reading address is random/different each time
VCRUNTIME140
UE4SS
UE4SS
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
UE4SS
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
UE4SS
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
UE4SS
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
Palworld_Win64_Shipping
kernel32
ntdll

Adding my voice to the list of people experiencing this.

Can confirm, I boot the game, close the intrusive popup, select multiplayer, close the second intrusive popup, select recent servers, select my server, password the box, hit the connect button, hit the redundant confirm button and at some point during the loading screen it crashes with the above crash log. Brute force reboot and retry eventually connects to server without crash.

I cleared my shader cache and I installed the zdev version of UE4SS.  The problem has not yet returned after several straight hours of gaming.

Can confirm, I boot the game, close the intrusive popup, select multiplayer, close the second intrusive popup, select recent servers, select my server, password the box, hit the connect button, hit the redundant confirm button and at some point during the loading screen it crashes with the above crash log. Brute force reboot and retry eventually connects to server without crash.

Those "instrusive popups" are the console and GUI windows, which is only enabled by default in the zDEV version of UE4SS - i.e. the version that is targeted at mod developers only. As you're clearly an end-user, I'm curious why you are using that version? Is the mod you're using telling you to use that?

They might've seen a reply by me to another issue where I asked someone to use the zDEV version because I couldn't get their dmp file working with the non-dev version.
I've also decided to always link to zDEV version to people whenever I request a user to test any particular build, just to make sure the dll & pdb match the ones I have so that I can actually do some debugging.

I believe that poster meant the two popups from the game, not UE4SS.

Oh actually that would make sense because in the 3.0.0 release there is a bug where closing the GUI window crashes the game (fixed in next release hopefully being done today btw).

Can confirm, I boot the game, close the intrusive popup, select multiplayer, close the second intrusive popup, select recent servers, select my server, password the box, hit the connect button, hit the redundant confirm button and at some point during the loading screen it crashes with the above crash log. Brute force reboot and retry eventually connects to server without crash.

Those "instrusive popups" are the console and GUI windows, which is only enabled by default in the zDEV version of UE4SS - i.e. the version that is targeted at mod developers only. As you're clearly an end-user, I'm curious why you are using that version? Is the mod you're using telling you to use that?

They might've seen a reply by me to another issue where I asked someone to use the zDEV version because I couldn't get their dmp file working with the non-dev version. I've also decided to always link to zDEV version to people whenever I request a user to test any particular build, just to make sure the dll & pdb match the ones I have so that I can actually do some debugging.

For clarification, I am the one who cleared their shader cache and installed the zDEV version, after digging into other bug reports for possible solutions.
BTW, clearing the shader cache fixes the "bending trees" problem that the game itself develops now and then.

The only "fix" I've found for this is a complete re-install of Palworld, and then, reinstalling UE4SS.

Can confirm, I boot the game, close the intrusive popup, select multiplayer, close the second intrusive popup, select recent servers, select my server, password the box, hit the connect button, hit the redundant confirm button and at some point during the loading screen it crashes with the above crash log. Brute force reboot and retry eventually connects to server without crash.

Those "instrusive popups" are the console and GUI windows, which is only enabled by default in the zDEV version of UE4SS - i.e. the version that is targeted at mod developers only. As you're clearly an end-user, I'm curious why you are using that version? Is the mod you're using telling you to use that?

They might've seen a reply by me to another issue where I asked someone to use the zDEV version because I couldn't get their dmp file working with the non-dev version. I've also decided to always link to zDEV version to people whenever I request a user to test any particular build, just to make sure the dll & pdb match the ones I have so that I can actually do some debugging.

For clarification, I am the one who cleared their shader cache and installed the zDEV version, after digging into other bug reports for possible solutions.
BTW, clearing the shader cache fixes the "bending trees" problem that the game itself develops now and then.

The only "fix" I've found for this is a complete re-install of Palworld, and then, reinstalling UE4SS.

So setting bUseUObjectArrayCache = false in UE4SS-settings.ini didn't work for either?

Following this threads advice it still errors with palworld_win64_shipping error on start up for me. Hopefully it's not a issue in further releases.


fix: types

Types

[ ] 🎉 New Features
[ ] 🐛 Bug Fixes
[ ] 📚 Documentation Changes
[ ] 💄 Code Style Changes
[ ] 💄 Style Changes
[ ] 🪚 Refactors
[ ] 🚀 Performance Improvements
[ ] 🏗 Build System
[ ] ⏱ Tests
[ ] 🧹 Chores
[ ] Other Changes

Background or solution
Changelog

@OpenSumi/developers pls see this


Update spin.js definitions for v2.3.2

spin/spin.d.ts
to authors(@borisyankov @theodorejb). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?


JFXSpinner not spinning in JFoenix 8.0.2

Hello, I'm migrating from JFoenix version 1.11.1 to 8.0.2, but when I run the program, the JFXSpinner does not move, this problem only happens with some Spinner that I have implemented.

In version 8.0.1 I also have the same problem, but in version 1.11.1 it works perfectly.
The implementation of the JFXSpinner is the same as any other.
Thanks :)

Hello,
How did you create the JFXSpinner?
please note that JFXSpinner now supports determinate mode

@TioCoding can you provide a SSCCE for this issue?

In this implementation the error is presented, add a JFXSpinner inside a JFXDialog:
public class SpinnerDemo extends Application {

    @Override
    public void start(final Stage stage) throws Exception {
        StackPane paneDialog = new StackPane();

        JFXDialog dialog = new JFXDialog();
        dialog.setDialogContainer(paneDialog);
        dialog.setOverlayClose(false);
        dialog.setContent(new StackPane(new JFXSpinner()));

        final Scene scene = new Scene(paneDialog, 300, 300);
        stage.setScene(scene);
        stage.setTitle("JFX Spinner Demo");
        stage.show();

        dialog.show();
    }

    public static void main(final String[] arguments) {
        Application.launch(arguments);
    }
}

The result (JFXSpinner not spinning):

will push the fix soon, thanks @TioCoding for pointing out the issue and the SSCCE 👍

@TioCoding, thanks for pointing this out: I thought I was going crazy because my spinner wouldn't spin.
@jfoenixadmin thanks for fixing!

Version 8.0.3 fixes the not spinning problem and is now on Maven Central.

@TioCoding ,Thank you for pointing this problem ,but my problem still the same.


Support for caching

Other vulnerability APIs such as PyPI's JSON API provide support for caching, allowing the client to avoid making multiple identical requests via ETag headers.
The OSV API's use of POST requests makes this somewhat challenging, as POST responses are not usually cached.
How should OSV support caching, and how should OSV clients cache responses?

We had another FR here.
My understanding of the relevant HTTP headers is weak, but my understanding is we could leverage two potential headers:

If-None-Match with Etags.
If-Modified-Since and re-using our modified timestamps in the returned OSV responses.

As @di mentioned though, POST may make this a slight misuse of these headers.
As for implementing them, I'm not sure if this is something ESPv2 woulds support today, so it may be a little challenging.

One other consideration here is that caching is not going to be saving much time with the actual query itself -- OSV.dev will still need to compute the actual query before any ETags or similar are returned. This would only save traffic and the costs associated with that.

Caching was touched on in #2239 as well.

Leaving this as a backlog item as we don't see this as enough of a priority to justify the large engineering investment here. Please comment here if anybody disagrees!


added Decode interface

this pull request adds an interface for custom decoding. This is similar to what you would find in the json package or in some other env packages, like https://github.com/kelseyhightower/envconfig#custom-decoders

Good idea!  Thanks for the patch.
Could you add a test for non-struct types that also implement the Decoder interface?  Since the codepaths differ for structs versus non-structs, I'd like to have a test to ensure I don't accidentally break it in the future.


[Request] Wireless charger to charge RF-receiving blocks in the world.

I'd like to have a wireless charger (that can be upgraded with capacitors for speed and range) to charge RF-receiving blocks placed in the world.
Would save conduits in places where they just don#t fit.

That has already been requested several times - and it will not be added.


Webpack and jest is add to project with basic files.

Webpack is add to project
jest is add for the testing the project.
Baisc files like index.html, index.js and style.css is add to project.

Hi @Hidar37,
You've done great work implementing the features, However, there are some Linters errors that need to be addressed.

Highlights

Webpack and Kanban implemented perfectly :heavy_check_mark:
Descriptive README file :heavy_check_mark:

Optional suggestions

[ ] [OPTIONAL] Please make sure to include a link to your license and fix  the links to your broken logo and remove info from microverse  in your readme to make your project more professional :+1:

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me (@codecaiine) in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @Hidar37  ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

Wenpack configured correctly :heavy_check_mark:
No linters errors :heavy_check_mark:

Optional suggestions

[ ] [OPTIONAL] Please make sure to include a link to your license and fix  the links to your broken logo and remove info from microverse  in your readme to make your project more professional :+1:

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me (@codecaiine) in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


awake pc from sleep

Description of the new feature / enhancement
A feature will be helpful which would wake up my pc from sleep in order to ring my alarm in the  mornings.
Scenario when this would be used?
this feaure will be helpful because my pc goes to sleep overnight and as a result my alarm doesnt ring in the mornings.  Okay..i know i can change sleep settings to never but that would drain my laptop battery a lot!!
Supporting information
No response

no plans currently


Implement autocd

In zsh I can change directory only by path (without cd command). In black screen it returns error:
> ./ekaragodin
zsh:1: permission denied: ./ekaragodin

I too have zsh as my default shell.  I am unable to cd at all.

anyone found a solution for this? +1

Anyone found a solution to this 3 years later? lol

Using zsh in any other terminal I can cd into directories using mydir.
Using Upterm I can only get there using cd mydir/.

Love everything else about Upterm but AutoCD would make it perfect


SHA1 for password verification

Hi, i have seen that for password verification LiteDB uses SHA1, but this hashing algorithm is weak...
it would be possible to make that a developer who uses liteDB can specify a hashing algorithm in the constructor of LiteDatabase class? perhaps through a class that implements a specific interface?
Thank you
(sorry for my english)

Hi @ANBlue, is SHA1 weak? I didn't know about this.  There is any other better? Maybe it's possible create more than 1 implementation about this.

https://www.bentasker.co.uk/blog/security/201-why-you-should-be-asking-how-your-passwords-are-stored
https://nakedsecurity.sophos.com/2012/12/09/sha1-brute-force-trimmed-21-percent/
the two links above explain why it is better not to use the sha1 for password storage/verification
better algorithms are:
https://en.wikipedia.org/wiki/Bcrypt
https://en.wikipedia.org/wiki/Scrypt

Yes, SHA1 is obsolete. SHA256 is a better alternative as the default. Having an overload to specify your own hash would be a good option also.

Bcrypt and scrypt should be used instead as it is stronger than sharing
family set for cryptography
On Jan 6, 2017 1:36 AM, "Kuiper"<EMAIL_ADDRESS>wrote:

Yes, SHA1 is obsolete. SHA256 is a better alternative as the default.
Having an overload to specify your own hash would be a good option also.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mbdavid/LiteDB/issues/409#issuecomment-270789780, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVQ2C6TATrMDL17bZsQE9qbftjLKpqoRks5rPX5wgaJpZM4LZRpG
.

Actually, I would agree.
https://rietta.com/blog/2016/02/05/bcrypt-not-sha-for-passwords/
http://security.stackexchange.com/questions/26245/is-bcrypt-better-than-scrypt

I write government software which requires high security and verification.
Which leaves myself to use 4096bit RSA private public key pair encryption
and bcrypt for verification.
This is my security recommendation.
On Jan 6, 2017 2:06 AM, "Kuiper"<EMAIL_ADDRESS>wrote:

Actually, I would agree.
https://rietta.com/blog/2016/02/05/bcrypt-not-sha-for-passwords/
http://security.stackexchange.com/questions/26245/is-bcrypt-
better-than-scrypt
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mbdavid/LiteDB/issues/409#issuecomment-270794573, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVQ2C0dk1VXXqaygteoHAroyzq0JZSbtks5rPYVvgaJpZM4LZRpG
.

I think it might be a good idea just to use what Microsoft thinks is safe (PBKDF2 with HMAC-SHA256, 128-bit salt, 256-bit subkey, 10000 iterations) and it's FIPS compliant:
This implementation is really easy to port
https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/PasswordHasher.cs

Hi @mkosieradzki, I like this "simple" solution :) I will read more about it, thanks.

@mbdavid  Yes, SHA1 was recently proven weak: https://security.googleblog.com/2017/02/announcing-first-sha1-collision.html
I was curious about your security implementation, so I briefly looked around the code.
I see that you store a SHA1 hash in the header page for verification to quickly feedback to caller that a password is wrong, rather than catching a decryption failure later on when reading/writing data. Correct?
Apart from that it seems you are already using PBKDF2 (Rfc2898DeriveBytes) for deriving the AES key from the password:
https://github.com/mbdavid/LiteDB/blob/944466cd6948e33433e0dc74f2c6a3001333ea4e/LiteDB/Utils/AesEncryption.cs#L25
So, while a malicious user may be able to supply a password that will result in a hash collision, decryption should still fail because you derive the AES key using Rfc2898DeriveBytes. With that, it doesn't seem to me that the SHA1 verification is really hurting anything. (someone double check me!)
However, you should shift your attention to other encryption concerns. I'll create separate issues for them.

The SHA1 hash of the password is not secure.
Any two data bases with the same password will have the same SHA1 hash.  Thus this leads to collisions.
Hashing the key generated by a suitable PBKDF would be slighly better, as the chances of collision is far reduced.  However, I would still not recommend that as it still provided an attacker with a short cut to verfiying a key.  The only acceptable solution is to switch to authenticated encryption using an EtM approach.

Are there still plans for a secure password encryption algorithm to be used replacing the current SHA1 functionality for the next appropriate version?

Hi @ANBlue, it´s on my plans change this. But to make it possible, I will need change datafile structure, and it's not easy job remove this compatibility. The next major version is likely to change this file structure and I will take advantage of this to fix some old decisions.

Hi! Quick note that the SHA1 is also referenced here:
https://github.com/mbdavid/LiteDB/issues/586
Basically, the SHA1 should be completely removed and use only the current PBKDF2 key derivation. However, I DO agree that it would be great to have settings to replace the PBKDF2. The current standard for local data hashing is argon2 (the winner of the Password Hashing Contest, used by Keepass, for example). However, Scrypt and Bcrypt are also pretty valid options(as they are resistant into GPU). PBKDF2 is not "bad", but it's not the greatest either, and people would surely like to have better options.
TL;DR: Linked ticket suggest removing the password validation and replace it with a Mac, which would be a better option cryptographically-wise. The Key Derivation Function(currently PBKF2) could be made a settings for people who would like stronger/customizable options(argon2, scrypt, bcrypt).
Also, I can help with all that stuff if you need it :)

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!


apex exec --help fails

as it understands "--help" as the command to be executed

Operating system: Linux zeno 4.9.43-1-MANJARO 1 SMP PREEMPT Sun Aug 13 20:28:47 UTC 2017 x86_64 GNU/Linux
Apex Version: Apex version 0.15.0

ah nice find, apex help exec still works at least  :D


Lengthy scrollview inside safe are element causes element to stretch

I have a large scrollView with lots of items
Root
->SafeArea
---->ScrollViewParent height: 100%, Grow: 1
------->ScrollView
----------->Items
if the ScrollViewParent  is not part of safe area element it works fine and covers just the screen and I can scroll items as expected.
But if its the child of Safe Area then it scales to cover the entire length of scrollview which makes it impossible to scroll.

If this issue remains, please open a new issue and attach content that reproduces the problem.


Stage import fails when several stages with the same name exist in different schemas of a database

Provider Version
0.17.1
Terraform Version
0.13.5
Describe the bug
If several stages with the same name exist in the same database under different schemas, the import of such schemas will fail.
The reason behind it is that the following statement is used to show the stages based on their ID:
return fmt.Sprintf(`SHOW STAGES LIKE '%v' IN DATABASE "%v"`, sb.name, sb.db)

Note that the terraform import itself won't fail, but it will import the stage from the first schema in alphabetical order.
Expected behavior
Stages with the same name in different schemas of a database can be imported successfully.
If the following statement was used to show the stages the issue would be solved:
return fmt.Sprintf(`SHOW STAGES LIKE '%v' IN SCHEMA "%v"."%v"`, sb.name, sb.db, sb.schema)

Code samples and commands

Importing stages with different resources:

resource "snowflake_stage" "my_stage_dev" {
  database            = "MY_DB"
  schema              = "MY_SCHEMA_DEV"
  name                = "MY_STAGE"
  url                 = "s3://my-bucket"
  storage_integration = "MY_INTEGRATION"
  file_format         = "FORMAT_NAME = MY_FILE_FORMAT"
}

resource "snowflake_stage" "my_stage_prd" {
  database            = "MY_DB"
  schema              = "MY_SCHEMA_PRD"
  name                = "MY_STAGE"
  url                 = "s3://my-bucket"
  storage_integration = "MY_INTEGRATION"
  file_format         = "FORMAT_NAME = MY_FILE_FORMAT"
}

terraform import snowflake_stage.my_stage_dev "MY_DB|MY_SCHEMA_DEV|MY_STAGE"
terraform import snowflake_stage.my_stage_prd "MY_DB|MY_SCHEMA_PRD|MY_STAGE"

Importing stages with same resource in different workspaces:

locals {
  env = {
    "default" : "dev",
    "production" : "prd"
  }[terraform.workspace]
}

resource "snowflake_stage" "my_stage" {
  database            = "MY_DB"
  schema              = "MY_SCHEMA_${upper(local.env)}"
  name                = "MY_STAGE"
  url                 = "s3://my-bucket"
  storage_integration = "MY_INTEGRATION"
  file_format         = "FORMAT_NAME = MY_FILE_FORMAT"
}

terraform workspace select dev
terraform import snowflake_stage.my_stage "MY_DB|MY_SCHEMA_DEV|MY_STAGE"
terraform workspace select prd
terraform import snowflake_stage.my_stage "MY_DB|MY_SCHEMA_PRD|MY_STAGE"

Looks like this was fixed in #291.

Thanks @elliottohara for fixing this and @ryanking for pointing it out!


Add LFortran Compiler Support

Hi,
This PR references issue #11 and adds LFortran support on Linux and macOS by installing LFortran using Miniconda.
The installation of LFortran on Windows using Miniconda is completed in the last commit. However, as I am not a Windows user, I couldn't set the paths. Perhaps someone else can perform the final steps for Windows.
Best,
Ali
[Edited]:

I excluded CC/CCX for LFortran.
The executable for LFortran on Windows is located at C:/ProgramData/Miniconda3/envs/lf/Library/bin/lfortran.exe. You can check it with cd C:/ProgramData/Miniconda3/envs/lf/Library/bin and then dir. However, -h results in an error.
Some work on setting paths on Windows using set and setx with /m doesn't work properly. After setting paths, it may be necessary to restart the current shell. I tried with cmd /c and powershell.exe -Command, but it doesn't work.

Thanks very much @gha3mi. A few notes

I think CC/CXX outputs and env vars should be set to preinstalled gcc/g++ for lfortran and other standalone fortran compilers, as anticipated in the readme, maybe others disagree
Not sure about installing via conda. Using brew, choco, and APT as done for other toolchains would be ideal — looks like this is possible for brew but not yet for APT or choco. Note miniconda is preinstalled on Ubuntu and Windows and may also have been setup via e.g. setup-miniconda or setup-micromamba. An lf env may already exist too, seems like this could cause collisions.

I think CC/CXX outputs and env vars should be set to preinstalled gcc/g++ for lfortran and other standalone fortran compilers, as anticipated in the readme, maybe others disagree

I considered this as well, but I was uncertain about which compiler should be set in this scenario. Since all the other compilers are available, it should be easy to figure out.

Not sure about installing via conda. Using brew, choco, and APT as done for other toolchains would be ideal

Personally, I'm not a fan of Conda. The issue is that LFortran consistently suggests using Miniconda.
By the way, I won't be able to work on it soon, maybe later. Someone else might find the time to do it. Feel free to make changes to this PR.

there was a brief thread in the lfortran mailing list last month, suggesting it will soon be available on APT as well

copying from #78
I think we can just use the builtin conda on linux/windows and install/use micromamba on mac, and move ahead with this, rather than wait for it on other package managers as I was thinking before. Some more things:

this now installs lfortran to the base environment, not a custom one
lfortran seems to pair better with MSVC on Windows — install MSVC toolchain with it and set cl as the C/C++ env vars and outputs, still use gcc/g++ on linux and mac though
tweaked the testing conditions, there is some issue with lfortran executable discovery on powershell and command prompt, so windows support is limited to bash for now unless it's invoked by abs path
updated the test matrix and compatibility files


ESP32 with 32feet on Windows 11, Bluetooth classic, can't connect

Background:
Finally, I want to use 32feet for connecting an ESP32 device (BT Classic) with Unity 3D (somewhere I found, that someone did this successfully, but no details provided). Since this did not work right way, as a simplified initial step, I used the

32feet (id="InTheHand.Net.Bluetooth" version="<IP_ADDRESS>") example “BluetoothClassicConsole” and the
ESP32 Example “SerialToSerialBT”

to get it up and running.
The ESP32 has been paired under Windows system settings -> OK
Using the Windows application “Bluetooth Serial Terminal”, I can connect to the ESP32 and can exchange data in both directions ->OK
Error:
Under 32feet I can’t connect to the ESP32. Error message:

System.Net.Sockets.SocketException,   HResult=0x80004005
Message=A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond 4022D8EAB23E:00001101-0000-1000-8000-00805f9b34fb (0)

Details:
The 32feet function “DiscoverDevices()” properly discovers the ESP32. The correct device name and address is retrieved.
The following: client.Connect(device.DeviceAddress, BluetoothService.SerialPort) fails with the exception mentioned above.
Before going in more depth, I like to clarify the system requirements. The32feet documentation (https://github.com/inthehand/32feet) clearly says: Bluetooth support requires .NET Framework v4.6.1 for desktop Windows 7, 8 and 10.
So, no Windows 11, correct? In that case we can stop for now. Any plans to support W11 in the future?
I found another thread  #202 , which is a different issue, but the guy reported, that he is also working under Windows 11.
Another “violation” to your spec is, that my system has .NET Framework v4.7.1 installed. When trying to install 4.6.1 I get the message “.NET Framework 4.6.1 or a later update is already installed on this computer.”. So I assume using 4.7.1 is Ok.
Would be great to get that thing clarified/fixed.
Thanks a lot for putting all these efforts in the maintenance of 32feet and providing it to public!!

The requirements text needs clarifying. For .NET Framework the minimum supported version is 4.6.1 but you can use the dll against any subsequent version - no need to install an older version.
There are two underlying implementations for Windows:-
Win32 supported on Windows 7 and above
WinRT supported on Windows 10 and above
.NET Framework 4.x will always have the Win32 implementation.
.NET 5.0 and above and targeting Windows 10.0.19041 or higher use the WinRT implementation. Older targets than this fallback to the .NET 5.0 Windows 7 Win32 version.
The UWP dll uses WinRT.
There are currently both .NET 5.0 and .NET 6.0 builds but the 6.0 dll doesn't offer any additional features. .NET 7.0 is supported but will use the .NET 6.0 dll.
It's not recommended using the library on .NET Core 3.x but it may work (on non-server Windows devices only).
I know that's a little overcomplicated but hopefully it explains it. In short you can use it with .NET 4.7.1 with no problem.

Thanks for the super-fast response. So seems my general setup is ok and I should stay with .NET (Framework) 4.7.1, since my final target system is Unity 3D and it (currently) only supports .NET Framework - not .NET (Core).
I will do some deeper investigations now, but since I am not a BT expert to be honest I do not know where to start ;-(. As I said:

works with a Windows application “Bluetooth Serial Terminal” => so I conclude, the general Windows setup (driver, dongle, ...) and ESP32 side is basically ok
does not work with 32feet (“DiscoverDevices()” works, but timeout during connect)

Did you observe s.th. comparable before? Any hint in which direction to investigate?
I will study your documentation and check, whether I find. s.th. there.
Thanks again for your support.

@schmitz-u that's great to see you are playing the same setup as I do. In my case I was able to connect Unity3d with the ESP32. On the ESP I've used the sample: https://github.com/espressif/arduino-esp32/blob/master/libraries/BluetoothSerial/examples/SerialToSerialBT/SerialToSerialBT.ino
and in my Unity environment I've entered in the Start method some code of 32feet sample: https://github.com/inthehand/32feet/blob/main/Samples/BluetoothClassicConsole/Program.cs
DiscoverDevices works well and detects the ESP32. After that I store the BluetoothDeviceInfo of my ESP32 to create a connection via BluetoothClient instance. You should not use BluetoothSecurity.PairRequest with a PIN, the communication will not work after that. Its because you can't confirm the pin and Windows is using a different PIN.
BUT, after all I was able to start Unity only one time. The second start blocks the editor, so I have to kill the Unity editor. Why is it so? I found out that the BluetoothClient close and any other disconnect methods will not keep free the used port. It seems the Unity Editor claims the port after closing the "Game".
I'm still in investigation how to solve this. But back to your point. Your setup should work, my is exactly the same (Win 11 and so on) and it works.

@schmitz-u btw, I found an example for your idea: https://github.com/jcs090218/32feet.Unity.Example

@jfellien, it sounds great, that you made the general connection work.
What worries me is that I think I used the same example code, that you mentioned (SerialToSerialBT.ino, BluetoothClassicConsole/Program.cs).... Have to go into more depth again. Won't be able to do this before the next weekend. In case I will take the initial hurdle, we can stay in contact regarding the Unity problem you mentioned.

btw, I found an example for your idea: https://github.com/jcs090218/32feet.Unity.Example

I see. Basically the same steps as in BluetoothClassicConsole/Program.cs The key point is: cli.Connect(mDevice.DeviceAddress, BluetoothService.SerialPort);
Equivalent to client.Connect(device.DeviceAddress, BluetoothService.SerialPort);  in BluetoothClassicConsole/Program.cs.
I confirmed, I read out the correct DeviceAddress, but this client.Connect times out....
Will get back on the weekend.

One of the complications with Unity is that while it is .NET 4.6.1 compatible it's the Mono runtime and the Mono version of System.Net.Sockets.Socket doesn't allow Bluetooth connections - there appears to be some extra checking of the endpoint passed  in and so the Win32 implementation of 32feet has a Win32Socket class for when it is used from Mono which P/Invokes into the WinSock APIs. If there is a problem with the behaviour of the Close() method which is causing the socket to be blocked from being used more than once I'll need to investigate this. However if the connect is not working in the first instance I'm less clear on why this might be.
It might be worth checking that there is nothing else paired with the device which may also try to open a connect to the same service. However that being said, I would expect another error message than a timeout. It might be useful if you could build with the source and step into the code and see where it is getting stuck and if there is any other diagnostic information.

@peterfoot you are a great developer. +100
I tested a lot of different things now and found out exactly what you describe. Also a few Async methods don't work well. But in the end you lib is the most stable solution for Unity at the moment. (If the "Game" will run unter Windows)
So, for my project I've solved anything.

@jfellien, it sounds great, that you made the general connection work. What worries me is that I think I used the same example code, that you mentioned (SerialToSerialBT.ino, BluetoothClassicConsole/Program.cs).... Have to go into more depth again. Won't be able to do this before the next weekend. In case I will take the initial hurdle, we can stay in contact regarding the Unity problem you mentioned.

btw, I found an example for your idea: https://github.com/jcs090218/32feet.Unity.Example

I see. Basically the same steps as in BluetoothClassicConsole/Program.cs The key point is: cli.Connect(mDevice.DeviceAddress, BluetoothService.SerialPort); Equivalent to client.Connect(device.DeviceAddress, BluetoothService.SerialPort); in BluetoothClassicConsole/Program.cs. I confirmed, I read out the correct DeviceAddress, but this client.Connect times out.... Will get back on the weekend.
BTW: which Arduinio ESP32 board package version do you use? I found a number of reports mentioning several BT issues with different versions. Right now I use 2.0.5. The newest one (2.0.6) raises some compile errors.

@schmitz-u  my ESP Board is an AZ Delivery WROOM-32.
I found out (today) that Bluetooth.cpp and .h in Arduino 2.0.6 has a bug. So I changed the stuff I need in the files but I would suggest to use 2.0.5


Make project-less ServerRuntime possible

Sometimes we want an empty Cayenne stack... Nothing wrong with that, the app should not fail. Currently it does:
Caused by: org.apache.cayenne.configuration.server.DataDomainLoadException: [v.4.0.M3 Feb 08 2016 16:38:05] Configuration resource "cayenne-project.xml" is not found.
	at org.apache.cayenne.configuration.server.DataDomainProvider.descriptorFromConfigs(DataDomainProvider.java:198)
	at org.apache.cayenne.configuration.server.DataDomainProvider.createAndInitDataDomain(DataDomainProvider.java:110)

Perhaps on bq-cayenne side we set the default dynamically. If "cayenne-project.xml" is present on classpath, we use that. If not, we revert to project-less stack (with a note in the logs).

UPGRADE NOTES
This is a breaking change. CayenneModule.builder is removed, as it no longer serves any useful purpose (its "noConfig" method is the same as the absence of cayenne-project.xml on classpath), but it made impossible clean implementation of this feature.


UIScreen bounds are not correct in Snapshop testing

I'm trying to use this framework for snapshot testing from some view controllers. It works fine in most cases, but I have one big issue. for example here is my test
func test_onboarding_iPhone13() {
    let vc = OnboardingViewController()
    assertSnapshot(matching: vc, as: .image(on: .iPhone13(.portrait)), record: true)
}

So, I'm trying to run a test on an iPhone 13. In the view controller, I use UIScreen.main.bounds to calculate some sizes, and it works fine when the test is not running. However, in the snapshot test, the value of UIScreen.main.bounds is different, causing issues.
Here is UIScreen.main.bounds when I run app on simulation (not testing)
(lldb) p UIScreen.main.bounds
(CGRect) $R0 = (origin = (x = 0, y = 0), size = (width = 390, height = 844))

but in UI testing
(lldb) p UIScreen.main.bounds
(CGRect) $R0 = (origin = (x = 0, y = 0), size = (width = 320, height = 480))

Am I doing something wrong here? Or it is a bug? Thanks you for your response in advance

Environment
swift-snapshot-testing version 1.10.0
Xcode 14.2
Swift 5.7.2
OS: 16.2

Same thing happens for me, I've got approx 1k test cases in the project, and some of them (very few 10-20) fails. I've tried several different approaches: comparing UIViewControllers with ViewImageConfig, comparing UIViews with custom size set and even writing custom snapshotting strategy. All of them somehow get down to the minimal size (width = 320, height = 480)
I finished with commenting out the failing tests 😔


Zombie virus delayed from 20-30 minutes from rule start.

About the PR

Zombie virus (for Patient 0) delayed from 20-30 minutes from rule start.
Limit zombification time to about 10 seconds
Those in hold of Zombification virus take more blunt instead of cellular damage
Warnings of zombification

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Zombie Patient 0 will get 20-30 minutes before they are forced to be a zombie. They can self-harm if they want to turn sooner.
tweak: Players zombifiying will spend less time in Crit, generally 10s at most. A lot less if they fought the initial infection.

Patient 0 turning into a zombie should be tied to an action, not their health state.

Patient 0 turning into a zombie should be tied to an action, not their health state.

It was previously an action. Has that changed?

Patient 0 turning into a zombie should be tied to an action, not their health state.

It was previously an action. Has that changed?

I didn't know that. I haven't changed it.

@GoodWheatley I have confirmed that patient 0 (but not later infected) can decide to succumb using an action (that I didn't know about) and have edited the changelog to match.

Thanks for the review @AJCM-git - I've removed the hardcoding around popups as requested, used an event (a much cleaner approach) and clarified some comments.

I know this is closed now but would it close #16341?

I know this is closed now but would it close #16341? If its possible for someone to test

Yes, this branch resolves #16341 - it was created to solve it.

the 20-30 minute part doesnt happen, i still see bozo turning into zombies 30 seconds into the round

the 20-30 minute part doesnt happen, i still see bozo turning into zombies 30 seconds into the round

It doesn't enforce a minimum time. This pr increased the MAXIMUM time zombies can wait (patient 0 was forced to turn almost immediately before).
Do we want a minimum time? If so, how long?

Easiest would be what traitors do: just delay selection for a few minutes

@deltanedas I posted these suggestions in Discord, if you had any thoughts let me know:
Zombie nerf ideas

Patient 0 cannot turn for 10 minutes after round start. If they suicide, their bodies remain crit / dead until this time has elapsed
Victims need to be dead 10+ seconds before turning, up to 30 (random)
Centcom announcement 8 minutes in (2 minutes before turn) that this is a zombie round (warning, virus detected) or (nearby stations have succumbed)
Bites of 2nd generation zombies have shorter range and lower infection chance
2nd gen Zombies have a chance to die entering crit (in addition to the entering die chance) so many zombies who get knocked down don't just get up again.

It's also possible that I create a second zombie event that is less dangerous and has these features. So we have deadly zombies and survivable zombies. The second is better for midround.
The Centcom announcement is a pretty heavy nerf as it gives staff time to arm up.

instead of fixed 10 mins do it like traitor where its X + random Y mins, X and Y are configurable
maybe announcement should happen 2 mins after people are made initial infected so they cant just ignore it
and separate rulesets for starting vs midround zombies is a very good idea


legends/plots overlapping in documentation

Hey,
I noticed that plots and legends are overlapping, e.g. here: https://squidpy.readthedocs.io/en/latest/auto_tutorials/tutorial_imc.html

thanks for pointing it out. I think this is low priority for now, but definitely to have a look for next release!

Small pitch: I'd redo the plot using barcharts instead of scatterplots - think it will be easier to interpret (esp. reorider the bars in ascending order according to their height). That way, cluster names could be on the x-axis (so no more legend) and  the bars would be colored by the cluster colors.

I think this is a very minor bug and I personally am not in favour of bar charts as the original visualization (as well as similar, see #349 ) are with line plots. I would close this


HDDS-6575. [Multi-Tenant] Clean up unused tenantDefaultPolicyName field in CreateTenantRequest protobuf message

What changes were proposed in this pull request?
This tenantDefaultPolicyName field is no longer in-use and didn't get removed in HDDS-6396.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6575
How was this patch tested?

[x] All existing test cases should pass.

Thanks @errose28 for the review!


Use DI to get the current User instance - Part 2

In #7434 we replaced all the new \Concrete\Core\User\User() calls with $app->make(\Concrete\Core\User\User::class).
We still have some new User() calls (maybe because I missed them, or because new code has been added after I created #7434): let's replace them too.

Can you update this with some fixes for the PHP syntax? Looks like the phpcs tests need to pass now on every PR in the 8.6.x-dev branch.

Done.


[WIP] Fix error/warning while building with android studio

min/target sdk should be defined in build.gradle, not in manifest.xml
updated : it's down to 16 now.
updated 2 : it's down to 1 now.
edit : working on other errors and warnings
Need to test

[x] Touches
[x] Sound
[x] Text input
[x] In-app purchase
[x] Download .obb expansion

If some of those fixes are trivial enough, it might be worth splitting them into another PR that could be merged for 3.1, while we keep the more complex/risky ones for 3.2.

Would likely fix #6626 BTW.

it's now down to 13 issues.
those seem tricky to fix

oh... it's by mistake...

#Weak RNG

../../src/com/google/android/vending/licensing/LicenseChecker.java:68: Potentially insecure random numbers on Android 4.3 and older. Read https://android-developers.blogspot.com/2013/08/some-securerandom-thoughts.html for more info.

  66     private static final int TIMEOUT_MS = 10 * 1000;
  67
  68     private static final SecureRandom RANDOM = new SecureRandom();
  69     private static final boolean DEBUG_LICENSE_ERROR = false;
  70
  71     private ILicensingService mService;

Now it's only 1 issue left.
it says this issue could be a problem on Android 4.3(api 18) or older.
based on https://developer.android.com/about/dashboards/

about 0.4% of devices could have a issue with it. (because we support api 18 or above)
if we must fix this issue, I would like to let someone fix it, I currenlty don't have a idea to fix it.
or we can ignore it by adding TrulyRandom to lint options.
@akien-mga what do you think?

@akien-mga I tested most functions.
it seems pretty stable state.

The changes to platform/android/java/src/com/android/vending come from upstream? Could you document the provenance and process to update them in a platform/android/java/README.md file?

yes. i updated it with https://github.com/google/play-licensing/tree/master/lvl_library/src/main/java/com/google/android/vending
I will add it to READMD.md

@akien-mga updated.

Thanks!


CQL updated

https://wandb.ai/tlab/CORL/reports/-Offline-CQL--VmlldzoyNzA2MTk5

also you need to resolve conflicts

@vkurenkov @Howuhh @Scitator


Categorizes alcohol, comments where recipe is missing

About the PR
Categorizes alcohol by strength, changes some alcohol values, makes Gargleblaster actually blast gargles.
Adds a comment to every cocktail with no recipe yet.
PS: If you'd prefer I move this upstream, I can.


fix: always return up to date IDP and Email Template in Management se…

…rvice
fixes gravitee-io/issues#6590

Here is a Postman collection with environment description in order to validate the fix.
Without this fix, the user creation may fail (404) or the email template may not be the one we send using the REST API call
With this fix, the user creation succeed and the email received uses the custom template.
Uploading SUPPORT_2222.postman.zip…


cannot import name 'six' from 'sklearn.externals' error while installing DXC-Industrialized-AI-Starter

The module is deprecated in version 0.21 and removed in version 0.23. So hotfix need to be done to avoid problem in boot camp usage.

Changes are done and published with new version of AI Starter.


Question: Master post/page view?

This is not an issue, it's just a question. I'm pulling a Patreon site with ~15 pages. Each post is saved (post #description.html, media, etc) and there's a few page#.json files too.
Should there a "master index" file that combines all this in a single scrollable rendered page generated, as I can't find anything (no index.html for example)?

The underlying library supports similar functionality, but this is not implemented in this application. Json files generated by --json command are mostly there for troubleshooting or usage by other programmers.


difficulty with closing ChatGPTEditWithInstructions window.

Seems like there is some difficulty with closing ChatGPTEditWithInstructions window.
instead of closing floating windows, hitting mapped keys triggers the normal mode on the prompt.
For now use :q to exit this windows (just for this window. For the other windows there is no issue)
Here are my mappings :
keymaps = {
    close = { "jk", "kj", "<Esc>" },
    yank_last = "<C-y>",
    scroll_up = "<C-u>",
    scroll_down = "<C-d>",
    toggle_settings = "<C-o>",
    new_session = "<C-n>",
    cycle_windows = "<Tab>",
  },

I have same issues, but for all of the ChatGPT windows

It seems you must be in insert mode first, then you can <C-c>. Esc doesn't work for me

Also my custom keybind to close buffer works for this:
nnoremap <silent> Q     :bd <CR>

I got the same issue

I can't even press <C-c> to close it from insert (or normal) mode, I have to use :bd.

I would also like to press <C-c> to close ChatGPTEditWithInstructions window instead of :q or :bd

PR https://github.com/jackMort/ChatGPT.nvim/pull/141 partially solve this issue mapping both insert and normal mode. I would like to give the user the ability to add its own key maps though. As soon as @jackMort accept this PR, I'll work on that


MINOR: Fix docs; client no longer uses yammer metrics

Yammer metrics is no longer a dependency of the client, but the Monitoring section of the docs contradicts this. For quite some time, clients have been using an internal metrics system unrelated to Yammer/Dropwizard metrics.
This contribution is my original work and I license the work to the project under the project's open source license.

Tagging some folks who have contributed to MetricsReporter.java: @ijuma @jkreps @arrawatia

Also pinging @guozhangwang for review.

Updated wording here to mention minimizing transitive dependencies and to introduce "Kafka Metrics".

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6119/
Test PASSed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6104/
Test PASSed (JDK 8 and Scala 2.12).


Parsing the serialized string in one shot

This MR would like to slightly improve performance by parsing the whole buffer in one go as unique fromCharCode String operation.

OK, it looks like in Chromium they are using btoa(fromCharCode(new UInt8Array(pngBuffer))) and in WebKit the arg size is an int32 which is plenty of chars ... I believe these days it's safe to just expect variable arity to deal with huge amount of arguments so that hacks like these needed in 2011 are not relevant anymore http://webreflection.blogspot.com/2011/07/about-javascript-apply-arguments-limit.html


Add publish command

Since users want to publish their static site to Github after generating their own profiles,
we should support the requirement adding publish command.


Documentation on creating a release

I snooped around the issues as best I could and didn't find anything-- As far as I can tell right now react-native relies on having a development server to run; could we get some information on when we can expect to be able to have standalone applications? Even if it's far off it would be awesome to get a bit more guidance / heads up in the docs.

Yup, we need to document this officially. In the meantime the instruction is in https://github.com/facebook/react-native/issues/240


Fixing excessive backoff timeout

It seems our CI just runs forever today. This PR tackles this:

Specifies request timeouts to all GET requests in lib.py
Fixes backoff sleep timeout of 1.5+ mins (when Semantic Scholar rate limit is 90), a bug introduced in https://github.com/blackadad/paper-scraper/pull/47
Upgrades ThrottledClientSession for readability and configurability, with test

Documenting formulas using constant class variables
Removes one-time configuration from inside try-except
Removes asyncio.TimeoutError suppression on close

Closing in favor of https://github.com/blackadad/paper-scraper/pull/69


Run plugin on bundle if no options.file

rollup-plugin-bundle-size breaks when bundling multiple code-split entry points (see output below).
npm run build

><EMAIL_ADDRESS>build /Users/fm813/github/manzt/zarr-lite
> rollup -c

loaded rollup.config.js with warnings
(!) Unused external imports
default imported from external module 'path' but never used
default imported from external module 'chalk' but never used
default imported from external module 'maxmin' but never used

src/index.js, src/indexing.js, src/httpStore.js → dist...
[!] (plugin rollup-plugin-bundle-size) TypeError: The "path" argument must be of type string. Received undefined
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at validateString (internal/validators.js:124:11)
    at Object.basename (path.js:1156:5)
    at Object.generateBundle (/Users/fm813/github/manzt/zarr-lite/node_modules/rollup-plugin-bundle-size/index.js:9:32)
    at /Users/fm813/github/manzt/zarr-lite/node_modules/rollup/dist/shared/rollup.js:18757:25
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `rollup -c`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/fm813/.npm/_logs/2020-12-09T16_35_26_761Z-debug.log

In this context options.file is undefined. This PR checks if options.file is present and if so will output the same as before. But if not, it will iterate through the bundle keys.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Project based settings

Hi
i'm using your tool everyday but there is this little thing that I either don't understand or not available.
We typically use new clients and port settings for every project,
So we have files called .p4config that contains:
P4PORT=:
P4CLIENT=
could you tool detect these files and use these settings so when we jump from one project to another
we do not need to start VSCode with new settings?
Otto

Hey Otto,
By default, the extension will let P4 decide what are the client and port settings. From my understanding, P4 will go up in the folder hierarchy until it find a specific filename, specified from the environment variables (usually .p4config).
If you already have such a file, it should already be working. I would suggest you try to run perforce.info - Display client/server information to get more info on what's happening.
Now, you can manually set perforce.client in your vscode user or workspace settings, but unfortunately the port is not exposed yet.

You are right, I accidentally removed my P4CONFIG variable so it no longer detects the .p4config files.
With that variable set everything works like a charm, thanks for the info and quick response.


Added all W3C input events to f7-input component. Bumped version.

Hello Vladimir,
keypress not working on mobile and i can't cancel keydown event.
beforeinput event required to me, because of this, i updated f7-input component's events. I used W3C docs.
I bumped the version to 0.7.7 and runned gulp dist command.

Merged, thanks!

By the way, there is a trick for the case you need some native HTML event if there is no such in any component by adding ".native" to the event name. You can add it like <f7-input @blur.native=...>

I tested direct events like keydown, but it not linked directly to  element, it linked to . I didn't tested @blur.native. if it's working with input element, my job completely not required 🤕 i can revert my changes if you want.
Thank you so much for your big efford Vladimir. I really respect you.

Ah right, it will link it to component parent element. It is good, no need to revert


[SDA-6886] Added redhatmanaged=true tag to operator roles in manual mode

SDA-6886

/lgtm

/lgtm


[Net Core 2.0] AccessDeniedPath ignored in cookie authentication

When using cookie authentication, the AccessDeniedPath config parameter seems to be ignored.  This has worked previously (though I haven't done the testing to determine if the behavior changed in 2.0 or 2.0.x).  It's always possible that I'm not doing it right, and I welcome correction if that's the case.
I've also noticed that the current cookie sample project exhibits this same problem (configured "/account/denied", but the action in AccountController is AccessDenied).
https://github.com/aspnet/AspNetCore/tree/master/src/AuthSamples/samples/Cookies
Here's the startup configuration in a trivial test app:
public void ConfigureServices(IServiceCollection services)
{
    services.AddMvc(config =>
    {
        var policy = new AuthorizationPolicyBuilder()
            .RequireAuthenticatedUser()
            .Build();
        config.Filters.Add(new AuthorizeFilter(policy));
    });
    services.AddAuthentication(CookieAuthenticationDefaults.AuthenticationScheme).AddCookie(coptions => new CookieAuthenticationOptions()
    {
        LoginPath = new PathString("/account/login"),
        AccessDeniedPath = new PathString("/account/forbidden"),
        ExpireTimeSpan = TimeSpan.FromMinutes(5),
        Events = new CookieAuthenticationEvents()
        {
            OnValidatePrincipal = async context => {  await ValidatePrincipal(context); }
        }
    });
}

Expected behavior: After logging in and when requesting a protected resource without the required role (i.e. action decorated with [Authorize(Role = "NotMyRole")]), should be redirected to configured AccessDeniedPath (i.e. "/account/forbidden").
Actual behavior: After logging in and requesting a protected resource without the required role, application redirects to "/account/accessdenied".
Workaround: simply handle "/account/accessdenied" by route or coded action.
TrivialWebAppAccessPathDeniedIssue.zip
To use the attached sample, just run it and then attempt to go to the About page.

Recreated this issue in the active aspnetcore repo and will close it here.


No such file or directory: 'combined_data/absa/ae/rest/train.txt'

Hi,
Facing below error,
"No such file or directory: 'combined_data/absa/ae/rest/train.txt'.  "
Kindly provide missing datasets

no idea why they have been removed, but you can git them from the git history:
https://github.com/rit-git/Snippext_public/tree/60620d013541d7a662233543809aee1e79eebc37

@yochail : yeah, thanks


Feat (number) / inRange function

Description
The equivalent of lodash's inRange function. lodash.inRange
Checklist

[x] Changes are covered by tests if behavior has been changed or added
[x] Tests have 100% coverage
[ ] If code changes were made, the version in package.json has been bumped (matching semver)
[x] If code changes were made, the yarn build command has been run and to update the cdn directory
[x] If code changes were made, the documentation (in the /docs directory) has been updated

Resolves
None. It is a new feature.

Hey @rayepps, I added overloading functions. If you have time to review, we can proceed the PR.

Thank you @mertdy finally merged it 🙃 🎉 will go out in next release of v12.0.0 (likely today)


Only change authorizer uris on TOKEN type authorizers

Fixes #51
The plugin should leave all non-Lambda authorizers as they are. Only Lambda authorizers have an URI that needs to be adapted for alias deployments.

Coverage decreased (-0.2%) to 50.52% when pulling 40f7bda32a4c02670a6dc5a07afb24f90d970ad5 on fix-cognito-authorizers into c2d4dba62d48f0b97684e2b8b17ac2c7a63a7903 on master.

Coverage increased (+3.4%) to 54.054% when pulling 6768270d947c5effd78591f226516ba1ec0aa48a on fix-cognito-authorizers into c2d4dba62d48f0b97684e2b8b17ac2c7a63a7903 on master.

Coverage increased (+3.4%) to 54.054% when pulling bab861b857dd138be928464c5945399286ef91ad on fix-cognito-authorizers into c27ce051c87436ab3f4daee3df28d983832249f5 on master.