mount_translated_uploader always populates with string of "Uploader" object

Hi there,
I've followed the instructions, and boiled down, the code in use is:
class Bob < ActiveRecord::Base
  translates :message, :button_text, :message_footer, :image_landing_url, :square_image
  extend CarrierwaveGlobalize
  mount_translated_uploader :square_image, SquareImageUploader
end

Uploader:
class SquareImageUploader < CarrierWave::Uploader::Base
  include CarrierWave::MiniMagick
  storage :file

  process resize_and_crop: 800

  private

   def resize_and_crop(size)
    manipulate! do |image|
      if image[:width] < image[:height]
        remove = ((image[:height] - image[:width])/2).round
        image.shave("0x#{remove}")
      elsif image[:width] > image[:height]
        remove = ((image[:width] - image[:height])/2).round
        image.shave("#{remove}x0")
      end
      image.resize("#{size}x#{size}")
      image
    end
  end

end

The problem happens when the resulting form (relevant snippet):
= f.fields_for :translations, bob_translations do |ft|
        .col-md-4
          .panel.panel-default
            .panel-heading
              = translation_locale_name(ft.object.locale)
            .panel-body
              = ft.hidden_field :locale
              = ft.file_field :square_image

The form has multipart also set.
But all of the values posted to the fields being saved as string representations of the upload file object, like "#ActionDispatch::Http::UploadedFile:0x00007fd5d44c99e0" - which is not useful.
Any hits for where this might not be working?

For anyone who comes across this: I found the same issue when integrating ActiveAdmin with Globalize + Carrierwave. It boiled down to the fact that mass-assignment to the nested translations attribute won't work because there's no mounted uploader on the Translation class. I've changed this gem like this:
https://github.com/foxtacles/carrierwave_globalize/tree/fix-carrierwave-globalize
and it works fine for me now.


Support "function-like" import()

The proposal for function-like import() is currently in stage-3, and I think it'd be nice if Typescript will support it in future versions.
For example, one such project that already supports it is webpack which checks for import() as a code-splitting construct (deprecating System.import in v2 and replacing it altogether in v3).
TypeScript Version:  2.0.10
Code
// A *self-contained* demonstration of the problem follows...

import('page/profile').then(() => /* load route */ );

Expected behavior:
The compiler allows the dynamic import syntax as above.
Actual behavior:
-> error TS1128: Declaration or statement expected.

https://github.com/Microsoft/TypeScript/issues/12364

Thanks @wallverb


Option to avoid queueing same task twice

It would be nice if there was a way to prevent a task from being queued multiple times, especially if given the exact same context/configuration. This would throw an alreadyQueued error when queued.
This would be optional, and off by default to prevent backwards compatibility issues.

I'm currently achieving this with my own check against Mongo:
let document = try await coll.findOne("category" == Self.category && "metadata.type" == type.rawValue && "status" != "dequeued")
return document != nil

If this returns true, then I don't queue another job since one already exists.

@Sherlouk would you mind hopping on our discord so that I can discuss the request a bit better? I don't see how you'd be looking to use this feature.

@Sherlouk would you mind hopping on our discord so that I can discuss the request a bit better? I don't see how you'd be looking to use this feature.


Feat: report proper error when a user enabled edge handlers, but accessing the directory fails

The current code for deploying edge handlers ignores errors when accessing the edge handlers directory:
https://github.com/netlify/cli/pull/1244/commits/f057acf91904a5edf3ab84e884b89c35721a430c#diff-895a6262de9045deceba0f1565662065R20
Currently enabling edge handlers is done by creating a directory, so it not being existent is not an error.
When we have a better way to detect when a user uses edge handlers we should properly report such errors.

We can follow up on this now and use build.edge_handlers !== undefined


Delayed, or no execution of queries

Issue Type: Bug
I frequently press (say) Ctrl-Enter to begin execution of a query and the status bar changes to "Executing query..." but the execution time doesn't move - often for many seconds. The "Cancel" button (oddly named) doesn't do anything! Sometimes the query runs after a random amout of time, and then the real execution time is correctly reported. However, frequently the query doesn't run at all, Cancel doesn't work, and the only way to proceed is to close Azure Data Studio altogether and start all over again. Making the tool pathetically useless.
Azure Data Studio version: azuredatastudio 1.28.0 (13e362762762e5cb76a5c8afd2bc650f48c2d3d8, 2021-04-15T00:24:15.710Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
AMD Ryzen 9 3900X 12-Core Processor             (24 x 3800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.95GB (8.17GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

agent
Mic
0.48.0

dacpac
Mic
1.8.0

import
Mic
1.3.0

profiler
Mic
0.11.0

schema-compare
Mic
1.10.0

sql-prompt
Red
0.2.6

sql-search
Red
0.3.1

this has been fixed in the latest insider build, please give it a try. similar to https://github.com/microsoft/azuredatastudio/issues/23145


Beckn-One: Trust Layer(Persistence Layer) API specification v0.1

The Associated Tickets: https://github.com/beckn/missions/issues/195

Restructured the above 3 documents based on additional context I got along the way.
Added more details in the example corresponding to the APIs defined.
separated the /order API definition sections from the original document and created a new doc for them.
Parent document: https://docs.google.com/document/d/1V5zCBbuljTYETW5VZ-ZzEY6CjKAkaEzDx3j-j1ZBLSQ/edit?tab=t.0
Links to the docs for API definitions:
APIs for tracking order data: https://docs.google.com/document/d/1_VwmFNdzFbnjzo6XFNX7Kxp4anf6aayxLUg4MwSfTbk/edit?tab=t.0#heading=h.pfn1wivtdtv3
APIs for tracking meter data: There are 2 versions, we will finalise one after the review.
Approach 1: https://docs.google.com/document/d/1ZBjFZKHnIWZu4yLOX3vFTBJDFY3LnLUBOz-1zresSek/edit?tab=t.0#heading=h.w6d10cx362je
Approach 2: https://docs.google.com/document/d/1kcpI6Ib4vbA5jZ2P7JJ6uK9rGElynxp2RpThd38n8j8/


fix: enable in and outs in the minimap. (fix issue #301)

Hi @ylmrx, your code looks fine. But sadly that would basically mean, that the graph is rendered twice. I believe the performance impact would be too large. I personally think that it's perfectly valid that the minimap is an abstraction of the actual graph.


Events method not usable with UNIX domain sockets

While testing my patch in #400, I noticed that the events() method does not work when connecting to the daemon through an UNIX domain socket:
>>> client = pylxd.Client()
>>> events = client.events()
>>> events.run()
Failed to receive data
Traceback (most recent call last):
  File "/nix/store/80xg5vr1095wihvvgppw9gjplhk1l0xr-python3-3.7.6-env/lib/python3.7/site-packages/ws4py/websocket.py", line 394, in once
    b = self.sock.recv(self.reading_buffer_size)
OSError: [Errno 22] Invalid argument

Hi @pstch,
Thanks for taking the time to play with pylxd and file this issue.
The problem you're having is that you forgot to events.connect() before trying to .run() - this is covered in the Events docs.

Hi @pstch,
Thanks for taking the time to play with pylxd and file this issue.
The problem you're having is that you forgot to events.connect() before trying to .run() - this is covered in the Events docs.


feat(lifecycle): hibernate implementation using cgroup freezer subsystem

Issue #, if available:
Description of changes:
Component lifecycle implementation of component hibernation using linux cgroup freezer subsystem. IPC interface changes in separate earlier PR.
Adding a change to run the linux maven github workflow as root since some of the tests for resource management require that
Why is this change necessary:
New feature for hibernate(pause/resume) components to save consumed resources
How was this change tested:
New unit and integ tests
Any additional information or context required to review the change:
Checklist:

[ ] Updated the README if applicable
[x] Updated or added new unit tests
[x] Updated or added new integration tests
[ ] Updated or added new end-to-end tests
[ ] If your code makes a remote network call, it was tested with a proxy

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Due to the upgrade in snapshot version, the compatibility check step fails since it looks for a snapshot that hasn't yet been created. Rest of the steps including all builds and tests have passed, ok to merge


Deeplab training & inference normalization

Hi, first of all great work!
I would like to try to retrain the deeplab_nchw.onnx model with more images to try to improve its accuracy. I understand that ´train.py´ generates the Model Type - 1 (billinear) but, how to train the one based on deeplab?
Apart, on inference, should the inputs of deeplab_nchw.onnx be normalized? If so, should I use the same parameters imgs -  np.array([0.50693673, 0.47721124, 0.44640532])) /np.array([0.28926975, 0.27801928, 0.28596011]?
Thanks

Hi, please refer the official deeplab repository on github for training. Also, you need to make few changes to train it for two class segmentation with custom dataset.
You can just divide the elements by 255.0 to normalize them to range 0...1 for training and inference.
Once its trained use the given script in ipynb notebook to convert it to onnx channel first format

https://github.com/anilsathyan7/Portrait-Segmentation#deeplab-quantization-aware-training-and-ml-accelerators (second paragraph)
https://github.com/tensorflow/models/tree/master/research/deeplab
https://github.com/anilsathyan7/Portrait-Segmentation/blob/master/deepstream_multiseg/onnx_nchw_conversion.ipynb

Ok I understand, thank you very much @anilsathyan7 !


Feature Request: Config for adding modded tools

I don't know how hard this would be too implement, but it would be nice to have the ability to add modded items to the mod through a config option, maybe somewhat like Project: War Dance (Though I think that may be a forge mod), and have the options for setting their percentage damage types, range, speed, etc.

I didn't want to add too much settings such as changing the values of stacks but as you ask I may add this on a future update.
For the modded weapons, this seem quite impossible but I just thought of something... Also it's a lot of config to setup and I already told about modded weapons that I won't do anything as it's impossible to configure globally BUT I just saif that I thought of something yeah? So I'll try this (even though it will be a struggle lol) but I can't tell you right now if it will possible: this is just an idea. Anyway, this will take some time to try so I'll let you know later. See you!

Okay, it seems it's not impossible. I'll make some tests but there is hope ;)


Upload files to S3

Describe the solution you'd like
To handle even larger file uploads over the HTTP limits, we want to allow users to upload their files to S3 first before they can choose which files to use to send to the backend. The S3 URL will be sent instead of the heavier JSON-converted file to the backend for better performance. As a consequence, the frontend will handle the S3 file upload independently
Setup Instructions (what branch to work off of)
Run the following commands
git checkout dev
git pull origin dev
git checkout -b s3-upload-files

FYI: If you are not able to immediately run git checkout dev, make sure you commit your changes in the current branch or run git stash and then execute the above commands

@farisdurrani who will we assign this task to?

no need to base dev branch off of nextjs @farisdurrani

@dwu359 progress on this?

Done


Add Serialize and Deserialize to Error

Any error that is meant out be propagated out of an enclave needs to
be able to be serialized and deserialized from the trusted side into the
untrusted side.
Clone was also added as the changes needed for Serialize and
Deserialize made it possible for the error to be cloneable.

Current dependencies on/for this PR:

main

PR #169

PR #170   👈

This comment was auto-generated by Graphite.

Merge Activity

Sep 21, 5:43 PM: @nick-mobilecoin started a stack merge that includes this pull request via Graphite.


Sqlite .delete('*') doesn't return deleted rows

Environment
Knex version: 3.0.1
Database + version: NPM package sqlite3 version 5.1.6
OS: MacOS 14.1 (ARM64)
Bug

Explain what kind of behavior you are getting and how you think it should do

The deleted row is returned, same as Postgres driver would do in this case or which Sqlite driver does when doing .insert(...).returning('*').

Error message

No error message, instead returning * is NOT added to the query, resulting in a number being returned instead of the rows.

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to MySQL or PostgreSQL, then single file example which initializes
needed data and demonstrates the problem.

const db = knex({
  client: 'sqlite3',
  connection: ':memory:',
  useNullAsDefault: true,
})

// Omitting creating table and adding a single record

const rows = await db.from('my_table').delete('*')
if (!Array.isArray(rows)) throw new Error('rows should be array of rows!')

Hello @code-ape, I took a look at the code, I think I can start working on this

Thanks so much @mustafabalila 😄


some modifications...

Here's what I did to get this as close to working as possible. FYI you don't need to create your own GSon instance, just declare it as a param or AutoInject it and you're good to go.

Well ok but now it's a spring boot application. The purpose of my "sample" project is to use spring-data-redis + redis-om-spring without spring boot magic.
In my opinion spring boot is good to bootstrap a new project, but sooner or later I'll have to switch to a standard spring application due to the high level of customization asked by my customers.

Another "problem": the Index created with your branch is not updated after inserts (also in my version BTW)
ft.info it.grogdunn.redis.RedisSaveMeIdx
1) "index_name"
2) "it.grogdunn.redis.RedisSaveMeIdx"
3) "index_options"
4) (empty list or set)
5) "index_definition"
6) 1) "key_type"
   2) "HASH"
   3) "prefixes"
   4) 1) "it.grogdunn.redis.RedisSaveMe:"
   5) "default_score"
   6) "1"
