Convert all Commands to Functions?

from
https://autohotkey.com/v2/v2-changes.htm

All commands can now be called as functions, except for control flow statements such as Return. The translation rules are:
If the command's first parameter is an output variable and the second parameter is not, it becomes the return value. Otherwise, RunWait and SendMessage return the value of ErrorLevel, and any other commands which set ErrorLevel return 1 on success and 0 on failure. ErrorLevel is still set as usual.
The remaining output variables are handled like ByRef parameters, except that an exception is thrown if the function is called incorrectly. Passing an empty string is equivalent to omitting the parameter, but any other value (which is not a variable reference) is not allowed.

So:
before:
WinGetTitle, OutputVar, A

after:
OutputVar := WinGetTitle("A")

This is only my personal preference, and perhaps an option could be provided to the user

in v2-a081, the optional comma after the command has been removed, and it is no longer optional, it is no longer allowed at all. So the command must be followed by a space.

I agree, the function syntax seems more familiar and readable.


Android devices ALWAYS prompt user for Write permissions (even when unnecessary)

Only some paths require write permissions to be explicitly prompted for--e.g. external storage, as is prompted here.
I am honestly unsure the actual use case here, but the request for permissions should be:

Only performed when actually required, e.g. a path outside the typical save area is specified.
Should only be shown when the SAVE action is taken--not upon initialization.

I've tried to find any fork that fixed that issue but couldn't succeed so I created my own containing only this fix.


Error 318 and 906

Even after following the troubleshooting section, it is not possible to execute the backup script.
I'm on bios 906 (laptop has been shipped with it), haven't tried any other script.
Also, in case of hardbrick, will the insydeh2o usb rescue procedure work (https://www.bios-mods.com/bios-recovery/insyde-bios-recovery/)?

If it is shipped with this BIOS it is a relatively new machine Wich means
that the scripts may not work! Also for a hardbrick I do not really know
what you can do! A hardbrick is always a shity situation! :/
Thomas Sermpinis (a.k.a. Cr0wTom)
Cell: +30<PHONE_NUMBER>
Email<EMAIL_ADDRESS>http://cr0wsplace.wordpress.com
(Sent from an android)
On Sun, Mar 31, 2019, 2:50 PM Nicola<EMAIL_ADDRESS>wrote:

Even after following the troubleshooting section, it is not possible to
execute the backup script.
I'm on bios 906 (laptop has been shipped with it), haven't tried any other
script.
Also, in case of hardbrick, will the insydeh2o usb rescue procedure work (
https://www.bios-mods.com/bios-recovery/insyde-bios-recovery/)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Cr0wTom/Mi-Notebook-Pro-Mods/issues/3, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AGx8U0uFukGq7ieRP46fkEqKAWoMVwidks5vcK74gaJpZM4cUStT
.

@Cr0wTom , I think you don't wanna leave those informations in the wild !!!


Update to Next.js 12.x features

Summary
We're waiting on next to release v12.1.7 so we can get some bug fixes in:

package.json error on npm install
move  from _app.js to _document.js

when this new version is released, let's upgrade and make the necessary changes and verify package.json error is no more
Job stories
What value will this deliver?

As a developer, I want to declare my Script tags inside my _document.js file, so I can be in compliance with next standards and have a more structurally sound application.
As a developer, I don't want to run into errors when doing an npm install

Acceptance criteria
What requirements must be met to complete this request?

Package.json error is not present anymore (if it is, let's open another ticket or reopen the old one)
Scripts tags are now located in _document.js file instead of _app.js and commonjs script still loads as expected

@jdharvey-ibm tagging you in case you want to add a better explanation to the packagejson error 😅

Hey team! Please add your planning poker estimate with ZenHub @alisonjoseph @andreancardona @francinelucca @jdharvey-ibm

@carbon-design-system/carbon-platform-devs I see the web app is now using Next ^12.3.2. It looks like we're no longer blocked to move scripts to a _document.js file if that's needed to close this issue.
Or, maybe we wait until Next 13 to take advantage of the new app directory?
https://beta.nextjs.org/docs/upgrade-guide#migrating-from-pages-to-app
https://beta.nextjs.org/docs/routing/fundamentals

I can take this and move the script so we can close this. Unless we want to hold off until we move to 13 and move to /app directory? It's currently in beta and they don't recommend using in production... CC: @jdharvey-ibm
https://beta.nextjs.org/docs/upgrade-guide#migrating-from-pages-to-app

Staying on 12 makes sense to me.

We definitely want to move to 13 soon since we have a dependabot PR now for that, but I agree that we can hold off with using the app folder for now. Should we use this issue to track the document.js changes as well as the effort to migrate to v13?

Could also close out this one and open a new issue to track the migration effort from 12 to 13.

https://nextjs.org/docs/upgrading
Actually looks like we're blocked there until Carbon supports React 18.

Worked on this a bit before EOY and was having trouble loading the script from _document.js , need to look further into it


QuickPay v10

This patch adds support for QuickPay v10 - an entirely new platform with a brand new API. It retains support for v4 to v7 (current AM version) and the update should be completely transparent for existing users.
The v7 platform is now deprecated in the sense that it will not receive any feature updates (bugs and security issues - if any - will naturally be resolved).
The v10 platform is already ahead with some important features like support for new and popular acquirers and a full-blown RESTful API - and more features will be added in a steady stream. New merchants will be created on this platform, so this is of course a priority update to us and our customers.
I'm always available for answering any questions you might have.

@ntalbott This PR is the successor of #1649 - Thx

+1

Added some inline comments RE formatting; overall looks pretty solid.

@ntalbott Thank you for the review - I've now incorporated the changes pointed out by you apart from requires! method (please see inline comment).

Hmmm, good point on requires!. I'll see if I can come up with a good fix when merging.

Thank you for responding so swiftly!

:tada: soon?

Can confirm that we're busy - I'm unlikely to get to this this week.

+1

Any ETA on this?

Is there any news, our Shopify shop only missing this to proceed. Please fix soon! :-)

Our integration goes through the v7 API; when I updated the code to reference this PR and ran our remote integration tests, I got the error "md5check did not match expected value" when attempting to capture an existing authorization. All the other options worked, and when I go back to the old v7 integration everything passes - is that change intentional?

@girasquid: No, that was not intentional. I've added some commits to this PR that fixes this and a few other minor issues. Thanks for testing and reporting - remote tests are now all green in my checkout.

Looks good now - can you squash this?

@girasquid: hereby squashed my changes to a single commit - ok?

I'm still seeing 10 here - do you need to push?

@girasquid: oh, you want me to squash into f6b9d73 ?

Into 78f5897148e37a9c3bd8508c98d6cc76e9ec5c91 - this entire PR upgrades the Quickpay integration to support v10, so I think we can keep it all in one commit.

@girasquid: +1, is now squashed into a single commit. Anything more I can do?

All done - thanks!

Waiting for this update to be released by QuickPay, our shop only need this! :+1:

@AlonsoGonzale: Technically we are waiting for Shopify to pull the updated AM into their applications ;-)

@ta There is a branch where I can test to integrate the v10 implementation ? And is the 3ds working with active merchant ?

@UgoMare: The code is merged into AM master (https://github.com/activemerchant/active_merchant/commit/44b47b6cb67cc879e47429138f98ea67c5bc0118). No support for 3D-Secure as of yet, sorry.

any news on this one?

Who are we waiting for? SHOPIFY or QUICKPAY? Does it take 4 months to make these fixes? :-)
Time is money...

Hi @GonzoDK, Shopify is working on this, v10 should be supported soon.

@andrewdunbar please let us know where to ship all the beers to, if "soon" is defined as "soon" :)

ETA?

Still working on it. I'll update this when it's ready to go.

Any news?

@stentebjerg: YES - v10 is live and in production at Shopify :)

Hi @ta. Just activated the v10 platform on the shop, but I can't see the Mobilepay anywhere and it's activated in QuickPay. Do I need to do anything else?


⚠ Bofrost.lu has degraded performance

In f2fc6bf, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1275 ms

Resolved: Bofrost.lu performance has improved in 1d465d9 after 8 minutes.


Getting java.lang.NoClassDefFoundError: io/vertx/core/file/FileSystemException

Hi I wrote a simple http server in Groovy.
However I am having some unexpected behavior when  running the server with maven, and without.
This is the repo to reproduce everything I'm going to describe https://github.com/michaelpog/vertx-http-server-groovy (https://github.com/michaelpog/vertx-http-server-groovy.git)
First of all when I run the HTTP server using a maven plugin, I'm getting this stack trace in the end:
mvn clean install
Exception in thread "Thread-13" java.lang.NoClassDefFoundError: io/vertx/core/file/FileSystemException
	at io.vertx.core.file.impl.FileSystemImpl.deleteInternal(FileSystemImpl.java:585)
	at io.vertx.core.file.impl.FileSystemImpl.deleteRecursive(FileSystemImpl.java:212)
	at io.vertx.core.impl.FileResolver.deleteCacheDir(FileResolver.java:315)
	at io.vertx.core.impl.FileResolver.lambda$setupCacheDir$1(FileResolver.java:304)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.ClassNotFoundException: io.vertx.core.file.FileSystemException
	at org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy.loadClass(SelfFirstStrategy.java:50)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.unsynchronizedLoadClass(ClassRealm.java:271)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.loadClass(ClassRealm.java:247)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.loadClass(ClassRealm.java:239)
	... 5 more

The plugin simply runs the http server Groovy script.
Looking at the exception it looks like it is trying to access the file system and it's failing. I don't want to / need to use the file system.
I looked through the documentation http://vertx.io/docs/vertx-web/java/#_disabling_file_caching_on_disk and tried everything
I started by simply running the http server directly in my IDE, and I see that after every time I exit the application I see a new folder created in my src directly under .vertx/ the name is something like file-cache-xxxxxxx...
I looked at this part of the documentation:
"To disable file caching you can provide your vert.x options the property fileResolverCachingEnabled to true"
And was completely confused, setting "fileResolverCachingEnabled" to true is supposed to disable? or enabled? I tried both:
def options = new VertxOptions();
options.setFileResolverCachingEnabled(false/true);
def vertx = Vertx.vertx(options)

