fix: Reduce error alerts by increasing caching tolerance

By submitting this PR, I am indicating to the Numberscope maintainers that I have read and understood the contributing guidelines and that this PR follows those guidelines to the best of my knowledge. I have also read the pull request checklist and followed the instructions therein.

Before this PR, all cached sequences loaded factorizations into the cache in strict lockstep with values. But many visualizers do not use the factorizations, and they require extra computations and/or http requests that can be wasteful if the results will not be used. This PR decouples the two caches, so that values can be cached without factorizations (although not vice versa, since you may need the values to obtain their factorizations).
This created a new issue: since the factorizations are not cached until their first request, every single OEIS sequence was reporting caching errors in the ShowFactors visualizer. That was because the ShowFactors simply loops over every value and gets its factorization. The first call to getFactors was triggering the cache fill, but for OEIS sequences the cache fill is asynchronous, so it returns immediately without having completed the operation. That meant the second call to getFactors was guaranteed to trigger an error that caching was in progress, since it completely certain was in fact in progress at that point.
The solution to this conundrum was to take advantage of p5's display loop. In the draw method, such caching errors are caught, and trigger a "progress spinner" cursor to appear. But of course p5 then simply gets to the next frame (after a delay dependent on the frame rate) and tries to access the cached values again. After a (typically very small) number of frames, the cache fill has completed, and the sequence is able to supply the factorizations without any CachingErrors. At this point, the draw() method clears the cursor and all seems well.
Running this PR in the frontend with https://github.com/numberscope/backscope/pull/67 in a local backend is producing really robust visualizer results for brand-new OEIS sequences for me. Hopefully the same will prove true in the review.
Resolves #260

Just a note that this appears to work perfectly on my machine, when paired with backend pr 67.

I read through the code and it seems clear to me, for the most part.  No particular questions.  Should I merge?


Need a waitForTextToDismiss(text) method

From<EMAIL_ADDRESS>on August 09, 2012 07:25:14
There are certain situations where the only visible indication that occurs is that the text disappears or changes after click.
In my case it's a listview click that needs to register.  I need to wait for the text to change before completing the next operation (clicking the back button).  The back button is pressed so quickly after the previous clickOnText that the app doesn't have time to fully process the clickOnText, causing an incorrectly failed test.  So for now, I've stuck a sleep command in there.
This would also work very well for dialogs (I saw how the waitForDialog to dismiss was implemented and it looks like it might have some reliability problems--it doesn't (can't?) actually look for the presence of a dialog, only a change in the view count.)
What this waitForTextToDismiss could do is first wait for the text to appear, and then for the text to disappear.
Original issue: http://code.google.com/p/robotium/issues/detail?id=308

From<EMAIL_ADDRESS>on August 09, 2012 23:05:16
You can use while(!solo.searchText()) with any searchText parameter to achieve what you want.
Status: WontFix
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on August 22, 2012 10:46:34
Right.  I would think this is a common enough use case to warrant its own readable method in Solo:
void waitForNoText(String text ) {
while (searchText(text)) {
sleep(200);
}
}


Star Rail - Add MoC and PF cycle count to the floors

The HoYoLab website and app display the cycles used for each floor

NVM its there just named something not obvious

Might be smart to alias them


Local instance of openhunt not logging in

After cloning app to local machine, running rails server, and then following instructions to add all the missing pieces (gem bundle install, creation of db, db migration, and adding my own settings.local.yml file), I got the app running at localhost:3000.
Tried to login and it goes to twitter, authenticates, redirects back to the application and after spending a few seconds on localhost:3000/login/callback# it changes the message from logging in to Login unsuccessful. Something went wrong.
Would really be grateful for some direction in where to go from here to make the app work :)
Thanks!!

References #17.

Having same issue as @kiriappeee here - I'm wondering if it has to do with the twitter key/secret. I'm getting a 500 error when trying to access /success with the following response: Twitter::Error::Unauthorized in SessionsController#auth_success

im switching out oauth to use omniauth. ill get that checked in soon and
some instructions on the README on how to set up the twitter app for dev
On Saturday, December 19, 2015, Jordan<EMAIL_ADDRESS>wrote:

Having same issue as @kiriappeee https://github.com/kiriappeee here -
I'm wondering if it has to do with the twitter key/secret. I'm getting a
500 error when trying to access /success with the following response:
Twitter::Error::Unauthorized in SessionsController#auth_success
—
Reply to this email directly or view it on GitHub
https://github.com/OpenHunting/openhunt/issues/36#issuecomment-166011805
.

Awesome, thanks @jacquescrocker

Brilliant. Looking forward to it :). :+1:

give this a try! https://github.com/OpenHunting/openhunt/blob/master/README.md

@jacquescrocker I'm getting this error after following your instructions:
https://www.dropbox.com/s/6vhpiqp4tzixolw/Screenshot 2015-12-20 13.04.22.png?dl=0

@speizerj The twitter API is throttled, the last time i checked, you can make 15 requests per 15mins. The message you are getting means you have exceeded this limitation and will need to wait for it to enter the next window to reset.

@nblackburn This was on the first request though. I deleted my comment because I just needed to pull in the latest from master - it's working now. Thanks


Documentation is missing a mutation example

As the title states, we have no example on how to create a mutation, which could be super beneficial for everyone. 👍

Would love to see that as well. Also, it would be cool to have an example how to insert multiple objects for example. I'm not sure how the syntax for that would look like.

@Bernold1 The documentation for a simple mutation is added now. If you see the tests we do have another example as well.

thanks, @vkolgi! is it actually possible to insert multiple items at once?

Awesome thanks @vkolagi


Generic function type not narrowed based on enum

TypeScript Version:  4.1.2

Search Terms: narrowing generic function type
Code
enum SyntaxType {
    A = 'a',
    B = 'b',
}

interface NodeA {
    type: typeof SyntaxType.A
}

interface NodeB {
    type: typeof SyntaxType.B
}

type SyntaxNode = NodeA | NodeB

type State<T extends SyntaxNode> = {
    node: T
}

const traverse = <T extends SyntaxNode>(
  state: State<T>,
  node: T,
): State<T> => {
  switch (node.type) {
    case SyntaxType.A:
      return handleA(state, node)
    case SyntaxType.B:
      return handleB(state, node)
  }
}

const handleA = (state: State<NodeA>, node: NodeA): State<NodeA> => state
const handleB = (state: State<NodeB>, node: NodeB): State<NodeB> => state

Expected behavior: I would expect the switch statement to narrow T to the concrete types NodeA and NodeB in the corresponding cases.
Actual behavior: Instead, the generic type T is not narrowed to the concrete types NodeA/NodeB leading to an error as the result of handleA/handleB does not cover all of Ts possible interpretations.
Playground Link: https://www.typescriptlang.org/play?ts=4.1.2#code/KYOwrgtgBAygniALgQwB4BU4AdhQN4BQUxUAglALxQDky1ANESQEKU0BGDBAvgQQJZJgAJwBmyAMa4AcgHsAJsHKESURNmAAuNRtmjYCFBg0A6Ujz6DEI8VKhzFrFSXU5tr4HoNI0mHCeYLAg9vIwdcKnDyAB97BWBA4I1YFGsAHnQoYFRrEHkAZ1C0cIA+NmdiEHjtdCCJWRB8xDVhZAA3EXyIqAysnNACotRSgAomJuRrbRhU4AySxkrqqHRGAEpp2fnKMud8gHd+RAkACygRqsUTDzX8JmIJZC6hv2AzTXuSYWBEMGEQKAnZB5AA2ShGE2s9Cgl2Aa0+j2e8B8xn8zA+qmI31+-0BwPkYOYENm0Nh8OIvF4BHqjWaQNBSjYxMmWhSLLSUQWMOWUQ2bPSnJ2UEhwGpDSaeIZrCozKm-Lm4WYXNh2kVfJm7MVZQoZRFQA
Related Issues:
Strictly speaking, this is not a bug, neither is it a feature request. It's something in between. 🙂
I also don't know how easy/hard this is to catch during type inference or whether narrowing the type here is even something that we want to do.