7) "attributes"
8) 1) 1) "identifier"
      2) "someText"
      3) "attribute"
      4) "someText"
      5) "type"
      6) "TEXT"
      7) "WEIGHT"
      8) "1"
   2) 1) "identifier"
      2) "indexMePlease"
      3) "attribute"
      4) "indexMePlease"
      5) "type"
      6) "TAG"
      7) "SEPARATOR"
      8) "|"
   3) 1) "identifier"
      2) "vectorSearchOnMe"
      3) "attribute"
      4) "vectorSearchOnMe"
      5) "type"
      6) "VECTOR"
   4) 1) "identifier"
      2) "id"
      3) "attribute"
      4) "id"
      5) "type"
      6) "TAG"
      7) "SEPARATOR"
      8) "|"
9) "num_docs"
10) "0"
11) "max_doc_id"
12) "0"
13) "num_terms"
14) "0"
15) "num_records"
16) "0"
17) "inverted_sz_mb"
18) "0"
19) "vector_index_sz_mb"
20) "0"
21) "total_inverted_index_blocks"
22) "0"
23) "offset_vectors_sz_mb"
24) "0"
25) "doc_table_size_mb"
26) "0"
27) "sortable_values_size_mb"
28) "0"
29) "key_table_size_mb"
30) "0"
31) "records_per_doc_avg"
32) "-nan"
33) "bytes_per_record_avg"
34) "-nan"
35) "offsets_per_term_avg"
36) "-nan"
37) "offset_bits_per_record_avg"
38) "-nan"
39) "hash_indexing_failures"
40) "10"
41) "total_indexing_time"
42) "0.152"
43) "indexing"
44) "0"
45) "percent_indexed"
46) "1"
47) "number_of_uses"
48) "1"
49) "gc_stats"
50) 1) "bytes_collected"
   2) "0"
   3) "total_ms_run"
   4) "0"
   5) "total_cycles"
   6) "0"
   7) "average_cycle_time_ms"
   8) "-nan"
   9) "last_run_time_ms"
   10) "0"
   11) "gc_numeric_trees_missed"
   12) "0"
   13) "gc_blocks_denied"
   14) "0"
51) "cursor_stats"
52) 1) "global_idle"
   2) "0"
   3) "global_total"
   4) "0"
   5) "index_capacity"
   6) "128"
   7) "index_total"
   8) "0"
53) "dialect_stats"
54) 1) "dialect_1"
   2) "0"
   3) "dialect_2"
   4) "0"
   5) "dialect_3"
   6) "0"


Fix loading in engines

When used in an engine, app.import was not available and failed. This PR fixes this and makes it possible to use ember-d3 in engines.

Thanks!


Add configuration options

Currently I identified the following things to configure:

[x] Max amount of cubes generated per tick (default 49*16)
[x] Max time spent generating cubes per tick (default 50 milliseconds)
[x] Vertical cube load distance (options: default=same as horizontal, and values 2-32)

[x] Make this option available in video settings and server.properties

[x] (maybe) Max client skylight update depth (default: 64)
[ ] Enable/disable ChunkGC (when normal chunk unloading is readded) not really possible
[x] Min/Max world height

[x] Dimension height and depth stored in NBT
[ ] Configurable in world type customization options (default value for new dimensions and separate values for dimension IDs)
[x] When cubic chunks toggle switch is added - Allow to set default values for new dimensions and separate values for dimension IDs
(list will be updated as I find more options)

https://github.com/Barteks2x/CubicChunks/issues/50

All the currently planned options have been added long time ago.


remove slugs

I believe this has no effect on the application, and will make new contributions have one less required step. Closes #144

I spot checked this and it all seems pretty much the same.  I assume you used a script to do this.  Good enough to approve?

Yeah a repo-wide regex search for slug:.*\n and replace with the empty string.


chore(factory): use precomputed hash for 'secret'

Use a precomputed hash of the word "secret" instead of random generated password.

I am not in favor of using pre computed hash coz. The idea of building apps is to progressively work on the features.

When you will use the factory for the 1st time, it will yell on you saying Model not found, which will force you to create the model.
After creating the model, you will get a user and realize the password is not encrypted, which will force you to add the model hook.

That's true, this should be done by the developper itself.
I'll probably add a statement into the documentation about that. Recommending to not use Hash.make() into the factory and instead use a precomputed hash.


Bump APIs and APPs

What?

Bump the following components

api-job-store: v1.1 -> v1.4.0
api-permission-manager: v0.2.1 -> v0.2.2
app-data-browser-next: v0.0.11 -> v0.1.0
app-user-manager: v0.1.7 -> v0.1.8

Add a convenient script utils/list_update_candidates.sh

Why?