Nothing really changed, every time I run the server in the IDE, and exit I get a new cache directory. And in maven I get the same stack trace dump.
After having a discussion about it here:
https://groups.google.com/forum/?fromgroups=#!topic/vertx/golo_ZibZts
Thomas suggested to use this flag -DdisableFileCPResolving (I'm guessing it's a system property for java), I tried that, and tried -DdisableFileCPResolving = true, nothing changed.
In fact I could not even find that documented anywhere, but I saw another flag documented:
"For backwards compatibility it will also default that value to the system property vertx.disableFileCaching"
Tried that and also nothing worked, regardless I need some programatic solution because the maven plugin I'm using is running the Groovy script directly from maven and doesn't pass any JVM properties.
Can you please help us  with all the above issues I'm having?
Thanks in advance

Has anyone had a chance to look into it?

can someone provide a reproducer ? I would be happy fixing this :-)

ah there is one linked :-)

I will have a look

this issue happens when the script ends when the JVM exits.
if you add a pause at the end of your script, it runs, for instance I've added:
    server.requestHandler(router.&accept).listen(8888)
    // Prevent the JVM to exit
    System.in.read()

it seems there is a classloading issue that prevents the shutdown hook to cleanup the cache dir. You can disable the cache dir with the vertx.disableFileCPResolving property.

The classloading issue may be related to the fact that it happens when the JVM shutdowns, perhaps that the ClassRealm class refuses to load some classes at this moment. If I set a reference to this class in the script it works ok.

Also you can try close the Vert.x instance in a try/finally block of your script, that will eagerly delete the directory without the shutdown hook.

@vietj thanks for looking into it, I'm trying to understand the suggestions that you provided:

vertx.disableFileCPResolving I am not sure where it's documented and how to use it. Thomas suggested -DdisableFileCPResolving you suggest  vertx.disableFileCPResolving, I just tried running with -Dvertx.disableFileCPResolving, doesn't seeem to make any difference. And I don't know how to even pass that to the maven groovy plugin.

I tried wrapping with try finally, it doesn't make any difference, neither when I run the server directly, nor when I run it from maven.

have you set a pause in your script ?

Yes I did, and It doesn't make any difference.
server.requestHandler(router.&accept).listen(8888)
System.in.read()