This is a correct error. You could legally call traverse this way:
declare const aNode: NodeA, stateB: State<NodeB>;
// Unsound, passes a stateB to handleA
traverse<NodeA | NodeB>(stateB, aNode);

That said, node could be more correctly narrowed, see #33014

Thank you! Overlooked that.


Failed to install

I don't know if I'm doing something wrong but after the installation process I get this error

/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:6: error: cannot find symbol
import com.facebook.react.ReactApplication;
^
symbol:   class ReactApplication
location: package com.facebook.react
/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:9: error: cannot find symbol
import com.facebook.react.ReactNativeHost;
^
symbol:   class ReactNativeHost
location: package com.facebook.react
/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:17: error: cannot find symbol
public class MainApplication extends Application implements ReactApplication {
^
symbol: class ReactApplication
/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:19: error: cannot find symbol
private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
^
symbol:   class ReactNativeHost
location: class MainApplication
/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:35: error: cannot find symbol
public ReactNativeHost getReactNativeHost() {
^
symbol:   class ReactNativeHost
location: class MainApplication
/home/garox/Documents/def/android/app/src/main/java/com/def/MainActivity.java:5: error: MainActivity is not abstract and does not override abstract method getPackages() in ReactActivity
public class MainActivity extends ReactActivity {
^
/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:19: error: cannot find symbol
private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
^
symbol:   class ReactNativeHost
location: class MainApplication
/home/garox/Documents/def/android/app/src/main/java/com/def/MainApplication.java:34: error: method does not override or implement a method from a supertype
@Override
^
8 errors
:app:compileDebugJavaWithJavac FAILED

I believe it's related to jitpack

Sorry for delay. I think you've already solved the problem, if not, please let me know.


Do Objects passed as parameters in functions change the original value?

Do Objects passed as parameters in functions change the original value?  (as in c#)
sercare and try
Whether you can remove the 'set' in these cases :

by 'set' I mean:
flags = updateFlags(flags, flag_keys) -> updateFlags(flags, flag_keys)
(best to keep return obj)

Yes, https://python.plainenglish.io/pass-by-object-reference-in-python-79a8d92dc493


First-class support for Deltas. Closes #101

This makes ReactQuill accept Deltas, in addition to HTML strings, for value and defaultValue.
Deltas are applied via setContents(), which means that plain objects are also supported. There is no official support for updateContents(), as it is imperative and destructive.
Deltas are kept around as the internal value whenever the value prop is also a Delta. Otherwise the HTML string is used, as was before. This might have some performance repercussions due to comparing objects instead of strings during update checks.
Notably, the delta object provided by the onChange event must not be used as value. Because it does only provide a changeset, rather than a full document, it will easily trigger an update loop. I see this mistake to be as likely as it is dangerous, so ReactQuill now prevents you from doing so.
I see this delta argument of onChange as being rather useless, if not outright misleading, in a React context, and I would definitely try to correct this in the next major. Same with source probably.
The change should only require a minor bump.

I see this delta argument of onChange as being rather useless, if not outright misleading, in a React context, and I would definitely try to correct this in the next major. Same with source probably.

It might be useful in some sort of event-sourced Redux-y model if the updateContents method was somehow used as the update mechanism instead of the declarative value API. Definitely can be addressed later.

The way I see this being done, is by keeping few full deltas around (or keyframes, if you want) and a number of diff deltas (frames) in-between, and combining them at selection time, using the delta composition functions, and feeding Quill via setContents.
Maybe, we could start accepting an Array of deltas, and performing updateContents whenever it changes, keeping track of which ones we have applied, maybe optimizing it by creating keyframes every n frames, etc. But you see where this might be going.
That said, the partial delta should probably be harder to access than the full delta. Unfortunately there's no way to distinguish the two.

Going off the keyframes idea - it might make sense to actually generate a new ops array for updateContents out of a diff of the new value and the result of getContents, to minimize DOM churn due to imperative reconstruction. At that point it's almost like creating a custom React renderer that targets Quill Deltas instead of the DOM. It would make sense to do a comparison of the goals of the project relative to DraftJS.


Make the lib no_std.

The lib compiles to WASM as there is a partial support for std in wasm. This PR makes it no_std.

I don't know of any deterministic and satisfactory test for no_std. Your previous main compiled to wasm as there is partial support for std in wasm.


Title is misleading

Anything that begins with "Download this app" or "create a new app and..." is not ADDING sign-in to an app.

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

ID: 175aa1f6-e1c7-12bb-6b87-a0ee6d948aca
Version Independent ID: 8a28a4a2-0798-627f-4ff3-23b4d730a659
Content: Quickstart: Add sign-in with Microsoft to an ASP.NET web app - Microsoft identity platform
Content Source: articles/active-directory/develop/quickstart-v2-aspnet-webapp.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

Hello @boomalator , thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

@boomalator , I followed Automatic configuration and Manual configuration in Register and download the app section and I was able to download the project with sign-in method added in the controller and startup.cs:

Can you let me know which areas specifically did not work for you?

I have no idea if it works or not. I needed to add authentication with AAD to an existing Web app. Starting with a prebuilt sample or even a template is not “adding” authentication.
It’s kind of like searching for “Add a deck to your house” and getting instructions that begin with “1. Buy a house with a deck. 2. Paint the deck”.
It’s (probably) not wrong, it’s just not what the title (and searches) would suggest.
Mobile: big fingers, little buttons, and don't get me started about autocorrect.
On Jun 1, 2021, at 1:35 AM, HarshitaSingh-MSFT @.***> wrote:

@boomalatorhttps://github.com/boomalator , I followed Automatic configuration and Manual configuration in Register and download the apphttps://docs.microsoft.com/en-us/azure/active-directory/develop/quickstart-v2-aspnet-webapp#register-and-download-the-app section and I was able to download the project with sign-in method added in the controller and startup.cs:
[image]https://user-images.githubusercontent.com/25202671/120284420-d9a39400-c2d9-11eb-96e4-5171bae27e42.png
[image]https://user-images.githubusercontent.com/25202671/120284477-e88a4680-c2d9-11eb-8430-a2f8253714d8.png
Can you let me know which areas specifically did not work for you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/76180#issuecomment-851898448, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEJIXXIZPLBHIARJWJK2RFTTQSEVPANCNFSM453ZZKMQ.

@boomalator, thank you for bringing this to our notice. We will be having internal discussions on this and will decide if we are going to take this forward or not based on our business.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
CC: @mmacy for your notice.

@HarshitaSingh-MSFT I agree 100% with @boomalator's call-out on this. Can you, @MarileeTurscak-MSFT, or @James-Hamil-MSFT please reopen this issue? Thanks.
@boomalator, thanks for the call-out on this. We recently made a pass through all the quickstart and tutorial intro paragraphs to more accurately describe them, but we didn't extend that to the article metadata.  I'll add this as a work item and make another pass through to update titles and descriptions in all quickstarts any anything else that uses the "download > edit vars > run" paradigm such that they more accurately reflect the content of the article.
#reassign:mmacy

New functionality for PRMerger bots - we can reopen our own issues! #please-open


Use auto from strawberry instead of define our own

This makes it possible to use "strawberry.auto" directly instead of
having to use own own auto.
We are still exporting auto in our __init__ file for backwards
compatibility, but we should remove that as well in the future.

Nice one! maybe we can also add a deprecation warning when someone uses auto from this package :)

Hey @patrick91 . Sorry for the delay, it has been a complicated week for me :(
I thought the same, but since I totally removed auto from this package, when someone tries to import it from fields/types.py it will actually be importing the one from strawberry. And there's no way to add a warning on that specific import also.
But this can be done as: Keep the current auto as well and support both, but raise a deprecation warning if it is used. Then we proceed to totally remove it in a future release.
What do you think?

@patrick91 totally forgot about this PR. Am starting to take a look at the open PRs and saw mine in the list :)
Just added the suggested deprecation to auto/is_auto from this package. Do you want to take another look at it?

Looks good to me 😊 feel free to merge 😊


Combining Error Stream now causes errors in processing

This commit causes failures in processing.

Hi @prabjohns-atex ,
Could you give more details? Why did you close the issue 🤔 ?


Is justfile-finding-when-in-symlinked-directory behavior correct?

Consider this!
$ mkdir foo
$ printf "default:\n  echo foo" > foo/justfile
$ mkdir foo/bar
$ ln -s foo/bar baz
$ printf "default:\n  echo outer" > justfile
$ cd baz
$ just
echo foo
foo

Here, even though the users shell will show baz as the current working directory, just uses the justfile in foo, since just follows .. to go upwards to the justfile.
This behavior is surprising to me, and I'm considering changing it to pop elements off of the current working directory, which would lead to finding the outer justfile instead.

Maybe you already know all this, but let me try and add some color here:
Working directories are complicated.
On all three major platforms (windows, mac, linux), the current working directory (cwd) is actually tied to an inode on disk (err... a vnode in memory), not a path - which means that when you cd baz, the actual, real ("physical") working directory, as far as the operating system is concerned, is foo/bar.  In fact, you can do crazy things like launch a process and then mv it's cwd somewhere else - the process will continue working, and all it's relative path accesses will be relative to the new location!
Some shells also set the PWD variable to be the "logical" working directory (baz, in this case) - and will sometimes display this in place of the "physical" working directory, as would be returned by the getcwd syscall.
You can see the difference between physical and logical with the pwd -P and pwd -L commands.  All pwd -L does is echo $PWD, but pwd -P actually calls the getcwd syscall.
The kernel will always use the physical cwd to resolve relative paths, never the logical one - which means if you would have to implement your own ".." path resolution.
Note, not all shells consider the logical and physical cwd differently.  For example, my favorite shell, fish, aggressively sets the PWD variable to the result of getcwd(), and so when I run your example, my shell shows foo/bar as the cwd at the end.

Hmmm, given the complexity, I'm thinking that just trying to do something different is probably a bad idea, even if it can be reasonably argued to be better behavior.
Working directory behavior is weird, but at least it's consistent and there's a model you can use to understand why it's weird.

Current OS behavior sucks, but trying to buck the trend is probably hopeless, so I'm going to close this.
Thanks again @joshuawarner32 for the nice explanation!


Transformation is not set correctly for dropdown menu when direction is up

When creating a dropdown menu with direction == 'up', items do not show correctly. In particular, the style bottom is not set to auto and the transformation that is supposed to push the items above the dropdown is wrong.

Cause
tl;dr this is caused by dash using react 15, not 16.
Long version:
Setting direction = "up" as a prop works in Demo.js.
By looking at the reactstrap source code, reactstrap injects a prop called x-placement in RSDropdownMenu that changes depending on the dropdown direction.
Bootstrap then uses a selector based on x-placement to move the popup to the correct place.
The x-placement attribute is not a standard DOM attribute. It's just a bootstrap thing. In HTML, I believe you can inject arbitrary attributes on divs, but the expectation is that you use data attributes. Your custom attributes should start with data-. Bootstrap ignores this convention, presumably because it predates it.
Up to React 15, any extra attributes passed to <div /> (or any other JSX DOM elements) just gets silently stripped out. In React 16, they get passed through. See here for changelog.
Thus, in Demo.js which uses React 16, the x-placement attribute makes its way into the DOM, where it is picked up by the bootstrap CSS.

When we use dash-renderer, the x-placement attribute gets stripped out. Because Dash silences React warnings, we don't see a console warning like Warning: Unknown prop x-placement on <div> tag (see this issue on Reactstrap for someone with similar problems), making it even harder to diagnose. The x-placement attribute is just missing from the div.

Moving forwards
I suggest finding out when dash-renderer is planning to move to React 16 and waiting till then. In the meantime, we can just disable direction = "up" in the prop types. @tcbegley If you're happy with that I can submit a quick PR.

Dash-renderer has a PR for migrating to React 16:
https://github.com/plotly/dash-renderer/pull/97

Closing this in favour of issue #65, which is more up-to-date. Thank you!


Retrieving Access Token from External Provider

Hi,
I have configure Microsoft authentication in IS4. Would like to ask if it's possible to retrieve access token for Graph API.
Thanks.

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use StackOverflow for that. This has the advantage that questions and answers can be easily found by search engines, and that there are more people answering questions than just us.
For IdentityServer3
https://stackoverflow.com/questions/tagged/?tagnames=identityserver3&sort=newest
For IdentityServer4
https://stackoverflow.com/questions/tagged/?tagnames=identityserver4&sort=newest
For commercial support
https://identityserver.io/

thanks @brockallen. yep, posted a question in StackOverflow.
would like to shoot one more question. :) is there some kind of a temporary store for the access token received from Microsoft in IS4? Looked into the OAuthEvents "OnCreatingTicket" in the MicrosoftAccountAuthentication MW. Saw that there is an access token returned and was wondering is it saved by IS4 and will need to retrieve it by accessing thru extension points.


Can we have more selective sync?

Hi,
I have been an Omnivore user for a month and so far I like it a lot. Because I am an Obsidian user, I really appreciate your efforts for creating a sync plugin.
Concerning how to plugin sync the content, I would like to suggest two improvements:

Can we have an extra option to sync just the highlights, instead of both the saved article and the highlights?

Can we have an extra option to sync just those annotated highlights (i.e. highlights + annotations)

Obsidian is my 2nd brain and I hope to feed in only the resonated content from the saved articles in Omnivore.
Thanks!

If I understand correctly, this option is already available here, in General Settings - Filter, select import just highlights.

I have selected this option but it still sync with full text and highlights.
I just updated to v 1.0.4 of the plugin, removed all the sync notes in Obsidian and re-sync, and still I have full text and highlights, not only highlights..

Just delete some fields in the template
{{#content}}
{{{content}}}
{{/content}}

Thanks for the suggestion, this hack works!


⚠ Processors has degraded performance

In e53bf67, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 48 ms

Resolved: Processors performance has improved in 2fd832a.


Covid19Data Portal missing Lineages handling

I noticed another problem after the Covid19DataPortal update. We get the lineage information from the portal directly and store it in the database. After the Data Portal update I looked at the lineages plot and saw that it does not go further than October last year. I then checked the samples from the update and saw that of the 462.957 new assemblies, 348.256 have no lineage in the repository. Of the remaining assemblies, the newest ones have a collection_date < 2022-02-28, so the plot shows no 2023 lineages.

Okay, we need to address the following items.

Update Covid19DataPortal accessor to be aware of missing lineage.
Update Processor and Pipeline module to optionally call Pangolin in case of missing lineage.

Pipeline call without '--skip_pangolin'
Add mode to check if mutations are present in database and VCF file exists --> run Pangolin only.

Update Covigator NGS pipeline to have a mode for Pangolin only.
Enable loading of Pangolin results from assemblies into database (code is present, needs to called based on call).


[full-ci] Fix the default document language for OnlyOffice

Description

Fix the default document language for OnlyOffice
Related Issue

Fixes https://github.com/owncloud/enterprise/issues/5807

Motivation and Context

How Has This Been Tested?

test environment: The local docker stack wopi setup:
checkout to the issue branch and build the dev image make -C ocis dev-docker
go to deployments/examples/ocis_wop
run export OCIS_DOCKER_TAG=dev and docker-compose up -d
test case 1: Change the language to desirable in the Account settings (Galego is not supported). Create the new Microsoft Word document.
Expected result: The UI and document language match to ocis language.

Screenshots (if appropriate):

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[X] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

tested. works fine to me  :thumbsup:


[Discussion] Support depending on fmt seperately

Currently we use the version of fmt bundled with spdlog. This is fine, unless of course the user wants to use their own version and links to it. This can cause issues since we include fmt via spdlog in our headers but the bundled version of fmt is not guaranteed to be the same version the user links to (especially if using the version fetched via cmake and linking to system fmt).
As far as I can see, we have a few options:

Remove all usages of fmt from malloy (I did a search, there are two, both added by me). I would prefer not to go for this option as it might be useful in the future.
Have malloy depend on fmt and pull it in like we do with spdlog currently, then let the user tell malloy to use the one found by cmake instead. We'd also have to toggle spdlog's external fmt flag
Wait for std::format to become supported by the main three. This is unlikely to be soon, afaik only msvc has support for it right now.

Personally I like option (2) best, but I am biased as I love fmt :p

I'm also voting for option 2.
fmt is great and I am actively waiting for proper support. As you stated that is not something to be expected any time soon tho :/


Reworking Compressor to use pre scanned data

Rework the Compressor to use the data from the scanner for compressing the data to an archive file.

Compressor is reworked to use the results from the scanner. Now the compressor will ignore files from the backerignore files.


Re-enabling tests that were flaky

Description
A lot of golden file web tests were disabled due to flakiness. https://github.com/flutter/flutter/issues/47003 was tracking this flakiness. It has been marked as resolved, so I am re-enabling these tests. If they still flake, we will have a recent reproduction. :)
Related Issues
https://github.com/flutter/flutter/issues/47003
Tests
are a go! 🚀
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

Gold has detected one or more untriaged digests on patchset 1.
Please triage them at https://flutter-gold.skia.org/search?issue=51666.

Gold has detected one or more untriaged digests on patchset 1.
Please triage them at https://flutter-gold.skia.org/search?issue=51666.

Some of these tests were previously not enabled, and chrome updates probably made slight changes here and there. Triaging...

LGTM but curious about two things:
Some of the tests you enabled here, were not enabled previously because I couldn't get some of them running, just want to make sure that you are able to get them running this time.
Also there are 4 more tests in bottom_navigation_bar_test.dart and 1 in opacity_test.dart not enabled in here, is there any reason why they are not enabled?

I wanted to try enabling them all and seeing what the results were, either reproducing the flake or identifying ones that weren't working.
So far they all look ok except for this one (the web version is on the left, and doesn't appear to render the nav bar):

Do you have an idea why the bottom nav bar wouldn't work?

Also there are 4 more tests in bottom_navigation_bar_test.dart and 1 in opacity_test.dart not enabled in here, is there any reason why they are not enabled?

I must have missed those! Thanks! I will see if those present issues like the bottom nav bar above.
Should I file bugs for each one that doesn't work out?

Also there are 4 more tests in bottom_navigation_bar_test.dart and 1 in opacity_test.dart not enabled in here, is there any reason why they are not enabled?

I checked, and these skipped tests aren't golden file tests. They don't have comments either saying why they are skipped. Perhaps running an audit of skip: isBrowser would be a good idea for another change. There are a lot of skipped tests that aren't clear why they are being skipped.

Gold has detected one or more untriaged digests on patchset 2.
Please triage them at https://flutter-gold.skia.org/search?issue=51666.

Do you have an idea why the bottom nav bar wouldn't work?

Sorry, it's bottom app bar. Not nav bar.

I am going to re-skip the golden file test for the BottomAppBar, as it appears that it is actually broken. Filed: https://github.com/flutter/flutter/issues/51675

Looks like the flake was reproduced as well on one test run: https://cirrus-ci.com/task/6066690064646144?command=main#L1244
I will re-open the flake.

Gold has detected one or more untriaged digests on patchset 3.
Please triage them at https://flutter-gold.skia.org/search?issue=51666.

We're not ready to enable web golden tests yet. Sorry for misunderstanding.

I will close this PR since the flake has not been resolved.


Release '0.0.8'

Signed-off-by: Yevhen Vydolob<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   70.02%   70.02%
=======================================
  Files          38       38
  Lines        2365     2365
  Branches      404      404
=======================================
  Hits         1656     1656
  Misses        709      709

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 bedcfaa...c595675. Read the comment docs.


Abillty to specify download directory

Clear and concise description of the problem

Be able to specify cert download location instead of default PLUGIN_DIR
This will allow me to easily use the certs from a central location of my choosing

Just use import { PLUGIN_DATA_DIR } from 'vite-plugin-mkcert/dist/lib/constant'

I'm trying to use the constant to set up live-server https configuration but am unable to do so. Is there a way to use this constant for config files in JS?
><EMAIL_ADDRESS>dev /Users/XXXX/frontend/apps/outlook
> live-server ./dist --port 8082 --no-browser --https=./live-server.config.js

node:internal/modules/cjs/loader:499
      throw e;
      ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './dist/lib/constant' is not defined by "exports" in /Users/XXXX/frontend/apps/outlook/node_modules/vite-plugin-mkcert/package.json
    at new NodeError (node:internal/errors:387:5)
    at throwExportsNotFound (node:internal/modules/esm/resolve:439:9)
    at packageExportsResolve (node:internal/modules/esm/resolve:718:3)
    at resolveExports (node:internal/modules/cjs/loader:493:36)
    at Module._findPath (node:internal/modules/cjs/loader:533:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:942:27)
    at Module._load (node:internal/modules/cjs/loader:804:27)
    at Module.require (node:internal/modules/cjs/loader:1022:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/Users/XXXX/frontend/apps/outlook/live-server.config.js:2:14) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

// package.json
//...
  "scripts": {
    "dev": "live-server ./dist --port 8082 --no-browser --https=./live-server.config.js",
  }

// live-server.config.js
var fs = require('fs')
var mkcert = require('vite-plugin-mkcert/dist/lib/constant')

/**
 * @type {import('tls').SecureContextOptions}
 */

const config = {
  cert: fs.readFileSync(mkcert.PLUGIN_DATA_DIR + '/certs/dev.pem'),
  key: fs.readFileSync(mkcert.PLUGIN_DATA_DIR + '/certs/dev.key'),
}

module.exports = config


BUG: "SWS/BR: Copy take media item source file path of selected items to clipboard" adds an invalid character to the path

After running the action and pasting the file path in an Open File Dialog window of another app, I get a message like the following:

Do you have any more info?  Is it a slash character that's causing the issue?  Does it work if you add quotes to the front and back of the pasted name?

Hey Tim,
There seems to be a newline character added at the end of the filename
because when I paste it into notepad++, the cursor ends up on the line
underneath the filename. See the gif below:

On Fri, Mar 24, 2017 at 4:00 PM, Tim<EMAIL_ADDRESS>wrote:

Do you have any more info? Is it a slash character that's causing the
issue? Does it work if you add quotes to the front and back of the pasted
name?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reaper-oss/sws/issues/846#issuecomment-289128833, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQOehPEY0ByNyhvUU9FT-_X76ebFZfJhks5rpCDjgaJpZM4MhUPx
.

Ah, thank you. Fixed for next pre.


Update jshint

Configure jshint to hint individual files, rather than the compiled library.  This should make syntax errors much more clear on their source, rather than some arbitrarily-high-numbered line in dist/ingress-model-viewer.js

Fixed in 0.18.0


Export layers to PDF

I've worked with autocad in the past and I remember exporting a drawing to PDF and still been able to enable and disable layers in it. This is very helpful to help visualize cluttered diagrams.

I agree.
Please also consider an option to export layers to PDF (as separate files for each layer). I've been using this feature in OmniGraffle...

I agree.
Please also consider an option to export layers to PDF (as separate files for each layer). I've been using this feature in OmniGraffle...

It is definitely an important feature to preserve layers for roundtripping workflows. On the other hand, during import of an SVG exported eg. from Adobe Illustrator Draw.io will not keep existing layers as well. This should be possible to fix. If you need example files for testing, let me know.

I would really appreciate it if this was developed, as I'm really stuck how to do this :)

See also Discussion jgraph/drawio#2458. I support this feature. A user currently would have to resort to Visio or some other software to get this feature.

It would be very nice to have this feature!

It'd be really great if this feature could be implemented!
Isn't there any voting functionality to show interest in feature requests raised by someone else alreay?

+1

I would like to add another aspect to the PDF Layer export feature. If you take into account, that there is a native approach to export the layered PDF in one rush, by writing the necessary structures into the PDF, you can take another approach (with some drawbacks).
Scripted Export Approach
Scripted export as seperate layers and merging multiple PDF into one using one PDF per layer.

Script draw.io to iterate through a list of all currently visible layers (omit invisible layers)
Export every visible layer as a single PDF with a basename and an LayerOrderID and the Layername as suffix with delimiter. Save the files in the tmp location of the OS.
Combine the exported PDFs into one Layered PDF using the given order by ID and the suffix als Layername using OpenSource commandline tools (my research on this has to be added later, due not reachable now from my place of writing).

The Watermark Trick
There is one trick you can use to add layers in PDF by using the watermark / background feature first.
Watermarks are actually a simple layer at the top with adjustable transformation and tranparency with a special flag.
Watermarks can be transformed to a simple PDF layer by removing the flag.

You add  layer by layer by repeating this process.
You can add even the invisible layers when preserving the visibility flag in a suffix and set the visibility of the layer in the combined PDF.

This approach is similar to Python Scripts often used in Inkscape etc. to add features not possible with the default functionality but not worth a plugin or changing the general functionality.
Remark on export preserving object structures in SVG and PDF
It is not helpful that the layers are not included even in the SVG at least as groups. This would offer another approach for scripted conversion not relying on Draw.io itselv.
Inkscape can still open the SVG only without any preservation of layer structures. The Draw.io layer names are not available in the SVG XML tree or even reflected as nested structure.

i dont know what is so complicated to mark resources for pdf layers.
it is as simple as wrapping the regular output in the following:
/OC /L101 BDC q <content-stream> Q EDC

and creating a layer resource like that:
101 0 obj
/Type /OCG /Name (L101)
endobj

and adding the resource to the page "Properties" resources.
<<  /Type /Page  /Resources <<  /Properties <<  /L101 101 0 R>>  >>

--
T

i dont know what is so complicated to mark resources for pdf layers.

Then submit a PR if it's that easy.

LOL

draw.io is also closed to contributions. We follow a development process compliant with our SOC 2 Type II process. We do not have a mechanism where we can accept contributions from non-staff members.

not allow by your own policies !

I will make an exception for this case. Please submit the PR.

@terefang Yes please. You would be my/our hero!

need to first ask my company if this is allowed by contract.

nonetheless while awaiting confirmation, i searched the codebase to to look how you generate pdf.
from the package.json files it looks like you use pdf-lib from npmjs.
i also found the PDFDocument initializer, but then i am lost.
i only found the pdfToPDF method but this is not part of the pdf-lib api.
so can you point me in the right direction (and file) where pdf-export actually starts ?
my javascript is a little bit rusty, since i wrote the last js i remember in 2010.

We convert HTML to PDF on line 714

We convert HTML to PDF on line 714

that would mean it is "technically impossible" to implement layers because you are rendering thru the browser implementation.

@terefang

that would mean it is "technically impossible" to implement layers because you are rendering thru the browser implementation and not some interfacing pdf library.
unless you change the rendering, of course !
Yes, unfortunately there is no documentationat all mentioning layers in the used library:

PDF-LIB · Create and modify PDF documents in any JavaScript environment.
url:: https://Hopding.github.io/
abstract: Create and modify PDF documents in any JavaScript environment.
If you are searching for other soultions (to render), I want to mention the term Optional Content Group (OCG)used in a layer features in PDF documents to organize content.

For me it is unfortunately beyond my (current) scope to support this on the javascript side.
Thanks for picking this up.


[WebPubSub] Adding feature support

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.
Supporting new features in Web PubSub

Add "groups" parameter when generating client access URI
Add a helper class SearchFilter for generating the filter string, referencing the practice of Cognitive Search
Add more tests

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Messaging.WebPubSub

Hi @jsquire do you have any expert looking into the CI failure on "Build Test Windows2019_NET461_ProjectRef_Release" complaining about some assembly missing? This is strange and should not be related to my change because my previous commit passed successfully however the next commit containing only a simple rename failed with this CI failure.

@vicancy:  I don't see any failures.   I'd suggest raising issues around pipelines in the "Engineering System" channel on Teams so that the right folks can assist.

Thanks for the review @ArthurMa1978 ! Could you also help approve the APIview https://apiview.dev/Assemblies/Review/c02c4162593a42a7b404ca7d927e4cef?


Feedback: Model discovery and installation

Running install with an unrecognised model gives the following output:
➜  models catai install gpt4all
$ cd /usr/local/lib/node_modules/catai
Model unknown, we will download with template URL. You can also try one of thous:7B, 13B, 30B, Vicuna-7B, Vicuna-7B-Uncensored, Vicuna-13B, Stable-Vicuna-13B, Wizard-Vicuna-7B, Wizard-Vicuna-7B-Uncensored, Wizard-Vicuna-13B, OpenAssistant-30B

Outputting a list of available models is excellent but perhaps also worth adding a catai install --list or similar command?
Also observe that the output appears to make no sense ... thous:7B? And what are the 7B,13B,30B models? (edit) Ahhh, the original llama models, doh!

Yes, it is a bit confusing naming something with 7B.
This is planned to be changed in the feature.
It is a good idea to list all the available models in a formatted way, with a simple command.
Thanks for sharing this :)

No worries, and thanks for this project. Obviously there is a lot of work going on in the LLM area and having something simple that doesn't take days of reading to understand is bound to increase interest and the ability of people to experiment.
Just downloading the models is an adventure given that it is not obvious that huggingface models are just git repos, nor is it obvious exactly what formats will work. catai install is bang where the complexity should lay.


Nginx: Allow to extends templates anywhere

This allows users to create their own Nginx template in their projects and extend Drifter defaults. The changes in the default templates are required when the user extends a template that itself extends another (like PHP extends default). Without this the provisioning fails because it is not able to resolve the parent template.
This PR is a: Improvement

[x] Documentation is written
[x] Changelog was updated

#187 #153 and #179 should be updated if this gets merged.


NullPointerException on Android

BLE scan sequence is failing with a NullPointerException on Android. It seems like channel is null (device got disconnected or something).
Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'void e.a.c.a.j.c(java.lang.String, java.lang.Object)' on a null object reference
       at com.pauldemarco.flutter_blue.FlutterBluePlugin$5.run(FlutterBluePlugin.java:10)
       at android.app.Activity.runOnUiThread(Activity.java:7095)
       at com.pauldemarco.flutter_blue.FlutterBluePlugin.invokeMethodUIThread(FlutterBluePlugin.java:7)
       at com.pauldemarco.flutter_blue.FlutterBluePlugin.access$400(FlutterBluePlugin.java)
       at com.pauldemarco.flutter_blue.FlutterBluePlugin$2.onScanResult(FlutterBluePlugin.java:83)
       at android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper$1.run(BluetoothLeScanner.java:498)
       at android.os.Handler.handleCallback(Handler.java:883)
       at android.os.Handler.dispatchMessage(Handler.java:100)
       at android.os.Looper.loop(Looper.java:214)
       at android.app.ActivityThread.main(ActivityThread.java:7397)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:935)

Same here

I have this as well, only comes in released app, shows up in ANR time-to-time in Play Store.
What's confusing to me is FlutterBluePlugin.java:10. If you look at https://github.com/pauldemarco/flutter_blue/blob/master/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java those line numbers are off. However if we just follow the call stack from onScanResult maybe this is a Protocol Buffer V8/ProGuard issue? I've seen that with Firebase plugin.

I have this as well, only comes in released app, shows up in ANR time-to-time in Play Store.
What's confusing to me is FlutterBluePlugin.java:10. If you look at https://github.com/pauldemarco/flutter_blue/blob/master/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java those line numbers are off. However if we just follow the call stack from onScanResult maybe this is a Protocol Buffer V8/ProGuard issue? I've seen that with Firebase plugin.

I have been seeing this in production as well. Looking at the stack trace, it looks like it's this call:
https://github.com/pauldemarco/flutter_blue/blob/c19bb9aadde250970353d22b3cd100db8bde28ff/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java#L998
This might be happening on startup when the activity isn't been registered for some reason. The activity object is not null checked here causing the exception (though a simple null check doesn't solve the issue).
https://github.com/pauldemarco/flutter_blue/blob/c19bb9aadde250970353d22b3cd100db8bde28ff/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java#L94

I have been seeing this in production as well. Looking at the stack trace, it looks like it's this call:
https://github.com/pauldemarco/flutter_blue/blob/c19bb9aadde250970353d22b3cd100db8bde28ff/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java#L998
This might be happening on startup when the activity isn't been registered for some reason. The activity object is not null checked here causing the exception (though a simple null check doesn't solve the issue).
https://github.com/pauldemarco/flutter_blue/blob/c19bb9aadde250970353d22b3cd100db8bde28ff/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java#L149

If that's the case then the root cause is deeper and more serious. With a simple null check at L149 you'd just prolong the inevitable crash, with a null Activity sooner or later you'd fail. The question is how the activity can be null. Maybe some race condition? Or the user changes orientation while starting? I don't have too many of these crashes, but they happen still every once in a while. I could never reproduce it on my end, and I believe this could be also one of those release-only bugs.

And I created a PR. I see this crash so regularly in the wild it is embarrasing. I thought about bringing back the MainActivity.kt (which I eliminated as part of https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects) so I can override runOnUiThread. But runOnUiThread is final, so I cannot even do a pacman style catch there...

The same is here, we facing crashes in production when channel is null, while calling channel.invokeMethod(name, byteArray);.
From my check it happens, when an app goes to the background for a long time, then onDetachedFromActivity is being called and following tearDown cleaning the channel. Please merge the fix that was proposed by @MrCsabaToth .

The same is here, we facing crashes in production when channel is null, while calling channel.invokeMethod(name, byteArray);.
From my check it happens, when an app goes to the background for a long time, then onDetachedFromActivity is being called and following tearDown cleaning the channel. Please merge the fix that was proposed by @MrCsabaToth .

That's great info. I've only seen this crash in the Play Store crashes, I was never able to reproduce it. The "when an app goes to the background for a long time" reproduction works for debug version or only for release?

Seeing this same error cropping up in my Crashlytics, Android only. Any fix? Hoping it is just an extended background time bug and not something causing people big issues, thanks!

Got this issue only on prod. Is there any workaround?

Got this issue only on prod. Is there any workaround?

I'm seeing a crapton of these in production. I was never ever able to reproduce in debug, or even with release builds testing. I cannot wait to release a long waited update to my app which has my own flutter_blue fork.

Any update on this?

Here's my fork https://github.com/CsabaConsulting/flutter_blue

@MrCsabaToth Do you think your fork is still stable? Also how can I use it?

@MrCsabaToth Do you think your fork is still stable? Also how can I use it?

This is how you refer a custom git repo within your pubspec: https://github.com/TrackMyIndoorWorkout/TrackMyIndoorWorkout/blob/cadd153f20277b8b8deb78e9088b8fa745873dd4/pubspec.yaml#L22
As you see I'm currently using my own fork still, but since flutter_blue got stuck in time, my fork is also not maintained and stuck in time. As Android 12 will take more market share it'll be more important to abandon flutter_blue. Within the upcoming months I gonna migrate over to flutter_blue_plus (by boskogs) and if that does not contain the Google ProtoBuffer related code exception catches, then unfortunately (and excrutiatingly painfully) I have to custom fork yet another f-ing time and rely on a custom fork again.

Here is my struggling: https://github.com/TrackMyIndoorWorkout/TrackMyIndoorWorkout/issues/210

Would you recommend me to switch directly to flutter blue plus?  Have you already had experiences with it?  Is it stable?

Il giorno 8 mag 2022, alle ore 22:20, Csaba Toth @.***> ha scritto:

Here is my struggling: TrackMyIndoorWorkout/TrackMyIndoorWorkout#210
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Would you recommend me to switch directly to flutter blue plus? Have you already had experiences with it? Is it stable?

I didn't have time to try it yet. But I also realize I don't have much more, other than another fork. I want to also tackle multiple issues, like the permission I mentioned, or Mac related ones by switching.

As far as I know flutter_blue_plus still haven't brought in macos support.
Also I don't think it is requesting permissions on android 12 and you'd have to use something else to request platform-specific permissions, perhaps permission_handler plugin.
I'm using this fork that at least addressed permission issues https://github.com/espresso3389/flutter_blue
Though macos support is one to one with the original - hence also broken.
@MrCsabaToth if you could propose and bring your changes in PRs to flutter_blue_plus that would be great. Especially if you fixed macos issues in yours.


Support mounting on SELinux

This should allow cross to work on both non-SELinux enabled systems
and SELinux ones.
I've tested on Fedora 29 for SELinux system.
Fixes #112 that was originally reported from Fedora as well I believe.

bors: try

Heya @Dylan-DPC - it seems that the build failed but for an unrelated reason - E: Unable to locate package linux-image-4.19.0-1-powerpc64:ppc64 - could you take a look ?

i'm not sure what's causing that. Will investigate.

Thank you !

Same problem here: https://github.com/rust-embedded/cross/pull/255
Marking this PR as blocked.

Heya @Dylan-DPC - I solved the conflicts in this PR and force-pushed - it seems that Bors is stuck perhaps ? Not sure what to do next tbh. Thanks btw :)

