Problem with [Function] patch, updates DOM incorrectly.

Hello,
So, i am trying to programaticaly generate a Virtual DOM through some other libraries, and render everytime i push some specific changes to the BODY.
But currently i am having issues with duplicated content and the ability of virtual-dom/patch to create the correct DOM representation.
Example:

hey

hey

I make the change to the following markup:

hey

hey

This is the result:

I get duplicated content, even i change the entire DOM to something different, the update does not happens.
It ads the content to the DOM but does not update/remove.
Everything is based off the Body tag, so its a single ROOT being rendered.

Help would be really appreciated, or if someone has faced a similar issue!

I'm having a similar issue, but I'm not able to reproduce it with a trivial example (so I suspect it may be in my code someplace.)
The interesting bit is that I've just come back from a two-week vacation, and I'm sure it was working before I left.  (I know, I know...)  Maybe a dependency broke?
Some screenshots that may help:
pre-update, paused in patchRecursive for the patch that failes -- you can see here that the patch has the correct target node, but index[nodeIndex] returns the wrong target.
post-update -- this is the failed update, which given the previous screenshot is what we'd expect.  The 'footer' is replaced with the new content.
I'm trying to figure it out... I don't see too much activity on this repo so not sure if we'll get any help.  I'll update here if I make any progress.

OK, figured out my issue, and it was my fault.  I was trying to be clever and automatically insert style nodes into my components, but in some cases the nodes had no content.  I tried to reproduce it with an empty div but that doesn't cause a problem; must be something about an empty style tag that throws off the indices in the patching process.
Check to make sure you aren't adding any garbage to the DOM that might be confusing the patching machinery.  I'd still call this a bug, but at least I can work around it.

@dashkb
In my case i had a different issue. The problem for me was that i was doing Tree dehydration and then pushing it into the server and then sending it back through sockets and then hydrating it. But in that case when i was doing Diff -> Patch, i had to give as an Argument the previous Tree to compare then, the Patch would generate the correct output.
At the same time, i also included the "key" property in the VNodes, so that might have fixed it as well, but for now its working fine.

Hey @sadiqevani! I tried following the steps from the issue above, but I couldn't replicate the issue. Can you share the problematic code?

Hey @jbucaran
I figured it out! The problem as mentioned above, was simply i had to compare the previous state, of the AST Tree with the new one and then apply the patches.
But i decided to go with VueJS and its Parser/Compiler for the thing i am working. They have done almost everything i was trying to do with the Virtual-Dom 😃


Add responsiveness to the home page.

Please assign this issue to me as I am a participant of LMG-SoC.

I am a participant of LGMSOC 2021.
Could you please assign this issue to me.

I would like to resolve this issue under LMG SOC 21. Please assign it to me. Thank you.

I would like to add some more songs
please assign it to me under LGM SOC.!

I would like to Add responsiveness to the home page.
please assign it to me under LGM SOC.!
@niloysikdar

/assign @nilisha-jais

Can you please elaborate more on the changes you want to make? @akitiashritha

I want to add the navbar and make a mobile view of it and as well want to modify the homepage
@niloysikdar

Seems like someone is already working on that. Please have a look at the open issues. If not, then we will definitely assign you.

This issue is assigned to me
On Mon, Jun 14, 2021, 14:51 Niloy Sikdar @.***> wrote:

Seems like someone is already working on that. Please have a look at the
open issues. If not, then we will definitely assign you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nilisha-jais/Musicophilia/issues/21#issuecomment-860535959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARVI6WLWH7AGTOBOHCJ5JATTSXC3DANCNFSM46EPEFAA
.

I'm almost done with that as it was assigned to me yesterday i started working on it but its ok i will just contribute my part of code
@niloysikdar  could you please suggest me your opinion

Then you go for it , I have no problem with it.
On Mon, Jun 14, 2021, 14:58 akitiashritha @.***> wrote:

I'm almost done with that as it was assigned to me yesterday i started
working on it but its ok i will just contribute my part of code
@niloysikdar https://github.com/niloysikdar could you please suggest me
your opinion
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nilisha-jais/Musicophilia/issues/21#issuecomment-860540039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARVI6WNFXTGRARB3FW32MG3TSXDTNANCNFSM46EPEFAA
.

Then you go for it , I have no problem with it.
…
On Mon, Jun 14, 2021, 14:58 akitiashritha @.***> wrote: I'm almost done with that as it was assigned to me yesterday i started working on it but its ok i will just contribute my part of code @niloysikdar https://github.com/niloysikdar could you please suggest me your opinion — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#21 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARVI6WNFXTGRARB3FW32MG3TSXDTNANCNFSM46EPEFAA .

okay !!!

No worries @jugeshraghav , @akitiashritha . We all can collaborate and contribute to the project. But as this issue was assigned to @akitiashritha , sorry but you have to wait @jugeshraghav . And you can go ahead @akitiashritha .

Ya sure 😇
On Mon, Jun 14, 2021, 15:05 Niloy Sikdar @.***> wrote:

No worries @jugeshraghav https://github.com/jugeshraghav ,
@akitiashritha https://github.com/akitiashritha . We all can
collaborate and contribute to the project. But as this issue was assigned
to @akitiashritha https://github.com/akitiashritha , sorry but you have
to wait @jugeshraghav https://github.com/jugeshraghav . And you can go
ahead @akitiashritha https://github.com/akitiashritha .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nilisha-jais/Musicophilia/issues/21#issuecomment-860544585,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARVI6WMIN3PJNA6EOXD37T3TSXEO7ANCNFSM46EPEFAA
.

No worries @jugeshraghav , @akitiashritha . We all can collaborate and contribute to the project. But as this issue was assigned to @akitiashritha , sorry but you have to wait @jugeshraghav . And you can go ahead @akitiashritha .

okay

This issue has been taken care of.


feat: add getCurrentPosition to component's ref

Summary
This function retrieves and returns the precise current position of the video playback, measured in seconds.
Motivation
The goal is to provide users with the ability to access the current position of the video playback. This allows for the creation of custom controls like forward and rewind buttons, enhancing user experience.
Changes
A new function getCurrentPosition has been added to the component's ref.
Test plan

Looks good to me, Just one note, I would prefer having the time in msec instead of sec. @KrzysztofMoch @YangJonghun what do you think of this point ?

As far as I know, onProgress callback exists for the same reason. I wonder for what purpose this method was implemented.

@YangJonghun  you're correct! In situations where the user only needs to get the current position when interacting with custom controls, such as the rewind and forward buttons, it's not necessary to use a callback function that is called every progress update interval.
Instead, the user can use this approach that is only called when the user clicks the rewind or forward buttons. When a button is clicked, you can get the current position and then seek forward or backward based on the button that was clicked.

@seyedmostafahasani
That makes sense!
However, I think it might be a bit difficult to use it for precise work due to the limitations of old architecture, since the current time value keeps changing on playback.
It will be great later when we're done with the new architecture :)
Thanks for your explanation.👍

@seyedmostafahasani
Oh sorry to late response.
I like second(specifically, second as a float value).
I prefer the same format as the value returned by onProgress, and I think many developers will be familiar with the web's video.currentTime format

I completely agree with you about second.

As C developer I prefer working with integer, but if you are ok for sec, then OK for me !

@freeboub @KrzysztofMoch @seyedmostafahasani
I think we need to deprecate this feature
ref command with return value not supported by Fabric
(save command also have same problem)

@YangJonghun I think we can use them with Fabric, but an important point is that these are deprecated methods on the native or JS side. Utilizing deprecated methods on the native or JS side may result in these functions not working correctly. I recommend checking out react-native-blurhash, as it employs Fabric. This library uses functions that return values via promises, such as the encode function, which you can observe in these URLs.
https://github.com/mrousavy/react-native-blurhash/blob/master/src/index.tsx
https://github.com/mrousavy/react-native-blurhash/blob/master/android/src/main/java/com/mrousavy/blurhash/BlurhashModule.kt

@seyedmostafahasani
BlurHash is NativeModule.
TurboModule and Fabric are different. TurboModule can be return values
What I meant was that it's not possible to use a View as a ref and return it.
Currently we use save or getCurrentPosition in react-native-video is to reference a reactTag in NativeModule(VideoManager).
We are using findNodeHandle for above behavior, but findNodeHandle will be deprecated, so it is not recommended to continue using this method. We need to migrate to command method to replace it.
However command does not support return or callback functions.

@YangJonghun Oh yes, you're right. Thanks for the detailed explanation. I was referring to the deprecated method findNodeHandle. Could you please take a look at the react-native-view-shot library? I think this library could help us.

@seyedmostafahasani
Thanks for give me reference
But unfortunately react-native-view-shot also using findNodeHandle now. (link)🥲
I guess I'll have to keep it for now and find a tricky way around it. Because we don't know when RN team get rid of findNodeHandle.
fyi) https://github.com/reactwg/react-native-new-architecture/blob/main/docs/enable-libraries-prerequisites.md

@YangJonghun anytime!
Did you see this PR
I agree with you, we can keep it for now. Also, if I find a better solution, I will share it with you.


Slackcat not working properly

