Fail earlier with DNS issues

I tried creating a DNS record on staging, and it's failing due to what appears to be a config/permissions error:
<EMAIL_ADDRESS>  | {"$fault":"client","$metadata":{"attempts":1,"httpStatusCode":403,"requestId":"a250980e-b074-41da-85e4-08cc5a7d9e66","totalRetryDelay":0},"Code":"AccessDenied","Type":"Sender","level":"warn","message":"User: arn:aws:iam::REDACTED is not authorized to perform: route53:ChangeResourceRecordSets on resource: arn:aws:route53:::REDACTED because no identity-based policy allows the route53:ChangeResourceRecordSets action","name":"AccessDenied","timestamp":"2023-03-17T15:28:22.328Z"}
<EMAIL_ADDRESS>  | {"$fault":"client","$metadata":{"attempts":1,"httpStatusCode":403,"requestId":"cc2ff934-ea00-4035-aa38-55c86e1b43bb","totalRetryDelay":0},"Code":"AccessDenied","Type":"Sender","level":"warn","message":"User: arn:aws:iam::REDACTED is not authorized to perform: route53:ChangeResourceRecordSets on resource: arn:aws:route53:::REDACTED because no identity-based policy allows the route53:ChangeResourceRecordSets action","name":"AccessDenied","timestamp":"2023-03-17T15:28:37.589Z"}
<EMAIL_ADDRESS>  | {"$fault":"client","$metadata":{"attempts":1,"httpStatusCode":403,"requestId":"99b035ba-1c17-4518-89db-54b69a2c63a3","totalRetryDelay":0},"Code":"AccessDenied","Type":"Sender","level":"warn","message":"User: arn:aws:iam::REDACTED is not authorized to perform: route53:ChangeResourceRecordSets on resource: arn:aws:route53:::REDACTED because no identity-based policy allows the route53:ChangeResourceRecordSets action","name":"AccessDenied","timestamp":"2023-03-17T15:29:07.734Z"}
<EMAIL_ADDRESS>  | {"$fault":"client","$metadata":{"attempts":1,"httpStatusCode":403,"requestId":"aa4a0070-fa9f-4b43-8d99-8c73d7e87fe6","totalRetryDelay":0},"Code":"AccessDenied","Type":"Sender","level":"warn","message":"User: arn:aws:iam::REDACTED is not authorized to perform: route53:ChangeResourceRecordSets on resource: arn:aws:route53:::REDACTED because no identity-based policy allows the route53:ChangeResourceRecordSets action","name":"AccessDenied","timestamp":"2023-03-17T15:30:08.063Z"}
<EMAIL_ADDRESS>  | {"$fault":"client","$metadata":{"attempts":1,"httpStatusCode":403,"requestId":"d1b53823-f8b8-4765-9fe0-b05c7c2c83b0","totalRetryDelay":0},"Code":"AccessDenied","Type":"Sender","level":"warn","message":"User: arn:aws:iam::REDACTED is not authorized to perform: route53:ChangeResourceRecordSets on resource: arn:aws:route53:::REDACTED because no identity-based policy allows the route53:ChangeResourceRecordSets action","name":"AccessDenied","timestamp":"2023-03-17T15:32:08.322Z"}

However, it never gets to the error state on the server or in UI:

I'm discussing the config/permissions with ITS, but I wanted to talk about the failure case in our DNS flow.  These 5 errors happen pretty quickly, and then nothing happens.
We should fail sooner I think.  In this case, no amount of waiting is going to fix the problem.
In the case of being Throttled by AWS, we'll get this:

Five requests per second per AWS account per Region. If you submit more than five requests per second in a Region, Resolver returns an HTTP 400 error (Bad request). The response header also includes a Code element with a value of Throttling and a Message element with a value of Rate exceeded.

If we get throttled, retrying later is good.  If we are failing some call to AWS, retrying forever isn't likely to fix things.
cc @Genne23v, @dadolhay

Can we throw UnrecoverableError based on error type from AWS? (Not sure which error we should throw this error though) Then we can stop the workflow without reducing waiting time.


Move RootPickerDialog and prohibit adding a new root if there is already a root inside

Move RootPickerDialogFragment here from Navigator
The functionality of the RootPickerDialogFragment and the ArkFilePickerViewModel.pinFile() are the same. In fact, we can add root/favorite from two different places. We need to reuse the general logic and prohibit adding root if there is already a root inside.
We also need to prohibit adding device root as ark root.

We also need to prohibit adding device root as ark root.

I'm thinking about this again and it seems there could be scenarios when it's ok, e.g. SD card if user doesn't have a lot of data on it. Do you think we should support it?


Unable to ACK while on Socket Mode + Non-Decorator functions

Hi Slack API team,
I am building a Slash Command bot in a class structure. With that, I have elected to use non decorator approach to break up the flat file.
I have review and attempted to duplicate this effort to gain a better understanding. While the example is not formulate in class structure, I do think it is a start.
Problem is that I am keep running into WARNING:slack_bolt.AsyncApp:Unhandled request when using non-decorator.
Maybe this is an operator error. I hope to gain some more insight from you :)
Reproducible in:
pip freeze | grep slack
python --version
sw_vers && uname -v # or `ver`

The slack_bolt version
slack-bolt==1.16.1
Python runtime version
Python 3.10.8
OS info
ProductName:	macOS
ProductVersion:	12.6.3
BuildVersion:	21G419
Darwin Kernel Version 21.6.0: Mon Dec 19 20:43:09 PST 2022; root:xnu-8020.240.18~2/RELEASE_ARM64_T6000
Steps to reproduce:
(Share the commands to run, source code, and project settings (e.g., setup.py))
import logging
import os

from slack_bolt.async_app import AsyncApp, AsyncRespond, AsyncAck
from slack_bolt.app.async_app import AsyncApp
from slack_bolt.adapter.socket_mode.async_handler import AsyncSocketModeHandler

logging.basicConfig(level=logging.DEBUG)
app = AsyncApp(token=os.environ["SLACK_TOKEN"])

async def test_command(logger: logging.Logger, body: dict, ack: AsyncAck, respond: AsyncRespond):
    logger.info(body)
    await ack("thanks!")
    await respond("HI")

app.command("/asset")(test_command)

async def main():
    handler = AsyncSocketModeHandler(app, os.environ["SLACK_APP_TOKEN"])
    await handler.start_async()

if __name__ == "__main__":
    import asyncio

    asyncio.run(main())

Expected result:
Expect the same behavior as I am using the listeners in decorator mode.
Actual result:
DEBUG:slack_bolt.AsyncApp:Applying slack_bolt.middleware.ignoring_self_events.async_ignoring_self_events.AsyncIgnoringSelfEvents
DEBUG:slack_bolt.AsyncApp:Applying slack_bolt.middleware.url_verification.async_url_verification.AsyncUrlVerification
DEBUG:slack_bolt.AsyncApp:Checking listener: test_command ...
WARNING:slack_bolt.AsyncApp:Unhandled request ({'type': None, 'command': '/assets'})
---
[Suggestion] You can handle this type of event with the following listener function:

@app.command("/assets")
async def handle_some_command(ack, body, logger):
    await ack()
    logger.info(body)

INFO:slack_bolt.AsyncApp:Unsuccessful Bolt execution result (status: 404, body: {"error": "unhandled request"})
DEBUG:slack_bolt.AsyncApp:Message processing completed (type: slash_commands, envelope_id: 91dd611a-0449-4740-a50d-3acf65cbf4d2, session: s_274867128)
DEBUG:slack_bolt.AsyncApp:Received message (type: PONG, data: bytearray(b'sdk-ping-pong:1676491922.412249'), extra: , session: s_274867128)
DEBUG:slack_bolt.AsyncApp:Received message (type: PING, data: bytearray(b'Ping from applink-6'), extra: , session: s_274867128)

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Is operator error...

The ordering of AsyncApp and AsyncSocketModeHandler (or any other handler) matters
#this needs to arrive first
app = AsyncApp(token=os.environ["SLACK_TOKEN"])

#this needs to arrive later
handler = AsyncSocketModeHandler(app, os.environ["SLACK_APP_TOKEN"])
   await handler.start_async()


Designer Problem

the changes on the colorscheme or theme of the form doesn't show up in the designer :/ .

Neither do mine, I believe the colors are all changed at run time.

If we want designer to update color scheme at design time, we have to make
color scheme a property of MaterialForm so it can be edited by designer and
get compiled at the time of design. Its changes must be saved on
"Form1.Designer.cs" (for example) by designer.
Or make a control that contains an instance of MaterialSkinManager.
I'm making an improvement on this project (implemented RTL and Modal
dialogs like MessageBox and SelectDialog, ...)
2015-07-28 22:00 GMT+04:30 Jason H Caruso<EMAIL_ADDRESS>
Neither do mine, I believe the colors are all changed at run time.
—
Reply to this email directly or view it on GitHub
https://github.com/IgnaceMaes/MaterialSkin/issues/64#issuecomment-125685897
.


