Not working

During the setup process, "Test" button says connection successful. But when I see the tasks, it says "Couldn't connect" with Configure server option.

I get this too

Any updates on this?

While you wait, you can build your own copy using my PR https://github.com/mayankmkh/linear-app-plugin/pull/211

It's been a while now that a PR has been merged for this, but I don't see any new version coming.
Is there a release plan, because I'm used to use this all the time with Jira and really wish I can switch to Linear.
Thanks in advance

@mayankmkh Sorry for the ping
Since the PR is merged, could u please prepare a release ? This is a good plugin, and I'd like to avoid forking it to make it works

@SirMishaa are you sure the PR is merged?

@SirMishaa https://github.com/Benbentwo/linear-app-plugin/ is a different repo, forked from https://github.com/mayankmkh/linear-app-plugin/issues

Correct, That link is my personal fork @SirMishaa. Feel free to use it, though I have no plans, nor time ATM, to maintain this. You can use my fork to mess around with the plugin, but ideally the above PR would be merged.
Sorry for any confusion on merges, I prefer to Link related issues.


Default drag & drop

I've implemented IDropTarget interface because I need to get drag event when moving a datagrid row in another position of the same datagrid.
I don't know how implement DragOver and Drop. Is there a way to call the DefaultDropHander methods ?
Thanks

I've found the solution myself
    public void DragOver(IDropInfo dropInfo)
    {
        GongSolutions.Wpf.DragDrop.DragDrop.DefaultDropHandler.DragOver(dropInfo);
    }

    public void Drop(IDropInfo dropInfo)
    {
        GongSolutions.Wpf.DragDrop.DragDrop.DefaultDropHandler.Drop(dropInfo);
        // DO STUFF
    }


some issues for insert word into sentence

i like to type for a sentences first before i bold it or make for the format,the problem is when i want to insert the ** or word into the sentence the word behide will be override. hope to solve the problem.tq.

This is editor related, suggest you send in an suggesting to Obsidian.md.


Naive guard against UITableView NSInternalInconsistencyException crash

This PR adds a preliminary and naive guard against performing updates on UITableView that are likely to raise an NSInternalInconsistencyException.

Codecov Report

Merging #6 into master will increase coverage by 30.49%.
The diff coverage is 94.91%.

@@             Coverage Diff             @@
##           master       #6       +/-   ##
===========================================
+ Coverage   55.19%   85.68%   +30.49%
===========================================
  Files          20       21        +1
  Lines         732      880      +148
===========================================
+ Hits          404      754      +350
+ Misses        328      126      -202

Impacted Files
Coverage Δ

Sources/BasicData/BasicDataSource.swift
88.23% <ø> (ø)
:arrow_up:

Sources/Updates.swift
94.11% <87.5%> (+44.11%)
:arrow_up:

Sources/UITableView/UITableView+Updates.swift
82.75% <90.47%> (+82.75%)
:arrow_up:

...es/UICollectionView/UICollectionView+Updates.swift
86.36% <94.11%> (+86.36%)
:arrow_up:

Tests/UIKitViewUpdateTests.swift
96.18% <96.18%> (ø)

Sources/BasicData/BasicSection.swift
100% <0%> (+42.85%)
:arrow_up:

...urces/UICollectionView/CollectionViewFactory.swift
48.71% <0%> (+48.71%)
:arrow_up:

Sources/UITableView/TableViewFactory.swift
49.15% <0%> (+49.15%)
:arrow_up:

... and 3 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 fd9c700...d5da35c. Read the comment docs.


Let BSD variants use sendmmsg and add GitHub action for compiling on FreeBSD

Code looks good, though there's been a bunch of commits post you asking for review, so I'm not 100% sure if it's ready to merge. Can you merge when ready?

Yeahhh. Wanted to test my new Github Action to test compiling on FreeBSD on every PR, but the only way to test was to commit and push and run here. Sooooo, hence the 10k commits lol. I did add the #do-not-review label to try to prevent this 😄

I still have a few things left to do (test using sendmmsg on FreeBSD in a VM), then I'll request a re-review and we can merge.
cc: @zakird

Sounds good. I missed label 🤦🏽 Usually just look at "Draft" status, but will keep my eye out!!


Not working in Release mode

I used stream listener for vpn connection detector. It is working on dubug mode but not working on release mode. Please have a look

@Sayed-Siam123  can you share more details like your flutter version and device model


timesave fork for testnet with constant spawnmap

This makes hunters and banks spawn always near harvest areas.
It also adds protection for newly spawned hunters and a spectator mode.
Fee for a new hunter and destruct fee is set to 1 HUC.
The refundable fee per hunter is set to 100 HUC.
Compared to the obsolete old version (branch "...timesave4"),
init.cpp is now completely unchanged, and the (player- and bank-)
SpawnMap is a constant array, similar to ObstacleMap.

Thanks a lot for the fixes, very nice!  ACK, merging.


Added new domain

This is what the new students use

E.g<EMAIL_ADDRESS>

@Jbjokke Pull request merged. Thank you!


Examples are missing for devfile UX when experimental is enabled

/kind feature

Which functionality do you think we should add?
Talkative examples for experimental feature for example devfile UX when experimental is enabled.
Why is this needed?
Because it is missing in UX and ofcourse for better user experience.
What is missing : devfile command example is missing in the example section when experimental is enabled.

$ ODO_EXPERIMENTAL=true odo push devfile --help
Push source code to a component.

Usage:
  odo push [component name] [flags]

Examples:
  # Push source code to the current component
  odo push

  # Push data to the current component from the original source.
  odo push

  # Push source code in ~/mycode to component called my-component
  odo push my-component --context ~/mycode

Flags:
      [...]
      --devfile string       Path to a devfile.yaml (default "./devfile.yaml")
      [...]
      --namespace string     Namespace to push the component to
      [...]

$ ODO_EXPERIMENTAL=true odo create --help
Create a configuration describing a component.

 [...]

Usage:
  odo create <component_type> [component_name] [flags]

Examples:
  # Create new Node.js component with the source in current directory.
  odo create nodejs

  # Create new Node.js component named 'frontend' with the source in './frontend' directory
  odo create nodejs frontend --context ./frontend

  # Create new Java component with binary named sample.jar in './target' directory
  odo create java:8  --binary target/sample.jar

  # Create new Node.js component with source from remote git repository
  odo create nodejs --git https://github.com/openshift/nodejs-ex.git

  # Create new Node.js component with custom ports, additional environment variables and memory and cpu limits
  odo create nodejs --port 8080,8100/tcp,9100/udp --env key=value,key1=value1 --memory 4Gi --cpu 2

Flags:
      [...]
  -n, --namespace string    Create devfile component under specific namespace
      [...]

/area devfile

Expected implementations

[ ] Add odo create examples for devfile component to the help page
[ ] Remove --namespace flag usage when experimental mode is disabled

/assign

@amitkrout there are already devfile specific examples included in UX for odo create -h and odo push -h .
for create :
https://github.com/openshift/odo/blob/master/pkg/odo/cli/component/create.go#L127
for push :
https://github.com/openshift/odo/blob/master/pkg/odo/cli/component/push.go#L37

Right, then why is not being reflect the in the UX. Will check it again, may me it has been updated through some pr.

any update on this @amitkrout

any update on this @amitkrout

checking

On master, i can see the missing devfile examples. It has been fix through some other pr i guess.


Handle update conflict in agentdeploy controller

We see these periodic error messages logged:
base_controller.go:159] "addon-deploy-controller" controller failed to sync "prod-on-prem/submariner", err: Operation cannot be fulfilled on managedclusteraddons.addon.open-cluster-management.io "submariner": the object has been modified; please apply your changes to the latest version and try again
Assuming no other controller is making frequent updates to the ManagedClusterAddon, the most likely reason for the update conflicts is due to caching. The controller obtains the ManagedClusterAddon from the informer's cache so it may not get the most up-to-date version of the resource which can cause the transient conflict errors when updating the resource. This can happen if the resource is immediately re-queued after an update and the updated version hasn't reached the cache yet when the controller's sync function is invoked again.
The log messages are benign but they are disconcerting and can fill up the log with noise. To avoid the logging, on conflict error, simply re-queue the resource instead of returning the error.

/cc @skitt @sridhargaddam

/assign @zhiweiyin318

@zhiweiyin318 @qiujian16 An integration test failed.  I don't see how my changes could've caused that, plus they all succeed locally for me. Has there been past issues with flaky test failures? Unfortunately I don't have the ability to re-run the job (not that I can see).

@tpantelis  We are trying to merge controllers to reduce conflicts. If the result is good, we can avoid modifying base_controller.

@tpantelis  I refactored agentDeploy controller PR-172 and fix issue in work applier PR-233 . the most of conflict has been avoided.  please update the latest addon-framework lib to have a try.


Static configuration via the enrollment list - Need more info?

Where I should store these static configurations?

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

ID: ec5e18dd-edbd-2978-d8bf-7739a1e54cea
Version Independent ID: ce89eed6-ef5a-bde4-9955-a5e7d99a6945
Content: Service concepts in Azure IoT Hub Device Provisioning Service
Content Source: articles/iot-dps/concepts-service.md
Service: iot-dps
GitHub Login: @nberdy
Microsoft Alias: nberdy

The static configuration is used by specifying the iotHubHostName of the IoT hub you want your device(s) to be registered to. It's stored in the enrollment object.

#please-close

@isharehappycode We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Install package error