Hi! I tried to use Slackcat to upload some files to Slack channel, but got some problems. I set SLACK_TOKEN to proper token I got, then I call echo "hello" | slackcat -c my-channel, but it results in error:
/Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/slackcat-0.2.5/bin/slackcat:17:in `block in get_objects': error retrieving channels from channels.list: missing_scope (RuntimeError)
    from /Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/httparty-0.13.6/lib/httparty/response.rb:35:in `tap'
    from /Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/slackcat-0.2.5/bin/slackcat:16:in `get_objects'
    from /Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/slackcat-0.2.5/bin/slackcat:22:in `channels'
    from /Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/slackcat-0.2.5/bin/slackcat:118:in `block in <top (required)>'
    from /Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/slackcat-0.2.5/bin/slackcat:117:in `map'
    from /Users/jenkins/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/slackcat-0.2.5/bin/slackcat:117:in `<top (required)>'
    from /Users/jenkins/.rbenv/versions/2.1.3/bin/slackcat:23:in `load'
    from /Users/jenkins/.rbenv/versions/2.1.3/bin/slackcat:23:in `<main>'

File upload with -m results in the same error. What should I do to fix that?

@silvansky Interesting, I've not see missing scope before, normally I'd expect invalid_auth if there was an auth problem. I will spend some time this afternoon and see if I can reproduce the problem here.

Got the same problem here, any news on it?
Thanks


Thumbnails required?

I have been looking for some time now after a package that can replace react-native-fast-image. This looks really good but i see in your documentations that thumbnails is required? Is it not possible to use it just like a Image replacement with the caching functionality?
There is not alot of maintained image packages out there right now that has immutable caching like this and is maintained.
Best regards.

@xerdnew please try using it without thumbnail as in the example. We will also change the docs


 We just OSSed http://github.com/googlecloudplatform/cloud-run-hello, can we re-do that? Also the quality on this one is pretty low –it would be good to re-shoot with a slightly higher quality maybe.

BTW, I'm just waiting on https://github.com/GoogleCloudPlatform/cloud-run-hello/issues/1

I've updated the gif to use the https://github.com/googlecloudplatform/cloud-run-hello repo.

Updated to use a static image on the readme

Looks good, though bit too much anti-aliased, I think I might recapture this later on a high-DPI display like Retina Display to avoid blurriness.


Don't force-install xargo

Previously miri used cargo install xargo -f which shouldn't be
necessary anymore since cargo install will now upgrade without -f.
The only reason I can see to use -f is from the cargo docs:

This is also useful if something has changed on the system that you
want to rebuild with, such as a newer version of rustc.

See the discussion on Zulip.

Sounds good, thanks!
@bors r+

:pushpin: Commit e9b8693aaae84d317d92de7c6438c3117af83122 has been approved by RalfJung

:hourglass: Testing commit e9b8693aaae84d317d92de7c6438c3117af83122 with merge c2fd8f05cc71892cc2875a6589ca37b37991beb6...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing c2fd8f05cc71892cc2875a6589ca37b37991beb6 to master...


Installation failing on macOs.

Hi,
I'm having some issues installing the package on macOs:
Kali:rez kelsolaar$ python install.py -v /Users/kelsolaar/.rez

installing rez to /Users/kelsolaar/.rez...
/Users/kelsolaar/.rez/bin/python
-c
import sys, pip; sys.exit(pip.main(["install", "--ignore-installed"] + sys.argv[1:]))
setuptools
pip
Traceback (most recent call last):
  File "install.py", line 133, in <module>
    create_environment(dest_dir)
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv.py", line 993, in create_environment
    install_wheel(to_install, py_executable, search_dirs)
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv.py", line 961, in install_wheel
    'PIP_NO_INDEX': '1'
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv.py", line 903, in call_subprocess
    % (cmd_desc, proc.returncode))
OSError: Command /Users/kelsolaar/.rez/bin/python -c "import sys, pip; sys.exit(pip.main([\"install\", \"--ignore-installed\"] + sys.argv[1:]))" setuptools pip failed with error code 1

While investigating I noticed that the output traceback from the sub-process is eaten, which is confirmed by printing all_output (print('\n'.join(all_output))) after that line here: https://github.com/nerdvegas/rez/blob/master/src/build_utils/virtualenv/virtualenv.py#L894
Kali:rez kelsolaar$ python install.py -v /Users/kelsolaar/.rez
installing rez to /Users/kelsolaar/.rez...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv_support/pip-1.5.6-py2.py3-none-any.whl/pip/__init__.py", line 9, in <module>
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv_support/pip-1.5.6-py2.py3-none-any.whl/pip/log.py", line 8, in <module>
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv_support/pip-1.5.6-py2.py3-none-any.whl/pip/backwardcompat/__init__.py", line 66, in <module>
  File "//anaconda/lib/python2.7/urllib2.py", line 94, in <module>
    import httplib
  File "//anaconda/lib/python2.7/httplib.py", line 80, in <module>
    import mimetools
  File "//anaconda/lib/python2.7/mimetools.py", line 6, in <module>
    import tempfile
  File "//anaconda/lib/python2.7/tempfile.py", line 32, in <module>
    import io as _io
  File "//anaconda/lib/python2.7/io.py", line 51, in <module>
    import _io
ImportError: dlopen(/Users/kelsolaar/.rez/lib/python2.7/lib-dynload/_io.so, 2): Symbol not found: __PyCodecInfo_GetIncrementalDecoder
  Referenced from: /Users/kelsolaar/.rez/lib/python2.7/lib-dynload/_io.so
  Expected in: flat namespace
 in /Users/kelsolaar/.rez/lib/python2.7/lib-dynload/_io.so
Traceback (most recent call last):
  File "install.py", line 133, in <module>
    create_environment(dest_dir)
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv.py", line 993, in create_environment
    install_wheel(to_install, py_executable, search_dirs)
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv.py", line 961, in install_wheel
    'PIP_NO_INDEX': '1'
  File "/Users/kelsolaar/Documents/Development/ThirdParty/rez/src/build_utils/virtualenv/virtualenv.py", line 903, in call_subprocess
    % (cmd_desc, proc.returncode))
OSError: Command /Users/kelsolaar/.rez/bin/python -c "import sys, pip; sys...d\"] + sys.argv[1:]))" setuptools pip failed with error code 1

I would suggest printing the output in the final raised exception.
Now coming back to my issue, I'm using multiples Python interpreters (System, Homebrew, Anaconda), my main one being the Anaconda one, however I noticed that the virtualenv you create for rez is using macOs shipped Python which is incompatible with the Anaconda one, any direct way to specify the interpreter to use?

Actually the question on being able to specify the interpreter rez uses is also valid on Windows.

Hey Thomas,
The python interpreter used in the rez install should have no bearing on
the python interpreter used anywhere else (including packages into rez
packages). Are you seeing otherwise? To control the python interpreter that
the install uses, just use a specific python interpreter when running
install.py, that should be the one that gets copied(?) into the
virtualenv-based installation.
Thx
A
On Wed, Feb 22, 2017 at 8:10 AM, Thomas Mansencal<EMAIL_ADDRESS>wrote:

Actually the question on being able to specify the interpreter rez uses
is also valid on Windows.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/387#issuecomment-281481350, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABjqSuky7fsBEBH03yFIRDXGxLGNaQG-ks5re1KugaJpZM4MGxcs
.

@nerdvegas, Allan: Given my system, rez is not picking the proper python interpreter, which should be the Anaconda one:
Kali:~ kelsolaar$ which -a python
/anaconda/bin/python
/usr/local/bin/python
/usr/bin/python

This is my default python and the one used for a vanilla shelf:
Kali:bin kelsolaar$ python
Python 2.7.11 |Anaconda custom (x86_64)| (default, Jun 15 2016, 16:09:16)
[GCC 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.11.00)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://anaconda.org
>>>

and this is the binary picked by rez on installation:
Kali:bin kelsolaar$ pwd
/Users/kelsolaar/Applications/rez/bin
Kali:bin kelsolaar$ ./python
Python 2.7.10 (default, Oct 23 2015, 19:19:21)
[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-7<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>>

I can likely submit a PR for the proper exception traceback too as it is quite critical to me to understand what is going on under the hood, need to check with upstream hierarchy before though :)
Cheers,
Thomas

Ack, if the installation is using a different interpreter to the one you
used on install.py, the issue is likely somewhere in virtualenv :/.
The installation process (under the hood, not for the user) is painful to
say the least. I've started to think that perhaps we could direct efforts
into platform-specific, simpler installers. For example I'm pretty sure I
can implement a much simpler linux-specific installation that doesn't use
virtualenv (and I'm pretty sure there's a small performance gain that way
too). That's been on my list for a while.
A
On Thu, Feb 23, 2017 at 8:16 AM, Thomas Mansencal<EMAIL_ADDRESS>wrote:

@nerdvegas https://github.com/nerdvegas, Allan: Given my system, rez
is not picking the proper python interpreter, which should be the Anaconda
one:
Kali:~ kelsolaar$ which -a python
/anaconda/bin/python
/usr/local/bin/python
/usr/bin/python
This is my default python and the one used for a vanilla shelf:
Kali:bin kelsolaar$ python
Python 2.7.11 |Anaconda custom (x86_64)| (default, Jun 15 2016, 16:09:16)
[GCC 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.11.00)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://anaconda.org>>>
and this is the binary picked by rez on installation:
Kali:bin kelsolaar$ pwd
/Users/kelsolaar/Applications/rez/bin
Kali:bin kelsolaar$ ./python
Python 2.7.10 (default, Oct 23 2015, 19:19:21)
[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-7<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.>>>
I can likely submit a PR for the proper exception traceback too as it is
quite critical to me to understand what is going on under the hood, need to
check with upstream hierarchy before though :)
Cheers,
Thomas
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/387#issuecomment-281806623, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABjqSud1SVcEU7z7U4hVcysEDJVVEUf1ks5rfKWygaJpZM4MGxcs
.

The installation process (under the hood, not for the user) is painful to
say the least.

Yeah I saw a lot of magic happening there :) Another related issue was Qt for rez-gui, I had to manually copy the needed packages to the rez virtual environment as it wasn't picking the one from my interpreter. I reckon that platform specific installers would be useful in the long term, especially from a provisioning / orchestration stand point, it would make deployment easy.
By the way I hope you don't mind the multiple issues I raised lately, if it can comfort you, even with its quirks I'm very happy with the tool so far and it is hopefully the solution we will adopt. Side question, do you have a Slack, Gitter, IRC, your_preferred_chat_client team for rez somewhere?

@KelSolaar did you managed to find a workaround? I basically have the same setup as yours. OSX and Conda

This issue is quite old. I'll consider it "fixed".


Scanner: Fix Scientific Notation

Scientific notation is not currently supported.

This is fixed in pull request https://github.com/benbjohnson/megajson/pull/20


ARC upgrade error

The app errored during latest upgrade.
Logs
Initializing upgrade
[1016] Checking if upgrade is required.
[1016] Deleting new databases if exists.
Maximum IPC message size exceeded.
UnknownError: Maximum IPC message size exceeded.

Versions
App: <IP_ADDRESS>5-stable
Chrome: 51.0.2704.106
Platform: Linux (-)

Hi, Thank you for the issue report. I'm on it right now and expect fix very soon.
The app will work with old setup. Sorry for the upgrade screen enabling itself every time when you open the app. I'll fix that too.
And I'm closing it as a duplicate of #830. But thank you again for the report.


OmniSharp does not set SolutionDir variable

Environment data
dotnet --info output:
.NET Command Line Tools (2.1.202)

Product Information:
 Version:            2.1.202
 Commit SHA-1 hash:  281caedada

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17134
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.202\

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.9
  Build    : 1632fa1589b0eee3277a8841ce1770e554ece037

VS Code version: Latest Stable
C# Extension version: 1.15.2
Steps to reproduce
Setup a CSproj with something like this for example:
  <PropertyGroup>
     <CodeAnalysisRuleSet>$(SolutionDir)\Example.ruleset</CodeAnalysisRuleSet>
     <Features>IOperation</Features>
     <NoWarn></NoWarn>
  </PropertyGroup>

Expected behavior
The extension should at least set the variable to your workspace root or have the option too. This is the default behavior of Visual Studio 2017 and will cause issues with anyone that thinks they can easily move to VSCode.
Actual behavior
You'll get errors that it can't find the file on build:
C:\Program Files\dotnet\sdk\2.1.202\Microsoft.CSharp.CurrentVersion.targets(134,9): warning MSB3884: Could not find rule set file "\Example.ruleset". [E:\Development\Example\Source\Cooker\Cooker.csproj]
    1 Warning(s)
    0 Error(s)

As a user fix, you can add this to your build task:
    "options": {
        "env": {
            "SolutionDir": "${workspaceFolder}"
        }
    },

I also believe it doesn't set the ConfigurationName

Environment variable can be set in tasks.json, but it doesn't seem like it can be set for running tests using Code Lens. Currently, "env" is not an option in csharp.unitTestDebuggingOptions config.
reference: run and debug unit tests

Environment variable can be set in tasks.json, but it doesn't seem like it can be set for running tests using Code Lens. Currently, "env" is not an option in csharp.unitTestDebuggingOptions config.
reference: run and debug unit tests

A lot has changed with O# project loading since this issue was opened. If you are still having problems, please open an issue at https://github.com/OmniSharp/omnisharp-roslyn/issues


Installing package with setup.py-based local dependencies creates wrong venv

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.

OS version and name: macOS 10.13.5
Poetry version: 0.11.4
Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/7755ad36b3e1fb47db67e9fbe5fd4807

Issue

When installing a package with local directories as dependencies, and one of them is setup.py-based rather then Poetry-based, the wrong virtualenv is created.
This caused by DirectoryDependency() calling Venv.create() for a setup.py-based package, which exports VIRTUAL_ENV into the environment, so that the subsequent call to Venv.create() from VenvCommand() for the actual package being installed returns a wrong virtualenv.
Simply removing VIRTUAL_ENV from the environment at the end of Venv.create() when no name argument is passed seems to solve the problem (or adding export=False with VenvCommand() calling Venv.create(export=True)), but I'm not sure what's the logic behind Poetry's use of the VIRTUAL_ENV variable, so I won't make a pull request.

And while I'm at it, I'm not sure if leaving the dependency's virtualenv that was created just for the purpose of parsing setup.py in the cache dir is the right approach.

A lot has been changes internally (especially on the develop branch) so closing this as it might have been fixed. Feel free to reopen if this is not the case.


Add Zach Dev to Contributors list

Thanks for helping me learn how to contribute!

Hi  @zribera18, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Bump version

@jakemac53 @sigmundch
Need to increment the version in order to publish the changes, right? I know I did this edit once before but it seems to have gotten lost.

Can you update the changelog as well, otherwise lgtm

Done. Do I need another LGTM or should I just merge it?

do you need one of us to publish or can you do that?

I also just added a tag v0.3.6+2

I'm trying to publish. It just keeps incrementing the timer. After 3
minutes I killed it and started over. It is up to 2m again and still going.
Is that normal?
On Tue, Mar 29, 2016 at 1:47 PM, Jacob MacDonald<EMAIL_ADDRESS>wrote:

I also just added a tag v0.3.6+2
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/smoke/pull/30#issuecomment-203094311

no its not :(. I can try in a little bit.

You should be able to use the git tag though

Ok, published


Controlling nodes positions

I'm currently working on a project that has been using Dagre for a long time. The lib is fantastic and always met the project needs until now but there is one new feature that i'm currently unable to implement.
I would like to know if in the current version of the lib is there any way i can control the position of nodes to keep them aligned.
For example:

I would like to keep some of the nodes in the graph above aligned like in the picture below:

I already tried using the clustering in the Dagre-D3, but the nodes inside the cluster are still scattered:

I think this would improve greatly the understanding of some graphs, even if it increased the number of crossing edges as a side effect.
Is there anyway i can achieve that with the current version of the lib ?
If not, how could i contribute changing the lib to achieve that ?

This is the kind of thing that WebCola is really good at - in fact they have an alignment constraint for exactly this.
However, it's a completely different layout model and does not do directed graph drawing as well as dagre.
The Holy Grail of graph layout is to combine all the features of all the layouts but it's probably impossible. 😁

hello，I have the same problem.Have you solved it?

Could you change the rankdir to LR and then give nodes 1-8 the same rank, node 9 a lower rank, and node 10 a higher rank. And maybe give the edges linking adjacent nodes, e.g. node4-node5, higher weigths to give them priority over longer edges such as node4-node7.

Hi,
I'm meeting the same problem. Could Dagre add API to allow manually specify certain node and edge's position first, and then the algorithm calculate other node/edge positions based on it?
In my thinking the positioning function have 2 types:

absolute positioning, which is directly specify one node's x/y value, or specify an edge's middle joint point (head and tail points are specified by node which the edge attaches to.
relative positioning, which is to specify one node's relative x/y value relative to another node.

In this way the user can have strong control over layouts, such as specifying node position in graph definition such as mermaid, or even drag&drop to fix certain node at ideal position and let the algorithm to calculate layout of other nodes based on it.

Is there any solution yet?

Is there any solution yet?

For the alignment of nodes you can try out nodeOrigin={[0.5, 0.5]} as a prop to reactflow itself. This will avoid the unnecesary bending of the edges.
For example:
<ReactFlow
nodes={nodes}
edges={edges}
onConnect={onConnect}
nodeOrigin={[0.5, 0.5]}
fitView

reviving this issue now that dagreJS is finally getting some love


BIGTOP-3313: Bump solor to 6.6.6

Bump solr to latest version of 6.x branch.
Also updated maven repo to https access
Change-Id: I7f09d72d51d6c89b172e796189261590b37444c4
Signed-off-by: Jun He<EMAIL_ADDRESS>

Passed build and smoke tests on local server (x86 and arm64)

+1, Build succeeded on CentOS 7 and 8, Debian 9 and 10, and Ubuntu 16.04 and 18.04.

+1.


Topics added using API don't display on explore page

Gitea version: 1.12.3
Git version: 2.7.4
Operating system: Ubuntu 16.04 (xenial) LTS
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Description
If you use the /repos/{owner}/{repo}/topics/{topic} API route to add a topic to a repository it will not be displayed on the explore page, but will show up properly on the repository page.
Steps to reproduce

Go to the API route on your Gitea installation (/api/swagger)
Click the Authorize button and provide some form of authentication (if the repository is private)
Expand the "Add a topic to a repository" section (under the "Repository" heading)
Click the "Try it out" button, fill in the fields, and click "Execute"
Go to the explore page and find the repository that you have added a topic to (notice that the topic is not displayed)
Click on the repository to go to its repository page (notice that the topics were added successfully)

Screenshots

This issue does not apply to the the /repos/{owner}/{repo}/topics API route. If you use that route to update the entire topic list for a repository the new topics will show up on the explore page after a page refresh.

I also just confirmed that this problem occurs with repositories that have previously added topics.
Previously I had only tested this with repositories that didn't have any topics at all.

Another interesting thing to note, if you add a topic using the web interface or using the /repos/{owner}/{repo}/topics API route the "last updated" time gets changed which will bump the repository to the top of the explore page. However, using the /repos/{owner}/{repo}/topics/{topic} API route will add a new topic to the list **but will not change the "last updated" value or bump the repository up to the top of the explore page.

I think I've figured out what's happening. Topics can be manipulated with multiple funcs, but not all are equal. 😅
Using the topics endpoint or the UI will invoke
https://github.com/go-gitea/gitea/blob/502e38c33c08cc6e9e7df86a934eb8b47ae26c49/models/topic.go#L242-L318
Whereas the API invokes
https://github.com/go-gitea/gitea/blob/502e38c33c08cc6e9e7df86a934eb8b47ae26c49/models/topic.go#L212-L224
https://github.com/go-gitea/gitea/blob/502e38c33c08cc6e9e7df86a934eb8b47ae26c49/models/topic.go#L100-L129
The top code invokes the bottom, but also seems to update the repo table as well, because we store topics on the repo table as well as the topics table. (Presumably to save on queries later??)

@jolheiser Hmm, I see what you mean. That's tricky. Ideally you'd have one function to rule them all (Lord of the Rings style ;) ). Do you think this will just be a simple function call replacement? Or are there other complications?

@mooror This should be easy to fix. Unfortunately we'll need to retrieve the topics for a repo, add/delete a topic, then serialize it back to the DB unless there's more context further up.

Unfortunately we'll need to retrieve the topics for a repo, add/delete a topic, then serialize it back to the DB unless there's more context further up.

Are you saying we will have to up the number of database queries for this API route?

At a glance I think so. Unless we have the data available earlier in the code. I've only taken a glance so far.

@jolheiser In this case, you gotta do what you gotta do I suppose.
Also I wish I could be of more use to you in fixing this issue but I haven't fully learned GO lang quite yet. Its on the list :smile:

@jolheiser:
Why do we still have the this Topics field?
If we still need it then I think:
diff --git a/models/topic.go b/models/topic.go
index 4a5bffa08..7f7c7cb9a 100644
--- a/models/topic.go
+++ b/models/topic.go
@@ -197,10 +197,13 @@ func FindTopics(opts *FindTopicOptions) (topics []*Topic, err error) {

 // GetRepoTopicByName retrives topic from name for a repo if it exist
 func GetRepoTopicByName(repoID int64, topicName string) (*Topic, error) {
+	return getRepoTopicByName(x, repoID, topicName)
+}
+func getRepoTopicByName(e Engine, repoID int64, topicName string) (*Topic, error) {
 	var cond = builder.NewCond()
 	var topic Topic
 	cond = cond.And(builder.Eq{"repo_topic.repo_id": repoID}).And(builder.Eq{"topic.name": topicName})
-	sess := x.Table("topic").Where(cond)
+	sess := e.Table("topic").Where(cond)
 	sess.Join("INNER", "repo_topic", "repo_topic.topic_id = topic.id")
 	has, err := sess.Get(&topic)
 	if has {
@@ -211,7 +214,12 @@ func GetRepoTopicByName(repoID int64, topicName string) (*Topic, error) {

 // AddTopic adds a topic name to a repository (if it does not already have it)
 func AddTopic(repoID int64, topicName string) (*Topic, error) {
-	topic, err := GetRepoTopicByName(repoID, topicName)
+	sess := x.NewSession()
+	if err := sess.Begin(); err != nil {
+		return nil, err
+	}
+
+	topic, err := getRepoTopicByName(sess, repoID, topicName)
 	if err != nil {
 		return nil, err
 	}
@@ -220,7 +228,25 @@ func AddTopic(repoID int64, topicName string) (*Topic, error) {
 		return topic, nil
 	}

-	return addTopicByNameToRepo(x, repoID, topicName)
+	topic, err = addTopicByNameToRepo(sess, repoID, topicName)
+	if err != nil {
+		return nil, err
+	}
+
+	topicNames := make([]string, 0, 25)
+	if err := sess.Table("topic").Cols("name").
+		Join("INNER", "repo_topic", "repo_topic.topic_id = topic.id").
+		Where("repo_topic.repo_id = ?", repoID).Desc("topic.repo_count").Find(&topicNames); err != nil {
+		return nil, err
+	}
+
+	if _, err := sess.ID(repoID).Cols("topics").Update(&Repository{
+		Topics: topicNames,
+	}); err != nil {
+		return nil, err
+	}
+
+	return topic, sess.Commit()
 }

 // DeleteTopic removes a topic name from a repository (if it has it)

would probably fix this but we'd also need a migration to ensure that the repository.Topics are up-to-date.
That would probably need to use something like SQLITEs GROUP_CONCAT to set the Topics.

Thanks @lafriks for keeping the issue from going stale

Thank you @zeripath for working on fixing this issue


Become a mouse driver instead

It would be great to have streams for mouseup, mousedown and click for the whole document. It can be useful in some components who want to know about mouse events in the outside world.

We could start with renaming the repo. And then transfer it to cyclejs-community where more of us can collaborate on this. :)

So, I need to update this driver so that I can finally release Cycle Color Picker  (this is driving me nuts, I just want to release it!).
My plan:

Rename repo
Convert to xstream
Add mouseup, mousedown, and click events
Update readme with examples for new methods
Release on npm (breaking change)

Later on we can look at issues #4, #6, and adding all events listed in #3. Basically I'd just like to get the list above done so that I can free up my time and brainspace for working on this and the keyboard driver.

I have changed this to be Cycle Mouse Driver on Github, and added up(), down(), click() methods. I've updated the readme and example, and published this to npm as cycle-mouse-driver https://www.npmjs.com/package/cycle-mouse-driver
It's not perfect, but it's a start!


Fix bad assumptions about state machine animation instances.

Whenever the StateMachine would encounter an AnimationState it would assume that the corresponding stateInstance would be an AnimationStateInstance. This wasn't always the case as AnimationStates can sometimes not point to an Animation (at edit time this is allowed). At runtime we had thought we could be smart about this and simply return a different "empty" State which doesn't do anything (what we call a SystemState) as the corresponding stateInstance, but this would trigger bad casts which a memory sanitizer helped point out.
The fix applied here is to ensure that those assumptions are valid by returning an AnimationStateInstance with an empty animation, which is exactly how the editor behaves too.

Updated, thanks @csmartdalton!


Error  running aks browse

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks browse
Errors:
HTTPConnectionPool(host='localhost', port=8888): Max retries exceeded with url: /closeport/8001 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x04E13110>: Failed to establish a new connection: [WinError 10061] No se puede establecer una conexión ya que el equipo de destino denegó expresamente dicha conexión',))
Traceback (most recent call last):
pip-install-wuxrdt2n\urllib3\urllib3\connection.py, ln 159, in _new_conn
urllib3\urllib3\util\connection.py, ln 80, in create_connection
...

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks browse --resource-group {} --name {}

Expected Behavior
Environment Summary
Windows-10-10.0.17763-SP0
Python 3.6.6
Shell: powershell.exe

azure-cli 2.0.64 *

Additional Context

Duplicate of #7686


Support CompletionItemKind.Method

Previously, if a registered completion provider returned items with
CompletionItemKind.Method, they would display in the list as properties,
due to not being converted to the corresponding SuggestionType.

Hi @sandyarmstrong, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Sandy Armstrong). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

I was thinking of doing the same in MyCompletionItem.convertKind, but I'm not as familiar with that area of the code.

LGTM


crash with Request Method:POST and http response 302 with no content or err500

The issue  is much like #750 , but it's a bit different .
In the environment nginx1.10.1 + ModSecurity2.9.1 + wordpress4.5.4, When publishing, editing, saving the article , the post request for the 302, into the non-response state . When deleting the article, delete the article for the first time normal, then delete the article will prompt an error .  there is no information in the modsec_audit.log, nginx in the error message:
[Alert] 19682 # 0: worker process 23877 exited on signal 11
[Alert] 19682 # 0: worker process 23878 exited on signal 11
[Alert] 19682 # 0: worker process 23879 exited on signal 11

When I set this:
SecAuditLog /var/log/modsec_audit.log
SecAuditLogType Concurrent
SecAuditLogStorageDir /opt/modsecurity/var/audit/

Nginx no longer has an error message, and modec_audit.log records a post request of 302 with an error status of 500.
Set SecRequestBodyAccess Off, SecResponseBodyAccess Off Problem remains.
Will in the end is where an error occurred? Is there a solution? Thank you!
New tracking information：
The above settings are not changed, the error is not resolved, but do not know why error_nginx.log there has been new information, not so regular before, it seems like intermittent, these new error messages appear in the absence of the login wordpress Background to operate when.
2016/11/08 04:09:55 [alert] 4295#0: worker process 4343 exited on signal 11
2016/11/08 09:55:18 [alert] 4295#0: worker process 4495 exited on signal 11
2016/11/08 11:46:03 [alert] 4295#0: worker process 4958 exited on signal 11
2016/11/08 13:34:31 [alert] 4295#0: worker process 5133 exited on signal 11
2016/11/08 14:05:57 [alert] 4295#0: worker process 5339 exited on signal 11
2016/11/08 15:34:48 [alert] 4295#0: worker process 5397 exited on signal 11
2016/11/08 19:14:33 [alert] 4295#0: worker process 5514 exited on signal 11
2016/11/08 20:26:32 [alert] 4295#0: worker process 5822 exited on signal 9
2016/11/08 20:26:37 [alert] 4295#0: worker process 5935 exited on signal 9
2016/11/08 20:30:14 [alert] 4295#0: worker process 5936 exited on signal 9
2016/11/08 20:30:18 [alert] 4295#0: worker process 5964 exited on signal 9
2016/11/08 20:37:30 [alert] 4295#0: worker process 5965 exited on signal 9
2016/11/08 20:37:37 [alert] 4295#0: worker process 5991 exited on signal 9

When the wordpress use https login, post data no response, send data to 0, can not login.
and error_nginx.log :
[alert] 9154#0: worker process 9813 exited on signal 11

When the wordpress use https login, post data no response, send data to 0, can not login.

I don't know if you can understand my problem, but I really can not solve these problems, I feel a little random.

Hi @edieterich,
Please try the ModSecurity-nginx connector available at:
http://www.github.com/SpiderLabs/ModSecurity-nginx


A link in repository description is broken.

This url returns 404.
http://storybooks.github.io/react-treebeard

I need an project admin/owner to edit project description, I updated the readme


Release Google.Cloud.Container.V1 version 2.3.0

Changes in this release:

Commit 3717e0d: Regenerate all APIs with generator change for deprecation

Created release for Google.Cloud.Container.V1-2.3.0


main two implementation

hello! i have a question, why you have two implementation of main function in aes.c and main.c? in codeblock throw an error

Hello @CDevelopEF ,
I don't think there are is a main function in aes.c. Please, check again.

Yes man I was wrong, my apologies


spatial coverage delete option and point/box toggle

The option to delete the spatial coverage  should be available whenever there is saved spatial coverage. But that seems to be not the case sometimes. In the screenshot below we have the option to delete temporal coverage but the same option is not available for spatial coverage.

If the resource already has box type spatial coverage, and I select the point coverage, the box does not get cleared from the map.

Originally posted by @pkdash in https://github.com/hydroshare/hydroshare/issues/5109#issuecomment-1642200976

@pkdash I wasn't able to reproduce 1 (option to delete spatial coverage missing). Was there anything unique that produced that state?


Brian region filter and any other search/filter field should be case-insensitive.

Brain region filter in the explorer window is case-sensitive which needs to be changed to become case insensitive.
Check cortical area and connectivity rule filter as well to ensure they are also all insensitive.

fixed


provide a Dockerfile for production

BEFORE THIS IS MERGED:

replace APP_KEY secret with app key of production transmorpher
replace testing aws credentials with production aws credentials
replace TRANSMORPHER_SIGNING_KEYPAIR with production signing keypair
add protector to transmorpher
add PROTECTOR_AUTH_TOKEN, PROTECTOR_PRIVATE_KEY to github secrets
create docker hub repo for transmorpher

should also add a .dockerignore to prevent env file from being copied into the image

@mszulik oh I see you've fixed the formatting already, and the env is running, great!
Let me know if there is anything else I can do for you :)

Hi @crohr, thanks for the quick reaction and the fix, I will try it out later :).

@crohr Hi, I have configured "@collaborators/push" as value for the "admins" input in the workflow, but only 1 collaborator's ssh key is added to the server instance. Did I misconfigure something?

@mszulik I got the same report so I guess something is not working as expected when retrieving collaborators. Not sure yet if it is a permission issue or if I misunderstood what that GitHub API provides. For now I would advise you to keep declaring admins explicitly.

@crohr Alright thank you!


chore: update PR template

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Update the PR tempalte
Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

More checkboxes don't seem good but let's try this new template for a while to see if it works


[_h_m_t_x] Read advanceWidth as unsigned short (uint16)

Print a warning on decompile if advance width (or height for vmtx) exceeds 0x7FFF (32767).
Upon compile, struct.pack will raise if the value is negative as it is out of range for the unsigned "H" format.
Fixes #645

Apparently both FontLab Studio (both 5 and VI) and Robofont allow these negative advances to occur in a font. UFO spec either does not enforce the value to be positive. I wonder why this is the case. Glyphs like that are definitely broken, as you can see from the mega-huge width they take when loaded in FreeType, MS and Apple rasterizers.
/cc @twardoch @typemytype @typesupply

OK, I don't mind changing both to tuple().

The only thing I'm worried is if anybody was relying on it being a list and was updating either the width or the sb in place, as in hmtx['a'][0] = 500...

True, although we could say that is poor style. I just checked my own code and ufo2fdk and ufo2ft and none of these do that. I'd be happy to take the risk.

@justvanrossum ok, I changed hmtx/vmtx metrics values to be tuples instead of lists.


Integration test failure: notebook workflow notebook cell actions

https://monacotools.visualstudio.com/DefaultCollection/Monaco/_build/results?buildId=76289&view=logs&j=a5e52b91-c83f-5429-4a68-c246fc63a4f7&t=8e2e103d-8d5d-54da-dc73-3c44bf1dc402&l=242
1 failing
  1) notebook workflow notebook cell actions:

      AssertionError [ERR_ASSERTION]: 1 == 2
      + expected - actual

      -1
      +2

      at Context.<anonymous> (extensions/vscode-notebook-tests/src/notebook.test.ts:324:10)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)

Fixes have been pushed to master, considering we are in the endgame, I'm not going to enable this particular integration test this iteration but have it turned on early next week.

It's already fixed.

No longer valid as the test is assuming that after one command execution, all events are sent the extension host correctly, which is not guaranteed.


Converted eclipse and idea scripts to Kotlin

I have tested the plugin thoroughly by diffing with the output of the Groovy scripts.
The behavior is improved over the Groovy scripts. The latter append content with every execution if the idea files already exist. This plugin does not. The getDefaultJunitVmParameter function reflects the ugliness with our current global configuration. Once we improve this, this method will be much nicer.
There is one issue that when an existing idea file is modified, the new lines in the copyright text are lost. I will think about how to fix this. I don't think this is a blocker. Otherwise everything is good to go and ready to be reviewed.

I ran ./gradlew cleanIdea idea, opened the IntelliJ project (via opening gradle.ipr).

No errors by the editor
I can successfully run an integration test. I have used GradleKotlinDslIntegrationTest. IntelliJ compiles the code successfully and the test passes.
Our run configuration are all properly defined.

I think this can be merged now.l

Feedback build https://builds.gradle.org/viewQueued.html?itemId=11181198


More on Activities and Record Types

I am creating an audit log retention policy. During the creation three questions came up:

what is AeD?
what activities are included for each record type?
will this increase the searchable date range for AzureAD user sign-in logs?

I found this referenced article. Unfortunately, that page lists more record types than are available when creating an audit log retention policy. For example, the feature "Azure Active Directory" uses these record types: AzureActiveDirectory, AzureActiveDirectoryAccountLogon, AzureActiveDirectoryStsLogon
Of the 3 listed, I only see AzureActiveDirectory as a record type when creating a retention policy.
For question 1, AeD appears to be related to eDiscovery, Ah! Advanced eDiscovery is my guess. These names need to be more friendly, or we need a place to find what the names mean!
For question 2, the only way I have found to determine what activities are included is to select one record type while creating the policy, then look at the list of activities. I tried to search the unified log and scroll through activities, but the groupings in the search page does not use the same language.
"Azure AD Group administration activities," is that an AzureActiveDirectory Record Type? Maybe.

For question 3, I think the answer is no. But there is also Workplace Analytics (isn't that called Viva Insights now?) which has log-in/log-out activities...
Maybe we should search the "UnifiedAuditLog user log-in activities" instead of the AzureAD User Sign-in logs.
Maybe they are the same thing and the AzureAD User Sign-in search will benefit from this retention policy too! Fingers crossed.

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

ID: e52e1877-191e-5dbe-b805-e755c736b985
Version Independent ID: 69df5ed6-a86f-06d7-1156-e1de211f8e83
Content: Manage audit log retention policies - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/audit-log-retention-policies.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@markjjo Please help us in resolving this issue. Thanks

@jonwbstr Thanks for your feedback! New issues and pull requests for this content are closed while we update this content to support upcoming feature improvements.


[WIP] Functional composition

Some useful functions for functional composition.
They are without tests and documentation now, because I want to discuss the topic first.

I generally like the idea of including functional composition capabilities

I've also thought of contributing a compose function, but why limit it to two functions? Here's what I came up with:
function identity($x) {
    return $x;
}

function compose(...$functions) {
    return array_reduce(
        $functions,
        function ($carry, $item) {
            return function ($x) use ($carry, $item) {
                return $item($carry($x));
            };
        },
        "identity"
    );
}

$plus2 = function ($x) { return $x + 2; };
$times4 = function ($x) { return $x * 4; };
$square = function ($x) { return $x * $x; };

$composed = compose($plus2, $times4, $square);

$mapped = array_map($composed, array(1,2,8,5));

assert($mapped == array(144, 256, 1600, 784));

@alexeyshockov I like @mschwager’s idea about compose() being n-ary

Closing, as we have a compose() now.


android browser support

Are there any plans to support android browsers in the future? I have tried 4 different browsers on android 4.2 and unfortunately prose is unusable.
Paste and select are extremely difficult or impossible and icons are often missing or do not appear to react.

I don't see how this is a dupe of #430. The problem described in this issue was not resolved when #430 was resolved.
Even now, we can not select text while editing a post in Chrome for Android. Editing/selecting does work fairly well in UC Browser.


Permission problems using Apache2 MPM ITK

With Apache2 MPM ITK, every virtual host can run under a separate uid and gid. Using Apache2 MPM ITK with ModSecurity results in some error entries in the audit log because of missing permissions.
The Problem is, that the DBM file can't be accessed/written, because the default permissions to the files are 0640. This results in the situation, that only one virtual host is able to initially write the DBM files (if they do not exist). All other virtual hosts are not able to read/write to the DB; files resulting in the following messages in the audit log

Message: collections_remove_stale: Failed to access DBM file "/tmp//global": Permission denied
Message: collections_remove_stale: Failed to access DBM file "/tmp//ip": Permission denied

When you set the DBM file world writeable, then the above errors are not logged any more and each virtual host can access the DBM file.
The audit log still shows the following error, which I was not able to reproduce/fix

Message: Audit log: Failed to lock global mutex: Permission denied

It seems there is a general problem when running virtual hosts as different users with mod MPM ITK or mod RUID, as the global mutex problem is already filed here https://github.com/SpiderLabs/ModSecurity/issues/454
It would be great if you could define SecDataDir also in the VHOST scope or if file permissions to all DBM files could be set in the configuration file. Also it would be great if the global mutex problem could be resolved.
If I can help debugging the global mutex problem, don't hesitate to contact me.

As of 4/27/2015 this issue is still not resolved.  Cpanel claims it is a SpiderLabs issue in this thread:
Hello,
Much of the issue is related to limitations with Mod_Security itself with regard to shared storage and handlers like Mod_ruid2 and MPM-ITK. It's not something that cPanel can really solve, but that being said, the case is still open in order to find the best possible way to avoid these types of issues for customers"
ref: https://forums.cpanel.net/threads/modsecurity-rule-processing-failed.453321

As for now 19/01/2016 the problem still remains.
ModSecurity: collection_store: Failed to access DBM file "/var/cpanel/secdatadir/ip": Permission denied
ModSecurity: Geo Lookup: Failed to lock proc mutex: Identifier removed

same problem here with mod_ruid2

Add patch in @ju5t comment [link], then recompile and it works!
My configuration was:

ubuntu 14.04.4
apache 2.4.7-1ubuntu4.11
libapache2-mod-ruid2  0.9.8-3
libapache2-mod-security2 2.7.7-2 (rebuilded)

Its a shame that now CPANEL is forcing you to disable modsec if you want to use ruid2 or itk due to this bug that they claim has not been addressed by Spiderlabs or others.  IMO this will open a gaping hole in security.  They refuse to even test the patch as they say it is not widely adopted yet.  I wonder if CPANEL is even watching this thread and hope the figure-pointing ends and a solution is developed.

Ju5t,
Which problem does your patch fix?   The Geo Lookup: Failed to lock proc mutex: Identifier removed issue or the Failed to access DBM file "/var/cpanel/secdatadir/ip": Permission denied  issue?   Thanks!

bump
my workaround is set all to 777 via cron_daily
this is insane

@Spork-Schivago it should solve the file permission denied issue. I know that the DirectAdmin control panel includes it in their ModSecurity implementation. I would assume that accounts for a reasonable number of servers.
We're using it ourselves too ever since 2014 and haven't seen any side effects.

Thank you Ju5t.   Still no fix for the modruid2 issues though, correct?   So even with the patch, I still won't be able to use modruid2.

@Spork-Schivago I wrote a patch for that too. I've raised a pull request for it but unfortunately it hasn't been accepted yet. See https://github.com/mind04/mod-ruid2/pull/8 for more details.

Marking @ju5t's patch to be discussed in our next team meeting. Candidate for v2.9.2. The date is was not set yet.

Zimmerle,
Do you know if @ju5t's first patch is already implemented?   If not, does your team plan on discussing implementing both patches in v2.9.2?   Thanks!

If anyone is interested.
We use this patch on hosting servers more 2 years, where total the number of users and sites >= 30000. Used this for apache2-itk and not found any problems all this time of use.

Thanks @Rendername .
Do not have any updates on this issue yet?
It's been three years!

@Rod-Gomes, Apparently not. In the main branch never no one sent a pull request, but this patch really solves problem as a minimum when using the apache itk.

I've never sent in a pull request because I think it can be improved. In the end I'm not a C developer and my knowledge of Apache's internals is limited. I can raise a pull request if it helps.
@zimmerle wrote in August that the patch would be up for discussion in version 2.9.2. This version isn't released yet. An update is more than welcome.

@ju5t your patch looks great, and would be really useful. I think you should raise a pull request!

The global mutex is optional since 112ba45 (included in 2.9.2 release)
@ju5t I've pushed your patch as PR #1912. It's tagged for review for 2.9.3.
Thanks all for your contributions :)

@victorhora awesome! Thanks!


Bump version to 0.17.3 and update CHANGES.md

@tarcieri could you review https://github.com/celluloid/celluloid-io/pull/166 ? Could be easily integrated in this release, or not?
Apart from that, +1 for new release .

@TiagoCardoso1983 I think I'd rather hold off on shipping that for now


Translate to PT-BR

Issue Description
Can i translate the README.md and the doct to pt-br?

Sorry, for now we want to keep the documentation English only, feel free however to publish blog posts describing certain aspects in PT-BR 😃


Build.SourceVersion not as part of the build number

why? would be great since this is the best version number

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

ID: 9ffebdc8-521c-44b5-cde4-4abdc1bd6f18
Version Independent ID: dc0effda-6d93-62aa-64c5-4e6f283b0826
Content: Predefined build variables - Azure Pipelines
Content Source: docs/pipelines/build/variables.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

It actually works. I think it's a new(-ish ?) feature. The docs should be updated to reflect that.
The trick is, it will only work for CI triggered builds not manually queued ones.
Would be great if we could have access to the short version of it though not the full GUID.

It actually works. I think it's a new(-ish ?) feature. The docs should be updated to reflect that.
The trick is, it will only work for CI triggered builds not manually queued ones.

nice, tested it and you are right! thanks for pointing that out 👍

Would be great if we could have access to the short version of it though not the full GUID.

agreed 😄

Any progress to gather short Build.SourceVersion ?

It's not yet implemented?

@schdief -- Steve, you may find answers here:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.
My apologies for the delay in responding.


Uncaught RangeError: Maximum call stack size exceeded

Issue Id: 76000c98-ba3d-f84d-60f4-a18061d87e94Versions - 0.10.6-release-  df352367df2efcfa9d602d471e4e2f42140a0f05-  45d69357c9eb068dd8e624f5b0fe461cd2078d88-  783ecf841a2b21edca6d86853670cae89b4c244f-  7cc0c602a2b802b09a3f56b68a1a61ee9e8f0066-  db71ac615ddf9f33b133ff2536f5d33a77d4774eStack RangeError: Maximum call stack size exceeded    at y.scan (out/vs/languages/sass/common/sassWorker.js:4:902)     at y.scan (out/vs/languages/sass/common/sassWorker.js:4:1041)     at y.e.scan (out/vs/languages/css/common/cssWorker.js:4:27925)     at y.scan (out/vs/languages/sass/common/sassWorker.js:4:1839)     at y.scan (out/vs/languages/sass/common/sassWorker.js:4:1041)     at y.e.scan (out/vs/languages/css/common/cssWorker.js:4:27925)     at y.scan (out/vs/languages/sass/common/sassWorker.js:4:1839)     at y.scan (out/vs/languages/sass/common/sassWorker.js:4:1041)     at y.e.scan (out/vs/languages/css/common/cssWorker.js:4:27925)     at y.scan (out/vs/languages/sass/common/sassWorker.js:4:1839)

duplicate of #3048


Write output file using UTF-8 encoding

Fixes #72, using from io import open for Python 2 support.

Looks good, I'll tack on a regression test after merging :D

:smiley:


Game crashing after using a big wand

Hi all,
I've been playing with my brother and everything was fine
In the first parallel world, when we hit the Temple of the Art
He had a very strong wand with some divide by spells, which caused him to crash. Upon reopening the game, he keeps crashing
My game does not crash. If he starts the lobby with the same save, he crashes when I connect (My game doesn't crash).
The log is something like this, but repeated thousands of times:
←[2m2024-12-12T23:51:27.686452Z←[0m ←[31mERROR←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Error occured while reading from websocket: IO error: An existing connection was forcibly closed by the remote host. (os error 10054)
←[2m2024-12-12T23:51:27.686640Z←[0m ←[31mERROR←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Likely that just means that the game has closed.
←[2m2024-12-12T23:51:28.234694Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::world←[0m←[2m:←[0m<PHONE_NUMBER>9556944 is gaining priority over ChunkCoord(289, 90) from<PHONE_NUMBER>2238966
←[2m2024-12-12T23:51:28.234804Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::world←[0m←[2m:←[0m Will request authority transfer
←[2m2024-12-12T23:51:28.235365Z←[0m ←[33m WARN←[0m ←[2mnoita_proxy::net::world←[0m←[2m:←[0m<PHONE_NUMBER>2238966 sent RelinquishAuthority for ChunkCoord(289, 90), but isn't currently an authority
←[2m2024-12-12T23:51:28.385881Z←[0m ←[33m WARN←[0m ←[2mnoita_proxy::net::world←[0m←[2m:←[0m Transfer failed, requesting authority normally
←[2m2024-12-12T23:51:28.385980Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::world←[0m←[2m:←[0m<PHONE_NUMBER>9556944 already has authority of ChunkCoord(289, 90)

Wherever he plays the same save, but without anyone entering the room, he gets this:
←[2m2024-12-12T23:56:35.838554Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m None
←[2m2024-12-12T23:56:35.985046Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Setting fonts...
←[2m2024-12-12T23:56:35.985147Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Installing image loaders...
←[2m2024-12-12T23:56:35.985355Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Initializing steam state...
←[2m2024-12-12T23:56:35.985423Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::util::steam_helper←[0m←[2m:←[0m Initializing steam client...
Setting breakpad minidump AppID = 881100
SteamInternal_SetMinidumpSteamID:  Caching Steam ID:<PHONE_NUMBER>9556944 [API loaded no]
←[2m2024-12-12T23:56:36.074407Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::util::steam_helper←[0m←[2m:←[0m Initializing relay network accesss...
←[2m2024-12-12T23:56:36.076895Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Checking if running on steam deck...
←[2m2024-12-12T23:56:36.076968Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::util::steam_helper←[0m←[2m:←[0m Spawned steam callback thread
←[2m2024-12-12T23:56:36.077056Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Creating the app...
←[2m2024-12-12T23:56:36.077192Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::save_state←[0m←[2m:←[0m Has savestate: false
←[2m2024-12-12T23:56:36.077518Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::save_state←[0m←[2m:←[0m Will save to: \\?\E:\Games\mods\NOITA\save_state
←[2m2024-12-12T23:56:36.170206Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Path is valid, checking mod now
←[2m2024-12-12T23:56:36.170312Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Appdata path: C:\Users\Kitsune\AppData
←[2m2024-12-12T23:56:36.170572Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Trying save path: C:\Users\Kitsune\AppData\LocalLow/Nolla_Games_Noita/
←[2m2024-12-12T23:56:36.170719Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Save path exists
←[2m2024-12-12T23:56:36.170791Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Found game save path: C:\Users\Kitsune\AppData\LocalLow/Nolla_Games_Noita/
←[2m2024-12-12T23:56:36.172337Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Trying to enable mod: Ok(())
←[2m2024-12-12T23:56:36.335390Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Mod path: E:\Steam\steamapps\common\Noita\mods\quant.ew
←[2m2024-12-12T23:56:36.335672Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Mod version: Some(Version { major: 0, minor: 27, patch: 7 })
←[2m2024-12-12T23:56:36.336205Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Mod is ok
←[2m2024-12-12T23:56:37.600797Z←[0m ←[33m WARN←[0m ←[2mnoita_proxy::bookkeeping::save_state←[0m←[2m:←[0m Could not read "run_info": The system cannot find the file specified. (os error 2)
←[2m2024-12-12T23:56:37.601008Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Using random seed:<PHONE_NUMBER>0374751385
←[2m2024-12-12T23:56:37.602961Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::bookkeeping::mod_manager←[0m←[2m:←[0m Found 188 progress entries
←[2m2024-12-12T23:56:37.603047Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Starting netmanager
←[2m2024-12-12T23:56:37.612809Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Listening for noita connection on <IP_ADDRESS>:21251
←[2m2024-12-12T23:56:37.612959Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Is host: true
←[2m2024-12-12T23:56:37.613440Z←[0m ←[33m WARN←[0m ←[2mnoita_proxy::bookkeeping::save_state←[0m←[2m:←[0m Could not read "world_chunks": The system cannot find the file specified. (os error 2)
←[2m2024-12-12T23:56:37.656278Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::util::steam_helper←[0m←[2m:←[0m Loaded avatar for SteamId(76561198129556944)
←[2m2024-12-12T23:56:37.896201Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::steam_networking←[0m←[2m:←[0m Lobby ready
←[2m2024-12-12T23:56:37.896299Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::steam_networking←[0m←[2m:←[0m Updating peer list
←[2m2024-12-12T23:56:37.897096Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::steam_networking←[0m←[2m:←[0m Switched to `creating mesh` state
←[2m2024-12-12T23:56:37.908221Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net::steam_networking←[0m←[2m:←[0m Switched to `all connected` state
←[2m2024-12-12T23:56:38.895407Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy←[0m←[2m:←[0m Starting the game now
←[2m2024-12-12T23:56:42.310874Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m New stream incoming from <IP_ADDRESS>:55579
←[2m2024-12-12T23:56:42.311062Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m New stream connected
←[2m2024-12-12T23:56:42.311840Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Chosen nickname: Kitsune
←[2m2024-12-12T23:56:42.312219Z←[0m ←[32m INFO←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Settings sent
←[2m2024-12-12T23:59:05.610180Z←[0m ←[31mERROR←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Error occured while reading from websocket: IO error: An existing connection was forcibly closed by the remote host. (os error 10054)
←[2m2024-12-12T23:59:05.610311Z←[0m ←[31mERROR←[0m ←[2mnoita_proxy::net←[0m←[2m:←[0m Likely that just means that the game has closed.

very useless issue without the wand

you guys are on a very old update


[BUG] Demo website broken?

Describe the bug
react import not being resolved for some reason

To Reproduce
Go to https://lukasbach.github.io/monaco-editor-auto-typings/ on Google Chrome latest version as of 9/15/23
Expected behavior
Expected react import to be resolved.

Environment (please complete the following information):

OS: macOS Version 13.4.1
Browser Version: Google Chrome | 117.0.5938.62 (Official Build) (arm64)
monaco-editor-auto-typings Version: ???
.

🌑


.gitignore: Ignore /.gopathok

We've been occasionally creating this file since 9c44933b (#410).  But it's recording information about the local environment, not part of our common source.  Adding it to .gitignore helps avoid having it accidentally committed.

LGTM

No reason to do E2E tests.


HBASE-24869 migrate website generation to new asf jenkins

Equivalent to our existing job on builds.a.o except:

moves to Jenkins Pipeline so we can configure some things that were done in plugins on the old jenkins
reduces the timeout from 4 hours to 1 hour. normal build times are ~15-20 minutes.
alters the message body for failure notifications because we can't set jenkins environment variables from our script.
adds an optional "force failure" parameter so we can test notifications

tested with a feature branch on ci-hadoop.a.o

build 3 was able to publish an update, archive the same artifacts, and clean up the workspace
build 6 was run with force fail set and produced a reasonable notification email to dev@hbase.

after this PR is merged we will need to update the jenkins job to point to the master branch instead of the feature branch for the pipeline definition.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 41s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  8s
Maven dependency ordering for patch

_ Other Tests _

2m 13s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2246/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2246

Optional Tests

uname
Linux 92117122cebe 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8646ac139d

Max. process+thread count
47 (vs. ulimit of 12500)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2246/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  7s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 20s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  7s
Maven dependency ordering for patch

_ Other Tests _

2m 28s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2246/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2246

Optional Tests

uname
Linux 3cb2e462d291 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8646ac139d

Max. process+thread count
49 (vs. ulimit of 12500)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2246/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 53s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 25s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  8s
Maven dependency ordering for patch

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

2m 34s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2246/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2246

Optional Tests
dupname asflicense

uname
Linux 2d9b2d82fb94 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8646ac139d

Max. process+thread count
52 (vs. ulimit of 12500)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2246/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Restrict audio casting to a specific application?

Hi
First of all, thanks a ton for this app. Totally awesome! Had some initial teething issues getting it to work, but the "run_as_admin" fix and "muting" the volume solved those. I use MediaMonkey as my mediaplayer and stream my songs from my folders/playlists organized there. Works great!

Is there some way to cast only audio being played through a specific application? I suppose ideally MediaMonkey should do that, but given that doesn't exist, wondering if there's some workaround to make that happen with this?
Thanks
Arun

Where can you set the input audio device?
Maybe it is possible to use http://vb-audio.pagesperso-orange.fr/Cable/index.htm
in order to direct spotify to a "virtual cable" and then have chromecast-stream read from a "virtual cable" thereby restricting the sound stream to a specific application

@limor1 I'll look into that! Looks good!

Not likely to happen


Docs missing alert close info

The alert examples and docs are missing data-dismiss="alert" resulting in the X not closing the alerts.
Should be:
×

or?

x

Resolved in 2.0.1-wip. Thanks!


winston error email notification while trying to send followup emails

not sure how the winston error is related with the email mechanism, but the issue is now resolved and addressed in the following commit https://github.com/unhangout/unhangout/commit/c778649fe825de4d42ffab06749776d65ea043ec, closing this for now.


Tests stuck without any error

Sometimes test get stuck without any error thrown.
Is there a way to see testcafe errors? how to distinguish which test/fixture gets stuck?
Is there a way to set test timeout and fail the test of the timeout passed?

Hi @sivanisentinel
The behavior your described looks like a bug. To find the cause of the problem we need to reproduce this behavior on our side. Could you please provide a small and complete example with which we can reproduce the problem?

It is really hard to reproduce. It sometimes happens and I don't know to predict when or what is causing it to suck.
Do you have testcafe debug logs? what about the test timeout option I asked about?

Do you have testcafe debug logs?

Internally TestCafe contains some kinds of debug information, but at present this is not enough to find the cause of the problem.
Could you provide a more details about your environment: Node.js and TestCafe version, browser and operation system name and version?

We are running with node 9.5 on linux with the latest testcafe version on headless chrome 75.0.
But we are looking for solution or problem solving option for future issues as well.
what about the test timeout option I asked about? this will help us a lot.

what about the test timeout option I asked about? this will help us a lot.

Usually, CI systems regulate the maximum time of test execution.

At present, we don't have this option in TestCafe.
You can define the maximum time of test execution programmatically:

const createTestCafe   = require('testcafe');
let testcafe           = null;
const timeLimitPromise = require('time-limit-promise');

createTestCafe('localhost', 1337, 1338, void 0, true)
    .then(tc => {
        testcafe     = tc;
        const runner = testcafe.createRunner();

        const runPromise = runner
            .src('test.js')
            .browsers('chrome')
            .run();

        return timeLimitPromise(runPromise, 5000, { rejectWith: 1 });
    })
    .then(failedCount => {
        console.log('Tests failed: ' + failedCount);
        testcafe.close();
    });

Thanks!
I do have the option to timeout the whole run from CI but it does not help up with defining what is stuck.
if even one test is stuck we wont get to reporter and wont get any details about the other tests.
I I could fail only the one test that is stuck I can easily identify it and also get the information about the other tests.

This issue seems to be a bug, however, it's still very unclear, so I'm afraid we are unable to provide any effective help without a working example. Please provide us with your working project and test code to reproduce the issue. If you cannot share it here on Github you can send it at<EMAIL_ADDRESS>

I noticed it's suck when I raised concurrency from 3 to 8. is that a known issue?
Either way it is impossible to know what causing testcafe to stuck when we dont have any trace logs.

We are not aware of this issue, however, if increasing the number of browsers leads to the hang, it can be a problem with RAM. It's just my assumption, because it's impossible to say anything precise without an example.

I also have this issue which is not random on my local machine. Here is my info:
node: v10.15.1
testcafe: 1.3.1
system: Win10
browser: Chrome 68.0

@Yaowenjie, we need an example to reproduce the issue. I'm afraid the provided information is not sufficient. We would really appreciate it if you share your project details with us.

Hi @AlexKamaev, thanks for reply!
My project is using MicrosoftOnline as login entry and using useRole for each test case, it will hang when login was just finished and redirecting just happen (Actually, as opposed to redirecting to our home page, It will be redirected to one wrong URL which is a static resource  path).
But if we try to fixture.page(URL).beforeEach(DO_LOGIN), this issue won't occur.
One difference I can see here is that URL is in fixture level or in Role

Thank you for providing the information. However, it is not sufficient to find the cause of the issue. We need to analyze your page or code to provide you with recommendations. Would you please give me a link to your site if it is public or create a simple example?

@LavrovArtem Thanks for reply! Unfortunately, it is not possible to give you a link, because that is a private project for enterprise. :(

We found more info. It seems like that once one request get a 403/500 response code with empty response body (which might be not the main requests of our app), testcafe will redirect us to blank as below. Sometimes it will be redirected to the SVG path as I said above. @AlexKamaev @LavrovArtem Does testcafe has some similar issues like this?

@Yaowenjie
Hello,
We cannot examine this issue without any reproducible example.
Feel free to open a new bug report with all required information.


Rebased GarNet branch

This PR supersedes #188 . Same functionality is being PRed but I've manually rebased the branch to the latest master.

Not to be merged yet - @vloncar is going to make a PR to this branch.


Status bar change to light content after Toast dismissed

Pre-requisites:

[x] Yes, I looked through both open and closed issues looking for what I needed.
[x] No, I did not find what I was looking for.

Expected Behavior
The status bar color scheme should not be changed after toast dismissed

Current Behavior
The status bar is in dark content when the app is running on dark mode with a light background.
When the toast is shown, the status bar changes to light content

Steps to Reproduce
https://github.com/user-attachments/assets/0c006b42-136e-45eb-a1bd-8eb79aa29c05
struct TestView: View {
    @State private var isLoading: Bool = false
    var body: some View {
        VStack {
            Button {
                isLoading = true
            } label: {
                Text("Hello World")
                    .padding()
            }
        }
        .toast(isPresented: $isLoading, dismissAfter: 2) {
            ToastView("Loading").toastViewStyle(.indeterminate)
        }
        .frame(
            minWidth: 0,
            maxWidth: .infinity,
            minHeight: 0,
            maxHeight: .infinity,
            alignment: .topLeading
        )
        .background(Color.white.ignoresSafeArea())
        .preferredColorScheme(.dark)
    }
}

Context
I want to keep the status bar color scheme after toast was dismissed.

Your Environment

Swift Version: 5
Xcode Version: 15.3
Operating System Version: 14.3.1
Device or Simulator: iOS 16 (dark mode)

Hi @thomas-gogox,
Thank you for your detailed report. I will look into this. Could you please confirm whether you are using the latest ToastUI version 4.0.0?

Hi @thomas-gogox,
Thank you for your detailed report. I will look into this. Could you please confirm whether you are using the latest ToastUI version 4.0.0?

I'm using the latest ToastUI version 4.0.0 @quanshousio


Fixing URLs throughout Building with Base Guides (#353)

What changed? Why?
Opened Issue (#353)
I have updated the relative paths in the markdown files for the Base Guides to ensure they lead to the correct locations when hosted on the production site. The original relative paths, which worked as expected in a local development environment, resulted in incorrect, nested URLs in the production environment, leading to 404 errors. This update changes the relative paths to root-relative paths, ensuring they function correctly in all environments.
Notes to reviewers
Please review the changes to ensure that the updated URLs correspond to the actual paths where the documents are located. This update is critical for maintaining the integrity of the documentation and ensuring that users can navigate it without encountering dead links.
I've made corrections to the following files under the apps/base-docs/docs/building-with-base/guides/ path:

advanced-frame-behavior.md
deploy-frame-on-vercel.md
hyperframes.md
linked-minting-frame.md
nft-minting-frame.md

Changes involved updating relative links such as ./deploy-frame-on-vercel to root-relative links like /building-with-base/guides/deploy-frame-on-vercel. This was done to ensure consistent and correct path resolution across both local and production environments.
How has it been tested?
The updated paths have been tested in a local development environment to ensure they correctly resolve to the intended documents. Additional testing in a staging or production-like environment is recommended to confirm that the changes resolve the navigation issues observed in production.
Does this PR add a new token to the bridge?

[X] No, this PR does not add a new token to the bridge
[ ] I've confirmed this token doesn't use a bridge override
[ ] I've confirmed this token is an OptimismMintableERC20

Please include evidence of both confirmations above for your reviewers.

Added another commit fixing broken links:
[mint.fun]: https://mint.fun/
[Frame]: https://docs.farcaster.xyz/learn/what-is-farcaster/frames
Assumed Frame was pointing here, but you all might want to verify that!


Measure exits as the 2nd hop

I think if the relay to be measured can exit to the destination, it should be the second (last) hop. If that ends up being the case, another relay should be chosen to take the first hop position such that it isn't likely to be a limiting factor, just like exits are chosen right now.
I will now summarize what I think should be done with pseudocode.
relay = get_next_best_priority_relay()
dest = get_a_destination()
min_bw = relay.bandwidth
if can_exit_to_dest(relay, dest):
    # find a 1st hop helper, put best priority relay 2nd
    helper = get_nonexit_with_min_bw(min_bw)
    circ = [helper, relay]
else:
    # find a 2nd hop helper, put best priority relay 1st
    helper = get_exit_with_min_bw(min_bw)
    circ = [relay, helper]
results = do_measurement_stuff(circ)
# only associating results with the chosen relay, not the helper too
store_results_for_relay(relay, results)

(Note that min_bw is not simply relay.bandwidth nor would it be if we agree on my suggestion. Currently there's an allowed range of 1.25x to 2x the relay's bandwidth, but I have reasons for wanting to change that which can/should be discussed in a different ticket, not here)

Closed by 2fc8f53329caf2406870007e53110060ad3a73f1


[stable/joomla] Fix appVersion

Remove distro from appVersion, this was an error in the pipeline. Already solved

/ok-to-test
/lgtm


Update graph-core to 2.0.1

About this PR
📦 Updates org.scala-graph:graph-core from 1.13.6 to 2.0.1 ⚠
📜 GitHub Release Notes
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-graph", artifactId = "graph-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-graph", artifactId = "graph-core" }
}]

labels: dependency, library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #1786.


Linux uhid fixes

This code predated the generic report handling, that's why it wasn't used.

This is what I guessed :-)

It also predated our USB PID assignment, which is why it is 0x9999:0x9999. We should update that too :sweat_smile:

The correct IDs are 1d50:616a , correct?
They should probably registered in the linux USB database: https://usb-ids.gowdy.us/read/?action=help?help=newitem

Yes. I think they haven't synced the list yet, our PID is registered on the vendor list.
https://github.com/openmoko/openmoko-usb-oui/blob/7d1a8b701fad1b023d712a3911eb4d65506cba7e/usb_product_ids.psv#L326

sorry the clang format check is complaining again :sweat_smile:, sorry, we have a pre-commit hook setup that helps with that


[RTM] ENH: Save text crashfiles in output directory

Fixes #595.
This creates a derivatives/fmriprep/logs directory and saves logs and text crashfiles there.
Some other minor cleanups I saw while in there. Probably most significant is only building the graph if we're going to run it or write the graph. This allows --reports-only without --write-graph to run more quickly.
Closes #240.

Hmm. If this worked, shouldn't log files have shown up in the artifacts? @oesteban would you mind seeing if you see something obvious? Like, should I move the configuration update to earlier/later in the program?


Major legal issue..

Are you checking the age of the people on the boyfriend list..?

so true bestie.


stevenblack-blocklist: 3.14.112 -> 3.14.115

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for stevenblack-blocklist is: Unified hosts file with base extensions
meta.homepage for stevenblack-blocklist is: https://github.com/StevenBlack/hosts
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.14.115 in filename of file in /nix/store/ckvhbrf9qyn33llx85wg3bvxvbbkc36m-stevenblack-blocklist-3.14.115

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

stevenblack-blocklist

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/6e375ae583216b55b88d9a8c49023a47311f87af#stevenblack-blocklist

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ckvhbrf9qyn33llx85wg3bvxvbbkc36m-stevenblack-blocklist-3.14.115
ls -la /nix/store/ckvhbrf9qyn33llx85wg3bvxvbbkc36m-stevenblack-blocklist-3.14.115/bin

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

6 packages built:

stevenblack-blocklist
stevenblack-blocklist.ads
stevenblack-blocklist.fakenews
stevenblack-blocklist.gambling
stevenblack-blocklist.porn
stevenblack-blocklist.social

Maintainer pings
cc @moni-dz @Guanran928 @Frontear for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Update Gradle version to 2.5 in cross-platform template

Also make sure the IDEs can deal with this.


Actually use the min keyframe interval from CLI

Fixes #1497

Coverage remained the same at 81.715% when pulling 5695b7289110aae605aa25215cad939cde5c520b on shssoichiro:1497-respect-min-keyframe-interval into 1e40ea6252cc91b69736e02e2415f79d83af88f2 on xiph:master.

Nice catch.


Pull request

Reference Issues/PRs

What does this implement/fix? Explain your changes.
Any other comments?

Pull request


x/crypto/acme: Client.GetAuthorization should return also Retry-After value

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.9.1 linux/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/tnozicka/go"
GORACE=""
GOROOT="/home/tnozicka/lib/go"
GOTOOLDIR="/home/tnozicka/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build118198130=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
Request
This function
https://github.com/golang/crypto/blob/541b9d50ad47e36efd8fb423e938e59ff1691f68/acme/acme.go#L355-L369
should return Retry-After as is used in
https://github.com/golang/crypto/blob/541b9d50ad47e36efd8fb423e938e59ff1691f68/acme/acme.go#L415
or there should be similar function that does return it.
I have a work queue and want to know at what time should I re-queue my object and I can't use active waiting as with the former function.

@tnozicka how would you propose for GetAuthorization method to return the  retry-after value?

@x1ddos A) seems nice but I don't really care that much except for having the value available
A) Extend Authorization struct (there is already Status inside)
type Authorization struct {
    // ...
    RetryAfter *time.Duration
}

B) Change return type
func (c *Client) GetAuthorization(ctx context.Context, url string) (*Authorization, error) {/* ... */}
// to
func (c *Client) GetAuthorization(ctx context.Context, url string) (*Authorization, retryAfter time.Duration, error) {/* ... */}

C) Second function if you want to avoid API change
func (c *Client) GetAuthorizationWithRetryTime(ctx context.Context, url string) (*Authorization, retryAfter time.Duration, error) {
    /* ... */
}

func (c *Client) GetAuthorization(ctx context.Context, url string) (*Authorization, error) {
    a, _, err := GetAuthorizationWithRetryTime(ctx, url)
    return a, err
}

@tnozicka acme.Client now has RetryBackoff field, which should allow for both access to Retry-After and have your own implementation of a retry backoff in methods like WaitAuthorization.
Would you now consider this issue fixed?

Closing since no response from original reporter.


Dialog & Modal Components [WIP]

Related issue
https://github.com/Royal-Navy/standards-toolkit/issues/207
Overview
Implement and document the Dialog and Modal components.
Reason

The Dialog and Modal are full screen components that focus the attention of the user. The Modal can be used in a variety of situations, including displaying forms. The Dialog is a confirmation modal, requiring the user to confirm or cancel an action.

Work carried out

[x] Implement the Modal component
[x] Implement the Dialog component (wrapper around Modal)
[x] Update Button exports
[x] Resolve some button styling issues
[x] Add automated tests
[x] Perform UAT down to Chrome 38 (and across a variety of form factors)
[ ] Add developer documentation
[ ] Add design documentation

Screenshot

Developer notes

We need to resolve tree shaking issues with the icon library ahead of merging and releasing this component. This is currently in progress.

There is a grey background (circle) on the close icon - it just doesn't seem to like to include that colour in the GIF.


BNF rule for setName + generated files

This fixes potential development problems as I didn't checked and included a proper BNF rule set for setName(). Sorry for that, but better soon than later :)

Thanks for that :)


Kraken upgrade and base image removal

This is a start on the Kraken images and combining base and daemon into a single image.
Untested other than checking that the 14.04 Dockerfile would build. Might need some more cleanup (and likely updates of symlinks). Thought I'd get the changes out there for discussion.
Do we also want to keep the demo image?

I was thinking or at least questioning myself if we shouldn't merge demo inside daemon as well. Thoughts?

And yes this is obviously breaking the CI...

Thanks @hunter