So now when I run mvn install, it stops and then when I press any key, mvn continues and I see this
Exception in thread "Thread-2" java.lang.NoClassDefFoundError: io/vertx/core/file/FileSystemException
	at io.vertx.core.file.impl.FileSystemImpl.deleteInternal(FileSystemImpl.java:585)
	at io.vertx.core.file.impl.FileSystemImpl.deleteRecursive(FileSystemImpl.java:212)
	at io.vertx.core.impl.FileResolver.deleteCacheDir(FileResolver.java:315)
	at io.vertx.core.impl.FileResolver.lambda$setupCacheDir$1(FileResolver.java:304)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.ClassNotFoundException: io.vertx.core.file.FileSystemException
	at org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy.loadClass(SelfFirstStrategy.java:50)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.unsynchronizedLoadClass(ClassRealm.java:271)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.loadClass(ClassRealm.java:247)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.loadClass(ClassRealm.java:239)
	... 5 more```

what I mean is this happen when the JVM exits and if you don't have a pause your server will be shutdown.
have you tried to close the created Vertx instance in a finally block ?

Ok that made some difference.
server.requestHandler(router.&accept).listen(8888)
    System.in.read()
}finally {
    server.close()
    vertx.close()
}

Now It works fine I don't see anything felt in the .vertx directory (not using any additional flags and maven doesn't dump that stack trace.
That's good progress however it's not a full solution since I need that System.in.read() to prevent from calling the finally and close the server right away, and mvn plugin is run by a Continuous Integration server and there's no human to press any key to move to the finally part.
On the other

the System.in pause seems necessary because of the maven plugin. For instance if you look at this netty example you can see it blocks as well https://github.com/netty/netty/blob/4.0/example/src/main/java/io/netty/example/echo/EchoServer.java

@vietj Thanks again for helping out with this.
I'm not sure I understand how we are expected to send anything into the standard input of an application ran by maven on a Continuous Integration server in an automated build context. And I also believe that we are not addressing the problem here.
The problem/bug that we need to address here is the fact that there's no way to turn off Vert.x writing to filesystem.
The solution should be a working configuration that I can set to VertxOptions that will prevent Vertx from accessing filesystem.
In fact it seems like this http://vertx.io/docs/apidocs/io/vertx/core/VertxOptions.html#setFileResolverCachingEnabled-boolean- is supposed to do exactly that, but it's not working.
This is the bug, this should be solved.
Adding delays in code is not the solution.
Can any of your developers please fix this bug?

@michaelpog I think there is confusion here.
I am just saying that if you don't add a pause, then your server will just not be launched because the groovy will exit even before the http server is started, making your integration test fail.
I said to use System.in.read() as pause but probably that in your case it should be a different pause, that prevents the script to exit until your test has finished to use the script.

besides in your case, you should close the vertx instance in a finally block which will properly close the file system, because the JVM exit hook set by Vert.x seems to fail to execute properly due to what I suspect the shutdown procedure of the maven plugin / classloading.
that being said I have not said that there is no bug related to filesystem caching.

the enableCaching flag does not control wether the .vertx directory is created or not. It control wether or not caching is using during file resolving.
the .vertx directory is used by the file cache and the classpath resolver (in .vertx/file-cache-$UUID). So to be sure that the .vertx dir is not created you should disable both caching and CP resolving

Ok thank you @vietj I understand what you mean, and if I do want to use caching then I will have to figure out a way to shut down the server in the finally. And I am sure there's a bug because even when I'm not using maven, and I run the vertx server in the IDE and then shut it down, I see the .vertx directory still remains.
So to be sure that the .vertx dir is not created you should disable both caching and CP resolving

So to my original question: How do I do that?
How do I disable caching and CP resolving?
How do I do it in code so that I can run the http server from maven, and I will be guaranteed that filesystem will not be accessed?

@michaelpog you should set the system property of the JVM : vertx.disableFileCPResolving=true and vertx.disableFileCaching=true, the later can also be set on the VertxOptions with fileCachingEnabled=false. Probably we should have something similar for the former.
Some other vertx projects uses the .vertx directory (like the http service factory).
I'm thinking that this directory should be retrieved from VertxInternal in a unified way and could also be a directory in the /tmp dir instead.

@vietj that worked. It would be great if this could be documented somewhere in your official documentation, and also it would be helpful if both could be set programatically.
Thank you for your help

@michaelpog will address this


Link to the docs on the Cloud Compute desktops from the policy page

This makes it easier to find for new contributors, since foundation.rust-lang.org seems to have better SEO than forge.

Ideally it would also be nice to link to it from the blog post: https://foundation.rust-lang.org/news/2021-11-16-news-announcing-cloud-compute-initiative/


Add report sending to DEV menu.

Add report sending to DEV menu.

Not needed, can be done from Contacts


CRAYSAT-1798: Preserve all port policies in sat swap

Summary and Scope

CRAYSAT-1798: Preserve all port policies in sat swap
When enabling or disabling a switch or a cable in sat swap switch or
sat swap cable, properly handle when a port has multiple port policies
applied to it. This fixes the issue where only the first port policy
associated with a port is kept through the disable/enable cycle.
The strategy for keeping a record of the port policies applied to each
port is unchanged. That is, for the disable action, for each port
policy associated with the ports to be disabled, append the
"sat-offline-" prefix to the policy name, create a new policy
with the prefixed name which has "state": "OFFLINE", and then apply
the policy to the port being disabled.
Note that for ports with more than one policy associated with them (e.g.
fabric ports which have a fabric policy and a qos policy), this will
result in multiple policies with the same "state": "OFFLINE" data
being assigned to those ports. According to the Slingshot team, this is
acceptable and should not cause issues.
Fix and improve the unit tests for the get_switch_port_data_list
method of the PortManager.  This now tests the method for a switch
with just edge ports, a switch with just fabric ports, and a switch with
both fabric and edge ports. It also does more consistent mocking by
mocking the get_port method instead of directly mocking the
FabricControllerClient class.
Fix unit tests for PortManager.get_jack_port_data_list. Fix the
swap_component unit tests for the CableSwapper and SwitchSwapper.
Fix minor issues in tests including PyCharm code inspection type
warnings and faulty assertLogs usages.
Test Description:
The following will be tested on baldar:

sat swap switch --action disable on an enabled switch
sat swap switch --action disable on an disabled switch
sat swap switch --action enable on an disabled switch
sat swap switch --action enable on an enabled switch

The same will also be tested for sat swap cable.

CRAYSAT-1798: Remove unused disruptive argument from swap_component
The disruptive argument was not used in the swap_component method of
the Swapper class. The value of the disruptive command-line option
is checked in the function check_arguments in sat.cli.swap.main
before calling the appropriate function for the cable, switch, or blade.
Test Description:
Unit tests pass. PyCharm is happier.

Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[x] HPC Product Announcement prepared, if applicable

I'm opening this for review, but I still have some testing to perform for sat swap cable, and I have to update the changelog still as well.

I have finished the testing of sat swap cable on the same system where I tested sat swap switch, and I have updated the CHANGELOG and declared a new version. This is ready for review and then merge.


Closes #1000735.

Compiler occiasionally throws duplicate error messages and we don't see it because console reporter ignores them.
We have to do the same.

Looks good to me! Please merge it in both master and 2.0.x

Hubert, has this been merged?

No, I will do it (along with finishing the other branch) when I find some extra time.

I just rebased it to master and checked that all tests pass with the current trunk. Hubert, do you want to do anything else, or I can merge it to master?

It is fine, you can merge it.


Agency logo update

@marcincichon @RobertWDLowe could one of you check when you have a chance. Thanks :)

looks good to me, @brown-a2


Update openstack-ansible SHA

This commit updates the openstack-ansible SHA in preparation for the
next release.
openstack-ansible has changed the way Python package constraints are
applied, see 7055f795095d8e19c052371259ca77bf42c3095f, to address an
issue where they were being applied incorrectly. This change means the
version of elasticsearch specified in the elasticsearch role is no
longer honoured by the repo_build role. This commit sets the var
repo_build_upper_constraints_overrides to allow the requirements of the
role elasticsearch to be met.
The following issues are addressed by this change:
Connected https://github.com/rcbops/rpc-openstack/issues/1119
Connected https://github.com/rcbops/rpc-openstack/issues/1201
Connected https://github.com/rcbops/rpc-openstack/issues/1211

Confirmed the following fixes are in the OSA SHA:
876b25 Provide ability to override upper-constraints
5cb4b9 Better control of mariadb restarts in k>l upgrades
bf8d8f Clean tempest dir before cloning.
a940f1 Fix config entry regression introduced in container restart patch
7055f7 Fix setting of upper_constraints fact

Which together address the 3 issues referenced above.
Also confirmed that providing an override for elasticsearch meant that the correct package version got built in the repo (i.e. was not overridden by global constraints):
-rw-r--r-- 1 nginx www-data 58336 Aug  1 09:20 /var/www/repo/pools/elasticsearch/elasticsearch-2.0.0-py2.py3-none-any.whl
Pending jenkins upgrade test completing successfully, I am 👍

👍


Cookie “__preview” has been rejected because it is in a cross-site

When I try to access my environment from sanity studio using sanity-plugin-iframe-pane. It seems we need a cookie set but that will not work in an iframe. Is there an alternative way to give content editors preview in the sanity studio?

The latest major version of this package is geared towards using Loaders and the Presentation tool to achieve live, visual editing of your storefront.
You can learn more in the Visual Editing section of Sanity's documentation


Support hotkeys from osu

When I open the browser-bar window I'd love to be able to toggle mods with the same hotkeys as ingame

Implemented in commit f6a662cc3cdcca15852107823be54b36448c3d49, available in version 1.3.2.


Update qcodes to support numpy 2.0

Numpy 2.0 is scheduled to be released this year (https://github.com/numpy/numpy/issues/24300). In this PR we adapt qcodes to work with numpy 2.0.
The numpy_non_concrete_floats_instantiable and numpy_non_concrete_complex_instantiable are now empty, perhaps it is better to just remove them. (they are public, but not used in the qcodes codebase)

The numpy_non_concrete_floats_instantiable and numpy_non_concrete_complex_instantiable are now empty, perhaps it is better to just remove them. (they are public, but not used in the qcodes codebase)

They are added to numpy_floats and numpy_non_concrete_complex_instantiable I think its better to remove them and add a breaking change notice.


Rename doctest_impl.h to doctest.cpp for less confusion.

Description
doctest_impl.h was not really a header file, it contains ODR definitions.
It was meant to be included by one and exactly one implementation file.
It was confusing for contributors. Now it is more clear.
Catch2 names its implementation files .cpp.
As a consequence, we simplified the creation of doctest with main.
Extra file doctest_main.cpp is not needed.

Good idea - thanks!


Minor fixes

Fix bug so API.volume(stream, volume) can actually set the volume.
Send output in android.exectue() though json.loads, so that when this is a dictionary, one actually gets a dict and not str. Using a try-except block for when this is just a string.
My editor also removed some whitespace and eol.

Thanks


Add public selector

Add a jQuery like selector :)
The link is Link

yes, added 'Ω' as selector. use Alt+234.  Committing that now


Add tvOS, watchOS targets

First time adding these targets, so a thorough review is appreciated. Everything appears to build, and tests pass on the new tv test target. I looked at Argo's setup as reference.

Can we also update bin/test to exercise all 4 targets? You can see an example of that in Argo

This CI failure is expected because of #79
👍


Setup LetterOpener gem

This handles the testing of emails for us locally without requiring us to use gmail or something in our development environment. https://github.com/ryanb/letter_opener

Closed by #45


Clean up / improve C++ code

I don't have a lot of experience with C++ so the C++ code is probably pretty awful. Someone with more experience could probably improve or clean up a few things.
Files in question: Everything in this directory
Comment if you want to work on this

Bump - Hacktoberfest just started


TSVD() as compression algorithm produces method error

Hello,
I'm trying to use HMatrices to compress a Matrix using TSVD. I followed the tutorial from the docs page and swapped out the compression algorithm PartialACA() for TSVD() but get the following error :
MethodError: no method matching (::TSVD)(::HMatrices.PermutedMatrix{…}, ::ClusterTree{…}, ::ClusterTree{…}, ::HMatrices.ACABuffer{…})
not sure what to do, any help is appreciated.
using HMatrices, StaticArrays

const Point2D = SVector{2,Float64}
# points on a circle
m = n = 10_000
X = Y = [Point2D(sin(i*2π/n),cos(i*2π/n)) for i in 0:n-1]

struct LaplaceMatrix <: AbstractMatrix{Float64}
    X::Vector{Point2D}
    Y::Vector{Point2D}
end

Base.getindex(K::LaplaceMatrix,i::Int,j::Int) = -1/2π*log(norm(K.X[i] - K.Y[j]) + 1e-10)
Base.size(K::LaplaceMatrix) = length(K.X), length(K.Y)

# create the abstract matrix
K = LaplaceMatrix(X,Y)

Xclt = Yclt = ClusterTree(X)
adm = StrongAdmissibilityStd()
comp = TSVD()

H = assemble_hmatrix(K,Xclt,Yclt;adm,comp,threads=false,distributed=false)

Hey @maltezfaria, no worries we are all busy nowadays :)
I ran ] update HMatrices and  the code above and it still produces the same error :/
I'm using it to compress a mutual inductance matrix of a electrical cable. The geometry is rather fixed so I only have to compute it once. A colleague of mine insisted I use this method for better accuracy of the compression.
Cheers,
Roel

You need to be on the main branch since I have not (yet) tagged a new version with the fix. ] update will only update to a stable version.

New tagged version is up. Running ] update should work now. Let me know if that is not the case.
Regarding your application: why not use the adaptive cross approximation (ACA) compression method in HMatrices (the default)? It is usually much faster than e.g. SVD, at the cost of being less robust...

Thanks for updating, I did some testing and while ACA compression is indeed much faster the TSVD compression has superior accuracy. As I said, I only have to compute it once so its worth the wait.

Fair enough, but keep in mind you can pass a tolerance parameter to all of these methods, so you can probably get good results from ACA by passing a more strict tolerance. But if TSVD suffices for you, great 👍 . I just never optimized it since I use that mostly for validation purposes.

One last thing in defense of TSVD I guess: it is the optimal low rank approximation (in Frobenius or spectral norm) and the only guaranteed method in HMatrices.


xywh_to_xyxy() crashes if 'xywh' is a numpy array: 'NameError: name 'xyxy' is not defined' (proposed fix included)

xywh_to_xyxy() crashes if 'xywh' is a numpy array: 'NameError: name 'xyxy' is not defined' (proposed fix included)
Python 3.6.13 with same packages as in the EPro-PnP-6PoF README.md
If I pass a numpy array to xywh_to_xyxy() it will crash with an 'NameError: name 'xyxy' is not defined' exception:
  File ".../EPro-PnP/EPro-PnP-6DoF/tools/../lib/utils/img.py", line 103, in xywh_to_xyxy
    if len(xyxy.shape) == 1:
NameError: name 'xyxy' is not defined

I tried to create a PR, but GitHub wouldn't let me.  Here is a proposed fix:
diff --git a/EPro-PnP-6DoF/lib/utils/img.py b/EPro-PnP-6DoF/lib/utils/img.py
index 3921598..9396b53 100644
--- a/EPro-PnP-6DoF/lib/utils/img.py
+++ b/EPro-PnP-6DoF/lib/utils/img.py
@@ -100,13 +100,13 @@ def xywh_to_xyxy(xywh):
         y2 = xywh[3] + y1
         return (x1, y1, x2, y2)
     elif isinstance(xywh, np.ndarray):
-        if len(xyxy.shape) == 1:
+        if len(xywh.shape) == 1:
             assert len(xywh) == 4
             x1, y1 = xywh[0], xywh[1]
             x2 = xywh[2] + x1
             y2 = xywh[3] + y1
             return (x1, y1, x2, y2)
-        elif len(xyxy.shape) == 2:
+        elif len(xywh.shape) == 2:
             return np.hstack((xywh[:, 0:2], xywh[:, 2:4] + xywh[:, 0:2]))
         else:
             raise

Thank you for your contribution. I'm not sure why PR is not working. I may look into the your proposed fix later, although I do not expect to revise the code of the CDPN network.
Btw, we do not recommend using the CDPN network directly for other applications, as we have not fully optimized the architecture and hyper parameters for EPro-PnP (since it's mostly used for comparison with the baseline in the paper). In fact we recently noted a design fault related to the global weight scale. An improved version with extended paper and code will be released in a few weeks.


[stable/prometheus-blackbox-exporter] Add PrometheusRule support

What this PR does / why we need it:
Adding prometheus rule support.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Thanks! Very appreciated. :)
/assign
/lgtm


Added Let's Encrypt clarification

Added clarification that Let's Encrypt needs to be able to connect to your server to verify the domain
Signed-off-by: Paul Rothrock<EMAIL_ADDRESS>

@crspeller can you help with a dev review, and then we can get this merged (after the one typo fix noted by Amy above)

Thanks! @icelander FYI on two small suggestions above, looks like I don't have push access to your branch to make them myself,

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Friendly reminder to help with the above two suggestions @icelander when you have a moment :)
Otherwise looks like this is ready to merge

Hi @icelander. I've committed the suggested changes as they were related to formatting. Re-requesting reviews so we can move the PR forward.

/update-branch


What happens if user has two subscriptions of the same name?

Dear Developer,
In my application I allow users to upgrade to another plan of the same subscription name while they have an existing subscription running. What would happen to the old subscription? Would it be canceled?
It seems like the newest subscription is the active one. Would the other (old) subscription still be charged?
Also: If a user upgrades, would there be a way to discount the remaining days of the old subscription from the first payment of the new subscription?
This is what it looks like in the DB:
subscriptions table
id: 17
name: main
plan: pro_monthly
ends_at: NULL
cycle_started_at: 2020-09-03 14:37:10
cycle_ends_at: 2020-10-03 14:37:10
id: 18
name: main
plan: flex_monthly
ends_at: 2020-09-03 15:04:32
cycle_started_at: 2020-09-03 14:49:32
cycle_ends_at: NULL
Thank you very much!
Sincerely,
zqstack

Or should I use swap()? How does it work exactly?

I'm affraid I would be charged after one year (unwantedly) :)

Thank you so much! So I need to use swap() only? It seems that it changes my subscription plan, but I did not receive any payment from either credit card or ideal.
And the double subscription, would the old one still be charged? I removed the previous subscription from the subscriptions table.

Thank you so much! So I need to use swap() only?
Yes :)

When would it be charged?
When executing cashier::run the order will be processed. If a payment should be incurred, it will be triggered then. Sometimes the user has enough credit balance or the payment amount is too low for Mollie to accept. Then it will be put on the User's balance.

And the double subscription, would the old one still be charged? I removed the previous subscription from the subscriptions table.
Instead of modifying the table directly, use cancelNow() to cancel subscriptions immediately. Any related scheduled OrderItems will still be processed.

Note that if you're using Mollie's test mode, payments will not automatically reach status paid.

Ah, that clarifies a lot!
What happens if the payment fails? For example, there is not enough money available on their account? Would the subscription be canceled?

Yes


[Compatibility] - MidiQOL

[x] Check for MidiQOL specific key mappings
[x] Decide how to deal with flags from other modules
[x] Advantage attribution on chat messages (not the normal MidiQOL adv attribution

Closing with #79
(more work might be needed).


Windows base image for ballerina

Purpose

$subject

Goals

Support windows platform docker images.

Codecov Report

Merging #215 into master will decrease coverage by 2.66%.
The diff coverage is 7.4%.

@@             Coverage Diff              @@
##             master     #215      +/-   ##
============================================
- Coverage     64.85%   62.18%   -2.67%
- Complexity      117      118       +1
============================================
  Files            13       13
  Lines           569      595      +26
  Branches         53       56       +3
============================================
+ Hits            369      370       +1
- Misses          172      196      +24
- Partials         28       29       +1

Impacted Files
Coverage Δ
Complexity Δ

...allerinax/docker/generator/DockerGenConstants.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...erinax/docker/generator/DockerArtifactHandler.java
46.95% <7.4%> (-8.13%)
15 <3> (+1)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3167718...7b61625. Read the comment docs.


allow_blank required for optional paramaters in serializers

Great, thanks! Fastest merge ever?

No, but probably one of the fastest :+1:


RestAPI doc is still using operationId and we do not have a solid way to guarantee they are the same

The docs team is using operationId to identify operations, and they decided to stick with swaggers.
The downstream SDK generators (at least dotnet SDK) want to provide extra information that our user could use to look up through the REST API docs. The solution that dotnet SDK provides is to write the operationId into xml docs.
TCGC did not pass through this operationId, therefore the downstream SDK generators need a concrete operationId instead of constructing an operationId again. But because of the existence of various name changing decorators, it is impossible for the SDK generators to calculate a correct operationId.
Similar reason that TCGC and the autorest emitter that emits the swagger calculates the operationId independently, therefore I think it is also problematic in this solution, because in independent calculation, there is no guarantee that they always get the same result which leads to an operationId in swagger to mismatch the operationIds in SDKs.
So either, we could convince docs team to use a different identifier for operations so that we could use in the downstream SDK generators.
Or TCGC and autorest could share a piece of code to calculate the operationIds and pass through this to SDK generators to guarantee the operationId for the same operation is always the same.

Having different operation_id is definitely not ideal. As autorest depends on TCGC, it may make sense to have TCGC to expose operation_id and let autorest uses that.
There is still a risk that TCGC one differs from what autorest was generating, but we can run against spec repo to see the diff and correct the logic to be consistent.

So the conclusion is for the immediate solution:

instead using client name to generate operation id, we will use TypeSpec name
scan repo for anything impacted and add explicit operation id override.

This may pending on getting RPSaaSMaster green & breaking change board review. @ArcturusZhang , any timeline pressure here?

So the conclusion is for the immediate solution:

instead using client name to generate operation id, we will use TypeSpec name
scan repo for anything impacted and add explicit operation id override.

This may pending on getting RPSaaSMaster green & breaking change board review. @ArcturusZhang , any timeline pressure here?

No. This issue has been there for a long time therefore its impact is relatively small since it only impacts docs of .Net SDKs


CollapseTransition please

Hi @50rayn thanks for the lib update to vue3, I am just wondering when the "CollapseTransition" will be published and what bug is it now. Maybe I can give some help.

Hello @zuo305 .
I have just release v.1.0.4.
I have done a more detailed research on this transition component and realised that the problem itself was in the documentation.
P.S. Thanks a lot for your contribution and feedback. If any other issues will be discovered, fill free :).
P.P.S. I'll wait your confirmation that everything is ok, then I'll close the issue :)


:construction: [WIP]: Detect solc version in config

Description
Detect solc version from package.json instead.
This feels ripe for bugs so it's a WIP
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #358

PR #359   👈

This comment was auto-generated by Graphite.


Version number of config tool visible

It would be nice to have the application version visible on the screen.
Thx!

And possibly also show a version from the firmware itself?

Added in the latest version.


Intermittent 70 second gaps in metrics

Hi! First, thank you for creating/sharing this tool! It's been very helpful to us on our journey with Prometheus and Google Cloud.
I'm hoping someone can help with an issue we've been running into. I noticed intermittent gaps in our Prometheus metrics that are pulled from the stackdriver-exporter. This occurs about 10 times each day and the gaps are consistently 70 seconds long.
The gaps correlate to Prometheus scrape timeouts of the exporter as well as log messages from the exporter that indicate timeouts during requests to the Stackdriver API, for example:
time="2019-04-24T20:09:52Z" level=error msg="Error retrieving Time Series metrics for descriptor `pubsub.googleapis.com/subscription/num_retained_acked_messages_by_region`: Get https://monitoring.googleapis.com/v3/projects/redacted/timeSeries?alt=json&filter=metric.type%3D%22pubsub.googleapis.com%2Fsubscription%2Fnum_retained_acked_messages_by_region%22&interval.endTime=2019-04-24T20%3A09%3A42.119800239Z&interval.startTime=2019-04-24T20%3A04%3A42.119800239Z: net/http: request canceled (Client.Timeout exceeded while awaiting headers)" source="monitoring_collector.go:182"
time="2019-04-24T20:09:52Z" level=error msg="Error retrieving Time Series metrics for descriptor `pubsub.googleapis.com/subscription/num_retained_acked_messages`: Get https://monitoring.googleapis.com/v3/projects/redacted/timeSeries?alt=json&filter=metric.type%3D%22pubsub.googleapis.com%2Fsubscription%2Fnum_retained_acked_messages%22&interval.endTime=2019-04-24T20%3A09%3A42.119800239Z&interval.startTime=2019-04-24T20%3A04%3A42.119800239Z: net/http: request canceled (Client.Timeout exceeded while awaiting headers)" source="monitoring_collector.go:182"
time="2019-04-24T20:09:52Z" level=error msg="Error while getting Google Stackdriver Monitoring metrics: Get https://monitoring.googleapis.com/v3/projects/redacted/timeSeries?alt=json&filter=metric.type%3D%22pubsub.googleapis.com%2Ftopic%2Funacked_bytes_by_region%22&interval.endTime=2019-04-24T20%3A09%3A42.087937084Z&interval.startTime=2019-04-24T20%3A04%3A42.087937084Z: net/http: request canceled (Client.Timeout exceeded while awaiting headers)" source="monitoring_collector.go:138"

Here's some details from our environment:

Prometheus

version: 2.7.1
scrape interval: ~10 seconds~ 60 seconds (for the stackdriver-exporter target)
scrape timeout: 10 seconds

stackdriver-exporter

version: 0.6.0
timeout: 10 seconds

Are these timeouts and/or metric gaps something that other people have seen? Any other ideas on how to troubleshoot this?

I recently realized that our scrape interval for the stackdriver-exporter is set to 60s (instead of our 10s default for everything else). That helps me understand why there's a 60-70 second gap instead of a 10-20 second one.
I'm still confused about the Stackdriver API timeouts themselves. Is anyone else seeing those when using the stackdriver-exporter? I'm running it from a container with GKE.

I discovered that the Google Cloud Console has metrics for each Google API. I looked at the metrics for the Stackdriver Monitoring API (used by the exporter) and found that these gaps corresponded to API degradation on Google's side (e.g elevated latency and error rates). I asked Google Support about this and didn't get an answer about why this happens but they did give some general guidance for the client side:

try breaking up each request into a set of smaller requests by setting label filters. Otherwise, they can retry timeouts with exponential backoff.

I guess there's something that could be done on the client side to improve this a little bit, but I don't know that it's worth it. The API seems to meeting its 99.95% SLO so I don't know if it's worth trying to optimize beyond that on the client side. I'm ok with a few intermittent gaps in my current use case.


mention slack callback_id

Signed-off-by: Arno Uhlig<EMAIL_ADDRESS>Update docs for slack callback_id after https://github.com/prometheus/alertmanager/pull/1592 is merged

@brian-brazil PTAL

Thanks!


Refactor to use IsCompleted instead of IsCompletedDeployment

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

👍
/lgtm

Here you go
/approve


[postgresql-extensions] Settings trait not thread safe

I'm trying to use the new install command from the postgresql-extensions crate but can't seem to pass the Settings object (regardless if I clone the Settings, reconstruct it or just use a new Settings::default()) without getting this error in my Rocket::launch method:
`dyn postgresql_commands::traits::Settings` cannot be shared between threads safely
the trait `Sync` is not implemented for `dyn postgresql_commands::traits::Settings`, which is required by `{async block@src/lib.rs:78:32: 78:33}: std::marker::Send`
required for `&dyn postgresql_commands::traits::Settings` to implement `std::marker::Send`rustc[Click for full compiler diagnostic](rust-analyzer-diagnostics-view:/diagnostic%20message%20%5B0%5D?0#file%3A%2F%2F%2Fhome%2Fspike%2FDocuments%2Fcoding%2FdRAGon_db%2Fsrc%2Flib.rs)
extensions.rs(136, 77): required because it's used within this `async` fn body
extensions.rs(39, 1): required because it's used within this `async` block
extensions.rs(39, 1): required because it's used within this `async` fn body
db_postgres.rs(130, 80): required because it's used within this `async` fn body
db_postgres.rs(81, 67): required because it's used within this `async` fn body
db_postgres.rs(77, 37): required because it's used within this `async` fn body
lib.rs(78, 32): required because it's used within this `async` block
lib.rs(266, 66): required by a bound in `async_main`

My code:
use pgvector::Vector;
use postgresql_embedded::{PostgreSQL, Settings};
use postgresql_extensions::{get_installed_extensions, install as install_extension};
use semver::VersionReq;
use std::path::{Path, PathBuf};

#[get("/hello/<name>/<age>")]
fn hello(name: &str, age: u8) -> String {
    format!("Hello, {} year old named {}!", age, name)
}

#[launch]
pub async fn start_server() -> _ {
	let database: db_postgres::VectorDB = db_postgres::VectorDB::new().await.unwrap();
	info!("Database initialized");
	rocket::build().mount("/", routes![hello])
}

impl VectorDB {
	pub async fn new() -> Result<Self> {
		Self::with_config(VectorDBConfig::default()).await
	}

	pub async fn with_config(config: VectorDBConfig) -> Result<Self> {
		let storage_dir: PathBuf = Path::new(&config.clone().path).to_path_buf();

		println!("Installing PostgreSQL v{}", PG_VERSION);

		let mut settings = Settings::new();
		settings.password_file = storage_dir.join(".pgpass");
		if settings.password_file.exists() {
			settings.password = std::fs::read_to_string(settings.password_file.clone())?;
		}

		let installation_dir = storage_dir.join("pg");
		let data_dir = storage_dir.join("pg_data");
		settings.installation_dir.clone_from(&installation_dir);
		settings.data_dir.clone_from(&data_dir);
		settings.temporary = false;
		settings.version = VersionReq::parse(format!("={}", PG_VERSION).as_str())?;

		info!("Starting PostgreSQL v{}", PG_VERSION);
		let mut postgresql = PostgreSQL::new(settings.clone());
		postgresql.setup().await?;
		postgresql.start().await?;

		if !postgresql.database_exists(DATABASE_NAME).await? {
			info!("Creating database '{}'", DATABASE_NAME);
			postgresql.create_database(DATABASE_NAME).await?;
		}
		let database_url = postgresql.settings().url(DATABASE_NAME);

		let pool = PgPool::connect(database_url.as_str()).await?;

		// For testing: Remove pgvector extension if it exists
		sqlx::query("DROP EXTENSION IF EXISTS vector CASCADE;")
			.execute(&pool)
			.await?;

		let db = Self {
			pool,
			config,
			postgresql,
		};

		db.setup_pg_vectors_extension(&settings).await?;
		db.setup_tables().await?;
		db.setup_indexes().await?;

		Ok(db)
	}

	async fn setup_pg_vectors_extension(&self, settings: &Settings) -> Result<()> {
		info!("Checking if pg_vectors extension is installed");
		if get_installed_extensions(settings).await?.is_empty() {
			info!("Installing pg_vectors extension");

			install_extension(
				settings,
				"portal-corp",
				"pgvector_compiled",
				&VersionReq::parse("=0.16.12")?,
			)
			.await?;
		}

		self.enable_pg_vectors_extension().await?;

		Ok(())
	}

	async fn enable_pg_vectors_extension(&self) -> Result<()> {
		let query = "CREATE EXTENSION IF NOT EXISTS vector;";
		sqlx::query(query).execute(&self.pool).await?;
		Ok(())
	}
// ...
}

Maybe the fix for this would be making the install method take a trait that must implement Send and Sync to force the Rust compiler to accept that the Settings type is threadsafe? Or wrap it in a Box?
#[instrument(level = "debug", skip(settings))]
pub async fn install(
    settings: Box<&dyn Settings + Send + Sync>,
    namespace: &str,
    name: &str,
    version: &VersionReq,
) -> Result<()> {```

