Add support for Spacelift (spacectl)

Overview
This PR introduces support for spacectl through API Keys. You need to pass your endpoint URL, API Key ID and Secret, that's it!
Type of change

[x] Created a new plugin
[ ] Improved an existing plugin
[ ] Fixed a bug in an existing plugin
[ ] Improved contributor utilities or experience

How To Test

There's a handy command to test if the credentials are indeed provisioned:
spacectl whoami

The output should contain JSON with the ID and name of the API Key as well as the endpoint URL.
Changelog

Hey @damianstasik , could we help you move this forward in any way?

@AndyTitu hey, sorry for the long delay, I'll be taking a look at it today!

@arunsathiya thank you for your review, and again sorry for long delay! I've applied your suggestions and finalized the api key importer, works great on my end! Let me know if this is closer to being shippable :)

Hey @damianstasik could you please sign your first few commits as well so we could merge this?

@AndyTitu done, thanks for taking a look at this PR!


Fix off-by-one bug for y-position of mouse cursor

Hi, it's me again. I fixed a small inaccuracy in the Input class, where the y-position of the mouse cursor at the very the top position was reported as 1, and not 0.

Hm, I'm not sure about this one. It sound odd to me that X is correct but not Y, and that it wasn't reported before as that sound pretty serious.
Could you post a minimal program that reproduces it so I can test on my computer? Thanks.

Hi, thanks for your quick response times.
This short program demonstrates the off-by-one bug for mouse y:
import java.io.IOException;

import lti.lwjgl.util.NativeExtractor;

import org.lwjgl.opengl.Display;
import org.lwjgl.opengl.DisplayMode;
import org.newdawn.slick.AppGameContainer;
import org.newdawn.slick.BasicGame;
import org.newdawn.slick.GameContainer;
import org.newdawn.slick.Graphics;
import org.newdawn.slick.Input;
import org.newdawn.slick.SlickException;

public final class OffByOne extends BasicGame {
  public OffByOne() {
    super("OffByOne");
  }

  @Override
  public void init(GameContainer container) {
    // empty
  }

  @Override
  public void update(GameContainer container, int delta) {
    Input input = container.getInput();
    if (input.isKeyDown(Input.KEY_ESCAPE)) {
      container.exit();
    }
  }

  @Override
  public void render(GameContainer container, Graphics g) {
    Input input = container.getInput();
    g.getFont().drawString(10, 10 + 20 * 1, "SCREEN: " + container.getWidth() + "x" + container.getHeight());
    g.getFont().drawString(10, 10 + 20 * 2, "MOUSE:  " + input.getMouseX() + "/" + input.getMouseY());
    g.getFont().drawString(10, 10 + 20 * 4, "PRESS ESCAPE TO EXIT");
  }

  public static void main(String[] args) throws SlickException, IOException {
    NativeExtractor.provideNatives();
    DisplayMode desktopDisplayMode = Display.getDesktopDisplayMode();
    AppGameContainer container = new AppGameContainer(new OffByOne(),
                                                      desktopDisplayMode.getWidth(),
                                                      desktopDisplayMode.getHeight(),
                                                      desktopDisplayMode.isFullscreenCapable());
    container.setVSync(true);
    container.start();
  }
}

The reason for being x right and y wrong is that lwjgl/sdl/gl coordinates have their origin at the bottom left corner, whereas java has top/left coords. So, the y coordinate gets flipped in Input, by the following formula:
mouseY = height - Mouse.getY()
The lwjgl/sdl/gl y coordinate runs from 0 to height - 1, resulting in java coordinate height - 0 to height - (height -1), which is the same as from height to 1 (flipped, as intended). Still, the range must be from 0 to height - 1, so subtracting one, as in the patch, would get things right.

Makes sense, thanks for taking the time to explain and provide a test program.


fix(datepicker): fix support for literal format on popup

Since the controller is used by old and new, I need to add an extra checking. I will add this to "What to remove" issue for when we remove deprecated stuff.
Fixes #4616

LGTM


Newline in Files search Regex not working

VSCode Version: 1.9.1
OS Version: Mac OS 10.11.6

Steps to Reproduce:

Go to the Search sidebar
Turn on Regex
Search for something like "module.exports.(.*);" (without the quotes) - that works OK
Change that to "module.exports.(.*);\n" - still works OK
Change that to "module.exports.(.*);\nfunction" - does only find in the open files.
Note that in the editor search/replace (CMD-F), it works perfect (both search and replace too)

To make things even worse, when the search sidebar finds something using a regex with newline, if you try to use Replace, the sidebar UI gets completely screwed.

This isn't supported yet - #15727


Validation in submission handler

First off, thanks for making this helpful library.
I'm running into a bit of difficulty with how to handle working with backends where validation and creation aren't necessarily separate steps.
Specifically, in the case of many user registration forms the only way to test if a username is taken is to try registering that user. So there is only a single backend API available.
As far as I can tell, there are two potential places for putting this logic:

In an asynchronous validation handler. This works for doing validation, but it doesn't seem to be possible to dispatch actions from there (and also feels slightly wrong). For example, I can't find a clear way of dispatching the server-provided ID into the Redux state when working through the async validator.

The submission handler. This works for submission and dispatching of state, but there doesn't seem to be any way of invalidating a form in case the submission failed.

Thanks for clarifying the best way to do this! (My natural instinct would be to allow the submission handler to return a validation object.)

Yes, this is an excellent use case. One that I've been meaning to implement. The submit should definitely be allowed to return validation errors. In fact, I also reuse the same client side validation on the server side to double check against ajax naughtiness (e.g. someone altering the request manually).
Expect this feature in the near future.

Fixed in v1.5.0.
Thanks for your feedback, @morgante.


i18n(de): Make translation of 'theme' consistent in German language: 'Theme'

…'Theme'

Description

Make translation of 'theme' consistent in German language: 'Theme'
discussion started in #2541

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

de
components/badges.mdx
Localization changed, will be marked as complete.

de
resources/plugins.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Update disabled.md to have valid server.host

As of Opensearch-dashboards 2.0, server.host: "0" is considered an invalid host address and errors. Changing to <IP_ADDRESS> seems to resolve this.
Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hey @brian-finisher , we recently added a DCO check that requires you to sign off your commits. For information on how to fix your commit, refer to this page.

