Add "friction" sliding when at the start/end slide to prevent the entire slider being dragged off-screen

Originally raised: https://github.com/ganlanyuan/tiny-slider/issues/364
This is for non-looping sliders, currently, if you continue sliding right on the last slide (or left on the first slide) it's possible to drag the entire slider off-screen.
Most modern sliders combat this with "friction" so that if you have reached the start or end of the slider, the slide speed reduces the further you drag it until it reaches 0.

If I understand the code correctly, we need to add some logic to the panUpdate function (https://github.com/ganlanyuan/tiny-slider/blob/489bb95fdbee172906e089a635f8f1b9db1262a3/src/tiny-slider.js#L2587).
We can alter the dist variable (multiply by 0.5 or something) whenever the dragging exceeds the boundaries of the container. If that works we can make it smooth by applying an easing-function (cubic/quadratic bezier)

I started working on this, when it's done I'll make a pull request :)

@kobius I got it working! https://github.com/ganlanyuan/tiny-slider/pull/604

@laurens94 incredible work! Your code looks great. Going to work it into my project now and test across multiple devices, viewport widths and slider types - will report back.

Thanks! Took me a few hours and a headache haha, looking forward to your test results!

@laurens94 it's working really well, I've tested on Desktop Mac and PC at multiple widths, and multiple generations of iPhone and a tablet. It's silky smooth and makes a huge difference to the UX.
The only thing I would suggest as a change/feature would be to start the friction a little earlier. After testing other native sliders on iOS and Android, they seem to end the drag at around 1/3 of the viewport width, so that only a 1/3 of the slider is blank when dragged from the start/end slide.
I tried adjusting the 'moveLimit' variable within the panUpdate function but have only managed to get it working for the left-side drag. I don't think it needs to be adjustable - it seems like the industry standard, even owlCarousel also seems to be around 1/3 for the easing to end the drag.

@kobius I'm glad to hear that your test results are positive! I completely agree with you, 1/3rd of the viewport would be better and conform the pattern.
I tried fiddling around with the code and I think the moveLimit variable is not that adjustable, because it's also used to define the easeTime variable, which has to be based on 100% viewport size. When I have some spare time I'll continue with it!

Got it working! :D https://github.com/ganlanyuan/tiny-slider/pull/604/commits/dbcf91e35f1463f6aa99908bb35273ff0416b36c

@laurens94 thank you! I've tested the new code across the same devices and viewport widths and it's working perfectly.
It's buttery smooth, especially on mobile - can I vote for this to be published to the main branch?

@kobius nice, you're welcome :)
It would be nice if it gets implemented in the main branch! I guess we have to wait until @ganlanyuan has some time to review it, right?


Fix for UTF-8 partials in function ConhostConnection::_OutputThread.

Summary of the Pull Request
ConhostConnection::_OutputThread shall take care of partial UTF-8 characters generated while buffering the stream read.

References
This PR may partially fix the occurrence of � characters as seen on screenshots in the following issues:
#386 #455 #666

PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #386

Detailed Description of the Pull Request / Additional comments
Code points are represented by a sequence of 1 to 4 bytes in UTF-8. Whenever UTF-8 text is getting buffered, code points that consist of multiple bytes may get split at the buffer boundaries. The buffer gets converted into a string of wchar_t where those partials are invalid and make MultiByteToWideChar replacing them with U+FFFD characters. The implementation needs to check whether or not the buffer ends with a partial character. If so, only convert the code points which are complete, and save the partial code units in a cache that gets prepended to the next chunk of text.
Remark:
The PR includes the removal of the UTF-8 Byte Order Mark if it is present at the beginning of the stream read.

Validation Steps Performed

Sorry I'm struggling with the update 😅 I'm still a GitHub newbie.

Have you encountered an incoming BOM as the first three bytes on the output pipe?

No. It was rather a guess. I removed the BOM check.

Found a misleading comment which was a leftover from one of my early tests.
If you still have any concerns please let me know.

You gave detailed instrunctions. That made it difficult for me to fail entirely 😃
BTW Why not asking contributors for support for small tasks like that. It's one of the advantages of open source. Although I would have a hard time to find the position in the code where to implement or update things. That's where you are experienced. So if someone opens an issue and you already have a rough idea then just leave a hint like "we have to have a look at this file or review this class or that function". I think this has the potential to save you some hours of coding and testing. And it would help to close issues even faster.

We’ve got the “Help-Wanted” tag for that! Not all of those issues have guidance or suggestions on how to proceed, of course, but we’re getting there. Thanks! 🙂

I hope I was able to address all change requests 🙂