Hello @spikecodes, I am not that familiar with rocket but it looks like for your example you might be able to address the issue by establishing the postgresql embedded instance prior to running rocket, just connect to it using whatever pattern rocket prescribes for connecting to a database.  I created an axum example that illustrates this approach #123.

Hello @brianheineman, thanks for sharing that Axum example. I will try the Rocket-equivalent to see if that fixes the thread safety issues.

@spikecodes are you still encountering issues using this with Rocket?

Hey Brian, I haven't actually tested it recently but I think I remember your suggestion working so I'd say it's safe to close this issue.

Thanks for the follow up @spikecodes!


Add Auto-Update switch to Windows client

What are you trying to do?
It would be good to be able to disable auto-updates for the Windows client
How should we solve this?
An environment variable, empty file for persistence -- checkbox in the taskbar as a stretch goal
What is the impact of not solving this?
Currently I am renaming ollama app.exe to ollama app.exe.disable and manually starting ollama via cmd and ollama serve (not great for restarts)
Anything else?
No response

@sebastianlau the windows tray app does not automatically upgrade, but it does detect an upgrade is available, and download it to get ready.  Once downloaded, it will report that an upgrade is available with a pop-up banner, and if you click on that banner (or select the menu item to upgrade) it will perform the upgrade.  If you don't click on the banner or menu item, the system will leave the upgrade staged, but not perform the upgrade.

