Allow trailing comma in enum value lists, per spec

See https://github.com/heycam/webidl/commit/291dcc397f48029ac3954b2b0e511f4c14c7631d

Yep, https://heycam.github.io/webidl/#idl-enums allows trailing commas when EnumValueListString expands to ε.
[19]	Enum	→	"enum" identifier "{" EnumValueList "}" ";"
[20]	EnumValueList	→	string EnumValueListComma
[21]	EnumValueListComma	→	"," EnumValueListString
                            | ε
[22]	EnumValueListString	→	string EnumValueListComma
                            | ε


The system cannot find the path specified. - Windows 10

I am sometimes getting this error message in windows 10 powershell:
The system cannot find the path specified.
Here is what I've ran into:
PS C:\Users\Username> scoop list
Installed apps:

  7zip 16.04
  git 2.15.0.windows.1
  nodejs 9.0.0
  openssh 5.4p1-1

PS C:\Users\Username> scoop which 7zip
'7zip' not found
PS C:\Users\Username> scoop which git
~\scoop\apps\git\current\cmd\git.exe
PS C:\Users\Username> scoop which nodejs
'nodejs' not found
PS C:\Users\Username> scoop which openssh
'openssh' not found

PS C:\Users\Username> scoop checkup
No problems identified!

PS C:\Users\Username> scoop update
Updating Scoop...
The system cannot find the path specified.
Update failed.
PS C:\Users\Username\scoop\apps\scoop\current>

PS C:\Users\Username> scoop which scoop
~\scoop\apps\scoop\current\bin\scoop.ps1

It seems like scoop can't find its packages, and it fails to update scoop itself.  Yet, the packages work fine when I run my scripts, for example: installing and running python and pip packages, and then running python scripts, however I have uninstalled python to reduce variables.
Here are my paths:
PS C:\Users\Username> ($env:Path).Replace(';',"`n")
C:\WINDOWS
C:\WINDOWS\system32
C:\WINDOWS\System32\Wbem
C:\WINDOWS\System32\WindowsPowerShell\v1.0\
C:\Program Files\Common Files\Microsoft Shared\Windows Live
C:\Program Files (x86)\Common Files\Microsoft Shared\Windows Live
c:\Program Files (x86)\Intel\iCLS Client\
c:\Program Files\Intel\iCLS Client\
C:\ProgramData\Oracle\Java\javapath
C:\Program Files (x86)\Windows Live\Shared
C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\
C:\Users\Username\scoop\apps\nodejs\current\bin
C:\Users\Username\scoop\apps\nodejs\current
C:\Users\Username\scoop\shims
C:\Users\Username\AppData\Local\atom\bin
C:\Users\Username\AppData\Local\Microsoft\WindowsApps
C:\Users\Username\AppData\Local\Microsoft\WindowsApps

I've tried reinstalling scoop, and moving paths up and down, but the problem still remains.  I am using the default bucket by scoop.

scoop which does work with the package name, instead it shows the path to the shim/executable. It's based on the which command from Linux.
λ scoop which 7z
D:\scoop\apps\7zip\current\7z.exe
λ scoop which node
Not a scoop shim.
D:\scoop\apps\nodejs\current\node.exe
λ scoop which npm
Not a scoop shim.
D:\scoop\apps\nodejs\current\bin\npm.cmd
λ scoop which git
D:\scoop\apps\git-with-openssh\current\cmd\git.exe
λ scoop which ssh
D:\scoop\apps\git-with-openssh\current\usr\bin\ssh.exe

But I can't figure out where the The system cannot find the path specified. can come from while updating...

gotcha, my results are now the same.  Why does it fail to update scoop?  Do you think it's related to The system cannot find the path specified.?

Is there .git directory inside C:\Users\Username\scoop\apps\scoop\current?

yes, there is

Can you try running these commands one by one from a new powershell window? This replicates the update process from line 73 to 79 in scoop-update.ps1.
> cd ~/scoop/apps/scoop/current
> . .\lib\core.ps1
> . .\lib\git.ps1
> $currentdir = fullpath $(versiondir 'scoop' 'current')
> write-host $currentdir
> pushd $currentdir
> pwd
> git_pull -q
> write-host $lastexitcode
> popd
> pwd

Interesting, I think you're onto something with the git:
PS C:\Users\Username\scoop\apps\scoop\current> cd ~/scoop/apps/scoop/current
PS C:\Users\Username\scoop\apps\scoop\current> . .\lib\core.ps1
PS C:\Users\Username\scoop\apps\scoop\current> . .\lib\git.ps1
PS C:\Users\Username\scoop\apps\scoop\current> $currentdir = fullpath $(versiondir 'scoop' 'current')
PS C:\Users\Username\scoop\apps\scoop\current> write-host $currentdir
C:\Users\Username\scoop\apps\scoop\current
PS C:\Users\Username\scoop\apps\scoop\current> pushd $currentdir
PS C:\Users\Username\scoop\apps\scoop\current> pwd

Path
----
C:\Users\Username\scoop\apps\scoop\current

PS C:\Users\Username\scoop\apps\scoop\current> git_pull -q
The system cannot find the path specified.
PS C:\Users\Username\scoop\apps\scoop\current> write-host $lastexitcode
1
PS C:\Users\Username\scoop\apps\scoop\current> popd
PS C:\Users\Username\scoop\apps\scoop\current> pwd

Path
----
C:\Users\Username\scoop\apps\scoop\current

So it's it the git_pull method.
What's the result of cmd /c git pull when run in C:\Users\Username\scoop\apps\scoop\current?

PS C:\Users\Username\scoop\apps\scoop\current> cmd /c git pull
The system cannot find the path specified.
Already up to date.
PS C:\Users\Username\scoop\apps\scoop\current> git pull
Already up to date.

Are you using Cmder? Output of echo %CMDER_ROOT%?

Oh wait a minute, I had installed it before to try it out, but I thought I uninstalled it, it is portable as I recalled.
PS C:\Users\samsclub06613\scoop\apps\scoop\current> echo %CMDER_ROOT%
%CMDER_ROOT%

Can you open a powershell and run cmd from it, does it show The system cannot find the path specified.?

I have been using powershell all along.  Right now I'm thinking that maybe hidden files from cmder are interferring.
PS C:\Users\Username> cmd
Microsoft Windows [Version 10.0.16299.19]
(c) 2017 Microsoft Corporation. All rights reserved.
The system cannot find the path specified.

