feat: add selection rules per upstream

Allow influencing how upstreams are selected to serve (or not) traffic.
Each upstream can have their own set of rules.
There are multiple rule types to combine as per use-case (excludeByDefault, includeWhenMetrics, etc).
Rules are evaluated from top to bottom; the final decision is based on the last rule whose condition is met

Solves #51
Examples
Make an upstream a fallback to use only if all other upstreams are down
- upstreams:
  - id: my-cheap-rpc-A
  - id: my-cheap-rpc-B
  - id: my-expensive-rpc-C
    selectionRules:
    - excludeByDefault:
        forMethods: '*'
    - includeWhenOtherUpstreams:
        forMethods: '*'
        activesAreLessThan: 1

Exclude an upstream if too much block lag from chain head compared to others
- upstreams:
    - id: my-good-rpc-A
    - id: my-good-rpc-B
    - id: my-bad-rpc-C
      selectionRules:
      - excludeWhenMetrics:
          forMethods: "*"
          evmBlockLagHigherThan: 100

so some comments here of "nice-to-haves"

for a specific upstream thats particularly "mixed" ie with archival/full nodes, and maybe some public nodes with rate limits, it would be nice if an archival/trace/debug request was routed to a node full node and returns an empty response, it gets "cordoned" for that method in the future, and will never be included to serve those heavy types of calls. (this would be the automated way)
second best would be to to mark each upstream as either full/archive , that way we can bundle all upstreams to one network, and only archival/debug/trace calls would be served by the corresponding upstreams (ie archive=true)
following this approach, maybe we can configure a block range limit per each upstream, so this would allow eth_getLogs on full nodes, provided its within that nodes block range


add josh instructions

This explains how to sync Miri changes to and from rustc.

I've read over this and it makes sense, though of course I've never tried to actually use josh.

@bors r+

:pushpin: Commit 11c593f4a5f9c36daa58b497516f24a3e17041a8 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 11c593f4a5f9c36daa58b497516f24a3e17041a8 with merge a74436e46fcc1dc020e9ad9aaa96e47fc236a088...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing a74436e46fcc1dc020e9ad9aaa96e47fc236a088 to master...


KAFKA-13654: Extend KStream process with new Processor API

KIP-820 implementation.
Changes:

Implement changes proposed in the KIP.
Adapt existing methods to implement FixedKeyProcessor instead of Processor, to guarantee fixed keys.

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thank you, @vvcephei !! I have applied most suggestions, let me know your thoughts  🙌🏽


Upgrade OpenRewrite's rewrite-java-17 8.40.0 -> 8.40.2

This PR contains the following updates:

Package
Type
Update
Change

org.openrewrite:rewrite-java-17
compile
patch
8.40.0 -> 8.40.2

Release Notes

openrewrite/rewrite (org.openrewrite:rewrite-java-17)
v8.40.2: 8.40.2
Compare Source
What's Changed

Keep multiple whitelines and same comments from the gitignore file by @Jenson3210 in https://github.com/openrewrite/rewrite/pull/4659
Feature - YAML search - find properties with a specific scalar value by @aamotharald in https://github.com/openrewrite/rewrite/pull/4657
Upstream RemoveMethodInvocationsVisitor by @nmck257 in https://github.com/openrewrite/rewrite/pull/4639
Feature/remove unused properties by @nmck257 in https://github.com/openrewrite/rewrite/pull/4636
Fixing the quoteEscaping test case for proper quote character escaping by @mccartney in https://github.com/openrewrite/rewrite/pull/4665
Account for a second common resource filtering pattern by @timtebeek in https://github.com/openrewrite/rewrite/pull/4666

New Contributors

@aamotharald made their first contribution in https://github.com/openrewrite/rewrite/pull/4657

Full Changelog: https://github.com/openrewrite/rewrite/compare/v8.40.1...v8.40.2
v8.40.1: 8.40.1
Compare Source
What's Changed

Catch non-semver compatible exception in case of version update by @Jenson3210 in https://github.com/openrewrite/rewrite/pull/4656
refactor: Update Gradle wrapper by @shanman190 in https://github.com/openrewrite/rewrite/pull/4662

Full Changelog: https://github.com/openrewrite/rewrite/compare/v8.40.0...v8.40.1

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade org.openrewrite:rewrite-java-17 8.40.0 -> 8.40.1

See:
- https://github.com/openrewrite/rewrite/releases/tag/v8.40.1

This PR can be closed once #1419 is merged.


Add default conformance to syntax collection

https://github.com/apple/swift/pull/40120
Added default conformance to syntax collection as discussed in #331

I saw that!
I have rebased 🚀

apple/swift#40120
@swift-ci Please test

apple/swift#40120
@swift-ci Please test


Update to reflex-platform staging branch with nixpkgs-21.05

Depends on https://github.com/reflex-frp/reflex-platform/pull/768. This can be taken out of draft state when that gets merged, which depends on the resolution of these issues.
I have:

[x] Based work on latest develop branch
[ ] Followed the contribution guide
[ ] Looked for lint in my changes with hlint . (lint found code you did not write can be left alone)
[ ] Run the test suite: $(nix-build -A selftest --no-out-link)
[ ] Updated the changelog
[ ] (Optional) Run CI tests locally: nix-build release.nix -A build.x86_64-linux --no-out-link (or x86_64-darwin on macOS)

As of https://github.com/obsidiansystems/obelisk/pull/972/commits/fe415ecb22845611f48b39eef2e4984a0375ac59 ob run works on this branch

Closing this as it has been merged as part of https://github.com/obsidiansystems/obelisk/pull/1005


Logic for "Resolve the conflicted Timeline Entry"

Description
The UI is done at https://github.com/toggl/toggldesktop/pull/3050
This ticket is connecting with the last criteria in https://github.com/toggl/toggldesktop/issues/2892
Allow solving overlaps. Clicking on overlap shows a context menu with 3 choices:
- change first entry stop time
- change last entry start time
- create a new entry of overlap
(The texts should be improved, they should be shorter)

We need to elaborate more. Examples are welcome 👍
💻 Environment
Platform: macOS

Basic Example:
TE 1: 06:45:10 - 06:55:10
TE 2: 06:55:00 - 07:00:00
If user picks "Change first entry stop time" ->
TE 1: 06:45:10 - 06:55:00
TE 2: 06:55:00 - 07:00:00
Else if user picks "Change last entry start time" ->
TE 1: 06:45:10 - 06:55:10
TE 2: 06:55:10 - 07:00:00


protein position and protein change are inconsistent for frameshift ins/del

http://www.genomenexus.org/annotation/6:g.26020799Adel

This happens to insertions too.

It looks like it's from vep: https://grch37.rest.ensembl.org/vep/human/hgvs/6:g.26020799Adel?content-type=application/json&hgvs=1

It's a known bug and they will fix it but only in grch38.

Maybe we should fix it in genome nexus?


add zookeeper RUOK check

just sends an RUOK to the given TCP port (which is assumed to be running ZooKeeper) and confirms that an IMOK comes back.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Contest's tool resets execution sometimes

Description
Running contest's tool sometimes resets several class generation executions. It is not very clear what exactly happens but looks like that instrumented process is dying and sends through log some "null" states to the runner. The latest processes these nulls as a command to stop and rerun the contest tool itself.
To Reproduce
Run the contest tool from the docker.
Expected behavior
Every class generation process should be processed without restarting.
Actual behavior
Some generations are restarted.
Additional context
ContestKt tool.

Possible cause: RD default log writes to stdout. As there was multuple entry points - #1741, may be in some places logs for RD was not configured correctly.  {
                }

            }
        }
    } )


HTML Drag and Drop

[X ] Feature request
Could HTML drag and drop be added to the web interface when adding attachments.
https://developer.mozilla.org/en-US/docs/Web/API/HTML_Drag_and_Drop_API/File_drag_and_drop
Ideally with the option of uploading multiple attachments / photos at once.

Dropzone appears to be a popular way of doing this, which just needs an AJAX endpoint to send the files to.

Thank you for your interest in NetBox. It looks like your feature request is missing some information. Per the contributing guidelines, please ensure that you have provided all of the following:

A detailed description of the proposed functionality
A use case for the feature; who would use it and what value it would add to NetBox
A rough description of changes necessary to the database schema (if applicable)
Any third-party libraries or other resources which would be involved

Please update the issue so that it meets all of  these requirements. If no response is received within a week, this issue will be closed. Thanks!


How can I override the  tag for my page?

Hi, may I know is there a way I can add some more <script> tags inside the <head> section of the default HTML>?
<!doctype html>
  | <html lang="">
  | <head>
  | <meta charset="UTF-8">
  | <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
  | <title>Example</title>
  | <meta name="description" content="Foo bar">
  | <link rel="icon" type="image/png" sizes="32x32"
 ...
  | <link rel="stylesheet"<EMAIL_ADDRESS>  | <link rel="stylesheet"<EMAIL_ADDRESS>  | </head>

<body>
  | <div class="pywebio">

If you want to include your javascript file, you can use the RequireJs built-in pywebio.
If you only want to execute some js code, just use pywebio.session.run_js() function.