bors r+


Advice for ledger to work from direnv using envrc

Not completely sure if this is the right place for this, but it works for me,
enabling the ledger binary to be found via direnv and shell.nix, locally.

Hmm, weird that this would be required, actually.

Without this, on opening my ledger file I get
File mode specification error: (error ‘ledger-binary-path’ (value: ledger) is not executable)

Whereas with this, all is well and flymake (I think) can find ledger. :shrug:

Thanks, I can reproduce this. What's happening here is that ledger-mode invokes the executable before any mode hooks run, and ultimately mode hooks are what cause the local environment to be established by envrc. I believe I've fixed this by instead setting ledger-mode-should-check-version.

Yes, that works, thanks!


fix: Error When using wait, Fatal driver exception occurred: Expected a value of type 'int', but got one of type 'String'

Description
when using wait shorthand action
- wait: 2000

I get this error:
 Fatal driver exception occurred: Expected a value of type 'int', but got one of type 'String'

or when I use this:
- wait:
  seconds: 2

I get
 Fatal driver exception occurred: Expected a value of type 'int', but got one of type 'Null'

looks like something is going horribly wrong when parsing integers, perhaps related to https://github.com/wolfenrain/fluttium/issues/365

As the param file is YAML, you should be careful on indentation. I had same issues, but after some tests, I figure out that we need more space indentation for wait:
- wait:
    seconds: 2