Search dropdown with allowAdditions submits form on enter key

The form should ignore Enter key presses from the dropdown's search input wheh allowAdditions=true too .
Reproducible jsfiddle, type on keybord new value, then push Enter to finish typing, form will submit:
http://jsfiddle.net/bojf4m1c/
Semantic UI - 2.4.2

Fixed since 2.7.0 https://github.com/fomantic/Fomantic-UI/pull/320

Please close this.


Updating techdocs-cli to fully facilitate a Backstage like environment using the --preview-app-bundle-path flag

Hey, I just made a Pull Request!
I discovered some issues when running techdocs-cli serve and using our Backstage app and the upstream Backstage app using the --preview-app-bundle-path flag.
The issues boiled down to a few things:

The HttpServer not proxying requests to

*/static/docs/default/component/local/
*/metadata/entity/default/component/local
*/metadata/techdocs/default/component/local
*/sync/default/component/local

The entity_metadata.json file was not created. This is requested by the request to */metadata/entity/default/component/local.
I have added the --catalog-file flag to the generate command. It will use the catalog file to generate the entity_metadata.json file and place it in the site folder

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Just waiting on a review

Bump. Waiting on a review

Not stale. Waiting on a review from techdocs maintainers.
@Rugvip is there any way I can get this looked at? I've added an issue to describe the issue: https://github.com/backstage/backstage/issues/18180

I'm facing this issue as well :(


Add scale retry in ScaleRC in test/e2e/util.go

Fixes (hopefully) #11790.
There are 2 timeouts that can be passed to kubectl/scale.go:Scale - one for the scale itself and the other for starting all replicas. test/e2e/util.go  pass only the second one (5 min). In such a case scale.go tries only once using very strict timeout (1ms). The timeout exceeded error in #11790 is thrown from ScaleRC almost immediately (the whole test fails after ~50sec) which suggests that a bigger timeout/retry needs to be passed.

cc: @ixdy

LGTM, thanks!


Add ReplicateLLM

Description
This PR adds the ReplicateLLM class to easily use Replicate Endpoints within distilabel.
Closes #122 (as a follow up of #47)
Things to discuss before merging

Generation kwargs definition is poor, because it's constrained to the cog container used, and it's most of the times different per each model hosted in Replicate, which makes it harder to use via kwargs
The fact that the pre/post processing happens internally in the cog container, the formatting of the prompt is tricky as sometimes that's expected to come pre-formatted on the client side, while other times that's handled by the container, but as that's controlled with kwargs, it's impossible to tell
After some testing I think that the latency is better in other competitors, so maybe we can keep this PR as a draft until we figure out whether there's a clean way we can make it work fine with distilabel because as of now, it's really messy

Thanks @alvarobartt , great work and analysis. I agree we can leave it as draft pr for now and see if there's interest from users


Advanced Shuffle Editor

This editor allows you to chain shuffles as they run. This PR still allows you to use the old system if you like.

So is it ready then @EmeraldBlock?

I haven't yet tested the new changes, but if you think it's all fine, yeah


How to debug custom loss function

This is likely a simple question as I'm a novice with Keras. I'm writing a custom loss function and I would like to debug it. How can I do this? The print() function doesn't work and loss/metric functions don't take any arguments beyond y_true and y_pred.

Hi @jamesseeman, I have the same problem with Keras at the moment. The problem is that the loss function is given to the model with the add_loss method or with the parameter loss= of the compile method. When the model is compiled a compiled version of the loss is used during training. It seems that even if you have a K.print_tensor() instruction inside the loss, when the loss is compiled the model doesn't need to eval that during training. I know, it's frustrating. This is what I found.

@bmxitalia Hi, I am having a similar problem. I am trying to implement Elastic weight consolidation in keras and in order to do that, I need a loss function which uses model trainable weights to calculate loss, so I need it such that every time weights are updated, different loss values should be encountered. However, I am not being able to get that. It might be because of what you said, that while training, a compiled version of loss is used.
Apart from that, I also haven't been able to debug the custom loss function.
Here's my custom loss function.

def custom_loss(fisher, star_vars, lamda):

    def loss(y_true, y_pred):
       # here "model" is a global variable
        var_list = []
        for layer in model.layers:
            for weights in layer.get_weights():
                var_list.append(weights)

        ewc_loss = keras.losses.categorical_crossentropy(y_true, y_pred, from_logits=False, label_smoothing=0)
        for v in range(len(fisher)):
            ewc_loss += (lamda / 2.0) * tf.reduce_sum(tf.multiply(fisher[v].astype(np.float32),
                                                                   tf.square(var_list[v] - star_vars[v])))

        return ewc_loss
    return loss

Any ideas on how to work around this?

You can call the function yourself and see what it returns.


Use AtomicIntegerFieldUpdater instead of AtomicInteger

Avoids one level of indirection when performing atomic
updates on the queueSize counter.

After an offline discussion, I've decided that the risk to maintainability doesn't justify the slight performance improvement here. Closing.


Allow Opus audio in MP4 container

