Tracking

Description
New tracking classes!
Based on (ongoing) work in scilpy: child classes of scilpy's Propagator, SeedGenerator, and Tracker.
Testing data and script
Ran with Learn2track: it runs. Results are ugly for now, testing with a properly trained model will be necessary, but at least it runs.
Have you

[ ] Added a description of the content of this PR above
[ ] Followed proper commit message formatting
[ ] Added data and a script to test this PR
[ ] Made sure that PEP8 issues are resolved
[ ] Tested the code yourself right before pushing
[ ] Added unit tests to test the code you implemented

People this PR concerns

Hello @EmmaRenauld! Thanks for opening this PR.

In the file dwi_ml/data/processing/streamlines/post_processing.py:

Line 119:27: E226 missing whitespace around arithmetic operator
Line 120:32: E226 missing whitespace around arithmetic operator
Line 120:34: E226 missing whitespace around arithmetic operator
Line 121:21: E126 continuation line over-indented for hanging indent

In the file dwi_ml/models/main_models.py:

Line 6:1: F401 'typing.Union' imported but unused
Line 6:1: F401 'typing.Iterable' imported but unused

In the file dwi_ml/tracking/tracker.py:

Line 59:9: F841 local variable 'seeds' is assigned to but never used

In the file dwi_ml/training/utils_trainer.py:

Line 67:80: E501 line too long (86 > 79 characters)

In the file please_copy_and_adapt/resume_training_from_checkpoint.py:

Line 68:13: F821 undefined name 'prepare_model'

In the file please_copy_and_adapt/track_from_model.py:

Line 185:9: F841 local variable 'seed_mask' is assigned to but never used

In the file please_copy_and_adapt/train_model.py:

Line 100:13: F821 undefined name 'prepare_model'

Do see the PEP 8 -- Style Guide for Python Code


Extra new mappings

A few new mappings added in and changed some regex to pick up some agents that were getting missed

Nice!

One more thing: if we want to release this version we should also increment the version in pom.xml


tests: remove legacy osd_scenario variable

As of stable-4.0 most of these references aren't needed anymore.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test centos-non_container-update

unrelated failures


Rename interval constants

Renamed the interval constants to match the README:
finance.Day   => finance.IntervalDaily
finance.Week  => finance.IntervalWeekly
finance.Month => finance.IntervalMonthly

Codecov Report

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

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   91.85%   91.85%
=======================================
  Files          10       10
  Lines         270      270
=======================================
  Hits          248      248
  Misses         11       11
  Partials       11       11

Impacted Files
Coverage Δ

history.go
91.83% <ø> (ø)
:arrow_up:

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 836c50e...0fd44d0. Read the comment docs.


Support redirectTo for the Auth component

Feature request
The sign in supports redirectTo. However, the Auth component currently does not support that. It would be great if the Auth component supports that.

Really need this one too!

Sorry for late response on this. I'll add it in 😄

:tada: This issue has been resolved in version 0.27.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Looks like this is still missing from the Props section of the docs https://ui.supabase.io/components/auth

@bakerfugu yep! we automatically plucked props with the website but it looks broken.
If anyone wants to take a look, the website is here https://github.com/supabase/supabase-ui-web


[New Port Request] opendht

Library name:  opendht
Library description: A C++11 Distributed Hash Table implementation
Source repository URL: https://github.com/savoirfairelinux/opendht
Project homepage (if different from the source repository): NA
Anything else that is useful to know when adding (such as optional features the library may have that should be included):  Provides CMake targets

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.

would like to add this port to vcpkg !


add optional links; manifests/bosh-lite now promote links/cloud-config

Think I still have some work to do to support etcd.cluster via links.
And locally having inter-node connectivity issues; not resolved yet.

And this issue:
$ curl https://etcd-0.etcd.service.cf.internal:4001/v2/keys -k
curl: (35) error:14094412:SSL routines:SSL3_READ_BYTES:sslv3 alert bad certificate

Hey @drnic. We are working on bosh 2.0 manifests now. Are you planning on working more on the issues you listed above in this PR or should we merge this and make those fixes?

@drnic We will merge this and resolve any issues as they come up.

My networking issues were local and fixed.
The only changes I've made since this PR were to expose properties via provides. Then I wanted to the remove to the etcd. namespace prefix but you can make the timing call on when to roll that out.
Thanks if this PR is helpful.


Unable to upload a file larger than 64KB

import fetch from 'node-fetch'
import { Blob } from 'buffer'

let body = new FormData()

body.append('file', new Blob([Buffer.from('a'.repeat(64 * 1024 + 1))]))

fetch('https://localhost/upload', {
  method: 'POST',
  body
})

It will throw the error.
TypeError [ERR_INVALID_STATE]: Invalid state: chunk ArrayBuffer is zero-length or detached
    at new NodeError (node:internal/errors:393:5)
    at ReadableByteStreamController.enqueue (node:internal/webstreams/readablestream:1122:13)
    at Object.pull (file:///code/node_modules/fetch-blob/index.js:161:42) {
  code: 'ERR_INVALID_STATE'
}

OS: debian 11 amd64
node: 18.10.0
node-fetch: 3.3.0