And not:
- wait:
  seconds: 2

The seconds shouldn't be at the same level as wait.
Hope this help.

@Thithip Thanks a lot! that was indeed the fix
still the issue remains when using
- wait: 2000


New files after compile

when I compile file this extension creates a new file for this class, and it does not care if have another point of view how I would like to store files. I have folders for packages and files for classes, and it just ignores my structure and creates files with the full name of the class, outside of packages folders.

Hi, daimor! :) Do you use options "cos.export.folder" and/or "cos.export.atelier" ?

All settings by default.
only ip, port,and namespace configured.

By default option "cos.export.atelier" is off and extension used "flat" name conversion logic. Please, enable it and check again.

Why it does use original file name all the time, without any options?


Update templates to match latest implementation

@miltontony ready for review :) :dancers:

@miltontony ready for review :)

not happy with this approach since it introduces 2 new templates which will make it tricky upgrading existing apps. Lets give it another go today and see if we can't make it work.

@miltontony ready for review sir :dancers: :dancers:

:+1: :cake: :bouquet:


Allow specifying a KMS Key ID for server-side encryption

@vlsi can you review when you have a sec? Thank you. :)

Thanks. I've pushed 1.8.2, enjoy


Suggestion: Multiple Kaplan Meier confidence interval options

Greenwood CI for KM estimator does not work very well with highly censored data and also has asymptotic properties, so doesn't work well for small sample sizes. After playing around with lifelines and getting many NaNs for CIs of highly censored data,  I ended up using "Beta Product Confidence Procedure for Right Censored Data" from the following paper:
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3769999/
There is an R package to calculate this (which also includes four other CIs):
https://cran.r-project.org/web/packages/bpcp/index.html
I'm a Python user so I wrapped this in rpy2. I understand that introduction of new libraries, wrappers or statistical methods may not be appealing, but just checking if there is any interest to incorporate this or any other CIs into lifelines?
Here's a sample of code that can create these CIs:
from rpy2.robjects import pandas2ri,r
import rpy2.robjects.packages as rpackages
from rpy2.robjects.vectors import StrVector
from rpy2.rinterface import RRuntimeError

