anchor free 疑问

您好，最近在深深入研究这项工作，也很感谢您的巨大贡献，请问yolox这种anchor free机制在loss计算的时候如何确保浅层特征预测小目标而深层特征预测大目标的呢？

It's simliar to FCOS, you could check get_assignment for more details.

You can just consider each grid as an "anchor" whose aspect ratio is 1 and area is downsample size. About  label assignment, use get_assignments

@violet2422 SimOTA会根据loss的匹配程度，对目标自动分层。关于这部分的分析，可以参考LLA


feat!: implement deterministic message hashing and use it for gossipsub message id

Ref #1208
TODO: Check the API around the seen cache key and ensure MUID is used for it.

Can only be merged once #1230 introduces the meta field.

Remove reviewers as PR is actually still draft

Now ready for review

TODO: Check the API around the seen cache key and ensure MUID is used for it.

Discussing with @jm-clius, we do not want to do the change on Relay at this stage. We will want to coordinate across implementations when we do it.


Load custom configs from $ZSH_CUSTOM before loading plugins defined in ~/.zshrc?

I'm new to oh-my-zsh. One of the first plugins I'm messing with is ssh-agent.
At first blush, my intuition is to keep the setting of zstyles - e.g. zstyle :omz:plugins:ssh-agent identities example-key - out of my main ~/.zshrc file, rather putting them into *.zsh files in my $ZSH_CUSTOM directory.
However, for that to work, I need the $ZSH/.oh-my-zsh.sh file to load those custom configs before loading plugins.
Am I stupidly breaking something by doing that? Or would that be a good idea for everyone - i.e. something I should submit as a PR?

Hey @rhbecker can you close this issue now that your use case is solved?

When you say "your use case is solved", are you referencing the workaround you suggested in your November 7th comment?