This avoids using MKV when video uses AVC and audio uses Opus, which
makes the resulting MP4 file playable in modern web browsers.
Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[ ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Fixes #27366

opus is mp4 container is only supported by the very recent versions of ffmpeg, 4.3 or so. Thus this will break most of the ffmpeg versions.

It would be even better to check the video and audio codecs info_dict['vcodec'] and info_dict['acodec'] against a list of supported codecs for the extension info_dict['ext'].

It would be even better to check the video and audio codecs info_dict['vcodec'] and info_dict['acodec'] against a list of supported codecs for the extension info_dict['ext'].


[unstable option] color

Tracking issue for color

Will be removed in 2.0.

This was removed in 2.x in https://github.com/rust-lang/rustfmt/pull/4006, but needs to be re-decided in 1.x.

With github actions, even with --color=always still the diff view is NOT colourful:
runs-on: ubuntu-latest
steps:
  - uses: actions-rs/toolchain@v1
    with:
      toolchain: stable
      override: true

  - uses: actions/checkout@v2

  - name: Test run bare cargo fmt
    run: >
      cargo fmt --all --manifest-path=rsmain/Cargo.toml
      -- --check --color=always

With github actions, even with --color=always still the diff view is NOT colourful:

That's because Actions has issues reporting tty/color support which has impacted myriad tools (https://github.com/actions/runner/issues/241). Some tools have implemented GitHub Actions-specific workarounds for this (e.g. by checking for the presence of the Actions-set env var), but I don't think that really makes sense for us. We utilize the term crate for detection and handling, so if that's something important to you then I'd suggest reaching out to them (https://github.com/Stebalien/term) to gauge their willingness/interest in supporting any such workaround for GHA.

Just a note. Cargo has CARGO_TERM_COLOR env var that can be set to always for colors on github ci. But cargo fmt is not using that.


SSL_get_peer_tmp_key - Session Resumption with Session ID - IPv6

SSL_get_peer_tmp_key works great for me even when Session Resumption is used with a Session ID on IPv4.
Even if tls_process_ske_ecdhe is not called again, when Session Resumption is used, s3->peer_tmp has the right value.
Currently I am searching for the lines of code where it is restored when Session Resumption is used, because it is not working when IPv6 is used. Hopefully this is not expected behavior and you can help me to find the problem.
I  am using OpenSSL 1.1.1k.

Current State: It looks like SSL_get_peer_tmp_key is never working when Session Resumption is used.
IP v4 only works for me, because there is a caching mechanism in Qt. This seems to fail with IP v6.
So this is expected behavior of OpenSSL or a general Bug?

So this is expected behavior of OpenSSL or a general Bug?

Assuming you are talking about TLSv1.2 then this is expected behaviour. In a TLSv1.2 session resumption there is no new key exchange so the tmp key isn't used.

I am talking about TLSv1.2 right.
Technically I fully agree with you.
But even if there is no new key exchange, the session is based on the first tmp key.
In our software we have strict rules and need to check, if the length of the tmp key is  sufficient.

But even if there is no new key exchange, the session is based on the first tmp key.

The "tmp" key is exactly that. Temporary. It is used as an input to derive the master key. Once the master key is derived the temporary key is no longer needed. When resuming a session the resumption is based on the master key. The tmp key is not used in any way at this stage.

In our software we have strict rules and need to check, if the length of the tmp key is sufficient.

There is no tmp key to check. If the tmp key was considered of sufficient length in the original session such that the derived master key is acceptable, then presumably the master key must still be acceptable in any resumption. Since you can't do a resumption without having completed an initial connection it seems that just checking the tmp key in the original connection should be enough.

So it is ok to accept a missing tmp key when a cipher algorithm with forward secrecy was used?
I try to find an API call to check if the session was resumed, but I can not finde one.

So it is ok to accept a missing tmp key when a cipher algorithm with forward secrecy was used?

If a <= TLSv1.2 session is resumed then, yes.

I try to find an API call to check if the session was resumed, but I can not finde one.

SSL_session_reused()
See:
https://www.openssl.org/docs/man1.1.1/man3/SSL_session_reused.html

Question has been answered, closing.

Thank you very much for your support. Problem was solved in https://bugreports.qt.io/browse/QTBUG-93295.


GH-3081: Add netcoreapp3.1 target to Cake

Tests already target netcoreapp3.1 so changes needed there.

Closes #3081

In phone, but this probably relates to / maybe duplicates #2873

@devlead It's definitely related, but #2873 is breaking and doesn't seem we'll merge until Cake v2.0. This PR is additive, only to help devs that don't have netcoreapp3.1 and higher only, and don't want to deal with roll-forward.

@augustoproiete back at computer, agreed 👍

@augustoproiete your changes have been merged, thanks for your contribution 👍


searx: fix for flask-babel 3.0

Description of changes
The update to flask-babel 3.0 (48958930) broke searx, despite the program apparently building.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested with searx.tests
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

despite the program apparently building.

That's the python experience ✨


HpackDecoder's maxHeaderListSizeGoAway kills stream unnecessarily

Consider this a bit of a TODO for myself, although @Scottmitch or @normanmaurer may have opinions about avoiding killing the connection in this case.

Actual behavior
When maxHeaderListSizeGoAway is exceeded, HpackDecoder will throw a connection error, killing the entire connection.
                            // Check new header size against max header size
                            if ((long) index + nameLength > maxHeaderListSizeGoAway - headersLength) {
                                headerListSizeExceeded(maxHeaderListSizeGoAway);
                            }

Note, I'm not talking about the separate check later against maxHeaderListSize and throws a stream error:
        // we have read all of our headers, and not exceeded maxHeaderListSizeGoAway see if we have
        // exceeded our actual maxHeaderListSize. This must be done here to prevent dynamic table
        // corruption
        if (headersLength > maxHeaderListSize) {
            headerListSizeExceeded(streamId, maxHeaderListSize, true);
        }

Even though both of those call headerListSizeExceeded(), they call different overloads of the method which will throw different types of exceptions (connection vs stream).
The value of maxHeaderListSizeGoAway is likely 1.25 the size of maxHeaderListSize, although I see there's some ways it could be something different and I didn't dig in more.
This killing of the connection seems unnecessary though, as:

It seems it would be pretty easy to throw away this batch of headers, by setting headers to a Http2Headers implementation that throws away further adds (similar complexity to EmptyHttp2Headers). This could be done any time headersLength > maxHeaderListSize.
When decode is called, the "entire header block is contained in {@code in}". So this protection doesn't protect us from buffering any more data; it only reduced the decoded headers size. That is, throwing away decoded headers as suggested in (1) does not seem to expose us to other problems.

Note also there are separate checks in DefaultHttp2FrameReader.HeadersBlockBuilder that protect against too large of compressed HEADERS+CONTINUATION. This is what constrains the size of in.

I think what you said @ejona86 makes sense... want to do a pr ?

I worked on the change last night :) but didn't send it out because I was investigating further changes. Sent it out now.


Error when installing from nuget in xamarin project

I am using vs2017 and xamarin.forms project, and this error showed when i start to install
"Could not install package 'LiteDB 3.1.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile111', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author"
So I assume it's the cause of vs2017? Not sure if there is future update for this version?
btw  thanks for this great db.

Hi @charleypeng, LiteDB v3.1 is was compiled in vs2017. To works with xamarin, you must reference LiteDB as a NetStandard reference.


Not showing complete product description on front-end.

Framework Version: 0.1.5

Probably product description HTML is not valid.

Framework Version: 0.1.5
Fixed


Adding global context secp256k1

This resolves #44
Before this for every multiplication you created the context. which is a very hungry operation (cpu, memory etc).
Instead we're creating a context only once and keep using that.
this context is thread safe. you can pass it around however you want.
the only constraint are:
There is no promise about the exact memory size and representation, meaning that you cannot save it to disk and reload later.
no one should try and modify it's memory(not a problem in rust)

@elichai Is the serde_json necessary? what is his purpose?
@gbenattar can I merge?

It's used in the tests so I moved it from the dev-dependencies to the dependencies

got it, thanks.
@gbenattar waiting for your ok

I am approving this PR given the following pre-condition:

This change has been tested in https://github.com/KZen-networks/kms-secp256k1 and in https://github.com/KZen-networks/gotham-city (gotham-server).


Replicating canvas-like behaviour

Hey,
I'm trying to achieve similar behaviour to the second example of this canvas-lib:
http://fabricjs.com/fabric-intro-part-5
Specifically this behaviour:

Multiple panable objects in one "canvas"
Zooming, while maintaining a scaled distance between the objects
Dragging of the entire "canvas", with e.g. alt + drag
Setting a initial layout of possibly nested objects

I've played around a bit with this library, but did not quite get the result I described above.
Could you maybe give me your assessment on wether you think this is doable with this library or if it is out of scope?

Thanks for opening an issue. This should be doable with multiple instances of Panzoom on the objects as well as a wrapper instance of Panzoom. Without trying it myself, I can't say for sure. But the default behavior is close aside from the multiple draggable children.

By the way, I think this example should be very close to what you're looking for. It has a panzoom instance within another. The zoom is controlled by the outer Panzoom. And more children could be added. In the case of clicking zoom buttons, I showed how to modify the children translations so that the child elements don't move when the parent is scaled.

Thanks for the fast response @timmywil
The linked example was the first one I tried to build upon but didn't quite get to where I wanted.
Your suggestion of a wrapping Panzoom instance did the trick for me tho.
This is roughly what I ended up:
js
/*
wrapper element acting as a canvas, to enable panning of the background
contain: outside to avoid panning out of boundaries
startCoordinates - (viewwidth|viewheight * (n-1)), where n is the vw/vh specified in the wrappers css to position it in the center
*/
const wrapper = panzoom(wrapper, {canvas: true, contain: "outside", startX: -window.outerWidth*2, startY: -window.outerHeight*2});
// attach zoom event listener to the wrapper element to keep relative positioning of child elements
root.addEventListener('wheel', (event: WheelEvent) => {
  wrapper.zoomWithWheel(event)
});

// create panzoom instance for every nested element
elements.forEach(element=> {
  panzoom(element);
});

html

      ...
      panzoom-elements
      ...

css
.root {
  width: 100vw;
  height: 100vh;
}
.wrapper {
  width: 500vw;
  height: 500vh;
}
/*absolute styling, to avoid the need for nested flexbox elements */
.panZoomElements {
  position: absolute;
  ...
}

The only minor issue is, that the drag behaviour of the nested elements is a little finnicky, when zoomed out all the way, which is due to the contain-option I assume?

Glad you got it working! I'm not sure about the nested elements. Doesn't look like you put contain on those. However, perhaps it's because the parent is scaled? If that's the case, the example linked above shows how you can override setTransform to divide the nested element transforms by the wrapper scale.


prioritize properly mapped read pairs.

I tested a read pair that both R1 and R2 are exactly reverse complement. This sequence has multiple copies in the reference genome. When mapped the read, strobealign return a paired of location that not properly mapped (flag 81 and 161). They are located in different chromosomes. But it would be more reasonable if a properly aligned location can be returned.
LH00139:14:225FNWLT3:3:1101:9636:13914  81      Nbe.v1.s09440   36226   60      94M     Nbe.v1.s11880   21622   0       CTTGTGTGTTGGCCGAAGTTACAATAGTAACATTGAACCCTCGAATATGTTCGAAGATCTCGAAATGATCTTCCAGTTCGGGGGAGAATTCGCA  FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF5FFFFFFFFFFFFFFFF5FFFFFFFFFFFFFFFFFNM:i:0  AS:i:208
LH00139:14:225FNWLT3:3:1101:9636:13914  161     Nbe.v1.s11880   21622   60      94M     Nbe.v1.s09440   36226   0       CTTGTGTGTTGGCCGAAGTTACAATAGTAACATTGAACCCTCGAATATGTTCGAAGATCTCGAAATGATCTTCCAGTTCGGGGGAGAATTCGCA  FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF5FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFNM:i:0  AS:i:208

Excellent with these kind of examples, thank you!
Could we also get the reference sequences you are mapping to? it would be easier to fix if we can reproduce it.
Also, do you have an idea of what the expected alignment should be? For example:
(i) Is the read pair from a duplicated region where they should map to the different copies (so R1 and R2 should map close to each other but not to the exact same position).
(ii) or are they sequenced in forward and reverse from the same short fragment (i.e., should map to the exact same reference position)

Btw, I should say that strobealign prioritises properly mapped reads, so this is likely a bug or some weird corner case.

Hi @ksahlin, thank you for your reply. The testing file is attached.
query_seq.zip
Ref

R1 and R2 are reverse and complement, so they should be mapped to exact the same reference position, rather than two different positions.

I see. Thanks for providing us with data.
Currently strobealign checks if two reads within a pair is proper or not. This is based on deviation from insert size etc. A guess is that the paired mapping to the same chromosome was too far out the threshold and was deemed 'not proper pair'. All non-proper pairs are scored the same, which could be why this happens. We could have a more nuanced approach that also takes the closest ones, occurring on the same chromosome, etc if all are non-proper.

Thank you for the information. But in this example, the distance of read pair is very close and not 'far away'. I am curious why this one was deemed 'not proper pair'.

Yes, I hope @marcelm can take a look in the coming week.
I should have clarified it better: I meant too far away from the mean of the fragment size distribution, i.e either < mu - X*sigma or  > mu +X*sigma. But we will se wht the data says.

Interesting! I debugged this and found the following. Both reads produce 316 seeds (NAMs) and both of the lists contain both of the mapping locations that we see in the SAM output (the one on Nbe.v1.s09440 and Nbe.v1.s11880). However, the program refuses to pair the seeds up. This happens in get_best_scoring_NAM_locations().
This is the relevant code:
https://github.com/ksahlin/strobealign/blob/93be0c284667bcac04f17e469b0114d881c30a29/src/aln.cpp#L612-L618
All except one condition are fulfilled:

:heavy_check_mark: Both seeds are on the same reference
:heavy_check_mark: One seed is forward, the other reverse
:heavy_check_mark: The distance between start positions is less than $\mu + 10\sigma$ (these have default values $\mu=300$ and $\sigma=100$ since this is the first read pair we look at)
:x: The distance between start positions is either positive or negative (a < b or b < a). This is not fulfilled since a==b in this case.

The result is that neither r1_r2 nor r2_r1 become true. The fix is to use <= and >= as comparison operators. I haven’t done this because there are some other places where this same or nearly the same code is repeated and I would like to factor this out into a function instead.
Also, we should think about the case that reads overlap in this way:
R1         -------->
R2   <--------

That is, the TLEN is shorter than the read length. Then the read ends should contain adapter sequence and should not contain seeds, but possibly it could happen by chance that a seed extends a little bit into the adapter.
Note that the above does not check whether TLEN is < mu-X*sigma, not sure if this is as intended.

Great, so <= and >= sounds good for fixing this.

Also, we should think about the case that reads overlap in this way:

Is this something that happens with PE reads? I did not implement a check for TLEN is < mu-X*sigma because I did not know about that case. Aren't a the mates read from opposite ends of a fragment?

This bug remind me the similar issue in bwa (https://github.com/lh3/bwa/pull/259).

cutadapt can get rid of most of the 'readthrough', but still some exceptions.

the default minimum match of adapter is 3,and 1 or 2 base would remain if the readthough part fall with this range.
sometime first several bases at the 5' of the reads are removed due to low complexity or extracted as UMI. This would also cause artificial readthough.

cutadapt can get rid of most of the 'readthrough', but still some exceptions.

Yes, sure. Sorry if it sounded like it; I didn’t want to suggest that using cutadapt would be the solution here. This needs to be fixed in strobealign.

On another note: Maybe it would be good to change strobealign such that it actually makes computations based on the insert size (TLEN).

Okay @marcelm , you know more about me on this one so I'll let you decide. I thought insert size was the inner distance between reads (negative in above case) and fragment length the length of the sequenced fragment (always positive), and furthermore, that TLEN was equal to fragment size.

I understand and remember being confused about this as well. Quoting from http://thegenomefactory.blogspot.com/2013/08/paired-end-read-confusion-library.html:

The main confusion is with "insert size". The name itself suggests it is the unknown gap because it is "inserted" between R1 and R2, but this is misleading. It is more accurate to think of the insert as the piece of DNA inserted between the adaptors which enable amplification and sequencing of that piece of DNA.

"Fragment size" appears to be a bit ambiguous because for some people, the fragment includes the adapters at either end, but I consider the terms insert size, fragment size and TLEN to mean the same thing.

With the fix suggested in #319, SAM output for the test read pair is as follows:
LH00139:14:225FNWLT3:3:1101:9636:13914  83      Nbe.v1.s00140   66357181        0       94M     =       66357181        -94     TGCGAATTCTCCCCCGAACTGGAAGATCATTTCGAGATCTTCGAACATATTCGAGGGTTCAATGTTACTATTGTAACTTCGGCCAACACACAAG       FFFFFFFFFFFFFFFFF5FFFFFFFFFFFFFFFF5FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF  NM:i:0  AS:i:208
LH00139:14:225FNWLT3:3:1101:9636:13914  163     Nbe.v1.s00140   66357181        0       94M     =       66357181        94      TGCGAATTCTCCCCCGAACTGGAAGATCATTTCGAGATCTTCGAACATATTCGAGGGTTCAATGTTACTATTGTAACTTCGGCCAACACACAAG       FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF5FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF  NM:i:0  AS:i:208

Hi @marcelm, I quote the specifications of TLEN in the sam format document, which might be helpful.

The intention of this field is to indicate where the other end of the template has been aligned without needing to read the remainder of the SAM file. Unfortunately there has been no clear consensus on the definitions of the template mapped start and end. Thus the exact definitions are implementation-defined.

In this criteria, the TLEN equal to the distance between R1 original start (R1 mapping start) and R2 original start (R2 mappinig end). But I found most tools calculate the TLEN as the distance of R1 mapping start and R2 mapping start.

Thanks for the pointer. So if we come back to the terms, only the older TLEN definition ("TLEN#1") would be equivalent to "insert size". (And of course TLEN can be negative as well.)
I fixed the TLEN computation in strobealign a while ago. I did not take into account the ambiguous scenario above at the time, but I checked the code again just now and it appears that we actually compute TLEN#2. That said, the scenario cannot arise in strobealign at the moment because reads that overlap in such a way are not treated as a proper pair.
This issue here is about the special case that R1 is identical to the reverse complement of R2, which will be fixed by #319, so to keep track of the more general issue of allowing the above "ambiguous scenario", I have opened issue #320. I suggest we continue discussion over there if necessary.

Hi @marcelm, I would like to thank you for this important update. Using the lastest branch. The ratio of properly mapped ratio increases significantly. (79.71% to 97.76%)
Before:
72320436 + 0 in total (QC-passed reads + QC-failed reads)
72320436 + 0 primary
0 + 0 secondary
0 + 0 supplementary
0 + 0 duplicates
0 + 0 primary duplicates
71695228 + 0 mapped (99.14% : N/A)
71695228 + 0 primary mapped (99.14% : N/A)
72320436 + 0 paired in sequencing
36160218 + 0 read1
36160218 + 0 read2
57643528 + 0 properly paired (79.71% : N/A)
71555882 + 0 with itself and mate mapped
139346 + 0 singletons (0.19% : N/A)
13351896 + 0 with mate mapped to a different chr
12795310 + 0 with mate mapped to a different chr (mapQ>=5)

After (I do not rerun all the data):
2739779 + 0 in total (QC-passed reads + QC-failed reads)
2739779 + 0 primary
0 + 0 secondary
0 + 0 supplementary
0 + 0 duplicates
0 + 0 primary duplicates
2720020 + 0 mapped (99.28% : N/A)
2720020 + 0 primary mapped (99.28% : N/A)
2739779 + 0 paired in sequencing
1369890 + 0 read1
1369889 + 0 read2
2678499 + 0 properly paired (97.76% : N/A)
2716063 + 0 with itself and mate mapped
3957 + 0 singletons (0.14% : N/A)
23214 + 0 with mate mapped to a different chr
15532 + 0 with mate mapped to a different chr (mapQ>=5)


#8303 set "Automatically end long idle transactions" to the true by d…

…efault; add wiki links

Option state isn't saved for connection types
https://user-images.githubusercontent.com/31996417/230039492-323c1855-95ae-4f78-971f-6e1186f8a730.mp4

Option state isn't saved for connection types

Fixed

the value changes are not saved if option state wasn't changed
https://user-images.githubusercontent.com/31996417/232028795-b60a1a65-ec84-4a52-b424-06b4d75ced43.mp4

the value changes are not saved if option state wasn't changed

fixed

the option isn't working in case it was set in data type preferences only.


Still maintained?

I'd like to use this but the pre-build binaries available seem to be for old node versions, the build is pretty ropey in our CI env - needs 4gb of c++ libs on windows agents etc...
The outstanding pull request is needed but hasn't been merged...
So just questioning if this is still maintained?

@rsouthgate this repo is still maintained, thanks for bringing this for our attention, I will take a look at existing PR and merge soon

@hectorhdzg Could you please build and publish the binary Assets? Last release version had 24 Assets. I'm seeing the following while running npm i in a node alpine container:
node-pre-gyp ERR! install response status 404 Not Found on https://github.com/Microsoft/ApplicationInsights-node.js-native-metrics/releases/download/v0.0.6/native_metrics-v0.0.6-node-v64-linux-x64-musl.tar.gz
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, musl) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error response status 404 Not Found on https://github.com/Microsoft/ApplicationInsights-node.js-native-metrics/releases/download/v0.0.6/native_metrics-v0.0.6-node-v64-linux-x64-musl.tar.gz

I'm required to add the following to my Dockerfile because of the binary assets not being published with the last released version:
RUN apk add git python make g++

New version including assets must be available now, let us know if you have any issue


Make builder API play well with hydration

Codecov Report

Merging #322 (df0e314) into master (7dc8cbe) will decrease coverage by 0.12%.
The diff coverage is 44.64%.

@@            Coverage Diff             @@
##           master     #322      +/-   ##
==========================================
- Coverage   73.33%   73.20%   -0.13%
==========================================
  Files          38       40       +2
  Lines        5337     5431      +94
==========================================
+ Hits         3914     3976      +62
- Misses       1423     1455      +32

Impacted Files
Coverage Δ

packages/sycamore/src/builder/agnostic/mod.rs
65.55% <38.77%> (ø)

packages/sycamore/src/utils/hydrate/mod.rs
79.16% <85.71%> (ø)

website/src/main.rs
0.81% <0.00%> (-0.01%)
:arrow_down:

packages/sycamore/src/builder/html.rs
100.00% <0.00%> (ø)

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 7dc8cbe...df0e314. Read the comment docs.


update API Management service creation form image

This PR is to update screenshot/illustration of Azure Management service creation.
This is the latest display on 30th July, 2018:

The current pricing tier combobox (drop down) contents:

I have updated the image and it's still using the same filename.
Please review.

@Erikre - Would you review the proposed changes, or suggest an author to review? Thank you.

Possibly @Juliako or @vladvino would be the best people to review this change. It looks like an image update to the following topic:
https://docs.microsoft.com/azure/api-management/get-started-create-service-instance

@Erikre

It looks like an image update to the following topic:
https://docs.microsoft.com/azure/api-management/get-started-create-service-instance

Yes, that link is correct. It is an image update, because the pricing tier has changed.

#sign-off

@vladvino: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

@Juliako: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Eclipse gate wand "eclipse gate" activation deletes iron doors.

Just Dire Things version
1.2.2
Minecraft Version
1.21.1
(Neo)Forge Version
21.1.37
Modpack & Version
No response
Do you have optifine or Rubidium installed?
None
Describe the issue
Eclipse gate wand "eclipse gate" activation deletes iron doors.
Steps to reproduce

Minecraft instance as described, only forge + jdt
new world, give necessary blocks + elipsegate wand.
construct a planks wall, 5 x 4, and put an iron door in the middle of it.
trigger eclipse gate in a 3x3x1 area that includes the iron door. planks + door phase out. I did set the range of the eclipse gate to 1 block deep.
observe only planks remain when blocks phase back in. Can walk right through where the door used to be.

Expected behaviour
Iron door isn't deleted.
Screenshots
No response
Log files
No response
Additional information
No response

I think the best solution here is to just blacklist all doors :). This should be done in the next version of the mod. Note: If you eclipse gate the block under the door, just like with torches, the door will pop off as an item entity.