What I wanted to do is to copy and paste the AdSense code in between the  tags of my ### site.
This is the example code snippet that I want to include:
<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=ca-pub-xxxxx"
     crossorigin="anonymous"></script>

Does this work?
run_js("""
$('head').append('<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=ca-pub-xxxxx"
     crossorigin="anonymous"></script>')
"""

Thanks for the quick reply man.
Just tried it out, it doesn't append the script tag inside , not even anywhere else hmm

Sorry, a redundant linebreak was incorrectly added to the code, the correct code is:
run_js("""
$('head').append('<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=ca-pub-xxxxx" crossorigin="anonymous"></script>')
"""

The above code does insert the script tag to head, you need use chrome's inspect panel to check the result rather than "View page source":

Oh yea, I see it now on the inspect panel. I kept looking from the page source. Thank you so much for your help. Let's see if Adsense works 🤞
Closing the issue now. Thanks.

If you want to include your javascript file, you can use the RequireJs built-in pywebio.
do you have some examples or documentation about this?
I'm not very familiar with frontend  and JS and need some clue.
To be specific, I'm looking for a way to have templates or themes for my PuWebIO website. (does not matter Bootstrap or Boilerplate JS)
Thanks


Cors Not Working: Apache and DRF

I installed django-cors-headers==1.0.0
I'm hosting just djangorestframework at localhost:8000
The webcode is hosted in localhost
My settings.py is
THIRD_PARTY_APPS = (
'corsheaders',
'bootstrap_admin',
'adminactions',
'rest_framework',
'rest_framework_swagger',
'taggit',
)
Also installed
MIDDLEWARE_CLASSES = (
'corsheaders.middleware.CorsMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'ccDjango.middleware.SetLastVisitMiddleware',
'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
)

CORS_ORIGIN_ALLOW_ALL = True
Anyone else having the same issue?

Also i've been trying to log through it and seems that only the request is being processed, but not the response

Can you perhaps paste some of those logs to help with the debugging?

@ottoyiu :
I got the settings like @thebenwaters , and I call my login url and pass the username and password arguements. But it returns a 403  "NetworkError: 403 FORBIDDEN - http://sso.acttao.com:8888/zgxcw/login/".
    My login view returns a url at first.   ------return redirect('zg_login')
    After, I return a response and still returns the 403.

    the CORS url must return a response?
    what might causes my issue?
    Thanks a lot.  Need help urgently.

It does not need to return a response object.
the reason is the shot of settings .
The settings of mine is below.
CORS_ORIGIN_ALLOW_ALL = True
CORS_ALLOW_CREDENTIALS = True
CORS_ORIGIN_WHITELIST = (
'zgxcw.acttao.com:8002',
'hostname.example.com'
)
At first I lost CORS_ALLOW_CREDENTIALS = True.
and I checked the source code middleware.py
line 57:
response[ACCESS_CONTROL_ALLOW_ORIGIN] = "*" if (
settings.CORS_ORIGIN_ALLOW_ALL and
not settings.CORS_ALLOW_CREDENTIALS) else origin

403 again!
I didnot find the real reason of 403.

I hope can help someone with my fail log.
Now, I pass the arguements (username and passwd) from a site to server site,
and called the login function.
But did not login successfully.
I thought might be the cookie did not set successfully.
If anyone has a suggestion is welcome.

There are a plethora of reasons you might be getting a 403. For me, the fix was to set the withCredentials XHR option to true. A jquery example:
$.ajax({
   url: a_cross_domain_url,
   xhrFields: {
      withCredentials: true
   }
});

@brad
Thanks for your suggestion.
I did my project whitout cross domain this time .
I would like to check it with my spare time.

I'm having the same problem! Any help would be appreciated!

Closing due to inactivity


Update CSV Export

Add following information to the csv export:

Index patient (first and last name)
Date & Place (#301)
Context
Test result

Closed in favor of #311


When Swift client use FSD(VATSIM) protocol will cause error

There are no exceptional situations with pyfsd except for error reporting. The client will indicate a connection closure.

can't reproduce with euroscope


1Password: make primary and secondary actions configurable

Description
Fixes #5083
Added two dropdowns to the extension's configuration to allow users to customize the primary and secondary actions for Login items.
The default behaviour is unchanged: "Open in 1Password" is the default primary action, and "Open in Browser" is the default secondary action.
Added keyboard shortcuts to the "Open in 1Password" and "Open in Browser" actions, for use when they are not the primary/secondary actions.
For "Open in Browser", the existing behaviour is maintained: the action is only available if the item has a primary URL.
For non-Login items, the behaviour is unchanged: the only available action is "Open in 1Password".
Note: the new configuration is only applicable when using 1Password v8.
Also:

refactored the action panel into a separate component
made the Item.url field optional to match the format of the 1Password CLIs' JSON output

Screencast
Command configuration:

Action panel with a customised primary action:

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Not sure if I should update CHANGELOG.md as part of the PR?

Is this ready to be merged @khasbilegt

LGTM!


Allow compile-time addressof expression applied to objects with static storage

Currently the addressof operator is forbidden in compile time expressions, and static slice objects have no way of being produced in a compile-time expressions.

[x] Alter struct address to have a base and offset member, the base being what currently exists in the struct and the offset being the number of bytes forward-offset from that base.
[x] Allow taking pointer expressions via the addressof operator in compile-time evaluation contexts.
[ ] (OPTIONAL) Allow dereferencing pointer expressions via the dereference operator in compile-time evaluation contexts.

This may require keeping a global map of static addresses and associated values in the compilation context. I believe we already store symbols this way so we might just be able to piggyback off that list.

Once you get into the realm of dereferencing at compile-time we may run into the situation where a user dereferences x bytes into object of size bytes where x is partially into the object or sub-object. For example dereferencing two bytes into a u32. Unless we want to define layout and basically have a virtual machine within the compilation context it might be okay to allow  taking the address of an object, but not dereferencing the resulting pointer. Indexing should work fine since slices and arrays all have struct value sub-objects which can be copied just fine.

Closed with bedb3129441267e5f36ba4b7aca621d0ada63696 and 8cdc587107cb587e3078cca144c2ac0ca9baf3bd.
The work from in the above commits revealed a lot of hidden bugs in code generation, so while this particular issue is "resolved" it is likely that more work will be required to fix misc. bugs not caught by the test suite.


Please use spectraProcessing.xAbsolute

Please ensure that we have a typedArray there otherwise sort would not work (it sorts as string).
And actually it seems that this is not a typed array !
https://github.com/cheminfo/nmr-displayer/blob/b20174938d4b88c0b7cfafe2264ffbec44b9f88f/src/data/data1d/autoRangesDetection.js#L20-L22
In the full project we need everywhere typed array for the data.

Actually there is the method xAbsoluteMedian and I used it.
The options for range picking were wrong and the result was not taking into account the frequency. I fixed it.


Ability to add annotations to the code

Is your feature request related to a problem? Please describe.
I would like to be able to add annotation to the source code (e.g. such as in a code review). Today, this is possible to change the color of a line, what I would like to be able is to add components/elements to the code (custom annotations).
Describe the solution you'd like
Have a function that, as for lineProps, let me specify a function that would return a component to attach to the formatted code.
For this function to work, wrapLines has to be true.
Describe alternatives you've considered
I believe it might be possible to hack the CSS to insert elements but that would come to limitations as to what CSS can do. Another way to do it is to modify the DOM using the function in lineProps but that would not be ideal/clean as well.
Additional context
Add any other context or screenshots about the feature request here.

One example of how it would look: https://codepen.io/anon/pen/VmQpYz

In case annotations would be something that the react-syntax-highlighter team would have interest in, I would like to propose another type of annotations:
Annotations for referencing certain tokens or lines of the code for teaching purposes. Eg. HTML and CSS on MDN or other platforms:

Source: my issue in code-surfer: https://github.com/pomber/code-surfer/pull/51

@simmerer would there be interest in this? Custom elements / React components on certain lines and pointing at certain parts of the code would be amazing!
If the react-syntax-highlighter team wouldn't be able to do this, I may even be able to implement this myself if I can get the time free in the next weeks and months.
I guess one place to start looking in createLineElement in src/highlight.js#L95-L129?
https://github.com/react-syntax-highlighter/react-syntax-highlighter/blob/efc3f7b7537d1729193b7a472067bcbe6cbecaf1/src/highlight.js#L95-L129

By the way, I hacked in a rudimentary version of this to code-surfer in https://github.com/pomber/code-surfer/pull/115, creates the ability to add code annotations (eg. to HTML, CSS, and JavaScript) like this:


Make controller's sync period configurable

Fixes #3915
Adds the sync-period flag (defaulted to 1s), that allows modification of the frequency at which the controller will
pull items off the workqueue for processing.
Adds --sync-period flag to modify the period of time between items being pulled from the controller's workqueue.

/kind feature
Signed-off-by: David Bond<EMAIL_ADDRESS>

/test all

I'm not convinced whether this should be a command line configurable option because I havent seen any good reasons why someone would want to change this timeout,

I'd echo this statement - as a user, when would I/should I configure this?


Make examples/dwarfdump.rs's output format match the "real" dwarfdump

Not sure if it belongs in this issue, but one possibility is that if we make the output an exact clone, then we could reuse some of the dwarfdump regression tests.

I've made and landed an initial mini clone.
It doesn't match the format of the output, but this is a swell idea. We could also compare performance between the two.

I'm starting to work on making the .debug_info format match dwarfdump.
For performance comparisons, I did some basic benchmarks at https://github.com/philipc/dwarf-bench, but I couldn't avoid IO for the libdwarf/elfutils benchmarks, so it was hard to get anything reliable and I'm not going to pursue that further currently. The benchmarks did show that gimli is already much faster (that's including IO in the gimli test too). I also think that both the libdwarf and elfutils APIs are much worse, at least for DIE iteration.

The benchmarks did show that gimli is already much faster (that's including IO in the gimli test too).

\o/

I also think that both the libdwarf and elfutils APIs are much worse, at least for DIE iteration.

\o/

We've been diverging from dwarfdump's output. While matching it was useful for early validation, I don't think it's as useful anymore. If we were going to match another tool's output, then I think readelf or llvm-dwarfdump would be better candidates.


this.random is not a function on 'node_modules\chance\chance.js:283:32'

Hi, when I am using
chance.prototype.string
I am getting below error-
this.random is not a function on 'node_modules\chance\chance.js:283:32'
which is this line in chance.js-
return Math.floor(this.random() * (options.max - options.min + 1) + options.min);
Although if I update the file and instead of this.random(), I call Math.random() it works fine.
Here is the function call that I made -
chance.prototype.string({length: span, pool: '123456789'});
I am a newbie in JS and trying to understand a lot of stuff in Chance :)
Can you please share your thoughts if I need to change something at my end.
Thanks in advance

Hi,
It seems that you are not loading the chance library correctly. you should use it as below:
var Chance = require('chance');
// Instantiate Chance so it can be used
var chance = new Chance();
// Use Chance here.
var my_random_string = chance.string();
and then there is no error!

for me, it happens when we use chance in jest test file, thethis object  is jest


Cannot get nav item to appear

Hello - testing out search space, and for whatever reason I cannot get the nav item to appear.
I created a page, and updated its url to "/search-space" but, it won't appear.
Any idea what I might be doing wrong? Do I need to put anything in the body of the page? Or, do I need a page at all?

Hi Rick, thanks for checking out my plug-in. 🙏 Search Space is automatically available after installation at the URL https://cogley.jp/search-space/. But it will not add itself to your blog's navigation. It's up to you to link to the page however you want.
How to add pages to the navigation in Micro.blog is described over at the help center. In short, create a new page and paste the URL in the page's content.

Ah right, I didn't have the fqdn in there, just the path. Thanks & donated!


IDEA 15 Preview

Can it be bumped so that we can try IDEA 15 Preview without breaking the project structure?

:+1:

Would be great ! :+1:

Yeah ! :+1:

+1

+1
WebStorm 11

+1

+1

@lylemoffitt so glad I came here from the notification of your insightful +1. Would have missed it if it weren't for github's amazing notification system.

So looks like IntelliJ Idea now has another new major version. Also looks like they're now just version by the year, as version 2016 was just released. Not sure if the issue name needs to get updated to reflect that.

What's the status on this?

Hi guys, instead of waiting for someone to resume the development of this plugin I made it by myself.
I worked around the deprecated calls by rewriting those snippet of codes that were using them.
I just make it build against my IntelliJIDEA Ultimate 2016.2 ( IU-145 , it ahould be IU-162 but this is the version that I have currently installed.. weired ... )
enjoy: https://github.com/tux-mind/CppTools/releases/tag/v145-atomic.bomb
:blush:

@tux-mind Is there a practical way to get this working?

@nicity What is your plan for this project?

@alexisvincent I'm working on it right now, it looks like that the generated zip it's a zipball of the sources instead of the compiled stuff.
My plans are to get this working because I don't want another IDE ( CLion ) when I already have a full licensed IU ( thanks to my university ) installed.
I need this plugin to work on a project of one of my courses.

@tux-mind Same here, really dislike having to maintain a separate config for CLion. Really appreciate the time you're taking to work on this. The intellij community is going to love you xD

Thank you @alexisvincent :blush:
I just make it works!
The way this plugin works it's a bit weired, you have to use it like a tool rather than a Language....
Anyway i didn't make another version singe the code is unchanged, I just updated the zip with missing stuff from previous releases and now it works like a charm :stuck_out_tongue_winking_eye:
https://github.com/tux-mind/CppTools/releases/tag/v145-atomic.bomb

@tux-mind This is awesome man. Would be really cool if you put it on the repo's. How exactly do i use it though, it fails on my side

Oh, hold on it seems to be working. Will verify in a moment

Whoop whoop 👍 Still having issues getting GOTO's working and stuff but Ill report back

@tux-mind Does this properly work for you? I don't get any intelligent IDE support


Exporting CPGs

Is your feature request related to a problem? Please describe.
I want to use joern to extract cpgs for input into a machine learning model but I'm finding it difficult to extract and keep track of all the snippets.
Describe the solution you'd like
Is it possible to parse a snippet and then write out the cpg to a singular file. And is it possible to parse the graph into a tempfile or stdout.

Have you tried the steps in the documentation? https://docs.joern.io/export/
Basically you can do the following steps (which can also be scripted)
Snippet:
~/b/j/joern-cli> cat /tmp/returntest/test.c
int func_with_multiple_returns (int x) {
  if (x > 10) {
    return 0;
  } else {
    return 1;
  }
}

Generating CPG for the snippet with CPG name :
test@device ~/b/j/joern-cli> ./joern-parse /tmp/returntest/ -o returntest.bin.zip
Parsing code at: /tmp/returntest/ - language: `NEWC`
[+] Running language frontend
[...]
[+] Applying default overlays
Successfully wrote graph to: /home/test/bin/joern/joern-cli/returntest.bin.zip
To load the graph, type `joern /home/test/bin/joern/joern-cli/returntest.bin.zip`

Loading the CPG for the snippet only:
 ~/b/j/joern-cli> ./joern
[...]
Version: 2.0.25
Type `help` to begin

joern> importCpg("/home/test/bin/joern/joern-cli/returntest.bin.zip")
[...]
val res0: Option[io.shiftleft.codepropertygraph.Cpg] = Some(value = Cpg (Graph [45 nodes]))

joern> cpg.method.name("func.*").dump
val res1: List[String] = List(
  """int func_with_multiple_returns (int x) { /* <=== func_with_multiple_returns */
  if (x > 10) {
    return 0;
  } else {
    return 1;
  }
}
"""
)


Simple HelloWorld Function in C# does not get deployed (no errors, deployment shows "completed" message)

Hi, I'm using VSCode 1.36.1 on MacOS, .NET Core SDK 2.2.401 (only SDK and runtime installed in my machine), and the Azure Functions Extension v 0.17.1. I create a simple function using the extension, in C# and with HTTP Binding. I try it locally and everything runs smoothly. I then tried to deploy this to my Azure subscription (both creating a function app from the extension and selecting an existing function app, always running in functions v2). The compilation and deployments seem to finish correctly, but when I go the portal I can't find the function deployed.
Everything I'm using is a "vainilla" as it gets, and followed this tutorial:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-develop-vs-code#enable-publishing-with-advanced-create-options

@argenacho can you copy the output of the deploy logs here?

Hi, thanks for reaching out and sorry for the delay. I've updated the functions extension to the latest version, and the error persists. Here are the output logs:
Running command: "git init"...

Deployment to "whcampus-test-fcn" completed.

@argenacho can you copy the .vscode/settings.json file in the root of your repo? The default azureFunctions.deploySubpath setting should be something like bin/Release/netcoreapp2.1/publish, but it appears you're deploying the root of the workspace instead.
You can run "Initialize Project for use with VS Code" from the command palette (view -> Command palette) to change settings back to their defaults

Hi, here are the file contents below. Looks pretty much like what you said:
{
"azureFunctions.deploySubpath": "bin/Release/netcoreapp2.1/publish",
"azureFunctions.projectLanguage": "C#",
"azureFunctions.projectRuntime": "~2",
"debug.internalConsoleOptions": "neverOpen",
"azureFunctions.preDeployTask": "publish"
}
I ran the  "Initialize Project for use with VS Code" and after that I was able to deploy correctly, though no visible changes appeared in my project.
I'm curious on why this didn't work out of the box. I'm going to create a new function from scratch and give it another try.
One more question: when I choose to deploy the function, I have to browse the folder of the function I want to deploy. What folder is that? root folder of the function project, or /bin/release/ ....?
Thanks again for looking at this!

One more question: when I choose to deploy the function, I have to browse the folder of the function I want to deploy. What folder is that? root folder of the function project, or /bin/release/ ....?

If deploySubpath is set, you shouldn't be asked for a folder and it should just deploy that folder


ci: stop testing against NodeJS v10, v12

BREAKING CHANGE: Drop support for NodeJS v10, v12

not needed, only types


Improve influxdb/client readme

This improves the readme for the go client. I added a short intro, a basic example of how to write a series to the db, and a few links along with an exercise for the reader(log system stats).
Closes #1225

@corylanou -- are we going to re-work the README.md file as part of your client overhaul? If so, perhaps this PR is no longer valid?

@otoolep I wasn't planning on re-working the readme yet.  We have to add in the UDP endpoint in a future PR, and I don't know if that will change anything significantly or not.
For sure the readme is probably going to get completely re-written as the client has completely changed.

OK, the client is changing completely, so I will close this as unmerged.
@epelc -- thanks for your contribution on this, but with the advent of 0.9.0, the text will not actually be valid. Please feel free to suggest improvements to the upcoming text however.


Add the let* monadic notation

Related issue: #95
Inspired by the notation introduced by OCaml 4.08, we propose to
introduce a new notation following the same principle.
    let* x := p in q

is strictly equivalent to
    x <- p;; q

Sorry, I misunderstood your comment on the issue.
I was not sure how to proceed, so I propose two approach:

Either always load _ <- _ ;; _, and load let* _ := _ in _ if desired (second commit)
Or load either one or the other (the third commit)

I personally prefer the latter, and it does not break the previous interface afaict. What do you think?

Thanks! The latter with MonadBaseNotation looks better. I'll try to figure out how to make CI happy.

The failure with the CI was due to a commit I had added which was breaking old Coq version (declaring the scope). I have removed it and now it appears that it works fine.
I should probably squash the two commits together if you prefer the approach with the base module (I do too! (: ). I think it would probably be wise to add some kind of test case too, right?

Yes we could add some Notation.v under /examples.

I think we are good to go now!


Double "period" (..) in message for System.InvalidOperationException

Not sure if this is classified as a bug, since it does not really affect functionality.
Steps to reproduce
try {Start-Process -FilePath "bogus.exe"} catch {Write-Host $PSItem.Exception.Message}

Expected behavior
(Only one period at end of sentence)
This command cannot be run due to the error: The system cannot find the file specified.

Actual behavior
(Two periods at end of sentence)
This command cannot be run due to the error: The system cannot find the file specified..

Environment data

Same results for both PS 7.0.0 and 7.1.0-preview.2 - I did not try preivew.1

Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Name                           Value
----                           -----
PSVersion                      7.1.0-preview.2
PSEdition                      Core
GitCommitId                    7.1.0-preview.2
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Most likely a typo in the resx file that string is being pulled from. Should be a fairly simple fix, thanks for reporting it! 😊

The template is "This command cannot be run due to the error: {0}."
.Net message "The system cannot find the file specified."
After substitution we get two dots at end.
We could add single quotes...

I wonder if we can simply remove the period from the template string. A majority of error messages that it receives will have periods at the end anyway, no?

I don't see why removing the period from the template would break things. You are right that the majority of error messages are properly formatted, and really, I for one won't notice a missing period now and then.

All template strings in ProcessResources.resx end with dot.

@iSazonov why can't we trim it...
ex.Message.TrimEnd('.')

As @vexx32 said we can remove final dot at all in the resx file.

Thank you.


remove anonymous function

WHY
BEFORE - What was wrong? What was happening before this PR?
Using the anonymous class to extend the request class was leading to problems when parallel testing. https://github.com/Laravel-Backpack/CRUD/issues/4389
I always thought that this anonymous class solution was either genious or stupid, guess what I found out ? 🤡
AFTER - What is happening after this PR?
I found out a better solution to merge the rules/messages in the request and validate it.
HOW
How did you achieve that, in technical terms?
Looked at it with a fresh perspective! 🙄
Is it a breaking change?
no
How can we test the before & after?
everything should keep working/validating when using this 👍

Haha that's some great copy there @pxpm 😅 I'll let @promatik review and test this. We gotta be sure nothing breaks.

@promatik I refactored as per our comments.
I think this cover all the cases, can you give this a new go and let me know if you found any inconsistency ?
Cheers

@pxpm let's please add unit tests for this. A bunch of them - covering all combinations of the types of validations we have... in all possible orders:

FormRequest
Array
Field

A new inspection was created.

The inspection completed: 3 new issues, 8 updated code elements

A new inspection was created.

Hello @tabacitu I've just finished writing the tests for the validation class and I merged them here.

I assigned @jorgetwgroup to review the PR, but I think we are done here and can merge this as it is.
Cheers

The inspection completed: 16 new issues, 15 updated code elements

hi @pxpm
First you know im not expert on test files, but always when i run this test i get this error.

I have this branches:
CRUD: refactor-request-anonymous-function (this)
PRO: main
Demo: main
Let me know if i miss something.
Cheers.

A new inspection was created.

The inspection completed: 12 new issues, 15 updated code elements

A new inspection was created.

The inspection completed: 12 new issues, 15 updated code elements


Add boosters and powerups

We need to add some goodies and powerups:

HTTPS should give you something (extra life?)
Service worker -> dual guns?
ressource hints being used?
Shields?
HTTP2?
...?

I started a simple mechanism for that, right now just giving out extra lifes for all kinds of audits. Whoever takes that issue feel free to build on that - we'll still need graphics for powerups, and different kinds of powerups (shields, dual lasters, whatever) etc.

Added shield powerup for ssl

Added sw powerup (faster shoot rate), and pwa powerup (destroys all asteroids currently on screen). Closing this out for now, minimum feature set is there now.


Adding a dependency with --git or --path doesn't add a newline to pypoetry.toml in 0.12.17

[x] I am on the latest Poetry version.

[x] I have searched the issues of this repo and believe that this is not a duplicate.

[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: Ubuntu 18.04.2 LTS

Poetry version: Poetry 0.12.17

Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/Heliozoa/2291a453d00abcfcf8e6515b76c71fcc

Issue
The gist is a result of running the following commands in 0.12.7
poetry init
poetry add pendulum --git https://github.com/sdispater/pendulum.git
poetry add my-package --path ../my-package/

After the first add, the empty line between poetry.dependencies and poetry.dev-dependencies is gone. The second one causes an error because the dev-dependency table now starts on the same line as the second dependency.
I remember it working very recently so I tried both 0.12.16 and the preview version 1.0.0a4. It issue is present in 1.0.0a4 but not 0.12.16.
Thanks for your work on poetry!

Same problem here with 0.12.17.

In develop branch, which uses tomlkit:
By changing https://github.com/sdispater/poetry/blob/aa2cc9b58b9ce869c0625463197e4799aca84c2e/poetry/console/commands/add.py#L54 into
        from tomlkit.items import InlineTable, Trivia
        from tomlkit.container import Container

and https://github.com/sdispater/poetry/blob/aa2cc9b58b9ce869c0625463197e4799aca84c2e/poetry/console/commands/add.py#L97 into
            constraint = InlineTable(Container(), Trivia(), new=True)

the issue goes away: the default function call used Trivia(trail="") instead.
I'd be happy to make a PR with the fix...


fix(ci): wait for mainnet fork deployment

Please read contributing guidelines and remove this line.

#5735  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spypsy and the rest of your teammates on  Graphite


[Bug]: the "connect with us" button is not visible on mobile screens

Describe the bug
Desktop screen

Mobile Screen

To Reproduce
No response
Expected Behavior
No response
Screenshot/ Video
No response
Desktop (please complete the following information)
No response
Additional context
No response

I can work on it. Please assign me

@Gauravtb2253 assigned to you

i am actually working on this right now and was about to ask for it to be assigned to me @pradeeptosarkar and then this happened

@Gauravtb2253 assigned to you

i am actually working on this right now and was about to ask for it to be assigned to me @pradeeptosarkar and then this happened

@Gauravtb2253 assigned to you

i have already identified the issue already and found another issue that the navbar in mobile view in the range of 700 to 900 width does not show any content in toggleable button

@Gauravtb2253 then pls don't move forward on this. closing this issue now. @keshav39 kindly raise a PR once you are done


Python 3.11 wheels

Python 3.11 beta 1 wheels were released on May 19, 2022. Python developer guide promise that python binary interfaces will be stable, so it allows starting to build wheel preparation now (wheels built with beta release should work with the stable release if depends only on public API).
As release wheels for python 3.10 take a long time and python 3.11 will bring really nice improvements for error reporting then many people may want to switch their base python development environment to python 3.11.
So I would like to ask if there are plans to release python 3.11 wheels close to the python 3.11 release?

Adding a link to PEP 664 – Python 3.11 Release Schedule . The current scheduled release data for Python 3.11 is 3.11.0 final: Monday, 2022-10-03.
We use Azure Dev Ops Hosted servers for the majority of the CI and packaging for SimpleITK. They maintain their virtual environments on Github. Having these VM support Python 3.11 will be a blocker to get the 3.11 wheels.
We have not specific timeline for 3.11 wheels, but anticipate being more timely with them than the 3.10 wheels.

cibuildwheel maintainers already working on having the possibility to build wheels with python 3.11 (installing from pkg and nuget) independently of python on the host runner.
I do not see your configuration (yaml), but if you want I may help with trying to use cibuildwheel.

While Windows and MacOSX are built natively on the AZP hosted systems. These first build the SimpleITK "core" C++ libraries, then as a separate build

Did I correctly understand that ctest call is building base libs?
https://github.com/SimpleITK/SimpleITK/blob/2ea09383411f7429fca348b9b0634989fb3c4382/Testing/CI/Azure/templates/package-win-job.yml#L22
Or is it in another place?

Yes, ctest is used to configure, build, and test the "base" or "core" SimpleITK C++ library and then submit the results to the cdash server. This core library is then used for each version python, java, C# etc.
Then ctest is used again for the python wheel: https://github.com/SimpleITK/SimpleITK/blob/2ea09383411f7429fca348b9b0634989fb3c4382/Testing/CI/Azure/scripts/win_build_python.sh

@blowekamp there are no linux and windows 3.11 wheels on pypi. I also do not see them on the release page. Could this be fixed?
https://github.com/SimpleITK/SimpleITK/releases/tag/v2.2.1

The wheels have been uploaded.


Use the new M1 runners for the macOS CI

See also: https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/

We have 2 CI errors here:

pipx is missing on the new runner.
http_parser is marked as deprecated in Homebrew and fails to install. We can either try to script brew edit to remove the deprecation or update to restinio 0.7.1. But I think a package for llhttp is missing in Homebrew. I'd try to go with the first option for a quick unblock.


Adds new blog post announcing opensearch hadoop

Description
Adds a new blog post announcing the availability of the hadoop client
Issues Resolved
[List any issues this PR will resolve]
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

This would be awesome to have!

This is a good start. I like having the compatibility matrices. Might be good, though, to also add a simple "Getting Started Examples"?
Maybe an example on how to write to a dataframe in scala?
e.g.,
val spark = SparkSession.builder().master("local[*]")
    .config("opensearch.nodes", "<IP_ADDRESS>").config("opensearch.net.http.auth.user", "admin").config("opensearch.net.http.auth.pass", "admin").config("opensearch.net.ssl", "true")
    .config("opensearch.batch.size.bytes", "1kb").config("opensearch.net.ssl.cert.allow.self.signed", "true")
    .getOrCreate()

or how to use it with pyspark like I demonstrate in my comment on #153.
I'm happy to add if you'd like.

Go for it!

Is anyone making any updates to this (@nknize )? We are targeting next week to publish it. Thanks!

Is anyone making any updates to this (@nknize )? We are targeting next week to publish it. Thanks!

Yes. I'll put the example in tomorrow.

Is anyone making any updates to this (@nknize )? We are targeting next week to publish it. Thanks!

Yes. I'll put the example in tomorrow.

Hi Nick, this is still awaiting your input. Thank you!!

@nknize @mnkugler @wbeckler - If you can make the final edits, update he blog date, and let @krisfreedain know when it's ready to go, we can get this posted to the blog tomorrow. Otherwise, we'll need to hold this until next Wednesday.

Otherwise, we'll need to hold this until next Wednesday.

Let's hold to Wednesday. I was working up the example with the published artifacts and noticed they don't support Spark 3. We may want to republish the Spark 3 artifacts before publishing the blog.

@mnkugler and @wbeckler - Are we good to publish this today?

Still waiting on @nknize's changes.

@pajuric The blocker right now is that the released OpenSearched-Hadoop artifacts are not compatible with Spark 3. Thus the compatibility matrix in this blog post is not correct and the example code I'm providing will not work for the users / readers:
e.g.,
[error] Modules were resolved with conflicting cross-version suffixes in ProjectRef(uri("file:/...
[error]    org.apache.spark:spark-core _2.13, _2.11

ThisBuild / scalaVersion := "2.13.0"

lazy val root = (project in file("."))
  .settings(
    name := "opensearch-spark-example"
  )

libraryDependencies ++= Seq(
  "org.apache.spark" %% "spark-core" % "3.2.4" exclude("javax", "servlet") exclude("org.apache", "hadoop"),
  "org.opensearch.client" % "opensearch-hadoop" % "1.0.1",
  "org.antlr" % "antlr4-runtime" % "4.8",
  "org.codehaus.janino" % "commons-compiler" % "3.0.8",
  "org.codehaus.janino" % "janino" % "3.0.8"
)

We need to publish the Spark 3 compatible version which is built and packaged with the artifacts from the spark/sql-30 module

I opened an issue to move this forward: https://github.com/opensearch-project/opensearch-hadoop/issues/304

@vagimeli @nknize - Just checking the status on this blog to see if there are any updates?

@vagimeli @nknize - Just checking the status on this blog to see if there are any updates?

@pajuric I've not heard from the authors in a while. I'm adding them to this comment, as they need to provide the update.
@nknize @harshavamsi Please update on the status of this blog. Is the text final and ready for an editorial review?

@wbeckler @Xtansia - Please provide an update on the blog, as I understand it has been transferred over to you both.

@wbeckler - Are we OK to close this blog?


feat(connector): (globalpay) add support for multilple partial capture

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Currently globalpay doesn't support multiple partial capture.
Merging this PR would bring that feature.
Flow

Create payment with capture_method = manual_multiple.
Partially capture the created payment.
Status changes to partially_captured.
Capture remaining amount. and status will go to succeeded

Capture can be done multiple times. Total captured amount cannot exceed authorized amount.
Additional Changes

[x] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Manual.

Create payment with capture_method = manual_multiple

Partial Capture the created payment

Capture remaining amount.

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

closing in favour of https://github.com/juspay/hyperswitch/pull/2035


Browser becomes unresponsive after dragging-and-dropping files.

UPDATE: it's this bug, which is being fixed by both Google and KDE: https://bugs.kde.org/show_bug.cgi?id=482142 and https://issues.chromium.org/issues/329703410
System Details

OS: Fedora 40
Thorium Version 122.0.6261.132 AVX2

Problem
The browser stops responding to mouse after dragging-and-dropping (DnD) a file onto a DnD area on a website.
Additional Notes
To reproduce:

Run Thorium on KDE Plasma 6 Wayland
Go to e.g. https://jsfiddle.net/kmturley/5tduyx6q/, but this happens on all websites
Drag a file from a file manager onto the DnD area
Observe the browser not responding to mouse events.
Keyboard still works, window still updates, it's only the mouse that gets disabled.

~Does not reproduce on Chrome Unstable.~
Hm, I managed to repro on Chrome Unstable, it looks like an upstream bug.

It looks like the browser gets confused whether the drag and drop session ended or not?

It seems that I have the same issue.
It only seems to happen when I am running wayland and have the --ozone-platform=wayland flag.
Try removing it.
However I didn't get the "Ignoring motion event received during dnd session." that you mentioned in the terminal
I personally really need the flag because I use pinch to zoom alot. Which does not work without that flag.

This is fixed in dev channel upstream chromium.
I use Version "127.0.6485.0 (Official Build) dev (64-bit)" with wayland and under KDE and I can upload photos to google image with no issues.


How to pass sting as the argument to the exported function

The code snippet
    [DllExport("Execute")]
    public static void Execute(string pname)
    {

        ProcessStartInfo startInfo = new ProcessStartInfo();
        startInfo.FileName = pname;
        Process.Start(startInfo);
    }

Is there any way by which we can pass the string values as the argument while executing the .net code using rundll32
  rundll32 Demo,Execute calc.exe

Firstly, wrong signature. You need offset to lpszCmdLine argument because rundll32 will call this like:
void CALLBACK
EntryPoint(HWND hwnd, HINSTANCE hinst, LPSTR lpszCmdLine, int nCmdShow);

Secondly, as you can see above, it will be pointer to allocated string. [? About data types]
I also remind everyone: Our DllExport supports export for executable modules ( .exe ). That is, you can try something related in your case but without rundll32. It also provides +x64 support.

Hi,
Thanks for pointing about lpszCmdLine.
I am able to read the complete command line string using GetCommandLineA() function and then use string functions to fetch the arguments.

Wait, why GetCommandLine ? you already have an access to LPSTR lpszCmdLine, eg.
// ~ rundll32 Demo,Execute calc.exe 1234 abc ...
string args = (CharPtr)lpszCmdLine
// "calc.exe 1234 abc ..."

Since I am not using EntryPoint function I dont think so I can directly access lpszCmdLine
Below is the final code snippet which worked for me.
Do let me know if there is another way to do the similar thing or I misunderstood your comments
    [DllImport("kernel32.dll", SetLastError = true)]
    static extern string GetCommandLineA();

    [DllExport("Execute", CallingConvention = CallingConvention.StdCall)]
    public static void Execute()
    {
        ProcessStartInfo startInfo = new ProcessStartInfo();

        string val = GetCommandLineA();
        string fname = "Execute";
        int sval = val.IndexOf(fname);
        int fval = sval + fname.Length+1;
        startInfo.FileName = val.Substring(fval);
        Process.Start(startInfo);
    }

Yes, because rundll32 will access to mentioned EntryPoint function instead of you. You just need to fix your signature due to +8 bytes offset to lpszCmdLine if we're talking about rundll32
~
[DllExport]
public static void Execute(IntPtr _, IntPtr __, IntPtr lpsz)
{
    string args = (CharPtr)lpsz
}

Or full signature. I'm not sure, but I think I also illustrated this moment in my screencasts to this project. Or where it was

I could not find how can we use (CharPtr) for converting the IntPtr lpszCmdLine to string.
If possible can you help me with some reference link or guide me on how can I use (CharPtr) ?
It will also help me to simplify my code

Oh right, sorry, (CharPtr)/(WCharPtr) are available if you have Conari engine https://github.com/3F/Conari
In general, it doesn't matter which way are you trying with lpszCmdLine. Even MarshalAsAttribute.
The important thing is that you have it :)
Again, please read our wiki.


Can't install the lib anymore: goawsroles/roles/roles.go:71: undefined: aws.Credentials

I can't use the library anymore because with the latest changes on goawroles lib the godynamo lib is broken:
running go get github.com/smugmug/godynamo I get:
github.com/smugmug/goawsroles/roles/src/github.com/smugmug/goawsroles/roles/roles.go:71: undefined: aws.Credentials

go get github.com/smugmug/godynamo
github.com/smugmug/goawsroles/roles
../../go/src/github.com/smugmug/goawsroles/roles/roles.go:71: undefined: aws.Credentials
same issue happens also for me

godeps save my life last night :D

I ran into this also and had to run
go get github.com/awslabs/aws-sdk-go/aws
I'm not sure why it didn't install automatically when this package is installed...any ideas?

Hmmm, the aws-sdk package is moving too fast, the bindings I made are already obsoleted by changes they have made...I think for now I just need to remove this dependency

I was wrong to try to bind to the aws-sdk-go, it is changing too quickly. I have removed references to it for   now and will add them back in at a later date when they have committed to a stable interface.


Failed to handle SubChannelFinalize message

Some apps have run into a broken state because of the coordinator being unable to process their SubChannelFinalize message. It's strange because the reason given is that the coordinator is in the Offered state, but one would expect the coordinator to be in Confirmed by then.
On top of that, the coordinator then receives a SubChannelAccept message (in theory because of a reconnect which rolls back the state). This seems to be a good idea, as the coordinator appears to have already rolled back to Offered, but handling this message fails with Invalid commitment tx signature from peer.
Logs here: https://itchysats.grafana.net/goto/0ckTe0gIR?orgId=1.

Related to https://github.com/get10101/10101/issues/1074.

This is the reason why our coordinator panics

See https://github.com/get10101/10101/issues/1074.
Every time these two peers reconnect the coordinator is restarted because of the panic.
Logs as proof.

Logs from one user[^1]
I've trimmed down the logs to only keep the most important bits.
2023-08-17T20:24:13Z	INFO Received peer Init message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:27:26Z	INFO Opening position peer_id=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:27:33Z	INFO Processing message kind=SubChannelAccept from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:27:34Z	INFO Sending message kind=SubChannelConfirm from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29

;; Reconnect
2023-08-17T20:27:55Z	INFO Received peer Init message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29

2023-08-17T20:28:24Z	INFO Processing message kind=SubChannelFinalize from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:28:24Z	ERROR Failed to process message: Failed to handle SubChannelFinalize message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29: Invalid state: Invalid state: Expected Confirmed state but got Offered

;; Reconnect
2023-08-17T20:28:24Z	INFO Received peer Init message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29

2023-08-17T20:28:29Z	INFO Processing message kind=SubChannelAccept from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:28:29Z	ERROR Failed to process message: Failed to handle SubChannelAccept message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29: Invalid state: Misuse error: Invalid commitment signed: Close : Invalid commitment tx signature from peer

;; Reconnect
2023-08-17T20:28:57Z	INFO Received peer Init message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29

2023-08-17T20:29:02Z	INFO Processing message kind=SubChannelAccept from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:29:02Z	ERROR Failed to process message: Failed to handle SubChannelAccept  message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29: Invalid state: Misuse error: Invalid commitment signed: Close : Invalid commitment tx signature from peer

2023-08-17T20:29:07Z	INFO Processing message kind=SubChannelAccept from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:29:07Z	INFO Sending message kind=SubChannelConfirm from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:29:17Z	INFO Processing message kind=SubChannelFinalize from=03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
2023-08-17T20:29:17Z     panicked at 'Commitment txids are unique outside of fuzzing, where hashes can collide'

;; Reconnect
2023-08-17T20:29:35Z	INFO Received peer Init message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29

2023-08-17T20:29:35Z     panicked at 'Commitment txids are unique outside of fuzzing, where hashes can collide'

2023-08-17T20:29:53Z	INFO Received peer Init message from 03365df35cd56a0f3b3b085039e22b5ec6e3d5f71c2a92266ca582bb0768d42b29
;; Reconnect

2023-08-17T20:29:53Z     panicked at 'Commitment txids are unique outside of fuzzing, where hashes can collide'

High level events

Lose connection after sending SubChannelConfirm.
Fail to handle SubChannelFinalize after reconnect because we've already rolled back to Offered (seems correct).
Peer sends SubChannelAccept (seems correct, peer must have rolled back correctly?).
Fail to handle SubChannelAccept! Seems like the reconnect logic failed us here somehow, even though we sent the right type of message.
After a couple of reconnects/attempts the peer sends SubChannelAccept again and it works (?) and we finally make it to processing SubChannelFinalize and then we panic!
After every restart we panic.

Why has this happened?
We have raised this with Tibo, providing full logs for the coordinator. Logs from the app are still pending.
[^1]: The logs from the other user look exactly the same.

see also https://github.com/p2pderivatives/rust-dlc/pull/142#issuecomment-1693511702


[FLINK-13155][hotfix][e2e] fix SQL Client end-to-end test

What is the purpose of the change
This PR fix SQL Client end-to-end test
Brief change log

Exclude kafka json files in flink-sql-connector-kafka
packaging flink-sql-parser into flink sql client jar

Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Please rebase the latest master and re-trigger the test.

The failure is irrelevant, passed in my own travis: https://travis-ci.org/KurtYoung/flink/builds/556831141, merging this.


HintsService and ReadCoordination can return incorrect labeling

As seen in https://github.com/k8ssandra/management-api-for-apache-cassandra/issues/512 there were metrics which have the IP + port in the metricName part itself, since the formatting had
"org\\.apache\\.cassandra\\.metrics\\.\\.(\\w+)\\.(\\w+)\\.(.+)$"
instead of "org\\.apache\\.cassandra\\.metrics\\.\\.(\\w+)\\.(.+)$"
HintsService and ReadCoordination are two metrics that do not have this special case taken care of and as such return incorrect metric naming:
# TYPE org_apache_cassandra_metrics_hints_service_hint_delays_10_244_2_14_7000 summary
# TYPE org_apache_cassandra_metrics_read_coordination_replica_latency_10_244_3_15_7000 summary

ReadCoordination metrics are DSE/HCD only.


RENDER TREE LayoutId-[<_ScaffoldSlot.floatingActionButton>]  when no FloatingActionButton

PROBLEM
How can I get rid of child 2:  LayoutId-[<_ScaffoldSlot.floatingActionButton>] ?
Indeed I do not understand what it is doing in the app
STEP TO REPRODUCE
This very basic app code as no FloatingActionButton
1____APP CODE
import 'package:flutter/material.dart';
void main() => runApp(ToTo());

class ToTo extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Container(
        color: Colors.orange,
      ),
    );
  }
}

However one is appearing
2____DEBUG PAINT
If you look at the bottom right corner of the screenshot (below)  you can see a little blue point
3____RENDER TREE
(DUMP rendering tree)
FloatingActionButton appears in the Render Tree:
rendering_tree.txt

Hi @benedicte69
I believe fab is part of scaffold
class Scaffold extends StatefulWidget {
  /// Creates a visual scaffold for material design widgets.
  const Scaffold({
    Key key,
    this.appBar,
    this.body,
    this.floatingActionButton, //! <= here !!
    this.floatingActionButtonLocation,
    this.floatingActionButtonAnimator,
    this.persistentFooterButtons,
    this.drawer,
    this.endDrawer,
    this.bottomNavigationBar,
    this.bottomSheet,
    this.backgroundColor,
    this.resizeToAvoidBottomPadding,
    this.resizeToAvoidBottomInset,
    this.primary = true,
    this.drawerDragStartBehavior = DragStartBehavior.start,
    this.extendBody = false,
    this.extendBodyBehindAppBar = false,
    this.drawerScrimColor,
    this.drawerEdgeDragWidth,
  }) : assert(primary != null),
       assert(extendBody != null),
       assert(extendBodyBehindAppBar != null),
       assert(drawerDragStartBehavior != null),
       super(key: key);

Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


502 Bad Gateway

Hello,
i try to install deployer on vagrant/homested machine (http://deployer.app/) - Ubuntu 14.04.3 LTS (GNU/Linux 3.19.0-25-generic x86_64).
After the install i try to configure my nginx server using the configuration found in examples/nginx.conf but when i log into the application i receive a "Server error" message.
The problem comes from the call to the http://deployer.app/socket.io/?.... who return 502 Bad Gateway nginx/1.9.7

Have you started the node server? node socket.js https://github.com/REBELinBLUE/deployer/blob/master/examples/supervisor.conf#L15-L25
and does your nginx config have the upstream https://github.com/REBELinBLUE/deployer/blob/master/examples/nginx.conf#L1-L3


Doomfist can uppercut enemies behind him

Uppercut hit detection is based on circular radius around the player. However, this should instead be a semi-circle in front of the player.

Redesigned doomfist uppercut since this issue was posted.


getPopup for layers other than markers

Not sure why this is a discrepancy, but it is - it would be nice to have getPopup on other types of layers that support bindPopup.

Any chance this will be backported to 0.7.x? I am missing getPopup on Path.

@anderius can you just use something like path._popup?

That would work, thank you!
On 28 April 2015 at 15:06, Vladimir Agafonkin<EMAIL_ADDRESS>wrote:

@anderius https://github.com/anderius can you just use something like
path._popup?
—
Reply to this email directly or view it on GitHub
https://github.com/Leaflet/Leaflet/issues/2292#issuecomment-97054232.


Use a refresh token to claim an access token?

Something that I would like to see in the README and if we can solve it here, I would gladly submit a pull request.
Given that a Web App has an expired access_token and a refresh_token, how does the client obtain a new access_token using that refresh token.

stored_user_credentials = {
refresh_token: 'TOKEN',
access_token: 'TOKEN',
expires_in: 3600,
issued_at: DateTime.parse('Fri, 15 Jan 2016 02:38:13 UTC +00:00')
}

client_secrets = GOOGLE_CALENDAR_CLIENT_SECRETS
client = Google::APIClient.new(application_name: GOOGLE_CALENDAR_APP)
calendar_api = client.discovered_api('calendar', 'v3')

client.authorization = client_secrets.to_authorization
    client.authorization.update!(
      scope: GOOGLE_CALENDAR_SCOPE,
      additional_parameters: {
        "access_type" => "offline",
        "include_granted_scopes" => "true"
      },
      redirect_uri: GOOGLE_CALLBACK_URL
    )

client.authorization.update_token!(stored_user_credentials)

#...

So far, I've not found an example of using the refresh token to obtain an access token anywhere. This setup step I've shown above works up until the access_token has expired and then no more.  In my real app, stored_user_credentials is coming from an ActiveRecord model that caches these credentials.

Generally speaking, you don't need to do much to exchange the refresh token. So long as you're properly serializing/deserializing the oauth2 client to restore the refresh token, all that needs to be done is call fetch_access_token!() to refresh. In this example, that would have to be after you call update_token!() to restore the refresh token.
That said, see https://github.com/google/google-auth-library-ruby for more current authorization code. It's a wrapper around signet. that makes things a bit easier. I'll put together a better sample at some point in the near future too.

The offending configuration seems to have been the
 additional_parameters: {
        "access_type" => "offline",
        "include_granted_scopes" => "true"
      },

Throwing a binding.pry to investigate, lead me to discover that calling client.authorization.refresh! directly was generating a message:
Signet::AuthorizationError: Authorization failed.  Server message: {   "error" : "invalid_request",   "error_description" : "Parameter not allowed for this message type: access_type" }
Removing those additional parameters seems to have lead to the client exchanging a refresh token for a new access token as expected in this instant example.

Thank you @sqrrrl. I'll be sure to take a look.

Ah, smallish bug in signet there.
For that error I'd suggest moving where you specify additional_parameters. Instead of setting them as properties on the client, just pass them in as options to authoriation_uri() so they're only used in that context (which is the case anyway, they're just used when building the initial authorization uri)

Can someone point me to an example of how I can get a refresh token working?
this is what I've got in my controller currently:
class Calendars::EventsController < ApplicationController
  def index
    if current_user
      client = ::Google::Apis::CalendarV3::CalendarService.new
      client.key = '...'
      client.authorization = {}
      client.authorization[:access_token] = current_user.oauth_token
      client.authorization[:refresh_token] = current_user.refresh_token if current_user.refresh_token
      @events = client.list_events(params[:calendar_id])
    end
  end
end

Added a small sinatra web app in the samples dir that hopefully helps.

@lifeiscontent did you ever figure that out?


Automatic inputs seem to add a load of accessories but not actual inputs

As is in the title, any help would be appreciated, happy to try any suggestions.
*edit: running latest version of the plug-in.

That sorted it, thanks


YARN-11609. Improve the time unit for FederationRMAdminInterceptor#heartbeatExpirationMillis.

JIRA: YARN-11609. Improve the time unit for FederationRMAdminInterceptor#heartbeatExpirationMillis.

LGTM.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 52s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
47m 36s

trunk passed

+1 :green_heart:
compile
0m 31s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
0m 27s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
0m 26s

trunk passed

+1 :green_heart:
mvnsite
0m 32s

trunk passed

+1 :green_heart:
javadoc
0m 34s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 27s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
0m 55s

trunk passed

+1 :green_heart:
shadedclient
39m 14s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 22s

the patch passed

+1 :green_heart:
compile
0m 23s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
0m 23s

the patch passed

+1 :green_heart:
compile
0m 21s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
0m 21s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 15s

the patch passed

+1 :green_heart:
mvnsite
0m 22s

the patch passed

+1 :green_heart:
javadoc
0m 22s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 21s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
0m 56s

the patch passed

+1 :green_heart:
shadedclient
38m  5s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 27s

hadoop-yarn-server-router in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

138m 49s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6254/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6254

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 8b85e6795040 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 367ca6c9a008421f741941b5d729d1c3c8ce3c70

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6254/1/testReport/

Max. process+thread count
536 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-router U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-router

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6254/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@YuanbenWang Thank you for your contribution!

@YuanbenWang Thank you for your contribution!

Thanks for the review!


CPU usage not fully reaching 100% with large vcpu values

While using the tool on a virtual machine to gain more computing power I noticed that not all the available CPU is used. The tool is using full CPU on 3 different local machines but they are just personal laptops with minimal setup.
The VM was an an AWS c6a.24xlarge EC2 with 96 vCPU and 192Go of RAM.
CPU usage was around ~21%
i lunched a second process and CPU usage got into ~46%
third process made it until ~65%
I tried a forth process but it was reaching 100% and impacting other processes op/sec.
The operations per seconds reached ~2.100.000 operations/second and this is impressive.
This is more of a experience than an issue since the tool managed to decrypt my 3 documents with 8 digits all capitals ( made some code changes since i didn't know how to pass that as a custom query ) in less than 18 hours.
Thanks guys for the tool.

Same here

I used it actually i set it to 20 if i set it to any larger number the op/sec go bellow the 2M mark. I don't know if the tool reached more than this barrier before ?

I used it actually i set it to 20 if i set it to any larger number the op/sec go bellow the 2M mark. I don't know if the tool reached more than this barrier before ?

You shouldn't exceed the number of threads your CPU has. Mine has 32, and I can see performance improvements until using 32 threads.

I guess that this is some kind of bottleneck I'm not sure if that's specific to EC2 instances or is it some code issue since I'm not familiar with rust. Using the tool on EC2 spot instances can be a great solution if the tool can use 100% CPU there too.

If nothing major has changed from when I wrote those parts then this tool is basically composed of 1 producer and multiple consumers. The worker threads are consumers, whilst a singular thread is a producer of passwords.
The idea is the workers get fed a password, attempt to crack the PDF with it, if successful it reports it back and the process ends.
Most likely the singular producer simply can't keep up with that many cores. So an improvement would likely be to allow multiple producers.
IIRC having multiple producers caused issues back when I wrote the code, mostly related to how the source of "candidate passwords" is basically just an iterator, which is sequential in nature. There are room for some potentially easy gains in this project however. IIRC the producer basically grabs 1 candidate before sending it through a pipe towards the workers. This likely involves a mutex somewhere along the line for each call, so perhaps having the producer prepare a "chunk" of candidates and sending them all at once yields better performance. But It'd probably need to be prototyped and tested to confirm that would actually be faster

Digging into the code in the "engine" crate then my memory was correct. I don't see any "quick and dirty" way to permit sending multiple passwords in each call without risking 1 worker picking up all of them, leaving the others "empty" until the next batch comes around. But perhaps that would be alright since adding some thoughts and prayers might mean they each get their own "batch" before the first worker gobbled up all of it's candidates. This requires our source of password candidates to be fast enough though.
So basically a quick change would be (in the "engine" crate) to change the messaging pipeline to be a Vec<Vec> (i.e. a list of passwords instead of just 1 password per message). Then have the engine consume X candidate passwords from it's source before sending a message containing them towards the workers. Assuming we set the "size" of each message to 30 passwords I do see some consequences:
Basically assuming we got more than 2 worker threads and our set of "candidates" is fewer than 30 that would basically mean only one worker will ever get to attempt to crack the PDF. i.e. if we want any reasonable performance then the size of our set of candidates should optimally be something like
# Number of workers consuming messages
WORKER_THREADS = 96
CANDIDATES: set[bytes] = {b"password", b"passwor1", ... }

# Maximum number of messages per "message" sent to each worker
MESSAGE_SIZE: int = len(CANDIDATES) // WORKER_THREADS

Sadly automatically calculating MESSAGE_SIZE is probably not a wise idea since there are situations where we would have to simply read and count all of them in order to know, e.g. if our source is a file where each line is a password. That'd likely be super slow assuming the file is large (and they tend to be in these applications...). But perhaps we can make it an argument the user can utilize for tuning.
Another option is doing what I think @VMordy did and split the password source (a file?) into multiple parts, and run multiple instances of PDFRip in parallel, since it "emulates" having multiple producers. I don't see how we can do this universally for each source of password candidates however

For parallel processing I tweaked in the source code to edit the password generator in the default_query so I made two binaries each one start the generation from a direction and when started in parallel the ETA is logically the half. I'm not familiar with rust but I'm used to tackle such bottlenecks with a partitioned channels pub/sub style where the producer publish passwords and consumers  try to do the work. I can see that in this case that the password producer is the main issue but we can work something up by dividing the pool of passwords by first partitioning it by first characters and then let each producer generate a password based on that first character. I'm just sharing my thoughts with you and I want to thank you for this great tool that you have made.


Edgedb Buildpack can no longer create Postgresql Extensions in Heroku

After changes made to Heroku on August 1st, edgdb's server command can't create any postgres extensions, as Heroku has changed their Postgres extension schema to their new "heroku_ext" schema. (according to: https://devcenter.heroku.com/changelog-items/2446)
Graphql-engine solved this by allowing a custom extensions schema variable during setup:
https://github.com/hasura/graphql-engine/issues/8734
See: https://github.com/hasura/graphql-engine/commit/f3dd1728213470b760a6672eacf9fec283cf9e35
See current error when trying to run edgedb's start-edgedb buildpack command.

Has there been any progress on this issue? I'm currently waiting to deploy a new version of an app that uses EdgeDB

Fixed in #5413


Sanitize numpy array to Python list for zmq Publisher

Sanitize numpy array to Python list for zmq Publisher so that it is JSON seralizable, since this will be removed at the RunEngine level.

Closed by #674


adds provisioning dialog directive

employed by marketplace and request details states this directive can both display and accept input while also displaying already selected values (input disabled)
this directive takes three parameters:

dialog (the dialog object)
isDisabled (boolean, if the dialog form is to be used for display only) optional
options (the options dialog object, if it exists, for indicating already selected values) optional

@eclarizio please review.

General feedback is that there may be several places you can add :: to remove things from the $watch list. For a very complex dialog this could trim a lot of items.

LGTM :+1:

@AllenBW let me know when the above comments are addressed . . . thx, Dan

G2G on this front

good catch @stackus, DialogFieldCheckBox wasn't employed to display previously selected values wasn't jiving


Add 4 nodes that relay inscription transactions

Adding these nodes as manual connections for peers when running a node for creating and broadcasting inscription tx would ensure tx is relayed properly. Some nodes on bitcoin p2p network are running a patch that rejects inscription txs in their mempool.

NACK

You should use your own DNS alias for this purpose. I've never made any promises that those nodes will relay inscriptions in the future.
There's no evidence that ordinal censorship is even widespread enough to matter. To have a high chance of ordinal transactions not propagating for nodes that don't listen and just make the standard 8 outgoing connections, you'd have to have more than a majority of nodes censoring them. For nodes that do listen, the situation is even better as they'll be even better connected.

Concept NACK
I am opposed to making mempoolfullrbf standard (would support including it in further documentation regarding node optimization/configuration) and I am STRONGLY opposed to providing static peers as a suggestion anywhere within the documentation.
The defaults (configuration documentation establishes defaults) should NEVER modify the peering configuration of a node unless a user explicitly chooses to do so (while being fully informed of why this is typically discouraged).
Assuming that the concept was approved, the proposed change (with Peter's addendum) is an appropriate way to encourage users to add these four specific peers to their nodes.

@petertodd

You should use your own DNS alias for this purpose. I've never made any promises that those nodes will relay inscriptions in the future.

Fixed DNS aliases in last commit. Sure, docs can always be updated and I added this after confirming on twitter once.

There's no evidence that ordinal censorship is even widespread enough to matter. To have a high chance of ordinal transactions not propagating for nodes that don't listen and just make the standard 8 outgoing connections, you'd have to have more than a majority of nodes censoring them. For nodes that do listen, the situation is even better as they'll be even better connected.

Agree, its not an issue at the moment however it might be too late to act if most nodes censor them at some point. It would affect not just inscriptions but bitcoin as well. So, I wanted to use these nodes as a precaution or best practice. I could have added other nodes here although I am assuming these nodes would be better connected to mining pools compared to others.
I had even reached out to some mining pools to get node addresses that could help, however there was no response.
@Psifour

I am opposed to making mempoolfullrbf standard (would support including it in further documentation regarding node optimization/configuration) and I am STRONGLY opposed to providing static peers as a suggestion anywhere within the documentation.

I dont see any downside of using mempoolfullrbf particularly for nodes running to just create inscriptions.
Can you explain the reasons for STRONG opposition as I dont think there are any issues with 4 manual connections to these nodes apart from automatic connections that bitcoin core makes (outbound/inbound)

The defaults (configuration documentation establishes defaults) should NEVER modify the peering configuration of a node unless a user explicitly chooses to do so (while being fully informed of why this is typically discouraged).

Manual connections are used in many bitcoin projects particularly the ones in alpha or beta phase. These aren't connections to some random nodes online. Peter Todd has been contributing to Bitcoin since years and also runs one of the DNS seeds used in bitcoin core.

@petertodd

You should use your own DNS alias for this purpose. I've never made any promises that those nodes will relay inscriptions in the future.

Fixed DNS aliases in last commit. Sure, docs can always be updated and I added this after confirming on twitter once.

Thanks.
With this change, ACK.

https://twitter.com/provoost/status/1641866584426790912
Cc: @casey @rot13maxi @raphjaph @RarePepeMaximalist @cbspears @Psifour @p


linter-flake8 not linting (not showing errors)

Folks,
I have installed linter-flake8 in atom and it does installed linter as dependency. I have my flake8 installed in my python virtual environment. I have my path to flake8 supplied as follows
/home/userlog/pythonvenv/bin/flake8. But, nothing happens when I edit a python file. Could some expert shed some light on where to look and how to troubleshoot what I am doing wrong? Or is it a issue, as I see linter in github with status build-failing.
Thanks in advance,

Hi @russellb23 ,
I had the same issue, in my case I 'solved' it by starting atom in a folder that has no git o virtualenv, like: atom path/to/empty/dir.
My guess is that per project/git/virtualenv settings confuse this package.
Hope it helps in your case too.
Cheers

Closing due to a lack of response.
If you are still having issues @russellb23 make sure that the file you have open is recognized as Python and is getting matched to this provider by running Linter: Debug from the Command Palette. You should see "flake8" in the list of Matching Providers, as well as "Open file is Ignored: No".


Add condition to avoid infinite loop for ask.com.

Fixes #258
Checklist

[ x] I have read the Contribution & Best practices Guide and my PR follows them.
[ x] My branch is up-to-date with the Upstream master branch.
[ x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

If page is empty with results, then ask.py scraper' parseResponse will return none.

Then after receiving through generalized.py' search function in newResults, It will check for 'None' condition. If true, it will break out of otherwise infinite loop.

I used following script to test the response, which was working fine if such case encountered.

On 7th page:

@shashank-sharma please review :).

Hi, Good going
Try to fix some error generating from travis-ci. Besides that I can see that it is not going in infinite loop which is great. But for different parameters I am getting different results, can you explain why ?
http://<IP_ADDRESS>:7001/api/v1/search/ask?query=fossasia&format=json&num=50
Giving: 50 results which is great
http://<IP_ADDRESS>:7001/api/v1/search/ask?query=fossasia&format=json&num=60
Giving: 45 results
http://<IP_ADDRESS>:7001/api/v1/search/ask?query=fossasia&format=json&num=80
Giving: 36 results

@shashank-sharma I cant open your provided links (I think different outputs can only be due to ask.com, isnt it?) , I have corrected travis errors. But what is this error showing up: "missing whitespace after ':'". Can you help me with this and indentation error please . Thanks.

Hey @rupav
As per what I understand your indentation on line 23 of ask.py is not proper. I cloned your repository and viewed this file.
See this image and see the error. Try fixing this and you will be able to clear few or all errors.

@shashank-sharma I had to clean everything from my forked repo too, which eventually closed this PR. Sending a new PR.


Remove elasticsearch.ssl.transport.enabled:false/true as it has no effect now

In the helm chart readme,
https://github.com/opendistro-for-elasticsearch/opendistro-build/tree/master/helm
is no longer relevant because transport tls is always enabled.

Hi @dmpe thanks for pointing this out.
The original change to disable this parameter is here: https://github.com/opendistro-for-elasticsearch/opendistro-build/pull/53
As of now, the HELM section is actively maintained by the community members.
Would you mind create a PR for this change in documentation? We would be very much appreciated.
Thanks.