utils = rpackages.importr('utils')
utils.chooseCRANmirror(ind=1);
packnames = ['bpcp']
names_to_install = [x for x in packnames if not rpackages.isinstalled(x)]
if len(names_to_install) > 0:
    utils.install_packages(StrVector(names_to_install))

pandas2ri.activate()

bpcp = rpackages.importr('bpcp')

def bpcp_survival(df,days_to_succeed = 5*365.):
    try:
        bfit = bpcp.bpcp(df['TherapyToLastDay'],df['Vital Status'],Delta=0);
    except RRuntimeError:
        return pd.Series(index=['survival','lower','upper'])
    surv_results = pandas2ri.ri2py(bpcp.StCI(bfit,days_to_succeed))
    return surv_results[['survival','lower','upper']].iloc[0]

For some comparative reason, I needed the "traditional" Greenwood's formula (which made me create a monkey patch for the _bounds method with the current code base). It would be nice if the user could choose optional methods for the confidence intervals.


Add title to desktop page

I think it would be a good idea to be able to add a title to the custom desktop page like we can in traditional desktop apps, i'm happy to open a PR for this, the only issue i can see with this is the menu items but it may be worth putting them on a grid column below?

I agree with the idea, moreover it would permit to use the DesktopPage with the sidemenu too.
I will do it this week because I'm incorporating an easy dialog system (using https://github.com/AvaloniaUtils/DialogHost.Avalonia ) in the DesktopPage, so I will have to update the documentation anyway 👍

