cups-browsed segfaults with recent glibc 2.32

Some Arch linux users are faced with broken driverless printing probably since our toolchain update with cups-filters 1.27.x and 1.28.0. I'm affected on one of my system where gtk based applications print "pdf-1.5" and  wild characters. I couldn't catch some useful info from debug log but get a repeatable segmentation fault when invoking like this:
/usr/bin/cups-browsed -v
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/libthread_db.so.1".
Wed Aug 26 15:07:17 2020 Reading command line option -v, turning on debug mode (Log on standard error).
Wed Aug 26 15:07:17 2020 cups-browsed of cups-filters version 1.28.0 starting.
Wed Aug 26 15:07:17 2020 Reading config: BrowseRemoteProtocols DNSSD,CUPS
Wed Aug 26 15:07:17 2020 Reading config: CreateIPPPrinterQueues Driverless
Wed Aug 26 15:07:17 2020 No "Browse..." line at all, accept all servers ("BrowseOrder Deny,Allow").
Wed Aug 26 15:07:17 2020 main() in THREAD<PHONE_NUMBER>24800
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7950edb in fclose@@GLIBC_2.2.5 () from /usr/lib/libc.so.6
(gdb) set logging file /tmp/trace.log
(gdb) set logging on
Copying output to /tmp/trace.log.
Copying debug output to /tmp/trace.log.
(gdb) thread apply all bt full
Thread 1 (Thread 0x7ffff660c400 (LWP 73791)):
#0  0x00007ffff7950edb in fclose@@GLIBC_2.2.5 () at /usr/lib/libc.so.6
#1  0x000055555555f82a in debug_printf (format=0x55555557dc86 "main() in THREAD %ld\n") at utils/cups-browsed.c:766
fp1 = 0x1
fp2 = 0x90
curtime =<PHONE_NUMBER>
buf = "Wed Aug 26 15:07:17 2020\000\000\000\000\000\000\000\000\016", '\000' <repeats 15 times>, "\060\000\000\000\000\000\000\000\003\000\000\000\000\000\000"
arglist = {{gp_offset = 16, fp_offset = 48, overflow_arg_area = 0x7fffffffe570, reg_save_area = 0x7fffffffe490}}
log_file_size = 1147032
#2  0x000055555555d36c in main (argc=2, argv=0x7fffffffe768) at utils/cups-browsed.c:12340
ret = 1
http =
i =
val =
p =
proxy = 0x0
error = 0x0
subscription_id = 0
action = {__sigaction_handler = {sa_handler = 0x7ffff7ce6ee0, sa_sigaction = 0x7ffff7ce6ee0}, sa_mask = {__val = {140737349773711,<PHONE_NUMBER>85208,<PHONE_NUMBER>48736,<PHONE_NUMBER>48652,<PHONE_NUMBER>48736,<PHONE_NUMBER>85208,<PHONE_NUMBER>05676,<PHONE_NUMBER>1123,<PHONE_NUMBER>2, 0, 0, 0, 0, 0, 0, 0}}, sa_flags = 0, sa_restorer = 0x0}
We have updated to glibc 2.32-2 / gcc-libs 10.2.0-1 / binutils 2.35-1 / libxcrypt 4.4.16-3.

Hi @andyrtr ,
would you mind sharing your /etc/cups/cups-browsed.conf? Did you have cups-browsed debug logging set to a file in the past?
In my opinion it is not connected to new glibc - lfp is NULL or corrupted - would you mind printing out its contents in gdb?

Attaching my cups-browsed.conf
As of today I can't reproduce it anymore. We had another toolchain update rebuilt against latest linux-kernel-headers 5.8, poppler 20.09.0, systemd and liblouis updates. Now I can print again to my printer using IPP. cups-browsed doesn't seem to crash anymore here. I've tried downgrading the toolchain and several releated packages of cups-filters but cannot reproduce it anymore.
Feel free to close this bug.

@tillkamppeter would you mind closing the issue?

No problem, not having a reproducer it is worthless. If there is actually a bug and someone else hits it, he will report it. Closing ...


[FLINK-29914][tests] Wait for Kafka topic creation/deletion

What is the purpose of the change
Kafka creation/deletion is an async operation. The API gives back future(s) which must be waited in order to be on the safe side. Additionally the APIs can throw exceptions while waiting to finish. In this PR I've added a more or less consistent Kafka topic creation/deletion behavior to the tests. Namely future wait to all the places where it was missing. This may or may not fix the issue (Kafka itself can be buggy) but I'm pretty sure w/o consistent topic creation/deletion API usage we're in trouble.
Brief change log

Wait for Kafka topic creation/deletion.
Add random to test topic names

Verifying this change
Existing unit tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

cc @XComp

This would justify couple of facts/behaviors. Namely test retry was introduced because of failures. I can simply imagine the following situation which justifies why retry introduced:

Test initiated topic creation
Topic not yet created
Test tried to read/write topic and failed (no topic auto creation)
Test retried
Topic creation finished
Test may passed

Now it died w/ the following exception:
Caused by: org.apache.kafka.common.errors.TopicExistsException: Topic 'partition_failure_recovery_EventTime' already exists.

Just to try it out I've temporarily cherry-picked #21189 to see how it behaves.

I'm just wondering whether adding the timeouts actually help our mission to stabilize the tests

If we can kick this to a green state we can remove the timeouts since the stability comes from waiting for the Java future via .all().get....

All in all we have multiple issues stacked up:

Creating the same topic exception was ignored via bad Kafka API usage (missing future wait)
Topics are not deleted in tests (which makes sense in case of exception)
Somehow the producer is not super happy and throwing timeout exception (this may be a Kafka which requires huge amount of time to debug)

I think the actual code should resolve the first 2 issues, the third may or may not disappear :/
Hope it will pass, such case we can just remove timeouts and ship it.

I've just fixed another layer of issue:
Nov 07 13:16:23 [ERROR] org.apache.flink.streaming.connectors.kafka.KafkaITCase.testMultipleTopicsWithKafkaSerializer  Time elapsed: 240.264 s  <<< ERROR!
Nov 07 13:16:23 org.junit.runners.model.TestTimedOutException: test timed out after 60000 milliseconds

However this issue was still there:
Caused by: org.apache.kafka.common.KafkaException: org.apache.kafka.common.KafkaException: Unexpected error in InitProducerIdResponse; The request timed out.

If the last change is not fixing it then I'm leaving this issue for now. That can be a tricky one if it's not generated by previous timeouts.

With fresh mind I think I've found the last issue :) Fixing it.
Additionally removing all timeouts as we've agreed and let's see...

Sounds promising. The change so far looks reasonable. Good job. :-) Could you ping me as soon as you're done?

I'm also wondering whether it would be reasonable to unify the Kafka test utilities (in a follow-up task). :thinking: multiple tests create their own admin client and deal with Kafka environments differently (KafkaSourceTestEnv, KafkaTestEnvironmentImpl, KafkaTableTestBase). It feels like we should have a JUnit5 Extension for that as well. We could also move the topic randomization in a single location at the end. WDYT?

I agree w/ the direction but as you mentioned in a follow-up PR. Now everything is scattered which is bad in general.
After we've fixed the tests w/ a fairly ugly code we could centralize the common tasks.
I've the following in my mind:

Topic creation
Topic deletion
Random name generation
AdminClient creation
Consumer creation
Producer creation

Could you ping me as soon as you're done?

Ping :)

FYI: I created a follow-up Jira issue for the Kafka environment topic: FLINK-29956

@PatrickRen can you have a look at it as well since it covers FLINK-24119

Some have still timeouts included (e.g. KafkaSinkITCase:438 and KafkaSinkITCase:432, KafkaSinkExternalContext:118, ...)

I've removed timeouts here too.

No special error handling (e.g. KafkaEnumeratorTest:217)

I think w/o deep understand what was the intention there I wouldn't change anything so leaving that as-is was intentional. W/o super deep consideration I would say we're not intended to test Kafka topic delete functionality. If topic can be created/modified and Flink is able to process the data, then we're fine. In my view we need a full concept w/ in-depth consideration to do proper refactoring in this area (I mean Kafka server/Consumer/Producer/AdminClient/Topic lifecycle).

Conceptually not all tests need random topics just where retry is happening. Of course adding random in general would be better design. If you guys asking me then later on we need to throw out huge amount of code in this area and just rewrite it to eliminate random failures. That mentioned we need a full concept and roughly several weeks to understand what each test does and kick through the design. What I've done here is a no-brainer copy-paste because I don't have now that time but just wanted to make it work. Adding any minor beautification to this area is waste of time from my perspective.

@flinkbot run azure

Couple of tests failed because they expect hardcoded name so rolled them back.

@flinkbot run azure

I will merge this unless you have any objections @XComp , let's get the tests back on track :)

If the other builds are also affected we should backport this I think 👍

Thanks guys, opened backport PRs...


False error message

This code should return a error message. But it doesn't.
var pizzaGuy = require('pizza-guy');
const files = [
  'https://httpbin.org/basic-auth/user/passwd',
];
pizzaGuy
  .deliver(files)
  .onAddress('./downloads')
  .onSuccess((info) => {
    console.log(`${info.fileName} succeed`);
  })
  .onError((info) => {
    console.log(`${info.fileName} failed`);
  })
  .start();

This is the same code from here, https://github.com/andrscrrn/pizza-guy/issues/58#issuecomment-257140103
In addition, I wonder what would happen if pizza Guy shows us the http response code in the info array too. I think that might solve many other problems.

@entrptaher I get what you are saying, but can you please change the title if the problem is that we are not showing any error message? Also, how do you think it is useful to return the http response code?

When you run it and check the info array, you can see that the error array has values under success. But it did not succeed downloading the file. The error array should only be filled when it counts that as error.
Having a http return code can always help. You can easily determine whether the page needs auth, or the page had problems loading data, or it ended successfully or not. Having a little more data will only help more.


onError/onSuccess for rules

Hello,
I have a long running workflows, and want to report the progress of execution to an external dashboard.
It would be nice to have rule-level onerror/onsuccess hooks so that a more granular reporting can be done. This call back can be used to execute pre- and post- logic.
I currently work around it by emulating start/stop of a rule by wrapping processing action in a call to an external utility (which in turn makes an HTTP call to the dashboard) but was hoping this can be done natively.
Thank you.