@miniksa Please have a look at @DHowett-MSFT 's comment: https://github.com/Microsoft/terminal/issues/386#issuecomment-495954828
I could try to move Utf8ToWideCharParser to types\ (in case I'm even allowed). Although one of the concerns was that we probably don't need to check the whole buffer. That was the reason why I proposed the check of the buffer end only.
I'd appreciate a more clear statement of how you want me to continue. If you don't see the chance to make any good out of this PR then I'm also okay to close it without merging. Just let me know 🙂

I am OK with a procedure that checks only the end of the buffer instead of the whole thing like the existing Utf8ToWideCharParser class. I just want it to be covered by a unit test somehow.
I'd like to either see it:

extracted into a method inside the TerminalConnection project and have a unit test applied there (which would probably require creating a unit test project UnitTests_TerminalConnection).
OR
extracted into a method inside the types project and add a test method, class, etc. to the existing test project Types.Unit.Tests (either its own Utf8Tests.cpp or just load it into UtilsTests.cpp as a new test.)

Option 2 is probably the easiest for you to accomplish.
Does that help?

Thanks @miniksa
Before I have seen your comment I already tried to find a way out of this stalemate.
I thought about a class for a static instance in the function.
Concept:
class UTF8OutPipeReader
    {
    public:
        UTF8OutPipeReader(wil::unique_hfile& outPipe) :
            _outPipe{ outPipe }
        {
        }

        [[nodiscard]] HRESULT Read(_Out_ std::string_view& strView)
        {
            DWORD dwRead{};
            bool fSuccess{};

            // in case of early escaping
            *_buffer = 0;
            strView = reinterpret_cast<char*>(_buffer);

            // copy UTF-8 code units that were remaining from the previously read chunk (if any)
            if (_dwPartialsLen != 0)
            {
                std::move(_utf8Partials, _utf8Partials + _dwPartialsLen, _buffer);
            }

            // try to read data from standard input
            fSuccess = !!ReadFile(_outPipe.get(), &_buffer[_dwPartialsLen], std::extent<decltype(_buffer)>::value - _dwPartialsLen, &dwRead, nullptr);

            dwRead += _dwPartialsLen;
            _dwPartialsLen = 0;

            if (dwRead == 0) // quit if no data has been read and no cached data was left over
            {
                return S_OK;
            }
            else if (!fSuccess) // reading failed
            {
                return static_cast<HRESULT>(-1);
            }

            const BYTE* const endPtr{ _buffer + dwRead };
            const BYTE* backIter{ endPtr - 1 };
            // If the last byte in the buffer was a byte belonging to a UTF-8 multi-byte character
            if ((*backIter & _Utf8BitMasks::MaskAsciiByte) > _Utf8BitMasks::IsAsciiByte)
            {
                // Check only up to 3 last bytes, if no Lead Byte was found then the byte before must be the Lead Byte and no partials are in the buffer
                for (DWORD dwSequenceLen{ 1UL }, stop{ dwRead < 4UL ? dwRead : 4UL }; dwSequenceLen < stop; ++dwSequenceLen, --backIter)
                {
                    // If Lead Byte found
                    if ((*backIter & _Utf8BitMasks::MaskContinuationByte) > _Utf8BitMasks::IsContinuationByte)
                    {
                        // If the Lead Byte indicates that the last bytes in the buffer is a partial UTF-8 code point then cache them:
                        //  Use the bitmask at index `dwSequenceLen`. Compare the result with the operand having the same index. If they
                        //  are not equal then the sequence has to be cached because it is a partial code point. Otherwise the
                        //  sequence is a complete UTF-8 code point and the whole buffer is ready for the conversion to hstring.
                        if ((*backIter & _cmpMasks[dwSequenceLen]) != _cmpOperands[dwSequenceLen])
                        {
                            std::move(backIter, endPtr, _utf8Partials);
                            dwRead -= dwSequenceLen;
                            _dwPartialsLen = dwSequenceLen;
                        }

                        break;
                    }
                }
            }

            // give back a view of the part of the buffer that contains complete code points only
            strView = std::string_view{ reinterpret_cast<char*>(_buffer), dwRead };
            return S_OK;
        }

    private:
        enum _Utf8BitMasks : BYTE
        {
            IsAsciiByte = 0b0'0000000, // Any byte representing an ASCII character has the MSB set to 0
            MaskAsciiByte = 0b1'0000000, // Bit mask to be used in a bitwise AND operation to find out whether or not a byte match the IsAsciiByte pattern
            IsContinuationByte = 0b10'000000, // Continuation bytes of any UTF-8 non-ASCII character have the MSB set to 1 and the adjacent bit set to 0
            MaskContinuationByte = 0b11'000000, // Bit mask to be used in a bitwise AND operation to find out whether or not a byte match the IsContinuationByte pattern
            IsLeadByteTwoByteSequence = 0b110'00000, // A lead byte that indicates a UTF-8 non-ASCII character consisting of two bytes has the two highest bits set to 1 and the adjacent bit set to 0
            MaskLeadByteTwoByteSequence = 0b111'00000, // Bit mask to be used in a bitwise AND operation to find out whether or not a lead byte match the IsLeadByteTwoByteSequence pattern
            IsLeadByteThreeByteSequence = 0b1110'0000, // A lead byte that indicates a UTF-8 non-ASCII character consisting of three bytes has the three highest bits set to 1 and the adjacent bit set to 0
            MaskLeadByteThreeByteSequence = 0b1111'0000, // Bit mask to be used in a bitwise AND operation to find out whether or not a lead byte match the IsLeadByteThreeByteSequence pattern
            IsLeadByteFourByteSequence = 0b11110'000, // A lead byte that indicates a UTF-8 non-ASCII character consisting of four bytes has the four highest bits set to 1 and the adjacent bit set to 0
            MaskLeadByteFourByteSequence = 0b11111'000 // Bit mask to be used in a bitwise AND operation to find out whether or not a lead byte match the IsLeadByteFourByteSequence pattern
        };

        // array of bitmasks
        constexpr const static BYTE _cmpMasks[]{
            0, // unused
            _Utf8BitMasks::MaskContinuationByte,
            _Utf8BitMasks::MaskLeadByteTwoByteSequence,
            _Utf8BitMasks::MaskLeadByteThreeByteSequence,
        };

        // array of values for the comparisons
        constexpr const static BYTE _cmpOperands[]{
            0, // unused
            _Utf8BitMasks::IsAsciiByte, // intentionally conflicts with MaskContinuationByte
            _Utf8BitMasks::IsLeadByteTwoByteSequence,
            _Utf8BitMasks::IsLeadByteThreeByteSequence,
        };

        wil::unique_hfile& _outPipe;
        BYTE _buffer[256]{ 0 }; // buffer for the chunk read (small size for debugging)
        BYTE _utf8Partials[4]{ 0 }; // buffer for code units of a partial UTF-8 code point that have to be cached
        DWORD _dwPartialsLen{}; // number of cached UTF-8 code units
    };

What do we have:

For @DHowett-MSFT I kept the check of the end of the buffer only.
@zadjii-msft will get names for the values. But we lost the WI_... macros.
@miniksa will get the helper object.

Any thoughts on that? Would it be worth to continue working on it?

I just want it to be covered by a unit test somehow.

That will become another challenge for me 😅

Thanks @miniksa Michael Niksa FTE
Before I have seen your comment I already tried to find a way out of this stalemate.
I thought about a class for a static instance in the function.
Concept:
class UTF8OutPipeReader
    {
    public:
        UTF8OutPipeReader(wil::unique_hfile& outPipe) :
            _outPipe{ outPipe }
        {
        }

        [[nodiscard]] HRESULT Read(_Out_ std::string_view& strView)
        {
            DWORD dwRead{};
            bool fSuccess{};

            // in case of early escaping
            *_buffer = 0;
            strView = reinterpret_cast<char*>(_buffer);

            // copy UTF-8 code units that were remaining from the previously read chunk (if any)
            if (_dwPartialsLen != 0)
            {
                std::move(_utf8Partials, _utf8Partials + _dwPartialsLen, _buffer);
            }

            // try to read data from standard input
            fSuccess = !!ReadFile(_outPipe.get(), &_buffer[_dwPartialsLen], std::extent<decltype(_buffer)>::value - _dwPartialsLen, &dwRead, nullptr);

            dwRead += _dwPartialsLen;
            _dwPartialsLen = 0;

            if (dwRead == 0) // quit if no data has been read and no cached data was left over
            {
                return S_OK;
            }
            else if (!fSuccess) // reading failed
            {
                return static_cast<HRESULT>(-1);
            }

            const BYTE* const endPtr{ _buffer + dwRead };
            const BYTE* backIter{ endPtr - 1 };
            // If the last byte in the buffer was a byte belonging to a UTF-8 multi-byte character
            if ((*backIter & _Utf8BitMasks::MaskAsciiByte) > _Utf8BitMasks::IsAsciiByte)
            {
                // Check only up to 3 last bytes, if no Lead Byte was found then the byte before must be the Lead Byte and no partials are in the buffer
                for (DWORD dwSequenceLen{ 1UL }, stop{ dwRead < 4UL ? dwRead : 4UL }; dwSequenceLen < stop; ++dwSequenceLen, --backIter)
                {
                    // If Lead Byte found
                    if ((*backIter & _Utf8BitMasks::MaskContinuationByte) > _Utf8BitMasks::IsContinuationByte)
                    {
                        // If the Lead Byte indicates that the last bytes in the buffer is a partial UTF-8 code point then cache them:
                        //  Use the bitmask at index `dwSequenceLen`. Compare the result with the operand having the same index. If they
                        //  are not equal then the sequence has to be cached because it is a partial code point. Otherwise the
                        //  sequence is a complete UTF-8 code point and the whole buffer is ready for the conversion to hstring.
                        if ((*backIter & _cmpMasks[dwSequenceLen]) != _cmpOperands[dwSequenceLen])
                        {
                            std::move(backIter, endPtr, _utf8Partials);
                            dwRead -= dwSequenceLen;
                            _dwPartialsLen = dwSequenceLen;
                        }

                        break;
                    }
                }
            }

            // give back a view of the part of the buffer that contains complete code points only
            strView = std::string_view{ reinterpret_cast<char*>(_buffer), dwRead };
            return S_OK;
        }

    private:
        enum _Utf8BitMasks : BYTE
        {
            IsAsciiByte = 0b0'0000000, // Any byte representing an ASCII character has the MSB set to 0
            MaskAsciiByte = 0b1'0000000, // Bit mask to be used in a bitwise AND operation to find out whether or not a byte match the IsAsciiByte pattern
            IsContinuationByte = 0b10'000000, // Continuation bytes of any UTF-8 non-ASCII character have the MSB set to 1 and the adjacent bit set to 0
            MaskContinuationByte = 0b11'000000, // Bit mask to be used in a bitwise AND operation to find out whether or not a byte match the IsContinuationByte pattern
            IsLeadByteTwoByteSequence = 0b110'00000, // A lead byte that indicates a UTF-8 non-ASCII character consisting of two bytes has the two highest bits set to 1 and the adjacent bit set to 0
            MaskLeadByteTwoByteSequence = 0b111'00000, // Bit mask to be used in a bitwise AND operation to find out whether or not a lead byte match the IsLeadByteTwoByteSequence pattern
            IsLeadByteThreeByteSequence = 0b1110'0000, // A lead byte that indicates a UTF-8 non-ASCII character consisting of three bytes has the three highest bits set to 1 and the adjacent bit set to 0
            MaskLeadByteThreeByteSequence = 0b1111'0000, // Bit mask to be used in a bitwise AND operation to find out whether or not a lead byte match the IsLeadByteThreeByteSequence pattern
            IsLeadByteFourByteSequence = 0b11110'000, // A lead byte that indicates a UTF-8 non-ASCII character consisting of four bytes has the four highest bits set to 1 and the adjacent bit set to 0
            MaskLeadByteFourByteSequence = 0b11111'000 // Bit mask to be used in a bitwise AND operation to find out whether or not a lead byte match the IsLeadByteFourByteSequence pattern
        };

        // array of bitmasks
        constexpr const static BYTE _cmpMasks[]{
            0, // unused
            _Utf8BitMasks::MaskContinuationByte,
            _Utf8BitMasks::MaskLeadByteTwoByteSequence,
            _Utf8BitMasks::MaskLeadByteThreeByteSequence,
        };

        // array of values for the comparisons
        constexpr const static BYTE _cmpOperands[]{
            0, // unused
            _Utf8BitMasks::IsAsciiByte, // intentionally conflicts with MaskContinuationByte
            _Utf8BitMasks::IsLeadByteTwoByteSequence,
            _Utf8BitMasks::IsLeadByteThreeByteSequence,
        };

        wil::unique_hfile& _outPipe;
        BYTE _buffer[256]{ 0 }; // buffer for the chunk read (small size for debugging)
        BYTE _utf8Partials[4]{ 0 }; // buffer for code units of a partial UTF-8 code point that have to be cached
        DWORD _dwPartialsLen{}; // number of cached UTF-8 code units
    };

What do we have:

For @DHowett-MSFT
        Dustin Howett
        FTE I kept the check of the end of the buffer only.

@zadjii-msft
        Mike Griese
        FTE will get names for the values. But we lost the `WI_...` macros.

@miniksa
        Michael Niksa
        FTE will get the helper object.

Any thoughts on that? Would it be worth to continue working on it?

I just want it to be covered by a unit test somehow.

That will become another challenge for me 😅

Yeah, that's great!
I'd put the entire class/file into the types library as a new cpp object and then add a corresponding file to the Types.Unit.Tests project called UTF8OutPipeReaderTests.cpp and make some tests for it there or something.

The following commit enables you to leave some comments in the code.

make some tests for it there

The TAEF is completely new to me. Seems I will have a hard time. Currently I already get errors if I only open a file such as UtilsTests.cpp in VS because the included "WexTestClass.h" can't be found.
Do we have any documentation about how to perform unit tests for the terminal?

Figuring out how to get the TAEF working was driving me crazy 🤪

@german-one Thanks for doing so much for this pull request! :smile: We really appreciate it.

I'm sorry TAEF and unit testing was trouble for you

Be assured that I fully understand that code needs proper testing and verification. Am aware that the console and terminal will be released to billions of machines out there. I certainly don't want to be the culprit for causing trouble all around the world. 😉 So don't worry. I'm learning something new every day. Thanks four your patience.

I had skipped that detail. But now I think I addressed all of your requsts.

It seems, there are still some issues left (blinking and improperly rendered characters):

Fortunately, that is not related to partial encoded utf-8 codepoints :smile:

@DHowett-MSFT Shall I open a new issue then?

Only if you can express exactly why those characters are "improperly rendered." Most of the people on my team don't know what they're supposed to look like.
HOWEVER: There's a chance it's already covered by a bunch of existing issues. We have known deficiencies in rendering Thai text, in rendering things with composing characters, in rendering things where there's a mismatch in number of cells to number of codepoints (we only support one codepoint -> 1 or 2 cells right now)

So, I assume, the blinking is Okay? ;)
Here is the Notepad rendering:

Nah I mean: If your report is "they don't look right," we need to know why. If your report is "they're blinking  and they absolutely should not be," that's completely and totally reasonable :smile:

@DHowett-MSFT

We have known deficiencies [...] in rendering things with composing characters [...]

Now that I read that, solving this problem should be also a task for UTF8OutPipeReader::Read I think. Something like "if the last complete code point was no Combining Diacritical Mark (U+0300..036F, U+1AB0..1AFF, U+1DC0..1DFF, U+20DC..20FF), put it in the cache, too". That's because we don't know if the partial or the upcoming character is a combining character.
(I just hope that winrt::to_hstring would make precomposed wide characters out of them afterwards.)
Do you think I should try to update the code accordingly?