Hi I am following the install instructions in the readme and after step where I enter the below command into my RPi, I get an error where it seems to not be installing some commands
I run the command:
sudo apt-get update && sudo apt-get install git apache2 php5 libapache2-mod-php5 php5-mcrypt expect geoip-bin
Then see this:
`Hit:1 https://deb.nodesource.com/node_12.x buster InRelease
Hit:2 http://archive.raspberrypi.org/debian buster InRelease
Hit:3 http://raspbian.raspberrypi.org/raspbian buster InRelease
Hit:4 http://raspbian.raspberrypi.org/raspbian bullseye InRelease
Reading package lists... Done
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package php5 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
Package libapache2-mod-php5 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
Package php5-mcrypt is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Package 'php5' has no installation candidate
E: Package 'libapache2-mod-php5' has no installation candidate
E: Package 'php5-mcrypt' has no installation candidate`
I thought maybe this was a non-issue, but then when I try to run
sudo nano /etc/apache2/apache2.conf
the file is blank
then when I try
sudo service apache2 restart
I see
Failed to restart apache2.service: Unit apache2.service not found.
Any ideas what I'm doing wrong? I already have PiVPN (via WireGuard) up and running.

Append this to your sources.list(run sudo nano /etc/apt/sources.list)
deb http://mirrordirector.raspbian.org/raspbian jessie main contrib non-free rpi
Then re-run the command.


MultiSelect without requirement to implement Display

Is your feature request related to a problem? Please describe.
I want to select paths, but PathBuf doesn't implement Display, but it has display() method
Describe the solution you'd like
Allow creating MultiSelect when elements doesn't implement Display, but if you provide custom formatter. Maybe something like
MultiSelect::new_with_formatter("text", [paths, ...], &|p| p.display()).prompt()?;

Describe alternatives you've considered
-
Additional context
-

You should be able to wrap a PathBuf in a newtype and implement Display for it quite easily:
use std::{fmt::Display, path::PathBuf, str::FromStr};

fn main() {
    let path = PathBuf::from_str("/home").unwrap();

    println!("{}", DisplayPathBuf(path));
}

// Newtype definition
struct DisplayPathBuf(pub PathBuf);

// Implementing Display for the newtype
impl Display for DisplayPathBuf {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        write!(f, "{}", self.0.display())
    }
}

Thank you @Baarsgaard, that is the recommended pattern to use with inquire.
Previously the approach was to pass a value formatter but the new type pattern allows for better structure and is more idiomatic


Webpack 2.3 is breaking builds