Doesn't snakemake already output progress of rule execution? At least it reports error/success I think. Would something that periodically probes an output log file of snakemake work?

I think snakemake run logs are not overly parser friendly for the environment I am in, both in structure and in timing (flush of output into a file). It would be cleaner to hook into the return code via callback. That way there is no separate process that watches the logs, and the callback is just another task I can instruct snakemake to execute on. Perhaps I may be missing some logging options?

Hello Everybody,
first of all: Thanks for the great tool!
We are using Snakemake to organize scientific, ETL-like pipelines and it has proven to be extremely valuable. Our workflows run regularly and integrate quite a few failure-prone steps (e.g. data transfer over mobile networks, http-requests, limited computing ressources) and are monitored by non-technical folks. In order to provide fine-grained reporting, a feature like rule-based onerror/onsuccess handlers would be really useful.
Is there any chance to get some momentum behind this enhancement request? If so and with a somewhat clear prospect to get such a feature merged eventually, I would be happy to take on the work on this and prepare a MR. What do you think?

I would also be interested in such a feature. I have a few core steps in a workflow where it would be convenient to have a rule-level onerror/onsuccess.

I would also add that I wouldn't mind having this!

A rule can complete successfully (exit code 0) but its outputs can be still be wrong. (e.g. if the invoked tool is buggy, if the user specified an incorrect argument, if the user provided incorrect inputs to the rule, etc)
One helpful use case for rule-level onsuccess would be to assert that the output data matches expectation.


SOLR-15917: Rename contrib -> modules

https://issues.apache.org/jira/browse/SOLR-15917
This is a big patch obviously, but I think I've curated into some reasonable commits.
I've run the following gradle commands, and all look good:

check
installDist
testDocker
testDockerfileOfficial

Happy to test whatever else.

It might be worth testing nightly?
Here is an example of testing all the hdfs related tests that I've been running for other stuff. It should take about 20 min:
/gradlew test --tests "*hdfs*" --tests "*Hdfs*" --tests "*HDFS*" --tests "*Hadoop*" -Ptests.jvms=4 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.nightly=true -Ptests.awaitsfix=false -Ptests.badapples=false -Ptests.file.encoding=UTF-8

I have no idea how long nightly tests take regularly. Feel free to play with with --tests

I'm fine with merging this and dealing with the merge conflicts in the Antora branch, I suspect these would be relatively minor. If this is ready, let's keep making progress IMO.

Ok after many rounds of testing the Nightly & HDFS tests, only 1 fails: SharedFSAutoReplicaFailoverTest. However this fails on main for me consistently, so I don't think it is relevant.
Will make a writeup on major-changes-in-9.adoc, then after a review of that I'll go ahead and merge (& backport) this.


Hanks Redemption Mod

Describe your bug here.
so i downloaded Hank's Redemption mod and i got an error, theres 3 track in the new version every track works except "Revenge" when i try to play the track i get an error please help me fix this
Command Prompt/Terminal logs (if existing)
flixel/tweens/misc/VarTween.hx (line 99)
flixel/tweens/misc/VarTween.hx (line 55)
flixel/tweens/FlxTween.hx (line 1263)
flixel/FlxGame.hx (line 750)
flixel/FlxGame.hx (line 682)
flixel/FlxGame.hx (line 556)
openfl/events/EventDispatcher.hx (line 444)
openfl/display/DisplayObject.hx (line 1400)

Uncaught Error: The object does not have the property "zoom"

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I mean, i just got the mod into the game

Not a Psych Engine related issue, post your issue on the page of the mod

it said to post issue here so i did it pls get me help or smth
czw., 25 lip 2024, 21:00 użytkownik ShadowMario @.***>
napisał:

Closed #15069
https://github.com/ShadowMario/FNF-PsychEngine/issues/15069 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/15069#event-13651808716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJYR3XUCGSRIFFDESJO7DPLZOFDLFAVCNFSM6AAAAABKUFJML6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGY2TCOBQHA3TCNQ
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


Make the user able to choose output file name when using export to XML

When you export to XML (to Premiere Pro, in my case), the names of all videos prepared for rendering will be "AUTO-EDITOR MEDIA GROUP." Something that can be very useful is a way to use the name that we want directly from the command line with arguments.

There are many ways and options to do that:

Set a name by default from the original file's name (for example, <filename>-auto-edited).
Allow the user to set an output name with arguments from the command line (for example, auto-editor example.mp4 --export premiere --output cut.mp4).

I'm processing videos in groups of 40 with script automation, but in the end, all files have the same filename. So, it's a bit frustrating to check each video just to rename it. So yeah, it would be something that I'll use, not something that can just be usefull sometime ahah.

You can already do this.
auto-editor file.mp4 '--export premiere:name="Hello"'

is there a problem with that syntax ?

seems that premiere cannot be 'premiere'


Could not load library "gurobi81"

Hi,
I installed Gurobi optimizer version 9.0.1 to use along with Julia. But when I try to use the package I get an error saying
LoadError: error compiling #GurobiMathProgModel#18: could not load library "gurobi81"
The specified module could not be found.
Can anyone please help me to resolve this?
Thanks.

If you've installed Gurobi 9.0 recently, you probably need to close Julia, open a new session, then run:
import Pkg
Pkg.build("Gurobi")
using Gurobi


[Contributor] Add Visit Expensify Classic Button to NewDot

Part of #wave9-collect-signups project
Main issue: Originated from this internal issue: https://github.com/Expensify/Expensify/issues/344901
Doc section: no design doc - this was pre-designed already.
Project: #wave9-collect-signups
Feature Description
Problem: As we start migrating users from OldDot to NewDot and point new signUps to NewDot, some users will require OldDot functionality to complete tasks, add secondary logins, use functionality not available on NewDot, access old expense/reports, and so on. If a user currently is in NewDot, the most straightforward way to access OldDot is to open a new tab and type in expensify.com to get to OldDot, which causes confusion. If they have the tryNewDot redirect, typing in expensify.com would not work and you’d have to send them a specific link or instructions how to do so.
Solution: Add a button in NewDot to allow user to easily navigate to OldDot.

Note: some users have the tryNewDot NVP applied to their accounts which means if they type in www.expensify.com, they are redirected to OldDot. This will not remove the NVP but will allow the user to be taken to the inbox.

Where will we add the button?
You’d find the button by going to Avatar > Settings. Then above Sign Out we’ll add the button Go to Expensify Classic (Image in thread of where it will go). Mock up here:

What will the button do?

Web (desktop/mobile) - opens up a new tab for https://www.expensify.com/inbox
NewDot App - leaves the app and opens up a new tab for https://www.expensify.com/inbox in your default browser
[HOLD] HybridApp - stays in HybridApp and goes to settings in OldDot. The hybrid app is not ready yet so we do not need to account for this use case yet, but just accounting that it will be here later.

I think this should have everything you need to complete this. Please feel free to ask me any questions.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0183075eef93ad0483
Upwork Job ID:<PHONE_NUMBER>197549568
Last Price Increase: 2023-12-11

Proposal
Please re-state the problem that we are trying to solve in this issue.
Add Visit Expensify Classic Button to NewDot
What is the root cause of that problem?
New Feature
What changes do you think we should make in order to solve the problem?
We will be adding another menuItem here, before Sign Out button.
https://github.com/Expensify/App/blob/165caf526ffcd2d00dbed4995d25981c7ab04b17/src/pages/settings/InitialSettingsPage.js#L259-L265
{
    translationKey: 'initialSettingsPage.goToExpensifyClassic',
    icon: Expensicons.Expensify,
    iconRight: Expensicons.NewWindow,
    action: () => {
        Linking.openURL(CONST.OLDDOT_URL);
    },
},

And add this CONST variable:
OLDDOT_URL: 'https://www.expensify.com/'

@neonbhai - This issue is labeled internal, which means it's not currently open to proposals. We will let you know if this changes in the future. Thanks!

@AndrewGable ah right. Thanks!

Hey, I'm Bartek from Callstack - expert contributor group - I would like to work on this issue

I haven't found a needed icon for this item, so I've converted https://github.com/Expensify/App/blob/main/assets/images/expensify-logo-round-transparent.png as best as I could 🙈
If exists that icon in SVG format or if You have it prepared please let me know

Hey @dubielzyk-expensify, could you please share a transparent svg of our round E logo? Like this one https://github.com/Expensify/App/blob/main/assets/images/expensify-logo-round-transparent.png but in SVG form. Thanks!

What about label translation for Spanish, is `'Ir a Expensify Classic' ok?

@kadiealexander : Can we use this?

What about label translation for Spanish, is `'Ir a Expensify Classic' ok?
Yes, that works @barttom

@dubielzyk-expensify that should work great thanks!

@barttom let us know if you need anything else for this one!

@kadiealexander Thanks a lot, that's all I need.

I saw in the other issue it doesn't look to be a blocker. @srikarparsi got assigned so I think should resolve shortly but I think we'll be okay to keep moving forward.

It looks like this worked on my expensify.com account when I'm already signed in to my www.expensify.com but when I signed up for a fresh account and tried to use the button, it is forcing me to sign into www.expensify.com when I would think I should just go right into expensify.com without needing to sign in.
This is reproducable by doing these steps:

Sign up on new.expensify.com using a gmail account. This requires you to use a magic code, which should validate the account.
Navigate to Settings > Go to Expensify Classic
Rather than than signing into expensify.com, it takes me to the below. The expected behavior would be that I'm signed in.

Can we confirm this should allow you to be signed in already? If so, we should update this to work properly.

@danielrvidal I might be wrong but I think we don't have any functionality to support logging in from newdot to oldot, only the other way around.

@barttom I reproduced the issue @danielrvidal is describing except I was required to enter another magic code when I clicked "Go to Expensify Classic"

In a similar line to what @pecanoro mentioned, is this expected for new accounts? Or should we expected to be able to navigate from NewDot to OldDot from a fresh account without having to sign in again?

As I understand, OldDot and New Dot are separated apps. Because of that, we have separated authentications.
If a user signed up in OldDot before, it should see inbox. In another case - it should be redirected to the inbox after login.
What I can do in NewDot is just redirect to another app.