@german-one I'd hold off on that, actually. We shouldn't do this just yet, because we need to make sure that conhost (the console host, which doesn't use WinRT at all; it also serves as the API server for the legacy Win32 console APIs) knows to store combining characters and grapheme clusters in the buffer. If we just handle them at the Windows Terminal layer there will be a mismatch/miscount in the number of cells on the screen.

Well, I already successfully tried to keep combining characters complete. And I think that, regardless of the behavior of conhost, combining characters must not be split. Correct me if I'm wrong, but won't that still be the task for the Windows Terminal if it reads the pipe?
FWIW winrt::to_hstring don't make precomposed wide characters out of them. Thus, forwarding only complete combining characters still doesn't solve anything. But it might be a precondition for further improvements.
Just get back to me whenever you think the time has come ...


Ability to join groups with moderation

This feature comprises two scenarios:

Ability to submit a join request option for visible groups

This is a request from uservoice:

Visible groups should have the option of sending a group join request to the group manager. This request should be visible in activity feed and email notifications of all managers of the group.

When dealing with private groups, managers are able to share a link to the group with users who then just need to click a button "join group" and that's it. This action should also have moderation.

All in all, there are several subtasks:

[ ] Join group request submission for scenario 1
[ ] Join group request submission for scenario 2
[ ] Notifications for group managers
[ ] Moderation for group managers

Hi,
I've finished the developments :)
You can find the pull requests here :

3akai-ux : https://github.com/oaeproject/3akai-ux/pull/4198
Hilary : https://github.com/oaeproject/Hilary/pull/1543

Tell me if you want some screenshots of the front (to tell me if there are elements to modify).

Tell me if you want some screenshots of the front (to tell me if there are elements to modify).

Yes, I do. I will test it anyway locally, but this way I'll know what to expect :)

All reviewed, feel free to ask me anything!


[Font bug] Limited support for Cyrillics

When using the Fritz Quadrata font, the same one the game should be using, as soon as Cyrillic characters show up in chat, the font changes and even stops using an outline as set by my settings.

Sadly, there's just nothing I can do about it due to how fonts are implemented, we simply don't have enough control over this stuff from Lua 😭


SSL cert chains can fail validation due to trailing whitespace

This Bug Report affects these Traffic Control components:

Traffic Ops

What did you do?

Tried to verify an SSL cert chain
What did you expect to happen?

SSL cert chain verification passes
What actually happened?

SSL cert chain verification fails due to trailing whitespace in the verification-generated cert chain

Fixed in #7628


Error setting state on unmounted component

It the async promise resolves after the component has unmounted, React complains in red at the console.
You could just test inside the resolve callback if the component is mounted before calling setState:
if(this.isMounted()) {
  this.setState({...});
}

...but it is considered a bad practice:
https://facebook.github.io/react/blog/2015/12/16/ismounted-antipattern.html
So you might want to wrap that promise with makeCancelable.

Yes this looks great, thanks for that. I'm working on it.

I published a version 1.1.2 which removes the warning


Recent changeset comments don't populate as suggestions when preparing to submit changeset

Description
A list of recent changeset comments, recent to older, previously popped up as dropdown when typing a changeset comment. It doesn't do that anymore and I miss it.
Version
2.0.1
What browser are you seeing the problem on? What version are you running?
Chrome v111.0
The OS you're using
Windows 10 Pro
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid#background=VBMPImagery_Most_Recent&datasets=msBuildings,a2b4bbc1bdbb4c8787a76ce787a948b8&disable_features=boundaries&map=19.96/38.21835/-78.83516&photo_overlay=mapillary,mapillary-map-features,mapillary-signs&poweruser=true
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

as of 2023-04-11 this feature is working.


Filter 'Running' phase in getResourcePercentDisplay

I am very sorry that the previous fix is not complete. I missed the resource calculation in getResourcePercentDisplay.
Before this patch,

After this patch, the default view is like,

And the sorters are also correct.

The image for test: https://hub.docker.com/layers/wizawu/k8dash/latest/images/sha256-7b0132749d780663b50e84731eb2ef33e57bb3a98e940c486f517ab0c01aa18c

Looks great! Thx @wizawu


Bug with uuid dependency in<EMAIL_ADDRESS>

With the current version 3.0.1 there was a bug introduced that makes the plugin unusable inside a node project. The module "uuid" is loaded in some files of node-cron but the package.json does not state it as a dependency<EMAIL_ADDRESS>will break on all systems were uuid is not installed in the main package.json of the node project.
Current workaround is either reverting to 3.0.0 or installing uuid alongside node-cron in your project until this is fixed.

#321 introduced the import but did not update dependencies. uuid should be either a peerDep or a dependency

Just basing off a hunch, but I'm also on 3.0.0 and I see the same issue.

Not sure why it didn't happen earlier, but this error happened right after I updated a single package

I don't know if this could be related, but version 3.0.1 sent my memory usage skyrocketing for my Node.js application on DigitalOcean. On version 3.0.0 the app runs at about 23% memory usage, but 3.0.1 sent it up to 100% and the app reset itself every 6 minutes or so to avoid resource exhaustion. Reverting node-cron back to 3.0.0 solved the problem.

@ugurgunes95
How did downgrad the package? I installed it with npm i --save<EMAIL_ADDRESS>and that worked.

Can't the built-in node's crypto.randomUUID be used?


Move symol length check out of rebinding loop

The symbol length check is loop invariant code, and so can be done once before looping over the rebindings. Also, replaces str_longer with strnlen.
This optimization was the work of @ocrickard in #26.

Omg thank you!

Yay!


How to use Alamofire in a Swift-3 app which support iOS-8.??

Hi,
I think Alamofire 4.0 support Swift-3 but the minimum target is iOS-9. And Alamofire 3.x versions support iOS-8 but this doesnt support Swift-3. So how can I use Alamofire in a Swift-3 app which support iOS-8.??

You cannot at the moment. We're bringing back iOS 8 and macOS 10.10 support in Alamofire 4.1.0 which should release today or tomorrow due to Apple removing Swift 2.3 support sooner than we expected.
Cheers. 🍻


cancel native events on hammer event trigger? eg touch

Hello,
I have the following issue: http://stackoverflow.com/questions/24998194/hammer-js-click-triggered-on-dom-elements-behind-tapped-after-tap-action
I hope to prevent it but cancelling native events to prevent them from bubbling/triggering on  other elements after they are used on a hammer instance. for example on touchend (which is used on eg tap) touch (and click) events should not be triggered on other elements.
is there a way to do this? perhaps with a custom listener? shouldn't it be the default behaviour anyway? I'm using hammer v1 but I can update if this issue can be solved in v2 but not in v1..
I can find a number of similar issues being reported on both this repository and on stackoverflow
thanks!

Closing in favor of #808


Fix code style to adhere to editorconfig rules

Azure.Core again

Azure.Core again

Was still working on the PR. Core changes are now removed.

You can use draft feature if PR is still work in progress


nodePackages: fix generate.sh failing on path with space

Motivation for this change
When the path to the Nixpkgs repo contains a space, generate.sh would fail. This can be esily fixed by adding quotes around the variable containing the path.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

I squashed the commits and rebased onto the latest master.

I squashed the commits and rebased onto the latest master.

I could have squash merged it.


Store version numbers in SQLite

and check that insert/update requests have increased the version number.

Passing cl/296355014


Improve performance of Command Prompt

if I run a maven build in a cmd window it seems that screen output eats more CPU than the actual build.

As seen on Reddit: https://www.reddit.com/r/java/comments/jxllez/java_development_on_windows_request_for/gcyg9od/

https://aka.ms/terminal

Maybe this should also be in Microsoft/windows-dev-performance


Should content-base-main-only.html move to layouts?

https://github.com/cfpb/cfgov-refresh/blob/flapjack/_includes/content-base-main-only.html appears to be a layout, should it go into /_layouts/?

Yes, please :)


Update LightGroup signature.