To support checking permissions in api-job-store
To find which component to update easily

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: d-hayashi
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [d-hayashi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @WatanabeToshimitsu

/uncc @yusukefs

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: d-hayashi, WatanabeToshimitsu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [WatanabeToshimitsu,d-hayashi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Multiple file issue in one gist

Hello guys,
At first, your plugin is awesome. I want to thank you four your efforts.
Could you add search for files that in gists that contains multiple files?
For example, gist which contains multiple files .
When, I search tkm, only tkm-docker-run is shown.

+1 for multiple file gists


add support for showing error/alert dialogs

initially this can just be dialogs that display an error message; at some point we will also want to have dialogs that present multiple options.

I'll work on this. More details on Zulip.


False positive for "shadowed" type

Describe the bug
When a module defines a type alias for a custom type (of the same name, from different module), the constructors of the custom type used in the module aren't taken into consideration in the NoUnused.CustomTypeConstructors rule.
SSCCE (Short Self-Contained Correct Example)
Steps to reproduce the behavior:

Create these two files:

-- A.elm
module A exposing (x)

import B exposing (Foo(..))

type alias Foo = B.Foo -- this "shadowing" is causing the bug, removing the alias removes the false positive

x = Foo 1 -- usage of the custom type constructor!

-- B.elm
module B exposing (Foo(..))

type Foo = Foo Int

Make your elm-review rules contain NoUnused.CustomTypeConstructors.rule []
Run elm-review
See false positive

Expected behavior
I'd expect not to see warning about the B.Foo constructor being unused, because I'm using it in A.x.

This was a confusing one! I tried it out, and you're right that this code works and should not be reported.
I have created failing test cases for this one, should anyone feel like taking this one
https://github.com/jfmengels/elm-review-unused/tree/failing-tests-for-issue-5
I noticed that this false positive becomes a true positive when you change A.Foo to type alias Foo = { name : Int } as that declaration adds a function to the scope, which overrides the import. I have added a test case for that one in the branch so that we don't introduce this problem :slightly_smiling_face:


fix(k8s-system-coredns.json): ability to filter on job

Pull Request
Required Fields
:mag_right: What kind of change is it?

fix

:dart: What has been changed and why do we need it?

Ability to filter k8s-system-coredns.json by job.

Optional Fields
:heavy_check_mark: Which issue(s) this PR fixes?

fixes #70

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


Demonstrate dev workflow

How do we allow typical developers' workflow of deploying to kubernetes?
One of the issues is discoverability of image registry.
Consider https://github.com/alexellis/k8s-tls-registry

refer .NET's https://dapr.io

Merge this with https://github.com/sagittaros/terraform-k3s-private-cloud/issues/7


npm audit on a brand new v17 Angular CLI generated application returns vulnerabilities

Command
new
Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
16.2.0
Description
After installing the latest Angular CLI and creating an application via ng new, running npm audit reports vulnerabilities and the suggestion to fix would require a non v17 version of @angular-devkit/build-angular
Minimal Reproduction

Install latest Angular CLI: npm install -g @angular/cli
Create new application: ng new npm-audit-fails --style scss --ssr false --skip-git
Change directory to new app: cd npm-audit-fails
Run audit: npm audit

Exception or Error
# npm audit report

axios  0.8.1 - 1.5.1
Severity: moderate
Axios Cross-Site Request Forgery Vulnerability - https://github.com/advisories/GHSA-wf5p-g6vw-rhxx
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/axios
  localtunnel  >=1.9.0
  Depends on vulnerable versions of axios
  node_modules/localtunnel
    browser-sync  >=2.24.0-rc1
    Depends on vulnerable versions of localtunnel
    node_modules/browser-sync
      @angular-devkit/build-angular  >=17.0.0-next.0
      Depends on vulnerable versions of browser-sync
      node_modules/@angular-devkit/build-angular

4 moderate severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

### Your Environment

```text
Node.js version v21.1.0 detected.
Odd numbered Node.js versions will not enter LTS status and should not be used for production. For more information, please see https://nodejs.org/en/about/previous-releases/.

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 17.0.0
Node: 21.1.0 (Unsupported)
Package Manager: npm 10.2.3
OS: darwin arm64

Angular: 17.0.2
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1700.0
@angular-devkit/build-angular   17.0.0
@angular-devkit/core            17.0.0
@angular-devkit/schematics      17.0.0
@angular/cli                    17.0.0
@schematics/angular             17.0.0
rxjs                            7.8.1
typescript                      5.2.2
zone.js                         0.14.2

Warning: The current version of Node (21.1.0) is not supported by Angular.

### Anything else relevant?

I noticed that the report somehow references version `>=17.0.0-next.0` of `@angular-devkit/build-angular` instead of simply v17.

Latest localtunnel (2.0.2) is still using a vulnerable axios (0.21.4). There's already an issue filed to localtunnel to update the dependency: https://github.com/localtunnel/localtunnel/issues/632. Until that's fixed, there's not much we can do about this.
browser-sync is only used for live reloading in ng serve and is used in local development only, so the risk here is fairly minimal. Rennovate should automatically pick up a version bump when it is available.

{
   ...
  "overrides": {
    "localtunnel": {
      "axios": "1.6.2"
    }
  }

Waiting on https://github.com/localtunnel/localtunnel/pull/633


Deploy versioning fix

Coverage remained the same at 100.0% when pulling 59112f7d1c0f4cea1d5c804cf2b333a85809bb3f on deploy-versioning-fix into 192e6cce036d612bf1746dd256f490fe964d0ac0 on master.


Upgrade/Reinstall Flynn?

Hi,
Is there a way to upgrade a Flynn cluster to a newer version? If not, is there an easy way to completely uninstall Flynn to after reinstall it?
And keep up the amazing work, I am really looking forward to the day its officially production ready!

@martensson we are currently working on an updater which will update Flynn components in place to a newer version, we are hoping to ship that very soon.
There is currently no easy way to uninstall Flynn (see #1020), but you can get a new version by stopping Flynn (i.e. sudo stop flynn-host) then re-running the installation script.
And thanks for the encouragement!

@lmars great to know you are planning to ship an updated and in the meanwhile I will just re-run the install script. Thanks for the help! :+1:

To others who find this issue, #1020 was closed by #1393 and the installer now includes an uninstaller with --remove. It also allows explicit install from scratch with --clean.

Thanks Guys! :+1:


onLongClick blocking clicked element

Continuing issue #540.
If using onLongClick = {} to popup dropdownMenu it locking clicked item, after dismiss ddMenu you need to click on it to unlock and once again to select.
This issue affecting only on selected with onLongClick element. So programm think what the pressed mouse button wasn't released.
https://user-images.githubusercontent.com/80458677/113281778-b0bd4d80-92ee-11eb-9cad-a2c3b8edac5b.mp4

@Exidel could you please share your code for such behavior?

I was reproduced this issue again.  The same result + ContextMenu lil bug.
https://user-images.githubusercontent.com/80458677/126193716-4440289a-1c9a-4f6b-9d08-52e2ba231f5f.mp4
diffirence with old one:
newer version of compose (0.5.0-build245)
newer version of Kotlin (1.5.10)
used ContextMenu instead of DropdownMenu + pointerInputFilter (for mouse position) in old

TestCode
import androidx.compose.foundation.ExperimentalFoundationApi
import androidx.compose.foundation.background
import androidx.compose.foundation.border
import androidx.compose.foundation.combinedClickable
import androidx.compose.foundation.layout.*
import androidx.compose.material.ContextMenu
import androidx.compose.material.Text
import androidx.compose.runtime.*
import androidx.compose.ui.Alignment
import androidx.compose.ui.ExperimentalComposeUiApi
import androidx.compose.ui.Modifier
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.unit.dp
import androidx.compose.ui.window.Window
import androidx.compose.ui.window.WindowSize
import androidx.compose.ui.window.application
import androidx.compose.ui.window.rememberWindowState

@OptIn(ExperimentalComposeUiApi::class)
fun main() = application {

    var mainWin = rememberWindowState(size = WindowSize(600.dp, 500.dp))

    Window(
        state = mainWin,
        title = "BugTest",
        onCloseRequest = { exitApplication() }
    ) {

        MainView()

    }

}

@OptIn(ExperimentalFoundationApi::class)
@Composable
fun MainView() {

    var index by remember { mutableStateOf(0) }
    var expand by remember { mutableStateOf(false) }

    Box(Modifier.fillMaxSize()) {

        Box(Modifier.padding(10.dp).size(250.dp, 500.dp).border(5.dp, Color.Red).align(Alignment.TopStart)) {

            Column(Modifier.size(200.dp, 400.dp).align(Alignment.Center).background(Color.Yellow)) {
                for (i in 1..20) {
                    Text( text = "item$i", modifier = Modifier.fillMaxWidth().combinedClickable(
                        onClick = { index = i },
                        onLongClick = { index = i; expand = true }
                    ) )
                }
            }

        }

        Box(Modifier.padding(10.dp).size(250.dp, 500.dp).border(5.dp, Color.Green).align(Alignment.TopEnd)) {
            Text(text = "$index", Modifier.align(Alignment.Center))
        }

        if (expand) {
            DropdownToContextMenu(expand, { expand = it })
        }

    }

}

@Composable
fun DropdownToContextMenu(expand: Boolean, expandLamb: (Boolean) -> Unit) {

    ContextMenu(
        expanded = expand,
        onDismissRequest = { expandLamb(false) }
    ) { Text("ForExample") }

}

By the issue you mean, that in case of long click, ordinary onClick is not triggered?

Thank you!
Played with the code a bit. Without ContextMenu clickable works fine. When contextMenu is added, the hover stays over text even when long click is completed. However adding it to ordinary click doesn't lead to such behavior.

I think opened ContextMenu get focus which prevent longClick to be completed. If i remember right ContextMenu and DropdownMenu based on Frame underhood like a Window and Dialog and when it in focus all mouse events got from this Frame.
When I was using pointerMoveFilter to got current cursor position for the DropdownMenu in test it showed diffirent numbers when cursor hover main window and DropdownMenu.

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


crudini: 0.9.3 -> 0.9.4

Description of changes
Tests fail on Python 3.11 due to pipes being deprecated. This was fixed in crudini 0.9.4 by https://github.com/pixelb/crudini/commit/e1650941054822faad4cda788bff6fe364eb4ca3.
https://github.com/pixelb/crudini/releases/tag/0.9.4
Discovered when testing https://github.com/NixOS/nixpkgs/pull/251878.
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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.

can I trouble you to add meta.mainProgram = "crudini"; as well please.

can I trouble you to add meta.mainProgram = "crudini"; as well please.

Done, thank you for the quick review!


First Study Group Meeting 5pm, July 30

Meeting will focus on future meetings, a meta meeting of sorts. Tentative topics:

Goals of future Hackey Hours
How we can help each other out on side projects and the like
Making this more accessible to others on the Minnesota Campus

Programming in academia

How to have a presence in a University setting?
Remove the focus on science and put it on technology and methods (programming)
How to not be another XXX_UsersGroup?
"You are alone together" in terms of programming.

How we can benefit?

Zero preparation presentations
"show your work" coding, and then looking at other peoples code
Rarely do you get feedback on your code/work
Learn from looking at other peoples stuff.

How to "Evangalize"?

To work well, we need more people
People need to benefit somehow
Reach out to other "Hecky Hour" groups?
Make sharing code less scary

Next Time

Overview of another peoples projects/Feedback mechanism
General technology discussion. socially focused.
Bigger projects?

Rob,
Sorry to miss the first meeting but I'm out of town.
Glad to be in the loop.
Thank you!,
Tim
On Thursday, July 30, 2015, Rob Schaefer<EMAIL_ADDRESS>wrote:

Programming in academia

How to have a presence in a University setting?
Remove the focus on science and put it on technology and methods
(programming)
How to not be another XXX_UsersGroup?
"You are alone together" in terms of programming.

How we can benefit?

Zero preparation presentations
"show your work" coding, and then looking at other peoples code
Rarely do you get feedback on your code/work
Learn from looking at other peoples stuff.

How to "Evangalize"?

To work well, we need more people
People need to benefit somehow
Reach out to other "Hecky Hour" groups?
Make sharing code less scary

Next Time

Overview of another peoples projects/Feedback mechanism
General technology discussion. socially focused.
Bigger projects?

—
Reply to this email directly or view it on GitHub
https://github.com/schae234/studyGroup/issues/2#issuecomment-126515384.

--
Thanks,
Timothy
<EMAIL_ADDRESS>

No problem Tim,
There will definitely be more!
Looking forward to hearing what you are up to lately.


author order scrambled

there seems to be some randomness to author order returning from the DOI call:
DOI("10.1080/00018730802564254")
returns
Bradley, A S; Davis, M J; Blakie, P. Blair; Ballagh, R.J.; Gardiner, C.W. Dynamics And Statistical Mechanics Of Ultra-Cold Bose Gases Using C-Field Techniques, Advances In Physics (2008) 10/btkg97, cited by 387
But the article is
P. B. Blakie, A. S. Bradley, M. J. Davis, R. J. Ballagh, and C. W. Gardiner, Dynamics and Statistical Mechanics of Ultra-Cold Bose Gases Using c-Field Techniques, Advances in Physics 57, 363 (2008).
Is this ShortCodes, or opencitations?

ShortCodes doesn't change the order, this looks like a data issue with opencitations:
https://opencitations.net/index/search?text=10.1080%2F00018730802564254

Thank you for reporting this problem. I confirm that the problem was with OpenCitations; there was a bug in the API. I fixed it

Thanks @arcangelo7


Backport "Refactor hasOnlyLocalInstantiation to use isLocalToCompilationUnit" to LTS

Backports #19886 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-19986.
Reference


:x::one::zero: delete booking modules

They are broken and not supported

I approve to merge it now


explicit versioned controller and DefaultApiVersion

consider this configuration
builder.Services.AddApiVersioning(options =>
{
    options.DefaultApiVersion = new ApiVersion(2023.5);
    options.AssumeDefaultVersionWhenUnspecified = true;
    options.ApiVersionReader = new HeaderApiVersionReader( "x-ms-version" );
    options.ReportApiVersions = true;
}).AddMvc();

and this controller
[Route("[controller]")]
[ApiVersion(2023.2)]
[ApiController]
public class WeatherForecastController : ControllerBase
{
    private static readonly string[] Summaries = new[]
    {
        "Freezing", "Bracing", "Chilly", "Cool", "Mild", "Warm", "Balmy", "Hot", "Sweltering", "Scorching"
    };

    private readonly ILogger<WebApplication4Versioning.WeatherForecastController> _logger;

    public WeatherForecastController(ILogger<WebApplication4Versioning.WeatherForecastController> logger)
    {
        _logger = logger;
    }

    public IEnumerable<WeatherForecast> Get()
    {
        return Enumerable.Range(1, 5)
            .Select(index => new WeatherForecast
            {
                Date = DateTime.Now.AddDays(index),
                TemperatureC = Random.Shared.Next(-20, 55),
                Summary = Summaries[Random.Shared.Next(Summaries.Length)]
            })
            .ToArray();
    }
}

now if i understood correctly WeatherForecast.Get should only reply to requests that are explicity from version 2023.2, but it is not the case. i found that requests that come without version as well as with version 2023.5 are accepted
what i am missing here?

Hmm... I don't think it matters here, but I presume the package versions are up-to-date. I incorporated your changes into the Basic Example and this is what I get:
Request
GET /weatherforecast HTTP/1.1
Host: localhost:5001
x-ms-version: 2023.2

Response
HTTP/1.1 200 OK
Content-Type: application/json; charset=utf-8
api-supported-versions: 2023.2

"json"

If I subsequent change the API version to 2023.5 or no API version at all, it behaves like this:
Request
GET /weatherforecast HTTP/1.1
Host: localhost:5001
x-ms-version: 2023.5

Response
HTTP/1.1 400 Bad Request
Content-Type: application/problem+json

{
    "type": "https://docs.api-versioning.org/problems#unsupported",
    "title": "Unsupported API version",
    "status": 400,
    "detail": "The HTTP resource that matches the request URI 'https://localhost:5001/weatherforecast' does not support the API version '2023.5'."
}

This is the response with or without x-ms-version because the assumed default version when unspecified is 2023.5.
This is exactly how I would expect things to behave. Can you elaborate on what you are seeing in responses? Is it a successful response?


fix(compiler-core): when whitespace is "preserve" compile the v-if/v-else will be error

close #6063
fix(compiler-core): when whitespace is "preserve" compile the v-if/v-else will be error

@liulinboyi do you have intentions in getting this PR ready to review or should it be closed?

@liulinboyi do you have intentions in getting this PR ready to review or should it be closed?

ok, i have no time to deal with it, i will close the PR.


Alt+Shift+Down conflicts with chrome shortcut

As per above, it needs a different shortcut, up, left and right works fine.

Conflict with extension instead of chrome.


Make ChooseRightsPage get urn from ServiceRegistry

Description
Currently there's only a hard coded value for setting the To-field in DelegationRequestDto. This task is about fixing that
Tasks:

[ ] Update the field 'urn:altinn:resource' in DelegationRequestDto to be set from the resourceReference field in the data from resource registry.

Additional Information
No response

Fixed together with issue #434


How to avoid updating uuid?

Hi @oliuz could you please assist me how I can avoid issue which I described above. In the latest version I can see that you moved this logic to boot method but it's not resolve my problem.
PS: I don't need uuid column in my teams table
I see 2 possible solutions:

don't extend TeamworkTeam model and just use TeamworkTeamTrait directly in my Team model.
try to suppress your attached listeners in the boot method

Hi @siarheipashkevich ,
What is the version of the package you use? Did you export the configuration file as I mentioned before?

8.* version because from this version you added this logic.
Yes, I did. but the main issue is forced added slug and uuid columns to table and the logic for these fields in the boot method.

Hi,
I understand that this is resolved with the PR you submitted.

@oliuz it's not resolved in my previous PR.


resourcePath no longer passed into loader from webpack correctly

partial fix for https://github.com/TheBrainFamily/cypress-cucumber-preprocessor/issues/192
Accidentally changed the loader function definition with the cucumber.json PR - revert to previous way it was called to sort out the 'this' resolution when called from webpack.
Note: there are still issues with the typescript webpack example due to the use of the jsonFormatter module browser side
To fix this and get the typescript webpack example running again I had to identify some modules in the webpack config in that example as empty
module.exports = {
  resolve: {
    extensions: [".ts", ".js"]
  },
  node: { fs: "empty", child_process: "empty", readline: "empty" },  <--- fixes the webpack example
  module: {
    rules: [
      {
        test: /\.ts$/,
        exclude: [/node_modules/],
        use: [
          {
            loader: "ts-loader"
          }
        ]
      },
      {
        test: /\.feature$/,
        use: [
          {
            loader: "cypress-cucumber-preprocessor/loader"
          }
        ]
      }
    ]
  }
};

see https://github.com/Asana/node-asana/issues/114#issuecomment-289156939 for discussion of this.
There may be a neater way to get the webpack loader example working again without this addition

Ok, I will look closer tomorrow, thanks for all the info!

Thanks @jcundill! Was walking into this exact same issue. I'll downgrade again until the release of this fix :)

:tada: This PR is included in version 1.13.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dataset.where performances regression.

What happened?
Hello,
I'm using the Dataset.where function to select data based on some fields values and it takes way to much time!
The dask dashboard seems to show some tasks repeating themselves many times.
The provided example uses a 1D array for which the selection could be done with Dataset.sel but with our real usecase we make selections on 2D variables.
This problem seems to have appeared with the 2022.6.0 xarray release, the 2022.3.0 is working as expected.
What did you expect to happen?
Using the 2022.3 release, this selection takes 1.37 seconds.
Using the 2022.6.0 up to the 2023.2.0 (the one from yesterday), this selection takes 8.47 seconds.
This example is a very simple and small one, with real data and use case we simply cannot use this function anymore.
Minimal Complete Verifiable Example
import dask.array as da
import distributed as dist
import xarray as xr

client = dist.Client()

# Using small chunks emphasis the problem
ds = xr.Dataset(
    {"field": xr.DataArray(data=da.empty(shape=10000, chunks=10), dims=("x"))}
)
sel = ds["field"] > 0

ds.where(sel, drop=True)

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
No response
Anything else we need to know?

I will look into it when I find time :)

I am a bit puzzled here...
The dask graph looks identical, so it must be the way the indexers are constructed.
The major difference I can find is:
The old version used np.unique while the new version uses xarrays cond.any(..)
Maybe someone with more experience in dask can help out?

The old code had:
nonzeros = zip(clipcond.dims, np.nonzero(clipcond.values))

This loaded the array once and then passed numpy values to the indexing code.
Now, the dask array is passed to the indexing code and is computed many times . #5873 raises an error saying boolean indexing with dask arrays is not allowed.
For here just do ds.where(sel.compute(), drop=True).  It's identical to what was happening earlier.
I think we should close this.

Just tried it and it does not seem identical at all to what was happening earlier.
This is the kind of dataset I'm working

With this selection:
sel = (dsx["longitude"] > 0) & (dsx["longitude"] < 100)
Old xarray take a little less that 1 minute and less than 6GB of memory.
News xarray with compute did not finish and had to be stopped before consuming my 16GB of memory.

Does sel.compute() not finish?

sel = (dsx["longitude"] > 0) & (dsx["longitude"] < 100)
sel.compute()

This "compute" finishes and takes more than 80sec on both versions with a huge memory consumption (it loads the 4 coordinates and the result itself).
I know xarray has to keep more information regarding coordinates and dimensions but doing this (just dask arrays) :
sel2 = (dsx["longitude"].data > 0) & (dsx["longitude"].data < 100)
sel2.compute()

Takes less than 6 seconds.

Yeah that was another change I guess. We could extract out the variable using .variable.
.where(sel2.variable.compute(), drop=True)

do your "_nadir" variables have smaller chunk sizes or are slower to read for some reason?

The .variable computation is fast but it cannot be directly used like you suggest:
dsx.where(sel.variable, drop=True)

TypeError: cond argument is <xarray.Variable (num_lines: 5761870, num_pixels: 71)>
... but must be a <class 'xarray.core.dataset.Dataset'>
or <class 'xarray.core.dataarray.DataArray'>

Doing it like this seems to be working correctly (and is fast enough):
dsx["x"]= sel.variable.compute()
dsx.where(dsx["x"], drop=True)

_nadir variables have the same chunks and are way faster to read than the other ones (lot smaller).

Hello,
I'm not sure performances problematics were fully addressed (we're now forced to fully compute/load the selection expression) but changes made in the last versions makes this issue irrelevant and I think we can close it.
Thank you!


Ensure that 'cfg' commands work when using debug script.

Tested:

imported the modified template in GTM
verified that ldfdr.registerTracker('lAxoEaKdNOA8OYGd') includes
enableAutoTracking: true set via cfg when using either _debug
suffix or when ticking debug mode

Related to https://github.com/Leadfeeder/issue-tracker/issues/18301

Review @Bajena


Fix error handling for getcwd

Quoting from the Linux man page for errno,
"The value in errno is significant only when the return value of the call
indicated an error (i.e., -1 from most system calls; -1 or NULL from most
library functions); a function that succeeds is allowed to change errno. The
value of errno is never set to zero by any system call or library function."
Successful calls to getcwd are allowed to set errno causing the compilation
database not to be written. Spurious failures of this nature were observed on
AIX.
Adjust the error handling for getcwd so that errno is only checked if the call
returned NULL.

Thank you very much!


File descriptor (socket) leak

Sockets opened in ICEGatherer seem to be leaked. They will never be closed even when the PeerConnections are closed. Finally, my server becomes unavailable with too many open files error.
I've investigated into webrtc's source code, and found that CandidateBase.close() is an no-op. Of course, it is because tokio's UdpSocket does not provide close(). Despite all, the sockets should be closed when it is dropped. Therefore, I guess that the socket is not dropped.
RTCPeerConnection holds Arc<PeerConnectionInternal> and there are other holders too. In v0.2.1, the peerconnection has no problem, dropped well. However, other holders seems not to drop it. I have no idea where the others are, but internal_rtcp_writer (peer_connection.rs, line 170) or pci (peer_connection.rs, line 1163, 1345, 1393) may have created reference cycle.
If possible, those references should be replaced by std::sync::Weak to break the cycle. Pion or other webrtc libraries written in garbage-collected languages may not suffer from this issue because the GC can detect and free those circular references well. Because Rust does not have one, we should use weak references to avoid this problem. It will also fix other memory leaks too.

I think I have made some progress on reference cycle caused memory leak.
reflect and data-channels-flow-control in examples v0.3.0 branch should be memory leak free now.
I have checked by using nightly rust version with following build
"RUSTFLAGS="-Z sanitizer=leak" cargo build --example reflect
It reports no memory leak.
I will keep checking and fixing other examples to make sure there is no  reference cycle caused memory leak

Unfortunately, I don't think the file descriptor leak is fixed yet (on master branch). I am going to look further into it on Monday

Unfortunately, I don't think the file descriptor leak is fixed yet (on master branch). I am going to look further into it on Monday

All examples in v0.3.0 branch of examples repo passed rc-cycle check. Please note that it requires some changes on callback function:
for example, here we have to use Weak pointer of peer_connection in its on_track callback, otherwise, it will cause reference cycle leak.
let pc = Arc::downgrade(&peer_connection);
peer_connection
        .on_track(Box::new(
            move |track: Option<Arc<TrackRemote>>, _receiver: Option<Arc<RTCRtpReceiver>>| {

Could you check your code whether have such callback?

No, RTCPeerConnection is dropped very well, and the file descriptor closes well on normal cases. It seems to be leaked when it fails to connect.
Steps to reproduce:

I added the following code to CandidateBase.close() (line 255)

        if let Some(conn) = &self.conn {
            let _ = conn.close().await;

            use std::sync::Weak;
            use std::time::Duration;
            let conn_weak = Arc::downgrade(conn);
            tokio::spawn(async move {
                let mut int = tokio::time::interval(Duration::from_secs(1));
                loop {
                    int.tick().await;
                    dbg!(Weak::weak_count(&conn_weak));
                    if dbg!(Weak::strong_count(&conn_weak)) == 0 {
                        break;
                    }
                }
            });
        }

Signal(offer) from other peer and forcefully kill the offer-side process
Server spams this log for a while
[2021-11-01T04:17:16Z TRACE webrtc_ice::agent::agent_internal] [controlled]: pinging all candidates
ICE connection state changes to failed
conn_weak still has strong_count of 1

[/Users/qbx2/ice/src/candidate/candidate_base.rs:263] Weak::weak_count(&conn_weak) = 1
[/Users/qbx2/ice/src/candidate/candidate_base.rs:264] Weak::strong_count(&conn_weak) = 1

I reproduced using ~50 clients.

RTCPeerConnection is dropped very well, and the file descriptor closes well on normal cases. It seems to be leaked when it fails to connect.
Steps to reproduce:

I added the following code to CandidateBase.close() (line 255)

        if let Some(conn) = &self.conn {
            let _ = conn.close().await;

            use std::sync::Weak;
            use std::time::Duration;
            let conn_weak = Arc::downgrade(conn);
            tokio::spawn(async move {
                let mut int = tokio::time::interval(Duration::from_secs(1));
                loop {
                    int.tick().await;
                    dbg!(Weak::weak_count(&conn_weak));
                    if dbg!(Weak::strong_count(&conn_weak)) == 0 {
                        break;
                    }
                }
            });
        }

Signal(offer) from other peer and forcefully kill the offer-side process
My program spams this log for a while
[2021-11-01T04:17:16Z TRACE webrtc_ice::agent::agent_internal] [controlled]: pinging all candidates
ICE connection state changes to failed
conn_weak still has strong_count of 1

[/Users/qbx2/ice/src/candidate/candidate_base.rs:263] Weak::weak_count(&conn_weak) = 1
[/Users/qbx2/ice/src/candidate/candidate_base.rs:264] Weak::strong_count(&conn_weak) = 1

I reproduced using ~50 clients.

@qbx2, I tried to reproduce this issue in ice crate with ping_pong example. From new ping_pong example, ice crate looks like no issue for such leak.
Therefore, I think the leak may come from webrtc crate itself, so I fixed one possible issue in https://github.com/webrtc-rs/webrtc/commit/0944adf54699752897f0bbac2251264da22036c2, where endpoint forgets to call underlying conn.close().
Could you try again?
In addition, in order to reproduce this issue easy, do you think any example in https://github.com/webrtc-rs/examples that can be hacked to reproduce this issue? With the same codebase, it is more easy to debug.
Thank you.

I tried webrtc on master branch, but had no luck. :(
Also, I am sorry about the codebase. I have a problem that my code is related to company stuff and that's why I could not share my code details. Therefore, I am going to make an example for reproduction to share with you. (I guess it is difficult to reproduce with the examples because it has different signaling protocol?)

@rainliu Please, check this out: https://github.com/qbx2/examples/commit/5f5c3a5001699059b82ce49431fe9c4ee7b11843

@rainliu Please, check this out: qbx2/examples@5f5c3a5

@qbx2, thanks for this example, I can reproduce this issue.

@qbx2, I think I found the root cause of the bug. https://github.com/webrtc-rs/webrtc/commit/e7d72d22875d05fe447c9336b6ada367400c4b91 should fix this issue.
The root cause is that cancel_tx signal should be set to internal.cancel_tx before call agent.dial or agent.accept, otherwise, calling agent.close() will have no-op on cancellation of dial/accept.
            let (cancel_tx, cancel_rx) = mpsc::channel(1);
            {
                let mut internal = self.internal.lock().await;
                internal.role = role;
                internal.cancel_tx = Some(cancel_tx);
            }

            let conn: Arc<dyn Conn + Send + Sync> = match role {
                RTCIceRole::Controlling => {
                    agent
                        .dial(
                            cancel_rx,
                            params.username_fragment.clone(),
                            params.password.clone(),
                        )
                        .await?
                }
                RTCIceRole::Controlled => {
                    agent
                        .accept(
                            cancel_rx,
                            params.username_fragment.clone(),
                            params.password.clone(),
                        )
                        .await?
                }
                _ => return Err(Error::ErrICERoleUnknown),

in your example, you still need to send done signal to call peer_connection.close().await, otherwise, it won't call self.internal.ice_transport.stop().await, which is used to cancel agent.dial/accept.
peer_connection
        .on_peer_connection_state_change(Box::new(move |s: RTCPeerConnectionState| {
            println!("Peer Connection State has changed: {}", s);
            if s == RTCPeerConnectionState::Failed {
                ...
                println!("Peer Connection has gone to failed exiting");
                // let _ = done_tx.try_send(());
            }
...

By the way, in your 50 clients call case, does each client have one peer_connection? If yes, once the following callback has indicated Failed, you may need to close the peer_connection for such client.
peer_connection
.on_ice_connection_state_change
or
peer_connection
.on_peer_connection_state_change

in your example, you still need to send done signal to call peer_connection.close().await, otherwise, it won't call self.internal.ice_transport.stop().await, which is used to cancel agent.dial/accept.

You're right. The reason that I commented that line out was to keep the process alive so that the leak could be confirmed. Thanks a lot for the advice, I am caring of closing peer_connection.

By the way, in your 50 clients call case, does each client have one peer_connection? If yes, once the following callback has indicated Failed, you may need to close the peer_connection for such client.

Yes, they have. However, peer_connections are designed to be dropped when it failed to connect (I use Weak). It has no problem, right?
Also, I've confirmed that https://github.com/webrtc-rs/webrtc/commit/e7d72d22875d05fe447c9336b6ada367400c4b91 fixes this issue. Thank you so much.

thanks for confirm. I just released v0.3.1 to include this fix.


Multilaser M11HC PC915 FIRMWARE recovery

I have a Chromebook that was company locked. I installed the mrchromebox firmware via the eprom burner, but I'm having problems with the operating systems. and I wanted an original firmware without the lock. I tried with the script utility but my fw is blocked and I have already done all the possible tutorials to unlock it and I was unable to do so. Is there any way to download and record via eprom?

GARG360 BOARD IS DALI9BMB6D0 REV:D

see: https://github.com/coreboot/coreboot/tree/main/util/chromeos
download crosfirmware.sh and run it: bash crosfirmware.sh garg360

I have a Chromebook that was company locked. I installed the mrchromebox firmware via the eprom burner, but I'm having problems with the operating systems. and I wanted an original firmware without the lock. I tried with the script utility but my fw is blocked and I have already done all the possible tutorials to unlock it and I was unable to do so. Is there any way to download and record via eprom?

You cabd sen-me this EPROM please, If some one have please??


⚠ Bofrost.at has degraded performance

In c65e1e2, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1257 ms

Resolved: Bofrost.at performance has improved in 725a0e0.


fix(semantic): incorrect symbol’s scope_id after var hoisting

Bug found in https://github.com/rolldown/rolldown/pull/1726

#4458  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite

cc @hyf0


Portable version of mitmproxy - how to build a standalone package?

Is it possible and how to approach creating a portable version of mitmproxy? I'd simply like to deploy it like e..g go binaries, without any additional dependencies and so on. Everything in a single package, dumped on a server and ready to go.

docker is good, but it requires installing docker itself
I once tried pex to bundle everything, but it did not actually work - maybe I need to check it again if there is no other better alternative

any other way to pack everything or build everything to a standalone package? How in general python packages should be deployed this way? without any dependencies and so on.

Hey,
We provide standalone binaries at https://github.com/mitmproxy/mitmproxy/releases/ - that should accomodate your needs. :)

Awesome thx! I was not aware of that. I always install it from pip/or dev mode. Additionally I can finally learn how to properly package python apps for Linux/Windows. thx

Where save the settings the provide standalone binaries?
If it save also in user .mitmproxy it is not portable.


feat(proto): add @opentelemetry/proto package

This is a WIP new package which may eventually be shared among the otlp protobuf/grpc exporters

Uses protobufjs
Uses long to support nanosecond timestamps with int precision
Works in browser and node
Implements hrTime and hexToBuf in order to avoid dependency on core

TODO:

[x] see if it is possible to remove grpc-js dependency for non-grpc exporters (optional peer dependency?)
[ ] create transformation functions for logs
[x] test traces
[x] test in browser
[ ] check bundle size vs current solution
[x] add README.md
[x] add ESM builds
[x] update package.json fields
[x] ensure windows compatibility

@vmarchaud @legendecas the protoc generator doesn't work with node 8 so i think we have a couple possible options:

create a small docker file/image with protoc and node 16 to generate the files - means taking a hard dependency on docker for the build process
drop support for node 8 - could be for just this package? not sure if the generated files cause problems with node 8 yet but if they don't at least the otlp exporters could still test for 8
don't use this module to generate proto files and hand-roll or similar
switch node versions during the CI run to build the generated ts with node 16 and test with node 8
maybe something else?

Does protobufjs work with nodejs 8? could be an alternative.

so i think we have a couple possible options:

I think we could use a docker image, its already used for the semantics convention codegen: https://github.com/open-telemetry/opentelemetry-js/blob/main/scripts/semconv/generate.sh

Does protobufjs work with nodejs 8? could be an alternative.

not sure, i'll give it a try

so i think we have a couple possible options:

I think we could use a docker image, its already used for the semantics convention codegen: main/scripts/semconv/generate.sh

I forgot we already had that dependency. I'm going to try protobufjs first, but if it doesn't work i'll do this.

+1 to protobufjs. It is web-compatible and it can pre-compile proto files to JSON (or even JS) to boost startup times. If I recall correctly @grpc/proto-loader is already using protobufjs (https://www.npmjs.com/package/@grpc/proto-loader).

Updated to use protobufjs which removes the dependency on google-protobuf and on @grpc/grpc-js and adds a dependency on protobufjs
Added tests

Looking forward to this! @dyladan pls give us know if we can help with anything

@morigs you already did by reminding me to get off my lazy butt and work on it 😆
Seriously though, reviews would be helpful. I've already done a quick PoC to prove out the concept of using this package in the otlp exporters with good results.
Open questions:

Is the use of long acceptable from a bundle size perspective (/cc @MSNev)? Without long, the timestamps lose precision (worth noting that this is already happening in other exporters because JS numbers can't hold timestamps in the nanosecond precision we collect them in).
Would appreciate a browser/esm expert to verify my esm packaging. I'm not really an expert here and just stole the implementation from another package.

Is the use of long acceptable from a bundle size perspective (/cc @MSNev Nev Wylie FTE)?

Based on the size of the Long package and how it's used here, I'd recommend not using it and instead extract the couple of functions that are required. This is primarily because everything is namespaced from the "Long" class which means any of the referenced function (depending on your packager) won't be able to tree-shake the unused code.
It's also not clear how the "Long" value would get serialized in the protobuf references.
And isn't a JavaScript number already a 64-bit value, what am I missing for the need to use this wrapper class?

Is the use of long acceptable from a bundle size perspective (/cc @MSNev Nev Wylie FTE)?

Based on the size of the Long package and how it's used here, I'd recommend not using it and instead extract the couple of functions that are required. This is primarily because everything is namespaced from the "Long" class which means any of the referenced function (depending on your packager) won't be able to tree-shake the unused code.
Would still prefer not to include all of the Long class.

Unfortunately it isn't that simple. If you don't include the long package, protobufjs will automatically pass all input to the Number function, losing precision.
We could potentially try to look at how protobufjs uses the given Long class (maybe it only uses a couple methods) and come up with our own class that implements them, but if protobufjs ever updates we run the risk of them changing their use of long.js, not to mention the risk that we might introduce bugs.
We could potentially export a function which takes the Long constructor as a parameter and registers it for use by protobufjs. This would leave it to the exporter to enable long precision mode manually (presumably by exposing a configuration option to the end-user).

It's also not clear how the "Long" value would get serialized in the protobuf references.

If long is included, protobufjs takes any input to a field with the s-/u-/int64 or s-/fixed64 types (proto types not js types) and passes it to Long.fromValue(). If long.js is included, it is passed to parseInt(input, 10)

And isn't a JavaScript Number already a 64-bit value, what am I missing for the need to use this wrapper class?
Answered my own question: Number is an ES6 object and therefore, it's not supported on all browsers -- comes back to the missing browser support matrix and I guess versions of Node as well.

Yes, but even Number is not a 64-bit integer and cannot safely store a timestamp of nanosecond precision without losing precision. Only 53 bits can be safely stored as an integer without losing precision.

Ok, re-looking at this an "assuming" that the web packages are "unlikely" to include the protobuf implementations (in preference for http/json), then as long as this is limited to the proto implementations (as it appears to be here) then it should be ok...
As I'm less concerned about the bundle size for node apps as these generally don't have PLT concerns, they will still have startup concerns (for things like Azure function apps that start/process/stop on-demand) but the overall package size is not quite as critical.

Ok, re-looking at this an "assuming" that the web packages are "unlikely" to include the protobuf implementations (in preference for http/json), then as long as this is limited to the proto implementations (as it appears to be here) then it should be ok...
As I'm less concerned about the bundle size for node apps as these generally don't have PLT concerns, they will still have startup concerns (for things like Azure function apps that start/process/stop on-demand) but the overall package size is not quite as critical.

I think that isn't necessarily a valid assumption. This package does all transformations required to do the JSON representations also, which wouldn't make sense to duplicate elsewhere.

@dyladan as I understand, Long.js requires Browserify to work on frontend

I'm looking into ways to reduce the payload size for the browser case. right now it uses the generated protobuf code to create an in-memory object, then converts that to JSON. the statically generated code is huge though, and we shouldn't need it for the browser at all. I'm looking into a way to have the conversions be able to live without the protobuf code.


[Frontend with HTML/CSS]: History and Versions of HTML

Description
I would like to provide documentation on history and different versions of HTML and the features they introduced.
Domain
Frontend Dev HTML/CSS/JS
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign this project


i cant import file in R please help me i am new to R language

View(GSE152418_p20047_Study1_RawCounts)
data <- read.delim('~/Desktop/demo/WGCNA/GSE152418_p20047_Study1_RawCounts.txt', header = T)
Error in file(file, "rt") : cannot open the connection
In addition: Warning message:
In file(file, "rt") :
cannot open file '/Users/deneez/Desktop/demo/WGCNA/GSE152418_p20047_Study1_RawCounts.txt': No such file or directory

setwd("~/Desktop/demo/WGCNA")
Error in setwd("~/Desktop/demo/WGCNA") : cannot change working directory


Metrics call to align_two_meters parameter order

Hi,
I've been playing around with your code according to your IPython Notebooks and came across an error when calling the metrics function. Down the line, this function calls nlimtks goodsections.py function _process_chunk (https://github.com/nilmtk/nilmtk/blob/412be54eefd81ae78d0f1ed3053bf8977063e61d/nilmtk/stats/goodsections.py#L36) which expects a DataFrame with parameter timeframe, which did not exist and therefore threw an error.
While trying to fix my error, I came across the documentation of align_two_meters which says it would take the timeframe from meter 'master', which is the first argument of that function. In your call you provide the function with (pred, ground), which seems off to me. After changing the order to (ground, pred) I no longer encountered the problem.
Can you assure me, that the order of your arguments is indeed correct?
This is the section I'm referencing, but the call is them same for all metrics files:
https://github.com/OdysseasKr/neural-disaggregator/blob/dec19c9beeb15e2c23a1dd2b1c52efafc7775810/DAE/metrics.py#L12

Hi @franzmueller
The first argument is pred since we are only care about the timeframe for which we have predicted data. This happens because ground might contain more samples than the pred (due to to dropped samples or other preprocessing).
Which dataset are you using? Are you sure that your dataframes are not empty? What is the exact error message?

Thanks for your quick reply!
I'm using the UK-Dale dataset and set the timeframe to a single day for testing. I am sure the dataframes aren't empty, I have checked using the debugger.
Sorry for not providing the full message in the first place. Here is what I got:

Traceback (most recent call last):
File "E:/github/kafka-operator-disaggregation-python/python/playground/playground.py", line 98, in
start="29-11-2015", switch="30-11-2015 00:00:00", end="12-01-2015 00:00:00")
File "E:/github/kafka-operator-disaggregation-python/python/playground/playground.py", line 86, in nn
rpaf = metrics.recall_precision_accuracy_f1(predicted, ground_truth)
File "E:\github\kafka-operator-disaggregation-python\python\neural_disaggregator\RNN\metrics.py", line 16, in recall_precision_accuracy_f1
for chunk in aligned_meters:
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\electric.py", line 829, in align_two_meters
sections = master.good_sections()
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\elecmeter.py", line 626, in good_sections
nodes, results_obj, loader_kwargs)
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\elecmeter.py", line 707, in _get_stat_from_cache_or_compute
computed_result = self._compute_stat(nodes, loader_kwargs)
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\elecmeter.py", line 759, in _compute_stat
results.run()
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\node.py", line 43, in run
for _ in self.process():
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\stats\goodsections.py", line 33, in process
self._process_chunk(chunk, metadata)
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\nilmtk\stats\goodsections.py", line 63, in _process_chunk
timeframe = df.timeframe
File "D:\Programme\Anaconda3\envs\nilmtk-env\lib\site-packages\pandas\core\generic.py", line 3614, in getattr
return object.getattribute(self, name)
AttributeError: 'DataFrame' object has no attribute 'timeframe'
Closing remaining open files:E:\github\kafka-operator-disaggregation-python\python\ukdale\ukdale.h5...done

This is the dataframe that gets to goodsections.py:


Invalid Password C during sync after Recover

Was a node with the blockchain a bit behind (off-time) and then running a recover - the sync screen asks to unlock the lightning wallet. Even its the correct passwort it comes back as "unvalid" an the WebUI does not offer a retry. When the WebUI gets reloaded thru the browser Lighting says its started ... so unlock may have worked in the background.

@cstenglein when does this "Invalid Password" notice gets created and can iz be remobved when there is no further password input box anymore?

Will show the correct error with https://github.com/cstenglein/raspiblitz-web/issues/387
I guess you entered the password wrong / had connection issues the first time & then it worked?

Should be fixed now in master, please test :)

Think it also has to do with the API or redis, just tested it with 1.8.0:
Unlocking the wallet returned an unknown error.
Trying to unlock it again, it now displays the following:

The main issue that the unlock input is still available is that I get ln_default_locked with 1 even after unlock:

On the Terminal, it still displays the unlock input as well. Trying to unlock it leads to the same error, but aborting the input on the ssh dialog then leads to the WebUI correctly getting ln_default_locked with ´0` and thus the lightning status.


chore(docs): fix links to React/HTML getting started docs

[x] Read CONTRIBUTING.md

The files were moved in #1933 and the links in README.md were not updated.

Hmmm. I don't know. If someone checks it, then we know they haven't read it. 😂

Hmmm. I don't know if we should approve this. If someone checks it, then we know they haven't read it. 😂

Haha! I followed the link in the README rather than in the PR template... but good call, I suppose we should fix the PR template as well :P

@TheCatLady : Thank you for fixing those links! :)

@TheCatLady : Thank you for fixing those links! :)

