Build pod fails during build phase

I have a ClusterStack that refers to Paketo Jammy Base Stack. Intermittently, the build init container is terminated with exit code 51. I can't figure out the root cause as the build container logs doesn't have any error logs, it just says signal: killed. How can I debug the issue? Any help in this regard is highly appreciated.
Logs from the build container - The build ran only for 12 milliseconds

Describe pod output - Terminated with Error 51

Kpack: <IP_ADDRESS>
Paketo Jammy Build pod: docker.io/paketobuildpacks/build-jammy-base:0.1.86

The CNB spec says that /cnb/lifecycle/builder exits with code 51 if there's a Buildpack build error. What buildpacks are you using (i.e. what does the DETECT phase print out)?
The fact that it's intermittent makes me suspect that there's some network issues that's causing a buildpack to error out. It would be nice if we had a way to turn on debug logs but unfortunately that's not supported right now.


Just a regular click on the edge

I see all the gestures you can make on a mobile device, but I'm just looking to click on the edge from the web app. Is there a way to do that?

I haven't tested how it works on web, but you may try to use onEdgeTapDown. Edges gestures works on top of https://github.com/nateshmbhat/touchable lib.

Yeah, I tried all of them. TapDown is basically a drag that starts above the line and you have to intersect it in order to trigger.


AuthModule : adding headers before authentication

As a solution to https://github.com/datawire/ambassador/issues/216, we decided to add a routing service between our many auth services. The idea was to route the request depending on a http header which ambassador would add depending on the AuthService configuration.
Sadly, add_request_headers or envoy_override are not available in the AuthService schema. This can be done at the Mapping level since it's passed the authentication.
Describe the solution you'd like
---
apiVersion: ambassador/v0
kind:  AuthService
name:  auth-router
auth_service: some-auth-service
add_request_headers:
- name: x-some-auth-header
  value: some-value

Describe alternatives you've considered
The envoy_override could be a more advanced option.
---
apiVersion: ambassador/v0
kind:  AuthService
name:  auth-router
auth_service: some-auth-service
envoy_override:
  request_headers_to_add:
  - append: true
    header:
      key: x-some-auth-header
      value: some-value

Third option
If a Mapping define the add_request_headers options, maybe add those headers to the authentication automatically.
Work around
In the end we used the host/:authority to route our traffic, but the solution would have been much more elegant if the routing could be determined through the service's annotation instead of hardcoding it in the authentication routing proxy.

Hi
I have also found this to be a real big issue when using a service mesh such as LinkerD where it requires a host header override to route traffic to the correct services.
Without the ability to elegantly set the l5d-dst-override header with the AuthenticationService capability that Ambassador provides we decide to go with an alternative API gateway. Ambassador would have been a perfect fit if it, in fact, supported the ability to override/provide headers on AuthenticationService as it does with Mapping

Thanks for the feedback. Note that this is pretty straightforward to do using the AuthService API itself (and in fact, we use this as part of the Filter mechanism in Ambassador Pro.)

Hi, where would one find the documentation on this? I have looked far and wide for a solution to this problem.

Why not just deploy the AuthService as a sidecar?


ETCD-416: bump etcd golang 1.16->1.19

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

/lgtm
/approve

overriding the remainder as they are unrelated to that change, they've passed before with the release config in the payload tests.
/override ci/prow/e2e-aws
/override ci/prow/e2e-aws-upgrade

/override ci/prow/e2e-aws-serial

/override ci/prow/e2e-aws-etcd-recovery

/test all

/retest-required

/lgtm
/approve

/override ci/prow/e2e-aws-serial
/override ci/prow/e2e-aws-upgrade

/jira refresh

/hold
waiting for the cachito guys to make the production builds work with local dependencies

already in #201


TractorFactory increasing code coverage

Description
Task: Tractor factory increasing test coverage
Tractor factory increasing test coverage.
Dependencies

#29

**Completion Deadline:**23/09
Documentation
-Main description of feature

JavaDoc
...](https://github.com/UQcsse3200/2023-studio-1/wiki/Tractor)

Member

Shanon Lakshan Chandrasekara @86LAK

will be done  after tractor fix in sprint 4


Convert JVM-based indexer to GraalVM native image

At present, we spend 30-45 sec just to start indexer (we need to download libs from maven, and so on). We can try to reduce this time to convert indexer to native image.
definition of done: we can manually build native image with 1 command, and all tests are passed.
This task depends on https://github.com/Hixon10/openjdk-mailing-lists-search/issues/7

This task is not so important anymore. We split indexer build and index process to 2 different jobs. So, at present we don't build indexer, when we just want to reindex content. Moreover, we spend majority of time, while git clone repo from github to ci agent, because we have almost 500 MB of artifacts (database).


Circular symlink in AppData\Local folder (windows-2019)

Description
I'm encountering an issue trying to onboard CI for Windows for our CMake Tools extension for VS Code. I've been trying to work around it, but haven't been successful yet, so I decided to open an issue for it.  There is a file watcher that ends up trying to index the %LocalAppData% folder during the test (I haven't figured out why yet since it doesn't appear to be our extension's file watchers).  This should normally be benign and not a problem, but it turns out that there is a circular symlink inside that folder in your windows-2019 image and it causes the test to fail.
Area for Triage:
Containers
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[x] Windows Server 2019

Image version
Environment: windows-2019
Version: 20210316.1
Included Software: https://github.com/actions/virtual-environments/blob/win19/20210316.1/images/win/Windows2019-Readme.md
Image Release: https://github.com/actions/virtual-environments/releases/tag/win19%2F20210316.1
Expected behavior
I expect there to be no circular symlinks in the %LocalAppData% folder
Actual behavior
The node module 'readdirp' complains that there is a circular symlink.
See the failure in this build
Repro steps
I'm not sure if you will be able to run the CI (Windows) action against the bobbrow/windowsCI branch, but that's the only way I can repro it now.
If there's a way you can just check the image and verify the bad symlink is there, it might be faster.

Hi @bobbrow!
We will take a look at it.

Actually, I may have successfully worked around it with https://github.com/microsoft/vscode-cmake-tools/commit/79c9ac8a2894e6fa0a704d3b362a2d0f38c4e9f2.  I'm seeing the tests proceed past this issue now.

Hi @bobbrow, as I see, this symlink in AppData/Local is default part on Windows for backwards compatibility: https://answers.microsoft.com/en-us/windows/forum/windows_7-files/why-are-there-multiple-layers-of-application-data/4ec2c92c-a691-432b-b86a-c76d2b9a3faa
I believe, you can just remove this symlink in runtime.


Adding CSS

It would be Better if you add a CSS properties to the button as well as the whole document body , As it would be nice :)
Also A Heading Like "Valentine " would be much better

that's a nice idea !! will try to implement that or in case if you want to contribute feel free to create a PR

Sure will create a PR and help you out 🙂


[Bug] Candlestick stick is not coming in middle when we have more no of data

Version
5.4.3
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=heatmap-cartesian
Steps to Reproduce

Current Behavior
Stick is not alligning at center.
Expected Behavior
Candlestick stick should be coming in the middle even for the huge
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

duplicate of #18503


Discussion: introduce new events API

Problem
Right now there are two event APIs:

The dataCallback and shouldSendCallback events, which can be specified using Raven.config or using their own dedicated set methods:

Raven.setDataCallback
Raven.setShouldSendCallback

An undocumented API using document-level DOM events:

document.addEventListener('ravenHandle', function () { ... });
document.addEventListener('ravenSuccess', function () { ... });
document.addEventListener('ravenFailure', function () { ... });

Difficulties with this system:

The two APIs cover almost completely different events, e.g. ravenHandle fires only on handled exceptions (read: not messages), whereas dataCallback fires for every sent event
Can only specify a single callback for dataCallback and shouldSendCallback [#428, #477]
DOM-level events don't work in certain environments, e.g. React Native which doesn't have a DOM/document
there is very little different between dataCallback and shouldSendCallback – they can both mutate the data object; the only difference is that only shouldSendCallback can "cancel" the sending of an event – can these be combined?
introducing new events like dataCallback and shouldSendCallback means adding more set API methods

Proposed solution
A new events API that looks something like this (aka very similar to Backbone.Events):
var handler = function () { /* do something */ };

Raven.on('send', handler); // set a handler
Raven.on('send', function () { /* do something else */ }); // set a different handler

Raven.off('send', handler); // remove a handler
Raven.off('send'); // remove all handlers

This API will also support chaining (like most other Raven methods), so you can do:
Raven
  .config('<your dsn>')
  .on('send', function () { /* do something */ })
  .install();

Proposed list of supported events:

send – fires when Raven is about to send data

first argument to handler is Raven's data object, which supports mutation
returning false cancels send (like shouldSendCallback)

success – fires when Raven transport is successful
failure – fires when Raven transport fails

Proposed rollout
We introduce this API in Raven.js 3.0, but continue supporting the old API methods – albeit with a deprecation notice that outputs using console.warn (similar to how React deprecates methods between minor revs).
In 4.0 (whenever that comes out), we remove it entirely.
Possible discussion points

should events be "read only", or should they have the ability to alter the control flow of Raven.js (see the proposed send event)
should we just stay the course with what we have, and deprecate the DOM events in favor of events similar to dataCallback, e.g. setSuccessCallback, setFailureCallback, setHandleCallback?

Anyways, this is what I'm thinking. Thoughts? Feedback?
Note there's already a partial PR up that implements most of this (#521).

What impact do you think this will have on the callbacks passed in Raven.config ?

See: "Proposed rollout" above
We'll keep those callbacks active for some time (with a deprecation warning). But we will remove them from the docs so that people don't use them going forward.

Will this also include a way to set and handle timeouts (as in #540)?

Discussion moved to #803.


Bump aws provider to v4

Depends on PR https://github.com/cloudposse/terraform-aws-s3-bucket/pull/131
what

Bump s3 bucket module to use v4 provider
Bump aws provider to v4

why

Team decision has been to not pin modules to major versions so better to role forward
Many errors with v4 in cloudposse/s3-bucket module

│ Error: Unsupported attribute
│
│   on .terraform-mdev/modules/mwaa_environment.mwaa_s3_bucket/main.tf line 166, in resource "aws_s3_bucket" "default":
│  166:         for_each = local.s3_replication_rules == null ? [] : local.s3_replication_rules
│
│ This object does not have an attribute named "s3_replication_rules".

references

https://github.com/hashicorp/terraform-provider-aws/releases/tag/v4.0.0

/test all

/test all

This has been moved to v4 already


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83561

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/85062

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/92878

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/92879

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/93259

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100527

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/102362

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103698

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/111196

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/112078

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/121524


Add k8s resource request by default

This PR adds an opinionated default resource requests to helm chart.
As explained in #8944, the request in CPU is necessary. Otherwise the JVM by default will optimize its parallelism to only use a single CPU, which impacts in the GC but also in the different executors Pinot uses. The memory request is set to 1.25Gi to leave some extra space from the 1G value used by default in -xmx.
Anyway, it should be recommended to customize these values on each environment, either by increasing the CPU resources or using -XX=ActiveProcessorCount

There are other places as well where appropriate defaults are not defined such as Zookeeper. For this PR, we can limit the scope to CPU and memory only. Overall, LGTM!

I've decided to remove the cpu requests and specify instead -XX=ActiveProcessorCount=2 in the default jvmOpts.

maybe we can reduce this to 1? some older laptops don't have 8 cores.

I think this approach is better because the request approach consumed at least 8 CPUs. This means that, by default, the chart would not start in clusters with less than 8 CPUs in total, which can be very common in local k8s installations (like minikube or microk8s) and also in small aws machines.

I am not familiar with now minikube/microk8s maps CPUs to host, just to clarify are these 8 cores virtual or the ones that are physical (include hyperthreading)?

Nodes with few cpus will have installations that use quite more threads than cpus. This could introduce performance issues due to high context switching when several queries are being executed at the same time, but the system should run well enough to do some tests, which is the common use case of this kind of installations.

I am less worried about running multiple queries at the same time, given that this setting is more for quickstart setups.

maybe we can reduce this to 1? some older laptops don't have 8 cores.

This is not going to be enforced by k8s. Instead, the OS is the one that is going to time slice between them. In most cases this isn't going to be a problem because, for example, when a query is being executed either the threads from the server or the ones from the broker will be active, but not both at the same time. If we decide to use only 1 thread then we would need to change the GC (doesn't make sense to use G1 with 1 thread).

I am not familiar with now minikube/microk8s maps CPUs to host, just to clarify are these 8 cores virtual or the ones that are physical (include hyperthreading)?

TBH I'm also not sure. But remember that by using -XX:ActiveProcessorCount we do not depend on k8s now.


Question about the VFHQ dataset

Fantastic work! I have a question about the dataset. Which version of VFHQ dataset did you use, the raw training set w/o resize or the resized 512*512 version? Thanks.

Thank you for your interest! We use the version  w/o resize.

Hello, is there a fast way to download VFHQ dataset? 百度网盘 is too slow~~~


Migrated deprecated React.PropTypes

Thanks @Andarist! This is the only dependency I have that still hasn't been updated. I hope this will get merged soon!


About dataset

hi, i'm confused by how you create the train20.txt and test20.txt, does it just randomly select from the all dataset
thanks

Hello, we randomly select training nodes (for ACM, UAI, BlogCatalog, Coraml, Flickr....) but make sure that training rate are 20/40/60 per class. And we use the last 1000 nodes for test. As for Citeseer, we select training nodes in order (without randomly shuffling) just to keep the same setting with GCN. I can further show the corresponding codes here:
import numpy as np
np.random.seed(20) #random seed
idx_train = []
idx_test = []
label = np.loadtxt('acm.label', dtype = int)
label_rate = 60  #or 40, 20

n = 3025
class_num = 3
train_num = class_num * label_rate
idx = list(range(n))
np.random.shuffle(idx)
count = [0] * class_num
for i in range(len(idx)):
    l = label[idx[i]]
    if count[l] < label_rate:
        idx_train.append(idx[i])
        count[l] = count[l] + 1
for i in range(len(idx)-1000, len(idx)):
    idx_test.append(idx[i])
np.savetxt('test' + str(int(label_rate)) + '.txt', idx_test, fmt = '%d')
np.savetxt('train' + str(int(label_rate)) + '.txt', idx_train, fmt = '%d')

Thanks


I want to how to do something like this to get two calls two fetch the result set

@web2style @kolpav thanks,
I used redux-api-middleware, but I had some problems.
This is my two action
function fetchAlbum() {
  return api(
    {
      url: 'api/album',
      actionTypes: [ALBUM_REQUEST, ALBUM_SUCCESS, ALBUM_FAILURE],
      schema: dataSchema
    })
}
export function loadAlbum() {
  return (dispatch, getState) => {
    return dispatch(fetchAlbum());
  }
}

function fetchCategory() {
  return api({
    url: 'api/category',
    actionTypes: [CATEGORY_REQUEST, CATEGORY_SUCCESS, CATEGORY_FAILURE],
    schema:dataSchema
  })
}
export function loadCategory() {
  return (dispatch, getState) => {
    return dispatch(fetchCategory());
  }
}

function doEverything() {
  return dispatch => Promise.all([
    dispatch(loadAlbum()),
    dispatch(loadCategory())
  ]);
}

How did this two and return a result set。
Thank you
#826

Can you elaborate a bit more? In the doEverything you can dispatch multiple actions like so
const doEverything = () => dispatch => {
  dispatch(loadAlbum())
  dispatch(loadCategory())
}

I have a feeling you want to return something from the doEverything function but thats not how redux works. Result of load album will be in action payload with type ALBUM_SUCCESS and result of load category in action payload with type CATEGORY_SUCCESS.

@kolpav
thank you. Thought I Thought

import { injectReducer } from 'store/reducers';

// Sync route definition
export default (store) => {
  return {
    getComponent: (nextState, cb) => {
      require.ensure([], (require) => {
        /*  Webpack - use require callback to define
            dependencies for bundling   */
        const Home = require('./containers/HomeContainer').default;
        const reducer = require('./modules/home').default;
        const actions = require('./modules/home').actions;

        injectReducer(store, { key: 'home', reducer });

        store
        .dispatch(actions.loadHome()) // set global spinner
        .then(() => cb(null, Home)) // success, data loaded, render component
        .catch(cb);

        /* Webpack named bundle   */
      }, 'home');
    }
  }
}

export function loadHome() {
  return dispatch => {         //  this must be a promise
    dispatch(loadAlbums()),
    dispatch(loadCategory())
  };
}

export const actions = {
  loadAlbums,
  loadCategory,
  loadHome
}

loadHome must be a promise

store
  .dispatch(actions.loadHome()) // set global spinner
  .then(() => cb(null, Home)) // success, data loaded, render component
  .catch(cb);

Is very bad idea you are not rendering anything till all of your data is loaded, imagine you open up facebook and you have blank screen for the entire duration of loading all posts, comments, images etc. only when everything is loaded you are shown the result. You would not like that I suppose. Exact opposite is done throughout the web. You are rendering as soon as you can and "entertain" user with something more interesting than blank screen while data is loading. Some examples:
Google showing dominant color of a thumbnail

Medium blog post is bluring up low res version of images

Facebook post placeholder on codepen
http://codepen.io/Mestika/pen/ByNYBa

For some global indication that you are loading data you can take inspiration from here

@kolpav thank you. you helped me.

Another option is redux-saga, if you're interested.  The main thing is that reducers should be 'pure' and not have side effects, so dispatch would not return the fetch results.  Sounds like you got it figured out, but I prefer redux-saga over thunk.  The thing with the thunk is you would need to write your own code, if you wanted to ensure both external calls succeeded.  Using the redux-saga you can timeout and cancel as well:
http://yelouafi.github.io/redux-saga/docs/advanced/RunningTasksInParallel.html
That will work correctly like the promise.All([,]) from your original question. Dispatching two store events won't have the same guarantees.

Looks like this is resolved, so closing for now. Thanks everyone.


How to use persistent store such as Redis

Is it possible/advisable to use the dataloader library to store data in a persistent, shared cache like Redis?
I've implemented a Redis package that satisfies the Cache interface but I'm hitting a snag when I want to set a value in the cache. The request hangs waiting for the thunk to be resolved.
func (c *Cache) Set(_ context.Context, key dataloader.Key, value dataloader.Thunk) {
	v, err := value()
	if err != nil { // this line is never reached, the request hangs.
		fmt.Println(err)
		return
	}
	if err := c.Client.Set(key.String(), v, 0).Err(); err != nil {
		fmt.Println(err)
	}
	return
}

I'm clearly doing something wrong. Any advice greatly appreciated. Thanks.

Just to be clear, I need to store a text representation of the value I'm caching, so I'm trying to extract the value so I can marshal it. The go-redis.Set() func will only accept a string as the value.

As of v5, we only support in memory caches because we cache the Thunk (function), and not the value it holds. The reason for this limitation is because we uses the cache for both caching AND batching items.  Eventually, I would like to decouple the batching cache from the value cache because it would allow more flexible caching strategies like the one you are suggesting.
That said, I do have a potential solution that I can offer you. I would suggest that you move your Redis logic into the batchFn so that you can retrieve items in bulk and fall back to more expensive calls only when you have to.
For example you could do something like:
func batchFunc(_ context.Context, keys dataloader.Keys) []*dataloader.Result {
  results := make([]*dataloader.Result, len(keys))

  // get what you can from redis
  values, _ := redisClient.MGet(...keys.Keys()).Result()

  // make a list of everything that was not found in redis
  var cacheMisses map[int]string
  for i := range keys {
    if values[i] == redis.Nil {
      cacheMisses[i] = keys[i].String()
    } else {
      results[i] = &dataloader.Result{values[i], nil}
    }
  }

  // get the missing items from more expensive location (like DB)
  for idx, key := range cacheMisses {
    value, err := db.GetValues(key) // Pseudo code!
    redisClient.Set(key, value)
    results[idx] = &dataloader.Result{value, err}
  }

  return results
}

This solution is nice because you can potentially batch both expensive DB calls and Redis calls which, depending on use case, COULD increase performance from one-off get calls.

That's awesome. Thanks so much for your help! I'll give this a shot.

I'm going to close this ticket. Feel free to re-open if you need more help.


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Current coverage is 83.60% (diff: 100%)

Merging #79 into master will not change coverage

@@             master        #79   diff @@
==========================================
  Files             1          1
  Lines            61         61
  Methods           7          7
  Messages          0          0
  Branches         17         17
==========================================
  Hits             51         51
  Misses           10         10
  Partials          0          0

Powered by Codecov. Last update 8357a78...64a5c35

Current coverage is 83.60% (diff: 100%)

Merging #79 into master will not change coverage

@@             master        #79   diff @@
==========================================
  Files             1          1
  Lines            61         61
  Methods           7          7
  Messages          0          0
  Branches         17         17
==========================================
  Hits             51         51
  Misses           10         10
  Partials          0          0

Powered by Codecov. Last update 8357a78...64a5c35


For Testing mkfatfs:Add Option to use BOARDCTL_FATDMAMEMORY

Summary
depends on https://github.com/PX4/NuttX/pull/173
Impact
Testing

Replaced by https://github.com/PX4/NuttX-apps/pull/14


Regen for polymorphic resources

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

This PR is still WIP.
Currently, the following RPs have compilation errors and I will fix them very soon:

automation
cosmosdb
dns
eventgrid
guestconfiguration
privatedns
websites

You will need to rebase the latest changes from main to resolve the conflicts in order to get pipelines to trigger.


[ANCHOR-247] Add profile running utility classes for IntelliJ and VSCode dev environment

…onment

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Add profile running utility classes for IntelliJ and VSCode dev environment.
Development on Windows is also supported.
Why
This is to reduce the friction to contribute to the anchor platform project.
Known limitations
Only Linux and Windows are supported.
For MacOS, follow the instruction: https://github.com/palantir/docker-compose-rule#running-on-a-mac

Reference Server Preview is available here:https://anchor-ref-pr825.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr825.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr825.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr825.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr825.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr825.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr825.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr825.previews.kube001.services.stellar-ops.com/


feat: update GitHub Action references in readme.md

Changes:

Create rough draft that updates references to GitHub Action version numbers in readme.md on renovatebot/github-action repository

Context:
Needs more work, so do not merge until ready! 😄
I accidentally closed my fork... So this is a retry of PR #20.

Is there a way to test this somewhere safe? I'm not sure if this code does what we want... 🙈

Is there a way to test this somewhere safe? I'm not sure if this code does what we want... 🙈

not easily, but at least it's validated. also risk is low for markdown files to break things.


GeneFuse ignores fusions in TCF3

Hello,
Thanks for developing this tool! I was testing it on some hematological samples with TCF3 gene fusions (confirmed by FISH), and it seems like GeneFuse is not picking them up. I have checked that the gene is correctly reflected in the reference, and the reads connecting TCF3 and its partners are uniquely mapped to its location in the corresponding BAM files. I have two samples with different fusions, and in both of them, the fusion was not predicted even as untranscribed. Can you suggest what else can I try to fix this problem?
Thanks in advance!
Cheers,
Nadezda

What's the fusion partner gene of this TCF3 fusion?
Please note that, GeneFuse only detects gene fusions that both two fusion genes are listed in the CSV file.

Hi, the fusion partners were HLF in one sample and PBX1 in the other, these fusions were experimentally confirmed by FISH. Both HLF and PBX1 were included in the fusion file along with TCF3, with the transcripts covering expected fusion junctions:
grep HLF test_fusions.csv

HLF_NM_002126,chr17:55264959-55325187
grep PBX1 test_fusions.csv
PBX1_NM_002585,chr1:164559183-164851831
grep TCF3 test_fusions.csv
TCF3_NM_003200,chr19:1609291-1652615

And these are the expected breakpoints:
sample 1:

chr19:1617928 - chr1:164683443, 59 supporting reads, TCF3-PBX1

sample 2:

chr19:1617921 - chr17:55318881, 77 supporting reads, TCF3-HLF
chr19:1617932 - chr17:55318898, 66 supporting reads, TCF3-HLF

Is there any other reason why this may be happening? Any hard coded filters on the reads/regions? Many thanks in advance!

Maybe reads contained breakpoint were filtered. In such a scenario which a mismatch occurred beside the breakpoint, all kmer contained the mismatch and the  breakpoint  would not be traced back to a genomic position and thus caused  more mismatch and be filtered eventually.


[HUDI-3742] Enable parquet enableVectorizedReader for spark inc query to improve peformance

Change Logs
add config to enable parquet enableVectorizedReader for spark inc query
Impact
improve incremental query for spark.
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

LGTM

@xushiyan @alexeykudinkin @nsivabalan
pls help me check this pr, thanks
for incremental read parquet file, we can achive 2.x faster

IIRC the vectorized reader was disabled for a reason. somehow the filtering was depends on the regular parquet reader.

IIRC the vectorized reader was disabled for a reason. somehow the filtering was depends on the regular parquet reader.

@garyli1019  thanks for your comments.
yes， i know the reason。so in this pr, I directly generated the code for the filter, vectorized the data, and directly filtered it after reading it

@xiarixiaoyao can you post any test results for this effort.

@xiarixiaoyao Main contention here is to introduce a config and keep existing behavior

@xiarixiaoyao can you post any test results for this effort.

Sorry for the late reply， let me do it now

@vinothchandar  @alexeykudinkin @garyli1019
addressed all comments
add config to keep original logical.
benchmark setUp

orignal table: 1000w row,  10 columns,
insert 100w
query begin incremental from commit time 0
benchmark result:

Java HotSpot(TM) 64-Bit Server VM 1.8.0_201-b09 on Windows 10 10.0
Intel64 Family 6 Model 158 Stepping 10, GenuineIntel
perf mor incr read:                       Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------------------------------
vectorized disable                                 1730           1796          94          5.8         173.0       1.0X
vectorized enable                                   739            769          26         13.5          73.9       2.3X

do we need add benchmark code to this pr, if needed, i will added

@xiarixiaoyao can you please rebase this one?
Also please fill out the description to explain the context.

@alexeykudinkin sorry to miss your message。 will update the code and add benmark result for it

@xiarixiaoyao : gentle reminder.

@hudi-bot run azure

@hudi-bot run azure

@alexeykudinkin @nsivabalan
rebase the codes，
Add parameters to control this behavior, which is enabled by default

@hudi-bot run azure

IIRC the vectorized reader was disabled for a reason. somehow the filtering was depends on the regular parquet reader.

garyli1019  has explained the problem: IIRC the vectorized reader was disabled for a reason. somehow the filtering was depends on the regular parquet reader.
Personally,i think we should enable vectorization by default

Personally,i think we should enable vectorization by default

+1. Users do not know when to enable vectorization. So provide a config to control this is meaningless.

@xiarixiaoyao : Can you address the comments in the PR ?
@garyli1019 : Any other concern about having vectorization for incr query for MOR  (with default turned off ? )

@bvaradar only partial columns will return by the vectorized reader, but the merging with log might need some columns not in the select statement, or the necessary columns using by hudi, such as precombine field or primary key. How do we handle this case?

Good point.  It doesn't look like the place where we set spark vectorized reading on has knowledge of read schema to safely enable vectorization for certain cases for MOR. @xiarixiaoyao : Thoughts ?

close this pr as we no longer need it


No path specified Error while using gulp cache with WebFundamentalsKit

Gulp v: 3.8.7 and gulp-cache (map-stream): 0.1.0
Here is the task in gulpjs file
// Optimize Images
gulp.task('images', function () {
  return gulp.src('app/images/**/*')
    .pipe($.cache($.imagemin({
      progressive: true,
      interlaced: true
    })))
    .pipe(gulp.dest('dist/images'))
    .pipe($.size({title: 'images'}));
});

and when I try using gulp images and this is the error I see.
[01:00:05] Starting 'images'...
Possibly unhandled Error: No path specified! Can not get relative.
    at File.Object.defineProperty.get (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/node_modules/vinyl/index.js:150:27)
    at DestroyableTransform.saveFile [as _transform] (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/lib/dest/index.js:40:48)
    at DestroyableTransform.Transform._read (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:184:10)
    at DestroyableTransform.Transform._write (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:172:12)
    at doWrite (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:237:10)
    at writeOrBuffer (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:227:5)
    at DestroyableTransform.Writable.write (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:194:11)
    at Stream.ondata (stream.js:51:26)
    at Stream.EventEmitter.emit (events.js:95:17)
    at queueData (/Users/rayabhagis/projects/nerdtogeeks.com/node_modules/gulp-cache/node_modules/map-stream/index.js:43:21)

But I can evade this one by just removing
.pipe($.cache($.imagemin({
      progressive: true,
      interlaced: true
    })))

part from the gulp task

Hi. Sorry to bother. But I am facing this The system cannot find the path specified error while running the images gulp task.
Here is my code
gulp.task('images', function () {
       $.cache.clearAll();
	return gulp.src('app/images/**/*')
		.pipe($.cache($.imagemin({
			progressive: true,
			interlaced: true,
			optimizationLevel: 5
		})))
		.pipe(gulp.dest('dist/images'))
		.pipe($.size({
			title: 'images'
		}));
});

I tried clearing the cache as suggested above but with no end result. Please help.

Do you have app/images/**/* and dist/images folders?  Is there a stack
trace pointing into the gulp-cache code?
On Tue, Dec 27, 2016 at 2:21 AM sabari venkatesh<EMAIL_ADDRESS>wrote:

Hi. Sorry to bother. But I am facing this The system cannot find the path
specified error while running the images gulp task.
Here is my code
gulp.task('images', function () {
$.cache.clearAll();
return gulp.src('app/images/**/*')
.pipe($.cache($.imagemin({
progressive: true,
interlaced: true,
optimizationLevel: 5
})))
.pipe(gulp.dest('dist/images'))
.pipe($.size({
title: 'images'
}));
});
I tried clearing the cache as suggested above but with no end result.
Please help.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jgable/gulp-cache/issues/30#issuecomment-269306785,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKCkboMRadYRG27atxpuR_FPAC941QLks5rMOaqgaJpZM4CcxkN
.


log bug

Expected Behaviour
log clean
Actual Behaviour
https://www.toptal.com/developers/hastebin/odipiqoxaz.properties

Minecraft Version:1.19

This Appears Normal

well, it doesn't give any problem in the game, but it floods the log a lot

Do you have the same issue if you replace modern elevators and escalators with regular mtr?


Dynamically link to subpages on runbook main page

Makes navigation simpler for people
What does this pull request do?
Dynamically link to subpages on the runbook main page

What is the intent behind these changes?
Had no humour in maintaining the subpage list, so make it semi-automatic.

It started simple, but I had to add fragments as I went:

next if resource.path == current_path to avoid self-reference
.sort_by { |r| r.data.weight } to retain the same sorting
I stopped at implementing safeguarding against different "levels" of resources

So instead, let's keep what we have and navigate on the left.


Fixes 3352 - copy local has no effect when opening project in vs2017

Fixes: #3352
Setting <ExcludeAssets>runtime</ExcludeAssets> in the .paket.props file for packages with copy_local: false to avoid VS2017 changing the props.assets.json when loading the project.
Tweaked the result of combining copy_local flags from lock and references file:

Setting only copy_local: true returns Some true which results in PrivateAssets=All
Combinations of copy_local: true and copy_local: false results in either Some false or None which results in either ExcludeAssets=runtime or no flags at all.

Moved the copy_local setting to element 5 in the .paket.resolved file in the obj folder and updated paket.Restore.targets to read the 5th element instead.

So, what's up with the Travis build? Does it break from time to time or did my changes break something?

Why I'm asking about PrivateAssets is that I have a scenario where I have copy_local: false for some packages in the paket.dependencies file and copy_local: true in some paket.references files (for a unit-test projects). This combination of setting copy_local ends up setting PrivateAssets=All for those packages. Why is the copy_local flag used to set PrivateAssets=All when it is ExcludeAssets=runtime that enables the expected behavior?

Other than the PrivateAssets thing, the changes committed fixes my issue. Should I remove the WIP and open another issue with the PrivateAssets thing instead?

I guess this is why: #2951

I'm done with this one as it fixes the problem I had with opening projects in VS2017 with copy_local: false.

Unless I have to change the PaketPropsVersion in RestoreProcess.fs since I'm now editing the .paket.props file

@forki @matthid Sorry for bothering you, but these changes fixes the issues I got when converting a C# solution with 180 projects (I know, I want to hang myself) to new SDK format. We're using the copy_local option quite extensively and would need this fix or something similar since VS2017 is changing the runtime entries in the project.assets.json when it loads the projects. This results in the .dlls with copy_local: false to be copied to the bin folders anyway.
Is there another way to fix this I should try instead?
Do you have some advice on how to add a test for this?
Running build.sh is all green locally, so I'm not sure why the builds are failing here.

@atlemann Thanks for the PR.
Regarding the Code: I think there should be a built-in to get the "package settings" after considering the paket.references file. So I'm a bit hesitant about doing that manually.
Regarding the Test-Suite: Yes it is probably broken again. I assume at least appveyor should be green again after adding the netcoreapp2.2 changes to the integration test suite (/cc @forki )

Um, not sure I follow what you mean by built-in?

@matthid Is there anything I can change to make it mergable? I basically have to postpone switching to new SDK format due to this.

please merge with master

At least the Appveyor build succeeded again now


Prefer v4.2 of chrony for nts support

merge with https://github.com/iris-GmbH/meta-iris-base/pull/69

superseded by https://github.com/iris-GmbH/meta-iris-base/pull/109


Im using FSCalendar in tableview cell, calendar is working fine but their is an issue while reusing the cell by scrolling up the tableview to the calendar. The tableview became glitchy. her is my code inside cell for row at indexPath:

let calendarCell = calendarTableView.dequeueReusableCell(withIdentifier: "calendarReuse")
let calendarView = calendarCell.viewWithTag(1) as! FSCalendar
calendarView.delegate = self
calendarView.dataSource = self
calendarView.register(DIYCalendarCell.self, forCellReuseIdentifier: "cell")
calendarView.reloadData()
return calendarCell
I am trying to remove "CalendarView.reloadData()" the table view is working fine, but the calendar not loading data anymore.

I also have this issue


repokitteh: add warning for draft PRs

@itayd thoughts?

also, since you renamed the file, you need to rename it also in the root file - repokitteh.star, otherwise it does not take effect and errors out. https://envoyproxy.slack.com/archives/CFTJA0ER1/p1626286039001900

adding a test comment: please ignore!

Ok I think this is ready for review - thank you so much for the pointer to slack error messages!

I'm seeing warnings on #repokitteh - I'm not convinced the rename works with old checkouts pointed at new code.
I'm inclined to keep the new code but revert the rename.  PR coming up


Jackson 2.10.0

Upgrade Jackson to 2.10.0. This should reduce the stream of CVE patches. For more details see
https://medium.com/@cowtowncoder/jackson-2-10-features-cd880674d8a2

Hmmm...I've seen what looks to be a webserver crash both in my pipeline and now on my local machine with this change. Investigating...

I've reproduced a failure locally:
java.lang.NoClassDefFoundError: com/fasterxml/jackson/core/TSFBuilder
        at io.smallrye.openapi.runtime.io.OpenApiSerializer.serialize(OpenApiSerializer.java:106)
        at io.helidon.openapi.OpenAPISupport.formatDocument(OpenAPISupport.java:224)

Looks like this class might not exist in 2.10.

Hmm, no, TSFBuilder exists only in 2.10 and later (see http://fasterxml.github.io/jackson-core/javadoc/2.10/com/fasterxml/jackson/core/TSFBuilder.html).

OK, the problem is microprofile/tests/tck/tck-opentracing forces some jackson dependencies to 2.9.0 so we end up with a blend of 2.9.0 and 2.10.0 on the classpath.


GitHub Actions build failing due to missing env var of frontend/docjson

This can be resolved by providing the following env vars to the directory:
PUBLIC_SUPABASE_KEY=xxxx
PUBLIC_SUPABASE_URL=https://xxxx.supabase.co

The build is successful with 39/39 in my local when the env vars are provided

Fixed by removing the need for them at build in 4cdb2273a59f23f2dc4fdd81fdbc56bf8ee81157


Branch revamp UI

Major rework on the UI.

Navigating contacts with mouse and arrow keys on the contacts panel.

Has the ability to navigate the contacts by clicking it in the panel list on the left with the mouse.
Clicking the display list and using arrow keys to select, hitting enter key to confirm selection
Although font size may be fixed and huge, every part of the text from the display card is reachable by scrolling around up, down left and right.

Added support for displaying really long names in the cards has been added by changing label to text, with wrap properties.

Workarounds made:
ScrollPane has issues detecting the height of a nested list view, hence not being able to scroll down to the true depth of the nested list view.
To fix this a split pane has been introduce to segregate the meeting list and the contact information to ensure that all of the information  on both sides can be seen and not cut off.

Limitations:
Although on smaller window sizes there may be more scroll bars around, the UI has been tweaked such that the scroll bars fit in well while still being able to see all of the content in the Staffbook.
As of now only able to select contacts via mouse and arrow keys.

select command is coming in a seperate pr to select contacts by command line.
closes #53
Preview of what the ui looks like:

Overall, I think it looks decent. 👍 I have added in some parts that you might have missed out and some suggestions to possibly enhance the current Ui. Also, I have also noted that the display of the contact details is slightly cramped up and requires scrolling to view.

This is the limitation of JavaFx itself, as it does not have native support for responsive layouts, it would also be troublesome to implement it from scratch too. Which presents the difficulty of making the UI look good in different screen sizes.
Hence the cheeky way of working around, is by setting a bigger font size to accommodate larger screen sizes/full screen, then when the window is resized to a much smaller window you are able to scroll around to view the contact card as well.
Also there is a center horizontal divider in the details panel that can be dragged up and down to show or hide the contact card and meetings card respectively.


worker_1   | 16:15:08 [Q] ERROR malformed node or string: <_ast.Name object at 0x7fb952093130>

Happens every 30 seconds. How do I debug this and why is this happening?
It seems to be caused internally and not by a task.

@viperfx did you ever figure out the cause of this?

Nope - I just stopped using this library. I stick with Celery and Huey.
On Thu, Oct 29, 2020 at 9:34 AM Adam Duren<EMAIL_ADDRESS>wrote:

@viperfx https://github.com/viperfx did you ever figure out the cause
of this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Koed00/django-q/issues/400#issuecomment-718755782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACQEQLT47Y32I47HP2JLXLSNFVOBANCNFSM4KEK2FJQ
.

--
Tharshan

FWIW: The malformed node or string issue I encountered was because I passed a UUID as an argument to schedule. Apparently that creates an issue when it gets serialized. The fix was to cast it to a string before calling schedule.
@Koed00


Only allow each flower to be visited once per nest cycle (per student)

Why thank you! :blush: Thanks for looking at it!


crashed randomly

Describe your bug here.
the title :trollface: :trollface: :trollface:
Command Prompt/Terminal logs (if existing)
I was playing the .exe from the 'export' folder so, there was nothing to see (also my command prompt wasn't opened)

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
nah, just copied my mod from 0.5.2h (which was not using source)

Learn how to report a bug


[types] Add missing string option type in toGamut

Only types are added to match functionality
https://github.com/LeaVerou/color.js/blob/f22556161af0caf03bff447ff86b918d125ccbe4/src/toGamut.js#L23-L28

@LeaVerou could you take a look at this? Otherwise it is creates errors in a typescript environment.


Add custom indoor picker level

Does any other open PR do the same thing?
PR #2189  , it has no return when Indoor Building is out of focus.
What issue is this PR fixing?
Added new events onIndoorBuildingFocused(also return empty array when building is out of focus), onIndoorLevelActivated.
Added command setIndoorActiveLevelIndex
Added new example for Indoor Map.
How did you test this PR?
All new features are for Google Map.
Tested on Android emulator (Nexus 5), iOS Simulator (iPhone X), real Android device (HTC U11), real iOS device (iPhone 6S) by running example.

@luciferes thank you ❤
LGTM @alvelig 🐽


New Plugin: linode-cli

A plugin for the Linode CLI.
Closes https://github.com/1Password/shell-plugins/issues/80

thank you! This is awesome.

@florisvdg I believe that I've merged main back into my fork, but since this is my first pull request on any open source project not certain. As to the importer, if we are to someday add the config file importer, maybe we should leave it as-is for now to be ready for that?


Pick a metrics library if none is selected and the prometheus-exporter feature is used

If a user uses the prometheus-exporter feature, there's a good chance they don't care which metrics library they're using. We should use one by default iff the prometheus-exporter feature is enabled and no metrics library is explicitly selected.
The question is which one to use. I'm somewhat tempted to make it prometheus-client because that's the official Rust prometheus client and the only one that currently supports exemplars.

Alternatively, we might want to use the prometheus client because it is already a dependency of the prometheus-exporter feature


How to suppress response schema for response code = 204 No Content?

I have an async Web API controller method that uses Task<IHttpActionResult> as the result type for convenience in generating detailed error responses.
Swashbuckle considers this a non-void function return type, and emits a response schema referencing "Object" in the output JSON.
How do I suppress that schema output, so that swagger-UI doc for this function will look the same as what is shown for a void function?
Example:
/// <summary>
/// description text
/// </summary>
/// <response code="204">No Content</response>
/// <response code="400">Bad Request - Invalid or missing parameter data</response>
[HttpDelete]
public async Task<IHttpActionResult> DeleteContract(Guid walletId, Guid contractId)
{ }

This currently outputs:
responses: {
204: {
description: "No Content",
schema: {
$ref: "#/definitions/Object"
}
},
400: {
description: "Bad Request - Invalid or missing parameter data"
},
}

But I want it to look like this:
responses: {
204: {
description: "No Content",
},
400: {
description: "Bad Request - Invalid or missing parameter data"
},
}

As a side note, shouldn't Swashbuckle do this return type suppression automatically for response code = 204?

Good catch - I think it's reasonable to say SB should do this automatically. Fancy submitting a PR?

Ya, I think I figure out a code fix this weekend.  Which class would be the best place to make the change?

The update needs to happen somewhere around the following line ...
https://github.com/domaindrivendev/Swashbuckle/blob/master/Swashbuckle.Core/Swagger/XmlComments/ApplyXmlActionComments.cs#L96

I worked around this by writing my own ResponseType attributes, starting with a base class:
[AttributeUsage(AttributeTargets.Method, AllowMultiple = true, Inherited = false)]
public class ResponseTypeAttribute : Attribute
{
    public ResponseTypeAttribute(HttpStatusCode statusCode)
        : this(statusCode, typeof(void))
    {
    }

    public ResponseTypeAttribute(HttpStatusCode statusCode, Type response)
    {
        StatusCode = statusCode;
        Response = response;
    }

    public HttpStatusCode StatusCode { get; set; }
    public Type Response { get; set; }
}

I could either use the class as is, or sub-class it for more conciseness and consistency:
[AttributeUsage(AttributeTargets.Method, AllowMultiple = false, Inherited = false)]
public class ResponseTypeNoContentAttribute : Attribute
{
    public ResponseTypeNoContentAttribute()
        : base(HttpStatusCode.NoContent)
}

I then wrote an IOperationFilter that reads the attributes and updates the definition accordingly, looking at the response type.
I find this far more powerful than using the XMLDOC comments - for starters (and I haven't shown it here) it allows me to use resources for internationlised descriptions, either by convention or explicitly provided. It also gives you compile-time checking for those times when someone renames a response class. I can also automatically add common response types, such as 500 or 503 to all responses.
Finally, for good measure, I later added the ability to provide example response data, in a similar manner to @mattfrear
So the final result is something like this:
[ResponseTypeNoContent]
[ResponseTypeBadRequest]
public IHttpActionResult Put(long id, UpdateCommand command)
{
// Do stuff here
return new NoContentResult();
}
And nary an XML comment to be seen, at least as far as the response types are concerned.


encoding/asn1: ASN1 UTC Time support should accept Time in "YYMMDDhhmmsshhmm" format

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.6.2 darwin/amd64

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/sthorne/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GO15VENDOREXPERIMENT="1"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fno-common"
CXX="clang++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

encoding/asn1 does not support both known ASN1 UtcTime formats as described here: http://www.obj-sys.com/asn1tutorial/node15.html.
asn1.parseUTCTime
Currently, only "YYMMDDhhmm[ss]Z" is supported.

What did you expect to see?
Support for "YYMMDDhhmm[ss]Z" or "YYMMDDhhmmsshhmm"

What did you see instead?
When using a server with the second format this is the error.

Error : Get https://ovirt.example.com/: tls: failed to parse certificate from server: asn1: time did not serialize back to the original value and may be invalid: given "141020123326+0000", but serialized as "141020123326Z"response Status:

CC @agl

UTCTime values MUST be expressed in Greenwich Mean Time (Zulu) and MUST include seconds (i.e., times are YYMMDDHHMMSSZ)

(From RFC 5280.)
If the certificate is issued by a public CA please include it and I'll complain to the CA.


@Column attributes to not insert, select or update not working

Hello,
I use some attributes, such as userId, in my main class with select, insert and update as false, just to get value and save in the audit in other fields.
Do you have some suggestions or fix the problem?
For example:
My main entity:
  @Column({ select: false, insert: false, update: false })
  userId?: string;

My audit entity, extends from main entity:
  @BeforeInsert()
  storeUserId(): void {
    if (this.userId) this._userId = this.userId;
  }

  @Column()
  _userId?: string;

Hi @roberto-souza,
I modified it to copy all but the column options below.
I have published version 0.5.3, so please check again. 🙏
generated,
generatedType,
generatedIdentity,
asExpression,
foreignKeyConstraintName,
primaryKeyConstraintName,
onUpdate,
unique,

@roberto-souza Sorry, There was a problem, so I published 0.5.4 in a hurry. Please check with version 0.5.4.

Now working perfectly! Thank you @kibae!


Pydocstyle options not passed through

In the code for the pydocstyle linter, params and meta are ignored .
Not even the options available for Pydocstyle config files work.
A really useful option to pass would be --add-ignore.
Or do I miss a way of achieving this through pylama?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Is it possible to have an update on this? I would like to use the --match, and --match-dir options, from pydocstyle, but I am not sure how to make it work.

I would also like this. I want to set the convention to google with convention = google but it's not working.


No values

Describe The Bug:
I get no values in % or lux
Plugin Config:
"accessToken": "token",
"priceIncTax": true,
"activatePriceGraphing": true,
"activatePriceSensor": true,
"activateRelativePriceSensor": true,
"_bridge": {
"username": "0E:E2:F6:50:58:B6",
"port": 57342
},
"platform": "HomebridgeTibberPrice"
Screenshots:

Hmm could be the same issue as #29 , do you have any logs from the plugin to verify?
I'll try and make the error handling a bit more robust!

Hi @FluGanz ! I've made the error handling a bit more robust.
Try updating to 1.0.3 and see if that fixes the problem.
If not, please provide some logs from homebridge 👍
Closing for now


fix: pad the leading zero bytes for ipv4 in ipv6 addresses

Fix: Leading zeros may not be set correctly for ipv4 in ipv6 addresses

1. Are you opening this pull request for bug-fixes, optimizations or new feature?
bug-fixes
2. Please describe how these code changes achieve your intention.

described in the related issue
3. Please link to the relevant issues (if any).

#319
4. Which documentation changes (if any) need to be made/updated because of this PR?

no
4. Checklist

[x] I have squashed all insignificant commits.
[x] I have commented my code for explaining package types, values, functions, and non-obvious lines.
[x] I have written unit tests and verified that all tests passes (if needed).
[x] I have documented feature info on the README (only when this PR is adding a new feature).
[x] (optional) I am willing to help maintain this change if there are issues with it later.

Thanks~


Additional error when there are issues with the applicable CET for a mapping constructor expression

Description:
$title.
Steps to reproduce:
public function main() {
    Foo f = {
        i: 1
    };
}

$ ballerina run qwe.bal
Compiling source
        qwe.bal
error: .::qwe.bal:2:5: unknown type 'Foo'
error: .::qwe.bal:2:13: a type compatible with mapping constructor expressions not found in type 'other'

Affected Versions:
slp1

Same for list constructor too.
public function main() {
    Foo f = [];
}

$ ballerina run test.bal
Compiling source
        test.bal
error: .::test.bal:2:5: unknown type 'Foo'
error: .::test.bal:2:13: incompatible types: expected 'other', found '[]'


UI Improvements

This issue aims to gather related idea/effort to improve the UI:
Filtering:

Avoid using directly gremlin

https://github.com/skydive-project/skydive/issues/1723

Make the filtering component a real component with different view/vuejs template. Can be a search box or a direct filtering button.

Topology view:

Use statically defined/template view, like container view, VM view, infra view. The goal is to defined static layers in order to place specific object to dedicated layers. Can make use of predefined gremlin filters.

Closing, new WebUI will handle this
https://github.com/skydive-project/skydive-ui


Feat: Important events in History in timeanddate.com

Describe the feature
timeanddate.com can be scraped to find the important events that happened on history on a specific day.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC 23 contributor
[X] I want to work on this issue

Go ahead @DyuthiVivek |


$compile error on ngMaterial

Angularjs version 1.5.7
Bootstrap version 1.3.3
Getting this error using availableParams as (ctrl is valid)
Error: $compile:ctreq
Missing Required Controller
Controller 'ngModel', required by directive 'key', can't be found
ctrl.availableSearchParams = [
    { key: "name", name: "Name", placeholder: "Name..." },
    { key: "city", name: "City", placeholder: "City...", restrictToSuggestedValues: true, suggestedValues: ['Berlin', 'London', 'Paris'] },
    { key: "country", name: "Country", placeholder: "Country..." },
    { key: "emailAddress", name: "E-Mail", placeholder: "E-Mail...", allowMultiple: true },
    { key: "job", name: "Job", placeholder: "Job..." }
];

instead of
ctrl.availableSearchParams = [];

that works but only with "query" parameter.
I'm unable to reproduce it on jsfiddle or plunkr.

Hi, I'm at a good point in this situation.
The attributes data-key is in conflict with one of my other plug-ins (datarangepicker I think).
If I change the template data-key attribute to data-nit-key (absolutely random but with reference to your nit- prefix) it works as normal.
I'll inspect more accurate when I get time.

Hi @marchrius,
the data-* stuff was a PullRequest to allow better testing of the UI. It is not required to have these attributes. Simply remove them from your template.


Scavengers: Update scav status for hosts

Screenshot of new layout

Code quality not really up to my standards, but oh well, this isn't core. @panpawn, if you have anything else to add?

Sorry, just got around to rebasing this, is anything else I can improve @Zarel? I'd love to improve the quality of this, just not quite what you mean...

It's not about your code, really, it's about the scavs plugin in general.

Bump


MZ-R37 set selection issues

As far as I can tell, there is a logic bug in the input_string() set selection code, that was causing some track names to be garbled on my MZ-R37.
return_current_set() was being called with current_set as the basis, but the call to enter_correct_set() a few lines beforehand caused us to enter wanted_set, so, depending on the set transitions, we may enter the wrong set. The phrase "Denis & Denis" reliably triggers this issue on my MZ-R37 - the '&' character and everything following it is garbled.
This PR fixes this issue on my player, but I do not have any other devices to test it with. If my understanding of the character-entry state machine is correct, I think it should apply to the others too.
I can confirm that with this change, gmdrec works perfectly on the MZ-R37.

Thanks for the report, this is the first report from a R37 user!
However. this change breaks later players such as my R90, so it looks like MZ-R37 will need a special treatment.
The issue may be related to a difference I saw in #8 - basically, when you enter a letter from a given set (for example 'D' from the set of uppercase letters) the player remembers this and the entry of the next letter begins also in the capital letters. The exception from this rule are what I call 'common set characters' which on most players are "'", ',', '/', ':', ' ', in this case the player remembers the last letter from any other set excluding this one.
Since the text becomes garbled after space, my guess is that R37 (kind of llike the early Japanese firmware R91 I had) returns to the uppercase set after space is entered. To test that, manually edit a track name and type lowercase a followed by space, then press Pause. If the next letter starts as a capital letter then that's the issue.

Sorry that I assumed there was a bug! I'll do a bit more experimenting tonight and see if I can find a nicer way to handle it.

updated the PR with changes that add R37 as an additional recorder type; it uses the definitions from definitions/r90.py but adds logic to handle the R37's character set selection differences, and a slight difference in how labelling mode is entered.

Oh also, I had to increase the HOLD time in settings.conf to 2.5s, to get my player to reliably recognize long button-presses. Would this be acceptable as a new default, or would you rather just document that the R37 needs a longer hold time?

Thanks for the PR. A couple questions:

can you describe the character set of the MZ-R37 a bit more in detail? If possible I'd prefer to modify the mappings in player definitions (like in r90.py, but specifically for R37) rather than add if clauses to the main logic
the change in enter_labelling - is the reason for it that otherwise a space is entered at the beginning of the track?

Yes, the enter_labelling change was because it was entering a space at the start of each track - the R37 only requires one press of STOP to get into labelling mode from the edit menu.
I took a look at #8 and did a bit of experimenting, the R37 (at least my US model) does always remember the character set after selecting a common character, so it's not that.
a - b is a minimal test case for the problem. Without the change in the call to return_current_set, it gets entered as a z b.
I added some basic logging to input_string() and push_button() and got the following difference:
With current_set = return_current_set(letter, wanted_set), produces 'a z b':
a: uppercase -> lowercase+0
Pause x 2
Right x 0
Stop x 1
 : lowercase -> uppercase-1
Pause x 3
Left x 1
Stop x 1
-: lowercase -> uppercase-6
Pause x 3
Left x 6
Stop x 1
 : numbers -> uppercase-1
Pause x 2
Left x 1
Stop x 1
b: numbers -> lowercase+1
Pause x 3
Right x 1
Stop x 1
Stop x 1 (long)

With current_set = return_current_set(letter, wanted_set), produces 'a - b':
a: uppercase -> lowercase+0
Pause x 2
Right x 0
Stop x 1
 : lowercase -> uppercase-1
Pause x 3
Left x 1
Stop x 1
-: uppercase -> uppercase-6
Pause x 1
Left x 6
Stop x 1
 : numbers -> uppercase-1
Pause x 2
Left x 1
Stop x 1
b: uppercase -> lowercase+1
Pause x 2
Right x 1
Stop x 1
Stop x 1 (long)

Yes, the enter_labelling change was because it was entering a space at the start of each track - the R37 only requires one press of STOP to get into labelling mode from the edit menu.

There is actually a reason for that, which I forgot about. It turns out that the players can tell whether the connected remote has a LCD display or not and behave differently in either case. For N1/R910/R909 we need to pretend to have a display (by pulling down the LCD pin unused on normal 3-wire connectors) and then two Stop presses are required. So it should be two only for those three devices. However, I'd like to refactor this function anyway -- I'll track it in #14

implemented with aeed7301d32fa9f32235f31040f4679f9f6c1405 and 19cd4221ca2958215eda2c2760041c024678ead0


Lathe Assembly Binning

Hi @elimoss
I am using your tool for assembling my long reads metagenome. I have a doubt in the procedure of assembly that does the lathe also bin the genomes before doing the circularization or do we have to do it separately after obtaining the final contig after the assembly is completed?
And I have few more questions that I wanted to ask:
Why are you using the very outdated version of FLYE assembler when they have upgraded to better versions where we don't have to give the genome sizes as they have told me?
I have done assembly of long read (ONT) using lathe and I do not see a very good result. The largest contig was of length 1130359. My sample is a diverse wastewater sample and I expect the largest contig to be much higher and of course, should be more in number as these have millions of bacteria and the minimum genome size of a bacteria should be more than 0.5 mega base pairs.
What parameters can be changed to achieve a better assembly result other than genome size? What genome size I should provide when I am expecting my sample to be diverse. I gave 100m and 250m, should I increase the genome size to 2500m and 4500m?
Thanks
Saraswati Awasthi

Hello Saraswati Awasthi
I am processing some long reads metagenome ,too . But from the workflow of Lathe ,  i don"t  how can get the bins from the metagenome. Have u solve your problems from using this workflow ?
Thanks
Binghe Zhao


⚠ Bofrost.de has degraded performance

In 6ec5ef7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 815 ms

Resolved: Bofrost.de performance has improved in de3b34d.


[Feature] Adding Cyber Radio Button

Description
I want to add a new button i.e. Cyber Radio Button... That on hover Shows the 3d cyber blurring text effect
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Can you show the demo screenshot of this button @Sakshamjain98

Show the hovering screenshot or demo video @Sakshamjain98

https://github.com/Rakesh9100/Beautiify/assets/137368917/8eb62eb3-a903-4346-b3f5-c7efa13fbcbf

Proceed @Sakshamjain98

PR status @Sakshamjain98


feat: Team Users

Feature Request
Describe the Feature Request
Implement the new Team Users API: https://youraccount.lacework.net/api/v2/docs#tag/TeamUsers
Is your feature request related to a problem? Please describe
I need to automate users provisioning in Lacework.
Describe Preferred Solution
Then end goal is to drive team users using this Python SDK.

Team Users have been implemented


feat: add virtualization objects support

:tada: This PR is included in version 0.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


decouple script and pickle

Currently, pickled data includes script. Further, some text does not change even after forcing script. Quick fix would be to save script-wise pickle, but a better solution would be to do a deeper fix.

http://www.benfrederickson.com/dont-pickle-your-data/ - use json (eg with jsonpickle, or via http://sanskrit-data.readthedocs.io/en/latest/sanskrit_data_schema.html#intro )

etaccha kRtam

Many many thanks. I am indebted for your contributions! Sincere apologies for the lousy code --- it kept growing randomly, without me spending quality time to clean it up :)
Another major issue with pickle -- being binary, can't see diffs, and always gets updated when I re-run the code (perhaps a timestamp is added)


bots: Notify users when their account is modified.

WIP
Fixes: #23641

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hello @laurynmm! I am wondering if you could possibly give this PR an early code review? I think I am mostly done with the logic sides of things and just need to fine tune and polish the final notification along with new test cases. I feel like the current code could be improved and would love another pair of eyes to look at it.

Thank you for the suggestions @laurynmm! I didn't actually realize that this PR is going to be that big. I'll take my time to digest your reviews.

In fact an easier staring place might be the name field and then the role field, and then doing the custom profile fields separately.

I didn't realize that I will also have to notify the users of changes in the name and role field. I'm wondering if I should create seperate commits for different changes in each fields.

I wonder if we want one message per field updated. For example, if the admin changes the role and the name, do we send one message or two.

I did actually take in consideration on how the messages are sent and formatted and felt like that was a question for Alya for sure. I agree we should open a discussion about this in #design.
I do agree about your 4th point about creating two database queries being expensive but I do think that it provides valuable information in the process. I'm not sure if the expense is worth the information but nonetheless I'll try my best to actively participate in the disscusion.
I do actually have one question where the issue states that there is automated changes to a user's profile. I'm not sure how the automated changes work and how to test the automated changes and would love some insights on it.
Again, thank you so much for the reviews, it has been quite helpful to me!

I'm wondering if I should create seperate commits for different changes in each fields.

Yeah, it could make sense do them as separate commits. Especially for when folks review the changes. Just make sure that each commit passes the relevant test suites for the changes.

I do actually have one question where the issue states that there is automated changes to a user's profile. I'm not sure how the automated changes work and how to test the automated changes and would love some insights on it.

Yeah, that was confusing to me too. It would be worth asking for clarification about that. I don't see any automated situations currently in the codebase for this path/endpoint. But maybe there are plans to add some as part of changes to admin management of user groups?

I do agree about your 4th point about creating two database queries being expensive but I do think that it provides valuable information in the process. I'm not sure if the expense is worth the information but nonetheless I'll try my best to actively participate in the disscusion.

Yeah, I can also see it being enough information to just show the updated value of the field in the message from the bot. But in the end, it's best to have those sorts of conversations on CZO so more people can have an opportunity to give their opinion.

@palashb01 @brijsiyag this is an initial draft and only covers a portion of the issues. Would love for you guys to take a first look and provide feedback.
@zulipbot add "buddy review"

@Joelute Thanks a lot for working on this. I haven't worked in the backend or with Django before, so I'm not entirely sure if the implementation is good enough. However, I did the manual testing and tried to cover as many cases as possible, and it seems to be working fine in all of them.
PS: It might be a good idea to seek a review from someone experienced in the backend. Perhaps @prakhar1144 or @brijsiyag could provide their insights when they have some time available :)

I'll try to review it today.


added labelId for StreamField FieldBlock / widgets labels’ missing semantic association with inputs

Fixes #11179
I have added an labelId variable and patched it in for attribute of label tag in client/src/components/StreamField/blocks/BaseSequenceBlock.js. I am trying to test this in microsoft edge web browser and google chrome.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Comment added with a bit more info about some potential approaches here https://github.com/wagtail/wagtail/issues/11179#issuecomment-1800254497
It looks like this may not be a good first issue as it will require a bit of digging into the block code, JS and jQuery plus accessibility.

@lb- thanks for your valuable response and guiding me through this issue.I will close this PR and make a new PR on this issue based on your response.


Cannot find type [System.Collections.Generic.List[CommandReferenceParameter]] error

After running the following:
Install-Module -Name Az.Tools.Migration;
New-AzUpgradeModulePlan -FromAzureRmVersion 6.13.1 -ToAzVersion 6.1.0 -FilePath [File Path/Name] -OutVariable ProposedPlan;

I'm getting the error:
New-Object : Exception calling ".ctor" with "0" argument(s): "Cannot find type [System.Collections.Generic.List[CommandReferenceParameter]]: verify that the assembly containing this type is loaded."
At C:\Program Files\WindowsPowerShell\Modules\Az.Tools.Migration\1.1.2\Functions\Private\Find-CmdletsInFile.ps1:125 char:30
+                 $cmdletRef = New-Object -TypeName CommandReference
+                              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [New-Object], MethodInvocationException
    + FullyQualifiedErrorId : ConstructorInvokedThrowException,Microsoft.PowerShell.Commands.NewObjectCommand

Same issue here. It went away after a restart.

I believe it is due to dotnet cannot unload loaded assembly. Please restart PowerShell. If issue is still there, please let us know.

close this issue since no feedback from author, please feel free to reopen if you have further questions.


mp4 format not working on 1080p youtube videos

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.09.28
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

$ youtube-dl -v --format mp4 https://www.youtube.com/watch?v=aEVb3rrooBQ
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--format', u'mp4', u'https://www.youtube.com/watch?v=aEVb3rrooBQ']
[debug] Command-line args: [u'-v', u'--format', u'mp4', u'https://www.youtube.com/watch?v=aEVb3rrooBQ']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.09.28
[debug] Python version 2.7.9 (CPython) - Linux-4.9.35-v7+-armv7l-with-debian-8.0
[debug] exe versions: avconv 11.12-6, avprobe 11.12-6, phantomjs present
[debug] Proxy map: {}
[youtube] aEVb3rrooBQ: Downloading webpage
[youtube] aEVb3rrooBQ: Downloading video info webpage
[debug] Invoking downloader on u'https://r4---sn-q4flrne6.googlevideo.com/videoplayback?expire=1570236235&ei=65KXXcGzG5GVwQGp0Z6wBw&ip=<IP_ADDRESS>&id=o-AGtG_i5zZXnM1b3ax5iNOXmO8M1UAHNqFxQ16DbmRv0V&itag=22&source=youtube&requiressl=yes&mm=31%2C26&mn=sn-q4flrne6%2Csn-vgqs7nlk&ms=au%2Conr&mv=m&mvi=3&pl=16&initcwndbps=1730000&mime=video%2Fmp4&ratebypass=yes&dur=928.798&lmt=1569687604289103&mt=1570214540&fvip=4&fexp=23842630&c=WEB&txp=3416222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cmime%2Cratebypass%2Cdur%2Clmt&sig=ALgxI2wwRQIhAI0fk4hWtPHkBAaCiTT8K8LJXvhug5N4Yfc6XfSsq-XtAiBBCbGvmkFEZjW-GBAe_Vz_yLXKSWz2rghOC2xReJKnPw%3D%3D&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHylml4wRQIhAKD3dcdvyJgCt75QdGheOJAAzwNRpDJtbOLwIDW7z-CjAiARuTnKNMaoetLx6ZgGeE25uTV8tWzbTth1pQAVzpKolA%3D%3D'
[download] Destination: spooky_time.mp4-aEVb3rrooBQ.mp4
[download]   0.2% of 166.33MiB at  1.09MiB/s ETA 02:33[download] Got server HTTP error: Downloaded 325879 bytes, expected 174404698 bytes. Retrying (attempt 1 of 10)...

ERROR: Did not get any data blocks
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 342, in real_download
    return download()
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 320, in download
    retry(err)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 225, in retry
    raise RetryDownload(e)
RetryDownload

Description

The playable codecs available to me are limited, so I typically run youtube-dl with --format mp4
When downloading youtube videos with 1080p, asking for mp4 causes the above failure
I Assume that this is because youtube is storing 1080p as video only, and explicitly calling --format mp4 doesn't handle that situation well
-- The video above has this format listed, and I assume it's what it attempts to download: 137          mp4        1920x1080  1080p 3748k , avc1.640028, 30fps, video only, 183.60MiB
-- if I run without --format mp4 it handles this situation very well, and the resulting mkv fits my needs, however videos without 1080 might default to a codec I cannot consume (like webm)
-- at this point I actually have a script that checks formats and runs it with --format mp4 if the video does not have 1080p...but I figured other people might want the --format mp4 option to do the merging and result in a mkv containing mp4 video...or somehow handle this format type smoother

Server side Youtube error for format 22.


Support for Websocket RFC 6455?

I was wondering if there are any news or update about the "planned" support for WS RFC 6455.
Thank you in advance

#13446


add the skip points for KeyRange

Search before asking

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

Motivation
Do we need to add a position to ignore the numkey integer because we only support a consistent range? For example, we add a wrong key currently in the watch command in zdiffstore and zinterstore.
Solution
No response
Are you willing to submit a PR?

[x] I'm willing to submit a PR!

@PragmaTwice do you think it is necessary?

I see the function iterates over range.key_step. And it will find the argument[i] to exec the watching.  https://github.com/apache/kvrocks/blob/8539ee50c7ed74dfd35a396d138efbc8bde65542/src/server/server.cc#L1659-L1670

The unnecssary numkey will be added into the finding. I think it's innocuous because the numkey shouldn't exist in watched_key_map_.

The current key range design is divided to two methods:

a static constant [start, end, step] if the key range can be determined without actual commands, e.g. GET key
a dynamic [start, end, step] according to actual commands, e.g. opname numkeys key... someflag

To generalize the key range design to a more pervasive form, I think we can add the third method:

a static constant [start, end, step] if the key range can be determined without actual commands, e.g. GET key -> [1,1,1]
a dynamic [start, end, step] according to actual commands, e.g. opname numkeys key... someflag -> [2, 1+numkeys,1]
a dynamic key position vector according to actual commands, e.g. opname numkeys1 key1... someflag numkeys2 key2... someflag -> {2,3,...,1+numkeys1,4+numkeys1,...,3+numkeys1+numkeys2}

Ok, I will add this method. That's a pretty immature skip points.

BTW, does we use the vec to store CommandKeyRange?

BTW, do we use the vector to store CommandKeyRange? like this {{2,1+numkeys,1},{4+numkeys,-1,1}}
I think it will show well.

Looks fine to me.


Display Resolution is Not Preserved

Love the tool but it does not preserve aspect ratio metadata so I currently need to modify the files afterwards.
Input video: 960x480,SAR 119:134, DAR 119:67
Output video: 960x480, SAR 1:1 DAR 2:1

Hmm, there's nothing specific here that would cause this, at least that I can think of.
Can you try using -e libx265 (to rule out svt-av1 encoder). If that still does it it must be an ffmpeg thing.
Try using ffmpeg directly and seeing if it occurs that way, if so maybe there are workarounds?

I can confirmed that this did not happen with --encoder libx264
Input video: 960x480, SAR 479:538, DAR 479:269
Output Video: 960x480, SAR 479:538, DAR 479:269

Same video ran with --encoder svt-av1:
Input video: 960x480, SAR 479:538, DAR 479:269
Output Video: 960x480, SAR 1:1 DAR 2:1

I also tried the same video with --encode svt-av1 --vfilter "setdar=dar=1.777":
Input video: 960x480, SAR 479:538, DAR 479:269
Output Video: 960x480, SAR 1:1 DAR 2:1

if so maybe there are workarounds?

For personal use I can write a script that just sets the output dar to be the input dar after the encode is done. But having the dar change with no specification to change it and not being able to force a dar with --vfilter "setdar=dar=#.###" is unexpected.

Try using ffmpeg directly and seeing if it occurs that way, if so maybe there are workarounds?

SvtAv1EncApp does not keep track of a DAR, all pixels are square. So, when the video stream from SvtAv1EncApp is sent to FFMPEG there is no display aspect ratio included. This is not affecting the libx264 and libx265 encoding options because those are both handled entirely within an FFMPEG call so FFMPEG can get the display aspect video from the input video.
I have not found a way to set the -aspect parameter from a mapped input video stream. Adding a .arg2("-aspect", var) to line https://github.com/alexheretic/ab-av1/blob/main/src/svtav1.rs#L164 would where var is a passed in as a command argument would help. I am not at all familiar with the Rust programming language but it I think the following sudo-code would completely fix this issue.
	let dar = Command::new("ffprobe")
        .kill_on_drop(true)
        .arg2("-v", "error")
        .arg2("-select_streams", "v:0")
        .arg2("-of", "default=noprint_wrappers=1:nokey=1")
        .arg2("-show_entries", "stream=display_aspect_ratio")
        .arg2(input)

    let to_output = Command::new("ffmpeg")
        .kill_on_drop(true)
        .arg("-y")
        .arg2("-i", "-")
        .arg2("-i", input)
        // use av1 video
        .arg2("-map", "0:v")
        // map everything but video from input
        .arg2("-map", "1")
        .arg2("-map", "-1:v")
        .arg2("-c:s", "copy")
        .arg2("-c:a", audio_codec)
        .arg2_if(downmix_to_stereo, "-ac", 2)
        .arg2("-c:v", "copy")
        .arg2("-aspect", dar)
        .arg2_if(audio_codec == "libopus", "-b:a", "128k")
        .arg2_if(add_faststart, "-movflags", "+faststart")
        .arg2_if(add_cues_to_front, "-cues_to_front", "y")
        .arg(output)
        .stdin(svt_out)
        .stdout(Stdio::null())
        .stderr(Stdio::piped())
        .spawn()
        .context("ffmpeg to-output")?;

Thanks, another option might be invoking svt-av1 through ffmpeg in the same way other encoders are. Previously it wasn't possible to pass svt-av1 args that way, so SvtAv1EncApp is called directly, but in recent ffmpeg versions it's possible.
This should address this issue by letting ffmpeg handle it. I just need to test that this path will work properly.

You can also test this path out by replacing --preset N with --preset N -e libsvtav1 --pix-format yuv420p10le to use the ffmpeg path.

Can you try #98 and see if it solves this issue?

I have some doubts about how this would work.
If ab-av1 uses ffmpeg libsvtav1 by default, I guess it means that the svtav1 version used is the one implemented in the ffmpeg version we have installed, is that correct?
How can we know what version of svtav1 is included in our ffmpeg version?
Will we still be able to run ab-av1 with SvtAv1EncApp like we do now?
I was compiling SvtAv1EncApp directly from their gitlab to take advantage of the latest updates, it would be a pity having to wait for ffmpeg to release a new version that includes an updated svtav1 version.
Besides, I was also trying to keep ffmpeg more regularly updated with some auto-builds, but have stopped doing so because it started to mess with ab-av1 sample split duration being absolutely inconsistently reported and very unreliable (for example, samples reported as 1h33m, or 15m, etc. very weird stuff that was resulting in nonsensical estimations). I tested with many different files, some were not so bad as others, but it seemed random and was overall unusable, until I found out that the only thing that had changed in my system was ffmpeg (a couple of weeks ago), so I rolled back to a previous version and the problem disappeared. Now I'm sticking to this build from 31st of October, it works great and I'd rather not touch it for a while, just in case.

If ab-av1 uses ffmpeg libsvtav1 by default, I guess it means that the svtav1 version used is the one implemented in the ffmpeg version we have installed, is that correct?

On linux it'll be the same version as SvtAvtEncApp it'll be dynamically linked, at least it is on Arch. If using a statically linked ffmpeg build it'll just be whatever they used, presumably a recent version in a recent build.

Will we still be able to run ab-av1 with SvtAv1EncApp like we do now?

In #98 no, I've removed the code. It's a win for code complexity as we'll no longer have 2 paths for encoding so I like it from that point of view. That it also fixes these aspect issues is another nudge in that direction.

it would be a pity having to wait for ffmpeg to release a new version that includes an updated svtav1 version.

ffmpeg don't need to release a new version for regular non-breaking updates, if statically linked ffmpeg will need to be rebuilt.
Overall I think it'll be worth unifying the encoder logic, but I'll do some more testing. @iPaulis if you have time take #98 for a spin and see if there are issues for your use case.

We can add install steps to the readme or more detailed stuff to the wiki.
On the other hand perhaps I've underestimated how easy it is to have ffmpeg setup with libsvtav1 (while it is indeed very easy on Arch). Perhaps that means we should hold fire on this change for a while, or add a fallback to the existing behaviour, though that would mean more code rather than less.

It sure seems way simpler in linux and its users are generally more willing or used to deal with this kind of things. For me this is not an issue either, but most windows or mac users probably won't know or want to bother compiling on their own, and that would greatly limit the number of users that could benefit from this tool.
I've been looking a little bit how it works for windows and I'm not so sure about shared libraries for the shared version of ffmpeg, but it is quite easy to find static builds that automatically include the latest library updates directly compiled from their github/gitlab repositories. So the svtav1 version included in those builds are the most updated ones according to the build date, which is nice.
There are many repositories storing this kind of builds, here is one that regularly updates not only ffmpeg including libsvtav1, but also SvtAvtEncApp among other binaries: https://jeremylee.sh/bins/
It may be helpful for windows users to include a link next to the requirements to meet the dependencies with downloadable binaries as an easier way than having to compile them yourself. Using Media Autobuild Suite seems the next easiest way to compile, which is a more automated and simple way of compiling these resources without needing to install cmake, microsoft c compilers, etc. on your own.
The only disadvantage of these builds is that they are compiled from the master branch, so they might not be as stable as some frozen release versions could be. I've had problems with some of those builds and ab-av1, maybe a bug was introduced, I have no idea, I'm just speculating, so I had to rollback and have not followed up.
As @georgemcarlson was explaining, there are not so many available premade/easy resources for mac, and there is also the added complexity between the intel x86 and apple silicon arm processors, so mac users might be forced to compile and would have more prerequisites to do so.

I've merged the change, as I think its the best way forward.
I think we could add some links in the readme to ffmpeg build locations for Linux, Windows & Mac to be helpful.


[2.0] Revisit init parameters

Right now init looks like this
function init(array $options = [], ?ClientBuilder $clientBuilder = null): void

as mentioned https://github.com/getsentry/sentry-php/pull/682#discussion_r233652663
this is a bit confusing.

One possible good solution that I see here would be reducing the scope of the ClientBuilder to a simpler TransportFactory, because that is the only proper dependency that needs to be handled outside of simple options.

As you said the serializer is another dependency that should not be handled as an option, so we still have the need of having a builder if we want to provide users with something they can quickly use to build up a client instance

I've put the serializer as an other dep of the client and as a property of the builder in #701