Are you seeing different behavior?


User Tag Filter

Add the ability to filter metric and user table views based on user tag.  Additionally, if feasible remember this setting as a user preference.

dup to #1861


Fix up USGS scaling and coefficient application that result in negative numbers

Scaling was being done, but the resulting array was not being stored, so it wasn't actually being applied
Coefficients for Landsat 8 can result in negative numbers, which breaks the algorithm. Added a step that clips values to 0-10,000 after applying coefficients
Minor fixes...

Note, Landsat 8 image that includes regression coefficients but doesn't clip to 0-10,000:

And after, which includes clipping from 0-10,000 after applying the regression coefficients:

Oh, forgot to add, love the other clean ups you've made!


Add support for command !exit to show last user to exit the server

this would be useful to see who last exited the server

You mean a list of players, who left the server during a running map?
E.g. !exit -> "Last players who left: PlyA, PlyB, PlyC"
And when a new map starts this map will be reset and filled when players leave during an active map?
In case the nobody left and you use !exit, the output shall be "No players left during this match" ??

that would be ok, I was mainly thinking the last person leaving the server! But the last 3 would be ok too!

Hmm only the last leaving person is so much easier to implement... Now I was fighting already how to get a dynamic list of only the last three... If I would know from the beginning, that only one person would be enough, I could already have finished this feature :-)
Since this is your idea: only the last or the last 3? Your decision...

Just the last would be fine!
and Thanks!!

Arf now when the code is almost finished :-)
Ok, only the last.
What would be the proper english:

option A: "The last player left: XX"
option B: "Last player left: XX"
option C: "Player last exited: XX"
option D: ??

And in case no player left: "No player left this match" ??

it doesn't really matter! I guess option b?

ok option B will be used

option D: "Last disconnected player:" is the most proper you can get I guess, or simply: "Last disconnect: PlayX".
Option A and B don't feel so smooth in english.

hmm yes you are right. but is "last disconnect:" correct regarding grammar??
"Last disconnected player: Fukushukumquat"
and UrT is already forcing a line-break :-)

and the name of the command? still "!exit" ? or "!disconnect"
if the command is "!exit", somehow the word "exit" should be in the output message or??

I think maybe !exit its fairly easy to remember ;)

command will be in release 1.9, which will be release soon. want to test support to UrT 4.3.2 first...

I'd love to test it!

Feature is now in the develop branch available.
Happy testing :-)
Reconnected players are automatically removed from the "last disconnected player list".


Please check requirements.txt for PyWin version

ERROR: Could not find a version that satisfies the requirement pywin32==306 (from versions: none)
ERROR: No matching distribution found for pywin32==306
Error: Process completed with exit code 1.

This error re

Closing this issue


refactor(loader): Change deepclone approach to reduce bundle size

Description
Change the way we do a deep clone for noticeable savings in bundle size.

@Abe27342  looks like attribution passes a complex object via options, but i don't think that should be loader options, it should be containerRuntimeOptions (if options at all)

@Abe27342 looks like attribution passes a complex object via options, but i don't think that should be loader options, it should be containerRuntimeOptions (if options at all)

@alexvy86 i would skip these test and file a bug. they are leveraging loader options incorrectly. loader options are not the correctly layer to manage a feature like attribution. it needs to be controlled by the runtime

@Abe27342
        Abram Sanderson (HE/HIM)
        FTE looks like attribution passes a complex object via options, but i don't think that should be loader options, it should be containerRuntimeOptions (if options at all)

@alexvy86 Alejandro Villarreal FTE i would skip these test and file a bug. they are leveraging loader options incorrectly. loader options are not the correctly layer to manage a feature like attribution. it needs to be controlled by the runtime

We were discussing it in Teams. @Abe27342 is it just the tests misusing loader options? From what you said I understood it was bigger than that. If it's just the tests then what Tony suggests seems fine (@ChumpChief , @markfields FYI).

Current POR:  Abandon this in favor of https://github.com/microsoft/FluidFramework/pull/15368


No vocalization of title after displaying the page

When a page is displayed, the first thing to vocalize automatically should be its main title.
This could be done by using announcements (notifications) to force the title page vocalization.

fixed


fix(server): can't reuse the port previously used to invoke listen when modify the config file.

Description
console.log('start vite server');
const server = await vite.createServer({
  root: process.cwd(),
});
server.listen(8000);

I use createServer directly like above, when I modify vite.config.js file, the port is no longer 8000 but the default 3000.

Or make listen not support the port param.

Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Could you link a bug issue together with a reproduction?

bug description https://github.com/vitejs/vite/issues/4464 @Shinigami92


qmk: 0.0.52 -> 1.0.0

Motivation for this change
Update qmk to 1.0.0
Things done

Built on platform(s)

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

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

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

[x] Fits CONTRIBUTING.md.

Thanks a lot for this!

The ci on linux says there is one failure in the tests
FAILED (failures=1)
Test failed: <unittest.runner.TextTestResult run=77 errors=0 failures=1>
error: Test failed: <unittest.runner.TextTestResult run=77 errors=0 failures=1>

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

3 packages failed to build:

python38Packages.qmk-dotty-dict
python39Packages.qmk-dotty-dict
qmk

Here is the test that appears to be failing
test_getitem_cache (tests.test_dotty_cache.TestDottyCache) ... FAIL

======================================================================
FAIL: test_getitem_cache (tests.test_dotty_cache.TestDottyCache)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/qmk_dotty_dict-1.3.0.post1/tests/test_dotty_cache.py", line 14, in test_getitem_cache
    self.assertEqual(dot.__getitem__.cache_info().hits, 9)
AssertionError: 10 != 9

Could we perhaps skip this test ?

@happysalada Thanks for the info! I will take a look at this to see if I can fix it!

tests are passing, thank you for this!


Update Set-SPSecurityTokenServiceConfig.md

Formatting. Fixed a couple of incorrect values.

Docs Build status updates of commit b58f9c8:
:clock10: Full build: the average full build time is 11 min(s) 53 sec(s), this is based on the last 846 build(s) for this repository.

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

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPSecurityTokenServiceConfig.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 2ef1c39:
:clock10: Full build: the average full build time is 11 min(s) 53 sec(s), this is based on the last 851 build(s) for this repository.

Closing, will submit a new change.

Docs Build status updates of commit 2ef1c39:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPSecurityTokenServiceConfig.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore: added esModules options

New options

esModules : to export content as ESM export

linting fixes

cc @evilebottnawi


docs: update license reference

PR Summary
Small PR - updates the server/README.md to point to the correct license file.

The licence file referenced in that README, though the same licence right now, isn't necessarily the same licence as this project. I would assume the original LICENCE file in server/ was removed, the README can likely just be deleted to be honest.