No problem!  Just pushed a fix for the PR template as well.


Enhancement: Add notifications

Description of enhancement
Add notifications for events such as your review being requested, your PR being reviewed, new PRs being opened. Make these notification types opt in/out on a settings page for the user.
Show the number of notifications in the taskbar on the app's icon (i.e. like Slack does). This may be automatic, not sure.
Rationale
Less need to manually check Pullp to see if there are changes you need to know about.

Completed https://github.com/rkclark/pullp/releases/tag/v3.0.0


realsense-net device in C# wrapper

Camera Model: D435i & D415
Language: C#/unity
In SDK version 2.34 realsense-net and rs-server got introduced. I successfully got the cpp examples working with a D435i connected to a Raspberry Pi 4 running rs-server. Now I wonder if it is possible to add a realsense-net device to a context and initialize the pipeline with the context in C# like in cpp, especially for unity. I couldn't find a C# wrapper for realsense-net. How could I get a C# app running with the realsense over ethernet?

Hi,
Will you be needing further help with this? If we don’t hear from you in 7 days, this issue will be closed.
Thanks

are there also other wrappers for that?
C, python etc? for using realsense-net towards network rs-server device?


[action] get_apple_pay_certificate

fixes: #11809
Generates Apple pay merchant certificates

@joshdholtz, @taquitos This is ready to review.