Patches the signature of the created LightGroup objects such that light groups can be successfully created under HA 2021.8.0 and above. (Fixes #131 .)

Thanks a lot!


Generate JSON tests even after failure against reference implementation

See: https://github.com/FHIR/sql-on-fhir-v2/issues/197#issuecomment-1953826419

Thanks, fixed in a107fa3.


Fix Tests

Try this
Passport 11 don't support laravel 8
PHPUnit 9.6.10 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.1.2
Configuration: /laravel-permission/phpunit.xml.dist

...............................................................  63 / 525 ( 12%)
............................................................... 126 / 525 ( 24%)
............................................................... 189 / 525 ( 36%)
............................................................... 252 / 525 ( 48%)
............................................................... 315 / 525 ( 60%)
............................................................... 378 / 525 ( 72%)
............................................................... 441 / 525 ( 84%)
............................................................... 504 / 525 ( 96%)
.....................                                           525 / 525 (100%)

Time: 00:13.213, Memory: 62.00 MB

OK ( 525 tests, 1240 assertions)

I think that with the tests fixed you will be able to continue your work

Hi @drbyte, Is this approach too much intricate?

@erikn69

Is this approach too much intricate?

This is probably the best approach for now.
I'd prefer to move the changes to missingTraitAuthorizable into a separate PR, because they're not entirely specific to this Passport PR.
I'll also push the fix to Show.php directly now.

Would it be convenient to leave Laravel 8 behind?

I'm not ready to do that in 6.0.  Willing to do it in 7.0.

Hi @SuperDJ,  try this version

@erikn69 I think this will meet most of the scenarios. The only one I'm not sure of is the following. Passport can login as different "types":

Authorization Code Grant with PKCE;
Password Grant Tokens;
Implicit Grant Tokens;
Client Credentials Grant Tokens;
Personal Access Tokens.

Each of these types should fallback to something. For example the Password Grant should just use the User model. The Client Credentials Grant, the one we are interested in, should use the Client model. And currently I'm not sure if we don't interfere with the others. I mean the change we are making should only apply for the Client Credentials Grant. But found this method that could help. I'm just not sure if it would be enough to check on those two values.

@erikn69 tested the changes locally and they work


resolve #54 by updating fields, other fixes

This update allows the use of the become_username, become_password, and other fields that allow privilege escalation by becoming another user through the su command, and others. This has been manually tested to the point of launching new jobs with an su-type privilege escalation.
In addition to that, the @resources.command decorator is added on the project create method overriding the generic resource version. This is a fix for a problem that a previous commit of mine created, which caused tower-cli to crash when creating a job template.

@AlanCoding looks great, thanks for the contribution.  Please merge when ready.


Update Set-NetIPv*Protocol NeighborCacheLimit description to be more accurate

The NeighborCacheLimit parameter to the Set-NetIPv*Protocol cmdlets is misleadingly named.  This PR updates the descriptions for the parameter across all Windows versions in this repo.

@e0i ping.

@tiburd The content can be merged￼. Thanks.

Docs Build status updates of commit cf608f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/nettcpip/Set-NetIPv4Protocol.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2012r2-ps/nettcpip/Set-NetIPv6Protocol.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2016-ps/nettcpip/Set-NetIPv4Protocol.md
:white_check_mark:Succeeded
View (WindowsServer2016-ps)

docset/winserver2016-ps/nettcpip/Set-NetIPv6Protocol.md
:white_check_mark:Succeeded
View (WindowsServer2016-ps)

docset/winserver2019-ps/nettcpip/Set-NetIPv4Protocol.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

docset/winserver2019-ps/nettcpip/Set-NetIPv6Protocol.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

docset/winserver2022-ps/nettcpip/Set-NetIPv4Protocol.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/nettcpip/Set-NetIPv6Protocol.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Slight hiccup in queue doxygen

The doxygen-created documentation here (for queue_is_empty) has gone a bit wonky!
Returns the highest level reached by the specified queue since it was created or since the max level was reset.

Parameters
q	Pointer to a queue_t structure, used as a handle
Returns
Maximum level of the queue
Reset the highest level reached of the specified queue.

Parameters
q	Pointer to a queue_t structure, used as a handle
Check if queue is empty

Parameters
q	Pointer to a queue_t structure, used as a handle
Returns
true if queue is empty, false otherwise
This function is interrupt and multicore safe.

Looking at the corresponding source, it seems that it's probably getting confused by the #if PICO_QUEUE_MAX_LEVEL ?

Fixed by #690


Fix the double typeguards

Collection of what it does / fixes

removes the typeguards from utils.ts, because there is an extra file for it named typeguards.ts

it got accidentally added in cf75bd0fc10b3d4909fd838c6c634b111229762c

Coverage increased (+0.8%) to 88.372% when pulling aff7367b12c42c6b6c66cce002862f2b1dde4cfc on hasezoey:feature/fixtypeguards into 0d68100051cef0e9ec2ff1346a574e0873f1873e on szokodiakos:master.

@Ben305 Bump


Add Findstemmer.cmake to find lib stemmer

This pr adds lib stemmer to setup script and allows cmake to find this installed lib during building velox.

Please update the PR title, drop [VL]

Please update the PR title, drop [VL]

@jinchengchenghh, thanks for your comment! Just updated.

@kgpai, could you also take a look? Thanks!

@PHILO-HE There are Fuzzer failures. Would you take a look?

@PHILO-HE There are Fuzzer failures. Would you take a look?

@mbasmanova, we will look into this issue. Assume it is not relevant to this pr.

@mbasmanova, I just rebased the pr. Thanks!

@mbasmanova, the CI failure is irrelevant, and it was just fixed by https://github.com/facebookincubator/velox/commit/4744994ae0f4c496ac65cf4637cab2626203ae9a.

@PHILO-HE We had to revert this PR because it broke pyvelox and fuzzer builds. It's because the docker images produced did not work well with pyvelox . I recommend we create another PR just to make changes to the images, see if they work and subsequently land them and then re-add this findstemmer change. Thank you !

@PHILO-HE We had to revert this PR because it broke pyvelox and fuzzer builds. It's because the docker images produced did not work well with pyvelox . I recommend we create another PR just to make changes to the images, see if they work and subsequently land them and then re-add this findstemmer change. Thank you !

@kgpai, I understand the reverting. Will fix that issue.


Adding IBM PC 8-sector formats

I've come across some early IBM PC disks that use 8-sectors per track rather than the more common 9.  This branch adds the format so I can manipulate them.

LGTM, thanks! Also, weird...


BUG:Rateus page on book_recommend page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Rateus page on book_recommend page.
we have to remove the rate us page form there.
https://github.com/anuragverma108/SwapReads/assets/155576040/36d849b2-d5dd-40f7-8cb5-f43d91ad0bb8
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

i want to work on this issue  can u please assign me ?


Refactor custom logger

Current custom logger only allows to log into console.
Use winston to refactor this and make a logger that logs to both console and log file. Maybe make it flexible enough to only use a specific transport in some case (only console or only file for example)
This logger should be applied everywhere so that we keep data of script execution.
See #42 for application to this

Base logger implemented, soon pushing to master
Closing for now.


OIDC authentication using authelia fails with no roles

Description
Using Authelia for OIDC Authentication, Version 2.0.0 allows the users to authenticate, though as authelia's OIDC implementation is not complete, there are no roles in the claim.
Version 3.0.0, has a failure point when there are no roles for the user. This means that upgrading from 2.0.0 to 3.0.0
will break existing systems.
Its not ideal that there are no roles, but i was able to get my user to be "admin" by disabling OIDC, logging in
as the auto created admin user and then making my user from authelia "admin", then re-enabling OIDC.
Related Issue

Fixes <issue_link>

Motivation and Context
Being able to use authelia to authenticate. Otherwise would require adding an extra authentication handler
like keycloak which is not ideal.
How Has This Been Tested?
Installed on my test container, built with the web ui included, was able to successfully authenticate and be presented with
the user interface.

...

Screenshots (if appropriate):
Types of changes

[X ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[X] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

Did some digging around, setting
PROXY_ROLE_ASSIGNMENT_DRIVER=default

allows the login to continue, closing this issue and PR

@mwinters-stuff Thank you! We should mention this in the 3.0.0 Release notes and the Upgrade documentation
@mmattel Can you take care?


Error saving, yields empty source file.

Hello
Today I had an issue, where VScode said that I could not save my RPG program.
I then closed the program and tried to open it again, and now it was all gone.
Is there any dumps or temporares I can recreate it from ?

⚠ REMOVE THIS LINE AND ANY SENSITIVE INFORMATION BELOW! ⚠

Context
Version

Code for IBM i version
2.2.1

Visual Studio Code version
1.82.2

Operating System
win32_x64

Active extensions
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.3.2
Db2 for IBM i (vscode-db2i): 0.4.1
Emmet (emmet): 1.0.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
Git Blame (gitblame): 10.5.1
GitHub (github): 0.0.1
GitHub Authentication (github-authentication): 0.0.2
GitHub Copilot Chat (copilot-chat): 0.7.1
GitLens — Git supercharged (gitlens): 14.3.0
IBM i Notebooks (vscode-ibmi-notebooks): 0.0.6
JSON Language Features (json-language-features): 1.0.0
Merge Conflict (merge-conflict): 1.0.0
Microsoft Account (microsoft-authentication): 0.0.1
Node Debug Auto-attach (debug-auto-launch): 1.0.0
Peacock (vscode-peacock): 4.2.2
RPGLE (vscode-rpgle): 0.21.4
RPGLE Free (vscode-rpgfree): 0.0.26
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0
WSL: Recommender (remote-wsl-recommender): 0.0.19

Remote system

Setting
Value

IBM i OS
V7R4M0

Tech Refresh
8

CCSID
277

SQL
Enabled

Source dates
Enabled

Enabled features

/QOpenSys/pkgs/bin
/usr/bin
/QSYS.lib/ILEDITOR.lib
/QSYS.LIB
/QIBM/ProdData/IBMiDebugService/bin

bash
attr
GENCMDXML.PGM
QZDFMDB2.PGM
startDebugService.sh

chsh
iconv
GETNEWLIBL.PGM

git
setccsid

grep
tar

ls

md5sum

sort

stat

tn5250

Shell env
BUILDLIB=ANLIB
CURLIB=ANLIB
HOME=/home/ANMET
HOST=mensa
LIBLS=ICEBREAK SYSAFD UNO700CFUT SMSDB NHO NHOPRG YXDB NHODATA YXDTA24 QGPL QTEMP ANLIB
LOGIN=anmet
LOGNAME=anmet
MAIL=/var/spool/mail/anmet
OLDPWD=/home/ANMET
PATH=/QOpenSys/usr/bin:/usr/ccs/bin:/QOpenSys/usr/bin/X11:/usr/sbin:.:/usr/bin
PWD=/home/ANMET
SHELL=/QOpenSys/pkgs/bin/bash
SHLVL=1
SSH_CLIENT=<IP_ADDRESS> 56390 22
SSH_CONNECTION=<IP_ADDRESS> 56390 <IP_ADDRESS> 22
TZ=<CET>-1<CEST>,M3.5.0,M10.5.0/03:00:00
USER=anmet
USERNAME=anmet
_=/QOpenSys/usr/bin/env

Variants
{
  "american": "#@$",
  "local": "ÆØÅ"
}

Errors
[
  {
    "command": "system \"CRTLIB LIB(ILEDITOR) TEXT('Code for i temporary objects. May be cleared.')\"",
    "code": 255,
    "stderr": "CPF2111: Library ILEDITOR already exists.",
    "cwd": "/home/ANMET"
  },
  {
    "command": "ls -p /QIBM/ProdData/IBMiDebugService/bin/certs/debug_service.pfx",
    "code": 2,
    "stderr": "/QIBM/ProdData/IBMiDebugService/bin/certs/debug_service.pfx not found",
    "cwd": "/home/ANMET"
  }
]

Please share the contents of your Code for IBM i output channel after you see the error.

Suggestion is to include that in the "report an issue" if possible of cause.
I no longer have this, since I reconnected to the session to try and fix the "error saving".
If it happens again, I'll include it :-)


Documentation for M-BPTREE does not match all functions

The documentation for M-BPTREE specialized methods indicates that there is a name_pop() function with a void return type. But the actual source currently has a name_pop_at() with a bool return type. I didn't spend time to audit the other interface functions but I was making use of this one.
The documentation should agree with the desired public API functions.

Indeed, the documentation was not correct. I have fixed it, and check other specialized functions.


Fb1451

Great, make the change.
On Fri, Apr 26, 2019 at 12:05 PM spiresheep<EMAIL_ADDRESS>wrote:

@spiresheep commented on this pull request.
In source/sorted_table_model.ts
https://github.com/eidolonsystems/clipped_table/pull/7#discussion_r279011624
:

}

public connect(slot: (operations: Operation[]) => void):
Kola.Listener<Operation[]> {

return null;

this.translatedTable.connect(slot);
return this.dispatcher.listen(slot);
}

private sort(tableModel: TableModel) {
const rowOrdering = [];
for(let i = 0; i < tableModel.rowCount; ++i) {

 rowOrdering.push(i);

}
rowOrdering.sort((a, b) => this.compareRows(a, b));
for(let i = 0; i < rowOrdering.length - 1; ++i) {

 this.translatedTable.moveRow(rowOrdering[i], i);

 for(let j = i + 1; j < rowOrdering.length; ++j) {

   if(rowOrdering[j] <= rowOrdering[i]) {

It's less than because the only value that could have been equal will
always be removed by that point.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/eidolonsystems/clipped_table/pull/7#discussion_r279011624,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD6MKGJVXVZFD6TY6IHKQPLPSMR4NANCNFSM4HIRCVPA
.

--
Kamal Mansouri
Computer Programmer
+1<PHONE_NUMBER>


[TS] Add isRequired to inputs

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
TS implementation of #1978

Implemented with #2547


fix issue #1453, disallow POST and HOST: commands to avoid proxy-redis connection to be dropped

https://github.com/CodisLabs/codis/issues/1453

Great work! Thank you so much!


[v.low] [usability] OOM results in "cannot start a container that has stopped"

This is very low priority: a hope that there's a way to provide a more helpful diagnostic for this error.
Setup: VM (m1.medium) with no swap. Force an OOM:
# podman --cgroup-manager=cgroupfs run -d -i -m 5246976 registry.access.redhat.com/rhel7/rhel:latest /bin/bash
a5f0b470febacb36beb34be61ea35011eb5f07fd83d2ea52dbc426d06b92189b

# podman --cgroup-manager=cgroupfs run -d -i -m 5246976 registry.access.redhat.com/rhel7/rhel:latest /bin/bash
cannot start a container that has stopped
`/usr/bin/runc start 447c64007114501935cad8be3cf0be0d02f18b6b421f0f468a7dd9e5a86454b1` failed: exit status 1

(Normally the second one fails, but sometimes it takes three). Actual error is from runc, and is from an OOM. System log shows:
Oct 02 16:22:41 esm-f29-1 kernel: Memory cgroup out of memory: Kill process 5379 (runc:[2:INIT]) score 1791 or sacrifice child
Oct 02 16:22:41 esm-f29-1 kernel: Killed process 5379 (runc:[2:INIT]) total-vm:481600kB, anon-rss:4296kB, file-rss:4756kB, shmem-rss:0kB
Oct 02 16:22:41 esm-f29-1 kernel: oom_reaper: reaped process 5379 (runc:[2:INIT]), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kB

This was entirely my fault: a mistake cleaning up my ansible setup code resulted in a swapless system. The error message, though, led me on some wild-goose chases. Is there any low-effort way to catch this and offer a more helpful message? If not, please close: my other reason for filing is in case someone in the future runs a web search. TIA.
podman-<IP_ADDRESS>-1.git1cd906d.fc29.x86_64
runc-1.0.0-55.dev.git578fe65.fc29.x86_64

Error message looks like a bug - we should be reporting the OOM from runc properly

@mheon Any movement on this issue?

Nope, never found the time to look at it

@haircommander Could you take a look at this?

I don't see this being fixed any time soon, even if there is a good way to fix it.

@baude @mheon What should we do with this one?

I think this is the kind of thing we expect to be fixed by ps --sync?

Or, really, cleanup processes in general?

runc doesn't seem to return an OOM error. Trying to reproduce I get:
starting container process caused "process_linux.go:446: container init caused \"read init-p: connection reset by peer\""
I'm under the impression that this must be addressed in runc. @mheon @edsantiago @rhatdan WDYT?

I think you have to try reproducing it on a super-low-RAM system. I'm still seeing it, podman-1.6.0-3.fc30.x86_64:
# podman --cgroup-manager=cgroupfs run -d -i -m 5246976 registry.access.redhat.com/rhel7/rhel:latest /bin/bash
ERRO[0000] cannot start a container that has stopped
cannot start a container that has stopped
Error: `/usr/bin/runc start 8477205ae163b71ea8543148362c21afc1fd79cf5c011402db9a5adea606d372` failed: exit status 1

...with the following in journalctl:
Oct 02 08:50:55 host-10-0-137-220 kernel: runc:[2:INIT] invoked oom-killer: gfp_mask=0xcc0(GFP_KERNEL), order=0, oom_score_adj=0
Oct 02 08:50:55 host-10-0-137-220 kernel: CPU: 0 PID: 18415 Comm: runc:[2:INIT] Not tainted 5.2.15-200.fc30.x86_64 #1
Oct 02 08:50:55 host-10-0-137-220 kernel: Hardware name: Red Hat OpenStack Compute, BIOS 1.11.0-2.el7 04/01/2014
Oct 02 08:50:55 host-10-0-137-220 kernel: Call Trace:
Oct 02 08:50:55 host-10-0-137-220 kernel:  dump_stack+0x5c/0x80
Oct 02 08:50:55 host-10-0-137-220 kernel:  dump_header+0x51/0x2ff
Oct 02 08:50:55 host-10-0-137-220 kernel:  oom_kill_process.cold+0xb/0x10
Oct 02 08:50:55 host-10-0-137-220 kernel:  out_of_memory+0x1d4/0x480
Oct 02 08:50:55 host-10-0-137-220 kernel:  mem_cgroup_out_of_memory+0xba/0xd0
Oct 02 08:50:55 host-10-0-137-220 kernel:  try_charge+0x72d/0x7b0
Oct 02 08:50:55 host-10-0-137-220 kernel:  ? __alloc_pages_nodemask+0x18e/0x340
Oct 02 08:50:55 host-10-0-137-220 kernel:  mem_cgroup_try_charge+0x99/0x1d0
Oct 02 08:50:55 host-10-0-137-220 kernel:  mem_cgroup_try_charge_delay+0x1d/0x40
Oct 02 08:50:55 host-10-0-137-220 kernel:  __handle_mm_fault+0xa59/0x1ac0
Oct 02 08:50:55 host-10-0-137-220 kernel:  ? __switch_to_asm+0x40/0x70
Oct 02 08:50:55 host-10-0-137-220 kernel:  ? __switch_to_asm+0x34/0x70
Oct 02 08:50:55 host-10-0-137-220 kernel:  handle_mm_fault+0xc4/0x1e0
Oct 02 08:50:55 host-10-0-137-220 kernel:  do_user_addr_fault+0x1f6/0x450
Oct 02 08:50:55 host-10-0-137-220 kernel:  do_page_fault+0x33/0x120
Oct 02 08:50:55 host-10-0-137-220 kernel:  ? async_page_fault+0x8/0x30
Oct 02 08:50:55 host-10-0-137-220 kernel:  async_page_fault+0x1e/0x30
Oct 02 08:50:55 host-10-0-137-220 kernel: RIP: 0033:0x7f467795dbc4
Oct 02 08:50:55 host-10-0-137-220 kernel: Code: 28 4c 8b 6c 24 20 49 39 d6 49 89 4e 60 0f 95 c2 48 83 c8 01 49 83 c0 10 0f b6 d2 48 c1 e2 02 48 09 ea 48 83 ca 01 49 89 50 f8 <48> 89 41 08 8b 05 96 d3 13 00 85 c0 0f 84 1c f7 ff ff e9 3a fb ff
Oct 02 08:50:55 host-10-0-137-220 kernel: RSP: 002b:00007ffe37a12280 EFLAGS: 00010202
Oct 02 08:50:55 host-10-0-137-220 kernel: RAX:<PHONE_NUMBER>019ff1 RBX: ffffffffffffffa8 RCX: 00005635ab15a010
Oct 02 08:50:55 host-10-0-137-220 kernel: RDX:<PHONE_NUMBER>0000f1 RSI:<PHONE_NUMBER>000000 RDI:<PHONE_NUMBER>000004
Oct 02 08:50:55 host-10-0-137-220 kernel: RBP:<PHONE_NUMBER>0000f0 R08: 00005635ab159f30 R09:<PHONE_NUMBER>000000
Oct 02 08:50:55 host-10-0-137-220 kernel: R10:<PHONE_NUMBER>0000f0 R11:<PHONE_NUMBER>00000d R12: 00007f4677a98cc0
Oct 02 08:50:55 host-10-0-137-220 kernel: R13:<PHONE_NUMBER>0000e0 R14: 00007f4677a98c60 R15:<PHONE_NUMBER>00000f
Oct 02 08:50:55 host-10-0-137-220 kernel: memory: usage 5124kB, limit 5124kB, failcnt 16
Oct 02 08:50:55 host-10-0-137-220 kernel: memory+swap: usage 5124kB, limit 10248kB, failcnt 0
Oct 02 08:50:55 host-10-0-137-220 kernel: kmem: usage 584kB, limit<PHONE_NUMBER>740988kB, failcnt 0
Oct 02 08:50:55 host-10-0-137-220 kernel: Memory cgroup stats for /libpod_parent/libpod-8477205ae163b71ea8543148362c21afc1fd79cf5c011402db9a5adea606d372: cache:0KB rss:4512KB rss_huge:0KB shmem:0KB mapped_file:0KB dirty:0KB writeback:0KB swap:0KB inactive_anon:0KB active_anon:4488KB inactive_file:0KB active_file:0KB unevictable:0KB
Oct 02 08:50:55 host-10-0-137-220 kernel: Tasks state (memory values in pages):
Oct 02 08:50:55 host-10-0-137-220 kernel: [  pid  ]   uid  tgid total_vm      rss pgtables_bytes swapents oom_score_adj name
Oct 02 08:50:55 host-10-0-137-220 kernel: [  18415]     0 18415   102038     2332   126976        0             0 runc:[2:INIT]
Oct 02 08:50:55 host-10-0-137-220 kernel: oom-kill:constraint=CONSTRAINT_MEMCG,nodemask=(null),cpuset=libpod-8477205ae163b71ea8543148362c21afc1fd79cf5c011402db9a5adea606d372,mems_allowed=0,oom_memcg=/libpod_parent/libpod-8477205ae163b71ea8543148362c21afc1fd79cf5c011402db9a5adea606d372,task_memcg=/libpod_parent/libpod-8477205ae163b71ea8543148362c21afc1fd79cf5c011402db9a5adea606d372,task=runc:[2:INIT],pid=18415,uid=0
Oct 02 08:50:55 host-10-0-137-220 kernel: Memory cgroup out of memory: Killed process 18415 (runc:[2:INIT]) total-vm:408152kB, anon-rss:4292kB, file-rss:5036kB, shmem-rss:0kB
Oct 02 08:50:55 host-10-0-137-220 kernel: oom_reaper: reaped process 18415 (runc:[2:INIT]), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kB
Oct 02 08:50:55 host-10-0-137-220 podman[18342]: 2019-10-02 08:50:55.662267073 -0400 EDT m=+0.361222308 container init 8477205ae163b71ea8543148362c21afc1fd79cf5c011402db9a5adea606d372 (image=registry.access.redhat.com/rhel7/rhel:latest, name=charming_robinson)

This issue is 1 year old today (happy birthday!), and I've long since disabled the memory cgroup test that triggered it, and I don't imagine anyone cares much about it. If it's not easy to fix runc to emit a more helpful message, it's probably ok to just live with this.

Happy birthday!
@edsantiago, I've seen the error in the journal as well but the error handling must have changed in the version you're using until now. Which podman are you running? In the current code, we're returning the error from runc directly which is starting container process caused "process_linux.go:446: container init caused \"read init-p: connection reset by peer\""; that's what I meant by "doesn't return an OOM error". So I think there's nothing we do in Podman to improve the error reporting. I'd guess that runc should detect the OOM and properly report it.

I'm still seeing it, podman-1.6.0-3.fc30.x86_64

I'll try rawhide in a bit.

Cannot reproduce on rawhide, even with 15 running containers. I assume this is a cgroups-v2 difference, but it could also be crun. Looks like I need to test on f30, which means I need to wait for a build.

@edsantiago Is this still an issue, or has it been fixed?  crun or runc?

Let's just close this; it's too much of a corner case to make sense spending time on.
FWIW with podman-1.6.2-2.fc30 I see:
# podman --cgroup-manager=cgroupfs run -d -i -m 5246976 registry.access.redhat.com/rhel7/rhel:latest /bin/bash
Error: container_linux.go:346: starting container process caused "process_linux.go:446: container init caused \"process_linux.go:412: setting cgroup config for procHooks process caused \\\"failed to write 5246976 to memory.limit_in_bytes: write /sys/fs/cgroup/memory/libpod_parent/libpod-1d1315ba431a19b3d071710cda0ba183c5e77600c325c4e97ace52bb301ccf3e/memory.limit_in_bytes: device or resource busy\\\"\"": OCI runtime error

With podman-1.6.2-2.fc31 I see no error, even with five containers running simultaneously.


undefined method flash=

I'm trying to use rodauth-rails in a rails api app. I've noticed a couple things about generating the rodauth install using the --json flag.
If I specify the generator to use --json it still enables :jwt in the rodauth_app file and includes a jwt_secret. I then get errors because it's expecting the jwt gem to be installed, which I'm not needing. The documentation makes it sound like I should only be getting all the jwt configurations if I specified --jwt with the generator.
Once I remove the jwt_secret and the enable :jwt, I stop getting jwt errors and start getting undefined method flash=' for #<ActionDispatch::Request:0x00007fabaea1fd90> when I try to post to /login. Since this is an api, I don't have a flash, and from reading through previous issues it sounds like this was fixed in version 0.4. I'm on 0.17.
This is on a fresh rails <IP_ADDRESS> api app. The only changes I have made are adding rack-cors, enabling cookies and session cookies in application.rb, and adding include ActionController::RequestForgeryProtection with protect_from_forgery with: :null_session in the application controller.
On a side note, how can I inspect a request being sent to a rodauth route? I can't find a place to put a byebug call. My client is sending an X-CSRF-Token header with a good value, but I'm still getting Can't verify CSRF token authenticity. so I'm needing to troubleshoot that further, but that's not your problem. :-)
Thank you!

It's strange. Everything related to Flash in the rodauth_app file is commented out. Is there another place I should be checking? It seems to be coming from the r.rodauth line in the routes block. That wasn't clear before, but it is now.
NoMethodError (undefined method `flash=' for #<ActionDispatch::Request:0x00007fd61a1059d0>):
app/lib/rodauth_app.rb:132:in `block in <class:RodauthApp>'

Line 132 is the r.rodauth inside the route block.
I did discover that if I remove protect_from_forgery from my application_controller.rb I no longer get the flash error, but instead I get:
NoMethodError (undefined method `new' for nil:NilClass):
app/lib/rodauth_app.rb:132:in `block in <class:RodauthApp>'

I created another fresh app and installed rodauth-rails from master. The --json update worked as expected this time. Thanks for fixing that!

It's strange. Everything related to Flash in the rodauth_app file is commented out. Is there another place I should be checking?

The issue is that rodauth-rails activated flash integration when it apparently shouldn't have. The error is most likely not coming from Rodauth but from rodauth-rails.

It seems to be coming from the r.rodauth line in the routes block. That wasn't clear before, but it is now.

The r.rodauth call routes Rodauth requests, so most Rodauth exceptions will originate from there. You likely have some backtrace filter that doesn't display the backtrace from gems, so that's why it appears as the first line.

I did discover that if I remove protect_from_forgery from my application_controller.rb I no longer get the flash error, but instead I get:

I'm missing a full backtrace for this.

I created another fresh app and installed rodauth-rails from master.

Would you mind sharing it? Being able to look at a Rails app that exhibits this error would guarantee me fixing it 😃

Of course, I'm sorry I didn't do this sooner. https://github.com/andyrue/rodauth-test Thank you for looking.


Doesn't work inside Electron

No idea why.

Possibly the same as #255?

@sdegutis: Does #274 fix this?

I don't know, sorry, I don't use shelljs or electron anymore.

So far I'm getting a Illegal Invocation everytime I use
var sh = require('shelljs');
sh.echo('Hello World');

inside the renderer index.js

@joselevelsup is that using the latest version of master (we merged #274 yesterday)? What does the following code do for you? This would isolate if it's more than just an issue with echo.
var sh = require('shelljs');
console.log(sh.pwd());

Turns out I was using it wrong. I had to use it in the Main process of Electron not the Render Process. Shelljs works fine!

Awesome to hear! I'll close this issue for now. Please reopen if there are any concerns.

Hi,
I tried to use it renderer and sh.pwd seems fine. But for some reason, when I package the app and then pwd still works but which command i not working.
Any idea why?
Thank you!


Time zones incorrectly allow the colon to be omitted

Bug Report
Steps to reproduce

Generate a sync function from the datetime-doc-definitions.js file: ./make-sync-function -j test/resources/datetime-doc-definitions.js build/output.txt
Insert the generated sync function into a Sync Gateway configuration file
Create a document with a date-time that has no colon separating hours and minutes in the time zone component. For example:

PUT /test/datetimeDoc HTTP/1.1
Host: localhost:4985
Content-Type: application/json

{
	"formatValidationProp": "2016-07T15:20:09.348-0700"
}

Actual behaviour
The document will be accepted even though the ECMAScript specification for Date Time String Format does not allow omission of the colon that should separate the hours and minutes of the time zone.
Expected behaviour
The document should be rejected with a status of 403 Forbidden and the reason "Invalid datetimeDoc document: item \"formatValidationProp\" must be an ECMAScript simplified ISO 8601 date string with optional time and time zone components".
Environment

Synctos: 2.2.1
Couchbase Server: 5.5 Developer Build
Sync Gateway: 2.0.0 Beta 2
Operating system: macOS High Sierra

Addressed by #296


Allow for empty/missing migrations

There shouldn't be significant performance implications of having some empty blocks and, right now, explicit empty blocks seem much clearer. Won't fix.


Unusual example in the inline-initialization.md

https://github.com/Kikobeats/js-mythbusters/blame/master/docs/v8-tips/inline-initialization.md#L44
var array = []
for (var i = 0; i < 10; i++) array[i] |= 0  // Oh no!

var array = []
array[0] = 0
for (var i = 0; i < 10; i++) array[0] |= i

The first one creates an array with 10 elements. The second one creates an array with a single element.
Maybe it's something like this:
var array = new Array(10)
for (var i = 0; i < 10; i++) array[i] = 0

Thanks!
The original resource was: https://www.html5rocks.com/en/tutorials/speed/v8/
The section tries to explain how v8 hidden class acts in an array initialization from a low level point of view.
I moved the high level part of the content into:
https://github.com/Kikobeats/js-mythbusters/blob/master/docs/array/preallocation.md
Also with a new example 🎉 .

Tanks, ´´´const n = 4
const array = Array(0).fill(0) // [ 0, 0, 0, 0 ]
´´´
´Array(n)´?


[5.9] Fix test that took seconds to type check

Cherry-pick of https://github.com/apple/swift-docc/pull/547

Explanation: This fixed an issue where one expression in a test was very slow to type check. In some cases this resulted in an error about the compiler being unable to type check this expression in a reasonable time.
Scope: Possible error about slow type checking during builds of test targets.
Radar: rdar://107960794
Risk: Low.
Testing: This change was verified manually. Debug builds opt in to warnings about slow type checking to avoid similar issues in the future.
Reviewer: @franklinsch

Note: the only reason for suggesting that we cherry pick this is to avoid potential compiler errors about type checking taking unreasonably long.

@swift-ci please test

@swift-ci please test


Unable to deserialize response from thriftrw-go

Please see https://github.com/thriftrw/thriftrw-go/issues/321
This appears to be an issue with thriftrw-node, though given the test cases I've been able to find, I think the issue could be with me using the library incorrectly. Even if that is the case, some docs to clarify this would be good.
See this archive for a reproducible test case: https://github.com/thriftrw/thriftrw-go/files/1218543/issue.tar.gz

@Willyham I'm not entirely sure if this is an issue with ThriftRW Node either. The corrupted payload I showed you was what Node saw before it was decoded by ThriftRW.
@kriskowal any ideas?

@Willyham Thanks for the really detailed repro!
I've seen something similar before where the body is converted to a string, which modifies the buffer, and then thriftrw isn't able to decode it.
I looked into the documentation for request, and found this little nugget:

encoding - encoding to be used on setEncoding of response data. If null, the body is returned as a Buffer. Anything else (including the default value of undefined) will be passed as the encoding parameter to toString() (meaning this is effectively utf8 by default). (Note: if you expect binary data, you should set encoding: null.)

I tried this out, and set encoding: null in options, and the issue no longer reproduces.
  const options = {
      url: 'http://localhost:8001',
      encoding: null,
      body: body.value,
      headers: {
        "Rpc-Caller": 'web',
        "Rpc-Encoding": "thrift",
        "Rpc-Service": "foo",
        "Context-TTL-MS": 5000,
        "Rpc-Procedure": "Foo::bar"
      }
  }

> node test.js
Result of small:
null <Buffer 00 00 00 00 00 00 00 7b>
Result of big:
null <Buffer 14 da db 63 ff b5 b7 52>


Move Median to functions-aggregate and Introduce Numeric signature

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks again @jayzhan211


Package for Debian

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=803413

The package has been accepted to Debian unstable (see bug link).
Big thanks to @cklin

woot!

https://packages.debian.org/sid/google-cloud-print-connector

Any plans to submit packages for other architectures?

Yes, I need to find out what's wrong with the others.

Further updates will belong to the Debian tracker


fix(focustrack): avoid failing when reading activeElement$ server-side

@quentinderoubaix @andreascorti Thank you for your review!


Usernames that should be blocked for non-technical reasons

It seems this list is focused on blocking technical terms that are potentially harmful. What's a good practice to block usernames that are otherwise illegal or inappropriate?
If the scope of this list wants to stay focused on technical terms, a pointer in the README could be beneficial. Happy to help contribute.

From the readme:

Please note that this list does not contain any curse words, there are other lists for that.

What lists could you point me (and hopefully others) to there?
Also, I'd be even more interested in best practices not just for profanity but actually legally problematic words (e.g. racist terms).

There is a link in the readme to https://github.com/LDNOOBW/List-of-Dirty-Naughty-Obscene-and-Otherwise-Bad-Words.

Also, I'd be even more interested in best practices not just for profanity but actually legally problematic words (e.g. racist terms).

That is very dependent on culture and geographic region and world wide not possible.

I'm closing this issue as I see the answer posted as resolved. Thanks @SuperSandro2000 for providing support!


rain loop configuration

RainLoop version, browser, OS:
Expected behavior and actual behavior:
Steps to reproduce the problem:
Logs or screenshots:

i configured rainloop
but when i just want to login for check email its say authntication faild or domain is not allowed.can any one tell me please where we have define that username or password that we can use for login and check our email inbox,sent box,.....i'm stuck here from past 4 days ....thnx in advance

When you define your login (actually you email address) you have to use the password of this mailbox.

@Amritmatti
Please check this guide: https://harrytang.xyz/blog/rainloop-webmail-directadmin


Update README.md

added Awesome TensorFlow list to the bottom of miscellaneous

+1 :)

i see there is a branch conflict.  @ujjwalkarn will this be added? if so , I can fix the conflict.

yes, please fix the conflict. Thanks :)


[Editor] Fix the resizing of an editor when it's rotated (bug 1847268)

There are 2 rotation we've to deal with: the viewer one and the editor one. The previous implementation was a bit complex and having to deal with these rotation would have potentially increase it.
So this patch aims to simplify the implementation and deal with all the possible cases.
The main idea is to transform the mouse deltas according to the rotations and then apply the resizing in the page coordinates system.

I just updated the integration test about the cursor name.

/botio integrationtest


Frontend login button

Fix the login button of the NavBar such that clicking on login redirects to the Tequila server (rather than the Login page)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
18 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 54.748%

Files with Coverage Reduction
New Missed Lines
%

services/dkg/pedersen/handler.go
18
86.92%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
3004

Relevant Lines:
5487

💛  - Coveralls


Correction for raw affiliation Elsa, Research Group for Environmental Lifecycle & Sustainability Assessment, Montpellier, France; ITAP, Univ Montpellier, INRAE, Institut Agro, Montpellier, France

Correction needed for raw affiliation Elsa, Research Group for Environmental Lifecycle & Sustainability Assessment, Montpellier, France; ITAP, Univ Montpellier, INRAE, Institut Agro, Montpellier, France
raw_affiliation_name: Elsa, Research Group for Environmental Lifecycle & Sustainability Assessment, Montpellier, France; ITAP, Univ Montpellier, INRAE, Institut Agro, Montpellier, France
new_rors: 03rnk6m14;051escj72;003vg9w96;03573dx38;00ftatp57
previous_rors: 03rnk6m14;051escj72;003vg9w96;03573dx38
works_examples: W4362637302;W4365459232
contact: acc3c7e1fb2125a2088669d51b9fdb8d:490f458b97049f50b610635150863da433 @ supagro.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Update ResourceOutputStream.php, bug: #14

https://github.com/amphp/byte-stream/issues/14

Not sure about that. Should we just is if ($this->chunkSize) { ... } else { ... }?

I am not sure, if PHP is doing it right, I get really strange errors.

@kelunik I think is if ($this->chunkSize) { ... } else { ... } is better

@kelunik Yes, I think so if we want to allow setting chunk size. Apparently passing null is coerced into 0 and causes weird behavior. I would also make the default size 65536 (needs updating in ResourceInputStream as well).

Close this ?

https://github.com/amphp/byte-stream/commit/ea135f26d04a110f15468a436be96df01f34252d fix this

@trowski Default size can be null, no?

Yeah, null is fine for the default.


Unable to find cloudformation manifest when stage is not dev

Setup of plugin in pom.xml:
             <plugin>
                <groupId>com.nimbusframework</groupId>
                <artifactId>nimbus-deployment-maven-plugin</artifactId>
                <version>0.13.5</version>
                <configuration>
                    <region>us-east-1</region>
                    <stage>development</stage>
                </configuration>
            </plugin>

When I trigger the deployment command: mvn nimbus-deployment:deploy (added -e -X for verbosity after)
I experience this:
[info] Starting
[info] Created File Service
[info] Got deployment information
Oct. 20, 2020 12:25:42 A.M. com.amazonaws.auth.profile.internal.BasicProfileConfigLoader loadProfiles
WARNING: Your profile name includes a 'profile ' prefix. This is considered part of the profile name in the Java SDK, so you will need to include this prefix in your profile name when you reference this profile from your Java code.
Oct. 20, 2020 12:25:42 A.M. com.amazonaws.auth.profile.internal.BasicProfileConfigLoader loadProfiles
WARNING: Your profile name includes a 'profile ' prefix. This is considered part of the profile name in the Java SDK, so you will need to include this prefix in your profile name when you reference this profile from your Java code.
Oct. 20, 2020 12:25:42 A.M. com.amazonaws.auth.profile.internal.BasicProfileConfigLoader loadProfiles
WARNING: Your profile name includes a 'profile ' prefix. This is considered part of the profile name in the Java SDK, so you will need to include this prefix in your profile name when you reference this profile from your Java code.
[info] Initialised
[info] Beginning deployment for project: controller, stage: development
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  1.245 s
[INFO] Finished at: 2020-10-20T00:25:42-04:00
[INFO] ------------------------------------------------------------------------
[ERROR] target/generated-sources/annotations/nimbus/cloudformation-stack-create-development.json (No such file or directory)
java.io.FileNotFoundException: target/generated-sources/annotations/nimbus/cloudformation-stack-create-development.json (No such file or directory)
    at java.io.FileInputStream.open0 (Native Method)
    at java.io.FileInputStream.open (FileInputStream.java:219)
    at java.io.FileInputStream.<init> (FileInputStream.java:157)
    at services.FileService.getFileText (FileService.kt:21)
    at services.CloudFormationService.createStack (CloudFormationService.kt:56)
    at mojo.DeploymentMojo.execute (DeploymentMojo.java:97)
    at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:210)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:957)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:289)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:193)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:566)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)
