Workaround for invalid resize event

Detects whenever a window resize event contains max u32 sizes as an invalid event that should be ignored.

Thank you so much! I'll merge this sometime later, when I can make sure that the fix is also ported to the v2.0 branch

This may have been fixed in winit since version 0.29.1 and backported to 0.28.7 (https://github.com/rust-windowing/winit/issues/2876), so since v2.0 updated its dependencies, this might not be a problem there.

Excellent! I'll close this then since it's fixed in v2.0

I take it you aren't doing a bug fix for this in v1.0?

Yep

Since I'm doing a 1.0 release anyway for the timer, I'll merge this fix as well.


edit results output text

i used a simple solution (using two string builders) because i dont think it's worth disrupting the order of psm and peptide analysis.
#1662

"targetText" change to "PsmPeptideProteinSummaryText"
"niceText" change to "TaskSummaryText"


Refactor settings module visibility controls

I've made changes to streamline the way we handle the visibility of certain settings in our settings module. After working on PR #46, I noticed we were repeating the same code for controlling visibility across multiple places. To clean this up, I introduced a new function, updateSettingVisibility(classSelector, visible), which consolidates the logic into a single, reusable method. This approach not only reduces duplication but also makes our code easier to maintain and understand.

LGTM


Dont assert in AddSrcMatchExtEntry and AddSrcMatchShortEntry

In the current version of SPINEL we maintain a local copy of src match tables, incase we need to do a RCP restoration. But if the local copy of the table is full, we assert irrespective of what RCP has returned as status. This PR changes this behaviour and instead of asserting radio SPINEL will just exit without updating the local copy of src match table.

@jwhui @abtink can you have a look at this please? A customer is hitting this assert when testing with large networks.

@jwhui @abtink Converting it to draft as we have some work to do on this. Will re-visit this soon.

Closing stale PR. Please feel free to reopen if/when you'd like to move this forward.


Avoid reloading protocol unless absolutely necessary

overview
Large python protocols can take 5 to 10 minutes to load, consuming operator time. Thus, reloading of protocols should be avoided unless absolutely necessary.
In particular, if a run is cancelled, and 'Reset Run' is clicked (per the UI prompt), the protcol currently gets reloaded. This seems unnecessary (though there might be deeper reasons).
current behavior
Protocol is reloaded on 'Reset Run' after run is cancelled.
steps to reproduce
See above.
expected behavior
After cancelling a run, a reset should be possible w/o reloading the protocol.
┆Issue is synchronized with this Wrike Task by Unito

Thanks for the request. The simulation + run system is a particularly gnarly (technical term) part of our software, and will be rebuilt in the coming months as part of a larger push to improve the calibration and run experiences.
Unfortunately, due to how the current system chooses to share state and object instances between calibration an runs (which is bad!), implementing this functionality in the current system would be a little too risky for us.
You may not see any movement on this ticket for a while as we plan out some of the lower level requirements for this re-build, but it is our intention to allow for the storage and retrieval of calibration data for a given protocol

Closing this in favor of #4360.


Getting Segment URLS for Dash, HLS

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
v3.0.6
Please ask your question
So I'm currently trying to get a list of segment URLS for the stream that's playing. I've managed to get a list of segments by using player.getManifest().variants[14]['video'].createSegmentIndex() and then using stream.segmentIndex. But even then, I've found no way of retrieving a list of segment URLS even though #1074 suggests that it should be possible using segment.createUris(). But, createUris() is deprecated now. Please note, that I'm currently testing this using the video and approach under basic usage, so it's a dash video stream, not a live stream. Ideally, I'd like to be able to do the same for both HLS and Dash, just so I can pre-warm the cache with segment Uris.

Hello,
Would you please try:
stream.segmentIndex.references[0].getUris() ? Let us know if it works for you. Thank you!

Hello,
Would you please try:
stream.segmentIndex.references[0].getUris() ? Let us know if it works for you. Thank you!

Not working. I'm getting an error:

myapp.js:73 Uncaught (in promise) TypeError: Cannot read property '0' of undefined
at onSegmentCreated (myapp.js:73)

Here's what I'm doing:
player.getManifest().variants[14]['video'].createSegmentIndex().then(onSegmentCreated, onSegmentCreationFailed)
and then in the onSegmentCreated function:
console.log(list.references[0].getUris())

Not working. I'm getting an error:

myapp.js:73 Uncaught (in promise) TypeError: Cannot read property '0' of undefined
at onSegmentCreated (myapp.js:73)

Here's what I'm doing:
player.getManifest().variants[14]['video'].createSegmentIndex().then(onSegmentCreated, onSegmentCreationFailed)
and then in the onSegmentCreated function:
console.log(list.references[0].getUris())

Try: const references = Array.from(stream.segmentIndex); references[0].getUris();.

Try: const references = Array.from(stream.segmentIndex); references[0].getUris();.

Not working. I'm now getting this error:

Uncaught (in promise) TypeError: references[0].getUris is not a function
at onSegmentCreated (myapp.js:75)

It looks like the getUris function isn't in the compiled bundle even though it is exported.  We'll look into it.


Update pytest.yml

Specify the strax to be v1.6.5
Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

Is this PR ready to be reviewed?

@chnlkx Please change the title of this PR to a meaningful one. :)

I will merge this but please @chnlkx update the title even after it is merged.


Relax restrictions.

User can view others' repo list and repo details
However, he cannot see the token or create the repo token.

Current coverage is 84.79%

Merging #49 into master will increase coverage by +0.30% as of a436bf7

@@            master     #49   diff @@
======================================
  Files           29      29
  Stmts          258     263     +5
  Branches         0       0
  Methods          0       0
======================================
+ Hit            218     223     +5
  Partial          0       0
  Missed          40      40

Review entire Coverage Diff as of a436bf7

Powered by Codecov. Updated on successful CI builds.


Updated Reuse Files tutorial with the feature to reference output of other tasks

This PR documents the ability to reference files from other tasks outputs to the existing Reuse Input Files Across Multiple Simulations tutorial.
I renamed the existing tutorial as Reuse Files Across Multiple Simulations since now it is also possible to reuse output files.

@maiyahershey I've added some text to the swan tutorial just to make sure the users know the limitations of this feature with swan.


Axios 409 Error Saving Workflow with SharePoint Connector

Describe the Bug with repro steps

In VS Code create workflow
Add Request Trigger
Add SharePoint HTTP Request action with existing SharePoint API connection from Azure
Save workflow. If API connection in Azure has a read-only lock, Axios 409 error results
If Azure resource lock changed to delete from read-only, workflow save will work

What type of Logic App Is this happening in?
Standard (VSCode)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
No
Workflow JSON
No response
Screenshots or Videos
No response
Browser
N/A - design time issue
Additional context
Just reporting this in case a) it help others or b) a more precise error message might be shown

Hi @anthonywhite thanks so much for raising this issue. I tried to repro in different ways but I couldn't. I added multiple locks and tried with different API connections. I might not be doing it in the right way. am doing it wrong. Could you explain a little bit more on how you added an existing SharePoint API connection from Azure into your local workflow?
I also have some questions

Which version of vscode extension are you using? Could you try with the latest?
Is this problem also occurring within the portal?

@anthonywhite I was able to repro the issue. There are a few notes about this.

Since the lock is for read-only. It does makes sense that the service doesn't allow to create new connections and therefore not allow to save the workflow
The vscode extension is showing what we are getting from the backend API call response when we call it, plus an insight that the error was when fetching the connection.
If we desire the get a more precise error message, we might need to move this issue to backend engineers


MSDK Installer Failure

I was installing all of our tools on a new install of ubuntu using the MSDK installer, and it seems that there is an issue with one of the commands. it looks like it used MkDir instead of mkdir in the shell. Second issue seems to be with a bad symlink. After that, seems like a chain of problems.
I'm not sure how doable it is, but if possible, we should add a workflow to test this process.

Thanks @EricB-ADI, this is the same issue as #516
I've had a fix to the installer package staged for release on the test server.  Just waiting for approval to push it out to production next week.
Agreed we need some auto-testing of the installer.  It's on my task list, should be able to get to it soon

Alright, im gonna close this then


[UWP][.NET] If icon url is not found, icons are rendered on top

Platform
What platform is your issue or question related to? (Delete other platforms).

UWP
.NET WPF

Details
The following card:
{
  "type": "AdaptiveCard",
  "version": "1.0",
  "actions": [
    {
      "type": "Action.Submit",
      "title": "Snooze",
      "iconUrl": "https://cdn1.iconfinder.com/data/icons/office-22/48/alarm-clock-512.png",
      "data": {
        "x": "snooze"
      }
    },
    {
      "type": "Action.Submit",
      "title": "Dismiss",
      "iconUrl": "http://bogusurl.png",
      "data": {
        "icon": "dismiss"
      }
    }
  ]
}

is rendered with only one icon on top, leading to different sized buttons:

Per #389 we should probably render buttons on the left in this case:
If iconUrl is null on ANY action within a set, the iconPlacement SHOULD BE leftOfTitle to ensure buttons are all the same height.
If you remove the "iconUrl": "http://bogusurl.png", the card renders as expected:

This would likely be tricky to fix as currently this logic switches on the presence or absence of the URL property and we don't know at that time if the url is valid.
(repros on UWP and .NET)

This is a low priority scenario that will be non-trivial to fix.


2FA- Error “missing Totp token”

Can’t log in to sh.it just.works, where I have 2FA enabled. Logging in to other instances where I don’t have 2FA is working

on further checking, it's a problem with Lemmy itself, so i'll co check there


Collect stacktraces for large number of IP violations

Fixes https://github.com/gradle/gradle/issues/29441
by increasing the limit for the number of collected stacktraces to 5000 when Isolated Projects feature is enabled

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.

The merge queue build has failed. Click here to see all failures.

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.


Component doesn't work in iOS 9

I've just installed XCode 7 and tested the project in iOS 9. It seems the component enters an infinite loop at the line 182 from UIScrollView+BottomRefreshControl.m: "[self brc_setContentOffset:contentOffset];"

It is strange that I only use it for CollectionView and TableView but it appears as calling setContentOffset on UITextView.

I got the same problem.It's so strange.