@barttom we'll get back to you on next steps, it sounds like this might be expected behaviour.

@barttom do you mind fixing these issues as follow-up? (as they're within regression period)

https://github.com/Expensify/App/issues/33284
https://github.com/Expensify/App/issues/33519

@situchan  I'm OOO till the 1st January. If  they can wait I can take them when I’ll be back

Just for reference, my comment here does not actually apply. I looked at this with our engineering team and if you are not signed into OldDot already, you should need to still sign in so please disregard my comment.
After the two fixes are done that @situchan I think it should be good.

Not overdue, waiting for @barttom to come back from OOO to deal with the remaining issues

Not overdue, Melvin, see above comment

@pecanoro @situchan Should I proceed mentioned issues here or send proposals for each issue separately?

They're regressions and I don't think you need to send proposals but directly create follow-up PR linking those issues.
Or if needed, you can post proposal here.

OK, so I go with PRs then

What is the status with this one? What do we still need to do to close it out?

@danielrvidal PR is on review, I reassigned it to our original C+ instead so we can speed it up.

Please re-add the payment date to the GH title when this is ready to pay. :)

@thesahindia don't forget the checklist please!

@thesahindia bump!

@thesahindia is OOO sick, no update.

Not overdue, please switch back to daily when the regression fix is deployed.

Not overdue, regression was merged 2 days ago.

I have updated the payment amount to reflect the regression. @thesahindia please request payment via NewDot.


[Dubbo-3370]Support merger annotation

What is the purpose of the change
Add merger for Reference annotation
This PR is related to this Issue ( #3370 )
Brief changelog
org.apache.dubbo.config.AbstractReferenceConfig
org.apache.dubbo.config.annotation.Reference
org.apache.dubbo.config.spring.beans.factory.annotation.AnnotationPropertyValuesAdapterTest
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3497 into master will increase coverage by 0.84%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #3497      +/-   ##
============================================
+ Coverage     63.37%   64.22%   +0.84%
- Complexity       71       73       +2
============================================
  Files           685      664      -21
  Lines         30447    28793    -1654
  Branches       5009     4834     -175
============================================
- Hits          19297    18492     -805
+ Misses         8828     8075     -753
+ Partials       2322     2226      -96

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/dubbo/config/AbstractReferenceConfig.java
88.23% <0%> (-5.52%)
0 <0> (ø)

.../apache/dubbo/qos/protocol/QosProtocolWrapper.java
64.1% <0%> (-17.95%)
0% <0%> (ø)

...che/dubbo/remoting/transport/mina/MinaChannel.java
43.42% <0%> (-10.53%)
0% <0%> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
70.73% <0%> (-9.76%)
0% <0%> (ø)

.../remoting/transport/netty4/NettyServerHandler.java
61.9% <0%> (-9.53%)
0% <0%> (ø)

.../dubbo/remoting/transport/netty4/NettyChannel.java
61.17% <0%> (-4.71%)
0% <0%> (ø)

...e/dubbo/remoting/transport/netty/NettyChannel.java
57.64% <0%> (-4.71%)
0% <0%> (ø)

...bo/rpc/cluster/support/FailbackClusterInvoker.java
67.21% <0%> (-3.28%)
0% <0%> (ø)

.../exchange/support/header/HeaderExchangeServer.java
65.09% <0%> (-1.89%)
0% <0%> (ø)

...pache/dubbo/remoting/transport/AbstractServer.java
47.91% <0%> (-1.05%)
0% <0%> (ø)

... and 27 more

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 9031a4c...da2031a. Read the comment docs.

Hi, sorry to keeping your pull request too long. Would you please resolve the conflict first?

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

why this pr hasn't been merged? Are there still some conflicts?

@xxlv hi, thanks for your contribution
pls resolve conflicts with the latest master branch

Close for long time no response.
Please feel free to reopen if you have any question.


Add Chinese translation file (editor.json)

My Chinese co-workers translated English messages of Node-RED editor into Chinese. This translation file will be useful for a lot of Node-RED users because Chinese is one of the most popular language in the world.

Coverage remained the same at 85.927% when pulling 63198b531516af7128a5f476aa68de7cd39c4ef9 on kazuhitoyokoi:master-zh-editor.json into 74ea85d19ce126d69cb521cf9abe37ca4c099944 on node-red:master.

Thank you :) @knolleary


preferinfer modifier for type parameters

This is not yet complete - it's an idea I have and one that I want to have feedback on since some way to allow partial inference is highly in demand by library author and type parameter modifiers seems to be a natural fit for this.
I think that in reality, people would most likely opt into partial inference for the whole signature - but I'm not sure if you'd like to invent a new syntax-level concept just for this purpose. One alternative, with a completely different syntax, that I can imagine would be something like this:
declare function fn<preferinfer: A, B>(a: A, b: B): [A, B]
fn<string>('foo', 100)

This kind of thing could serve as a modifier for the list of type parameters.
The implementation here builds heavily on the @weswigham's work from https://github.com/microsoft/TypeScript/pull/26349 . The mechanism to opt into partial inference is different though. The other PR required opt-in from the caller - whereas this one here requires opt-in by the declaration author. I would consider this a major DX improvement. The downside is that this PR here only allows for inferring trailing type parameters (in the partial type parameters lists) - but I think it's a reasonable limitation/a fair tradeoff.
supersedes https://github.com/microsoft/TypeScript/pull/26349
closes https://github.com/microsoft/TypeScript/issues/26242
closes https://github.com/microsoft/TypeScript/issues/10571.

The other PR required opt-in from the caller - whereas this one here requires opt-in by the declaration author. I would consider this a major DX improvement.

This is half of the story the other half is where this is worse than #26349. It's worse because now I have to rely on the callee to "allow me" to pass in partial type arguments, and why should the callee be allowed to control my ability to pass in partial type arguments or not, it's irrelevant to the callee. That is let's say there's a function g: <T, U>(...) => ... here I wish to pass T and infer U, with #26349 I can simply write g<T, _>(...) but with this PR I need to "ask" to the callee to mark U as preferinfer and "allow me" to pass partial type arguments as g<T>(...), why should one need to drag the callee here (imagine people opening PRs for this), the caller wanting to infer U has almost nothing to do with the callee.
But I also understand the other half where it's an improvement, eg if we have something like f: <T, InternalU>(...) => ..., the improvement here afaiu would be that we need not rely or educate callers of f about partial type arguments ie we need not tell them "Hey that InternalU is an internal thing that gets inferred anyway but to allow inferring it you'll have to pass in _ so your call would look like f<MyT, _>(...)", all this messaging is not needed with this PR because now the callee can simply mark InternalU as preferinfer and the caller just passes MyT ie writes f<MyT>(...) as usual without having to deal with anything extra.
But we can achieve what this PR achieves with #26349 also, simply by declaring f as f: <T, InternalU = _>(...) => ... this would be equivalent of writing f: <T, preferinfer InternalU>(...) => .... I'm not sure if #26349 implements this already or not but it can done.
So what I'm saying is that this PR doesn't just solve a problem, it also creates another one (ie having to "ask" the callee to "allow" partial type parameter inference). But if #26349 allows (if it already doesn't) _ as the default type parameter type then it solves the same problem this PR solves but doesn't create a new one.
I hope I've got things right, just my two cents.

But I also understand the other half where it's an improvement, eg if we have something like f: <T, InternalU>(...) => ..., the improvement here afaiu would be that we need not rely or educate callers of f about partial type arguments ie we need not tell them "Hey that InternalU is an internal thing that gets inferred anyway but to allow inferring it you'll have to pass in _ so your call would look like f<MyT, _>(...)", all this messaging is not needed with this PR because now the callee can simply mark InternalU as preferinfer and the caller just passes MyT ie writes f(...) as usual without having to deal with anything extra.

This is my primary motivation behind this PR.

But we can achieve what this PR achieves with https://github.com/microsoft/TypeScript/pull/26349 also, simply by declaring f as f: <T, InternalU = _>(...) => ... this would be equivalent of writing f: <T, preferinfer InternalU>(...) => .... I'm not sure if https://github.com/microsoft/TypeScript/pull/26349 implements this already or not but it can done.

This probably isn't as easy as it sounds here because InternalU might need a reasonable default in case it fails to be inferred.

imagine people opening PRs for this

I agree that I didn't think this part through.

The bottom line for me is that some mechanism for this is needed. This is one of the most requested TS features - especially desired by library authors. I don't care much about the actual mechanism that will allow this - I could even live with _. The PR for _ has been open since 2018 and didn't move forward - I'm not sure why that is. I'm just trying to revive the conversation about this pressuring need with this new take on the feature.
I also mention in the PR description that this isn't a 1-to-1 replacement for _. Both could co-exist. Since my use case is closer to InternalU, I prefer not to force _ on the caller - it's my callee that decides about this parameter being "internal". _ would certainly be a workable (but not ideal) solution for us though.

This probably isn't as easy as it sounds here because InternalU might need a reasonable default in case it fails to be inferred.

I don't follow this, what would happen in case of an inference failure is what happens today, ie the type parameter gets resolved to the constraint or whatever. I'm not suggesting a new behavior, g<T>() would be just a sugar for writing g<T, _>() and we understand the behavior of latter.
In case you're asking if <preferinfer A extends B>(...) is equivalent to <A extends B = _>(...) then we what is the equivalent of <preferinfer A extends B = C>(...) then the answer if we can make up a way/syntax to express that, eg we can say it'd be <A extends B = _ | C>(...), ie here | doesn't actually mean a union it's just a hint for compiler.
But again the mechanism and behavior of what should happen when is already there it's just about bringing a new syntactic aid, so it's definitely easy, or at least that's what I call easy.

I also mention in the PR description that this isn't a 1-to-1 replacement for _.

I couldn't find where you mention or indicate that. Moreover you also explicitly write...

supersedes #26349
closes #26242
closes #10571

...which indicates otherwise.
Overall I agree there's a need for this kind of feature, but as I said it's better to extend #26349 with = _ that'd be simple and straightforward.

