My 3 questions

Is msrp software only for asp.net users?
I think it can only be started from the asp.net project now.
can start msrp from the command line?
Will it provide the command line to load the listener, cluster, and endpoint parameters?
Even more complex commands?
Some details
How many backends are in your application?

[ ] 10+

How do you host your application?

[ ] Other Hosting model (please include details below)

What did you think of YARP?
good

Is msrp software only for asp.net users?
I think it can only be started from the asp.net project now.

YARP is first and foremost a library for use with ASP.NET Core. However...

can start msrp from the command line?
Will it provide the command line to load the listener, cluster, and endpoint parameters?
Even more complex commands?

Yes. Today if you follow the Getting Started guide the end result will be an exe that you can run from the command line and configure via a the appsettings.json file. See this sample as well.
In the future we do plan to ship a pre-built version of that exe with additional built in features. See https://github.com/microsoft/reverse-proxy/issues/261.

the end result will be an exe that you can run from the command line and configure via a the appsettings.json file.-----This helped me a lot, thank you！

Duplicate of #261


[Event Change] codeunit 12 "Gen. Jnl.-Post Line"

Please change OnBeforeInsertVATForGLEntry event as below :
local procedure InsertVATForGLEntry(var GenJnlLine: Record "Gen. Journal Line"; VATPostingSetup: Record "VAT Posting Setup"; TaxJurisdiction: Record "Tax Jurisdiction"; GLEntryVATAmount: Decimal; SrcCurrGLEntryVATAmt: Decimal; SrcCurrCode: Code[10]; UnrealizedVAT: Boolean)
    var
        GLEntryVATAmountNotEmpty: Boolean;
        IsHandled: Boolean;
    begin
        IsHandled := false;
//EVENT CHANGE
        OnBeforeInsertVATForGLEntry(GenJnlLine, VATPostingSetup, TaxJurisdiction, GLEntryVATAmount, SrcCurrGLEntryVATAmt, SrcCurrCode, UnrealizedVAT, IsHandled, VATEntry);
//EVENT CHANGE

        if IsHandled then
            exit;

 [IntegrationEvent(true, false)]
    local procedure OnBeforeInsertVATForGLEntry(var GenJnlLine: Record "Gen. Journal Line"; VATPostingSetup: Record "VAT Posting Setup"; TaxJurisdiction: Record "Tax Jurisdiction"; GLEntryVATAmount: Decimal; SrcCurrGLEntryVATAmt: Decimal; SrcCurrCode: Code[10]; UnrealizedVAT: Boolean; var IsHandled: Boolean; var VATEntry: Record "VAT Entry")
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Fix compile errors and warnings

I had to fix some errors to make it compile.. here it is if you want to save a few minutes. :)

thanks!


How to use the plugin with Grafana Cloud

Hello.
I have been trying to understand how to send logs with pino-loki straight to Grafana Cloud (I have no local instance of Loki installed on my production or environment machines) but so far I don't really understand if the use case is supported or not or if I need to default to using promtail.

Hello!
I myself have tested with Grafana Cloud successfully with the following parameters :
{
	  host: "https://logs-prod-eu-west-0.grafana.net",
	  basicAuth: {
	    username: "181770",
	    password: "apiKey",
	  },
}

These values can be found on the Loki details page on Grafana cloud :

And regarding the API key, you will have to create one from your "my account" page on grafana.net and use it as password in "basicAuth" object.
Feel free to reopen the issue if needed

@Julien-R44  Is it still working that way? I can't make it work with Grafana Cloud. The password should be thie base64 encoded string that Grafana Cloud calls API Key?

@Julien-R44 Is it still working that way? I can't make it work with Grafana Cloud. The password should be thie base64 encoded string that Grafana Cloud calls API Key?

Works well in 2023! I'm using pino-http too and does not need to convert to Base 64.
     pinoHttp: {       name: version,       level: 'info',       transport: {         target: 'pino-loki',         options: {           singleLine: true,           messageFormat: '[{context}] {msg}',           ignore: 'context',           colorize: true,           labels: { application: 'nest-app' },           host: process.env.LOKI_HOST,           basicAuth: {             username: process.env.LOKI_USER,             password: process.env.LOKI_PASSWORD,           },         },       },       genReqId: () => randomUUID(),     }

Turns out I had some errors in my config. Thanks @lesimoes for the confirmation. I had again a look and found the problem at least ;)


remove coredns netpol

As coredns got applied for new clusters via the AppPlatform and we are doing the adoption of all coredns-components via the cluster-shared app we don't need netpol/coredns managed via cluster-resources-app anymore.

This PR:

adds/changes/removes etc

Testing
Description on how cluster-resources can be tested.

[ ] fresh install works

[ ] AWS
[ ] Azure
[ ] KVM

[ ] upgrade from previous version works

[ ] AWS
[ ] Azure
[ ] KVM

Other testing
Description of features to additionally test for cluster-resources installations.

[ ] check reconciliation of existing resources after upgrading
[ ] X still works after upgrade
[ ] Y is installed correctly

Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

I thought it may be dangerous to delete this directly since there may be incompatibilities in consumers' side.  Therefore, it might be better to add a parameter to disable this manifest and keep rendering it by default.
Then I checked default-apps-aws and default-apps-gcp. This app is not used there. Interesting.
@bavarianbidi @giantswarm/team-hydra Do you know why?

I thought it may be dangerous to delete this directly since there may be incompatibilities in consumers' side. Therefore, it might be better to add a parameter to disable this manifest and keep rendering it by default.

I'm currently implementing something to keep coredns up and running during the adoption phase for existing clusters.
The biggest issue i currently have with the coredns App is the fact, that this app use total different label selectors but most of the k8s object names keep the same (expect the service object itself). And the difference from a kubadm orchestrated coredns differs a lot from the coredns we get via our app platform :disappointed:

Then I checked default-apps-aws and default-apps-gcp. This app is not used there. Interesting.
@bavarianbidi @giantswarm/team-hydra Do you know why?

on gcp (and i guess same for aws) the coredns-App was used from the beginning and this app is shipped with their own network policy.
The netpol/coredns in this repo is matching on coredns-pods, shipped by kubeadm where the label selector is different.

is now the case?

is now the case?