Maybe u use UITextView+Placeholder or other projects which have method +(void) load { [super load]; //}

Yes, that is exactly what am I using. Interesting. Do you have any ideas on how to fix it to be able to use both Categories?

Ok, that was exactly the problem. I've fixed it. Thanks!


[FEATURE REQUEST] Allow specifying a Postgres database for each validator/actor

As an operator user, I want to be able to specify a postgres database via an argument for CRD object, to provision the database outside the pocket-operator lifecycle (such as RDS).
We should make use of secretKeyRef/SecretKeySelector type - https://github.com/kubernetes/api/blob/669318b25bc70fee14165aa37ffb3f354e58a71f/core/v1/types.go#L2064 to enforce utilizing secret objects.

Could this noe be done via the configs in the core repo directly?

@okdas CLosing this out as done!


Python 3 support

Now pyrst uses suds library which is outdated (uploaded to pypy in 2010-09-15).
Suds is not compatible with python 3. Maybe it is time to switch to https://pypi.python.org/pypi/suds-jurko/0.6 or to https://pypi.python.org/pypi/suds-py3?

Good idea.
I am no longer using Birst (I have moved to a different company, one that does not use Birst much), but I welcome upgrades!


CompositeByteBuf.toString(int index, int length, Charset charset) fails when the buffer has more than two components

Using Netty 4.0.33:
CompositeByteBuf.toString(int index, int length, Charset charset) ultimately ends up in ByteBufUtil.decodeString(ByteBuf src, int readerIndex, int len, Charset charset):
if (src.nioBufferCount() == 1) {
     // Use internalNioBuffer(...) to reduce object creation.
    decodeString(decoder, src.internalNioBuffer(readerIndex, len), dst);
} else {
    // We use a heap buffer as CharsetDecoder is most likely able to use a fast-path if src and dst buffers
    // are both backed by a byte array.
    ByteBuf buffer = src.alloc().heapBuffer(len);
    try {
        buffer.writeBytes(src, readerIndex, len);
        // Use internalNioBuffer(...) to reduce object creation.
        decodeString(decoder, buffer.internalNioBuffer(readerIndex, len), dst);

in the last line we try to decode from readerIndex on the newly allocated buffer, but we should decode from 0.
A small test to reproduce the bug:
    @Test
    public void testToString() {
        ByteBuf a = Unpooled.copiedBuffer("First part,", CharsetUtil.US_ASCII);
        ByteBuf b = Unpooled.copiedBuffer("then part two", CharsetUtil.US_ASCII);

        ByteBuf concatenated = Unpooled.wrappedBuffer(2, a, b);

        concatenated.toString(1, 1, CharsetUtil.US_ASCII);
    }

This test thows an IndexOutOfBoundsException.

I think this was fixed already. Can you try 4.0.34-SNAPSHOT ?

On 26 Jan 2016, at 15:45, Rolf Arne Corneliussen<EMAIL_ADDRESS>wrote:
Using Netty 4033:
CompositeByteBuftoString(int index, int length, Charset charset) ultimately ends up in ByteBufUtildecodeString(ByteBuf src, int readerIndex, int len, Charset charset):
if (srcnioBufferCount() == 1) {
// Use internalNioBuffer() to reduce object creation
decodeString(decoder, srcinternalNioBuffer(readerIndex, len), dst);
} else {
// We use a heap buffer as CharsetDecoder is most likely able to use a fast-path if src and dst buffers
// are both backed by a byte array
ByteBuf buffer = srcalloc()heapBuffer(len);
try {
bufferwriteBytes(src, readerIndex, len);
// Use internalNioBuffer() to reduce object creation
decodeString(decoder, bufferinternalNioBuffer(readerIndex, len), dst);
in the last line we try to decode from readerIndex on the newly allocated buffer, but we should decode from 0
A small test to reproduce the bug:
@Test
public void testToString() {
    ByteBuf a = UnpooledcopiedBuffer("First part,", CharsetUtilUS_ASCII);
    ByteBuf b = UnpooledcopiedBuffer("then part two", CharsetUtilUS_ASCII);

    ByteBuf concatenated = UnpooledwrappedBuffer(2, a, b);

    concatenatedtoString(1, 1, CharsetUtilUS_ASCII);
}

This test thows an IndexOutOfBoundsException
—
Reply to this email directly or view it on GitHub https://github.com/netty/netty/issues/4757.

@normanmaurer verified it does not throw IndexOutOfBoundsException with 4.0 branch.

@johnou thanks!

Yes, verified that it is fixed in 4.0.34.SNAPSHOT. Any idea of when 4.0.34 will be released?

This week

Am 26.01.2016 um 16:28 schrieb Rolf Arne Corneliussen<EMAIL_ADDRESS>Yes, verified that it is fixed in 4.0.34.SNAPSHOT. Any idea of when 4.0.34 will be released?
—
Reply to this email directly or view it on GitHub.

@normanmaurer Great!


net/http: Content-Length: 0 is not set for PATCH requests with empty body

What version of Go are you using (go version)?
1.15
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
darwin amd64
go env Output
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/tharindu/GoCode/bin"
GOCACHE="/Users/tharindu/Library/Caches/go-build"
GOENV="/Users/tharindu/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/tharindu/GoCode/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/tharindu/GoCode"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.15/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.15/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/3l/cw1f6lvn6p1c_vcg0cnbmqwc0000gp/T/go-build547496291=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

package main

import (
	"fmt"
	"io/ioutil"
	"net/http"
	"strings"
)

func main() {

	url := "https://myapi.net/logs/file/a?action=aa"
	method := "PATCH"

	payload := strings.NewReader("")
	client := &http.Client {
	}
	req, err := http.NewRequest(method, url, payload)

	if err != nil {
		fmt.Println(err)
	}
	req.Header.Set("Authorization", "Bearer my-token")
	req.Header.Set("Content-Length", "0")

	res, err := client.Do(req)
	defer res.Body.Close()
	body, err := ioutil.ReadAll(res.Body)

	fmt.Println(string(body))
}

What did you expect to see?
Content-Length header getting set to 0 when the request is observed from the server side.
What did you see instead?
Content-Length header is not getting set for PATCH requests with empty/nil payload. It is getting set without any issue for other http methods (POST, PUT).
It is also getting set for PATCH requests if the payload is non empty.
Also setting req.Header.Set("Content-Length", "0") seems to have no effect on the actual content length getting set in the request.

Thank you for filing this issue @tharinduwijewardane and welcome to the Go project!
I’d like to understand the motivation here. Was there a failure on some intermediate or front end server, say Nginx or is your service breaking due to lack of that 0? I ask because in 9+ years no one had reported this being a problem. Did you discover this perhaps just by a code audit, or an actual failure? Please let us know. The impetus for adding that 0 for POST and PUT is that RFC 2616 says we can include 0 only if the method doesn’t prohibit it, and Nginx apparently needs that too. However, nothing has been reported/requested for DELETE nor PATCH in the past 9 years (almost decade).

@odeke-em I came across this when invoking an Azure data lake API (https://docs.microsoft.com/en-us/rest/api/storageservices/datalakestoragegen2/path/update).
They deliberately reject the PATCH request when the Content-Length is not set to 0 even if there is no content. Maybe that is something they should not do.
However, at-lease we should be able to manually set the content-length header right?
In this case req.Header.Set("Content-Length", "0") is not honoured by http package.
I think we should be able set whatever the header we need.

Thank you for providing the background, @tharinduwijewardane! I have cited your case in the fixing CL https://go-review.googlesource.com/c/go/+/250039. To get around this, firstly am away from my computer for a couple of hours but I’ve written for you a repro with a stand-alone server using my iPad here https://play.golang.org/p/43_KR2riBiT, please let me know what it prints out on your computer.

@odeke-em "In server" the content-length header is not received.
PreRequest PATCH / HTTP/1.1
Host: <IP_ADDRESS>:65209
Content-Length: 0

In server PATCH / HTTP/1.1
Host: <IP_ADDRESS>:65209
Accept-Encoding: gzip
User-Agent: Go-http-client/1.1

Response HTTP/1.1 200 OK
Date: Mon, 24 Aug 2020 08:17:40 GMT
Content-Length: 0

Thank you for running them! Given that you are the sole reporter in the past 9 years, I don’t think that this could qualify for a backport, but would it be alright for you to use go tip for the next 3+ months until Go1.16 is released?  From my research it was proposed in 2010 in https://tools.ietf.org/html/rfc5789

It just says that as long as the method doesn’t forbid sending the
Content-Length, it can be sent which is my interpretation to say that it
doesn’t forbid it.
On Mon, Aug 24, 2020 at 1:28 AM Dave Cheney<EMAIL_ADDRESS>wrote:

Are you sure that rfc2616 defines patch? From my research it was proposed
in 2010 in https://tools.ietf.org/html/rfc5789
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/40978#issuecomment-678986077, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFL3V4B3A7EMLKVNIAVOOTSCIQC5ANCNFSM4QIEEM2Q
.

@odeke-em Thank you. But this is no longer a blocker for me as there is a workaround/hack.
By setting req.TransferEncoding = []string{"identity"} I can get it to add the header.

It just says that as long as the method doesn’t forbid sending the Content-Length, it can be sent which is my interpretation to say that it doesn’t forbid it.
…

That’s probably for connection reuse to work.
What’s the use case for sending a PATCH with an empty body, surely the server will reject it?

Thanks Dave for the citations. In regards to the use case, the original
reporter stated that they need to send an empty PATCH request in order to
flush the contents of a file that was previously uploaded, in reply to my
similar question as per
https://github.com/golang/go/issues/40978#issuecomment-678946195
On Mon, Aug 24, 2020 at 1:36 AM Dave Cheney<EMAIL_ADDRESS>wrote:

It just says that as long as the method doesn’t forbid sending the
Content-Length, it can be sent which is my interpretation to say that it
doesn’t forbid it.
… <#m_-5160185177206370588_>
That’s probably for connection reuse to work.
What’s the use case for sending a PATCH with an empty body, surely the
server will reject it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/40978#issuecomment-678990325, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFL3V443H3YAWDWSSQSOS3SCIRCHANCNFSM4QIEEM2Q
.

Ok, thanks for confirming


Error when I try to use exchange like Yobit, Cryptopia...

When I try to analyze one echange like Yobit or others it show this error :
File "/app/exchange.py", line 59, in get_historical_data
app_1  |     if time_unit not in self.exchanges[exchange].timeframes:
app_1  | AttributeError: 'gateio' object has no attribute 'timeframes'
any suggestions?
Thanks.

It's possible that these exchanges don't have sufficient support in ccxt which is the library we use to fetch exchange data. That said, there is a function available to us that we don't check which would tell us if the exchange / ccxt has support for dealing with their candle timeframes.
We should implement better error handling so that it doesn't crash and burn in mysterious ways.

Fixed crashing without meaningful message in https://github.com/CryptoSignal/crypto-signal/pull/208, waiting for review.


feat:add an ignore option for 'no-extraneous-dependencies'

If the alias module starts with "@", the current "no-extraneous-dependencies" rule will report an error in the case of "@custom/XXX".

Coverage increased (+0.007%) to 97.363% when pulling 9a57101ca88b412faf91b52038df9156fc63afcb on wheasy:master into db471a85573a88e0bb9f4a1d53f40fed603651d1 on benmosher:master.

Coverage increased (+0.007%) to 97.363% when pulling 9a57101ca88b412faf91b52038df9156fc63afcb on wheasy:master into db471a85573a88e0bb9f4a1d53f40fed603651d1 on benmosher:master.

Coverage increased (+0.007%) to 97.363% when pulling 9a57101ca88b412faf91b52038df9156fc63afcb on wheasy:master into db471a85573a88e0bb9f4a1d53f40fed603651d1 on benmosher:master.

Coverage increased (+0.007%) to 97.363% when pulling 9a57101ca88b412faf91b52038df9156fc63afcb on wheasy:master into db471a85573a88e0bb9f4a1d53f40fed603651d1 on benmosher:master.

Duplicate of #1227 (please try not to open duplicate PRs)

This seems like a strange approach to fix the problem of being unable to have an alias that’s a valid package name (with an @). Maybe that should be fixed instead?

This seems like a strange approach to fix the problem of being unable to have an alias that’s a valid package name (with an @). Maybe that should be fixed instead?

This is really not a good solution. I will close this pr.


Recommended heuristic for integrating "policy tree" with "honest causal forest"

The below post is more of a methodological question than a technical one.
Based on what I've gathered, the honest causal forest and policy tree are two distinct yet related methods. Both can evidently yield actionable insights on the effects of treatment within a heterogeneous population. However,

The honest causal forest can estimate conditional average treatment effects across non-prespecified and heterogeneous subgroups
Whereas, the policy tree can identify a data-driven optimal treatment rule for a given sample based on the observed data

So, while the honest causal forest is a bit more exhaustive (since it estimates pretty granular subgroup-specific causal estimates), the policy tree provides a bit more of "broad brush" strategy to these heterogeneities by identifying an optimal treatment rule that can be applied to a population for making treatment decisions.
Question:
Given that both the honest causal forest and policy tree address similar research objectives (i.e., to help understand heterogeneities that exist in a population, so that we can make better decisions), has your research group developed any standard protocols or heuristics for incorporating the results of the honest causal forest into the inputs of the policy tree model? For example, would it be reasonable to develop a protocol whereby we only input covariates into the policy tree that were listed among the top 10% of the most "Important" variables for heterogeneities within the honest causal forest, or something like that?

Hi @njawadekar

For example, would it be reasonable to develop a protocol whereby we only input covariates into the policy tree that were listed among the top 10% of the most "Important" variables for heterogeneities within the honest causal forest

Yes, that's a perfectly fine heuristic and is suggested here https://github.com/grf-labs/policytree/issues/46 in order to make a setting with many covariates feasible for policy_tree.
Our research group (@halflearned) has been working on an online tutorial for ML-based HTE estimation, you might find the section on policy learning useful: https://bookdown.org/stanfordgsbsilab/tutorial/policy-learning-i-binary-treatment.html

Also, if you're looking for a real-world empirical application, @hhsievertsen has a paper using causal forest + policy tree here https://github.com/hhsievertsen/hhsievertsen.github.io/raw/master/mat/wp/chx_sep2021.pdf

Dear @erikcs  ,
First thanks a lot for the package and the guidance!
I have a question related to the one raised by Njawadekar, and I was hoping you could provide some additional intuition.
I'm running the policy tree algorithm, where I only input covariates that are among the top 10% of the most "important" variables based on the DR scores from that causal forest that I took these variables from. I followed the tutorial you recommended (https://bookdown.org/stanfordgsbsilab/tutorial/policy-learning-i-binary-treatment.html). Before running the policy tree, I adjusted the double-robust scores by multiplying them by -1, since I am aiming to minimize dropout rates. However, there are still a few points that seem unclear to me, and I might be misunderstanding some aspects.
When I estimate the value of the learned policy, I observe for example a decrease of about 4 percentage points. However, when I examine the tree and test whether the treatment effects vary across the leaves, I run into some confusion. If I understand correctly, a "Leaf node action = 2" indicates the individuals for whom the policy is recommended. However, I never see a decrease of more than 1 percentage point for any leaf node = 2 (so how can this be an optimal tree and still have an overall value of 4 percentage point). Most of the larger percentage decreases occur in leaf node = 1, and even then, for relatively small sample sizes. This makes it difficult for me to reconcile the estimated policy value of a 4 percentage point decrease?
Am I misinterpreting the results of the tree in relation to the estimated value of the learned policy, or might there be something else I am doing incorrectly?
Thank you so much!

Hi @hanneleer, sorry, I'm having some trouble understanding exactly what you are asking for, if you post a simple code example along with toy/real data that illustrates your question, that could probably help.

Hi @erikcs - thanks a lot for coming back to me!! (I am not sure how to get it in perfect tables and codes in github - I am really sorry about the way the code is provided!)
So I started with just fitting my causal forest - and retrieved the double robust scores. As I look at the impact of the policy on the probability of dropping out (aim: lower dropout = better) - I also multiplied my gamma.matrix with -1 to be sure that I minimize the rewards.
Fit a policy tree on forest-based AIPW scores
Forest <- causal_forest(data$X, data$Y, data$W, cluster = data$school_level)
CATE_Forest <- average_treatment_effect(Forest, target.sample = "all")
"Estimate: -0.0106, Std. Error: 0.0013"
Gamma.matrix <- double_robust_scores(forest)
Gamma.matrix <- double_robust_scores*-1
head(Gamma.matrix)
[,1]             [,2]
[1,]  -2.0112      -0.03817
[2,]  0.01469     -0.01055
[3,]  -0.0071       0.00363
[4,]  0.05937      -0.04230
[5,]  0.02668      -0.01810
[6,]   0.01027    -0.00745
[reached getOption ("max.print") -- omitted 135000 rows ]
Divided the data into train and evaluation sets
Train <- sample(nrow(data), size = nrow(data)*0.7, replace = F)
Test <- data[-train,]
Fit the policy tree
policy_tree <-  hybrid_policy_tree(X[Train,], Gamma.matrix[Train,], depth = 5)
plot(policy_tree)
#policy_tree object
Tree depth:  2
Actions:  1 2
#Variable splits:
#(1) split_variable: X4  split_value: 1
#(2) * split_variable: X3 split_value: 0
# (4) split_variable: X2  split_value: 0
# (8) * action: 2
# (9) * action: 1
# (5) split_variable: X5  split_value: 0
# (10) * action: 2
#(11) * action: 1
#(3) split_variable: X1  split_value: 0
#(6) * action: 2
#(7) split_variable: X5  split_value: 1
#(12) * action: 2
#(13) * action: 1
``

Predicting treatment on test set
pi.hat <- predict(policy, X[Test,]) - 1
Predicting leaves
Leaf <- predict(policy, X[Test,], type = "node.id")
Num.leaves <- length(unique(Leaf))
Estimate the value of the learned policy**
gamma.hat.1 <- Gamma.matrix[Test,2]
gamma.hat.0 <- Gamma.matrix[Test,1]
gamma.hat.pi <- pi.hat * gamma.hat.1 + (1 - pi.hat)  * gamma.hat.0
value.aipw.estimate <- mean(gamma.hat.pi)
value.aipw.stderr <- sd(gamma.hat.pi) / sqrt(length(gamma.hat.pi))
"Estimate [AIPW]: -0.0574 Std. Error: 0.001"
Is treatment effect different across treatment or control group?
ols <- lm(gamma.hat.1 - gamma.hat.0 ~ 0 + factor(pi.hat))
coeftest(ols, vcov=vcovHC(ols, 'HC2'))[1:2,]
estimate           standard error
factor(pi.hat) 0     0.0089              0.0048
factor(pi.hat) 1      0.0116              0.002

So when I estimate the value of the learned policy, I observe for example a decrease of about 5.7 percentage points. However, when I examine the optimal tree and test whether the treatment effects vary across the leaves, I run into some confusion. If I understand correctly, a "Leaf node action = 2" indicates the individuals for whom the policy is recommended. However, I never see a decrease of more than 1.8 percentage point for any leaf node = 2 (so how can this be an optimal tree and be a subgroup recommended to treat?). This makes it difficult for me to reconcile the estimated policy value of a 5 percentage point decrease?
Moreover if I look if the treatment effect is different across treatment and control group - I also do not see negative results for the estimate for the treatment group (0.0116). When I check whether the tree is optimal:  all(apply(Gamma.matrix,1,which.max) == predict(policy_tree, X)) - I also get a FALSE response but I am really confused how this is possible. Am I misinterpreting the results of the tree in relation to the estimated value of the learned policy, or might there be something else I am doing incorrectly?
Thanks a lot!!


Model disappears during staging & camera change on iOS (since v2.0.0)

Description
After updating to v2.0.0 on iOS the model disappears during staging & camera change (rotation/zoom/pan etc.)
It reappears after the interaction finished. This includes user interactions and also programatically initiated ones.
The issue 100% reproducible with all modelviewer.dev examples too on iPhones (with iOS 15.2)
https://user-images.githubusercontent.com/19240196/193558721-3a748cc6-6c60-435d-80f7-f058c61e3941.MP4
Same issue happens with animated models too without user interaction:
https://user-images.githubusercontent.com/19240196/193560967-c5566e92-272e-4796-b951-85551cf0e926.MP4
Reproduce:
Visit modelviewer.dev/examples with an iPhone running iOS 15 and rotate the models.
Expected behavior:
Model should be visible during the rotation.
Live Demo
visit this with iPhone running iOS 15.2:
https://modelviewer.dev/examples/stagingandcameras/
Version

model-viewer: v2.0.0

Browser Affected

[x] Chrome
[ ] Edge
[x] Firefox
[ ] IE
[x] Safari

OS

[ ] Android
[x] iOS, version: 15.2  ( it can't be reproduced with iOS 16.0 or iOS 15.4.1 )
[ ] Linux
[ ] MacOS
[ ] Windows

Indeed v1.12 works properly and v2.0 not that's the interesting part.

I can confirm we also started observing this exact issue after upgrading v1.12.1 to v2.0.2.

Considering iOS 15.2 didn't even make it onto this version share chart, I'm not going to prioritize fixing this, especially since I don't have an iOS 15.2 device to repro with. If any of you would like to debug and submit a PR, I'd be stoked though!


Image_Augmentation Added

@ShambhaviCodes Is this WIP? I can see a lot of things missing from here

Okay yes, I didn't include a training model.
Will add it.

Anymore changes required?

@ShambhaviCodes We've to exactly mirror all the sections from here I can see you've condensed many sections into one. If possible try to use the same image of cat used in d2l. The model and the training dataset also differ but I think that maybe acceptable.
/cc @AnirudhDagar

Hi @ShambhaviCodes sorry for the staleness of the project. Maintainers are busy nowadays. I can merge your PR if you can deliver the following things:
-[ ] Mirror the sections of d2l original repo as closely as possible without merging it into one. You can find utilities like show_images  in the d2l package in the repo. If you need any other utility feel free to implement one.
-[ ] Download and use the image of turtle from original repo
-[ ] If Cifar-10 dataset is possible try to run the model on it. If not then MNIST is fine given the limited resources we have
PS: I can understand Multi-GPU training will not be possible at all

Hello @CaptainDredge
Will make the changes and push over this weekend.
Thanks


ZigBee/ZHA - Multiple errors in home-assistant.log

Home Assistant release with the issue:

0.95.0
Last working Home Assistant release (if known):
Unknown - first use of a Hue Sensor in ZHA
Operating environment (Hass.io/Docker/Windows/etc.):

Rasbpery Pi 3B+ 32bit HassOS / HassIO image.
Component/platform:

ZHA Componet.
Description of problem:
I have recently added a Philips Hue Motion Sensor to ZHA and multipel errors seem to be present in the Home-Assistant.log file.
The sensor appears to function, other than the red LED lighting when motion is detected (which usually indicates an error communicating with the bridge).
I use a Digi Xbee2 USB co-ordinator and the errors are multiple errors generated by zigpy_xbee.zigbee.application
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
No YAML code to relate to this at this time.

  Not sure if it is related or a different issue:
Sat Jul 06 2019 12:37:12 GMT+1000 (Australian Eastern Standard Time)
Error doing job: Exception in callback <bound method SerialTransport._read_ready of SerialTransport(<uvloop.Loop running=True closed=False debug=False>, <zigpy_xbee.uart.Gateway object at 0x7144b070>, Serial<id=0x7144b2f0, open=True>(port='/dev/ttyUSB0', baudrate=57600, bytesize=8, parity='N', stopbits=1, timeout=0, xonxoff=False, rtscts=False, dsrdtr=False))>
Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 66, in uvloop.loop.Handle._run
  File "/usr/local/lib/python3.7/site-packages/serial_asyncio/__init__.py", line 106, in _read_ready
    self._protocol.data_received(data)
  File "/usr/local/lib/python3.7/site-packages/zigpy_xbee/uart.py", line 78, in data_received
    self.command_mode_rsp(rsp)
  File "/usr/local/lib/python3.7/site-packages/zigpy_xbee/uart.py", line 55, in command_mode_rsp
    data = data.decode('ascii')
UnicodeDecodeError: 'ascii' codec can't decode byte 0x91 in position 3: ordinal not in range(128)

the 1st one is a confirmed bug and shouldn't affect anything, as it just Device announce zdo message. Fixed in https://github.com/zigpy/zigpy-xbee/pull/49
For the second one, there was another report of this behavior in https://github.com/zigpy/zigpy-xbee/issues/48
if you can reproduce it, please post the full logs (enable debug log for xbee: zigpy_xbee: debug) into that xbee issue or PM me the log files on Discord.
I'm going to close this issue here, as it belong to zigpy-xbee


stop using unwrap() in value parsers

description
the value parsers for the db drivers uses unwrap a lot
proposed solution
maybe fall back to an empty string or something else to indicate if the type can't be cast to a string

I just tried with sqlite file saved from QGis and it crashed  here:
https://github.com/achristmascarl/rainfrog/blob/0159133eda01ca9fb6ddfd7de9d71dca5899d524/src/database/sqlite.rs#L125
on table that has column of type MULTIPOLYGON. I think sqlx does not support that type, but it would be nice if it just ignored those values and displayed other columns.

@platlas let me know if you have a chance to test this with the MULTIPOLYGON type after the new release and if it still crashes 🙏

It works now without crashes. Thanks!


Support for async refinements

When using a .refine that returns a Promise I get the following error from Zod:
Async refinement encountered during synchronous parse operation. Use .parseAsync instead.

I assume nestjs-zod uses .parse under the hood. Is there a way to tell nestjs-zod to use .parseAsync instead?

I also would like to make some database verification, and should to be async validation. How to switch from .parse to .parseAsync?

I managed to have just that, I describe it here: https://github.com/anatine/zod-plugins/issues/34#issuecomment-1450823908
It would be nice to get proper support for that from one of those nestjs-zod projects though.


Enable iOS h264 support

A few weeks ago, h264 hardware support was added. Currently, this requires setting the 'rtc_use_objc_h264' compile time flag. It would be great if this could be added to the build scripts and the pre-compiled builds.

I just tried building the latest library (9554). I have been using a much older library from January but wanted to see how the latest version works. (The new library needed the VideoToolbox.framework added to link properly.)
With the latest library, I am getting just a black rectangle for my local video. Could this be related to this issue or does not having this flag simply mean it doesn't use h264 acceleration? Are you seeing a black video screen for local video as well?
Thanks.

Usually black local view means the local camera wasn't opened correctly. Are you using any video constraints?

I don't believe so. Using the older library works fine.

I take that back. You are absolutely correct. I do have some constraints in place to lower the resolution and frame rate in order to stop the video from lagging. Commenting them out does show the local view.
With some experimentation, it appears that it is setting the resolution to 320x240 that is causing the failure. Given setting the frame rate does not do anything (and the frame rate is what I really want to change), it looks like this is a problem for me until the Google folks fix the issue where setting a frame rate constraint on iOS doesn't actually change anything.
Thanks for your on target help. Sorry for adding an unrelated topic to this thread.

To link correctly, I had to add VideoToolbox.framework. Is this related?

as bwilks128 confirmed, the issue was related to the video constraints. The current version will only do 640x480

Hi all,
If I'm reading this correctly, @bwilks128 bug was unrelated to the original feature request.
The original request:

A few weeks ago, h264 hardware support was added. Currently, this requires setting the 'rtc_use_objc_h264' compile time flag. It would be great if this could be added to the build scripts and the pre-compiled builds.

Is it possible this is still in the pipeline?
Thanks in advance!

I am trying to set the video codec to h.264, for this I changed GYP_DEFINES="chromium_ios_signing=0" to GYP_DEFINES="chromium_ios_signing=0 use_objc_h264=1" in function wrbase in build.sh. However the offer still has vp8 in it instead of h264, not sure what I am missing here.

nvm got it to work in the debug lib

Did anyone figure this out?

Is there any progress on this?

Same problem here. I don't find any build/common.gypi in which I can set use_objc_h264


Update collector.go

Add handling of nil values returned by pem.Decode()
This fixes this issue for prometheus bosh release

thanks for the PR @damzog
I'll release a patched version and submit a PR to prometheus-boshrelease asap


Rename all occurrences of Tone to Pitch

Quite a large PR here but there were a lot of uses of "tone".
Also fixed related file names, comments and made sure to correctly pluralize tones to pitches.
I'm not sure what the class diagram is, that is mentioned in #113

I updated the class diagram for you. It uses the software: UMLet

Makes sense. Waiting on completion of #118


Revert "Replace some more characters in format_value."

Reverts w3c/testharness.js#188

@inseoun What's the motivation to revert this change?

This is not the first time there was a destructive PR from this person, I suggest ignoring.


Separated Slack bot docs from README

This helps removed the cluttered part of the README file by separating Slackbot documentation in docs/
Closes #3

@neodark LGTM


Add the ability to install a gem from a local file in sensu_gem LWRP

I was trying to fix an issue in a existing plugin and after building the new version (a new gem) I would like to copy the file to a cookbook and tell chef to install that gem from a local file, like what is described in the chef docs for the gem resource
gem_package "right_aws" do
  source "/tmp/right_aws-1.11.0.gem"
  action :install
end

https://docs.chef.io/resource_gem_package.html
Probably, it isn't a very common use case for most of the users, but I think that it can be useful.

+1
or install from github

Added in #408 which will be part of the next release.


RC commands are yaw dependent (in manual mode)

In Manual mode, when I monitor the output (_actuators.control[]) of the attitude stabilization of my quadcopter (in mc_att_control_main.cpp) I see that for a constant roll RC input command the _actuators.control[0] and _actuator_control[1] changes when I turn the quadcopter in yaw (by hand).  For example when I turn it by 180° in yaw the _actuators_control[0] changes to be moreless the opposite...
In Acro mode this weird behavior doesn't happen.
Why?
This seems to be as if the RC commands were given in the World Reference Frame. But in manual mode shouldn't it be in Body Reference Frame?
I am using PX4 v1.8.0 with the pixracer board tilted by 90° in pitch (ROTATION_PITCH_90) but I am disabling the magnetometers.
Thanks

Background
@alexcherpi Correct me if I misunderstood you. How PX4 currently does it is it generates a complete 3D attitude setpoint orientation including yaw from your sticks https://github.com/PX4/Firmware/blob/72b898fbf6d9f61c60da872dabed66a82974aaea/src/modules/mc_att_control/mc_att_control_main.cpp#L413-L443. The yaw is generated by moving the absolute yaw around with the rate given by the stick here: https://github.com/PX4/Firmware/blob/72b898fbf6d9f61c60da872dabed66a82974aaea/src/modules/mc_att_control/mc_att_control_main.cpp#L421
After that the attitude controller gets called to execute on that setpoint: https://github.com/PX4/Firmware/blob/72b898fbf6d9f61c60da872dabed66a82974aaea/src/modules/mc_att_control/mc_att_control_main.cpp#L521 The controller will always take the "shortest path" to reach the desired attitude but will prioritize roll and pitch over yaw with the yaw weight applied here https://github.com/PX4/Firmware/blob/72b898fbf6d9f61c60da872dabed66a82974aaea/src/modules/mc_att_control/AttitudeControl/AttitudeControl.cpp#L84.
Note that the strategy of the algorithm didn't change for a long time (not new in 1.8) the only thing that changed in 2017 is the quaternion attitude controller executing the setpoint https://github.com/PX4/Firmware/pull/8003.
Answer
I would say that should explain why when you turn your vehicle in a bench test it will try to reach back to the original absolute yaw it locked to and not just correct roll. This algorithm strategy things has the big advantage that the yaw angle will not drift over time given your yaw estimate is correct and doesn't drift (which it might not be anymore without magnetometer). As a result you can fly very straight lines in manual and usually don't have to touch the yaw stick. I assume what you expected to see is only roll and pitch being controlled in an absolute manner and for yaw only the rate controller being ran.

This seems to be as if the RC commands were given in the World Reference Frame. But in manual mode shouldn't it be in Body Reference Frame?

Conclusion
I'm open to finding an appropriate solution for example an option for your use case or eventually after in-depth testing convince myself of a better default way to handle yaw in manual. Could you explain what your goal is? That would help to understand the requirements.
Ideas
@bresch Just told me about the use case where you have a vehicle that has very big yaw errors while flying normally or even doesn't care about the yaw at all and has disabled yaw control. In that case correcting roll pitch according to a setpoint which assumes a different yaw angle to be locked than the vehicle actually has is totally confusing. Is that your use case?
There are multiple actions we can take:

(Optionally) switch stick setpoint generation for roll and pitch to the current yaw vehicle state and test the real world accuracy difference
Expose the yaw weight of the attitude controller such that reducing significance of or even disabling yaw tracking is easily possible without screwing up the output gains (that is anyways on my todo list).
(Optionally) switch yaw control to rate control only for testing. That would get rid of any yaw lock and just try to reduce drift.

Dear MaEtUgR,
Problem solved! But thank you for your complete answer!
Actually I rebased my branch from v1.8.0 to v1.8.2 and the issue disappeared.
The cause are possibly the following: issue in v1.8.0 OR mess in my code (I would go for the 2nd option).

Problem solved! But thank you for your complete answer!
Actually I rebased my branch from v1.8.0 to v1.8.2 and I changed my custom mixer to the 'Generic Quadrotor X' mixer and the issue disappeared.
The cause are possibly the following: issue in v1.8.0 / mess in my code / mess with my custom mixer.
Sorry if it cannot help PX4 to be improved but thanks for your help

Nice to hear! Thanks for the update and feel free to reopen and ask back if you have further questions or ideas. I will close this issue now.

@alexcherpi I don't understand why you don't see the issue anymore, the logic did not change and as @MaEtUgR  explained, there is a reason why it is like that right now.
There is a workaround for VTOL planes that have bad yaw tracking here:
https://github.com/PX4/Firmware/blob/90758258723f837dcadf1d4c303229a722f64ec9/src/modules/mc_att_control/mc_att_control_main.cpp#L455-L490
Maybe we can put a parameter instead of if(is_vtol) so you can also use it with any "free to yaw" vehicle.

Hi bresch, the cause are possibly the following:

issue in v1.8.0 (doesn't seems to be the case)
mess with my custom mixer (normally not as it is used after creating the _actuators.control[] uorb message)
mess in my code
So I think the poblem was a mess in my code, sorry if it cannot help PX4 to be improved.


Plutus contact accessibility for development

@j-mueller has created a Plutus contract that we can compile to target x86. This can be run in our Docker execution service.
To facilitate and test this, we should:

Write a static bundle that represents this contract. This will include the base64 encoded executable and the graphQL schema
Create a development service in our docker-compose file serves this bundle
Do an end to end test across the services to ensure things are configured as we expect (API -> contract proxy -> docker execution)

Jann's Plutus PR that contains this work is here: https://github.com/input-output-hk/plutus/pull/730

👍

Create a development service in our docker-compose file serves this bundle

Let's treat the service as a first-class component Bundle Server src/bundle_server, which we can include in the dev stack, but it should also be a useful tool for production deployments

#46


Catch error in write to remove broken file

When the write command errors it still sometimes construct a broken file on disk.
This change should make sure, that  the write function would tidy up these broken files after faliure, because they are in the way for later analysis.
Do you think, that this is enough to capture all broken files?
Should this also be tested for other backends?
Locally I get some spurious failures which have nothing to do with the changes but are coming from some download time out errors.

Codecov Report

Merging #483 (17c4cac) into main (2603bd1) will decrease coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #483      +/-   ##
==========================================
- Coverage   81.15%   80.98%   -0.17%
==========================================
  Files          49       49
  Lines        4113     4108       -5
==========================================
- Hits         3338     3327      -11
- Misses        775      781       +6

Files Changed
Coverage Δ

src/write.jl
89.79% <100.00%> (+0.90%)
:arrow_up:

... and 7 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I think we need to first check the file didn't already exist!
One error could be that force=true was not used and the file existed.

Closing as stale, feel free to continue


Get Notifications from Opener

Would it be possible to get a notification if the ring of the opener has been triggered? Nuki Web offers this feature, however I was unable to find any information regarding this feature on the Bridge Http Api.

See #29


Add stopWatch as controler in visual studio toolbox

Issue Type

[ ] Bug Report
[ ] Feature Request
[ ] Sample request
[ ] Documentation issue or request
[ ] Other – Please describe:
UserVoice link

Current Behavior

Expected Behavior

Steps to Reproduce (for bugs)

Your Environment

SDK #:
Operating System build # :
Visual Studio Version # :

Hi @khoshroomahdi,
Thanks for the feedback. For UWP platform feature suggestions, could you instead please open an item in either the Windows 10 Feedback Hub or the UWP UserVoice feedback forum:
https://wpdev.uservoice.com/forums/110705-universal-windows-platform
If you do submit feedback or a suggestion through those channels, could you also provide more details on what you're trying to accomplish with a stopwatch?


Log the IP address in audit.log

Created from https://wso2.org/jira/browse/IDENTITY-4389
Currently, IS 5.1.0 does not support ip-address in audit.log.
Needs to log the IP address in <IS_HOME>/repository/logs/audit.log.

Logging support for user id and the authenticator should also be added.

In IS-5.10.0, IP address can be enabled in audit logs by adding the following config in <IS_HOME>/repository/conf/deployment.toml file.
[audit.log.contextual_param]
params = ["remote_address"]

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add ticket-* icons/improve metadata

Following on from #1292. Related to #1192.

hmm not sure about these icons, Do we have real-life usecases?

hmm not sure about these icons, Do we have real-life usecases?

I came across ticket-x in the wild this weekend on either Booking.com or Airbnb (can’t remember which)… ticket-slash (redeemed) and ticket-check (booked) seem like a no brainer also. The others less so… We could drop ticket-plus/minus if you want @ericfennis ?

Adding and removing tickets is also a common use case.
I vote for keeping them all.


"Contact us" email to feedback author

Description
Attempts to close https://www.wrike.com/open.htm?id=656177298

New account created in SendGrid (please contact me to obtain credentials)
General feedback submissions also generate and send an email that goes to the email address entered in the form
New library sendgrid added to requirements.txt
Message body is the following:

Thank you for your feedback

Your message:

<submitted_form_data>

Important

Email address<EMAIL_ADDRESS>needs to be verified as valid sender for SendGrid. Someone with access to that email inbox should be able to locate an email coming from SendGrid with a link to verify the address. Until this is done, this feature won't work.
A new env variable SENDGRID_API_KEY must be present and set to the correct API key. Please, contact me to obtain it.

Type of change

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

How Has This Been Tested?
Manual testing
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added unit tests that prove my fix is effective or that my feature works

Nice work on using SendGrid, PM me the API key when you get a chance and I can add it to Heroku. Once I have that set then I will merge these changes in. As for the owner of<EMAIL_ADDRESS>do you know who that is? If not, I'll ask Joost

Address has been validated and also set as the email address of sparc-portal account in SendGrid. Also important is that the env variable SES_SENDER needs to be set to<EMAIL_ADDRESS>Let me know if this needs to be changed. Thank you!


how to show overlayScreen?

Hi, Dear @kanzitelli. how can i fix this problem?
PinOverlay: {
  component: PinOverlay,
    options: {
        layout: {
          backgroundColor: '#12469A',
        },
        overlay: {
          interceptTouchOutside: true,
        },
    },
},

This doesnt work. it says "Type '"PinOverlay"' has no properties in common with type 'Layout'."
screens.N.showOverlay('PinOverlay');
My goal here is to show pinScreen to hide content.

Hi @Yasir5247! Please, provide some extra code, it's hard to say with the provided snippet. Also, screens.N is a plain react-native-navigation instance and while using that, you have to check their docs. From what I see, you're just providing a string (screen name) which is not the way of doing that, please consider checking the docs -- https://wix.github.io/react-native-navigation/api/overlay.

export const screens = generateRNNScreens({
    LoginScreen: {
      component: LoginScreen,
      options: {
        bottomTabs: {
          visible: false,
          drawBehind: true,
        },
        topBar: {
          visible: false,
        },
      },
    },
    PinOverlay: {
      component: PinOverlay,
      options: {
        layout: {
          backgroundColor: '#12469A',
        },
        overlay: {
          interceptTouchOutside: true,
        },
      },
    },
    ScanScreen: {
      component: ScanScreen,
      options: {
        topBar: {
          ...withRightButtons('flashOn'),
        },
        hardwareBackButton: {
          bottomTabsOnPress: 'first',
        },
        ...withBottomTab(
          services.t.do('section.navigation.screens.scanScreen'),
          'scan',
        ),
      },
    },
}, [withGestureHandler, withSS, withClient, withAppearance])

suppose i have these screens in my app. in the scan screen i would like show "PinOverlay" as soon as the user
navigates to that screen
//scanScreen.tsx
//this works. i was just wondering if this is the best way u to show the overlay screen.
useEffect(() => {
    screens.N.showOverlay({
      component: {
        name: 'PinOverlay',
      },
    });
  }, []);

@kanzitelli also, can you please tell me how to change the layout from right to left vice versa.
my local language is written from right to left. so if i change to my local language i want to be able
to change the layout "direction" also. is it also possible without app reload?
src/navigation/options.ts
export const navDefaultOptions = (): Options => {
  return {
    layout: {
      orientation: ['portrait'],
      componentBackgroundColor: DesignSystem.themeColor('bgColor'),
      backgroundColor: DesignSystem.themeColor('bgColor'),
      direction: stores.ui.language === 'dh' ? 'rtl' : 'ltr',  <--------------------- this doesnt work
    },
    ...screenDefaultOptions(),
    ...tabsDefaultOptions(),
    ...tabDefaultOptions(),
  };
};

@kanzitelli my local language is from right to left. is it possible to change layout direction without app reload also?
export const navDefaultOptions = (): Options => {
console.log('LOGG', stores.ui.language === 'dh' ? 'rtl' : 'ltr');
return {
layout: {
orientation: ['portrait'],
componentBackgroundColor: DesignSystem.themeColor('bgColor'),
backgroundColor: DesignSystem.themeColor('bgColor'),
direction: stores.ui.language === 'dh' ? 'rtl' : 'ltr',   <-----------------------here
},
...screenDefaultOptions(),
...tabsDefaultOptions(),
...tabDefaultOptions(),
};
};

@kanzitelli just hoping to get a reply. sorry for the trouble.

hey @Yasir5247. I am very busy with the project as am preparing it for the release, and don't have enough time for GitHub life.
About overlay, yeah I would just do it inside useEffect or using this hook -- https://underscopeio.github.io/react-native-navigation-hooks/docs/use-navigation-component-did-appear.
And I have never worked with rtl and ltr and not sure how it works with RNN. But all dynamic changes for RNN are happening here https://github.com/kanzitelli/rnn-starter/blob/master/src/services/navigation/index.ts#L25 when you change UI values.

@kanzitelli thank you. and best of luck


Code cleanup and added metadata for ML Vision

This PR is also missing a rebase. I'm working on rebasing it correctly. The HEAD for this branch is 9c2af92.


Add new area and need-X labels

Issue #, if available:
Description of changes:
Add these new labels:

area/helm: Issues or PRs related to Helm charts
area/testing: Issues or PRs related to testing
area/resource-references: Issues or PRs related to resource references
need-investigation: Indicates an issue needs some investigation.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


add flask oauth library

add the oauth flask library to requirements.txt which otherwise fails.

Thanks @kayiwa!


can you write a demo ? thank you

i used it
it can not work
can you write a demo ? thank you

var a=new TodoRESTClient()
alert(a.getTodos())

I'm wokring on a demo, but it needs more time than I thought. Hope to finish it shortly.

thank you very much,i used http://jsonplaceholder.typicode.com/  testting my project :)

I have created a plunker. See if it helps

@mmrath thank you very much :)

Hi,
First of all, thanks for your module. That's the only REST module i've found for Angular2.
But I'm a bit confused. Your joined plunker is not relevant, because it does not show a proper usage as a node module.
I've included the angular2-rest/angular2-rest.js file in my index.html  and I'm stuck on an "Uncaught ReferenceError: require is not defined" (angular2-rest.js:46)
Can you provide more example please ? At least, a simple usage considering angular2-rest has been installed with npm install ?

After some time of workaround, I finally decided to use the code present in angular2-rest.ts you shared on Plunker, including it as a .ts file in my project (building it with the rest of my application), and it works !
It seems the version available among NPM is not up-to-date.
I'm still interested to deploy your solution as a node module (through NPM), and hope you'll can provide more information about how to use it.


Correct VS workload name and fix VS installer arguments

PR Details
This fixes a problem where Stride installation always fails for the machines which do not have the Development Tools plus .NET Core 2.1 workload installed and then shows an incorrect error message telling the user to install the incorrect workload .NET Core 2.1 Runtime (LTS) (inside .NET core cross-platform development) manually.
Description
This problem was introduced with b73b2a956422171c4173cecc7d5433d2d9c67d36 - the VS installer is invoked incorrectly due to invalid generated arguments. The commit changes the array type containing necessary VS workloads from string to tuple. However, not all places that referred to the tuple were updated to access the appropriate tuple item, which in turn resulted in the tuple being consumed directly by its ToString() method instead of the actual workload ID. This PR changes the code in the remaining places.
Furthermore, VS installer workload Microsoft.NetCore.ComponentGroup.DevelopmentTools.2.1 was incorrectly labeled as .NET Core 2.1 Runtime (LTS) (inside .NET core cross-platform development). This PR changes it to Development Tools plus .NET Core 2.1 (as seen in the official documentation here).
Related Issue
#653, #985
Motivation and Context
The motivation is to fix the installation problem and the incorrect error message, so that the newcomers do not have to waste their time trying to figure out why the installation is constantly failing for their machine.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

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

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

Good catch, thanks for the fixes!

Good catch, thanks for the fixes!

Thanks a lot, this one was bugging new users since a while.

Thanks a lot, this one was bugging new users since a while.

I'm glad I could help. Thanks for merging!

I'm glad I could help. Thanks for merging!


blockchain.info/address does not support bech32, causes error when viewing payment details

The View payment details link in the moderator view of a dispute opens the contract payment address in blockchain.info, which causes an error if the address is bech32:
https://blockchain.info/address/bc1qfl894qr5f4xs0pczd9vlsrrlulczcc7a069xfsueq6patkdl9pgselad40
I was gunna write you a PR that switched the link to the payment transaction instead but I'm guessing since that txid is not included in the contract, it's not sent to the moderator?
I'm hope blockchain.info will support bech32 soon for this case it might be worth switching to an explorer that does support bech32 addresses:
https://en.bitcoin.it/wiki/Bech32_adoption

@tyler-smith Am I recalling correctly that you fixed this one recently?

@rmisio No, I fixed an issue with the blockdozer URLs being wrong. We might have to do what @pinheadmz said and change to a block explorer that support bech32.

This is closed by #1334


Improve error message: "

Looks like the same as https://github.com/microsoft/bond/issues/1017. Closing in favor of that.


[Improvement] unify alsa and pulseaudio run scripts

Is your feature request related to a problem? Please describe.
There are two distinct runners for alsa and pulseaudio, leading to duplicate code
Describe the solution you'd like
Unify the two scripts.
Describe alternatives you've considered
No alternative is available
Additional context
Just respect the d.r.y. principle
We can start from the branch create for issue #102

Progresses!

Most variables are now supported in pulseaudio mode
No more duplicated code
DSD excluded forcefully in PulseAudio mode (would have resulted in some stuttering)

I need to test in alsa mode extensively before releasing.

Available in release 2022-11-05.3


sys/fuchsia: description for fuchsia.io FIDL interfaces

This CL adds syscall description of fuchsia.io interfaces.
(the FIDL description is available here: https://fuchsia.googlesource.com/zircon/+/master/system/fidl/fuchsia-io/io.fidl)
This time, I enumerated request handles as a separate structure, and added support for FIDL enums by converting them into syzkaller flags.
I will upload generated files (and FIDL compiler changes) after getting this description reviewed. Can you please take a look and see how the description looks?

Flags and input handles look good to me.
We could also generate exact types of output handles, but no need to do this in this in change. One step at a time.
Please update docs/fuchsia.md with steps required to regenerate these files.
You also need to run 'make generate', but otherwise looks good.

Ran make generate and updated docs/fuchsia.md. Can you please take a look?


Kinematics: Store virtual marker definitions inside rigid body definition

which would make sense since every markers that relate to a given rigid body would be defined in the same structure. Related to issue #63.
Also, add an option to track_rigid_body that would reconstructs all these markers while tracking the body.

Now done in 9b37c9b.


Allow users to load their mocked endpoint via json spec file

I've just discovered this library recently and it's quite nice, but there is a feature I was wondering if we could get add. The feature would be allow users to load their endpoints using a json file, similarly to what json server does. Thinking about user experience, maybe follow the below example:
var httpServer = new HttpServer();
httpServer.Config.FromJsonFile("path/to/json_spec.json");

Considering a json spec file in the below format:
{
  "Routes": [
      {
        "Method": "GET",
        "Url": "/hello-world",
        "Body": "Hello from route.json!",
        "StatusCode": 200,
        "RequestHeaders": {
          "Authorization": "Bearer ey..."
        },
        "ResponseHeaders": {
          "CorrelationId": "00000000-0000-0000-0000-00000000"
        }
      },
      {
        "Method": "POST",
        "Url": "/hello-world",
        "Body": "{\"Data\": { \"Fullname\": \"JOHN DOE\", \"Age\": 42 }}",
        "StatusCode": 200
      }
    ]
}

We can get the proposed experience with the following changes (using .Net 6 sintax):

Create a model for the JsonSpec file:

public sealed class JsonSpec
{
    public IEnumerable<JsonSpecItem> Routes { get; init; } = default!;
}

Create a model for the route definition

public sealed class JsonSpecItem
{
    public string Method { get; init; } = default!;
    public string Url { get; init; } = default!;
    public string Body { get; init; } = default!;
    public int StatusCode { get; init; } = default!;
    public Dictionary<string, string> RequestHeaders { get; init; } = default!;
    public Dictionary<string, string> ResponseHeaders { get; init; } = default!;
}

Implement an extension method to handle the route registration

public static void FromJsonFile(this IRequestBuilder builder, string jsonSpecPath)
{
        var specAsJson = File.ReadAllText(jsonSpecPath);

        var spec = JsonSerializer.Deserialize<JsonSpec>(specAsJson);

        foreach (var route in spec.Routes)
        {
            builder.Request(route.Method, route.Url, route.RequestHeaders)
                .Send(route.Body, route.StatusCode, route.ResponseHeaders);
        }
}

Calling the endpoint, for example, the "hello-world" from the json example with the following curl command:
curl -v -H "Authorization: Bearer ey..." http://localhost:5000/hello-world

would produce the following output:
*   Trying <IP_ADDRESS>:5000...
* Connected to localhost (<IP_ADDRESS>) port 5000 (#0)
> GET /hello-world HTTP/1.1
> Host: localhost:5000
> User-Agent: curl/7.83.1
> Accept: */*
> Authorization: Bearer ey...
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Server: Kestrel
< Transfer-Encoding: chunked
< CorrelationId: 00000000-0000-0000-0000-00000000
<
Hello from test route.json!

I would be happy to implement the pointed changes (with maybe some changes) in the project 😊

Sorry for the very late response. Was super busy recently. Looks good to me but a bit difficult to find a time to implement. Pull requests are welcome.


Update homework.md

adding a team to locked queue
Describe your changes
Checklist

[ ] I have read the [CONTRIBUTING]
[x] I have performed a self-review of my own code
[x] I have run the code locally and it works as expected
[x] I have commented my code, particularly in hard-to-understand areas

Screenshots (if appropriate)

locked queue


Extremly slow uploads to hubiC

Hi,
I face extremly slow transfer rate to upload files to hubiC.
From the hubiC web ui, I have usually around 16Mbps
BUT from Swift, the maximum I face is 1.6Mbps.
10 times less !!
Any clue how to solves this ??

Upload speed of segmented files will likely be improved after the release of the next version of JOSS (see https://github.com/javaswift/joss/pull/70).


refactor(client): use multicast observable for invitations

TODO:

fire cancelled and timeout properly on the backend

Current dependencies on/for this PR:

main

PR #2846   👈

This comment was auto-generated by Graphite.

@wittjosiah queued this pull request to merge with Graphite.

@wittjosiah merged this pull request with Graphite.


New version: Bloqade v0.1.22

Registering package: Bloqade
Repository: https://github.com/QuEraComputing/Bloqade.jl
Created by: @weinbe58
Version: v0.1.22
Commit: e77f99305f999550a5eca9d1318a57cdfa1059dd
Reviewed by: @weinbe58
Reference: https://github.com/QuEraComputing/Bloqade.jl/commit/e77f99305f999550a5eca9d1318a57cdfa1059dd#commitcomment-107516711
Description: Package for the quantum computation and quantum simulation based on the neutral-atom architecture.
Release notes:

Release Note
changes to BloqadeSchema

Renaming QuEraSchema objects to remove redundances: https://github.com/QuEraComputing/Bloqade.jl/pull/540
fixing bug caused by that change: https://github.com/QuEraComputing/Bloqade.jl/pull/541

@weinbe58
please see the message above.  Once you fix the automerge issues you can register again the new revision without changing the version number and this pull request will be automatically updated.
[noblock]


Create routing policy for empty deployment spec MERGEOK

Manual deployments can have a empty deployment spec, without any instances. This
change ensures that we create a routing policy and update DNS for such
deployments.

Will this help when application has submitted a spec with instance "a", then deploys to dev for the first time under a new instance "b"?

I took the liberty of adding a test with this scenario which fails.

Instances.......................!
@jonmv PTAL.

Java build passed.


Adding clang/15.0.0 to list of include dirs

This is needed for ROCm 5.2+

retest Ubuntu-sanity please

retest Ubuntu-GPU-multi please


Log command line parameter settings for service invocations

Description
I want to be able to see in the service logs what CLI parameters the various services were invoked with, a la various Kubernetes components.

This was only relevant for the golang code, the Rust binaries do this automatically.


LAB_AK_01_Lab1_Ex1_Explore_M365_Defender

Contact Details
<EMAIL_ADDRESS>What should we change?
PROPOSAL
At the Task3-Step5, after selecting "Turn new features on", the message "Hang on! We're preparing new spaces for your data and connecting them" may not appear. So I recommend to add following step.

If  the message "Hang on! We're preparing new spaces for your data and connecting them" does not appear, from the navigation menu, under Assets area, select Devices.

Lab
Lab 01 Exercise 01 Explore Microsoft 365 Defender
Relevant screenshots
paste here 😉

Yes. I saw this in the labs.

It shows up, but after an hour or longer.


Improved solution and instructions of operator exercise

Insist on non member operators and use friend (only) when needed
Closes #264

Btw, in reaction your our discussion here I extended the slides on hidden friends and gave guidance for operator overloading #288.


Use Go's built-in DNS resolver to avoid DNS issues when building Acorns

Checklist

[ ] The title of this PR would make a good line in Acorn's Release Note's Changelog
[ ] The title of this PR ends with a link to the main issue being address in parentheses, like: This is a title (#1216). Here's an example
[ ] All relevant issues are referenced in the PR description. NOTE: don't use GitHub keywords that auto-close issues
[ ] Commits follow contributing guidance
[ ] Automated tests added to cover the changes. If tests couldn't be added, an explanation is provided in the Verification and Testing section
[ ] Changes to user-facing functionality, API, CLI, and upgrade impacts are clearly called out in PR description
[ ] PR has at least two approvals before merging (or a reasonable exception, like it's just a docs change)

I don't think this makes any difference. AFAIK we are already using the pure go resolver because the binary is not linked to glibc at all (at least on linux).  I could be wrong, but if you get
$ ldd bin/acorn
	not a dynamic executable

I don't see how it could not be using pure go.

oh wow. this 100% fixes it for me locally

i didnt think you had to mess with -tags netgo anymore

For me, on macOS, I never hit the issue with -tags netgo. I sometimes hit it without that. I agree that it's strange...


Initial documentation

Closes #34

Thoughts on amendments to the README @adlerjohn ? The escrow will need to be merged into master first.


fix: resolve issue where resolutionrequest defaulted to v1alpha1 vs v1beta1

/kind cleanup
Changes
It was identified that the default APIVersion was v1alpha1 in pkg/apis/resolution/v1beta1/resolution_request_defaults.go when it should be v1beta1. This PR fixes this incorrect default API version
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] [N/A] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] [N/A] Has Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Resolved issue where resolutionrequest defaulted to v1alpha1 when it should be v1beta1

/kind bug

recorded my questions in https://github.com/tektoncd/pipeline/issues/7447 to unblock this fix
thanks @aaron-prindle
/lgtm


fix bug #188 - [closed]

In GitLab by @luotian-github on Sep 14, 2017, 07:00
Merges devs -> dev
fix bug #188

In GitLab by @wknet123 on Nov 13, 2019, 09:58
closed


look at annotating the color key

Reported by Ansley B Manke on 29 Nov 2004 17:20 UTC
Some possibilities and issues
-- Maybe an option on the /KEY qualifier?  /KEY=labelled
-- Or, an addition to SHAKEY?
-- If the key is labelled, do we want to remove the units that are automatically
part of the title?
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1104

Comment by ansley.b.manke on 4 Mar 2010 00:29 UTC
Done.  This from the docs:
The last argument to SHAKEY (new with Ferret v6.2), turns on annotation of the shade key. This puts the minimum and maximum value of the data field, at each end of the color key.  The Ferret alias KEYMARK can be used to turn this feature on and off.  The KEYMARK setting persists in the Ferret session until it is changed with another KEYMARK command.
yes? KEYMARK,1  ! turns on key annotation
yes? KEYMARK,0  ! turns it off

Comment by ansley.b.manke on 31 Mar 2011 19:50 UTC
Ticket is done, but was never closed.


Canada: forgot password flow error

When resetting user password using one-time link for returns the following error:

The e-mail address<EMAIL_ADDRESS>is already taken.

Tried locally both with local and production TIG API: everything works.
Now this issue only reproducible on intl QA.


Is there a pre-trained model?

Hello, have you finally trained to the fid of the final result of the paper? Is there a pre-trained model?

Hi @wflrz123,
Thank you so much interest in the repo! While training, I saved the FID score results here. I will provide the pre-trained model later.

Thank you very much!
------------------ 原始邮件 ------------------
发件人:                                                                                "asarigun/TransGAN"                                                                                @.>;
发送时间: 2021年7月23日(星期五) 下午5:27
@.>;
<EMAIL_ADDRESS>主题: Re: [asarigun/TransGAN] Is there a pre-trained model? (#4)
Hi @wflrz123,
Thank you so much interest in the repo! While training, I saved the FID score results here. I will provide the pre-trained model later.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Auth packets

Does anyone have the auth packets for protocol version 5?

@AdaptCharm Check out https://github.com/issy123/agario-protocol/blob/master/protocol-v2.1.5.md
I'm pretty sure the auth packets would be the same, because miniclip whet crazy about changes in protocol 6.

FE 05 00 00 00
FF 01 00 00 00


Rewrite to use xmltree/xmlparser

In the future, will add opening, modifying, and retrieving data from epubs.

Will test image support before merging.


Bring in ColorUtils, take out TextraTypist dep.

TextraTypist is somewhat of a large dependency if people don't want to use it, and only one class was used from it anyway (and it was only used by PopColorPicker). ColorUtils is from TextraTypist and was copied almost in full; the color name description code is too ingrained to copy easily, so I just didn't bring it over. It's up to you if ColorUtils belongs in Stripe so other code of yours can use it, or just in the color picker library since that deals with colors primarily.

Thank you so much! I appreciate you. Merry Xmas.


Allow Anti-Aliasing to be disabled

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Drawing 1-pixel thick lines results in blurry, fuzzy outputs while using DrawingContext.DrawLine().

I believe this is due to anti-aliasing.
Describe the solution you'd like
A clear and concise description of what you want to happen.
A way to disable anti-aliasing for the drawing context (which I believe is included in native WPF).
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Right now, I can add 0.5 to all my x,y values to prevent this behavior.

But this is less than ideal as I can't add a centre line through the square without it being blurred.
Additional context
Add any other context or screenshots about the feature request here.
Blurry lines:
public void Draw(double x, double y, DrawingContext drawingContext)
{
    Pen _pen = new(new SolidColorBrush(Color.FromRgb(202, 205, 169)));

    List<Point> Points = new List<Point>()
    {
        new Point(x - 4, y - 4),
        new Point(x + 4, y - 4),
        new Point(x + 4, y + 4),
        new Point(x - 4, y + 4),
        new Point(x - 4, y - 4),
    };

    for (int i = 0; i < Points.Count - 1; i++)
    {
        drawingContext.DrawLine(_pen, Points[i], Points[i + 1]);
    }
}

Non-blurry lines:
public void Draw(double x, double y, DrawingContext drawingContext)
{
    Pen _pen = new(new SolidColorBrush(Color.FromRgb(202, 205, 169)));

    List<Point> Points = new List<Point>()
    {
        new Point(x - 4.5, y - 4.5),
        new Point(x + 4.5, y - 4.5),
        new Point(x + 4.5, y + 4.5),
        new Point(x - 4.5, y + 4.5),
        new Point(x - 4.5, y - 4.5),
    };

    for (int i = 0; i < Points.Count - 1; i++)
    {
        drawingContext.DrawLine(_pen, Points[i], Points[i + 1]);
    }
}

I guess this one is related to a problem I came across... I initialized a 200x200 pixel WriteableBitmap with 4 colors, I did not want a blurry edge between them. I assume anti-aliasing is doing the same here:

(this is a crop of the transition region)
But in this case it would be a property on Image instead of DrawingContext.

@kolbyd @HoneyTauOverTwo you can set RenderOptions.BitmapInterpolationMode property:
<Image Source="{Binding MyBitmap}" RenderOptions.BitmapInterpolationMode="LowQuality" />

It has quite confusing naming for enum values, because we copied Skia naming.
https://github.com/AvaloniaUI/Avalonia/blob/d1afd3bb591c266015dcdecc9611148ca08c2ee4/src/Skia/Avalonia.Skia/SkiaSharpExtensions.cs#L16
https://github.com/AvaloniaUI/Avalonia/blob/4c77f1eb3d11308412b915a82460e4408ce360a0/src/Windows/Avalonia.Direct2D1/Media/Imaging/WicBitmapImpl.cs#L28

@maxkatz6 I was digging into Avalonia's code and found this BitmapInterpolationMode enum. But did not find where to set it yet.
I changed my code to this (I am not using data bindings):
<Image x:Name="DisplayImage" RenderOptions.BitmapInterpolationMode="LowQuality"/>

Then in the code behind I have:
WriteableBitmap bmp = new WriteableBitmap(new PixelSize(200, 200), new Vector(96, 96));
// bmp initialization omitted
DisplayImage.Source = bmp;

but it still produces the same result.
And sorry for cluttering this here, probably not the place to discuss.

Related to #199

Is there a way to do this from C#? Couldn't find the property

@Hanprogramer If I understood correctly there is currently no easy easy to do. As you can see the issue is also still open. There is a PR to implement this, see #9584 . You may want to try this PR out and see if your issue is resolved.


Do OCR-det and OCR-rec have plans to support onnx ？

The issue has been passed on to the model owner and will be addressed in upcoming scheduling, please stay tuned


version bump to v1.4.4

Let's please ship a new version to fix an issue (#324) introduced in v1.4.3.

Please note that the appveyor failures can be ignored for now, I've fixed them in #320 which isn't quite ready for review yet.


feat: add support for tgz archive files

This PR is to support tgz files as asked in #132 and #131

I think PR is not ready yet ...
as while I am doing a couple of manual tests, I faced this error :

However the file is still there and I can cat it normally
Any ideas?

I think PR is not ready yet ...
as while I am doing a couple of manual tests, I faced this error :

However, the file is still there and I can cat it normally.. this happens on Linux and not windows, on windows it works just fine.
Any ideas?

I figured out the issue when I checked the WS response (server log was silient though)...
the issue was related to logs path config... I was customized it to look into some specified folder in '/mnt/logs'
So to fix the issue, I added
  path: "/tmp/**/*"
  }```
in my config file as well.

So it has nothing to do with PR, unless the user should be aware of the paths config in such case!

Thanks! Good idea.
But unpacking all log files automatically is not a safe operation, logs may take a lot of disk space. Let's add log-viewer.unpack-archive configuration property and disable it by default.
The ideal solution is ask the user if he want to unpack log because it is not expected. Adding asking of the user is not a trivial fix, I'll implement it later.

realFile

I searched the entire code base but didn't find a track for realFile.. can you elaborate more on this?

I've updated the PR by suggested comments:

introduce log-viewer.unpack-archive
check the gzip file extension

I am working on a quick fix

realFile

I searched the entire code base but didn't find a track for realFile.. can you elaborate more on this?

I mean you have to add Log.realFile to com.logviewer.data2.Log class. Now com.logviewer.data2.Log has file field, your code replace the value to the temporary file. I propose to keep the original file name in Log.file , but put unpacked file to Log.realFile.
I can add it after merge.

BTW, I did other changes as well to suit needs
https://github.com/sevdokimov/log-viewer/compare/master...mhewedy:log-viewer:develop
Let me know if any interesting feature that I can contribute back

I added the following changes after merge:
1.) renamed the configuration property to log-viewer.unpack-gz-archives
2.) Fixed checking of log visibility: no error if .gz file is visible, but the unpacked '/tmp/log-viewer*.tml' is not.
3.) Added the details error message when the user is trying to open .gz file with disabled log-viewer.unpack-gz-archives
4.) added tests

Good, Thanks 🌹


Added fr_FR base

not 100% perfect but a good start

 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.


test(multiple): refactor unit tests of several components

Opening this PR as it is required to:

Refactor variables in various component unit tests to have constant type instead of let as this variables are not getting re-initialize during the test life-time.
Added type annotations to variables instead of any.
Renamed variables.
All tests within a spec should be consistent with all other tests in repo.

Fixed it.


error while building debian package

Using debian stretch and docker v. 17.05.0-ce
While running make, there were errors in the final step:
Successfully built 2415b2a17a8e
Successfully tagged miniflux/debbuild:latest
docker: Error response from daemon: invalid volume spec ":/pkg": invalid volume specification: ':/pkg'.
See 'docker run --help'.
Makefile:2: recipe for target 'all' failed
make: *** [all] Error 125

I guess this is the $PWD variable that don't get replaced on this line: docker run --rm -v ${PWD}:/pkg miniflux/debbuild https://github.com/miniflux/package-deb/blob/master/Makefile#L3
I do not have a Debian right now to test, but try to find the equivalent. Or, run the same commands as the ones in the Makefile but directly from your shell.


chore: increment python library version

Description
The description of the main changes of your pull request
Related Issue(s)

Documentation

Validated the aarch64 release works in https://github.com/delta-io/delta-rs/actions/runs/6307097128/job/17123215869?pr=1664


Ledger Integration

Please, where I could find the content about "Ledger Integration"? It looks like it's missing.
Is there some documentation about inventory posting configuration for D365FO like as the "Appendix A: Postings based on recommended settings" in the document "Cost Management in Microsoft Dynamics AX 2009"? (https://www.microsoft.com/en-us/download/confirmation.aspx?id=1041)
Tks in advance!

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

ID: f223e1d2-3d7a-592a-a404-6cfe30fecbbf
Version Independent ID: 63478ba4-0e82-63ce-3d80-a7ec5d39e84b
Content: Cost management home page - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/supply-chain/cost-management/cost-management-home-page.md
Service: dynamics-ax-applications
GitHub Login: @AndersGirke
Microsoft Alias: aevengir

@Chrisslima thank you for posting this.
@AndersGirke can you please review the comment and provide assistance as needed?

Bug 318221 has been logged to track this issue.


how to pass Jenkins parameters as input to replace global variables

Hi Hitesh ,
I want end user to enter  values for Global variables like bigip_pool_member and port in Jenkins. These values need to be override  the globalvars mentioned in f5-newman-build files.

Repository Archived


Allow any log level to create breadcrumbs

Creating a breadcrumb with an invalid log level would cause an exception, resulting in the log being missing in the breadcrumbs. This pull request adds a log level adapter method, which translates common log levels to suitable breadcrumb levels, defaulting to the info level.
Fixes #297

FYI: I've rebased this PR because some other conflicting PR's got merged.

Thanks for taking the time to create the PR! Much appreciated! 👍


Use feature name as id for flag analytics

At the moment, a lot of our clients are sending flag analytics data in the format {"<feature id>": <evaluation count}. The javascript client is using the feature name and the dashboard expects the feature name in the response data.
SDKs to correct:

[x] Java
[ ] Python
[ ] Elixir
[ ] Go
[ ] Node
[ ] Ruby
[ ] Rust
[ ] PHP
[ ] .NET
...More TBC

Python PR here: https://github.com/Flagsmith/flagsmith-python-client/pull/29

Node JS PR here: https://github.com/Flagsmith/flagsmith-nodejs-client/pull/48

Ruby PR here: https://github.com/Flagsmith/flagsmith-ruby-client/pull/17

PHP PR here: https://github.com/Flagsmith/flagsmith-php-client/pull/28


datadir should be /var/lib/NAME not /home/NAME

As the [http://www.pathname.com/fhs/](FHS 2.3) states:

An application (or a group of inter-related applications) must use a subdirectory of /var/lib for its data.`

/home is definitely not right for this. One might argue that /srv/NAME fits the bill, but /var/lib is better IMO.

:+1:, especially for following the FHS!


Will WPA2-Enterprise authentication be supported?

It depends on esp8266 core sdk.
Please search or open issue there.
https://github.com/esp8266/Arduino/issues


Integration of IoBroker in NethServer

I came across the Open Source Project nethserver https://www.nethserver.org/ while searching the net.
I also tested the whole thing. Is really easy to use and easy to install additional systems / packages from there. It might be interesting to offer iobroker as a system to be installed as well. The whole thing is when I have seen the law a thing from Italy and possibly a possibility to bring iobroker to Italy or to make it better known there. You can also find instructions on how to do this here. https://docs.nethserver.org/projects/nethserver-devel/en/latest/

Please create here https://github.com/ioBroker/ioBroker/issues :-)


Implement reactive support for server side code generation

https://github.com/quarkiverse/quarkus-openapi-generator/pull/482 adds support for generating server stubs from OpenAPI definitions. This only supports for now classic. The underlying library, Apicurio Codegen, also supports reactive generation. Use the capabilities of the library to implement reactive support in the extension.

Is there any indication of when this could be available? I'm currently generating code, but changing it manually every time to make it reactive. Would be great if it's going to work automatically :-)


bambooagent group is not created

It looks like the role doesn't try to create the bambooagent group before creating the user, resulting in the following error:
TASK [mimacom.bamboo-agent : Add service user] *************************************************************************************************************************************** fatal: [xxx]: FAILED! => {"changed": false, "msg": "Group bambooagent does not exist"}
Happy to create a PR once this is confirmed as an issue.

Actually the task creates a group by default, see:
https://docs.ansible.com/ansible/latest/modules/user_module.html
What ansible version and Linux distribution do you use?


Cannot install on Python version 3.12.0

Hello,
Great work for this package, do you plan to make it accessible for those using Python 3.12 ?
KRegards

are you sure its not working on 3.12?, I got the error message below today, it seems like it should work on 3.12.
RuntimeError: Cannot install on Python version 3.13.0; only versions >=3.9,<3.13 are supported.


[Bug]: Script fails if no enviroment variable is set

What happened?
Hi,
Running the script straight forward does not work.
./pipeline.sh: line 118: exclude_list[@]: unbound variable
To make it work I need to set any value in the env. variable.
Expected behavior
Script should work without setting the Environment Variable.
How do we reproduct the bug?
Using bash in OS X.
Run the script in a folder with a json file.
Relevant log output
No response
Screeenshots
No response
Additional information
No response
Code of Conduct

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

I cannot replicate this issue - what version of bash are you running?
It was written and tested on bash 5 (of a Mac) - and the GitHub actions all use the latest ubuntu image.
I vaguely remember issues with the pre-installed version of bash I had on my development Mac and switched to the GNU bash from homebrew.

Ok, I just located the original version of bash on my Mac (/bin/bash which is version 3.2.57) and now I can replicate the issue.
However, I have also in the same place found my notes on the issue, so this is a bash issue with bash pre 4.2.
Arrays that are defined within a function are not available outside of the function unless you declare with a -g (bash 4.2 ) or by default in bash 5+
I will look into other options, maybe defining the arrays globally at the top and then using them instead of defining them within the function might work.

Ok, a little more digging and the issue seems to be a little different to what I had in my notes.
bash 3 needs
exclude_array=( )
instead of
declare -a exclude_array=()
I can add in some code which does a work round based on bash version I think, give me 24 hours to to some testing and I will get a new version out.
This issue actually will impact all of the repos in this org as they are based on the same basic pipeline

Temporary working around:
EXCLUDE_FILES="nothing" ./pipeline.sh
This will set the exclude files array and allow the script to run cleanly.

Possible solution - going to run a battery of tests
if [[ -n "${EXCLUDE_FILES-}" ]]; then
    IFS=',' read -r -a exclude_list <<< "${EXCLUDE_FILES}"
    echo " Excluded: ${EXCLUDE_FILES}"
    parameters=true
else
    if ((BASH_VERSINFO[0] < 4)); then
        exclude_list=("")
    else
        # shellcheck disable=SC2034
        declare -a exclude_list=()
    fi
fi

basically bash 3 does not see an empty array as a bound variable so it is failing, this adds an array with an empty string, it also doesn't like the declare -a hence that being missing to, brief testing seems to suggest this will solve the issue.

Ok, I just located the original version of bash on my Mac (/bin/bash which is version 3.2.57) and now I can replicate the issue.
However, I have also in the same place found my notes on the issue, so this is a bash issue with bash pre 4.2.
Arrays that are defined within a function are not available outside of the function unless you declare with a -g (bash 4.2 ) or by default in bash 5+
I will look into other options, maybe defining the arrays globally at the top and then using them instead of defining them within the function might work.

I can confirm, this is the version I have by default (3.2). I also installed bash 5, and in this version it worked flawlessly.

New version has been pushed to master which should now support base 3.

Well done. Thanks.