I don't follow this, what would happen in case of an inference failure is what happens today, ie the type parameter gets resolved to the constraint or whatever. I'm not suggesting a new behavior, g() would be just a sugar for writing g<T, _>() and we understand the behavior of latter.

The constraint is usually the default too - but that's not always the case. You can use different values for both (within the boundaries of the allowed subtype relationship). It's not necessarily something that I'd like to do but this proposal removes such a possibility (since you use _ as a default type argument).

In case you're asking if (...) is equivalent to (...) then we what is the equivalent of (...) then the answer if we can make up a way/syntax to express that, eg we can say it'd be <A extends B = _ | C>(...), ie here | doesn't actually mean a union it's just a hint for compiler.

Yeah, this is what I was getting at. I don't think that overloading the unions to add hints for the compiler would be a good syntax for this but perhaps something different could be used.

I couldn't find where you mention or indicate that. Moreover you also explicitly write...

Oh, sorry! I could swear that I included such a note somewhere in the description but apparently, I didn't. It is a given though - since this doesn't preclude merging that other PR as well. As you have observed - this PR mainly fixes the problem of "internal" type parameters, which is a slightly different problem than the one solved by that other PR. I mostly copied the issue references from the other PR as the overlap is, IMHO, sufficient enough. I've seen PRs addressing some of the issues by providing alternatives etc and not necessarily by implementing the requested feature/bug fix "literally". I don't have a problem with tweaking those links to smth like "relates to" though. That's not really that important to me here 😉

I don't think that overloading the unions to add hints for the compiler would be a good syntax for this but perhaps something different could be used.

Yeah that's a secondary issue. Another option is instead of <A = _> we could use <A?> that actually looks much more clear and well understood. And then <A extends B = _ | C> simply becomes <(A extends B)? = C>. (? outside following that it's [(infer A extends B)?] and not [infer A? extends B])

Oh, sorry! I could swear that I included such a note somewhere in the description but apparently, I didn't.

Haha, no problems.

@Andarist I'm marked this as a draft since the design of partial inference was never finished.

@sandersn that’s understandable. Is there any chance that we could revive the design process of this feature anyhow? It’s highly requested and I would love to work on the actual implementation. Perhaps as a 5.1 goal? 😉

@Andarist the first step in the design process is to open a bug to discuss the design.


Android 6.0 only Media issue

This is a

[x] Bug
[ ] Feature Request

Which plugin does this impact:

[ ] Battery
[ ] Connectivity
[ ] Contacts
[ ] DeviceInfo
[ ] ExternalMaps
[ ] Geolocator
[x] Media
[ ] Permissions
[ ] Settings
[ ] Text To Speech
[ ] Vibrate
Other:

Version Number of Plugin: latest
Device Tested On: Android 6 devices
Simulator Tested On: Android 6
Expected Behavior
When I click on a button to load a photo gallery, I get asked for permission, and the photo gallery should pop up.
Actual Behavior
After you give permission for Android 6 to access the camera, the photo gallery doesn't pop up. Even after you press the button multiple times.
Steps to reproduce the Behavior
See above.

This issue was moved to jamesmontemagno/MediaPlugin#14


python3Packages.bandit: 1.7.5 -> 1.7.6

changelog: https://github.com/PyCQA/bandit/compare/1.7.5...1.7.6
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 273208 run on aarch64-linux 1

6 packages built:

bandit (python311Packages.bandit)
bandit.dist (python311Packages.bandit.dist)
prospector
prospector.dist
python310Packages.bandit
python310Packages.bandit.dist

@ofborg build python311Packages.bandit

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

6 packages built:

bandit (python311Packages.bandit)
bandit.dist (python311Packages.bandit.dist)
prospector
prospector.dist
python310Packages.bandit
python310Packages.bandit.dist


Added index and appendix to neuroscience

Issue: 5284

looks like neuroscience spec files could use an update. you can update with

run neuroscience test locally with rspec spec/recipes_spec/main_spec.rb:L#
run ./scripts/update_recipes_spec_data
check that expected_output for neuroscience has changed in the expected way

Unfortunately, while fixing the expected output, ran into some errors with injected exercises update from main. Will keep an eye on main for the fix, then proceed.


Fixed player name formatting for ausclanwar

Before fix:

After fix:

Also closes #53 I've updated the README accordingly


Bug in get_between_tokens()

In https://github.com/HazyResearch/snorkel/blob/master/snorkel/lf_helpers.py
should grab left tokens of span 0 if span 0 is second, not left tokens of span 1.

Closed in v0.5.0


ADBDEV-4936-21: Add assertions to return from dynamic_cast.

Add assertions to return from dynamic_cast.
dynamic_cast may return NULL, so I added assertions.

https://github.com/arenadata/gpdb/pull/807


[wip] Add upload functionality to send methods

I'm working towards implementing #3.
The first commit in this pull request implements a servant type for endpoints expecting a request with multipart/form-data, since that's what the Telegram API uses.
Due to type incompatibilities between Network.HTTP.Client.MultipartFormData (which works directly with Request from http-client) and servant-clients Req data type), I've had to copy two functions from servant-client-<IP_ADDRESS> and add a function argument to them.
I'd appreciate feedback on this approach. Please note that the code compiles but hasn't been tested yet.

I had small investigation on servant support for multipart data a few months ago. There is an open issue 133 regarding file upload. So I decided to leave uploading feature pending until servant will support it. If you provide working solution for telegram api it would be great! Of course it's better to contribute to servant, but temporary solution is also nice to have.

I've implemented and tested file upload for photos.
@klappvisor Can you do a code review, please?
I will implement uploading for the other media types when we agree on the API.

@klappvisor Done. Is there anything else that you would like to have changed?


Retire legacy script

We can now retire this from the repo:
https://github.com/tenstorrent/tt-metal/blob/main/scripts/build_scripts/build_with_profiler_opt.sh
It is only used in one workflow, and its usage can be replaced by either build_metal.sh or with direct CMake usage.
cc: @mo-tenstorrent

build_metal.sh is not building programming examples. build_with_profiler_opt.sh does.
Can we add building programming examples as an option too? cmake --build build --target programming_examples
build_with_profiler_opt.sh is referenced at multiple places in docs. Docs assume programming examples are built as well.

Good catch.
Is there no
‘add_subdirectory(programming_examples)’ anywhere?
I think it’s a good idea to add an option for that.

Searched build_metal.sh and didn't find anything.


Add prettier to eslint config

Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

[x] I am running the latest version
[x] I checked to make sure that this issue has not already been filed

Problem

Our client-side code formatting is inconsistent.

Solution

We can use prettier along with eslint to handle formatting.

I am working on this


contrib: Remove CHARTS_PATH dependency

This script would check for CHARTS_PATH and CHARTS_TOOL, but it never
actually invoked them. We've renamed the script now so it's out of date.
Remove the check altogether, we can automate running the Helm bits later.
Related: https://github.com/cilium/charts/pull/114

Marking for backporting since I hit this while doing current releases.

@nebril I'd highly suggest using the scripts from main during releases, we don't generally backport / maintain older branch versions of the scripts. Hopefully soon we'll migrate it all over to cilium/release so we can delete them from this tree altogether.

I'd highly suggest using the scripts from main during releases

yeah i think this makes sense, maybe we just need to improve the release check list. i always get confused from which branch i'm running what command 😵


Escape single quote

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Link to JIRA: https://issues.jenkins.io/browse/JENKINS-66658
Corrects merged PR: https://github.com/jenkinsci/workflow-job-plugin/pull/214
"peuvent sétendre" -> "peuvent s'étendre"

Merge conflicts. Not sure why this PR is currently shown as adding a file.

Looks to be abandoned.


Fix typo

a -> an

@changeworld : Thanks for your contribution! The author, @msangapu, has been notified to review your proposed change.

thanks @changeworld .


don't set quota-project

TODO: probably revert the entire prior commit instead

also TODO: set up easier iteration on cloud build / terraform tomorrow to root cause the lingering issues.

what we can see after this is the SA doesn't "have access" to the project.
seems to be resource manager API b/c cross project. discussed with arnaud, will likely resolve with enabling the API

setting the project works fine after enabling resource manager api


The namespace for Controller

The namespace for Controller is  Microsoft.AspNetCore.Mvc but it is in Microsoft.AspNetCore.Mvc.ViewFeatures package !

Indeed, assemblies/packages and namespaces don't/can't always match. With MVC generally we recommend using the Microsoft.AspNetCore.Mvc package, which brings in all the requirements for ASP.NET Core MVC applications.


[sv/alert] add alert support in cip_base_scb

In cip_base_scoreboard, add the following alert_esc_agent support:

Add tlm_analysis port to detect alert sent from DUT
Use process_alert_fifos task to filter out illegal alert_items (ping
responses and sig_int_err). If real alert happened, or real alert
handshake finished, both condition will call "process_alert_sig"
function.

Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

@rasmus-madsen i hope this PR helps with supporting alert_handler in AES.
To add alert in TB, you can add list_of_alerts in aes env_cfg like hmac did here
Regarding the conversation we had through email, I would like to clarify a little bit about the idea to "add anything in place to handle the alerts with appropriate actions?"
I would like to get more details about your suggestion here.Currently in cip_base_vseq.sv, we have "run_alert_rsp_seq_nonblocking" task to response to alert_p/n
Current way is to manually add this task to body() task.
If we want to support alert_response in place, are you suggesting adding it in body() in cip_base_vseq.sv and let all the extended IPs call super.body()? (Right now I believe non of the IPs call super.body() from cip_base_vseq.sv)


Cache ticker - CIK mappings

Instead of consulting the SEC service every time that we need a CKI from a ticker, we should just call the service once and cache the results. Further consumptions of that info can read from the cache.
Caching possibilities:

Redis
Memcache
Node-cache

Done! caching is done when the webapp initialises and the data is ready for use on further requests.


Auto install workplace extensions

I would like a way to install workplace extensions from the code CLI. The use would be to automate install of extension when users use yarn install or npm install using a postinstall script in the workplace. That way we can avoid users manually needing to install extension.

@sandy081 Are you open to contributions for this change?

Yes, we do!


Upgrade drone to 0.8