[ERROR]

However, there are the present files:

The default stage for nimbus is dev. If you want some resources to deploy in other stages you need to add the stage parameter to the annotation:
@HttpServerlessFunction(method = HttpMethod.POST, path = "update", stages = ["dev", "development"])

If you want to deploy all your resources to another stage or change the default stage, you can change the nimbus.yml file and add a defaultStages parameter, e.g.
#nimbus.yml
projectName: MyProject
defaultStages:
  - dev
  - development

Thanks a lot!
I've added the stage to nimbus.yml and it solved this issue.


Call quteLanguageServer.stop() when deactivating

Closes #576
Signed-off-by: David Thompson<EMAIL_ADDRESS>

Thanks @datho7561


Some mods are installed using an older MC version even though a newer one is available

When using multiple acceptable game versions, some mods are installed using an older MC version even though a newer one is available. Example:
minecraft = "1.19.3"
acceptable_versions = ["1.19.2", "1.19.1"]
where the 1.19.2 version of the mods Debugify, Not Enough Crashes and Dash Loader are installed, when the 1.19.3 versions should be the most appropriate version.
My guess is that this occurs because some mods either:

have a version number higher for the older game version such as Debugify 1.1 (1.19.3) compared to 2.8.0 (1.19.2), and Not Enough Crashes 4.2.0+1.19.3 compared to 4.2.1+1.19.2; or
the publish date for an older game version is more recent than for the current game version such as Dash Loader; or
the version formatting recently changed such as Iris Shaders 1.5.1+1.19.3 compared to 1.19.2-1.4.2.

