Provide reasonable error when first argument to EnumField is not an Enum

Hello!  The error message didn't help :)
Modified example from front page:
from marshmallow import Schema
from enum import Enum
from marshmallow_enum import EnumField

class Something:
   foo = 1

class StopLight(Enum):
    green = 1
    yellow = 2
    red = 3

class TrafficStop(Schema):
    light_color = EnumField(Something)

schema = TrafficStop()

print(schema.dump({'light_color': StopLight.red}).data)
print(schema.load({'light_color': 'red'}).data)

Expected failure on schema instantiation:
RuntimeError: Received "Something" which is not of type Enum

Actually got failure on schema.load call:
{'light_color': 'red'}
Traceback (most recent call last):
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow_enum/__init__.py", line 95, in _deserialize_by_name
    return getattr(self.enum, value)
AttributeError: type object 'Something' has no attribute 'red'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "test.py", line 20, in <module>
    print(schema.load({'light_color': 'red'}).data)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow/schema.py", line 580, in load
    result, errors = self._do_load(data, many, partial=partial, postprocess=True)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow/schema.py", line 660, in _do_load
    index_errors=self.opts.index_errors,
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow/marshalling.py", line 295, in deserialize
    index=(index if index_errors else None)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow/marshalling.py", line 68, in call_and_store
    value = getter_func(data)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow/marshalling.py", line 288, in <lambda>
    data
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow/fields.py", line 265, in deserialize
    output = self._deserialize(value, attr, data)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow_enum/__init__.py", line 82, in _deserialize
    return self._deserialize_by_name(value, attr, data)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow_enum/__init__.py", line 97, in _deserialize_by_name
    self.fail('by_name', input=value, name=value)
  File "/opt/miniconda/envs/hermes-api/lib/python3.7/site-packages/marshmallow_enum/__init__.py", line 100, in fail
    kwargs['values'] = ', '.join([str(mem.value) for mem in self.enum])
TypeError: 'type' object is not iterable

Would definitely review a PR for this


Undefined index: QUERY_STRING

Hi,
When I am trying to test my code I get the error : Undefined index: QUERY_STRING.
Please change the line 87 in the FilterableTrait.php to:

I added the following to just before this block as a quick fix:
  if (!isset($_SERVER['QUERY_STRING'])) {
    $_SERVER['QUERY_STRING'] = '';
  }


Always show:Initializing ActionScript & MXML language server...

Always show:Initializing ActionScript & MXML language server...
and High CPU usage

When it never completes initialization, it means that something about your project is causing code intelligence to crash or hang. Would it be possible for you to share your project with me?
Additionally, can you give me some more information? Which operating system? Which version of Java? Thanks!

Thank you for reminding me.
I found the problem. Because there are too many files in the imported external class folder

@fireflyhy You may be able to fix that by increasing the memory available to the JVM when running the extension.
Take a look at the as3mxml.languageServer.jvmargs setting. Something like -Xmx1024m might be just what you need.


Problem in transition with 2 images

Transition doesn't behave correctly when there are only 2 images: the second image is displayed for too short time.

I solved the issue. Check this link
https://github.com/smarteist/android-image-slider/pull/6#issue-210848091

venky97vp solved this issue.
Dowload zip file from https://github.com/venky97vp/android-image-slider/tree/bugfix-twoimages and import into your project.

Hi @Nicolas2106 its much unclear, using com.github.smarteist:autoimageslider:1.2.0 the issue still exists.
Do you mean to manually manage the dependency by importing the zip from the provided link?


get the vmi status

hello
how can i custom define the output of oc get vmi
like following, we have 5 items: NAME,AGE,PHASE,IP and NODENAME, but i'd like to add more item.
[root@infra1 ~]# oc get vmi
NAME      AGE       PHASE     IP            NODENAME
ceshi     8m        Running   <IP_ADDRESS>   master1.sonic.com
in which code file i need to modify
thank you
majorin

Hi @majorinche, what you see there is configured by us server-side. You can change it on your running deployment if you like. When updating KubeVirt you will always have to perform that custom step though. To add more fields, do something like this:
kubectl edit customresourcedefinitions virtualmachineinstances.kubevirt.io

You can change this section so that it suits your needs:
  additionalPrinterColumns:
  - JSONPath: .metadata.creationTimestamp
    name: Age
    type: date
  - JSONPath: .status.phase
    name: Phase
    type: string
  - JSONPath: .status.interfaces[0].ipAddress
    name: IP
    type: string
  - JSONPath: .status.nodeName
    name: NodeName
    type: string

You can solve this also purely on the client side: https://kubernetes.io/docs/reference/kubectl/overview/#custom-columns.


truncate table during running sysbench insert，one tikv panic four times

Bug Report

What version of TiKV are you using?
TiKV
Release Version:   5.2.0
Edition:           Community
Git Commit Hash:   dbefa5ec66ccb4cb0cfeef3ad05d5932da2aeb36
Git Commit Branch: heads/refs/tags/v5.2.0
UTC Build Time:    2021-08-23 16:05:41
Rust Version:      rustc 1.56.0-nightly (2faabf579 2021-07-27)
Enable Features:   jemalloc mem-profiling portable sse protobuf-codec test-engines-rocksdb cloud-aws cloud-gcp
Profile:           dist_release
What operating system and CPU are you using?
k8s
2TiDB、3PD、4TiKV（4core、8G mem）
Steps to reproduce
TiKVWorkloadStress004
1、import data (32 tables)
/br restore full --pd tc-pd.glh-hqgsq.svc:2379 --storage s3://benchmark/sysbench-32-300G-release-4.0 --s3.endpoint http://minio.pingcap.net:9000 --send-credentials-to-tikv=true
2、run sysbench prepare
tabNum=512
threads=8
3、after 5min, truncate table which step1 created (32 tables)
What did you expect?
tikv is normal
What did happened?
one tikv（tikv0） panic four times（from monitor，the memory usage is not high）

/assign hicqu


[AutoDiff] Numerical accuracy of standard library derivatives

Describe the bug
Currently, there is a bug in the standard library that produces incorrect derivatives. This is the VJP derivative of FloatingPoint.addingProduct, as mentioned in #41423. That exposes a vulnerability in the floating-point differentiation test suite, which could allow for similar mistakes in the future. Perhaps someone could expand on my work-in-progress and revise the test suite to ensure that pullbacks always propagate. This would also catch numerical accuracy errors caused by the compiler itself, not just the Standard Library's Swift code.

CC: @BradLarson @asl