Ha, there it is. Okay, something is up with your cmd profile then. But I really don't know how to solve this now.
I can only remove the cmd part from the git_pull() function and use powershell instead. (This will not fix the The system cannot find the path specified.-problem, but we don't get an exit code 1 anymore 😁)

I had to clear autorun from the regedit, apparently cmder's modifications still remained.
https://superuser.com/questions/727316/error-in-command-line-the-system-cannot-find-the-path-specified
Now I can update:
PS C:\Users\Username> cmd
Microsoft Windows [Version 10.0.16299.19]
(c) 2017 Microsoft Corporation. All rights reserved.

PS C:\Users\Username> scoop update
Updating Scoop...
Scoop was updated successfully!

What do you think, is the problem fixed?

I had the same problem (The system cannot find the path specified) but in my case clink was guilty. The issue appears if you use clink autorun and you removes it without doing the following:
clink autorun uninstall
I installed clink using scoop.

This appears to be resolved. If not, let us know. Thanks.

I've got the same error message i.e. The system cannot find the path specified. when trying to run an Allure app from PowerShell. The solution for me was to install Scoop and its apps to another path which has no spaces in it.


Space invaders blockly - first draft

First issue of blocks for space invaders course consisting of the following:

block to create an img element using a specific drop-down set of images
block to set the position [top/left] of an element (using the style) to a certain number of pixels
block to change the position [top/left] of an element by a certain number of pixels (may want to modify to look like the change-variable block)

As a side-effect of this, the getAllSpaceships block is using code-gen for a more generic querySelectorAll so I've added this as a block just like the existing querySelector block -- this returns a list.
I've asked for review from @taybenlor (to get an opinion about the change position block) and @jimmo, to spread the review load wherever someone has time. I have more blocks to do that relate to calling custom functions that will release on this same branch.
See example:

With code-gen looking like:

I think this is now looking pretty solid. Karsten also approved the requestAnimationFrame technique, so we can include the new blocks as follows:

'on next animation frame'
'starting animating with'

I think this has now addressed remaining review issues. Are we all good with this? My one remaining question is whether Karsten will realise where the function is for defining animate() -- this is located in the "Events" tray and looks like an event handler.
@jimmo do you think it might be sufficient to get Karsten to supply this is a read-only block on the workspace, predefined? I can export the playground question I'm using to test the blocks here.


Throw separate type of exception when user must restart com.docker.service

Are there particular scenarios where this is needed? There are too many possible points of failure, failed pipe between docker and docker service, Moby VM gets into a bad state, Service attempting to reconnect to Moby, etc. There is no status that can be pulled from the service or the VM.
I have added, user permission related exception, and a field in ContainerServiceNotRunningException with the service name.

https://github.com/Microsoft/RTVS/pull/4145


[503] "error":"SearchPhaseExecutionException[Failed to execute phase [query_fetch], all shards failed]

I am trying to write tests for ES setup we have:
setup do
    Rubygem.__elasticsearch__.create_index! force: true
 end

context 'filter' do
    def setup
      example_gem = create(:rubygem, name: "example_gem")
      example_gem_other = create(:rubygem, name: "example-gem-other")
      create(:version, rubygem: example_gem, indexed: false)
      create(:version, rubygem: example_gem_other)
      Rubygem.__elasticsearch__.refresh_index!
    end

    should "filter yanked gems from the result" do
      response = Rubygem.elastic_search "example_gem"
      assert 1, response.results.size # <= line 20
    end
  end

However, I keep getting:

Elasticsearch::Transport::Transport::Errors::ServiceUnavailable: [503] {"error":"SearchPhaseExecutionException[Failed to execute phase [query_fetch], all shards failed]","status":503}
test/unit/rubygem_searchable_test.rb:20:in `block (2 levels) in class:RubygemSearchableTest'

On two of the issue @karmi asked if they were getting the error (SearchPhaseExecutionException) when they were running test, unfortunately they were not so he didn't mention what needs to be done.
I have strong feeling that I am getting exception because Rubygem.__elasticsearch__.refresh_index! was not finished before response.results.size was executed. If I add sleep 1 after refresh_index, I don't get the error, ie let the reindexing finish?

hi, is following the solution:
Rubygem.__elasticsearch__.client.cluster.health wait_for_status: 'yellow'

I just found it here: https://github.com/elastic/elasticsearch-rails/blob/a714d5931182501531d2ce6b767bcb3c97ce56a7/elasticsearch-model/test/integration/active_record_import_test.rb#L33


Issue #216, #239 & #240 - add order by support of multiple fields, ASC/DESC and NULLS FIRST/LAST; move orderby to context; fix to ensure recalc when order by field changes

This PR is an alternative to PR #238 (#238 or this PR should be merged, not both) and incorporates Issue #239 moving the order by to the context, eliminating any default orderby applied when none is specified (except for RelationshipField__c) and updating the helptext of FieldToOrderBy__c .  It also includes #216 and #240.
Since this was all fresh in my mind, I decided to implement #239 in case it's decided to use that approach instead of keeping the order by on the RollupSummaryField.  Based on working through #216 and the other issues identified that were addressed around orderby, I believe the approach in this PR should be chosen as it offers the benefits mentioned in #239.  The downside to this PR as opposed to #238 is that it does change signatures of RollupSummaryField and Context constructors. However, since the orderby did not make it to LRE project, the surface area of the impact is minimized to DLRS internally and anyone using DLRS source only.  The change to use the context will result in different rolled up results when no order by is specified compared to the ThenBy approach when RollupSummaryField contains the orderby.  However, in both cases it is a non-deterministic result so the net effect is essentially the same.  Moving to context seems to be a more natural way to manage the order by based on DLRS usage and avoids a lot of unnecessary processing.
Note

If the decision is made to NOT implement #239, then PR #238 should be merged and PR #241 cancelled.
If the decision is made to implement #239, then PR #241 should be merged and PR #238 cancelled.

Please see comments in https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/239#issuecomment-136645150 prior to merging this PR.
Only 1 of the following PRs should be merged: #238, #241, #243, #244


Please update releases page

I was trying to get revolute joints to work by following #139 and could not get it to work because I only just now realised I was using the "stable" 0.10.0 version. This is seen to new people under "Install" -> "get a stable release". Please update this to show the latest stable version.
The demos also use a version above 0.10.0. I only realised I was using an outdated version because I was trying to duplicate the double pendulum example and realised that Render.startViewTransform was not defined, despite it being in the docs.
Thank you!

Apologies for the confusion, I have revisited this and have switched to using only tags rather than creating entries for the github releases page, as you can now see on the releases section which now matches that on npm.

Thank you!


Create the Wiki for the Project Milestone 2

This been documented at the wiki https://github.com/airavata-courses/airavata-nextcoud/wiki/Project-Milestone-2


PowerToysRun's system command not found in result

Microsoft PowerToys version
0.41.3
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce

enable "PowerToys Run"
enable "Windows System Command" plugin
use alt+space open search bar
search shutdown

✔ Expected Behavior
Shutdown command show in result.
❌ Actual Behavior
is not.
Other Software
Windows 11 (insider preview)
No response

@ZeddG93
Those commands were localized. Try to use a name according to your locale.
I'm closing it. Feel free to reopen if it doesn't solve your issue.


Add release notes for incremental compilation improvements

What the title says
Fixes https://github.com/gradle/gradle-private/issues/3375

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


UIIcon problem

JVx.mobile
demo
user: features
open Contacts (DB)

not reproducable as described.


Fix error handling in batch operations

Description
This PR fixes two issues:

The errors (if present) is a field under response.data instead of under response
We should always remove from the write buffer. We currently do not remove operations that fail in Hasura, so they are always kept around in the front of the buffer.

Type of change

[X] Bug fix
[ ] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

Looks good. Thanks for debugging this and making the change @ypc-faros !

Thank you @ted-faros !!!


API v2 deprecated

Hello - thanks for making this software. I was trying to play with it today and I got the following error:
{12:48}~ ➭ ytsearch test
Error: Non-whitespace before first tag.
Line: 0
Column: 1
Char: N
    at error (/usr/local/lib/node_modules/ytsearch/node_modules/sax/lib/sax.js:642:8)
    at strictFail (/usr/local/lib/node_modules/ytsearch/node_modules/sax/lib/sax.js:662:22)
    at Object.write (/usr/local/lib/node_modules/ytsearch/node_modules/sax/lib/sax.js:934:11)
    at Parser.exports.Parser.Parser.parseString (/usr/local/lib/node_modules/ytsearch/node_modules/xml2js/lib/xml2js.js:273:29)
    at Parser.parseString (/usr/local/lib/node_modules/ytsearch/node_modules/xml2js/lib/xml2js.js:6:61)
    at exports.parseString (/usr/local/lib/node_modules/ytsearch/node_modules/xml2js/lib/xml2js.js:296:19)
    at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/ytsearch/node_modules/youtube-search/index.js:45:7)
    at emitNone (events.js:72:20)
    at IncomingMessage.emit (events.js:166:7)
    at endReadableNT (_stream_readable.js:905:12)

So I dug a little deeper and found that youtube-search/index.js is querying gdata.youtube.com/feeds/api/videos?q=test&start-index=1&max-results=1 and getting "No longer available" as a result. I guess this is because APIv2 is deprecated.
Am I correct? If so, are there plans to move to v3 or any other current work arounds?

No, I'm not interested in supporting YouTube or their crappy authenticated APIs. I will probably make a module for vid.me soon instead. If anybody else wants this package name for a replacement I'd be happy to give it to them.


Face Unlock not working after explicit locking the screen in Linux Mint

Please describe the issue in as much detail as possible, including any errors and traces
Howdy works well for terminal application on my Dell Inspiron 7560 laptop with only camera.
But there's issue with gui version of linux mint lock screen. At first login, howdy detects the face and give the option of login after verifying the face, but text similar to "Face Recognised" message is displayed with ugly placement. I would suggest after verifying face, it must automatically login to desktop.
Second after first login, If I lock the screen explicitly, then howdy doesn't recognise the face, even my camera light blinks. This was tested numerously by me in all identical situations, where howdy used to work.
Linux distribution (if applicable): Linux Mint 19 Cinnamon Edition
Howdy version: Latest version

How did you solve this?

Hello, at first I was not able to get howdy to start on the lock screen however, after making these changes it starts now and the camera comes on but it fails every time to unlock from the lock screen.  All other application of howdy works perfectly with exception of the lock screen.  does anyone have any ideas?

Hello, at first I was not able to get howdy to start on the lock screen however, after making these changes it starts now and the camera comes on but it fails every time to unlock from the lock screen.  All other application of howdy works perfectly with exception of the lock screen.  does anyone have any ideas?

same thing here, ubuntu xfce4 desktop, light-locker
Mar 23 09:33:52 dell /lib/security/howdy/pam.py[10210]: Traceback (most recent call last):
Mar 23 09:33:52 dell /lib/security/howdy/pam.py[10210]:   File "/lib/security/howdy/pam.py", line 93, in pam_sm_authenticate
Mar 23 09:33:52 dell /lib/security/howdy/pam.py[10210]:     return doAuth(pamh)
Mar 23 09:33:52 dell /lib/security/howdy/pam.py[10210]:   File "/lib/security/howdy/pam.py", line 39, in doAuth
Mar 23 09:33:52 dell /lib/security/howdy/pam.py[10210]:     pamh.conversation(pamh.Message(pamh.PAM_TEXT_INFO, "Attempting face detection"))
Mar 23 09:33:52 dell /lib/security/howdy/pam.py[10210]: PamException: Conversation error

Hi @boltgolt in 2023 :)
is there any progress regarding unlocking Mint (21.1) from the screensaver?
It works in Ubuntu 22, so maybe there is a way to make it run in Mint too?

@jkatnik @SeidlEnt This comment worked for me on Mint 21.2 :)


Add EXPIRED status to TransactionStatus enum

The TransactionStatus enum should be updated to match the latest Access API specification: https://github.com/onflow/flow/blob/master/protobuf/flow/entities/transaction.proto#L5-L12

Implemented in https://github.com/onflow/flow-go-sdk/pull/57


Validate data type as part of validating blueprint parameters.

@geirst, please review
@havardpe, FYI

@geirst, PTLA


Sticky header is disappearing when scrolling

When scroll and the header reaches the top, it disappears
adapter.layoutManager = StaggeredGridLayoutManager(...)
adapter.setDisplayHeadersAtStartUp(true)
adapter.setStickyHeaders(true)

@jrcacd, please more information.

After this error has fixed the above started.

I might experience the same bug, can't tell exactly when it started.
Scenario is: sticky headers and under the current header there are more items than fit on the screen.
When scrolling down, the next header does not show up in place but replaces the current header at the sticky position while leaving an empty space at its correct position. Once the new header is scrolled to the top, everything looks ok. Scrolling up works fine.
A temporary workaround for me is to set a 0-sized recycler pool for the header view. (Probably has performance impact, but not relevant for my scenario.)

The same bug

If you use 5.0.1, do you still have the issue?

With 5.0.1 the issue is not reproducible for me.
I never call onViewRecycled() manually.

@mattlaabs, I, have to recycle manually. So you confirm that is related to the change I made in 5.0.2.
I have put back the old code in the StickyHeaderHelper where I set the view not recyclable, the issue doesn't appear anymore, this means the view must not be recycled, and I will recycle manually when not visible anymore.


Minor UX improvements (fix #187)

For now, the only user-facing change is that it adds new case for notifications, if that's it, you may want to change the title.

About componentize too much, look at:
    getLibraryComponent() {
        if(this.props.library === null) {
            return <FullViewMessage>
Loading library...
</FullViewMessage>;
        }
        if (this.props.library.length === 0) {
            return <EmptyLibrary />;
        }
        if (this.props.tracks.length === 0) {
            return <FullViewMessage>
Your search returned no results
</FullViewMessage>;
        }
        return (
            <TracksList
                type='library'
                tracks={ this.props.tracks }
                trackPlayingId={ this.props.trackPlayingId }
                playlists={ this.props.playlists }
            />
        );
    }

    render() {
        return (

                { this.getLibraryComponent() }

        );
    }

don't you feel it's just too much abstraction ? We're calling FullViewMessage multiple times, why make a separate component that renders just a new FullViewMessage ?
Imho, it really should be:
    getLibraryContent() {
        if(this.props.library === null) {
            return (
                <FullViewMessage>

Loading library...

                </FullViewMessage>
            );
        }
        if (this.props.library.length === 0) {
            return (
                <FullViewMessage>

Too bad, there is no music in your library =(

                        nothing found yet, but that's fine, you can always
                        <Link to='/settings/library'>add your music here</Link>

                </FullViewMessage>
            );
        }
        if (this.props.tracks.length === 0) {
            return (
                <FullViewMessage>

Your search returned no results

                </FullViewMessage>
            );
        }
        return (
            <TracksList
                type='library'
                tracks={ this.props.tracks }
                trackPlayingId={ this.props.trackPlayingId }
                playlists={ this.props.playlists }
            />
        );
    }

    render() {
        return (

                { this.getLibraryContent() }

        );
    }

And I'd put everying in the render method. But that's another thing, and I'm ok with calling class methods to get component content.
Abstracting things is a good idea when components gets too long or too complex, but not everwhere/everytime

I agree that no abstraction is better than bad abstraction, but that does not mean we shouldn't seek for   a good abstractions.
We are not in a hurry, right? We don't have to ship anything, and can just sit back and think, right?

Tbh, I don't really worry about how the code looks, because the codebase looks good anyway. I'm just a bit more focused on features for now.

I think we broke appveyor :stuck_out_tongue_winking_eye:

Tbh, I don't really worry about how the code looks

Well, if you only start worrying when it looks bad, then it is a bit late.

Well it's often late in that case ;)
I propose we pass once on the code before releasing, but now, let's focus on the features.

About notifications ect, I think we can create a user-story/issue, even if ftm it's ok. I'd need some use-case first.

Well it's often late in that case ;)

Just wanted to make the point, nothing more.

I propose we pass once on the code before releasing, but now, let's focus on the features.

and

About notifications ect, I think we can create a user-story/issue, even if ftm it's ok. I'd need some use-case first.

Sounds good to me.


Add enter handler for the textbox

Fixes: https://github.com/h2oai/wave/issues/372

@mturoci - this might conflict with #348?

Yes, I suppose. I suggest addressing both issues in this PR.

The implementation unconditionally calls the keyup handler. We need a properly-named attribute on the textbox that conditionally enables this behavior.
@mturoci - this might conflict with #348?

@lo5 @mturoci Introduced a new attribute called submit to the textbox to conditionally the keyUp handler and addressed the requested changes.


Command 'auth login\r\n' not found

Hi,
I'm testing mailslurper with a Java application and see the following in the log:
ERRO[5915] Problem finding command from input            error="Command 'auth login\r\n' not found" input="AUTH LOGIN\r\n" who="SMTP Worker 10"

mailslurper works with other programs, so I'm guess the Java app might speak SMTP wrong, but I'm no expert.
Any hint is welcome.
kind regards,
secbob

Hello @secbob.
I found this very nice repo today. But, I can anwser you.
The authentication is not for SMTP, but only for the web page admin.
See https://github.com/mailslurper/mailslurper/blob/master/pkg/mailslurper/SmtpCommands.go
The command AUTH LOGIN is not present.
But it would be very good for someone to implement this, because when our applications went into production, they will use authentication and, it would be nice to be able to test this.
Still, congratulations to the developers.
(sorry for my english)

Thanks for the clarification. I've been since using rnwood/smtp4dev which works with my Java app.


Enhancement - Improve method tooltips.

Hello
This pull request contains the following changes:

The method tooltip now uses divs instead of spans. This resolves the issue where only the first line of the description and summary would have a margin when it spanned multiple lines.
The method tooltip now shows the parameters and return type of a method (without their description, for now).
The long description will now no longer trim out extra newlines that you added in the docblock for clarity.
The summary and long description will no longer filter out any asterisks or slashes inside the string itself.
DocParser will now differentiate between no docblock or a docblock without a @return tag. This is because, according to the phpDocumentor documentation, a docblock without a return tag implies @return void, whilst for constructors it implies @return self. When no docblock is available, we can't assume the method returns void so it now just leaves the return type out during autocompletion to explicitely indicate that it is unknown.

I also wanted to ask for your input on the tooltips: as you can see, the tooltip is growing larger and larger as more information is added. I was thinking that perhaps we should only show the method signature and the summary by default, and show all the information if you hold down the alt modifier key while hovering over the method? What do you think?

Hi thank you.
I'll try it tomorrow. I think that adding a key to see more is not the best choice because some people could miss the feature. Perhaps just a configuration option ?

That would indeed be an option. I think e.g. PHPStorm doesn't even show a tooltip on hover, it just shows all the information it has when you press a certain shortcut. IMHO our approach feels better as the tooltip doesn't get in the way and you don't need to press a key each time, which is cumbersome.
I think we need something you can trigger easily, i.e. see the basic information in the tooltips and have a way to 'view the rest of the information' whenever you want to. With a config option it's all or nothing in the sense that you either get all information all the time or basic information all the time. Would using a modifier key and displaying a note about it in the basic tooltip be any help? You'd have something like:
void myMethod($param1)

This is the summary.

Hold alt for more information.

(The last sentence could be styled to e.g. have a smaller font.)

I tested it, and there something that I don't like : you print "void" when the method doesn't return anything.
I did it at first, but with the feedback from some users, I decided to not print it because you can see faster which method returns something, and which not when there's not keyword at all. With void, you must read it to be able to know it
For the tooltip size, perhaps both solutions could be cool. If you want the full description always, there's a parameter in configuration to set. Otherwise, you will have a message to press a key ?

Regarding the tooltip size: having both an option and the modifier key seems like the best solution, i.e. something like Always show detailed tooltips.. Is this something you want in this pull request before it is accepted or can I delegate this to an enhancement ticket and a separate pull request at a later date?
About the returning of void when the @returntag is omitted, I did this because both phpDocumentor and the draft PSR-5 dictate it (see also [1] and [2]). The draft PSR-5 recommends always specifying a return type, but allows leaving it out for methods that return nothing for convenience. (I however didn't do this for methods that are missing a docblock.) If however, this is something you absolutely don't want, should I change it to return nothing at all or perhaps mixed?
[1] https://github.com/phpDocumentor/fig-standards/blob/master/proposed/phpdoc.md#815-return
[2] http://www.phpdoc.org/docs/latest/references/phpdoc/tags/return.html

For the tooltip size, it could be done later.
About the return tag, I'm not against the "void" (for the tooltip for example), I just don't want it into the suggestion list because it's harder to see the difference between methods that returns something and those that return nothing (or don't have a doc)
So, perhaps the proxy could return "void" to show it somewhere else in the plugin, but in the autocomplete providers, when there's "void", nothing should be show (in my opinion)
Is that a good thing in your opinion ?

I see your point; if you show nothing for void in the autocompletion suggestions, you can then immediately deduce that nothing will be returned, without having to read the word void (because the return type in the suggestions list is empty, you have a visual constrast with suggestions that do have a return type). This would also mean that when you see an empty return type, it could mean any of:

The method doesn't have a docblock. In reality, the method could be returning anything (void or non-void). (None of the built-in PHP functions have a docblock so this seems like the appropriate action.)
The method has a docblock, but no return statement. As previously stated, it should (and that is an important word in loose typing ;-) be retuning void.
The method has a docblock and explicitely indicated void to be its return type.

In contrast, if we explicitly print void, when you see void, you know that:

The method is documented.
The method should not be returning anything (or you shouldn't expect a useful return value).

... and when you see nothing, you know that the method is not documented at all and you shouldn't be relying on autocompletion for the return type. You should then probably either document the method (if it's your own code) or resort to some other form of documentation (in the case of the built-in PHP functions), as they can return pretty much anything and for us it is an 'unknown'.
What do you think?

You convinced me, even if I don't like it^^


Gighub copilot chat for vscode on linux through OOD Apps

Copilot Chat Extension Version: GitHub.copilot-chat-0.21.2024092601.vsix
VS Code Version: 1.92.2
OS Version: Rocky-8-Linux
Logs:
[spsoni@ln003 ~]$ code-server --install-extension GitHub.copilot-chat-0.21.2024092601.vsix
Installing extensions...
Error: Unable to install extension 'github.copilot-chat' as it is not compatible with VS Code '1.92.2'.
at G.install (/global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:172:75857)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async a.h (/global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:145:1092)
at async /global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:144:9774
at async Promise.all (index 0)
at async a.installExtensions (/global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:144:9711)
at async /global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:186:76641
at async se.run (/global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:186:76404)
at async Q (/global/home/users/spsoni/.local/lib/code-server-4.92.2/lib/vscode/out/vs/server/node/server.main.js:186:78861)
Failed Installing Extensions: file:///global/home/users/spsoni/GitHub.copilot-chat-0.21.2024092601.vsix

Steps to Reproduce:
1.Installed VS Code using following steps
VERSION=4.92.2
mkdir -p ~/.local/lib ~/.local/bin
curl -fL https://github.com/coder/code-server/releases/download/v$VERSION/code-server-$VERSION-linux-amd64.tar.gz
| tar -C ~/.local/lib -xz
mv ~/.local/lib/code-server-$VERSION-linux-amd64 ~/.local/lib/code-server-$VERSION
ln -s ~/.local/lib/code-server-$VERSION/bin/code-server ~/.local/bin/code-server
PATH="~/.local/bin:$PATH"
code-server
Now visit http://<IP_ADDRESS>:8080. Your password is in ~/.config/code-server/config.yaml
2.Downloaded githubchat vsix file on my mac moved it to linux server and used command "code-server --install-extension GitHub.copilot-chat-0.21.2024092601.vsix"

Unable to install extension 'github.copilot-chat' as it is not compatible with VS Code '1.92.2'.

Looks like you use the prerelease version of chat with stable vscode. That isn't supported and you should use the extension's release version or vscode insiders

I am using code-server version 4.91.1 and copilot-chat 0.21. I fixed above using by changing the project.json file. Modified vscode:'^4.94.1' with lower version. Now I am getting another issue. Chat is not working properly and give me the error message "Right-hand side of 'instanceof' is not an object" when I ask any questions.

How do I download specific version of copilot chat vsix file?


Is it possible to export data to BigQuery arrays with to_gbq?

I'm trying to export a DataFrame to BigQuery with to_gbq, and a few columns have array values.  Unfortunately, these get coerced into strings if I rely on the built-in schema inference.  If I manually specify ARRAY type in table_schema I get this error: "Invalid value for type: ARRAY is not a valid value."
I see that ARRAY is not listed as a valid type here, but I'm having trouble finding any further documentation.  Is this not supported?

There's very limited support given pandas' support for nested arrays. @tswast will know better but one reasonable approach is to convert to a better supported format (unnest / another table / string), and then normalize into arrays with a BQ query over your uploaded data

@mtchllbrrn
Good question, I don't think this is possible. Under the hood, to_gbq uses standard GBQ client with CSV as an intermediate format. Repeated fields are not supported for this ingestion format, see https://cloud.google.com/bigquery/docs/loading-data-cloud-storage-csv#limitations.

Got it, thanks for the help y'all.  I'll plan on exporting as strings and converting to arrays with a BQ query.


Replace various HTTP client implementations with symfony/http-client

This replaces the various implementations of HTTP clients (curl, Guzzle, Box\Curl) with symfony\http-client, dropping both the guzzle and curl dependencies (though cURL should be recommended for performance reasons).
It also removes a lot of outdated code, and code that needs to be maintained, in favour of a well maintained auto-loaded library (which should modules/adapters should be encouraged to use). Hopefully this helps moves toward a more stable base.

Note that in the current state tests have not been updated - are they still used?
Additionally, the adapters under library/Payment and library/Registrar may need updating.

Note that in the current state tests have not been updated - are they still used?
Additionally, the adapters under library/Payment and library/Registrar may need updating.

Yes, the tests are still used. You also need to generate a lock file for PHP 8.0 as that's the minimum supported for FOSSBilling. You also need to fix the PHPStan error, as there's an undefined class being referenced

@BelleNottelling Sorry about that, I was running a much newer PHP which I totally forgot about - sorted now.
For the PHPUnit tests at /tests, could you advise further on this? The tests seem to fail based on existing changes alone.

@BelleNottelling Sorry about that, I was running a much newer PHP which I totally forgot about - sorted now.
For the PHPUnit tests at /tests, could you advise further on this? The tests seem to fail based on existing changes alone.

The test is now passing, so it looked like you fixed the issue

@BelleNottelling Sorry about that, I was running a much newer PHP which I totally forgot about - sorted now.
For the PHPUnit tests at /tests, could you advise further on this? The tests seem to fail based on existing changes alone.

The test is now passing, so it looked like you fixed the issue

Yeah, apologies again, I was running PHP 8.2. 😅
So are the PHPUnit tests at /tests not currently manually ran? I'm guessing not as they're mostly broken, which is fine with me - I just don't want to spend time updating unused code.

@BelleNottelling Sorry about that, I was running a much newer PHP which I totally forgot about - sorted now.
For the PHPUnit tests at /tests, could you advise further on this? The tests seem to fail based on existing changes alone.

The test is now passing, so it looked like you fixed the issue

Yeah, apologies again, I was running PHP 8.2. 😅
So are the PHPUnit tests at /tests not currently manually ran? I'm guessing not as they're mostly broken, which is fine with me - I just don't want to spend time updating unused code.

It depends:
https://github.com/FOSSBilling/FOSSBilling/tree/main/tests/integration they are currently not working...
See other PR
Other should work fine as it...

@BelleNottelling Sorry about that, I was running a much newer PHP which I totally forgot about - sorted now.
For the PHPUnit tests at /tests, could you advise further on this? The tests seem to fail based on existing changes alone.

The test is now passing, so it looked like you fixed the issue

Yeah, apologies again, I was running PHP 8.2. 😅
So are the PHPUnit tests at /tests not currently manually ran? I'm guessing not as they're mostly broken, which is fine with me - I just don't want to spend time updating unused code.

It depends: https://github.com/FOSSBilling/FOSSBilling/tree/main/tests/integration they are currently not working... See other PR
Other should work fine as it...

Interesting, in which case tests/modules/Extension/ServiceTest.php should have failed as I've yet to update it.
Though, looking back through the Actions history the tests have failed (but passed) for a while with:
Run php ./src/vendor/bin/phpunit ./tests/modules/ Error #[0] occurred in [/home/runner/work/FOSSBilling/FOSSBilling/src/vendor/phpunit/phpunit/src/TextUI/Command.php] at line [99]: [syntax error, unexpected token "}"] THE ERROR HANDLER HAS CHANGED!

As I said, the tests that are run in the action are in use.  That should be the modules and library tests. Outside of that, that's really all I know. Those tests come from BoxBilling and are largely incomplete, don't find issues, and prove to be troublesome.
If I remember correctly, BoxBilling awhile ago had codecov used and it only scored like 45% coverage. We'd hugely benefit from reworked tests.
You now know pretty much all I do about the tests 🤷♀

@BelleNottelling Fair enough 😅. I'll not bother with any tests not passing/tested by Actions then.
I'm sure we can work on improving the testing infrastructure once the legacy stuff has been removed.

Now that the module tests are fixed, I'll rework the related tests for this change and rebase on main once #867 is merged.

This is virtually complete but I need to complete the PayPal payment gateway changes once I better understand what's actually going on in there, and I ideally want to remove the use of file_get_contents() to get remote content, so this will likely be toward the end of the weekend now. 😅

This is pretty much ready to go, bar the file_get_contents() changes which can be addressed at a later date. 😃


assume unfound classes to be in the same package as the current class

Professor,
This is the PR to fix #49 and #48.
For classes that could not be found among import statements, instead of throwing an exception, Specimin will now assume those classes to be in the same package as the current class.

LGTM

Professor,
I have updated the codes based on your suggestions. Please take a look.


fsspec wrapper

Wrapper around the existing object store API we present to Python to be used
E.g. something like this should work, but doesn't quite yet:
import object_store_rs as obs
import pyarrow.parquet as pq
from object_store_rs.fsspec import AsyncFsspecStore

store = obs.store.HTTPStore.from_url("https://github.com")
fs = AsyncFsspecStore(store)
url = "opengeospatial/geoparquet/raw/refs/heads/main/examples/example.parquet"
test = pq.read_metadata(url, filesystem=fs)

File ~/github/developmentseed/object-store-rs/.venv/lib/python3.11/site-packages/fsspec/spec.py:1918, in AbstractBufferedFile._fetch_range(self, start, end)
   1916 def _fetch_range(self, start, end):
   1917     """Get the specified set of bytes from remote"""
-> 1918     raise NotImplementedError

cc @martindurant

Here, you are using the file-like API. I can make a thin wrapper which calls cat_file with a range for any input filesystem, and then this should work.

I figured that I only needed as a minimum to implement the methods on the AsyncFileSystem base class that raised NotImplementedError, and that everything else would be supported out of the box?

Yeah, a fallback implementation in fsspec would totally make sense. This does the business
--- a/fsspec/spec.py
+++ b/fsspec/spec.py
@@ -1915,7 +1915,7 @@ class AbstractBufferedFile(io.IOBase):

     def _fetch_range(self, start, end):
         """Get the specified set of bytes from remote"""
-        raise NotImplementedError
+        return self.fs.cat_file(self.path, start=start, end=end)

     def read(self, length=-1):

together with
--- a/object-store-rs/python/object_store_rs/fsspec.py
+++ b/object-store-rs/python/object_store_rs/fsspec.py
@@ -55,7 +55,7 @@ class AsyncFsspecStore(fsspec.asyn.AsyncFileSystem):
             resp = await obs.get_async(self.store, path)
             return await resp.bytes_async()

-        if start and end:
+        if start is not None and end is not None:
             return await obs.get_range_async(
                 self.store, path, offset=start, length=end - start
             )

(because start can be and often is 0, and actually None means 0 anyway).
Leads to:
In [1]: import object_store_rs as obs
   ...: import pyarrow.parquet as pq
   ...: from object_store_rs.fsspec import AsyncFsspecStore
   ...:
   ...: store = obs.store.HTTPStore.from_url("https://github.com")
   ...: fs = AsyncFsspecStore(store)
   ...: url = "opengeospatial/geoparquet/raw/refs/heads/main/examples/example.parquet"
   ...: pq.read_metadata(url, filesystem=fs)

Out[1]:
<pyarrow._parquet.FileMetaData object at 0x10a8539c0>
  created_by: parquet-cpp-arrow version 16.1.0
  num_columns: 10
  num_rows: 5
  num_row_groups: 1
  format_version: 2.6
  serialized_size: 6685

Until https://github.com/fsspec/filesystem_spec/pull/1732 is merged and released, is it expected for AsyncFsspecStore to subclass both from fsspec.asyn.AsyncFileSystem and AbstractBufferedFile?

is it expected for AsyncFsspecStore to subclass both from fsspec.asyn.AsyncFileSystem and AbstractBufferedFile?

That sounds ... painful. The linked PR has no negative side effects, so I only need to write some kind of test for it.

@martindurant Do you have any interest in pushing this PR over the line, since you know fsspec so well?

Sure! What needs doing besides the fsspec PR?

I'm not fully sure how to test it. I'm also not sure if there are any provided methods in the fsspec base class that we should be implementing, that could be more efficiently provided from the Rust side (without adding a bunch of new Rust code; I'd like this fsspec wrapper to ideally be python-only)

we would love to contribute to this as well and make it work with flyteorg/flyte - wdyt?

I'm happy to accept contributions

@kylebarron, When will this be merged? I'd like to help work on it as well.

I'd be happy to accept a PR with tests. There was some progress from @martindurant in #60

Superseded by https://github.com/developmentseed/obstore/pull/63


Error: unknown flag: --from-file

Hey! When I run this command from the book for example 14-3:
kubectl apply cm --from-file ghost-config.js ghost-config
I get this error:
Error: unknown flag: --from-file
I'm on kubectl version:
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.3", GitCommit:"f0efb3cb883751c5ffdbe6d515f3cb4fbe7b7acd", GitTreeState:"clean", BuildDate:"2017-11-08T18:39:33Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.4", GitCommit:"d6f433224538d4f9ca2f7ae19b252e6fcb66a3ae", GitTreeState:"dirty", BuildDate:"2017-06-22T04:31:09Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

Do I need to be on an older version for that invocation to work?
For anyone else with this problem, I used this command to get it working:
kubectl create configmap ghost-config --from-file ghost-config.js

Thanks so much for making this correction!

Thanks for posting this work-around for this error in the book!

@ftripier ,
kubectl create configmap ghost-config --from-file ghost-config.js

This command would fail if a configmap ghost-config already exists.
kubectl apply is used mainly to update resources if there is change in manifests. So how can we update an existing configmap ?

@karthikc911, I know this issue isn't too fresh, but I'll post this here anyways for future readers. You could try to do something like this: https://blog.atomist.com/updating-a-kubernetes-secret-or-configmap/

Looks like this has been addressed. Feel free to reopen if that's not the case.


Add more details to session

Add details discussed in #265.
Also added a link to the specific comment regarding EEG considerations from our discussion.

I can build it locally so not sure what is wrong regarding the build-book fail.
Also, I couldn't find a new link to replace [✖] http://www.fieldtriptoolbox.org/reference/data2bids/, it seems to be gone from that website..

Also, I couldn't find a new link to replace [heavy_multiplication_x] http://www.fieldtriptoolbox.org/reference/data2bids/, it seems to be gone from that website..

Maybe best to link to the code itself?
https://github.com/fieldtrip/fieldtrip/blob/release/data2bids.m
Also those are not changes you have made, so you did not break it so I would not worry about it.

@SamGuay updated jupyter book version in #267 and that fixed it. :magic_wand:


Slack notification

Mandatory parameters:
--slack_notifier true --slack_webhook_url WEBHOOKURL
Optional:
slack_username & slack_channel, default values webhookbot and general
Ex: command:
github_changelog_generator -u test -p test -t TOKEN --slack_notifier true --slack_webhook_url WEBHOOK_URL --slack_channel random --slack_username testUser

Thanks so much for spending time on this amazing patch. We really appreciate it. However, I do not think this is something we want to add right now, since Slack notification can be done as a command-line invocation, on the content that we generate.
Perhaps we can add a README section on things one can do with the generated CHANGELOG.md when it's been produced, and then a "Send it to Slack" recipe, using another tool, for example slackcat.
Thanks again! I'll close this PR.


many: update kirkstone branch

Update the kirkstone branch with the changes from dunfell. Update the kernel patches for strict confinement.
Verified locally. Both 5.15 and 5.10 linux-yocto kernels are built with apparmor support. Snap reports strict confinement on both kernels.
FYI when you attempt a local build, the AppArmor recipe may fail. I pushed a patch to my fork which I intend to upstream at some point. The patch is right here: https://github.com/bboozzoo/meta-security/commit/d98555ba2f72f103b955aeed934f91933b753f1c

I had a number of commits on my local branch that are now obsoleted by this
PR, so if you want to merge this, I'm happy to re-calibrate my work on top
of that.
rday
On Tue, Jul 19, 2022 at 12:16 PM Omer Akram @.***>
wrote:

@.**** approved this pull request.
Great progress. LGTM
—
Reply to this email directly, view it on GitHub
https://github.com/snapcore/meta-snapd/pull/10#pullrequestreview-1043722845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEUVBYHYMTFEZ4RNCEUX43VU3IHJANCNFSM54AQ2XKA
.
You are receiving this because your review was requested.Message ID:
@.***>


IPython parameter autocompletion breaks if parameter has empty docstring

Simplest example:
class A(param.Parameterized):
    a = param.Boolean(default=False, doc="")

Solution: don't have empty docstrings? :-)

Solution: don't have empty docstrings? :-)

Agreed :) Still shouldn't break though. Simple fix but I'll leave it to Jean-Luc.

Ok sure! I'll have a look at this shortly...
I think parameters shouldn't ever have empty docstrings but I also agree that a nicer warning should be generated should such an empty docstring be encountered.

Check if still a problem

Autocompletion now based on the signature so no longer has this issue.


In reader, slide shows do not work anymore

Only the first image is shown.
See for example page 25 in wind355, which worked properly before (tested in #34)
@serverfire Can you fix this?

@libreliodev Fixed.

@serverfire Thanks.


Can anyone tell me how to call insert_roles() method to initialize the "roles" table？

Can anyone tell me how to call insert_roles() method to initialize the "roles" table？
I have encountered some error:
"application not registered on db instance and no application bound to current context"
when I call Role.insert_roles()

Closing as a duplicate.


Fixed unsupported ibmq_manila in test

Description

Fixes tests for QPU ibmq_manila

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

The tests fails because of the AWS login (fixed in #287 )


Connection failed when Redis master down

What happened:
Connection to Redis failed when master went down.
redis: 2022/03/17 05:37:39 sentinel.go:536: sentinel: GetMasterAddrByName name="redis" failed: dial tcp {old-master}:26379: i/o timeout
2022/03/17 05:37:39.428142 juicefs[7] <ERROR>: error: redis: connection pool timeout

What you expected to happen:
Automatically connect to the new master.
How to reproduce it (as minimally and precisely as possible):

Redis sentinel mode
Power down the Redis master node

Anything else we need to know?
Environment:

JuiceFS version (use juicefs --version) or Hadoop Java SDK version: 1.0.0-beta1
Metadata engine info (version, cloud provider managed or self maintained): Redis server v=6.2.6

@SandyXSD we should update the docs on how to adjust these connection parameters.

I'm afraid it's a too advanced topic.

I added a proxy (k8S service or any other load balancer with health check) in front of Sentinel, which effectively solved the sentinel Timeout problem


[Bug]: Lack of module "importlib.resources" in python 3.6 env

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Lack of module "importlib.resources" in python 3.6 env:
>>> from towhee import pipeline
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/__init__.py", line 19, in <module>
    from towhee.engine.engine import Engine, start_engine
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/engine.py", line 18, in <module>
    from towhee.engine.pipeline import Pipeline
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/pipeline.py", line 18, in <module>
    from towhee.engine.graph_context import GraphContext
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/graph_context.py", line 21, in <module>
    from towhee.engine.operator_context import OperatorContext, OpStatus
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/operator_context.py", line 24, in <module>
    from towhee.engine.thread_pool_task_executor import ThreadPoolTaskExecutor
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/thread_pool_task_executor.py", line 22, in <module>
    from towhee.engine.operator_pool import OperatorPool
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/operator_pool.py", line 19, in <module>
    from towhee.engine.operator_loader import OperatorLoader
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/engine/operator_loader.py", line 25, in <module>
    from towhee.hub.file_manager import FileManager
  File "/Users/binbin/Towhee_projects/towhee-test-env/lib/python3.6/site-packages/towhee/hub/file_manager.py", line 20, in <module>
    import importlib.resources
ModuleNotFoundError: No module named 'importlib.resources'

Expected Behavior
Install this module automatically
Steps To Reproduce
1. create python 3.6 virtual env
2. install towhee
3. from towhee import pipeline

Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):0.4.1.dev87
- OS(Ubuntu or CentOS):mac
- CPU/Memory:
- GPU:
- Others:

Anything else?
No response

/assign @Chiiizzzy


Show error message upon each retry failure

Summary
this is part of https://github.com/harvester/harvester/issues/5288, after https://github.com/harvester/harvester/pull/5386 merged to master, error message updated upon each retry failure. Before this pr, error message pops up only when upgrad CR imageReady is False.
Occurred changes and/or fixed issues
related to https://github.com/harvester/harvester/issues/5288
Technical notes summary

N/A
Areas or cases that should be tested

Harvester upgrade
Areas which could experience regressions

N/A
Screenshot/Video
Test upgrade

Screencast from 2024-03-18 17-17-20.webm
Test Image

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[ ] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[ ] The PR has a reviewer assigned
[ ] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[ ] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

Remember to backport to release-harvester-v1.4 after merge.

@mergify backport release-harvester-v1.3 release-harvester-v1.4


add visual feedback when editing relations that something happened

currently the icon is not changed when hovering the edit icon and if you click the icon the form is loaded, but visually nothing changes.
change the icon on hover and click
scroll to form on click

I noticed the edit button doesn't have its usual colour (yellow) in the relations tables, but is blue, which makes me think that that may be connected to it not appearing pressed/clicked. I.e. maybe it's integrated differently into that table than in others places and is accidentally overlayed by another element or similar.

I.e. maybe it's integrated differently into that table than in others places and is accidentally overlayed by another element or similar.

Its a totally different edit column that does some weird ajax form loading voodoo. The whole triple table only started to inherit from GenericTable a one or two weeks ago

The new relations (1799ea63c3c222c075a2006495675dbdc7651315) use different forms, so I'm closing this


Self-removal/muting support MVP

Added by Rich:
MLS allows you to publish proposals and commits. In some systems, the proposals are sent separately, and then a later commit is published by someone else that commits all of the previous proposals. In our current implementation, all proposals are immediately committed by the proposer, so we don't use separate proposal messages, just commits that include the proposals.
The problem with removing yourself is that MLS prohibits commiting a Remove proposal that removes the committer. I think the reason is that generating commits requires deriving secrets to use for the next epoch of the group, and therefore anyone being removed from the group should not have access to those secrets.
It seems there are two ways to solve this problem:

Add support for separating out proposals and commits. So to remove yourself, you publish a proposal, and then somebody else commits it. This requires a bunch of changes to how we do proposals/commits today. We will need to chat about how to design this.

You remove yourself from the group, but don't update the group encryption state (in essence, you've effectively 'muted' the group without changing the group membership). I don't like this so much, because you still need a way to notify your other devices you've done this, otherwise you're only muting for your current device.

Here's how I suggest doing this task.
There are two parties here, the ‘proposer’ and the ‘committer’.
Logic to add for the proposer:

Create a IntentKind::LeaveGroup intent type here.
Cmd+F for IntentKind::RemoveMembers, and follow the same lifecycle:

Just like remove_members(), add a method called leave_group() that constructs the LeaveGroup intent and then syncs.
In get_publish_intent_data(), add another match arm for LeaveGroup. This case should be much simpler, as there is no IntentData to fetch. Use this OpenMLS method to create a leave group proposal. Return it from get_publish_intent_data and let it be sent on the group.
In process_own_message(), add another case for LeaveGroup. Extract the PrivateMessageIn from message: ProtocolMessage, then decrypt it via openmls_group.process_message(). If the decryption fails, call conn.set_group_intent_to_publish() to reset the intent. If the decryption is successful, go ahead and delete the group from the database. conn.set_group_intent_committed() will be called at the end of the function.

Logic to add for the committer:

In process_external_message(), fill in the match arm for proposal messages:
Validate that the proposal is an attempt by a member to leave, discarding the proposal if not. You could consider using this.
Store the pending proposal as described here.
Create a IntentKind::CommitPendingProposals intent type. Implement the lifecycle events:

I wrote all of this up, but with XIP-46 coming, I just realized it may actually be easier to wait for that, and have the proposer just propose an update to the members list on the group context, and have the committer handle resolving the list of people to remove. That might simplify the above significantly. In fact, it may be worth considering rolling this into the identity work.

Simpler metadata-based approach without floating proposals:

Commit a metadata change to remove your inbox ID
When any other installation processes metadata changes:

If they remove yourself, go ahead and update the group state to LEFT_GROUP
If they remove someone else, publish a commit to remove all of their installations from the group

PR started here: https://github.com/xmtp/libxmtp/pull/1090


twister: pytest harness requires custom parameters even it does not exist

Describe the bug
The sample samples/subsys/testsuite/pytest/shell/sample.pytest.shell fails.
This test uses harness: pytest.
But the underlaying tests seem to pass just fine.
To Reproduce
Steps to reproduce the behavior:

connect nrf52840dk/nrf52840
run scripts/twister -p nrf52840dk/nrf52840 -s sample.pytest.shell -v --inline-logs --device-testing --device-serial /dev/ttyACM0
See error

Expected behavior
No reported failure
Impact
Failure in CI
Logs and console output
Traceback (most recent call last):
  File "/usr/lib/python3.10/multiprocessing/process.py", line 314, in _bootstrap
    self.run()
  File "/usr/lib/python3.10/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/kagi/zephyrproject/zephyr/scripts/pylib/twister/twisterlib/runner.py", line 1320, in pipeline_mgr
    pb.process(pipeline, done_queue, task, lock, results)
  File "/home/kagi/zephyrproject/zephyr/scripts/pylib/twister/twisterlib/runner.py", line 674, in process
    self.run()
  File "/home/kagi/zephyrproject/zephyr/scripts/pylib/twister/twisterlib/runner.py", line 1129, in run
    harness.pytest_run(instance.handler.get_test_timeout())
  File "/home/kagi/zephyrproject/zephyr/scripts/pylib/twister/twisterlib/harness.py", line 303, in pytest_run
    cmd = self.generate_command()
  File "/home/kagi/zephyrproject/zephyr/scripts/pylib/twister/twisterlib/harness.py", line 344, in generate_command
    self._generate_parameters_for_hardware(handler)
  File "/home/kagi/zephyrproject/zephyr/scripts/pylib/twister/twisterlib/harness.py", line 385, in _generate_parameters_for_hardware
    for param in hardware.runner_params:
TypeError: 'NoneType' object is not iterable

Environment (please complete the following information):

OS: Ubuntu 22.04
Toolchain: Zephyr SDK 0.16.5
Commit SHA or Version used: a1dd9a5f6e562a11edc238e3af76dd64caddca83

Additional context
The hw map was not used.
Add checking the exist of the custom parameters does not trigger above traceback.

Fix is provided by @katgiadla, ready to merge.


Updated Hungarian translation

Translated the welcome message

Might look different in the GitHub mobile view, but it looks great on desktop, so don't worry:


Function for splitting cellhash data by sample

Once we have cellhash barcode data integrated with Alternative Experiments as described in https://github.com/AlexsLemonade/scpcaTools/issues/28, we will need a function to split those data into separate SCE objects by sample.
This function should do some testing for doublets where cells come from multiple samples, and other QC as appropriate.
Of note, the cellhash tag matrix is likely to have generic names (MUTLI1, MULTI2, etc.) rather than specific sample names, so we will probably want to incorporate changing those to sample names here.

With some of the results from https://github.com/AlexsLemonade/alsf-scpca/pull/164, I think it makes sense to rescope this issue to not split by cellhash results, but rather to add columns to the colData for the object with statistics and assignments from the various potential methods for demultiplexing, while leaving the full library as a single SCE object.
This would likely mean adding an add_cellhash_demux() function similar to add_miQC(), which would add extra columns to the colData, while also recording any specific parameters in the metadata.
This function should probably add demux results from both DropletUtils::hashedDrops() and Seurat::HTODemux().

I think I would consider this closed by #84 and  #89 (with a little mention to #93). I am no longer sure a monolithic function that does both at once is required, so I am closing this, though we could reconsider.


feat: Make autoUpdate and autoLaunch features configurable in MakerWixConfig

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] The changes are appropriately documented (if applicable).
[ ] The changes have sufficient test coverage (if applicable).
[ ] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
feat: Make autoUpdate and autoLaunch features configurable in MakerWixConfig

Added features property to MakerWixConfig to enable configuration of autoUpdate and autoLaunch features

(see electron-wix-msi's Features definition here)

These features can then be configured by adding features to @electron-forge/maker-wix's config in forge.config.js
Example: features: { autoUpdate: true, autoLaunch: false }

@MarshallOfSound and @malept Is this the correct way to submit a PR to the repository? The option to add reviewers is disabled for me so I'm tagging you here to take a look. Thank you!

@MarshallOfSound @malept I see that the slow-tests for ubuntu are failing repeatedly, but as someone pointed out in the Electron HQ Slack thread, these tests are also failing on master. Can this PR be merged then since the test issue is not caused by this PR? Thanks!

Ah, I missed this. In the future the PR title should look something like
feat(maker-wix-msi): add features to config

Gotcha. Thank you Mark!


Feat: Adding Link to The Social icons instead of website opening in a new tab

Describe the feature
i want to add different links in each icon given on page. For example like if we click on github icon then it opens your repo in a new tab. That will help users to get more known about the project admin or owner.please give this issue sir to me! Thank you.
Add ScreenShots

as i go to any of the link given below in the social icons again the webpage opens in a new tab

Record

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

Sorry @Devanshu53 but this website doesn't have any social presence now. Feel free to choose from any other issue.


🐛 BUG: The package "perf_hooks" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

Which Cloudflare product(s) does this pertain to?
Wrangler
What version of Wrangler are you using?
wrangler 2.8.1
What operating system are you using?
linux
Describe the Bug
I have an issue with wrangler i am using @vercel/node to create a service it wrangler I have added the platform in the wrangler.toml file like
`compatibility_date = "2023-01-30"
name = "my-worker"
account_id = "9a203996171bcf2f8bf9647631470d56"
node_compat = true
platform = 'node'
workers_dev = false`
But i still get this error

Hi! We're going to close this for now, since we don't have a good way to support the perf_hooks package (given security constraints around the workers environment).


Can't resolve 'reakit'

Hello!
I am getting an error when running this script. Did anyone have the same problem? What am I doing wrong?
./node_modules/villain-react/dist/villain.js
Module not found: Can't resolve 'reakit' in '...reader\node_modules\villain-react\dist'
import React from "react"
import Villain from "villain-react"

import 'villain-react/dist/style.css'

const Reader = () => {
    const url = '~/archives/archive_5247_115849.cbr'
    const workerUrl="./libarchive.js/dist/worker-bundle.js"

    return <Villain source={url} workerUrl={workerUrl} />
}

export default Reader

@Daniil-Solovyev You need to include reakit as a dependency in your project. it is not bundle in build.
https://reakit.io/

I was working on version 2.0 ( It allows you to create a custom UI and some other improvements ) see: https://github.com/btzr-io/Villain/issues/634
But I'm not sure if this module is used enough to keep maintaining it.
Please let me know if you are interested on v-2.0.

Now my code is
import React from "react"
import Villain from "villain-react"

import 'villain-react/dist/style.css'

const Reader = () => {
    const url = '~/archives/archive_5247_115849.cbr'
    const workerUrl="~/node_modules/libarchive.js/dist/worker-bundle.js"

    return (
        <Villain source={url} workerUrl={workerUrl}
            // options={{
            //     theme: "Dark",
            //     maxPages: 500,
            //     mangaMode: true,
            //     forceSort: false,
            //     allowFullScreen: true,
            //     autoHideControls: false,
            //     allowGlobalShortcuts: false,
            // }}
        />
    )
}

export default Reader

And error

This files are going to be accessed by the browser so it needs to be an absolute path:

Try to move this files to the public or static files directory in your server ( Usually where index.html is located )

Something like this
const url = 'http://localhost:3001/static/archives/archive_5247_115849.cbr'
const workerUrl="http://localhost:3001/static/js/worker-bundle.js"

Alternative: you can use an iframe to embed the reader.
See: https://villain.js.org/#/embed

Ok. Than you.


fix(KFLUXBUGS-902): disable pipeline actions for contributors

Fixes
KFLUXBUGS-902
Description
Disable pipeline actions for contributors
Type of change

[ ] Feature
[x] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[x] Edge

/lgtm

/retest


how to build this program,and there is not up-to-date mirror in dockerhub

I have a problem
API V3 is no longer supported. Use API V4 instead
this bug have fixd, but the image in dockerhub is not update

I've had a similar problem and make the following modification to solve it.
Change "go1.8.3" to "go1.11".
https://github.com/rancher/pipeline/blob/11e4b3f18f9a3502637fcf161bc1463cf74d5a99/Dockerfile.dapper#L14


Issue-Adds updated readme

Automated Review Comments
Issues in README.md:

(8, 'Line exceeds 80 characters')
(9, 'Line exceeds 80 characters')
(10, 'Line exceeds 80 characters')
(11, 'Line exceeds 80 characters')
(22, 'Line exceeds 80 characters')
(48, 'Line exceeds 80 characters')

I got it.


Add several fields linking

example:
@dataclass
class User:
    first_name: str
    last_name: str

@dataclass
class UserResponse:
    full_name: str

converter_user_to_response = get_converter(
    User,
    UserResponse,
    recipe=[
        link(
            P[User].first_name & P[User].last_name,
            P[UserResponse].full_name,
            coercer=lambda first_name, last_name: f"{first_name} {last_name}"
        )
     ]
 )

It is implemented with https://github.com/reagento/adaptix/pull/275


Use modern Java

Can't run LicenseChecker for project where compiler target version is later than 8.

@jpoikela We have bumped the JDK version to 12 as 13 seems incompatible with some gradle stuff. Due to this change, there are now some gradle failures when using jruby 9.1. 9.2 still works are expected.


Release CI scheduled action is triggered on wrong branch

The nightly scheduled release CI action gets triggered on the default branch (develop), leading to this failure. It should be triggered only on release branches, however. But, the on.schedule attribute does not support specifying a specific branch to schedule on.
I see two ways to solve this issue:

One can specify a branch to check out for the checkout action (in this case master). However, this would conflict with the on.push trigger for release/** branches, i.e. those would also be tested on the master branch. So I would need to either split off the scheduled action into its own action, or figure out whether one can pass the ref argument to the checkout action only if the action is scheduled.

Just remove the scheduling for the release CI. After all, the release versions are not going to suddenly change and break other branches. Release versions are always coordinated to work with each other.

I am currently in favor of the first option and just splitting the scheduled action off into its own action. I think its nice to frequently be checking the release version for compatibility, even if there shouldn't be any problems. But maybe I am just paranoid.
This issue also applies to all the other projects with a release CI, i.e. Predenc, Math, Mclwrap, and Groupsig.

I'm okay with either option 👍

vote for 1.


Udemy download issue.

[x ] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ x ] Bug report (encountered problems with youtube-dl)

Note it's a duplicate of https://github.com/rg3/youtube-dl/issues/18126,  but, I can't see there something which was merged into the current version.
C:\Users\TOMER\Downloads\Python_Tut_Udemy_Course>youtube-dl.exe --cookies cookies.txt --verbose --sleep-interval 127 -o "./%(playlist)s/%(chapter_number)s-%(chapter)s/%(autonumber)03d-%(title)s.%(ext)s" https://www.udemy.com/python-tutorial-for-beginners/learn/v4/content
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--cookies', 'cookies.txt', '--verbose', '--sleep-interval', '127', '-o', './%(playlist)s/%(chapter_number)s-%(chapter)s/%(autonumber)03d-%(title)s.%(ext)s', 'https://www.udemy.com/python-tutorial-for-beginners/learn/v4/content']
[debug] Encodings: locale cp1255, fs mbcs, out cp862, pref cp1255
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17134
[debug] exe versions: none
[debug] Proxy map: {}
[udemy:course] python-tutorial-for-beginners: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Unauthorized (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp36zpl0n4\build\youtube_dl\extractor\common.py", line 605, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp36zpl0n4\build\youtube_dl\YoutubeDL.py", line 2215, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

If the purpose of this issue is a site support request please provide all kinds of example URLs
All U-demy courses

But I can't see anything on the other issue.
https://github.com/rg3/youtube-dl/issues/18126

Repository owner locked and limited conversation to collaborators on 11 Nov 2018


Fix broken hashlink for toMatchSnapshot

The link for toMatchSnapshot died when the optional string was added to the matcher.

Thanks for this PR, but we're rewriting the docs and this is already addressed there, as TOC is autogenerated based on sections headings.


Revert "Make it just start"

Reverts AEGEE/oms-docker#17
Reason: dev environment requires absolute paths and doesn't allow having repos outside of containers

As agreed, it is better to split the files


wrong parent version in passport-service pom.xml

Great Code! I cloned it to try it out and mentioned that it could not compile: The pom.xml of passport-service refers to parent pom bootiful.microservices-parent in version 1.0.0-SNAPSHOT but it should be 1.0.0.BUILD-SNAPSHOT. Just a small typo...

Is it still applicable? @jnehring


[FLI-856] Support Alpine/Musl

Currently the engine is only compiled against GLibc which means the clients will not work in Alpine env
We should create musl compatible versions, targeting x86_64-unknown-linux-musl and aarch64-unknown-linux-musl
I think there are two paths we could take:

Continue to bundle these new libs along with the glibc versions so that the native clients (ie flipt-client-java) should 'just work' on all architectures
Create new musl/alpine versions of each flipt-client-x library, putting the responsibility on the user/integrator to select the correct SDK for their use case.

I'm leaning on 2 as it might be very difficult for us to figure out which version of the fliptengine Clib to 'load' at runtime when the clients initialize (ie likely not all languages allow you to know which libc your os supports/requires). Also each additional fliptengine we package increases the overall library side by avg of 20-30mb (perhaps we can find a way to slim these down though?)
I'd be interested to know what others think, also if there are any examples of other SDKs built with FFI and run on both libc and musl based OSes
From SyncLinear.com | FLI-856
SDKs to Support

[x] Go<EMAIL_ADDRESS>[ ] Node
[ ] Python
[ ] Ruby
[ ] Java

See: https://github.com/flipt-io/flipt-client-sdks/actions/runs/7962787687/job/21736987230?pr=140#step:5:1395 for POC of failing test using musl based docker image

cc @piclem

@markphelps I would go with the second option also.

What would option 2 look like in practice?
For example, if I import the go library, would I have to choose between an glibc version or a musl version? That would be somewhat problematic in scenarios where local development environments are glibc, but the production environment is musl.

@markphelps Spent some time looking into this.
I think a good example repo to look at is the confluent-kafka-go library since they use the librdkafka C-library under the hood.
It looks like they use Go build tags to support musl vs libc, which seems like a better option than different imports.
If you look at the librdkafka_vendor directory each included version of the library is 13MB+, so doesn't look like that's a huge issue.
Also, they see the build_*.go files (e.g. build_darwin_amd64.go) for how they use build tags to include the right version of the library.
It seems like the biggest downside to this approach is that it will download all the different pre-build binaries during the build phase, but the final binary should be statically linked to only the version the platform that you're building for needs.

Just checking in on this. We'd love to use the client-side evaluation SDK, but this is a blocker for us (not just musl, but also automatic arch selection).

He @jalaziz sorry for the delay in replying. We can definitely take a look at building a MUSL compatible build and packaging like you mentioned above. Or if you are able to contribute we would ofcourse ❤ that as well!
We are also actively exploring leaning more into WASM, and building a proper WASM impl of the evaluator engine (not just for JS like we have now) and then using WASM runtimes in the various languages we support to call the evaluator. The main benefit of this approach would be that it would be architecture independent.
First one would likely be Go since that's the language we use the most internally. Would this work for you and your team?

He @jalaziz sorry for the delay in replying. We can definitely take a look at building a MUSL compatible build and packaging like you mentioned above. Or if you are able to contribute we would ofcourse ❤ that as well!
We are also actively exploring leaning more into WASM, and building a proper WASM impl of the evaluator engine (not just for JS like we have now) and then using WASM runtimes in the various languages we support to call the evaluator. The main benefit of this approach would be that it would be architecture independent.
First one would likely be Go since that's the language we use the most internally. Would this work for you and your team?

That would work great. Our main use case is Go as well.
I don't know if I'll have time to contribute the changes for the approach outlined above, but I'll definitely try!

@jalaziz @piclemx we just released<EMAIL_ADDRESS>see #332 for details
Can you try:
go install<EMAIL_ADDRESS>
It should support both x86 and arm on Linux

I'm implementing Flipt at alpine based Java microservices for my company. Is there some workaround or a roadmap planned for the Java version? This seems to be a big blocker for now

I'm implementing Flipt at alpine based Java microservices for my company. Is there some workaround or a roadmap planned for the Java version? This seems to be a big blocker for now

I can get a Java version of the sdk built that works on musl. Will work on it today and keep you posted

@willyw0nka we just released https://central.sonatype.com/artifact/io.flipt/flipt-client-java-musl
could you try it and let me know if it works for you?

Hi @markphelps , I'm a colleage of @willyw0nka and tried your solution. It still doesn't work for us.
I integrated the dependency and built the project just fine, but when I tried to run it on my development machine (OSX) it throws me these errors.
nested exception is java.lang.UnsatisfiedLinkError: Unable to load library 'fliptengine': dlopen(libfliptengine.dylib, 0x0009): tried: 'libfliptengine.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibfliptengine.dylib' (no such file), '/Users/david/.sdkman/candidates/java/17.0.10-tem/bin/./libfliptengine.dylib' (no such file), '/Users/david/.sdkman/candidates/java/17.0.10-tem/bin/../lib/libfliptengine.dylib' (no such file), '/usr/lib/libfliptengine.dylib' (no such file, not in dyld cache), 'libfliptengine.dylib' (no such file) dlopen(libfliptengine.dylib, 0x0009): tried: 'libfliptengine.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibfliptengine.dylib' (no such file), '/Users/david/.sdkman/candidates/java/17.0.10-tem/bin/./libfliptengine.dylib' (no such file), '/Users/david/.sdkman/candidates/java/17.0.10-tem/bin/../lib/libfliptengine.dylib' (no such file), '/usr/lib/libfliptengine.dylib' (no such file, not in dyld cache), 'libfliptengine.dylib' (no such file) dlopen(/Users/david/Library/Frameworks/fliptengine.framework/fliptengine, 0x0009): tried: '/Users/david/Library/Frameworks/fliptengine.framework/fliptengine' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/david/Library/Frameworks/fliptengine.framework/fliptengine' (no such file), '/Users/david/Library/Frameworks/fliptengine.framework/fliptengine' (no such file) dlopen(/Library/Frameworks/fliptengine.framework/fliptengine, 0x0009): tried: '/Library/Frameworks/fliptengine.framework/fliptengine' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Library/Frameworks/fliptengine.framework/fliptengine' (no such file), '/Library/Frameworks/fliptengine.framework/fliptengine' (no such file) dlopen(/System/Library/Frameworks/fliptengine.framework/fliptengine, 0x0009): tried: '/System/Library/Frameworks/fliptengine.framework/fliptengine' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/System/Library/Frameworks/fliptengine.framework/fliptengine' (no such file), '/System/Library/Frameworks/fliptengine.framework/fliptengine' (no such file, not in dyld cache)
Next thing I tried was to run the app.jar inside our Alpine Linux docker image and the errors are the same as with the regular java version.
Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'flagController' defined in URL [jar:file:/app.jar!/BOOT-INF/classes!/org/company/services/nexus/medias/controller/FlagController.class]: Unsatisfied dependency expressed through constructor parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'featureFlagService' defined in class path resource [org/company/services/featureflags/config/FeatureFlagConfig.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.company.services.featureflags.service.FeatureFlagService]: Factory method 'featureFlagService' threw exception; nested exception is java.lang.UnsatisfiedLinkError: Error relocating /root/.cache/JNA/temp/jna8663586253287976616.tmp: swapcontext: symbol not found
hope you can help us.

Hi @markphelps no worries. If you need me to test something dont doubt to reach me.

I just released flipt-client-java-musl-v0.0.2 which should have fixed both issues above ☝🏻
it seems to take a bit for it to make it to maven central however, so maybe can try in 30m or so? Im about to be traveling for the next couple hours but can follow up later today

No worries! :)

I just finished testing it and it worked like a charm in both scenarios. Well done!

Thanks for your patience and for testing @SultanICQ !!

Hi @markphelps, today one of our developers with a windows machine tested the new library and, as everybody would expect, it didn't work because the windows libraries are not included.

Could it be possible for the musl version to add also de win dlls?

Hey @SultanICQ yes I will work on getting a windows version built this week

@SultanICQ does your colleague need 32-bit or 64-bit Windows support?

Work being done over 👉🏻 https://github.com/flipt-io/flipt-client-sdks/pull/433

Hey @SultanICQ we just released 0.2.0-rc.1 that should have Windows support. I say should because I cant easily test since I don't have a Windows machine. We're working on getting more comprehensive testing after release via GitHub Actions though.
Can you have your coworker try 0.2.0-rc.1 on Windows? Thanks!

Hi @markphelps !
I'll prepare a test for my colleage to try. I assume she'll do the test on monday at the earliest.
I keep you posted.

Hi @markphelps .
It worked perfectly. Thanks! I think that for now we have all the scenarios well covered.
Will the version of the library change to 0.2.0?

Hi @markphelps ,
I deleted my previous message, maybe you have received an email from github.
With Windows now there is no library error but now there is a subtle error with the evaluateBoolean response. I see that now is not returning a Result.
So I think that overall it works but I have to adapt my code with this method signature change.
I keep you posted.

Hi @markphelps ,
I deleted my previous message, maybe you have received an email from github.
With Windows now there is no library error but now there is a subtle error with the evaluateBoolean response. I see that now is not returning a Result.
So I think that overall it works but I have to adapt my code with this method signature change.
I keep you posted.

Yes apologies we simplified the response type by not returning the wrapped Result type re: #353
Since these are pre 1.0.0 releases we decided to bump the minor version for this change in method signature.. sorry about that

Well, all good and working fine with this new version from my side.
Nice work!!

@jalaziz @piclemx we just released<EMAIL_ADDRESS>see #332 for details
Can you try:
go install<EMAIL_ADDRESS>
It should support both x86 and arm on Linux

Just getting back to this, but unfortunately a musl vs glibc dependency won't work for us long term. While all our built docker images are Alpine, we generally test locally on our machines which use glibc. It would be great if we could install one dependency that is universal.
As I mentioned in a previous post, that should be doable with Go and build tags. Yes that means downloading both the musl and glibc libraries when building, but I don't think that's a major issue.

Gotcha. I'll create a universal version to try this approach and ping you once ready


Wrong type inference for inherited static class factory

TypeScript Version:  3.4.0-dev.20190228
Search Terms:
static property this as constructor type inference (in any order/inclusion)
Problem:
Let's say, we have a class that must be instantiated using a static factory function:
class Test {
  private constructor(public x: number) {}

  static create(x: number) {
    // As `this` here is of type `typeof Test` (constructor), this code is perfectly valid.
    return new this(x);
  }
}

Then:
class TestChild extends Test {
  // Here, we introduce a property only `TestChild` has to trigger an error later
  get xIsNumber() {
    return typeof this.x === 'number';
  }
}

const testChild = TestChild.create(2);

testChild; // TS tells that testChild is of type `Test`.
// Actual console output:
// ▶ TestChild {x: 2}

console.log(testChild.xIsNumber); // TS tells that "Property 'xIsNumber' does not exist on type 'Test'."
// Actual console output:
// => true

Expected behavior:
testChild should be of type TestChild, because the static method create uses realtive this to construct the instance of a class. When the method create is invoked from the TestChild, it uses TestChild's constructor as this, not Test's. For TestChild, this in the inherited method create is of type typeof TestChild.
Actual behavior:
However, TS has solidified the return type as Test for some reason.
Therefore, testChild is of type Test.
Playground Link:
https://www.typescriptlang.org/play/#src=class Test {
  constructor(public x%3A number) {}

  static create(x%3A number) {
    return new this(x)%3B
  }
}

class TestChild extends Test{
  get xIsNumber() {
    return typeof this.x %3D%3D%3D 'number'%3B
  }
}

const testChild %3D TestChild.create(2)%3B

console.log(testChild.xIsNumber)%3B
Related Issues: Found none...

You can easily get this to work using a generic type parameter to the create function and a type annotation for this.
class Test {
  constructor(public x: number) {}

  static create<T>(this: new (x: number)=> T, x: number) {
    return new this(x);
  }
}

class TestChild extends Test{
  get xIsNumber() {
    return typeof this.x === 'number';
  }
}

const testChild = TestChild.create(2);

console.log(testChild.xIsNumber); // ok

@dragomirtitian, this is a perfectly valid workaround, yes. But this workaround requires to retype constructor in the factory every time there was a change to this constructor. And if the factory is introduced to the inheritance chain sometime later (in hypothetical ChildOfTestChild, for example), refactoring this can quickly become a pain.
The issue is, anyway, related to the implicit type inference. In this workaround, however, the constructor's return type is explicitly stated, thus the resulting type not being a properly inferred one anyway.
This can be demonstrated by this code:
class Test {
  constructor(public x: number) {}

  // This example shows that TS cannot properly infer the return type of a constructor,
  // because type argument `U` falls back to `Test` in any case, whereas type `T` is inferred correctly
  static create<U extends Test, T extends new (x: number) => U>(this: T, x: number): U {
    return new this(x);
  }
}

/// ... TestChild here ...

const testChild = TestChild.create(2);

console.log(testChild.xIsNumber); // Error here again

Thus, the offered workaround kinda explicitly "shows" TypeScript where to look for the constructor's return type, which only demonstrates the issue in question: lack of type inference.

Duplicate #5863

Oh. Okay.
Gonna add my case to the issue then.


Specify items schema for customNodeLabels

What this PR does / why we need it
Adds the missing schema for items of the customNodeLabels array.
Depends on https://github.com/giantswarm/schema/pull/2
Towards https://github.com/giantswarm/roadmap/issues/1181
Checklist

[x] Update changelog in CHANGELOG.md.

LGTM but can you kick off the tests again please? Not sure why they're failing.

Error: values don't meet the specifications of the schema(s) in the following chart(s):
cluster-gcp:
Could not read schema from HTTP, response status is 404 Not Found

Looks like the schema was not available yet when the test ran.

@AverageMarcus I never know what to do about those "create" ad "upgrade" tests here. Do I have to run them?

/test all

/test create
/test upgrade

@marians Those are our e2e tests that run in tekton. You should run them and the commands to run them should actually already be part of the PR template.


Sharp Support for Bun install

Due to the use of custom scripts, Bun install is currently not compatible.
The bun runtime is already compatible with sharp:

#3511

This work in sharp could resolve this:

#3750

This work in bun could help resolved this:

https://github.com/oven-sh/bun/pull/4263

Please subscribe to https://github.com/lovell/sharp/issues/3750 for updates relating to the removal of the custom install script.


Address Tx table column improvements

Change 'Amount XEC' to 'Amount' (header)
Give breakpoint priority to 'Amount' rather than 'Inputs' and 'Outputs' on smaller screens
Right-align 'Amount' column
Introduce some sort of colour/font/indicator change to make it easy to see at a glance that the tx was a net positive or negative to the address balance

added


TypeScript support (squashed)

Closes systemjs/issues#454
This is just #366 rebased on master. Have squashed things for convenience, but evidently messed up something - the transpiler is not loading correctly.
NOTE: @guybedford abusing closes here so that waffle groups things correctly, refs doesn't work, so watch out with merging

Awesome, thanks for this @crisptrutski. Finishing off the merge manually.


Modify Getting Started to recommend the 5.10 toolchain temporarily.

Due to a Swift compiler bug in the main branch, swift-testing cannot currently be used with it. Until that issue is resolved, we should direct developers to the 5.10 toolchain which is still a happy path.
We can revert this change once the Swift compiler issue is resolved.

@swift-ci please test

Thanks! Can you apply the same recommendation to CONTRIBUTING.md ?

@swift-ci please test

@swift-ci please test


Created specs for most of the built in Dapps including rpc specs

This added specs for most of the built in dapps

Pretty awesome works! Thanks! :tada:


[WIP] Simplify indexed fields

Adds types for indexed fields
class DocumentExample(Document):
   example1 : Indexed(str, pymongo.ASCENDING)

would become
class DocumentExample(Document):
   example1 : Indexed(str, pymongo.HASHED)

would become
class DocumentExample(Document):
   example1 : HashedIndex[str]

The names of the types are a bit confusing, as they're indexed fields, suggestions?
Do you like this API?
This solves #105
This could be extended for GeoJSON GeoSphere and Geo2D indexes, but that example is a bit more complicated

Can this PR be fixed to pass the test and get merged at some point ?

Hey @jsenecal
I forgot about this. Yup, this is a good one. I'll pick it up soon


dns_filter: Initial commit for dns_filter skeleton

Description: This is the skeleton for a DNS filter. This filter attaches to a UDP listener and responds to DNS queries for configured domains.
The filter is marked as WIP.
 This code will consume a minimal configuration, parse, and and store it internally. It does not handle any network data at the moment.
We plan on adding request/response handling along with unit tests, and extending the filter to perform recursive queries in subsequent pull requests.
Signed-off-by: Alvin Baptiste<EMAIL_ADDRESS>Risk Level: Medium
Testing: unit tests
#Issue: https://github.com/envoyproxy/envoy/issues/6748

@sumukhs Can you take a look and suggest any other changes?  Thanks.
I closed https://github.com/envoyproxy/envoy/pull/10331 because merging form master made the commit list quite large.

I closed #10331 because merging form master made the commit list quite large

In the future please don't do this as we lose all the review history. All commits squashed during merge. Thank you!

I'm assigning this over to @sumukhs for review for now. Once approved I'm happy to take a look. Thank you!

/retest

clang-tidy was recently fixed and is now failing. Some of the errors are legit but the exception ones I think are bogus and will need to be fixed in master. cc @lizan @derekargueta
/wait

LGTM. Thank you


Inconsistent Failed to instrument test executable error

Describe the bug
Running on WSL 1 (ubuntu), and sometimes tarpaulin works fine, but most of the time the tests run successfully and then I receive this error:
ERROR cargo_tarpaulin: Failed to get test coverage! Error: Failed to run tests: Failed to instrument test executable
Error: "Failed to get test coverage! Error: Failed to run tests: Failed to instrument test executable"

To Reproduce
Run tarpaulin under WSL 1
cargo tarpaulin --frozen --locked --out=html --no-fail-fast --follow-exec

I am running in a Cargo workspace, not sure if that affects tarpaulin.
Often after running the same command 5 or 6 times will finally result in tarpaulin succeeding once, sometimes tarpaulin never will.

I've recently done a lot of windows improvements, in the latest tarpaulin release. Not sure if they'll hit WSL but if you have time to test it again I'd be interested in hearing back!

I've recently done a lot of windows improvements, in the latest tarpaulin release. Not sure if they'll hit WSL but if you have time to test it again I'd be interested in hearing back!

Great work! Running
cargo tarpaulin --frozen --locked --out=html --no-fail-fast --engine llvm

has worked beautifully for me.
Thanks for all your work!


ARM Template missing Settings

Need to add:

WEBSITE_CONTENTAZUREFILECONNECTIONSTRING
WEBSITE_CONTENTSHARE

Same issue second user -- this appears after deployment -- also behavior after deployment isn't consistent with documentation -- returns a SAS token for some non-existent container.


Add use_scm_version=True to setup.py

Should fix https://github.com/conda-forge/h5netcdf-feedstock/issues/42 for newer releases.

[ ] Closes Issue #xxxx
[ ] Tests added
[ ] Changes are documented in CHANGELOG.rst

@basnijholt Thanks for taking care. I've added a PR with possible fix over at https://github.com/conda-forge/h5netcdf-feedstock/pull/44.
For h5netcdf I think moving to a pyproject.toml-only approach discarding setup.py and setup.cfg seems appropriate. I'll add the necessary changes later this week.

The issue is resolved over at conda-forge with https://github.com/conda-forge/h5netcdf-feedstock/pull/44.