@RishabhTayal Seriously dude, can you approve the changes, since someone else made some modifications? This has been sitting around for a month, it's a pretty important feature for anyone using apple pay.

@tmaly1980 I am not one who can approve the pull requests.

@tmaly1980 I am not one who can approve the pull requests.

Not the PR. But something mentioned above:
The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

@googlebot Who else is needed to approve the changes?

@janpio thanks for jumping in. I will address the comments later tomorrow.

@tmaly1980 if you want to use action right now you can point to my fork and branch.

There was another comment on this PR here:
https://github.com/fastlane/fastlane/issues/11809#issuecomment-438319150

@janpio I think that something we can add later. That would be an enhancement on this feature.

@RishabhTayal I don't understand the line about creating a CSR. In my case (and I'd imagine many others), I have to provide an EXISTING CSR from my Stripe account in order to make Apple Pay work at all. How exactly does your code work? Wouldn't the Apple Pay calls require that the merchant (ie Stripe) has authority (via the CSR) to make them?

(I don't know anything about Apple Play and those signing certificates - and I suspect other reviewers will have the same problem. One of you might want to summarize the process that the user has to go through and where he gets what CSRs etc.)

@janpio @RishabhTayal I've done this a few times and Stripe generates the CSR for you via their dashboard. But I'd imagine the process varies for every merchant provider.
https://stripe.com/docs/apple-pay/apps

@tmaly1980 I can create a separate pr for that feature. But I don't want to mix that with this pr. This pr works as intended. We can add an option later to pass the csr file.

I think it's an essential feature and not worth waiting for.

You can create a PR based on @RishabhTayal's branch and create a PR that adds the functionality into his PR. Just fork his repository and branch, do the work, then open a PR onto his branch.
If you are done before we get this one here merged, it will just be integrated into this PR and included in the merge when we do merge this one. If not, you can still change the merge base to master of fastlane itself without losing any work.

Resolved the merge conflict.
Could you take another look at the open comments here @RishabhTayal? Thanks.

@janpio All the comments are addressed. This is ready for review again.

@googlebot I consent.

Do we have plans to merge this PR soon?

@gholias thank you for bump this.
While this PR does have some outstanding comments, and could be cleaned up, perhaps "perfect" is the enemy of "done" and we should move ahead.
pinging @mollyIV @joshdholtz for thoughts if this contains any hard blockers.

@gholias @snatchev I will test it today/tomorrow and see what we can do to push it through 😊 Just want to make sure those Apple APIs are still working as we expect 💪

thanks @joshdholtz

Any updates on this PR?

@joshdholtz @RishabhTayal Any updates on this? I can help if needed!

@yhkaplan Hey! A feature like this will probably end up getting slightly rewritten due to the new App Store Connect API endpoints. I'm planning to start the module that includes this sometime next week probably!

@joshdholtz any update? Do you need help?

@ryechus Wooooowwww... things slipped away from me didn’t they 😅 I... actually don’t know if there are new API endpoints for this yet so I just revisit this PR and probably merge it 😊 Have you been able to test this PR to see if it works for you?

Still needed

Please see my PR for a simple addition that makes this all possible


incase of escaped characters in path

if there is a space in the filename ,  will got a 404 error.

Hi! The tests are now failing, probably because this PR makes no sense; decodeURI is already called, just inside the send call, so your call here will cause a double decoding.
Please look into the test failures. In addition, to accept the PR, the PR needs to include a test that failed without your change and passes with your change, to prove that it is fixing an issue.

For now, I'm going to close this PR as invalid for the following reasons:

The change is causing uncaught exception to occur, which introduces a huge security hole allowing one to DoS a server with this change on it.
The change will now cause double-decoding of URLs, i.e. it makes the URL /foo%2525bar.txt load the file foo bar.txt when it should be loading the file foo%25bar.txt.
There are no tests with the change.
There is already a test that should prove loading things with a space works (https://github.com/expressjs/serve-static/blob/v1.10.0/test/test.js#L61-L65).
This now incorrectly calls the send module. You can read the documentation at https://github.com/pillarjs/send#sendreq-path-options , which states "the path is a urlencoded path to send (urlencoded, not the actual file-system path)". You cannot decode the path before handing to send, or it's in violation of the API contact.

Please feel free to either open a new PR with a fix along with a test that fails without the change and passes with the change, or open an issue instead providing all the details for how we can reproduce thew issue.

sorry for my stupid mistake...


Abort on playlist page error but not on playlist entry download error

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

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

Please make sure the question is worded well enough to be understood
Currently using ef8fb7f029b816dfc95600727d84400591a3b5c5
When downloading playlists/channels it sometimes happens that downloading the playlist pages (getting all the playlist metadata/playlist entries) has an error while trying to get one of those pages. I would like for yt-dlp do abort when that happens.
I know that can be achieved using --abort-on-error, however that has the downside that it also aborts when there is an error while trying to download a playlist entry. I would like for it to continue when there is an error with a playlist entry, but abort when there is an error while downloading the playlist pages.
Is there some way to achieve that?
Aborting when a playlist entry can't be downloaded is problematic for my use case, because some playlists/channels have entries that never or rarely work, and so aborting on those would mean that all the later entries that actually do work would never get downloaded.
The problem with not aborting on playlist page error is that the playlist_index/playlist_autonumber as well as the playlist_count are then wrong, which I use as part as the filename.
If there is currently no way to achieve that, this can be considered a feature request.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Is there some way to achieve that?

No

The problem with not aborting on playlist page error is that the playlist_index/playlist_autonumber as well as the playlist_count are then wrong,

Shouldn't happen. Log?

The behavior of playlist_index seems to have changed since the last time I've paid attention to that, it doesn't reverse the index when using playlist_reverse, but I'm pretty sure it used to. playlist_autonumber reverses numbering though, so I'll switch my script over to that.

The problem with not aborting on playlist page error is that the playlist_index/playlist_autonumber as well as the playlist_count are then wrong,

Shouldn't happen. Log?

I've used the Bitchute channel Liberum Arbitrium for testing because it has 27481 videos, so yt-dlp has to download over 1000 pages (high chance of something going wrong).
Log where it starts with index 2 (?) even though it's clearly not the second video on the channel, and yt-dlp even knows that there are 27481 videos, but because there was an error on page 19 it only got the latest 476 and bases it's numbering on that.
I think the correct number would be 27006. But since according to the documentation playlist_autonumber is based on the download queue, starting at index 1 might be the wanted behavior in this case?
I've tried using playlist_count+1-playlist_index instead, but got NA instead of a number, despite yt-dlp clearly knowing how many videos there are in total even with the downloading error (as can be seen in the


Eliminate the standalone console window on Windows

Goal
This can safely be hidden.

Hi @mottosso
Has this console window been hidden on you guys end ?
I find that using windowless pythonw.exe and subprocess.Popen to run avalon-setup/avalon.py could do the trick:
pythonw -c "from subprocess import Popen; Popen(['avalon'], shell=True)"

Hey @davidlatwe, no I haven't looked into this yet, and admittedly it hasn't been an issue; quite the opposite, I find the information is quite useful at times.
Maybe showing it could be an opt-in, like an environment variable?
$ set AVALON_SHOW_CONSOLE=1

and admittedly it hasn't been an issue; quite the opposite, I find the information is quite useful at times.

Oops, that's true, sorry, I forgot to mention that I was writing a .bat file for artists to start the launcher, so was looking for a magic cmd to hide this for them :P

Maybe showing it could be an opt-in, like an environment variable?
$ set AVALON_SHOW_CONSOLE=1

Ummm, I'll think about this, thanks !


Recreated conceptual docs folder for azps 4.2.0

Docs Build status updates of commit f8df54b:
:clock10: Pending: waiting for processors (36 builds ahead of you)
:warning: Docs Build is busy, currently there are 36 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit f8df54b:
:clock10: Full build: the average full build time is 28 min(s) 1 sec(s), this is based on the last 427 build(s) for this repository.

Docs Build status updates of commit f8df54b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-4.2.0/authenticate-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/azureps-in-docker.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/azureps-vm-tutorial.yml
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/context-persistence.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/formatting-output.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/get-started-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/index.yml
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/install-az-ps-msi.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/install-az-ps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/manage-subscriptions-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/migrate-az-1.0.0.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/migrate-az-2.0.0.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/migrate-az-3.0.0.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/migrate-az-4.1.0.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/migrate-from-azurerm-to-az.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/new-azureps-module-az.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/queries-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/release-notes-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/toc.yml
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/uninstall-az-ps.md
:white_check_mark:Succeeded

docs-conceptual/azps-4.2.0/using-psjobs.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Modal fix + Attended status for non-admins

PR deployed at: https://pr46-app-noblocknoparty.surge.sh


Known issue with Routr UDP connections in DigitalOcean Kubernetes (DOKS)

This thread documents a known issue and workaround for using Routr with UDP connections in DOKS.
Issue:
The DOKS load balancer prematurely closes UDP connections, impacting applications relying on long-lived UDP sessions. This leads to clients becoming unreachable despite having a valid registration time.
Example:
An SIP Agent sending a REGISTER request with a 60-second expiry will become unreachable within 15 seconds due to the closed connection.
Workaround:
Currently, the only known workaround is to set the expiry time for UDP registrations to a significantly lower value, such as around 10 seconds.
Additional Notes:
While this issue happens with DOKS it is possible it is present with other vendors, and further research will be needed.

Notes to self:

externalTrafficPolicyUDP = Local with NodePort: This approach might offer progress for issue # [number]. We'll need to provide the nodePort configuration via the chart.
sessionAffinity = ClientIP: This setting didn't improve the situation.
internalTrafficPolicy to Local: Changing this setting had no effect on the issue.
TCP connection: This works perfectly every time.
Using NodePort as the service type works perfectly for UDP which makes m believe the issue is with the load-balancer


Parser error for final ';'

Hi all,
Is this an error or an expected behaviour ?
Using this code :
import pydot

print(pydot.graph_from_dot_data('''graph G {
  A--B
}''')[0].to_string())

print(pydot.graph_from_dot_data('''graph G {
  A--B;
}''')[0].to_string())

I can see :

The comma at line 8 seems to create a node "\n" in the graph ??
Thanks,
PS: windows, python3.10, pydot1.4.2, pyparsing3.0.7

I've been able to reproduce this bug locally too and it looks like it's an issue caused by the later releases of pyparsing. Using 'graph {\n0 ;\n1 ;\n2 ;\n0 -- 1 ;\n1 -- 2 ;\n2 -- 0 ;\n}\n' as a test string I was able to confirm that running with pyparsing<=3.0.1 pydot parses the string correctly but running with pyparsing>=3.0.3  the extra newline node is added to the parsed output (pyparsing 3.0.2 raised an error trying to parse the string).

I hit this same issue and did a little digging into the root cause. I think the issue is the \n after the edge is matched by the ID (node_id), alphastring_ option, of node_stmt.
Excluding \n from alphastring_ seems to fix the issue.
alphastring_ = OneOrMore(CharsNotIn(noncomma + " \n"))
If this is the correct fix, \r and \t should probably be excluded too.
Here is a unit test that can be added to the test suite for this issue:
def test_parse_white_space(self):     g = pydot.graph_from_dot_data('graph {\n1;\n2;\n2 -- 1;\n}')     self.assertEqual(2, len(g[0].get_node_list()))
FWIW, I think the reason this doesn't happen with a node at the end of the graph is because the node_stmt includes Optional(semi.suppress()) which matches the ';' and then the Optional(semi.suppress()) in stmt_list skips the '\n'.

just ran into the same problem with a ghost "\n" node before graph closing '}'
            'name': '"\\n"',
            'parent_graph': <pydot.Dot object at 0x7fe11c6eee48>,
            'parent_node_list': None,
            'port': None,
            'sequence': 66,
            'type': 'node'}],