List of known mods that I encountered where an older game version was unappropriated installed (for the above versions):

Debugify (1)
Not Enough Crashes (1)
CraftPresence (2)
Dash Loader (2)
Essential Client (2)
3D Skin Layers (3)
Iris Shaders (3)

*Tested only with Modrinth

Your guesses 1 and 3 are correct - packwiz doesn't use publish date, only the version number. A good improvement would be to prefer files using the primary Minecraft version, and treat the acceptable versions list as being in order of priority.

Also happens with MoreCulling

I will note that I've implemented this in the reverse order (most preferred acceptable version is last) since I think most users have this set in ascending order - though of course the primary version is still used first


Dont use IndexOf as that has Cuture issues

Theres currently some culture issues using IndexOf as mentioned in this issue https://github.com/dotnet/msbuild/issues/5502.
https://github.com/dotnet/msbuild/issues/5502 fixes it, but its only enabled in a late wave, that we are not depending on yet.
This PR uses contains instead, it seem that fixes the issue with Culture for now.

Ah Contains dont work in all MSBuild versions, ie the one is used in CI here.
@jeffkl do you have any good ideas? What is the IsTraversal property used for? AFAICT its set after ie. Directory.Build.props is imported, so users don't use it to conditionally do logic in those.

Its easy for tooling to differentiate between a solution file and a project because they have different extensions.  However, its much harder if two projects end in .proj.  The idea behind this property is so that tooling can treat a project different if its really just a traversal project.  I don't think a ton of tooling has a dependency on it but we certainly can't get rid of the property.  I know I used it in SlnGen.
It is set after Directory.Build.props but before the project and Directory.Build.targets.  Most tools just evaluate a project and then inspect the properties.
From the issue you linked, it looks more like not properly converting the 0 to a string: https://github.com/dotnet/msbuild/issues/9757
Are you seeing issues because of this call to IndexOf()?