implemented some best practices in app.js

changed module requirements using var to const (ln 2-4)
grouped some var initializations to a single line (ln 6)
changed some var inits to let
changed certain strings enclosing char from " to '

the errors coming from Travis seem to not be a result of my changes. Not sure if this is from something else that needs to be fixed or if Travis results are currently being ignored.
Would love more information!
-Cooper

just looked more closely through the Travis logs, looks like the errors are due to my changes. Im investigating a fix

huh, Travis is taking a very long time... currently at 5 minutes

The build worked fine, but something cause the build to not exit, this is probably something to do with the way Travis has been setup for this project...
I have tested this locally, it works (see screenshots)
screenshots: https://imgur.com/a/AYHhy


clear out interval timers in willDestroyElement in case not completed…

… in finish(), to avoid ongoing errors throughout experiment as the referenced element has been destroyed

fixes #64


Update package.json license

As the title says

I'd like to see this merged too if possible!

LGTM

Sorry folks. I don't think I'm getting notifications for this repo at all. 😬
Better late than never. 😞

Ace! Thanks 💃


Sam header when mapping to transcriptome

I tried mapping to the transcriptome, and it seems to have worked. However, while the reads are mapped to the individual chromosomes, the sam header still consists of the transcript IDs.
Example:
@HD     VN:1.0  SO:unknown
@SQ     SN:ENST00000456328_1    LN:1657
@SQ     SN:ENST00000515242_1    LN:1653
@SQ     SN:ENST00000518655_1    LN:1483
@SQ     SN:ENST00000450305_1    LN:632
@SQ     SN:ENST00000438504_1    LN:1783

Since the reads have the expected "chromosome" identifiers in the sam fields, this results in many error messages such as printed below when converting the sam file to bam or calling samtools flagstat:
[W::sam_parse1] urecognized reference name; treated as unmapped

Hi,
Ugh, I missed that one.
I'll fix it soon and let you know.
Thank you for the report!
Best regards,
Ivan.

Sorry it took a bit...
Could you try pulling the latest commit and giving it a spin?
Best regards,
Ivan.

Sam/bam files are okay now! Alignment looks fine. I'm going to rerun with some trimming at the end of the reads, I get the idea that for many reads the end of read is softclipped because of lesser quality at end of sequences/adapter sequence.

I seem to be having this similar error with genome alignment.
Getting many lines with:
[sam_read1] reference 'U00096.2' is recognized as '*'.


Update Clarity - Docusaurus V3

Add Installation shortcut in the upper barre and explanation of the Documentation organization.
Have to update Docusaurus to v3 to be able starts a local development server on my computer => the current version is depreciated anyway so i think this update is a good thing.
Need to verify if the update doesn't broke the web site and the dev of other people.

I remember we attempted a v3 upgrade at some point in the past and it failed.  I agree that it should be done at some point, however, if we manage to get the components to work. At the moment I am stuck with trying to test this PR:
testuser13@yggy:~/pr13$ git branch
* PR-update-clarity
  master