Done in the last commit, see the update readme

sorry to be a pian, but what about the ability to set position of the title bar? for example personally i would have it centered.

That's fine, I'm really open of making the library better so that's the process
We can do a TitleAlignment property + TitleFontSize + TitleFontWeight ?

i agree :), also whilst i think about it, any perticular reason you use usercontrols for the custom controls instead of creating a control say like avalonia core does? im not 100% sure but i expect there is a reason why they do it that way.

Hi, sorry, I missed some emails 😓
Could you summarize the situation in order to help us? Thank you very much sorry again
Diego Battistini

Il giorno 19 apr 2022, alle ore 21:02, sabuto @.***> ha scritto:

i agree :), also whilst i think about it, any perticular reason you use usercontrols for the custom controls instead of creating a control say like avalonia core does? im not 100% sure but i expect there is a reason why they do it that way.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Hey, no worries :)
so for example you have CircleProgessBar here with the properties but it is a user control. and then you have the styles defined in the xaml for that user control but avalonia itself makes controls like this ProgessBar which is not a user control, and then styles like this ProgessStyle so it is contained in a style page. I am not sure of the implication but i would assume there is come benefit.

new properties :

Title
TitleHorizontalAlignment
TitleFontSize
TitleFontWeight

have been added in the new nuget package