This not only upgrade drone to 0.8 but also greatly improve Drone life-cycle management. All the docker configuration is in one file for both the server and the agent.
One can simply restart drone with the docker-compose up command.
This is the recommended way to install Drone. http://docs.drone.io/installation/
The main difference with the previous installation is that both the server and the agent run on the same machine. I don't see a good reason for running on two machines that justify the added complexity. I guess this is up for debate.
A temporary instance of drone0.8 is running here.

LGTM


MRI Reconstruction [R]

App name
dashr-brain-surface-viewer
App

[ ] This app is currently deployed on DashR Gallery: [deployment URL]
[x] This app is new

Source code repository
[source code repo URL]
Current appearance
[attach screenshot of current app here]
Redesign
Link: https://github.com/plotly/streambed/issues/12068#issuecomment-462442220

Link: https://github.com/plotly/streambed/issues/12879#issuecomment-488130571

Playground deployment
[url of playground deployment]

MRI reconstruction.zip

@KPhans bump. Is this relevant still?

@lundstrj nope!


Question regarding dealing with automatically paging queries...

When I call a function like Tentacat.Issues.list it automatically blocks until all the pages are downloaded.  This is awesome - except if possible I'd like some kind of indication as to how it's going.
If there isn't a way to increase the verbosity - could we maybe get an optional callback that's called every time a HTTP request is made so we can instrument something.
Thanks,
Red

Hi @redvers , have you checked the pagination options and see if one of those satisfy your requirements? https://hexdocs.pm/tentacat/Tentacat.html#get/4

Hey there! Im trying to set pagination: :none to the filter function from the Pull module, but I I can't make this work... is it not available at that level?
Tentacat.Pulls.filter(client(), "myrepo", repository, %{
        "state" => "close",
        "per_page" => 1,
        "page" => 1,
        "pagination" => "none"
      })

Thanks


Latest version missing from npm

Was trying to figure out why the new <!-- action-docs-usage --> syntax wasn't working for me no matter what I tried when I realized that that was added in v1.3, but npm only has the package up to v1.2: https://www.npmjs.com/package/action-docs

I also ran into this.  I wanted to use this project, but I guess I will have to look at alternatives for now.

@nnellanspdl, I've just updated the package. You can now install the package with the action-docs binary from GitHub (see https://github.com/npalm/action-docs/pull/505)

This is no finally solved. Version 1.3.0 never made it to npm. The new release is no again on NPM.


Feat/sketching initial ideas

I have just started to sketch out some of my ideas for the architecture. Im no Java Export so it would be very nice to hear your thoughts on my current idea. Feel free to change what you want to change, this pull request should just help us think about the architecture we want.

Can we please decide on whether we want to use this structure as a base and merge it into the main branch ASAP, so that we can start building the UI as well as the back end in separate branches with this structure.

Yes, let's go ahead with that! I haven't heard any objections, so I think we can start with this structure. If we have any concerns along the way, we can always make adjustments. 👍🏻

I just can't approve it myself. You are welcome to approve and merge it 👍🏻


Getting "Document is not defined" when using in Nextjs project

Describe the bug
I am using just the Cropper element and it is giving me a "document is not defined" error in  my nextjs project. is there something additional i need to do to configure this project?

I suspect https://github.com/ValentinH/react-easy-crop/releases/tag/v4.5.1 being the issue. Could you please try version 4.5.0 and let me know if it works for you? If yes, we can easily add a fix I think.

I am using "react-easy-crop": "^4.7.0",
I actually fixed it by putting <Cropper into a component and then dynamically including it
const ImageCropper = dynamic(() => import('components/reuseable/image-cropper'), { ssr: false })


Failed connection to server

I followed the description but it didn't work because the server is not reachable.
I comformed that Docker is running.
Can anyone teach me how to get out of this problem.

open the server terminal and press r + enter to refresh the connection

Are there any logs from the server?

If you've already troubleshooted server socket status, please ignore below steps.
Here are steps to troubleshoot server connectivity and websocket issues:

Check if the server is listening:

Confirm if the server is correctly set up to listen for connections. You can verify this using the following command on Mac and Linux (note: sudo access required):lsof -nP -i4TCP:3000

Example output indicating the server is listening:COMMAND   PID    USER   FD   TYPE DEVICE SIZE/OFF NODE NAME
Python  24241 iamuser  3u  IPv4 123456      0t0  TCP <IP_ADDRESS>:3000 (LISTEN)
Python  24248 iamuser  3u  IPv4 123456      0t0  TCP <IP_ADDRESS>:3000 (LISTEN)

Server console log check:

Check the server's output to ensure it's running correctly? You should see something like this:uvicorn opendevin.server.listen:app --reload --port 3000

With the expected output being:INFO:     Uvicorn running on http://<IP_ADDRESS>:3000 (Press CTRL+C to quit)
INFO:     Started reloader process [24241] using WatchFiles
INFO:     Started server process [24248]
INFO:     Waiting for application startup.
INFO:     Application startup complete.

Testing websocket connection:

If the server is listening, try testing a websocket connection using the following commands:brew install websocat
websocat ws://<IP_ADDRESS>:3000/ws

Keep an eye on the server logs for a new websocket connection. The logs should indicate a successful connection:INFO:     ('<IP_ADDRESS>', 59395) - "WebSocket /ws" [accepted]
INFO:     connection open

Reporting Errors:

If you encounter a stack trace due to an exception on the server side after following the above steps, please paste the stack trace here for further analysis.

If you've already troubleshooted server socket status, please ignore below steps.
Here are steps to troubleshoot server connectivity and websocket issues:

Check if the server is listening:

Confirm if the server is correctly set up to listen for connections. You can verify this using the following command on Mac and Linux (note: sudo access required):lsof -nP -i4TCP:3000

Example output indicating the server is listening:COMMAND   PID    USER   FD   TYPE DEVICE SIZE/OFF NODE NAME
Python  24241 iamuser  3u  IPv4 123456      0t0  TCP <IP_ADDRESS>:3000 (LISTEN)
Python  24248 iamuser  3u  IPv4 123456      0t0  TCP <IP_ADDRESS>:3000 (LISTEN)

Server console log check:

Check the server's output to ensure it's running correctly? You should see something like this:uvicorn opendevin.server.listen:app --reload --port 3000

With the expected output being:INFO:     Uvicorn running on http://<IP_ADDRESS>:3000 (Press CTRL+C to quit)
INFO:     Started reloader process [24241] using WatchFiles
INFO:     Started server process [24248]
INFO:     Waiting for application startup.
INFO:     Application startup complete.

Testing websocket connection:

If the server is listening, try testing a websocket connection using the following commands:brew install websocat
websocat ws://<IP_ADDRESS>:3000/ws

Keep an eye on the server logs for a new websocket connection. The logs should indicate a successful connection:INFO:     ('<IP_ADDRESS>', 59395) - "WebSocket /ws" [accepted]
INFO:     connection open

Reporting Errors:

If you encounter a stack trace due to an exception on the server side after following the above steps, please paste the stack trace here for further analysis.

I've got same issue, so I followed your guide, and I received the error message as below.
(OpenDevin) root@a451579031d9:/OpenDevin# uvicorn opendevin.server.listen:app --port 3000
INFO:     Started server process [8064]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:3000 (Press CTRL+C to quit)
INFO:     ('<IP_ADDRESS>', 52480) - "WebSocket /ws" [accepted]
INFO:     connection open
Please check Docker is running using `docker ps`.
Error! Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
Failed to stop container: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
Error creating controller.

Error message left on terminal running websocat ws://<IP_ADDRESS>:3000/ws
(OpenDevin) root@a451579031d9:/OpenDevin# websocat ws://<IP_ADDRESS>:3000/ws
{"error":true,"message":"Error creating controller. Please check Docker is running using `docker ps`."}

Thank you for posting advices.
Because  I am novice, I cannot find server terminal, I cannot find logs.
I tried measures shown above, but unfortunately my system is windows11 and I couldn't use "losf" command.
I retried and found that no container is running.
"docker ps" command shows no containers running.
I assume that this is the main cause of this problem.
How do you guys think?
Here's my computer setups:
Windows 11 Pro
intel Core i5-8400
Mem 40GB

I also have the same problem, running on Ubuntu 22 on WSL2

@hyeonmini
In the case of macOS:

Ensure that Docker Desktop is running
In Docker Desktop, under advanced settings check the case “Allow the default Docker socket to be used (requires password)”.

This should resolve the issue.

I also have the same problem, running on Ubuntu 22 on WSL2

@shahab00x , Were able to troubleshoot using https://github.com/OpenDevin/OpenDevin/issues/344#issuecomment-2027840656?
@Fujimon416 , I haven't used Windows machine for a while. Here is a link to understand the commands
Get-NetTCPConnection -State Listen

You want to find if 3000 and 3001 are on the list.

@neeleshbisht99 has a good point. Check if your OS allows socket.

I also have the same problem, running on Ubuntu 22 on WSL2

@shahab00x , Were able to troubleshoot using #344 (comment)?
@Fujimon416 , I haven't used Windows machine for a while. Here is a link to understand the commands
Get-NetTCPConnection -State Listen

You want to find if 3000 and 3001 are on the list.

I tried that but everything looks fine. I think the problem is with docker using a different subnet than wsl and the main network. Is there a way to modify uvicorn opendevin.server.listen:app --port 3000 to access the server over the network rather than the localhost?

Is there a way to modify uvicorn opendevin.server.listen:app --port 3000 to access the server over the network rather than the localhost?

Yup! You should be able to add --host <IP_ADDRESS>

This is becoming a megathread of failed connections 😅 As such it's probably not super helpful.
If anyone is still stuck, please open a new issue with:

The commands you ran to start the app
The environment/config variables you used (redacting your API keys)
Everything output from uvicorn after running that command
A screenshot of the /ws item in the network tab of your browser's console (CMD+Shift+C or CTRL+Shift+C ususally)

Is there a way to modify uvicorn opendevin.server.listen:app --port 3000 to access the server over the network rather than the localhost?

Yup! You should be able to add --host <IP_ADDRESS>

Yes! It worked.
I changed the VITE_WS_URL environment variable to the external IP address of WSL and also ran this command uvicorn opendevin.server.listen:app --port 3000 --host  <IP_ADDRESS>

