Improve completion item sorting in already completed statements

Description:
Right now, LS sorts the completion items correctly for partially written statements. Ex:
public function main() returns error? {
    int val = get<cursor>
}

function getInt() returns int {
    return 1;
}

function getStr() returns string {
    return "";
}

At <cursor>, functions that return an int and int type variables will be sorted first.
But if the user has already completed the statement similar to the below case, the sorting of completion items are not correct.
public function main() returns error? {
    int val = get<cursor>Str();
}

function getInt() returns int {
    return 1;
}

function getStr() returns string {
    return "";
}

In the above example, getStr() will be the 1st while getInt() comes 2nd. As per the variable declaration statement, user expect a function/var of type int to come first.
Describe your problem(s)
See description.
Describe your solution(s)
Need to improve the LS ContextTypeResolver to derive the contextually expected type for already completed statements.
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

In this particular scenario, at the cursor point, what is the resolved completion provider? What if we try to resolve to the same provider in both the cases?

In this particular scenario, at the cursor point, what is the resolved completion provider? What if we try to resolve to the same provider in both the cases?

@nadeeshaan Here the issue is with the completion item order. In 2 cases (partial and complete statement), we get 2 different results (int and function () returns string) as the context type from the ContextTypeResolver. That's causing the difference when we sort by assignability.


[NEW]: Late Confession in Austria Crypto Trial

Date
2023-12-13
Description
After weeks of courtroom drama, the main accused in a crypto fraud trial suddenly reversed course and confessed. After first pleading innocent when the trial started in September, then confessing to some things, then retracting some of that Wednesday morning, he suddenly admtted Wednesday just before lunch that the EXW (exchange wallet) and EXW-Token (ECR20 on Ethereum) were always meant to be a fraudulent ponzi scheme from inception.
The prosecution knows of 40.000 victims and 17,6 mio Euro in money stolen. A witness, the main accused's ex-girlfriend and mother of his child, testified that actually 80 to 120 mio Euro were stolen by the EWX group.
Links
https://www.heise.de/news/Ploetzliches-Gestaendnis-Kaerntner-Krypto-Token-war-von-Anfang-an-Betrug-9574145.html
https://kaernten.orf.at/stories/3236486/
Here's a video about the opening of their office in Klagenfurt, Austria, some 4 years ago
https://www.youtube.com/watch?v=mzRRVtaAdso
Reportedly, such events were used to raise money. Bags full of cash are said to have been handed over at such events.
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
Daniel AJ Sokolov
Personal link
https://social.heise.de/@newstik

There's more reporting in Austrian media about this.
https://www.sn.at/wirtschaft/oesterreich/exw-prozess-hauptangeklagter-150071365
https://www.derstandard.de/story/3000000199475/exw-prozess-hauptangeklagter-bekennt-sich-vollumf228nglich-schuldig
https://kaernten.orf.at/stories/3236486/

There is some interesting collected background information and what kind of person was behind this here
https://behindmlm.com/companies/exw-wallet-indictments-arrests-criminal-trial-in-austria/

Herzog was arrested last year after attending a party in “Upper Austria”. Upon being arrested, Herzog purportedly feigned being autistic.

https://kurier.at/chronik/oesterreich/betrug-exw-klagenfurt-bandion-ortner-kryptowaehrung/402608954

[Der Kurier:] According to prosecutor Czedik-Eysenberg, he acted as an autistic person. But he managed to reset his phone to factory settings in time.

I'm rolling my eyes at the many times crypto scammers have abused autism to either signal "look at me, I'm a math genius, invest with me" or try to use it as "but I didn't understand I was hurting people".

Thanks so much for all the detail and translation! https://web3isgoinggreat.com/?id=exw-fraud


How to access special folders?

Hi, I'm trying to access my app folder (using the permission onedrive.appfolder). According to the documentation, this is a special folder. How can I access such folders using the Python SDK? I have tried calling:
client.drive.special.get()

But it throws an exception:
Traceback (most recent call last):
  File "unodrive.py", line 24, in <module>
    client.drive.special.get()
  File "/usr/local/lib/python3.6/dist-packages/onedrivesdk/request/special_collection.py", line 118, in get
    return self.request().get()
  File "/usr/local/lib/python3.6/dist-packages/onedrivesdk/request/special_collection.py", line 39, in get
    return self._page_from_response(collection_response)
  File "/usr/local/lib/python3.6/dist-packages/onedrivesdk/collection_base.py", line 47, in _page_from_response
    return response.collection_page
  File "/usr/local/lib/python3.6/dist-packages/onedrivesdk/request/special_collection.py", line 145, in collection_page
    self._collection_page = SpecialCollectionPage(self._prop_dict["value"])
KeyError: 'value'

To debug this issue I modified http_provider.py to print HTTP requests and reponses and got the following output:
https://api.onedrive.com/v1.0/drive/special
{"message":"No HTTP resource was found that matches the request URI 'https://api.onedrive.com/v1.0/drives('me')/special'."}

I'm not sure what's wrong here. Any ideas or suggestions?

I ended up sending the request manually as a workaround:
import requests, json
endpoint = "https://api.onedrive.com/v1.0/drives/me/special/approot"
headers = {"Authorization": "Bearer " + access_token}
response = requests.get(endpoint, headers=headers).json()
appfolder_id = response["id"]

@olav-st what do you get if you do client.drives.get() instead of drive.get()? I only get one drive, personal, but try exploring with autocomplete.

Thank you for reaching out and for your patience. This SDK is being officially deprecated. See #209 for more information


Getting started with Kubernetes Clusters

Article checklist
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #2143

@zolomohan, I would like to review this if you haven't started yet.

yeah @lalith1403. Sure, go ahead.

Getting started with Kubernetes Clusters #2207.pdf
@munubi254
Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.

@hectorkambow  Thank you, I'll rectify the wording as soon as possible though some flags raised were for Kubernetes commands which one cannot change. How about that?

@hectorkambow I've done some changes. please check

@munubi254 There are a lot of grammatical errors. Ping me once you have rectified all the errors. There are various tools, such as Grammarly, Hemmingway App, that help with this. Refer to other articles for more information and the standard expected of authors.

@lalith1403 I'm really sorry for that. I've rectified a bit I had forgotten on the first paragraph..the other errors flagged are for Kubernetes commands. Please check.

@lalith1403 @hectorkambow

@lalith1403 @hectorkambow

Will review it today. Thank you.

@lalith1403 I've tried making some fixes. Please check

@lalith1403 Am closng this for conformity with the new repo. My new PR Is at #2360

Thank you v much for your patience as we worked through our Repo re-structuring process.
The purpose behind this restructure - was to better allow us to handle more volume and allow students to publish more articles and get more involved with the community (with open-source contribution and suggestions).
With most of the changes coming to completion - we will be sure to get your article published asap.
Thank you again - and we are looking forward to more amazing articles.

@lalith1403

great work @munubi254
awesome intro article - easy to follow along - looking forward to more in depth ones as well 🚀
cc @lalith1403


iwd: 0.7 -> 0.8

https://git.kernel.org/pub/scm/network/wireless/iwd.git/tree/ChangeLog
Things done

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

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

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

cc @Mic92

Success on aarch64-linux (full log)
Attempted: iwd
Partial log (click to expand)
wrapping `/nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/bin/connect-network'...
shrinking RPATHs of ELF executables and libraries in /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8
shrinking /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/libexec/iwd
shrinking /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/bin/iwmon
shrinking /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/bin/iwctl
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/lib  /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/libexec  /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8/bin
patching script interpreter paths in /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8
checking for references to /build in /nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8...
/nix/store/zffndcyd3qkh1b7qhpbbnff4rmqwrdw2-iwd-0.8

Success on x86_64-linux (full log)
Attempted: iwd
Partial log (click to expand)
wrapping `/nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/bin/connect-network'...
shrinking RPATHs of ELF executables and libraries in /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8
shrinking /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/libexec/iwd
shrinking /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/bin/iwctl
shrinking /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/bin/iwmon
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/lib  /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/libexec  /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8/bin
patching script interpreter paths in /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8
checking for references to /build in /nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8...
/nix/store/zf9gn18yq6p73xk2z3f6n0sqglzvw70p-iwd-0.8


Add the PUBLIC keyword to the target_link_libraries call

Recent changes broke the target_link_libraries call inside FindCUDA.cmake. CMake requires the use of the keywords in the target_link_libraries call either nowhere, or everywhere.

@cyrush @white238 can I get a review on this?

I'll defer to @white238 -- he and i had a conversation related to this.

No problem -- @white238 and I actually chatted about this yesterday and he pointed out the fix.


Update test-and-publish.yml

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Idea] component testing for Vite

According to<EMAIL_ADDRESS>the dev server:

will load support files
will load the current spec from the url
will start the test when both files are done loading

The setup is the following for vite dev server in cypress/plugins/index.js
import { startDevServer } from '@cypress/vite-dev-server'

module.exports = (on, config) => {
...
  if (config.testingType === 'component') {
    on('dev-server:start', (options) => {
      return startDevServer({
        options,
        viteConfig: {
          configFile: path.resolve(__dirname, '..', '..', 'vite.config.ts'),
          clearScreen: false,
        },
      });
    });
  }
...
}

As Vite config can be overridden there is potential to add plugin for the cucumber transform, see https://vitejs.dev/guide/api-plugin.html#transforming-custom-file-types , similar to the compilation step in esbuild plugin: https://github.com/badeball/cypress-cucumber-preprocessor/blob/master/esbuild.ts .
Current behavior
Component testing does not work with this pre-processor, as Cypress uses dev servers to serve the tests, not pre-processors, in Component mode.
Desired behavior
A Vite plugin exist for this cypress cucumber project, to run in component testing.
Will give it a try in next few days, and see if I can validate the concept.

If you can provide a minimal example that contains Vite, a component, a feature file and step definition(s), then I might be able to show you how to integrate a loader.

Sure, will do so tomorrow.