and came up with same fix for alphastring_ to include newline. also had issues with the keywords not being registered as such instead of literals.

Fixed


Adding Contributing.md with proper instructions to contribute to the project

The file will make the job of new contributors easier since they can follow the instructions in much better way

Sounds good. What are the specific details that should be included in a contributing.md?

First would be steps to fork the project
Then create a branch
Add upstream
Work on the issue
Push it to local
Create a PR
Add the details of your changes
End would contain links to useful git resources for beginners
Each of the above steps would contain the code associated with it in case someone does not want to read through the docs of git

@FireCubeStudios
Please review PR #53

Closing as PR #53 is adding this


Nil pointer

https://github.com/taigrr/elevenlabs/blob/3e3b7004b807a52642b3589b66e766e45d36cd57/client/tts.go#L77
Check the error here please, can cause nil pointer down the line

good catch, thanks!

@taigrr just FYI, you have two more of those in the same file


Continue when the changelog cannot be generated

This can happen if the repo has never have a versioned PR merged.
Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>Resolves: https://github.com/product-os/flowzone/issues/172

@klutchell this looks great! I went through the code and it looks reasonable (given my unfamiliarity with the full code base). Is there a way for me to tell my PR to use this branch to test it before I put any sort of "approval" review on it?

Don't worry about the approval yet, I'm going to re-review with fresh eyes
today. I also have some questions about your PR.
To test this PR on your PR, you can change @.*** to a
ref from my PR, like the branch or commit.
On Fri, Sept 2, 2022, 7:56 a.m. Micah Halter @.***>
wrote:

@klutchell https://github.com/klutchell this looks great! I went
through the code and it looks reasonable (given my unfamiliarity with the
full code base). Is there a way for me to tell my PR to use this branch to
test it before I put any sort of "approval" review on it?
—
Reply to this email directly, view it on GitHub
https://github.com/product-os/flowzone/pull/173#issuecomment-1235410237,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4CWIBKTN4MMZU2V4MELVDV4HTOZANCNFSM6AAAAAAQCW4N4M
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ah thanks for the reply @klutchell I noticed that and have been giving it a try :) I think it's running, I ran into a new issue but it is definitely on my end haha I set the version in the package.json to the initial version I wanted 0.1.0 and it was erroring because the "current version" is the same as the "updated version" hehe

https://github.com/balenaltd/security/pull/1
Looks like i got passed that hurdle though! Also could you disable versionbot on the balenaltd/security repo for me? :)

Oh @klutchell one thing i did come across, in the workflow file for Flowzone the steps for parsing the old version and parsing the new version are both named Parse version. It might be helpful/more clear to specify that one is parsing the old version and one is parsing the new :)

It might be helpful/more clear to specify that one is parsing the old version and one is parsing the new :)

Good call @mehalter , in fact I'm going to change the first version check to look for existing tags (since that's what versionist does) and we can avoid cases where the initial version was set in package.json but there is no tag.