From the issue you linked, it looks more like not properly converting the 0 to a string: https://github.com/dotnet/msbuild/issues/9757
Are you seeing issues because of this call to IndexOf()?

Yes in this case its because of the call to IndexOf - we could also make the Property not use that, as its traversal by nature when this props file is imported?

I'm a little nervous about changing behavior.  Would comparing a string work better?
- <IsTraversal Condition=" '$(IsTraversal)' == '' And $(TraversalProjectNames.IndexOf($(MSBuildProjectFile), System.StringComparison.OrdinalIgnoreCase)) >= 0 ">true</IsTraversal>
+ <IsTraversal Condition=" '$(IsTraversal)' == '' And '$(TraversalProjectNames.IndexOf($(MSBuildProjectFile), System.StringComparison.OrdinalIgnoreCase))' != '0' ">true</IsTraversal>

@jeffkl - good idea, that seem to work as well, so im changing the code to do that instead


Using browser autofill in admin text input field changes theme from dark to light

There's a minor style bug that changes the theme on input fields when using the browser auto-complete feature:

Start position - focus on text input with autocomplete value

Hover over suggested value - input changes to light theme (input text font changes to system default too, but I think this is a Chrome issue)

Select value. Theme remains on light, including after moving focus away from input

If I edit the input box after using the selected value, the theme returns to dark:

Looking at the color style before and after, I see the following:
Before:

After:

Looks like it's the input:-internal-autofill-selected that changes the colours due to the !important flag.
input:-internal-autofill-selected {
    ~~appearance: menulist-button;~~
    background-image: none !important;
    background-color: -internal-light-dark(rgb(232, 240, 254), rgba(70, 90, 126, 0.4)) !important;
    color: fieldtext !important;
}

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project
Technical details

Python version: 3.12
Django version: 4.2.7
Wagtail version: 5.2.1
Browser version: Chrome 119

I can confirm the issue exists.


Not working with keyboardShouldPersistTaps.

Thank you for this plugin. It's really nice.
I caught a bug while doing some tests. It seems that it doesn't respond to a ScrollView with keyboardShouldPersistTaps='never' while a TextInput does.
Any thoughts?
Thanks

Hey there @luco, could you create a simple repo that would replicate this behavior or paste your example here?

I have created one in the snack. https://snack.expo.io/rJ7NGqQmG please check. It seems so weird.
Its actually dismissing the keyboard but takes a second to dismiss the keyboard.

I remember somehow I had a similar issue. I'll take a look at my code and get back to you this week.

Any update @umhan35 ?

Any update about this? I have the same issue

+1

+1


provide better names for sample configs and document what's different in each of the configs.

one of the recent PRs closes this issue.


Add doc diff to CI

Adds doc build and diff to CI for Storage.

Note to self: check out scripts/setup_spm_tests.sh to fix the version mismatch error

Please close, merge, or comment. We plan to close stale PRs on November 28, 2023.

I'll revisit this another time. We haven't moved to docc yet because it's unable to generate ObjC docs from Swift.


pinocchio: 2.0.0-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository pinocchio to 2.0.0-0:

upstream repository: https://github.com/ipab-slmc/pinocchio_catkin.git
release repository: https://github.com/ipab-slmc/pinocchio_catkin-release.git
distro file: melodic/distribution.yaml
bloom version: 0.7.2
previous version for package: null

Holding for sync https://discourse.ros.org/t/preparing-for-melodic-sync-2019-02-06/7766


Node Request Middlewares

When developing the node / network interface for wasm modules, create a way for middlewares to be inserted as well. This will allow projects like Pocket Network to develop said middlewares and send requests to supported networks (Ethereum, Cosmos, Polkadot, etc) to a Pocket network node.

This has been implemented as URI Redirects

This has been implemented as URI Redirects


contrast RealBasicVSR and BasicVSR_PlusPlus

I would like to know which is better between RealBasicVSR and BasicVSR_PlusPlus performance, accuracy, speed and time, and what is the difference between the two?

Did you ever find out?


Machine Driver UI Issues

Rancher Version:
v1.1.0-dev4

[ ] No place to add name for new machine drivers. I found a valid URL and it autmoatically picked up a name, is this normal for machine-drivers?