Here is an example repo, run yarn install then yarn cy:ct to launch Cypress component runner.
Test is added in cypress/integration. Have not tried yet to create the Vite plugin, so if you get a chance to give it a go, let me know.
cucumber.zip

Thanks for the example. I took a good look at it and was able to get it to compile correctly. However, the difference in how component tests are bundled compared to regular tests (or rather that they're not, but each module loaded separately by the browser), unfortunately makes the preprocessor currently incompatible with component testing.
For the curious: some design is required to support "Run all specs" while maintaining the isolation between tests and their step definitions, IE. despite "running all specs", specs still shouldn't share all step definitions. Supporting this test mode has been a major struggle and it continues to be so for plugin developers in general. See https://github.com/cypress-io/cypress/issues/1586.
The design required entails calling require(..) for each file containing step definitions, in a particular context. However, vite-plugin-commonjs, necessary to make this compile, will lift all calls to require(..) to the top, thus breaking this context.

In order to support "Run all specs", component testing with Vite will remain unavailable, at least as long as the above-mentioned issue goes unsolved. This is true specifically for Vite, as Vite serves source code over native ESM (ref).
Component testing using a different bundler might still work, but I have not investigated it.

You know what, I’m not quite ready to abandon this. I need to sleep on it though.

Thx for looking into it, I also stumbled on the ESM stuff with require warning.
In addition, I am not sure about the upcoming changes in Cypress 10 where they merge component and standard runners.

Oh, where can I read about the changes you’re referring to?

Getting documentation on upcoming releases is a challenge with Cypress.
But looking at PRs from time to time, I see that they are preparing a unified runner, as a lot of code was duplicated when they created the component runner. This includes a lot of re-factoring on the backend side as well.

I'm facing similar problem did you manage to find a workaround?

This turns out to be why. It can be bypassed with a small change to Cypress though, we’ll see how that goes.

Thanks @badeball . Let's see if this can be fixed vite side :)

Support for component tests using Vite as bundler has been fixed and released as v7.1.0.

Hey @badeball with the configuration listed here: https://github.com/badeball/cypress-cucumber-preprocessor/blob/3fcba98d03a72cde3f6a682a7dfac0cddb22bfed/examples/ct-vite-ts/cypress.config.ts
I am unable to make it work... in fact, I think it cannot work as it is a problem on vite side not pre-processing non-JS files... See https://github.com/vitejs/vite/pull/3828#issuecomment-1540529716 and https://github.com/vitejs/vite/pull/9981
Even if you create a "dumb" custom rollup plugin like this:
const customPlugin = () => {
  return {
    name: 'cucumber-to-cypress',
    transform(code, id) {
      console.log('##############');
      console.log(id);
      return null;
    },
  };
};

module.exports = defineConfig({
  component: {
    devServer(devServerConfig) {
      return devServer({
        ...devServerConfig,
        framework: 'react',
        bundler: 'vite',
        viteConfig: {
          plugins: [react(), createRollupPlugin(devServerConfig.cypressConfig), viteCommonjs(), customPlugin()],
        },
      });
    },

it will only console.log on requests to ts/js files (or similar)
See:
##############
/Users/PRIVATE_PATH/cypress/support/commands.ts
GET /__cypress/src/cypress/support/component.ts 304 1.386 ms - -

and on the .feature files it's just printing:
GET<EMAIL_ADDRESS>304 1.668 ms - -

As explained here, this requires a patch to @cypress/vite-dev-server and the example illustrates how that can be achieved.

Thanks for the support on this @badeball !


feat: update builtin policy

Codecov Report

Merging #7 (8feb560) into master (5283001) will increase coverage by 0.73%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   86.53%   87.26%   +0.73%
==========================================
  Files          11       11
  Lines         683      683
==========================================
+ Hits          591      596       +5
+ Misses         92       87       -5

Impacted Files
Coverage Δ

src/judger/interactive.c
74.71% <100.00%> (ø)

src/judger/traditional.c
80.00% <100.00%> (ø)

src/lib/builtin_hook.c
98.86% <0.00%> (+5.68%)
:arrow_up:

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 5283001...8feb560. Read the comment docs.


Incompatible mod on 1.20.1

Mod Name
replay mod
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/replay-mod-fabric-and-forge
Minecraft version
1.20.1 (LTS)
Describe the bug
When i boot Create mod with Replay mod it gives me an error.
Steps to reproduce
1.Install Create Mod On Forge
2.Install Replay Mod On Fabric
3.Boot the game
Logs
https://gist.github.com/Alcryv/b82e4f566e8961dcd361aea41612bdee
Additional context
No response

nvm fixed it by changing version of replay mod


Add documentation for advanced search

Add the documentation about the new sorting and filtering features on https://docs.microsoft.com/en-us/nuget/consume-packages/finding-and-choosing-packages

Once this is done, we should also update the .NET Core documentation. See: https://github.com/nuget/nugetgallery/issues/8074


ansible-zabbix-agent fails on Unzip on Windows with Ansible 2.9.x

Describe the bug

I have the following issue with Ansible 2.9.1 and zabbix-agent 2.1.0 (Ansible Galaxy):
failure: Error unzipping 'c:\zabbix\\zabbix_agents-4.0.0-win-amd64.zip' to 'c:\zabbix\'!. Method: System.IO.Compression.ZipFile, Exception: Exception calling "Open" with "3" argument(s): "End of Central Directory record could not be found."
{
"changed": false,
"dest": "c:\\zabbix\\",
"msg": "Error unzipping 'c:\\zabbix\\\\zabbix_agents-4.0.0-win-amd64.zip' to 'c:\\zabbix\\'!. Method: System.IO.Compression.ZipFile, Exception: Exception calling \"Open\" with \"3\" argument(s): \"End of Central Directory record could not be found.\"",
"removed": false,
"src": "c:\\zabbix\\\\zabbix_agents-4.0.0-win-amd64.zip"
}

This happens on this task:
Description[w19-m32-zab466] converge: dj-wasabi.zabbix-agent : windows | unzip file src={{ zabbix win install dir }}\{{ zabbix win package }}, dest={{ zabbix win install dir }}, creates={{ zabbix win exe path }}
Test class/tmp/zabbix/roles/dj-wasabi.zabbix-agent/tasks/Windows.yml:114

Installation method/version

Ansible Version
ansible 2.9.1
  config file = None
  configured module search path = ['/home/molecule/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.8/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 3.8.0 (default, Nov 23 2019, 05:36:56) [GCC 8.3.0]

Targetted hosts
Concerns the following OS(es):

Windows

Expected behavior

I have no issues on Ansible 2.8. So something must have changed, but I didn't see anything in the Porting Guide for 2.9 in that regard.

I'll try  to investigate this a bit more, as I know the Windows support is best effort! :)
I created the ticket for tracking purposes.

Ok, I tracked down the issue to the fact that the URL I'm using to download the package contains a redirect (302, Moved permanently).
win_get_url in Ansible 2.9 has a bunch of new parameters introduced.
"follow_redirects" is one of them. "safe" is the default option, meaning it should follow redirects on GET requests.
Now, it does look like the win_get_url operation, is only downloading the HTTP response  (372 bytes) instead of following the redirect to download.
So it means it's not following the redirect to download the target file.
I'll investigate a bit more.

So using "follow_redirects: all" on the download task is the approach.
Now, I need to check if it's retro-compatible with Ansible 2.8/2.7 :)

I opened a bug report with Ansible: https://github.com/ansible/ansible/issues/65556

I'll close this, as this is clearly an Ansible issue that's going to be fixed in 2.9.3.
Might want to make it clear in the README of the role that 2.9.{0,1,2} are not compatible. :)

Hi @rnsc
I've reopened this issue so I know that I need to update the documentation. Thank you for providing comments and a solution/workaround. 👍

Documented it.


[FEATURE] Remove gas fee estimation feature

Detailed description
I may receive an error message similar to the following:
cannot estimate gas; transaction may fail or may require manual gas limit (error={"message":"execution reverted","code":-32000}, method="call", transaction={"to":"0x1A2B49e10013C40AAC9b6f9e785837bfd329e5e0","data":"0x56891412","accessList":null}, code=UNPREDICTABLE_GAS_LIMIT, version=providers/5.5.0)

I suggest removing the gas fee estimation feature itself.
Context
Due to multi-network support, the gas fee estimation feature often fails. This is often the case with many early-stage L2s such as Arbitrum.
Is it correct to display "Estimation failed" on the UI when the gas fee estimation fails? I think it only makes users uneasy.
Therefore, I think it is better to remove the gas fee estimation feature.
Possible implementation
No response
Additional information
No response
Code of Conduct

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

Assuming that this is actually connected, after that error message was shown the whole page stops working - and for me personally that error message is shown on every navigation while being logged in, multiple times after each other.
screenshot of the pool overview
As I have not found out how to look at the L2 pools without being logged in yet, I cannot use any L2-related features on the page at all - it only works while I'm logged out.


B-Spline Interpolation order 0

B-Spline interpolation supports orders from 0 to 5, however only 1 to 5 are present in the interpolator enum here : https://github.com/SimpleITK/SimpleITK/blob/36c0ed9512bb5141c6e7d85bf1bb68d31191c3a5/Code/Common/include/sitkInterpolator.h#L30
How to go about using B-spline interpolation with order 0?

Hello @kmalhotra30,
Order zero in the underlying ITK implementation is equivalent to nearest neighbor (see code here). In SimpleITK just use sitk.NearestNeighbor instead of a BSpline of order 0.

Alright, thanks for the help.

Sorry to re-open the issue @zivy , in general spline order 0 is not the same as nearest neighbor ? or in ITK the implementation is same?

Hello @kmalhotra30,
Generally speaking, you are correct, spline order zero is not the same as nearest neighbor, but in ITK it appears to be that order zero is nearest neighbor, at least according to the documentation. From a practical standpoint, what matters is the implementation, as the theory is on paper and does not necessarily match what was implemented.

@zivy I see,  thanks for your response.


Provide integration test toolkit

The current NodeJS server implementation is useful, but hard to use by external packages. Maybe we should provide a separate toolkit (maybe written in Go, replacing the current implementation).

@sagikazarmark I think you can use this: https://github.com/jcchavezs/httptest-php


Log package installation history

I just advised an ipm user on how to track a history of installations of their package through <Invoke> elements, but really this is something the package manager could/should just do itself.
History should include at least:

Version
Timestamp
$Username

Possibly also (note - these are a minor wrinkle from a transaction handling perspective; if install rolls back we still need the history):

Success (%Status)
Committed (e.g., 0 in dev mode if installation fails)

History should be logged for both "load" and "install" (including as a dependency).

Clarifying: this is a local history on the instance where the packages are installed.

Got you. Such a functionality is already implemented in zpmhub: https://openexchange.intersystems.com/package/zpmhub

Would be great to expose this via zpm "history" or something like that, and also to provide a public API for programmatic access.

@isc-kiyer @isc-rkuszews @isc-eneil - could you chime in if you think anything is missing from the spec? This is very important for v1 / inclusion in IRIS.

@isc-tleavitt I don't know what Committed is but the rest makes sense to me. I like the idea of using the IRIS Audit log since we can then leverage its existing UI and capability to turn auditing on/off for specific Audit events (for example, a user may only care about failed installs).
We can still have zpm "history" to wrap around the IRIS audit log but have that as the common source of info.


Update user task properties

Hi @timvandervoornmx ,
For what version is this PR?

It’s for 9.21 😊
From: Maria Shaposhnikova @.>
Date: Friday, 16 December 2022 at 16:58
To: mendix/docs @.>
Cc: Tim van der Voorn @.>, Mention @.>
Subject: Re: [mendix/docs] Update user task properties (PR #5436)
Hi @timvandervoornmxhttps://github.com/timvandervoornmx ,
For what version is this PR?
—
Reply to this email directly, view it on GitHubhttps://github.com/mendix/docs/pull/5436#issuecomment-1355132665, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXYMCDRIOHDFIFF6EYMRCVLWNSGRVANCNFSM6AAAAAATBDF6OA.
You are receiving this because you were mentioned.Message ID: @.***>

@timvandervoornmx , we do not use dark mode for our screenshots. Can you please replace the updated screenshot with the light mode one?

@MariaShaposhnikova Done!


Request for pretrained weight

Hi,
I want to request a pre-trained weight if you don't mind. I just want to have quick results on my dataset.

Hi, this repo is a piece of work I did at school, and I didn't make a copy of the pre-trained weights before graduating, so I'm not able to provide them. Having said so, it is easy and cheap to train a network from scratch using this repo. If you have any further questions, please feel free to open an issue.


$baddog not appearing in osmosis frontend

$baddog was taken off front end about an hour ago, right when $bad was added. Concerning need response. thank you.
### Tasks

Resolved by now


extra tests for the tie break merge option

Hmm, what could that be? Is the comparison always true?

Can probably close this one now, fused in #422


The sample template is clipped

[Enter feedback here]
The ARM template sample doesn't include all the parts mentioned, but it clips in the middle. At the same time when you upgrade this, you could add the Bicep version of the sample as well. as it is available in the quickstart-template on which this is based on. Thanks!

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

ID: a6bb7895-266b-fb1b-e733-1e3133b2c0e9
Version Independent ID: 5d25c9c9-8674-3831-64c0-700556b172f0
Content: Quickstart: Create an Azure Route Server by using an Azure Resource Manager template (ARM template)
Content Source: articles/route-server/quickstart-configure-template.md
Service: route-server
GitHub Login: @halkazwini
Microsoft Alias: halkazwini

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@sympaJN
Thank you for your feedback! I've just updated the template, it'll be reflected in the public doc in the next hours.
#please-close


fix(select): allow value to be set with reactive forms in on init

Closes #1355

It's very useful feature.


Add Webhook documentation and sample JSON bodies

Hey,
this is supposed to solve: #1016
Scope of changes

added some sample JSON bodies (returned by v0.0.8) of the webhooks.
added the newly added "VISIBILITY-CHANGE" webhook to the table.
add short docs on how to test webhooks locally (+ ref to ngrok)
minimal md cleanup

While writing this I found this "behavior" #1302. For now I just documented it, as is. If it's decided to be changed accordingly, I'd be more than happy to do this as well.
Cheers,
Ruffy

Thank you for putting this together.  I think it'll be a big help for people looking to start with the API. :1st_place_medal:


Memory leaks (GPUImageMovie processImage:)

Guys can anyone help me solve those memory leaks.. I dont understand why there are many leaks in GPUImage library..

I was having an issue with memory leakage while using GPUImageChromaKeyBlendFilter with GPUImageMovie. What seemed to fix it was calling endProcessing on the GPUImageMovie object when the view disappears or just whenever I was done with it.


Using uninitialized variables

Hi.
Is this expected behavior? If I append an uninitialized variable to print, it shows a random value. If I try to print the variable on its own, it gives me an error:
> print("hello")
hello
> print("hello "..x)
hello<PHONE_NUMBER>854775807
> print(x)
syntax_error: stdin:1: 'x' undeclared (first use in this function)

It seems like the system should throw an error for the appended print, too.

Indeed, that's an unwanted side effect.

Wow, this fix made me find a hard to find bug in my Matter code. Thanks !

Fixed in https://github.com/berry-lang/berry/pull/415 (waiting for merge)


Spin up a multi container app with a database

This adds our real app to the deployment.

This PR is on hold till we get some other higher priority items done


[entropy_src/rtl] Cleanup timing enable network

As discussed in PR #14660 there are many places where enable-clearing RNG data or masking internal control signals can cause verification challenges. This is a primary motivation for keeping delayed copies of the enable signals, to apply delayed enable signals at the appropriate points in the pipeline.
Such fine-tooth timing management of the enable/disable signals is not good for maintainability.  This commit aims to simplify the network of enable signals in the entropy_src core, while fixing a number of regression failures.
Data pathways that had previously been blocked the instance the module was disabled have now been allowed to flow, letting data propagate until it stalls at the SHA3 conditioner or gets dropped at the bypass FIFO.
At the same time, this commit simplifies and documents the timing of the module enable signals, both the es_enable level signal as well as the module_enable_pulse.
The module_enable_pulse is now one cycle long, and generated from an explicit flop. (As opposed to using a prim_mubi4_sync),  AsyncOn=1. The es_enable level signal starts in the cycle right after the module_enable_pulse, but falls immediately when the module_enable register is set to MuBi4False.
Since both the enable pulse, and the enable level signal are derived from the same Mubi field, they are fanned-out in MuBi representation.
All of the prim_mubi_sync's now use AsyncOn(0).
Signed-off-by: Martin Lueker-Boden<EMAIL_ADDRESS>

this lgtm assuming i understood your last point correctly.
Just as a note the fpga is showing a failure on an entropy_src_ast test. I imagine this one can be reproduced in sims as well.

Yes I saw that.   I took some time to switch to a last few toggle coverage points, but it this is fixed now.


Fix highlighter color

Right now the yellow on white highlight color is not cute nor accessible, gotta fix this across light and dark mode.

When you refer to cute, do you mean you would like a different color or a tint/shade of yellow that corresponded with each theme?

Hahaha sorry for the lack of clarity @NeffCodes, I wasn't expecting many folks to see this 😆
So right now, when you highlight text by itself, it looks like this:

But when you have a highlighted section using Obsidian highlights , it looks like this:

This looks good in light mode, but in dark mode it looks like this:

No problem! Do you mind if I take a look at it? I took the liberty of quickly looking up some color contrast ratios, but since I don't know the full scope of what you want to achieve, what level do you want the accessibility ratio to be?
For example, color yellow #756b13 would provide a fairly strong contrast against the text, but not against the background.
However, based on the contrast between your red color and the white text in dark mode, you could go lighter to have a similar ratio with yellow #877B16 to stand out more against the background. Or, if you would rather have a different color, you go with a blue #4779c5.

Hey @NeffCodes that #756b13 looks great! I think it's okay if the text has the strongest contrast, I think that's most important. I appreciate you looking at this, thank you!

No problem!


fix error type in ReadNode when file was not found

The method ReadNode in the ocis storage didn't return the error type NotFound when a file was not found.

@ishank011 this should help with https://github.com/owncloud/ocis/pull/755#issuecomment-721087510 in owncloud/ocis
Can you review please?

Thanks @ishank011. :love_you_gesture:


Ratio information does not appear in the HELM string

If I draw the following:

The ratio of 2:3 does not appear in the HELM string.

Fixed in 2017-11-13 update

The HELM has changed, but it does not include the ratio numbers.

I don't think there should be a bond connecting the two if there is a ratio.

The ratio information is better recorded against the bond since it is possible to have multiple components connected and the approach above does not work well with more than two fragments.
The UI should enforce the rule that ratios cannot be added to the group if the groups are connected by a bond.

2017-12-09 update: ratio will be removed if the group is connected by a bond

Ratio information is removed if you change the ratio after drawing the bond, but drawing the bond does not remove existing ratio information.

Ironically the ratio information now does appear in the HELM string, so the original issue is fixed. Perhaps we should discuss tomorrow?

decision from today's meeting:
When drawing a bond connecting the two group, their ratio should be cleared.

Ratio number should be displayed as "ratio: 3"

Fixed in 2017-12-13 update

Looks great and the warning message and change of text are good. Just one thing, only one of the ratios is cleared.

Sorry.  there is typo.  Fixed in 2017-12-14 update.

Confirmed fixed.


Build warning due to vulnerable transitive System.Text.Json dependency

When using Silk.NET.Core as a dependency and compiling a project with the .NET 9 SDK, warnings regarding a vulnerability in the System.Text.Json package are shown:
> Project.csproj: Warning NU1903 : Package 'System.Text.Json' 8.0.0 has a known high severity vulnerability, https://github.com/advisories/GHSA-8g4q-xg66-9fp4

This is very annoying when multiple projects are built as the warning will be shown for each where Silk.NET.Core is a (transitive) dependency.
The cause for this seems to be that Microsoft.Extensions.DependencyModel (8.0.0), which is referenced from Silk.NET.Core, depends on System.Text.Json ≥ 8.0.0. As per the security advisory, the first patched version is 8.0.5, so bumping Microsoft.Extensions.DependencyModel to at least 8.0.2 should fix this warning.

PRs welcome.


ProteoRE tool 'get_unique_peptide_srm_method'

Another ProteoRE tool ; this one is 'get_unique_peptide_srm_method'
Thanks for the review !
Still 16 tools to come ...

@bgruening we are all set on our side for this PR.
If you could please review it ... Thank you very much for all the help !

Human_SRM_atlas_2016-04.csv is super big. Does it really need to be shipped with the tool, or should the admin download it on demand? Will this file change over time? Can we compress it?

Thanks Björn, this file does not change over time. I gziped it and lade modifications in the wrapper (loc file) and Python script to take gzip csv file as input.


Update java agent versions

Changes
Update java agent versions.
With the old versions, there're problems with NaN metrics from Kafka/Fraud detection service. I noticed they get fixed with this version update.
otel-col  | Metric #85
otel-col  | Descriptor:
otel-col  |      -> Name: kafka.consumer.partition_assigned_latency_max
otel-col  |      -> Description: The max time taken for a partition-assigned rebalance listener callback
otel-col  |      -> Unit:
otel-col  |      -> DataType: Gauge
otel-col  | NumberDataPoints #0
otel-col  | Data point attributes:
otel-col  |      -> client-id: Str(consumer-frauddetectionservice-1)
otel-col  | StartTimestamp: 2023-09-18 12:53:47.837352255 +0000 UTC
otel-col  | Timestamp: 2023-09-18 12:54:47.726008797 +0000 UTC
otel-col  | Value: NaN

It appears the new java agent didn't, after all, fix the NaN metrics. That's why a new issue was created for it:
https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/9498


python310Packages.hahomematic: 2023.2.7 -> 2023.2.8

Diff: https://github.com/danielperna84/hahomematic/compare/refs/tags/2023.2.7...2023.2.8
Changelog: https://github.com/danielperna84/hahomematic/releases/tag/2023.2.8
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.hahomematic
python311Packages.hahomematic


Mark v1.19.x as stable when we are happy with v1.19

Because v1.19 is a new milestone release and a new k8s minor release -- we have held back as marking this as stable in the channel server. Normally, we mark releases as stable within one week.
Because stable to us means ready for production we will stand by, let v1.19.x bake for a bit and then when we are happy specify a v1.19.x release as stable.

Hi, there! New to the project. How long do releases usually stay as latest until they reach stable? Is there a formal release cycle?

Where can I track what issues are open and blocking this version from hitting stable?

We are mostly keeping a close eye on any etcd issues that get entered and need to wait until we get a couple other project releases out in about a week from today before we can have QA review these. You can search for k3s issues with the area/etcd label which will show a few that we may look into at our earliest ability.
We are also keeping an eye out in other channels such as the rancher users slack for anything.

Sounds good, thanks for the info!

https://github.com/rancher/k3s/pull/2487

Closing. v1.19.3+k3s2 is now stable!
I want to notify that etcd is not quite yet out of experimental -- nor is SELinux support.
So - we're marking it as stable because we feel good about K3s v1.19 but etcd and SELinux are still experimental in v1.19.3+k3s2.
Here's the plan (subject to change)

v1.19.4+k3s2 release (slated for late November) moved etcd to certified (take it out of experimental)
v1.19.4+k3s1 release (slated for 11/11) SELinux moves to certified (take it out of experimental)


Simplified react-player DEMO

Hello:
Since I am new to react, I am not quite sure how to go about editing the code so I can have a simplified version to show only the player's window and play/pause fullscreen buttons. I tried to edit the app.js from the demo website, but the code is huge and is all merged into one line.
I can install the code on my own AWS EC2 server. Just need a little advice.
Any pointers to help me get started?
Thanks

Hi @inglesuniversal,
Editing the app.js bundle is a bad idea. That file is just a minified version of App.js that serves the demo page.
This jsFiddle example will hopefully get you started. It renders a simple ReactPlayer instance with play/pause/fullscreen buttons. Notice under External Resources on the left that it is pulling in the React development files, the ReactPlayer dist file, and the screenfull util for the fullscreen function.
If you wanted a standalone HTML version to use, you could paste the code from the fiddle into React's HTML file example, like so:
<!DOCTYPE html>

    <meta charset="UTF-8" />
    <title>ReactPlayer Example</title>

Note that both the fiddle and the HTML example use Babel to transpile the code inline. This is fine for demo purposes, but for a production app you should compile/bundle your files using webpack or similar.


ADD Scroll To Top Button

Description
When I go through the latest version of this website .  I found that when user goes to bottom of the page then there is no any scroll button to return to the top of the website .
Screenshots

Do you want to work on this issue?
Yes
Additional information
No response

Thank you for raising. This has been raised multiple times and closed for the reasons mentioned here: https://github.com/EddieHubCommunity/BioDrop/issues/8717
In future we would ask that you check both open and closed issues before raising a new issue, to avoid duplication.


Instructions for setting up with rails not complete?

Running rails generate que:install just installs the migration, right?
Isn't there something that needs to do Que.connection = ActiveRecord in a rails app that uses que?

The railtie does that automatically if ActiveRecord is defined.

Doesn't Que.mode = :async need to be set somewhere?

The railtie does that too :smile:
I actually have a half-finished gem called que-rails that basically is the extracted railtie, but with specs that make sure that the rake task to create the migration works, and the worker pool boots properly in server mode, and all that. Basically those Rails integration things that aren't specced. Another thing I just haven't had the time to finish. If anyone wants to take it over I'd be happy to push it - I dislike having the Rails-specific stuff in the gem proper.

Ah, found the problem. I'm using rails-api, and rails-api doesn't define Rails::Server.
Not sure if the railtie should use a different constant or method to figure out if it can set Que.mode = :async?


width/height option lost in 1.0.0-alpha.1 ?

hey,
how to set width & height of the rendered canvas?
in 1.0.0-alpha.1 these options show no effect... the canvas is always rendered with the size of the window
are there new options to define width/height of the rendered canvas?
thx
mike

They are currently missing (it's still WIP, and some other features are also missing), but will be part of the 1.0.0 release.

hey,
all right… but i can say already that this version is working very well.

offscreen rendering is working
colors, shadows are working

thanks for your effort
mike

Am 27.09.2017 um 06:40 schrieb Niklas von Hertzen<EMAIL_ADDRESS>They are currently missing (it's still WIP, and some other features are also missing), but will be part of the 1.0.0 release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/niklasvh/html2canvas/issues/1230#issuecomment-332406940, or mute the thread https://github.com/notifications/unsubscribe-auth/AbxP1YzuNIqvZl9b8qSlCH9z3A-vPNJUks5smdHFgaJpZM4Pke_P.


How to get underlying Sequelize / Config ?

Hey !
Great module.. been looking for this from a while.  Just wondering how to get underlying sequelize module, or how to configure additional config like pool etc.

Never mind i figured it out. ! very well written library this one. 👍

@vikrantpogula thank you very much :)


Improve window handling (release and close)

Hi @felixrieseberg
Eventually, this PR might take care of the following issues: #239 and #251
I already commented it here: https://github.com/TryGhost/Ghost-Desktop/issues/239#issuecomment-322464549
We had similar issues in one of our Electron apps, and using these two code blocks helped to resolve the problem of proper window close and release across Windows, Linux and macOS.
It seems that the behaviour of Ghost Desktop on Windows has been improved already and the Electron process(es) are terminated as expected. Eventually, you shall ignore this PR but consider to close the referenced issues.
Please let me know what you think about that.
Thanks and regards, JoKi

You know, nevermind the event handlers. I think this is a good change and if I find any issues, I'll just fix 'em as I find them.
Thank you very much!


Execution order based on variables

Description
Replaces the current calculation of the model execution order in init and update - based on the input configuration - with another one based on the information on which variables each model initialises and which ones they require.
Using this method has identified also an inconsistency in the variables used by the soil model, which I guess it is a good thing!
The depends section of the example config files have been removed, and I had not spotted it being mentioned anywhere else, including the documentation, but let me know if I missed anything.
In principle, this is the last bit of the implementation of the variables system.
Fixes #389
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Relevant documentation reviewed and updated

Codecov Report
Attention: Patch coverage is 96.87500% with 1 line in your changes missing coverage. Please review.

Project coverage is 94.96%. Comparing base (f3c6f53) to head (316a367).

Files with missing lines
Patch %
Lines

virtual_ecosystem/core/variables.py
96.66%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #548      +/-   ##
===========================================
- Coverage    94.98%   94.96%   -0.02%
===========================================
  Files           74       74
  Lines         4145     4152       +7
===========================================
+ Hits          3937     3943       +6
- Misses         208      209       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I'm in no rush, but if this is Ok, it will be best to merge it. The longer a PR is open, the more challenges it brings merging it for ongoing work. It is approved but I would prefer if you merge it, so it is done when it is a good time (eg. to avoid conflicts).


Need modification in "Makefile" and "miopen.hpp"

As per latest release 1.6.4 of rocm driver , library name of rocblas and hipblas needs to be changed from ""-lhipblas-hcc , -lrocblas-hcc"  to ""-lhipblas -lrocblas"
Hence, Makefile needs modification as below
LD_FLAGS=-L$(ROCM_PATH)/lib -L$(ROCM_PATH)/opencl/lib/x86_64 -lMIOpen -lOpenCL -lmiopengemm -lhipblas -lrocblas
Besides this, 1 minor modification requires in miopen.hpp as below
#include <hipblas/hipblas.h> to #include <hipblas.h>
Only after above 2 modification, we can able to build the miopen-benchmark tests on rocm release 1.6.4

can you send this as a pull request?

I havae created pull request for below:
Besides this, 1 minor modification requires in miopen.hpp as below
#include <hipblas/hipblas.h> to #include <hipblas.h>
But , somehow pull request was not coming for Makefile changes and it went directly, not sure why.

Can you please merge the changes asap.

Done.

Thanks.


[RESOLVED] High ICMP ping response time

Problem started at 15:15:16 on 2023.05.19
Problem name: High ICMP ping response time
Host: ST-FG
Severity: Warning
Operational data: Value: 947ms
Original problem ID: 20967
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23135&eventid=20967

Problem has been resolved in 8m 0s at 15:23:16 on 2023.05.19
Problem name: High ICMP ping response time
Host: ST-FG
Severity: Warning
Original problem ID: 20967


Document supported languages

Description
Document supported languages (where applicable) for the DevX frameworks.

@aliciakrebs Need re-approval 🙏


VSCode extension: Add a change I missed for environment listing

Missed something in https://github.com/Azure/azure-dev/pull/2789.

VSCode Extension Installation Instructions

Download the extension at https://azuresdkreleasepreview.blob.core.windows.net/azd/vscode/pr/2800/azure-dev-0.8.0-alpha.1.vsix
Extract the extension from the compressed file
In vscode
a. Open "Extensions" (Ctrl+Shift+X)
b. Click the ...\ menu at top of Extensions sidebar
c. Click "Install from VSIX"
d. Select location of downloaded file


add vscode to .gitignore

Fixes #527 added .vscode/ to .gitignore.

I signed it!

@nworden please review


Add Erigon Execution Client

Changes:

Add erigon Execution client to stack

Types of changes
What types of changes does your code introduce?

[x] New feature (non-breaking change which adds functionality)

Testing

[x] Yes
In case you checked yes, did you write tests?

[ ] Yes

[x] No

Comments about testing , should you have some (optional)
Run smoke tests

Codecov Report
Base: 83.14% // Head: 83.14% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (892727a) compared to base (caa2fe3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #156      +/-   ##
===========================================
- Coverage    83.14%   83.14%   -0.01%
===========================================
  Files           37       38       +1
  Lines         2409     2426      +17
===========================================
+ Hits          2003     2017      +14
- Misses         321      323       +2
- Partials        85       86       +1

Flag
Coverage Δ

unittests
83.14% <ø> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/pkg/keystores/keystore_external_utils.go
83.00% <0.00%> (-0.17%)
:arrow_down:

configs/init.go
100.00% <0.00%> (ø)

internal/utils/version.go
83.33% <0.00%> (ø)

configs/logging.go
76.92% <0.00%> (ø)

internal/pkg/generate/generate_scripts.go
96.58% <0.00%> (+0.01%)
:arrow_up:

cli/cli.go
68.75% <0.00%> (+0.49%)
:arrow_up:

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.

@AntiD2ta just solved what you requested, please review it again!


Don't reindex objects in Postgres

Description
No need to log and load objects with Postgres
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI, plus will do a bounce test

/retest


Acquire WDL for posting to GATK protected, if possible

Need to get from @dlivitz
Put into scripts/wdl/
Should coincide with #383

Making changes to the original to break it out into separate tasks (and take advantage of parallelization) opportunities.


Install requires libzstd-dev

I'm running Piaware image 1090&978 with adsbexchange standard (only 1090).  Following your script attempting to add 978 feed, compilation error on install.
When running the install script on Pi 3B+, image piaware 5.10.60-v7+ on buster:
In file included from readsb.h:101,
                 from interactive.c:54:
threadpool.h:36:14: fatal error: zstd.h: No such file or directory
     #include <zstd.h>
              ^~~~~~~~
compilation terminated.

Added libzstd-dev to the required $packages in install.sh and installation succeeded.

Yeah, this package isn't really needed anymore, the normal feed scripts should send 978 as well now.
But will still fix.

Interesting, I will check that out.  I recently noticed that my 978 traffic was not reflected on ADSBx, hence looked for a simple addition.  I did see that some scripts combine 1090 & 978 into dump1090 and that's not what I want.  Since I'm running 1090-fa and 978-fa, I thought I needed a separate task.
Still tinkering a bit, as I want ADSBx to pick up location from gpsd, not a file.  Thank you!

just update the main feed scripts.
for MLAT changing location is annoying.
so yeah i'm not sure if i ever will get around to implementing gpsd support.

for MLAT changing location is annoying.

Yeah, I've disabled the mlat service for this reason and it also doesn't apply to my use case.
I see your combined solution at ADSBexchange/feedclient@a88d549; I'll work on integrating into my setup.  Seems when I set this up a few months ago, I followed your advice on reddit for feeding gpsd into readsb/tar1090.
I agree, probably not worth adding gpsd support for normal users who set up a feeder. Mine is an edge case and I have the know-how to tinker with bash scripts and service files.
I'm primarily using this feeder to sample if additional coverage is beneficial in various remote or low altitude locations, like airport surface and over water where more sites would pick up traffic lower to the ground.  While I'm gathering this data for myself / clients, it is valuable to share it with the community as often I capture traffic that is otherwise uncollected.
Thank you for your work.


Train Custom Dataset

Hey, I am coming too. infer_mdt.py works now, but when I trained my dataset, I had a new problem as below.

Do you guys know why it has a segmentation error? Please help me!

When I ran the MDTv1, I met the same problem.

My GPU is 3090 24GB.

It seems like a environment issue. Can you try to reinstall your pytorch environment and also the cuda and cudnn?

Can I add you to WeChat communication?

Can I add you to WeChat communication?

give me your email address or wechat account

<EMAIL_ADDRESS>Thanks!!

Can I add you to WeChat communication?

give me your email address or wechat account

<EMAIL_ADDRESS>

Can I add you to WeChat communication?

give me your email address or wechat account

<EMAIL_ADDRESS>
已发送

Hey, I want to train on my custom dataset. But its unconditional data set, just the images, no class id no annotations. Can I still train? How can I do that?
@gasvn @xv994 @liwenyang-911 @mavenlin @P2333 Can you please help me out here if you know?

Hey, I want to train on my custom dataset. But its unconditional data set, just the images, no class id no annotations. Can I still train? How can I do that?
@gasvn @xv994 @liwenyang-911 @mavenlin @P2333 Can you please help me out here if you know?

I'm sorry for replying late.
This model is a supervised deep learning. Maybe using conditional dataset is not a right choice.


OSB logo policy

What is our reuse policy regarding the OSB logo? I guess it is covered by the Apache license, but should we be more precise here? For example, we could add text like this:

Everyone who supports the Open Service Broker API is encouraged to download and use the logo without cost or restriction. Downloading and/or using the Open Service Broker logo implies acceptance of the following conditions. It is expected that any person or entity using the Open Service Broker logo does so in an appropriate fashion, the logo is not modified, the design colors and aspect are retained and space around the logo is sufficient so as to prevent the logo from being construed as part of another graphic element.
The Open Service Broker API logo is not intended to imply certification or any official compliance status.

@chipchilders what does the CFF (or CF) do for stuff like this?

The TM policy is here > https://www.cloudfoundry.org/trademark/
The OSBAPI reference to that is here > https://www.openservicebrokerapi.org/governance
Specifically >
Licensing and Trademarks
The licensing and copyright of the API specification and any software produced within the PMC will follow the CFF Intellectual Property Policy, including:
All contributions to the Open Service Broker API and related projects require the individual contributing to be covered by a CLA.
All work of the Open Service Broker PMC will be licensed using the Apache Software License version 2.
Labeling of all artifacts as “Copyright (c) YYYY CloudFoundry.org Foundation, Inc. All Rights Reserved.”
The use of the CFF trademarks and word marks is governed by CFF’s Trademark Usage Guidelines.
Implementations (consumer or provider) of the Open Service Broker API are encouraged to document which version(s) of the Open Service Broker API they support. References to the API itself should be referred to as the “Open Service Broker API”.

Have a look and let me know if there are any thoughts for improvement or clarity!

@fmui does @chipchilders's response answer your question?

Technically, everything is there. But I know from other standard working groups that people want/need an extra statement about the logo. Can they put it on their websites or in publications? Can it be used for marketing? Is there some kind of certification process to use the logo? The "Licensing and Trademarks" section talks about "API specification and any software". Does this cover the logo? How do I label the logo with a copyright statement?
This is all about clarity.
Additionally, at some point somebody will ask for his-res versions of the logo...

@avade on that note, is there a high res version of the logo you can add to the repo?

@fmui do you have any specific suggestions for what we should change?
@mattmcneeney @duglin if the PMC wants to have us change the policy to support a specific goal, happy to help with that if the goal can be articulated!
@christopherclark Adding you to this thread... can you please work with Melissa L to get a high-res OSBAPI logo to @mattmcneeney so that he can add to the repo?

@chipchilders I have sent Matt the logo.

Logo added in #463. Thanks @christopherclark !


Add vitest init like storybook init

Clear and concise description of the problem
There are so many different ways to set it up that it can be confusing.
Suggested solution
Storybook has show one solution.
This has auto generation feature of .storybook/* and src/stories/*.
And then, auto detect TS or JS, and auto edit package.json.
Auto generate feature of vitest.config.ts like pnpm dlx vitest@latest init.
It would be convenient if it were there.
Alternative
If this is not within the scope of what vitest should do,
or if vitest already has this functionality,
please close this issue.
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

What would this package actually do?
As far as I know, @storybook/builder-vite was a separate package that was used to integrate Vite into Storybook. Nowadays it's no longer needed as it's built-in into Storybook. Using this logic I have no idea what @vitest/builder-vite would do. Integrate Vite into Vitest? 😕

@AriPerkkio Maybe I was tired when I post this issue. I was seeing this command pnpm dlx storybook@latest init , auto generate .storybook folder and src/stories. I will rewrite Issue.

It looks like Jest has jest --init https://jestjs.io/docs/cli#--init and it's recently moved to create-jest package https://github.com/jestjs/jest/pull/14453
This essentially just ask 5 questions https://github.com/jestjs/jest/blob/7e33cd77ccab163ea525f5c7b45c3c48a3458630/packages/create-jest/src/questions.ts#L10-L50 and I'm not so sure if it's practically useful.
I think having the examples https://vitest.dev/guide/#examples https://github.com/vitest-dev/vitest/blob/main/examples/README.md is easier to maintain and practically more useful, so maybe we just need to tweak documentations so that new users can discover it easily.
Btw, I just found that create-vitest package is already taken by https://github.com/dolanmiu/create-vitest/ But it doesn't look like this one generates config file vitest.config.ts.

As a team, we decided that it would require too much maintenance on our side to support this. We believe that frameworks themselves should integrate testing scaffolding since it is very dependent on the codebase.
If you just need a simple example of a javascript application, you can already use existing examples: https://vitest.dev/guide/#examples


Fixes ignored Hugo versions

Looks like old Netlify environment paths.
Not sure if this will fix the issue when it is one click deployed, but the HUGO versions are not set using your environment paths.
I bumped my version to 0.29 and it worked just fine also.
Reference

thanks @talves, still catching up in gitter.


No fusion generated using demo fastq files

No fusion detected with the demo data
genefuse -r hg19.fasta -f genes/druggable.hg19.csv -1 R1.fq -2 R2.fq -h report.html > result

I have updated the demo dataset, which contains a valid EML4-ALK fusion with ultra-high frequency.
Please download it again from: http://opengene.org/dataset.html

Hi Shifu,
I downloaded the new demo data and tried again with no fusion generated, the same issue. Additionally, I tried the gen_fusion_file.jl file and got a lot errors, please see attachments. Would you please check the version that you use is the same as that on the github. Many Thanks.
error_Julia.txt
TRFPgenes.txt

Could you let me know how many lines do R1.fq and R2.fq have? They should be 15000000 lines for each file.
For the second issue. Which genes do you want to include? I can generate the CSV file for you. Julia language is evolving so fast, so the package may encounter compatible problems.

Hi Sifu,
Yes, R1 and R2 each has 15,000,000 lines. I ran fusion analysis for your R1 and R2, and there are ALK-EML4 fusion and others in the R1/R2 fastq files; that is no problem for the R1 and R2 fastq file. Is the version of GeneFuse that you use different from the one you posted?
I work on illumina RNA Fusion Panel, attached is the gene list of the Panel, I need hg19 for the CSV file. It is greatly appreciated for doing that. Additionally, please let me know if you need test (positive)  fastq files of that panel.
TRFPgenes.txt


csim error

I run the command:    ./make-hw.sh lfcW1A2 pynqZ1-Z2 h
I got below error. I am running it on linux centos machine with vivado 2020.2
INFO: [SIM 211-1] CSim done with 0 errors.
INFO: [SIM 211-3] *************** CSIM finish ***************
ERROR: [HLS 200-644] The 'csim_design -compiler' hidden command is not supported.
INFO: [HLS 200-111] Finished Command csim_design CPU user time: 10.22 seconds. CPU system time: 0.67 seconds. Elapsed time: 9.42 seconds; current allocated memory: 66.645 MB.
command 'csim_design' returned error code
while executing
"csim_design -argv "$directory_params $test_image 10 $expected_result" -compiler clang"
(file "/root/BNN-PYNQ/bnn/src/network/hls-syn.tcl" line 79)
invoked from within
"source /root/BNN-PYNQ/bnn/src/network/hls-syn.tcl"
("uplevel" body line 1)
invoked from within
"uplevel #0 [list source $arg] "

Can you try by removing the -compiler clang from line 79 of hls-syn.tcl file?

I run by removing the "--compiler clang" option from line 79.  I get below output :
INFO: [SIM 211-2] *************** CSIM start ***************
INFO: [SIM 211-4] CSIM will launch GCC as the compiler.
Compiling ../../../../../../../library/host/rawhls-offload.cpp in debug mode
Compiling ../../../../../../../library/host/foldedmv-offload.cpp in debug mode
Compiling ../../../../../../lfcW1A2/sw/main_python.cpp in debug mode
Compiling ../../../../../../lfcW1A2/hw/top.cpp in debug mode
Generating csim.exe
Up to there its fine. Then it has generated below errors:
ERROR: [HLS 214-134] in function 'WidthAdjustedInputStream<64u, 64u, 13u>::WidthAdjustedInputStream(hls::stream<ap_uint<64>, 0>&, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:735:85)
ERROR: [HLS 214-134] in function 'WidthAdjustedOutputStream<64u, 64u, 32u>::WidthAdjustedOutputStream(hls::stream<ap_uint<64>, 0>&, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:768:7)
ERROR: [HLS 214-134] in function 'WidthAdjustedInputStream<64u, 64u, 32u>::WidthAdjustedInputStream(hls::stream<ap_uint<64>, 0>&, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:735:85)
ERROR: [HLS 214-134] in function 'WidthAdjustedOutputStream<128u, 128u, 16u>::WidthAdjustedOutputStream(hls::stream<ap_uint<128>, 0>&, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:768:7)
ERROR: [HLS 214-134] in function 'WidthAdjustedInputStream<128u, 128u, 16u>::WidthAdjustedInputStream(hls::stream<ap_uint<128>, 0>&, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:735:85)
ERROR: [HLS 214-134] Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:752:0)
ERROR: [HLS 214-134] in function 'WidthAdjustedOutputStream<16u, 64u, 4u>::WidthAdjustedOutputStream(hls::stream<ap_uint<64>, 0>&, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/streamtools.h:752:87)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:108:60)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:109:59)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:108:60)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:109:59)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:108:60)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:109:59)
ERROR: [HLS 214-134] in function 'DoCompute(ap_uint<64>, ap_uint<64>, unsigned int)': Pointer to pointer is not supported (/root/BNN-PYNQ/bnn/src/library/finn-hlslib/fclayer.h:109:59)
ERROR: [HLS 214-135] Syn check fail!
INFO: [HLS 200-111] Finished Command csynth_design CPU user time: 11.68 seconds. CPU system time: 0.8 seconds. Elapsed time: 14.53 seconds; current allocated memory: 90.104 MB.
Pre-synthesis failed.
while executing
"source /root/BNN-PYNQ/bnn/src/network/hls-syn.tcl"
("uplevel" body line 1)
invoked from within
"uplevel #0 [list source $arg] "
INFO: [HLS 200-112] Total CPU user time: 22.2 seconds. Total CPU system time: 1.85 seconds. Total elapsed time: 24.83 seconds; peak allocated memory: 89.753 MB.
INFO: [Common 17-206] Exiting vitis_hls at Tue Apr 13 06:54:51 2021...

Have you modified the script to execute vitis_hls instead of vivado_hls?

Yes, I made changes at two places in make-hw.sh..
Line 59  :   PATH_TO_VIVADO_HLS=$(which vitis_hls)
Line 145:   vitis_hls -f $HLS_SCRIPT -tclargs $NETWORK-$PLATFORM $HLS_SRC_DIR $PARAMS $TEST_INPUT $TEST_RESULT $PLATFORM_PART $TARGET_CLOCK

I would strongly suggest moving back to vivado_hls, given all the flow has been validated with that tool (so installing 2020.1 or earlier versions)
If you need to use vitis_hls, you should change the branch of the finn-hlslib under bnn/src/library/ to vitis_hls.


recomb synonyms?

Shouldn't
decreased mitotic recombination
decreased DNA recombination during vegetative growth
They are currently child/parent....but gene conversion and DNA repair via homologous recombination cab both be types of mitotic recombination right? (maybe I'm missing something)

No - in GO gene conversion and recombinational repair are directly is_a DNA recombination, not under mitotic recombination.

ok, I still don't understand what the difference is.
Is there one?

Well, I'm certainly no expert myself! I think the differences may be bigger in other species, especially multicellular euks, than in pombe. GO has a term for meiotic gene conversion, and I have no idea whether it would be applicable to fission yeasts (GO's also got some other types of gene conversion that I know don't apply to pombe).

Hmm I think they can as DNA repair via homologous recombination is active during meiosis. I think holliday junction formation is the main pathway  for repair but SDSA (which can/does result in gene conversions?) is also active.
This is an area where I feel I have a basic overall understanding but the details are a bit wooly.

In light of our non-expertise and uncertainty, I'll leave the terms separate for now. That feels like the way to play it safe.


put BotInfo inside Agent

similar to agent.policy we can use the Agent class to store optional extensions to agent behavior. For instance, they can be assigned BotInfo only if needed, otherwise it can be None.

put it in its own file in elfpy/bots in #546


Why not to put exceptions from the data layer into domain layer?

Why not to put exceptions from the data layer into domain layer and make them as a part of business logic? In that case the presentation layer will be completely separated from the data layer: now the ErrorMessageFactory class uses NetworkConnectionException and UserNotFoundException from the data layer.

Because those exceptions belong there. An exception like, for example, "CacheKeyMissingException" should reside in the same layer that it is used, ie: data layer. The exception "ButtonTextEmptyException" is obviously related to ui, so it belongs in the presentation layer. Keep exceptions where they belong. Exceptions are not part of any "logic", nor should be used as a control flow. Also, because the way this project is set up, you need one "entry point" to start the app and know about all the code, and in this case, this is the presentation layer.
I suspect your question pops up because "Exception Handling" is more of a cross-cutting concern, as it something that touches all the layers, but is not exclusively part of one layer.

So each layer has specific tasks related to "Exception Handling".

The presentation layer makes sure the end user is properly notified (via a dialog or something) and cleans up any UI and state problems.
the business layer cares about the business side of errors. So there we make sure we don't show any sensitive info to the user, maybe roll back the action that caused the error and returning to a correct state of the app.
the data layer is all about problems with data. This means shielding the rest of the app of exceptions that your SQL db might throw, all different types of network errors, data objects that are not available, and more.

So, let's imagine an app where i can order a puppy with the press of a button. I press the button, but i'm on a train driving through a tunnel. I'll press the button, Presentation notifies Business of said press. Business logs "Trikke wants a puppy" via Data in a file. Business call Data to make a request to the API and order a puppy. I'm in a tunnel, so the network request fails miserably after a timeout. Data catches this and notifies Business with a general "ApiNotAvailableException". Business notices and removes the log "Trikke wants a puppy". Business notifies Presentation that an unrecoverable error has happened. Presentation shows me a nice messages stating "We couldn't order you a puppy because the app is experiencing network troubles".
And now we've seen a round trip of an action and each layer is doing it's own part in handeling an error in this action.

In your example the "ApiNotAvailableException" will be converted to "We couldn't order you a puppy because the app is experiencing network troubles" message in the presentation layer? (in that case "ApiNotAvailableException" is a property of data layer, but the presentation layer has to know about that exception)

Please read up on my other comments on another question about layering.  It's a bit of a hard concept, because layering in this project's setup != layers in Clean Architecture. Parts of both the data and presentation layer of this project's setup exist on the same layer in Clean Architecture, so there are no dependency issues.
@android10 has set the project up like this due to a number of advantages, and it is both easier to explain the concept of Clean Architecture and enforce the rules. But i could make a repo with only one project module and have everything in the same namespace and still have a correct architecture in accordance to Clean Architecture. It would just be harder to enforce the rules, i would have more of a mental challenge to respect the boundaries.

@Trikke, could you help me, please, to place the things in the right way in some specific scenario? :-)

Trikke wants to download a puppy
Business calls Data to make a request.
Data makes request and notices, that the Puppy with some Id is already in the database.
How to provide an exception to the Presentation and which exception. Because on one hand, there is an "PrimaryKeyException" from database framework, which will be thrown if we will try to add the puppy into database. On the other hand we can define our custom "PuppyAlreadyExist" exception in the data, and throw it before we cache the puppy in the database(simply call subscrier.onError if the puppy was already cached).

Like i said, the "Exception Handling" part of the data layer consist of shielding the app from any problems of external sources (SQL, File storage, Network, ...) which all each have their own set of exceptions and/or problems.
This would mean that indeed, you would create a "PuppyAlreadyExistException" which the data layer throws whenever this scenario happens. Which could be "PrimaryKeyException" in an DB scenario, or "AlreadyInMemoryException" if you would happen to use a MemCache. Maybe you could be using both, first MemCache and then DB, so you catch both exceptions as a "PuppyAlreadyExistException" because they are both the same scenario.
So the presentation layer gets the general "PuppyAlreadyExistException", but not some specific "PrimaryKeyException".

@Trikke, thank you for your help!


Support disks up to 2 TB for AWS

For VMs in AWS,
AWS supports up to 2TB Volume size on OS disk but Cloudbreak only allows to create 1TB volume size for all cloud providers. Also Cloudbreak supports up to 12 disks maximum.
This is not sufficient for future needs so please consider supporting more disk and volume size.
Please consider supporting up to 24 disks or more with 2-TB of capacity each.
Thanks

+1

Hi there,
As we expect to have more than 1TB data too, if we initiate the cluster with a 1TB storage volume and then expend it to our needs from the AWS console, is there an issue with that ? Of course, it can't apply for autoscaling nodes only for "static" ones.
It's the only workaround I see.
Thanks,
Nicolas

Hi,
Cloudbreak release 1.2.1 already supports more than 12 disks and more than 1T disks from the UI for ephemeral volumes, like for a d2.8xlarge instance with 24 attached 2T disks.
For EBS volumes, more than 12 disks that are larger than 1T is supported from the cloudbreak shell, but not from the UI. You can start cloudbreak-shell with cbd util cloudbreak-shell and create a template from there, then use the template from the UI.
Also please note that EBS magnetic volumes cannot be larger than 1T, it's a limitation by AWS.
Marton

Hi @martonsereg
Ok, understood.
Will have a look into cb-shell then :)
Thanks !
Nicolas


Store registry data on Persistent Volume

Storing docker registry's data on PV as discussed in https://github.com/fabric8io/fabric8/issues/6688#issuecomment-270776017

This is great - thanks @novakg!  Just the one comment about the storage request size to address and we can merge this.


boundary: 0.3.0 -> 0.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for boundary is: "Enables identity-based access management for dynamic infrastructure"
meta.homepage for boundary is: "https://boundaryproject.io/"
meta.changelog for boundary is: "https://github.com/hashicorp/boundary/blob/v0.4.0/CHANGELOG.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/dxl6n8r1s3s8hmwb0hc8h9rr6q23n481-boundary-0.4.0/bin/boundary passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 0.4.0 with grep in /nix/store/dxl6n8r1s3s8hmwb0hc8h9rr6q23n481-boundary-0.4.0

found 0.4.0 in filename of file in /nix/store/dxl6n8r1s3s8hmwb0hc8h9rr6q23n481-boundary-0.4.0

directory tree listing: https://gist.github.com/d835896063546a337f85a68ef145362b

du listing: https://gist.github.com/9dd99bcd2b72a9e60925574c6b7a3fb7

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
boundary

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dxl6n8r1s3s8hmwb0hc8h9rr6q23n481-boundary-0.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A boundary https://github.com/r-ryantm/nixpkgs/archive/f6f1aa41908a5a70eaabc256b5f05dcde0145c77.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dxl6n8r1s3s8hmwb0hc8h9rr6q23n481-boundary-0.4.0
ls -la /nix/store/dxl6n8r1s3s8hmwb0hc8h9rr6q23n481-boundary-0.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

boundary

Maintainer pings
cc @06kellyjac for testing.

Duplicate of #128957


S3CrtClient::GetObject() return no error when fails to write data to a stream

Describe the bug
To get an object you need to set up ResponseStreamFactory which return a stream for output. However this stream is not verified.
It could fails opening file, it could fails to write data, and there is no way how to test it.
see:
    request.SetResponseStreamFactory([&]() {
        return new Aws::FStream("/dev/full", std::ios_base::out | std::ios_base::binary);
    });

    auto outcome = s3Client->GetObject(request); // here is stream created and also destroyed
    // no way how to test if stream si OK
    // the outcome must be error

Expected Behavior
if write fails the outcome is error
Current Behavior
if write fails the outcome is success
Reproduction Steps
On Linux could be simulated using full device.
# echo "Hello world" > /dev/full
-bash: echo: write error: No space left on device

{
    auto executor = Aws::MakeShared<Aws::Utils::Threading::PooledThreadExecutor>("s3-executor", 4);
    auto s3Client = Aws::MakeShared<Aws::S3Crt::S3CrtClient>(
        "s3-client",
        Aws::Auth::AWSCredentials{accessKey, secretKey},
        [&]() {
            Aws::S3Crt::ClientConfiguration configClient;
            configClient.endpointOverride = endpoint;
            configClient.executor = executor;
            configClient.enableHostPrefixInjection = false;
            configClient.scheme = boost::starts_with(endpoint, "https") ? Aws::Http::Scheme::HTTPS
                                                                        : Aws::Http::Scheme::HTTP;
            return configClient;
        }(),
        Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy::Never,
        false /* useVirtualAddressing */,
        Aws::S3Crt::US_EAST_1_REGIONAL_ENDPOINT_OPTION::NOT_SET);

    Aws::S3Crt::Model::GetObjectRequest request;
    request.WithBucket(bucketName).WithKey(objectPath);

    request.SetResponseStreamFactory([&]() {
        auto streamBody = std::make_unique<Aws::FStream>(
            "/dev/full", std::ios_base::out | std::ios_base::binary);
        if (!streamBody->is_open()) { // opening will be successful
            throw std::runtime_error{"Failed to open /dev/full"};
        }
        if (streamBody->fail()) { // also ok until any attempt to write
            throw std::runtime_error{"Stream is fail"};
        }
        return streamBody.release();
    });

    auto outcome = s3Client->GetObject(request);

    if (outcome.IsSuccess()) {
       throw std::runtime_error{"Test fail"};
    }
}

Possible Solution
No response
Additional Information/Context
No response
AWS CPP SDK version used
1.9.363
Compiler and Version used
gcc (Debian 10.2.1-6) 10.2.1 20210110
Operating System and version
Linux

Closing the issue. The created stream is moved into outcome, so it can be tested, however still a bit tricky.
    if (!outcome.IsSuccess()) {
      // error
    }

    // if filesize is less then buffer then stream will be good
    // normally would be flush into disk when outcome is destroyed
    outcome.GetResult().GetBody() << std::flush;

    if (!outcome.GetResult().GetBody()) {
        // error
    }


Windows - Feature Idea: "Blink" Windows Taskbar icon on new messages

In skype, when a message is received and it doesn't have the focus, the icon on the taskbar will "blink" until the user goes to the app. I think the same behavior should be implemented (and configurable, so some people could enable it while others disable it)

The better way is to show a tray icon on the bar, and show the number of unread new messages on the tray icon when new messages arrived. That will help a lot when we set the OS X Dock to auto-hide. Hope to see this new feature in the next version.

I have understood this is issue for Windows.
@Johnich, could you make a new issue to split topic? (yours is for osx)
btw, please indicate platform to understand the point of topic. It's very helpful because I have to think about all platforms(win, osx, linux). :(

I had raised it with Windows in mind as I don't know if other platforms support that behavior on the icons in the taskbar. If they do, and it's easy, can't see why not implement it in all platforms. If it's only possible to implement it in Windows, then it's a Windows only request :)

Got it. I will new an issue.

Yes, we recently switched to mattermost, 10% of users complain that without blinking they can't notice new messages.

Currently on Windows 7 this blinking notification stuff isn't working, as I understand this right: https://github.com/electron/electron/blob/master/docs/tutorial/desktop-environment-integration.md
So, that I only has win7 on my computer, I can't really help out here :disappointed:
Otherwise we should wait til electron add a full support for blink notifications.

Seems that this type of notification doesn't work on windows. Maybe electron isn't so far yet?

I had thought browserWindow.flashFrame() might realize this, however I have never tested this function yet.

Tried it in a quick test. You're right, it works. I will make a Pull Request to implement this.

Jep, works but maybe something we would want a setting for?

I think it's the common way to blink the icon in the taskbar when a notification show up. Every chat messenger do that.

Well slack isn't doing it.

@MetalCar
for reference, the slack settings. afaik this is the default there:

Have some problems with the "Idle"-Setting. Is there a possibility to get how long the user is idle? Or should I build this for myself? I tried to build it for myself, using the blur-Event on the BrowserWindow. But it is fired twice :confused:

Let's just start with Never and Always, we can always expend the settings in the future.

Sure, it seems a good way

Is merged in master, should probably be closed.

On Windows 10, there is an option in settings to "Flash window when receiving a notification". The taskbar icon flashes but only 4 times (4 seconds), then it goes back to normal, and there is no way to know that you have a missed message. Can we please add an option to "Flash continuously until at least one message is read"? I love Slack, but I hate that I keep missing messages due to focusing on other work, or being away from the computer.


alertlog changes

I am not sure if you will accept this PR as it introduces some changes which you may not want but here it is anyway

can now parse GoldenGate logs for OGG-123 type errors
can now parse multiple log files
normalised paths to files passed as options to the binary
removed "description" field from the exported metric
removed "ignore" field as well and instead errors listed to be ignored in the config file are actually ignored and not exported
renamed ignoreora option to ignore (as it wont be always ORA but OGG as well)
added ogg option to config file

Great changes you have made.  In principle i have no problems with this pr, but i need the description and ignore lables in my Grafana dashboards.
In my opinion it is better to make it configurable in the config file.
Please add options in the file section like "description: true/false" and "showignorederrors: true/false"

@floyd871 are you going to share the grafana json for the exporter within the repo?

Put some Grafana Dashbords in the "grafana" folder.

\o/ kudos @floyd871


Update CONTRIBUTING.md

@clebsonf Please merge this PR and label with "hacktoberfest-accepted" label

@fineanmol thanks for contributing.


[BUG] CUDA out-of-memory when using DISK-OFFLOAD, but CPU-OFFLOAD is fine

CUDA out-of-memory when using DISK-OFFLOAD, but CPU-OFFLOAD is fine
command:
~/DeepSpeedExamples$ deepspeed ./inference/huggingface/zero_inference/run_model.py --model facebook/opt-6.7b --batch-size 4 --prompt-len 512 --gen-len 32 --disk-offload --quant_bits 4
it seems that there's omthing wrong when deepspeed uses pin-mem in gpu
after change "return tensor.pin_memory()" to "return tensor", it finishes inference successfully
[2024-10-22 20:35:12,903] [INFO] [comm.py:652:init_distributed] cdb=None
[2024-10-22 20:35:12,903] [INFO] [comm.py:683:init_distributed] Initializing TorchBackend in DeepSpeed with backend nccl
load model
[2024-10-22 20:36:53,218] [INFO] [config.py:733:init] Config mesh_device None world_size = 1
Installed CUDA version 12.2 does not match the version torch was compiled with 12.1 but since the APIs are compatible, accepting this combination
Traceback (most recent call last):
File "/home/xiaoguo/DeepSpeedExamples/./inference/huggingface/zero_inference/run_model.py", line 383, in
run_generation(
File "/home/xiaoguo/DeepSpeedExamples/./inference/huggingface/zero_inference/run_model.py", line 224, in run_generation
model = get_ds_model(
File "/home/xiaoguo/DeepSpeedExamples/./inference/huggingface/zero_inference/run_model.py", line 148, in get_ds_model
model = OPTForCausalLM.from_pretrained(
File "/home/xiaoguo/anaconda3/envs/awq/lib/python3.10/site-packages/transformers/modeling_utils.py", line 3451, in from_pretrained
init_contexts = [deepspeed.zero.Init(config_dict_or_path=deepspeed_config())] + init_contexts
File "/home/xiaoguo/anaconda3/envs/awq/lib/python3.10/site-packages/deepspeed/runtime/zero/partition_parameters.py", line 1021, in init
self.param_swapper = param_swapper or AsyncPartitionedParameterSwapper(_ds_config, self.dtype)
File "/home/xiaoguo/anaconda3/envs/awq/lib/python3.10/site-packages/deepspeed/runtime/swap_tensor/partitioned_param_swapper.py", line 44, in init
self._configure_aio(ds_config)
File "/home/xiaoguo/anaconda3/envs/awq/lib/python3.10/site-packages/deepspeed/runtime/swap_tensor/partitioned_param_swapper.py", line 126, in _configure_aio
self.buffers = get_accelerator().pin_memory(torch.empty(int(self.aligned_elements_per_buffer *
File "/home/xiaoguo/anaconda3/envs/awq/lib/python3.10/site-packages/deepspeed/accelerator/cuda_accelerator.py", line 293, in pin_memory
return tensor.pin_memory()
RuntimeError: CUDA error: out of memory
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
[2024-10-22 20:36:56,877] [INFO] [launch.py:319:sigkill_handler] Killing subprocess 80369
[2024-10-22 20:36:56,878] [ERROR] [launch.py:325:sigkill_handler] ['/home/xiaoguo/anaconda3/envs/awq/bin/python', '-u', './inference/huggingface/zero_inference/run_model.py', '--local_rank=0', '--model', 'facebook/opt-6.7b', '--batch-size', '4', '--prompt-len', '512', '--gen-len', '32', '--disk-offload', '--quant_bits', '4'] exits with return code = 1

after change "return tensor.pin_memory()" to "return tensor", it finishes inference successfully
but got wrong answer:
0: Paris is the capital city of

after change "return tensor.pin_memory()" to "return tensor", it finishes inference successfully
but got wrong answer

Have you installed the dependencies in the directory DeepSpeedExamples/inference/huggingface/zero_inference/ using pip install -r requirements.txt?


*: force convert to int64 to support 32bit os. Fix #1202

Fix #1202
~Still WIP, since I am currently setting up things to valid the gitea crossbuild with this fix.~

This PR does fix linux/386 builds

This PR does reverse a change in https://github.com/src-d/go-git/commit/e5c9c0dd9ff1f42dcdaba7a51919cf43abdb79f9 from #1199 and cherry-picked from #1187


Grouping DTOs according to Databricks API documentation sections.

Extending ObjectInfoDTO with objectId field.
Removed redundant @Getter and @Setter annotations (already provided as a part of Lombok @Data annotation).

closes https://github.com/edmunds/databricks-rest-client/issues/44

https://github.com/edmunds/databricks-rest-client/pull/64 shall be merge first, then I'll do a little bit more work on this PR


unkown admin restrict and problem of changing places of applications in launcher

Describe the bug or issue
I have the chromium os with an unknown admin that is blocked: "connect android phone" and entering developer mode on the first boot. It came from the first boot. It has only one account registered. Also, the placement of the applications on the launcher forcibly resets/changed every time I stop change the placement/reopen launcher.
Describe your setup

Raspberry Pi 4B with 4GB RAM
Your Chromium OS for Raspberry Pi release version, e.g. Chromium OS r86.04240.186 (32-bit) r4b

To Reproduce
Steps to reproduce the behavior for admin blocking:

Go to 'Settings'
Click on 'Connected devices' / Scroll down to 'Connected devices'
See you can't turn on the Android Phone column.
Steps to reproduce the behavior of resetting of placement of applications on launcher.
drag the application(s) to another place.
wait or reopen the launcher
see if the placement of application(s) reset/changed.

Screenshots and/or hardware setup photos
Pictures and explanations of the problems.zip
Please open this zip file.
Additional context
So basically, you must debug:

delete the administer setting
debug the code on change the application placement code on the launcher(so applications stay in the place where you drag)

Please fix them on next update.

“Connected device” feature isn't available in vanilla Chromium OS, unfortunately, like many other items. Google Assistant, for example, is another one of those.
There are no "admin restrictions" per-se, if there was any, they'd be shown up in chrome://policy
You will see the hollowed, non-functional frontend of those unavailable items if the chromium project is unchanged from upstream. However they really bother you that much, it's easy to hide them by commenting out the relevant sections in chromium and rebuild.

I am unable to reproduce your reported issue 2 about changing the order of the applications. In our test kits, if you drag and place any app from posA to posB, they stay there.
Closing this issue now, further comments welcomed.


Fix oauth reset secret

Coverage decreased (-0.01%) to 82.402% when pulling 1dac2d071c5c1261c0119842748c9b4918e74bb3 on eatonphil:fix-oauth-reset-secret into 7a64a432ff7d3933657de5363d33b75c5a8fdc7c on linode:master.

Thanks!


Now you can swipe at the clients section to navigate! :D

Added a new feature, enabled mobile support for swiping left and right at the clients section to navigate.

@travisneilson so I will not merge that, right?


Patch/Nimbus: Do not update parameter while changing it

Hi
First of all thanks a ton for the awesome new Nimbus firmware. I wanted to port clouds to daisy patch since forever but never found the time. I am so glad you did this!
Currently, when using the encoder to update a parameter <-> input mapping, the parameters get updated during the selection process (i.e. while 'scrolling' through the list). This makes it basically impossible to set a parameter to a fixed value without keeping an active input mapped to it.
This changes the behavior to only update the parameter after exiting the selection process.
Cheers

When I was initially testing it, I thought I remembered being able to set a parameter and then change without it updating. Maybe something happened between then and now.
We'll review, and test your fix in the next few days and get it merged :)
Thanks for the feedback, and contribution!

We looked at it, and it's all good!
@corvusprudens had a pretty good idea for reversing the highlight color highlighting a parameter when it's being changed, and tried it out with this snippet on line 209:
paramControls[i].getName((i == cursorpos) ? increment : 0), Font_7x10, !(selected && (i == cursorpos) && mymod(increment, NUM_PARAMS) != 0));

Seems like a cool  idea. So if you want to add that we can get it merged (or we can add that at a later time).

If we want to add the additional text inversion we can do that in a later update. For now this is fine :)
Thanks for the fix, @baumgartnerniels !!