Query-frontend: Set value of remote_user field in Slow Query Logs from HTTP header

Fixes #7360.

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Add new flag to query-frontend command.
Set value of remote_user field from header if flag is set.
Update docs.

Verification

Build thanos binary locally.
Start Query Frontend component. Point it to an instance of Query. Pass --query-frontend.slow-query-logs-user-header=<test header> and --query-frontend.log-queries-longer-than=1s.
Send request to /api/v1/query that contains a long-running query and sets the <test header>.

Thanks for the review and the merge.


Update comments to match function parameter names

The parameter names of some functions (in particular, BN_is_negative and EC_KEY_get0_engine) in the comments didn't match the ones in function prototype. I've updated the comments so that they're in agreement.

Checklist

[X] documentation is added or updated
[ ] tests are added or updated

I'd argue that it's trivial change. In such case you'd be advised to amend commit message adding CLA: trivial line and force push. This would remove "cla-check" and review can proceed.

@dot-asm Yep, that's what I thought. Updated.

richsalz removed the need-cla label

So it's not done automatically? Ought to be bug...


Upgrade Chakra-UI to latest version

Upgrade Chakra-UI to latest version and test/check for any negative side effects/errors.

The following error is received on updating Chakra UI (and it's dependencies) to the latest version:
./node_modules/framer-motion/dist/es/components/AnimatePresence/index.mjs
Can't import the named export 'Children' from non EcmaScript module (only default export is available)
This problem is documented here - mjs is not supported by default in CRA v4. This has been fixed in v5.
We can either upgrade to CRA v5, use Framer Motion v4 (although this seems unwise as it is a dependency of Chakra UI), or attempt to patch/override the webpack config to add support for mjs.


Support ledger wallet

Issue with one relayer-operator using wallet mode —> I think it's actually 10, 5, 0, 0 (encoins minted). And flow was, Wallet tab -> 'Coins to Mint' (10, 5, 5, 0, 0) -> 'Send Request' (Worked) -> Transfer tab -> Select 10, 5, 0, 0, and 'Send to Wallet' -> Copied codes to Word page -> Sent and recieved in Eternl HWW -> switch active wallet -> reload page -> import (which I tried just 1 key)
Even pasting the minting key their encoins were not showed up
He tried with a regular wallet and worked smoothly
So probably Eternal with hardware wallet is generating some trouble
Yeah, it might be something with HWW's. Sent hot -> hot and worked perfectly, then hot -> HWW and getting the same issue where I click import and nothing comes up.
edit You are able to send hot -> HWW -> hot and be able to import them. The HWW user just wont be able to tell what the value of each NFT is unless the sender tells them which key goes with each asset ID.

A similar bug report and the solution.
Proposal
Library
Bundle it 1
Bundle it 2


Add google-cloud-cpp v2.29.0

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (google_cloud_cpp) have been updated in this PR. Please review the changes.

@meteorcloudy @Wyverald could you please trigger the tests? Thanks.

@meteorcloudy @Wyverald could you please trigger the tests? Thanks.

Done.


create hello_jaborjkath.php

Hi @jaborjkath!
Thank you for contributing to hello-open-source. Merging your PR. Keep learning!

Thank you too @siddharth2016 for an awesome guide and PR approval. Yes, will definitely do.


fix: add totalCount to show the number of rows

Description
Add totalCount props to show the number of rows. Now the totalCount is returned from the useTable hook.

Let's mark it as fix since it's something that I forgot to add!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


inappropriate Text Style

The text style in the hero section requires modification for better suitability.

Before update

after update


Update support.md

@HeidiHanZhang : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Tribe rosters

Add tribe rosters, at least original tribes. Not sure how to do post-merge tribes

Tribe colors now outline profile pictures in https://github.com/mawilson/hello_django/pull/27, but an actual page for tribe rosters is yet to be implemented

Add Tribe name to Survivor card, which would also require adding tribe names to the Season model (so they're not just Blue, Yellow, Red)

Tribe colors & names have been added in https://github.com/mawilson/survivor_draft/pull/84, but are not yet visible in Survivor card. That's an easy addition. A tribe roster feature would probably want to be its own page though.

Added tribe name & color to Survivor page in https://github.com/mawilson/survivor_draft/pull/87.
Thinking I may make a different view on the homepage to sort by Tribe instead of Team, rather than a brand new page.

Tribe view on homepage implemented on https://github.com/mawilson/survivor_draft/pull/88.


Fields inherited from a base can now be narrowed.

This is useful if, for example, an inherited object specifies a
string literal in place of the more generic 'str'.
Shape(type=str)
Rectangle(type='rect')
Triangle(type='tri')

fixes #108

Looks good to me.
In theory it should be possible to raise an error if a conflicting type is used in a base class (e.g str & int) since this will create a schema that won't validate against anything. But this is good enough for now.

Thanks!


James m ballard patch 1

This should fix the mac build errors. My mac is currently committed to other work, so I am testing on Windows. I'll merge as long as the Windows build is good.


Don't eat lines with pure prompt

Don't know if this change is correct, but it fixes this issue for me.
Fixes #1524

Yes, this probably fixes the issue with pure, but I'm not sure this should not be removed, as this code was added to fix other functionality, though I don't remember what exactly.
CC @johnpneumann

I'm gonna take a look at this again and see why this theme and this particular issue continues to crop up. There have been quite a few issues around editor in particular since zsh 5.3.1 (I won't rehash that convo here but feel free to check #1488 and #1245) came out which introduced the awesome bug that it did. I've seen several issues lately that's focus was around the editor module in particular.
For context, when this was added in, it was to address #1246 (merged in #1272). In the ticket it says that the issue is related to 5.3.1 which is known to have issues. I wasn't paying attention at the time and didn't know about the regression. As such it may be that this is no longer needed. However if that is the case there's still the issue of not seeing the context for your commands.
I have some time this weekend and will look at this further then. If anyone else wants to hack on it before then to see what works across zsh versions skipping the ones we know are bad (I think there's more than 5.3.1 but I have to check my notes), across distros then just lemme know what you find and we can work it out together.

Thanks for taking a look. This is one part of the codebase I haven't messed with much just yet.
One thing worth noting: I'm also seeing a different oddity with zsh 5.4.2 on macOS:
~ % if true; then
then else> echo true
then else> else
else else> echo false
else else> fi
true

Moving the conversation back here since most folks are not going to care about the how or why with this. So I looked at this further today. The proposed fix does fix it for pure, but it breaks the following prompts:

damoekri
giddie
kylewest
paradox
sorin

This is due to the way that the fix introduced in #1272 was put together. It addressed the bug, but it was addressing a bug that we believe to be an issue with 5.3.1 (among other things that have been fixed over the last several months). To get on with it, the fix should be more specific and should address specific prompts that utilize editor_info, which are basically all of the prompts listed above. No other prompts do not utilize the editor_info, so they do not need to be reset. The biggest reason that we cannot merge this "as is", is due to the fact that it will break the sorin prompt (the default) and that's gonna give users a bad experience, so we don't want to do that.
There are 7 files that need to be updated to address this, (6 prompts [addressing smiley as well, because it's affected by the issue referenced] and the editor). I've tested the fixes all day and found everything to work as expected. Additionally, we should deprecate the ps-context, since the default should be to show it - and the only reason it wasn't being shown was because of how we were handling it to begin with.
@yevhen-m - I don't want to stomp on your current PR, so I can throw the diffs of what I changed to make it work on here, I can just point you to the bits that need to change, or I can throw a PR together. Let me know what you'd like to do and we'll get it handled.

@johnpneumann thanks help. Let's close my PR so you can create a new one with all appropriate changes.

Rgr. I'll have the PR later tonight or tomorrow morning.


[Bounty]Setup Request

I would like to pay a bounty for someone to setup Tribeca on a Docker VPS for me so I can just use a webpage link from any browser anywhere and with a username and password to login. After I login in I want to go directly  to the webclient of Tribeca. I will need to have instruction on how to add api connects to my exchanges.
Bounty = $30 USD

Hello,
Contact me on skype: joey.antonisse
Greetings

Any luck with this?
Would like the same help. Not sure how to set this up.

Hello @briancrypto2076 and @JPAntonisse
If you guys get any success, I'm also willing to pay / donate some btc bugs.
here is my raised issue: #219

Everyone,
I have moved on from Gekko.. just to much for me.. I prefer to keep it
simple stupid.. moved on to this..
Making 3% daily on my portfolio over 5 pairs using this method...
https://youtu.be/wd2zmwzPE2w
I have tried...
This method of auto trading (to me) is easy to setup, most profitable in
all markets conditions and is the cheapest even with a VPS ($5 mth),
subscription to tradingview ($15mth) and subscription to autoview ($5 mth).
Tradingview script that I am using:
https://www.tradingview.com/script/87e6lWzO-Best-Holy-Grail-Indicator-V3/
https://l.facebook.com/l.php?u=https%3A%2F%2Fwww.tradingview.com%2Fscript%2F87e6lWzO-Best-Holy-Grail-Indicator-V3%2F&h=ATMlJt704S9p1e5IfVlev0G8WqSw2wm6FJj9jcivS74e_F5OQqssW0uul9ZpUHiUrqprqrcOKHvTt4p_RGpx5WYl5IBTHkdlevKfoiBfEUQYJxFpjWxv5YSwYL3Rj3O_fV83_FrjaPmMsVxRFt7NPA9gtLMJ
Setup on a VPS from here:  https://clients.databasemart.com/cart.php?a=
confproduct&i=1
Got Autoview extention from Google Chromeand use this coupon:
B3CAB-1XTCP-YWEHH
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
Thank You --
Brian Sinclair
c:<PHONE_NUMBER>
On Wed, Feb 14, 2018 at 7:37 AM, Shahid Rasool<EMAIL_ADDRESS>wrote:

Hello @briancrypto2076 https://github.com/briancrypto2076 and
@JPAntonisse https://github.com/jpantonisse
If you guys get any success, I'm also willing to pay / donate some btc
bugs.
here is my raised issue: #219
https://github.com/michaelgrosner/tribeca/issues/219
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/michaelgrosner/tribeca/issues/213#issuecomment-365594787,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AicOvcZGsUJ805KUcei6xlRxcXtqpvq-ks5tUtOQgaJpZM4R4XA8
.


Feature/home page text

 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.:x: shekharncst sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Outside Circle rectangle display

Hi Denis Bogatyrev,
Thank you for the DBMapSelectorViewController. It is working good in previous versions up to iOS 9. But when it is running in iOS 10 the rectangle around the circle in segment OutSideCircle. It is not covering all the map. You can se the difference in the below screen shots. I tried to make that spread to all the map but i can't solve.
Please update the code about this issue. Thanks in advance.
Hemaraju.
iOS 10

iOS 9

Hi Denis Bogatyrev,
Can you please update the repo.
Thanks,
Hemaraju.

Same issue in iOS 9.3

I have the same issue in iOS 9 and 10.
How I can solve?
Thanks

Any update on this ?

Find 'DBMapSelectorOverlayRenderer.m' ---> #pragma mark Helpers --->
add new function:

(BOOL)canDrawMapRect:(MKMapRect)mapRect zoomScale:(MKZoomScale)zoomScale {
return true;
}


Unresolved callback on FindAppropiateTeam

Don't exists else for this [if] (https://github.com/howdyai/botkit/blob/master/lib/SlackBot.js#L154 ) and callback is not called

I found fixed this in #887 before realizing it was recorded as an issue. I included the team_id there which is an added help for debugging.

@peterswimm 🚫


Add support for %load magic

Fixes #581.

Looks great to me, and works well. Thanks!

Nice, 75% complete on the 0.9 Milestone, w00t!


While rubocop -a reports having corrected a problem it does not change the file

16:32 % cat xxx.rb                                                                                2016-07-05 16:32:05
module M
  def a # rubocop:disable Metrics/MethodLength
    puts 'M'
  end
end
robert@rd ~/tmp   (ruby-2.3.1@tools), node-5.6.0
16:32 % rubocop -V                                                                                2016-07-05 16:32:09
0.41.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-linux)
robert@rd ~/tmp   (ruby-2.3.1@tools), node-5.6.0
16:32 % rubocop -a xxx.rb                                                                                2016-07-05 16:32:14
Inspecting 1 file
W

Offenses:

xxx.rb:1:1: C: Missing top-level module documentation comment.
module M
^^^^^^
xxx.rb:2:9: W: [Corrected] Unnecessary disabling of Metrics/MethodLength.
  def a # rubocop:disable Metrics/MethodLength
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1 file inspected, 2 offenses detected, 1 offense corrected
robert@rd ~/tmp   (ruby-2.3.1@tools), node-5.6.0
16:32 % cat xxx.rb                                                                                2016-07-05 16:32:19
module M
  def a # rubocop:disable Metrics/MethodLength
    puts 'M'
  end
end

I cannot reproduce this using 0.41.1, after copying your code. It correctly removes the disable for me.

@Drenmi  I guess you meant 0.41.2 where it is fixed :+1:

@RobertDober: I had RuboCop version 0.41.1, but from master branch, where the fix was already committed. 😁

Kudos for the fix BTW


Use PATCH API to update display names of entities

Describe the task
Currently we are relying on the PUT API to update the display names of the entities
This update does not happen when using ingestion-bot token for authentication
Instead we will use the PATCH API to updated the display names and use force override to update already present display names if any.

fixed by https://github.com/open-metadata/OpenMetadata/pull/14224


support a simple session without JWT

Similar to https://github.com/ueberauth/guardian/pull/297, I've been thinking about something like guardian, but without any tokens. For a simple session JWT is pretty heavyweight. I really like the common set of methods, but this might be too far out of Guardians use case. I basically just store the claims in the session and let Plug handle the security. Then there's a plug to load those claims into the private claims. This won't work for all cases (API, Channels, others?), but I think there's a use case for something that provides a simple serializer for storing a serialized resource and permissions in the session.

Thanks @ericsullivan, please see my comment on #297

Ok, thanks for taking a look. I figured Guardian wasn't the place for it but I wanted to check before creating another authentication solution

@ericsullivan there are quite a few authentication libraries already for Elixir, Guardian just happens to be the JWT solution.
https://github.com/h4cc/awesome-elixir#authentication


Remove the blocklist for aten.div.Tensor

Ticket

Resolves #479
Resolves tenstorrent/tt-metal#15630

Problem description
Remove the blocklist for aten.div.Tensor and make sure these cases pass
What's changed

[x] Further optimize tensor / const to tensor * (1 / const)
[ ] Resolve tensor / scalar where the scalar is unknown

This PR seems no remove the blocklist?

I cannot remove the block list yet.  It still does not work when the denominator is an unknown scalar.  This is the main reason why this PR is still a draft.

okok, thanks for remind

I'm marking this PR as ready.  I remove block list entries that should work since #507.  The failing cases are now tracked by tenstorrent/tt-metal#15630.  I also make them xfail test cases.

@swimdi, how do we regenerate the block list for status quo?  Some worrisome test cases seem to fall through.

You can

Disable all the blocklist
pytest tests/autogen_op/ALL to get which cases are failed
python3 tools/generate_guard_function_from_input_var_metric.py

@jerrysky3 is added some conv OOM case on to_tt_guard_autogen, so need to add it on the new one

Missing dependency?
/home/ubuntu/actions-runner/_work/_tool/Python/3.8.18/x64/lib/python3.8/site-packages/_pytest/config/__init__.py:329: PluggyTeardownRaisedWarning: A plugin raised an exception during an old-style hookwrapper teardown.
Plugin: helpconfig, Hook: pytest_cmdline_parse
ConftestImportFailure: ImportError: libGL.so.1: cannot open shared object file: No such file or directory (from /home/ubuntu/actions-runner/_work/pytorch2.0_ttnn/pytorch2.0_ttnn/tests/conftest.py)
For more information see https://pluggy.readthedocs.io/en/stable/api_reference.html#pluggy.PluggyTeardownRaisedWarning
  config = pluginmanager.hook.pytest_cmdline_parse(
ImportError while loading conftest '/home/ubuntu/actions-runner/_work/pytorch2.0_ttnn/pytorch2.0_ttnn/tests/conftest.py'.
tests/conftest.py:6: in <module>
    from tests.utils import calculate_accuracy
../../_tool/Python/3.8.18/x64/lib/python3.8/site-packages/tests/__init__.py:3: in <module>
    from ultralytics.utils import ASSETS, ROOT, WEIGHTS_DIR, checks, is_dir_writeable
../../_tool/Python/3.8.18/x64/lib/python3.8/site-packages/ultralytics/__init__.py:11: in <module>
    from ultralytics.data.explorer.explorer import Explorer
../../_tool/Python/3.8.18/x64/lib/python3.8/site-packages/ultralytics/data/__init__.py:3: in <module>
    from .base import BaseDataset
../../_tool/Python/3.8.18/x64/lib/python3.8/site-packages/ultralytics/data/base.py:12: in <module>
    import cv2
../../_tool/Python/3.8.18/x64/lib/python3.8/site-packages/cv2/__init__.py:181: in <module>
    bootstrap()
../../_tool/Python/3.8.18/x64/lib/python3.8/site-packages/cv2/__init__.py:153: in bootstrap
    native_module = importlib.import_module("cv2")
E   ImportError: libGL.so.1: cannot open shared object file: No such file or directory
Failure: Tests failed with exit code 4.


Support CSP Restrictions Without Nonce Attribute

I have a project that I would like to explore using a css-in-js library like glamor, glamorous, etc.
However, the Content-Security-Policy restrictions for this project are such that style tags are not allowed to be injected onto the page, and must be served in the initial static html file.
One solution around this would be to provide a nonce attribute. Styled components accomplishes this through an undocumented feature of webpack (See Issue).
However, IE11 does not support nonce attributes, and CSP Level 2 in general, which is a browser I need to support for this project.
More info on this security policy here.
I had an initial discussion on this topic on Twitter. But I believe the discussion on this problem goes deeper than Twitter allows for. NOTE: I had mentioned the idea of being able to target an already existing style tag, I believe I was mistaken, and I don't believe this will work for the given CSP restrictions.
I've created a minimal server example that should (theoretically) produce the same CSP restrictions for loading styles.
There may be another way around these CSP settings that are equally as secure, but allow for dynamic styles from a trusted source such as glamor. But I am not aware of any. (I'm also not an expert in CSP 😞 )
I suppose one way around this would be to somehow statically analyze the styles and generate a stylesheet at build time. However, I think we would lose out on one of the very powerful features of glamor and related libraries: dynamically re-defining styles based on props, state, and theming.

Came across this via a discussion about emotion. I am not using glamor, but thought I would add some thoughts, especially as I think people looking at stricter CSP rules is a good thing, and that css-in-js libs working within those envs would be great!
Two points:

IE11's CSP support is basically 0, so there's no point working around it. It will just ignore CSP rules regardless, so style tags will work just fine there.

I think a nonce is a good solution to this problem, as discussed https://github.com/emotion-js/emotion/issues/403 there are two approaches that I can see (one of which styled-components is using already).

This is a great point. I actually had a similar discussion on styled-components about this. I agree, the solution is a nonce attribute with the understanding that there's simply no way to prevent inline style injection in IE


Suggestion : add % to the crowns

Suggestion : add % to the crowns same way as % xp to next level.  This will give an indication for tree (course) completion

Is this the sort of thing you had in mind?

I have not bothered for the bonus skills as it would be fairly pointless, note that they are not included in to percentage calculation for normal skills either.

Sorry my request  was not clear. I meant % in total count, not in individual break down.  This is too much info :).
In the crown itself
249 / 320
77.8%

Ah I see. Putting the overall crown percentage shouldn't be too hard, just need to find a good place to put it. I'll have a play around and see.

Is this more like it?

yes ! you are the best !

This has been added in 638ccd8.


Re-check examples

Some1 needs to re-check the examples, they are not currently correct and doesn't correspond to the text.

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

ID: b777fcfe-c7da-0165-d6ac-74fc2db07daf
Version Independent ID: 12045879-94af-fd90-66a0-8f3db00a579f
Content: Get-PnPListItem (SharePointPnPPowerShell)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Get-PnPListItem.md
GitHub Login: @Techwriter40
Microsoft Alias: kirks

Hello @zumiek
how are you?
Even though the PnP documentation articles are on this repository, I need to kindly redirect you to open PnP cmdlets issues here, in that repository, where a member of the PnP community will be able to correct the article with your feedback.
Thank you!

@get-itips Thank you very much for the contribution and sharing this explanation. @zumiek Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

This seems to be an issue with Github/MSDocs (?) and the way it is parsing the code in those examples. The below text was copy/pasted from https://pnp.github.io/powershell/cmdlets/Get-PnPListItem.html, but on render it changes to the text seen on the page that is the subject of this comment thread.

EXAMPLE 5
Get-PnPListItem -List Tasks -Query "bd6c5b3b-d960-4ee7-a02c-85dc6cd78cc3"
Retrieves all available fields of list items based on the CAML query specified
EXAMPLE 6
Get-PnPListItem -List Tasks -Query ""
Retrieves all list items modified today, retrieving the columns 'Title' and 'Modified'. When you use -Query, you can add a clause to retrieve specific columns (since you cannot use -Fields)

This seems to be an issue with Github/MSDocs (?) and the way it is parsing the code in those examples. The below text was copy/pasted from https://pnp.github.io/powershell/cmdlets/Get-PnPListItem.html, but on render it changes to the text seen on the page that is the subject of this comment thread.

EXAMPLE 5
Get-PnPListItem -List Tasks -Query "bd6c5b3b-d960-4ee7-a02c-85dc6cd78cc3"
Retrieves all available fields of list items based on the CAML query specified
EXAMPLE 6
Get-PnPListItem -List Tasks -Query ""
Retrieves all list items modified today, retrieving the columns 'Title' and 'Modified'. When you use -Query, you can add a clause to retrieve specific columns (since you cannot use -Fields)


GraphLab Create requires a license to use in linux

Hello I have problem when I want ti import the bellows
I got the error ([ERROR] graphlab.connect.main:  ========================================
GraphLab Create requires a license to use. To get a non-commercial  license for academic use only, visit https://turi.com/register. )
I'm using Linux Ubuntu 18.04.6 LTS and python 2.7 and I have follow the whole procedure of [LINUX_INSTALL.md] and also I using anaconda and jupyter
I have installed theses
pip install -U turicreate
would you please help me
import graphlab as gl
import numpy as np
import graphlab.aggregate as agg
import matplotlib.pyplot as plt
from matplotlib import rcParams
import datetime as dt
from graphlab import degree_counting
from graphlab import connected_components
from graphlab import pagerank
from graphlab import shortest_path
from graphlab import triangle_counting
from graphlab import label_propagation
from graphlab import kcore
from graphlab import graph_coloring

You should be using TuriCreate not GraphLab Create. TuriCreate is open source (i.e. does not require a license).
TuriCreate is largely a drop-in replacement for GraphLab create. Once TuriCreate is installed, all of these imports should work if you just replace graphlabwith turicreate.


[ADDON] - Flux

Add support for Flux as an add-on
https://fluxcd.io/docs/installation/#bootstrap-with-terraform

Hi. Is any work being done in this area?
Flux seems quite easy to add (I already installed as helm chart via the addon (singular) module, and the existing examples make already establish a pattern for software with more than 1 SA).
My questions would be more on which (if any) policies to add - I can think of at least ECR pull permissions on the same account and region for the different controllers).
The bootstrapping could be outside the scope, unless there is a simple way to include it here.
@bryantbiggs, not sure if this one is with you, but if there is any work being done, and you (or others) could point me there, I could try to contribute in the next few days, as I will be doing exactly this internally.
Thx

cc: @csantanapr who is working on this.
@joaocc we are working on finalizing the revamped design and blueprint for gitops-bridge which will include support for both argocd and flux. You can expect flux support to be added soon after. In the mean time, if you want to take a stab at adding a module flux I think we would welcome that.

I was trying to understand if there were any changes to the current v1.9.x design of this blueprints addons model (ie, single monolith-file), and/or if there was any branch already being tested. If not, I can try to create one.
I was also thinking of proposing one as submodule to be called from the current main.tf (similarly to how https://github.com/aws-ia/terraform-aws-eks-blueprints/tree/v4.32.1/modules/kubernetes-addons were structured, but only creating one level) to get feedback. If current maintainers prefer to stick with a single file, it would be trivial to merge that content back to main.tf, as I would keep variables and outputs identical.

I am almost ready to push an initial draft version to a branch fork.
I have a current question regarding IRSA.
As per the documentation (https://fluxcd.io/flux/installation/configuration/workload-identity/#aws-iam-roles-for-service-accounts), the recommended approach is to create 2 separate IAM roles (mapped to 3 of the 6 controllers).
While I would prob 3 being better for environments with more strict separation needs, I would like to keep with the recommended approach.
However, in my understanding, the terraform-aws-eks-blueprints-addon module only supports one IAM role for each addon.
If it is confirmed that we can only map one role in terraform-aws-eks-blueprints-addon, and wanting to keep with the same approach as others (like aws_efs_csi_driver) it seems that we will be forced to have to map all 3 controllers to the same same role Also tried to find how to do this mapping and wasn't able to find it, since terraform-aws-eks-blueprints-addon only receives a list of annotations on which it will write the value of aws_iam_role.this[0].arn.
Is my understanding correct (@csantanapr,  @askulkarni2, @bryantbiggs)? Or is there any way to configure terraform-aws-eks-blueprints-addon to create different roles?
I already created an issue (https://github.com/aws-ia/terraform-aws-eks-blueprints-addon/issues/23) to discuss the ability to specify extra roles, but it doesn't seem to be easy to do in a way that doesn't break compact with existing modules. Maybe @bryantbiggs can support a v2.0 where this is possible (will add ticket to discus this)
In the meanwhile I will try to propose an initial implementation with a single role, as it is better than not having any :)

I think this is getting to be a bit of an xy-problem. What method have you taken to enable FluxCD support?

I originally installed it directly via the addon module as it wasn't supported in the addons module.

Per FluxCD:

The recommend way of installing Flux on Kubernetes clusters is by using the bootstrap procedure.

Of which they list two methods - CLI or w/ their Terraform provider. Why not use the bootstrap Terraform resource https://registry.terraform.io/providers/fluxcd/flux/latest/docs/resources/bootstrap_git ?

Per FluxCD:

The recommend way of installing Flux on Kubernetes clusters is by using the bootstrap procedure.

Of which they list two methods - CLI or w/ their Terraform provider. Why not use the bootstrap Terraform resource https://registry.terraform.io/providers/fluxcd/flux/latest/docs/resources/bootstrap_git ?

Yeap this is what we use. There is no need to have Flux bootstrapibg as part of this project.

There is no need to have Flux bootstraping as part of this project.

I'm starting to agree with that as well - I hadn't spent time looking into FluxCD integration until just now and after seeing this, I wonder what value it provides here
What permissions do the Flux IRSA roles require? Are those going to be custom to each specific implementation?

@brenwhyte, if you look at the same doc (https://registry.terraform.io/providers/fluxcd/flux/latest/docs/guides/install-helm-release) the also mention some valid reasons for installing it via helm:

Using the Flux Helm chart is a better option when Flux needs to be installed without any bootstrap configuration. The Helm Terraform provider can be used to install the chart in a Kubernetes cluster. Custom resource definitions will be applied along with all of the Flux components.

For context on why we are not using the CLI for bootstrapping, and this may be just our case, and maybe for historical reasons (or lack of time to find a better way), bootstrapping from CLI to a git repo which already had content lead to flux creating a parallel git tree, which forced us to manually fix it. At some point we decided it was better to install via helm early in the EKS setup process, and create the secrets and initial seed resources via kubernetes_manifests.
Since we were already using other addons we are already deploying (cert-manager, external-dns, external-secrets, efs-csi, cluster-autoscaler, ...), and since the IRSA bits are very nice in avoiding work and ensuring consistency, we started looking into adding it to the addons, and sharing this back to the community.
If @brenwhyte, @bryantbiggs and others in the project think approach of installing FluxCD doesn't have a place in the addons project, we can simply retire the PR, and keep using it in our fork for internal uses.
Answering to @bryantbiggs question, the documentation of FluxCD defines 2 roles on 3 sa-accounts, for access to ECR and for access to KMS (https://fluxcd.io/flux/installation/configuration/workload-identity/#aws-iam-roles-for-service-accounts).

From their docs

to grant Flux controllers access to cloud resources such as container registries, KMS, S3, etc.

I don't think those are the permissions they are stating as required, but just as examples

Yes, indeed not required, but certainly arguable that accessing ECR (and OCI on S3) via IRSA would be one of the desired goals when deploying to EKS.
Providing useful defaults for these common cases (as for instance is done on the AWS EFS CSI driver where access is given to all EFS on the AWS account - maybe a bit excessive but prob a very useful default for most users but overridable via some vars)
Based on the feedback above, I did a bit more research, and there seems to be a set of work in providing exactly this type of capabilities for FluxCD (listed in https://github.com/fluxcd/flux2/issues/3003 and in the docs https://fluxcd.io/flux/security/contextual-authorization/).
in the case of AWS, they identify and support 7 use cases, 5 of them already supported.
The original roles/SAs i mentioned above (source, kustomize, image-reflector) are indeed mentioned. some use cases seem generalizable (source-controller obtaining charts via OCI and Helm access to ECR), kustomize accessing SOPs secrets with KMS, and image-reflector also with regards to ECR...
Supported      Source Controller             Bucket Repository Authentication      Guide
Supported      Source Controller             OCI Repository Authentication         Guide
Supported      Image Reflector Controller    Container Registry Authentication     Guide
Supported      Kustomize Controller          SOPS Integration with Cloud KMS       Guide
Supported      Source Controller             Helm OCI Repository Authentication    Guide
Not Supported  Source Controller             Git Repository Authentication (RO)    fluxcd/source-controller#835
Not Supported  Image Automation Controller   Git Repository Authentication (RW)

But maybe I am not understanding the intent or strategy for the addons, and I am trying to add things that are too specific.
What would be the desired approach for adding FluxCD to the addons?

The future plan is to not include argocd or fluxcd in this terraform module.

ArgoCD addon/feature would be remove in the next major release 2.0 of this module.
FluxCD user space patterns would be provided similar as the ones we already have for argocd using new gitops-bridge pattern


Support request to the Meteor issue queue.

Using public/private email while authenticating on apinf with github

Here is the support request to meteor project - link