I've the same problem on mac m1. I use colima as container. Finally fixed it by soft link the socket file as:
sudo ln -s -f /Users/david/.colima/docker.sock /var/run/docker.sock
sudo chmod +r /var/run/docker.sock

Great find! Docker socket issue is common. Did you keep the error message
that pointed you to the docker socket issue? Sharing it could help others
facing the same problem.
On Sun, Mar 31, 2024, 12:40 PM david l euler @.***>
wrote:

I've the same problem on mac m1. I use colima as container. Finally fixed
it by soft link the socket file as:
sudo ln -s -f /Users/david/.colima/docker.sock /var/run/docker.sock
sudo chmod +r /var/run/docker.sock
—
Reply to this email directly, view it on GitHub
https://github.com/OpenDevin/OpenDevin/issues/344#issuecomment-2028818280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKR3B4RNXVUHJW7IZA46MYTY3A4ATAVCNFSM6AAAAABFOTVCE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRYHAYTQMRYGA
.
You are receiving this because you commented.Message ID:
@.***>

We might need an "alternative setups" section for colima, windows, ollama etc


Support for siafund API calls in siac

Plus the addition of the SpendSiafunds API call.
The calls should be correct but I haven't verified that they work as expected. I imagine it would be easy to test them with the .siakey files using testing constants, yes?
btw, if you assign PRs they'll show up when you click the "Pull requests" link the header. Maybe we should start doing that.

I like the idea of assigning pull requests.
That's the reason the types package has keys now, very easy for you to test the siac commands manually by loading those keys into the wallet and then spending from them.

Points of confusion:

Siafunds can't be sent to siacoin addresses, and there's no way for siad to check whether you're sending to a siacoin address by mistake. Need to educate users on what siafund addresses are and how to get them.
No zero-conf, and other weirdness -- need to mine a block to clear up certain problems.

how badly would it break things if I added a checksum and address type right now?

it wouldn't be terrible, but I think it would make more sense to merge this first


New package: Li.CXMLDecompiler version 11

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?
closes #201708

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.9 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Manual Validation ended with:

2024-12-23 10:45:15.593 [FAIL] Installer failed security check. Url: https://silica.codes/Li/cxml-decompiler/releases/download/v11/CXMLDecompilerv11.zip Result: 0x80004005

(Automated response - build 931.)

Manual Validation ended with:

2024-12-25 13:23:33.146 [FAIL] Installer failed security check. Url: https://silica.codes/Li/cxml-decompiler/releases/download/v11/CXMLDecompilerv11.zip Result: 0x80004005

(Automated response - build 931.)

@wingetbot run

Manual Validation ended with:

2024-12-27 09:48:54.879 [FAIL] Installer failed security check. Url: https://silica.codes/Li/cxml-decompiler/releases/download/v11/CXMLDecompilerv11.zip Result: 0x80004005

(Automated response - build 931.)

@wingetbot run


fix: switch tx count to number

change tx_count to (int) number

:tada: This PR is included in version 1.0.0-brc-20.25 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Issue is resolved

Description
(Write your answer here.)
Checklist

[x] I am making a proper pull request, not a spam.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
(Write your answer here.)
Add relevant screenshot or video (if any)
(Add here)

Please add the issue you were assigned

#252 Footer should be improvised @nilisha-jais

Hey @tanya070820 just check in the preview of the deployed link. The footer does'nt look fine

Ok Ma'am

@niloysikdar What happen Sir I am not getting your point.

Go to this link and see the changes.

Sir very sorry for the mistake but can you please help me how can I improve it

No worries @tanya070820 , we can definitely fix that.  Just pull the changes first. Then you are good to go.

@niloysikdar Please have a look on the final one

I have seen the changes, but there are so many things which aren't working. You can also see through the deployed link.

@niloysikdar @nilisha-jais Please have a look .

Conflict is resolved please have a look.

Hey @tanya070820 , if you please open the deployed link you can see that the preloader, navbar, footer, nothing is working, it is causing too many bugs on the website. @nilisha-jais  @harshita2216 @Keerthanam1029 please review this pr.

Sir as I have compared it from original one then the things which were working their are also working here

Okay @tanya070820 ,don't worry. Other reviewers will review this.

@tanya070820 still not working fine. Please  Check this. The preloader and navbar are disrupted.
Do one thing close this pull request, update your fork and then push the changes


Reductions won't accept lists anymore

Solves #3038 doing what was advised on #3050 by @jakevdp

So, I am allowing tuples as input, otherwise we will lose jnp.prod(shape), which would be a shame

So, right now it's failing on the masking test and parallel test, with the same error, can you guys help me out? @shoyer @hawkinsp
Sorry for bothering

From the test failures, it looks like the tests are passing a list to a jax function that no longer accepts lists. One of the issues comes from the fact that reductions attempt to convert their inputs into arrays: https://github.com/google/jax/blob/fd886b17a48f23637f3fcae810117721f155e16c/jax/numpy/lax_numpy.py#L1520
But the test of reductions is passing a list: https://github.com/google/jax/blob/e36c72b9839efe8a98a75d93f87661f55b585ddb/jax/experimental/jax_to_tf/tests/tf_ops_test.py#L283-L284

A general comment: I'm a bit wary of this pull request as written. The problem is, it removes support for codepaths that are likely employed by many users (passing lists to jax functions), and so will almost certainly introduce breakages into user code.
For any change like this, we should be displaying a deprecation warning for a few releases: that is, if a user is passing input that currently works and will not work after this change, it should not break immediately, but rather raise a FutureWarning with instructions about how to fix the issue. Then, once that warning has been present for a few release cycles, we can make the breaking change.

Great idea Jake, this would be quite disruptive to codebases as it stands indeed, I opened a PR with the warning

Hey folks. I just encountered the future warning about reductions on list and tuples in the future. However, (and I only read quickly) - most of the chatter in here seemed to be about lists and excluded tuples from this change.
I just wanted to check in if this was intended. I've got a quite a few cute little tuple comprehensions that use jnp.sum and I'll rewrite them now (they are under review) if this is the direction JAX is taking. But if not, ..., well they are cute.

Hi @btanner - we are indeed moving to a world where tuples & lists will not be silently converted to device arrays by jax functions. If this is really what you want, you can always do it explicitly with something like jnp.sum(jnp.asarray(tup)). But if your tuples are small, the overhead of converting them to device arrays before summing is probably not what you want, and if your tuples are large enough that it is what you want, the overhead of constructing them as Python tuples (versus, say, native jax array operations) is probably not what you want.

Hi @btanner - we are indeed moving to a world where tuples & lists will not be silently converted to device arrays by jax functions. If this is really what you want, you can always do it explicitly with something like jnp.sum(jnp.asarray(tup)).

To be explicit, the other option (ideal for short tuples) looks like functools.reduce(jnp.add, tup)


Follow recommendations and try both diff-lcs version on 2.7.1

After Jon's comment on https://github.com/rspec/rspec-support/pull/430#discussion_r495807025
Related: https://github.com/rspec/rspec-support/pull/421/commits/548624db45f93676f2d56f134d86d1dc9d6933a6

It should work given that's what rspec-support had, want to open the PRs?

I should open the related PR at the same time. I will do it next time.


Create a job posting form

Related to #59

Develop a user-friendly form where employers can input job details such as job title, description, responsibilities, qualifications, experience level, location, and application deadline.
Include validation checks to ensure that all required fields are filled and that the entered information meets the specified criteria.

Work in progress for the employer Job post feature.

The task has been completed hence closing it for the sprint.


Add documentation for installing rustup completion as non root user

The instructions for adding rustup completion for bash as a non root
user were missing. Said instructions were already added for the other
shells.

