feat: cancel running tasks

Fixed: https://github.com/merico-dev/lake/issues/451
Part 1 - Cancelling tasks
We want to cancel tasks while they are running on the server. This is so we can stop long running processes with ants.
New technology we use
Simple golang event bus - https://nats.io/
How the code works:

Run docker compose with the new nats server.
Run a task for collection with Github
Cancel the task by POST to localhost:8080/task/cancel?taskName=github
The api call hits a route that sends a NATS message to a subscriber in the github plugin
The plugin subscriber calls scheduler.Release() (An ants function to stop the ants worker pool)
See the logs stop collecting data

TODO:

[x] All plugins need to implement this in their execute function
[x] Create a core method that each plugin can use called ListenForCancelEvent()
[x] Docs on how to cancel tasks
[ ] Cancel task by ID and not just task name

@klesh @hezyin looking for feedback
Part 2 - Getting tasks by id and showing status
The api saves the tasks in the db now and return the ID. You can now query by ID to get the task you want with a valid status.

@joncodo shall we use Context instead of nats, it seems too bloated...

@joncodo shall we use Context instead of nats, it seems too bloated...

I agree for sure. I tried to use it but could not get it at first. I will try to get it again on Tuesday.


Fix invalid return type in docblock

For "fluent" chaining, can't you use @returns self , or is that only recognized by certain IDEs?

@DHager Yes. For the moment I just fixed the one that were incorrect so know all method returning $this have QueryBuilder as return type so in IDEs we have no more issues reported. @returns $this  and @returns self are not used a lot in the project so I would let you guys decide the right choice.

@Koc @DHager I updated in favor of @return $this as @Koc as recommended in the draft of PSR-5

@dag-io can you check consistency in other classes, like https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/AbstractQuery.php and others?

@Koc, it's all good now

:+1:

I manually applied the patch after a rebase and reverting $this to self or static (depending on context).
Thanks @dag-io, sorry for the long delay!

Thank you @Ocramius


Let your little LCD shine. "Balls stay in court".

I'm really liking the pico display.  I went to do a Pull Request, couldn't work out how to do it for the examples. Here's a tweaked version that takes the ball size into account so each bounces as it touches the edge of the screen. It doesn't give the viewer the false impression that the screen is somehow cropped. It also sets the speed of the ball in proportion to its size. Smaller = faster.
If you like it, feel free to use it.
new-balls.py.txt

Love it!
Appreciate it very much!

You’re welcome! Loving your products. You’re now my go-to shop over all others.
Regards,
Mark

Mark Emery Limited<EMAIL_ADDRESS>Tel: 077 6997 3757 Incorporated in England and Wales number 7065343. Registered Office: C/o Calder & Co, St Vincent House, 30 Orange St, London WC2H 7HF.
Appreciate it very much!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Section: Setting the Stage

@eswan18, How would you feel about renaming this section to something like "Setting the Stage" or "Introductory Review" rather than being pandas-specific? I think it'd be useful to review lists and dictionaries here, so I'd like to be more generic.

Ready for review, @eswan18.

Small typo

Overall I'm very happy with this section and will close this issue once the typo (above) is fixed.

Thanks, made fixes and ready for merge to develop.

Merged


[Improvement]: Confusing error message when collect sum is used with non sequence reference

Description
import ballerina/io;

public function main() {
    string s = "12manu245xyz";
    int numbersCount = from var m in s
        let int code = m.getCodePoint(0)
        where code >= 48 && code <= 57
        collect sum(1);
    io:println(numbersCount);
}

Above code gives undefined function 'sum' error.
Describe your problem(s)
This error is confusing, since sum function is available and works if I put code in sum.
Describe your solution(s)
It should give something like non-sequence reference not not supported in collect clause error.

Moved to #42036


added terminal_length_per_neurite feature extraction

allow for getting the path lengths to all terminal
point per neurite

Seems good to me.


XAudio improvements