I where about to start writing about something completely different. (about how it's better to just use blob's backed up by the filesystem but then i tough 64KB was nothing. it's so small... so it would not matter.
then i investigated it and it further and tried to reproduce it (preferable without fetch - if possible)
b/c it's most likely not a problem with node-fetch, maybe it was just something with our toIterator to web stream converter.
So i manage to reproduce it with just:
import { Blob as fetchBlob } from 'node-fetch'
import { Blob } from 'buffer'

const buf = Buffer.from('a'.repeat(64 * 1024 + 2))
const blob = new Blob([buf])
const file = new fetchBlob([blob])

for await (const c of file.stream()) {
  console.log(c)
}

(therefore i'm going to close this as this issue dose not belong here - but feel free to keep commenting)
This is more or less what's going on under the hood.
the hole FormData is converted into a Blob and then sent to the destination.
kind of like formData2blob(fd).stream().pipeTo(dest)
Anyhow that's beside the point, (bit of topic)

readable web byte-streams have this kind of step where it detaches the underlying ArrayBuffer when using:
ReadableStream({
  type: 'bytes',
  async pull (ctrl) {
    ctrl.enqueue(uint8array)
  }
})

And once the uint8array have been enqueued then the buffer will be detached and the byte length will become zero so you can't use that uint8array again for anything else.
that's the root cause of this error: chunk ArrayBuffer is zero-length or detached. it have been detached.
then i tried something else:
import { Blob } from 'buffer'

const buf = Buffer.from('a'.repeat(64 * 1024 + 2))
const blob = new Blob([buf])

for await (const c of blob.stream()) {
  console.log(c)
}

Uint8Array(65536) [
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97, 97,
  97, 97, 97, 97,
  ... 65436 more items
]
Uint8Array(2) [ 97, 97 ]

all looks good ✅
then i had a look at the underlying ArrayBuffer and tried something else
for await (const c of blob.stream()) {
  console.log(c.buffer) // log the arrayBuffer
}

ArrayBuffer {
  [Uint8Contents]: <61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 ... 65438 more bytes>,
  byteLength: 65538
}
ArrayBuffer {
  [Uint8Contents]: <61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 ... 65438 more bytes>,
  byteLength: 65538
}

that dose not look right... ❌
should have been
ArrayBuffer {
  [Uint8Contents]: <61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 61 ... 65438 more bytes>,
  byteLength: 65538
}
ArrayBuffer {
  [Uint8Contents]: <61 61>,
  byteLength: 2
}

I took an extra look at the NodeJS source code: console.log(Blob.toString()) cuz i knew that .stream() impl in NodeJS it isn't perfect.
stream() {
    if (!isBlob(this))
      throw new ERR_INVALID_THIS('Blob');

    const self = this;
    return new lazyReadableStream({
      async start() {
        this[kState] = await self.arrayBuffer();
        this[kIndex] = 0;
      },

      pull(controller) {
        if (this[kState].byteLength - this[kIndex] <= kMaxChunkSize) {
          controller.enqueue(new Uint8Array(this[kState], this[kIndex]));
          controller.close();
          this[kState] = undefined;
        } else {
          controller.enqueue(new Uint8Array(this[kState], this[kIndex], kMaxChunkSize));
          this[kIndex] += kMaxChunkSize;
        }
      }
    });
  }

the issue here is that it's using:
controller.enqueue(new Uint8Array(this[kState], this[kIndex], kMaxChunkSize));

instead of doing things such as:
controller.enqueue(new Uint8Array(this[kState].slice(start, end));

This is unexpected from a web spec perspective of reusing the same underlying ArrayBuffer (but with an uint8Array with a byte offset & size instead)
So really this isn't a issue with fetch-blob either, but with NodeJS own implementation of theirs Blob class
it should not even be doing:
    return new lazyReadableStream({
      async start() {
        this[kState] = await self.arrayBuffer();

as that is a bad memory hog to allocate the hole arrayBuffer like that instead of creating some more stream friendlier variant
I have already reported it to NodeJS as said it's bad. https://github.com/nodejs/node/issues/42264
i even reported this same exact problem in one of the comments: https://github.com/nodejs/node/issues/42264#issuecomment-1293524461
But i totally forgot about even posting them to NodeJS earlier.
I have always been reluctant to use NodeJS own Blob implementation and doing things such as export default globalThis.Blob || polyfill, cuz it well... sry for saying this: NodeJS blob impl. kind of sucks...

it isn't stream friendly,
casting the blob.type to all lowercase is bad
and there is no way to create blob backed up by the filesystem so they are completely useless - you might just as well use ArrayBuffer or Typed Arrays then, b/c it's hold in memory anyway...


Made tooltip component reusable

Date: 17 feb 2024

Developer Name: @skv93-coder

Refrence
https://ant.design/components/tooltip
Issue Ticket Number

Description
Makes tooltip component more reuseable

Documentation Updated?

[ ] Yes
[x] No

Under Feature Flag

[x] Yes
[ ] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots
screen-capture (7).webm

Screenshot 1

Test Coverage

Screenshot 1

Additional Notes

Looks good to me mostly but I got this issue when checking in my local, does it happen for you too?
2024-02-18.19-37-04.mp4

Looks good to me mostly but I got this issue when checking in my local, does it happen for you too?
2024-02-18.19-37-04.mp4

It was a mistake from my side. corrected good review.


Breaking change in pascal-case in 3.x.x

const { pascalCase } = require("pascal-case");
pascalCase("IQueryAArgs")

This code used to give IQueryAArgs before 3.x.x, but now it gives IQueryAargs.

Thanks for the flag. This looks related to merging the regexes in no-case together into a single regex, so the first would split it apart but the same fails to also split AA. Looking at a fix today.

Fixed with https://github.com/blakeembrey/change-case/commit/d945e02b515cb8f7797328bcc5d6a29eb6c5832d.


[irods/irods#7220] Do not explicitly build against libc++ (main)

In service of irods/irods#7220
Merge irods/irods#7526 before merging this.

#'d

Missed one @alanking


Protect Anya pending block system

Things seem to have tentatively settled. This PR can be easily reopened if necessary, but for now I don't think it is.


Lack of nodejs inside devcontainers could be an issue

I'm going to have to test this for both the PHP and Ruby environments.

I've moved the installation of Node.js from the base image into the development containers. You can configure which version of PHP or NODE to install using the PHP_VERSION and NODE_VERSION arguments respectively.


use a single server-info for all mapper types

          Improvement: Shall we use one `server-info` for all the mappers, and start the mode based what is written in the file?

Originally posted by @whynowy in https://github.com/numaproj/numaflow/pull/1778#discussion_r1671666859
### Tasks
- [ ] https://github.com/numaproj/numaflow/issues/1836
- [ ] https://github.com/numaproj/numaflow/issues/1838
- [ ] https://github.com/numaproj/numaflow/issues/1837
- [ ] https://github.com/numaproj/numaflow/issues/1846
- [ ] https://github.com/numaproj/numaflow/issues/1849
- [x] Testing single-server info for map

Closing as all tasks completed.


Display a different Cosmos DB API between Portal and Storage Explorer

Storage Explorer Version: 1.9.0_20190702.3
Platform/OS: Linux Ubuntu/macOS High Sierra/Windows 10
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Create a Cosmos DB account with Core(SQL) API in Portal.
Open the new created account in Storage Explorer
Check the Cosmos DB API on actions & Properties panel.

Expect Experience:
Display the same Cosmos DB API between Portal and Storage Explorer.
Actual Experience:
Display a different Cosmos DB API between Portal and Storage Explorer.

Azure Storage Explorer support for Azure Cosmos DB has been discontinued as of v1.24.0. We recommend users to use Azure Cosmos DB’s Data Explorer, which can be found in Azure Portal or as a stand-alone version in http://cosmos.azure.com/ for the best Azure Cosmos DB experience. You can also find additional Azure Cosmos DB tooling here: Azure Cosmos DB Tools


PPYR_07918-PA (P450)

PPYR_07918-PA chimeric (CYP4AA1)

Not relevant to CYP303 tree. Won't fix until next manuscript


Can't login to admin-panel

Always get error:
BadMethodCallException in Controller.php line 107: Method [hasTooManyLoginAttempts] does not exist.
laravel on local host with running MAMP PRO

@berrymoor, Is this vanilla Voyager, or did you add it to an existing project?
hasTooManyLoginAttempts() is defined in the Trait: \Illuminate\Foundation\Auth\ThrottlesLogins
Are you using Auth in your project?

Closed due to inactivity.


cannot reshape array

Hi,
I'm  trying to convert darknet weights into pb model, but I got this error on reshaping array.
I tried yolov4-tiny.weights and coco.names
Traceback (most recent call last):
File "convert_weights_pb.py", line 52, in
tf.app.run()
File "C:\Users\Asus\Anaconda3\envs\env-openvino\lib\site-packages\tensorflow\python\platform\app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "C:\Users\Asus\Anaconda3\envs\env-openvino\lib\site-packages\absl\app.py", line 300, in run
_run_main(main, args)
File "C:\Users\Asus\Anaconda3\envs\env-openvino\lib\site-packages\absl\app.py", line 251, in _run_main
sys.exit(main(argv))
File "convert_weights_pb.py", line 42, in main
load_ops = load_weights(tf.global_variables(scope='detector'), FLAGS.weights_file)
File "D:\ds\prosa\OpenVINO-YOLOV4\utils.py", line 117, in load_weights
(shape[3], shape[2], shape[0], shape[1]))
ValueError: cannot reshape array of size 554878 into shape (256,256,3,3)
I also got same error on reshaping array, when converting 1 class darknet weights model into pb model.
Would you help me what's going wrong? Thanks

Hello! @devipramita
which command did you use?
python convert_weights_pb.py --class_names cfg/coco.names --weights_file yolov4-tiny.weights --data_format NHWC --tiny
I think you may miss “--tiny"

Hello！ @TNTWEN
I also meet same situation. I have trained a yolov4-tiny by myself dataset(image size is 608*608).But when I use the "convert_weights_pb.py" to convert. I got this error on reshaping array.
Traceback (most recent call last):
File "D:/summer/code_article/code/OpenVINO-YOLOV4/convert_weights_pb.py", line 52, in
tf.app.run()
File "C:\Users\jji\AppData\Roaming\Python\Python36\site-packages\tensorflow_core\python\platform\app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "C:\Users\jji\AppData\Roaming\Python\Python36\site-packages\absl\app.py", line 299, in run
_run_main(main, args)
File "C:\Users\jji\AppData\Roaming\Python\Python36\site-packages\absl\app.py", line 250, in _run_main
sys.exit(main(argv))
File "D:/summer/code_article/code/OpenVINO-YOLOV4/convert_weights_pb.py", line 42, in main
load_ops = load_weights(tf.global_variables(scope='detector'), FLAGS.weights_file)
File "D:\summer\code_article\code\OpenVINO-YOLOV4\utils.py", line 115, in load_weights
(shape[3], shape[2], shape[0], shape[1]))
ValueError: cannot reshape array of size 446994 into shape (256,384,3,3)
I have write " tf.app.flags.DEFINE_bool('tiny', True, 'Use tiny version of YOLOv4')" in convert_weights_pb.py

Hello!@lijunnankman
https://github.com/TNTWEN/OpenVINO-YOLOV4/blob/534cca38d1e3bf95dce98daa216a57f62a2f83ce/convert_weights_pb.py#L24-L25
You also need to set --size 608

yes, I also set.Can we keep touch in mail?My mail is<EMAIL_ADDRESS>a lot.

I meet this error too,  but after I change the file cfg/coco.names, it works for me


add day 1

Trickier than expected!  Still not really happy with my implementation, it kind of got away from me halfway in.

Yes, being tricky was intended :)


Replace deprecated project with projected type

Ref: https://github.com/taiki-e/pin-project/blob/v0.4.21/CHANGELOG.md#0421---2020-06-13

Also open to migrating to pin_project_lite which is supposed to be lighter-weight, but as-is this was a pretty quick change.
Original error:
Compiling actix-interop v0.1.1 (/home/jon/src/oss/actix-interop)
error: use of deprecated item 'project': consider naming projected type by passing `project` argument to #[pin_project] attribute instead, see release note <https://github.com/taiki-e/pin-project/releases/tag/v0.4.21> for details
   --> src/local_handle.rs:54:7
    |
54  |     #[project]
    |       ^^^^^^^
    |
note: the lint level is defined here
   --> src/lib.rs:144:23
    |
144 | #![deny(missing_docs, warnings)]
    |                       ^^^^^^^^
    = note: `#[deny(deprecated)]` implied by `#[deny(warnings)]`

Good catch-- done.

Thanks! 👍


In attempting to open an '.etl.zip' file with TraceLog, FileNotFound is thrown

In trying to create an etlx file from an etl.zip archive with the TraceLog.CreateFromEventTraceLogFile a FileNotFoundException is thrown for existing files
This is because TraceEventDispatcher accepts .etl.zip files, but when it gets to ETWTraceEventSource.GetMergeAllLogFiles via ETWTraceEventSource.Initialize there is not a case for .etl.zip files.
If .etl.zip files are meant to be supported, it should be fixed so this exception is not raised. If they are not supported, the code should fail faster and raise a more helpful exception
Here is a stack trace from my program that triggered this error:
Unhandled Exception: System.IO.FileNotFoundException: Could not find file     E:\OneDrive\App Compat Traces\perfcollectdata\P0_Sub P0\Advanced Combat Tracker\AdvancedCombatTrackerTest01\AdvancedCombatTrackerTest01.etl.zip
   at Microsoft.Diagnostics.Tracing.ETWTraceEventSource.GetMergeAllLogFiles(String fileName) in E:\CPUStats\PerfView\TraceEvent\ETWTraceEventSource.cs:line 586
   at Microsoft.Diagnostics.Tracing.ETWTraceEventSource.Initialize(String fileOrSessionName, TraceEventSourceType type) in E:\CPUStats\PerfView\TraceEvent\ETWTraceEventSource.cs:line 355
   at Microsoft.Diagnostics.Tracing.TraceEventDispatcher.GetDispatcherFromFileName(String traceFileName) in E:\CPUStats\PerfView\TraceEvent\TraceEvent.cs:line 2714
   at Microsoft.Diagnostics.Tracing.Etlx.TraceLog.CreateFromEventTraceLogFile(String filePath, String etlxFilePath, TraceLogOptions options) in E:\CPUStats\PerfView\TraceEvent\TraceLog.cs:line 63
   at CPUStats.CPUStats.ComputeStats() in E:\CPUStats\CPUStats.cs:line 489
   at CPUStats.Program.Main(String[] args) in E:\CPUStats\Program.cs:line 56

This path is not expected to work with an etl.zip file.  The contents of the zip file are expected to extracted first.


Add/edit descriptions of existing playlists?

It is already possible to create playlists with descriptions with
Webclient.create_playlist(name, description, public=False)

but would it be possible to edit/add descriptions to existing user created playlists in the future? I can edit/add descriptions on the Android Play Music app.

If it can be done from the app, then it should be pretty straightforward to add.

That would be cool! It's also possible to edit the playlist title from the app:

blocked on #303

request
PUT https://mclients.googleapis.com/sj/v1.10/playlists/<id>?alt=json&hl=en_US
{
	"creationTimestamp": "-1",
	"deleted": false,
	"description": "...",
	"id": "...",
	"lastModifiedTimestamp": "1420400606246000",
	"name": "...",
	"ownerName": "...",
	"shareToken": "...",
	"type": "USER_GENERATED"
}

response is the current state (same as request assuming serial requests)

unfortunately, it doesn't look like this endpoint handles patch requests. It might be easier to do this through the batch_mutate call that change_playlist_name currently uses.

Is there a reason that change_playlist_name isn't in the documentation?
PS I quickly added a change_playlist_description method and added a description param to Mobileclient.create_playlist while on a conference call this morning : P

Also, there doesn't seem to be any testing done for playlist descriptions. Should something be added for that? Or is it fine to leave it?

Is there a reason that change_playlist_name isn't in the documentation?

Whoops; nope. We should add it.

Yeah, I'll do that, too.

@simon-weber: So, as I was checking out create_playlist, I noticed that in gmusicapi description is required in the Webclient. This is inconsistent with Google's behavior which lets you create a playlist with no description. What do you think about changing that and having the Mobileclient version follow suit?

As I'm thinking about it more, you can also toggle a playlist as public or not. Wondering if all these should be wrapped up into one method or should I add a method for setting that separately?

Alternatively, would you want all of the individual methods and a combined edit_playlist method?

I like to keep the client library as close to the actual calls as possible. So, if the request supports all of these changes in one call, let's expose all of them in one call too.

PS I quickly added a change_playlist_description method

Where is change_playlist_description, I cannot find it (maybe because I'm not that familiar with github)?
change_playlist_name works fine for me

Here it is: https://github.com/simon-weber/Unofficial-Google-Music-API/blob/e7e2e3d9ee90ede66356163f2a3f752a4a34fa14/gmusicapi/clients/mobileclient.py#L301.

Whoops, I misread you. It's on @thebigmunch's fork; you either need to wait for it to be merged or use his code directly.

Thanks for the quick reply Simon, but still cannot find it haha.
https://github.com/thebigmunch/Unofficial-Google-Music-API/search?utf8=✓&q=change_playlist_description

@thebigmunch probably just didn't push it yet.

Yeah, I meant locally. Sorry, my back started acting up so I stopped spending so much time on coding work this week. I'll make Pull Requests for a few things today, for sure.
Here is a branch that contains a few playlist-related changes. They include a fix for setting playlists as public, an optional description parameter to Mobileclient.create_playlist, and a new Mobileclient.edit_playlist method.

Just tested out your Mobileclient.edit_playlist method and it works great for names, descriptions and public/private @thebigmunch! Thank you both!

Closed by b45e1113.


Condition selection & quaternary (left/right)

On one of my indicators to obtain the correct alert i need to select the Pair in the condition section, not the indicator name. The indicator is selected in the tertiary & quaternary  sections.
A couple of questions:

Can i select the pair in condition section even though it changes dynamically with each alert created (as per the CSV pair list file) when the tool is run?  Is it even possible or can the config.yml file be setup to just select the drop down position in condition not the pair name which in my case would be 1? Unfortunately the default sets to indicator name

.

Can i edit the config.yml file to have a quaternary left  / right ? As you can see in the screenshot i wish to alert entering a channel which needs Upper and Lower bounds

Condition: Pair (current - position 1)
Secondary: Entering Channel
Tertiary: Left = Indicator Right = Indicator
Quaternary: Left = Indicator Right = Indicator
Thanks for your help and btw awesome job.

Hi again - pointing out a couple flaws I didn't realize..

Can you help me understand more why you need to select a particular position? That would be a tad brittle.
Otherwise, I could modify the tool so each condition can have dynamic placeholders..
For example, consider the following where {{symbol}} would be replaced with "MATICUSDTPERP" from the .csv file:

alert:
  condition:
    primaryLeft: "{{symbol}}"
    primaryRight: Volume
    secondary: Crossing
    tertiaryLeft: "BitMEX Funding and Premium Index"
    tertiaryRight: Funding Rate

I had no idea that there could be even more inputs...gonna have go back to the drawing board here and work on a solution...maybe I could create a quick "quaternary" solution, but now that starts to sounds weirder than "tertiary"

Hi there,

the reason for selecting a particular position is because the condition has a drop down and shows the pair and indicators. For my particular case i need to chose the pair which is in the first position (blacked out ones are indicator names). Your suggestion i think could work.

Yeah quaternary does sound weird.. was going for consistency lol. I don't have experience in programming these things but if its not too hard a quick 4th option would be good.

So it turns out you can actually just leave "primaryLeft" blank and the script skips over it..
was able to add "quaternary" left and right (will never remember how to spell that!)
Grab the update (from your alerts-home directory):
tradingview-alerts-home $ npm install<EMAIL_ADDRESS>
make your config look something like this:
alert:
  condition:
    primaryLeft:
    primaryRight:
    secondary: Entering Channel
    tertiaryLeft: Value
    tertiaryRight: 5000
    quaternaryLeft: Value
    quaternaryRight: 4000

Legend! thanks for that. Shall i use the {{symbol}} syntax for the condition pair selection?
I'll give it a try and keep you posted.

gonna go ahead and assume everything's running fine..


Install Failed

When attempting to update the extension installed to Azure Data Studio:
When v1.36.2 Azure data studio and vs code preview, are both closed, then double click the (Unblocked) vsix.
Message is: Install Failed 'One or more extensions are for Visual Studio Code. Try installing them in Visual Studio Code'
Log contents:
10/06/2022 12:51:26 PM - Microsoft VSIX Installer
10/06/2022 12:51:26 PM - -------------------------------------------
10/06/2022 12:51:26 PM - vsixinstaller.exe version:
10/06/2022 12:51:26 PM - 17.3.22-preview1
10/06/2022 12:51:26 PM - -------------------------------------------
10/06/2022 12:51:26 PM - Command line parameters:
10/06/2022 12:51:26 PM - C:\Program Files (x86)\Microsoft Visual Studio\Installer\resources\app\ServiceHub\Services\Microsoft.VisualStudio.Setup.Service\VSIXInstaller.exe,C:\Users\MYUSERNAME\Downloads\extra-sql-script-as-0.5.0.vsix
10/06/2022 12:51:26 PM - -------------------------------------------
10/06/2022 12:51:26 PM - Microsoft VSIX Installer
10/06/2022 12:51:26 PM - -------------------------------------------
10/06/2022 12:51:26 PM - Initializing Install...
10/06/2022 12:51:26 PM - Extension Details...
10/06/2022 12:51:26 PM - 	Identifier         : extra-sql-script-as
10/06/2022 12:51:26 PM - 	Name               : Extra Sql Script As
10/06/2022 12:51:26 PM - 	Author             : pacoweb
10/06/2022 12:51:26 PM - 	Version            : 0.5.0
10/06/2022 12:51:26 PM - 	Description        : This extension adds several missing options to the context menu of the object tree: Script Table as INSERT, Script Table as UPDATE...
10/06/2022 12:51:26 PM - 	Locale             : en-US
10/06/2022 12:51:26 PM - 	MoreInfoURL        :
10/06/2022 12:51:26 PM - 	InstalledByMSI     : False
10/06/2022 12:51:26 PM - 	SupportedFrameworkVersionRange : [0.0,2147483647.2147483647]
10/06/2022 12:51:26 PM -
10/06/2022 12:51:26 PM - 	SignatureState     : Unsigned
10/06/2022 12:51:26 PM - 	Supported Products :
10/06/2022 12:51:26 PM - 		Microsoft.VisualStudio.Code
10/06/2022 12:51:26 PM - 			Version :
10/06/2022 12:51:26 PM - 			ProductArchitecture : x86
10/06/2022 12:51:26 PM -
10/06/2022 12:51:26 PM - 	References         :
10/06/2022 12:51:26 PM - Signature Details...
10/06/2022 12:51:26 PM - 	Extension is not signed.
10/06/2022 12:51:26 PM -
10/06/2022 12:51:26 PM - Searching for applicable products...
10/06/2022 12:51:26 PM - Found installed product - Global Location
10/06/2022 12:51:26 PM - Found installed product - Visual Studio Professional 2022 Preview
10/06/2022 12:51:26 PM - VSIXInstaller.NoApplicableSKUsException: One or more extensions are for Visual Studio Code. Try installing them in Visual Studio Code.
at VSIXInstaller.ExtensionService.GetInstallableDataImpl(IInstallableExtension extension, String extensionPackParentName, Boolean isRepairSupported, IStateData stateData, IEnumerable1& skuData)    at VSIXInstaller.ExtensionService.GetInstallableData(String vsixPath, String extensionPackParentName, Boolean isRepairSupported, IStateData stateData, IEnumerable1& skuData)
at VSIXInstaller.ExtensionPackService.IsExtensionPack(IStateData stateData, Boolean isRepairSupported)
at VSIXInstaller.ExtensionPackService.ExpandExtensionPackToInstall(IStateData stateData, Boolean isRepairSupported)
at VSIXInstaller.App.Initialize(Boolean isRepairSupported)
at VSIXInstaller.App.Initialize()
at System.Threading.Tasks.Task`1.InnerInvoke()
at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.VisualStudio.Telemetry.WindowsErrorReporting.WatsonReport.GetClrWatsonExceptionInfo(Exception exceptionObject)

Hi
You have to use the option INSTALL FROM VSIX in Extensions section, in Azure Data Studio

Tested with this version

Thanks


Private team cloud snapshots

Background
Currently, cloud snapshots are public to the internet. With the concept of a team, we now have a natural access control.
Description
Create a team snapshot and access it from the Tilt Cloud team page.
From Tilt:

From the Tilt Cloud team page:

If you are a team member or a team owner, you can create a snapshot that will be associated with the team. (See https://docs.tilt.dev/teams.html about team membership.)
Maybe we will also allow the None role to create a team snapshot as well, but they won't be able to access it immediately until they become at least a team member. This will likely be a future improvement.
All team members and team owners can view all team snapshots. But nobody else can view these team snapshots.
A team owner can delete any team snapshot.
If you created the team snapshot, you can delete it.

Related
https://github.com/windmilleng/tilt/issues/3067 talks about private snapshots, but that's outside of a team setting, and more generic. This is about sharing at once to a team only, similar to how for a Google doc, you can share it within your entire organization if you are using Google apps for your business/domain.

A user has suggested that a team owner should be able to delete team snapshots (even if they are created by another user). Logging this here as an idea/ feature.

A user has suggested that a team owner should be able to delete team snapshots (even if they are created by another user). Logging this here as an idea/ feature.

a driver for this is that if an admin becomes aware that some keys or other sensitive material happened to be logged, we would want a quick way to purge past affected snapshots without having to interrupt individual application engineers and request that they delete them

Thanks @andymartin-sch , that makes a lot of sense!

@andymartin-sch : I've updated the description to be explicit who can delete a snapshot. Feel free to take a look and let me know if you'd prefer something else. I also created https://github.com/windmilleng/tilt/issues/3260 and https://github.com/windmilleng/tilt/issues/3261. Do take a look and comment there / upvote if they are of interest.

Looking at the mock, it seems like private snapshots are opt-in at the point of creating the snapshot.
In the interest of preventing information leaking, it would be useful to be able to disable public snapshots on a Tiltfile entirely.

Looking at the mock, it seems like private snapshots are opt-in at the point of creating the snapshot.
In the interest of preventing information leaking, it would be useful to be able to disable public snapshots on a Tiltfile entirely.

+1, was going to leave a similar comment. We would prefer no public snapshots ever for our team, but if others need this, maybe a block_public_snapshots-like flag could be added to set_team or disable_snapshots?

Thanks @theothertomelliott and @andymartin-sch ! I should've considered this more closely. I updated the description with saying that only private snapshots are available in the context of a team. @andymartin-sch thanks for mentioning those flags / functions. We'll definitely dig into it further when we get to detailed implementation on a final design. But my initial thought is that it would be hardcoded so we wouldn't even need those flags. But we'll see!

Closing this since we have https://docs.tilt.dev/snapshots.html#private-snapshots-aka-team-snapshots now.


[nrf fromtree] samples: sysbuild: hello_world: fix broken references/…

…args
README for this sample had references to the initial sample name/target.
Signed-off-by: Gerard Marull-Paretas<EMAIL_ADDRESS>(cherry picked from commit 598ab6b39a5e38466a286fb317f39bcd34b8ad20)

@kylebonnici , this fixes the issue with the link you've reported.


📎 Publish nightlies as pre-releases instead of separate extension

Summary
The stable and nightly versions of Biome are currently published as two separate extensions.

biomejs.biome (stable)
biomejs.biome-nightly (nightly)

One of the goals of publishing a nightly release of the extension is to allow early adopters to test new features and help us catch issues before they land in a stable release. In order to reduce friction, switching from the stable release to the nightly and vice-versa should be as straightforward as possible.
When I originally petitioned to split both releases into separate extensions, my hope was to avoid dealing with the fact that the VS Marketplace does not support SemVer. To cope with that limitation, extensions that publish pre-releases usually have a system in place that consists of publishing stable releases with an even-numbered minor version and pre-releases with an odd-numbered minor version.

2.0.0 (stable)
2.1.0 (pre-release)
2.2.0 (stable)

This has the effect of producing non-linear versions within a channel (2.0.0 -> 2.2.0), which is something I'm admittedly not a fan of. That being said, and having discussed it among ourselves and with community members, we feel that the benefits of having a single extension with two release channels outweigh the cons.

Easier to switch between stable and nightly
Users need to maintain only one set of settings
Easier upgrade path
No more conflicts between both versions

Upgrade paths

Users who subscribe to the stable release channel expect to be upgraded to the next stable version, not the next pre-release version.
Users who subscribe to the pre-release channel expect to be upgraded to the next pre-release version, but won't mind being upgraded to the next stable release if there isn't yet a pre-release version available.

Version numbers
To prevent confusion, the version number of the latest stable release should always be higher than the latest pre-release, even if this means that we end up with gaps.
For example, if we publish a pre-release (2.1.0), and some days later we publish another pre-release (2.3.0), the next stable release should not be 2.2.0, but 2.4.0.
To do

[ ] Stop packaging nightly
[ ] Stop publishing nightly
[ ] Mark pre-releases with vsce
[ ] Refactor code meant to prevent conflicts with nightly
[ ] Use CI to ensure the version pattern matches the corresponding channel
[ ] Use CI to ensure stable version numbers are always higher than the version of the latest pre-release

Nice!
Are you sure going into the

major.EVEN_NUMBER.patch for stable release and major.ODD_NUMBER.patch for pre-release way

instead of the

Non-linear patch for stable release

?

Nice!
Are you sure going into

major.EVEN_NUMBER.patch for stable release and major.ODD_NUMBER.patch for pre-release way

instead of

Non-linear patch for stable release

?
References

Deprecate biomejs.biome-nightly in favor of pre-release versions of biomejs.biome #123 (reply in thread)
Full Semver Standard Support microsoft/vscode-vsce#932

Not sure yet.
We might even contemplate the YYYY.M.NNNN versioning scheme for nightlies. That's what GitLens does.

We might even contemplate the YYYY.M.NNNN versioning scheme for nightlies

That would require having also the versioning for stable to be with this non-semver versioning, forever

We might even contemplate the YYYY.M.NNNN versioning scheme for nightlies

That would require having also the versioning for pre-release to be locked into this non-semver versioning forever.

True. I’ll keep it in mind.


Inline styles with Radium

How can I use this with Radium?

I’m not familiar with Radium, but from its documentation it looks like it works with pure inline styles and doesn’t use CSS selectors. cq-prolyfill relies on CSS selectors, so in my opinion you cannot use these two projects directly together.
Radium has support for a small subset of CSS selectors: :hover, :focus, :active and @media. If they implement the :container(…) selector, it might be possible to use it in conjunction with cq-prolyfill.


Add org.opencontainers.image.source comments to Dockerfiles

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
796

Relevant Lines:
941

💛  - Coveralls


Requests to sentry.io after disabling telemetry

Steps to reproduce:
List the minimal actions needed to reproduce the behaviour.

Install influx
Set the '--reporting-disabled' flag
Install grafana
Setup a dashboard
Use your browsers developer console to look at network requests

Expected behavior:
No external contact even telemetry.
Actual behavior:
Requests to "https://o1313669.ingest.sentry.io/api/XXXXXX/XXXXXXX/XXXXXX" seen.
Environment info:

System info: Linux 5.15.0-58-generic x86_64
InfluxDB version: InfluxDB v2.6.1 (git: 9dcf880fe0) build_date: 2022-12-29T15:53:07Z

Config:
/usr/lib/influxdb/scripts/influxd-systemd-start.sh
line 3 has been changed to read: /usr/bin/influxd --reporting-disabled &
Screenshot:

You're seeing these in your browser console while in the Grafana dashboard? This is coming from Grafana and not from InfluxDB. We do not send telemetry requests to sentry.io, they are sent to telemetry.influxdata.com and are sent by the server process not in browser.

You're seeing these in your browser console while in the Grafana dashboard? This is coming from Grafana and not from InfluxDB. We do not send telemetry requests to sentry.io, they are sent to telemetry.influxdata.com and are sent by the server process not in browser.

My mistake. Thank you for the clarification.


Do not use /tmp as a userConfRoot

Description
vip dev-env create uses Lando in such a way that /tmp (os.tmpdir()) is chosen as a Lando user root configuration directory. This means that /tmp (or its equivalent) is mounted as /lando into containers.
The issue here is that Lando tries to change the owner of /lando and its files and subdirectories to match the UID/GID of the user running Lando. In Linux, this unfortunately succeeds, and as a result, /tmp and all of its files and directories change their ownership. This becomes catastrophic for, say, Firefox (at some point, all network requests freeze, the browser crashes), Chromium (it fails to start), and for some software installed from Snap.
For containers created by vip dev-env create, docker inspect will show something like this:
            {
                "Type": "bind",
                "Source": "/tmp",
                "Destination": "/lando",
                "Mode": "cached",
                "RW": true,
                "Propagation": "rprivate"
            },

Lando's scripts/user-perm-helpers.sh has function, perm_sweep(), which becomes destructive when used on host's /tmp:
nohup find /lando -not -user $USER -execdir chown $USER:$GROUP {} \+ > /tmp/perms.out 2> /tmp/perms.err &

This PR offers a dedicated temporary directory to Lando so that the chown operation does not break the host system.
Steps to Test
Outline the steps to test and verify the PR here.
Example:

Check out PR.
Run npm run build
Create and start a test environment.
Observe that /tmp's permissions/ownership were not changed.

r2259-stacks

Great thanks for this!


The commands Set-UserAnalyticsConfig and Get-UserAnalyticsConfig are depreciated.

[Enter feedback here]

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

ID: 412cc99c-71d3-b2ad-f2e6-38b6270522ed
Version Independent ID: adf90afb-7fe5-9c46-0992-e030b8b995e1
Content: Personal insights deployment guide
Content Source: Viva/insights/personal/Setup/deployment-guide.md
Service: viva
Sub-service: viva-insights
GitHub Login: @madehmer
Microsoft Alias: v-lilyolason

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


GetAllUserKeys or GetAllUserInfo. I need a hybrid.

For me in my current app I could really use a hybrid of GetAllUserInfo that just gets and stores Key and DisplayText. Can you show me where I might add that?
if (!scope.getAllInfo) {
  //var keys = peoplePicker.GetAllUserKeys().split(';');
  ngModelCtrl.$setViewValue(peoplePicker.GetAllUserKeys());
} else {
  ngModelCtrl.$setViewValue(peoplePicker.GetAllUserInfo());
}

I tried changing the above to
if (!scope.getAllInfo) {
  //var keys = peoplePicker.GetAllUserKeys().split(';');
  ngModelCtrl.$setViewValue(peoplePicker.GetAllUserKeys());
} else {
  var obj = peoplePicker.GetAllUserInfo();
  ngModelCtrl.$setViewValue( {Key: obj[0].Key, DisplayText: obj[0].DisplayText} );
}

but no go.

I tested making your change, and I could get a model out no problem, but propogating the picker from code, by way of the model didnt work, which turned out to be a bug in the $render function.
either way, I think you would want to change the render function and the formatterss function if you just wanted a simplified model of the user.

Can you show me an example?


Upgrade to react-native 0.44

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Backup manager status poller

Not calling APIs for backup status, instead calling internal methods
Changes corresponding UTs

Pull Request Test Coverage Report for Build 1058

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 88.568%

Totals

Change from base Build 1054:
-0.03%

Covered Lines:
6541

Relevant Lines:
7154

💛  - Coveralls


No more covers in details view.

Bug Description
I started Playnite again after a long time and I noticed that no more covers appear in details view. I don't know which update it was because I haven't started Playnite for about half a year.
When I switch to the original eMixedNite theme, the cover appears.
To Reproduce
No response
Extensions log
No response
Screenshots

This problem has been fixed in version 3.75, the update will release later today or tomorrow! If you want more info on this issue you can refer to this.

Thank you so much


[SPARK-16709][CORE] Kill the running task if stage failed

What changes were proposed in this pull request?
At SPARK-16709,  when a stage failed, but the running task is still running, the retry stage will rerun the running task, it could cause TaskCommitDeniedException and task retry forever.
Here is the log:
16/07/28 05:22:15 INFO scheduler.TaskSetManager: Starting task 1.0 in stage 1.0 (TID 175, <IP_ADDRESS>, partition 1,PROCESS_LOCAL, 1930 bytes)
16/07/28 05:28:35 INFO scheduler.TaskSetManager: Starting task 1.0 in stage 1.1 (TID 207, <IP_ADDRESS>, partition 1,PROCESS_LOCAL, 1930 bytes)
16/07/28 05:28:48 INFO scheduler.TaskSetManager: Finished task 1.0 in stage 1.0 (TID 175) in 393261 ms on <IP_ADDRESS> (3/50)
16/07/28 05:34:11 WARN scheduler.TaskSetManager: Lost task 1.0 in stage 1.1 (TID 207, <IP_ADDRESS>): TaskCommitDenied (Driver denied task commit) for job: 1, partition: 1, attemptNumber: 207
16/07/28 05:34:40 INFO scheduler.TaskSetManager: Starting task 1.1 in stage 1.1 (TID 227, <IP_ADDRESS>, partition 1,PROCESS_LOCAL, 1930 bytes)
16/07/28 05:40:08 WARN scheduler.TaskSetManager: Lost task 1.1 in stage 1.1 (TID 227, <IP_ADDRESS>): TaskCommitDenied (Driver denied task commit) for job: 1, partition: 1, attemptNumber: 227
16/07/28 05:42:28 INFO scheduler.TaskSetManager: Starting task 1.2 in stage 1.1 (TID 250, <IP_ADDRESS>, partition 1,PROCESS_LOCAL, 1930 bytes)
...

1 task 1.0 in stage1.0 start
2 stage1.0 failed, start stage1.1.
3 task 1.0 in stage1.1 start
4 task 1.0 in stage1.0 finished.
5 task 1.0 in stage1.1 failed with TaskCommitDenied Exception, then retry forever.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Test build #63421 has started for PR 14557 at commit 1a1ea2f.

Test build #63421 has finished for PR 14557 at commit 1a1ea2f.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63421/
Test FAILed.

Test build #63424 has started for PR 14557 at commit 9263678.

Test build #63424 has finished for PR 14557 at commit 9263678.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63424/
Test PASSed.

Test build #63434 has started for PR 14557 at commit 9ea08e8.

Test build #63434 has finished for PR 14557 at commit 9ea08e8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63434/
Test PASSed.

Test build #63768 has started for PR 14557 at commit fbe31eb.

LGTM

Test build #63768 has finished for PR 14557 at commit fbe31eb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63768/
Test PASSed.

Merged build finished. Test PASSed.

There are multiple issues with this PR.  Some are at a more stylistic level, but some include deeper issues -- e.g. see SPARK-17064.  Most fundamentally, this PR is the wrong solution at least in the sense that it does not implement a minimal fix without other side effects.  The problem is that TaskCommitDenied is not being handled properly when a duplicate Task tries to commit a result that has already been successfully committed by another attempt of this Task.  The proper fix needs to be at that point of committing duplicate results, not by making the larger, unnecessary change in how we handle cancellation/interruption of other Tasks in a TaskSet when one of them produces a FetchFailed.

@shenh062326 I would rather like to propose to close this if there is no argument against ^.


Plugin E2E: Add support for testing alert query

What this PR does / why we need it:
This PR makes it possible to write e2e tests that verifies that a backend data source plugin can return data frames in a format that is compatible with alerting.
The process for testing alerting compatibility is as follows:

Use alertRuleEditPage fixture to open a new, empty alert rule edit page OR use the gotoAlertRuleEditPage fixture to open an already existing alert rule.
Get query editor row, set data source and fill in the query
Click the Preview button to evaluate the alert query. If successful, the data source is compatible with alerting.

This AlertRuleEditPage model follows the same pattern as all the other page models - it can be instantiated with or without an id. As mentioned above, it's exposed through alertRuleEditPage fixture (new alert rules) and gotoAlertRuleEditPage fixture (existing alert rules). I've also added a readProvisionedAlertRule that makes it possible to read a provisioned alert rule.
In practice, I think this will be useful to check for regressions in backend data source plugins. In the past, it has happened that changes to the backend code and how the data frames are being constructed leads to breakages in alerting. Running e2e tests that checks for breakages should be able to catch these regressions.
Which issue(s) this PR fixes:
Part of https://github.com/grafana/plugin-tools/issues/796
Will follow up with a docs PR later.

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

@josmperez do you want to have a look at the docs?

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

@josmperez can you take a look at the docs?


Bitmap mask and Graphics at higher depth causes error

Version

Phaser Version: 3.17.0
Operating system: Windows 10

Description
When using a BitmapMask and Graphics with a higher depth it causes an error:
Uncaught TypeError: Cannot set property 'TL' of undefined
Example Test Code
https://jsfiddle.net/ryhfg7zk/
var config = {
  type: Phaser.WEBGL,
  parent: 'phaser-example',
  width: 640,
  height: 480,
  scene: {
    preload: preload,
    create: create
  }
};

var game = new Phaser.Game(config);

function preload() {
  this.load.image('bunny', 'https://labs.phaser.io/assets/sprites/bunny.png');
  this.load.image('phaser2', 'https://labs.phaser.io/assets/sprites/phaser2.png');
  this.load.image('checker', 'https://labs.phaser.io/assets/pics/checker.png');

}

function create() {

  var checker = this.make.image({
    x: game.config.width / 2,
    y: game.config.height / 2,
    key: 'checker',
    add: true
  });

  var bunny = this.make.sprite({
    x: game.config.width / 2,
    y: game.config.height / 2,
    key: 'bunny',
    add: true
  });

  var phaser2 = this.make.sprite({
    x: game.config.width / 2,
    y: game.config.height / 2,
    key: 'phaser2',
    add: false
  });

  bunny.mask = new Phaser.Display.Masks.BitmapMask(this, phaser2);
  bunny.depth = 200;

  graphics = this.add.graphics();
  graphics.depth = 300;

  this.input.on('pointermove', function(pointer) {
    graphics.clear();
    graphics.lineStyle(3, 0xff0000, 0.8);
    graphics.strokeCircle(pointer.x, pointer.y, 10);
    phaser2.x = pointer.x;
    phaser2.y = pointer.y;

  });

}

I tested this on the lastest dev/master and the same issue occurs.

Looking into this, the depth makes no difference, it just has to be a Graphics object in the display list following something with a BitmapMask on it, to break it.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Leo westebbe/issue 304 show user data

In response to #304. UserTable now loads a list of Users (aka surveyors), and clicking on a user then loads that user.
I'm not sure if this is a good way to set up with react-hook-form. I set the value prop for each <TextField/>  to conditionally load its data from the RTK-query hook, or the initial value if null. If someone on our team is an expert with this library, please assign as a reviewer!

Could you rebase this off main without the jsonserver changes? It'll make it easier to review.

Could you rebase this off main without the jsonserver changes? It'll make it easier to review.

@mattdelsordo Done

It still looks pretty janked up, would it be alright if I had a go at it?

So I think some git nonsense has occurred and these changes got included in your other PR and are already on main, but in that case they've been reviewed and everything looks good, so we can probably just close this as well.

So I think some git nonsense has occurred and these changes got included in your other PR and are already on main, but in that case they've been reviewed and everything looks good, so we can probably just close this as well.

You're right Matt. I'm going to close this issue.


HTML Framerate Dependent on Window Size, Despite "viewport" stretch mode

Tested versions

Reproduceable in: 3.5.3, 4.2.1

System information
Windows 10, Godot 4.2.1 and 3.5.3, RTX 3070, Driver 546.01, Vulkan
Issue description
If I set display stretch mode to "viewport" and stretch aspect to "keep", resolution 640x480. I get a loss of frames dependent on my browser window size. 44 fps for 1080p, 24 fps for 1440p. Since the game is render at 640x480 always, shouldn't the frames be the same? This is with a blank 3d scene, just a camera.
Steps to reproduce
New project:
resolution = 640x480
stretch mode = viewport
stretch aspect = keep
make a 3d scene with a camera, and a canvas layer with a label, attach a script to the label with
func _process(delta):
	text = str(Engine.get_frames_per_second())

run web export, and change the size of the browser window to watch framerate go up and down.
Minimal reproduction project (MRP)
bezier.zip

related to:

#36507
https://github.com/godotengine/godot-proposals/issues/6221

I didn't have hardware acceleration on.... This issue is basically non-existent when you actually use your dang hardware.

I didn't have hardware acceleration on.... This issue is basically non-existent when you actually use your dang hardware.

For future reference, you can check whether hardware acceleration works in Chromium-based browsers by opening chrome://gpu in the browser's address bar.


Block: IDJKids

Channel ID
IDJKids.rs
Reference
https://github.com/iptv-org/iptv/issues/6486
Notes (optional)
IDJKids is owned by United Media, which has requested a removal request in the past.

https://github.com/iptv-org/database/pull/7543

Reopening. https://github.com/iptv-org/database/pull/7543#issuecomment-1806942897

https://github.com/iptv-org/database/pull/7543#issuecomment-1806945693


Add gamepad support for desktop navigation, as well as built-in TDP adjustment

Suggested new feature or improvement
Users of PC gaming handhelds are having difficulty navigating around the Windows desktop compared to Valve's SteamOS distro of Linux. Both Asus and Lenovo have developed their own solutions but neither support the main Windows desktop interface. It would be nice if Microsoft could make gamepads work better for basic tasks on the Windows desktop so that OEMs do not have to do their own heavy lifting. A pop-up virtual keyboard controlled by the gamepad would also be useful whether in-game or in basic tasks like web browsing. And TDP control through a quick menu pop-up (and also accessible via gamepad) would help enthusiasts gain better control over power consumption.
I know the dev team is unlikely to comment on speculation regarding the supposed Windows 12 release, but I would love to see this feature trickle down to Windows 11 as well.
At the end of the day, the gamepad isn't ready to substitute a mouse and keyboard for Windows's most advanced features, but it should be available for the most essential features, so that many users would not need to use either a mouse and keyboard, or the touchscreen, to be able to use Windows. The gamepad can even be mapped to all 10 digits to unlock Windows with the 4-digin PIN, as demonstrated on PS5 and Steam Deck. This configuration would not only benefit PC gaming handhelds, but also PCs that are plugged into a television, perhaps for a home entertainment setup.
If this feature request has reached the wrong repo, I would love to be redirected to a more appropriate repo should a link be available. 🙂 I did submit this as feedback via the Feedback app in Windows, but I'm worried devs don't notice it there so I chose to post here as well.
Scenario
Navigating the desktop through a gamepad can allow a user to launch the Xbox app, Steam, and/or other 3rd party game launchers similar to how a user would with a traditional mouse and keyboard configuration. It would also allow the user to perform simple tasks such as web browsing without the need for either a mouse and keyboard, or a touchscreen.
A virtual keyboard controlled by the gamepad would allow for text input in-game or in web browsing, without depending on an actual mouse and keyboard nor a touchscreen.
TDP control via a quick menu accessible via gamepad would allow power users to control power consumption, perhaps to extend battery life when using applications of low power demand.
Additional details
No response

Thanks for the request! The Windows team actively looks at Feedback Hub, so I'm sure they saw it :)
Given that this is a Windows enhancement, I don't think this makes sense on this repo. My best recommendation would be to file on Feedback Hub, which you already have 😄


BDay offset behaviour vs docs & implementation

apologies if this has already been raised but couldn't find an answer.
from the docs :
DateOffsets can be created to move dates forward a given number of valid dates.  For example, Bday(2) can be added to a date to move it two business days forward.  If the date does not start on a valid date, first it is moved to a valid date.
pd.Timestamp('20150830')+BDay(0) == pd.Timestamp('20150830')+BDay(1)  is True
it doesn't move it first to a valid date and the code seem to confirm that. not sure what's the intended behaviour here but would be great to clarify. thanks in advance.

The doc-string of DateOffset is pretty clear on this. The 0th day is defined to rollforward.
In [2]: pd.offsets.DateOffset?
Init signature: pd.offsets.DateOffset(self, n=1, normalize=False, **kwds)
Docstring:
Standard kind of date increment used for a date range.

Works exactly like relativedelta in terms of the keyword args you
pass in, use of the keyword n is discouraged-- you would be better
off specifying n in the keywords you use, but regardless it is
there for you. n is needed for DateOffset subclasses.

DateOffets work as follows.  Each offset specify a set of dates
that conform to the DateOffset.  For example, Bday defines this
set to be the set of dates that are weekdays (M-F).  To test if a
date is in the set of a DateOffset dateOffset we can use the
onOffset method: dateOffset.onOffset(date).

If a date is not on a valid date, the rollback and rollforward
methods can be used to roll the date to the nearest valid date
before/after the date.

DateOffsets can be created to move dates forward a given number of
valid dates.  For example, Bday(2) can be added to a date to move
it two business days forward.  If the date does not start on a
valid date, first it is moved to a valid date.  Thus psedo code
is:

def __add__(date):
  date = rollback(date) # does nothing if date is valid
  return date + <n number of periods>

When a date offset is created for a negitive number of periods,
the date is first rolled forward.  The pseudo code is:

def __add__(date):
  date = rollforward(date) # does nothing is date is valid
  return date + <n number of periods>

Zero presents a problem.  Should it roll forward or back?  We
arbitrarily have it rollforward:

date + BDay(0) == BDay.rollforward(date)

Since 0 is a bit weird, we suggest avoiding its use.
File:           ~/pandas/pandas/tseries/offsets.py
Type:           type

the problem is not with 0th day but with 1th and this is the part of the doc that is confusing : If the date does not start on a valid date, first it is moved to a valid date.  so you would expect to have
pd.Timestamp('20150830')+BDay(1) to add 1 BD after its moved to a valid BD and its not.

this has been this way for quite some time, eg.. I think since this was written.
so BDay(0) == BDay(1) if the anchor is not a business day.
will reopen for some comments
cc @sinhrks
cc @chris-b1
cc @cancan101

It's kind of inconsistent with the Period behavior e,g:
In [59]: d = pd.Timestamp('20150830')

In [61]: (d.to_period('B') + 1).to_timestamp()
Out[61]: Timestamp('2015-09-01 00:00:00')

In [64]: (d + BDay(1))
Out[64]: Timestamp('2015-08-31 00:00:00')

But I also feel like the ship may have sailed - I believe all of the other anchored offsets work like this too:
 In [66]: d + pd.offsets.MonthEnd(1)
 Out[66]: Timestamp('2015-08-31 00:00:00')

the period transform is more like:
In [5]: (d+pd.offsets.BDay(0)) + pd.offsets.BDay(1)
Out[5]: Timestamp('2015-09-01 00:00:00')

so actually ok with that. I think this could use some docs expanding on exactly this.

It is related to #10575, whether business day should implicitly anchor to the next valid date. CC  @maximilianr

As far as the Period implementation goes: I find it unintuitive on gaps. If you ask for the business day and pass in a Sunday, do you really want Monday?
https://github.com/pydata/pandas/issues/10575

OK, we crossed there @sinhrks. Great minds think alike (but fools seldom differ?)


Add adapter for X-ray reasoner

We'd like to experiment with slurping slices of the X-ray graph (already conformant with tkg standard), and if they are amenable, allowing them to slurp slices in.
This should turn out to be a generic cypher call, no need to transform as in the monarch case

Hi Chris,
Absolutely.  Team X-ray is happy to engage on this.  It is a good proof-of-concept that the KG standard will facilitate KG ingestion or merging.
Steve

This seems to be done as part of the transformers written for Neo4j.
@cmungall Correct me if I am misunderstanding this issue.


There is no app to download

Hello.
There is no app to download here Application download

Incorrect link was fixed

Thank you so much!


can't 'jbuilder install' automatically imply 'jbuilder build @install'

Of course, when I order to install things, things should be build before.
$ jbuilder install
The following <package>.install are missing:
[...]
You need to run: jbuilder build @install

The rationale for jbuilder install not building things is that sometimes people build as their user and install as root. It's not uncommon to do: make && sudo make install. In this case, we don't want the build to run as root

In that case, maybe an option to the install command would do the trick.
For example:
jbuilder install --build-if-necessary / jbuilder install --build

Also, let's assume that jbuilder does nothing when things are already build.
Your use case would still work:
jbuilder build @install && sudo jbuilder install

alias 'jbuilder install --build-if-necessary'='jbuilder build && jbuilder install' should solve the problem personally for you.
Personally, I don't think such an option is a good idea.

I think that in most makefiles, make install triggers the build if necessary.

I suppose we could  make it more makefile like by making uninstall/install aliases. But then you can't really pass extra params to install. And you always want to avoid buildings things with sudo as diml mentioned. The benefits seem quite low...

If other users want/expect this, let's hope they will manifest themselves here.
If I am the only one bothered, it is fine to close this after some time.

I personally think the present behaviour is better, but the GNU standards, for example, recommend that make install should build the program if make all hasn't been run (with the caveat that if make all has been previously run, then make install should not alter the build tree so that make all; sudo make install shouldn't build anything as root). See their standards document for more info.

I'm also in favor installing implying building. This is the behavior I would expect, and the opposite can lead installing stale files when one forgets to run build. It would also be consistent with the behavior of jbuilder exec after #152 is implemented.

@copy you're referring to the enhancement that $ jbuilder exec foo should rebuild foo first?

@copy you're referring to the enhancement that $ jbuilder exec foo should rebuild foo first?

Yes.

As a compromise between the security consideration of not wanting to build as root but wanting the convenience, perhaps we could emit a warning if the build artefacts are stale. Or possibly emit the warning if the artefacts are stale and the various detections for sudo/su/root reveal that jbuilder may be being run elevated, but otherwise rebuild first? These would be hard to do for make but relatively easy to do for jbuilder.


Determinant Calculator

@Sulagna-Dutta-Roy I wish to contribute to your repo by adding a determinant calculator which will calculate the determinant of an nXn matrix

Kindly use feature request for raising the issues


Investigate changes to Formulas (Formulas 2.0)

Notion recently launched Formulas 2.0, which have been  ported to the API in September.
We should look into what it’d take to add support.

I’d like to collaborate on this!


Speech to text lab with Raspberry Pi

The cart contains a Raspberry Pi, along with a USB speakerphone containing a microphone and speaker.
One of the labs we need to use this is speech to text based off the Azure cognitive services speech service, all coded in Python.
There isn't a Speech SDK in Python for the Raspberry Pi at the moment, so this lab should use the REST API.
Hardware

Raspberry Pi 4
M0 USB speakerphone

Lab parts
This lab should focus on the Speech to Text scenario, with output to the console. As this lab needs to explain all the steps it needs to be documented either as markdown, or a Jupyter notebook that can be run on the Pi.
The lab should have a structure something like:

Setting up the Pi (ideally with links to existing content such as the Microsoft RPI resources
Setting up Jupyter Notebooks (if needed) - this can link to the existing documentation on this in this repo
Setting up the Cognitive Service - ideally using a link to the relevant documentation in Microsoft Docs
Recognizing speech and outputting the text

All labs should follow the lab contribution guidelines.

Implemented by #30


XSS in examples

Hello,
just been playing around with the examples and i discovered XSS in the route-map example.
starting the app like so
~/express$ node examples/route-map/
get /users
delete /users
get /users/:uid
get /users/:uid/pets
delete /users/:uid/pets/:pid
Express started on port 3000

Then browsing to the following causes the injected javascript to load
 http://ip:3000/users/%22%3E%3Csvg%20onload=prompt()%3E
full request
GET /users/%22%3E%3Csvg%20onload=prompt()%3E HTTP/1.1
Host: <IP_ADDRESS>:3000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate
Connection: close
Upgrade-Insecure-Requests: 1

the issue seems to be with the following in the codebase
var users = {
  list: function(req, res){
    res.send('user list');
  },

  get: function(req, res){
    res.send('user ' + req.params.uid);
  },

  delete: function(req, res){
    res.send('delete users');
  }
};

where req.params.uid is not sanitised
thanks

While I am not too worried because the examples are just to show express features, not real use, I think it would be good to do one of two things here:

Add a comment about why rendering user content without escaping is bad
Escape the input

I would approve a PR which did either of the above.

https://www.owasp.org/index.php/Data_Validation#Sanitize
uh....

If it's within the examples odds are this is out in production somewhere. As a developer would look to examples either first starting out with the framework or just day to day. I can speak from experience on developers using example routes to build out there applications. Therefore since there is the possibility of it existing at least once within the base code, it's not a stretch for it to have been implemented into someone's production application.

By all means, fix it and submit a PR :)

Hi @Caprico1 and no one is claiming otherwise. But of course fixing the example won't somehow fixes those theoretical production deployments, for example. That's also why the suggestion was as part of a pull request to fix the issue, to add information for the users who are looking at the examples to understand escaping and why the example is doing it, etc.

Right, it was to explain in an issue why this would be bad if in prod (owasp link)
and @jthorpe6 and I's thought process on how this is an issue.
I'll look into making a solution or at the least adding an example to sanitize inputs within routes within the framework.

Adding sanitization may not fix the existing codebases problem but it will address future users.

Speaking of security issues in the examples, it is possible to read any file (directory traversal) on a server running downloads example.
%2E%2E%2F is decoded as ../ and causes path.join() to go to a parent directory.
https://github.com/expressjs/express/blob/e1b45ebd050b6f06aa38cda5aaf0c21708b0c71e/examples/downloads/index.js#L19-L20
And for example, it could be used to read a session secret:
http://localhost:3000/files/..%2F..%2Fsession%2Fredis.js


Include sparkle and grey overlay effect on post-list sidebar too.

On the homepage, the sideblock has a really attractive grey overlay and star pattern. Is it possible to have this on the post-lists page too?
It doesn't need to be included upstream, I'd just appreciate instructions on how to achieve this myself.

I managed it by copying the values for the homepage to the values for the posts list page. Sparkle wasn't added.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/176 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit f1c4ab6:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-11-13-35-20 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-11-13-35-20

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Tutorial  example: Virus network cannot animate

Trying to replicate the animation for virus network:

ttributeError                            Traceback (most recent call last)
 in
15 fig, axs = plt.subplots(1, 2, figsize=(8, 4)) # Prepare figure
16 parameters['population'] = 50 # Lower population for better visibility
---> 17 animation = ap.animate(VirusModel(parameters), fig, axs, animation_plot)

~/monocondaNew/lib/python3.7/site-packages/agentpy/analysis.py in animate(model, fig, axs, plot, steps, skip, fargs, **kwargs)
136         plot(m, axs, *fargs)  # Perform plot
137
--> 138     ani = matplotlib.animation.FuncAnimation(
139         fig, update, frames=frames, fargs=(model, axs, *fargs), **kwargs)  # noqa
140
AttributeError: module 'matplotlib' has no attribute 'animation'
what am I missing here?
Thanks

Please try to update your matplotlib library to the latest version (3.3.3) and let me know if the error persists.
In the next update I will try to declare version-specific dependencies so that this shouldn't happen anymore.

Please try to update your matplotlib library to the latest version (3.3.3) and let me know if the error persists.
In the next update I will try to declare version-specific dependencies so that this shouldn't happen anymore.

Yup, thank  you. That did the trick.
Thank you for authoring agentpy. The python native integrated visualization is great. I dont need to write JS like in mesa.
Well done.

Yup, thank  you. That did the trick.
Thank you for authoring agentpy. The python native integrated visualization is great. I dont need to write JS like in mesa.
Well done.


Fix #919

Closes #919
Bump pipeline version to 3.11

Subworkflow updated on nf-core/modules in https://github.com/nf-core/modules/pull/2711


Made a PKGBUILD and added to Archlinux AUR repositories

https://aur.archlinux.org/packages/virtwold

That's great!  I updated the readme with that info.  Thanks for the help!


don't mix slots and attributes in Database

good idea, but this might not fix the problem in which FIeldResearchSite wasn't quotes in forward references


Add support for Signify plugin

I realise a feature of this plugin is that its simple and fast, with a low code line count. That realised, this pull request squeezes in support for Signify.
The first commit adds that support.
The following three commits are refactoring, separated to show the idea behind each change. These can be squashed if the pull request is approved. The last commit, avoiding duplicate code, follows the same style that is used for errors, warnings and information (in the same function).

Outstanding contribution.
Very well structured; even nicer than what I had before (due to my poor Vimscript skills).
I am more than happy to merge.
Question; I am about to embark on a new project named nvim-linefly which will be a pure Lua re-imagining of this existing Vimscript statusline plugin but for modern Neovim only (0.8 and above). Aims will be to be even simpler and smaller than this plugin so I was planning to exclude support for some plugins. My plan was to only support:

Native Neovim diagnostics (no ALE, no Coc)
Only support Gitsigns (no GitGutter, and now no Signify).
Only support nvim-web-devicons (no vim-devicons)

I am hoping to have this available for Neovim 0.9 which is due in about a month or. So I kind-of want Neovim 0.8 and 0.9 users to not use mistfly, but use the pure linefly instead. I may emit a warning message of sorts. The user community of this plugin is so small that I don't think that will be that big a deal.
Any objections? Would you require Signify in a pure Lua plugin; it seems to me that most Neovim folks who have gone pure Lua will not be using Signify. My feeling, no evidence for that.
This plugin, mistfly, will continue to be maintained since I still use it for my rare Vim sessions; I am a Neovim user, but I do still use Gvim occasionally.
Cheers, and thanks for the contribution.

Thank you for merging.
I have just started learning Vimscript, hence could only rate my skills at sub-poor :)
Half a year ago I was introduced to Vim plugins and Neovim. I am slowly working through all that these offer. I went for over a decade with a very minimal vimrc and a single plugin (for space highlighting). I have a lot of catching up to do!
While learning and comparing plugins I came across a few support issues, then decided to make some minor contributions.
I am moving towards being a Neovim user, and currently using Gitsigns. For Vim I still use Signify. So no objections to linefly only supporting Gitsigns.
Would you mind if I continue to contribute towards mistfly?
I am looking forward to trying out linefly!

Would you mind if I continue to contribute towards mistfly?

Sure, maybe even linefly as well. Sensible contributions are always welcome, such as this Signify PR.
Note, the philosophy will always be about minimalism, 300-500 lines-of-code; small selection of plugins, high performance; that type of thing.
My statusline plugins are not intended to compete with: lualine, lightline or vim-airline.
Best regards.

I am looking forward to trying out linefly!

Probably ready in about a week or two; stay tuned at linefly.
It will basically look and feel the same as this plugin; but be for modern Neovim-only written exclusively in Lua; so even better performance than mistfly via Neovim Lua JIT and lazy.nvim byte-compilation.
"Blazingly fast" as some YouTubers say :)

linefly plugin is now available, if of interest, if not, that's ok too.

Sorry for my late reply. Recent lack of free time.

Sure, maybe even linefly as well.

Awesome. Thanks. Hopefully I can contribute to linefly as well.

Note, the philosophy will always be about minimalism,

Yes, I appreciate that. I started out with vim-airline since it was used in a colleague's configuration. Then only recently took the time to go through its documentation and set it up to my liking.
Next I decided to take a look at mistfly, due to its simplicity and how it looked. There're a couple of features I miss, however I'm cautious to suggest additions, since no doubt other users have different sets of desired features. I'm currently seeing if I can do without, or find workarounds.

linefly plugin is now available

Great! Congratulations. Hopefully soon I'll give it a go.

There're a couple of features I miss, however I'm cautious to suggest additions

Out of curiosity, what are they?
vim-airline and mistfly / linefly are not operating in the same space. The former is a comprehensive statusline infrastructure, the later two are just a little more advanced then the stock out-of-the-box statusline.
vim-airline better matches up with lualine; if one is thinking about moving from one statusline to another.
Cheers.

Out of curiosity, what are they?

I will definitely get back to you on this.
I had a PR ready with a minor tweak to the selection of the Git status plugin, however I will need to rework it with the introduction of mistflyWithGitStatus.

Yes, sorry, there have been changes due to the introduction of nvim-linefly. I wanted internal option name consistency as well as commonality between this plugin and the new nvim-linefly plugin.
Good news, is that all the work is now complete.
So you should be safe to proceed.
Cheers.

No problem at all. My one line tweak became a two line tweak.
Just need to test the change.

Out of curiosity, what are they?

Hi. Sorry it took a while to get back to you.
I haven't had a chance to try out linefly. At work I'm using Neovim, and still with vim-airline. While at home I'm using Vim with mistfly and nightfly.
(I have a question about use ofnightfly with Vim verses Neovim, is there a particular place where I should ask that?)
The two features I've missed going fromvim-airline tomistfly are:

Spell status and Paste mode (I use the latter far less)
Search count

For spell status and paste mode the workaround I've used is to show the value after changing it, e.g., :set spell! spell?. This works well enough, with a double tap of the mapped key to show the current value.
Spell status - I don't have a strong reason to add it to mistfly, other than I find it quite useful.
Paste mode - I believe it's in Vim's standard status line (so why not throw it in with spell status).
For search count, I don't have a workaround for Vim (I see that Neovim displays search count).
I've created the commits to add spell status, paste mode and search count tomistfly and will create the pull request if there's hope of at least search count getting in (and if there's not a workaround for it).

(I have a question about use ofnightfly with Vim verses Neovim, is there a particular place where I should ask that?)

Open an issue at the nightfly repo.

The two features I've missed going fromvim-airline tomistfly are:

Spell status and Paste mode (I use the latter far less)
Search count

I am interested in these, especially for linefly, I even created this issue a couple months ago; specifically I want:

Macro status (recording started, recording ended)
Search count
showcmd status

This is especially useful in Neovim when cmdheight=0 is set. If possible, adding them here in mistfly would also be possible (though showcmd via showcmdloc is only available in modern Neovim I believe).
My query would be, one flag for all these items or multiple flags individually? Maybe a single item which is a dictionary (Vimscript) or table (Lua) for the options of interest?

I've created the commits to add spell status, paste mode and search count tomistfly and will create the pull request if there's hope of at least search count getting in (and if there's not a workaround for it).

Firstly, I want to get a feel of what an appropriate API would be, and where these items would be displayed in the statusline.
One of the core principles of mistfly / linefly is no jank when navigating and doing work; so the existing left and right sides of mistfly / linefly should stay as is. Any of these new items should be displayed somewhere in the centre or a bit left / right of centre in the statusline.
To-be-determined.
Yes, I am interested in incorporating these extra items, especially in linefly.

Open an issue at the nightfly repo

Thank you. Will do.

I am interested in these

Oh great. As mentioned, I could get by without spell and paste status, however I am missing search count.

one flag for all these items or multiple flags individually?

Currently I've got one option for spell and paste, and one for search count. I also tried three options, one for each.

Maybe a single item which is a dictionary (Vimscript) or table (Lua) for the options of interest?

That might be nice. Although I don't have knowledge of option standards to weigh in.

where these items would be displayed in the statusline.

Currently I have them all on the right side. That is near where Neovim displays search count (at least for me). For paste mode status, the default Vim status line has it with the mode. vim-airline does the same and also places spell status there. However that messes with the fixed layout of the left side (file name and branch name).
For example:
                19:1 ⎪ 42 ↓All ⎪ Spc:2

        Spell ⎪ 19:1 ⎪ 42 ↓All ⎪ Spc:2
        Paste ⎪ 19:1 ⎪ 42 ↓All ⎪ Tab:8
  Paste Spell ⎪ 19:1 ⎪ 42 ↓All ⎪ Tab:8

        [5/8] ⎪ 19:1 ⎪ 42 ↓All ⎪ Spc:2
[5/8] ⎪ Spell ⎪ 19:1 ⎪ 42 ↓All ⎪ Spc:2

Any of these new items should be displayed somewhere in the centre

I wonder if search count could be a single addition to the right (similar to how Git status pops up and disappears on the left as one works), and the other new items in a new group near the centre?
My branch currently has a commit for spell and paste status, and on top of that a commit for search count. I could reorder these commits and create a pull request for only adding search count as a place to start?
For search count, I also wondered about having simply 5/8 rather than [5/8], however kept the brackets since it seems standard.

In the interest of brevity, I am inclined to not implement either paste nor showcmd support.
That leaves the following which are of prime interest to yourself and myself:

Macro recording -> mistflyWithMacroState
Search count -> mistflyWithSearchCount
Spell -> mistflyWithSpellStatus

Your positioning of search count and spell status seems ideal (in the previous post).
Macro recording state seems better positioned in the middle of the statusline.
I will devote time this week to implementing each of these in mistfly and linefly plugins, especially so because this has been in the back of my mind for a couple months now; so it is time for me to just get it done.

Great!
Leaving out paste sounds right. I think I included it for fairness, and probably since vim-airline has the two grouped together (with mode). Even without its status displayed via paste? I don't think I'd miss it; I usually end up pasting, undoing, toggle paste, and pasting again.

I will devote time this week to implementing each of these in mistfly and linefly plugins

I removed adding paste mode status from my commit and created a pull request. I hope you don't mind; I didn't want the work to go to waste, and I'm glad to be able to contribute. Even if you only use this as a reference or as a base for your work, or if it's close enough to the mark I would be more than happy to fix it as required.
I'll add some remarks about the implementation in the pull request.

with_macro_status, with_search_count and with_spell_status have now been ported to linefly.
Good result. I am even using some of these new options myself.
Many thanks.

Great!
I'm about to move to linefly for my Neovim configuration.

Quick note, Signify (and certain other Vimscript plugins) is/are not supported by linefly. I narrowed down plugin support to ones that the majority of Neovim users are using.
In Neovim land Gitsigns is the defacto Git decoration plugin. It is actually very nice, and it is maintained by one of Neovim's core contributors (Lewis).

Quick note, Signify is not supported by linefly.

Thank you for the heads-up. I'm using Signify for Vim, and Gitsigns for Neovim, so all good.


dnsmasq: enable nftset support

Description of changes
Enable nftset feature of dnsmasq. dnsmasq supports adding to nftsets in addition to ipsets since 2.87.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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.

I didn't do test compilations on all dependent packages because there're 700+ of them, but I use nixpkgs with this PR for my own system and it all builds fine.

Thanks!


Not working on IOS

Doesn't appear to work with the native browser or chrome on iOS.

Commit a6b966b may have fixed this - need to test on an ios device to confirm:
a) App is starting on press.
b) App is filling as much of the display as possible (full screen not possible on iOS).

Commit 03d5600 has finished the fixed.


SGImageCache is always reading from disk!

I'm using SGImageCache but while I'm scrolling I see there is a little low frame rate on my iPhone 6! I checked in Xcode debug navigator, and noticed disk reading won't rest! means SGImageCache all the time is reading from disk asynchronously!
customImageView.setImageForURL(imageUrl, placeholder:UIImage(named: "food-default"), crossFadeDuration: 0.2)

Hi @hashemp206 Can you let me know where in the SGImageCache code the disk access is happening for you?  Can you get the stack trace from the debugger?
Cheers

disk reading never rest until I stop scrolling!
here is the stack trace:

Do you mean it's calling [SGImageCache getImageForURL] for every pixel scrolled, or once every time a collection view cell comes on screen?
If it's happening for every pixel moved, the call to get the image might be in a bad place.
Are you able to paste the code where you're calling the image fetch from?

I'm calling the getImageForURL when cell appears on screen (checked).
func configureForItem(menuItem: MenuItem, withQuantity quantity: Int) {
if let imageUrl = menuItem.imageUrl {
            SGImageCache.getImageForURL(imageUrl).swiftThen({[unowned self] (object: AnyObject ) -> (PMKPromise?) in
                self.customImageView.image = object as?
Curious what you're expecting to see here.  Assuming these images are already cached to disk, I expect to see each image loaded in from disk on a background queue once as each collection view cell comes on screen.

image size averagely about 35 KB on disk. I expected these images cached from disk to memory and not all the time fetched from disk.

Would you be able to give this pattern a try? (sorry for the obj c pseudo-code but should be easily translatable into swift)
if ([SGImageCache haveImageForURL:url]) {
    imageView.image = [SGImageCache imageForURL:url];
} else {
    imageView.image = nil;
    [SGImageCache getImageForURL:url].then(^(UIImage *image) {
        if (image) {
            imageView.image = image;
        }
    });
}

nothing changed! still issue exist!

I'm running out of ideas ;)  I don't expect the disk usage on small images to cause stuttering (we even use it with no stuttering on iPhone 4 on reasonably large images).  What does the Xcode profiler say is using up cpu / gpu / causing the framerate stuttering?

guess what? in prepareForReuse methods  I was setting imageView.image with placeHolder image on the main thread!
self.customImageView.image = UIImage(named: "food-default")

@hashemp206 Just a quick comment, you may want to use fewer exclamation points when making github comments. It makes it sound like you are yelling at us, when what we're trying to do is help you out. Thanks :)


Prevent overwrite of existing TCP stream connection

Removed $socket->connect();
On line 109 the function authenticateBolt is called that calls the Bolt function init.
Init does the also a $socket->connect(); and overrides the existing PHP TCP Stream resource.
The lost (overwritten) resource is never closed, resulting in an warning of a not nicley closed connection.
WARN [o.n.b.t.TransportSelectionHandler] Fatal error occurred when initialising pipeline, remote peer unexpectedly closed connection.

Great work :100:,
I don't know why some tests fail. It seems like a CI problem. I'll merge and release it later today!
Thank you very much :pray:

It seems the Ci is missing credentials or so, maybe secrets are missing because it's my fork?

Seems like an fair assesment! I'll run the tests locally and turn of the full tests for PRs as a short term solution
Sent from ProtonMail mobile
-------- Original Message --------
On 6 Dec 2021, 11:16, Tim wrote:

It seems the Ci is missing credentials or so, maybe secrets are missing because it's my fork?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

The fix is now released in version 2.3.2 :tada:


Can not add staticman_v2 comments

[x] This is a question about using the theme.
[ ] This is a feature request or enhancement.
[x] I believe this to be a bug with the theme.
[x] I am using the latest version of the theme.
[ ] I have updated all gems with bundle update.
[ ] I have tested locally with bundle exec jekyll build.

Environment informations

**Minimal Mistakes version: forked yesterday
GitHub Pages hosted: yes

Expected behavior

I want to be able to use staticman comments.
Steps to reproduce the behavior

add staticman and authorize it
configure staticman
changed the entry of jekyll.environment == production to jekyll.environment != production because it doesnt seem like you can change the environment from inside github pages
add comments: true to your post
provider: staticman in the config
still no comments

layout single: https://hastebin.com/axovasoyaq.scala (used in my posts)
a post: https://imluc.ga/blog/first-test/
_config.yml https://hastebin.com/oxahebofow.bash

edit: the comments js shows up in the f12 menu

GitHub Pages automatically sets the environment to production when they build your site.
So you changing the conditional to != likely broke it.
Is this a public repo? I need to see the rest of your files so I can reproduce and run a build locally. I can almost guarantee it's not a bug but a setting that was omitted.

yes, that fixed it! I likely forgot to enable comments or something along those lines


Make OCF Perceiver_Pytorch pip installable

Adds OCF's pytorch to PyPi with name perceiverio so that we can then use pip install perceiverio the same as Skillful Nowcasting and MetNet models.

Is it worth a little tidy up of the following files
from perceiver_pytorch.perceiver_pytorch import Perceiver
from perceiver_pytorch.perceiver_io import PerceiverIO, PerceiverLM
from perceiver_pytorch.multi_perceiver_pytorch import MultiPerceiver
Some of them have duplicate functions in them.
Also would love a bit of descirption for each one, maybe + paper reference.
I think Perceiver was made by the original repo - so maybe just concentrate on description of the new models

Is it worth a little tidy up of the following files
from perceiver_pytorch.perceiver_pytorch import Perceiver
from perceiver_pytorch.perceiver_io import PerceiverIO, PerceiverLM
from perceiver_pytorch.multi_perceiver_pytorch import MultiPerceiver
Some of them have duplicate functions in them.
Also would love a bit of descirption for each one, maybe + paper reference.
I think Perceiver was made by the original repo - so maybe just concentrate on description of the new models

Okay, yeah, I'll make changes to those

@peterdudfield I've added a bit more documentation and paper links, and refactored some more of the common functions. I also removed the experimental and PerceiverLM as I don't think either of those are useful to what we are doing.


details about parameters and hyperparameters

Hi, could you share some hyper-parameters details while training biLM？like optimizer, dropout rate? Thanks.

We used Agagrad with learning rate = 0.2, batch size = 128 * 20 tokens on each GPU across 4 GPUs (with synchronous gradient updates).  Dropout rate was 0.1 on input/output to each LSTM (no recurrent dropout, following Zaremba et al (https://arxiv.org/abs/1409.2329).


Change MSI name to ‘CLI 2.0 for Azure’

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2737   +/-   ##
=======================================
  Coverage   61.13%   61.13%
=======================================
  Files         480      480
  Lines       25679    25679
  Branches     3886     3886
=======================================
  Hits        15698    15698
  Misses       9055     9055
  Partials      926      926

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 8829361...967299d. Read the comment docs.


Delete spurious XX Collection Record

Deletes XX Collection Record line on show Observation page when the herbarium collection is not searchable via a web link.

coverage: 93.524% (-0.002%) from 93.526%
when pulling b1b9455a8c0f19bf349a79ed6711f852210d4105 on jdc-false-collection-record
into f762dddfdfda2c3cb336d876c04f53d2974bbe92 on main.


stable/23.2 - Retry CRI requests when containerd is not initialized

Containerd does not report correct error code in this case yet.
Thus we have to check error message.
Link: https://github.com/containerd/containerd/pull/9565
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/296
(cherry picked from commit 2bd7999c6c09fef0ecabfbe730bfd16ca3507144)

Shipped!


Arbitrary files overwrite

Hi guys!
Just checking this very nice project.
While trying to build using npm, a high severity bug alert popped up:

Is this dependency going to be updated so that we can sleep safe and sound? 😀

Fixed :)

Fixed :)


Update README to clarify project status

Thanks @hmhealey updated, nice find :)

Skipping Jenkins because it's just a README change