Strange, this instructions didn't work on my machine.
$ bash --version
GNU bash, version 4.4.19(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Hmm, it works for me when I copy and paste the two commands as-is. I tested it on Fedora 29 and Ubuntu 18.04.2 LTS. The bash version on Ubuntu is the same as yours:
GNU bash, version 4.4.19(1)-release (x86_64-pc-linux-gnu)

Troubleshooting steps:

Do you have the bash-completion package installed and working?

Does your shell source /etc/profile in ~/.bashrc?
Did you apply any manual modifications to /etc/profile, the files in /etc/profile.d, or the files in /usr/share/bash-completion/?

Do you have either of the following environment variables set?

$BASH_COMPLETION_USER_DIR

Defaults to $XDG_DATA_HOME/bash-completion when not set

$XDG_DATA_HOME

Defaults to ~/.local/share/ when not set

@ArifRoktim It's my fault for not sourcing /etc/bash_completion in .bashrc.
Maybe you could add that caution in this PR.
It is weird! Last time I use bash interactively, I don't need to source /etc/bash_completion.
In /etc/skel/.bashrc, it states that
# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if ! shopt -oq posix; then
  if [ -f /usr/share/bash-completion/bash_completion ]; then
    . /usr/share/bash-completion/bash_completion
  elif [ -f /etc/bash_completion ]; then
    . /etc/bash_completion
  fi
fi

But in /etc/bash.bashrc, those lines has been commented:
# enable bash completion in interactive shells
#if ! shopt -oq posix; then
#  if [ -f /usr/share/bash-completion/bash_completion ]; then
#    . /usr/share/bash-completion/bash_completion
#  elif [ -f /etc/bash_completion ]; then
#    . /etc/bash_completion
#  fi
#fi

I'm on Ubuntu 18.04.2 (fresh install).

@lzutao So I spun up a fresh install of Ubuntu in a VM to see if bash completion works by default. After running the rustup completions ... command and opening a new shell, rustup completion worked.
I looked at the default shell config files and at the very end of ~/.bashrc(and also /etc/skel/.bashrc) were the following lines:
# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if ! shopt -oq posix; then
  if [ -f /usr/share/bash-completion/bash_completion ]; then
    . /usr/share/bash-completion/bash_completion
  elif [ -f /etc/bash_completion ]; then
    . /etc/bash_completion
  fi
fi

I took a look at /etc/bash.bashrc and also found those commented out lines. I guess Ubuntu decided that it's better to have the completion-enabling code in the user .bashrc to make it easier for users to turn off the feature if they wanted to.
Fedora takes a different approach. The default .bashrc just sources /etc/bashrc. That file then sources all files that fit the glob /etc/profile.d/*.sh, and /etc/profile.d/bash_completion.sh matches that glob.

I think in general, bash completion usually "just works" out of the box for most Linux distros. The exact method that the shell uses to enable bash completion is distro specific. Perhaps I should add that users should first check to make sure that completion works for their specific shell?

:umbrella: The latest upstream changes (presumably #1704) made this pull request unmergeable. Please resolve the merge conflicts.

Hi @ArifRoktim
Would you mind rebasing your change rather than having a PR with merge commits in?
I think that if you do that, we should be in a position to merge this pretty soon.
Thanks,
D.

:umbrella: The latest upstream changes (presumably #1646) made this pull request unmergeable. Please resolve the merge conflicts.

Hi @ArifRoktim
I merged some changes which put you in need of a rebase again.  Since it was in part my fault for taking so long to process your PR I've rebased it for you and think it's good.  Could you please check what I've done and if it's good then we'll try and merge your submission :D
D.

I might have already covered this with #1646. If I overshadowed your work, I'm sorry.

@yodaldevoid
Indeed you have. But it's fine, these things happen.
I've also been annoyed by not having cargo completion for a while so I'm happy for your pr being merged :smile:
@kinnison
There are some issues with the rebase but since #1646 obsoletes this pr, I'm going to close it instead. Thanks for working with me on this pr even if it is closed.


Use common OIDC token env vars for live tests.

Support oidc az cli login in post deploy scripts for multi-cloud. Requires https://github.com/Azure/azure-sdk-tools/pull/9099

API change check
API changes are not detected in this pull request.

/azp run js - identity - tests

/azp run js - identity - tests-weekly

/azp run js - identity - tests-weekly

/azp run js - identity - tests-weekly


Replace sparklines data & line items in Fabric view tabs

Currently, the sparklines data in the fabric view tabs is just displaying # of microservices, which is not very interesting.

I literally have no idea what sort of metrics would make sense here. I can't think of any data that we have that would be helpful to the user in this context.

@tilleryd - Any ideas?

If there's no useful data to display, then this is my problem to fix. I'm not happy with the way these tabs look when there's just a title and single detail line, but maybe it'll improve when icons are in place.

Once we have proper icons in place, lets remove the sparklines.


Typescript error in DOMUtils.d.ts?

With the following dependencies:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
I get the following typescript error:
node_modules/@types/history/DOMUtils.d.ts(3,81): error TS2304: Cannot find name 'EventListenerOrEventListenerObject'.
I also have "lib": ["dom", "es2017"] set in tsconfig.json. This name exists in node_modules/typescript/lib/lib.dom.d.ts - why would it not be picked up by @types/history?

We don't maintain @types/history in this repo.


Update from v0.9.2 to v1.0.0

This PR creates the v1.0.0 release of terrainbento on conda-forge. It is associated with publication of a paper in GMD.

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Stuff that is not great and we should address some day

This is a place to write down stuff that doesn't make a ton of sense and should be changes one day.

Border colors is using extraMuted as default. But also has name called default
<Pane appearance="dark" /> is never used.
<Pane appearance="selected" /> is only rarely used.
Positioner (maybe) should use Popper.js just like BlueprintJS.  It will be more reliable.
Our icons should allow for any svg
Our icons should not be wrapped in a Box I think. I like how react-icons exports their icon works.
We should consider deprecating SubHeading since it's never used.

<Button> doesn't accept String height, e.g.: height={majorScale(4)}

That's because the height is used to calculate the padding etc. We should change majorScale() to return numbers instead since ui-box automatically converts numbers to px values anyway.

minorScale and majorScale should be exported from evergreen-ui.

The Text component should contain a class name to style following elements inside of it. Which would avoid having to manually import and configure Strong, Small etc.
<Text>You can use the mark tag to highlight text.</Text>
<Text>This line of text is meant to be treated as deleted text.</Text>
<Text><s>This line of text is meant to be treated as no longer accurate.</s></Text>
<Text>This line of text is meant to be treated as an addition to the document.</Text>
<Text>This line of text will render as underlined</Text>
<Text>This line of text is meant to be treated as fine print.</Text>
<Text>This line rendered as bold text.</Text>
<Text>This line rendered as italicized text.</Text>

SideSheet should have the same API as Dialog with onCloseComplete.

Show types in documentation for Dialog
Show clearer onConfirm example in code

Alerts should use Paragraph not Text for children


collection inputs from the notebook

see nbinspect  from nb2workflow

see renku functionality for types can be used. Maybe additional plugins hooks can be used for domain-specific input annotations.

there's a little tool that adds some additional info of a notebook: https://github.com/agoose77/jupyterlab-knowledge-graph


git, curl, openssl: Refactor $NIX_SSL_CERT_FILE handling

Motivation for this change
The way NIX_SSL_CERT_FILE (and previously SSL_CERT_FILE) support leaks from openssl to git, passing by curl.
This changes shows that

we can avoid patching git by fixing curl's patch
we can avoid patching curl by making it rely on openssl's default CA
we can enable the feature proposed in #23723 and dropped with refactor at point 2.

These points correspond to the three commits. The commits should be reviewed separately to make sense.
It is an improvement/rewrite of #24121
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] 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/)
[?] Fits CONTRIBUTING.md.

/cc @shlevy @domenkozar @edolstra @vcunat

cc @shlevy

This should also go to staging-17.03 if it works.

Nice!

Thanks @shlevy :-).
Would you mind reviewing this on darwin, when hydra is ready ? Especially check that it does not introduce a regression for #23723. I have no darwin machine at hand.
PS: Is there a way to add regression tests for stuff like #23723 ?

I don't have a darwin box either, sadly. @copumpkin ?


update deprecated interface

This change is

Coverage decreased (-0.01%) to 43.63% when pulling a1d04a7397e93e7bf52987e73d6b4891161d1aad on hzxuzhonghu:master into 7b4d8cb1c30aaf9224ea8ad1981bad458d3b8892 on kubernetes:master.

/lgtm

@hzxuzhonghu thanks!


What happens after "Tracker is lost" warning?

Hello,
I guess the title is self-explaining. Just to add to my question - is it possible to re-gain tracking pose somehow after this warning is sent? Because I get enormously big numbers after tracker is lost in rectified_pose. Let's say 4x10^27 in translation axes.

One more addition - smooth_pose continues to work correctly (with accumulated drift ofc).

Hi @eMrazSVK,
It is possible that tracking can resume, however, be aware that during the period where tracking is lost VO can drift. If it drifts too much, it is advisable to restart the VO. For the smooth pose, as you mentioned in the question, it will continue to give out the pose with accumulated drift.


[Feature request] invoke ag interactively

Category

[X] fzf binary

OS

[X] Linux

Shell

[X] bash

I've used fzf and fzf.vim and they are great! Ag command in fzf.vim will allows us to:

:Ag <pattern> will return the search result and pipe it to fzf
fzf will allows us to narrow down the above result to get what we want.

However, in ag's case, more often, we may want to change the <pattern> dynamically instead of changing the query to narrow down the result of a specific <pattern>. I'll give an image as example of this:

skim is a rust version of fzf, which provides an "interactive" mode to dynamically invoke the commands.
I understand the role of fzf is a general purpose "filter", and the above feature may be far from what fzf tries to be. Though as an end-user, I'd really like fzf to have this feature. Please consider about it, thanks!

I don't have enough time these days, so I can make any promises, but I'll think about it.
Anyway, here are some concerns that come to my mind.

Rebuilding the input list from scratch can feel clunky. The list will be cleared, cursor will always be brought down to the bottom of the screen, tab-selected entries will be lost. And depending on the command used, there can be noticeable initial delay which will make the user wait until the first entry from the new command pops up.
So restarting ag command every time you press a key can be an overkill, especially when the selectivity of the initial query is good. Isn't it silly to restart ag command when there are only a handful of matches?. It may suffice to rebuild the list only when some key is pressed such as CTRL-R (r for refresh). Note that it is possible to implement the idea externally using --expect and --print-query options, though it requires closing the current fzf window and opening a new one. So the experience will not be as nice, but in that way, we can keep the simplicity of fzf.
Search syntax of fzf is not compatible with the regex used by ag command. Passing fzf query directly to ag command may or may not make sense. (e.g. !test 'run)

Since I don't have answers to the above questions, I'll close the issue for now.
Actually, it's a bit hard for me to see the point of the example. If we restart ag on every change of the query string, fzf no longer works as a filter, but simply becomes a selector interface for external process, which is not the problem fzf was designed to solve. It would make more sense if we restart the ag process not on every update of query, but on an explicit request. But in that case, we can already implement the feature using --expect as I mentioned above.

I think it would be good to look back into this feature request. There are cases when having possibility to actually rerun a command makes more sense, e.g. when initial amount of lines is really big. For instance when using some full text search engine instead of grep. I found some workarounds to for such case, but they are making it more cumbersome. Instead, taking inspiration from skim would be an interesting option.
I've created a PR to give this idea a try. Here's some initial observations:

Rebuilding the input list from scratch can feel clunky.

It seems that the performance is pretty good on Linux and Mac.

So restarting ag command every time you press a key can be an overkill

In certain situations yes, but then it should be noted that "interactive mode" is better suited in cases when we could actually benefit from having a subset of results already provided by a command itself. E.g. when initial result set is huge.

Search syntax of fzf is not compatible with the regex used by ag command.

Definitely yes. It's hard to imagine that other commands would have the same syntax for filtering. In this case "interactive mode" shouldn't assume the "extended-search mode" and pass everything as-is or maybe it should implement a key binding for switching off interactive mode and then filtering based on fzf syntax. I think skim does that.
I've created a pull request to give it a try.


"Can not find compilation library location for package.." error right after installing .NET Core 2.0 SDK

I have an ASP.NET Core 1.1 website that targets .NET Framework 4.6.1. Now after installing .NET Core SDK 2.0, I can't view any of the pages because of the error:

InvalidOperationException: Can not find compilation library location for package 'xxx'

Below is the stack trace. Not sure if it's really Razor-related. The site works after uninstalling .NET Core 2.0 SDK.

Microsoft.Extensions.DependencyModel.CompilationLibrary.ResolveReferencePaths()
Microsoft.AspNetCore.Mvc.ApplicationParts.AssemblyPart+<>c.b__8_0(CompilationLibrary library)
System.Linq.Enumerable+d__16.MoveNext()
Microsoft.AspNetCore.Mvc.Razor.Compilation.MetadataReferenceFeatureProvider.PopulateFeature(IEnumerable parts, MetadataReferenceFeature feature)
Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationPartManager.PopulateFeature(TFeature feature)
Microsoft.AspNetCore.Mvc.Razor.Internal.RazorReferenceManager.GetCompilationReferences()
System.Threading.LazyInitializer.EnsureInitializedCore(ref T target, ref bool initialized, ref object syncLock, Func valueFactory)
Microsoft.AspNetCore.Mvc.Razor.Internal.RazorReferenceManager.get_CompilationReferences()
Microsoft.AspNetCore.Mvc.Razor.Internal.DefaultRoslynCompilationService.CreateCompilation(string compilationContent, string assemblyName)
Microsoft.AspNetCore.Mvc.Razor.Internal.DefaultRoslynCompilationService.Compile(RelativeFileInfo fileInfo, string compilationContent)
Microsoft.AspNetCore.Mvc.Razor.Internal.RazorCompilationService.Compile(RelativeFileInfo file)
Microsoft.AspNetCore.Mvc.Razor.Internal.CompilerCache.CreateCacheEntry(string relativePath, string normalizedPath, Func<RelativeFileInfo, CompilationResult> compile)
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Mvc.Razor.Internal.CompilerCache.GetOrAdd(string relativePath, Func<RelativeFileInfo, CompilationResult> compile)
Microsoft.AspNetCore.Mvc.Razor.Internal.DefaultRazorPageFactoryProvider.CreateFactory(string relativePath)
Microsoft.AspNetCore.Mvc.Razor.RazorViewEngine.CreateCacheResult(HashSet expirationTokens, string relativePath, bool isMainPage)
Microsoft.AspNetCore.Mvc.Razor.RazorViewEngine.OnCacheMiss(ViewLocationExpanderContext expanderContext, ViewLocationCacheKey cacheKey)
Microsoft.AspNetCore.Mvc.Razor.RazorViewEngine.LocatePageFromViewLocations(ActionContext actionContext, string pageName, bool isMainPage)
Microsoft.AspNetCore.Mvc.Razor.RazorViewEngine.FindView(ActionContext context, string viewName, bool isMainPage)
Microsoft.AspNetCore.Mvc.ViewEngines.CompositeViewEngine.FindView(ActionContext context, string viewName, bool isMainPage)
Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewResultExecutor.FindView(ActionContext actionContext, ViewResult viewResult)
Microsoft.AspNetCore.Mvc.ViewResult+d__26.MoveNext()
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__30.MoveNext()
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__28.MoveNext()
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ResultExecutedContext context)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(ref State next, ref Scope scope, ref object state, ref bool isCompleted)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__22.MoveNext()
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ResourceExecutedContext context)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(ref State next, ref Scope scope, ref object state, ref bool isCompleted)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker+d__20.MoveNext()
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Builder.RouterMiddleware+d__4.MoveNext()
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Session.SessionMiddleware+d__9.MoveNext()
Microsoft.AspNetCore.Session.SessionMiddleware+d__9.MoveNext()
System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware+d__7.MoveNext()

@pranavkm can you take a look to see what's going on here? Wrong SDK being used maybe?

Do you have an repro application that you could share with us?

I created a very simple repro below. The site won't run if .NET Core 2.0 is installed. Apparently it only happens when you add a reference directly to a .NET Framework dll (point to the bin/*.dll of ClassLibrary2). It works if you add a reference to a project.
https://github.com/jomargon/DotNetCore20IssueRepro

@jomargon thanks for the repro. This is essentially a dup of https://github.com/aspnet/Home/issues/2126. I have a work around for it here: https://github.com/aspnet/Home/issues/2126#issuecomment-322325498

@pranavkm Thanks! Any chance we won't need the workaround on the next minor update? I'd rather wait for a fix than apply the hack.

@pranavkm should this bug be closed as a dup of https://github.com/dotnet/core-setup/issues/2981 ?

@Eilon yup. I'll go ahead and close this.
@jomargon I'm working with the CLI team to get the fix in for the 2.0.1 release.

This issue is still happening and not resolved in 2.0.2.
At the same time I do not know for sure if this is resolved in 2.0.2 or not? I see many still seem to have the issue - there are several comments. workarounds, closed as duplicate threads etc., but there is nothing that I could find that clearly says if this is fixed in 2.0.2 or when it will be fixed..
Any help and understanding of our pain will be much appreciated.

I am also facing the issue in 2.0.2. I logged the defect in #https://github.com/dotnet/core-setup/issues/3397


There is no python2 image with nightly tag

Currently compose setup OpenWhisk with python2 runtime, but there is no docker image with nightly tag for python2.
So users are unable to invoke python2 action.
2019-08-07T08:31:09.972Z] [ERROR] [#tid_4AmAKSd56LiUSH63j2NI1GULHlSjqaT1] [DockerClientWithFileAccess] info: command was unsuccessful, code: 125 (unsuccessful), stdout: , stderr: Unable to find image 'openwhisk/python2action:nightly' loc
/usr/bin/docker: Error response from daemon: manifest for openwhisk/python2action:nightly not found.

We need to drop python2 for compose based setup or add "nightly" image for python2.

I believe the decision was made to stop building the python2 image a while back because python2 is end-of-life January 1, 2020.

we should retag the "latest" image to "nightly" though - there's still a lot of p2 code out there.

Why not use 1.13.0-incubating as the tag?  it's the same bits as 'latest'.   I could add a 'nightly' as well I suppose, don't really care.

Nightly tag added as alias to 1.13.0-incubating/latest.

I confirmed we can invoke python2 action with docker-compose based setup.
Thanks @dgrove-oss


Bug fix: Resuming after pause

Currently, pausing breaks resets all the timers and counters.
As one may expect from the word "Pause", it'd be better to keep them as they were after the pause period is over.

Hey @hovancik. What is your take on this? If this is free to take, I would like to work on it. Said that, if you think I should start with some other issue please do let me know.
Thanks :D

Hey @hovancik. What is your take on this? If this is free to take, I would like to work on it. Said that, if you think I should start with some other issue please do let me know.
Thanks :D

Hi @armudgal , this is free to take, but it requires some thinking. Stretchly does not have any "state" and I am pretty much sending a lot of events up and down. And it's all connected(with DND, idle time and such).
Here are some related cards: https://github.com/hovancik/stretchly/issues/503, https://github.com/hovancik/stretchly/issues/772
Feel free to take a look at the code and write here how would you solve it?

Hi @armudgal , this is free to take, but it requires some thinking. Stretchly does not have any "state" and I am pretty much sending a lot of events up and down. And it's all connected(with DND, idle time and such).
Here are some related cards: https://github.com/hovancik/stretchly/issues/503, https://github.com/hovancik/stretchly/issues/772
Feel free to take a look at the code and write here how would you solve it?

Wouldn't it be better if you were prompted to take a break immediately after the pause? Or at least if the countdown continues running while paused but once it reaches zero it's paused there? I imagine that if you're using a productivity app and don't want to be interrupted and spend a significant amount of time using it, that after you're done would be a good time to take a break. In my use case (after zoom meetings) I'd like a break reminder then instead of 15 minutes later.

Well, I guess depends how you use pause, whether it means you take your own break, or it's a time when you don't wanna be disturbed.

closing in favor of #1248


Parse/Generate call signature with keeping argument name

I want to do like this.

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

This would be very helpful when combined with #26190 as we could fix the problem that the inferred parameter names are off by one there.

Seems to me this can be accomplished using ReturnType<T> (added to lib.d.ts in 3.0) and the Parameters<T> type I suggest here https://github.com/Microsoft/TypeScript/pull/24897#issuecomment-397463626. The only one thing we don't support is extracting the parameter names, but that doesn't appear to be significant in the example.

@ahejlsberg the Parameters<T> type doesn't preserve the original argument names or TSDoc comments. I think what @whatasoda wants is very similar to the new feature request I just created #30351.


Container level diagrams are not linked to components

When looking at the container level digram the components within that container are not linked to the relevant component view if it exists. It would be nice if I could click on a component and have it move to that diagram automatically.

Yeah, this is a good idea. Lets wait first how https://github.com/avisi-cloud/structurizr-site-generatr/pull/149 / https://github.com/avisi-cloud/structurizr-site-generatr/pull/149#issuecomment-1451393506 turns out for the tab support which is imho also useful here.


icons dont load sometimes in safari

overview
when using safari sometimes the icons dont load
investigate why and see if we can fix it
or at a minimum, if icons dont load hide them entirely instead of displaying the label
seems like it could be an issue with ligatures?
adding text-rendering: optimizeLegibility may help

[Error] Origin https://demo-validator-a2.chef-demo.com is not allowed by Access-Control-Allow-Origin.
[Error] XMLHttpRequest cannot load https://telemetry.chef.io/segment/api_keys due to access control checks.
[Log] Error retrieving Segment API key: 0/undefined (main-es2019.16d125b6df3242c1fba9.js, line 1)
[Error] Failed to load resource: Origin https://demo-validator-a2.chef-demo.com is not allowed by Access-Control-Allow-Origin. (api_keys, line 0)

Selected Element
< …

I have notice that clearing the browser cache has resolved the issue temporarily.

folks think this may be related to...
https://github.com/panthershark/safari-cache-bug

Closing due to Safari cache bug stated above


Css snapshot

Resolves https://github.com/opentelekomcloud/ansible-collection-cloud/issues/143

recheck


Add Horizontal Cards

I think would be good to add horizontal cards.

How so? What do you mean by horizontal cards?

Do you mean like this: https://getbootstrap.com/docs/4.0/components/card/#card-groups or this: https://getbootstrap.com/docs/4.0/components/card/#card-decks ?

Please search first next time. Duplicate of #20794.

No I mean something like this http://materializecss.com/cards.html#horizontal

Yeah, that's the same as the issue I referenced.