Resolved (see https://github.com/apple/swift/pull/41423#issuecomment-1133053762)


Incorrect spacing in deployment.yaml

https://github.com/localstack/helm-charts/blob/049c866100eb4d3637f259e8db53740eba3080e6/charts/localstack/templates/deployment.yaml#L84
This should be 14, or the other mounts should be indented by 2

@alexrashed Can I get push access on this repo so I can open PR?

If not, here is the corrected volumeMounts key in templates/deployment.yaml
          volumeMounts:
            {{- if and (.Values.mountDind.enabled) (.Values.mountDind.forceTLS) }}
            - name: dind-tls
              mountPath: /opt/docker/tls
            {{- end }}
            {{- if .Values.enableStartupScripts }}
            - name: {{ template "localstack.fullname" . }}-init-scripts-config
              mountPath: /docker-entrypoint-initaws.d
            {{- end }}
            {{- if .Values.persistence.enabled }}
            - name: data
              {{- if hasPrefix "0." .Values.image.tag }}
              mountPath: /tmp/localstack
              {{- else }}
              mountPath: /var/lib/localstack
              {{- end }}
            {{- end }}
            {{- with .Values.volumeMounts }}
            {{- toYaml . | nindent 12 }}
            {{- end }}
          {{- end }}
          ```

Hi @corinz!
Thanks for the hint and thanks for pointing us towards the fix!
We are generally not giving anyone push permissions to the repo (for obvious security reasons), but that's not necessary! 😃
The process of proposing changes to a repo in GitHub is usually to:

Fork the repo.
Create a branch in that repository for your fix.
Implement the fix in that branch.
Create a Pull Request from your fork's branch to the main branch of the "upstream" repository.

This is why pull requests are called pull requests (because you request that we pull your changes from your fork into our main repo / branch).

Here's a tutorial on how to create a PR from a fork: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork
Let me know if you want to give it a try. Otherwise I'm happy to take your suggested changes and create a PR.

@alexrashed Would you create the PR? The fail case is simple to test: enable persistence and add a custom volume like so
volumeMounts:
  - name: init-scripts
    mountPath: /etc/localstack/init/ready.d
    readOnly: true
persistence:
  enabled: true


Introduce "Locate Popup"

Goal
Introduce a new popup that enables the user to locate signals in the resource tree.
For this reason, we want to introduce a new popup that allows to configure criteria for signals. When the user configures a set of cirteria and clicks "apply", the resources that have attached signals that match these criteria are highlighted in the resource tree.
The visuals as well as some additional details are described in the following mock:

A first implementation is provided by #1935

This is completed.


Health data for Find in files/instant search

Closing as branched out from release. will retarget once release is merged back to master.


Update sbt-kubeyml to 0.3.9

Updates org.vaslabs.kube:sbt-kubeyml from 0.3.8 to 0.3.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.vaslabs.kube", artifactId = "sbt-kubeyml" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #641 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #641   +/-   ##
=======================================
  Coverage   81.38%   81.38%
=======================================
  Files         114      114
  Lines        3873     3873
  Branches       90       90
=======================================
  Hits         3152     3152
  Misses        721      721

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c6aea7...6680b1c. Read the comment docs.


switch Quick Fix fails on lack of curly braces

Dart SDK version: 2.19.0-444.4.beta (beta) (Tue Jan 10 10:24:44 2023 +0000) on "windows_x64"
When creating a switch statement and using the Quick Fix helper in VSCode, if curly braces are not included, the helper produces syntactically invalid code. Consider the following code:
enum ExampleEnum { one, two }

void example() {
  final ExampleEnum exampleEnum;

  switch (exampleEnum)
}

After running the Quick Fix, the code becomes:
enum ExampleEnum { one, two }

void example() {
  final ExampleEnum exampleEnum;

  switch (exampleEnum)

    case ExampleEnum.one:
      // TODO: Handle this case.
      break;
    case ExampleEnum.two:
      // TODO: Handle this case.
      break;
  }

It works fine with switch (exampleEnum) {}:
enum ExampleEnum { one, two }

void example() {
  final ExampleEnum exampleEnum;

  switch (exampleEnum) {
    case ExampleEnum.one:
      // TODO: Handle this case.
      break;
    case ExampleEnum.two:
      // TODO: Handle this case.
      break;
  }
}

Expected behavior is that either the Quick Fix adds the curly braces (preferred), or it doesn't show the suggestion until curly braces have been added manually by the programmer.

https://dart-review.googlesource.com/c/sdk/+/278913


No methods in my generated JAXB class. continuation of the issue https://github.com/highsource/maven-jaxb2-plugin/issues/91

Sorry for not including the screenshot for CustomerAccountType where i have the the below comments. I don't know how to do the property customizations using bindings.jaxb file.
/**
     * Gets the rest of the content model.
     *
     *

     * You are getting this "catch-all" property because of the following reason:
     * The field name "CustomerPurchase" is used by two different parts of a schema. See:
     * line 28 of file:/C:/00_PROJECTS/99_workspace/c360-canonical-models/src/main/xsd/com/schema/physical/customeraccount/v9_03/customeraccount.xsd
     * line 26 of file:/C:/00_PROJECTS/99_workspace/c360-canonical-models/src/main/xsd/com/schema/canonical/customeraccount/v9_03/customeraccount.xsd
     *

     * To get rid of this property, apply a property customization to one
     * of both of the following declarations to change their names:
     * Gets the value of the rest property.
     *
     *

     * This accessor method returns a reference to the live list,
     * not a snapshot. Therefore any modification you make to the
     * returned list will be present inside the JAXB object.
     * This is why there is not a set method for the rest property.
     *
     *

     * For example, to add a new item, do as follows:
     *

     *    getRest().add(newItem);
     *

     *

This has nothing to do with the maven-jaxb2-plugin. The Javadoc gives you a precise hint on why this is happenning as well as how to resolve this:
You are getting this "catch-all" property because of the following reason:
The field name "CustomerPurchase" is used by two different parts of a schema. See:
line 28 of file:/C:/00_PROJECTS/99_workspace/c360-canonical-models/src/main/xsd/com/schema/physical/customeraccount/v9_03/customeraccount.xsd
line 26 of file:/C:/00_PROJECTS/99_workspace/c360-canonical-models/src/main/xsd/com/schema/canonical/customeraccount/v9_03/customeraccount.xsd

You can use jaxb:property customization to rename one of the CustomerPurchase properties.
I don't see it in the scope of this project to provide pro-bono consulting on all the JAXB and XJC-related questions, sorry. Please consider asking on StackOverflow.


bug: images broken for some example notebooks on github

Describe the bug
Images not visible for below notebooks on github.

https://github.com/aws/amazon-braket-examples/blob/main/examples/quantum_annealing/Dwave_Anatomy.ipynb
https://github.com/aws/amazon-braket-examples/blob/main/examples/getting_started/1_Running_quantum_circuits_on_simulators.ipynb
https://github.com/aws/amazon-braket-examples/blob/main/examples/getting_started/2_Running_quantum_circuits_on_QPU_devices.ipynb
https://github.com/aws/amazon-braket-examples/blob/main/examples/advanced_circuits_algorithms/Simons_Algorithm/Simons_Algorithm.ipynb

To Reproduce
Steps to reproduce the behavior:

Go to examples folder in amazon-braket-examples repository
Click on particular example notebook i.e. ipynb file listed above.
Scroll down to view the embedded images.
Check if images are broken.

Expected behavior
We should see images for circuits/graphs properly.
Screenshots
If applicable, add screenshots to help explain your problem.
System information (please complete the following information as applicable):

Package Versions [e.g. amazon-braket-sdk==1.9.5]: N/A
Working Environment: [e.g. Local, Braket Notebook Instances]: Local/Github
Browser [e.g. chrome, safari]: Firefox
Region [e.g. us-west-2]: N/A

Additional context
Images are visible in notebook instance provided by Braket.

This is fixed with https://github.com/aws/amazon-braket-examples/pull/148, closing this issue.


How can I check Duration without make 2 requests?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Hi!
I'm using yt-dlp since beginning. Everything works fine but I have a problem. I make 2 requests

--get-title, get-duration (So if Video is longer than 60 Min I down download it because of Server Resource)
yt-dlp "YouTube-video"

My question is: Is it somehow possible to make yt-dlp https://www.youtube.com/watch?v=.... (if duration >60 min) don't download it - else download it.
So that I don't have to make 2 requests - because google will ban IP.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

yt-dlp --match-filter "duration>3600" <url>


implement futures Sink/Stream/AsyncRead/AsyncWrite semantics for webtransport datagrams

use the abstractions in futures to be able to transparently send and receive streams of bytes using webtransport datagrams under the hood
for example, ws_stream_wasm does this for websockets

nice!! are you planning on taking on this task?

someday :sweat_smile:

nah screw it


Feature request:  DHCP Dynamic DNS Key Algorithm

Hi,
after struggeling with DHCP Dynamic DNS updates I found out that OPNsense is only supporting the key algorithm hmac-md5. Therfore I would like to request adding a drop-down box allowing to select the key algorithm  hmac-md5 or hmac-sha512.
Thanks for the greate work!
Regards
Ingo

Closed. Merged with #2651


Users lack of account numbers

Every user should have a unique account (8 digits) in a way that every other user in the system could transact with this account number.

Fixed in de0c5b54e8c176868c377669a11f16d1da638620


Failed to generate vector assets

Hi,
I have installed the necessary packages.
yarn add react-native-vector-image @klarna/react-native-vector-drawable
I have added the following line to the React Native Build Phase
../node_modules/react-native-vector-image/strip_svgs.sh
When I run the yarn react-native-vector-image generate command, I get the following error
While trying to resolve module `nanoid/non-secure` from file<EMAIL_ADDRESS>the package `/Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/package.json` was successfully found. However, this package itself specifies a `main` module field that could not be resolved (`/Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/index.cjs`. Indeed, none of these files exist:

  * /Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/index.cjs(.native|.ios.js|.native.js|.js|.ios.json|.native.json|.json|.ios.ts|.native.ts|.ts|.ios.tsx|.native.tsx|.tsx)
  * /Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/index.cjs/index(.native|.ios.js|.native.js|.js|.ios.json|.native.json|.json|.ios.ts|.native.ts|.ts|.ios.tsx|.native.tsx|.tsx)
Error: While trying to resolve module `nanoid/non-secure` from file<EMAIL_ADDRESS>the package `/Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/package.json` was successfully found. However, this package itself specifies a `main` module field that could not be resolved (`/Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/index.cjs`. Indeed, none of these files exist:

  * /Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/index.cjs(.native|.ios.js|.native.js|.js|.ios.json|.native.json|.json|.ios.ts|.native.ts|.ts|.ios.tsx|.native.tsx|.tsx)
  * /Users/someuser/Documents/workspace/shop-mobile/node_modules/nanoid/non-secure/index.cjs/index(.native|.ios.js|.native.js|.js|.ios.json|.native.json|.json|.ios.ts|.native.ts|.ts|.ios.tsx|.native.tsx|.tsx)
    at DependencyGraph.resolveDependency (/Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/node-haste/DependencyGraph.js:436:17)
    at Object.resolve (/Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/lib/transformHelpers.js:317:42)
    at resolve (/Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/DeltaBundler/traverseDependencies.js:629:33)
    at /Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/DeltaBundler/traverseDependencies.js:645:26
    at Array.reduce (<anonymous>)
    at resolveDependencies (/Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/DeltaBundler/traverseDependencies.js:644:33)
    at /Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/DeltaBundler/traverseDependencies.js:329:33
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (/Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/DeltaBundler/traverseDependencies.js:137:24)
    at _next (/Users/someuser/Documents/workspace/shop-mobile/node_modules/metro/src/DeltaBundler/traverseDependencies.js:159:9)
Failed to generate vector assets
✨  Done in 9.03s.

What am I missing here? I'm using react-navigation package for screen navigation.

Hi, what version of react-native are you using and which package manager (npm/yarn) + version are you using?

I'm using the following
yarn: 1.22.4
react: 17.0.1 => 17.0.1
react-native: 0.64.1 => 0.64.1

Hmm, it's same as the Example app so should be fine. Can you post the contents of your metro.config.js?

Sure. Here is the metro.config.js
module.exports = {
  transformer: {
    getTransformOptions: async () => ({
      transform: {
        experimentalImportSupport: false,
        inlineRequires: true,
      },
    }),
  },
};

That looks good, and if you run yarn why metro?
Does the app run fine by itself otherwise? Any custom setups?

Anyways, I'm able to replicate this locally, so I'm fairly sure it's not your setup. Will dig deeper later!

I was also able to reproduce it with the Example app. The problem appears when I use react-navigation package.

Fixed in 0.3.2.


SW-4410 Add planting seasons to planting site API

Return the list of planting seasons as part of the planting site payload,
and update the site's planting seasons when the site is edited.

Current dependencies on/for this PR:

main

PR #1492

PR #1493

PR #1494

PR #1495

PR #1496

PR #1497

PR #1498

PR #1499   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 28, 10:17 AM: @sgrimm started a stack merge that includes this pull request via Graphite.


Test static assertions should show a meaningful message

Use BOOST_STATIC_ASSERT(e) or have your own BEAST_STATIC_ASSERT(e) that uses the string form of the expression 'e' as the message.

willfix


[stable/nginx-ingress] Apply default backend service command line argument if one is configured

Is this a new chart
No
What this PR does / why we need it:
PR #20297 changed the deployment behaviour of the chart if it is configured with:

.Values.defaultBackend.enabled=false
.Values.controller.defaultBackendService=my-namespace/my-default-backend-server

As a result, the controller no longer uses the default backend that is configured with the values mentioned above but the default 404 error page of nginx.
That configuration combination is useful if you want to deploy your own default backend independently of this chart and you don't want to configure the default-backend-service command line argument through .Values.controller.extraArgs
This PR restores the previous deployment behaviour while keeping the intent of PR #20297 intact.
Which issue this PR fixes
None
Special notes for your reviewer:
None
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test
Thanks for the pr!

/lgtm


Update munit to 0.7.28

Updates org.scalameta:munit from 0.7.22 to 0.7.28.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "munit" } ]

labels: test-library-update, semver-patch

Superseded by #142.


[Auto Bench] Integrate push-based CDN to scripts of auto benchmarking on AWS

Closes #2827

This PR:

Adds script of running benchmarks with push-based CDN together with a README on running locally
Does some minor changes to benchmark output structure and input parameters
Adds some benchmark results

This PR does not:

Key places to review:

Converted to draft for now as I have some more things to change...should be back to "ready for review" by EOD


Dynamic Bathroom Marker Color

Bathroom marker will have different background color depending on their average rating. The worst one will get a red background and the best one will have a green background.
Bathroom marker is also changed to circle and circle.fill, with custom coloring function that can adjust the saturation and brightness of a given color.

close #62

LGTM!


misc/test_softmax_sharded.py mismatch

Command to run:
pytest tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax_causal_mask[bfloat8_b-in0_DRAM-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax[float32-in0_DRAM-causal-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax[float32-in0_DRAM-no-causal-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax[bfloat8_b-in0_DRAM-causal-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax[bfloat8_b-in0_DRAM-no-causal-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_scale_mask_softmax_rm[float32-in0_DRAM-causal-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_scale_mask_softmax_rm[bfloat8_b-in0_DRAM-causal-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax_with_sharded_mask[float32-in0_DRAM-CM-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax_with_sharded_mask[float32-in0_DRAM-RM-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax_with_sharded_mask[bfloat8_b-in0_DRAM-CM-device_params0]
PASSED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_softmax_with_sharded_mask[bfloat8_b-in0_DRAM-RM-device_params0]
FAILED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_scale_mask_softmax_rm[float32-in0_DRAM-no-causal-device_params0] - AssertionError: FAILED: 0.5278782609430358
FAILED tests/tt_eager/python_api_testing/unit_testing/misc/test_softmax_sharded.py::test_scale_mask_softmax_rm[bfloat8_b-in0_DRAM-no-causal-device_params0] - AssertionError: FAILED: 0.5277648629045888

The compute kernel is ttnn/cpp/ttnn/operations/normalization/softmax/device/kernels/compute/softmax_sharded.cpp
I was able to reduce the test to a single tile input for bfloat8_b. The PCC improved to 0.8, but is still too low. The numbers in the comments can be used to get the original dimensions. The test still fails with bfloat16 as well although that's not tested in the test in the repo.
@ncvetkovicTT hopefully the LLK team will be able to take a look.
from loguru import logger
import torch
import pytest
import math

import ttnn

from tt_lib.utils import (
    pad_weight,
    tilize_to_list,
    untilize,
    is_close,
)
from models.utility_functions import print_diff_argmax, comp_pcc
from models.utility_functions import torch2tt_tensor, tt2torch_tensor, pad_by_zero
from models.utility_functions import is_grayskull, skip_for_blackhole

def test_scale_mask_softmax_rm(device):
    in_dtype = ttnn.bfloat8_b
    in0_mem_config = ttnn.MemoryConfig(ttnn.TensorMemoryLayout.INTERLEAVED, ttnn.BufferType.DRAM)
    causal_mask = False
    torch.manual_seed(0)
    N = 32 # 384

    fuse_head = 1

    grid_size = (1, 1) #(8, 7)
    batch = grid_size[1]
    num_cores_r = grid_size[0]
    input_shape = (batch, num_cores_r, fuse_head * N, N)

    hidden_dim = 1024
    num_heads = 16
    scale = 1 / math.sqrt(hidden_dim // num_heads)

    attention_mask = torch.rand(batch, 1, 1, N)
    attention_mask = (attention_mask > 0.5).float()
    attention_mask = attention_mask.reshape(batch, 1, -1, 32)
    attention_mask_t = ttnn.from_torch(
        attention_mask, dtype=ttnn.bfloat16, layout=ttnn.ROW_MAJOR_LAYOUT, device=device
    )

    input_tensor = torch.randn(input_shape).bfloat16().float()
    in1_t = ttnn.from_torch(
        input_tensor, dtype=in_dtype, layout=ttnn.TILE_LAYOUT, device=device, memory_config=in0_mem_config
    )
    grid_coord = ttnn.CoreCoord(grid_size[0] - 1, grid_size[1] - 1)
    shard_grid = ttnn.CoreRangeSet({ttnn.CoreRange(ttnn.CoreCoord(0, 0), grid_coord)})
    shard_shape = [fuse_head * N, N]
    shard_spec = ttnn.ShardSpec(shard_grid, shard_shape, ttnn.ShardOrientation.ROW_MAJOR, False)
    sharded_mem_config = ttnn.MemoryConfig(ttnn.TensorMemoryLayout.HEIGHT_SHARDED, ttnn.BufferType.L1, shard_spec)
    in1_t_shard = ttnn.to_memory_config(in1_t, sharded_mem_config)

    # subblock_size would need to be at most 4 for float32, but this test is only for bfloat8_b
    subblock_size = 1 #6
    block_size = 1 #12
    program_config = ttnn.SoftmaxShardedMultiCoreProgramConfig(
        compute_with_storage_grid_size=grid_size,
        subblock_w= subblock_size,
        block_h=block_size * fuse_head,
        block_w=block_size,
    )

    tt_output_sharded = ttnn.scale_mask_softmax_in_place(
        in1_t_shard, scale, attention_mask_t, program_config=program_config, is_causal_mask=causal_mask
    )
    print(f'in {in1_t_shard}\nout {tt_output_sharded}')

    tt_output_tensor = ttnn.to_layout(tt_output_sharded, ttnn.ROW_MAJOR_LAYOUT, memory_config=in0_mem_config)
    tt_output_tensor = ttnn.from_device(tt_output_tensor)
    tt_output_tensor = ttnn.to_torch(tt_output_tensor)

    attention_mask = attention_mask.reshape(batch, 1, 1, N)

    golden_output_tensor = input_tensor * scale + attention_mask
    golden_output_tensor = torch.softmax(golden_output_tensor, dim=-1)

    allclose, output = comp_pcc(
        tt_output_tensor,
        golden_output_tensor,
    )
    logger.info(output)
    assert allclose, f"FAILED: {output}"

Update: Was looking at a wrong reader kernel, the fix is in https://github.com/tenstorrent/tt-metal/pull/15993

Merged by https://github.com/tenstorrent/tt-metal/pull/15993.


updated requirements

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (81ea262) 0.00% compared to head (8e9da82) 0.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #41   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          6       6
  Lines        542     542
=====================================
  Misses       542     542

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


use an ActorProxy (subclass) instead of a method_missing definition

... on every Call.new
some micro-benchmarks (run against an older AHN version and JRuby 1.7.4): https://gist.github.com/kares/3576e272250204eb66d1
with 1000 calls we're getting ~ 37% improvement, while with 500 around 33%

Coverage remained the same at 97.38% when pulling 34d1941297320520f7ff17ba52bc56c1506ae9ae on kares:no-def-at-runtime into 897b66b088eeb578a13c3e77bb7ca30d189ed2fa on adhearsion:develop.

Awesome, thank you Karol! Would you mind, perhaps, adding a changelog entry with these details and squashing?

Merged at https://github.com/adhearsion/adhearsion/commit/d1896853d03cfab7196b6b6bd9628298751d1a4a

Thanks Ben, got stacked on other projects.


Add systemd-specific start/restart tasks/handlers for el7/infernalis

This makes the roles work on el7; the old sysv-initscript invocations no longer work. Unfortunately the "restart osd with systemd" handler is now a "restart everything on the node" handler at the moment (to fix this a list of OSD-ids would need to be generated somehow, there no longer is a "restart all osds" option like in SysV and Upstart).

If I need to restart osd or mon or cento7 with playbook, what would be the commands for this?

@rokka-n
For a monitor: systemctl restart<EMAIL_ADDRESS>For an OSD: systemctl restart<EMAIL_ADDRESS>


Add optional IP selection handler

Add an optional IP selection handler to the SDK client initialization step which accepts the request context and expects returning an IP.
The default handler, if no handler is specified by the user, will simply return the IP attached to the request object.
An additional handler should be available within the library, which looks into the X-Forwarded-For header and gets the first IP in the list if it exists, falling back to the IP attached to the request if the header isn't available. This is an implementation that users can optionally use instead of making their own.

Available in 1.1.0.


Can you update @azure-tools/azcopy-node with latest dependencies

Can you update the package to latest version of dependent packages and publish a new npm package. Its very old and refer to packages that are outdated.
Thanks

@khanaffan which dependent  packages are you referring to?

It is keytar that has been lock down in package.json to specific version. Newer version of electron use latest version of keytar.
The one in there only work with electron 5,6,7,8,9(beta) and not 10 or 11. We have electron 11.
This causes
Module did not self-register: '\\?\D:\bsw\imodeljs\common\temp\node_modules\.pnpm\registry.npmjs.org\keytar\5.6.0\node_modules\keytar\build\Release\keytar.node'.

Here is package json from azcopy-node. Also note that azcopy are also old at least 6 months old.
    "@azure-tools/azcopy-darwin": "10.5.0",
    "@azure-tools/azcopy-linux": "10.5.0",
    "@azure-tools/azcopy-win32": "10.5.0",
    "@azure-tools/azcopy-win64": "10.5.0",
    "@storage-explorer/macos-keychain": "1.0.0",
    "json-stream": "^1.0.0",
    "jsonwebtoken": "^8.4.0",
    "keytar": "5.6.0",
    "tslib": "^1.9.0",
    "uuid": "^3.3.2"

Thanks

@khanaffan Ah ok, yep. With regards to keytar and any other modules used to do the JWToken sharing between azcopy-node and the actual AzCopy executable, we think we're going to remove those dependencies and leave it up to the consumer to implement the toking sharing instead. So instead of upgrading the dependences we'll remove them.
Any not related to that, including the AzCopy exe, we will get those updated to newer versions.

That make sense. Is there any ETA on this change?
Thanks.

@khanaffan I'll be making this my priority tomorrow, but just incase something comes up, let's say ETA of by end of next week. I'll definitely keep you up to date on progress.

@khanaffan, this is what we'll be shooting for with in the next version with regards to dependencies. Let me known if anything looks off to you.
  "dependencies": {
    "json-stream": "^1.0.0",
    "jsonwebtoken": "^8.5.1",
    "tslib": "^2.1.0",
    "uuid": "^8.3.2"
  },
  "optionalDependencies": {
    "@azure-tools/azcopy-win32": "10.8.0",
    "@azure-tools/azcopy-win64": "10.8.0",
    "@azure-tools/azcopy-linux": "10.8.0",
    "@azure-tools/azcopy-darwin": "10.8.0"
  }

@MRayermannMSFT  Yes that look good.
Currently all azcopy binaries are installed on all platforms. I assume when you made them optional you also added os spec so only one of them get installed, right?

@khanaffan they are all already optional in that way? And they all already have an os spec?

You are right. I just check i was seeing win32 and win64 being installed. I thought it say something else. My bad.
Thanks

@MRayermannMSFT currently this package is referencing a non-existent optional dependency "@storage-explorer/macos-keychain"
this is causing failure to install using yarn for instance
can you please remove it?

@moro85 This is what @MRayermannMSFT doing if you read on the top. He is getting rid of those. The new package.json look like this. It has no keytar or keychain dependencies.
"dependencies": {
    "json-stream": "^1.0.0",
    "jsonwebtoken": "^8.5.1",
    "tslib": "^2.1.0",
    "uuid": "^8.3.2"
  },
  "optionalDependencies": {
    "@azure-tools/azcopy-win32": "10.8.0",
    "@azure-tools/azcopy-win64": "10.8.0",
    "@azure-tools/azcopy-linux": "10.8.0",
    "@azure-tools/azcopy-darwin": "10.8.0"
  }

@khanaffan we've been having signing issues today so I haven't been able to upload to NPM today. >.< Sorry for the delay.

@khanaffan and @moro85 azure-tools/azcopy-node version 2.0.0 has been published! Unfortunately the npmjs.com page hasn't updated yet but I can see it with npm view and I was able to install it:
$ npm view @azure-tools/azcopy-node

<EMAIL_ADDRESS>| MIT | deps: 8 | versions: 2
This package allows you to invoke [AzCopy v10](https://github.com/Azure/azure-storage-azcopy) from NodeJS.
https://github.com/Azure/azure-storage-azcopy#readme

dist
.tarball<EMAIL_ADDRESS>.shasum: ccadfa65bac0243d00bccd5df9bbb1549de28798
.integrity: sha512-HerDOjI+mYftnytOkanamDHZ/ZUCxbDrnfg7AoubKiiTtxEkShQYvzQ7CSNImpw6dKHJ/rlZsCPmfC4zdoLoNA==
.unpackedSize: 645.3 kB

dependencies:
@azure-tools/azcopy-darwin: 10.8.0 @azure-tools/azcopy-win32: 10.8.0  json-stream: ^1.0.0                tslib: ^2.1.0
@azure-tools/azcopy-linux: 10.8.0  @azure-tools/azcopy-win64: 10.8.0  jsonwebtoken: ^8.5.1               uuid: ^8.3.2

maintainers:
- azure-sdk<EMAIL_ADDRESS>
dist-tags:
latest: 2.0.0

published 9 minutes ago by azure-sdk<EMAIL_ADDRESS>
You can see the lower number of runtime dependencies and the usage of AzCopy 10.8.0. Thank you both for your patience!

The NPM page is now up-to-date as well.


portfolio: 0.70.4 -> 0.71.0

Description of changes
https://github.com/portfolio-performance/portfolio/releases/tag/0.71.0
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

We might to push up to https://github.com/portfolio-performance/portfolio/releases/tag/0.71.1

env -i ldd is happy


Update README.md

Additions to describe how to use the custom data modal

The "encode" jsons are special hacks,  I don't think we should reference them in examples.   In fact I think we should remove this hack and use our new configurable data modal instead.

The "encode" jsons are special hacks,  I don't think we should reference them in examples.   In fact I think we should remove this hack and use our new configurable data modal instead.


Add IP reuse e2e tests

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Superseded by #905
/close


Unnecessary use of Webdriver in tests

Why You use Webdriver in tests. Because of this tests cannot be executed after download of this project just like that. For me it will be enough if You use in tests simple asserts just to show that mocha works with allure.
For few days I am looking for simple example with use of feature, story and steps, because it doesn't work in my project, but your mocha-allure-example it is not what should be - helps new developer to start working with allure reports :/

Well, thank you for the feedback!
I have updated examples, now there is a simple test file that doesn't contain any logic, only some essentials. Also, the rest is also now will be much easier to set up. Just run npm install && npm test.
I hope it helps!

Great job! Thanks 👍


rav1e assumes 4:0:0 is monochrome

header.rs contains:
let monochrome = seq.chroma_sampling == ChromaSampling::Cs400;

I don't think is is true, and it causes failure when trying to feed the encoder grayscale videos.

In AV1 spec, monochrome == grayscale. Could you elaborate on the failure?

Or do you mean using a monochrome color space with 4:2:0?

Oh, I see, OK. The failure is just unimplemented! (which doesnt so nice when calling from C).
I thought this was a bug since monchrome usually doesn't mean grayscale... (I was trying to use 4:0:0).


Add defaults and fix reference bug

Changes in the update -

Fixed reference bug in column-formatting schema that showed an error in VS2019
Added some defaults for improved authoring experience

@octogonz Can you please help by merging this?

@asnow003 Can you please help by merging this?

@microsoft/json-schemas-admins Can someone please help by merging this ?

Reviewed and merged, next time @ me and I'll hop on it sooner! Thanks, Linda


LM Studio Support

Hi,
Great work you've done!
I hope you can add LM Studio support for local LLMs since LM Studio has many AI models and more support than Ollama.
Regards,

Fixed in #5


[FLINK-28137] Introduce SpeculativeScheduler

What is the purpose of the change
This PR adds a SpeculativeScheduler which is in charge of speculative execution.
It is based on the work of #20221.
Verifying this change

Added unit test SimpleExecutionSlotAllocatorTest and SpeculativeSchedulerTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

Thanks for reviewing! @reswqa @wanglijie95
Merging.


Extension of the use cases to classify a tileset with a B3DM tileset

Extension of the use cases to classify a tileset with a B3DM tileset.
Removed restrictions:

Only one node is supported for classification and it must have a mesh.
Only one mesh is supported when using b3dm for classification.
Only one primitive per mesh is supported when using b3dm for classification.

Updated restrictions:

The mesh primitive must have a position attribute.
The mesh primitive must have a batch id attribute

Thanks for the pull request @ppoulainpro!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Thanks Pascal! Do you have a Sandcastle example or test case to help the reviewer test out these changes?

@OmarShehata. I need to do some checks with Draco based B3DM. I have detected some issue at update that I don't have when I don't apply Draco at tiling level.
I will upload after an example that will look like to:

Hi @OmarShehata, I have uploaded an update with Draco support and an example to test it.
It should be ok now.

Replaced by #8316


[BUG] autoAssign: false is ignored.

Describe the bug
setting todo.autoAssign: false is not working, is always assign the generetaed issue.
To Reproduce
todo.autoAssign: false
do a Pr with a todo
and then merge it.
Expected behavior
expected that the generated issue is assigned to none.

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
This issue has been automatically marked as stale because it has not had recent activity.


instancetype: Introduce PreferredInterfaceMasquerade

/area instancetype
/hold
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

TBD

/unhold

/approve

/retest-required

/retest-required


Add a license to allow others to build upon the codebase

I'd like to fork and extend the codebase (I'm happy to collaborate, too) to create an educational piece of content using Twilio SMS and Hookdeck to handle inbound webhooks and demonstrate live voting via SMS (and potentially email using SendGrid). My hope is that the content will be published on the Twilio blog (I'm in touch, but I haven't shared a specific idea yet).
However, without a license in the repo, the default applies:

without a license, the default copyright laws apply, meaning that you retain all rights to your source code and no one may reproduce, distribute, or create derivative works from your work

From https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/licensing-a-repository
Would you be willing to add a license to allow others to build upon this codebase?

Great suggestion, I added it and also push the new migration file as well. You can start the project easily.

@Chensokheng - thanks very much!
The folks at Twilio like the idea, so I'm going to fork the repo and do a proof of concept (PoC) that adds:

Phone number registration and verification
SMS voting

Once the PoC has been done, I'll get an agreement for a tutorial outline with Twilio. I'll keep you updated on progress.

@leggetter Great! I am looking forward to that.


Rule-idea:space-around-operator

https://github.com/brigade/scss-lint/blob/master/lib/scss_lint/linter/README.md#spacearoundoperator

Thanks! I guess this could be useful for someone, so I'll add this to the next milestone.

Perhaps called calculation-no-unspaced-operator as "calculation" seems to be a thing in SCSS) so it can be used in tandem with stylelint's core rule

@jeddy sure, no problem.
As about the rule. I am goint to make it exactly a -no- rule, meaning prohibiting spaces around operators seems to be no-go. At least because $var--lol is a valid variable name in Sass, and if we prohibit spaces we will force users to write $var - 10px as $var-10px, which would lead to a totally incorrect result.
I'd also save ourselves an effort distinguishing calc from all the other functions and would rather check operators inside calc() here as well.

I am goint to make it exactly a -no- rule

I agree. Well reasoned.

i think best options: "always" and "never".

@evilebottnawi with "never" users won't be able to write:

$var - 10px; they will be forced to write it as $var-10px, which Sass interprets as a variable name (like in $var-10px: #faaac0;)
font-family: sans- + "serif";; they will have to do it as font-family: sans - + "serif"; which is compiled into font-family: sans-+"serif";

@dryoma oops, i forget ;) right rule must have no

Some more things I'd like to bring up:

Make operator rather than calculation the thing here, as the rule would work with conditions and logical operations (@if ($i == 10 and $j != 1)) as well.

Users (me included) would want control over where newlines are allowed: before or after an operator. Yet I think it's best to do it in a way of eslint's operator-linebreak. I.e. not forcing linebreaks on every instance of an operator, but just specifying where they are allowed (before of after the operator) if a used wants to set them. Also, no distinguishing on whether it is a multiline operation (just like function-calc-no-unspaced-operator doesn't do it).
So I'm thinking of a set of rules:

operator-no-unspaced - require a single whitespace on both sides of an operator;
operator-no-empty-line-before - disallow empty line before an operator;
operator-no-empty-line-after - disallow empty line after an operator;

Does each rule need to address indentation itself?

Nope, indentation should be the sole domain of the indentation rule.
Originally the indentation rule just ignored the indentation within function arguments and other parentheses. Now there is an option to check. We putting it behind an option to avoid requiring a major release. With 7.0 on the horizon this behaviour might change.

Make operator rather than calculation the thing here

That looks elegant and well-reasoned to me.

Wow, totally missed those options, thanks!

Published with 1.3.0.


No badge is shown on the non-compact Titlebar even though Option is set

Describe the bug
When not using compact menu and search bar option, the titlebar badge is not displayed.
To Reproduce

Have installed: Microsoft Visual Studio Enterprise 2022 (64-bit) - Preview
Version 17.7.0 Preview 2.0
Install version 1.1.44 of Solution Colors by Mads Kristensen
Open Tools...Options..Environment...General
Ensure Use compact menu and search bar is unselected.
Open Tools...Options...Environment...Fonts and Colors...Solution Colors
See that Show title bar is defaulted to True.
Press OK.
Restart if necessary to apply compact menu change from step 4.

Expected behavior
I expect to see the solution name as a badge on the titlebar, just as I do for [PREVIEW | ADMIN]
Screenshots

Additional context
Microsoft Visual Studio Enterprise 2022 (64-bit) - Preview
Version 17.7.0 Preview 2.0

I just did some tests, and if I understand your issue correctly, it's due to VS, not to the extension.
When VS is is non-compact mode, there is no badge for the solution name, because that name is displayed to the left of the title bar.


Fix: update disklessset.spec with required dependencies and paths

The SPEC file of disklessset.spec needed some work so that the RPM worked as intended.
This update ensures that the disklessset RPM creates the same files as the disklesss role in bluebanquise/roles/core.
The following features were tested:

Add a new kernel
Create initramfs for kernel
Create livenet image
Mount/unmount livenet image

Dear @ginomcevoy
Thanks a lot for this fix ! I did the spec file on a non fresh node, so I probably miss many things.


[bug] gazebo setup error with colcon_release

Bug
After run ./colcon_release , when try to
$ rosrun vehicle_gazebo_simulation_launcher setup.sh

as in https://github.com/CPFL/Autoware/tree/develop/ros/src/simulation/gazebo_simulator, it fails with the following errors.
Install osrf_citysim models
cp: cannot stat '/home/horibe/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher/scripts/../../../worlds/external/osrf_citysim/models/*': No such file or directory
Install car_demo models
cp: cannot stat '/home/horibe/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher/scripts/../../../worlds/external/car_demo/car_demo/models/*': No such file or directory

cp: cannot stat '/home/horibe/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher/scripts/../../../worlds/external/osrf_citysim/worlds/citysim_gazebo7.world': No such file or directory
cp: cannot stat '/home/horibe/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher/scripts/../../../worlds/external/osrf_citysim/worlds/citysim_gazebo9.world': No such file or directory
cp: cannot stat '/home/horibe/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher/scripts/../../../worlds/external/car_demo/car_demo/worlds/mcity.world': No such file or directory
cp: cannot stat '/home/horibe/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher/scripts/../../../worlds/external/car_demo/car_demo/worlds/mcity_r1.world': No such file or directory

I couldn't start gazebo simulation. This works correctly with ./catkin_make_release.
what I did is
$ git checkout develop
$ rosdep update
$ rosdep install -y --from-paths src --ignore-src --rosdistro $ROS_DISTRO
$ git submodule update --init --recursive
$ ./colcon_release
$ source ./install/setup.bash
$  cd ~/workspace/Autoware/ros/install/vehicle_gazebo_simulation_launcher/share/vehicle_gazebo_simulation_launcher
$ chmod +x download_model.sh integrate_world_files.sh setup.sh upgrade_gazebo_stable_version.sh
$ rosrun vehicle_gazebo_simulation_launcher setup.sh

Please wait a moment because I am trying to reproduce it


Index Error while starting arduino platform

Home Assistant release (hass --version):
0.42.4
Python release (python3 --version):
Python 3.4.2
Component/platform:
Arduino
Description of problem:
While starting HA i have unexpected exception and arduino platform can not be used:
17-04-18 23:25:31 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform arduino
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/helpers/entity_component.py", line 153, in _async_setup_platform
    entity_platform.schedule_add_entities, discovery_info
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/sensor/arduino.py", line 48, in setup_platform
    sensors.append(ArduinoSensor(pin.get(CONF_NAME), pinnum, CONF_TYPE))
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/sensor/arduino.py", line 63, in __init__
    arduino.BOARD.set_mode(self._pin, self.direction, self.pin_type)
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/arduino.py", line 72, in set_mode
    self._board.ANALOG)
  File "/home/pi/.homeassistant/deps/PyMata/pymata.py", line 743, in set_pin_mode
    self.INPUT
IndexError: list index out of range

Expected:
I should be able to see a button in HA dashboard.
Problem-relevant configuration.yaml entries and steps to reproduce:
arduino:
  port: /dev/ttyUSB0

sensor:
  - platform: arduino
    pins:
      22:
        name: Arduino Mega switch
  - platform: yr

According the description you want to use that pin 22 as digital output aka as a switch but you are using the sensor implementation instead of the switch platform.
arduino:
  port: /dev/ttyUSB0

switch:
  - platform: arduino
    pins:
      22:
        name: Arduino Mega switch


Identify Selected Value Point

How to identify which value is being selected

look at the "vals" variable
var slider = new rSlider({
target: '#slider',
values: [2008, 2009, 2010, 2011, 2012, 2013, 2014, 2015],
range: true,
set: [2010, 2013],
onChange: function (vals) {
console.log(vals);
}
});


[Feature Request] per_user / per_usergroup metric

Hey,
we need to get the run duration of each instance per user/usergroup. So that we can report the sum of total duration in minutes per user / month.
Can you report such a metric? E.g. anka_instance_state_per_user or anka_instance_state_per_usergroup
Thanks

Hi @dennispost , what do you mean by user/usergroup? Can you give me an example?

Hey @NorseGaud , my idea was to calculate the sum of every minute where a user has an anka_instance in running_state.
So that we can report the sum of all running minutes per month.
I know that this does work in Enterprise Plus license tier only.

Do you mean the users of a certificate using certificate authentication? What sort of authentication method are you using?

Got it -- thanks for the clarification.
In order to achieve this, the gitlab runner executor we created needs to post metadata with all of the available/relevant information that you want available in the exporter. This is because the exporter is a consumer of the API JSON from the Controller. If you start an instance with specific metadata (see API docs), it will be available on the instance object.
While we could modify both the runner executor to do this and then set up the exporter to consume it, this creates a situation where a feature of the exporter will only work if you're using the gitlab runner. I'm not sure if our dev team wants to accomplish it this way. I'll have a conversation with them about it and see what they recommend.

The runner and Controller are only able to communicate over the Controller's API (when it goes to request or terminate a VM). It only receives and sees what we pass into the API by the runner anka executor code.


Cloud firestore 0.10.0 FieldValue.increment not working

I got a problem with concurrent transactions.
I'm using flutter with cloud firestore plugin version 0.10.0
I got a collection called counters with a document named test. Inside this document i got a field called counter ( number type).
This counter is increased pushing a button from different devices, that's when I understand that a transaction like this makes sense (I attach the piece of code I'm using).
DocumentReference counterRef = Firestore.instance.document("counters/test");

new FlatButton(
                            color: Colors.green,
                            child: new Text("ADD"),
                            onPressed: () {
                              Firestore.instance.runTransaction((Transaction tx) async {
                                DocumentSnapshot Ref = await tx.get(counterRef);
                                if(Ref.exists){
                                  print("Updating counter!");
                                  await tx.update(Ref , <String, dynamic>{
                                    'counter': FieldValue.increment(1),
                                  }).catchError((e){
                                    print("tx update err:$e");
                                  });
                                }else{
                                  print("No document found in transaction!");
                                }
                              });
                            },
                          ),

Using only 2 physical devices, hitting the button at once on both, the counter is only increased 1 time. And i can see on the logs that each device has called the transaction ( i see the Updating counter print on the console )
Can anyone check FieldValue.increment functionality from 2 devices at the same time?

I didn't try my code on multiple devices but I'm running an asynchronous function incrementing a value which is called multiple times in a short period of time and I don't have any issues.
I believe that your mistake is wrapping the update in a transaction. I read through the commited code from the latest commit, which added the FieldValue.increment(num) and I believe that it guarantees an atomic increment on the server-side of Firestore, which means that you don't need a client-side transaction to safely increment your value.
Example method:
void incrementCounter(int num) async {
    Firestore.instance.
            collection('your_collection').
            document('your_document).
            updateData(<String, dynamic> {
      'your_value': FieldValue.increment(num),
    });
}

What is also worth noting is that you can use increment() with a negative number and thereby decrement a value.

Thanks for the advice, when I have a moment I'll try it. The problem only happens if the 2 transactions are executed in less than 1 second between each one. If they are executed with more time everything works correctly.
You can test it yourself from several devices at once, so we can confirm that something is wrong with the transactions.
About the counters, take a look on this links
https://firebase.google.com/docs/firestore/solutions/counters
https://firebase.google.com/docs/firestore/manage-data/transactions

I can confirm that transactions are unreliable. I previously used them (also on multiple devices) and increments would sometimes not be registered at all (if the transactions are in a short period of time).
However, this has already been mentioned exactly 2 months ago in #28714, so I'm not sure what can be done about that.

It would be good to know if the problem is on the firestore plugin or is a more serious problem with transactions in firebase.

I'm guessing that it's a problem with the cloud_firestore plugin. It's still in development and there is a chance somebody missed a detail in the implementation of a transaction.
It's unlikely for Firestore to have a bug since it's already in production (and not in a beta) and there are countless other working implementations of transactions for different languages and frameworks.

Also i can confirm that transactions on firestore are working fine. I deployed 2 cloud functions to perform a transaction on the same field. ( Just increasing a counter )
9:25:31.704 a. m. test Function execution started
9:25:32.710 a. m. test2 Function execution started
9:25:36.663 a. m. test Transaction success!
9:25:39.392 a. m. test2 Transaction success!
It's just a bug in the cloud firestore flutter plugin.

You can modify a document in firestore only once per second. So FieldValue.increment() won't update the counter twice if it is called on the same document more than once within a period of one second (see firebase documentation).
Since you are using transaction, you can first read the latest value of the counter from the server and then increment the counter manually using Transaction.update() (that's what transaction are made for: set of read and write operations which are atomic in nature).
Note: Transactions will still not overcome the firestore's inherent limitation of not being able to update a document more than once per second. But one of the advantage of using transactions for incrementing counters rather than FieldValue.increment is that transactions are retried on abortion automatically (see firebase documentation for transactions), therefore you might not see the updated counter value instantly but they will get registered after a short while once the failed/aborted transaction gets successfully retried.

Dear MohammedMaaz, the issue is related to the transactions behavior in flutter.  Transactions are unreliable using flutter.

@kataklysmo I suspect the issue was resolved with https://github.com/flutter/plugins/pull/1780. How's it with cloud_firestore 0.12.9+5?

Hi @kataklysmo
for what I can see the issue is related to a FlutterFire plugin,
please open the issue in the dedicated github.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
thank you


Date wrong for next launch (Starlink 9)

Next launch date is currently showing May 31, 2020, but it looks like the Starlink 9 expected launch date is actually June 22, 2020 at 6:20am EDT.

Updated, thanks!


fixes #2323 summernote field extra options

Fixes #2323 - you could not have multiple Summernote editors on one page, with different options

Merged into master. It'll be available with composer update later today, as 4.0.30, after a few more non-breaking changes get merged.


Can you please provide me code snippet for saving a video?

Please help me with saving a video stream to a file.

StackOverflow for support


Wine windows render as invisible or still frames with borders

Please describe the bug
Certain Wine windows do not render their content, and instead show a still frame of other content visible on the workspace. Window borders are still rendered. Can also disappear entirely at times; whilst still being visible as running processes.
Steps to reproduce:
I don't expect you to install League of Legends, but as I do not run other Wine applications I don't know how widespread this is, but these issues do not exist in dwm, Gnome or Openbox for me, on the same system, and should therefore be Hypr-related rather than a problem with the Wine installation. Regardless, to reproduce:
For Windows rendering a still frame with a border

Install League of Legends, for example via Lutris or this repository.
Open the client. During this process you should see 1-2 wine-related windows render as a still frame showing what's behind it with a border around it (make sure to have borders enabled in Hypr).

For Windows disappearing entirely
After following the previous steps queue up for a match and swap to a different workspace (leaving the client where it was).
When the queue pops the client will disappear if you're not on the same workspace. If you are on the same workspace as the client it will work as expected.
The client will also disappear after a match is complete, rather than showing the post-game lobby. The process is still running, but it is either not rendered at all or hidden somewhere, requiring killing the wine process for it to reappear (will of course happen next match and so on).
Expected behavior
Windows rendering their content.
Screenshots
This is a screenshot showing the desktop wallpaper behind a Wine window rather than its content. Notice the window borders are still rendered.

This is what it should look like:

Anything else?
Different states like floating or tiled or specified size/position rules do not make a difference.
Possibly useful xprop of the window pictured above:
WM_STATE(WM_STATE):
		window state: Normal
		icon window: 0x0
_NET_WM_DESKTOP(CARDINAL) = 2
_NET_FRAME_EXTENTS(CARDINAL) = 6, 6, 6, 6
_NET_WM_BYPASS_COMPOSITOR(CARDINAL) = 0
_NET_WM_STATE(ATOM) = _NET_WM_STATE_SKIP_PAGER, _NET_WM_STATE_SKIP_TASKBAR
_NET_WM_NAME(UTF8_STRING) =
WM_ICON_NAME(STRING) =
WM_NAME(STRING) = "hypr"
_NET_WM_ICON(CARDINAL) = 	Icon (32 x 32):

WM_HINTS(WM_HINTS):
		Client accepts input or input focus: True
		Initial state is Normal State.
		bitmap id # to use for icon: 0x3200017
		bitmap id # of mask for icon: 0x3200019
		window id # of group leader: 0x3400004
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_NORMAL
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x3, 0x24, 0x0, 0x0, 0x0
WM_NORMAL_HINTS(WM_SIZE_HINTS):
		program specified location: 1024, 612
		program specified minimum size: 512 by 216
		program specified maximum size: 512 by 216
		window gravity: Static
_NET_WM_USER_TIME_WINDOW(WINDOW): window id # 0x320000c
XdndAware(ATOM) = BITMAP
_NET_WM_PID(CARDINAL) = 17989
WM_LOCALE_NAME(STRING) = "en_GB.UTF-8"
WM_CLIENT_MACHINE(STRING) = "void"
WM_CLASS(STRING) = "leagueclient.exe", "leagueclient.exe"
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW, _NET_WM_PING

Log:
log.txt It was too long to post on pastebin.

That's weird, as I've used many wine apps with no issues, havent used league though... Will test.

Can't reproduce, all of my wine games and programs (tested: photoshop, homm5, civ4, civ5, terraria, hyperdimension neptunia) work properly. Have you tried changing the window's size when it goes blank?

Blank window resizing:

As for the 'invisible' ones I've discovered that if I can locate it and move/drag the window, it will become visible again. Does not work with the blank ones though.

As for the 'invisible' ones I've discovered that if I can locate it and move/drag the window, it will become visible again. Does not work with the blank ones though.

Yeah this one makes sense, some windows do that and I've yet to find out why (my example is always ario)
with the blank ones I don't know what's causing this. The app itself should spew errors if that happens though, have you tried launching it from a terminal and seeing the output?

I had a look at the wine logs but to be honest interpreting them is a bit beyond my scope.
I realize this entire issue is difficult to troubleshoot when there are no direct log entries and you're not able to reproduce it in your wine applications, so if you want to close this that's fine.
Ideally someone else would report their experience with League of Legends and Hypr so we could rule out my setup being the cause.

I'm experiencing similar issues however I have achieved a somewhat usable setup with the following rules in hypr.conf:
windowrule=float,class:leagueclientux.exe
windowrule=size 1286 726,class:leagueclientux.exe
windowrule=workspace 2,class:leagueclientux.exe
windowrule=move 650 360,class:leagueclientux.exe

windowrule=float,class:explorer.exe
windowrule=size 0 0,class:explorer.exe
windowrule=workspace 2,class:explorer.exe
windowrule=move 650 360,class:explorer.exe

windowrule=float,class:riotclientux.exe
windowrule=size 1286 726,class:riotclientux.exe
windowrule=workspace 2,class:riotclientux.exe
windowrule=move 650 360,class:riotclientux.exe

windowrule=workspace 2,class:league of legends.exe

With this windows will spawn floating and in their correct sizes preventing odd rendering issues when Hypr attempts to tile them. It is not sufficient to set it to floating as this results in a ~20x40px window.
There are still odd problems which do not arise in other WMs.

All in-game video modes (i.e. borderless, fullscreen, windowed) behave poorly.

Windowed mode results in the camera being unable to move (you move it by dragging the cursor to the edge of the screen). This is expected as it happens in other WMs too.
Borderless (the recommended setting for Linux) allows camera movement. However, switching to a different workspace and back cuts off part of the window. This does not happen on other WMs. Video.
Fullscreen mode results in other workspaces being non-interactive and maintaining the League cursor.Video.

Assigning a fullscreen window rule in Hypr's config does not solve these issues. The somewhat workable solution is to use borderless and either continue to switch back and forth between workspaces until it fixes itself or toggle Hypr's full-screen setting on and off. Not ideal but at least you don't have to restart the game.

As OP described the client is still invisible after closing the main game. If you find its space and attempt to move it you can sometimes make it render. Video.

Unfortunately I do not know what is causing these issues to arise in Hyper but not other WMs.

videos are broken, but all of this seems to arise from some miscommunication issue between hypr and Xorg (damn it, xorg!), where windows do not get their initial size and pos (state) correctly applied.
I can reproduce the partially visible bug on fullscreen with vlc, although moving the mouse over the invisible area fixes it. It's been annoying me for ages, but I have no idea either on what causes it.
I might try to see if I'm missing something - If you say it works on other WMs, I'll try spectr and bsp

Fullscreen mode results in other workspaces being non-interactive and maintaining the League cursor.

I have suspicions this is partially due to Xorg being Xorg - does this happen on, say, dwm?

videos are broken for me (cant play them).
Seem to be broken for me too. Opening their links with mpv e.g. mpv https://user-images.githubusercontent.com/102994494/164995159-77de14ea-489a-49ac-8e73-9af7d997c1af.mp4seem to work.

I can reproduce the partially visible bug on fullscreen with vlc, although moving the mouse over the invisible area fixes it. It's been annoying me for ages, but I have no idea either on what causes it.

Does not work with League as moving the cursor to the edge of the screen moves the camera (I do move the cursor there in the video).

I have suspicions this is partially due to Xorg being Xorg - does this happen on, say, dwm?

Yes this is entirely possible. Since borderless is the go-to setting when it works I have not tried fullscreen in other WMs. I will fire up bsp and check.

I have suspicions this is partially due to Xorg being Xorg - does this happen on, say, dwm?

You are correct. This one occurs on bspwm and leftwm as well. The others do not.

ok. will look into.

partially visible fixed in 8bad02d1155bcbc83a6f7fe6a6ed71c53febd3c2

partially visible fullscreen fixed in 8bad02d

Can confirm. League no longer cuts off parts of the screen. Thanks!


How can I close the dropdown with the ESC key?

I am using the select2 theme.  The dropdown toggles the way I want, but I also want the dropdown to close when the user presses ESC.

This looks like another bug that's occurring because the dropdown isn't focused. Take a look at #1040, it will fix this issue.

Ryan:
How do I implement the fix?  Or is there a newer version that fixes this?
Thanks so much.
Eric

Should be fixed with https://github.com/angular-ui/ui-select/pull/1040


Startup failed on version of  1.9.1 and 1.9.2

This happens when there is a mismatch between the helm chart version and the docker image version. These two versions must match exactly.
When installing the helm chart, only specify the --version ... flag to set the helm chart version, which pulls the matching docker image version automatically.
Using --set version=..., which overrides the docker image version from the helm chart, is not supported and will lead to various problems.


Add support for test scopes

This a small suggestion of adding the ability to pass in a scope object (fixture) per-test.
It's supported by both Specs2 and Scalatest and is convenient for removing code that generates test data out of the test, allowing the test code itself to focus on the logic being tested.
It's a pretty minor addition that's easy to implement, for example for StringSpec it would look like this:
fun <T> String.inScope(scope: T, test: T.() -> Unit): TestCase = this.invoke { test(scope) }
and tests using it would look like this with StringSpec:
class SomeTest : StringSpec() {
    inner class Scope {
        val data = Gen.string().nextPrintableString(length = 5)
    }

    "Test something".inScope(Scope()) {
        data.length shouldBe 5
    }
}

I don't understand sorry. What's the different between that and say:
class SomeTest : StringSpec() {
    inner class Scope {
        val data = Gen.string().nextPrintableString(length = 5)
    }

    "Test something") {
        val scope = Scope()
        scope.data.length shouldBe 5
    }
}

You're right, it's practically the same. I tried your approach before posting the suggestion - it's fine for small examples, but for test scopes that contain a lot of data, prefixing each variable access with scope. is tedious.
The same effect can be achieved by using with(Scope()) {... }, but that would potentially add another nesting level.
So this suggestion doesn't add any new functionality that can't already be achieved, but I think it'll allow a bit more convenience.

I think it is a balancing act to decide what features to add and what to leave out.
The example above could also be expressed via an ordinary property.
class SomeTest : StringSpec() {

    val data: String
        get() = Gen.string().nextPrintableString(length = 5)

    "Test something" {
        data.length shouldBe 5
    }
}

Do you want scopes to group several objects? Like in
inner class Scope {
  val data = Gen.string().nextPrintableString(length = 5)
  val other = ...
}

If the grouping is relevant, why not use table-driven testing, then? If the grouping isn't relevant, could also use independent functions/properties as shown above.

Agreed - this offers nothing new and is a matter of style. I'll just keep using my own extension method :)


Support for type extensions in Query and Projection DSLs

@solnic wdyt? If it looks fine overall I'll add more operations.

@solnic I added Initializer to Attribute, now extensions are evaluated via a default proc. This means they will be preserved in DSL calls. Does it solve the performance concerns?

@flash-gordon I think this will be fine

@solnic OK, it's finished


font color in menu item

font is not showing in menu section

@santoshsharma143 Please can I be assigned ?

@santoshsharma143  u can start working on it


Syntax update to match Lightkurve 2

Lightkurve has been updated to Lightkurve 2, which brought with it some syntax changes.
This is a fix for function calls we use in PBjam to match the new syntax.
This means PBjam will no longer work with versions earlier than Lightkurve 2 Are we happy with this?
The biggest change users will see is probably that cadence -> exptime, in the session  init. PBjam now raises a (informative) value error if you try to use cadence instead of exptime. And a warning if you don't set exptime. Units for exptime is seconds, and for Kepler short cadence exptime  is 60 and 1800 for long cadence.
This should fix Issue #260
In addition, I made some minor fixes when using spectra as input and a few other things.

Ran Pytest locally without issues.
Ready to be merged

Travis seems to be having issues with Pymc3 for Python 3.6 and Python 3.7.
Seems fine for 3.8 and 3.9.


Add tox

This pull-request adds a tox config file.
I took some notes from #118 and have refrained from changing unnecessary files.
You can take advantage of tox by installing either tox or detox and running either command from the base directory.
Learn more about tox: https://tox.readthedocs.io/en/latest/

Current coverage is 100% (diff: 100%)

Merging #347 into master will not change coverage

@@           master   #347   diff @@
====================================
  Files           8      8
  Lines        1244   1244
  Methods         0      0
  Messages        0      0
  Branches      193    193
====================================
  Hits         1244   1244
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 319f2d0...a835520

Hi @crsmithdev, can you merge this PR so contributors can easily run tests.

Done, thanks for the PR


bypass fips tests because we need a build in current

its been days, as soon as we get a build, i'll revert


Using calloc in var initialization. Fixes #472

Close #472

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


how to hide user@hostname

how to hide user@hostname

Hi @LoogCP - that is the context segment. Simply remove it from your segment list, and you should be good. See the README for defining the left and right segments.


How to install PNP SharePoint online (office365)?

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[X] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
How to install PnP on SharePoint online?
https://pnp.github.io/sp-dev-fx-webparts/gettingstarted/
Issue description
How to install PnP on SharePoint online?
https://pnp.github.io/sp-dev-fx-webparts/gettingstarted/

Hi Beau,
Thanks for your reply, actually I want to know to install on the SharePoint online site?
Thanks,
Raj

From: Beau Cameron @.>
Sent: Wednesday, 8 February 2023, 10:10 pm
To: SharePoint/sp-dev-docs @.>
Cc: Thamaraj Maharajah @.>; Author @.>
Subject: Re: [SharePoint/sp-dev-docs] How to install PNP SharePoint online (office365)? (Issue #8758)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Hi, can you clarify you're question? Are you wondering how to install the samples repository (the one that you links) to your computer?
—
Reply to this email directly, view it on GitHubhttps://gbr01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-dev-docs%2Fissues%2F8758%23issuecomment-1422658402&data=05|01|raj%40idtworld.com|f3bf3edfc84f49297aab08db09de4344|eb713521f6d6467ebd8d9dba3203402b|0|0|638114622510789232|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2F65syHUxORCyF3YO69aWyG43HiIhvfSl%2FNaEctjw7nA%3D&reserved=0, or unsubscribehttps://gbr01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAX6IQPT5HXNY7MPQ24U2BVTWWOSOBANCNFSM6AAAAAAUUXK6NE&data=05|01|raj%40idtworld.com|f3bf3edfc84f49297aab08db09de4344|eb713521f6d6467ebd8d9dba3203402b|0|0|638114622510799695|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=J8pT6JSh5In0PKTJxFO7h9vrLHrJ3CVkyHhYDFoOGuo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

Hi! You'll need to download the repository, and build / package the solutions. Then you can install them in SPO.
Please check out the SPFx Documentation on Deploying Solutions.
https://learn.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/get-started/hosting-webpart-from-office-365-cdn


Add the possibility to highlight specific lines

To just like GitHub does.

We can add a parameter to a paste URL (/paste/foo123&ln=1-2+42 for instance)
We can pass the hl_lines flag to Pygments when coloring the paste text

What do you think about ln=1&ln=2&ln=3 ? Can flask handle this correctly?

I don't think this is a good idea :) I'd rather have some kind of DSL to specify ranges and specific lines!

Something like that, yeah :)


Add options: auto-assign and assignees

Closes https://github.com/sqren/backport/issues/188
Example: assign yourself to the backport pull request
backport --auto-assign

if you always want to assign yourself to your backport PRs you can add the following to .backport/config.json:
{
  "autoAssign": true
}

Example: assign another user to the backport pull request:
backport --assign watson

Released in v5.4.3


add database seeds. update models

I updated the models a bit to fix some of the table relationships (I think the way I've set it up will work :/)
I also created seeds so we can test things out

I also updated the get route for homepage the homepage and added examples of the information available in the homepage handlebars so you can use the information and format it

I also updated the get route for the race page and added examples of the information available in the race handlebars so you can use the information and format it


Setting the http-addr option on consul.watch

How can I override the default of <IP_ADDRESS>:8500 as the address of the consul agent on a watch command?
In the Consul cli, there is a -http-addr option, but the below code does not seem to work (I still see the watch attempting to connect to <IP_ADDRESS>:8500)
`
var watch_options = {  'type' : 'services', 'datacenter' : 'ord', 'http-addr' : '<IP_ADDRESS>:8500'};
console.log ("Trying to reach agent ", watch_options["http-addr"]);
var watch = consul.watch({ method: consul.catalog.service.list, options: watch_options});
watch.on('change', function (data, res) {
console.log ("BOOM");
console.log('data:', data);
});
`

Not sure why the code-block formatting only kind of worked in the above post...

You need to set the host/port when you create the consul client, example:
var consul = require('consul')({host: 'example.org', port: 8500});

var watch = consul.watch({ method: consul.kv.get, options: { key: 'test' }});

Thanks!


feat: nft claimer

Fixes https://github.com/snapshot-labs/pitches/issues/1
To-Do

[ ] https://github.com/snapshot-labs/snapshot/issues/3884
[ ] https://github.com/snapshot-labs/snapshot/issues/3888
[ ] https://github.com/snapshot-labs/snapshot/issues/3889
[ ] https://github.com/snapshot-labs/snapshot/issues/3890
[ ] https://github.com/snapshot-labs/snapshot/issues/3891

Self-review checklist

[ ] I have performed a full self-review of my changes
[ ] I have tested my changes on a preview deployment
[ ] I have tested my changes on a custom domain
[ ] I have run end-to-end tests yarn cypress:test:e2e, and they have passed

Additional notes or considerations
(Include any other relevant information or context that may be helpful for the reviewer)

Also I didn't see anything about votes claiming in the demo, are we still doing that?

I didn't see anything about votes claiming in the demo, are we still doing that?

Not in this cycle


Add a shortcut to create a new graph from a template from templates list

It could be nice to have a button on templates entries in the administration panel to speed up template-based graph creation.

Shipped in<PHONE_NUMBER>c84bdff3de38b01acd1c8ecf1c587d.

👌🏻


[BUG] Service Bus: Unknown error after calling GetNamespaceInfo

Describe the bug
When I call client.GetNamespaceInfoAsync it is returning an exception. The message shows an "unknown error".
Expected behavior
Throw an error that is not unknown or successful.
Actual behavior (include Exception or Stack Trace)
Returning unknown error.
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

start the clientManagement
call GetNamespaceInfoAsync

   at Microsoft.Azure.ServiceBus.Management.NamespaceInfoExtensions.ParseFromContent(String xml)
   at Microsoft.Azure.ServiceBus.Management.ManagementClient.<GetNamespaceInfoAsync>d__9.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()

Environment:

Name and version of the Library package used: Microsoft.Azure.ServiceBus 5.1.3
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): [e.g. Azure AppService or Windows 10 .NET Framework 4.8] netstandard2.1, called by netcoreapp3.1
IDE and version : [e.g. Visual Studio 16.3] VS16.10

//cc: @shankarsama

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Hi @ramya-rao-a ,
I just changed and now I receive this: Unknown error. (GeneralError)
and the inner exception:
<feed xmlns="http://www.w3.org/2005/Atom"><title type="text">Publicly Listed Services</title><subtitle type="text">This is the list of publicly-listed services currently available.</subtitle><id>uuid:9f875c19-3997-4acb-838f-5233bc5dff57;id=87</id><updated>2021-10-08T00:34:10Z</updated><generator>Service Bus 1.1</generator></feed>
Status: 200 (OK)

Content:
<feed xmlns="http://www.w3.org/2005/Atom"><title type="text">Publicly Listed Services</title><subtitle type="text">This is the list of publicly-listed services currently available.</subtitle><id>uuid:9f875c19-3997-4acb-838f-5233bc5dff57;id=87</id><updated>2021-10-08T00:34:10Z</updated><generator>Service Bus 1.1</generator></feed>

Headers:
Date: Fri, 08 Oct 2021 00:34:10 GMT
Server: Microsoft-HTTPAPI/2.0
Transfer-Encoding: chunked
Strict-Transport-Security: REDACTED
x-ms-client-request-id: 77693b4a-9b5a-462f-8f29-afe726bbc2f2
Content-Type: application/atom+xml; type=feed; charset=utf-8

Hi @eddynaka,
Can you include a code snippet so we can attempt to reproduce the issue?

Hi @JoshLove-msft ,
below the snippet:
var serviceBusOptions = new ServiceBusAdministrationClientOptions();
serviceBusOptions.Retry.MaxRetries = (retry == bool.TrueString) ? 3 : 0;
var client = new ServiceBusAdministrationClient(serviceBusConnectionString, serviceBusOptions);
NamespaceProperties ni = await client.GetNamespacePropertiesAsync();

Hi @JoshLove-msft , since both use ServiceBus, I was expecting both to work or at least to ServiceBus generate some specific message like "u are using the wrong thing".
Don't know if you have a use case for that, for accepting any kind of SB.
Thanks :)

HI @JoshLove-msft ,
Below, a connection string from NotificationHub Resource:
Endpoint=sb://resource.servicebus.windows.net/;SharedAccessKeyName=DefaultListenSharedAccessSignature;SharedAccessKey=long_key

Below, a connection string from a ServiceBus Resource:
Endpoint=sb://resource.servicebus.windows.net/;SharedAccessKeyName=DefaultListenSharedAccessSignature;SharedAccessKey=long_key

So, I imagined, "hey, both have the same connection string, would that work?"
:)


The popup border is invisible

Question or Suggestion
Now:

Should looks like:

I'm using vim-go, when I execute :GoDoc, the border is invisible, how can i fix this ?

I think I found the problem - there's no border on the window and there also doesn't appear to be any way to set the Windows highlights either.
https://github.com/fatih/vim-go/blob/master/autoload/go/doc.vim#L129
I'm not sure there's a whole lot I can do to make this look better without affecting all other floats in Neovim.
I would suggest looking elsewhere for go dev in Neovim, it doesn't look like this plugin is even maintained anymore.
This one here is a great option if you want something that works great out of the box:
https://github.com/ray-x/go.nvim

alright..
I'll try to use the go.nvim , tks a lot.

I found an other problem, when I open a "quickfix" pane, the border/split-line is invisible too

and the also happen on the help-pane :help vim

Do you have hide_fillchars set to true in your cyberdream config? If not it could be your statusline setting. We found that to be the case here:
https://github.com/scottmckendry/cyberdream.nvim/issues/81#issuecomment-2197777969

hide_fillchars set to true works !
and after switching to go.nvim + lspsaga, i found that lsp_keymaps=false must be set, or it will fallback to using the lsp hover function which display without border by default.


fix jupyter notebook extension

1. More friendly hint for user:

2. Add content for how to edit your cluster settings:

3. Lazy loading of resources to accelerate the submitter:

HI,  Whether to provide a new version of the adaptation update


[Bug]: It is not possible to add "Name & Location"

What happened?
There is no input field in NodeRed version 3.0.2 (node-red-contrib-zwave-js v 7.1.2)

How to reproduce?
Add or edit name of node
Version
7.1.2
Node-RED Version
3.0.2
What hardware are you seeing the problem on?
Raspberry Pi
Relevant Z-Wave-JS log as a text file
new entrys by dialog action "save"
2022-08-19T18:38:41.766Z NDERED « [CONTROLLER] Payload received.
2022-08-19T18:38:41.767Z NDERED « [API: ControllerAPI] [Method: setNodeName]
└─[params]
0: 17
1:
2022-08-19T18:38:41.769Z NDERED » [Node: 17] [NODE_NAME_SET] [DIRECT] Forwarding payload...
2022-08-19T18:38:41.787Z NDERED « [CONTROLLER] Payload received.
2022-08-19T18:38:41.788Z NDERED « [API: ControllerAPI] [Method: setNodeLocation]
└─[params]
0: 17
1:
2022-08-19T18:38:41.790Z NDERED » [Node: 17] [NODE_LOCATION_SET] [DIRECT] Forwarding payload...

Hi @KristiansBox,
This has been fixed in V8 - which in currently in BETA.
I'll close this in favour of https://github.com/zwave-js/node-red-contrib-zwave-js/issues/231 which is for the same issue.
if you want to BETA test - please see change log before doing so.
https://github.com/zwave-js/node-red-contrib-zwave-js/blob/8.0.0/CHANGELOG.md.

Hi Marcus,
how can I use the beta 3 in NodeRed? I installed the package beta 3 via npm but it is not activate in NodeRed. There ist always version 7.1.2 active.
Kristian

Hi @KristiansBox  - restart Node RED, and it should apply.

Unfortunately that does not work. Even a restart didn't help. Does version 7 have to be uninstalled beforehand?

There are some Bugs in 3.0.2 (https://discourse.nodered.org/t/enotempty-when-installing-node-from-palette-since-nodered-3-0-directory-not-empty/65968/21)
I had to delete all folder with a dot at the first position and now is the beta active an I can change the name.
Thanks a lot. Have a nice weekend ;-)
Kristian

Anytime.
When I release the 3.0.0 as stable - it should show as an update.
Have a great weekend

No, uninstalling went automatically. At the beginning I was in the wrong folder and in between on the wrong server :')

But you're all good right?

Yes all good


web-vitals is undefined

Description

web-vitals has been imported but all methods are undefined, please see the image below.

Please help me to investigate
Steps to reproduce

trigger initializeFaro will face the issue

import { trace, context } from '@opentelemetry/api'
import {
  ReactIntegration,
  ReactRouterHistory,
  ReactRouterVersion,
  initializeFaro as coreInit,
  getWebInstrumentations,
} from '@grafana/faro-react'
import {
  FaroSessionSpanProcessor,
  FaroTraceExporter,
  TracingInstrumentation,
} from '@grafana/faro-web-tracing'

export const initializeFaro = () => {
  const faro = coreInit({
    url: config.FARO_URL,
    apiKey: config.FARO_API_KEY,
    app: {
      name: packageJSON.name,
      version: packageJSON.version,
    },
    sessionTracking: {
      enabled: true,
      samplingRate: 0.25,
    },
    instrumentations: [
      ...getWebInstrumentations({
        captureConsole: true,
      }),

      new TracingInstrumentation({
        instrumentationOptions: {
          propagateTraceHeaderCorsUrls: [],
        },
      }),
      new ReactIntegration({
        router: {
          version: ReactRouterVersion.V4,
          dependencies: {
            history: history as ReactRouterHistory,
            Route,
          },
        },
      }),
    ],
  })

   faro?.api.pushLog(['Faro was initialized'])
   faro?.api.initOTEL(trace, context)

   return faro
 }

Expected behavior

Worked

Actual behavior

Environment
"@grafana/faro-react": "^1.3.8",
"@grafana/faro-web-tracing": "^1.3.8",
"react": "^18.3.1",
<EMAIL_ADDRESS>

Hi @tiennguyen1293 first time I see this and unfortunately can't reproduce the error.

Do you use Faro with a standard react web-app or maybe with some mobile framework?
Do you use the CDN version or the NPM package?
When/where do you initialize Faro?
Since you set to auto update Faro to the most current minor or patch version I assume you are on the most recent Faro version which is 1.7.3 ?

Hi @codecapitano thank for quick response:

I'm using react web-app.
use NPM package.
when init Web-app I will initialize Faro.
I tried upgrade to the latest 1.7.3 version but it has the same issue.

Ok thanks.
The issue happens during build right?
If yes, can you wipe node_modules and install and build again (i know not the coolest answer but I want to exclude that kind of issues)

the issue happens during runtime.
I ran rm -rf node_modules yarn.lock and re-install. And then re-started the App, but the issue still happens.

Does it happen on every browser?
Is there maybe something running which prevents the script to work? Like a content filter?
Maybe an issue in regards to content security policy?

it is happening to build runtime to run App on localhost. Maybe it is related to building library or compiling code?

Aaah at build runtime 👍

Maybe it is related to building library or compiling code?

Ok yeah this might be causing the issue.
What is your build tool of choice?

i'm using Webpack version 5

i found the root cause, it is related to webpack config.
I configured
resolve.mainFields: ['browser', 'main', 'module'],

I will close this issue
Thanks for you support

Great thank you for the update. 🙏