@mehalter can you retry this branch with your PR again? I changed the logic to use tags for the old version, and since your repo doesn't have any versioned tags yet it shouldn't matter what is in package.json (it will be overwritten by versionist anyway).

@klutchell looks like it worked perfectly!

@balena-ci I self-certify!


"Strejești" is listed in the drop-down menu

Description:
When user writes ”Strejești" the location appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running. After every step delete the text you typed from the search bar.

Step 1
Type "Strejești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Strejești, OLT".


feat: First draft of AEAD, DAEAD, DSA (Signatures), HKDF, MAC, JWS

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0f81087). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #31   +/-   ##
=======================================
  Coverage        ?   59.15%
=======================================
  Files           ?       88
  Lines           ?     9168
  Branches        ?        0
=======================================
  Hits            ?     5423
  Misses          ?     3745
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Added Option to Enable/Disable Driver Logging

Modified Contact, Switch, Siren, and Temp & Humidity drivers to permit user to enable/disable logging.
Combined with changes contributed by @borristhecat, 9 of 12 drivers will have enable/disable logging.

+1 - the fact that this has not gotten any attention is disappointing and concerning. A simple merge. Please address @konnected


✨ Feature Suggestion | Add software/covid-19 page

Description
In accordance with comments on issue #2301 @lrq3000 and me would suggest to add a collection of covid-19 apps. Covid-19 apps are often not privacy friendly but required to join venues. On F-droid there are already some alternative apps which could replace the the more evil once. This will help people not to get excluded from society and being able to protect their privacy.
It might also be nice to add apps that could store QR codes like Catima.

Yes I support this suggestion. I can make a PR (it will be the first time I design a new section but it will be a good training) if @editorial sees value in this proposition.
I would also suggest that this section adds mentions of how to uninstall automatically installed covid apps such as what Google did with Android phones in Massachusetts.

@privacytools/editorial I like this idea - what are your thoughts?

Good idea


Incident does not return username affected by Create/Lock

How do I show in Incidents the username that was created/blocked for example? Here in my environment it is working normally but it only brings the name of the incident and not the name of the affected user. Would I be able to bring the username when the trigger is triggered in Incidents?

Hi Ferspider3,
The event log details will be reflected in the "value category" when you select the specific event. Please recheck once.