See #768.
The ^ in the webpack config is resulting in the use of 2.3.x for Webpack and that gives this error:
-react-jsx-self<EMAIL_ADDRESS>the API schema.
- configuration.output.path: The provided value "../app/assets/webpack" is not an absolute path!
icromatch@^2.1.5ap@0.0.1-browserify<EMAIL_ADDRESS>Fetching packages...
warning<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
e/rof/src/github.com/shakacode/react_on_rails/gen-examples/examples/basic/client && $(yarn bin)/webpack --config webpack.config.js
info No lockfile found.
[1/4] Resolving packages...
<EMAIL_ADDRESS>the API schema.
- configuration.output.path: The provided value "../app/assets/webpack" is not an absolute path!
rake aborted!
Command failed with status (1): [cd /home/rof/src/github.com/shakacode/reac...]
/home/rof/src/github.com/shakacode/react_on_rails/rakelib/task_helpers.rb:20:in `block in sh_in_dir'
/home/rof/src/github.com/shakacode/react_on_rails/rakelib/task_helpers.rb:20:in `each'
/home/rof/src/github.com/shakacode/react_on_rails/rakelib/task_helpers.rb:20:in `sh_in_dir'
/home/rof/src/github.com/shakacode/react_on_rails/rakelib/examples.rake:40:in `block (3 levels) in <top (required)>'
/home/rof/src/github.com/shakacode/react_on_rails/rakelib/examples.rake:34:in `block (4 levels) in <top (required)>'
Tasks: TOP => examples:build_webpack_bundles_basic
(See full trace by running task with --trace)

https://app.codeship.com/projects/187011/builds/23733772
The short-term fix is to lock Webpack at 2.2.1.
We'll be updating to 2.3.1 shortly.

The issue is caused by breaking change in Webpack itself.
The output.path required to be an absolute path in webpack v2
In webpack 2.2.x there was no absolute path validation, but in 2.3.x it is added.
Here is my PR to fix this issue
https://github.com/shakacode/react_on_rails/pull/771


Rename .NET Core to .NET

Rename .NET Core to .NET
.NET Core is legacy name and was renamed to .NET in 5.0 release

In that case, also change the URL to be canonical -- the existing URL redirects. The Framework URL my need to be changed too?

Link updated in 6640eb4b52c0d7effe9de986a88afe105ec26fd7
It still redirect to language specific version, but I would keep it as it is.
Change for .NET Framework are not needed.


Fix membership resolver used in integration tests

What changed?

Fix membership resolver used in integration tests

Why?
Matching simulation test fails because the identity of matching hosts are the same and it fails the tasklist ownership check.

How did you test it?
Existing tests and matching simulation test

Potential risks

Release notes

Documentation Changes

This makes a heck of a lot of sense and looks correct and reasonable to me, but: what was failing before? I can guess how this fixes it, but this all probably belongs in the commit message / PR description.

The matching simulation test is failing on some API calls after this PR got merged.
The AddDecisionTask and PollForDecisionTask API fails due to TaskListNotOwnedByHostError


Support hash join in substrait-to-velox conversion

I have finished rebasing my changes to Velox. You can raise your PR to https://github.com/oap-project/velox.


Need the ability to query a product

As a user
I need the ability to query the product information based on personalized filters I set up
So that I can get the detail information of a product I'm looking for.
Details and Assumptions

Create RESTful API that has query function
Add query string support to List function

Acceptance Criteria
Given I have a product service
When the GET /products request is made
Then a JSON file is returned which contains the details of a product I query

We currently have list by rating and list by price, perhaps we can add a bit more

Hey team! Please add your planning poker estimate with ZenHub @dd3053 @dggjsc @hxwang-463 @mpourostad


[Bug][master] After subtask fault tolerance, 2 task instances are generated，The process instance status always is executing

1.run sub_process workflow
2.stop master
3.start master,2 task instances are generated

The master log is as follows  （master日志如下）

The worker log is as follows （worker日志如下）

Which version of Dolphin Scheduler:
-[1.3.2-release]

i am working on it.


Update logic used to detect the use of pyspark

pyspark: name.toLowerCase().indexOf('pyspark') >= 0,
Needs to change to:
pyspark: pythonPath.toLowerCase().indexOf('spark') >= 0,
That's better, as we have users creating launch.json manually.
https://stackoverflow.com/questions/56400048/how-do-you-make-spark-submit-arguments-come-right-after-spark-submit-when-debu

Closing in favor of #6091


Option for menu not to return focus to the assoicated button

Feature request:
An @Input() option for not returning focus to the associated button when the menu closes.
What is the expected behavior?
When the menu closes, not to return the focus to the button. This could be an optional behavior.
What is the current behavior?
When the menu closes after clicking a menu item, the focus returns to the associated button. However, if the command of the menu item is to focus on an input element, current behavior just breaks the intended feature which for example enables editing in the input box right away.
What are the steps to reproduce?
With the tutorial link at https://stackblitz.com/angular/pbndqaomepr?file=app%2Fmenu-overview-example.html
Copy and paste the following:
<button mat-button [matMenuTriggerFor]="menu">Menu</button>
<mat-menu #menu="matMenu">
  <button mat-menu-item (click)="focusInput()">Focus Input</button>
  <button mat-menu-item>Item 2</button>
</mat-menu>

a
b

<button mat-raised-button (click)="focusInput()">Focus Input</button>

c

d

    <mat-form-field fxFlexFill>
        <textarea matInput placeholde="MyInput" #textDetail name="note" ></textarea>
    </mat-form-field>

import {Component, ElementRef,ViewChild} from '@angular/core';

/**
 * @title Basic menu
 */
@Component({
  selector: 'menu-overview-example',
  templateUrl: 'menu-overview-example.html',
  styleUrls: ['menu-overview-example.css'],
})
export class MenuOverviewExample {
  	@ViewChild('textDetail') noteInput: ElementRef;

focusInput(){
  (this.noteInput.nativeElement as HTMLTextAreaElement).focus();
}
}

In contrast, when running the same command focusInput() in the button, the input box gets focused as expected.
What is the use-case or motivation for changing an existing behavior?
Menu command to focus on any element that can be focused. For example, adding or editing some text in an input box.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 6.0.4, Angular Material 6.2.1, TypeScript 2.7.2
Chrome and other browsers
Is there anything else we should know?
At the moment, the workaround is to use
focusInput(){
  setTimeout(()=> (this.noteInput.nativeElement as HTMLTextAreaElement).focus(),
       280);
}

280ms is for Chrome, but for Edge, the delay has to be around 400 ms. Anyway, people with eagle eyes could see the blinking of the focus changing.

Adding an option like this will break the accessibility of the menu, because it'll throw the user's focus back at the root when they close the menu. Closing the issue, because in general we don't add APIs that would inhibit accessibility. You should be able to hack around it, but at that point it would be up to you to maintain it.

refer this
[(https://www.interfacecreator.com/2018/09/material-menu-focus-issue-fixed.html)]

thanks vijaynet10. However, with the posted workaround, the menu is not closing, though the input does get focus. I had also tried to fire $event.stopPropagation() is menuClosed or closed events of the menu, not good either.

hi zijianhuang, refer this   https://stackblitz.com/edit/angularmaterialmenufocusissue

Hi @crisbeto, can I reopen this? It would be great to have such an option. Maybe we can mark the menu element to not return focus it is not feasible for the menu as a whole?
We have some use cases when we want to set focus on our own, for example, menu option to rename element(double-click is reserved for other interaction).

yep ....reopen the issue @VVKot


Log error messages without stack trace https://cloud.google.com/error…

https://cloud.google.com/error-reporting/docs/formatting-error-messages#log-error

@amineck do you want to revisit this again?


Added Bulgarian

Made keyboards, dictionary from here.
Same as this.

By the way, I saw on my other pull requests that you added screenshots and native titles/descriptions for the play store after they were merged, would it be helpful for me to add those on subsequent pull requests?

If you can add those in your original PR, that would be great.
If the PR does not contain screenshots, I take the old screenshots from the Play Store.

I've added screenshots and added more to the description.

thanks!


Modified provider name to align with partner name

Provider trades under eir evo, updating name to match, as per website already linked to service.

#sign-off


When I reboot the phone, I need to jailbreak again, otherwise I won't be able to open Untether.

When I reboot the phone, I need to jailbreak again, otherwise I won't be able to open Untether.
Me iphone11, iOS version 14.5.1
Can we solve this problem?

Same issue for me iphone 11pro and ios 14.4

iphone 11 pro on ios 14.4
After reboot, the phone is not Jailbroken.
I try it for 10 times with Unc0ver app and also with Untether app, I get the same result, the iphone reboot not Jailbroken.

This is why Fugu14 is called in Germany a Mogelpackung. Fugu14 is not untethered - it is semi-untethered 🤫

This is why Fugu14 is called in Germany a Mogelpackung. Fugu14 is not untethered - it is semi-untethered 🤫

Incorrect, Fugu14 is fully untethered. u0 is not. u0 requires two reboots to detect the untether sometimes, the 7.0.x releases were poorly made

i jailbreak it 4 times and made every time a reboot but still not jailbroken after reboot.🤷♂

Same issue here iPhone 11 Pro on iOS 14.4

Incorrect, Fugu14 is fully untethered. u0 is not. u0 requires two reboots to detect the untether sometimes, the 7.0.x releases were poorly made

i jailbreak it 4 times and made every time a reboot but still not jailbroken after reboot.🤷♂

Because unc0ver is not untethered.


Replace gas price with fee

Part of https://github.com/massalabs/massa/pull/3173#discussion_r1014817564

Should we merge this? What is the procedure for that? @Thykof

@gterzian I am asking @gregLibert for review

thanks @gterzian !

This change will only be effective in the testnet 17. This makes the latest version of massa-as-sdk not compatible with the current testnet (used for hackathon). Is that a problem ? @gregLibert @Thykof

Yes it is. We need to reverse...

Ok, restored the branch and will re-open later when we can merge...


ACM for CSP

I would like to know if CSP customers will have access to Azure Cost Management

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

ID: 59edbf44-87b8-7b7a-92a5-6ba12bdd8aff
Version Independent ID: 1d7974d8-a0f2-e2b1-cc38-565e2d7fec40
Content: Understand Azure Cost Management data
Content Source: articles/cost-management/understand-cost-mgt-data.md
Service: cost-management
GitHub Login: @bandersmsft
Microsoft Alias: banders

@mtrangel89 Hi. I can't publicly answer this today. However, we'll have news about this next week.
#please-close

@bandersmsft Thanks for the super quick response. :+1:

@bandersmsft Very well, thank you for answer fast.

@bandermsft What is the status of this now? Went from a pay-as-you-go account to CSP today without knowing cost management wasn't available.

@bcdunbar @mtrangel89 Have you seen https://docs.microsoft.com/azure/cost-management/get-started-partners?

Hello unable to activate in the subscription of my clients.

hmmm looks like "next few months" for CSP customers https://azure.microsoft.com/en-us/updates/azure-cost-management-features-in-development-october-2019/

Is there any update on this for CSP with Offer: MS-AZR-0145P ?
I would love to hear about upcoming updates on this. It has been awhile since we have heard anything. Thanks!

It looks like you are able to select Management Groups and Subscriptions as the Scope now. Can anyone confirm work is being done for the MS-AZR-0145P offer? Thanks! @bandersmsft


feat: Add 500 error page

Description
fixes #87
Demo

Checklist:

[x] I have read and followed the Contributing Guide
[x] Add error page

Few comments on this:

Hopefully, we don't have any issues causing 500 errors on production .)
The way I was developing it was to set dev: false on nuxt config. Then I removed some imports to cause an error.

the visual emoticons have not been implemented as discussed.

By the visual material there should be the title Not found but I don't think it's correct. There should be an Internal server error or Unexpected Error

The previous error pages have left alignment and here was the center. So I propose to keep it consistent as shown in the demo.


Quicksilver is Buggy on MacOS 14 (Sonoma)

There are a fair number of really bad Quicksilver bugs on MacOS Sonoma. We are working on them, but it is a slow process.
If Quicksilver is critical to your use of MacOS, I would recommend not yet updating to Sonoma if you can help it.
If you are already on Sonoma, several of the bugs can be worked around by disabling "Superfluous visual effects" (settings -> appearance), followed by a restart of Quicksilver.

If you're unable to do so (for example some people can't get past the installation screen), paste the command below into Terminal.app (followed by return), which has the same effect, followed by a restart of Quicksilver.
defaults write com.blacktree.Quicksilver "Use Effects" -bool NO

Thanks for your patience while we try to sort things out.

Thank you for working on this, I was not able to even get through the setup on a new machine.  The Terminal command above helped quite a lot.

Glad to see this isn't a sign I broke something.  (I'm having the "detached pane" issue.) I'm so grateful for the team/community's continued work on this project.  I'm a longtime user and I look forward to the continued improvement (and resolution of these issues).

Thank you for looking into this.  I just bought a new Mac and the installation process for QS was difficult.
For anyone else: if you can't click with the laptop trackpad, try an external mouse.  It's what got me to a usable state.  The trackpad makes the window jump away like crazy.

Thank you for this. I struggled a lot to get through the first screen itself, it was not just buggy, it is practically unusable. Thanks for the workaround, I had uninstalled it, now got it back.

Thank you for looking into this. I just bought a new Mac and the installation process for QS was difficult.
For anyone else: if you can't click with the laptop trackpad, try an external mouse. It's what got me to a usable state. The trackpad makes the window jump away like crazy.

unfortunately nothing worked for me. Most windows use to show up and by the time I try to go to the window, it vanishes. It was a mess, with trackpad and with mouse. I thought I will use stage manager still similar issues. Somehow i figured if i put my mouse at right place i can click through the agreement, and that helped me get past 1 screen - but this is a persistent pain.

Running QS 2.4.2 smoothly on (4041) Sonoma 14.2.1
Thanks a ton for the work and update.  Before the update I had problems with launcher begin off center and top right of screen.  No more.  Working as expected.

@winstonford that's great news, thanks for the report!

Still seeing frequent crashes with 2.4.2 on Sonoma 14.2.1. Crash log indicates SIGSEGV:
Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x0000000000000000
Exception Codes:       0x0000000000000001, 0x0000000000000000

2.3.2 is more stable for me on Ventura, I did get a crash this morning, is there a good/better place to post crash reports to help rather than become noise?
Crashed Thread:        5  Dispatch queue: quicksilver.qslibrarian.scanning
Exception Type:        EXC_BAD_ACCESS (SIGSEGV)

I'm hoping to use my Sonoma machine more today, so may have more insight.
I presume it's a separate issue, but my biggest issue in Ventura is typing ~/doc/gi/test to complete to ~/Documents/git/test and it ends up getting ~, then the / for doc gets to the / root path - as-if a path searcher returns too late and it falls back to / (using the → key instead of / works (~→doc→gi→tes), but is less convenient)

Sorry for late response.

is there a good/better place to post crash reports to help rather than become noise?

Here on GitHub is great. Submitting the crash reports will hopefully be eventually helpful, but the backlog of issues is long.

my biggest issue in Ventura is typing ~/doc/gi/test to complete to ~/Documents/git/test and it ends up getting ~, then the / for doc gets to the / root path

Perhaps open a separate issue, but this seems odd. ~/git/fauxmo selects the expected directory for me. In settings -> command, what is your Reset search after settings? Mine is 1.5s.

Not sure if I should open a separate issue, but I've noticed that after upgrading to Sonoma, my QS Catalog entries are randomly emptied of their contents.
The two that I've noticed this for are for the Web Searches plugin, and a custom folder that I've added. At random times they seem to need to be refreshed in the QS UI, because their contents aren't in the catalog anymore (UI shows no number next to the entry).
The other odd thing I've noticed since updating to Sonoma is that sometimes a custom trigger that I've set (command-option-T to open Terminal) doesn't work until I open Quicksilver to the Triggers section. Once I've done that, it works even when QS is not activated, as expected.

I'm going to close as macOS 15 is out, and 2.4.3 has several relevant fixes.


SAK-30500 Set new message event to recipient id

What is the request merge flag for? For Sakai 11? If so, what is the Sakai-11 flag for?

I thought request merge was just to bring git hub PRs to the attention of people, not a specific request to merge into 11.x or anything.


hark: allow frontend to make notifications

this PR:

allows the frontend to send a poke request to create a notification.
makes the "id" and "time" params optional for the %add-yarn action

for https://github.com/tloncorp/landscape/issues/154
Fixes LAND-505

This is actually pretty sick, thank you for front-running.

not leaving you hanging, just trying to think about how we roll this out in a way that doesn't change the api. @midsum-salrux @Fang- could use some help w/ that if y'all have thoughts.

@patosullivan thought occurred to me today, this should move to tloncorp/landscape#192 and that PR should be based off of the final desk migration so we don't lose these changes in the move.

@arthyn done

@patosullivan transitioned this to a new type + mark so that this would be backwards compatible and we can deploy much easier. @midsum-salrux if you have a sec to review would be appreciated


Integration of XUnit and TeamCity is broken

Description
XUnit has built-in integration with TeamCity
Steps to reproduce

Create xunit test project dotnet new xunit
Set environment variable SET TEAMCITY_PROJECT_NAME to some value to simulate running under TeamCity, for instance: SET TEAMCITY_PROJECT_NAME=my
Run tests using some verbosity level: dotnet test --verbosity Minimal or dotnet test --verbosity quiet

Expected behavior
You should find messages like ##teamcity[testStarted in stdOut
for all verbosity levels
Actual behavior
There are no any messages like ##teamcity[testStarted in stdOut
Environment
dotnet --version:
2.1.200
2.1.300-rc1-008673

@smadala, @martincostello please take a look at this problem. In SDK 2.1.104 all works fine, since 2.1.200 we're able to reproduce it.

@mayankbansal018, any progress on it?

@singhsarab , can you please take a look at this issue

@singhsarab, @smadala, we're getting more and more tickets about this breaking change. Please arrange the priority of this issue.

@dtretyakov This is by design as verbosity minimal and quiet, the expectation is that no informational logs should be shown for these levels.
We fixed the behavior as part of https://github.com/Microsoft/vstest/pull/1396
Recommendation is use normal or detailed verbosity for full detailed logs.

Using normal or verbose logging is not working. I can see the additional log messages, but nothing that is picked up by teamcity.

Passing --verbosity=normal as an argument to dotnet test outputs test results to TeamCity (2017.1.4) as it did in previous versions of the SDK when using 2.1.300 of the .NET Core SDK.

@singhsarab Verbosity level should not make any influence on CI integration. We should take in account that not only people run tests (they can use some IDE for this), vice versa in the most cases a CI runs tests via a command line.
Now

in the quiet mode there are now any integration at all
in the minimal (default) mode the integration works only for failed tests

It seems to be definitely breaking change, very confusing behavior and useless, thus the only thing we can recommend: "to not use these modes" by large red letters.

To illustrate this change, here is a screenshot from my application's CI build in TeamCity.
Build 1.0.1105 uses --verbosity=normal - the test results are shown.
Build 1.0.1104 uses the previous behaviour of just running dotnet test without any explicit verbosity arguments - no test results are shown at all.

From the perspective of a TeamCity user, #1396 was a breaking change.

@martincostello You're work around is not working for me.  Any preview nuget packages?
The verbosity change is not a work around to the problem, unless I'm missing something.
Here is an example "dotnet new xunit" project with the TeamCity.VSTest.TestAdapter package and couple example tests added: https://github.com/zone117x/DebugTeamCityTestReporting
Created a new build configuration with the dotnet CLI runner using test command and verbosity set to normal. Attached build logs with: attempt 1) first run, attempt 2) run with /nodeReuse:false parameter, attempt 3) running on a Windows build agent with latest stable dotnet SDK/runtime.
DebugTestReporting_Tests_attempt-1.log
DebugTestReporting_Tests_attempt-2_disable-nodereuse.log
DebugTestReporting_Tests_attempt-3_win-build_agent.log

@zone117x I'm not using the built-in .NET Core TeamCity runner - I'm using a vanilla PowerShell step that calls into my own .ps1 that invokes dotnet test itself.

Just created a custom script build step with dotnet test DebugTeamCityTestReporting.csproj --verbosity Normal /nodeReuse:false
Doesn't work.

What about if your put an equals between the parameter name and value like in my example above? Otherwise, I’m not sure why it would work for me but not for you.

Wow. I tried --verbosity normal instead of --verbosity Normal and it worked..
dotnet test should probably not require that to be case sensitive, or at least output a warning about an invalid command line parameter.
@NikolayPianikov In the mean time, Teamcity .NET build agent needs to use lowercase variables for the verbosity option.

@zone117x @martincostello thank you for info, done. Try updating to new version

@nikolayPianikov which version specifically?

@akatz0813 from this page for appropriate TeamCity version

FYI I've found that using this argument instead means that test verbosity is increased to that required but build verbosity isn't, so then the behaviour is actually like-for-like:
--logger:"console;verbosity=normal"

@NikolayPianikov May I go ahead I close the issue?

@singhsarab we just found some workaround for (https://github.com/JetBrains/teamcity-dotnet-plugin/issues/127), but I hoped it is temporary solution. Also some related issues (like https://github.com/JetBrains/TeamCity.VSTest.TestAdapter#visual-studio-test-platform) have only partially workaround.
Are you not planing to fix? It is your choice, but please note that it is quite important for any CI systems.

@NikolayPianikov Unfortunate that this came as a breaking change for you guys, but glad that you have a solution in place now. As I mentioned earlier, this change in behavior is actually a fix we made to ensure we are consistent. The current behavior is by design.
The solution you guys have now seems appropriate for specification.
This change is not a take back as such, because logger verbosity can be configured for detailed logging.
Please let me know if you disagree. Thanks.

@martincostello I just bumped into the same issue! Saw your other tickets as well. I just upgraded from 2.1.101 to 2.1.300 and was getting no tests on TeamCity as well. I agree that this is a breaking change and I don't having to pass more arguments than before. It works passing --logger:"console;verbosity=normal" but.. was better before.

@NikolayPianikov @dtretyakov Can you please consider printing messages to console in teamcity logger?

@smadala Yes but the test information will be duplicated depending on verbosity level

@NikolayPianikov Can we add ##teamcity prefix only in logger itself rather than in xunit adapter?

From  https://github.com/Microsoft/vstest/issues/1606 There are various issues and PRs floating around that set the console logger's verbosity to...

Console logger behavior has been changing almost in every release as per different requirements of  vstest.console.exe and dotnet test scenario.
We really don't want to change the behavior of vstest console logger until it is very much required.
I think teamcity CI integration can be achieved without depending on console logger behavior.

@NikolayPianikov

TeamCity has a lot of clients and we treat them with respect. Many of them have already established CI processes and we must not allow any things to make their lives more difficult.

We all are on same page here.
Can you please provide more details about your scenarios? We will work on a solution which will have little or no impact on Teamcity customers.
If you are using ##teamcity only for test reporting, why can't we use trx logger?

With the console, TeamCity streams the test results to CI jobs in real time as they run. With .trx, don't you have to wait for the job to complete to get all the results at once from the XML file? Knowing a long-running test job is failing early on, rather than having to wait for the whole job to complete, is much more valuable.

TeamCity service messages are useful for a set of scenarios like gathering statistics, code coverage, code inspections, attaching artifacts, build process's flow control. And as @martincostello mentioned it provides real-time feedback to a TeamCity server and users can save a lot of time.
TeamCity sends service messages when:

user runs his build under TeamCity using .NET CLI Plugin (TeamCity specifies additional test adapter)
user manually adds the reference to TeamCity.VSTest.TestAdapter into test project. In this case the TeamCity integration is working from any TeamCity runner (for example command line runner).

xunit (for dotnet core version only) sends TeamCity service messages when it finds the TEAMCITY_PROJECT_NAME environment variable for its proccess. It's like a way to determine if the xunit tests are runing under TeamCity server (to not publish TeamCity service messages from the users' command line). For the similar cases TeamCity is watching for output here and when it finds some serviece message it turn off its own propagation of service messages to avoid a duplication for test information.
There is an intresting moment: when verbosity level is quiet and users cannot see TeamCity service messages created by xunit and sended by console logger this code still RECEIVES notifications about actually filtered service messages.
Summarizing, TeamCity cannot process the case with xunit for dotten core properly Meanwhile other cases are working as expected.
To minimize the consequences we have made verbosity levels as 'normal' by default for .NET CLI Plugin
For TeamCity.VSTest.TestAdapter we have made verbosity levels as 'normal' by default using the paramer VSTestVerbosity, excepted the case when user manually specifies other verbosity level.
But anyway it does not look very good

@NikolayPianikov Thanks for detail info. I will get back to you.
/cc @cltshivash @pvlakshm @mayankbansal018 @singhsarab FYI

@NikolayPianikov The verbosity levels of quiet and minimal shouldn't log information for successful tests. It was a bug that was fixed. Unfortunately the dependency has been taken on it and it is better to correct rather than let it remain for perpetuity.

Hi, our team has just hit this problem. I agree with @joaopgrassi, this is a breaking change for users. dotnet test used to correctly report tests to TeamCity, and now it does not. I imagine dotnet test was supposed to make it easier to run tests, but the xunit and TeamCity integration has had multiple bugs.

@JamesFenton, did you tried using logging verbosity parameter mentioned above? https://github.com/Microsoft/vstest/issues/1590#issuecomment-393460921

@dtretyakov I did try it, and it does work. Its just annoying that dotnet test does not work out the box - I had to find a Github issue so that my build worked.


Models folder can not found?

You can find the pre-trained models in the log/save_models. We will modify this misleading problem soon.


0.5.x

Jason:
I have the examples (blinkled.js, analog.js analog2.js, fade.js, input.js) working.
Do you have some formal way of testing?
--Mark

I'm miss typed the branch name initially and didn't catch it until I had committed so things to my repo.  I thought I had recovered.

package.json.version is a place I was recording current version numbers.  It's not easy to add comments to package.json files.


model filed have zero values,but why inserting SQL include those fields?

Your issue may already be reported! Please search on the issue track before creating one.
What version of Go are you using (go version)?
go version go1.12 windows/amd64
Which database and its version are you using?
mysql 5.5.53
Please provide a complete runnable program to reproduce your issue. IMPORTANT
I read docs : https://gorm.io/docs/create.html#Default-Values.
but the zero values insert into sql. How to fix it?
My code:
        userProfile := new(proto.UserProfile)
	userProfile.Nickname = swmUtil.GetRandomString(15)
	fmt.Println(userProfile)  // nickname:"c2c56284dyZ2utt"

	// INSERT  INTO `user_profile` (`nickname`,`created_at`,`updated_at`,`deleted_at`,`status`,`xxx_unrecognized`,`xxx_sizecache`)
	// VALUES (?,?,?,?,?,?,?)[c2c56284dyZ2utt  0 0 0 [] 0] 0
	if err := tx.Create(userProfile).Error; err != nil {
		tx.Rollback()
		return err
	}

Looking forward to your reply.

I think it is case by !field.IsPrimaryKey.
In addition to the primary key will be inserted into sql.
Is there anything else that can be solved?
@jinzhu  Looking forward to your help.
 if !field.IsPrimaryKey || !field.IsBlank {
  columns = append(columns, scope.Quote(field.DBName))
  placeholders = append(placeholders, scope.AddToVars(field.Field.Interface()))
}

callback_create.go#L66

If you want to avoid this, consider using a pointer type or scanner/valuer
You can get this from doc

I have tried this program.
I used this library last year and it is normal.
But now the code is the same and the result is wrong.
I do not know why.
Temporary plan !field.IsPrimaryKey delete .
        var profile = proto.UserProfile{Nickname: "99",}
	if err := u.db.Create(&profile).Error; err != nil {
		return err
	}

Change code can solve my problem. I don't know if it is correct.
for reference only
 } else if !field.IsPrimaryKey || !field.IsBlank {
    columns = append(columns, scope.Quote(field.DBName))
    placeholders = append(placeholders, scope.AddToVars(field.Field.Interface()))
}

To
} else if !field.IsBlank {
	if !field.IsPrimaryKey {
	  columns = append(columns, scope.Quote(field.DBName))
	  placeholders = append(placeholders, scope.AddToVars(field.Field.Interface()))
	}
}


Cannot start the program with F5

Hi,
I just installed vscode insiders for windows and cloned the repo. When opening it as hitting F5 here's what I am getting
src/extension.ts:67:7 - error TS2345: Argument of type '{ text: string; range: vscode.Range; trackingId: string; }' is not assignable to parameter of type 'CustomInlineCompletionItem'.
  Object literal may only specify known properties, and 'text' does not exist in type 'CustomInlineCompletionItem'.

67       text: rs.completions[i],
         ~~~~~~~~~~~~~~~~~~~~~~~

[10:25:11] Found 1 error. Watching for file changes.

Not knowing nodejs, i have no clue where to start

@willmendil i had the same issue and fixed by replacing text: rs.completions[i] to insertText: rs.completions[i] it seems the latest vscode insider change the parameters signature of CustomInlineCompletionItem

ok tis worked but not fully now i get these in the new window

Got the same issu

@willmendil @Vesloo  yeap me too, it seems this repository hasn't been updated for a while and is using old calls to vscode extension api,, i added --enabled-proposed-api into the .vscode/launch.json arguments, but it doesnt work

I got the exact same error while a new insider window is being opeaned.

Suggestions by the software does not work. So I checked the code-clippy's window while waiting for the code to generate, I saw this:

The suggestion array was generated as empty. The model's suggesting empty values

It's giving me the exact same issue. I've tried normal/insiders both. It didn't work for either of them.

I'm not sure if this repository is still being maintained, try this : https://github.com/CodedotAl/code-clippy-vscode/pull/13/files

I'm not sure if this repository is still being maintained, try this : https://github.com/CodedotAl/code-clippy-vscode/pull/13/files

@MHuiG I've cloned the repo and started the extension. I was able to open the debug mode without any errors. But while writing a code, it was giving me some errors. But it was giving me some suggestions too.
These are the error messages:
error: 'Model flax-community/gpt-neo-125M-code-clippy-dedup-2048 is currently loading'
error: 'Rate limit reached. Please log in or use your apiToken'
Here is the full DEBUG CONSOLE output:
capabilities resolved immediately
assistant is closing
capabilities resolved immediately
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Model flax-community/gpt-neo-125M-code-clippy-dedup-2048 is currently loading', estimated_time: 21.041305541992188}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['impl OpenOption for Option {
fn parse_open_options', '// ===================================================================

fn main() {

}
', '// IFC2 calls the following method:
//
//']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['}
}

//
//WKwidgets', 'fn parse_open_options_with_options() {
', 'f:f
}

fn parse_open_options']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['//
// Copyright 2014-2020 The Kubernetes Authors.
//', '//
//
//
//
//
//
//
//', '//
// Copyright 2019 The Kubernetes Authors. All rights reserved.']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['}

fn main() {

::_f', '// This call is taken from the compiler and passed to the library
//', 'extern "C" {
	fn main(argc, arg']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['void:
::<IP_ADDRESS> :
', 'f f
:f
:f
:f
:f
:', 'class:
:class:
']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['fn main_with_c() {
// I don't', '// Use a test for testing the behavior.
//
// Usage:', 'describe "Powar_S3_Dwip/P']
(3) ['// Read the first byte of the string
', 'if (strlen(tagname) == 0) {', 'bool is_thread_safe = false;
']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(2) ['write_data(write_data_info, write_data', 'ichar_unittest() {
	ichar_unittest']
assistant is closing
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(2) ['write_data(write_data_info, write_data', 'ichar_unittest() {
	ichar_unittest']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['{
// I'm doing the following so that when I call
', '{
	print!("\n");
	print!("{:', '{
for f in os::file_operations().to_']
(3) ['// use std::io::
//', 'impl <1> <2> File::WriteFile for <2', 'fn main()
{

}']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['// I think I understand what I'm doing wrong but I'm not', '#include <stdio.h>

// #include <', '//
//
//
//
//
//
//
']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['}

#include "clang_core/clang_assert', '#[test_func_Mangle_1_Mangle_', 'use std::io::unittest;
use std::ruby']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(3) ['fn main()
{
  assert_eq!(', 'fn main()
{
 ::testing::random_un', '// This is a program to give you
// a list of the']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(1) ['() {
self.write_file("']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
(1) ['{
let mut output = _;
_ =']
(3) ['fn test_std::io::Write(in: &std::', 'fn main_do_stuff() {
if!!', '#: src/main/index.js:20
#:']
(3) ['{
write_one_file("c:/users/user1', '{
if!(::_std::is_io_fn', '{
let mut i = 0;
']
(3) ['fn main_func() {
	fmt!("fmt', 'let mut reader = ::Reader;
let mut obj = ::Obj', 'fn main()
{}

// Return the address of']
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
flax-community/gpt-neo-125M-code-clippy-dedup-2048
flax-community/gpt-neo-125M-code-clippy-dedup-2048
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}
{error: 'Rate limit reached. Please log in or use your apiToken'}

@SohamMalakar This is over the rate limit because it uses the Huggingface account's API.
Even with the Huggingface account's API token, there are limitations and a Pro plan is required.
This limitation can only be removed if the model is deployed locally or on your own server without using the Huggingface account's API.
However, local deployment consumes local resources, And the VSC plug-in requires a local version to be refactored.

If you want to get rid of the API, I think the first thing to do is to encapsulate gpt-code-clippy as a service (code clippy as service) like clip-as-service , and then refactor the plug-in here to use the service encapsulated above.
There's still a long way to go, and the repo hasn't been updated in half a year.

@MHuiG Thanks for the information. 😄

The actual/parent version of the copilot-clone works for me.

Hi y'all thanks for the active discussions. We are hoping to start back up maintenance of this repository in the next few weeks and will look to incorporate some of the ideas y'all have put here. PRs are welcome!

Is anyone here interested in helping with refactoring the code to work similar to @MHuiG's suggestion for "code clippy as a service"? I think wrapping up HF models similar to this repo: https://github.com/moyix/fauxpilot where you can locally or remotely create a docker container that hosts a optimized version of the model for deployment is a good evolution for this repository.


Consider adding maplibre as a source and target

The MapLibre project is a FOSS continuation of the source code of Mapbox' client libraries when they got closed-sourced.
The MapLibre Style Specification is backward compatible with the style specification of Mapbox. Consider adding maplibre as a --source and --target.

Yeah that sound like a great idea. We should also enhance the README of the parser itself to make clear that it is also suitable for maplibre.


Sort tests before shuffling

I found, that sometimes tests go in different order on different machines, so let's sort them to get identical after-shuffle order on every machine.

Would you, please, update package version and upload it to pypi?

@Tayamarn I've uploaded 1.1.0 to pypi


Warn when using deprecated listeners as loggers

Fixes #23360

@bot-gradle test BD, ACC

I've triggered the following builds for you:

BuildDistributions build
AllConfigCacheTestsPullRequestFeedback build

@bot-gradle test ACC

I've triggered the following builds for you:

AllConfigCacheTestsPullRequestFeedback build

@bot-gradle test and merge

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

I've triggered a build for you.

@bot-gradle test and merge

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

I've triggered a build for you.


RFC: Client side testing of forms

Idea
It'd be nice to use testing frameworks for JavaScript.
Proposals

Library to fake XMLHttpRequest and AJAX calls: http://sinonjs.org/

Behavior Driven Development testing framework for JavaScript. It does not rely on browsers, DOM, or any JavaScript framework. http://jasmine.github.io/2.0/introduction.html

Testing
Our custom JS libraries for the forms.
=============
Ported from https://github.com/inspirehep/inspire-next/issues/79

Jasmine has been implemented and adopted in Invenio so I'm closing this RFC.


Fix #393: make async method non async

if it lacks the await keyword.
There is only a codefix, no analyzer, since it's based on the compiler's
CS1998 warning. The codefix just removes the async keyword and replaces
all occurrences of return x with return Task.FromResult(x).

@thomaslevesque Nice one. I liked the rewriter approach, we have not used that before here.
I have added you to the collaborators team, so next time we can assign the issue to you.


Creating Yara Rules

when trying to create a yara rule i get an error Syntax - 'rule  {' not found!
but all is in order. i provide a rule name and the rule content

@adimenia I need to open up the restrictions on the syntax check, but if you format your rules similar to the way I have below, it should work. Assuming you don't have any comments preceding the rule keyword.
rule php_in_image {
    strings:
        $gif = /^GIF8[79]a/
        $jfif = { ff d8 ff e? 00 10 4a 46 49 46 }
        $png = { 89 50 4e 47 0d 0a 1a 0a }

        $php_tag = "<?php"
    condition:
        (($gif at 0) or
        ($jfif at 0) or
        ($png at 0)) and

        $php_tag
}


lomiri.lomiri-ui-toolkit: Fix compatibility with versioned QML import paths

Description of changes
LUITK uses QML2_IMPORT_PATH for finding themes, both additional ones & the default. If it can't find its default theme, most LUITK apps are displayed as solid-black.
Fix by adding the new variable to its list of considered envvars.
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.

lomiri.lomiri-terminal-app (the only currently-merged graphical user of this) stops complaining about the default theme missing, so this appears to fix the issue.


Portfolio Deployment Link

See Link Below
My Portfolio

See Link Below
My Portfolio

Hi @mrprotocoll ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
Linters are passing 👍
The project is running successfully 👍
PR is well descriptive 👍
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @iambenkis in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Fix task collection with sqlite

Close #661 (again)

For the record:
With #664, we tried to be very general and support background tasks with some flexibility (that is, they could use either sync or async db session). For some unclear reason, we were relying on async sessions for task-collection and on sync session for workflow execution.
With this PR we simplify things a bit, and we switch to always using sync DB sessions in background tasks, which has never lead to errors.


Fix inconsistency and grammar in help messages

Add periods for consistency where missing.
Fix grammer.
Update jsr to JSR.

@roj1512 Thanks for your suggestions! But I am not sure if these changes are appropriate, and I feel they are outside the scope of this PR, so I will leave it to the maintainers to decide if they should be merged.

 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.

Thanks for the PR, please sign the CLA so we can merge it :)

@bartlomieju Signed it! 😁


nextcloud26: 26.0.1 -> 26.0.2

https://nextcloud.com/changelog/#26-0-2
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.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.

Sorry for the noise I forgot that it was backported already and my local release-22.11 was kinda old so I wrongly assumed this wasn't the case :facepalm:


Remove bridge for EDGI lobby

edgi-public-lobby channel is not used and has been archived. I'm also testing making changes to config

Going to just merge this tomorrow, if no reviews


Fix offsets for the support image maker in the Friend screen

Fixes #149

I'm not sure whether to use *2 for script coordinates or not, so it's done one way in one line, and then another way in another line.

I'm not sure whether to use *2 for script coordinates or not, so it's done one way in one line, and then another way in another line

That depends on whether we're taking an image from screen or cropping an already taken image.
Ideally, both should've been done in the same way, but:

For Servant and CE, we're cropping an image which has already been taken from screen.
For Friend, we're taking image directly from screen, so script coordinates.

That depends on whether we're taking an image from screen or cropping an already taken image.

I think you're misunderstanding. Why does the old code specify the supportBound with 2 instead of just using the multiplied values to begin with, for example 106 instead of 532?

Oh, now I get what you were asking.
There is no solid reason really. I just wrote it that way when I was tweaking the values. Feel free to change it to use the final multiplied values.


[Bug] | White Screen of Death

Are you using the latest release?

[X] Yes, I am using the latest release.

Have you checked a bug report doesn't already exist?

[X] Yes, I have checked the existing bug reports.

What happened?
When users and myself now go to the URL they get a white screen.  No data at all.  works fine when accessing from IP but doesnt work from URL.  invite.mysite.io
Virtualisation
Docker
What operating system are you seeing the problem on?
Linux
What browsers are you seeing the problem on?

Steps to Reproduce
visit url
white screen
Relevant Log Output
No response

my web url is behind nginx proxy manager if that helps.

Same issue with CloudFlare Zero Trust. Some diagnosis on the client side shows that it's attempting to load javascript and connect to socket.io but cloudflare (or whatever reverse proxy) is redirecting to login but an error is thrown. I don't have this with other applications so it seems to be something with perhaps the login page cached on the browser-side. Better would be not to cash this so that the login HTML is refetched. This would cause the reverse proxy to send a login. Just my 2c, I could be wrong.

Quick work around.
Navigate to wizarr.yourdomain.com/favicon.ico
Revert back to white screen and hit F5.
Suggest wait and see what Oidc looks like as it might fix this issue, or provide a better way for a proper fix.

Let me see if i can find my  discussion as i had the same problem a while ago and someone here walked me thru what to change in cloudflare

Let me see if i can find my discussion as i had the same problem a while ago and someone here walked me thru what to change in cloudflare

That would be awesome. I'm shooting in the dark here and it's something that should be documented for all.

it was on Discord if your on there discord i can send u the link it has pictures and he made it very easy for me to make the needed changes. It actually fixed alot of other issues with other applications i was having
its under support and V3 invites not working. Half way down the page

it was on Discord if your on there discord i can send u the link it has pictures and he made it very easy for me to make the needed changes. It actually fixed alot of other issues with other applications i was having
its under support and V3 invites not working. Half way down the page

I'm on Discord and there is a reference to nginx config. I commented there I want to see if forcing client cache TTL to be less than Oidc TTL solves the problem by forcing an origin pull and refreshing client cache.

Disable this, it lives within Speed -> Optimisation -> Content

 Disable this, it lives within Speed -> Optimisation -> Content
Also whilst being deprecated, disable these:

I have done this but still same issue

So it does appear to be a cache issue.  I disabled the following and seems to be working again

So it does appear to be a cache issue. I changed the following and seems to be working again

I have turned cache off completely for wizarr but still every now and then experience this issue.


Color feedback for Taipy visual elements

What would that feature address
The goal would be to make the experience of Taipy Gui more enjoyable and make it easier and user-friendly. Having a visual feedback on visual elements makes understanding the code far more effortless and enjoyable.
Example:

Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%
[ ] Propagate any change on the demos and run all of them to ensure there is no breaking change
[ ] Ensure any change is well documented

This was requested again by users

Partly done by @dinhlongviolin1


List US Government URLs

The doc lists URLs in the Azure Commercial cloud that need to be reachable for on-premises installations of the AMA. Please also include the URLs for the Azure Government cloud.

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

ID: bd3dca2f-629d-36e9-f0f0-8c7e64cc04ab
Version Independent ID: f40ee991-6cb8-1eee-9a35-ebc5833d7c7f
Content: Azure Monitor agent overview - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-overview.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@SvenAelterman Thanks for your feedback.
Assigning to content author @bwren for review.
@shayoniseth FYI.

Apologies for the late reply. Looks like this got added to Firewall requirements.
#please-close


ErrImagePull when using latest docker image tag & embedded registry in air-gapped cluster

Environmental Info:
K3s Version:
k3s version v1.29.3+k3s1 (8aecc26b)
go version go1.21.8

Node(s) CPU architecture, OS, and Version:
Linux hp1 5.4.0-176-generic #196-Ubuntu SMP Fri Mar 22 16:46:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.6 LTS"

Cluster Configuration:
3 servers, 0 agents
$ sudo cat /etc/systemd/system/k3s.service
[Unit]
Description=Lightweight Kubernetes
Documentation=https://k3s.io
Wants=network-online.target
After=network-online.target

[Install]
WantedBy=multi-user.target

[Service]
Type=notify
EnvironmentFile=-/etc/default/%N
EnvironmentFile=-/etc/sysconfig/%N
EnvironmentFile=-/etc/systemd/system/k3s.service.env
KillMode=process
Delegate=yes
# Having non-zero Limit*s causes performance problems due to accounting overhead
# in the kernel. We recommend using cgroups to do container-local accounting.
LimitNOFILE=1048576
LimitNPROC=infinity
LimitCORE=infinity
TasksMax=infinity
TimeoutStartSec=0
Restart=always
RestartSec=5s
ExecStartPre=/bin/sh -xc '! /usr/bin/systemctl is-enabled --quiet nm-cloud-setup.service 2>/dev/null'
ExecStartPre=-/sbin/modprobe br_netfilter
ExecStartPre=-/sbin/modprobe overlay
ExecStart=/usr/local/bin/k3s \
    server \
	'--cluster-init' \
	'--embedded-registry' \
	'--disable=traefik' \
	'--disable=servicelb' \
	'--tls-san=<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>' \
	'--node-ip=<IP_ADDRESS>' \
	'--node-external-ip=<IP_ADDRESS>' \
	'--flannel-iface=eno1' \

The file /etc/rancher/k3s/registries.yaml on all server nodes is:
mirrors:
  "*":

Describe the bug:
Given busybox.yaml with the following content:
apiVersion: v1
kind: Pod
metadata:
  name: busybox
spec:
  terminationGracePeriodSeconds: 1
  containers:
    - name: busybox
      image: busybox # or, add :latest
      command:
        - sleep
      args:
        - "infinity"

and given that the referenced image is in the embedded registry:
$ sudo ctr i ls | grep busybox
docker.io/library/busybox:1.35.0                                                             application/vnd.oci.image.manifest.v1+json           sha256:4b9dd8bbde7eb5d60bc86ed64100692745310912e023e49fc85fd04ca82d7d10 4.3 MiB   linux/amd64 io.cri-containerd.image=managed
docker.io/library/busybox:latest                                                             application/vnd.oci.image.manifest.v1+json           sha256:f19676f4de546985ff6d516bd5e772f2cacfae61699078c93cda3b4472371267 4.3 MiB   linux/amd64 io.cri-containerd.image=managed
docker.io/rancher/mirrored-library-busybox:1.36.1                                            application/vnd.docker.distribution.manifest.v2+json sha256:3ebd70b65321b7d390b8031ff280fbf59cd747e7a80490c9ee1bda13d6463e4e 4.3 MiB   linux/amd64 io.cattle.k3s.pinned=pinned,io.cri-containerd.image=managed,io.cri-containerd.pinned=pinned

then executing k3s kubectl apply -f busybox.yaml deploys the pod, but I get an ErrImagePull.  Describing the pod in k9s shows:

Steps To Reproduce:

Create a k3s cluster with an embedded registry and the above /etc/rancher/k3s/registries.yaml file.
Copy the busybox.yaml content above.
Ensure busybox:latest is present in the embedded registry.
Apply with kubectl -f busybox.yaml

Expected behavior:
Pod should be able to pull image.
Actual behavior:
Pod unable to pull image, in ErrImagePull condition.
Additional context / logs:
If you omit the image tag or explicitly set tag to latest, this bug is present.  However, if you use a tag other than latest, like 1.35.0, everything works ok.

This is covered in the documentation: https://docs.k3s.io/installation/registry-mirror#latest-tag

Oh wow.  I must've missed that.  I had no idea!  Sorry for the noise.


Unable to filter a string field with numeric values

I wanted to make an example showing filtered data but I'm getting an error because the attribute I want to filter by is a string that holds numeric values. This is a common occurrence and in my example below, it is not a case where the field can be changed from string to number as it would "break" the joining process with other datasets.
In the map below, I have a field geoid_state that is a unique identifier for each state in the US that is commonly used in conjunction with geoid/fips county values to join tables of census data with their associated geometries.
Any geoid whether for state, county, etc. should always be a string even though it contains numbers. The reason is because if a value 01001 is converted to number, it becomes 1001 and the link to join is broken.
I'm not sure if I am overlooking something and/or don't have the proper syntax (fingers crossed haha), but it seems like this isn't possible as I keep getting the error Error: Maps API error: {"errors":["operator does not exist: text = integer"],"errors_with_context":[{"type":"unknown","message":"operator does not exist: text = integer"}]}
Here is the map that I was trying this with:
https://cartodb.github.io/carto-vl/examples/editor/index.html#eyJhIjoidGFibGVfNXlyX2NvdW50eV9hY3MiLCJiIjoiIiwiYyI6Im1hbWF0YWFrZWxsYSIsImQiOiJodHRwczovL3t1c2VyfS5jYXJ0by5jb20iLCJlIjoiY29sb3I6IHJhbXAoZ2xvYmFsUXVhbnRpbGVzKCRwZXJjX2hpc3BhbmljLDcpLHN1bnNldClcbnN0cm9rZVdpZHRoOiAwXG5maWx0ZXI6ICRnZW9pZF9zdGF0ZSA9PSAnNTUnXG4iLCJmIjp7ImxuZyI6MCwibGF0IjowfSwiZyI6MCwiaCI6IkRhcmtNYXR0ZXIiLCJpIjoiZGF0YXNldCJ9

I think is a failure of our SQL filters: the generated SQL, geoid_state = 55 should have the literal quoted: geoid_state = '55'. We should be able to do that since we now the type of the properties by the backend metadata.

Hi,  is there currently a suggested workaround for this?
it seems I'm running into the same issue while trying to construct a filter with an "in" expression.
filter: exp.in(exp.prop('geoid'),
        ['36061003300', '36061009200'])

Results in the error:
Uncaught (in promise) CartoMapsAPIError: [SQL] ["operator does not exist: text = bigint"]

Interestingly, if I cast the prop to a String, the errors are squashed (but the filtering still isn't working):
filter: exp.in(String(exp.prop('geoid')),
        ['36061003300', '36061009200'])


make template project

Comments from Jérôme:

if you add documents, don't add full articles but just a couple placeholder very short documents.

remind that those are not generated manually but by pubget.

if you add labels, point out that they can be generated in labelbuddy rather than editing the file directly.
if you just want to add the directories without example files, you can put a short readme in each directory (if the directory is completely empty git will not do anything with it, as it tracks regular files and symbolic links but not directories)
as you add the template readme you can also check and edit the "contributing to this repository" docs: https://neurodatascience.github.io/labelbuddy-annotations/contributing_to_this_repository.html#adding-a-new-project

@jeromedockes ,
I added the template project README and skeleton structure. If you think it's okay, then I'll go ahead and update the README's for each project. see this commit
I also changed to code to make the database so that it would ignore the template project. I'm not sure if I did this in the best way - could you check? see this commit

thanks! yes the README looks good, I just made a couple of small changes and added a mention of the datasets.json file. I also updated the database module


MPR not working in v3 with dicomjson

Good evening team,
I was using MPR on OHIF v2 with dicomjson.
I noticed it is not working on v3.
You can check it using the demo link in the documentation :
https://v3-demo.ohif.org/viewer/dicomjson?url=https://ohif-dicom-json-example.s3.amazonaws.com/LIDC-IDRI-0001.json
Once the demo open, click on the MPR button.
The window will divide in three, but the views will not be loaded.
The below error appear in console :
Unhandled Promise Rejection: TypeError: undefined is not an object (evaluating 'e.split("/studies/")[1].split')

Below how it load :

Any suggestion or comment would be Welcome.

Please review the latest code in the master branch. I am confident that this issue has been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


Is DepthEffect Working?

Description of the bug
Hey everyone!
I'm new to GLSL and postprocessing, just playing with custom effects, and noticed that DepthEffect always shows white just background with no details.
Is DepthEffect Working with latest version?
To Reproduce

// ...

const depth = new DepthEffect({

});

composer.addPass(new RenderPass(scene, camera));
composer.addPass(new EffectPass(camera, depth));

// ...

Expected behavior
A grey scale image where dark is the far layer and white is closest layer
Library versions used

Three: 0.155.0
Post Processing: 6.33

Desktop

OS: MacOS
Browser Arc Browser (Chromium)
Graphics hardware: M1

Hi, this is working as expected.
The DepthEffect is usually used for debugging. Objects that close to the camera will appear black/grey since their fragments have low depth values. Everything else will be mostly white due to the depth distribution of the depth buffer (see https://www.reedbeta.com/blog/depth-precision-visualized/)


MVVM Example?

Can we have a MVVM example?

I don't know if I should make a new issue just to mask this question but...
How do ICommand's work?
From the examples I find online It looks like ICommand only covers the main event aka

Click for Buttons
Click for MenuItems

How would I bind a ICommand property to a paint event?

I cannot suggest to use Binding for now. It is slow and will be rewritten in future.

I cannot suggest to use Binding for now. It is slow and will be rewritten in future.

ok thankyou

@Shadowblitz16, I am moving this to discussions as this will be done after we redo Binding. Also, do you know any good simple MVVM examples? Is there any in Wpf or Maui?


WIP: Fix deadlock during shutdown which prevented leader election cleanup

Before this fix, the deadlock would prevent the leader pod from giving up its lease, which would make it take several minutes for new pods to be allowed to elect a new leader. During that time, no Pinniped controllers could write to the Kube API, so important resources were not being updated during that window. It would also make pod shutdown take about 1 minute.
After this fix, the leader gives up its lease immediately, and pod shutdown takes about 1 second. This improves restart/upgrade time and also fixes the problem where there was no leader for several minutes after a restart/upgrade.
The deadlock was between the post-start hook and the pre-shutdown hook. The pre-shutdown hook blocked until a certain background goroutine in the post-start hook finished, but that goroutine could not finish until the pre-shutdown hook finished. Thus, they were both blocked, waiting for each other infinitely. Eventually the process would be externally killed.
This deadlock was most likely introduced by some change in Kube's generic api server package related to how the many complex channels used during server shutdown interact with each other, and was not noticed when we upgraded to the version which introduced the change.
Release note:
TBD but needed

Nice find!


Exclude Jps from discovered JVMs

[x] Implement code
[x] Add tests

Does not work because Process#pid has only been introduced in Java 9


Build update

Issues fixed:
#17 cleanup mess with jtorctl-briar.jar
#18 Build Android Tor (this is no longer needed since we are dependent of tor-android-library for this)
#19 Debugging data
#30: Could not find proxy universal library
#35: x86 build for Tor Binary
#38: Problem installing gradle
Build changes
Updated Gradle to 4.5 across project
Updated Android Gradle build to 3.0.1
Require JDK version 1.7
Require Android SDK Version 21+
Include tor-android-library for Android install
Include jtorctl:02 binary library (deleted jtorctl-briar lib)
Code Changes
OnionProxyManager allows an implementing class to decide how to setup Tor.
Moved java specific Tor install code from universal to java module.
Use tor-android-library for Tor install in android module
Some signature changes related to latest jtorctl import
Added parameter to allow logging of tor control processing
Todos:
Tested on Mac and Android device. Still needs testing on linux/windows/android_emulator
Unit tests
The threading logic in existing code should move to a publish/subscribe pattern.
Need to maintain different versions of Tor binaries per platform version. Currently project is only using latest version which isn’t good for backwards compatibility.

This change is

 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.

Thank you so much for this! I just had one minor question and then we can commit. Thanks!

Reviewed 48 of 48 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

universal/src/main/java/com/msopentech/thali/toronionproxy/OnionProxyContext.java, line 61 at r1 (raw file):

    }

    public final File getGeoIpv6File() {

Why does this need to be final?

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

universal/src/main/java/com/msopentech/thali/toronionproxy/OnionProxyContext.java, line 61 at r1 (raw file):
Previously, yaronyg (Yaron Y Goland) wrote…
Why does this need to be final?

I was inferring the intent from the original code, where the fields are final. So it would make sense that the method accessors for those fields should also be final.
I could see the case where we would want some configuration file not to be in the working directory. There are couple of solutions that jump out.

Marking these method accessors as not final. I would suggest in that case to also remove the final flag on the respective fields so they could be overriden in the subclass.

Create a ConfigBuilder class that allows the application to specialize config locations. Then we would have: OnionProxyContext(ConfigBuilder builder). The default builder would be WorkingDirConfigBuilder(File dir)

I’m more in favor of (2), not much work either way. Thoughts?

Comments from Reviewable

I was inferring the intent from the original code, where the fields are final. So it would make sense that the method accessors for those fields should also be final.
I could see the case where we would want some configuration file not to be in the working directory. There are couple of solutions that jump out.

Marking these method accessors as not final. I would suggest in that case to also remove the final flag on the respective fields so they could be overriden in the subclass.

Create a ConfigBuilder class that allows the application to specialize config locations. Then we would have: OnionProxyContext(ConfigBuilder builder). The default builder would be WorkingDirConfigBuilder(File dir)

I’m more in favor of (2), not much work either way. Thoughts?

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

2 sounds great, thanks!

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Should have the ConfigBuilder complete and ready for review by 2/10

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Looking forward to it!

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Ready for review and commit.
General:

TorConfig/TorConfig.Builder - makes easy to configure Tor for different environments or to use existing platform installations
TorInstaller - abstracted out installer code into its own class [Java/Android]
Improved logging
Improved concurrency handling on files

Android:

Add sl4j android logging library so logs will display

This is great! I'm happy to ignore my last comment and commit if you would like since the comment is about an issue that predates your code. Just let me know. Thanks!

Reviewed 20 of 20 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

universal/src/main/java/com/msopentech/thali/toronionproxy/FileUtilities.java, line 83 at r2 (raw file):

    public static boolean setToReadOnlyPermissions(File file) {
        return file.setReadable(false, false) &&

I believe this is actually code that you just moved here but doesn't it look wrong? Why we are setting writable and executable to true for a function that says it is setting read only permissions? And no, I don't remember. :)

Comments from Reviewable

.
The FileUtilities.setToReadOnlyPermissions is correct behavior. It's not clear from Javadocs for the File class, but the first three lines remove all permissions (user/world/group). Then last three set them to user only. Without first three lines, the last three would leave group permissions on top of current file permissions (probably only a *nix issue).

public static boolean setToReadOnlyPermissions(File file) {
    return file.setReadable(false, false) &&
            file.setWritable(false, false) &&
            file.setExecutable(false, false) &&

            file.setReadable(true, true) &&
            file.setWritable(true, true) &&
            file.setExecutable(true, true);
}

If everything else looks good, I'd say its ready for a merge from my side.

Comments from Reviewable


🏁 Race of Sloths: The Open Source Developer Challenge is about to start

As a team of Rust developers from Race of Sloths,  we recognized this repo to be highly valuable in the open-source movement.
Race of Sloth makes the open-source contributions more fun and rewarding.

This issue is an invitation to check how Race of Sloths works
To accept the invitation, comment @race-of-sloths accept
Once the invitation is accepted, your contributors are eligible participate in the Race
Contributors just need to mention @race-of-sloths in their PR within your normal contribution flow
We invite you for collaboration and making open source development more engaging together!

Thank you for the recognition! However, I don't with to participate at this time.


FS cache

Might be worth considering some method of caching interactions with the file-system.
Primary benefits revolve around performance and reducing IO load on the OS.
There's some precedence already:

Webpack (I think) merges synchronous calls when handling async calls. Not sure how much benefit this would be, given how little of our code is synchronous.
React native crawls the file system and maintains an in-memory store. Probably the fastest approach, but requires you to use file-watchers to invalidate the data.

#116 would help a lot with lowering our FS load.

Should also consider dependency resolution and the strain that it introduces as well.

Seems to work well based on some initial performance tests.
Data set contains ~1200 programatically generated files that link amongst one another with a mixture of relative and module based identifiers.
Running without the file system cache
~/Projects/unfort $ node test.js
Completed build in 27810ms
1209 files processed

Running with the file system cache
~/Projects/unfort $ node test.js
Completed build in 13235ms
1209 files processed

Using a chokidar instance's crawling to pre-populate the file system cache.
~/Projects/unfort $ node test.js
Completed build in 12293ms
1209 files processed

The final test is a proof of concept for #127


Query Cache - Recreating Cache or Dual Cache

Your library has been a great way for me to set-up a cache when primarily utilizing LINQ queries. The cache helps execute queries more quickly, but I would like to use it as a fallback in the case my database is down.
I would like to be able to expire my cache and recreate it, or even better, have the ability to run two cache simultaneously. This way I can constantly have a fresh cache in the case of downed database or a fallback cache that will present data regardless of it being stale. Using tags for expiration seems to also prevent any future run of that query with the same tag. Any tips or further documentation?
Thanks!

I will send this by route of email as this is not really an issue but a support question.


ElementList is no longer a subclass of list

Fixes #680

@andrewsmedina The changes in this PR fix the majority of the remaining test failures. If you don't have any concerns, merging will help to cut down the noise when looking at the remaining failures.

@jsfehler I'm testing this in my local machine as part of review process. Finishing this step I will approve and merge this pr.

@jsfehler thanks for contribute


Dart 3 compatability

This package has not been actively updated to Dart 3. Is it still maintained?
https://github.com/google/dart_cli_script/blob/main/pubspec.yaml#L9

Yes, it's still maintained. Dart 3 only released a few weeks ago; please give us time.


Move if name == 'main' body to main function

You should create function for main:
def main() -> NoReturn:
    ...

if __name__ == '__main__':
    main()

This will allow to comment just one string to start debugging.

Done!


Multicursor with consecutive lines + moving of lines

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  1.84.2
OS Version: Windows 10 22H2

Steps to Reproduce:

Set up multiple cursors that are on consecutive lines (any pair number of cursors will behave similarly)
Move lines up (resp. down) with Alt + UpArrow (resp. DownArrow)

Result: When moving up (resp. down) the pair (resp. odd)  number of consecutive lines behave as expected, but the odd (resp. pair) ones' cursors disappear and stay in place.
It is still possible to select multiple lines and move them with Alt+Up/DownArrow though.
Thanks for the support!

Duplicate of https://github.com/microsoft/vscode/issues/11809


fixed labels in object manipulator scene

They were still referring to primitive manipulator instead of object manipulator

Made a small layout update - moved the text description onto the back panel, updated the text 'our example scene' to 'an example scene'.
Thanks!

/azp run


Output a warning if React version is missing in settings

Resolves #1789
This message will appear when running some of the tests. Not sure if that should be avoided.

Another idea is to add "latest" as react version. :D

@kud i could see adding a separate option for "detect" (using the "version" as a fallback), which did require('react').version - want to open an issue or PR that adds that?

(Note that it'd have to use process.cwd(), and the resolve package, to get the proper require path)

I think I got the "detect" option to work. I found something I didn't expect though: when extending airbnb's config (as opposed to the recommended config from the plugin), there is a version there in context.settings even if you don't set it in .eslintrc. I believe it comes from here https://github.com/airbnb/javascript/blob/02b4eea3452b0ae6da666b01f9da23f26148f5b6/packages/eslint-config-airbnb/rules/react.js#L464
Is this intended? I found this surprising and non-obvious 🤔

Yes, absolutely it's intended.

Oh. Is this just the version used at Airbnb?
@ljharb you mentioned elsewhere you would make not specifying the version throw. But then (correct me if I'm wrong) if most of plugin's users use it via extending airbnb, they would never observe that because of this implicit (for them) setting.

Yes, that is true. However, with that behavior, the airbnb config would probably remove that setting.
With "detect", it'd probably use that, though.


Adding Java Security Manager Testcase:IDENTITY-3340

Test class name is not according to the standard convension. Please follow this and do the necessary changes accordingly.
https://docs.wso2.com/display/TA430/Best+Practices+for+Writing+Integration+Tests

Still "Carbon3340TestCase.java" is not compliant with the naming convention, please check