testuser13@yggy:~/pr13$ git remote -vv
origin	[https://github.com/ThimotheeV/treeppl.github.io.git](https://github.com/ThimotheeV/treeppl.github.io.git) (fetch)
origin	[https://github.com/ThimotheeV/treeppl.github.io.git](https://github.com/ThimotheeV/treeppl.github.io.git) (push)
testuser13@yggy:~/pr13$ npm install
npm error code ETARGET
npm error notarget No matching version found for<EMAIL_ADDRESS>npm error notarget In most cases you or one of your dependencies are requesting
npm error notarget a package version that doesn't exist.
npm error A complete log of this run can be found in: /home/testuser13/.npm/_logs/2024-12-10T08_37_02_277Z-debug-0.log
testuser13@yggy:~/pr13$

For comparison
git checkout master
npm install

succeeds

I think I have corrected the problem, can you try again pls ?

Seems to work now:
testuser13@yggy:~/pr13$ npm list @docusaurus/core
<EMAIL_ADDRESS>/home/testuser13/pr13
├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>deduped

testuser13@yggy:~/pr13$

The test site runs.
I like.

Working on Mac!
Ready to go.


add keybind to close selected file in opened file search

#116

after a while using this nice feature, i noticed that i constatly need to clean up some already unnecesary files. The idea is from nvim again, there the most popular pickers like telescope.nvim have this ability and i miss it so much in obsidian =)

LGTM👍

Supplemental Information
I considered removing closed files from the selection options, but I decided to keep them for consistency with other commands.

@svonjoi
Released in v12.1.0 🚀

hey, thanks for the update, i tested and saw that element after closing it still apearing in the list untill the list is opened again

@svonjoi
Thank you for checking. So far, it’s working as expected.
https://github.com/tadashi-aikawa/obsidian-another-quick-switcher/issues/256#issuecomment-2466084362

i'm blind :D


Update event-011.md

Another time added (I forgot to add this one last time)

Sorry about the format. I assumed that the double dash was only for repeated events.


Astrto Colony

Module Request
Game Information:
Game Title: Astro Colony
How to obtain: Astro Colony Dedicated Server Setup
Suppoted OSs: Windows
Supports RCON/Console input? Not Sure
I confirm:

[x] that I have searched for an existing module request for this application.
[x] that the server is an official server distributed by the games publishers/developers.
[x] that the server is available to the general public.
[x] that the server can be run headless (without a GUI).
[x] I have already attempted to create a configuration myself using the configuration tool

Not aware of the game having dedicated servers

I've created a draft in Greelan/AMPTemplates:dev, but wow, this server is janky. A bunch of stuff does not seem to work?

Afaik, the multihome argument has not been implemented for this server. In any event, binding to <IP_ADDRESS> is fine. The issue with the server list is that, last time I checked a couple of weeks ago, it only shows around 60 servers at one time, and doesn't have a search/filter function. The dev has said he will increase the number to 150 or 200 and add the search/filter function, making it easier to find your server.
Make sure you have forwarded the query port.


Page shifting when hover over backlink

Hi, thanks for building this SSG template for digital gardening, I really like it. I found a small bug, It's very easy to reproduce and happens 100% of the time:
On the demo site, simply go to notes page, click on "easter egg" and then in the backlink section, when you hover over"Notes" the whole page will shift position when the preview pops up, creating a effect as if the page is shaking.
I think it happens because the scroll bar shows up during the hover.
Hope this can be fixed in the future iteration, thanks again!
Jarod

@jarodise Hey, thanks so much for the feedback.
I was able to reproduce the bug you found. I think that there are some areas where the mouse is only half in the hover zone. That might be why it the scrollbar keeps disappearing. Thanks again for the issue.

Hi, seems like the problem still exists.

Same situation?

Yeah, basically it's caused by the scroll bar showing up during the hovering, scroll bar is hidden by default and shows up when the hover popup goes below the bottom of the page.

I think the easy solution would be either complete hide the scroll bar or make it show all the time.

I think the easy solution would be either complete hide the scroll bar or make it show all the time.

I notice that when I hover over the link, the footer shifts down as well. The modalset to position: relative, so when it comes appears it affects the page length. Setting it to position: absolute should fix this.

@jarodise Try it now.

I'm not sure if you pushed the update to your demo site, but from my end, the problem still persists, see the screen capture:

Weird. What's your screen size? What device?

I'm on Macbook Pro 13 inch Late 2015, however, I don't think it's device relevant, like I said earlier, the problem is caused by scrollbar showing up when the popup goes beyond the extend of the page bottom, so as long as the content is long enough to reach the bottom of the page, this problem will show up. no matter how big the screen or what the resolution is.
If everything is limited to the top part of the page where the bottom of the page is never reached, of course this problem will not show because the scroll bar won't show in that case, but however we cannot rule out of possibility of having longer form notes or larger photos that will push the content to the bottom page , which eventually will cause the scroll bar to show and then cause the shifting of the page.
The solution remains, either having the scroll bar show all the time, or not showing at all, only by doing this you won't have the shifting caused by the sudden emergence of scroll bar, hope I explained myself clearly.

Notice the scroll bar:

Suggestion: If we move the popup away from the bottom of the screen, we might avoid the issue altogether.

I'm hesitant to have the scrollbar always present, because most websites don't act like that.  That's why I'm looking for a workaround.

@jarodise what do you think? :point_up:

@jarodise what do you think? ☝

yeah, I think this is a good solution.
another idea would be, having an on/off switch for preview popup, since this is the reason that's causing it.

yes, it works for me! thanks for the effort!

One last question: Do you know how I could apply the update on my forked repo?

One last question: Do you know how I could apply the update on my forked repo?

Great question. I'm really not sure. Template repositories are pretty new to me. I'd imagine you could connect your repository to this one using git. I'll play around.

One last question: Do you know how I could apply the update on my forked repo?

Great question. I'm really not sure. Template repositories are pretty new to me. I'd imagine you could connect your repository to this one using git. I'll play around.

Yeah, if you find an easy option, please kindly let me know, I've played with a few git commands I found on stackoverflow, but came across some errors about merge conflict, as someone who's really not technical, I don't know how I could go forward with this, I hope I don't have to delete and refork the template every time you push an update.
thanks!

@jarodise I think I found a way for you to update it. Lmk if this works.
$ git remote add upstream git@github.com:binyamin/eleventy-garden.git
$ git stash
$ git merge upstream/master -m "Update template"
$ git stash apply

You may see merge conflicts (ie. lines of code which we both changed). VS code can help you resolve them.


How to customize the default URL(/socket.io/?EIO=...) in client and server?

I want to customize the URL such as /mySockketIO/?EIO=...

Server Options has a context attribute

Thanks! I have found it.
In client:
IO.Options options = new IO.Options();
options.path = "/subscribe.socket.io";
In cerver:
com.corundumstudio.socketio.Configuration config = new com.corundumstudio.socketio.Configuration();
config.setContext("/custom.socket.io");

@Tracy0725 thanks a lot!

This context works properly and it sets up the context for handshake - but for actually sending the messages it does not work. Any idea how this can be done?


Cepo

Do we also want to maybe edit the ReadMe.md here: https://github.com/CerebrasResearch/cb_optillm/blob/cepo/README.md#implemented-techniques ?


Transformation plugins missing documentation

Where is the problem?
https://docs.konghq.com/hub/kong-inc/request-transformer-advanced/configuration/
What happened?
In the older document you could see that the headers could be added to the request transformation advanced plugin. The information also provided how customers can reference values stored in vaults like Hashicorp or AWS secrets manager.
What did you expect to happen?
I expected to see documentation on how to set up referenced headers.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

In the older document you could see that the headers could be added to the request transformation advanced plugin.

The doc that you're asking about is here: https://docs.konghq.com/hub/kong-inc/request-transformer-advanced/how-to/

The information also provided how customers can reference values stored in vaults like Hashicorp or AWS secrets manager.

I believe this is the doc you're referring to: https://docs.konghq.com/hub/kong-inc/request-transformer-advanced/how-to/templates/
Both of these documents have links from the plugin landing page: https://docs.konghq.com/hub/kong-inc/request-transformer-advanced/#get-started-with-the-request-transformer-advanced-plugin
I understand the confusion with the link text, so I'll adjust that to be more descriptive of the contents of the page it's linking to.


required() of value false throws fieldname is null

When trying to require boolean fields, value false should not thow an error.
Haven't found any related issues.
It's independent PR.

Thank you!


[Improve][Manager] Not pass the type field when querying sources and sinks

Description
Not pass the type field when querying sources and sinks.
InLong Component
InLong Manager
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I will deal with this problem.


User ride viewing: figure out metric/imperial switch

metric/imperial switching for user ride viewing is only possible via URL parameters.  Figure out better way.

Resovled now with new button.


Open URL

Hello, could you add a right-click option to open the URL when showing the list of breached entries?

This is probably possible. I may even just be able to hook into the default right-click menu that you get in the main KeePass window.

I think @CoolAEW is looking for a way to launch the URL associated with the breached entry so they can go to change the password for that service. Right now, you double-click the breach item which selects the entry in the main window, then you can right-click that item and launch the URL from there. This is more-or-less a shortcut for that.


Could the discussions tab be opened for non questions/bugs?

This place may not be the best place. I was wondering how does this package differ from the standard Provider package and Riverpod? Would like to see an extensive explanation as a beginner.

Hi, please read readme. This package saves hours and days of heavy work for me. Try and tell how it's going =)

@machinescream The readme does not explainthe difference between the standard Provider package and Riverpod.

@machinescream The readme does not explainthe difference between the standard Provider package and Riverpod.

And why should it?

@machinescream Don't know why you are being defensive....I am merely trying to know the practical differences between them without having to use all of them extensively.


Add support for Swagger

It would be great if we could use add  Swagger in the go-json-rest package.
Swagger is a specification and complete framework implementation for describing, producing, consuming, and visualizing RESTful web services.
Are there any plans to implement this? If not: could you please give me a hint where I should start implementing this?
Thanks in advance!

+1

+1

+1 Swagger support would be great

+1

+1

+1

+1  github.com/emicklei/go-restful supports swagger very well, users of go-json-rest needs it definitely!

Swagger2.0 is being renamed Open API Initiative (https://openapis.org/).
And @casualjim the author of go-swagger has started go-openapi https://github.com/go-openapi
This is very interesting because the implementation is now broken down in a lot of small pieces that could be much easier to reuse. Anybody wants to glue Go-Json-Rest and Go-OpenAPI ? :)

Any updates on this?
This issue was opened 2 years ago. Maybe at that time swagger was not as important. but in 2016, Open API is a crucial part of a rest application, it makes it possible to plug your API to AWS Gateway, or the new Google Endpoints in no time.
IMO Open API support must be a first class citizen of this library. @ant0ine would you clarify if you do, or do not have the intention of making this happen? We are in the process of finding a suitable Go library for our rest app, and Open API support is an important factor for us.


perlPackages.ClusterSSH: init at 4.13.2

Motivation for this change
Adds ClusterSSH with one dependency and myself as a maintainer. ClusterSSH is a tool that lets you interactively manage multiple servers over SSH at once.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

May be from CPAN instead of GitHub to simplify auto-update ?
It has the same version https://metacpan.org/pod/App::ClusterSSH::Cluster

You're right, thanks! I've been looking at CPAN before, but I thought it's just some library and not the program itself.
I've also added installation of script for bash completion and missing dependency on openssh.

@GrahamcOfBorg build perlPackages.ClusterSSH
@aither64 Doing a quick scan of the perl source code I found this: https://metacpan.org/source/DUNCS/App-ClusterSSH-v4.13.2/lib/App/ClusterSSH/Config.pm#L464
Without looking into it in detail, did you test the application in such a way that you confirmed this code works as intended?

Success on x86_64-linux (full log)
Attempted: perlPackages.ClusterSSH
Partial log (click to expand)
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wbp00fgb02vyq50gavpa7xa5amch0q2h-perl5.28.0-App-ClusterSSH-4.13.2/lib  /nix/store/wbp00fgb02vyq50gavpa7xa5amch0q2h-perl5.28.0-App-ClusterSSH-4.13.2/bin
patching script interpreter paths in /nix/store/wbp00fgb02vyq50gavpa7xa5amch0q2h-perl5.28.0-App-ClusterSSH-4.13.2
checking for references to /build in /nix/store/wbp00fgb02vyq50gavpa7xa5amch0q2h-perl5.28.0-App-ClusterSSH-4.13.2...
shrinking RPATHs of ELF executables and libraries in /nix/store/2zsrgyrs6xiikz1191rs6vbnk5pr3pbk-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
gzipping man pages under /nix/store/2zsrgyrs6xiikz1191rs6vbnk5pr3pbk-perl5.28.0-App-ClusterSSH-4.13.2-devdoc/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/2zsrgyrs6xiikz1191rs6vbnk5pr3pbk-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
checking for references to /build in /nix/store/2zsrgyrs6xiikz1191rs6vbnk5pr3pbk-perl5.28.0-App-ClusterSSH-4.13.2-devdoc...
/nix/store/wbp00fgb02vyq50gavpa7xa5amch0q2h-perl5.28.0-App-ClusterSSH-4.13.2

Success on aarch64-linux (full log)
Attempted: perlPackages.ClusterSSH
Partial log (click to expand)
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/lqsvcdn8lpwzdivx458f4dbambqwxvhl-perl5.28.0-App-ClusterSSH-4.13.2/lib  /nix/store/lqsvcdn8lpwzdivx458f4dbambqwxvhl-perl5.28.0-App-ClusterSSH-4.13.2/bin
patching script interpreter paths in /nix/store/lqsvcdn8lpwzdivx458f4dbambqwxvhl-perl5.28.0-App-ClusterSSH-4.13.2
checking for references to /build in /nix/store/lqsvcdn8lpwzdivx458f4dbambqwxvhl-perl5.28.0-App-ClusterSSH-4.13.2...
shrinking RPATHs of ELF executables and libraries in /nix/store/l0haqanl5825gypqfrdv5641xp6naid8-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
gzipping man pages under /nix/store/l0haqanl5825gypqfrdv5641xp6naid8-perl5.28.0-App-ClusterSSH-4.13.2-devdoc/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/l0haqanl5825gypqfrdv5641xp6naid8-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
checking for references to /build in /nix/store/l0haqanl5825gypqfrdv5641xp6naid8-perl5.28.0-App-ClusterSSH-4.13.2-devdoc...
/nix/store/lqsvcdn8lpwzdivx458f4dbambqwxvhl-perl5.28.0-App-ClusterSSH-4.13.2

@aanderse, yes, I'm using it and it works for me. The code you linked is used to find xterm (or some other configured terminal), so the hardcoded paths would be used if it weren't found already in $PATH, i.e. it will not find it on NixOS and exit. I guess we could add /run/current-system/sw/bin to the hardcoded list, but I think it is sufficient to rely on $PATH being set properly and having the terminal in it.

Success on x86_64-linux (full log)
Attempted: perlPackages.ClusterSSH
Partial log (click to expand)
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xsn0s5g49650y89241vdal28wz1l7knk-perl5.28.0-App-ClusterSSH-4.13.2/lib  /nix/store/xsn0s5g49650y89241vdal28wz1l7knk-perl5.28.0-App-ClusterSSH-4.13.2/bin
patching script interpreter paths in /nix/store/xsn0s5g49650y89241vdal28wz1l7knk-perl5.28.0-App-ClusterSSH-4.13.2
checking for references to /build in /nix/store/xsn0s5g49650y89241vdal28wz1l7knk-perl5.28.0-App-ClusterSSH-4.13.2...
shrinking RPATHs of ELF executables and libraries in /nix/store/gzynavpxaw0d87w3qk0yg23ddk26h39b-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
gzipping man pages under /nix/store/gzynavpxaw0d87w3qk0yg23ddk26h39b-perl5.28.0-App-ClusterSSH-4.13.2-devdoc/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/gzynavpxaw0d87w3qk0yg23ddk26h39b-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
checking for references to /build in /nix/store/gzynavpxaw0d87w3qk0yg23ddk26h39b-perl5.28.0-App-ClusterSSH-4.13.2-devdoc...
/nix/store/xsn0s5g49650y89241vdal28wz1l7knk-perl5.28.0-App-ClusterSSH-4.13.2

Failure on x86_64-darwin (full log)
Attempted: perlPackages.ClusterSSH
Partial log (click to expand)
  TODO passed:   160-163, 168-171
t/30cluster.t   (Wstat: 512 Tests: 59 Failed: 0)
  Non-zero exit status: 2
  Parse errors: No plan found in TAP output
Files=16, Tests=1916, 11 wallclock secs ( 0.23 usr  0.09 sys +  8.08 cusr  2.26 csys = 10.66 CPU)
Result: FAIL
Failed 1/16 test programs. 0/1916 subtests failed.
make: *** [Makefile:1025: test_dynamic] Error 255
builder for '/nix/store/csp92ks9w95rm3p9pkl60w9qsdq9bc33-perl5.28.0-App-ClusterSSH-4.13.2.drv' failed with exit code 2
error: build of '/nix/store/csp92ks9w95rm3p9pkl60w9qsdq9bc33-perl5.28.0-App-ClusterSSH-4.13.2.drv' failed

Success on aarch64-linux (full log)
Attempted: perlPackages.ClusterSSH
Partial log (click to expand)
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/58lissxzppdn2cy62abhfq98ln83lvqd-perl5.28.0-App-ClusterSSH-4.13.2/lib  /nix/store/58lissxzppdn2cy62abhfq98ln83lvqd-perl5.28.0-App-ClusterSSH-4.13.2/bin
patching script interpreter paths in /nix/store/58lissxzppdn2cy62abhfq98ln83lvqd-perl5.28.0-App-ClusterSSH-4.13.2
checking for references to /build in /nix/store/58lissxzppdn2cy62abhfq98ln83lvqd-perl5.28.0-App-ClusterSSH-4.13.2...
shrinking RPATHs of ELF executables and libraries in /nix/store/8308zlb51l4i74xjmdh2k2l4gfqn46i7-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
gzipping man pages under /nix/store/8308zlb51l4i74xjmdh2k2l4gfqn46i7-perl5.28.0-App-ClusterSSH-4.13.2-devdoc/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/8308zlb51l4i74xjmdh2k2l4gfqn46i7-perl5.28.0-App-ClusterSSH-4.13.2-devdoc
checking for references to /build in /nix/store/8308zlb51l4i74xjmdh2k2l4gfqn46i7-perl5.28.0-App-ClusterSSH-4.13.2-devdoc...
/nix/store/58lissxzppdn2cy62abhfq98ln83lvqd-perl5.28.0-App-ClusterSSH-4.13.2

@aanderse, yes, I'm using it and it works for me. The code you linked is used to find xterm (or some other configured terminal), so the hardcoded paths would be used if it weren't found already in $PATH, i.e. it will not find it on NixOS and exit. I guess we could add /run/current-system/sw/bin to the hardcoded list, but I think it is sufficient to rely on $PATH being set properly and having the terminal in it.

Makes sense to me. In this situation you should avoid adding /run/current-system/sw/bin as it is always better to explicitly declare dependencies when you can, like this situation. Looking good.

@GrahamcOfBorg build perldevelPackages.ClusterSSH

@volth do we need to take special care if packages do not build for specific perl version?

Failure on x86_64-linux (full log)
Attempted: perldevelPackages.ClusterSSH
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/w0gyc5xkj81ywrchsb0gdiazw4r5knn3-perl5.29.4-X11-Protocol-Other-30/lib
patching script interpreter paths in /nix/store/w0gyc5xkj81ywrchsb0gdiazw4r5knn3-perl5.29.4-X11-Protocol-Other-30
checking for references to /build in /nix/store/w0gyc5xkj81ywrchsb0gdiazw4r5knn3-perl5.29.4-X11-Protocol-Other-30...
shrinking RPATHs of ELF executables and libraries in /nix/store/if38vapx8q8nl685gbynhbjin474l9wp-perl5.29.4-X11-Protocol-Other-30-devdoc
gzipping man pages under /nix/store/if38vapx8q8nl685gbynhbjin474l9wp-perl5.29.4-X11-Protocol-Other-30-devdoc/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/if38vapx8q8nl685gbynhbjin474l9wp-perl5.29.4-X11-Protocol-Other-30-devdoc
checking for references to /build in /nix/store/if38vapx8q8nl685gbynhbjin474l9wp-perl5.29.4-X11-Protocol-Other-30-devdoc...
cannot build derivation '/nix/store/3q5ccjgvx8l2bksvj5qib54pwjdinbpd-perl5.29.4-App-ClusterSSH-4.13.2.drv': 2 dependencies couldn't be built
error: build of '/nix/store/3q5ccjgvx8l2bksvj5qib54pwjdinbpd-perl5.29.4-App-ClusterSSH-4.13.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: perldevelPackages.ClusterSSH
Partial log (click to expand)
/nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034/lib/perl5/site_perl/5.29.4/aarch64-linux-thread-multi/Tk/pTk/Tcl-pTk: interpreter directive changed from "/usr/bin/perl -w -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034/lib/perl5/site_perl" to "/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/bin/perl -w -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034/lib/perl5/site_perl"
/nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034/lib/perl5/site_perl/5.29.4/aarch64-linux-thread-multi/Tk/pTk/mkVFunc: interpreter directive changed from "/usr/bin/perl -w -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034/lib/perl5/site_perl" to "/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/bin/perl -w -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4/lib/perl5/site_perl -I/nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034/lib/perl5/site_perl"
checking for references to /build in /nix/store/31ynnqhjxafdvinsvx45q7vxvkllgrqb-perl5.29.4-Tk-804.034...
shrinking RPATHs of ELF executables and libraries in /nix/store/hlzxixmfb9rpr90p8gh391zqhyj8k7bs-perl5.29.4-Tk-804.034-devdoc
gzipping man pages under /nix/store/hlzxixmfb9rpr90p8gh391zqhyj8k7bs-perl5.29.4-Tk-804.034-devdoc/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/hlzxixmfb9rpr90p8gh391zqhyj8k7bs-perl5.29.4-Tk-804.034-devdoc
checking for references to /build in /nix/store/hlzxixmfb9rpr90p8gh391zqhyj8k7bs-perl5.29.4-Tk-804.034-devdoc...
cannot build derivation '/nix/store/bm5l0xi19zpg5hkhq8cm9j7h91jvg2rf-perl5.29.4-App-ClusterSSH-4.13.2.drv': 2 dependencies couldn't be built
error: build of '/nix/store/bm5l0xi19zpg5hkhq8cm9j7h91jvg2rf-perl5.29.4-App-ClusterSSH-4.13.2.drv' failed

@volth do we need to take special care if packages do not build for specific perl version?

I'd put meta.broken = versionAtLeast perl.version "5.29"

@GrahamcOfBorg build perlPackages.ClusterSSH

@GrahamcOfBorg build perldevelPackages.ClusterSSH

Sorry for the delay. I've rebased it on nixos-unstable, added meta.broken for perl 5.29 and restricted platforms to x86_64-linux based on the build failures. Is it suitable for merging? I have no idea what to do about those build failures at this time.

This is obsoleted by  #65043 and should be closed.


Gamboa's Module_1 pull request

Pull request for the Module_1 JSHINT.com assignment.

I did take off some points (not that many), but you can open a new pull request with more corrections if you wanted to get the points back. :+1:


Persist only neccesary files to VCS

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I need to persist the uptime-kuma configuration to replicate on diferent deployments.
I dont want to upload to my project all the content of uptime-kuma data directory.
What are the required files to store and upload to VSC service?
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
Latest
💻 Operating System and Arch
Ubuntu 22.04
🌐 Browser
Google Chrome
🖥 Deployment Environment

Runtime:
Database:
Filesystem used to store the database on:
number of monitors:

All files in the data directory are required for it to work correctly.

All files in the data directory are required for it to work correctly.

Any form to export the configuration for import?
Upload to repository a .db file is the finally solution?

Upload to repository

Aaah so you don't mean the Vienna Scientific Cluster Service (first result if you search for VSC Service)...
We might not currently support what you are looking for.
I think you are looking for a monitoring solution which is infrastructure as code. That simply is not us.
While you can use the deprecated backup functionality or write your own scripts do do what you need, I think what you are looking for is likely a different solution.
Checking your db into git might work as well if you are using git LFS, but be aware that the db-size will be larger the more heartbeats need to be retained. (see #4500)
Please have a look at gatus instead if you want IAC.

After publish the comment and save the repository that you mentioned, github suggest me this project: https://github.com/BigBoot/AutoKuma
I leave this link here to help who search this questions.
Ty!!


Compiler Panic When Constructing Dict

While solving day 8 of AoC I ran across a compiler crash when trying to create a dictionary with the type signature Dict {x: I32, y: I32} Nat from a List (T {x: I32, y: I32} Nat) using Dict.fromList
The code in question is here
Replacing this piece of code with the helper function woodFromInput (as seen here) solves the issue.
Using a List.walk instead of doing Dict.fromList directly does not make any difference.
The error message in question is:
thread '<unnamed>' panicked at 'called `Option::unwrap()` on a `None` value', crates/compiler/mono/src/ir.rs:5710:56
stack backtrace:
   0: rust_begin_unwind
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/panicking.rs:142:14
   2: core::panicking::panic
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/panicking.rs:48:5
   3: roc_mono::ir::late_resolve_ability_specialization
   4: roc_mono::ir::with_hole
   5: roc_mono::ir::from_can
   6: roc_mono::ir::specialize_variable
   7: roc_mono::ir::specialize_external_help
   8: roc_mono::ir::specialize_all
   9: roc_load_internal::file::run_task
  10: core::ops::function::FnOnce::call_once{{vtable.shim}}
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

The compiler also hangs instead of exiting.

I suspect this is because Hash is not yet implemented for Hash, although the type checker assumes that it is.

Nevermind, this reproduces on latest Roc. The symptom in debug mode is
thread '<unnamed>' panicked at 'assertion failed: `(left == right)`
  left: `3402`,
 right: `3306`', crates/compiler/mono/src/ir.rs:1178:33

resolving to failures here:
https://github.com/roc-lang/roc/blob/662eb5895eab36af4550a2c00d2bf089f0ef9340/crates/compiler/mono/src/ir.rs#L1169-L1182

Root cause is likely the same as https://github.com/roc-lang/roc/issues/4717


[Bug]: Preview Deployment containers are not removed after merging the PR

Description
After the PR is merged, the container keeps running. AFAIR it was working properly in 28X versions
Minimal Reproduction (if possible, example repository)

Add a repo-based resource; in my use case it's using Nixpacks deployment
Configure auto PR deployments in "Advanced" Configuration
Create PR in your repo
Merge into main
Either run

docker ps

or go to the Configuration: Preview Deployments

And you'll see that the one you've closed is still running
Exception or Error
No response
Version
v4.0.0-beta.294

This will be fixed in the upcoming version.


Moving the project to typescript

This is not the complete work, but it builds correctly and the test pass. With this PR, I'm proposing to move to project to typescript to introduce type-safety, and move the tests to jest, which supports very similar syntax to chai at this stage.
Let me know what you think - if we can merge this, I'll continue improving the package.
I'm purposefully not bumping up the package version - but if you need me to, I can.
Future work can include:

Actual type safety - currently we're allowing any.
Moving the tests to TS as well / and don't reference the dist folder for running tests.
Improve maintainability and community support by adding github actions (build/release).

Thank you for this! Merge and made some updates. Take a look here https://github.com/claude-abounegm/angel-eval/tree/feature/typescript


Images misdisplayed  in 0.3.0

Hi again,
Playing in my demo site with the new 0.3.0 commit, I've found what is like to be a malfunction.
When it goes to product/show after select a product in the index, it shows the buttons for the size and color variants (using the RoR BB jersey as demo), but below the main default picture, it adds all the images "thumbnails" for all variants and when you select an specific color, doesn't change the main image and still show all the thumbnails.
In another test, in Admin site, when I set the image for the different option_type it display the new partial to update the images files names, but in the show product of the store it uses that images to set the content of the variant button instead of leaving the name of the variant.
Please let me know if you need further information
I hope it helps!

This should be fixed on master @ 1f807b0