About the UserControl/Control debate I did it just because of habits
But I don't think it makes a lot of changes anyway - from documentation :
UserControl control is a Control that represents a reusable collection of controls in a predefined layout.
UserControl actually provides very little functionality on top of ContentControl. The difference is that you will not usually create instances of the UserControl class directly; instead a new subclass of the UserControl class is usually created for each "view" to be shown by an application.

ok cool :)


Remove an unnecessary conditional

This PR removes the if chunks: conditional and dedents the conditional block, since if chunks: is always True. Removing the conditional allows the test coverage fail-under value to increase to 100%; previously it was at 99% because the conditional branch never evaluated to False, so branch coverage wasn't complete.

It's easy to confirm that the condition will always be True, even when the string being split is empty:
>>> "".split("a")
['']
>>> bool("".split("a"))
True

Great, thx again!


Add popcon

xref https://github.com/galaxyproject/galaxy/issues/1469

just remembered, sorry @erasche :/
it has been rpetty hectic lately

No worries @martenson it's been miserably hectic here too :) Sorry if I came off as annoyed :/

@erasche you didn't, cheers


Update request

Please, update instacodes-sublimetext-plugin, that plugin to work correctly in 2016. Now name of service — instaco.de. Thanks.

I wrote a letter to the author Instaco.de, he corrected errors. Now instacodes plugin works for me.
Thanks.

Ah fantastic!  I would have gotten to this issue sooner but I'm out traveling and managed to shatter my laptop screen.  Currently writing this from an old Android with a broken screen as well!  I don't travel well with electronics...
Let me know if the instaco.de updates are still broken or any other issues

@0x4445565A, when you will have the opportunity and free time, please, to change in the description of the plugin and in plugin files Instacode.es → Instaco.de. Thanks.

Made the changes and pushed (Also replaced the screen on my laptop, so cheers all around!)


UPSTREAM: 127400: apiserver/admission/webhook: construct static CEL compiler only once

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/lgtm
/approve


Looking for info on making a Windows console server work using Unity and its threading