Well, I guess re-opening the PR didn't work and now it's stuck in a weird state.
this is a lot of effort for a single-string change to some documentation :-(
I'm not sure it's possible to do any --signoff within the GitHub web UI - one must do a CLI clone which adds quite a bit of friction to this process.

@brian-finisher If it would be easier, you can create a new branch, make your commit again (but with a sign-off this time), and then create a new PR. The DCO check checks your entire branch for the sign-off, so if you have two commits in the branch and one of them doesn't have the sign-off, the whole branch is "bad". If you use the Git CLI, use the -s flag for sign-off and -m for your message. GitHub Desktop the client does not work, but if you use Chrome, you can download an extension to sign off. Here's the repo for the extension: https://github.com/scottrigby/dco-gh-ui
One more important note that the email you use to sign off must be included in your GitHub account. For example, if your GitHub account's email is<EMAIL_ADDRESS>you can't sign off with<EMAIL_ADDRESS>Here's a little more info about emails in GitHub: https://docs.github.com/en/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-email-preferences/setting-your-commit-email-address

Thanks @keithhc2. I have submitted a new PR #675


[1.1.x] Adding dummy.txt to storage dirs | Fix test_delay list nesting | Add test for delay func where step > len(series) | Add tests for delay render function with negative step | Simplify logic of delay fun

Backports the following commits to 1.1.x:

Adding dummy.txt to storage dirs (#2259)
Fix test_delay list nesting (#2262)
Add test for delay func where step > len(series) (#2262)
Add tests for delay render function with negative step (#2262)
Simplify logic of delay function (#2262)
Support negative steps in delay render function (#2262)
fixed missing static location in nginx + uwsgi conf #2260 (#2264)
graphite functions: handle max on empty sequences better (#2266)

Codecov Report

Merging #2270 into 1.1.x will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            1.1.x    #2270      +/-   ##
==========================================
+ Coverage   79.58%   79.61%   +0.02%
==========================================
  Files          85       85
  Lines        8848     8840       -8
  Branches     1894     1891       -3
==========================================
- Hits         7042     7038       -4
+ Misses       1545     1543       -2
+ Partials      261      259       -2

Impacted Files
Coverage Δ

webapp/graphite/render/functions.py
96.22% <100%> (+0.17%)
: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 94d4200...1c5892e. Read the comment docs.


add CSS prop in readme

I see the use of css prop in the source code, but it is not written in readme. We should write an example to let the user know about this feature

Hey :)
Thanks @SGAMERyu ; I will for sure add this in documentation.
Keep in mind Pinceau is still in heavy work, but I plan to release a proper documentation website at some point!
This is a single (and vast) project, but I will try my best at making a proper communication once I'm satisfied with the 1.0 release.

Hey :)
Thanks @SGAMERyu ; I will for sure add this in documentation.
Keep in mind Pinceau is still in heavy work, but I plan to release a proper documentation website at some point!
This is a single (and vast) project, but I will try my best at making a proper communication once I'm satisfied with the 1.0 release.

Thank you for your time. pinceau is a great project and I'm looking forward to the 1.0 release


[TablePagination] Support native selection

Closes #13736.

@jsdev It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


add options for subscription cancellation

add cancel_at_cycle_end option for subscription cancellation api.

api for undo cancellation  of subscription when cancel_at_cycle_end is enabled would also be helpful

api call for undo subscription cancellation when cancel_at_cycle_end is enabled will be helpful. (api call must be made before the cycle ends)

@msonowal  workaround for now,
dependency https://github.com/rmccue/Requests
$url = 'https://api.razorpay.com/v1/subscriptions/'.$subscription_id.'/cancel';
$payload = ['cancel_at_cycle_end' => 1];
$headers = array('Accept' => 'application/json');
$options = array('auth' => array(self::$razorpaySecretId, self::$razorpaySecretKey));
$response = Requests::post($url, $headers,$payload, $options);

@SaikumarChintada
yes I am also doing it like that only for now

#83 was a duplicate of this, but got merged first.
Merging this anyway, because it was the first proposed PR to fix the options issue :)


2023.3.x feature auth bg image

Add support for personalized auth background screen .

 Auth background image


Version directories specifications

In the command, is there a way to specify where the version directories will be created?  Or, does it just create the directories in the location where you run it from?
sudo jbvcs -u enuggetry -r jbvcs-test -w 1000 0
It might be a good idea to have an option to specify the target directory, like: -d /mydir

The CLI doesn't come with any new features. It's basically an interface which calls the defined functions of the module. Because the directory is hard coded for now within the module, there's no scope of adding a directory parameter.
On 22-Aug-2016 11:05 AM, Eric Y<EMAIL_ADDRESS>wrote:In the command, is there a way to specify where the version directories will be created?  Or, does it just create the directories in the location where you run it from?
sudo jbvcs -u enuggetry -r jbvcs-test -w 1000 0
It might be a good idea to have an option to specify the target directory, like: -d /mydir
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

The lack of ability to specify the target dir is rather limiting.


added ios link

What does this PR do?
Adds IOS link to footer
Description of Task to be completed?
Add IOS link to footer
How should this be manually tested?
Load deploy preview
What are the relevant Trello Card stories?  Trello Card Item
Pull Request Type (check one)

[ ] Feature
[ ] Bug Fixes
[x] Chore

Developer Checklist

[x] New code has been scanned for ESlint issues
[x] I have tested my code locally
[x] I made sure that there are no other pull requests that solve for this issue
[x] My code follows the same structure within the guidelines

Screenshots (if appropriate)

Can you change the pull request to the development branch @ashleygraf101.
The process is localbranch > development > master.
Also, there's a Coming soon that needs to be removed on the Download section on the footer component.

Sorry, just fixed it. Just realised how that happened.
I'll make the footer component change.


Add grid.viewAsChart and grid.goToNextGrid keyboard shortcuts for editor

We already have shortcuts for save as CVS, Json, and excel in the grid when ResultsGridFocusCondition. I have added 2 more shortcuts which are view as chart and go to next grid (in the case where we have multilple result set)
grid.viewAsChart - CtrlCmd + KEY_R, CtrlCmd + KEY_V
grid.goToNextGrid - CtrlCmd | KEY_R,  CtrlCmd + KEY_N

I would recommend not doing the actual refactor now, I think the impact is too large for such a small desired feature, It should be it's own PR. I was just making that comment as a one off. I would revert to what you had before.

@anthonydresser I agree! I really have little confidence on this refactoring. The code in gridParentComponents, query.component and editData.components are not very organized. We need to refactor in the future


Support for raw XML?

Is there a way to include raw XML for some properties.
Maybe something like this:
class Widget(BaseXmlModel):
    id: str = attr()
    body: etree.Element = element()

Implemented in version 2.1.0.
See for more information.


There is a note that says there used to be a recommendation to disable the Administrator account, however there are still references in this doc that say to check that the account is disabled.

[Enter feedback here]

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

ID: a861650d-0499-c46b-7ca8-d1fb04546192
Version Independent ID: 3967b664-f419-6e8e-5ad8-f98cdcb21733
Content: Appendix D - Securing Built-In Administrator Accounts in Active Directory
Content Source: WindowsServerDocs/identity/ad-ds/plan/security-best-practices/Appendix-D--Securing-Built-In-Administrator-Accounts-in-Active-Directory.md
Product: windows-server
Technology: ad-ds
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

I removed the disabled account references. The update will publish soon.
Closing issue. #please-close.
Pat


QR Code doesn't work with Dashlane

While Google Authenticator and Authy are able to scan and use the QR Code just fine, something in the secret make Dashlane not be able to scan it.
Any idea of why?

Unfortunately my crystal ball is out for repairs, so you'll have to fill us in into some of the details. What are the parameters you're creating the QR code with for example, so that we can try to reproduce and have a look at your actual values and see if there's something wrong with them.

I'm creating the QR Code with nothing else than the default parameters, also using default parameters for the
new TwoFactorAuth();
It's nothing but default.

Could you show us a QR code that doesn't work? Or the DataUri?

The problem is kinda solved by itself.
There is two methods of adding a 2FA code on Dashlane, one of them don't work, but the other do, so it's not a you problem.

Could you elaborate on what the two methods are and which of them works? For future reference...

If you go to the "Authenticator" page and click "Add a 2FA token", then scan the QR Code, the reading will fail and Dashlane will spill out an error
If you, however, go to your existing IDs, click the one from your website, edit it, and click "SETUP" on the 2FA part, then, the code will scan fine and be added to your account.
I hope I was clear enough.

That's really weird.

Here is one of the QR code generated

It scanned fine for me, first try? Deleted it, scanned it again, no problem. Are we talking iOS, Android?

I am talking Android, two of my friend plus me couldn't scan it in the "Authenticator" tab, but editing our profile with our existing info, then scanning the code there worked for us.

I'm sorry, that must be a Dashlane issue then. It scans perfectly fine for me on iOS. Maybe someone else can chime in?

seems fine for me, the URI encoded in the qr code looks fine too, you can use the Google Authenticator app to test it out with a different app on the same phone without having to set up a whole new account to confirm.


Intersnack - The number of jobs is the same as on the company's official website

Description: Check if the number of displayed jobs on "https://peviitor.ro/" of company "Intersnack" is the same as on their website "https://www.intersnack.ro/cariere/oportunitati-de-cariera".
Precondition: The website (https://peviitor.ro/) is up and running

Step 1 : Open "https://peviitor.ro/"
Expected result : You can see the homepage with the searching bar in the middle

Step 2 : Verify if the searching bar is functional by clicking inside it.
Expected result : The text can be typed inside the searching bar

Step 3 : Type the name of the company "..." and use the button "Cauta"
Expected result : You can type the name of the company without problems and after using button "Cauta" the company jobs will be displayed

Step 4: Verify the number of jobs.
Expected result: above the boxes with the jobs you can see the the number of jobs.

Step 5: Go to the company's official website and see if the number of jobs is the same with the jobs number displayed on "peviitor.ro"
Expected result : The number of jobs from the official company website "https://www.intersnack.ro/cariere/oportunitati-de-cariera" is the same with the number of jobs on "peviitor.ro"


argument 'keypairs': failed to extract enum Signer

hello, i have an issue when i want to swap -->argument 'keypairs': failed to extract enum Signer ('KeypairWrapper | PresignerWrapper | NullSignerWrapper')

variant KeypairWrapper (KeypairWrapper): TypeError: failed to extract field Signer::KeypairWrapper.0, caused by TypeError: 'Signature' object cannot be converted to 'Keypair'
variant PresignerWrapper (PresignerWrapper): TypeError: failed to extract field Signer::PresignerWrapper.0, caused by TypeError: 'Signature' object cannot be converted to 'Presigner'
variant NullSignerWrapper (NullSignerWrapper): TypeError: failed to extract field Signer::NullSignerWrapper.0, caused by TypeError: 'Signature' object cannot be converted to 'NullSigner'
i actually use the same code give in the exemple  -->signature = keypair.sign_message(message.to_bytes_versioned(raw_transaction.message)), what's wrong ?

the error came because i tried to rewrite messageV0 to obtain recent blockhash. i fix this.


add reactions and message interactions

Summary
Adds methods to react, unreact, and delete all reactions on messages.
Adds handlers for react and unreact events.
Adds interactions to send_message arguments
Checklist

[x] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.

[ ] This PR fixes an issue.
[x] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, …)

does revolt refer to them as MessageInteractions or MessageReactions? I don't exactly want to introduce a breaking change if and when revolt adds interactions

I've just checked the Revolt api docs and https://developers.revolt.chat/api/#tag/Interactions, looks good


handling  requests for food and other items supply in camps

Subject of the issue
How  to   add location of new  relief  camp (  not  sure if all locations  are added  )
How  to ask  for supplies   for a relief camp  ( getting  messages saying large number of people camps  and less supply )
How to add  a camp contact person .
Your environment

version of your python
which browser and its version
OS Type (iOS/Android/Linux/Windows)

Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo as a GIF, if possible.
Expected behaviour
Tell us what and how exactly is it supposed to  happen
Actual behaviour
Tell us what happens instead or what went wrong

could see  it already handled


Uncaught TypeError: Path must be a string. Received undefined

[Enter steps to reproduce:]

Have gitbook serve running in background whilst editing gitbook
edit page and save
wait for gitbook serve to reindex pages (and possibly load page in browser)
Atom raises above exception

I can also replicate by:

Run atom
open suitable (gitbook) project
"touch" any file in project, e.g. run "touch SUMMARY.md"
Atom raises above exception

Atom: 1.28.0 x64
Electron: 2.0.3
OS: Mac OS X 10.12.6
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Path must be a string. Received undefined
At path.js:28

TypeError: Path must be a string. Received undefined
    at assertPath (path.js:28:11)
    at Object.join (path.js:1251:7)
    at onEvents.events.map.event (/Applications/Atom.app/Contents/Resources/app/src/path-watcher.js:283:46)
    at Array.map (<anonymous>)
    at handler (/Applications/Atom.app/Contents/Resources/app/src/path-watcher.js:276:34)

Commands
  3x -1:50.9.0 core:move-right (input.hidden-input)
     -1:50.1.0 core:backspace (input.hidden-input)
     -1:49.3.0 core:move-down (input.hidden-input)
     -1:48.9.0 core:backspace (input.hidden-input)
 13x -1:48.6.0 core:move-left (input.hidden-input)
     -1:46.5.0 core:backspace (input.hidden-input)
     -1:46.1.0 core:move-up (input.hidden-input)
  2x -1:45.7.0 core:move-right (input.hidden-input)
     -1:45.2.0 core:backspace (input.hidden-input)
     -1:44.9.0 core:move-up (input.hidden-input)
     -1:44.6.0 core:move-left (input.hidden-input)
     -1:44.2.0 core:backspace (input.hidden-input)
     -1:43.9.0 core:move-up (input.hidden-input)
     -1:43.6.0 core:move-right (input.hidden-input)
     -1:43.2.0 core:backspace (input.hidden-input)
     -1:39.2.0 core:save (input.hidden-input)

Non-Core Packages

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/14227 so you can subscribe there for updates if you'd like.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.

(Sorry for delay!)

mkdir Test
Atom Test/Test2
Enter some data, and save (cmd-S)
touch Test/Test2
Enter some more text, and save (cmd-S)

and I get the path.js exception mentioned.
Anything else I can do to help track/fix this?

Opps. Wrong thread


Buildpack's release method called twice

With the 1 change, the bin/release method of a buildpack is called twice during staging. That's because start_command() and effective_procfile() both end up invoking the installer.release_info(). The duplicate invocation of "release" may result in some unexpected behavior in certain buildpacks.

This has been delivered.


[Don't merge till travis passes]Improve help docs and fix related test cases

Purpose
$subject
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18936 into next-release will increase coverage by 5.44%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           next-release   #18936      +/-   ##
================================================
+ Coverage         15.09%   20.53%   +5.44%
================================================
  Files                48       52       +4
  Lines              1259     1383     +124
  Branches            198      216      +18
================================================
+ Hits                190      284      +94
- Misses             1055     1086      +31
+ Partials             14       13       -1

Impacted Files
Coverage Δ

tool-plugins/vscode/src/core/extension.ts
27.55% <0%> (-0.63%)
:arrow_down:

...-plugins/vscode/src/project-tree-view/activator.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/extension.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/overview/activator.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/overview/renderer.ts
0% <0%> (ø)
:arrow_up:

tool-plugins/vscode/src/core/messages.ts
100% <0%> (ø)
:arrow_up:

...s/vscode/src/project-tree-view/project-overview.ts
0% <0%> (ø)
:arrow_up:

...ool-plugins/vscode/src/debugger/config-provider.ts
0% <0%> (ø)
:arrow_up:

...ugins/vscode/src/project-tree-view/project-tree.ts

composer/packages/tracing/src/ToolBar.tsx
71.05% <0%> (ø)

... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3687014...2e8a972. Read the comment docs.


[feat] - Locust version update

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Upgrade version of locust tool

What is the current behavior? (You can also link to an open issue here)
dojot/dojot#1494

What is the new behavior (if this is a feature change)?
updated version of paho MQTT, locust tool

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:

Please resolve conflicts and merge the to branch feature

Please resolve conflicts and merge the to branch feature


update all k.Authority checks to return govtypes InvalidSigner error

this error type is more explicit than our current return value of ibcerrors.Unauthorized
it looks like theres an invalid signer error in `govtypes`. maybe we can update all of these to use that error return, looks like that's what the 08-wasm client returns in `StoreCode`.
Originally posted by @charleenfei in https://github.com/cosmos/ibc-go/pull/4773#discussion_r1345666125

is this a dupe of https://github.com/cosmos/ibc-go/issues/4815?

ah, sorry, i just realised that issue was closed -- i thought that it was still missing!


UI broken in latest dev builds (4122)

The layout is kinda broken, with no variables pane visible (or misplaced)

Will also ask maybe related question here. Is the Terminal view supposed to be scrollable?

I can't access the beginning of the error right now.
Also worth noticing that the whole panel is scrollable (intended) but it doesn't help for this case.

@rchl regarding the issues with the terminal. For now you can click on the tab when its already open and it will open the terminal in an output panel.
I have tried a bunch of things to get an integrated terminal working nicely and scrollable and just cannot do it inside a phantom. No way to copy stuff really sucks as well.
I have been slowly working on replacing the integrated terminals with a view that appears when you are debugging and gets removed when you click away from it after debugging. This works great for me since I have a really wide monitor but probably isn't going to be all that great for smaller screens. Probably still better than the phantom approach.

Also worth noticing that the whole panel is scrollable (intended?) but it doesn't help for this case.

Scrolling is intended and it's useful for the other panels since they expand downwards when they have lots of content. Unfortunately not so useful in the case of a terminal.


fix scrolling on static pages

I don't think any of these pages exist on patricbrc atm, so i'm closing.


Fix Return 1 as a Minimum of Satoshi amount.

As mentioned in lnbits#655 :

I changed the code when you did not accept to change the output type of the "fiat_amount_as_satoshis ()" function to float.
because of When the fiat currency price like IRT is lowest from satoshi, the function returns 0 and addon like BTCPOS can't calculate correct amount of satoshi and create invoice.
For example : 1 IRT => 0.0104 Sat then Function return 0,

Add Nobitex exchange, because many Iranians use this exchange and it is almost a price reference for digital currencies of this exchange.

Hey!
If 1 IRT => 0.0104 sat then we shouldn't assume that it's 1 sat. At that point, currency conversion doesn't make any sense anyway.
Another point: Adding an exchange just for one ticker (IRT) seems like a bad idea since it will receive requests for all other tickers as well. I think one exchange for IRT was added already. As long as we don't specify in the code which currency should be requested from which exchange, I would be against adding another one for a single ticker.

Closing.

Hello
Why dont merge ?


Particle Effects System

Description
The goal for this feature is to implement a generic and reusable Particle Effect system which can be used throughout the game to provide further depth to the visual style.
Examples

Explosion - particle effects will allow for explosions to be created which could be used for future Turret variants or landmines in the future.
Enemy Death - particle effects could be used to provide more visual variety in enemy deaths.

Plan
This system will use the LibGDX ParticleEffect class for drawing particles. This also allows us to use the pre-existing particle editor tool.
The plan is to add a LoadParticleEffects method to the ServiceLocator so that it can be loaded the same way that other assets can be. Loading particle effects will also need to be added to the GameArea and its config file to allow them to be loaded through configs using the current system.
A component/set of components will then be used to be able to begin and stop each effect. An event driven system will be used to allow other components to start a particle while maintaining a level of abstraction. This will ensure dependencies remain low.
Milestones
List of sub-features you intend to have done and in Master by the end of the sprint.

[x] Add LoadParticleEffects to the ServiceLocator
[x] Allow for a list of particle effects to be loaded via the GameArea config files
[x] Implement a ParticleComponent to allow for particles to be triggered
[x] Implement a particle effect into the game as a proof of concept
[x] Testing
[x] Javadoc's
[x] Documentation

Completion Deadline: Sep. 29
Documentation

TBC

Members

Rowan Gray (@rowan-gray)

Idk why I closed this since I still need to do testing and documentation

Completed


[Request]: GitHub Desktop

Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used (e.g. if it's a GitHub project, it should have at least 100 stars and/or 50 forks)
[X] English interface (or at least English documentation)
[X] Latest stable version
[X] Full version (i.e. not a trial version)
[X] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
github-desktop
Description
GIT GUI Client
Homepage
https://desktop.github.com
Download Link(s)
https://desktop.github.com/download/
Some Indication of Popularity/Repute
https://git-scm.com/downloads/guis - 1st option
It's not found in scoop official/known buckets,

extras/github


Why does handler need to be Sync?

I have a situation where I want to pass a sender to the request handler:
use std::sync::mpsc::Sender;
let tx: Sender<Foo> = get_sender();
rouille::start_server(ADDR, move |req| handle(req, tx.clone()));

Unfortunately, this doesn't work because unlike std::thread::spawn, start_server(addr, handler) requires that handler is Sync.
Why is this the case? Is there any way to get around it without resorting to a Mutex?

I actually ran into this exact case today!
Rouille uses a thread pool internally, which means that multiple requests could be happening simultaneously. That concurrent access means that any types you access from your handler need to be Sync!

Hi, I ran into this issue too. The problem, from what I can tell, is that there is no way to actually clone a Sender such that you can move it into one of the threads that will process a request.
Often in cases like these, you can get around it by cloning then moving in an extra block:
{
    let tx = chan.clone();
    move |req| {
        // use tx
     }
}

But even this doesn't appear to work. There should be a way to share Sender clones to request handlers. Is there any workaround in place here except for adding an extra Arc on the Sender?


🐛 CostInsights: Abnormal scaling of cost chart along y axis

Plugin Name
CostInsights
📜 Description
I have cost data coming from aws of the range 300 to 400 dollars. The cost chart displayed in the cost insights page has y axis values like 450,000,000 ,  900,000,000, etc. Due to this, unable to see the data visually.

👍 Expected behavior
It should be in somewhat 0 to 1000 range based upon my data. Or it should be like 0 to max value of my data.
👎 Actual Behavior with Screenshots
It has enormously high y axis values like 900,000,000.

👟 Reproduction steps
Go to cost insights page and view the daily cost graph.
📃 Provide the context for the Bug.
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Hi @esw-afabiano, any ideas on this one?


Item Dupe

When you have Soulbound on armor while keep inventory is on, the armor will try and duplicate itself in the slot below it, due to TFC changing the back slot to take the first armor inventory slot id. This can cause items to either become copies, or items to be lost. Seems to only affect equipped armor.

Removed Soul Bound Enchantments
This issue / recipes will be fixed within the next release of the modpack.


update: optimize memory allocation and refactor next/next_batch helpers

Signed-off-by: Jason Dellaluce<EMAIL_ADDRESS>Co-authored-by: Leonardo Grasso<EMAIL_ADDRESS>What type of PR is this?
/kind design
/kind feature
Any specific area of the project related to this PR?
/area plugin-sdk
What this PR does / why we need it:
This PR refactors the sdk helpers used in the next and next_batch plugin symbols. The new design proposes the usage of a new object interface PluginEvents that brings the following value propositions:

It's easier to use and more elastic. It does not force developers to use the current callback method of the NextBatch wrapper, and can be used inside next too.
Allocates memory only once, and does not perform any buffer copy on the fly. The benchmarks bundled in the PR shows that in the general case the next_batch flow performs zero memory allocations.
Hides the complexity of the underlying C-memory management. At the same time, the object gives pointer access to the C buffers for advanced uses or for passing them to the plugin framework.
Enables safe and protected memory accesses by implementing the io.Reader and io.Writer interfaces. For instence, one may access event data in write-only mode during the next flow, and in read-only mode in the field extraction flow.
Disambiguates the C memory governance in the plugin framework. C-memory related to the plugin events is totally managed inside the plugin by the sdk.

Additional context:
See https://github.com/falcosecurity/plugin-sdk-go/issues/9.
Additional notes for the reviewer:
The changes of this PR affect the go sdk only, but need few minor changes inside libs too. This is due to the fact that libscap currently frees the produced event data at each iteration of next, which makes it impossible to reuse the same memory buffer and makes the memory governance ambiguous. In order for this to work, the only code locations to remove  in libs would be the following (we refer to the plugin-system-api-additions wip branch of libs:

libscap/scap.c#L1710
libscap/scap.c#L1801

Here's the patch 👇🏼  intended to be used on top of https://github.com/falcosecurity/libs/pull/93
memory.patch.txt
Which issue(s) this PR fixes:
Fixes https://github.com/falcosecurity/plugin-sdk-go/issues/9
Does this PR introduce a user-facing change?:
update: optimize memory allocation and refactor next/next_batch helpers

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jasondellaluce
To complete the pull request process, please assign kris-nova after the PR has been reviewed.
You can assign the PR to them by writing /assign @kris-nova in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @jasondellaluce. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Welcome @jasondellaluce! It looks like this is your first PR to falcosecurity/plugin-sdk-go 🎉

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1b7bb00 chore: update size checking

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1b7bb00 chore: update size checking

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jasondellaluce: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-plugin-sdk-go
9bb4afb93de05c93aceea8cb95b2d413d5588c45
link
true
/test build-plugin-sdk-go

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Hey @mstemm
Here is the patch for the docs :point_right:  https://github.com/falcosecurity/falco-website/pull/493#pullrequestreview-781907448


LDAP - failing to bind anonymously

Hi all,
I've been trying to setup LDAP, but not having much luck with anonymous bind. Has anyone actually managed?
All I can find is most people have used a bind account
I've tried debugging but I can't see any errors and the AD server does allow anonymous bind.
AUTH_METHOD=ldap
LDAP_SERVER=example.com:389
LDAP_BASE_DN=DC=example,dc=com
#The full DN and password of the user used to search the server
#Can both be left as false to bind anonymously
LDAP_DN=false
LDAP_PASS=false
LDAP_DISPLAY_NAME_ATTRIBUTE=cn
LDAP_TLS_INSECURE=false
#Active Directory
LDAP_USER_FILTER=(&(sAMAaccountName=${user}))
LDAP_VERSION=3
LDAP_ID_ATTRIBUTE=BIN;objectGUID
#Debugging if you have issues, comment out after LDAP is working
#APP_DEBUG=true

What is your LDAP Server type? I'm assuming AD but looking to confirm.
Can you try using a tool like JXPlorer to perform an anonymous bind?

What is your LDAP Server type? I'm assuming AD but looking to confirm.
Can you try using a tool like JXPlorer to perform an anonymous bind?
LDAP_DUMP_USER_DETAILS=true might be a great way to get info as well, otherwise watch the server and find out what it is responding with. It's been a while since I've worked with AD, more so OpenLDAP but they are one of the same, just different ways to get access to the logs.

@tiredofit thank you for your reply. I basically gave up and used a service account :)


Inconsistent comprehension syntax

Every loop is structured with some variation of for in or for of. Doing it this way keeps us from being able to [f .. for of obj] even forgetting about object comprehensions. I see that as a separate bug. The other point of inconsistency was it in lambdas. Cascades are nicer for property access and appear in both with and comprehensions.

@gkz ?

This and ! were my biggest complaints. I've since moved on to ES2016.

I'm gonna try one last time... @gkz?

Sorry I closed it because of lack of support. I thought inconsistencies (this issue, ! operators, ambiguous implicit lists, function application associativity, etc.) would be addressed by LS2 but never culminated. With do expressions in ES2016 making switch more functional along with many features adopted from Coffee I think the community will be best served by building on Babel. I'm starting working on a partial application symbol and currying prototype methods inspired by LS.

That's okay, I understand. I still believe LS to be a better option and this issue should have a conclusion, though :). (I'm very biased)

So, you want to add '[f .. for of obj]'?
Please keep issues open even if you are no longer interested as they might be revisited in the future.

Basically, yes. I guess we could allow for in obj for consistency (the in would thus be optional)

And it would be cascade for consistency as well.

That's something I disagree with, and if it needs to change, I'd rather see it the other way around... But I'd be against such a breaking change for LS 1.x anyway...


cask-repair: use system perl

Fixes https://github.com/vitorgalvao/tiny-scripts/issues/135

Thank you.


upgrade aws-sdk to latest version

Fix #388

Codecov Report

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

@@           Coverage Diff           @@
##           master     #408   +/-   ##
=======================================
  Coverage   38.71%   38.71%
=======================================
  Files          29       29
  Lines        2278     2278
=======================================
  Hits          882      882
  Misses       1277     1277
  Partials      119      119

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 d8fd464...4f9858e. Read the comment docs.

Thanks for your support @gianrubio
LGTM 👍


http://phisix-api.appspot.com/stocks.xml is Over Quota CRITICAL

There are times that when i opened and access this page (http://phisix-api.appspot.com/stocks.xml) and other related pages (either JSON OR XML) it shows  the following:
Over Quota
This application is temporarily over its serving quota. Please try again later.

Please kindly fix it.

Try http://phisix-api2.appspot.com/stocks.xml http://phisix-api2.appspot.com/stocks.xml

Edge Dalmacio

On Mar 25, 2015, at 10:49 AM, heyou93<EMAIL_ADDRESS>wrote:
There are times that when i opened and access this page (http://phisix-api.appspot.com/stocks.xml http://phisix-api.appspot.com/stocks.xml) and other related pages (either JSON OR XML) it shows the following:
Over Quota
This application is temporarily over its serving quota. Please try again later.
https://cloud.githubusercontent.com/assets/8090574/6817516/7a5f04e6-d2dc-11e4-9c9c-f626aa01abda.png
Please kindly fix it.
—
Reply to this email directly or view it on GitHub https://github.com/edgedalmacio/phisix/issues/14.

I've setup a load balancer between phisix-api and phisix-api2. That should help fix the "Over Quota" issue.


deps: Use python2 explicitly, to avoid FTBFS on modern systems

Modern Linux distributions do not alias python to python2. Ptyhon's
PEP394 goes into further detail, but currently you should point to the
Python version you expect to find as python.
This fixes various FTBFS on modern Linux/macOS systems with no python
alias.
See: https://www.python.org/dev/peps/pep-0394/
See: https://github.com/mapbox/node-sqlite3/issues/1328
See: https://github.com/mapbox/node-sqlite3/issues/584
See: https://github.com/mapbox/node-sqlite3/issues/1443

Please note that just aliasing python with a package (like Debian's python) or manually is not a real solution, and leaves the user/developer's system open to many funny bugs, which the "No default python thing is precisely trying to avoid".

Thanks for the change.


looks like this action is not supported...

Hello,
Please add steps to recreate, the version of the SDK you're using and the details about the device you're using so we can investigate this bug.

I have a similar problem .
The SDK I use is com.paypal.checkout:android-sdk:0.5.2
My Client ID is  AeJ407jpGvJ0xtTdt1JtDpKZJMSKLSk-6UK-Lh-JCtywuVZYxLXneOUXV8rv2hTH5YrXwTAeeoKkXRq3

Hi @YBZzhangyanbin - could you create a new GitHub issue and provide these details? I'll be able to help you there.
@shixiangyu - would you be able to share the version of the SDK you're using?

Hi @YBZzhangyanbin - could you create a new GitHub issue and provide these details? I'll be able to help you there.
@shixiangyu - would you be able to share the version of the SDK you're using?

OK

Thanks @YBZzhangyanbin , we are looking forward to following up

Hi @shixiangyu and @YBZzhangyanbin - I'll be closing this issue due to inactivity. Please comment here if you're still having issues. Thanks!


On STM32,Multiple TPDOs uses event-driven transmission,  How to use the function "OD_requestTPDO(uint8_t *flagsPDO, uint8_t subIndex)" and "uint8_t *OD_getFlagsPDO(OD_entry_t *entry)" sends TPDO?

I think that you should use "CO_TPDOsendRequest(TPDO[idx])"

https://github.com/CANopenNode/CANopenDemo/blob/master/demo/CO_application.c#L112

But I'm not really sure: if I have e.g. 16 temperature sensors which I have set that can be mapped to TPDO, do I have to follow this approach for every of the sensors? if this the intended method?

Yes. With analog values you could also use some hysteresis this way. But with event driven PDOs you may also set some long interval on event timer and also set inhibit timer.
Event driven PDOs may be very fast in CANopen, but they could also overload the traffic. So more reliable and deterministic approach could be with SYNChronous PDOs.

Event driven PDOs may be very fast in CANopen, but they could also overload the traffic.

But isn't this exactly the point of the inhibit time?
Also if we are talking about bus load, this is heavily dependent on your use case. A temperature will not change often, and sync telegrams and responses of the same value over and over, is way more load for the bus than.

So the current process seems less than optimal:

I have to add an "extension" to all my mapped OD entries. To add some flags. But just knowing all mapped entries can be hard, as this even could change at runtime.
I have to manually keep track of changes on the mapped objects.

So, the only thing TPDOs currently do well, are timed and synchronous sending. Which is not working in event driven environment.
Why not just have another (uint64_t, as this is at max 8 Byte long) field in every TPDO object, that stores the last transmitted value that get's compared to the current state? So basically what every user now has to do for every mapped value individually?

Event driven PDOs may be very fast in CANopen, but they could also overload the traffic.

But isn't this exactly the point of the inhibit time? Also if we are talking about bus load, this is heavily dependent on your use case.

Yes, everything depends on use case. But inhibit time may not prevent overload. Synchronous transmission is more deterministic. For example, if some exceptional event happens on whole system, devices may try to send many different PDOs at the same time. And this may overload the bus at the very inconvenient moment. In case of synchronous PDOs there is no problem. However, that is just one point of view.

So the current process seems less than optimal... Why not just have another (uint64_t, as this is at max 8 Byte long) field in every TPDO object, that stores the last transmitted value that get's compared to the current state?

That may be true for one user case, when we want to automatically send all or some PDOs after they change. That is quite simple solution, which was implemented in v1.3. But application could have no control over it.
"New approach" is current, where application has full control, not over the TPDO, but over the OD variable, which may be mappded to a TPDO...

For example, if some exceptional event happens on whole system, devices may try to send many different PDOs at the same time. And this may overload the bus at the very inconvenient moment. In case of synchronous PDOs there is no problem.

Yes, both have up- and downsides, depending on the use case. My prior comment was about overload from one single client, but you are totally right.

That may be true for one user case, when we want to automatically send all or some PDOs after they change.

Imo this is not a "use case" but indicated by the "event driven" transmission types of the TPDO (254 and 255).

"New approach" is current, where application has full control, not over the TPDO, but over the OD variable, which may be mappded to a TPDO...

Well of course it's not working now, but I don't see how this makes implementing this harder or impossible? In the TPDO_process routine, just check the last sent value against the current value (calculated from the mapped OD entries). This would remove the need for flags on every mapped value and the need to mark OD entries changed in the first place.
So just some pseudo code. Just to get my idea across:
TPDO_process() {
	for (TPDO in all TPDOs) {
		//.... checking sync flags, count down timers etc.
		uint64_t currentValue =  getCurrentTPDOValue(TPDO);
		if (TPDO.lastSentValue != currentValue && inhibitTime == 0) {
			sendTPDO(TPDO);
		}
	}
}

getCurrentTPDOValue(TPDO) {
	// Probably I switched LSB and MSB here or did some weird inbetween
	uint64_t current = 0;
	uint8_t index = 8;
	for (mappedEntry in TPDO) {
		index = index - mappedEntry.size;
		current |= mappedEntry.value << index;
	}
}

Can you tell me, why this couldn't work?

For sure your proposal could work, actually he just tell you it was how it worked before.
I'm pretty sure that a conditional compile flag and/or additional TPDO internal config could be set for this behavior to be implemented in the library and hence no TPDO request flag should be set. May be both methods can co-exist
This should have a table of pros and cons of each memory, mostly:

Control over OD and TPDO: I still do not understand clearly what @CANopenNode means with this but seems most important
OD Read function (probably related to previous point)
Memory (the extension memory would be saved maybe but this certainly will add some other memory? plus the 64 bits to store last sent value)
Performance (building the value to compare on every TPDO process will add a bit of overhead)

What is actual problem with the current implementation?

Everything about event driven TPDOs:

@glmnet hypothetical szenario: When having a lot of TPDOs, and values, storing the data and comparing the change is a lot of logic, but pretty redundant.
Event driven TPDOs currently are pretty static. You can't just put any OD index into the mapped values, because the OD entry needs to have the extension preconfigured. And value changes must be observed by the application, so the application has to know to mapped values and also update the observer when the mapping changes at runtime. Configuring event driven TPDOs dynamically is absolutly not trivial.

Events are not "automatic", they requires some programming effort. Is this the problem?

If you want to call it that, yes. Because timers are also not "automatic", yet this library provides functionality for them.

We could add also the alternate method. But, will the alternate method spend less processing power? ("getCurrentTPDOValue" is more complex as in example shown above.)

Probably more processing necessary but less development time and less CANOpen function specific code inside my application (what is, imo, the main reason to use a library like yours - not having to care about function or technology specifics).
I guess another, a bit less processing intesive way, would be to provide an OD entry write function, that checks for difference and set's a flag. As I understand this, this is already possible using these extensions, but imo that's not really intuitive (and the documentation about extensions even warns you about their powerfulness).

I finally got arround to create a custom write function which does the check automatically. Maybe this also will help someone in the future.
But @CANopenNode could you take a look at it? It feels kind of verbose and assumes, you add the OD_entry_t* as object in the OD_extension_t for the entry. Is there a better way to request the TPDO in a custom write function without being vulnurable for future changes in the library?
ODR_t OD_writeTPDOCheckChange(OD_stream_t* stream, const void* buf, OD_size_t count, OD_size_t* countWritten) {
    // Only stream is currently relevant for us. All other values will be checked in original method
    if (stream == NULL) return ODR_DEV_INCOMPAT;

    // Check if original data exists
    if (stream->dataOrig == NULL) return ODR_SUB_NOT_EXIST;

    // Copy original data in temporary buffer
    uint8_t* dataOrig = malloc(sizeof(uint8_t) * stream->dataLength);
    memcpy(dataOrig, stream->dataOrig, stream->dataLength);

    // Call original write function
    ODR_t ret = OD_writeOriginal(stream, buf, count, countWritten);

    // If it wasn't successfull we don't need to bother
    if (ret != ODR_OK) {
        // Before returning, we need to free the temporary buffer
        free(dataOrig);
        return ret;
    }

    // Get or find the OD_entry_t*
    OD_entry_t* entry = stream->object;
    if (entry == NULL) {
        entry = OD_find(OD, stream->index);
        // Still can't find it? Leave.
        if (entry == NULL) {
            free(dataOrig);
            return ODR_OK;
        }
    }

    // Check if the value changed and request TPDO if it is
    if (memcmp(dataOrig, buf, stream->dataLength) != 0) {
        OD_requestTPDO(entry, stream->subIndex);
    }

    free(dataOrig);
    return ODR_OK;
}

Hi,
Can i ask why you are doing malloc(), memcpy() then memcmp(), is there a
reason you cannot simply memcpy(stream->dataOrig,buf,stream->dataLength)?
Would then reduce the malloc/free calls as these will be called a lot. I
may have missed something but a quick glance looks like there is an
optimisation.
On Wed, 18 Dec 2024 at 14:23, Daniel @.***> wrote:

I finally got arround to create a custom write function which does the
check automatically. Maybe this also will help someone in the future.
But @CANopenNode https://github.com/CANopenNode could you take a look
at it? It feels kind of verbose and assumes, you add the OD_entry_t* as
object in the OD_extension_t for the entry. Is there a better way to
request the TPDO in a custom write function without being vulnurable for
future changes in the library?
ODR_t OD_writeTPDOCheckChange(OD_stream_t* stream, const void* buf, OD_size_t count, OD_size_t* countWritten) {
// Only stream is currently relevant for us. All other values will be checked in original method
if (stream == NULL) return ODR_DEV_INCOMPAT;
// Check if original data exists
if (stream->dataOrig == NULL) return ODR_SUB_NOT_EXIST;

// Copy original data in temporary buffer
uint8_t* dataOrig = malloc(sizeof(uint8_t) * stream->dataLength);
memcpy(dataOrig, stream->dataOrig, stream->dataLength);

// Call original write function
ODR_t ret = OD_writeOriginal(stream, buf, count, countWritten);

// If it wasn't successfull we don't need to bother
if (ret != ODR_OK) {
    // Before returning, we need to free the temporary buffer
    free(dataOrig);
    return ret;
}

// Get or find the OD_entry_t*
OD_entry_t* entry = stream->object;
if (entry == NULL) {
    entry = OD_find(OD, stream->index);
    // Still can't find it? Leave.
    if (entry == NULL) {
        free(dataOrig);
        return ODR_OK;
    }
}

// Check if the value changed and request TPDO if it is
if (memcmp(dataOrig, buf, stream->dataLength) != 0) {
    OD_requestTPDO(entry, stream->subIndex);
}

free(dataOrig);
return ODR_OK;

}
—
Reply to this email directly, view it on GitHub
https://github.com/CANopenNode/CANopenNode/issues/489#issuecomment-2551451715,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG2GE7BKPBLQVTSZOZQTT2GGAOZAVCNFSM6AAAAABMOFTLVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJRGQ2TCNZRGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


refactor(ipld): rework Retriever

Retrieve is now blocking forever until either context is canceled or square is reconstructed
Quadrant requesting is now async and detached from reconstruction attempts
session provides a done channel that notifies users if there enough shares were downloaded to try reconstruction

Tested with #702
Closes #706

Tests failing

Oh yeah, that's another flakey one which his now fixed

Ok, goroutines limit again... aaaa

@liamsi, mind to re-revouz?

Rebase on master

The goroutine limit issue in tests is now fixed.

The failed test is https://github.com/celestiaorg/celestia-node/issues/787 and I won't fix this in this PR.

Looks good 🚀


DataFrame.cumsum()

Codecov Report

Merging #500 into master will decrease coverage by 2.88%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #500      +/-   ##
==========================================
- Coverage   93.74%   90.86%   -2.89%
==========================================
  Files          30       30
  Lines        3948     3961      +13
==========================================
- Hits         3701     3599     -102
- Misses        247      362     +115

Impacted Files
Coverage Δ

databricks/koalas/missing/frame.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
93.05% <100%> (-2.07%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
23.58% <0%> (-70.76%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/conftest.py
87.87% <0%> (-9.1%)
:arrow_down:

databricks/koalas/__init__.py
76.31% <0%> (-7.9%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

databricks/koalas/namespace.py
88.23% <0%> (-1.69%)
:arrow_down:

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 74e62e9...9e85654. Read the comment docs.

Im going to take this over at https://github.com/databricks/koalas/pull/521


Clean up some code, fix some stuff (untested)

I see you love your comments :stuck_out_tongue:
Just cleaned up some code, and now using struct instead of writing/reading bools independently for better efficiency and reliability. Removed all of that "static" crap, you don't need it unless it's for a buffer inside of a function.

Woooweeee! That's a lot of fixing! Thank you, and yeah I love my comments , it keeps me on track of what the function does. :P (The YouTube link I will keep ;)) The Bluetooth setting will not work by the way because I have yet to implemented it in the JavaScript, because I do not know how to add more than 1 setting! Will upload the html tonight also if you want to look at it?
Again thank you so freaking much!

No problem! Glad you're this excited :)
I'd be happy to look at the code again w/ HTML tonight/early tomorrow.

I tried to compile it but it says that settings is undeclared, I have added an index.html also to master branch :)

Add:
Settings settings;
To the top of the file, that will fix that. I can look at the html when I'm
not mobile
On May 28, 2015 4:01 PM, "Aborgh"<EMAIL_ADDRESS>wrote:

I tried to compile it but it says that settings is undeclared, I have
added an index.html also to master branch :)
—
Reply to this email directly or view it on GitHub
https://github.com/Aborgh/CUTOFF/pull/1#issuecomment-106581222.

Added Settings settings; at the top , did nothing though. I will make a much pretier version of the html on a later point but you get the idea :P


How can I use this extension in Google Chrome?

Hello, I installed this extension in Google Chrome but I have some problem.
If I go to https://gist.github.com/mbostock/963c4d246e96963856bf I can see the button "bl.ocks" at the right of the "Download ZIP" button but if I go to https://gist.github.com/alessandro-gentilini/c07039f3d5dea29f767c the button "bl.ocks" is not there.
What am I missing? Do I need to be "registered" on http://bl.ocks.org/ in order to use it?
Thank you.
Alessandro Gentilini

The issue is that you have a hyphen in your username, and there’s a bug where the extension doesn’t detect Gists when the username has a hyphen. Changing the title of this issue accordingly.

More precisely, the username regular expression is: /^[a-z0-9][a-z0-9]*$/i, and this should be made more lenient.

Thank you!


Can't increase max concurrent Gdx.net HTTP requests

Please ensure you have given all the following requested information in your report.
Issue details
My app is triggering two HTTP requests, one of which might take a few seconds to load. I noticed however that the second request is not issued, until the first one completes. I would like for these HTTP calls to be made in parallel.Please provide the details of your issue
Reproduction steps/code
Looking in the code, the NetJavaImpl  uses an new AsyncExecutor(1);, where 1 is the max concurrent requests. I would like to increase this (ie. to 2 or 3), however it doesn't seem to be possible since it is hard coded to 1.
https://github.com/libgdx/libgdx/blob/master/gdx/src/com/badlogic/gdx/net/NetJavaImpl.java#L127
What would you recommend in order to make it possible to shoot multiple parallel HTTP requests to the server?
Version of LibGDX and/or relevant dependencies
1.9.6
Stacktrace
Please select the affected platforms

[x] Android
[x] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

Maybe a regression introduced here : https://github.com/libgdx/libgdx/commit/46eb90103f8b51ac5b90d74a47f3d577e7b17b91#diff-ead460e624940f939f21200edb7e5897R127
A solution could be revert back to Executors.newCachedThreadPool(factory); with factory setting threads as daemon.

It probably makes sense to allow for a limit to be specified, would it be preferred to default to 1 as it is today or go back to the unbounded newCachedThreadPool?

I would go for unbounded newCachedThreadPool which is IMO the common use case.
Only few edge cases need to limit thread pool.


Rescan blocks over and over again.

Then if you have a current blockchain and chainstate data it loops the rescan on start over and over again.
It wear level my SSD by 3.5TB in not even 2h of working. This is complete crap seriously.
2017-02-09 20:59:20 ZCoin version v<IP_ADDRESS>-unk (2017-02-08 02:29:10 +0100)
2017-02-09 20:59:20 Using OpenSSL version OpenSSL 1.0.1j 15 Oct 2014
2017-02-09 20:59:20 Default data directory C:\Users\user1\AppData\Roaming\zcoin
2017-02-09 20:59:20 Using data directory C:\Users\user1\AppData\Roaming\zcoin
2017-02-09 20:59:20 Using at most 125 connections (2048 file descriptors available)
2017-02-09 20:59:20 Using 12 threads for script verification
2017-02-09 20:59:20 scrypt: using scrypt-sse2 as detected.
2017-02-09 20:59:20 init message: Verifying wallet...
2017-02-09 20:59:20 dbenv.open LogDir=C:\Users\user1\AppData\Roaming\zcoin\database ErrorFile=C:\Users\venom\AppData\Roaming\zcoin\db.log
2017-02-09 20:59:20 Bound to [::]:8168
2017-02-09 20:59:20 Bound to <IP_ADDRESS>:8168
2017-02-09 20:59:20 init message: Loading block index...
2017-02-09 20:59:20 Opening LevelDB in C:\Users\user1\AppData\Roaming\zcoin\blocks\index
2017-02-09 20:59:20 Opened LevelDB successfully
2017-02-09 20:59:20 Opening LevelDB in C:\Users\user1\AppData\Roaming\zcoin\chainstate
2017-02-09 20:59:20 Opened LevelDB successfully
2017-02-09 20:59:20 LoadBlockIndexDB(): last block file = 0
2017-02-09 20:59:20 LoadBlockIndexDB(): last block file info: CBlockFileInfo(blocks=12100, size=34906947, heights=0...11120, time=2014-10-31...2016-12-10)
2017-02-09 20:59:20 LoadBlockIndexDB(): transaction index disabled
2017-02-09 20:59:20 LoadBlockIndexDB(): hashBestChain=f1d0747b36df0a7149982bbcb5d9f76b46c43863c3dff37099db14d93b18691c  height=11120 date=2016-12-10 00:13:30
2017-02-09 20:59:20 init message: Verifying blocks...
2017-02-09 20:59:20 Verifying last 288 blocks at level 3
2017-02-09 21:05:42 No coin database inconsistencies in last 289 blocks (1190 transactions)
2017-02-09 21:05:42  block index          381817ms
2017-02-09 21:05:42 init message: Loading wallet...
2017-02-09 21:05:42 PROCESS BLOCK = 0
2017-02-09 21:05:42 PROCESS BLOCK = 1
2017-02-09 21:05:42 PROCESS BLOCK = 2
2017-02-09 21:05:42 PROCESS BLOCK = 3
2017-02-09 21:05:42 PROCESS BLOCK = 4
2017-02-09 21:05:42 PROCESS BLOCK = 5
2017-02-09 21:05:42 PROCESS BLOCK = 6
2017-02-09 21:05:42 PROCESS BLOCK = 7
2017-02-09 21:05:42 PROCESS BLOCK = 8
And so on..ever time and it never starts in the and it repeats this crappy procedure after it processed all blocks.. seriously will you replace me a drive?

You could try with -checkblocks=0 for every restart. It checks your wallet once and after finish it won't check anymore. We are working on solution for better performance. Thank for the patience.

Hey tk, this is LBCMiner.

Could you give me ur config ? This line from your log surprises me a little bit and I think it's the main culprit for the heavy IO.
2017-02-10 18:43:27 Verifying last 11478 blocks at level 3
Thanks.

This is now solved with the new core update.
https://github.com/zcoinofficial/zcoin/tree/core_upgrade.
Please reopen if still an issue.


[DP-240][DP-246] Record warehouse read and query latency

DD link

Unfortunately, it is not comparable between query history in snowflake for our warehouse_read_latency and warehouse_query_latency. https://docs.snowflake.com/en/sql-reference/account-usage/query_history I wonder if they shorten the total_elapsed_time or something like that since the number is lower than what I see in spark also their snowflake connector log 🤷 for example the query time is 4 seconds here but shows less than 3s in their db.

SELECT
Query_text, start_time, end_time, total_elapsed_time, compilation_time, execution_time,
  FROM SNOWFLAKE.ACCOUNT_USAGE.QUERY_HISTORY
  where query_tag = 'dff91a884042c97f' and query_type = 'SELECT'

https://app.datadoghq.com/logs?query=%40trace_id%3A8796541c931cd376 snowflake &agg_m=count&agg_m_source=base&agg_t=count&cols=host%2Cservice%2C%40trace_id&event=AgAAAY9jwqzQOBEzuAAAAAAAAAAYAAAAAEFZOWp3c0EtQUFEaE15cThYQ0trMndBQwAAACQAAAAAMDE4ZjYzYzItZjFjMi00YWFhLWI1NmEtYTJmMGQwYzc0ODY0&fromUser=true&messageDisplay=inline&refresh_mode=sliding&storage=hot&stream_sort=desc&viz=stream&from_ts=1715279163627&to_ts=1715365563627&live=true


Select formats based on index

As a precursor for the chunk7 reader, use the same format ordering as in lsl and read in the native datatype (line 293)

Does this also do what https://github.com/sccn/xdf/pull/38 proposes? If not, it would be nice if you could integrate that into this PR.

Does this also do what sccn/xdf#38 proposes?

Yes, I didn't see that but it does everything sccn/xdf#38 does (and a tiny bit more). Plus, it's already in the correct repository :-)

OK great, that's what I wanted to know - so we can safely close https://github.com/sccn/xdf/pull/38 once this gets merged.

We really need to set up a CI service. I don't have the time to verify if these changes (especially the switch from struct.unpack to np.frombuffer) did not introduce regressions, but by just looking at the code it LGTM. @cboulay feel free to do more checks and/or merge.

I created an example file repository with a tiny (2kB) example file.
It's not a replacement for a CI, but the most basic test (python -c "import pyxdf; print(pyxdf.load_xdf('../example-files/minimal.xdf'));") produces the sample output except for the changed dtype in the output time_series:
([{'info': defaultdict(<class 'list'>, {'name': ['SendDataC'], 'type': ['EEG'], 'channel_count': ['3'], 'nominal_srate': ['10'], 'channel_format': ['int16'], 'created_at': ['50942.723319709003'], 'desc': [None], 'uid': ['xdfwriter_11_int'], 'effective_srate': 10.000000000000021}), 'footer': {'info': defaultdict(<class 'list'>, {'writer': ['LabRecorder xdfwriter'], 'first_timestamp': ['5.1'], 'last_timestamp': ['5.9'], 'sample_count': ['9'], 'clock_offsets': [defaultdict(<class 'list'>, {'offset': [defaultdict(<class 'list'>, {'time': ['50979.76'], 'value': ['-.01']}), defaultdict(<class 'list'>, {'time': ['50979.86'], 'value': ['-.02']})]})]})}, 'time_series': array([[192, 255, 238],
       [ 12,  22,  32],
       [ 13,  23,  33],
       [ 14,  24,  34],
       [ 15,  25,  35],
       [ 12,  22,  32],
       [ 13,  23,  33],
       [ 14,  24,  34],
       [ 15,  25,  35]], dtype=int16), 'time_stamps': array([5. , 5.1, 5.2, 5.3, 5.4, 5.5, 5.6, 5.7, 5.8])}, {'info': defaultdict(<class 'list'>, {'name': ['SendDataString'], 'type': ['StringMarker'], 'channel_count': ['1'], 'nominal_srate': ['10'], 'channel_format': ['string'], 'created_at': ['50942.723319709003'], 'desc': [None], 'uid': ['xdfwriter_11_int'], 'effective_srate': 10.00000000000004}), 'footer': {'info': defaultdict(<class 'list'>, {'writer': ['LabRecorder xdfwriter'], 'first_timestamp': ['5.1'], 'last_timestamp': ['5.9'], 'sample_count': ['9'], 'clock_offsets': [defaultdict(<class 'list'>, {'offset': [defaultdict(<class 'list'>, {'time': ['50979.76'], 'value': ['-.01']}), defaultdict(<class 'list'>, {'time': ['50979.86'], 'value': ['-.02']})]})]})}, 'time_series': [['<?xml version="1.0"?><info><writer>LabRecorder xdfwriter</writer><first_timestamp>5.1</first_timestamp><last_timestamp>5.9</last_timestamp><sample_count>9</sample_count><clock_offsets><offset><time>50979.76</time><value>-.01</value></offset><offset><time>50979.86</time><value>-.02</value></offset></clock_offsets></info>'], ['Hello'], ['World'], ['from'], ['LSL'], ['Hello'], ['World'], ['from'], ['LSL']], 'time_stamps': array([5.1, 5.2, 5.3, 5.4, 5.5, 5.6, 5.7, 5.8, 5.9])}], {'info': defaultdict(<class 'list'>, {'version': ['1.0']})})


rename "some other race" to "unrepresented race"

Preview Link

Click to demo updates
Description

Rename the "OTHER_STANDARD" race from "Some other race" to "Unrepresented race"
Motivation and Context

[Closes #1236]
How has this been tested?

Screenshots (if appropriate):

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Options for multi category with screenshots

Two or more races & Unrepresented ra

ce (Non-Hispanic)

Two or more races or Unrepresented race (Non-Hispanic)

Two or more races & 'Unrepresented race' (Non-Hispanic)


Offer more languages

The current version only offers an Arabic version, it would be great to have more languages (quran-json offers 10 languages at the moment).

This is a great first step

Adding the English version in https://github.com/AmmarCodes/obsidian-quran-vault/releases/tag/v1.0


Restrict user

Hi,
I have a question, It is possible to restrict only one user to apply some restriction with iptalbes, I'm thinking to configure a new ip range for this user, but not sure if I can do this, Is possible?
If not any other options to apply some ip restriction to only one user, other are right with actual default config with default network /24
Thanks

This can be achieved but is unsupported by the script.
I suggest you to read the OpenVPN documentation.


Weekly Builds?

Is your feature request related to a problem?
I don't have an environment to build Files from source.
Describe the solution
So I would love to have weekly builds.
Describe alternatives you've considered
N/A
Scope
Should? (at least I want)
Additional comment
N/A

if you join the discord you can request for insider. that will get you more updated builds.

@josh-65 I'm already in the insider :)
But it's not weekly

I want to have weekly builds like Microsoft Edge if we can...

The insider channel is already updated about once a week. Sometimes we run into blocking issues so we can't release an update, but we usually let you know when that happens.


nvm use fails to detect latest minor version when executed with major version only

Run:
nvm use 8

Fails with:
node vv8.17.0 (64-bit) is not installed or cannot be found.

Note the "double v" in the reported version.
It seems that it finds the suitable version, but add an extra 'v' to the version which fail it to use it properly.
Probably this is the culprit:
https://github.com/coreybutler/nvm-windows/blob/5803c4f8c7766c670996fbce3fc113a9b0d95050/src/node/node.go#L47
It should replace the leading 'v' in the version argument before checking the file existence which already adding a leading 'v' to the file path. (or similar fix)

Closing as a duplicate of #708.


EllipsoidTerrainProvider.requestTileGeometry wrong return type

TerrainProvider#requestTileGeometry returns a Promise<TerrainData>, but EllipsoidTerrainProvider returns the TerrainData directly (not a Promise).  This breaks sampleTerrain / sampleTerrainMostDetailed because it expects a Promise.  The net effect is that you can't use those functions if you're not sure at runtime whether the provider will be a "real" terrain provider (that has to make a server round-trip and thus returns a Promise) vs the Ellipsoid provider.
In the short term I just avoid making the call to sampleTerrain when the provider is the Ellipsoid provider, but in the long term it should be fixed.

I just noticed, I GoogleEarthEnterpriseTerrainProvider is guilty of this as well.  If the cache misses, it returns a Promise, but if the cache hits, it just returns the data directly (which would also break sampleTerrain).

Thanks @thw0rted, I've opened this PR: #6824

That fix was merged and will be included in the Cesium 1.48 release available on August 1st.  Thanks again @thw0rted!


VSCode with High Contrast mode

VSCode Version: 1.4.0
OS Version: Windows

I would like to know how have you guys implemented High Contrast color theme ? Is it a native support from Electron ?
My app is using the latest version of Electron. However, if i turn on the Windows High contrast mode my app doesn't align well.
I would appreciate if you guide here

All with CSS.

thanks


Add modified libcxxabi ebuild for other LLVM versions

Self explanatory, as of writing only an ebuild for 20230820 (https://github.com/llvm/llvm-project/commit/87a14216cd4058154fe59a4031e6fb962ebf0462) exists.

Partly finished; created copies for 20230810 & 20230803.
Left are live versions (so that 18 can finally be unmasked) and LLVM 17.


Update SwiftPM support to Swift 5

iOS related files which imports UIKit is currently excluded from SwiftPM
using conditional compilation

thanks for contributing!


Create database

I have a collection of fn files for bacteria,How do I create my reference genome as a database?

Hello @gewenlong365
Do you mean you want to create a new database containing these genomes? (E.g., to make it publicly available, or to compare genomes from other projects against this database)
Or, do you mean you want to obtain a reference database to compare your genomes against? (E.g., to classify your genomes taxonomically, or to find the closest relatives published elsewhere).
Also, do you want to use the CLI locally or do you prefer using web-based interfaces?
Best wishes,
Miguel

Thanks for reply
yes, i want to obtain a reference database to compare unknown genomes against for classification
how can i do ?
i try to use miga add,but it seems to have failed
Thanks again~

@lmrodriguezr The process is as follows:
miga new -P ./te_14 -t genomes
miga add -P ./te_14/ -t genome -i assembly  ../test/db/genome/*
miga classify_wf -l ./te_14/ -o out_test ./assembly.fasta

@lmrodriguezr hello!MiGA team~
I think I've found a good enough example to do what I need
https://miga.gitbook.io/main-1/10WEtzu06EHwzW762VwX/miga-aws-cli-projects/classify-a-genome/using-your-own-project
but Run command miga ls -P . -p
is that everything appears to be done,still have .pid files
ls daemon
d  daemon.json  maintenance  MiGA:pseudo.output  MiGA:pseudo.pid  p  status.json

miga ls -P . -p
         name  raw_reads  trimmed_reads  read_quality  trimmed_fasta  assembly  cds   essential_genes  mytaxa  mytaxa_scan  taxonomy  distances  ssu   stats
         ----  ---------  -------------  ------------  -------------  --------  ---   ---------------  ------  -----------  --------  ---------  ---   -----
P_alcaligenes  -          -              -             -              done      done  done             done    done         done      done       done  done
P_fluorescens  -          -              -             -              done      done  done             done    done         done      done       done  done
  P_mendocina  -          -              -             -              done      done  done             done    done         done      done       done  done
     P_putida  -          -              -             -              done      done  done             done    done         done      done       done  done
   P_stutzeri  -          -              -             -              done      done  done             done    done         done      done       done  done
   P_syringae  -          -              -             -              done      done  done             done    done         done      done       done  done

ok,sorry
After waiting some time, it was done
I seem to understand that this project can't create its own reference genome library, right?
it would be great if it had that feature
thanks!

Hello @gewenlong365 ,
If you want a pre-indexed general-purpose reference database for classification, you can use TypeMat_Lite. To download TypeMat_Lite, you can use:
miga download -v

That database includes all available genome representatives for validly published species. If you prefer the GTDB taxonomy instead, you can also obtain the GTDB_Anchor_Lite with the same action:
miga download -n GTDB_Anchor_Lite -v

Or, you can see all the available databases:
miga download --list


.download() throwing malformed uri error (via empty data uri - "data:")

I've been troubleshooting a download error for a while and could use any nuggets of information you can provide.
I am trying to download an excel file. We are calling .click() on a button to download, and then immediately calling .download(), but on occasion, download will throw an error from the data-uri-to-buffer package because the uri passed is 'data:', essentially empty. I have squashed the error, and download eventually writes 'data:' to the file using the provided path, as expected.
This seems to happen randomly. Theories: the site we are interacting with has a heavier load than at other times, causing download to miss the file in some way; we are utilizing cookies/sessions in a way that causes the site we are interacting with to not provide the file; the page loaded slowly, causing a timing issue.

Additional information: when multiple xls files are downloaded during same horseman process, they seem to be overwriting one another on occasion.


Solving issue #1782 -  solution to 3171. Find Subarray With Bitwise OR Closest to K

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
#1782
Description
[Please include a brief description of the changes or features added]
This description is for the solution of Find Subarray With Bitwise OR Closest to K
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Security enhancement
[x] Other (specify): Leetcode solution_______

Screenshots / Videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have followed the code style guidelines of this project.
[x] I have checked for any existing open issues that my pull request may address.
[x] I have ensured that my changes do not break any existing functionality.
[x] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[x] I have read the resources for guidance listed below.
[x] I have followed security best practices in my code changes.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Please read the following resources before submitting your contribution:

[x] Code Harbor Hub Community Features
[x] Docusaurus Documentation
[x] React.js Documentation
[x] Markdown Guide
[x] MDX Documentation
[x] Mermaid Documentation

Resources for DSA Features on Our Site:

[x] Tabs: Switch Languages
[x] Line Numbering
[x] Math Equations

Contribution Guidelines
Thank you for considering contributing to our project! To ensure smooth collaboration and effective contribution management, please adhere to the following guidelines:
Issue Creation

Limit on Issues:

Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.

Contribution Levels

Basic Contributions:

This project is primarily focused on documentation. Most of the setup has been completed, so contributors will generally need to work on basic code tasks, such as writing tests.
For these tasks, issues will be assigned the level1 label.

Acknowledging Hard Work:

If a contributor puts in significant effort on a task, the issue will be upgraded to level2. This is our way of recognizing and appreciating extra effort.

Feature Additions and Component Work:

Contributors working on new features or components using JSX/TSX will be assigned a level based on the complexity and quality of their work.
The more complex and valuable the contribution, the higher the level assigned.

Level Definitions

Level1:

Tasks are straightforward, such as fixing minor bugs, writing tests, or making simple documentation updates.

Level2:

Tasks require more effort, such as addressing complex bugs, improving existing features, or making substantial documentation improvements.

Level3:

Tasks are highly complex and involve significant new feature development, major refactoring, or extensive contributions to the project’s core components.

We look forward to your contributions and appreciate your effort in helping us improve the project!

@Ajay-Dhangar
Kindly review and merge


[Visual Studio Code - Project Manager] Unable to load remote project fold when there's "." in the folder path

Extension
https://www.raycast.com/MarkusLanger/vscode-project-manager
Description
When connecting to remote project, if the folder path contains ".", it will only connect to the ssh server but will not open the project fold. This is a bug of vscode CLI but can be easily solved by just adding a "/" at the end of path.
Steps To Reproduce
No response
Current Behaviour
No response
Expected Behaviour
No response

@sunrisewestern Could you check if the code in the branch fixes your problem and if it also works for projects without a "." in the path
https://github.com/MarkusLanger/extensions/tree/5411

@sunrisewestern Could you check if the code in the branch fixes your problem and if it also works for projects without a "." in the path https://github.com/MarkusLanger/extensions/tree/5411

Yes. it fixed my problem. And works on normal projects without dot.

PR created #5413

@raycastbot close this issue

Strange, let me check why it didn't work

@MarkusLanger @pernielsentikaer sadly this breaks search in vs code remote over ssh,
also my projects show up double now in vs code recently used. See screenshot:


Before building, first check for valid connection to App Store Connect

Describe the solution you'd like
It would be great that, before building, testing and releasing the build, to check if the connection to App Store Connect is still active. I've already had several times that the login to App Store Connect is expired for some unknown reason. You only get that warning at the end of a build, which is in my case 15-20 minutes later.
It's better to know this upfront, so you can re-connect your account before building. This could possibly save an extra 15-20 minutes in my use case, as I don't have to run the whole build twice.
Describe alternatives you've considered
The only alternative would be to manually check on the Distribute > Stores page if the connection is still active or expired.
Additional context
This problem usually happens in step 23 of the build:
2019-08-16T09:37:54.4007730Z ##[section]Starting: Create distribution
2019-08-16T09:37:54.4012570Z ==============================================================================
2019-08-16T09:37:54.4012700Z Task         : App Center distribute
2019-08-16T09:37:54.4012840Z Description  : Distribute app builds to testers and ***s via Visual Studio App Center
2019-08-16T09:37:54.4012940Z Version      : 3.156.0
2019-08-16T09:37:54.4013010Z Author       : Microsoft Corporation
2019-08-16T09:37:54.4013110Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/deploy/app-center-distribute
2019-08-16T09:37:54.4013200Z ==============================================================================
2019-08-16T09:44:50.7225470Z ##[error]{"code":"two_factor_authentication_error","message":"Error: Failed to publish to store. Error: apple connection is not valid for ***: XXXX"}
http response code: 400
2019-08-16T09:44:50.7243110Z ##[section]Finishing: Create distribution

This is unrelated to the actual feature request but fastlane, an open-source mobile build automation tool, handles this really well by managing your App Store Connect session and automatically logging back in if it's broken.

@jvandenaardweg Thank you for the feature request, this is good feedback and I hope we get some traction on it so we can implement it.

@jvandenaardweg , how do I renew the token?

@Sutil Select your app in AppCenter, then: on the left sidebar: Distribute -> Stores. Then you should see a red banner like the one below, which allows you to sign in again to the app store. After doing that, you can build again.

@jvandenaardweg Thanks! Works fine!

Or at least the build shouldn't completely fail. Just because the distribution step fails doesn't mean that there's anything wrong with the app package. I could reconnect my store and publish it in 30 seconds, but instead I have to repeat the 30 minutes build process...

Or at least the build shouldn't completely fail. Just because the distribution step fails doesn't mean that there's anything wrong with the app package. I could reconnect my store and publish it in 30 seconds, but instead I have to repeat the 30 minutes build process...

I just had this error today and would love to see a solution similar to the one suggested.

Maybe also send an email that the auth token is going to expire, or at least that it has done?

Hello guys, there's any way to don't need re-authenticated every month?

I second this, it shouldn't be very hard to check the connection and it would save a bunch of time. I can imagine this would be great for the resources of the build servers too since they don't need to needlessly build something multiple times.

Still getting this error. after 29 min the build is finished and then it crashes with this error, very unnecessary. Alternatively if it could flag the build as successful, then we can manually re-publish it after fixing the issue without having to run another full build.

🤖

We're maintaining an app together with another company. Due to security reasons, we don't have access to their account with 2FA. It happens often that the person triggering the build does not have permissions to update the connection. To top it off, this person will also not get an email saying the build has failed.
It would be great if at least this banner is visible somewhere on the pages used to trigger a build.

No, this is still an outstanding and frustrating issue.

Can confirm this is still an issue.

This hit our team just the other day.

same here.. is there away to solve it ?

@Oddj0b

would love to see any of the solutions above! 👀

And 3 months later... Still the same.

Still happens! It's a constant issue, our team is very small so everyone has a bunch of issues to solve. Having one issue not being tested by this is very frustating.

This issue is still here in 2024.

We have actually moved away from appcenter since they don't seem to care. We're using bitrise now.
Good luck everyone!


refactor: Alternative Effect.Service suggestion

since we combine tags and default layers, we can go one step further and grab and pass the dependent services to the constructors.
NOTE: api is not yet finished, I would suggest we have overloads without dependencies as second argument.
at least I couldn't make it work to infer the dependencies and provide them within the same options object.

closing as not planned