not yet :-( as our MCs aren't updated to the lastest cluster-release yet.

will close the PR as we don't have to take care of these clusters.


bvar::Collector::run_dump_thread coredump

Describe the bug (描述bug)
bvar::Collector::run_dump_thread coredump
To Reproduce (复现方法)
Expected behavior (期望行为)
Versions (各种版本)
OS: CentOS Linux release 8.1.1911 (Core)
Compiler: gcc (GCC) 8.3.1 20191121 (Red Hat 8.3.1-5)
brpc: brpc-0.9.8_rc01-1
protobuf: protobuf-3.5.0-7
Additional context/screenshots (更多上下文/截图)

触发场景是什么？

We close this issue because it is irreproducible and inactive for a long time.
If you can reproduce this issue with the latest version of bRPC, please reopen this issue and tell us how to reproduce.


Update metrics-jmx to 4.2.18

About this PR
📦 Updates io.dropwizard.metrics:metrics-jmx from 4.2.10 to 4.2.18
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (4.2.10).
You might want to review and update them manually.
http/baker-http-dashboard/package.json

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.dropwizard.metrics", artifactId = "metrics-jmx" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.dropwizard.metrics", artifactId = "metrics-jmx" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #1511.


Graduate Egress feature from Alpha to Beta

Egress will be graduated to Beta and enabled by default starting with
v1.6.
Signed-off-by: Quan Tian<EMAIL_ADDRESS>

/test-all
/test-windows-all

/test-all
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all

/test-all
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all

Codecov Report

Merging #3509 (4f7f45b) into main (c2a664b) will decrease coverage by 22.15%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #3509       +/-   ##
===========================================
- Coverage   65.51%   43.36%   -22.16%
===========================================
  Files         278      210       -68
  Lines       27771    25171     -2600
===========================================
- Hits        18195    10915     -7280
- Misses       7656    13166     +5510
+ Partials     1920     1090      -830

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
43.36% <0.00%> (-0.02%)
:arrow_down:

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


Module build failed: SyntaxError: 'with' in strict mode (8:5784)

ERROR in ./~/uploadcare-widget/uploadcare.min.js
Module build failed: SyntaxError: 'with' in strict mode (8:5784)

This is what my code looks like
import React from 'react'
import Col from 'Components/Layout/Col'
import Label from 'Components/Layout/Label'
import ResponsiveImage from 'Components/Layout/ResponsiveImage'
const uploadcare = typeof window !== 'undefined' ? require('uploadcare-widget') : { Widget: () => {} }

/**
 * TODO: make this so it does not trigger error about with() in Strict Mode
 */
class FileUpload extends React.PureComponent {
  componentDidMount () {
    const { id, onChange } = this.props
    const widget = uploadcare.Widget(`#${id}`)
    if (widget && onChange && typeof onChange === 'function') {
      widget.onChange((file) => {
        if (file) {
          file
            .done(info => onChange(info.cdnUrl))
            .fail(() => onChange(null))
        } else {
          onChange(null)
        }
      })
    }
  }
  render () {
    const { id, name, imagesOnly, crop, shrink, multiple, ...attrs } = this.props
    return <input
      id={id}
      data-multiple={multiple}
      data-images-only={imagesOnly}
      data-crop={crop}
      data-image-shrink={shrink}
      name={name}
      {...attrs}
      type='hidden'
    />
  }
}

export default FileUpload

const removeCommonAtts = ({ tabPort, ...rest }) => {
  return rest
}

const renderFileUpload = ({ input, label, required, style, meta: { error, touched }, ...props }) => (
  <Col {...props} style={style}>
    {input.value && <ResponsiveImage src={input.value} />}
    {label}
    {touched && error && {' ' + error}!}

      <FileUpload
        {...removeCommonAtts(props)}
        {...input}
        id={input.name}
        placeholder={label}
      />

  </Col>
)

export { renderFileUpload }

I use Webpack and Babel to compile my code and it is a Universal App hence why I am Checking for Window Object.
Any help would be appreciated.

The widget use with function that not support in strict mode =(.
Can you off strict mode in webpack build?

Webpack doesn't support that because strict mode is implied in ES modules. This is a showstopper for using the uploadcare widget with modern tooling.

fix this in https://github.com/uploadcare/uploadcare-widget/pull/523


Calculate Ship Hardpoints as Math.floor(tons/100)

Add attack and damage roll to Animal as per PC/NPC
Add animated fade-ins for panels

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[x] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Calculate Ship Hardpoints as Math.floor(tons/100)
Add attack and damage roll to Animal as per PC/NPC
Add animated fade-ins for panels

Character Sheet Info Roll20 Internal Use only.


Validating Blank Dates

Please validate the section about validating against blank dates. You're stating that one should compare against 1900, but this doesn't seem to work always, one example for this can be seen on Yammer these days:
https://www.yammer.com/dynamicsaxfeedbackprograms/#/Threads/show?threadId=1594212369743872
Maybe different Excel settings or locales make a difference here, I've had very strange effects in my own Excel parsing code over the years.
Any insight in how to do this best would be appreciated, based on how RSAT actually reads those values.
Thanks!

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

ID: 3333aae9-4afa-d53f-13f8-e495a451c3c3
Version Independent ID: 59e4ae15-534b-eb98-84be-23de51d02241
Content: Troubleshoot the Regression suite automation tool - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/perf-test/rsat/rsat-troubleshooting.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @FrankDahl
Microsoft Alias: fdahl

@torbennehmer, thank you for providing this feedback.
@FrankDahl, please review and updated the content as needed.

Closed as won't fix.


Fix SLO callback URL

Reported here - https://wordpress.org/support/topic/sso-logout-issues/
SLO is using home_url(), which some sites don't have in the "Allowed Callback URLs" field for the Application. Changed to the main Auth0 callback URL which is always there. Callback is not used as a redirect so does not need to change for Implicit, just needs to be valid.
Also adds SLO check to the admin + login pages and changes the redirect URL used after logout to point to an external page if there is one or the wp-login.php page.

@lbalmaceda - home_url() always returns something but what it returns is not always something that /authorize will accept (i.e. it's not always set in the "Allowed Callback URLs"). We have a URL we can use that's always there (saved when the App is created) so it's better to use that.

Alright. I'm not so convinced though. Like I said, this looks like something we should "help fix" from the documentation side. A good doc on "how to achieve SLO" would mention this url needs to be set as a requirement, here and whitelisted in the tenant. That's what most of the SDKs do AFAIK. But again, I'm not going to block you for this. Your call :)


Bad PowerShell code to get IP address

The code:
$ip = @{
    Name = 'myVM1'
    ResourceGroupName = 'CreateVNetQS-rg'
}
Get-AzPublicIpAddress @ip | select IpAddress

in the Connect to a VM from the internet section doesn't work.
This code specifies the name of the VM as the parameter, but Get-AzPublicIpAddress expects the name of a public IP address. See this snip from the output of
Get Help Get-AzPublicIpAddress

-Name <System.String>
Specifies the name of the public IP address that this cmdlet gets.

possible workaround:
Get-AzPublicIpAddress -ResourceGroupName CreateVNetQS-rg | select Name, IpAddress

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

ID: 21a03427-b70e-ab8e-1977-bac5066dbc96
Version Independent ID: bcb8091d-0c21-d462-0216-b53c21733de0
Content: Create a virtual network - quickstart - Azure PowerShell - Azure Virtual Network
Content Source: articles/virtual-network/quick-create-powershell.md
Service: virtual-network
GitHub Login: @KumudD
Microsoft Alias: kumud

Hello @jwilliams45-aug
Thank you for your feedback! We will review and update as appropriate.

after looking at this more, the issue is that the tutorial assumes there are public IP addresses created when the VMs are created, but that is not the case.

Hello @jwilliams45-aug, I could replicate the issue faced here. After following the tutorial, I was successfully able to fetch the public IP address as shown below.

Below might be possible troubleshooting steps you can try.

As per the Prerequisite section of this tutorial you need to have Azure PowerShell module version 5.4.1 or later. Can you please check if this is the issue here and update the module.
Another possible issue might be as VM1 is created with -AsJob option which creates the VM in the background, and does not give any error if VM deployment has failed. Can you validate if this was case and take needful steps.

Regarding your comment above.

after looking at this more, the issue is that the tutorial assumes there are public IP addresses created when the VMs are created, but that is not the case.

Azure provides a default outbound access IP for Azure Virtual Machines which aren't assigned a public IP address. This is mentioned in the notes section of the tutorial.

Hope this helps! Since we primarily monitor this channel for any doc enhancements or documentation bugs, we will now proceed to close this issue out. If you are still unable to get rid of this issue, I will suggest that you please create a thread on our Q&A platform. Thank you! Please tag me in your response if there are any additional concerns. Thank you!


Streaming results from PipedStreams

Hi there! Currently, I have a pipeline (PipedInputStream and PipedOutputStream) and I'd like to be able to stream the results from the PipedInputStream as I gradually write to the PipedOutputStream.
An example of what I'm trying to do would be
ByteArrayOutputStream temp = new ByteArrayOutputStream();
temp.write("a random message".getBytes());

PipedInputStream in = new PipedInputStream();
final PipedOutputStream out = new PipedOutputStream(in);

temp.writeTo(out);
out.flush();

context.result(in)      // should return "a random message"
// out.close()              // want to remove this

Thread.sleep(5000);
temp.write(" + additional text".getBytes());
temp.writeTo(out);
out.flush();           // ideally, after 5000 ms, it should now now return "a random message + additional text"

However, I can't seem to get Javalin to return without the first message first calling out.close() right after context.result(in). But that would not allow me to further write to out afterwards.
Sorry if this was confusing, please let me know if there's anything I should clarify. Thank you in advance!

Hi, thank you so much for getting back to me. I'm not too familiar with context.res.getOutputStream(), how would I use it here? Looking at documentation I'm guessing I would have to make ctx_.res.getOutputStream my OutputStream instead (the out variable). Is that what you're suggesting?

@KevinSaigon The default JavalinJackson implementation does something similar for streaming json:
PipedStreamUtil.getInputStream { pipedOutputStream ->
    objectMapper!!.factory.createGenerator(pipedOutputStream).writeObject(obj)
}

fun getInputStream(userCallback: (PipedOutputStream) -> Unit): InputStream {
    val pipedOutputStream = PipedOutputStream()
    val pipedInputStream = object : PipedInputStream(pipedOutputStream) {
        var exception: Exception? = null // possible exception from child thread
        override fun close() = exception?.let { throw it } ?: super.close()
    }
    executorService.execute { // start child thread, necessary to prevent deadlock
        try {
            userCallback(pipedOutputStream)
        } catch (userException: Exception) {
            pipedInputStream.exception = userException // pass exception to parent thead
        } finally {
            pipedOutputStream.close()
        }
    }
    return pipedInputStream
}

@tipsy Thank you for your response. I'm currently using Javalin 3x so I don't think I have access to those functions. Just as a general question, is there a way to return a result gradually - that is line by line ("a random message" and then "+ additional random text") without having to close the output pipe? Thank you.

Hi, thank you so much for getting back to me. I'm not too familiar with context.res.getOutputStream(), how would I use it here?

The OutputStream in ctx.res is directly the output stream used by Jetty to communicate with user. I'm not quite sure what you're trying to do, but as far as I see you're trying to keep the connection and send some data to user over the time, so I'd go for sth like:
Executor ioExecutor = Executors.newCachedThreadPool();

Javalin application = Javalin.create()
    .get("/test", ctx -> {
        CompletableFuture<Void> result = new CompletableFuture<>();
        ctx.future(result, (value) -> {});

        ioExecutor.execute(() -> {
            try {
                ctx.res.getOutputStream().write("a random message".getBytes()); // user receives it immediately
                Thread.sleep(5000);
                ctx.res.getOutputStream().write(" + additional text".getBytes()); // write some extra data to the stream that is still open
            } catch (IOException | InterruptedException exception) {
                // do sth
            } finally {
                result.complete(null);
            }
        });
    })
    .start(8080);

Yes, it's the same method, but in Javalin 4.x internals have been slightly rewritten to eliminate various bugs & vulnerabilities. In general I recommend to bump Javalin to 4.x. API is pretty much the same and you'll spend probably up to an hour migrating to the newer version.

By default response is chunked into smaller pieces, so if you'd like to force server to push it to the client (because your feature is more about notifying user rather than just lazy evaluation of a stream for like e.g. generator purposes) you can use flushBuffer +with disabled acceleration:
https://user-images.githubusercontent.com/4235722/172470592-4d45d1cc-59a5-4c20-b329-266b9a851ea5.mp4
Code:
ctx.res.setCharacterEncoding("UTF-8");
ctx.res.addHeader(Header.X_ACCEL_BUFFERING, "no");
ctx.res.flushBuffer();

OutputStream outputStream = ctx.res.getOutputStream();
CompletableFuture<Void> result = new CompletableFuture<>();
ctx.future(result, (value) -> {});

ioExecutor.execute(() -> {
    try {
        outputStream.write("a random message".getBytes()); // user receives it immediately
        ctx.res.flushBuffer();
        Thread.sleep(5000);
        outputStream.write(" + additional text".getBytes()); // write some extra data to the stream that is still open
    } catch (IOException | InterruptedException exception) {
        // do sth
    } finally {
        result.complete(null);
    }
});

If that's your goal, you may actually want to use server-side events:

https://javalin.io/documentation#server-sent-events

SSE handler uses long living async connection to send various events to the client. It's handled by Javalin and you wouldn't have to worry about manual impl.

I'll look into it. Thank you for all your help!


AbstractLogMinerEventProcessor DEBUG to TRACE loglevel

https://issues.redhat.com/browse/DBZ-5833
Current issue is that it logs tons and tons of log data at DEBUG level when knowing that other tables had changes is not helpful.
I need debug level on this class to get other useful data

@Naros This make sense to me, @tooptoop4 Could you please create a Jira issue for the change?

I've logged https://issues.redhat.com/browse/DBZ-5833, I'll adjust the commit message.  Thanks @tooptoop4

Applied to main (=2.1).

@tooptoop4 Could you please share your name so we can create record in COPYRIGHT file? Thanks


Timestamps not exporting properly

I'm trying to run an export but the timestamp special datatype doesn't seem to be working.. I'm testing in the Emulator and Production. When performing the export, the data looks like the following:
{
  "5Ie1pOL8": {
    "updated_at": {
      "_seconds":<PHONE_NUMBER>,
      "_nanoseconds": 172000000
    },
    "__collections__": {}
  }
}

It's just exporting the raw Firebase timestamp and not the formatted special type.
Any ideas?

Just an update -
Did some digging and it found the issue resides in the serializeSpecialTypes function. Seems like when checking if the data is an instanceOf a FieldValue, all those checks fail as the instanceOf thinks it's just an Object.
Not sure if something broke in a newer version of firebase?


helm cilium encryption settings

    --namespace kube-system \
    --set encryption.enabled=#{settings.networking.cni.enabled.to_s} \
    --set encryption.type=wireguard \
    --set encryption.nodeEncryption=#{settings.networking.cni.enabled.to_s} \
    --set routingMode=tunnel \
    --set tunnelProtocol=vxlan \
    --set ipam.mode="kubernetes" \

in src/kubernetes/software/cilium.cr
can this be right?
config has these:
cni:
    enabled: true
    encryption: false
    mode: flannel

Shouldn't cni.encryption be in both?
Also, you think vxlan is the best for hetzner, or should it be configurable maybe?

Well spotted! That should indeed be encryption instead of enabled. I guess I didn't notice during my testing because I had both values set to true.  I just pushed this change so I am closing this one but feel free to reopen or open another one if you still see an issue related to this.  Thanks :)


Return address object instead of a String in Geocoder.address

Hello!
I have a question. Is it possible to use geocoder with PORO? I want to fetch country using coordinates, so I do the following:
class Geo
  attribute :latitude
  attribute :longitude
  attribute :address

  def initialize(attrs)
    @latitude = attrs[:lat]
    @longitude = attrs[:long]
    @address = Geocoder.address([@latitude, @longitude])
  end
end

And I get the address as a string, but I'd like to have an object Geocoder::Result::Google, to be able to fetch country by a method address.country and not by parsing the string "R. Duques de Bragança 10, 1200-026 Lisboa, Portugal"
I'm ready to make a PR, but I'm not sure that it is a good idea to change the returning value of Geocoder.address from string to object. Maybe it can be done with existing tools?

Sorry, I've read the whole and understood that the thing I want is Geocode.search


Implement Simple BLS (Boneh–Lynn–Shacham) Signature API.

This is to implement substrate/primitives/core/src/bls.rs API similar to what we have for ed25519 and sr25519. This among others is to equip BEEFY with BLS signatures which is essential for the accountable light client protocol.
BLS crypto to be provided by https://github.com/w3f/bls This library is tuned to provide the specific flavor and the efficiency of the BLS signatures required by future protocols for example to have public keys on both pairing curves.
I have started some initial work here:
https://github.com/drskalman/substrate/commits/skalman-bls-beefy
and perhaps continue with the help of imagination till someone from parity take ownership and offer guidance on this issue.

We'll do a DLEQ proof version so we can avoid complicating the gossip rules?

Next step: https://github.com/w3f/bls/issues/46


[DOCS] variable name mismatches

Description
While setting up backend locally even after providing key and secret I was getting error then checked variable name used in docs are different from actual one.
Screenshots
Name in docs

Actual name

🥦 Browser
Brave
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

I want to work on this issue. Also Can I add "How to set up razorpay keys and setup" raising another issue?

Awesome, I think the variables changed accidentally when I was messing with the payment gateway in this commit .
Also instead of another issue, you can do all the changes related to the Razorpay docs in this issue; make a PR once done !

Done ✅


Add support for the Pyramid framework

As we discussed at re:Invent, there is currently no support for the Pyramid web framework. I am willing to take a stab at an implementation of this. This issue is for discussion and tracking.

Thank you for the follow-up. You can see a reference implementation for Flask middleware here: https://github.com/aws/aws-xray-sdk-python/blob/master/aws_xray_sdk/ext/flask/middleware.py. You can see in the ref implementation that there are helper methods to deciding segment name/sampling decisions etc based on incoming requests. The middleware just need to implement certain hooks in a request/response lifecycle and extract proper fields and capture unhandled exceptions resulting in 5xx.
I'm happy to work with you to add Pyramid support.


Add yunjie-lu as addtional /feature/aft codeowner

Pull Request Functional Test Report for #3599 / 43f974087083a12ebb8ddac48e9db070c2ae0312
No tests identified for validation.
Help


fix(app-bar): app-bar props handling fix

Please read and mark the following checklist before creating a pull request:

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

I don't think we need to support this

OK


Include scheme argument for ios

https://github.com/underscopeio/underscope-ci/pull/499

Closing, implemented in #500


OldFilmFilter doesn't work

I'm trying to apply the OldFilmFilter to a sprite:
        img.filters = [new OldFilmFilter()];

however, the filter doesn't look like it looks like in the demo - the image just becomes slighly gray, no noise animations or anything else.
I've also tried re-rendering the sprite every tick, still nothing:
props.app.ticker.add(() => props.app.renderer.render(img));

Pixi version: ^6.2.1
Old film filter version: ^4.1.5

I figured it out - you have to animate it yourself by changing the seed value. The demo is sorta misleading.


Bump Heroku stack from heroku-16 to heroku-18

This PR has an empty commit. It's just to trigger a Heroku release.
I've run this in my terminal:
❯ heroku stack:set heroku-18 -a davidrunger
Stack set. Next release on ⬢ davidrunger will use heroku-18.
Run git push heroku master to create a new release on ⬢ davidrunger.

Heroku did say this during the deploy:
Purging Cache. Changing stack from heroku-16 to heroku-18

Hooray, the app still works. :) 👍


Host admin can add funds to organizations

User story
As a host admin, I want to manually add funds to an Organization, so that I can process bulk transfers.
Best solution for this problem
An 'add funds' option on the Organization page that only host admins can see.
I'm not sure where in the process the platform and host fees should come into it. With the current 'add funds' feature for adding to Collectives, you can put in the host and platform fee % manually. But if those fees will be charges automatically when the Org allocates them to Collectives, then we don't want to double charge them.

This feature is hardcoded for my user atm (I cant' remember how we ended up here). Ideally any host admin can do this.

We still need this but we should solve #2311 first.

I'm wondering what could happen if someone malicious plays with that, especially because any user can become a host admin (aka we cannot trust host admins).
Implementing this issue means that any user will be able to create a prepaid payment method for any organization out there without their consent. Do we want that? Can this be abused?
We should also be careful with what that means when we'll implement #2441: with this issue implemented, any self-hosted collective will be able to lie on its available balance so this information will not be trustable for sensitive operations anymore.
@piamancini @znarf @alanna @flickz  Can we think of other scenarios that may be considered as vulnerabilities?

Implementing this issue means that any user will be able to create a prepaid payment method for any organization out there without their consent.

I didn't exactly think about the consequence like this when I opened a PR for this issue, now that I think about it, it is actually scary.  Can we have an implementation where the host admin can do this but a notification email will be sent to the root user for confirmation, the fund will never be added until the root user confirms it?

Any self-hosted collective will be able to lie on its available balance so this information will not be trustable for sensitive operations anymore.

Can we or do we have any means of confirming the authenticity of self hosted collective balance?

Yeah you're right @Betree I don't think we can go ahead with this in such a simple way. Maybe something like, host admin can add funds to organization but that organization can only contribute those funds to Collectives under that host? I think it's too complicated, though.
For now let's just stick with root users can add funds to Organizations. If we get demand in the future from other host admins about this feature in the future, we can tackle it. Really, the system should have some way of tracking a balance for a host and then the host can allocate those funds to Organizations or anyone in the system, but that's a whole other idea.

For now let's just stick with root users can add funds to Organizations.

It may be safer to do that until we're ready to properly tackle it, we should keep this issue open so that host admins can +1 if they're interested in it.

host admin can add funds to organization but that organization can only contribute those funds to Collectives under that host?

That's already the case, prepaid are limited to the host they are created by

This is done


Calculate the Maximum and Minimum for all Genes in All Pathways to set basicStyle.js

Calculate the maximum and minimum for all genes in all pathways to set basicStyle.js

Calculated the average and median of all the genes in the pathways whose data is available in the dataset, they are 213 genes, Max Average is  0.02993, and the minimum is -0.03452. I have calculated it to 5 decimal places for better understanding as to 3 decimal places many averages collided to 0.000.
I am attaching the link to the google sheets with averages and medians of these genes.
Link -> Gene Data


Adding License and code of conduct and coverage tracking

What Are We Doing Here
Adding a license file and a code of conduct so that we can make the repo available publicly.
Also switched travis-ci to the opensource version and added coverage tracking into the app.
How to Verify

No code changes, just ensure tests still built on travis-ci.org now instead of .com.
Coverage is uploaded.

Apparently some of the repo changes I made caused #9 to become busted. Seems like a GitHub bug, maybe. Opened this one to add the additional changes I made.


GW2Mounts folder and config file not being created

GW2Mounts folder is not being created in addons folder inside main GW2 folder, and config file is not automatically created/saving so every time i re-log I have to set up all the keybindings again...

Yes, this is the only solution. Unfortunately, ArenaNet requires addon developers to put the settings files in that folder, which goes against Microsoft's best practices and therefore is often blocked by the OS for security reasons.


#5 Set up CI systems

#5
Warning: This PR alters config files, please remember to make a backup of yours before you pull this (good practice in any case tbh). And git update-index --assume-unchanged config/envConfig.json and such after
This was way more complicated than I expected due to the nature of our setup. This PR is far from ideal but I think for the most part the work required to get tests set up is now complete. I may also look into coverage and styling/quality before we merge.
This PR enables Travis CI to run our tests, to do this I created a test server (not to be confused with staging) and test bots which exist solely for the purpose of running tests. This required Travis access to tokens which proved very difficult, to enable this I changed our envConfig.js file to a JSON file which could be edited through use of a bash script ran by Travis pre-tests. The relevant variables are kept by Travis and called by our builds to be fed to the script. Obviously, this could pose a security threat if someone chose to console.log these values or perform some nefarious task (such as nuking the test server) while running the bot in a test environment so Travis very wisely only enables tests to use our tokens if the code is on our repo. Basically, this means that tests will only run on PRs and branches on this repo and if anyone wishes for whatever reason to develop their feature branches off this repo they will need to merge to a feature branch on our repo before PRing the develop branch
Also added now is the Codacy CI. Codacy automates some code review in that it detects and points out objectively bad or repetitive code, it also enforces styling guides. We will configure this enforcement through the .eslintrc.json config file in the projects root. The exact configurations of this file need to be discussed so we can agree on our style moving forward.
Work that I would like to complete in this PR before merge:

[ ] More and better comments in relevant files
[ ] Requiring passing tests for all PRs through branch protection
[ ] If it is agreed by reviewers that the work specified below may be postponed for our stability patching then the work needed should be accurately issued

Work needed that I would not like to complete in this PR but instead issue:

Coverage reports (Having taken a quick glance it appears this could take quite a bit of time to set up, would be nice to have pre v1 but not necessary for our infrastructure patch)
Failing tests and any disabled tests
.replace(/\/g,'') was necessary to remove "\" from tokens and IDs but rather than run this everytime we a token/ID value in sub bots some setup should probably be done just once on startup
In CISetup.sh I believe I have essentially created and renamed 13 copies of the envConfig.json file to change the 13 values. There may be a better way to accomplish this, the script takes 26 seconds to compete which is a significant portion of build-time but at least it's only for the CI
MCS disabled as it seems to be completely broken and was complicating tests
Discuss rules included in .eslint.json file which will be read from the develop branch for CI review

It looks good, though I would make some changes. For the Dockerfile and the .sh files(mostly the ones not used for testing). I would get rid of the .sh files as they seem kind of redundant and change the Dockerfile to:
FROM node:7.6

# Create and cd into the directory where the bot will live
WORKDIR /usr/src/digbot

# Copies the project into the container
COPY . . #maybe this should be COPY ../ . (because this file is not in the root of the project)

# Install dependencies(dev-dependecies are not installed, I believe)
RUN npm install

# Run the bot
CMD ["node", "src/bot.js"]

Also some files need to be added to the .dockerignore file(like the node_modules dir).
The reason I would propose this change is that now the dependencies are installed when the docker container is build making it more predictable what dependencies are installed without them updating unnoticed. Also before it was needed to call the run.sh script after starting the container. Now when the container starts the bot starts as well (which is kind of logical as the container runs the bot). This together will make the container autonomous(it starts itself) and stateless(when the bot is ran it will not alter the container, like installing dependencies (with the exception of log files)).
I would also update node to version 8 as the version now used has some vulnerabilities. It seems that the bot is fully compatible and we will need to do this anyway as discord.js(11.4.2+) requires version 8 or higher(check stable version on github). Note that we will lose erlpack unless python is installed and this will also require the package-lock.json to be committed. That last one is important as it will lock all versions of all dependencies, that results in the docker container always being compiled the same way. Now it is possible that I compile it and later compile it again and that it will have installed updated versions of certain dependencies which lead to compatibility issues the second time.
Also I know it was kind of a big unannounced push, but the changes I made to the config might be of help when it comes to travis. You can basically set private environment variables in travis. The SUBBOTS variable accepts a json object with the configuration for the subbots. I know it seems like a big pull request and that there are merge conflicts, but it seems bigger then it is and the conflicts can be resolved by ignoring those changes on that pull request. The only thing that needs to be done is that the right environment variables need to be set, like NODE_ENV for testing container.
Also as testing will do for now, I think for the future we should push for more individually testable modules. Right now we basically test a chain of modules which rely on a live discord server. This makes it harder to debug and test. To best do this we need to start using classes and to make or lives easier a service container.

ESlint is used for the syntax, not sure here. In that case I would go with the standard Google has created for javascript, it is not perfect but it is widely used.


I don't understand celluloid/culture and it's breaking my bundler

@digitalextremist can you please explain why this is necessary? It seems to make celluloid more complicated and even more of a "special snowflake" as it were. Sorry, it might be a great solution to some problem, I just don't get it.

@ioquatix are you talking about the submodule hell? I was under the impression that it was no longer present in the master branch.

Yeah, it's been removed from celluloid/celluloid master. We can remove it from the other Celluloid projects as well.


Spyder can't start because of socket issues

I am using Anaconda 1.5. Created a new environment for Py3.6. Spyder won't open from Navigator when new environment is selected and the spyder is the one in that environment.
(C:\ProgramData\Anaconda2\envs\Python36) C:\Users\Wuffalo\AnacondaProjects>spyder Traceback (most recent call last):   File "C:\ProgramData\Anaconda2\envs\Python36\Scripts\spyder-script.py", line 3, in <module>     import spyder.app.start   File "C:\ProgramData\Anaconda2\Lib\site-packages\spyder\app\start.py", line 7, in <module>     import socket   File "C:\ProgramData\Anaconda2\envs\Python36\lib\socket.py", line 53, in <module>     from enum import IntEnum, IntFlag ImportError: cannot import name 'IntFlag'
And now when I switch back to my 2.7 environment I cannot open Spyder there without using terminal.
Is this a python installation problem? I have tried restarting but haven't tried reinstalling python yet.
Versions and main components

Spyder Version: 3.1.3
Python Version: 3.6
Qt Version: 5.6.2
PyQt Version:
Operating system: Win10

It seems an error with the sockets module. We'll try to add a workaround for this in Spyder 3.2

Closing as a duplicate of #4132.


gh #200: dsDisplay L3 Python test cases

Python helper class and test suite  implementation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Sumip98:x: Bhanu PrakashBhanu Prakash seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Possible to use test operators in if>?

I initially wanted to dynamically include a file based on a variable, e.g.
for_each>:
  my_class: [hub]
  _do:
  +postmake:
      !include : ${my_class}_scrub.dig

But I found that doesn't work.  So I'm wondering if it's possible to somehow compare that variable to a value then execute a task?
Something like:
+postmake:
  if>: Boolean(${my_class}.isMatch(hub))
  _do:
    !include : hub_scrub.dig

+postmake:
  if>: Boolean(${my_class} === 'hub')
  _do:
    !include : hub_scrub.dig

These both fail with errors similar to

Can not construct instance of boolean from String value 'Boolean(hub.isMatch(hub))': only "true" or "false" recognized

Is it possible to do what I want?  Maybe someone has a better idea?

Right after I posted a colleague found this:
+postmake:
  if>: ${my_class === 'hub'}
  _do:
    !include : hub_scrub.dig


Fix tests

This resolves the CI build errors with v3.X .(at least I hope so) I upgraded to the latest ember-cli version and made changes which made sense to me. If anything needs chanings. I'd be happy to change them.
After this #186 and #165 should be able to be green and merged after that.

@spruce thanks for getting this done. Just one change I hope will work.

@jrjohnson gonna do it tomorrow. Thanks for the quick response.

I don't know why some are failing and others are not. Do you have any idea?

Sorry @spruce looks like I sent you down a bad path. I was under the impression that useYarn was the fix for the 2.12 scenario failures. If you want to go ahead and remove that last yarn commit I will merge this and we can go forward without yarn here.

ok, I removed it again. I'm a little puzzeled as to why bower is not getting installed with yarn ...

I think we may not need the framebuffer stuff in travis since testem is using chrome headless, but in the interests of progress I'm going to merge this as is.

@jrjohnson could you cut a new release?


Stratsyllabus

Adding the strategy syllabus (in no apparent order apparently) keen to see if I'm doing this right so I can go ahead and do the rest :)

I see you 😉
On Thu, 20 Aug 2020 at 06:43, Kimberly Bailey<EMAIL_ADDRESS>wrote:

@KimmyBailey https://github.com/KimmyBailey requested your review on:
#494 https://github.com/Umuzi-org/tech-department/pull/494
Stratsyllabus.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Umuzi-org/tech-department/pull/494#event-3673483662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDBL7ZRHSIUYCXICT7A6I3SBSSWXANCNFSM4QFUJ6YQ
.
--

Thandiwe Nyandeni-Sikhosana
Senior Web Development Manager
c: (+27) 84 335 1010
Looking for skills? Hire our talent https://www.umuzi.org/hire-talent/
Check out our Umuzi Website http://www.umuzi.org/ / Visit our Studio in
Jeppestown
https://www.google.co.za/maps/place/Umuzi+Photo+Club/@-26.20681,28.0618713,17z/data=!3m1!4b1!4m5!3m4!1s0x1e950e883f464a13:0xf5a78a7d74f3f331!8m2!3d-26.20681!4d28.06406
--

@KimmyBailey Looks ok for the most part, but need to take you through how the syllabus is structured so we build that at the same time as well as gitignore and how we need to apply it here :)


feat: upgrade operator and revamp module

I'll mark this PR as draft since for the time being I need to troubleshoot this with @modridi because I could not bootstrap the devops-stack realm using the new CRD.


the href encoding

Hi Jake and all,
Is the href encoding supported in Altair? I can't tell by the docs how I would use it. Vega lite docs uses "as" in the spec and of course that is a key work in Python:
{
  "data": {"url": "data/cars.json"},
  "mark": "point",
  "transform": [{
    "calculate": "'https://www.google.com/search?q=' + datum.Name", "as": "url"
  }],
  "encoding": {
    "x": {"field": "Horsepower", "type": "quantitative"},
    "y": {"field": "Miles_per_Gallon", "type": "quantitative"},
    "color": {"field": "Origin", "type": "nominal"},
    "tooltip": {"field": "Name", "type": "nominal"},
    "href": {"field": "url", "type": "nominal"}
  }
}

Yes, it is supported. For example, here we modify a scatter plot of car properties so that clicking on any point opens a google search for the car name:
import altair as alt
from vega_datasets import data

try:
    # Python 3
    from urllib.parse import urlencode
except ImportError:
    from urllib import urlencode

def make_google_query(name):
    return "https://www.google.com/search?" + urlencode({'q': '"{0}"'.format(name)})

cars['url'] = cars['Name'].apply(make_google_query)

alt.Chart(cars).mark_circle(size=60).encode(
    x='Horsepower',
    y='Miles_per_Gallon',
    color='Origin',
    href='url',
    tooltip=['Name', 'url']
)

This would probably be worth adding as an example in the docs

#1085 adds this as an example in the docuementation

That is a big help Jake and an extremely useful feature. Thank you very much. I see that the hyperlinks work perfectly when the chart is saved as an html file; however, in the JL notebook I just get a blank screen even when saving the chart and loading it into an Iframe. Perhaps I am doing something wrong. Either way, I'm quite happy using the standalone html file if need be.
import altair as alt
from vega_datasets import data
from IPython.display import HTML, display, IFrame

c=alt.Chart(data.cars.url).transform_calculate(
    url='https://www.google.com/search?q=' + alt.datum.Name
).mark_point().encode(
    x='Horsepower:Q',
    y='Miles_per_Gallon:Q',
    color='Origin:N',
    href='url:N',
    tooltip='Name:N'
)

c.save('test_chart.html')
IFrame('test_chart.html', 500, 500)

Sorry for the confusion. If I do the following in a JL notebook, the hyperlink works (in other words, I don't need to save the chart and then load it into an iframe). However, in a Colab notebook, I do get a blank cell and no errors in the JS console.
alt.Chart(data.cars.url).transform_calculate(
    url='https://www.google.com/search?q=' + alt.datum.Name
).mark_point().encode(
    x='Horsepower:Q',
    y='Miles_per_Gallon:Q',
    color='Origin:N',
    href='url:N',
    tooltip='Name:N'
)

Are you using http://colab.research.google.com/?
I pasted your code into a comment in a Colab notebook there, and it works correctly.

Yes, I get the blank cell. How odd.

Just so I'm clear: the chart doesn't render at all when you execute the cell?

Because it works fine for me:

Could you share the link to the notebook where the chart fails to render?

No no, the chart reders perfectly. The blank cell appears when the hyper link is clicked.

Ah, I see. That was not clear to me.
That is due to the fact that Colab renders outputs in their own iframe, and so the hyperlink is being followed in that iframe, but http://google.com returns a blank page if you try to load it in an iframe.
To follow the hyperlinks when using colab, you can shift + click to open them in a new window.
Unfortunately, Vega-Lite has no way to specify the href target to make hyperlinks open a new window by default; a previous request about that was closed as will-not-fix: https://github.com/vega/vega-lite/issues/3795

Awesome – best of luck with the workshop!

Is it possible to turn off href for some of the points?  For example, if some of the points have links, but some do not.
I tried setting my url variable to None but then I wind up with a hyperlink to null.
Apologies for posting to this old thread, but it seemed somewhat germane.  Happy to open an issue if called for.

Ah, I see if I set the masked rows to an empty string, it's effectively what I want (the hyperlink seems active in the browser, but it's a non-op when you click it.)   This is good enough for me!  Thanks so much for a great tool.

Does href no longer work on mobile?
I tried it on the example shared above and it works, but when I recreated the chart locally it didn't, that is, I can click on it through my laptop, but if I try going in mobile mode it doesn't execute the action.

There were some changes lately in Vega affecting mobile tooltips described in https://github.com/vega/vega/issues/3893#issuecomment-1998790778. Maybe the behavior you are seeing is related to the same changes?

I've noticed that difference as well, but I'm not able to tell whether the issue or the solution that they implemented is directly affected by this, as they talk about hover and tooltips rather than the url clicks. This referenced issue seems to specifically address the tooltip behaviour.


[INTERNAL] Add and update badges

Add dependency status badges
Replace npm version badge with a more reliable one (shields.io kept on
showing "invalid")

Pull Request Test Coverage Report for Build 126

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.683%

Totals

Change from base Build 123:
0.0%

Covered Lines:
423

Relevant Lines:
618

💛  - Coveralls


Create the release_packaging recipe for dart-internal

Actually we will be reusing the same packaging recipes but with some changes to the packaging script to separate the operations of the fetch, compile and upload stages.


Update mime-db to 1.31.0 in v1.x branch

If you have an issue with a specific extension or type
Locate the definition for your extension/type in the db.json file in the mime-db project.  Does it look right?
[ ] No. File a mime-db issue.
[x] Yes: Go ahead and submit your issue/PR here and I'll look into it.
Please upgrade the mime-db dependency to 1.31.0 in the v1.x branch. This will add support for WebAssembly which is needed by webpack-dev-middleware. See https://github.com/webpack/webpack-dev-middleware/issues/229

Looks like my problem in https://github.com/expressjs/express/issues/3486 would be solved too by backporting mime-db in v1.x.

Tagged & published v1.5.0, which is built on<EMAIL_ADDRESS>cc https://github.com/expressjs/express/issues/3486


album video custom filter no sound

I read the album by reading the video file, after a custom filter, the video file is exported without sound, very confused, for the time being did not find an effective solution to the problem, I hope you can help or tips, the following is my relevant code :
//Render display
renderView = RenderView(frame: CGRect(x: 0, y: UIScreen.Screen().height*0.08+0.5, width: UIScreen.Screen().width, height: UIScreen.Screen().width))
view.addSubview(renderView)
movie = try MovieInput(url: url, playAtActualSpeed: true, loop: true)
movieFilter = Normalfilter()
movie --> movieFilter --> renderView
movie.start()
//Sandbox directory address
tmpFileURL = NSURL.fileURL(withPath: "(NSTemporaryDirectory())tmp(arc4random()).mp4")
print(tmpFileURL)
//Export
output = try! MovieOutput(URL: tmpFileURL, size: Size(width: Float(UIScreen.main.bounds.height), height: Float(UIScreen.main.bounds.width)))
movie --> movieFilter --> output
output.startRecording()
Finally, the export of the video file is no sound, is where I have a problem?
@gillygize @wallerdev @mz2 @datskos @rounak @SoundandStuff

@gillygize @wallerdev @mz2 @datskos @rounak @SoundandStuff

@henvyluk did you tried, to set camera.audioEncodingTarget ?
camera.audioEncodingTarget = output

My previous commend was a misunderstanding, sorry :|

@henvyluk In MovieInput is a commend from @BradLarson:
// TODO: Audio here
So, I guess there is no sound setup for MovieInput component yet.

Yes, I have noticed. I also have access to relevant information, try to do this, but not yet achieved. I have seen someone to put forward the demand, it was also a Mac version [https: //github.com/BradLarson/GPUImage2/pull/140!],  but the iOS version I was trying to be unsuccessful

Yes, I have noticed. I also have access to relevant information, try to do this, but not yet achieved. I have seen someone to put forward the demand, it was also a Mac version [https: //github.com/BradLarson/GPUImage2/pull/140!](https: //github.com/BradLarson/GPUImage2/pull/140), but the iOS version I was trying to be unsuccessful.

Yes, I have noticed. I also have access to relevant information, try to do this, but not yet achieved. I have seen someone to put forward the demand, it was also a Mac version
[https: //github.com/BradLarson/GPUImage2/pull/140] (https: //github.com/BradLarson/GPUImage2/pull/140) , but the iOS version I was trying to be unsuccessful.

Yes, I have noticed. I also have access to relevant information, try to do this, but not yet achieved. I have seen someone to put forward the demand, it was also a Mac version
(https: //github.com/BradLarson/GPUImage2/pull/140) , but the iOS version I was trying to be unsuccessful.


logseq: 0.7.0 -> 0.7.5

Description of changes
closes https://github.com/NixOS/nixpkgs/pull/176842/.

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)
[ ] Tested, as applicable:

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

[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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

1 package built:

logseq

Around this version logseq added support for the logseq protocol (logseq://), shouldn't we have a handler for it?
Something like:
xdg.mime.defaultApplications."x-scheme-handler/logseq" = "logseq.desktop"


[terra-form-select] Invalid onBlur events triggered when selecting an option

Bug Report
Description

The Select component is triggering inappropriate onBlur events when selecting options.
Switching the variant produces different onBlur results.
Default Variant:
A default variant will fire onBlur every time an option is selected. (Mobile and Desktop)
Multiple/Search/Tag/Combobox:
The onBlur fires when the first option is selected and the dropdown menu closes. (Mobile)
Example Code:
import React, { useState } from 'react';
import Select from 'terra-form-select';

const Example = () => {
  const [value, setValue] = useState(['blue']);
  const [count, setCount] = useState(0);

  return (

{`Blur Count: ${count}`}

      <Select variant="multiple" onBlur={() => { setCount(count + 1); }} value={value} onChange={setValue}>
        <Select.Option value="blue" display="Blue" />
        <Select.Option value="green" display="Green" />
        <Select.Option value="purple" display="Purple" />
        <Select.Option value="red" display="Red" />
        <Select.Option value="violet" display="Violet" />
      </Select>

  );
};

export default Example;

Steps to Reproduce

Create a demo page using the code snippet above.
Click on the arrow button of the Select to open the dropdown.
Select an option.
Observe the onBlur event count.
Repeat steps 2-4 with each variant and observe they behave differently.

Note: The behavior is different when the dropdown is opened via clicking the arrow button vs clicking into the text/selection region.
Note: The behavior is different on desktop and mobile. Each variant should be testing on both desktop and mobile.
Additional Notes
The mobile workflows were tested on mobile iOS Safari using an iPhone X.
Expected Behavior

The onBlur event should only fire when focus navigates away from the Select. For the tag/multiple variants the dropdown menu should not close when selecting the first option.

@StephenEsser
I tried the steps mentioned by you in the issue description. I, however, do not see blur events triggering multiple times upon an option selection across all variants. Could you let me know which version of form-select, you are seeing the issue in?
Note: I tested the snippet on iOS mobile Safari on an iPhone XS Max.

@pranav300
The blur event is only being fired once for specific actions, but it should not be firing at all.
For default variants onBlur will fire once each time an option is selected. (Mobile and Desktop).
For multiple/search/tag/combobox variants (on mobile) the onBlur fires when the first option is selected and the dropdown menu closes. The dropdown menu should stay open and the blur event should not fire.
Are you not seeing the onBlur event being fired at all in your testing?

@StephenEsser, thanks. We weren’t aware that you meant the arrow button in Step 2 🤷🏻♂

@nramamurth I've updated the reproduction steps to include directions for clicking the arrow button. I wasn't aware clicking the arrow button behaved differently until I double checked after @pranav300's comment. Both workflows should be checked while working on this issue for both mobile and desktop.

We are seeing some undesirable behavior in prod that seems tied directly to use of the drop down arrow and we definitely listen to the onBlur event, so I'm guessing this is related. Is there any ETA on this work or a way I could get notified when there is a snapshot to test against?

@pranav300 @saket2403 (Issue assignees). This issue came up during office hours, are there any updates? Are you both or either of you working on this issue?


maximumResultsReached - get_everything method

I have been trying to get all the news article related to say money laundering using the get_everything method, but after a certain number of results have been accessed, it gives up this error.

Note: I have a paid account, not a developers account. I can access a max of 9900 articles only when let's say the number of articles related to money laundering is 50K. How can I access the remaining articles?

Hi @Deadshot20,
While this is a wrapper for News API, this library is not official and I have no real affiliation with News API. Therefore, I can't comment on rates or limits related to types of accounts. I suggest you contact<EMAIL_ADDRESS>.
Personally, when using the API, I use more specific queries in order to have a limited result set.
Best of luck


[terraform] Allow to overwrite ami id with variable

We encountered a problem recently with cluster_test when test runner and validator nodes are crashing with kernel panic with recent AWS ami.
This diff allows to overwrite ami id with variable aws_ecs_ami_override - this allows to use previous ami, without upgrading to last one

Remove usages of var.cluster_test

:hourglass: Testing commit ac13b5d876a63006167d50d648142b4ee1d90e4e with merge 3a993ece576a346ddf5a79bcdfa6b12f969dd7d0...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: mgorven
Pushing 3a993ece576a346ddf5a79bcdfa6b12f969dd7d0 to master...


Invisible search in select

I'm submitting a ...

[x] feature request

What is the motivation / use case for changing the behavior?

The idea is that normal search in select is too disruptive, yet in many cases we want to have some searchability in select. And that should be enabled by default.
So the proposed solution is that we still keep not disruptive select, while allowing user to search.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

We need some investigation on how it's done. I think in native select you have some kind of timeout.

Add to input issue in ui-framework


[REQUEST] Automaticaly go to the Lolicode's part

When I select a block and then I switch to the Lolicode section, I would like to go automaticaly to the part of the code. It will be much more convenient, especially when you deal with huge config ...
Thanks

OB Settings > On config edit navigate to > LoliCode

Sorry I misread the issue and closed it by mistake 😅

Sorry I don't think I will do this, please give labels to blocks and then simply CTRL+F to find it. Implementing this would take too much time for the small benefit it will give.


IDB benchmark would probably be faster with getAll()

Hey, this is an awesome project and really technically impressive. Hats off!
Just wanted to point out that the IDB benchmark for summing each item would probably be faster using getAll() rather than cursors. It avoids the back-and-forth of iterating through each item in the object store with a cursor.
Now of course, for the 100,000 case, you probably don't want to read 100,000 items into memory at once. But even in that case, you can use batching with getAll() to implement a faster "cursor" (e.g. fetch 100 items at a time instead of just 1).
I did this years ago for PouchDB (https://github.com/pouchdb/pouchdb/pull/6033); it was a ~50% speedup at the time. So absurd-sql would probably still win! :slightly_smiling_face: But it might be interesting to see a comparison with a more "optimized" IDB usage.

@nolanlawson you are correct! Switching to getAll and summing 400,000 items takes ~3100ms instead of a wild ~8100ms. That's Chrome btw, with cursors Firefox only takes ~5100ms (I've found cursors to be a lot faster in Chrome). I've been using 400,000 items as my stress test.
I really expected cursors to be the fastest possible. I'll have to think about this. The problem is that when we are at the read level, we don't know which blocks to expect in the future. We could "prefetch" 10 blocks at a time or something like that, but we'd easily overfetch as SQLite sometimes jumps around and does random access.
Not sure what the right heuristic is. It's probably worth exposing options so apps can fine-tine it for their app.
And yeah, absurd-sql still only takes ~1200ms to sum 400,000 items :)
It's hard to know what's a fair benchmark. In SQLite's case, it's not having to keep everything in memory, so it's not really fair to call getAll and shows those numbers side-by-side. Not sure what the right call is.

I'll do some more digging, but I think cursors might be find in absurd-sql's case. The problem with cursors seems to be the jumping back and forth between IDB and JS. But because we are effectively batching reads, we're not jumping back and forth nearly as much. If I sum 400,000 items, there's not nearly the same amount of jumping back and forth, there's just the occasional block read when it needs more data. Using a cursor is definitely faster than using individual get requests, so the only other option would be calling getAll with a range that load the next 5 blocks or something. But I'm skeptical that will show much improvement; it might be slightly faster, but there's probably enough cases where it overfetches that voids out the perf benefit.
(I know this isn't what you're saying, just thinking about the right heuristics we should use)

That's interesting, yeah. If you know what kind of data you're working with, then you can probably choose a reasonable batch size. But for a generic database like absurd-sql it's kind of hard to make that call on behalf of the user. In that PouchDB issue I mentioned overfetching but said I thought it was worth the tradeoff (maybe it was; can't recall :sweat_smile:).
As for what's a fair comparison, I think it's fine either way. You used IDB cursors because that's the idiomatic approach – most IDB users are probably not implementing batched cursors unless they run into perf problems and even know it's a solution. The browser vendors should make cursors faster!


PR-1794 - Cache tweaks.

Two changes:

If an object is older than the current version reset all the WKS indices. This means that changes in the WKS index table can be handled by incrementing the cache DB minor version.
If an object has a more recent version, it is presumed from the future and treated as if it were corrupt and not used.

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/436/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/446/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/2025/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/434/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2129/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/691/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/468/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/455/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/2151/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/2047/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/581/

clang-analyzer build failed! https://ci.trafficserver.apache.org/job/clang-analyzer-github/712/


Added SpotBugs plugin to Java client build

Issue #, if available: N/A
Description of changes:

Added SpotBugs Gradle plugin version 6.0.18 (for Java Client)
Configure SpotBugs tasks for the Java main source set
Set up HTML and XML report generation
Configured SpotBugs to ignore failures
Automatically upload spotbugs artifacts after each CI build

NITS:

Updated formatting in GitHub actions to keep it consistent (not functional / no impact)

Additional Test:
i. Spotbugs reports generated

Clean-up java build directories
Run each command

./gradlew build
./gradlew :client:buildAll
./gradlew :client:buildAllRelease

Check for XML/HTML reports
ER: 2 files are generated: build/reports/spotbugs/main/spotbugs.html and
build/reports/spotbugs/main/spotbugs.xml
AR: Files are present

ii. XML report is valid

Install Spotbugs plugin in IntelljIDes
Open generated XML file via Spotbugs plugin
ER:  IntelljIdea recognized all findings and provided highlights and code navigation
AR: Mathing ER

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Consider adding reminder to check new report to the developer guide.

Updated documentation.


Add Polish translations

Suggested by AUTUMN WIND via email.

I do not intend on adding additional translations at this time, it is already getting hard to maintain.


add mod and obs domain averages to json files (currently only bias is available)

@jservonnat  Now in master


Truncated string in log outpout

Example: hydrosphere appears as ydrosphere in the main console log.

It's log4j "feature". Switching to logback can fix that.


fixremove literal partition by and order by expression in window function

pick from master https://github.com/apache/doris/pull/26899
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Data not shown in ItemDetails Form

Describe the bug
Hi,
Bug: empty fields in itemDetails Form
To Reproduce
Steps to reproduce the behavior:

Go to 'Companies'
Open a item from data-table and remember the ID
no need to do anything just close it again
Go to 'Company Types' without doing anything go back to 'Companies'
Open the same row from Data table the Form is empty

i use chrome browser
best regards
michi

I think that issue is related to #56

Hi,
Thanks for finding this bug.
I have solved this issue in this commit https://github.com/what-crud/vue-crud/commit/eae86bf62df344f16958676f8833609ee17be773
When user went back to the item which he watched previously app didn't react because it didn't catch any changes in stored item. I had to reset this item after table destroy.
@michigit-dev  Could you check if problem has been solved?


Updating merge request api

Added scope to merge request api and have made project_id optional on the merge request which was added in Gitlab 9.5 https://docs.gitlab.com/ee/api/merge_requests.html#list-merge-requests

Ping @biggins.

rebased as new PR: https://github.com/m4tthumphrey/php-gitlab-api/pull/503

Moving discussions to the new PR.


cloc: 1.90 -> 1.92

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AlDanial/cloc/releases.
meta.description for cloc is: "A program that counts lines of source code"
meta.homepage for cloc is: "https://github.com/AlDanial/cloc"
meta.changelog for cloc is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/6m5n508rw0cwliyb3d95zlbk2zm64y7i-cloc-1.92/bin/.cloc-wrapped passed the binary check.
/nix/store/6m5n508rw0cwliyb3d95zlbk2zm64y7i-cloc-1.92/bin/cloc passed the binary check.
1 of 2 passed binary check by having a zero exit code.
2 of 2 passed binary check by having the new version present in output.
found 1.92 with grep in /nix/store/6m5n508rw0cwliyb3d95zlbk2zm64y7i-cloc-1.92
found 1.92 in filename of file in /nix/store/6m5n508rw0cwliyb3d95zlbk2zm64y7i-cloc-1.92

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
cloc
docbookrx

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

cloc

Maintainer pings
cc @rycee for testing.

This is now broken because of

Hmm, curious. Why would Darwin see a different hash compared to x86? I haven't seen that happen before.

I asked for advice in #macos will see if any ideas pop up.

@berdario Fixed in https://github.com/NixOS/nixpkgs/pull/149460


[polarion] Each result dir equals polarion test run

Each of the dirs found under the cifmw_polarion_jump_result_dir will be understood as a new test run in polarion. The polarion role will merge all .xml result files found inside each of the directories. Then it will iterate over all merged files and will upload them to polarion as a test run.
The reason behind this is that the same test can be executed several times within one job, each time with a different configuration or in a different cloud state (e.g. before/after service reboot). However, if polarion receives multiple results of the same test it will take into account only the last one.
The results from the tests that are executed repeatedly need to be in separate result directories so that they are reported as separate test runs.

this is ready, just note, it has to be merged around the same time as the downstream patch (ci-framework-jobs repo, MR 1022), ideally if the downstream patch was merged first and this one would follow within minutes.

/lgtm

/lgtm

This is a good result in the way it introduces the concept, but there is no direct correspondence between a stage and the results. If we just split by step, we may end up introducing more reports than needed. The way I see (and the way I thought when I raised the issue that led to this) is that we need a new concept of "phase" and aggregate the test results by phase.

This is a good result in the way it introduces the concept, but there is no direct correspondence between a stage and the results. If we just split by step, we may end up introducing more reports than needed. The way I see (and the way I thought when I raised the issue that led to this) is that we need a new concept of "phase" and aggregate the test results by phase.

stage/phase .. all the same to me, test-operator (the role in ci-framework or test-operator itself) and this polarion role don't have all the context to tell what results belong to what stage or phase - test-operator works with stages, a phase as i understand is a concept above that, a test run in polarion is also a concept above that .. now, the mapping between stages, phases and tests runs isn't simple, one test run may include more stages and thus more phases. I don't see any part of the whole automation process that could tell what should be in which test run.
Therefore, I proposed this (honestly, I think it's clever :D), everything that is in one directory (the dir is most likely created by test-operator .. the stage) is gonna be one test run. Therefore it doesn't matter what a phase or a stage is, we don't have to define those terms and find a mapping between them, simply what's in one directory it's gonna be reported as one test run. We may end up with a bit more test runs then we need, so what? Polarion can handle that, if not, it's time to get rid of that. Having potentially more test runs than needed is the price to pay for being able to run the same test multiple times during the same job.
Of course, feel free to propose a different soltution if you have a better idea how to address this.

Can we get this approved and merged?  What is holding it up?

Can we get this approved and merged? What is holding it up?

As I mentioned, I don't think the proposed change address the issue that led to its creation. Even if this is merged, we will need to work to redo this differently because different test-operator stages doesn't mean different test runs. IMHO. Others will disagree.

different test-operator stages doesn't mean different test runs

I agree, this patch is not saying that. In fact, it was written this way so that we don't have to deal with that. But I already said it, twice, once in the commit message and once in the comment I wrote earlier. So far I haven't heard any different ideas or better solutions. As far as I know, this resolves the issue and can be merged right after we merge the downstream patch (which is btw left in the review queue .. seems like owners of that repo need to revisit their review processes).

Please lets get this resolved (sync up if necessary) so we can support adoption, etc.


update section about IAM

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface
/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Update of the IAM section for AWS Security Lake post
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [Issif]

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

LGTM label has been added.  Git tree hash: 10617cdd7aa27f6e9efb12ca7e3efd3098b3d717


Bumping CAPC to v0.4.9-rc1 to fix VM deletion bug

Issue #, if available:
Description of changes:
Bumping CAPC to v0.4.9-rc1 to fix VM deletion bug
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.12


enhancement

i want to see more the features like a mining features
my address : 0x63Cbc34d9a7829dDAb71ba8D7618100a73F7b4aD

@Gungogut Thank you for your testing and we hope to continue delivering high quality features and improvements!


Swipe display incomplete in a list when rotating device

Process a swipe left on OpenPaaS (Android, or iOS) Unified Inbox with the smartphone in portrait mode (see portraitSwipeAction1.png)

Rotate the screen in order to be in landscape mode (see landscapeSwipeAction2.png)
The swipe (orange) is not displayed completely NOK

Thanks for reporting this bug. I will take care of it once I have time (but for sure in the near-future)

Fixed #19 fix swipe on rotation


German Translations for Login Messages

I'm don't feel like downloading an editor for this, so here are the relevant bits:
#: __init__.py:23
#, python-format
msgid "Login succeeded. Welcome, %s."
msgstr "Sie wurden reibungslos angemeldet. Willkommen, %s."

#: __init__.py:24
#, python-format
msgid "You are logged in as %s."
msgstr "Sie sind als %s angemeldet."

Awesome.
Can you send a pull request ?

Possibly later today. I don't want to download a .po editor, so I'll look into finding a free online resource. If I can't find one, I'll just add in the file and edit the metadata by hand.

From: Ming<EMAIL_ADDRESS>Sent: 27.10.2015 22:14
To<EMAIL_ADDRESS>Cc: Alexander<EMAIL_ADDRESS>Subject: Re: [django-cas-ng] German Translations for Login Messages (#58)
Awesome.
Can you send a pull request ?

Reply to this email directly or view it on GitHub:
https://github.com/mingchen/django-cas-ng/issues/58#issuecomment-151727172

@kavdev I made a pull request (#59) out of your translation.

In fact, I wonder if including .mo in the repository is really pertinent.
Maybe would it be better to add the command django-admin compilemessages in the Makefile

agreed, .mo file should not be in repository.

Cool.


heuristic_atmosphere bound failure

In some instances, I'm getting breaks at line 95 of inverse_simple.py, inside of heuristic_atmosphere, where:
bounds = (h2os[0]+0.001, h2os[-1]-0.001)
best = min1d(lambda h: abs(1-p(h)), bounds=bounds, method='bounded')
This stems from non-safe bound definition, and should get fixed.
That said, this failure is almost certainly caused from something earlier on, as x_RT probably shouldn't hold multiple entries of the same (or nearly identical values).  Usually, this occurs when I have some bad configuration settings, and am relying on the presolve h2o grid in apply_oe.  So noting that while this is a bug, it's only a bug because I almost certainly messed something up previously.  I'm considering adding some basic check to the presolve results, to ensure that they're reasonable, and throw exceptions earlier to prevent wasted LUT time if not.
Self-assigning both of these items for future release.

Solved during 2.0->3.0 pathway, forgot to close.


感觉离agent落地还很遥远

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
感觉离agent落地还很遥远
Use case
感觉离agent落地还很遥远
Related issues
感觉离agent落地还很遥远
Feature Priority
None
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

快了，gpt5出来，agent会更快一点的


attempt to support clusterAllReplicas for clickhouse datadog agent

What does this PR do?
https://github.com/DataDog/integrations-core/issues/14199
Update clickhouse datadog agent client to have ability to specify a cluster name, and query system tables with clusterAllReplicas option.
Motivation
Datadog agent currently is not working with clickhouse clusters running across more than one node. This PR is an attempt to support monitoring multi-node clickhouse deployment by datadog agent.
Additional Notes
I am yet to build/test locally.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

You can build and test your changes using our CLI tool ddev. After installing ddev, you can spin up a test environment for clickhouse using ddev env start clickhouse <environment>. To see the possible ddev test environments we currently support, run ddev env ls clickhouse.

Hi there, I tried ddev, but got bash-3.2$ ddev env ls clickhouse No testable checks found for: clickhouse instead.
How may I set up the test environment for ClickHouse first?

Hi there, I tried ddev, but got bash-3.2$ ddev env ls clickhouse No testable checks found for: clickhouse instead.

The most likely reason for that is that you didn't configure ddev to point at your copy of the repo. You can run ddev config edit to edit the config and make sure that it's pointing to the path where you cloned the integrations-core repo:
[repos]
core = "~/path/to/integrations-core"

Hey @sichenzhao, did you get to try my tip above? Let us know if there is anything we can do to help you towards getting this into shape.


Bring over l2_utils from large PR

Change Summary
Closes #1207
Overview
3 Utility functions which will be needed for Ultra L2 Processing:

build_solid_angle_map to determine the solid angle for evenly angular spaced rectangular grid of the sky
build_az_el_grid to build evenly angular spaced rectangular grid of the sky
rewrap_even_spaced_el_az_grid to take an unwrapped (raveled) rectangular grid of the sky in 1D and turn it back into a 2D grid.

New Files

imap_processing/ultra/l2/l2_utils.py

Utility functions for use in Ultra L2 processing.

imap_processing/tests/ultra/unit/test_l2_utils.py

Tests for above functions

Testing

Adds unit tests for all new functions.

Looks good to me. The biggest ask would be to move the build_az_el_grid to be a common util.

Thanks @laspsandoval! I made all the smaller changes - I think the amount of overlap with build_spatial_bins functionality is smaller than the differences between their outputs -  all that they both do is divide 2pi or 360 by a given spacing.
I can definitely call build_spatial_bins at the start of build_solid_angle_map, but then I need to convert back and forth from degrees.
In terms of making these shared utils, how about I move this file to be in ultra/utils/spatial.py, and bring build_spatial_bins there as well?


New feature for controls

I have a feature request about the controls.
Being able to scroll through the items in a target with the arrow keys, and select a item with enter.
It would be awsome to have and easiyer to control fast.
Reach out if any questions.

How does moving your hand from mouse to keyboard arrows makes you faster?

How does moving your hand from mouse to keyboard arrows makes you faster?

I would say it's faster than using Buttons. I personally sometimes have problems with the camera and it still turns around even though it should not when selecting, and that would not be an issue with arrow keys.

I personally sometimes have problems with the camera and it still turns around

Some other resource is conflicting, enabling controls in their own thread.


Adding remove breakpoint by location

Adding a new command to remove a breakpoint by location. This command already exists for adding a new breakpoint.

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


feat(tdhttp): request creation methods/functions init Host field

Accepted target can contain scheme://host:port prefix. If it is the case host:port is used to set Request.Host. If not and the header Host is present, it is used to set Request.Host.

coverage: 99.866%. remained the same
when pulling a927c540571bf518174cbeb6a29352418436524d on tdhttp-host
into 7dc6c8266e80af42dbcec0eb3198c150fc8b7dca on master.


Sometimes they are several offers in the offer_response. We just get the last updated one.

Sometimes they are several offers in the response from Packt website.
For example, yesterday :
{
"count": 2,
"data": [
{
"amountAvailable": null,
"availableFrom": "2020-05-22T00:00:00.000Z",
"createdAt": "2020-05-20T09:02:39.679Z",
"deletedAt": null,
"details": null,
"expiresAt": "2020-05-23T00:00:00.000Z",
"id": "35f3ef1f-e878-4743-b417-df902260eca9",
"limitedAmount": false,
"priority": 0,
"productId": "9781838987862",
"updatedAt": "2020-05-20T09:02:39.679Z"
},
{
"amountAvailable": null,
"availableFrom": "2020-05-22T00:00:00.000Z",
"createdAt": "2020-05-20T09:01:41.269Z",
"deletedAt": null,
"details": null,
"expiresAt": "2020-05-23T00:00:00.000Z",
"id": "803ca6c5-5daf-453c-80a8-75b49614dbaf",
"limitedAmount": false,
"priority": 0,
"productId": "9781788836210",
"productId": "9781788836210",
"updatedAt": "2020-05-20T09:01:41.269Z"
}
]
}
The one to get is the last updated one. This pull has been tested during a few days with success.

Looks good but you need to resolve git conflicts. Can you rebase with master?

Hello Marcin
I have tried to rebase it, tell me if it is Ok now.

Looks OK, thank you - I'll make a new release soon.


Two questions about the experimental results in Tabel 1 of the paper.

Hi, I would like to ask you two questions about the experimental results in the paper's Table 1.
I would like to ask where the acc 53.97 of full tuning of ssv2 was obtained?

When I read VideoMAE, I found that pretrain on ssv2 and then finetune on ssv2 can get 69.3 results. I know your paper is using the K400's pretrain parameters, but I also did experiments and I can achieve 65+ results with 50 rounds finetune on ssv2:

so my first question is where did you get 53.97 from?
The second question is that the data in the chart below I also did not find in the table, is it written wrong?

Hi,
Thanks for your raised question.

May I know your detailed configuration including command and pre-trained weights?

Good catch. We are sorry for that typo. We updated the table while missing the main text. Thanks again for pointing it out.

Hi,
1 pre-trained weights：
The pre-training weights I use is
https://drive.google.com/file/d/1JfrhN144Hdg7we213H1WxwR3lGYOlmIn/view
the location where the picture is shown in Annex 1.
2 shell：
I basically did not change the shell relative to VideoMAE, but only changed the batch size from 256 to 64. The equivalent is now using the parameters of K400 pretrain 800 rounds and fintune 50 rounds on ssv2. The final results obtained are compared with the same case using ssv2 pretrain 1600 rounds, as the picture in Annex 2 shows, they both exceed 65+ acc1. And the shell is in the Annex 3.
From：Yang Zhen
Phone：15951873323
WeChat：15951873323
| |
浙大_软工_杨震
|
|
@.***
|
---- Replied Message ----
| From | Shoufa @.> |
| Date | 10/15/2022 09:18 |
| To | @.> |
| Cc | yz is all you @.> ,
@.> |
| Subject | Re: [ShoufaChen/AdaptFormer] Two questions about the experimental results in Tabel 1 of the paper. (Issue #16) |
Hi,
Thanks for your raised question.
May I know your detailed configuration including command and pre-trained weights?
Good catch. We are sorry for that typo. We updated the table while missing the main text. Thanks again for pointing it out.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks for your reply. Did you experiment with the VideoMAE codebase?
I guess you experiment with strong data augmentation and optimizer (e.g. AdamW). For fair comparison to linear probling, We experiment with the same setting as linear probe, which uses SGD and does not contain strong data augmentation.
Please let me know if I miss something.

Thanks for your reply. I will do the experiment to verify it again！

@ShoufaChen @yangzhen1997 I was also experiencing the same problem. Even though you removed those augmentations and AdamW optimizer, will your method still be able to improve the results? Based on my experiments, adding augmentations and AdamW optimizer did not improve (and sometimes degraded) the performance. This is because in full fine-tuning, they are used to reduce the model overfitting when tuning many parameters. However, in VPT and your method, since we are only tuning a small fraction of parameters, it does not improve model performance. Therefore, would it be fair to report the full fine-tuning results without any augmentations and sophisticated optimizers?


Should Media3 use media duration to calculate frame rate rather than track duration?

Background
When I contribute this code: https://github.com/androidx/media/pull/1235, I find there is a potential issue in Mp4Extractor.java.
Technical Details
(1) Using AndroidX Media to parse mp4 file. For example, one Media3 test media: https://github.com/androidx/media/blob/release/libraries/test_data/src/test/assets/media/mp4/dolbyVision-hdr.MOV
(2) This test media includes one video track with 5 samples and frame rate is 30fps (You can check it by MediaInfo or other tools.). The key point here is this mp4 file has 'elst' box with segment duration = 64 (timescale = 600, so it is ~0.107s.) If you check the "TRACK DURATION" (moov->trak->tkhd→duration), it is same. But the "MEDIA DURATION" (moov->trak->mdia->mdhd->duration) is 100 (timescale  = 600, so it is ~0.167s.) All of above follow ISO spec and there is NO problem.
(3) In AndroidX Media, Mp4Extractor::processMoovAtom(), https://github.com/androidx/media/blob/release/libraries/extractor/src/main/java/androidx/media3/extractor/mp4/Mp4Extractor.java#L553, it calculates framerate by "TRACK DURATION" rather than "MEDIA DURATION", https://github.com/androidx/media/blob/release/libraries/extractor/src/main/java/androidx/media3/extractor/mp4/Mp4Extractor.java#L622
So the result is framerate = 5 / (106667/1000000) = 46.875fps, which is very different from the file's actual framerate (30fps).
(4) If the "MEDIA DURATION" is used, framerate = 5 / (166667/1000000) = 30fps, which is same as the file's actual framerate.
Thought
I think, as long as there is 'elst' box in MP4 file, the calculated frame rate will be wrong here.
Do you think we should change the behavior here (replace track duration with media duration) in Media3?

Hi, could you have a look whether this is an issue? :)

Do you think we should change the behavior here (replace track duration with media duration) in Media3?

Yes, you're correct. It is more accurate to derive the frame rate using the media duration stored in the mdhd box of each track. We have added a fix for this. Thank you for raising this issue!


Implement Nuxt Site Config

Clear and concise description of the problem
The convention for managing site urls is changing.
Suggested solution
Use nuxt-site-config to manage site url.
Alternative
No response
Additional context
No response

beta 60 will include this


Added Support to Call Handlers on Fatal

This pull request enhances the exported API with two new functions which allow for Exit Handlers to be invoked by any Fatal call.
New Functions
AddExitHandler( func() { ... } )
Exit(1)

While GO offers the ability to recover from panic there is no way to intercept an os.Exit event. To allow graceful shutdown and clean-up or programs which use Logrus to exit using a Fatal call I've borrowed the idea of the atexit package and enhanced Logrus to use it's own Exit() function which invokes handlers before quitting.
Usage

When setting up a logger one may optionally call RegisterExitHandler( func() {...} ) to add a function which will be invoked for any Fatal call to the logger before ultimately calling os.Exit(1).

I'm not sure how to test this properly without some means of sharing state between the test application and the logrus software under test.
Question:
Would you be ok with adding a build tag to the test file I introduced so that the test only runs when disk is available?

Need to update documentation and add examples before merging.

@aarongreenlee I had some concerns about adding this in https://github.com/sirupsen/logrus/pull/385#discussion_r95447494.

Specifically, how is support for this orthogonal to hooks?