[x] After adding a machine driver, it's inactive, but doesn't have the "Inactive" bar on it. This seems to only occur if there is an error in the Download URL. Can we somehow show an error?

[x] In Infrastructure -> Hosts, cllicking on "Manage available machine drivers" directs back to the Admin->Settings pages instead of the new Admin -> Machine Drivers page

Name is not a settable field anymore, we get the name from the downloaded driver.

Version - master 6/16
Verified fixed


Flexible plug version

Problem
Current version specification for Plug is very restrictive and does not allow using latest 1.6.x versions.
Solution
Remove the bug fix-level digit from the version specification, allowing any version >= 1.5 and < 2.
Also, upgrade all dependencies to ensure that tests still pass after the upgrade.

Coverage remained the same at 92.308% when pulling a12b36f29bfb3ab20d8921b64e0210940c24eb2c on amuino:flexible-plug-version into 47d0efdbd214756819473dee24876e23b033c009 on tommy351:master.


Dynamic config file

Maybe its useful to install a config handler, to centralize all app configurations.

Added https://github.com/mustangostang/spyc/ as Config-Loader, can also build config files, feature will come soon


Guideline editor

Noah you are FUCKED

OH LAWD PLEASE HELP

What audio jungle stuff? Should this also come with automatic BPM detection and shit?

too much work, wont be worth it for now

For another day™


002 - OSGI_Security_Framework update: new package names

Signed-off-by: Romain Siche<EMAIL_ADDRESS>

Signed-off-by: Romain Siche<EMAIL_ADDRESS>

There is no point in putting the Signed-off-by in Pull Requests as they are not part of the repo's history.
Also the proper format, as described in the CONTRIBUTING document, would be
Signed-off-by: Romain Siche<EMAIL_ADDRESS>git commit -s will add this properly for you.


WindowState.manage(win) makes BrowserWindow.loadURL() reject promise in electron 5.0.0+ (using 5.0.1))

the URL will be successfully loaded, but the returned promise (feature from electron 5.0.0) will be rejected.

I cant reproduce this. Do you have a sample where this happens?
My minimal test:
const { app, BrowserWindow } = require('electron');
const windowStateKeeper = require('electron-window-state');
let win;

function createWindow() {
  let mainWindowState = windowStateKeeper({
    defaultWidth: 1000,
    defaultHeight: 800,
  });

  // Create the browser window.
  win = new BrowserWindow({
    x: mainWindowState.x,
    y: mainWindowState.y,
    width: mainWindowState.width,
    height: mainWindowState.height,
  });

  win
    .loadURL('https://google.com')
    .then(() => console.log('resolve'))
    .catch(e => console.error(e));

  mainWindowState.manage(win);
}

app.on('ready', createWindow);

let me try to reproduce it as well in a minimal setting, my code base isnt big yet, so maybe you can check it in there if you want with the public repository (let me first clean up the code quickly)

for now this is the code, i didn't have time for the minimal version yet:
https://github.com/mithom/streamerbuddy/blob/feature/auto-update/main.js

i should have tested this before making this issue, but with google it indeed works fine.
I will investigate my issue more, but for now i wouldn't  say it is electron-window-state causing causing it anymore. My appologies for looking in the wrong place (initially the error disappeared when i commented the .manage(win) out, but now even that doesn't work)


Fix typo

Fixed a typo in your docs

[ ] Tests added / passed
[x] Passes black dask / flake8 dask

Thanks!


[YUNIKORN-970] Add queue metrics with queue names as labels

What is this PR for?

Update the queue metrics documentation to describe the new queue metrics with labels to be introduced in 1.5.0
Add deprecation (1.6.0) and removal (1.7.0) notices for the subsystem metrics that are being replaced

What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[x] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-970
How should this be tested?
Local build with ./local-build.sh run
Screenshots (if appropriate)

Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

LGTM, thanks Will, I will merge it.


HMR support to multi entries

What kind of change does this PR introduce?

ref #7829

Did you add tests for your changes?

not yet

Does this PR introduce a breaking change?

no

What needs to be documented once your changes are merged?

no

Should work with webpack@latest. Feel free to report new issue with reproducible repo.


keepass: improvements

*.desktop file now:

Refers to an icon.
Is placed in the proper category (based on comparison with keepassx2.
Has proper mime type (also based on comparison).

Also, now use icoutils to extract icons from the application ressouces
and transform them from *.ico to *.png. Created and used a generic
script that has the ability to move the extracted *.png to their
appropriate standard freedesktop location.
Tested this on nixos. keepass now has a icon and is categorized
in the same bin as keepassx2. The program still execute and
function prefectly.

CC @amorsillo @obadz

No objections here, thanks.

You should also add yourself as maintainer :)

@obadz : Thanks for the quick response.
Not a bad idea to add myself as a maintainer as this script is by no mean lean. Would be a
first time. Any pre-requisite to be met for that purpose?
Also I though this script might eventually be adapted so that it can be used by other mono
package (that most likely won't be provide pre-packaged free desktop icons). The script
can also extract free desktop icons from an executable or a dll which is what I was doing
before the package changed to building from source.
Maybe it could be trigged by an optional field to be used with the new architecture for mono
application (specific mono derivation). The script could use some improvements so that it
require less information.

@jraygauthier I'm afraid I don't have the expertise (nor do I have commit rights on this repo) to help with your changes.
AFAIK there is no pre-req for adding yourself as maintainer.  You just need to add an identifier to lib/maintainers.nix

@obadz : I saw an earlier commit named: "Lay down the foundation for packaging the .NET echosystem" for which you seem the author for which I had a failed merge to solve (which is
done in this version). When I first saw this commit, I thought that it may have automated this
icon extraction problem which I noticed after testing it didn't.
Now that we compile from sources
instead of deploying binaries, there should always be an application icon in the source package
which we could exploit for conversion to free desktop png. The minimal requirements assuming
the *.ico contain only the application's icons (which it should) would be that the user provide
its location and the name of the target icons (without extension).
Adding this feature would provide us with an almost automated freedesktop icon compliance
for mono packages which I my view would be great for desktops users. What do you
think?
I'm no mono developer so I'm unsure whether mono packages always resort to
*.ico ressource for their icons. Anyone could confirm/infirm this?

@obadz : As suggested added myself as a maintainer.

I'd be happy to merge this if you rebase it.

@joachifm: Done and tested. Everything still work as it did.

Thanks


Version 3

There's still some work missing on this - we should not merge this atm. I'll try to finish it over this weekend.


amazing shell tab completion

this worked in the template I used at https://github.com/note/teleport-scala

inspiration
https://github.com/scalacenter/bloop/tree/master/frontend/src/main/scala/bloop/cli/completion

More inspiration:
https://github.com/remkop/picocli
and
https://picocli.info/autocomplete.html


Add files via upload

add Zarqa Private University to the universities list, thank you

@SajidSadaqa I am sorry, the domain zu.edu.jo is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


corretto 20 based on AL2023

Hi
Since AL2023 is already in GA would be nice to have corretto:20 version based on 2023 aside of AL2.
Is there a chance for such image?
Thanks!

Thanks for bringing this up, we'll see when we can get that added and we will update this issue soon.

Hi @klubi , we finished creating Docker image for 20 on al2023. It's now available on Dockerhub and ECR public. You can pull the image via:
# from ECR public
docker pull public.ecr.aws/amazoncorretto/amazoncorretto:20-al2023-generic

or
# from Dockerhub
docker pull amazoncorretto:20-al2023-generic

Callout: 20 doesn't have al2023 binary(see corretto 20 download page) so we built the docker image via installing generic linux corretto 20 on al2023. You might need to install using dnf or yum if any dependencies are missing.
I'm closing this issue since the image is available. Feel free to reopen if you have any issues.


Widen 3D images for array-image copy

According to the OpenCL spec for clEnqueueCopyImageToBuffer (and similar for the reverse), the driver is allowed to fail requests that would result in using resource dimensions or pitches that are not supported:

CL_INVALID_IMAGE_SIZE if image dimensions (image width, height, specified or compute row and/or slice pitch) for src_image are not supported by device associated with queue.

For OpenCLOn12, we would like to require 256-byte row pitches and 512-byte slice pitches, because are the limits enforced by the ID3D12GraphicsCommandList::CopyTextureRegion API. When the CTS gets down to one-byte-per-pixel image formats with a 128 texel wide image, that results in row pitches of 128 bytes. The simplest way to allow OpenCLOn12 to pass would be to just use wider textures.

FWIW, I think I wouldn't accept a patch doing this as there is nothing that constrains the minimal size of images in the specification (and the conformance process requires that some test be run with small_images as per https://github.com/KhronosGroup/OpenCL-CTS/blob/master/test_conformance/opencl_conformance_tests_12_full.csv#L49). The error you quoted is only there to catch the case where an image has been created with valid dimentions for at least one device in the context but not the device the copy command is being submitted to. Granted, it's not very clear. I'll raise a spec issue to look into clarifying this.
Two ways forward that come to mind:

Revert to a less efficient copy method when images don't meet D3D constraints.
Propose spec changes for a future version of OpenCL (that wouldn't be backwards compatible though)

Thanks @kpet, I've already done your 1st proposed suggestion. I'll go ahead and close this.


Unable to detect the image corners

I have integrated the library in my project, it's opening the camera but i need to give permission for camera and local storage manually to app. and it's also not detecting the document's corners  and giving the exact image clicked as result.

i have the same issue , can u give me your app.js file


Document return types of authTest.controller.ts error responses

[x] ApiForbiddenResponse
[x] ApiUnauthorizedResponse

:tada: This issue has been resolved in version 5.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


improve the building system

Description

Move testenv from test/material to deploy.
Decide whether to change /etc/hosts on every testenv_init or just document it.
Fix caching layer in Dockerfile

Related Issues

https://github.com/SuperDuperDB/superduperdb/issues/1324
Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

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

Comparison is base (34830a7) 80.33% compared to head (0cc6525) 67.09%.
Report is 71 commits behind head on main.

Files
Patch %
Lines

superduperdb/backends/ibis/query.py
21.73%
72 Missing :warning:

superduperdb/components/stack.py
41.53%
38 Missing :warning:

superduperdb/server/app.py
56.75%
32 Missing :warning:

superduperdb/vector_search/interface.py
48.64%
19 Missing :warning:

superduperdb/ext/openai/model.py
37.03%
17 Missing :warning:

superduperdb/misc/server.py
26.08%
17 Missing :warning:

superduperdb/vector_search/server/service.py
48.48%
17 Missing :warning:

superduperdb/ext/utils.py
27.77%
13 Missing :warning:

superduperdb/cdc/cdc.py
25.00%
12 Missing :warning:

superduperdb/vector_search/server/app.py
65.71%
12 Missing :warning:

... and 19 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1326       +/-   ##
===========================================
- Coverage   80.33%   67.09%   -13.25%
===========================================
  Files          95      103        +8
  Lines        6602     7072      +470
===========================================
- Hits         5304     4745      -559
- Misses       1298     2327     +1029

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