Hey there,
I apologize in advance for the length that this question may become but I have a Windows console server application that uses things like ThreadPool, message Fibers, etc and I am trying to get it to work with Unity properly so that I can compile it as a Linux headless server instead of being tied to Windows. It uses this fiber framework and is quite complex, with a fairly large amount of interfaces and files associated with it that use things like Enqueue, IExecutor, Execute, toExecute, DispatchAdapter. Unfortunately, I am just not familiar enough with threading to be able to figure out exactly how to accomplish what I am after.
I was able to get a test working using a dispatcher I found called  UnityMainThreadDispatcher. There was a place that used the call of "this.requestFiber.Enqueue(Method);" to, of course, queue up a method to run but when I would run the server within Unity it would not do anything once it ran it, so I took the Dispatcher UMTD and just swapped it out for UnityMainThreadDispatcher.Instance().Enqueue(Method); and it worked. The only issue is, that was one single thing out of many, many places in which call Enqueue and I think that it kind of defeated the purpose of the whole fiber and enqueue system to begin with by simply sending that one particular thing out to be processed.
Because the system is rather complex I figured that using something a bit more feature-filled and comprehensive such as your UnityThreading might help, I am just unfortunately not sure where I should start or what things I should try to replace of what it currently uses with the Methods UnityTheading provides. What I am trying to figure out / find information on is, in a system that uses Execute, ThreadPool, ThreadFiber, etc, is there a specific place I can / should swap out for UnityThreading to handle anything that is sent to be Enqueued so that I am not trying to go through the code and find everything that uses the Enqueue method and changing it individually or is that actually what I should do? I could technically just do a Find / Replace on anything using the current Enqueue method, but that seems like it would not work well because of the "this.requestFiber.Enqueue" part of the call that are specific to the instance of that particular class. so I am guessing it ends up needing to be something farther down the line that needs to be replaced.
I am hoping that perhaps you might be able to give me an opinion or any info you might have? This is the very last part I need to get working in my game framework to finally be able to move on with my development, I have spent 2 months straight working on it and I am so close, yet I can't find any info on how to make this work the way I need. : (
Any assistance would be extremely appreciated!
Thanks,
-MH

Hello MostHated,

In your opinion, do you think it might not work for what I am trying to accomplish? Would using coroutine end up being more of a performance hinderance? I unfortunately do not know much about threading, so it is difficult for me to try to guess on what may work or not.

Unity's classes can work only in the main thread, and you can't start it in another thread. For this issue Unity create coroutines. If you didn't use Unity classes in threads it will be work. You can use Coroutines for this. It works IEnumerator (like foreach) and executing in the main thread. It started work after Update function. https://docs.unity3d.com/Manual/ExecutionOrder.html
But also, you can use Unity Networking for Multiplayer


Peer feedback

Here is some feedback for your dash!

move the dropdown to the right side so it is easier for the user to find
size of the titanic photo could be smaller
add a note stating that the cabin location was from Class 1
change the default deck level
change the grey colour in the bar plots
add note stating you can hover over the points for passenger information

Thank you Carlina,
Great feedback! We have made notes and will be implementing some of your suggestions, stay tuned


Implement Acquire for PgListener

I have a binary that mostly listens to notifications for a few channels, then does some work and writes some results back to Postgres when it's done. PgListener already implements Executor, so it's possible to borrow the listener's connection in order to do this for simple writes (and let this binary only use a single PG connection, since they're somewhat expensive and limited), but there's currently not a way to open a transaction via PgListener's connection. So, I implemented Acquire for it.
The test seems a little silly, I guess it's not catching many issues that the type system doesn't, but it felt reasonable to assert that it's actually possible to open up a transaction from the listener's connection.
Thanks! Feedback welcome!

Looks like someone beat you to it (#3550)

That's true! I'll just close this one out then, thanks!


modify paths for generic packaging

I would like to help getting the "deb/rpm packaging" #129 ticket move forward. Since I don't want to pollute that ticket, I am suggesting a rather huge change here. Let me briefly explain "generic packaging".

Whonix's packages (that mostly contain just scripts, config files and package dependencies) are mostly packaged using "generic packaging". It's about where files are stored, how the makefile works and distribution specific packaging tools.
In short, files in etc/... in root source folder will be installed to /etc/..., files in usr/... will be installed to /usr/... and so forth. This should make renaming, moving files around, packaging, etc. very simple. Packaging of most of Whonix's packages (list) looks very similar.
The makefile, the make-helper.bsh script are the same for any package and other files (such as licensing stuff) look very similar.
This makes packaging and maintenance real simple.
Does that make sense?

We also have a few python packages.

example python tool
example python lib

Using that method - using the canonical, standard debhelper dh_python2 way - we are able to produce Debian policy conform .deb packages, that means lintian is reporting zero warnings.

Proposal:
You'd need to move files accordingly. Maybe you want several packages? Such as tuf-shared, tuf-server, tuf-client? In that case, create these three folders in the root of TUF's source folder. (Or alternatively split into three git repositories.) Then inside the tuf-client folder, add tuf-client/etc/, tuf-client/usr/, etc. folders accordingly and move files there. Please have a look at the example repositories above to find out where to best place files.
I'd then do my creating a pull request that adds the generic packaging. Can't promise anything, but I think this should be relatively easy. We could also contact debian-mentors, they got a mailing list and irc channel helping with debian packaging questions.

@adrelanos
Did you get a chance to test stdeb?  Is stdeb and the debian build tools (as outlined in Debian's Guide for Packaging Python Libraries) enough to address issue #129?  I would think that our standard directory layout wouldn't be a problem for these tools.
The feature request outlined here is to support Whonix's "generic packaging" scheme.  Correct?
I'm just not sure if you're requesting we change our directory structure on our master branch, or to help with porting the code base to the directory structure required for Whonix.  That is, a TUF w/ "generic package" repository like the whonix-repository tool: https://github.com/Whonix/whonix-repository.

Did you get a chance to test stdeb?

Yes. Found it too complicated and confusing. Preferring debhelper dh-python2 method.

Is stdeb and the debian build tools (as outlined in Debian's Guide for Packaging Python Libraries) enough to address issue #129?

Might be. Can't be of help for stdeb.

The feature request outlined here is to support Whonix's "generic packaging" scheme. Correct?

Wipe the "Whonix" word. Unrelated to Whonix, which just happens to use "generic packaging" method. It's a suggestion to change the path for the "generic packaging" scheme that would help with packaging for any Debian (and maybe others) based distro.

directory structure used by Whonix

It's not something to suit Whonix. It's something to suit the "generic packaging" solution.

whonix-repository tool

That is totally unrelated here. It's just a tool to conditionally populate /etc/apt/sources.list.d/whonix.list or not.

Looks like no paths are being modified and we're going the dh-python2 packaging (#259 #262) route. Boldly closing this for cleanup, but feel free to reopen.


Support GitHub Enterprise

@m5rk: Hey, I think 719f4f6e2882cf850c94a5a6cf1e690df72e3192 adds proper support for GitHub Enterprise, but I have no way of testing this. If you still use this, do you mind testing it out for me?
I'm also not sure if issues/pulls for different domains will open in the same window (because it's separate requests and I didn't look carefully @ the old code).
If you don't use this anymore just disregard this. Thanks.


[SPIKE] Metric registration

Goal
Create tickets to encompass functionality for: displaying, creating, updating  and deleting bias metric configurations from Trusty AI service
Dependency issue

Mocks

Itemized goals

Create necessary tickets when mocks are available and requirements agreed.

Outcome - created tickets:

#1269
#1274
#1275

Outcome - created tickets:

#1269
#1274
#1275