Yeah exactly. It's conceptually an aberration but I think it might work... unless the init script calls it again after sourcing oh-my-zsh. I don't know...
I think we merged quite recently a PR to allow using custom lib files, so you could have a particular $ZSH_CUSTOM/lib/something.zsh sourced before the rest. Take a look at the init script for more info.
In the end, I think none of these solutions is good enough. Maybe we could standardise some file in the custom directory to run before everything else.. like a pre-init.zsh file. But I'm not a 100% on that either. You might want to reconsider putting it in the zshrc file.
BTW your applied solution of loading custom files first breaks the use case of overriding plugin or libs settings (I'm talking aliases and such).
Sorry for the big reply :grin:

This was merged in #3617 / #3809. Custom configs are now loaded before plugins.
Could you close this issue?

Cool! I haven't confirmed this solves my issue yet, but I'll assume so and re-open if not. Thanks!


LazyRow in desktopMain is not scrollable

LazyRow in commonMain does not scroll on desktop. Works on Android
For reference LazyColumn works correctly on both platforms
Affected platforms

Desktop

Versions

Kotlin version*: 1.9.23
Compose Multiplatform version*: 1.6.10-beta01
OS version(s)* : Windows 11 23H2 build 22631.3447
OS architecture: x86
Device (model or simulator for iOS issues): n/a
JDK (for desktop issues): openjdk version "17.0.9" 2023-10-17 build -11185874

To Reproduce
Create a LazyRow:
LazyRow(userScrollEnabled = true)  {
itemsIndexed(msc) { index, _ ->
.....
}
Expected behavior
Row is scrollable in desktop with mouse wheel
Works in Android but not desktop

It's already discussed in #2528 and #2690
It's not about desktop vs android but about touch vs mouse.
Long story short: A mouse might have multiple wheels or it might be a 2-directional touchpad, so scrolling delta is a vector, but just a number. A regular mouse wheel produces only y-axis delta. To scroll horizontally, hold the shift button - it will invert delta to x-axis.
This is a common practice even outside Compose, but we're thinking how to improve it
Closing as duplicate

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


during saving icons,the wrong use of 'IconFactory.Save' save 7 images,So the final save icon is a icon file with only 16 pixels

Thank you for your report. However, this is not an issue; it has already been explained here: #1


bean defined in null, could not be registered, A bean with that name has already been defined in null and overriding is disabled.

我们鼓励使用英文，如果不能直接使用，可以使用翻译软件，您仍旧可以保留中文原文。
We recommend using English. If you are non-native English speaker, you can use the translation software.
Which Component
eg. Nacos Discovery, Sentinel
Describe what problem you have encountered
A clear and concise description of what you want to do.
Describe what information you have read
eg. I have read the reference doc of Sentinel
Componet i now use are Nacos 1.0.0,SpringCloud Alibaba 0.9.0,SpringCloud Green,Boot 2.1.4
i download package from offical site,and startp up it in cluster mode.i try to start my own spring cloud services in this service i code some feign remote services.and  i got error like this
2020-02-21 10:18:03.850  WARN 14952 --- [           main] o.m.s.mapper.ClassPathMapperScanner      : Skipping MapperFactoryBean with name 'sysSocialDetailsMapper' and 'lcloud.netplan.admin.mapper.SysSocialDetailsMapper' mapperInterface. Bean already defined with the same name!
2020-02-21 10:18:03.857  WARN 14952 --- [           main] ConfigServletWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.support.BeanDefinitionOverrideException: Invalid bean definition with name 'remoteAdminService.FeignClientSpecification' defined in null: Cannot register bean definition [Generic bean: class [org.springframework.cloud.openfeign.FeignClientSpecification]; scope=; abstract=false; lazyInit=false; autowireMode=0; dependencyCheck=0; autowireCandidate=true; primary=false; factoryBeanName=null; factoryMethodName=null; initMethodName=null; destroyMethodName=null] for bean 'remoteAdminService.FeignClientSpecification': There is already [Generic bean: class [org.springframework.cloud.openfeign.FeignClientSpecification]; scope=; abstract=false; lazyInit=false; autowireMode=0; dependencyCheck=0; autowireCandidate=true; primary=false; factoryBeanName=null; factoryMethodName=null; initMethodName=null; destroyMethodName=null] bound.
2020-02-21 10:18:03.867  INFO 14952 --- [           main] ConditionEvaluationReportLoggingListener :

Error starting ApplicationContext. To display the conditions report re-run your application with 'debug' enabled.
2020-02-21 10:18:03.873 ERROR 14952 --- [           main] o.s.b.d.LoggingFailureAnalysisReporter   :

***************************
APPLICATION FAILED TO START
***************************

Description:

The bean 'remoteAdminService.FeignClientSpecification', defined in null, could not be registered. A bean with that name has already been defined in null and overriding is disabled.

Action:

Consider renaming one of the beans or enabling overriding by setting spring.main.allow-bean-definition-overriding=true

i had try set AllowBeanDefinitionOverriding property in bootstrap.yml ,but during debug i found loadBeanDefinitionsFromRegistrars method will be called twice.so i found setAllowBeanDefinitionOverriding attribute will be true at  the first time,and in the second time it is false.
someone can help me？

How did you use it? pls put some code snippets


Cannot read property 'top' of undefined(…)

Hey,
inView('.someSelector').check();
inView.is(document.querySelector('.someSelector'));
both returns: Uncaught TypeError: Cannot read property 'top' of undefined(…)
I am using Chromium 53.0.2785.92 & latest in-view.min.js

Yup, you're right. That's related to a recent API change. Will be fixing this ASAP. Thanks!

That should be fixed in 0.4.1. Thanks again!

Seems to be happening again (^0.6.1)


Adding CI to module

Added

TravisCI configuration added and now passing.
badge added to docs. (will alter this to use TravisCI under terraform-aws-modules once reviewed)
permissions section now in the example readme.
placeholder shell script added for CI deployment. Eventually this should conditionally release to the registry when those APIs become available.

Changed

altered tf variable aws_region to region.
replaced hardcoding the region to instead use a random region as retrieved by an awscli docker container within CI.
example cert is now a regionally-specific resource enabling tests to run in various regions at once and not collide.
ruby version bump means Rhcl becomes rhcl.

Resolves https://github.com/terraform-aws-modules/terraform-aws-alb/issues/3 once merged.

This is the build of the PR: https://travis-ci.org/run-at-scale/terraform-aws-alb/builds/290279127
I suppose in this PR state, it's probably worth setting up Travis for this project. Doing that shortly.

Thanks for the travis permissions @antonbabenko but it looks like one more step is needed: making me admin of the repo so I can both do the registry publish and enable the TravisCI build.
I get the following within TraivsCI and can't enable due to perms:

I see the following in the terraform registry:

Unfortunately it looks like kitchen terraform doesn't fully support version 0.10.x just yet so I'm getting some spotty results. I'm going to disable nightly builds when this is up to avoid the flip flop of status due to incompatibility.

Done.

Hmm since the incoming branch is from a forked repo I can't run the build until this is merged. Doing that now.


Add devcontainer "postStartCommand" support to Codespaces

Please add support in Codespaces for the devcontainer postStartCommand config property https://code.visualstudio.com/docs/remote/devcontainerjson-reference

I believe this should be filed against Codespaces (https://github.com/MicrosoftDocs/vscodespaces), no?

@aeschli oh yup you're right, thanks! found https://github.com/MicrosoftDocs/vscodespaces/issues/403
it's kind of confusing what features belong to which project :sweat_smile:


bob build ignore file & dir targets of child tasks

Imagine you have this bob.yaml
  generate-api:
    ~
    target: |-
      server/rest-api/generated/types.gen.go
      server/rest-api/generated/server.gen.go
  build:
    input: *
    cmd: go build -o ./build/server main.go
    dependson:
      - generate-api
    target: /build/server

This would also add the targets from generate-api to the inputHash of the build-task.
Current State
Input hash is created multiple time. Once right after the  playbook is generated and in the case of a task having child dependencies it might also be recomputed + updating inputs from the filesystem. This is problematic for performance & correctness reasons.
readBobfile => computeInputs => aggregate =>  build => computeInputs => build
Target State:
InputHash computation should be able to be executed right before processing the task.
Reading inputs from the filesystem should be only done before the playbook (building) starts. Inputs should not be read from the filesystem during a build.
readBobfile => aggregate => playbook => computeInputs (filteredInputs()) => computeInputHash => build
Things TODO:

[x]  Write a test in which targets from a child task are ignored in the parent tasks input list.
[x] Add another member to the task to store child targets
[x] Ignore child targets in filterInputs()
[ ] Add a function to the playbook to initiate computeInputs()

Closed by #173


How to generate an UMD library?

I'm trying to create a library that can be used both in node.js and the browser, a standalone UMD library. With some puzzling with the configuration of package name and globals I managed to generate a bundle that can be used from node.js, but I can't manage to get the library working in a browser by exposing it on window. This is my configuration:
const { FuseBox, BabelPlugin } = require('fuse-box')

const fuse = FuseBox.init({
    homeDir: './src',
    output: './dist/$name.js',
    package: {
        name: 'myLibrary',
        main: 'src/index.js'
    },
    globals: {
        myLibrary: '*'
    },
    plugins: [
        BabelPlugin({
            presets: ['es2015']
        })
    ]
});

fuse.bundle('myLibrary')
    .instructions(`> index.js`)

fuse.run()

In node.js I can use the library like:
const myLibrary = require('./dist/myLibrary')
console.log(myLibrary.myFunction)

In the browser I try:
<!DOCTYPE html>
<html>
<head>
  <script src="./dist/myLibrary.js"></script>
</head>
<body>

<script>
    console.log(myLibrary.myFunction) // doesn't work
</script>

</body>
</html>

But myLibrary is not exposed on window. Any idea what I'm overlooking? Following the docs I expected the globals to do the trick.
I saw #522 which may be related, I tried<EMAIL_ADDRESS>but no luck.

Hi!
I think your main file is wrong
You homDir points to src
 main: 'src/index.js'

Could you try it without src?

Thanks for your fast reply. I tried main: 'index.js' and main: './index.js', both no success in the browser (and all work fine in node.js)

Could you copy paste the last bit of your bundle? It can find by searching for FuseBox.expose
I will check the functionality later on today, to verify it.

I think you mean this part of the bundle:
FuseBox.expose([{"alias":"*","pkg":"myLibrary/index.js"}]);
FuseBox.main("myLibrary/index.js");
FuseBox.defaultPackageName = "myLibrary";

@josdejong thanks a lot for your patience. I confirm the bug related to the recent change to the root
It points to an external FuseBox reference where all modules are stored. That will be fixed today/tomorrow.

That's good news, thanks for all your work!

@josdejong https://twitter.com/_developit/status/844260377546440705

@aretecode I don't use AMD myself but when publishing a library I think it's still relevant to have it working in AMD too.

@josdejong false alarm!
After an investigation, it turns out you are using globals wrongly.  By setting * you are actualy exposing every single exported object to window. To achieve want you want, please change * to any suitable name.
I have prepared a github example and updated the docs to make it clear.
I hope this will solve your problem

Changing myLibrary: '*' to myLibrary: myLibrary indeed solves the issue in the browser, but that gave a problem with node.js: you then have to load the library like const { myLibrary } = require('myLibrary') instead of const myLibrary = require('myLibrary'). I found the configuration I'm looking for by using default global and removing the package name:
const { FuseBox, BabelPlugin } = require('fuse-box')

const fuse = FuseBox.init({
    homeDir: './src',
    output: './dist/$name.js',
    package: {
        // !no package name, use default!
        main: 'index.js'
    },
    globals: {
        default: 'myLibrary'
    },
    plugins: [
        BabelPlugin({
            presets: ['es2015']
        })
    ]
});

fuse.bundle('myLibrary')
    .instructions(`> index.js`)

fuse.run()

Thanks for your help and for clarifying the docs! I still see the sentence "In order to expose your package, a bundle must have a package name", which apparently is not the case? I had to remove it to get my library exposed correctly.

@josdejong indeed that is not the case, I will fix that ;-)
Closing the issue now, feel free to re-open elsewise ;-)


cachekey: added --key-type to allow parent selection URL to be modified

Added ability in the cachekey plugin to apply all transformations, available for modifying
the cache key, to parent selection URL:

--key-type=cache_key - apply transformations to  cache key
--key-type=parent_selection_url - apply transformations to
parent selection URL

TODO/TBD: After this change all transformations can be applied not
only to the cache key but to parent selection URL as well. It would
make sense to give the cachekey plugin a new, more suitable name.

[approve ci autest]

[approve ci autest]

[approve ci autest]

[approve ci autest]

@jrushford, @alficles appreciate your comments.
Please let me know if there is more info needed.

Cherry-picked to v9.0.x branch.


"pattern not matched" error in Splunk connect logging pods

We have deployed Splunk Connect for Kubernetes on Openshift 4.6 using helm and are getting the below error in the logging pods.
Please let us know how to fix this.
2021-04-22 08:31:34 +0000 [warn]: #0 [containers.log] pattern not matched: "2021-04-22T08:31:17.647698673+00:00 stdout P 2021-04-22 08:31:17 +0000 [warn]: #0 [containers.log] pattern not matched: \"2021-04-22T08:31:08.789002238+00:00 stdout F \\\\\\\\\\\\\\\\\\\\\\\\\\\\\
Reference bug: https://github.com/fluent/fluentd-kubernetes-daemonset/issues/434
Environment:

Openshift 4.6.21
Kubernetes version v1.19
Splunk Connect for Kubernetes helm chart version: 1.4.7

Please share your helm values.yaml that you used to deploy SCK. Be sure to remove credentials.

helm values.yaml attached.[
values-yaml.txt
](url)

Please find the values.yaml attached.
values-yaml.txt

OpenShift 4.x logtype is cri not json. Change logFormatType to cri.
splunk-kubernetes-logging:
  containers:
    logFormatType: cri

Also have a look at the example:
https://github.com/splunk/splunk-connect-for-kubernetes/blob/develop/helm-chart/splunk-connect-for-kubernetes/examples/openshift4-logging-only.yaml

Thank you, that was it! Do you also know how to send logs only from 2 namespaces? I tried  to do this (see below) in the values.yaml. Is this the correct way to do it? I additionally annotated the namespace (oc annotate namespace kube-system splunk.com/index=k8s_events)
---
objects:
    core:
      v1:
        - name: pods
          namespace: kube-system, openshift-logging
        - name: namespaces
        - name: nodes
        - name: events
          mode: watch
---
objects:
    core:
      v1:
        - name: pods
          namespace: kube-system
        - name: namespaces
        - name: nodes
        - name: events
          mode: watch
---

objects are not about logs. objects are about sending information about currently existing k8s resources (AKA objects) to splunk. at least as far as I know. never used it myself yet.

Do you also know how to send logs only from 2 namespaces?

set the splunk.com/exclude=true annotation on all namespaces that you don't want to send log output to splunk.
https://github.com/splunk/splunk-connect-for-kubernetes#managing-sck-log-ingestion-by-using-annotations

Thank you @vinzent !

Hi I am using https://github.com/splunk/splunk-connect-for-kubernetes/tree/develop/manifests/splunk-kubernetes-logging to implement splunk on my AWS eks cluster version  1.21
This is my values.yaml
global:
licenseKey:
cluster:
kubeEvents:
enabled: false
webhook:
enabled: true
prometheus:
enabled: false
logging:
enabled: false
ksm:
enabled: true
Recently i updated my k8s version to 1.27 and the splunk stopped working and i am now getting the error
2023-09-12 13:01:50 +0000 [warn]: #0 [containers.log] pattern not matched: "2023-09-12T13:01:40.19427752Z stdout P 2023-09-12 13:01:40 +0000 [warn]: #0 [containers.log] pattern not matched: "2023-09-12T13:01:23.359524644Z stdout P 2023-09-12 13:01:23 +0000 [warn]: #0 [containers.log] pattern not matched: \"2023-09-12T13:01:03.253198428Z stdout F \\\\\\
can anyone help


Alternative type parameters for Mock

Currently, given function type F, a type of a mock of this function is Mock<Parameters<F>, ReturnType<F>>. Wouldn't it be more convenient to write Mock<F>?

Yup! Totally makes sense. mockFn already supports both signatures.

I don't think there's a way to support both signatures for Mock. (I mean, I tried declaration merging without success.) This would be a breaking change, right?

What if we introduce another type like MockFn that would take fn instead? I am not sure if having only function type variant is strictly better option that the current situation.

Released in https://github.com/dethcrypto/earl/releases/tag/earljs%400.1.11


flatpak-builder: 1.0.0 -> 1.0.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/flatpak-builder/versions.
meta.description for flatpak-builder is: '"Tool to build flatpaks from source"'.

Checks done (click to expand)

built on NixOS
/nix/store/h460y6c99l9595z4grkgx9gzqfqm5m7m-flatpak-builder-1.0.1/bin/flatpak-builder passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 1.0.1 with grep in /nix/store/h460y6c99l9595z4grkgx9gzqfqm5m7m-flatpak-builder-1.0.1
directory tree listing: https://gist.github.com/748740aab1d127f7a7731fbf5eaa1233
du listing: https://gist.github.com/641fb136fb79ad92e54e1371f4957ac3

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:
nix-shell -p cachix --run 'cachix use r-ryantm'

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/f78fa3f28846d2a9919684236f0f2c50b4ff75c1.tar.gz -p flatpak-builder
ls -la /nix/store/h460y6c99l9595z4grkgx9gzqfqm5m7m-flatpak-builder-1.0.1

cc @jtojnar for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: flatpak-builder
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: flatpak-builder
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/cpdrn730k4r3psb7d9jqmvfhlxn94qpa-flatpak-builder-1.0.1-doc
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/cpdrn730k4r3psb7d9jqmvfhlxn94qpa-flatpak-builder-1.0.1-doc
checking for references to /build in /nix/store/cpdrn730k4r3psb7d9jqmvfhlxn94qpa-flatpak-builder-1.0.1-doc...
shrinking RPATHs of ELF executables and libraries in /nix/store/p399iwp1bq4qhjknp9dj7rm3686i6y00-flatpak-builder-1.0.1-man
gzipping man pages under /nix/store/p399iwp1bq4qhjknp9dj7rm3686i6y00-flatpak-builder-1.0.1-man/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/p399iwp1bq4qhjknp9dj7rm3686i6y00-flatpak-builder-1.0.1-man
checking for references to /build in /nix/store/p399iwp1bq4qhjknp9dj7rm3686i6y00-flatpak-builder-1.0.1-man...
/nix/store/gwxjzsh9053sygdpmhwqaky2y52y03v5-flatpak-builder-1.0.1

Success on aarch64-linux (full log)
Attempted: flatpak-builder
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/jqa0g8d3iwwqjf3sfnvgd55dzimqzi3r-flatpak-builder-1.0.1-doc
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/jqa0g8d3iwwqjf3sfnvgd55dzimqzi3r-flatpak-builder-1.0.1-doc
checking for references to /build in /nix/store/jqa0g8d3iwwqjf3sfnvgd55dzimqzi3r-flatpak-builder-1.0.1-doc...
shrinking RPATHs of ELF executables and libraries in /nix/store/7zrdw1xw0r5ksy3318y0hzlhcnmkw8pq-flatpak-builder-1.0.1-man
gzipping man pages under /nix/store/7zrdw1xw0r5ksy3318y0hzlhcnmkw8pq-flatpak-builder-1.0.1-man/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/7zrdw1xw0r5ksy3318y0hzlhcnmkw8pq-flatpak-builder-1.0.1-man
checking for references to /build in /nix/store/7zrdw1xw0r5ksy3318y0hzlhcnmkw8pq-flatpak-builder-1.0.1-man...
/nix/store/zhfqmr6k28kn239zxscp6px7zvvj0zb8-flatpak-builder-1.0.1

https://github.com/flatpak/flatpak-builder/releases/tag/1.0.1


Only read unread samples count no callback has been set

Reading the count of unread samples should only be necessary in case there is now callback registered, as the count of unread samples is only read when a callback is set.
It turns out that couting the number of unread samples is a somewhat costly operation, so it should only be executed if necessary.

Ok, it seems, that this changes don't make any sense according to https://github.com/ros2/rmw_connextdds/pull/76/files#r832450207


Use metadata to structure analysis "SalgueiroPereiraMarie2020"

replaced by #3, since the PR should target the development branch, and this PR also includes additional notebooks


Allow users to update their Private Tangle

extends current script private_tangle.sh to allow to
stop services
update services
restart services
remove services' data

fixed in the Chrysalis version


Modify the absolute paths into relative paths

When we change the environment, the absolute paths are not consistant with the new workspace.

Modified


Incorrect word usage

The page has on-premise instead of on-premises.  There is no such word as on-premise.  It is always on-premises when referring to a location.  A premise is a belief.  Premises is one ore more locations.

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

ID: 49421ed5-6252-b24f-2983-ae58cea26df2
Version Independent ID: 0dd3e8b1-319c-1694-4cdd-bb050d0e85f5
Content: Azure Firewall forced tunneling
Content Source: articles/firewall/forced-tunneling.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@duanecr , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@duanecr , Appreciate your patience. The doc has been edited accordingly and it would take some time for the changes to get reflect over the doc.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Botania Integration

Issue description:
[Feature Request][Cross Mod][Long Term]
When you get around to trying RS with Botania, please test out using RS with the Botanical Brewery, Petal Apothecary, Runic Altar, etc..  Like many of the Botania "devices", they can easily get jammed up if you request multiple items rather than just one.
The issue is that the Botania devices require you to "drop" items onto them using blocks like Open Crates.  This means that there is no reliable way to tell that the "device" is done other than to check for the output product arriving back into the system.   That, in fact, is the solution -- provide a flow control method that gates the next crafting operation based on the product being returned from the machine.

Making RS "smart" enough to do that would kill the complexity (and fun) of botania (and in-world crafting in general).

I've already done this with AE2, but I had to add extra non-functional items to recipes and use them as "tokens" to ensure that each recipe completed before the next item request was sent.   That works, but it was huge pain and only Botania had this sort of non-sense.   That was NOT fun.

But it would destroy the whole "difficulty" of the mod (eg.
That's a mod pack developer choice OR a player choice of what game he wants to play.  Saying that type of game/gaming will "destroy" the mod is nonsense and frankly none-of-your-business.

whatevah, i'll do what i want

Of course you will, it’s your mod.  I wasn’t even replying to you with that last comment.  Apologies if you thought it was for you.
I was speaking to the other guy who thinks he should decide which mods are together in a mod pack, because they ‘destroy’ each other.
From: Raoul<EMAIL_ADDRESS>Sent: Thursday, October 06, 2016 2:13 PM
To: raoulvdberge/refinedstorage
Cc: TK Backman; Author
Subject: Re: [raoulvdberge/refinedstorage] Botania Integration (#421)
whatevah, i'll do what i want
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/raoulvdberge/refinedstorage/issues/421#issuecomment-252090345, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHlvxsSRqyXAMPMD0VoZZE3HK159jMO_ks5qxWRYgaJpZM4KOodJ.

 Also it is as much the modders choice as it is the player (or pack makers, who is just a player sharing his instance).
And for the record I'm not "just a gamer". I made a decent amount of contributions to RS and have been doing  MC modding for quite some time.

Vazkii made a design choice with Botania.  Most mods will not go out of their way to bypass something that is clearly intentional.
I'm pretty certain Vazkii has said before that Botania is fully automate-able with only Vanilla Minecraft and Botania.  It should be pretty darn simple to strap on something like RS or AE on top of that.  Spend some time figuring out how to do that, rather than complain that a mod hasn't added a "screw Vazkii's decision" option.
A lot of people out there find the in-world crafting of Botania to be an interesting challenge and quite fun to try to figure out.  If you don't find that fun, then clearly you are not part of the intended audience.

@Sorbe Please  show some respect to other people contributing to this issue. I'll be locking this issue for now, as I think that everything that has to be said has been said.


html sidebar item should be translatable

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Docusaurus offers the possibility to add a "sidebar-title" using the HTML type. However, there is currently no option for a simple translation.
{
type: 'html',
className: 'sidebar-title',
value: 'Placeholder',
defaultStyle: true,
},

The function itself works again with the latest Canary version, but it can only be used in one language and is limited if you offer docs in several languages.
Reproducible demo
No response
Steps to reproduce
Add the sidebar-title, build it and check it. It's only available in the hardcoded language or rather the value that is set.
Expected behavior
The sidebar-title should be available in other languages as well, so there should be some sort of support for this.
Actual behavior
The sidebar-title (value) is hardcoded and therefore the same for all languages.
Your environment

Public source code:
Public site URL:
Docusaurus version used: Canary
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

I'll consider it as a duplicate of https://github.com/facebook/docusaurus/issues/4542
Suggested workaround, not an ideal api but will work to translate html navbar items: https://github.com/facebook/docusaurus/issues/4542#issuecomment-1434839071

This works on our own website:

I just ran yarn start:website --locale de
If it doesn't work for you please create a runnable repro including the command you use to start in de lang

Runnable proof that it does: https://stackblitz.com/edit/github-zcvn3b?file=sidebars.js,docusaurus.config.js
Please create a minimal repro next time

Thank you, it seems our issue was that instead of export default sidebars; we still had module.exports = sidebars; at the end of the sidebars config, updating this has indeed resulted in the expected behaviour.


Tizonia not able to play youtube music video

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.11.07. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2018.11.07

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[x ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.11.07
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.11.07
[debug] Python version 3.6.5 (CPython) - Linux-4.18.15-1-pae-i686-with-glibc2.3.4
[debug] exe versions: ffmpeg 4.0.1, ffprobe 4.0.1
[debug] Proxy map: {}
Usage: youtube-dl [OPTIONS] URL [URL...]
youtube-dl: error: You must provide at least one URL.

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Using KDE on OpenSuse tumbleweed
youtube-dl to support Tizonia (version 0.15.0) which was installed as a snap
Here is error report when I attempt to load a music video
tizonia --youtube-audio-stream https://www.youtube.com/watch?v=8Pa9x9fZBtY
`tizonia 0.15.0. Copyright (C) 2018 Juan A. Rubio
This software is part of the Tizonia project <http://tizonia.org>

ERROR: Signature extraction failed: Traceback (most recent call last):
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1221, in _decrypt_signature
    video_id, player_url, s
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1130, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1193, in _parse_sig_js
    jscode, 'Initial JS player signature function name', group='sig')
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 978, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
 (caused by RegexNotFoundError(u'Unable to extract \x1b[0;34mInitial JS player signature function name\x1b[0m; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[YouTube] (IOError) : ERROR: Signature extraction failed: Traceback (most recent call last):
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1221, in _decrypt_signature
    video_id, player_url, s
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1130, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1193, in _parse_sig_js
    jscode, 'Initial JS player signature function name', group='sig')
  File "/snap/tizonia/211/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 978, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
 (caused by RegexNotFoundError(u'Unable to extract \x1b[0;34mInitial JS player signature function name\x1b[0m; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

tizonia exiting (OMX_ErrorInsufficientResources).

 [OMX.Aratelia.audio_source.http:port:0]
 [OMX_ErrorInsufficientResources]`

dstftw
I verified I am using version 2018.11.07
Please explain the outdated version label that you applied...
Thanks in advance...

You're using != tizonia is using. The label means exactly what it spells.


One pass availability reporting

replaced property 1password_extension_available with property one_password_extension_available for Viewed Login tracking event. Kept property 1password_extension_available for User Login event intact because it is marked as deprecated and will be removed..
Reference Trello card: Update 1password property

Nice Boris! Merge away 👍


Commenting on unchanged lines in a pull request

Summary
Review comments are the primary way reviewers provide feedback to the author and other reviewers during the review phase of a pull request, but currently comments can only be added to the 3 lines before or after a changed line. We intend to allow users to add comments and suggest changes on any line in any file changed in the pull request.

Intended Outcome
This will improve the review experience and increase review quality by allowing users to comment on lines that were not changed (but maybe should have been). Users will no longer need to add a comment to a random changed line and reference a line number when providing feedback about missing a change.
How will it work?
A users can add a comment, reply to a comment, or suggest changes to any line of a changed file, not just the 3 lines surrounding a change.

Please continue to refer to our updated Public Roadmap for the latest ships, including updates on the continuation of these projects. For more information, please check out this Discussion post.


Bump dbt version to 1.4

bump DBT version

something is not yet fully working:
dbt compile --target dev
15:45:48  Encountered an error:
'_MISSING_TYPE' object is not callable
15:45:48  Traceback (most recent call last):
  File "python3.10/site-packages/dbt/main.py", line 135, in main
    results, succeeded = handle_and_check(args)
  File "python3.10/site-packages/dbt/main.py", line 198, in handle_and_check
    task, res = run_from_args(parsed)
  File "python3.10/site-packages/dbt/main.py", line 225, in run_from_args
    task = parsed.cls.from_args(args=parsed)
  File "python3.10/site-packages/dbt/task/base.py", line 163, in from_args
    return super().from_args(args)
  File "python3.10/site-packages/dbt/task/base.py", line 98, in from_args
    config = cls.ConfigType.from_args(args)
  File "python3.10/site-packages/dbt/config/runtime.py", line 265, in from_args
    return cls.from_parts(
  File "python3.10/site-packages/dbt/config/runtime.py", line 83, in from_parts
    .get_default_quote_policy()
  File "python3.10/site-packages/dbt/adapters/base/relation.py", line 60, in get_default_quote_policy
    return cls._get_field_named("quote_policy").default_factory()
TypeError: '_MISSING_TYPE' object is not callable

https://github.com/dbt-labs/dbt-core/issues/6451

This issue is fixed now and it looks like dbt-impala is compatible with dbt 1.4.x now.

However, I still cannot use dbt-impala due to https://github.com/cloudera/dbt-impala/issues/146 (for both 1.3.1 and 1.4.0)

File "python3.10/site-packages/dbt/adapters/base/impl.py", line 691, in expand_target_column_types
    self.expand_column_types(from_relation, to_relation)
  File "python3.10/site-packages/dbt/adapters/sql/impl.py", line 98, in expand_column_types
    reference_columns = {c.name: c for c in self.get_columns_in_relation(goal)}
  File "python3.10/site-packages/dbt/adapters/impala/impl.py", line 163, in get_columns_in_relation
    columns = self.parse_describe_extended(relation, rows)
  File "python3.10/site-packages/dbt/adapters/impala/impl.py", line 187, in parse_describe_extended
    pos = ImpalaAdapter.find_table_information_separator(dict_rows)  # ensure that the class method is called
  File "python3.10/site-packages/dbt/adapters/impala/impl.py", line 219, in find_table_information_separator
    if row['name'].startswith('# Detailed Table Information'):
KeyError: 'name'

is still open for incremental models

After changing all references of row['name'] to row['col_name'] and row['type'] to row['data_type'] , most of these errors go away. But, the generated SQL code is weird:
strange output:
insert overwrite heilerg_db.example (foo, Detailed Table Information) partition(baz)
    select foo, Detailed Table Information
    from heilerg_db.example__dbt_tmp

most likely expected output:
insert overwrite heilerg_db.example (bar) partition(baz)
            select bar
            from heilerg_db.foo

input:
{{
    config(
        materialized='incremental',
        incremental_strategy = 'insert_overwrite',
        partition_by = 'baz',
    )
}}
{% set partition_key = '2023-01-01' %}

SELECT * FROM heilerg_db.foo

{% if is_incremental() %}

  -- this filter will only be applied on an incremental run
  WHERE baz = '{{ partition_key }}'

{% endif %}

Observations:

why is a non existing column foo copile into the select foo?
why is detailed table observations rendered into the SQL and killing the SQL?

@hpasumarthi can you take a look at this?

This is closed by https://github.com/cloudera/dbt-impala/pull/175


doesn't work on Fedora

[arothste@drewfus-systemmax ~]$ julia
               _
   _       _ _(_)_     |  A fresh approach to technical computing
  (_)     | (_) (_)    |  Documentation: http://docs.julialang.org
   _ _   _| |_  __ _   |  Type "?help" for help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 0.4.1 (2015-11-08 10:33 UTC)
 _/ |\__'_|_|_|\__'_|  |
|__/                   |  x86_64-redhat-linux

julia> Pkg.init()
INFO: Initializing package repository /home/arothste/.julia/v0.4
INFO: Cloning METADATA from git://github.com/JuliaLang/METADATA.jl

julia> Pkg.add("HttpServer")
INFO: Cloning cache of BinDeps from git://github.com/JuliaLang/BinDeps.jl.git
INFO: Cloning cache of Compat from git://github.com/JuliaLang/Compat.jl.git
INFO: Cloning cache of HttpCommon from git://github.com/JuliaWeb/HttpCommon.jl.git
....
julia> Pkg.test("HttpServer")
INFO: Computing test dependencies for HttpServer...
INFO: Cloning cache of BufferedStreams from git://github.com/BioJulia/BufferedStreams.jl.git
INFO: Cloning cache of Codecs from git://github.com/dcjones/Codecs.jl.git
INFO: Cloning cache of FactCheck from git://github.com/JuliaLang/FactCheck.jl.git
INFO: Cloning cache of JSON from git://github.com/JuliaLang/JSON.jl.git
INFO: Cloning cache of Libz from git://github.com/BioJulia/Libz.jl.git
INFO: Cloning cache of Requests from git://github.com/JuliaWeb/Requests.jl.git
INFO: Installing BufferedStreams v0.0.2
INFO: Installing Codecs v0.1.5
INFO: Installing FactCheck v0.4.2
INFO: Installing JSON v0.5.0
INFO: Installing Libz v0.0.2
INFO: Installing Requests v0.3.3
INFO: Building HttpParser
INFO: Building MbedTLS
INFO: Testing HttpServer
HttpServer utility functions:
  > `write` correctly writes data response
5 facts verified.
HttpServer runs
  > using HTTP protocol on <IP_ADDRESS>:8000
Listening on <IP_ADDRESS>:8000...
ERROR (unhandled task failure): ArgumentError: Array dimension must be 0 ≤ dim ≤<PHONE_NUMBER>854775807, got<PHONE_NUMBER>3709551615 for dimension 1
 in pointer_to_array at pointer.jl:41
 in on_body at /home/arothste/.julia/v0.4/HttpServer/src/RequestParser.jl:86

As an interesting data point:
julia> big(typemax(UInt64))
18446744073709551615

So thats probably related

(so, then the implication then is maybe something like a -1 is filtering through somehow from maybe httpparser? and getting stuffed into a UInt64)

Possibly bindeps is picking up on an old/incompatible version of httpparser?

Thats a good guess too.
@andrewrothstein, any way you could throw some debug printlns in there? Which Fedora version?

I'm using Fedora 21. If ur feeling courageous enough and have docker installed you can replicate/fiddle with my setup. The HttpParser module exhibits the problem so I'm happy to move this issue to that github repo if you'd like.
drew ~ > docker run --rm -ti andrewrothstein/docker-julia:fedora_21 julia -e 'Pkg.init(); Pkg.add("HttpParser"); Pkg.test("HttpParser");'
INFO: Initializing package repository /root/.julia/v0.4
INFO: Cloning METADATA from git://github.com/JuliaLang/METADATA.jl
INFO: Cloning cache of BinDeps from git://github.com/JuliaLang/BinDeps.jl.git
INFO: Cloning cache of Compat from git://github.com/JuliaLang/Compat.jl.git
INFO: Cloning cache of HttpCommon from git://github.com/JuliaWeb/HttpCommon.jl.git
INFO: Cloning cache of HttpParser from git://github.com/JuliaWeb/HttpParser.jl.git
INFO: Cloning cache of SHA from git://github.com/staticfloat/SHA.jl.git
INFO: Cloning cache of URIParser from git://github.com/JuliaWeb/URIParser.jl.git
INFO: Installing BinDeps v0.3.19
INFO: Installing Compat v0.7.8
INFO: Installing HttpCommon v0.2.4
INFO: Installing HttpParser v0.1.1
INFO: Installing SHA v0.1.2
INFO: Installing URIParser v0.1.1
INFO: Building HttpParser
INFO: Package database updated
INFO: Testing HttpParser
ERROR: LoadError: ArgumentError: Array dimension must be 0 ≤ dim ≤<PHONE_NUMBER>854775807, got<PHONE_NUMBER>3709551615 for dimension 1
 in pointer_to_array at pointer.jl:41
 in on_body at /root/.julia/v0.4/HttpParser/test/runtests.jl:93
while loading /root/.julia/v0.4/HttpParser/test/runtests.jl, in expression starting on line 129
=============================[ ERROR: HttpParser ]==============================

failed process: Process(`/usr/bin/julia --check-bounds=yes --code-coverage=none --color=no /root/.julia/v0.4/HttpParser/test/runtests.jl`, ProcessExited(1)) [1]

================================================================================
ERROR: HttpParser had test errors
 in test at pkg/entry.jl:803
 in anonymous at pkg/dir.jl:31
 in cd at file.jl:22
 in cd at pkg/dir.jl:31
 in test at pkg.jl:71

tried stepping through the code but the tracks sort of end at the julia<->native boundary.

added some println(s) per...
julia> Pkg.test("HttpParser")
INFO: Testing HttpParser
INFO: Recompiling stale cache file /root/.julia/lib/v0.4/HttpParser.ji for module HttpParser.
request: 'GET /favicon.ico HTTP/1.1
'
size: 27
request: 'Host: <IP_ADDRESS>=5000
'
size: 20
request: 'User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9) Gecko/2008061015 Firefox/3.0
'
size: 90
request: 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
'
size: 73
request: 'Accept-Language: en-us,en;q=0.5
'
size: 33
request: 'Accept-Encoding: gzip,deflate
'
size: 31
request: 'Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
'
size: 48
request: 'Keep-Alive: 300
'
size: 17
request: 'Connection: keep-alive
'
size: 24
request: '
'
size: 2
ERROR: LoadError: ArgumentError: Array dimension must be 0 ≤ dim ≤<PHONE_NUMBER>854775807, got<PHONE_NUMBER>3709551615 for dimension 1
 in pointer_to_array at pointer.jl:41
 in on_body at /root/.julia/v0.4/HttpParser/test/runtests.jl:93
while loading /root/.julia/v0.4/HttpParser/test/runtests.jl, in expression starting on line 129
===========================================================================================[ ERROR: HttpParser ]============================================================================================

failed process: Process(`/usr/bin/julia --check-bounds=yes --code-coverage=none --color=yes /root/.julia/v0.4/HttpParser/test/runtests.jl`, ProcessExited(1)) [1]

============================================================================================================================================================================================================
ERROR: HttpParser had test errors
 in test at pkg/entry.jl:803
 in anonymous at pkg/dir.jl:31
 in cd at file.jl:22
 in cd at pkg/dir.jl:31

My C mojo might be weak but type + flags is 9 bits != 1 byte. Not sure if there's some magical bit fiddling going into decoding the Parser data structure between C and Julia.
type Parser
    # parser + flag = Single byte
    type_and_flags::Cuchar

    state::Cuchar
    header_state::Cuchar
    index::Cuchar

    nread::UInt32
    content_length::UInt64

    http_major::Cushort
    http_minor::Cushort
    status_code::Cushort
    method::Cuchar

    # http_errno + upgrade = Single byte
    errno_and_upgrade::Cuchar

    data::Any
    id::Int
end

and
struct http_parser {
  /** PRIVATE **/
  unsigned int type : 2;         /* enum http_parser_type */
  unsigned int flags : 7;        /* F_* values from 'flags' enum; semi-public */
  unsigned int state : 7;        /* enum state from http_parser.c */
  unsigned int header_state : 8; /* enum header_state from http_parser.c */
  unsigned int index : 8;        /* index into current matcher */

  uint32_t nread;          /* # bytes read in various scenarios */
  uint64_t content_length; /* # bytes in body (0 if no Content-Length header) */

  /** READ-ONLY **/
  unsigned short http_major;
  unsigned short http_minor;
  unsigned int status_code : 16; /* responses only */
  unsigned int method : 8;       /* requests only */
  unsigned int http_errno : 7;

  /* 1 = Upgrade header was present and the parser has exited because of that.
   * 0 = No upgrade header present.
   * Should be checked when http_parser_execute() returns in addition to
   * error checking.
   */
  unsigned int upgrade : 1;

  /** PUBLIC **/
  void *data; /* A pointer to get hook to the "connection" or "socket" object */
};

That certainly looks very suspicious, I wonder why this is only arising now

The version of http_parser that comes with this package has
struct http_parser {
  /** PRIVATE **/
  unsigned char type : 2;     /* enum http_parser_type */
  unsigned char flags : 6;    /* F_* values from 'flags' enum; semi-public */

so is that the header for the version on your system?

I guess thats the most recent version, at least.
A fix might be just to do https://github.com/JuliaWeb/HttpParser.jl/pull/36/files, if its ready


Send query templates to the client, so we can pre-validate

This would be really nice. Obviously, validators can't be done this way since they require knowledge of the data in the database, but the query templates could be used to provide client validation.
A couple of questions:

Would we want to hide certain query templates from this feature?
Should we do this, maybe it doesn't help anyone enough to be worth the trouble

This sounds like it might not be worth the trouble to me.  It doesn't improve the error messages at all, and it's only a performance gain if queries that don't match templates are common, which they shouldn't be because they should be failing basically 100% of the time during development.  The additional complexity will also make it more likely that we'll do something incorrectly when people change their templates live.

I'm not sure about query templates. Can you share me the concept?


Format vec! macro using brackets

Closes https://github.com/nrc/rustfmt/issues/391.

Looks good. I wonder if longer term we can look at the macro definition to determine the kind of brackets used for macro uses? I guess that will be complicated if they are in different crates, etc.


issue #38: replace URLs in README.

This pull request replaces URLs (https://qualitativedatarepository.github.io/... to https://globaldataversecommunityconsortium.github.io/...) in README to prevent 404 errors in Dataverse. See more details in #38.

OK – The previewers should now be restored at their original locations and available at the new one as well. I’ll update the readmes to note the move, hot to do it via API, and will get a db update script in as well.
Thanks again Peter for the rapid feedback and fixes! Apologies to all for the mistake – I’ll be sure to think about the consequences w.r.t. anything tied to a repository in the future.
-- Jim
From: Király Péter<EMAIL_ADDRESS>Sent: Thursday, February 06, 2020 7:32 AM
To: GlobalDataverseCommunityConsortium/dataverse-previewers
Cc: Subscribed
Subject: [GlobalDataverseCommunityConsortium/dataverse-previewers] issue #38: replace URLs in README. (#39)
This pull request replaces URLs (https://qualitativedatarepository.github.io/... to https://globaldataversecommunityconsortium.github.io/...) in README to prevent 404 errors in Dataverse.

You can view, comment on, or merge this pull request online at:
https://github.com/GlobalDataverseCommunityConsortium/dataverse-previewers/pull/39
Commit Summary

issue #38: replace URLs in README.

File Changes

M README.mdhttps://github.com/GlobalDataverseCommunityConsortium/dataverse-previewers/pull/39/files#diff-0 (32)

Patch Links:

https://github.com/GlobalDataverseCommunityConsortium/dataverse-previewers/pull/39.patch
https://github.com/GlobalDataverseCommunityConsortium/dataverse-previewers/pull/39.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/GlobalDataverseCommunityConsortium/dataverse-previewers/pull/39?email_source=notifications&email_token=ABTLRTYQQWUWGUD7P46IXWLRBP7LPA5CNFSM4KQ4CPI2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILP5NVQ, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABTLRTYMFI7GMUWLQKVOGCLRBP7LPANCNFSM4KQ4CPIQ.


Consider supporting backwards compatiblity between ophys and pophys

A recent change made the previous tag ophys incompatible with the new pophys modality. In order to support previously serialized schemas, it would be nice to have some custom logic to coerce the previous definition to the current one.
This coercion mechanism should ideally make things compatible at the level of the python API (e.g. calling Modality.OPHYS) and at the level of deserialization. In both cases, the output should coerce to the new Modality.POPHYS object.

Using the new codegen, if I just add Modality.OPHYS pointing to the _Pophys() model is that sufficient to support this?

Naively I would say no. Assuming I understood what you suggested,
from a technical point of view, the abbreviation map would somewhat break since only the abbreviation of the last entry would be added
Modality.abbreviation_map = {m().abbreviation: m() for m in Modality.ALL}

At a more conceptual level, I also think that the biggest point is NOT making it compatible at the level of the python API but instead at the level of deserialization. In fact, I think we should slowly deprecate the use of OPHYS in the python API but support the deserialization of the OPHYS into POPHYS.
You can check the tests on the open PR to have a sense what cases I believe we should support. (https://github.com/AllenNeuralDynamics/aind-data-schema-models/pull/61/files#diff-a238cb580608af7c9b598c6b5bb3bacf973927b080ad7c8919e0ea104c1947cd)

Got it thanks for the clarification -- in that case it might be easier to just run an upgrader on all the old metadata and replace the variable wherever we can find it. Unless there are places that OPHYS was used outside of the metadata but that need to be deserialized into one of the metadata classes, then we do need to implement as you've done here.

Waiting on #94

After poking at this a bit I'm not sure this is possible, the discriminated Union forces you to rely on the abbreviation field and after an hour of messing with this I can't find a way to get it to deserialize properly. The issue is that the ONE_OF field only has a limited list of options. You can get around this by adding an additional class with abbreviation="ophys" and you can get it to pick up that class during deserialization, but then if you run a model_validator on top of that to swap ophys->pophys it fails because it already knows the type of the class at that point and it expects abbreviation: Literal["ophys"].
One way around this would be to implement the validators in classes that include a call to Modality.ONE_OF (rig, session, quality_control, and data_description). I think this isn't worth it when we can go back and replace old uses of "ophys" -> "pophys".
I'll open a ticket in aind-data-migration to migrate all previous uses of "ophys" -> "pophys" in the metadata.
If there's additional concerns with metadata that isn't stored in DocDB let me know


Use an upsert strategy instead of the current replace all for scores

This was a valid issue for the 2021 edition. Future core revamps make this irrelevant.


Type of return value from indexing

The current spec is silent on what the return value from an indexing operation should be: https://data-apis.github.io/array-api/latest/API_specification/indexing.html
I propose that the return value should always be another array object, i.e., so __getitem__ could be type annotated as __getitem__(self: Array, key: IndexKey) -> Array. However, this is inconsistent with NumPy, which returns NumPy scalars in some cases when indexing with integers.

Thanks @shoyer. I agree with that proposal. This is what most libraries do I think - PyTorch, JAX, TensorFlow, Dask and MXNet (from memory, will verify later), with only CuPy agreeing (?) with NumPy here.

Thanks @shoyer. I agree with that proposal. This is what most libraries do I think - PyTorch, JAX, TensorFlow, Dask and MXNet (from memory, will verify later), with only CuPy agreeing (?) with NumPy here.

CuPy always returns array too.

CuPy always returns array too.

Thanks Leo. I didn't have a GPU machine at hand and couldn't find it in the docs. Turns out it's documented after all: https://docs.cupy.dev/en/stable/reference/difference.html#zero-dimensional-array

Thanks Leo. I didn't have a GPU machine at hand and couldn't find it in the docs. Turns out it's documented after all: https://docs.cupy.dev/en/stable/reference/difference.html#zero-dimensional-array

This was fixed in gh-125, closing.


Added date selector accessibility

I don't see any problem with accessibility.
Can you explain please why we need use button instead of div ?

Hi wakirin
The scenario is when we are using embedded calendar view (where calendar input field is hidden), user should be able to navigate and select date via keyboard. Let me know if you have better suggestion to handle that. May be I am missing something.
Thanks
Sachin Singh

Hi @wakirin ,
I haven't heard from you in a while. Let me know if I need to update anything in the PR.
Thanks
Sachin Singh

Closing this PR


Addresses #726 obsolete valve cell type

edits to 'cardiac valve cell'
#gogoeditdiff

Related Uberon PR: https://github.com/obophenotype/uberon/pull/2730

add replaced_by: valve cell

Thanks for reviewing, @dosumis. I added "term replaced by 'cardiac valve cell'" to the three obsoleted classes.


Module not using job options?

Check if already reported
Couldn't see it in the issues list
Environment, versions (OS, Ansible, Rundeck, Plugin) etc.
Rundeck 2.8.4-1 cafe cubano purple gift 2017-07-12
Ubuntu 14.04.4 LTS
ansible-plugin-2.0.2.jar (manually dropped into the lib path)
How to reproduce
Sorry if this is dumb I'm really new to rundeck.
I'm trying to run an ansible playbook from rundeck and do it locally (i.e. not on a node - ansible already has all the details/keys etc it needs to call out and deploy).
I added job options to use my own inventory:
  sequence:
    commands:
    - configuration:
        ansible-become: 'false'
        ansible-playbook: /vagrant/ansible/playbook.yml
      nodeStep: false
      type: com.batix.rundeck.plugins.AnsiblePlaybookWorkflowStep

And it just seemed to be getting ignored by the plugin:
procArgs: [ansible-playbook, gather-hosts.yml, -vvvv, --extra-vars=@/tmp/rundeck/ansible-runner9148971650549642535extra-vars]

The contents of extra-vars:
facts: True
tmpdir: '/tmp/rundeck/ansible-hosts7801713943532825501'

I added some 'error' level logging to the module to try and figure it out - it seems like the options are getting passed through to the AnsibleModuleWorkflowStep.executeStep method:
2017-07-18 10:34:05,965 [Thread-80] ERROR com.batix.rundeck.core.AnsibleRunnerBuilder - {ansible-become=false, ansible-extra-param=, ansible-playbook=/vagrant/ansible/playbook.yml, ansible-debug=True, ansible-limit=localhost}
I did try adding the debugging option to /etc/rundeck/profiles (at the bottom) but I don't see much  more in the logs:
RDECK_JVM="-Djava.security.auth.login.config=$JAAS_CONF \
           -Dloginmodule.name=$LOGIN_MODULE \
           -Drdeck.config=$RDECK_CONFIG \
           -Drundeck.server.configDir=$RDECK_SERVER_CONFIG \
           -Dserver.datastore.path=$RDECK_SERVER_DATA/rundeck \
           -Drundeck.server.serverDir=$RDECK_INSTALL \
           -Drdeck.projects=$RDECK_PROJECTS \
           -Drdeck.runlogs=$RUNDECK_LOGDIR \
           -Drundeck.config.location=$RDECK_CONFIG_FILE \
           -Djava.io.tmpdir=$RUNDECK_TEMPDIR \
           -Drundeck.server.workDir=$RUNDECK_WORKDIR \
           -Dserver.http.port=$RDECK_HTTP_PORT \
           -Dansible.debug=true"

Here's the job definition including the options I'm trying to set:
- description: Deploy to the test environment from the test WordPress branch
  executionEnabled: true
  id: 3cb5599b-d5bd-4d20-a7a6-74a729248ab4
  loglevel: DEBUG
  name: Deploy Test
  nodeFilterEditable: false
  options:
  - name: ansible-inventory
    value: /vagrant/ansible/ansible_hosts_test
  - name: ansible-limit
    value: all
  scheduleEnabled: true
  sequence:
    commands:
    - configuration:
        ansible-become: 'false'
        ansible-playbook: /vagrant/ansible/playbook.yml
      nodeStep: false
      type: com.batix.rundeck.plugins.AnsiblePlaybookWorkflowStep
    keepgoing: false
    strategy: node-first
  uuid: 3cb5599b-d5bd-4d20-a7a6-74a729248ab4

Is there anything I can do to help diagnose this issue further?

The execution you are seeing is not of your playbook, but of the gather-hosts.yml which is an which runs as part of the resource process.  You need to add a workflow step which runs that playbook, which I don't see in your configuration file.
Closing due to inactivity.


Updated the MIME detection technique. Replaced the deprecated function...

... "mime_content_type" with "finfo" functions.

@sibtainnorain Hi thanks for Pull Requesting!
Tested succesfull.

New release available


Release 0.2.1

Could you make another release with the latest changes?
Sidenote:
I tried to build it myself, but could not run the executable built as shown below, probably line-feed problems in the code?
Error I got when running "docker run -it wp2reg:latest"
standard_init_linux.go:211: exec user process caused "no such file or directory"
Dockerfile I used:
#################
## Build
#################
FROM golang:1.16-buster AS build

RUN git clone https://github.com/hansmi/wp2reg-luxws.git
WORKDIR wp2reg-luxws/luxws-exporter
ENV GOOS=linux
ENV GOARCH=amd64
RUN go build -o /bin/luxws-exporter

#################
## Runtime
#################

FROM alpine:latest
RUN apk add bash
RUN apk --update add busybox-extras
RUN mkdir -p /run

COPY --from=build /bin/luxws-exporter /run/

EXPOSE 8111
CMD ["/run/luxws-exporter", "-verbose", "-controller.address", "<IP_ADDRESS>:8214", "-controller.language", "de", "-web.listen-address", ":8111"]

@dagerber, I released version 0.3.0 with the corresponding binary packages.
A local test with the following Dockerfile was successful:
FROM golang:1.18-buster AS build

RUN git clone https://github.com/hansmi/wp2reg-luxws.git
WORKDIR wp2reg-luxws/luxws-exporter
ENV GOOS=linux
ENV GOARCH=amd64
RUN go build -o /tmp/luxws-exporter

Command used:
docker build --rm --tag lebuild . && docker run lebuild /tmp/luxws-exporter -help

I also realized what the problem is with the ominous "no such file or directory". A plain go build produces an executable still linking against libc. Alpine uses musl, not glibc as Debian does. You either need to adjust the build flags for a fully static binary (there are many guides on the web) or use a runtime environment with the correct glibc version (e.g. Debian).

Thank you very much!
I now have your exporter running in a Docker container on my Synology NAS, along with Prometheus and Grafana.
Something else: I get the following output (Alpha Innotec, LW150A):
# TYPE luxws_up gauge
luxws_up{status="collection via LuxWS protocol failed: item with name \"Wärmemenge\" not found"} 0

Can I ignore it or should I open another issue?

@dagerber, please open another issue and include the relevant output produced with the -verbose flag.


[INFINITY-576]C*, kafka, hdfs, elastic, overlay network support and tests

[x] Cassandra overlay tests and documentation
[ ] Kafka overlay tests and documentation
[ ] HDFS overlay tests and documentation
[ ] Elastic overlay tests and documentation

retest this please

retest this please

retest this please (ccm)

retest this please

retest this please (unable to repro)

retest this please (marathon)

retest this please (ccm)

retest this please (xpack)

@loren has a fix out for the xpack flakiness: https://github.com/mesosphere/dcos-commons/pull/1131

@ArtRand Xpack fix was merged.

retest this please (xpack round 2)


[bug]: Minting New Batches of Grouped, Normal (Version 0) Assets when --decimal_display is Used

Background
I'm attempting to mint a subsequent batch of a normal, Version 0 asset where --decimal_display was used in the initial batches mint command. There was no meta_bytes associated with the original mint either.
Using the proper group key, it does not appear there is anyway for the next group of the asset, in a different batch, to inherit the --decimal_display setting of the initially minted group.
Your environment
Lightning Terminal integrated mode v0.13.993-exp
tapcli getinfo                       # version of `tapd`, `lnd`, and network

{
    "version":  "0.4.1-alpha commit=v0.4.2-0.20240815180811-2110839696cb",
    "lnd_version":  "0.18.0-beta",
    "network":  "regtest",
    "lnd_identity_pubkey":  "028d846445f80f8f53385b36f3d1217b3cc10d066221e7a95fc3a1823eb26ff0ac",
    "node_alias":  "alice",
    "block_height":  291,
    "block_hash":  "255f55c955c1392c4cfbfc94ad3bfcea7ad69d2e8d9763244eaa588fda6cb4cc",
    "sync_to_chain":  true
}

uname -mrsv                          # operating system

Linux 5.15.49-linuxkit-pr #1 SMP Thu May 25 07:17:40 UTC 2023 x86_64

bitcoind --version || btcd --version # version of `btcd`, `bitcoind`, or other backend

Polar default.

Steps to reproduce
(1) Mint the first batch, with the initial group of the asset
tapcli assets mint --type normal --name regtestCoin --supply<PHONE_NUMBER>00 --new_grouped_asset true --decimal_display 4

(2) Finalize the batch and mine some blocks.
(3) Attempt to mint the second group of the asset in a new batch. The error output indicates that the decimal_display should be 0 (incorrectly)
litd@alice:/$ tapcli assets mint --type normal --name regtestCoin --supply 100000000 --decimal_display 4 --grouped_asset --group_key 036c5e19efc591dd1e63f209b7431499bc82e5ffa7ba3
bea2f32f79282acfc362c --meta_type json --meta_bytes '{"decimal_display": 4}'

[tapcli] unable to mint asset: rpc error: code = Unknown desc = unable to mint asset: seedling decimal display does not match group anchor: 4, 0

(4) A second attempt to mint the second group, using a decimal_display of 0 works
litd@alice:/$ tapcli assets mint --type normal --name depixTestnet1 --supply 100000000 --grouped_asset --group_key 036c5e19efc591dd1e63f209b7431499bc82e5ffa7ba3bea2f32f79282acfc362c --meta_type json --meta_bytes '{"decimal_display": 0}'

{
    "pending_batch":  {
        "batch_key":  "034432ecf70ecac7396ac0fb2d95a09f0722798c60d149afe29423577de050bb85",
        "batch_txid":  "",
        "state":  "BATCH_STATE_PENDING",

        ...

Expected behavior
Expect step (3) from above to work and step (4) from above to return the error message returned from step (3).
Actual behavior
The output indicates that the daemon expects the new group to have a decimal_display of 0, potentially looking down a path of group_anchor rather than group_key.

Thanks for the report.
Just a hunch, but I think the group anchor isn't created correctly because there's a parameter too much. The command line library we use never takes true as an argument for a boolean flag. Just by specifying the flag itself you turn it on or set it to true (all boolean flag are by default false because of that).
So --new_grouped_asset true probably causes the parser to think the flags are done and interprets true as the first non-flag parameter (then ignoring --decimal_display 4).
Could you please test if that's the case? So just replace --new_grouped_asset true with --new_grouped_asset.

@guggero Thanks for the snappy response!
Your suggestion satisfied what I was trying to achieve.


Timezone - formatter - model->save()

in php.ini timezone is UTC. system timezone is UTC. yii defaultTimeZone is UTC. But my datetime attribute gets converted to my timezone "Asia/Kolkata" before saving into db.
Eg: UTC time 12:00Hrs
my input 17.30hrs
what I expect in db is 12:00hrs and in view 17.30hrs
But what I got in db is 17:30hrs and in view I got 23:00hrs.
web.php:
'formatter' =>
        [
			'class' => 'yii\i18n\Formatter',
			'dateFormat' => 'php:d-m-Y',
			'datetimeFormat' => 'php:d-m-Y H:i a',
			'timeFormat' => 'php:H:i A',
			'timeZone' => 'Asia/Kolkata',
		],

@cebe need to convert from user timezone to UTC. formatter does not do any conversion on beforeSave.

Looks similar with #9937

@sreenadh are you using datevalidator to validate the value?

@cebe yes

http://www.yiiframework.com/doc-2.0/yii-validators-datevalidator.html#$timeZone-detail

If this property is not set, yii\base\Application::$timeZone will be used.

which is UTC, so your input is assumed to be 17:30 UTC.

yii defaultTimeZone is UTC

do you mean Yii::$app->timeZone?

my input will be in Asia/Kolkata and I should get that value stored in db as UTC and show in view as Asia/Kolkata

then why did you set Yii::$app->timeZone to UTC? You have to either set Yii::$app->timeZone to your timezone or configure $timeZone property of the date validator to be Asia/Kolkata.

web.php
$config = [
    'id' => 'basic',
    'basePath' => dirname(__DIR__),
    'bootstrap' => ['log'],
    'timeZone' => 'Asia/Kolkata',
    'components' => [
		'formatter' =>
        [
			'class' => 'yii\i18n\Formatter',
			 'nullDisplay' => '',
			'dateFormat' => 'php:d-m-Y',
			'datetimeFormat' => 'php:d-m-Y h:i A',
			'timeFormat' => 'php:h:i A',
			'defaultTimeZone' => 'UTC',
			'timeZone' => 'Asia/Kolkata',
		],

in model
 public function rules()
    {
        return [
           [['created_datetime'], 'safe'],
            [['created_datetime'], 'date', 'format' => "php:d-m-Y h:i A" ],
        ];
    }
``

But Date time is saved in database as "0000-00-00 00:00:00" !


Implement POST "/oid/iid" as UPDATE instance

Implement POST "/oid/iid" as UPDATE instance according
OMA-DM-LightweightM2M-2015-0054R02-CR_Commands_Definition_Conflicts.
Signed-off-by: Achim Kraus<EMAIL_ADDRESS>

Sounds good.
We could also add some tests in leshan-integration-tests.
if we want to handle this correctly we should also add the code which manage UPDATE and REPLACE in ObjectEnabler

OK, I will add test (and remove whitespace :-) .

An other alternative for removing not longer contained resources in REPLACE may be the introduction of a "onInitialize" method in BaseObjectEnabler.

Tests is ok for me.
About REPLACE/UPDATE, I should have chosen the DELETE/CREATE.
I mean intuitively, It seems that REPLACE means delete the instance and replace it by this new one. (so this seems to be a creation)
With your solution we keep the previous instance.
The spec is really don't clear about this:
" - Replace: replaces the Object Instance or the Resource(s) with the new value provided in the “Write” operation."
But maybe you're right, your solution avoid some troubles:

e.g. On Server Object a delete/create means that we will lost the shortID
e.g. With mandatory single instance a delete/create will be problematic as a delete is forbidden (For this one we can choose to use the SYSTEM identity ...)

If someone else has an idea about that ?
In all case, if we chose your solution, I prefer we avoid the write(null) stuff and use the DeleteRequest on the resource.

I have to confess, that currently my "Objects Instances" are more "wired" and it would be very hard to change that right now to delete / create (I know, this means also no bootstrap :-) ).
I will go for a Delete on a resource

(Forget this ...

Why? Delete ("/o/i/r") works, may be a more radical then expected :-).

Using DELETE on resource level only for the SYSTEM and not for DM SERVER makes a dependecy to the PR #95.
It would be cleaner to use the newly introduce ExtendedIdentity.

After reanalysing, I choose the name reset() for cleanup the none mandatory writable resources in case of write / UPDATE.
For a more "far solution" I started thinking about something with

startWrite(update/replace)
finishWrite(success/rollback)
onDelete()
on InstanceEnabler base.
This introduce the possibility to support
"atomic instance writes" (with rollback)
collect changes for multi resources changes caused by instance writes
may protect reads for changing instances.
release resource, when the instance is deleted.

The issue for such a "far solution" would be, that it should have a defined
"Threading Model" (see Californium Resource.getExecutor() ). Otherwise
I would guess, that it may get to complicated.

Integrated in master (commit 37fb24f499269fcff4a7744407f5e7d4d343fd97)
Thx again @boaks.

WARNING:
The REPLACE is not proper working for Bootstrap!
It doesn't write the server ID in the server instance ...
I'm going to fix this.

:scream:

Yes, but usually a BS will first delete all security and server object instances.
So the REPLACE is a rare case for BS.


fix: Unskip and fix failing EC tests

Update the assertions to match the new conftest output produced by
https://github.com/hacbs-contract/ec-policies/pull/40 so the tests
pass.
Note: This requires that
https://github.com/redhat-appstudio/build-definitions/pull/124 is merged and
visible in the task bundle being used by the e2e tests. I'm not sure what the process is to make that happen.

Not sure what the sast-go failure is and what to do about it, and I can't see what the error message is. Advice appreciated.

/retest

/retest

/retest

/test redhat-appstudio-e2e

test case failed with error.
• [FAILED] [16.254 seconds] [chains-suite Tekton Chains E2E tests] /go/src/github.com/redhat-appstudio/e2e-tests/pkg/framework/describe.go:23   test creating and signing an image and task   /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:53     verify-enterprise-contract task     /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:128       [It] succeeds when policy is met       /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:164   Begin Captured GinkgoWriter Output >>     [BeforeAll] verify-enterprise-contract task       /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:133     [BeforeEach] verify-enterprise-contract task       /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:146     [It] succeeds when policy is met       /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:164   << End Captured GinkgoWriter Output   Expected       <[]v1beta1.TaskRunResult | len:0, cap:0>: nil   to contain elements       <[]*tekton.TaskRunResultMatcher | len:2, cap:2>: [           {               name: "OUTPUT",               value: nil,               jsonValue: <string>"[\n\t\t\t\t\t\t{\n\t\t\t\t\t\t\t\"filename\": \"/shared/ec-work-dir/input/input.json\",\n\t\t\t\t\t\t\t\"namespace\": \"release.main\",\n\t\t\t\t\t\t\t\"successes\": 2\n\t\t\t\t\t\t}\n\t\t\t\t\t]",               jsonMatcher: nil,           },           {name: "PASSED", value: "true", jsonValue: nil, jsonMatcher: nil},       ]   the missing elements were       <[]*tekton.TaskRunResultMatcher | len:2, cap:2>: [           {               name: "OUTPUT",               value: nil,               jsonValue: <string>"[\n\t\t\t\t\t\t{\n\t\t\t\t\t\t\t\"filename\": \"/shared/ec-work-dir/input/input.json\",\n\t\t\t\t\t\t\t\"namespace\": \"release.main\",\n\t\t\t\t\t\t\t\"successes\": 2\n\t\t\t\t\t\t}\n\t\t\t\t\t]",               jsonMatcher: nil,           },           {name: "PASSED", value: "true", jsonValue: nil, jsonMatcher: nil},       ]   In [It] at: /go/src/github.com/redhat-appstudio/e2e-tests/tests/build/chains.go:180

Added the new task bundle at https://github.com/hacbs-contract/ec-policies/pull/58 .

/retest

/lgtm

@flacatus Looks like we're green now. Could you give it a lgtm?

/lgtm

/approve


Reconsider using cd in shell scripts if avoidable

I find the scripts relying on cd fragile. I prefer that all scripts are executed under a constant PWD. There is almost always a way to do it such as mvn -f scala2.13
From @NvTimLiu:
We use cd scala2.13 in our build scripts/docs/CICD instead of maven -f scala2.13 as below everywhere,
do you think need we file a following up issue to the updating? @gerashegalov
https://github.com/NVIDIA/spark-rapids/tree/branch-24.10/scala2.13#building-with-scala-213
https://github.com/NVIDIA/spark-rapids/blob/branch-24.10/build/buildall#L236
https://github.com/NVIDIA/spark-rapids/blob/branch-24.10/.github/workflows/mvn-verify-check.yml#L307
https://github.com/NVIDIA/spark-rapids/blob/branch-24.10/jenkins/spark-nightly-build.sh#L29
https://github.com/NVIDIA/spark-rapids/blob/branch-24.10/jenkins/spark-premerge-build.sh#L194
Originally posted by @NvTimLiu in https://github.com/NVIDIA/spark-rapids/issues/11427#issuecomment-2335196874

@gerashegalov Thanks for help creating the issue
I'm working on v24.10, I suppose the issue is not impact v24.10 release,
Let me target it to 24.12, I'll handle it later,
Thanks!


#2191 MySQL encrypt test

Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 8022

0 of 6 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 60.719%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/extractor/impl/dml/PredicateExtractor.java
0
2
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/encrypt/dml/EncryptOrPredicateFiller.java
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/encrypt/dml/EncryptOrPredicateFiller.java
1
0.0%

Totals

Change from base Build 8019:
0.0%

Covered Lines:
8667

Relevant Lines:
14274

💛  - Coveralls


[Feature] Encapsulates the annotation position acquisition capability to support customized html annotations

What problem does this feature solve?
to enable custom marker
What does the proposed API look like?
markWithPoint(datum, content:html/jsx)

将获取数据位置与重绘时更新位置逻辑封装，支持用户通过api传入jsx或者html内容，实现自定义数据标注的能力

1.11.0支持了html和react


Any recent plan to update code?

Hi, @mlzxy
I've read your new draft and found that the code implementation is a little bit confusing. Is the code consistent with the new arxiv draft (which is uploaded on 7 March 2024)? If not, any plan to release them?

Yes, I will release a new version in October.

Look forward to that.


Python 3/2 compatibility

Most of the issues are related to map and filter returning iterables (instead of lists, as in Py2), the removal of reduce and tuples not being expanded on lambdas anymore.
I'll still do some review to avoid calling list in some places, but submitting to let @michaelbarton figure out how to run py3 on Circle =]

Thanks for your PR Luiz. I appreciate you making the effort to make this python
2/3 compatible. I believe this was also something @fungs was interested in too.
I took a quick look at why the build is failing on circleci. The failing
feature test is for the login command. Specifically a Docker image is created,
logged into, a couple of commands are run, then the output is inspected to
ensure the process was successfully logged in. The output looks different from
what is expected an that is what is causing the feature failure.
If you are able to debug that, that would be great. Otherwise no worries, what
you've already done is very useful and I will try to resolve this.

Hi @luizirber, I'm sorry for the double work but I had already fixed those parts in a local branch, replacing some of @michaelbarton 's functional programming paradigms to their pythonic equivalents. I just didn't have the time to finish and upload this branch.

@fungs that's fine, looking forward to use it =]
@michaelbarton closing this, since @fungs already have it covered.

Hey, I'm going to reopen this, @luizirber has a working version that we can merge in once it passes the login feature tests. @fungs lets discuss when you have your PR in.

I created issue #71 related to making the CLI python 3 compatible. I will close this PR as I see that it is a request to merge into master, where as we would instead create a new release branch following semantic versioning, and then merge this into master.

I added additional tests to insure the python 3 distribution installs. This along with luis's commits are on a feature branch feature/python-3-compatible


Preview versioning

1、add object tagging
2、add bucket tagging
3、add versioning
4、add bucket encryption

Coverage decreased (-3.2%) to 88.167% when pulling 2fe7728ce04078fd1ecbd10d08682c3ff6a35310 on preview_versioning into 08079eb9f6aaa13ee00b8fade6ceeffa138bf877 on master.


Navigator change

Broadcasting popRoute and pushRoute methods via SystemChannels.navigation. These messages will be used in flutter_web to detect the route
Description
In order to upstream WebOnlyNavigatorObserver in flutter web we would like to it's functionality with SystemChannel messages. There will also be another change which will alter: https://github.com/flutter/flutter_web/blob/master/packages/flutter_web_ui/lib/src/engine/window.dart#L101 to parse the new route and also deletes:
https://github.com/flutter/flutter_web/blob/master/packages/flutter_web/lib/src/widgets/web_navigator.dart altogether.
Related Issues
https://github.com/flutter/flutter/issues/34272
Tests
I added the following tests:
Replace this with a list of the tests that you added as part of this PR. A change in behaviour with no test covering it
will likely get reverted accidentally sooner or later. PRs must include tests for all changed/updated/fixed behaviors. See Test Coverage.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[ ] No, this is not a breaking change.

Also: Looks like the analyzer is unhappy: https://cirrus-ci.com/task/5980540199501824

Thanks for upstreaming this change to flutter!
Please add tests to make sure the correct platform messages are being sent. For reference, you can look at two places:

navigator_test.dart for examples on how to test the Navigator in general (this is probably where you want to add your tests).
title_test.dart for examples on how to test platform messages (look for setMockMethodCallHandler).

Thanks for the recommendations! I also added the tests now.

@goderbauer @mdebbar I addressed the comments! Can you please have another look?

It looks like the tests are failing for widgets with this change.  Can you take a look?
Also, in the future, please open a branch on your own fork of flutter and open pull requests from that fork.  See the https://github.com/flutter/flutter/wiki/Tree-hygiene page for more information.

Thanks for approving @yjbanov I accidentally closed this one.


Forecast.io is now DarkSky.net

Copied from an email to forecast.io registered accounts:

First, we’re consolidating our branding under the Dark Sky name, getting rid of Forecast.io entirely. We’ve found that people often don’t realize the two are related, and this causes a lot of confusion.

This means a few things for you as a developer:

The developer portal is now https://darksky.net/dev/.
The attribution requirement of the Terms of Service have changed. Attribution should:

Display “Powered by Dark Sky”
Link to https://darksky.net/poweredby/.
Please make this change at your earliest convenience!

The API endpoint has changed from https://api.forecast.io/ to https://api.darksky.net/.

Thanks for the heads up! It looks like the old API endpoint still works, so I'm going to wait and push out a major version update sometime this weekend including the new API endpoint. As far as names go, this project will still be called ForecastIO (unless there are serious objections).

 Update API endpoint


UNC_IDD forecast, 4 hours late

Thank you for the submission, we will accept it later than the deadline only for this submission period since we do not plan to score this week.

Thanks @Annabella-Hines !


Aggregation function window setting is ignored in monitoring check

Steps to reproduce:

create monitoring check with 1h schedule and 1m aggreation fn window for a metric that is reported frequently.

Expected behavior:
There should be about 60 statuses in the history per hour.
Actual behavior:
There is just 1 status per hour.
Environment info:
Influxdb 2.0-beta6

@alespour can you add some more detailed steps as to how to reproduce this so it can begin to be addressed? Thanks

@glinton sure

Have some metrics being uploaded to InfluxDB 2.x, eg. telegraf with default configuration will do.
Next create a threshold check

query any metric, eg. cpu measurement, field usage_user and leave the default aggregation setting on (ie. function mean with 1m window).
configure the check with 5m schedule

There should a check status per every aggregation window (1m) in the history, but there is just one per every check schedule (5m).
If I enable check query customization and open custom query script editor, I can see
check = {
  _check_id: "0591e88a4fe93000",
  _check_name: "Name this Check",
  _type: "custom",
  tags: {},
  every: 5m
}

option task = {
  name: "Name this Check",
  every: 5m, // expected to match check.every
  offset: 0s
}

...
data = from(bucket: "telegraf")
  ...
  |> aggregateWindow(every: check.every, fn: mean, createEmpty: false)

The 1m aggregation window settings is nowhere to be found in the script.


chore: suspend arb bots

Fix for https://github.com/Badger-Finance/devops/issues/27

Codecov Report

Merging #354 (fa2b16a) into develop (c2d8426) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #354   +/-   ##
========================================
  Coverage    43.03%   43.03%
========================================
  Files           23       23
  Lines         1645     1645
========================================
  Hits           708      708
  Misses         937      937


G4 events: Staking & Bonding

Bond Modal › Should Execute Successfully test is failing both locally and on CI with the introduction of this change.

Bond Modal › Should Execute Successfully test is failing both locally and on CI with the introduction of this change.

Fixed.


Add sort function to Transformer.afterParse sources prior to visitation

I just added a small change to the Transformer class, specifically to the afterParse method. I added a sorting function to ensure that user scripts are visited last during transformation. This is important because user scripts may reference other scripts in the project, and we want to ensure that those dependencies have been transformed first/all supporting classes encode statements have been populated.
Please let me know if you have any questions or concerns about this change.

Fixes: #43

I commented in #43


better equals/hashcode impls

DO NOT MERGE, YET
I created this PR to take a sneak peak on the diff. After finishing it, it can be merged.

Towards #1235

Current coverage is 95.97%

Merging #1242 into master will decrease coverage by -0.07% as of b7a3e29

@@            master   #1242   diff @@
======================================
  Files           83      83
  Stmts         9531    9507    -24
  Branches      1766    1759     -7
  Methods          0       0
======================================
- Hit           9154    9124    -30
- Partial         98     100     +2
- Missed         279     283     +4

Review entire Coverage Diff as of b7a3e29

Powered by Codecov. Updated on successful CI builds.


add enabled in values.schema.json

fix: https://github.com/open-telemetry/opentelemetry-helm-charts/issues/228

@JaredTan95 Please bump the patch version of the chart and run "make generate-examples".

@JaredTan95 Please bump the patch version of the chart and run "make generate-examples".

ok


No responses to HTTPS requests when using wrk with 64 or more connections

There seems to be an issue where no responses are sent if a large number of https requests are sent at the same time.
Repro Steps

Run Kestrel\samples\SampleApp on a public port (e.g. https://+:5001).
From a Linux machine, run wrk -c 1 -t 1 -d 1 https://server:5001.
Responses should be sent correctly.
Increase the client connections to 64 (wrk -c 64).
No responses will be sent.

We've sorted out what it was . Discussion here: https://github.com/dotnet/corefx/issues/14954


AIRSplitL2Memref: Disables L2 memref splitting for small designs

Disables the AIRSplitL2Memref pass if none of the air.segments need to map to more than one memtiles.
A new option clNumTilesPerL2Tile is added to pass in architecture information: the number of compute tiles with close affinity to one memtile.
For example, a column of 4 tiles have close affinity to 1 memtile on NPU.

The existing docs say:

Thanks for the comment. Agreed that the previous doc was vague and some parts of it are obsolete. I hope that the updated doc better describes the pass.

The existing docs say:

Thanks for the comment. Agreed that the previous doc was vague and some parts of it are obsolete. I hope that the updated doc better describes the pass.

It is much better, thanks!


colorCustomizations per folder on multi-root workspace

Working with multi-root workspace and it would really help to be able to customize the colors of each root folder. Specifically coloring the tabs differently, as there might be files with the same name on the different folders.
This is a re-iteration of #164368, I assume there may be more interest now that multi-root workspaces is probably used more often.  It's a pitty that the original ticket did not get upvoted sufficiently.
Hopefully more luck this time.

@aeschli  @TylerLeonhardt
You may want to upvote this request


DPI adjust restored window position

Raw cursor position should be DPI-adjusted before being used by WPF to position the restored window.
Issue #196

Awesome, thankyou!


ndt7: spec should clarify the closing protocol

The sane closing protocol is that the source (i.e. the server during the download) sends a CloseMessage when done, then the sink (i.e. the client during the download) finshes uploading its counter-flow measurements, if any, and then also sends a CloseMessage. This shall unblock the server reader and then terminate the test. Note that this also poses some constraints on how the code is written.
(Learned this the hard way when writing the reference Go client for ndt7.)
Well, actually the correct thing to do is to write code that is fault tolerant with respect to the closing protocol, and does not declare failure once the test is started. This is especially useful because either end should be able to RST the connection to stop the peer, as a last resort solution, and the test should still be declared successful in this case.
So, there is still need to document the closing protocol, but differently than originally planned.
[Edited on 29 August 2019]

This is basically another angle of #72, so I'm going to close this as duplicate.


Empty Screen Issue

Hi @jsuarezruiz
Thanks for the awesome work.
I have created 3 buttons in Figma and used the generated code as you guided in the YT video.
but got the below empty screen on android.

Figma design :

please advise where I am wrong.
thanks

Have you set the root element position at 0,0? https://github.com/jsuarezruiz/figma-to-maui-graphics#known-limitations-or-issues
Could you share the Figma Document to test it?

Hi @jsuarezruiz ,
Thanks for your reply.
Here it is,
https://www.figma.com/file/fywpf5xn7AfhSEpuUpLIhc/Untitled?node-id=3%3A13
please advise where I am wrong.

HI @jsuarezruiz,
I have step into further to create some more controls (login form) and this time I got the output below,

actual figma document :
https://www.figma.com/file/fdhhxqGnkF31TI7XYrPSR5/Untitled?node-id=0%3A1
Please advise what going wrong here

HI @jsuarezruiz,
I have step into further to create some more controls (login form) and this time I got the output below,

actual figma document : https://www.figma.com/file/fdhhxqGnkF31TI7XYrPSR5/Untitled?node-id=0%3A1
Please advise what going wrong here

https://github.com/jsuarezruiz/figma-to-maui-graphics/issues/2#issuecomment-1294637735
Probably you are missing images. Images are still not generated.


Typo in Ubuntu 18.04+ clean up instructions

Dear MicrosoftDocs,
Under the heading "Step 2: (Optional) Remove the Azure Linux Agent artifacts", there is a typo on the first line of the Ubuntu 18.04+ instructions:
sudo pt -y purge walinuxagent sudo cp -rp /var/lib/waagent /var/lib/waagent.bkp sudo rm -rf /var/lib/waagent sudo rm -f /var/log/waagent.log
The command "pt" should actually be "apt".
Sincerely,
Alex

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

ID: 77fb799a-68f0-119a-bac0-3ad65a5365a5
Version Independent ID: 253b4303-9ae5-2e77-f5c9-125afd2658d8
Content: Disable or remove the provisioning agent - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/disable-provisioning.md
Service: virtual-machines
Sub-service: imaging
GitHub Login: @danielsollondon
Microsoft Alias: danis

@Alex-W1
Thanks for your feedback! We will investigate and update as appropriate.

@Alex-W1
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


SMAConfigParser no attribute

after installing automator running manual.py gives the error
local@hedonismbot:/opt/Automator$ python manual.py Manual processor started. Python 64-bit 2.7.18 (default, Mar  8 2021, 13:02:45) [GCC 9.3.0]. Guessit version: 1.0.3. /usr/bin/python Python 2 is no longer officially supported. Use with caution. Loading config file /opt/Automator/config/autoProcess.ini. Traceback (most recent call last):   File "manual.py", line 430, in <module>     main()   File "manual.py", line 368, in main     settings = ReadSettings(logger=log)   File "/opt/Automator/resources/readsettings.py", line 591, in __init__     self.readConfig(config)   File "/opt/Automator/resources/readsettings.py", line 740, in readConfig     for key in config[section]: TypeError: 'SMAConfigParser' object has no attribute '__getitem__'

2bb656b98cb1e3ec6f9048807ca728bffb12a288
This is an error because Python 2 isn't supported anymore and this method isn't backwards compatible. Should probably upgrade to Python 3. It is however an easy fix to include backwards compatibility so the above change is a small patch that should fix this error and allow it to be Python 2 and 3 friendly
Long term though you're going to want to upgrade
Let me know if that fixes things and close the issue