Try to create XAudio 2.9 even on Windows10 desktop.
Map Volume property to Voice.
Handle CoInitializeEx on Desktop platforms (dotnet/corert#6046).

Thanks!


How to continuously integrate the theme with travis-ci?

I note that you have continuously integrated this theme with travis-ci, can you teach me how to do it? Because I am a completely new travis-ci user, especially don't know how to write the .travis.yml file. It will be very appreciated if you give me some guidance. Thanks!

I don't know what you mean with "integrate this theme with travis-ci". I use Travis CI in this project to release the generated theme to the master branch.
If you want to use this theme you must put it in the themes folder of your Hexo project and specify colos as theme in the Hexo configuration file.
But I think what you want is to deploy your site automatically to the gh-pages branch of your repo instead of executing hexo deploy on your workspace. If that's the case I found this great guide by @sazzer with a simple search.
To learn about Travis CI and how to write a .travis.yml I recommend reading the Travis CI docs.

Thank you for reply, but I think you misunderstood me, I just want to trigger
Travis-ci each time I update a hexo theme repository in my github, because I am
also a theme maker, but I just don't know how to use Travis-ci to test,build and
pass my project. Thank you!
On 周二, 11月 3, 2015 at 06:30, ahaasler/hexo-theme-colos
<EMAIL_ADDRESS>wrote:
Closed #9 [https://github.com/ahaasler/hexo-theme-colos/issues/9] .
—
Reply to this email directly or view it on GitHub
[https://github.com/ahaasler/hexo-theme-colos/issues/9#event-452563022] .[https://github.com/notifications/beacon/AGec2anl3AEbQSUAz19X2hdV_r33NeYNks5pB9uVgaJpZM4GaKhF.gif]

Travis CI automatically triggers builds if a repo changes. You just have to log in and add the repository (it's integrated with GitHub).
If you want to trigger a build on a repo if another one changes you can use the Travis CI API on the one that changes.


Missing closing parenthesis on data value in cq.sjon

Hi, in the cq.json file (Northern Mariana Islands) the language data value is missing a parenthesis for the ["People and Society"]["Languages"]["text"] attribute.
"Languages": {       "text": "Philippine languages 32.8%, Chamorro (official) 24.1%, English (official) 17%, other Pacific island languages 10.1% (includes Carolinian (official), Chinese 6.8%, other Asian languages 7.3%, other 1.9% (2010 est.)"     },
(includes Carolinian (official)
should be
(includes Carolinian (official))
Also in the mz.json the language string is missing a % symbol for the "Nyanja 8.1" list item
Thank you for the repo, it is extremely useful!

Thanks for reporting.
Sorry not trying to be funny - but not sure what I can do - please, report the data errors at the CIA - see https://www.cia.gov/the-world-factbook/ for the source that gets reformatted 1:1 with errors and all :-).

Hi, sorry, did not realize the original data was erroneous :( Thank you


Frontend pages should have title rather than URL.

Currently the page title on the frontend is just the URL route; We should display proper ones instead.

Duplicate of info in #327. Closing in favor of that issue. There is a lot of debt / enhancements building up on the front end. A good sig topic


Update prepare-permissions.md

Please add an image to choose the Organization for which the PAT is being created for Azure DevOps Server versions.
Currently there is an active bug due to which only PAT for "All accessible Organizations" works. Here is the Bug: https://dev.azure.com/mseng/AzureDevOps/_workitems/edit/1942506
Customer mentioned that the documentation does not indicate to choose PAT for "All accessible Organizations".
Actually, PAT should work for the specific collection but due to the active bug, for now, the workaround is that customers have to create PAT for "All accessible Organizations".

@steved0x
Can you review the proposed changes? If approved, the new image will need to be moved into an internal repo and renamed to follow the file naming guidelines.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I am closing this and moving it to a PR in the private repo.


MySQL APP Update and Delete statements

Hey Team,
Looking for some assistance regarding adding UPDATE, DELETE, and INNER JOIN statements via the current MySQL app. Current QUERY and INSERT statements work great, just looking for the ability to further expand upon statements we can run.
Thanks again for all you guy's hard work! Please let me know if I can help test in any way.
Best Regards,
Taylor

@dhaval055 could you help Jaydeep or Bhavik add this? The one who knows SQL best preferably :)

Sure


Expand attributes in extension

Asciidoctor does not expand the attributes (variables) for extensions.
This needs to be done at least for the URL in the screenshot macro.
Maybe we need to also do this for the content of the geb block

@lordofthejars Do you think we need to do this also for the geb block?
Or would it be sufficient to add a target to the geb block (similar to the screenshot macro) so that we do not need to search the entire geb code for the attributes
[geb, url={mySite}]
// no need to 'go' to site, as url parameter will cause an implicit 'go'
$('#button').click()

yes I think that in every place where Asciidoctor does not expand the attributes, it should be done. In this case you mean the attributes value of a geb block

yes maybe it is easier to just do it in the attributes of the geb block and not in the entire block content. because the content is in the end groovy code we would need to ensure that we do not accidentally expand some groovy block.

Yes I think the same.

I think that block attributes are expanded, as long as you surround the value in single quotes:
[geb, url='{mySite}']

Asciidoctor doesn't expand attributes in the target or attributes of a block macro, though. In those cases, you have to expand them explicitly.
In the future, we may have a declarative way to control this behavior. It wasn't something I considered when I first wrote the parser.

Keep in mind that it is necessary to pass on the subs attribute. For instance,
[geb,subs=attributes+]

Asciidoctor Diagram recently dealt with this issue. See https://github.com/asciidoctor/asciidoctor-diagram/issues/105

I added a bunch of integration tests and substitution is now working fine except for the geb block content.
Especially if it is a literal block.
I will try to get a hold of some groovy guys at my workplace to see how we can expand attributes within groovy code. The tricky part is to distinguish attributes from closures...

@lordofthejars I opened a pull request with tests and some refactoring for the attribute expansion. Can you have a look at it. Thx

I will try to get a hold of some groovy guys at my workplace to see how we can expand attributes within groovy code.

One thing you might be able to do is pass the document attribute map to Groovy and allow the use of Groovy syntax for referencing those attributes. Something like:
[geb]
$("input", name: "q").value("${attrs['asciidoctor-version']}")
waitFor(5){true}

In other words, you are exposing the attribute map to Grovoy. This would work without modifying the subs on the block.

@mojavelinux thanks for the hint. It works like a charm
expanding of attributes has been merged with pull request #7

Cool!
I had a chance to review the PR. I'd like to make a suggestion. The variable adocAttrs doesn't feel very elegant to me. Since you are using uppercase for Browser, Dimension, Keys, perhaps you can bind to the variable Attributes instead (implied that they are from AsciiDoc). I'd also be open to Attrs. wdyt?
I also noticed that you are substituting attributes using a custom routine. We should set it as a goal in the future to use the AbstractNode#apply_subs method once AsciidoctorJ makes it available. All the logic for performing substitutions, attributes or otherwise, is handled by this method.

The difference between Browser, Dimension, Keys on one side and the attributes on the other side is that for Browser, Dimension, Keys I am binding the class object where as for attributes I bind an instance object.
The difference is that Keys.ENTER will refer to the static field ENTER where as adocAttrs.get('foo') refers to the instance method get()
I was considering calling them just 'attributes' or 'attrs' but that felt to generic and I was afraid this could lead to name collision. That is why I have chosen the more verbose 'adocAttrs'.
One could though argue that the geb code is written specifically for the documentation and therefore it can adapt to the naming convention given by the screenshot extension. Do you think this is a legitimate restriction to put onto the users?

I now see the distinction from the class names. However, I still think a more readable variable name should be chosen. I like "attrs" or "attributes". I think it's pretty clear given the uniqueness of that term that it is coming from AsciiDoc.

Another idea is to use the variable name "asciidoc", which is understood to be a map of variables.

I like asciidoc it is both, simple and specific. No extra fuzz and also little chance for name collision

:+1:


fix utility_meter for 2022.4.3

Upstream TariffSelect initializer now requires unique_id parameter (https://github.com/home-assistant/core/pull/68457)
Unsure why there's no default set on it, but there we go.

build the unique_id from that of the utility_meter component, defaulting to None.

Thanks a lot for the fix. Looks good.

Unless I'm mistaken, https://github.com/home-assistant/core/pull/68457 was released in HA 2022.4.0, not 2022.4.3. Shouldn't the version check be for 2022.4.0 instead of 2022.4.3?

#680 😅

@bdr99 You are sharp mate!

@whi-tw thx


Creation of 404 not found page.

Creation of 404 not found page.
@Manishkr1007 plz assign me this issue.

Creation of 404 not found page. @shauryaq05 Please assign this issue


More compliant, more dense template cards

Before:

After:

One significant change, as can be seen, is that the CardMedia sets background-size: cover which I think is what we actually wanted in the first place - without it, the waves (if they'd been smaller, or on a very large screen) would be cut off.

Tugboat has finished building the preview for this pull request!
Link:

https://pr4759-uixofbcqfvuw8ao6591v3h7tahnqhif3.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/603e3294fcb22d9bfc6498a3

Tugboat has finished building the preview for this pull request!
Link:

https://pr4759-uixofbcqfvuw8ao6591v3h7tahnqhif3.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/603e3294fcb22d9bfc6498a3


Topic-recognition-prostate-segment-3Dunet

implement 3D unet to segment 3D MR image of prostate

Document

Title:
Good
Address problem:
Good
Working Principle:
Not mentioned. Description of Unet should be provided
Figure visualization:
No figures of UNET. Does have pipeline figure
dependency:
Good
inputs and outputs description:
Not provided. No visualization of segmented output and no dsc metric table
training plot:
Good
readme using markdown:
Good
preprocessing description:
Good
dataset split description
Good
Description of usage:
Usage of the files provided. However, no description for them.

Check the files:

modules.py
Good
dataset.py
More comments should be provided within loading function
train.py
Single class DSC was used instead of multi-class
More comments should be provided within the training function
Using testing images as validation? Could not find validation set
predict.py
DSC score calculation does not seem right.
no visualization function for outputs

Minor tips (nothing to do with marking)
Overall:

solve the problem:
Not sure. No output visualization provided
TF/torch implementation:
Yes
Design quality:
Code looks good, not sure about the performance
code comments:
Only brief comments provided
Difficulty:
Unet3D - Normal

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks
Recognition Problem
Solves problem (no output visualised, unknown what results are) -2
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Normal -5
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe OK
Model/technical explanation -2
Good Description and Comments but could be better -1
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description minimal -1


Error box when starting program

When starting up the program from the 0.11-pre release .exe it gives me these error boxes:

This seems to be caused by this line in Updater.cs pointing to a dead link - I assume you must have changed your GitHub username or something. The update.json in this repository also seems to be full of dead links.

That's the problem, yea, I forgot to update it when I changed my name and everything, I'm going to fix this right now

I think it's fixed, now !


fix: screenshot example can now run > 1times on same machine

I think previously we were just exercising this example with cold AWS Lambda functions.

nice!


Failed to save database password: failed to unlock correct collection （not wsl）

Describe the bug
when i execute supabase link --project-ref xxxxxx
Failed to save database password: failed to unlock correct collection '/org/freedesktop/secrets/aliases/default'
Finished supabase link.
Additional context
Supabase cli version: 1.123.4
OS:  Ubuntu 22.04.3 LTS (Jammy Jellyfish)

This is just a warning when native credentials store isn't available. It's safe to ignore.


Collapsable subassets for a grail card

A better way to list sub-assets that belong to a grail card. Sub-assets that do have a grail will be hidden by default. See the screenshot:

The reasoning behind this change is to save space in the "balances" list. This became an issue especially with some of the sub-asset heavy projects like PHOCKHEADS.

Overall the code looks decent (haven't had a chance to test it yet), but I would prefer it if you could update the code to stop referring to the parent asset as the "grail" card and refer to it as the parent asset, since it is easier to understand the parent-child subasset relationship than it is to understand what a "grail card" is.
Please change any reference to "grail" to be "parent" or "parent_asset"... this will make the code easier to understand what is going on... just a personal preference.
Once this update is made i'll spend some time testing this update and get it merged into the next release if all looks good.
Thanks for this PR... seems like cool functionality that will enhance the wallet a bit :)

Thanks for the feedback. I will look into it, however I'm gonna travel extensively in the next coming months, so it may take a while.

No worries, I can merge and tweak the variable name when the time comes if it’s not already done😀👍🏻
Sent from my iPhone

On Apr 22, 2022, at 2:23 AM, Mario Dian @.***> wrote:

Thanks for the feedback. I will look into it, however I'm gonna travel extensively in the next coming months, so it may take a while.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.


updating the UI of the contact us page. Issue number #169

Hi @danishsheikh1122
read this - #64
also please follow the guidelines, and don't make a PR without getting assigned.

ok sorry sir


Zone updates in 1.18 break a number of Angular zone tests

All succeed in 1.17
See @kevmoo for repro details
Suspect commit: dart-lang/sdk@6a7c037

CC @mit-mit

Reverted the suspect commit and it seems to be working for me. Landed on Dart bots now.

There seems to be a bug in Angular2 that the el function is defined in two places, so it doesn't compile with dart2js. If I fix that it runs there as well.

There seems to be a bug in Angular2 that the el function is defined in two places, so it doesn't compile with dart2js. If I fix that it runs there as well.

Awesome!

Fixed by 30b489578d9ee587d2ca622346e2051a869fb994 – thanks @alan-knight
Will open a merge request separately


How to control flex lines

Hello there.
I would like to implement the following functionality:
I have a HorizontalScrollView, inside which a FlexBoxLayout. Items should fit screen width, but if there are more than four lines, I would like to increase FlexBoxLayoutwidth in order to keep 4 lines (in this case some items will be outside of the screen)
<HorizontalScrollView
            android:id="@+id/horizontalScroll"
            android:layout_width="match_parent"
            android:layout_height="0dp"
            android:layout_weight="1.2"
            android:fillViewport="true"
            android:scrollbarSize="1dp">

            <com.google.android.flexbox.FlexboxLayout
                android:id="@+id/bottomFlexBox"
                android:layout_width="wrap_content"
                android:layout_height="match_parent"
                android:paddingBottom="32dp"
                android:paddingEnd="32dp"
                android:paddingStart="32dp"
                app:alignContent="flex_end"
                app:alignItems="flex_end"
                app:dividerDrawable="@drawable/divider"
                app:flexDirection="row"
                app:flexWrap="wrap"
                app:justifyContent="center"
                app:showDivider="middle" />

        </HorizontalScrollView>

It's not supported at this moment. #156 is on the TODO list.
That feature may be the solution.


[BUG] LocalStorage is not immediately accessible.

Describe the bug
It was fine when using version 2.9.5, but error occurs in version 2.10.0 (latest one)
in version 2.9.5, there is no deserializer, it works fine without delay.
but in the latest version 2.10.0, there is deserializer wrapped with useCallback.
so, we can't get localStorage value immediately.
You can check this bug right here
To Reproduce
in version 2.10.0
Expected behavior
There should be no delay when reading value from LocalStorage.
Additional context
No response

HI, thanks for the bug report.
We have updated the API to make it SSR-friendly, so you'll be able to use it as previously. See #451
It will be released this week


Refactor of appforms raincatcher strucure

Motivation

It is necessary to refactor the structure of the raincatcher appforms module to be more modular. This makes it easier to understand and test each of the views, directives and controllers)

Related to https://github.com/feedhenry/fh-js-sdk/pull/182, The number field was updated to bind values from the controller to the Submission model. This approach should also be useful for other rendering approaches (Backbone, Ionic etc)

Changes
New  Structure
Keeping an Angular project structure of isolating different functional areas.
E.g.
Services that are shared
- angular
-- app.js (initialisation)
-- shared
--- appformsClient (Exposing a service that will deal with the interaction between the angular app and the $fh.forms API)
-- form
--- formDirective.js (The Directive for rendering the form)
--- formController.js (The Controller for the form directive)

Binding Between Angular Fields and the Submission Model
The idea here was to push as much of the logic for dealing with the form structure and values to the Submission Model in the $fh.forms Client SDK.
In the Submission Model (https://github.com/feedhenry/fh-js-sdk/pull/182), a function was added to build the full state of the form. This function binds the submission values to the field so the user is presented with the full structure of the form including any existing values in the field.
When rendering the form, the full structure can be exploited to more easily render the form as all of the data binding and initialisation has already been done by the Submission model.

Looks good

Took a look too after our meeting yesterday, I think this structure is looking very nice!
One question I have is: Should we keep the dist/ folder under version control? Isn't it generated by wfm-template-build or something similar?


Update aws-java-sdk-kinesis, ... to 1.11.723

Updates

com.amazonaws:aws-java-sdk-kinesis
com.amazonaws:aws-java-sdk-s3
com.amazonaws:aws-java-sdk-sqs

from 1.11.717 to 1.11.723.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws" } ]

labels: library-update, semver-patch

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   46.26%   46.26%
=======================================
  Files          24       24
  Lines         281      281
  Branches        2        2
=======================================
  Hits          130      130
  Misses        151      151

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 aef6f51...3e50e33. Read the comment docs.


chore: clean-up various config files

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


creating sacredboard core module.

Adding usage example in the bootstrap.

Hmm, some tests have failed. I have to check why and then I'll do the rest :-)

I fixed my tests and wanted to push the changes also to your branch in your repo (it should work, but perhaps I am missing permission or doing something wrong). So I have set up a new branch in my repository that includes both your changes and my fixed tests. I hope to finally finish it soon.


Virtual dataset level metadata not showing

I notice the virtual dataset level metadata doesn't seem to be displaying. See screenshot.
Here's the closed issue, where you added this feature https://github.com/Labcas-NIST/Labcas-ui/issues/49 (for reference)

Hi Heather, this is indeed a bug. Although I see you're not in incognito mode, it doesn't matter since this issue should not occur either way. Can you try test? I updated the code since the expand button was having issues as well. I'm hoping that may have fixed things.

@yuliujpl Seems to be working great! thanks!


Application resource plans hitting RequiresReplace can fail with "application already exists"

Description
Depending on speed of hardware, plan changes requiring a destroy of an application and new creation, such as changing constraints will fail the first time as the application is not removed from juju fast enough.
However waiting a little bit after the failure, for the application to be removed, and running the plan a 2nd time will succeed in creating the new application.
Urgency
Annoying bug in our test suite
Terraform Juju Provider version
0.8.0
Terraform version
1.5.5
Terraform Configuration(s)
terraform {
  required_providers {
    juju = {
      version = ">= 0.7.0"
      source  = "juju/juju"
    }
  }
}

provider "juju" {
}

resource "juju_model" "this" {
  name = "unit-test"
}

resource "juju_application" "this" {
  model = juju_model.this.name
  name = "ubuntu-lite"
  charm {
    name = "jameinel-ubuntu-lite"
  }
  trust = true
  expose{}
}

Reproduce / Test
With the above plan, run:
`terraform init && terraform plan && terraform apply`

Once deploy is finished, update the plan to include in the application resource:
`constraints = "arch=amd64 cores=1 mem=4096M"`

Now run `terraform plan && terraform apply`, the plan should be:
`Plan: 1 to add, 0 to change, 1 to destroy.`

Type `yes` to run the plan. Depending on the speed of the hardware used, this should trigger:

juju_application.this: Destroying... [id=unit-test:ubuntu-lite]
juju_application.this: Destruction complete after 0s
juju_application.this: Creating...
╷
│ Error: Client Error
│
│   with juju_application.this,
│   on heather.tf line 17, in resource "juju_application" "this":
│   17: resource "juju_application" "this" {
│
│ Unable to create application, got error: cannot add application "ubuntu-lite": application already exists

### Debug/Panic Output

_No response_

### Notes & References

_No response_

There may still be scenarios where this is a problem. However the majority of cases should be fixed. Open a new issue if this occurs again.


vue-cli create a new app,and this app can't run with 'npm run serve'

Version
3.5.3
Environment info
Environment Info:

  System:
    OS: Linux 4.15 Deepin 15 15.9.2
    CPU: (6) x64 Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz
  Binaries:
    Node: 10.15.3 - /usr/local/bin/node
    Yarn: Not Found
    npm: 6.9.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 70.0.3538.77
    Firefox: Not Found
  npmPackages:
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0-beta.3
    @vue/babel-plugin-transform-vue-jsx:  1.0.0-beta.3
    @vue/babel-preset-app:  3.5.3
    @vue/babel-preset-jsx:  1.0.0-beta.3
    @vue/babel-sugar-functional-vue:  1.0.0-beta.3
    @vue/babel-sugar-inject-h:  1.0.0-beta.3
    @vue/babel-sugar-v-model:  1.0.0-beta.3
    @vue/babel-sugar-v-on:  1.0.0-beta.3
    @vue/cli-overlay:  3.5.1
    @vue/cli-plugin-babel: ^3.5.0 => 3.5.3
    @vue/cli-plugin-eslint: ^3.5.0 => 3.5.1
    @vue/cli-service: ^3.5.0 => 3.5.3
    @vue/cli-shared-utils:  3.5.1
    @vue/component-compiler-utils:  2.6.0
    @vue/eslint-config-standard: ^4.0.0 => 4.0.0
    @vue/preload-webpack-plugin:  1.1.0
    @vue/web-component-wrapper:  1.2.0
    eslint-plugin-vue: ^5.0.0 => 5.2.2
    vue: ^2.6.6 => 2.6.10
    vue-eslint-parser:  2.0.3
    vue-hot-reload-api:  2.3.3
    vue-loader:  15.7.0
    vue-router: ^3.0.1 => 3.0.2
    vue-style-loader:  4.1.2
    vue-template-compiler: ^2.5.21 => 2.6.10
    vue-template-es2015-compiler:  1.9.1
    vuex: ^3.0.1 => 3.1.0
  npmGlobalPackages:
    @vue/cli: 3.5.3

Steps to reproduce
vue create xxx
cd xxx
vue run serve

What is expected?
it's run
What is actually happening?
npm run serve                                                                                1 ↵
><EMAIL_ADDRESS>serve /home/fungleo/Sites/TMS/draggable-test
> vue-cli-service serve

 INFO  Starting development server...
 10% building 1/1 modules 0 activeevents.js:174
      throw er; // Unhandled 'error' event
      ^

Error: ENOSPC: System limit for number of file watchers reached, watch '/home/fungleo/Sites/TMS/draggable-test/public'
    at FSWatcher.start (internal/fs/watchers.js:165:26)
    at Object.watch (fs.js:1253:11)
    at createFsWatchInstance (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:38:15)
    at setFsWatchListener (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:81:15)
    at FSWatcher.NodeFsHandler._watchWithNodeFs (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:233:14)
    at FSWatcher.NodeFsHandler._handleDir (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:429:19)
    at FSWatcher.<anonymous> (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:477:19)
    at FSWatcher.<anonymous> (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:482:16)
    at FSReqWrap.oncomplete (fs.js:154:5)
Emitted 'error' event at:
    at FSWatcher._handleError (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/index.js:260:10)
    at createFsWatchInstance (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:40:5)
    at setFsWatchListener (/home/fungleo/Sites/TMS/draggable-test/node_modules/chokidar/lib/nodefs-handler.js:81:15)
    [... lines matching original stack trace ...]
    at FSReqWrap.oncomplete (fs.js:154:5)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>serve: `vue-cli-service serve`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>serve script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/fungleo/.npm/_logs/2019-03-29T10_24_44_708Z-debug.log

I'm chinese, and my english is poor, so...
I'm try reinstall<EMAIL_ADDRESS>but the problem alway there...

See https://github.com/guard/listen/wiki/Increasing-the-amount-of-inotify-watchers

just change your port from :8080 to :xxxx


(Feature Request) Auto Mute as well as .mute length

I would like to suggest the feature of including an option length in mute like .mute name m. Having that would push her abilities for auto mod.

Something like this? https://github.com/Kwoth/NadekoBot/commit/93c28918d0e3d40919ddf4790e70f4684614e1c9

if I understood that right then yes

it exists. .mute 5 @someone


SDK Wrappers aren't triggered by Release Libraries action

This action didn't trigger the other, dependent actions. Possibly due to the required: true parameter?

Better solution is now available with the workflow_call trigger, so the release-libs.yaml can call the other required libraries.


Icon buttons are not reachable with arrow key navigation

Clarity Version: 3.1.1
WCAG 2.1 Guideline: 2.1.1 Keyboard
Test Environment:

Test URL: https://clarity.design/documentation/buttons
OS: Windows 10
User Agents/Assistive Technologies: Firefox 76.0
Screen reader: NVDA 20.1

Steps to reproduce:

Open the test url.
Go to the icon buttons section and try to navigate through list of horizontally placed icon buttons using down arrow keys.

Expected result:
Screen reader user should be able to navigate through the list of icon buttons using arrow keys or tab key navigation.
Actual result:
Screen reader user is able to navigate through icon buttons using tab key navigation only. Even tried  switching between NVDA browse/focus mode, still navigation happens with tab key only.

Hi there, this is an automated message.
In an effort to dedupe and track accessibility reports consistently, this issue is being closed and tracked internally with our Accessibility team. It will still be triaged and addressed as part of ongoing work.


Backport changes to release-1.16

This PR backports some changes on master to the release-1.16 branch, namely #110, #114, #115, #116.

The release-1.16 branch will be used to reproduce the API ref or not?

@kbhawkey We took a snapshot when fixing the API ref generator for 1.17. The snapshot starts the release-1.16 branch, in case we need to regenerate the API ref for whatever reason. Since we have reworked the directory layout for gen-apidocs, we are only porting back some bug fixes manually.
I have already regenerated the 1.16 API ref to fix an issue found at k8s/website. (https://github.com/kubernetes/website/pull/18587).

@kbhawkey We took a snapshot when fixing the API ref generator for 1.17. The snapshot starts the release-1.16 branch, in case we need to regenerate the API ref for whatever reason. Since we have reworked the directory layout for gen-apidocs, we are only porting back some bug fixes manually.
I have already regenerated the 1.16 API ref to fix an issue found at k8s/website. (kubernetes/website#18587).

OK, was news to me.

/lgtm


[CLEANUP] use const wherever possible

If a parameter or local var can be const it should be const, or * const, or const * const

Agreed. const * helps us see what function params are input vs potential output, and prevents us from accidentally writing to things we shouldn't be writing to.
And since we can't have private class members in C, having struct member pointers as const * const is an alternative that makes sure we don't change these member vars.


Update: Neovim.Neovim.Nightly version 0.9.0

Pull request has been automatically created using 🛫 WinGet Releaser.


Annotation selection

Annotation selection

ViewMapAnnotation extended for selectedContent = content to display when the annotation is selected
Map initializers extended for selectedItem binding - allows to observe the selected item and set it from code (apart from the user tap manual selection)

Hi @BucekJiri
Thank you for you contributions to this repository. I've found your additions very helpful, especially the clustered content.
However, having another size for the selected content leads to this behavior:
https://user-images.githubusercontent.com/5942764/199796300-55492a61-1ace-4a86-88ee-a4c83b67cffb.mp4
The selected content does not seem to be centered like the unselected content. Did you notice this too? Any idea how to fix this? I could not find a way so far.
By the way: Any idea how to animate the change between unselected to selected?
Thank you very much!
-Roman

Hey @rmnblm
great to hear you found it helpful. This PR is a bit outdated and does not contain all the recent changes.
Check this issue. It might help you with the problem.
When it comes to animating the selection, I have not tried that but I believe triggering an animation in the overridden setSelected method in the MKMapAnnotationView could do the trick.


ARM support

Hi. I'm just wondering if bk agents can run on arm instances?
I'd love to run my agents on the new m6g instance types to save money.
Edit: I just noticed this PR https://github.com/buildkite/elastic-ci-stack-for-aws/pull/690. Can this be approved and merged ?

Hi @nitrocode! We're hoping to look at it and merge it soon. There are some issues on the branch which need to be resolved, first. Please subscribe to that PR to follow progress. :-)


Epic: provide pg_analytics

Motivation
Requested by @acervantes23
DoD
pg_analytics extension available for customers
Implementation ideas
Tasks
- [ ] Evaluate extension, gather use cases
- [ ] Gather dependencies
- [ ] Add extension to compute image
- [ ] Rollout extension

Other related tasks and Epics

https://blog.paradedb.com/pages/introducing_analytics
https://github.com/neondatabase/neon/issues/7015

Request by a prospective customer (TagniFi). They plan to bring in around 2 TB of data.

They sell data via a console, downloadable dataset, api and a excel plugin. So their prod use case is mostly to enable this.

Timeline is more towards end of April.


How to load IPL to a specific player?

Hey, I wanted to know how can I load an IPL to a specific player?

The same way as you do to trigger other things to a specific player (show markers, allow a player to press a key at a certain point, etc).
You need to trigger the client function on the desired player from the server.
I don't really know much about this since I've never ran a proper FiveM server.

Okk i see, is it possible to use your function server side ?
RegisterServerEvent('loadIPL')
AddEventHandler('loadIPL', function()
   Citizen.CreateThread(function()
     BikerClubhouse1 = exports['bob74_ipl']:GetBikerClubhouse1Object()
      BikerClubhouse1.Walls.Set(BikerClubhouse1.Walls.plan, BikerClubhouse1.Walls.Color.brown, true)
   end)
end)

Something like that ? and after i just need to call the event ?

I suggest you post your question on FiveM forum since I never implemented this ^^
From what I see, I think you could write it simply this way:
RegisterServerEvent('loadIPL')
AddEventHandler('loadIPL', function()
     BikerClubhouse1 = exports['bob74_ipl']:GetBikerClubhouse1Object()
     BikerClubhouse1.Walls.Set(BikerClubhouse1.Walls.plan, BikerClubhouse1.Walls.Color.brown, true)
end)

But I guess you get the spirit :)

Okk thx ! <3


Documentation Offline

Describe the bug
Your current documentation links are all offline. Cloudflare Forbidden error. It has been like this for a couple weeks.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://docs.crawlab.cn/'
See error
403 Forbidden
cloudflare

Screenshots
If applicable, add screenshots to help explain your problem.

Due to the ICT policy in Mainland China, the site's beian has expired which resulted in the inaccessibility. You are recommended to check out the documentation in the Github repo https://github.com/crawlab-team/crawlab-docs.

Fixed


SNMPv2-SMI.py deprecation warnings

Expected behavior
When i run pysnmp based SNMP agent i got deprecation warnings (see attachment).
Actual behavior
When i run pysnmp based SNMP agent i got deprecation warnings (see attachment).
Detailed steps
When i run pysnmp based SNMP agent i got deprecation warnings (see attachment).
Python package information
7.1.13
Operating system information
SLES 15
Python information
3.11
(Optional) Contents of your test script
No response
Relevant log output
/usr/lib/python3.11/site-packages/pysnmp/smi/mibs/SNMPv2-SMI.py:1259: DeprecationWarning: isFixedLength is deprecated. Please use is_fixed_length instead.
  if impliedFlag or obj.isFixedLength():
/usr/lib/python3.11/site-packages/pysnmp/smi/mibs/SNMPv2-SMI.py:1231: DeprecationWarning: isFixedLength is deprecated. Please use is_fixed_length instead.
  elif obj.isFixedLength():

This is by design and preparation for the next major PySNMP release.


logs by date

Description
Is there a possibility of logging the errors and output files name in date format?

Hello, you can use custom logger or use external tools like logrotate.


[DesignProposal]: Run Operator with towhee.op()

Background and Motivation
There is no way to run op in Towhee directly, so should run Operator with towhee.op() like towhee.pipeline().
Design

Run op in hub or local file

from towhee import op

resnet_op = op(operator_src='resnet-image-embedding', init_args={'model_name': 'resnet50'}, tag='main') # tag defaults to 'main'
result = resnet_op(call_args='{image_path': './test.png'})

Run op with class(notebook)

from towhee import op

resnet_op = op(operator_class='ResnetImageEmbedding', init_args={'model_name': 'resnet50'})
result = resnet_op(call_args='{image_path': './test.png'})

Pros and Cons

Users can run op in hub directly.
Develop can test op with towhee.op() instead of run their local class file.
Support to run and test op in notebook directly.

Anything else? (Additional Context)
No response

It will be much simpler if we drop the metaprogramming tricks:
resnet_op = op(operator_class='ResnetImageEmbedding', model_name=resnet50)
result = resnet_op(image_path ='./test.png')

metaprogramming tricks:

maybe need to refer to the API towhee.pipeline()
def pipeline(pipeline_src: str, tag: str = 'main', install_reqs: bool = True):

metaprogramming tricks:

maybe need to refer to the API towhee.pipeline()
def pipeline(pipeline_src: str, tag: str = 'main', install_reqs: bool = True):

It's better to have an API towhee.operator that aligns with towhee.pipeline, but make towhee.op a syntax sugar for convenience.

Done with #549


Refactor functions.CalcThermalLoads

Refactor parts of code from CalcThermalLoads into separate small functions that can be called by alternative functions to 'CalcThermalLoads'

this has been merged into master. thank you!


version: 0.2.0

Proposed changes
version 0.2.0
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #71 (f0d4997) into main (0725075) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #71   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           60        60
  Lines          666       666
  Branches        99        99
=========================================
  Hits           666       666

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 0725075...f0d4997. Read the comment docs.


Go To Definition fails if containing file is referenced with forward slash

UPDATE: Issue with forward slashes, not junctions.
See https://github.com/Microsoft/visualfsharp/issues/4016#issuecomment-348663946.
Go To Definition fails for a symbol if the containing file is in a directory that is linked via NTFS junction or symbolic link.
Go To Definition fails for a symbol if the containing file is in a directory that is referenced with forward slashes in the .fsproj file.
In a .NET Standard F# project, this fails with error "Cannot navigate to the symbol under the caret."
In a .NET Framework F# project, this works.
In a .NET Standard C# project, this works.
Repro steps
In Visual Studio, create a Class Library (.NET Standard) F# project.
Create a .fs file in a subdir.
Create a junction to the subdir. (e.g. in Powershell run: New-Item myjunction -ItemType Junction -Value origsubdir)
Edit your .fsproj and manually reference the file using a forward slash.
In your main Library.fs, reference something from that file and then try to F12 to it. Results in error: "Cannot navigate to the symbol under the caret."
(Note that intellisense/compilation are working; just F12 fails.)
Related information
Visual Studio Community 2017 Preview
Version 15.5.0 Preview 4.0
Visual F# Tools (Experimental) <IP_ADDRESS>12401 (from feed: https://dotnet.myget.org/F/fsharp-preview/vsix)
dotnet 2.1.1-preview-007165

Update:
This is not an issue with junctions. It is an issue with forward slash path separators in the project file. (In trying to create a minimal repro I had hand-edited the fsproj and used forward slashes without realizing that could be the issue.) However it is still true that only .NET Standard F# fails with forward slashes; I verified with hand-edited F# "classic" and C# projects that they worked with forward slashes.
The navigation failure essentially stems from here:
https://github.com/Microsoft/visualfsharp/blob/afb8b75ae9523f69d20b7f853c0c8d939ddd089d/src/fsharp/NameResolution.fs#L1463
The equality fails because one side has forward slashes and the other backslashes.
The root cause seems to be here:
https://github.com/Microsoft/visualfsharp/blob/afb8b75ae9523f69d20b7f853c0c8d939ddd089d/vsintegration/src/FSharp.Editor/LanguageService/LanguageService.fs#L229-L233
The sources coming in retain the forward slashes verbatim from the project file.
Would the following fix for the above nested function be suitable? Using Path.GetFullPath (which uses winapi GetFullPathNameW) to convert to a canonical form with backslashes.
let fullPath p =
	if Path.IsPathRooted(p) then p
	else Path.Combine(Path.GetDirectoryName(path), p)
	|> Path.GetFullPathSafe

@chadunit I think that's a good thing to try.  @KevinRansom for FYI as he implemented this.

Oh woah, thanks a lot for figuring this out! I'd been using forward slashes to avoid any potential issues with the Linux build server, and I had just assumed F12 was still buggy.

This is still the problem with latest versions of VS 2017 and 2019 Preview.
In addition to "Go To Definition", it reversely affects "Find All References".
And dotnet new ... F# template for ASP.NET Core inserts forward slash in fsproj.
I'd gladly try to create a PR, but given the fix is already suggested, that feels like stealing credit 🙂


Willroman/builder

Fixes #17779
I would have created this pull request between my copy of the master branch and the master branch on aspnet/SignalR-Client-Cpp but, unfortunately, I added some commits from this branch (builder) to my master and didn't think they should be included as part of the pull request.
I have actually done my testing against the builder branch as it seems to be more similar to the current client libraries of the other languages.
This is my first time contributing code to any open source projects so please forgive any missteps in proper etiquette.

If you make a new branch from master and git cherry-pick the single commit you want then you can get off my builder branch which will make the PR easier to merge :D

Thanks for the info on using git cherry-pick.  I've pretty much used git/github via Visual Studio so, unfortunately, I'm not that well versed in the command line utilities.  I will try to see if I can pull off that feat ;-)

Replaced by https://github.com/aspnet/SignalR-Client-Cpp/pull/23


Illegal Update to non-static final field - Gradle 7.6

Hello,
I'm seeing the following error when utilizing Strict-Logging:
java.lang.IllegalAccessError: Update to non-static final field com.xyz.logger attempted from a different method (com$typesafe$scalalogging$StrictLogging$_setter_$logger_$eq) than the initializer method <init>
It works on Gradle 6.9.2, but not when I upgrade to 7.6 (which I need to).  Any ideas?
Additional Info:
Gradle: com.typesafe.scala-logging:scala-logging_2.12:3.9.2 (scala-logging_2.12-3.9.2.jar)
JDK11

seems pretty weird/inexplicable that Gradle version could possibly make any difference


Generic processing pipeline

Problem

To allow new transaction type, we need to switch types (or make generic) in our processing pipeline

Summary of Changes

Make (as much as reasonably possible at this time) the processing pipeline generic

Fixes #

From here, block-verification (replay) and block-production (banking) can be made generic independently.

Is it necessary to have SVMTransactionAdapter? Can't we add those methods to SVMTransaction instead?

Is it necessary to have SVMTransactionAdapter? Can't we add those methods to SVMTransaction instead?

SVMTransaction is meant to be stuff necessary for svm. converting into our legacy formats for our internal/plugin interfaces elsewhere isn't necessary for that.

The goal is for SVMTransactionAdapter to go away eventually

What will the interfaces taks SVMTransaction when Adapter goes away? If so, can the adapter's methods be added to SVMTransaction as @jstarry suggested, then remove them when time comes?

What will the interfaces taks SVMTransaction when Adapter goes away? If so, can the adapter's methods be added to SVMTransaction as @jstarry suggested, then remove them when time comes?

Yeah they will take SVMTransaction when the time comes. BUT SVMTransaction is part of the public interface of svm. We need to be careful with what we're adding to that interface, and shouldn't add things in that aren't necessary. (cc @buffalojoec)

Yeah they will take SVMTransaction when the time comes. BUT SVMTransaction is part of the public interface of svm. We need to be careful with what we're adding to that interface, and shouldn't add things in that aren't necessary. (cc @buffalojoec)

Yeah it gets a little hairy when you have methods on SVMTransaction that are never used within SVM. I really only skimmed this PR, but something like trait RuntimeTransaction: SVMTransaction could help.

Yeah it gets a little hairy when you have methods on SVMTransaction that are never used within SVM. I really only skimmed this PR, but something like trait RuntimeTransaction: SVMTransaction could help.

Yeah not opposed to an approach like this; think naming can't be RuntimeTransaction since that's a concrete type (at least rn 😉 )
Something like
pub trait RuntimeTransactionTrait: StaticMeta + SVMTransaction {
    // temporary conversion fns we need
}

Then for many of our processing functions we don't even need RuntimeTransaction<TRAIT> we can just do impl RuntimeTransactionTrait 🤔
@tao-stones any thoughts on a name here, do you like this approach more?

Then for many of our processing functions we don't even need RuntimeTransaction<TRAIT> we can just do impl RuntimeTransactionTrait

I like this approach - taking trait instead of concrete RuntimeTransaction or SVMTransactionAdapter.
As for naming, I feel RuntimeTransactionTrait isn't descriptive enough, is it? but I can't come up better one 🤔

As for naming, I feel RuntimeTransactionTrait isn't descriptive enough, is it? but I can't come up better one

How about ExecutableRuntimeTransaction?
I just completely made up the executable bit.

Sorry I still don't really understand why we need TransactionWithMeta. Why don't we just use RuntimeTransaction<SanitizedTransaction>?

Sorry I still don't really understand why we need TransactionWithMeta. Why don't we just use RuntimeTransaction<SanitizedTransaction>?

Chatted with @apfitzge and I think I'm understanding things better now. Kinda spaced on the goal of being able to pass the upcoming RuntimeTransaction<ResolvedTransactionView> through the tx pipeline. So rather than using RuntimeTransaction<Tx> where Tx: SVMTransactionAdapter everywhere we are going with Tx: TransactionWithMeta which is nice because then when we need to convert to geyser / block entry tx types we can make use of the static metadata rather than needing to compute that metadata from the types that impl SVMTransactionAdapter.


[Bug] Discord Canary Stuck on Update Failed Loop (M1 Mac)

Describe the Bug

Exactly what the title says, it keeps on retrying every 10 seconds, and fails. I tried repairing and reinstalling, didn't do anything.
To Reproduce

All of a sudden discord froze, so i closed it, and when i open it, this happens.
Expected Behavior

Discord opens normally. after perhaps an update of some sort because it is discord canary.
Screenshots

Discord Version

Canary
Additional Context

Is there any evidence this is a BetterDiscord issue?


what is the  "per set sequence lengths"

Hi,
I am not so clear from the manual about "per set sequence lengths". Is it about the target genome or the query sequence?
I am comparing two genome assembly from minicap2, and I am using the minidot.R to plot the paf result from the minimap2, but I don't have another legnth file. Is it okay to take the header from the sam file from minimap2? Is there a file format for the length file?
It seems the executable minidot in the bin folder only works with minimap version 1, and it is recommended to use minicap2 now....
Thanks,
Cui

Hi Cui,
it is a 3 column file gives the length of each contig of each genome:

genome_id - name of genome fasta file without .fa
contig_id
contig_length

I usually create it using samtools faidx, and then just by adding the genome_id in the first column.
If you run bin/minimap it will do that automatically. I should work with minimap2 too, but I never had the time to really test it. Hope that helps!

Hi Thomas,
Thank you for your quick and clear explanation.  I have one more question, should  I follow the order that target file should be placed in the first half of the file, and query genome in the second half of the file. Or the order doesn't matter?
I generated the paf file using minimap2, would this file compatible?
I couldn't find the installation instruction from the minicap website, so I am not sure if i could run it at all....
Thanks,
Cui

The order shouldn't matter - I think it is just joined by genome_id. minimap2 paf files should work too. But let me know if they don't.

Hi,
I made a length file as instructed, and a paf file generated in minimap2, and using minicot.R script to construct the dot, but got an error message like this:
Error in Math.factor(x$V2) : ‘cumsum’ not meaningful for factors
Calls: cbind -> lapply -> FUN -> Math.factor
Execution halted
Do you know what could be the reason? Or maybe there is something wrong with my steps?
thanks,
Cui

@smallfishcui were you able to fix this problem?
I am in the same situation.
Thanks!


HttpRequest.Builder is not thread-safe

As a follow up to https://github.com/open-policy-agent/opa/issues/5720, we noticed that the HttpRequest.Builder inside AllowCallable was not thread-safe. Not sure if Guava cache will allow concurrent loading when calling cache.get(cachableRequest, new AllowCallable(request, opaUrl, allowOnError)), if yes, then the current implementation might hit the same issue.

The builder is only ever used in the call invocation, so moving the construction of it there seems reasonable to me. PR welcome 🙂

Fixed in v1.5.1. Thanks for reporting and fixing, @xhl1988!


Fix test for biggus 0.14.

Tiny fix needed as private element got moved.

thanks @pp-mo

Does this not mean the minimum dependency needs to be updated?

Does this not mean the minimum dependency needs to be updated?

No, this is not an API change :  The _Elementwise class was, and remains, a private element.

But there is no biggus._init in Biggus 0.13, though I guess it is only a test...

You could use a try/except to avoid this, e.g.:
try:
    from biggus._init import _Elementwise
except ImportError:
    _Elementwise = biggus._Elementwise


Problem with Dagger.@spawn over scenarios

Running a testmode pipeline with Dagger.@spawn inside the do_pipeline function seems to stop (although local activity is occurring).
This could be a problem with the result printing, or a lack of understanding of the spawning process.

Closed by #359


Adding basic implementation of CreateTableFromSnapshot

Basic Implementation of CreateTableFromSnapshot, Unit test case creation is in progress

This change is

getting some issues during creation of unit test case, so currently working on that, mean time I pushed the basic API implementation code.

It seems build has failed due to timeout and download issue, looking into it

The builds have been super flaky in the last few days.  I am restarting the ones that failed for you.

One of the travis build failed with some timeout issue, may need to restart.

Hey roopak,
I restarted the build and it failed again because of this reason.
[WARNING] /v/third_party/googletest/googletest/src/gtest-death-test.cc:836:: Death tests use fork(), which is unsafe particularly in a threaded context. For this test, Google Test detected 2 threads.
Aborting because exceptions are disabled: unrecoverable error in MakeCall(): TableAdmin(parent=projects/the-project/instances/the-instance) uh-oh [7=PERMISSION_DENIED] -
Let me know if I should restart it again.

PTAL

One of the build is failing, but not getting, is it related to my change or  need to restart the build.

@roopak-qlogic I restarted that build.  Please let me know when this PR is ready for another review.  I still think the tests do not cover enough of the PollLongRunningOperation() function

Did not get chance to add new test cases, but in next couple of days I will add and push for review.

Code is ready for review

PTAL


Implementation of Reverse:v1

To review the changes only for the new Reverse op, see https://github.com/NervanaSystems/ngraph/compare/mkarzyns/opsets_poc...tomdol/reverse

It would make sense to make this a PR agains the opset branch and later change the base branch, when that branch is merged.

I changed the base branch, the PR is a bit cleaner, but there are conflicts due to changes in the opset branch.


Update Ruby to 2.6.5

Heroku supports v2.6.5.
https://devcenter.heroku.com/articles/ruby-support#ruby-versions
I updated .ruby-version and Gemfile.

Thank you for your pull request!
Could you also update Dockerfile?
https://github.com/pocke/ruboty-ruby-jp/blob/8fe496d27a22f2f3e4a7c75b4b7c8efc9eb933ee/Dockerfile#L1

Oops!
I updated ruby version on Dockerfile.

Thank you!


feat: show stderr output from code execution

Closes https://github.com/mfontanini/presenterm/issues/251.
This uses the os_pipe crate to capture both stdout and stderr from code execution, displaying them interleaved as you would see it in the terminal. The interactivity of seeing each output line as it happens (instead of all at the end) is retained.
https://github.com/dmackdev/presenterm/assets/79006698/0b93e4b3-96dd-4e98-9ac0-e03dd1b43514

Awesome, thanks for the fix!


Data Catalog not loading with MySQL adapter for remote servers (using IP)

Describe the bug
I am trying to connect to a MySQL Server using IP as host with password, username, and database.
I am able to run queries, but data catalog doesnt load at all, or it gives error

To Reproduce
Try to connect to any MySQL remote server
Expected behavior
Shows Catalog
Actual behavior
Sometimes it gives error as shown in screenshot or it doesnt load at all, but queries still can be run.
Additional context
What is the output of harlequin --version?
harlequin, version 1.9.0
Installed Adapters:

duckdb, version 1.9.0
sqlite, version 1.9.0
mysql, version 0.1.0

(I am using ubuntu 23, with zsh shell. using mysql adapter for AWS RDS)

https://stackoverflow.com/questions/29772337/python-mysql-connector-unread-result-found-when-using-fetchone
This might be useful


Flutter app not running.

This the error:
FAILURE: Build failed with an exception.

Where:
Build file 'C:\Users\siddh\AndroidStudioProjects\opinion_frontend\android\app\build.gradle' line: 26

What went wrong:
A problem occurred evaluating project ':app'.

Could not open cp_dsl generic class cache for script 'C:\src\flutter\flutter\packages\flutter_tools\gradle\flutter.gradle' (C:\Users\siddh.gradle\caches\6.7.1\scripts\5mfyzrmhp6txkku6greg87vhl).
BUG! exception in phase 'semantic analysis' in source unit 'BuildScript' Unsupported class file major version 60

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 8s
Exception: Gradle task assembleDebug failed with exit code 1
Flutter Doctor:
`
[√] Flutter (Channel stable, 2.2.1, on Microsoft Windows [Version 10.0.19042.985], locale en-US)
• Flutter version 2.2.1 at C:\src\flutter\flutter
• Framework revision 02c026b03c (4 days ago), 2021-05-27 12:24:44 -0700
• Engine revision 0fdb562ac8
• Dart version 2.13.1
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at C:\Users\siddh\AppData\Local\Android\sdk
• Platform android-30, build-tools 30.0.2
• Java binary at: C:\Program Files\Common Files\Oracle\Java\javapath\java.exe
• Java version Java(TM) SE Runtime Environment (build 16.0.1+9-24)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
[√] VS Code (version 1.56.2)
• VS Code at C:\Users\siddh\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.22.0
[√] Connected device (3 available)
• GM1911 (mobile) • c1caeb86 • android-arm64  • Android 11 (API 30)
• Chrome (web)    • chrome   • web-javascript • Google Chrome 90.0.4430.212
• Edge (web)      • edge     • web-javascript • Microsoft Edge 91.0.864.37
`

@BSiddharth , I think installing android studio will solve this problem as running flutter doctor is showing an issue - Android Studio (not installed)

@Atharva-Vijay-Khade actually i do have android studio not sure why suddenly vscode is not able to find it

@BSiddharth, found one similar issue, check this once https://github.com/flutter/flutter/issues/67986

@Atharva-Vijay-Khade  It does not work for me. In the image below it says to restart the ide but the result is same. Interesting thing is that flutter changes the path name automatically and put  '.plugins'  after the path

@BSiddharth : Install Android SDK Command-line tools from Android Studio.then re run flutter doctor.

@mehtahardikr Already done that. This has to be because of the java jdk update. I will try downgrading

Solved AS not found. The command that worked for me was flutter config --android-studio-dir "C:\Program Files\Android\Android Studio" ( No = before the path) but still apps are not running. flutter vscode shows install extension message on run start debugging. I am same problem as this person https://stackoverflow.com/questions/67565475/flutter-vscode-shows-install-extension-message-on-run-start-debugging

App is running on android studio but not on vscode

Hi @BSiddharth
> Could not open cp_dsl generic class cache for script 'C:\src\flutter\flutter\packages\flutter_tools\gradle\flutter.gradle' (C:\Users\siddh\.gradle\caches\6.7.1\scripts\5mfyzrmhp6txkku6greg87vhl). doesn't seem to Android Studio or Flutter
Can you please provide a complete reproducible minimal code sample, and complete flutter run --verbose logs running the minimal code sample?
Thank you

Hello @TahaTesser  that problem had been solved a while ago and  5 hours ago code was still not working, vscode was not able to find any device to run but flutter doctor was showing 3 devices connected but I randomly started the app again just now and everything seems to be working. I did not touch anything , just ran the code on AS where it was working as mentioned in the last comment. The code was just for the simple counter app that is automatically generated by flutter.
I can still provide any details you want or we can close this one.


fix(config-ui): change jira board selection box to be searchable

Summary

Change the Jira board selection box to be searchable, and avoid the interface only returning 50 boards limit.
Does this close any open issues?
Closes #3254
Screenshots

Other Information
Nothing.

Maybe we can change this tip "Select the boards you would like to sync."

I find a bug again:

type mapping is wrong when edit board

some other bugs:

when open modal agian, the board name lose
@likyh This was an advanced timing issue that's existed for sometime. JIRA API data is not ready on the first modification request so the active board names are not applied, on the second modification request the proxy data is ready so true board name are displayed. Our Design requirements have only ever identified Boards using "Board [ID]", our blueprint only stores board ID as well. So this is why I've left this behavior intact until the timing issue is resolved. The Important thing is that Board ID is still preserved, regardless if actual name or generic board name is displayed.

(main also exist) When I delete a board or adjust the order of the boards, the transformation of the second page will not match. For example, the original requirement of board1 is tagA, and the requirement of board2 is tagB. After 1 is deleted, board2 becomes tagA
This was recently fixed by my last PR

some other bugs:

when open modal agian, the board name lose

@likyh This was an advanced timing issue that's existed for sometime. JIRA API data is not ready on the first modification request so the active board names are not applied, on the second modification request the proxy data is ready so true board name are displayed. Our Design requirements have only ever identified Boards using "Board [ID]", our blueprint only stores board ID as well. So this is why I've left this behavior intact until the timing issue is resolved. The Important thing is that Board ID is still preserved, regardless if actual name or generic board name is displayed.

(main also exist) When I delete a board or adjust the order of the boards, the transformation of the second page will not match. For example, the original requirement of board1 is tagA, and the requirement of board2 is tagB. After 1 is deleted, board2 becomes tagA

This was recently fixed by my last PR for "Blueprint Settings Degraded..."

1 is after the bug fix you said and it's a new bug on this PR and it fixed.
2 is https://github.com/apache/incubator-devlake/issues/3294

Where does the board limit of 50 come from in the first place?


Added missing _time_average_seconds per server metrics

haproxy_server_http_connect_time_average_seconds
haproxy_server_http_total_time_average_seconds
haproxy_server_http_queue_time_average_seconds
haproxy_server_http_response_time_average_seconds
Added the above metrics, these are reported by haproxy, but exporter doesn't collect them. They're important identifying outliers at a quick glance in larger pools.

I wish haproxy had proper counters for this data, rather than pre-computed decaying averages.

any chance getting this merged?


Helm Chart - repository

Hello.
Is there a plan to put the helm chart into some repository?
I can see the helm chart is in https://github.com/lucky-sideburn/KubeInvaders/tree/master/helm-charts but it would be great if it is in some repository.
The ArtifactHub doesn't have the Readme:
https://artifacthub.io/packages/helm/kubeinvaders/kubeinvaders
Thank you...

Hi @ruzickap!
Yes I should do it! I can create a separate repo only for the helm chart! what do you think?
Thanks!
E

Make sense to me...
There are some GitHub actions which helps publish helm charts as GH pages.

Done! https://artifacthub.io/packages/helm/kubeinvaders/kubeinvaders


TCP Margin calcuation for subscription liveness check.

TCP isn't instant though.  RTO * retries can be hundreds of seconds if a link exhibits a high round-trip time.  Even a single lost packet could lead to a breakage for TCP here, whereas this code might accommodate such a thing for UDP.
I don't see that it makes sense to branch like this PR attempts to.  In this context, the only thing that is special about UDP+MRP as compared to TCP is that we have immediate access to retry parameters with MRP, whereas these are likely obfuscated for TCP.  But they still absolutely exist.
Originally posted by @msandstedt in https://github.com/project-chip/connectedhomeip/pull/11795#discussion_r749855682

Also note the maximum idle timeout computation in #11795 is not compliant with the current spec, which has jitter and exponential backoff.

copy from #11795,
from @msandstedt
The simplest thing would be to decouple from the exact nature of the underlying transport's reliability features. We could instead say: transport attempts to be reliable. Therefore, we should allow 1 strike before assuming the subscription is dead. So given mMaxIntervalCeilingSeconds, fail at 2x mMaxIntervalCeilingSeconds.
As an aside, this PR's computation for maximum idle-state MRP timeout is not spec compliant.

we need to discuss here for the suitable timeout value

@bzbarsky-apple @mrjerryjohns @msandstedt

The suggestion in https://github.com/project-chip/connectedhomeip/issues/12009#issuecomment-976093089 might be pretty reasonable....

@bzbarsky-apple @yunhanw-google - does this need to be fixed for 1.0?

Going to redirect to @msandstedt

I feel like the code @yunhanw-google has in place is pretty reasonable for 1.0, and particularly so if TCP is out of scope.  This is how we compute the liveness timeout:
    System::Clock::Timeout timeout =
        System::Clock::Seconds16(mMaxIntervalCeilingSeconds) + mpExchangeCtx->GetSessionHandle()->GetAckTimeout();

And this is GetAckTimeout:
System::Clock::Milliseconds32 ExchangeContext::GetAckTimeout()
{
    System::Clock::Timeout timeout;
    if (IsUDPTransport())
    {
        timeout = GetMRPConfig().mIdleRetransTimeout * (CHIP_CONFIG_RMP_DEFAULT_MAX_RETRANS + 1);
    }
    else if (IsTCPTransport())
    {
        // TODO: issue 12009, need actual tcp margin value considering restransmission
        timeout = System::Clock::Seconds16(30);
    }
    return timeout;
}

So because we don't have a good way to know link conditions for TCP, we are just adding 30-seconds margin.  But I don't believe we are going to be supporting TCP in 1.0.

Agree, thanks @msandstedt . Moving to post V1.0

@pidarped You might be interested in this one...


fix(50435): Duplicate seeming Code Actions for convert const to let

FIxes #50435

Would you rather I file a separate issue for the fact that this refactoring discards trivia?
/// <reference path="someFile.d.ts" />

// @ts-check

// all of these comments should be preserved
const x = 123;
x = 456;


The connection is already closed error

I am using the next code to maintain connection errors and reconnections.
self.client = redis.createClient(self.cfg.port, self.cfg.host, {retry_strategy: function (options) {
    if (options.error) {
        if (options.error.code === 'ECONNREFUSED') {
            // End reconnecting on a specific error and flush all commands with a individual error
            return new Error('The server refused the connection');
        }
        if (options.error.code === 'ECONNRESET') {
            return new Error('The server reset the connection');
        }
        if (options.error.code === 'ETIMEDOUT') {
            return new Error('The server timeouted the connection');
        }
    }
    if (options.total_retry_time > 1000 * 60 * 60) {
        // End reconnecting after a specific timeout and flush all commands with a individual error
        return new Error('Retry time exhausted');
    }
    if (options.attempt > 5) {
        // End reconnecting with built in error
        return new Error('Retry attempts ended');
    }
    // reconnect after
    return 1000;
}});
self.client.on('error', function (err) {
    log.error({err: err}, 'EventParser.redis.client error: %s', err);
    process.exit(1);
});

But recently I received 28Mb logs of  trying to execute SMEMBERS command on a closed connection. retry_strategy was not triggered. Another workers which use the same redis connection did not produce any errors at all.
{"name":"myworker","hostname":"2cc64ce4ff8a","pid":30450,"level":40,"err":{"message":"SMEMBERS can't be processed. The connection is already closed.","name":"AbortError","stack":"AbortError: SMEMBERS can't be processed. The connection is already closed.\n    at handle_offline_command (/node_modules/redis/index.js:843:15)\n    at RedisClient.internal_send_command (/node_modules/redis/index.js:873:9)\n    at RedisClient.smembers (/node_modules/redis/lib/commands.js:62:25)\n    at bound (/node_modules/q/q.js:1942:29)\n    at Promise.apply (/node_modules/q/q.js:1165:26)\n    at Promise.promise.promiseDispatch (/node_modules/q/q.js:788:41)\n    at /node_modules/q/q.js:1391:14\n    at runSingle (/node_modules/q/q.js:137:13)\n    at flush (/node_modules/q/q.js:125:13)\n    at _combinedTickCallback (internal/process/next_tick.js:67:7)\n    at process._tickDomainCallback (internal/process/next_tick.js:122:9)","code":"NR_CLOSED"},"msg":"MyWorker.processData error: AbortError: SMEMBERS can't be processed. The connection is already closed.","time":"2016-07-22T12:45:02.600Z","v":0}

Version: node_redis v.2.6.2 and Redis 3.0.7
Platform: Node.js v5.10.1 on Ubuntu 14.04.4 LTS
Description: Already closed connection does not trigger retry_strategy handlers

Looks like I'm experiencing the same problem. Getting The connection is already closed having retry_strategy in options

hey guys, any suggestions on how to resolve it?

Please if anyone can update on some settings we are missing

let me check how to import "self.client = redis.createClient()" from you code.
in my image:
`
yourNodeModuleForRedis.js
-- self.client = redis.createClient()
you are importing "yourNodeModuleForRedis" to another module as below :
callingRedis.js
-- var redis = require("yourNodeModuleForRedis")
`
then you may meet "PUBLISH(or SUBSCRIBE) can't be processed. The connection is already closed"

I believe I'm hitting this issue when upgrading ElastiCache. Connection dies, and no notification comes through. Connection just stops working. locking up the entire node process. Am I missing something else simple to fix this, or I am indeed seeing the same?

Did anyone have any luck resolving this issue?

recently I've got this error and can't understand how options.error can be null

if (options.error.code === 'ECONNREFUSED') {
^
TypeError: Cannot read property 'code' of null
at Object.self.client.redis.createClient.retry_strategy (/var/daemon/script.js:86:24)
at RedisClient.connection_gone (/node_modules/redis/index.js:606:41)
at Socket. (/node_modules/redis/index.js:286:14)
at Socket.g (events.js:273:16)
at emitNone (events.js:85:20)
at Socket.emit (events.js:179:7)
at endReadableNT (_stream_readable.js:906:12)
at nextTickCallbackWith2Args (node.js:474:9)
at process._tickCallback (node.js:388:17)

If the connection is closed the client will not reconnect at all anymore. Therefore the retryStrategy will not be triggered either.
As soon as you return a non number from the retryStrategy no retry will happen anymore and all further commands will be rejected with the known error message.
@pavelsc you have a different issue, please check if there's a error currently.


feat: suggestions for miss spelled sub commands

Notes for Reviewers

I have created a function that finds the closest sub-commands for the miss spelled sub commands entered by the user. The function uses the concept of Levenshtein distance to find the closest matched argument. This function has been added in theutils package for later use under the name of closest-arg.go

Added checks for argument overflow for mesheryctl perf apply command.

This PR fixes #6269
Screenshots:

Signed commits

[x] Yes, I signed my commits.

Codecov Report
Patch coverage has no change and project coverage change: -4.28 :warning:

Comparison is base (958a787) 16.47% compared to head (eb0081d) 12.19%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7308      +/-   ##
==========================================
- Coverage   16.47%   12.19%   -4.28%
==========================================
  Files         118      119       +1
  Lines       16300    16359      +59
==========================================
- Hits         2685     1995     -690
- Misses      13128    14041     +913
+ Partials      487      323     -164

Flag
Coverage Δ

gointegrationtests
?

unittests
12.19% <0.00%> (-0.05%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/perf/apply.go
43.14% <0.00%> (-0.53%)
:arrow_down:

mesheryctl/internal/cli/root/perf/error.go
36.50% <0.00%> (-1.83%)
:arrow_down:

mesheryctl/internal/cli/root/perf/perf.go
21.87% <0.00%> (-4.06%)
:arrow_down:

mesheryctl/pkg/utils/closest-arg.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/status.go
1.51% <0.00%> (-53.79%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.53% <0.00%> (-53.69%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.00% <0.00%> (-46.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.65% <0.00%> (-42.98%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
4.00% <0.00%> (-42.67%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.66% <0.00%> (-42.67%)
:arrow_down:

... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

This is looking pretty great, @gurpreet-legend

@alphaX86 what do you think?

This is looking pretty great, @gurpreet-legend

Thanks @leecalcote , today I'll be working on apply command as well. And will try to improve this further if possible :)


iOS: Crash on  @objc func windowDidBecomeHidden(_: Notification) {     removeKVObserver()   }

Describe the bug
We have a crash that occurs on iOS when reading the reanimated keyboard height.
Code snippet
  const { height: keyboardHeight } = useReanimatedKeyboardAnimation()

Repo for reproducing
It's in our main application. I can't provided a project to reproduce at this time.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

mobile[62515:8433225] *** Terminating app due to uncaught exception 'NSRangeException', reason: 'Cannot remove an observer <KeyboardMovementObserver 0x600007c9cba0> for the key path "center" from <UIInputSetHostView 0x1342e19e0> because it is not registered as an observer.'
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000180437330 __exceptionPreprocess + 172
	1   libobjc.A.dylib                     0x0000000180051274 objc_exception_throw + 56
	2   Foundation                          0x0000000180b2f1e0 -[NSObject(NSKeyValueObserverRegistration) _removeObserver:forProperty:] + 616
	3   Foundation                          0x0000000180b2f5bc -[NSObject(NSKeyValueObserverRegistration) removeObserver:forKeyPath:] + 132
	4   Foundation                          0x0000000180b2f4d0 -[NSObject(NSKeyValueObserverRegistration) removeObserver:forKeyPath:context:] + 184
	5   mobile                              0x00000001033eaf6c $s32react_native_keyboard_controller24KeyboardMovementObserverC16removeKVObserver33_98D42DC26656EA4CABF65DBC7E7C9563LLyyF + 192
	6   mobile                              0x00000001033eabb4 $s32react_native_keyboard_controller24KeyboardMovementObserverC21windowDidBecomeHiddenyy10Foundation12NotificationVF + 36
	7   mobile                              0x00000001033eac38 $s32react_native_keyboard_controller24KeyboardMovementObserverC21windowDidBecomeHiddenyy10Foundation12NotificationVFTo + 120
	8   CoreFoundation                      0x000000018036bff8 __CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__ + 140
	9   CoreFoundation                      0x000000018036bf1c ___CFXRegistrationPost_block_invoke + 84
	10  CoreFoundation                      0x000000018036b424 _CFXRegistrationPost + 404
	11  CoreFoundation                      0x000000018036ae10 _CFXNotificationPost + 664
	12  Foundation                          0x0000000180b5309c -[NSNotificationCenter postNotificationName:object:userInfo:] + 88
	13  UIKitCore                           0x0000000116dd3314 -[UIWindow _setHidden:forced:] + 680
	14  UIKitCore                           0x0000000116c380ec -[_UIRemoteKeyboards setWindowEnabled:force:] + 324
	15  UIKitCore                           0x0000000116c3a6b4 -[_UIRemoteKeyboards setWindowLevel:sceneLevel:forResponder:] + 588
	16  UIKitCore                           0x000000011683f8f8 -[UIKeyboardSceneDelegate setWindowLevel:sceneLevel:forResponder:] + 144
	17  UIKitCore                           0x000000011684048c -[UIKeyboardSceneDelegate setTextEffectsWindowLevelForInputView:responder:] + 864
	18  UIKitCore                           0x0000000116843cc4 -[UIKeyboardSceneDelegate setKeyWindowSceneInputViews:animationStyle:] + 524
	19  UIKitCore                           0x0000000116843a84 -[UIKeyboardSceneDelegate setInputViews:animationStyle:] + 132
	20  UIKitCore                           0x0000000116844a54 -[UIKeyboardSceneDelegate setInputViews:animated:] + 72
	21  UIKitCore                           0x0000000116844aa4 -[UIKeyboardSceneDelegate setInputViews:] + 52
	22  UIKitCore                           0x0000000116842b48 __102-[UIKeyboardSceneDelegate _reloadInputViewsForKeyWindowSceneResponder:force:fromBecomeFirstResponder:]_block_invoke.314 + 24
	23  UIKitCore                           0x0000000116c10d0c __65-[UIPeripheralHost(UIKitInternal) queueDelayedTask:forKey:delay:]_block_invoke + 156
	24  libdispatch.dylib                   0x0000000106bb1d50 _dispatch_client_callout + 16
	25  libdispatch.dylib                   0x0000000106bb5208 _dispatch_continuation_pop + 756
	26  libdispatch.dylib                   0x0000000106bcc8d4 _dispatch_source_invoke + 1676
	27  libdispatch.dylib                   0x0000000106bc2634 _dispatch_main_queue_drain + 848
	28  libdispatch.dylib                   0x0000000106bc22d4 _dispatch_main_queue_callback_4CF + 40
	29  CoreFoundation                      0x000000018039a784 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 12
	30  CoreFoundation                      0x0000000180394de4 __CFRunLoopRun + 1912
	31  CoreFoundation                      0x0000000180394254 CFRunLoopRunSpecific + 584
	32  GraphicsServices                    0x0000000188eb7c9c GSEventRunModal + 160
	33  UIKitCore                           0x0000000116d9aff0 -[UIApplication _run] + 868
	34  UIKitCore                           0x0000000116d9ef3c UIApplicationMain + 124
	35  mobile                              0x00000001027f7ed8 main + 96
	36  dyld                                0x0000000106579514 start_sim + 20
	37  ???                                 0x0000000106699f28 0x0 +<PHONE_NUMBER>
	38  ???                                 0x3a17800000000000 0x0 +<PHONE_NUMBER>152520704
)

Smartphone (please complete the following information):

iPhone 14 Pro, iOS 16.4, Xcode 14.3

@gtokman I guess it happens on latest version of the library, right?
As an intermediate solution I can suggest to revert back to 1.4.4
In a meantime I'll try to find a solution and provide a fix 🙂

@kirillzyusko yes, on the latest version of the library and 2.14 for reanimated.

@gtokman could you please also tell more about how do you use the library and when this exception occurs?
How many KeyboardProviders do you have? When this exception occurs (I guess when keyboard becomes hidden, but how long do you need to show/hide keyboard before you meet an exception?

@gtokman also, if you are able to test, could you please try to change:
keyboardView?.removeObserver(self, forKeyPath: "center", context: nil)

to
_keyboardView?.removeObserver(self, forKeyPath: "center", context: nil)

Just add _ to keyboardView variable.
I have an assumption, that when we refer to keyboardView it may re-run a process of finding a view and return a new instance without a listener 🤷♂

@gtokman could you please check whether #157 fixes the problem?

@kirillzyusko I had the same issue and applying the patch from your PR fixes the crash for me. Thank you 👍

Awesome @cgav Glad to know the fix is actually working!
I'm going to merge this PR within this week and prepare a new release, since it's quite impactful change 😎

FYI I'm also having this issue. I'm just using the KeyboardProvider in my App.tsx, and then in one place in my app:
 React.useEffect(() => {
    const show = KeyboardEvents.addListener('keyboardWillShow', () => {
      bottomSheetRef.current?.expand();
    });

    return () => {
      show.remove();
    };
}, []);

The thing is that I can't reproduce this myself but Sentry has reported over 2.5k users experiencing this. What's weird to me about that 2.5k number is that the feature where the above code is, isn't much used. Could it be that the crash occurs when just applying the KeyboardProvider?

Hi @gunnartorfis
Yes, the issue will occur when you've wrapped app into KeyboardProvider (underhood it'll render KeyboardControllerView which setup all listeners and etc.).
@gunnartorfis could you please try to apply patch from https://github.com/kirillzyusko/react-native-keyboard-controller/pull/157 and see whether the amount of errors will be decreased?

Hi @gunnartorfis
Yes, the issue will occur when you've wrapped app into KeyboardProvider (underhood it'll render KeyboardControllerView which setup all listeners and etc.).
@gunnartorfis could you please try to apply patch from #157 and see whether the amount of errors will be decreased?

I see, thanks. Since I'm unable to reproduce the issue myself I don't want to release the patch to production until it has been released. If you have any ideas on how I might reproduce

this myself then I'd be happy to. It seems to be hitting quite the range of iOS versions, etc - I'm on 16.3 myself.

@gunnartorfis that's the problem that I don't know how to reproduce this crash 🤷♂ I can not reproduce the problem in example app and I don't have reproducible example.
I have only guesses, that the reference to the keyboard view is changing and I'm trying to remove an observer which doesn't exist anymore... I really hope this patch can fix everything - if not, then I'll be happy to continue investigation on how to resolve the problem. The new version of the library 1.5.4 will be published today.
@gunnartorfis since you are not using other functionality apart of keyboardWillShow you can safely downgrade keyboard-controller version to 1.4.4 - this version will not have crashes, since I'm not using KVO mechanism in this version.

@kirillzyusko Thanks for the info. I will downgrade for now and keep a close eye on this.
Feel free to tag me if there's anything I can do to help out.

I'm seeing this issue in bugsnag with 1.10.3 Here's the stack trace:
Hardware Model:     iPhone14,5
Process:            App
Identifier:         com.app
Role:               Foreground
OS Version:         iOS 16.5

NSRangeException: Cannot remove an observer <KeyboardMovementObserver 0x283160310> for the key path "center" from <UIInputSetHostView 0x10b613c70> because it is not registered as an observer.

0  CoreFoundation +0x9cb0      ___exceptionPreprocess
1  libobjc.A.dylib +0x183cc    _objc_exception_throw
2  Foundation +0x6621c         -[NSObject(NSKeyValueObserverRegistration) _removeObserver:forProperty:]
3  Foundation +0x65f5c         -[NSObject(NSKeyValueObserverRegistration) removeObserver:forKeyPath:]
4  Foundation +0x65e6c         -[NSObject(NSKeyValueObserverRegistration) removeObserver:forKeyPath:context:]
5  App +0x571850           KeyboardMovementObserver.removeKVObserver() (KeyboardMovementObserver.swift:98:20)
6  App +0x572b50           @objc KeyboardMovementObserver.keyboardWillAppear(_:)
7  CoreFoundation +0x3758c     ___CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__
8  CoreFoundation +0xdb824     ____CFXRegistrationPost_block_invoke
9  CoreFoundation +0xbe8b4     __CFXRegistrationPost
10 CoreFoundation +0x4baf8     __CFXNotificationPost
11 Foundation +0x5cd34         -[NSNotificationCenter postNotificationName:object:userInfo:]
12 UIKitCore +0xb7ed9c         -[UIInputWindowController postNotificationName:userInfo:]
13 UIKitCore +0x39245c         ___68-[UIInputWindowController postValidatedStartNotifications:withInfo:]_block_invoke
14 UIKitCore +0x3b2e28         -[UIInputWindowController postValidatedStartNotifications:withInfo:]
15 UIKitCore +0x314424         -[UIInputWindowController postStartNotifications:withInfo:]
16 UIKitCore +0xb7fad4         ___77-[UIInputWindowController moveFromPlacement:toPlacement:starting:completion:]_block_invoke_4
17 UIKitCore +0x3d8ad0         +[UIView(UIViewAnimationWithBlocksPrivate) _modifyAnimationsWithPreferredFrameRateRange:updateReason:animations:]
18 UIKitCore +0xcd2ac          +[UIView _setupAnimationWithDuration:delay:view:options:factory:animations:start:animationStateGenerator:completion:]
19 UIKitCore +0x5538c0         +[UIView(UIViewAnimationWithBlocks) _animateWithDuration:delay:options:animations:start:completion:]
20 UIKitCore +0x2ce2b4         -[UIInputViewAnimationStyle launchAnimation:afterStarted:completion:forHost:fromCurrentPosition:]
21 UIKitCore +0x2ce174         -[_UIViewControllerKeyboardAnimationStyle _launchAnimation:afterStarted:completion:forHost:fromCurrentPosition:]
22 UIKitCore +0x2cde80         +[UIView(Animation) _performWithAnimation:]
23 UIKitCore +0x2cddc8         -[_UIViewControllerKeyboardAnimationStyle launchAnimation:afterStarted:completion:forHost:fromCurrentPosition:]
24 UIKitCore +0xb807d0         ___77-[UIInputWindowController moveFromPlacement:toPlacement:starting:completion:]_block_invoke.459
25 UIKitCore +0x36f4e4         -[UIInputWindowController chainPlacementsIfNecessaryFrom:toPlacement:transition:completion:]
26 UIKitCore +0x36ca9c         -[UIInputWindowController moveFromPlacement:toPlacement:starting:completion:]
27 UIKitCore +0x36bc00         -[UIInputWindowController setInputViewSet:]
28 UIKitCore +0x36b638         -[UIInputWindowController performOperations:withAnimationStyle:]
29 UIKitCore +0x27b92c         -[UIKeyboardSceneDelegate setKeyWindowSceneInputViews:animationStyle:]
30 UIKitCore +0x27ae20         -[UIKeyboardSceneDelegate setInputViews:animationStyle:]
31 UIKitCore +0x204678         -[UIKeyboardSceneDelegate setInputViews:animated:]
32 UIKitCore +0x20168c         -[UIKeyboardSceneDelegate setInputViews:]
33 UIKitCore +0x8ea2e4         ___102-[UIKeyboardSceneDelegate _reloadInputViewsForKeyWindowSceneResponder:force:fromBecomeFirstResponder:]_block_invoke.329
34 UIKitCore +0x8e9d74         -[UIKeyboardSceneDelegate _reloadInputViewsForKeyWindowSceneResponder:force:fromBecomeFirstResponder:]
35 UIKitCore +0x8e8a30         -[UIKeyboardSceneDelegate _reloadInputViewsForResponder:force:fromBecomeFirstResponder:]
36 UIKitCore +0x27c8c0         -[UIResponder(UIResponderInputViewAdditions) reloadInputViews]
37 UIKitCore +0x21a44c         -[UIResponder becomeFirstResponder]
38 UIKitCore +0xfdb73c         ___99-[_UIRemoteViewController _serviceWantsKeyboardEventsWithPromotedFirstResponder:completionHandler:]_block_invoke

@bcgilliom can you please test 1.11.0 and if it's still reproducible, then open a new issue? 🙏


iOS: New option for changing height of story titles

Add this option as a 4 part segmented control in the story titles settings menu. This allows you to increase the height for story titles to get more space for story titles and story content previews.
4 parts:

2 line title only (this is built but is exposed in a show_content_preview preference)
2 line title + 2 line story content (todays default)
2-4 line title (variable height) + 2-4 line story content (variable height)
2-4 line title (variable height) + 2-6 line story content (variable height)

We'll have to use the new smaller thumbnail to make this work, but it should be top-right positioned, not center-right positioned.

This is great! Can you add a segmented control to the menu on story titles? I think this should be changeable when you're on the view. Widen the popover menu if necessary.
[Title only | S | M | L]
[Title | Short | M | L]
[Title | S | Medium | L]
[Title | S | M | Long]

I don't remember if that's how we do segmented controls. But if it looks bad with changing titles, just use [TITLE ONLY | S | M | L].

Scratch that, make it [TITLE ONLY | 1 | 2 | 3 | 4]

Sure, I can do that. I agree that we don't want S | M | L, since those are already used for the font size. Shouldn't change the segment titles with the selection, as that'd make the tap targets move (and would look strange).

On iPad, tapping on the new "title only" control shows the infrequent stories message. Also, there's a visual glitch on the segmented control on the right side.
Also, can you add another segmented control for displaying images: [No Image | Small | Large]

Also, since we're here working on the titles, can we add an HTML decode for the story content preview? Use Outside magazine to test it, as it has many » html entities.

Also any chance we can make the 2 and 3 options use dynamic heights? I want to avoid titles like this.
(Ugh I cant upload screenshots on iOS, so imagine or find a story with a single line of title and a single line of content taking up a whole lotta white space)

The visual glitch (the segmented control not the same length, I assume you mean) is due to using proportional widths for the titles (the TITLE ONLY segment is wider than the numbers).
With three kinds of small/large options here (preview, image, text), and the struggle to find titles that work, I think it might be time to consider a design tweak. Some suggestions:

A minimal change, prefix each of the segmented controls with a prompt, e.g.

ORDER: [NEWEST | OLDEST]
SHOW: [ALL | UNREAD]
PREVIEW: [NONE | S | M | L]
IMAGE: [NONE | S | L]
TEXT: [XS | S | M | L | XL]

A variation could be to use images instead of text for the segmented control titles, along with the prompt. Perhaps just for the PREVIEW, IMAGE, and maybe TEXT ones, e.g. 1, 2, 3, or 4 stacked horizontal lines for PREVIEW, small or large image representations for IMAGE, and increasingly large letter A or T for TEXT.

Another option could be to replace the segmented controls with disclosure rows that push to a list of options, and show the selected option, like in the preferences (consistency is a good thing):

ORDER: NEWEST >
SHOW: UNREAD >
PREVIEW: MEDIUM >
IMAGE: SMALL >
TEXT: MEDIUM >

Should make the corresponding changes to the story menu too. What do you think?

I think what I've done is satisfactory, so may not need to tweak the design after all.  So I'll close this. But let me know if you do want to change it to add prompts, or other ideas.

Looks like Large Images and a [2] on story titles is causing overlapping issues.

I haven't seen any issues of that with those (or other) settings. Can you show an example, so I can reproduce it?

Happens on Nancy.

Here's another one I see pretty consistently.
https://www.newsblur.com/site/2186/apartment-therapy-saving-the-world-one-room-at-a-time

I'm still getting this bug on a few titles, I think always with an image. Check the NYT feed. If you like, use the android_speed account (password is newsblur) and you should be able to see a huge variety in the river.


broken dependencies in pc file

The tcti-tabrmd pkg-config file needs to list gio as a dependency. Without this (the current state) builds will fail against the TCTI since the tcti-tabrmd.h file includes gio.h which is usually under ${includedir}/glib-2.0/gio/ and this won't be on the compiler search path.

merged in #79


Adjust export order Lucide Svelte

Closes #1536
What is the purpose of this pull request?

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

Description
SSR in Svelte Kit was not working due to a change in #1499. Thanks to @JanTrichter for the fix
Before Submitting

[x] I've read the Contribution Guidelines.
[x] I've checked if there was an existing PR that solves the same issue.

Beta released on<EMAIL_ADDRESS>and it works according to comments in #1536

Beta released on<EMAIL_ADDRESS>and it works according to comments in #1536

lgtm :+1:


⚠ Bofrost.de has degraded performance

In 5b99153, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 856 ms

Resolved: Bofrost.de performance has improved in 9f97ef1 after 304 days, 1 hours, 52 minutes.


famous create [name-of-project] Error

Hello,
I am behind a proxy.
I can install famous cli, but when I run to create project. It shows error:
events.js:85
throw er; // Unhandled 'error' event
^
Error: getaddrinfo ENOTFOUND code.famo.us
at errnoException (dns.js:44:10)
at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:94:26)
Nodejs and npm can works nomally in other project.
Please help to solve it.
Thank you

@whatisor Thanks for filing this.
I have migrated this question to the repo for the CLI tool where they are better suited to help you out.  https://github.com/Famous/famous-cli/issues/56