pollymc-unwrapped: init at 8.0

Description of changes
added s0me1newithhand7s  into maintainers-list.nix.
added new package pollymc-unwrapped into nixpkgs/pkgs/by-name/po/pollymc-unwrapped.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

i'm really sorry for being such a dumb person but this is like my second time working with git not for creating dots or smth like this.. thanks y'all for patience!

Result of nixpkgs-review pr 325889 run on x86_64-linux 1

1 package built:

pollymc-unwrapped

Result of nixpkgs-review pr 325889 run on aarch64-linux 1

1 package built:

pollymc-unwrapped

so.. what do i need to do?
close PR?

There are Discord clients like Vencord (allows some paid features such as screensharing in 1080p) available in Nixpkgs using that obviously goes against Discord's TOS. Hosting modded versions of popular applications have been common to GitHub. If Minecraft wants to take down pollymc and its derivatives, I think they would strike the code repo first.
I am all for changing the license to Minecraft launchers in nixpkgs to lib.licenses.unfree.

There are Discord clients like Vencord (allows some paid features such as screensharing in 1080p) available in Nixpkgs using that obviously goes against Discord's TOS. Hosting modded versions of popular applications have been common to GitHub. If Minecraft wants to take down pollymc and its derivatives, I think they would strike the code repo first.
I am all for changing the license to Minecraft launchers in nixpkgs to lib.licenses.unfree.

you just came and save me, thank you sooo much. :D

so.. what do i need to do?
close PR?

This is what i recommend. I am by no means an arbiter in this, though

There are Discord clients like Vencord (allows some paid features such as screensharing in 1080p) available in Nixpkgs using that obviously goes against Discord's TOS.

Did I ever say that wasn't also a legal grey area and isn't a risk to nixpkgs?

Hosting modded versions of popular applications have been common to GitHub.

Let's call a spade a spade: this isn't a "modded" version of a popular application, this is a cracked version. The main reason it exists is to remove DRM inline with Mojang's EULA and allow for online play with no ownership of an account via alternative auth servers. I'm not sure why we're trying to act as if it's anything different
Regardless, how does something being hosted on GitHub mean it's ok? ..I don't understand your point here

If Minecraft wants to take down pollymc and its derivatives, I think they would strike the code repo first.

They could at any point, and I wouldn't be surprised if they did. Why should we wait for this and risk an incident with them?
I'd also be concerned by setting this as a precedent. Following the logic here and above: should we just start packaging any code off GitHub? What if the code is stolen? Leaked from a proprietary company? I don't think basing these decisions off companies not taking down repositories yet is at all viable long-term.

I am all for changing the license to Minecraft launchers in nixpkgs to lib.licenses.unfree.

This wouldn't make much sense -- plenty of launchers are FOSS and do not include unfree assets, including this one. The actual issue with this package and why it shouldn't be built is not because it's "unfree", but because it would make nixpkgs complacent in breaking a third party's EULA and it may be illegal in some users' areas. Conflating unfree software with this is a misuse of the license attribute
I would also like to reiterate: this is unmaintained software. It is out of date by almost a year and has been forked by a common contributor. Even if you can look aside the many legal issues with this package, I would highly advise against merging this as it will be dead on arrival. As an alternative I would recommend waiting on (or making) the follow up PR to https://github.com/NixOS/nixpkgs/pull/313011, where Fjord - the aforementioned fork of this project - was packaged. It no longer has many (any?) of the legal issues mentioned here as it re-added DRM, but still keeps the more legal features of pollymc such as support for FTB modpacks and alternative auth servers

I would also like to reiterate: this is unmaintained software.

let's ignore DRM part and focus on this. pollymc works? yep. user can play minecraft? yep.
in case fn2006 does not show in 4 mounths - someone, if not me, will fork it. isn't delete package easier than add?

let's ignore DRM part and focus on this

I don't think we should really ignore it considering it's implications...but sure, for the sake of this reply

pollymc works? yep.

Some things work. Many things also don't. From it's last release based on Prism, polly has not received fixes/updates required for

Not showing Microsoft refresh tokens in the log
Working "Sort by" filters in the Modrinth dialog
Support for Neoforge modpacks from Curseforge
Resolving issues with settings resetting after saving
Support for mod info from Neoforge versions >= 20.5
Preventing multiple crashes
Fixing many encoding issues
Importing vanilla Technic modpacks
Uploading screenshots to imgur

This is just a handful of changes from the last few minor releases (keep in mind a major release of Prism is also soon) of Prism upstream and doesn't include any bugs that may be unique to polly. These issues will also continue compounding long into the future with how rapid the pace of the modding community is
pollymc in this PR specifically also isn't in a great state, as only having an unwrapped variant basically restricts this to more advanced users who know how to override Prism's wrapper

user can play minecraft? yep.

*without owning the game

in case fn2006 does not show in 4 mounths - someone, if not me, will fork it.

As I said in my previous response, this has already happened. Once again:

As an alternative I would recommend waiting on (or making) the follow up PR to #313011, where Fjord - the aforementioned fork of this project - was packaged. It no longer has many (any?) of the legal issues mentioned here as it re-added DRM, but still keeps the more legal features of pollymc such as support for FTB modpacks and alternative auth servers

isn't delete package easier than add?

Considering it could break end user configurations, I wouldn't say it's much easier at the very least; I don't see how this matters though. Requesting to add a package you already plan on possibly removing just a few months later (which may or may not be before the branch off into 24.11) isn't a very good practice, and imo the discussion of it is another good sign this shouldn't be merged

the ONLY reason to close i agree with, from this point, is PollyMC being outdated. np! fork => package => PR. of course this will take some time, but provide thr option. needed option, whatever you say, my friend.

Playing devil's advocate a bit here — we have launchers that offer offline mode like HMCL in Nixpkgs already. If we were to follow the EULA completely, we have to remove those launchers too, or render them unfree.
However, I agree that an unmaintained fork of an existing launcher should not be accepted here — having a maintained fork would be a much more welcome alternative

Playing devil's advocate a bit here — we have launchers that offer offline mode like HMCL in Nixpkgs already. If we were to follow the EULA completely, we have to remove those launchers too, or render them unfree.
However, I agree that an unmaintained fork of an existing launcher should not be accepted here — having a maintained fork would be a much more welcome alternative

i just need to clarify, if i drop PR with working, maintained fork that still do not follow the Mojang's EULA - that's will be OK?

@getchoo already highlighted a previous PR where a package for the maintained fork of PollyMC was being proposed: https://github.com/NixOS/nixpkgs/pull/313011
So there already is a maintained fork of this software.

@getchoo already highlighted a previous PR where a package for the maintained fork of PollyMC was being proposed: #313011
So there already is a maintained fork of this software.

you'll see what i mean as soon as i get back home from work. :D


remove the unnecessary dependency pins in nlp-feature-extractors package

Description
Remove the unnecessary dependency pins in nlp-feature-extractors package.
The pin on tqdm caused issues with other dependencies during image create, even though it is completely unnecessary (there is no specific API used which needs a particular version, it's using the most basic functionality to create the status bar).  The pins are unnecessary and create a lot of dependency issues with other packages.  Also removing protobuf as it's not needed at all anywhere.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1759 (01b9348) into main (f3fad40) will increase coverage by 0.42%.
The diff coverage is n/a.

:exclamation: Current head 01b9348 differs from pull request most recent head 6bfcd9d. Consider uploading reports for the commit 6bfcd9d to get more accurate results

@@            Coverage Diff             @@
##             main    #1759      +/-   ##
==========================================
+ Coverage   88.30%   88.72%   +0.42%
==========================================
  Files          89       70      -19
  Lines        5250     4844     -406
==========================================
- Hits         4636     4298     -338
+ Misses        614      546      -68

Flag
Coverage Δ

unittests
88.72% <ø> (+0.42%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

raiutils/raiutils/common/retries.py

raiutils/raiutils/models/__init__.py

raiutils/raiutils/data_processing/__init__.py

...sk/rai_core_flask/environments/base_environment.py

raiutils/raiutils/dataset/__init__.py

..._flask/environments/credentialed_vm_environment.py

.../raiutils/data_processing/data_processing_utils.py

...ai_core_flask/environments/azure_nb_environment.py

..._core_flask/rai_core_flask/environment_detector.py

..._core_flask/environments/databricks_environment.py

... and 9 more

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


First 5Gb are not 5Gb 🤔

Some datasets*, for example vivym/midjourney-messages and Open-Orca/OpenOrca are not fully converted or copied(?) to parquet export.
These two are originally both in parquet format

midjourney original parquet files are about 8-9Gb in total, so the part of the data for datasets-server should be about 5Gb for sure but the viewer shows 1.56Gb:

original OpenOrca is two parquet files about 4Gb in size total but the viewer shows 2.85Gb:

Am I misunderstanding something or is it a bug?
*idk i assume there might be more of them but i have these two examples in mind.

Indeed, it sounds like a bug. For example, https://huggingface.co/datasets/vivym/midjourney-messages/tree/refs%2Fconvert%2Fparquet/default/partial-train shows only 10 160MB-Parquet files.
cc @lhoestq

Or maybe it's 5GB of decompressed data?

It's 5GB of uncompressed data indeed

@lhoestq is this what we want?

I think so, otherwise it could be possible to have a 1TB dataset stored in a 1MB Parquet file and most of our jobs couldn't handle it ^^

Good point. So: it's an issue for moonlanding, more than for datasets-server. But... I'm not sure we want to increase the message length AGAIN
https://github.com/huggingface/moon-landing/pull/8593#issuecomment-1883553457
Currently, on https://huggingface.co/datasets/timm/imagenet-12k-wds:
Size of the auto-converted Parquet files (First 5GB per split):
9.99 GB

where First 5GB must be understood as First 5GB of uncompressed data.
What should we show?

Maybe
Size of the auto-converted Parquet files (First 5GB per split) (?):
9.99 GB

with extra info on hover that shows "The Parquet export only contains the first 5GB of each split (uncompressed)"

otherwise it could be possible to have a 1TB dataset stored in a 1MB Parquet file and most of our jobs couldn't handle it

@lhoestq how is this possible?

otherwise it could be possible to have a 1TB dataset stored in a 1MB Parquet file and most of our jobs couldn't handle it

@lhoestq how is this possible?

I made up the numbers but if a dataset is made of 1TB of the same data, then a compression algorithm can theoretically compress the data to just two items (value, length) which could fit in <1MB.
Anyway this was just a way to illustrate than the compressed size of data can't give much information abotu the real data size in general.


One memory map storage

As we discussed:

One memory map storage per module call
Basis to have feature lists, mass lists, raw data files in ram only
taks need to work out where to get the storage from
some ims modules get their own storage
temp file deletion executed b MemoryMapStorage in the discard() method. That method requires an instance of theUnsafe, though.

Should i update the list in MZmineCore to a list of WeakReference in case the files could get garbage collected?


what is the default login info for amdin?

After
Clone
git clone https://github.com/Mosasauroidea/GazellePW.git
cd GazellePW
Start
docker-compose -p gazelle up
trying to login as：
user: gazelle
password: password
on
http://ipv4:9000/
( ! ) Fatal error: Uncaught DB_MYSQL_Exception: MySQL: Invalid Query: INSERT INTO login_attempts (UserID, IP, LastAttempt, Attempts) VALUES ('', '<IP_ADDRESS>', '2023-05-31 23:45:17', 1) SQL error: 1364 (Field 'BannedUntil' doesn't have a default value) in /var/www/classes/mysql.class.php on line 199
( ! ) DB_MYSQL_Exception: MySQL: Invalid Query: INSERT INTO login_attempts (UserID, IP, LastAttempt, Attempts) VALUES ('', '<IP_ADDRESS>', '2023-05-31 23:45:17', 1) SQL error: 1364 (Field 'BannedUntil' doesn't have a default value) in /var/www/classes/mysql.class.php on line 199

which os did you use?

Distributor ID: Ubuntu
Description:    Ubuntu 22.04.2 LTS
Release:        22.04
Codename:       jammy

which os did you use?

Distributor ID: Ubuntu
Description: Ubuntu 22.04.2 LTS
Release: 22.04
Codename: jammy

Distributor ID: Debian
Description:    Debian GNU/Linux 10 (buster)
Release:        10
Codename:       buster
fresh debian has the same errors with same install method of docker.

Change permissions of mysqld_sql_mode.cnf to 0444:
chmod 0444 mysqld_sql_mode.cnf
Then the mysql config could work


Donate button on website redirects to paypal.com, does not go to SWS team

I wanted to donate to the team at large specifically because of the utility I get from the groove tool (It's awesome). However when I clicked the big donate button on sws website it attempts to follow the link https://www.paypal.com/cgi-bin/webscr but gets redirected to paypal.com , maybe that account has been deleted or is inactive? I donated to Breeder instead for now.

but none of the donations go to the active contributors anyway

Out of curiosity, may i ask who the beneficiaries of the donations are? The original devs?

As far as I know, that account belongs to @swstim.


Amina/feature

Hello Team!
Can I proceed with a pull request here (or it's not a good idea :D) ?
Thanks.

For test.


chore: bdd-sbhg-1590202872 to 0.0.1

chore: Promote bdd-sbhg-1590202872 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
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

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


Add an option to sync via wifi only

Could you add an option to sync via wifi only please ?

We do have an option to download attachments via WiFi only.
Why do you need to stop sync completely without WiFi?

Yes we already have download attachments via wifi only and it's great !
The option to sync timeline via wifi only is usefull when you want to sync your timeline frequently at home but don't want on 3G/4G to prevent using data (even if andstatus doesn't use a lot of data) or to save battery (I am in this case).

Thank you for explanation, will definitely add this.

Thank you !

Any updates?

Implemented in v.27.00 (180), available at the Open Beta testing channel https://play.google.com/apps/testing/org.andstatus.app


cups-browsed.c: Introduce NotifLeaseDuration directive

The PR implements two things:

NotifLeaseDuration directive for cups-browsed.conf - it will make
lease duration for notifications configurable by users. IMO it is not
useful for regular users, but it is helpful during sanity testing
(for verifying that we actually renew the subscription when time
comes). The current hardcoded 1 day is unusuable for that :( .
I implemented the lowest threshold to 300s to prevent a possible DoS.
Subscription renewal is set to happen in the middle of NotifLeaseDuration,
not one minute before lease expiration. This was a problem on busy servers,
where cups-browsed was busy and wasn't able to renew the subscription
before cupsd removed it. Then if some jobs had come before the subscription
was created again, the queue got disabled. The proposed approach is based
on behavior of DHCP.

Connected to Fedora issue .

Thank you very much, your change is merged now.

Cherry-picked to 1.x branch as ff8aa21836.

Thank you, Till!


Dynamically adding multicheckbox to the forn does not show label and value

I'm submitting a bug  (check one with "x")
[x ] bug report

Current behavior

when dynamically adding adding a multicheckbox, it does not show correctly. Cab seen here
https://stackblitz.com/edit/angular-2ouh7u?file=src/app/app.component.ts
   {
      key: 'multiselect',
      type: 'select',
      templateOptions: {
        label: 'Multi-select',
        multiple: true,
        options: [
          {label: 'Iron Man', value: 'iron_man'},
          {label: 'Captain America', value: 'captain_america'},
          {label: 'Black Widow', value: 'black_widow'},
          {label: 'Hulk', value: 'hulk'},
          {label: 'Captain Marvel', value: 'captain_marvel'},
        ],
        change: (field, $event) => {
            this.getChange();
          }
      },
    },
  ];

  getChange() {
    this.fields.push ({
        key: 'demo',
        type: 'multicheckbox',
        templateOptions: {
          label: 'Demo',
          options: [{
            key: 'me',
            value: 'you'
          }]
        }
      })

  }

Expected behavior

it show show the passed value as label in the UI, in this case "you"
Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

mac, webstorm, npm, real rest endpoints server

Angular version: 2.0.X

angular 8

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

chrome

Language: [all | TypeScript X.X | ES6/7 | ES5]
typescript 3.5.3
Node (for AoT issues): node --version =
12.x.x

@aniruddhadas9 hey buddy!
If you want this to work under the current circumstances don't use
this.fields.push()
and instead use
this.fields = [...this.fields, newField];
or do a push and rebind the entire fields array with.
That being said, I'm not sure why just directly appending a field as long as change detection is occurring isn't properly propagating things but what I've provided to you now will fix that.
@aitboudad should this.fields.push() have no ill effects?

@aitboudad upon further investigation it looks like no formControl gets generated when you use this.fields.push as opposed to rebinding the entire fields array.  I'm not sure where but I was able to figure that out from console errors.
@aniruddhadas9 this is the reason why it doesn't get displayed properly.  console error causes break in the JS due to exceptions and you get a ton of "formcontrol isn't defined" errors.  Until that is corrected, you'll simply have to do what I previoiusly suggested.

push doesn't change the fields reference which angular rely on to detect changes, spread assign is the right way to solve the issue.
this.fields = [...this.fields, newField];


Incorrect VP8X Canvas Dimensions in Animated WebP Files

Summary
Piexif incorrectly sets the VP8X canvas dimensions for certain animated WebP files, leading to a "Frame exceeds canvas" error when frames exceed the dimensions of the last frame.
Steps to Reproduce:

Process an animated WebP file using Piexif where the frames vary in size.
Ensure that the last frame is smaller than some of the previous frames.
Run a tool like webpinfo on the output file to observe the error.

This can be reproduced with the pil_animated2.webp file in my fork: https://github.com/hMatoba/Piexif/commit/5a12974343580d5f6fbbed74966bc56f2859ffe9
Input Test File
→ webpinfo -diag tests/images/pil_animated2.webp
File: tests/images/pil_animated2.webp
RIFF HEADER:
  File size:  37342
Chunk VP8X at offset     12, length     18
  ICCP: 0
  Alpha: 1
  EXIF: 0
  XMP: 0
  Animation: 1
  Canvas size 400 x 400
Chunk ANIM at offset     30, length     14
  Background color:(ARGB) ff 00 00 00
  Loop count      : 0
Chunk ANMF at offset     44, length   5190
  Offset_X: 0
  Offset_Y: 0
  Width: 400
  Height: 400
  Duration: 70
  Dispose: 0
  Blend: 1
Chunk VP8L at offset     68, length   5166
  Width: 400
  Height: 400
  Alpha: 0
  Animation: 0
  Format: Lossless (2)

  ...

Chunk ANMF at offset  34824, length   2518
  Offset_X: 0
  Offset_Y: 6
  Width: 320
  Height: 382
  Duration: 70
  Dispose: 0
  Blend: 0
Chunk VP8L at offset  34848, length   2494
  Width: 320
  Height: 382
  Alpha: 1
  Animation: 0
  Format: Lossless (2)
No error detected.

Output Test File
→ webpinfo -diag tests/images/out/i_pil_animated2.webp
File: tests/images/out/i_pil_animated2.webp
RIFF HEADER:
  File size:  37400
Chunk VP8X at offset     12, length     18
  ICCP: 0
  Alpha: 0
  EXIF: 1
  XMP: 0
  Animation: 1
  Canvas size 320 x 382
Chunk EXIF at offset     30, length     58
Chunk ANIM at offset     88, length     14
  Background color:(ARGB) ff 00 00 00
  Loop count      : 0
Chunk ANMF at offset    102, length   5190
  Offset_X: 0
  Offset_Y: 0
  Width: 400
  Height: 400
  Duration: 70
  Dispose: 0
  Blend: 1
Error: Frame exceeds canvas in ANMF chunk.
Errors detected.

Expected Behavior:
The VP8X chunk should reflect the maximum dimensions across all frames to correctly represent the entire canvas, preventing any "Frame exceeds canvas" errors.
Actual Behavior:
The VP8X chunk dimensions are set based on the last frame processed. If this frame is smaller than earlier frames, the overall canvas size is incorrect, leading to errors.
This issue affects users processing animated WebP files with frames of varying sizes. Ensuring the correct canvas size in the VP8X chunk is critical for maintaining compatibility and avoiding errors in tools that validate WebP file structure.
Technical Details
The issue arises because the set_vp8x function in the piexif library sets the width and height based on the last frame it processes. This approach fails to account for previous frames that may be larger, resulting in an incorrect canvas size.
WebP Image (OK)
For non-animated WebP files, the VP8X chunk correctly represents the image dimensions:
+--------------------------+
|   WebP Container          |
|                          |
|  +--------------------+  |
|  |     VP8X Chunk     |  |  <-- Flags and size info
|  +--------------------+  |
|                          |
|  +--------------------+  |
|  |    VP8L Chunk      |  |  <-- Image data (Lossless)
|  +--------------------+  |
|                          |
+--------------------------+

Animated WebP (Broken)
For animated WebP files, the VP8X chunk incorrectly sets the canvas dimensions based on the last frame processed, even if earlier frames are larger:
+--------------------------+
|   WebP Container          |
|                          |
|  +--------------------+  |
|  |     VP8X Chunk     |  |  <-- Flags and canvas size info (incorrectly set)
|  +--------------------+  |
|                          |
|  +--------------------+  |
|  |     ANIM Chunk     |  |  <-- Animation control info
|  +--------------------+  |
|                          |
|  +--------------------+  |
|  |    ANMF Chunk      |  |  <-- Frame 1 (larger than last frame)
|  |  +--------------+  |  |
|  |  | VP8L Chunk   |  |  |  <-- Image data
|  +--------------------+  |
|                          |
|  +--------------------+  |
|  |    ANMF Chunk      |  |  <-- Frame 2 (smaller, last frame processed)
|  |  +--------------+  |  |
|  |  | VP8L Chunk   |  |  |  <-- Image data
|  +--------------------+  |
|                          |
+--------------------------+

Proposal:
Modify the set_vp8x function to track the maximum width and height across all frames. The VP8X chunk should be updated to reflect these maximum dimensions, ensuring that the canvas size accommodates all frames.
I have a commit in my fork that appears to resolve this: https://github.com/hMatoba/Piexif/commit/7c6522a312b0c95553ec19254ea88ed33efe12f3

Modify the set_vp8x function to track the maximum width and height across all frames.

That isn't necessarily enough. Some webp files have a canvas that's larger than all frames.
Here's a file where the canvas is 16x16 but the two animation frames are 12x10 and 8x2 respectively:
test.webp.zip
 % ~/Downloads/libwebp-1.4.0-rc1-mac-arm64/bin/webpinfo ~/Downloads/test.webp
File: /Users/thakis/Downloads/test.webp
RIFF HEADER:
  File size:    154
Chunk VP8X at offset     12, length     18
  ICCP: 0
  Alpha: 1
  EXIF: 0
  XMP: 0
  Animation: 1
  Canvas size 16 x 16
Chunk ANIM at offset     30, length     14
  Background color:(ARGB) ff ff ff ff
  Loop count      : 0
Chunk ANMF at offset     44, length     64
  Offset_X: 2
  Offset_Y: 4
  Width: 12
  Height: 10
  Duration: 100
  Dispose: 0
  Blend: 1
Chunk VP8L at offset     68, length     40
  Width: 12
  Height: 10
  Alpha: 1
  Animation: 0
  Format: Lossless (2)
Chunk ANMF at offset    108, length     46
  Offset_X: 4
  Offset_Y: 4
  Width: 8
  Height: 2
  Duration: 100
  Dispose: 0
  Blend: 1
Chunk VP8L at offset    132, length     22
  Width: 8
  Height: 2
  Alpha: 1
  Animation: 0
  Format: Lossless (2)
No error detected.

(This images was drawn by myself and saved using aseprite. Feel free to use it in tests, I hereby put it in the public domain.)

That isn't necessarily enough. Some webp files have a canvas that's larger than all frames.
Here's a file where the canvas is 16x16 but the two animation frames are 12x10 and 8x2 respectively:

@nico Thanks for the follow-up and that interesting test case! It's right to include examples of WebPs with canvases bigger than their frames - definitely adds another layer to think about.
I've taken a good look at your test fix, and I think it's actually handling these cases pretty well for spec-compliant WebPs. Here's the gist:

The code checks all chunks (VP8X, VP8, VP8L, ANMF) and finds the biggest dimensions.
If there's already a VP8X chunk (like in your test.webp), it considers those dimensions too.
It then creates a new VP8X chunk using the largest size it found, so all frames should fit.

For your 16x16 canvas example with smaller frames:

If there's a VP8X chunk saying it's 16x16, we'll use that size since it's bigger than the frames.
If there's no VP8X chunk, we'll use the size of the largest frame, which works too.

I've thrown a bunch of different WebPs at it, including some funky animated ones, and it seems to be handling them fine.
That said, if you've got any other tricky WebPs or a nuance to this case that I've missed, please let me know.
Thanks again for diving into this!


How many versions of MNE should we officially support?

Originally posted in https://github.com/lina-usc/pylossless/pull/161#issue-2423857408 :
Since v1.7 (April 2024), MNE uses edfio for writing EDF files. So we no longer should depend on edflib-python, but should use edfio.
There are a couple of ways we can go about this.

Add a lower cap on the MNE version in our requirements.txt file, i.e. mne>=1.7. This assures that people don't have an older version of MNE installed that will try to use edflib-python.
Follow SPEC0 and officially support all MNE versions that were released within the last 2 years (so that goes back to mne 1.4). Thus, we would need to add code that checks what MNE version the user has installed. If it is <1.7, check if edflib-python is installed. If it is >1.7, check if edfio is installed, etc.

Or I guess we could do a middle ground and continue to support mne v1.6, and thus add code to check if edflib-python or edfio needs to be installed, as previously described.
Personally I vote for 1, because I think the maintenance burden of 2 is currently too much for this project. Also, MNE only officially supports their latest stable release, so I don't want to support people using older versions that MNE doesn't even support themselves.
@Andesha @jadesjardins @christian-oreilly What do you think?

@scott-huberty : Duplication of what I answered in #161

SPEC0 is a nice initiative. I did not know about it. In general, I favor the requirements being as flexible as possible to avoid creating dependency conflict in downstream projects but I am good with 1 too because this is a relatively young project so I think we need to keep the maintenance burden relatively low until there is more buy-in from the community.

@scott-huberty Should we close this issue with a PR updating the requirements for mne>=1.7?

+1!


HDDS-10086. Intermittent timeout in TestSafeMode

What changes were proposed in this pull request?
HDDS-8982 added a new assertion in TestSafeMode and set timeout of 1 minute for the test case.  Encountered the following problem in a recent run:
Tests run: 2, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 103.8 s <<< FAILURE! -- in org.apache.hadoop.fs.ozone.TestSafeMode
org.apache.hadoop.fs.ozone.TestSafeMode.o3fs -- Time elapsed: 72.90 s <<< ERROR!
java.util.concurrent.TimeoutException: o3fs() timed out after 60 seconds

Initial selectContainer has correctly found none:
2024-01-08 10:08:41,553 [main] WARN  container.ContainerManagerImpl (ContainerManagerImpl.java:getMatchingContainer(344)) - Container allocation failed on pipeline=Pipeline[ Id: 30c296b9-71b9-4744-8977-9b77b35a0eb3, Nodes: e6c09ac7-730a-4058-a1a8-e64ffc2fb789(fv-az1117-812.frmogvfxo1lepnjzqgguis2xib.cx.internal.cloudapp.net/<IP_ADDRESS>)d591e5af-e8f5-464c-8a55-75d5e8fc5b83(fv-az1117-812.frmogvfxo1lepnjzqgguis2xib.cx.internal.cloudapp.net/<IP_ADDRESS>)6df5c950-b1f8-41e9-a8bd-47cd1228c3d6(fv-az1117-812.frmogvfxo1lepnjzqgguis2xib.cx.internal.cloudapp.net/<IP_ADDRESS>), ReplicationConfig: RATIS/THREE, State:OPEN, leaderId:e6c09ac7-730a-4058-a1a8-e64ffc2fb789, CreationTimestamp2024-01-08T10:08:39.438Z[Etc/UTC]]
java.lang.IllegalArgumentException
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:129)
	at org.apache.hadoop.hdds.scm.node.SCMNodeManager.minHealthyVolumeNum(SCMNodeManager.java:1204)
	at org.apache.hadoop.hdds.scm.pipeline.PipelineManagerImpl.minHealthyVolumeNum(PipelineManagerImpl.java:669)
	at org.apache.hadoop.hdds.scm.container.ContainerManagerImpl.getOpenContainerCountPerPipeline(ContainerManagerImpl.java:351)
	at org.apache.hadoop.hdds.scm.container.ContainerManagerImpl.getMatchingContainer(ContainerManagerImpl.java:331)
	at org.apache.hadoop.hdds.scm.pipeline.WritableRatisContainerProvider.selectContainer(WritableRatisContainerProvider.java:193)
	at org.apache.hadoop.hdds.scm.pipeline.WritableRatisContainerProvider.getContainer(WritableRatisContainerProvider.java:163)
	at org.apache.hadoop.hdds.scm.pipeline.WritableRatisContainerProvider.getContainer(WritableRatisContainerProvider.java:92)
	at org.apache.hadoop.hdds.scm.pipeline.WritableContainerFactory.getContainer(WritableContainerFactory.java:74)
	at org.apache.hadoop.fs.ozone.TestSafeMode.lambda$testSafeMode$0(TestSafeMode.java:123)

Pipeline creation failed, since no datanodes were available:
2024-01-08 10:08:41,553 [main] WARN  pipeline.WritableRatisContainerProvider (WritableRatisContainerProvider.java:getContainer(106)) - Pipeline creation failed for repConfig RATIS/THREE Datanodes may be used up. Try to see if any pipeline is in ALLOCATED state, and then will wait for it to be OPEN
org.apache.hadoop.hdds.scm.exceptions.SCMException: Ratis pipeline number meets the limit: 3 replicationConfig : RATIS/THREE
	at org.apache.hadoop.hdds.scm.pipeline.RatisPipelineProvider.create(RatisPipelineProvider.java:153)
	at org.apache.hadoop.hdds.scm.pipeline.RatisPipelineProvider.create(RatisPipelineProvider.java:57)
	at org.apache.hadoop.hdds.scm.pipeline.PipelineFactory.create(PipelineFactory.java:89)
	at org.apache.hadoop.hdds.scm.pipeline.PipelineManagerImpl.createPipeline(PipelineManagerImpl.java:255)
	at org.apache.hadoop.hdds.scm.pipeline.PipelineManagerImpl.createPipeline(PipelineManagerImpl.java:241)
	at org.apache.hadoop.hdds.scm.pipeline.WritableRatisContainerProvider.getContainer(WritableRatisContainerProvider.java:100)
	at org.apache.hadoop.hdds.scm.pipeline.WritableContainerFactory.getContainer(WritableContainerFactory.java:74)
	at org.apache.hadoop.fs.ozone.TestSafeMode.lambda$testSafeMode$0(TestSafeMode.java:123)

However, one pipeline was found to be ALLOCATED, so the call waited for that to be opened:
2024-01-08 10:09:41,554 [main] WARN  pipeline.WritableRatisContainerProvider (WritableRatisContainerProvider.java:getContainer(122)) - Waiting for one of pipelines [PipelineID=57157ebf-cb57-4f69-817d-8bea082c3750] to be OPEN failed.
java.io.IOException: Pipeline 57157ebf-cb57-4f69-817d-8bea082c3750 is not ready in 60000 ms
	at org.apache.hadoop.hdds.scm.pipeline.PipelineManagerImpl.waitOnePipelineReady(PipelineManagerImpl.java:772)
	at org.apache.hadoop.hdds.scm.pipeline.WritableRatisContainerProvider.getContainer(WritableRatisContainerProvider.java:120)
	at org.apache.hadoop.hdds.scm.pipeline.WritableContainerFactory.getContainer(WritableContainerFactory.java:74)
	at org.apache.hadoop.fs.ozone.TestSafeMode.lambda$testSafeMode$0(TestSafeMode.java:123)

The problem is that both timeouts are 60 seconds, thus the test may be aborted just before getting the expected IOException.
This PR increases test timeout to 2 minutes.  At first I tried to reduce pipeline report time to avoid unnecessary wait, and it has fixed the original issue, but hit another intermittent timeout shutting down datanodes (which is part of the original test, before the getContainer call).
https://issues.apache.org/jira/browse/HDDS-10086
How was this patch tested?
Passed in 10x20 runs:
https://github.com/adoroszlai/ozone/actions/runs/7447762180

cc @SaketaChalamchala

Thanks @duongkame for the review.


[QUESTION] Error: Could not open libcuda.so.

A problem of running the warp, here is the detailed information:
opencv-contrib not installed, some features will be disabled.
Please install with `pip3 install opencv-contrib-python`
Error: Could not open libcuda.so.
Warp initialized:
   Version: 0.3.1
   CUDA device: NVIDIA RTX A6000
   Kernel cache: /home/yuran/.cache/warp/0.3.1
2024-08-28 15:25:23,037 - mani_skill2 - INFO - load sdfs from file
Traceback (most recent call last):
  File "src/test_maniskill_env.py", line 7, in <module>
    env = gym.make("Pinch-v0", obs_mode="rgbd", control_mode="pd_joint_delta_pos")
  File "/home/yuran/anaconda3/envs/mani_skill2/lib/python3.8/site-packages/gym/envs/registration.py", line 235, in make
    return registry.make(id, **kwargs)
  File "/home/yuran/anaconda3/envs/mani_skill2/lib/python3.8/site-packages/gym/envs/registration.py", line 129, in make
    env = spec.make(**kwargs)
  File "/home/yuran/anaconda3/envs/mani_skill2/lib/python3.8/site-packages/gym/envs/registration.py", line 90, in make
    env = cls(**_kwargs)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/mpm/pinch_env.py", line 36, in __init__
    super().__init__(*args, **kwargs)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/mpm/base_env.py", line 95, in __init__
    super().__init__(*args, **kwargs)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/sapien_env.py", line 132, in __init__
    obs = self.reset(reconfigure=True)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/mpm/pinch_env.py", line 46, in reset
    return super().reset(*args, seed=seed, **kwargs)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/mpm/base_env.py", line 297, in reset
    return super().reset(*args, **kwargs)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/sapien_env.py", line 402, in reset
    self.reconfigure()
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/mpm/base_env.py", line 111, in reconfigure
    self._setup_mpm()
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/mani_skill2/envs/mpm/pinch_env.py", line 56, in _setup_mpm
    self.mpm_model = self.model_builder.finalize(device="cuda")
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/warp_maniskill/mpm/mpm_model.py", line 458, in finalize
    m0: MPMModel = super().finalize(device)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/warp_maniskill/warp/sim/model.py", line 1872, in finalize
    m.particle_q = wp.array(self.particle_q, dtype=wp.vec3, device=device)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/warp_maniskill/warp/types.py", line 534, in __init__
    dest = empty(shape, dtype=dtype, device=device, requires_grad=requires_grad)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/warp_maniskill/warp/context.py", line 1056, in empty
    return zeros(shape, dtype, device, requires_grad=requires_grad, **kwargs)
  File "/home/yuran/Projects/Mani_Derfom_GS3d/ManiSkill2-docs/warp_maniskill/warp/context.py", line 984, in zeros
    raise RuntimeError("Trying to allocate CUDA buffer without GPU support")
RuntimeError: Trying to allocate CUDA buffer without GPU support

does anyone meet the similar problem, is there any suggestion about this? Thanks a lot!

Hey @MarcYu0303, this is quite an old version of Warp which could make it difficult for us to diagnose. Is it possible to try running the latest version of Warp in a different Conda environment? It doesn't have to be this warp_maniskill problem, maybe just see if you can allocate an array on the GPU.

Can you also clarify if this is WSL2?

Thanks for your reply. I find the similar problem when running some other simulator, which means this problem is not related to Warp itself. And finally I solved it. Thanks again for your time!


first commit - modify slide

This is a modification to the slide

This is a modification to the first slide


Support disc numbers for sorting

Added as a track attribute in Subsonic API 1.8. Otherwise, sorting a multi-disc album is very mangled.

Could use a transformer, but the implicit disc numbering is visible in sorting now.


Issue: Understanding inconsistencies in data recording for LangSmith and LangChain JS/TS

Issue you'd like to raise.
Hello, I am using LangSmith to evaluate platform tools for cost and performance tracking and I am noticing some inconsistencies I do not understand.
For azure-openai and openai JS/TS packages exported from LangChain, I get inconsistent cost tracking. When I use the azure-openai package, costs are not recorded. Token counts seem to be consistent, but I get no cost information at all. I have seen similar unrecorded costs making AWS Bedrock calls. When I use the openai package, I get costs, but I'm not confident they are accurate.

No LLM calls record time to first token either, unless it is a local ChatOllama call. LangSmith reports the other values as "this run did not stream output", which is interesting as I am using LangChain's Ollama exported chat model object.
It doesn't matter if I try this in a RunnableSequence or direct invoke call - Azure calls do not get cost information.
When I inspect traces for these Azure runs, I see the following message when trying to interact with the Playground:

And OpenAI runs are correctly derived as OpenAI for the purposes of the Playground.
What am I doing wrong?

We need to relax the default model-matching rules we have for cost estimation. (I think azure openai returns gpt-35-turbo instead of gpt-3.5-turbo for instance)
You can customize the cost rules yourself if you'd like FYI, in case your pricing doesn't match the default ones

Hi @hinthornw thank you for your reply!
I discovered the list of models as well, and noticed the related regexes, and I've been using the model name specified in my deployment, which should match to the second regex on the unpinned model name (without a version, at least that is the OpenAI behavior)

And this is what the deployments in Azure show for their model names:

But it sounds like you are saying adding/cloning the model with a looser regex should address this - because on closer inspection we are using the legacy 0613 version.
Thanks again

Hello @hinthornw, i'm reopening this issue because it doesn't seem as if adding new regexes has helped, at least when using the exported AzureChatOpenAI object from @langchain/azure-openai.
Even for gpt-4, no cost attribute is collected at all. It isn't that they don't match, they simply are not returned at all.
const azureChatModel = new AzureChatOpenAI({
    azureOpenAIEndpoint: my-cool-endpoint
    azureOpenAIApiKey: super-secret
    azureOpenAIApiDeploymentName: also-super-secret,
    modelName: "gpt-4"
  });

For 3.5 turbo, the other params are the same but the modelName argument I've tried values that regex testers indicate should match, like just gpt-3.5-turbo, but they don't work, at least not for the AzureChatOpenAI object.
is there something else i should be inspecting?

Thanks for raising - will pass this on

@elliotmrodriguez Hey there, have you perhaps already figured why the stream requests are not tracked properly?
I have a related issue: No matter which way I trace my Custom LLM using WrapOpenAI / traceable / RunTree from the Typescript SDK, no first tokens is monitored. Any ideas? :)

Oh the typescript SDK may not track that event rn - I'll sync with the owner

I had the same issue. I managed to track cost by configuring a LangSmith model where the rule matches with  ls_model_name metadata (which was filled with the deployment name) 😉

https://docs.smith.langchain.com/how_to_guides/tracing/log_llm_trace


Use Option.when, whenever applicable

Getting nullable or non-existent values often required a method which returns an Option as a result.
This was often constructed via ternary or an if statement, and often repeated the logic the parent method contained (which threw an exception if not valid).
This was unified in: https://github.com/javaslang/javaslang/pull/1269

As stated on the PR, the code size is reduced by applying this change but we create an additional Supplier instance. This changes the runtime behavior of our collection. I'm afraid, therefore we can't apply this change.


add simple head/tail modifiers

Checklist

[x] I have added tests
[-] I have updated the docs and cheatsheet
[-] I have not broken the cheatsheet

cc @pokey -- not for review yet, a 📌 for a conversation for a possibly terrible idea i had about handling non-simple head/tail

I believe we discussed this and decided that it would be better to have one token to indicate the start of a head or tail modifier and one token to indicate the end. If nothing falls in between then it is a head or tail of line, otherwise it's a head or tail of whatever modifier sequence falls in between. Does that line up with your recollection?

Decided to opt for making it easy to tail one modifier because I think "head inside" is at least as common as "head line" for me tbh. See https://github.com/cursorless-dev/cursorless/pull/2428


Do you need a strong correspondence between the input text and lang_id

Hello, I would like to ask about the use of text and language ID as input. If the text and ID do not match, there may be repeated audio conversations, such as
{
"language": "KO",
"text": "你好，你叫什么名字，很高兴认识你",
}
Can this situation be monitored or monitored through a parameter setting in the model, or does it require checking the correspondence between text and lang_id before inputting into the model?

There is no way to figure out the language of the text once you put it into the model, so you need to make sure that the text and the language match before putting it in.
Here are some more details: There are two language settings internally in the model. For simplicity, we always change both to be the same in the demos, but if you write the code to run the model yourself, you can modify them individually if you want.
The first one always has to be the same as the language of the text and it can be changed with the following method:
https://github.com/DigitalPhonetics/IMS-Toucan/blob/6355e679207917b68486649bfb0065b7f556cacd/InferenceInterfaces/ToucanTTSInterface.py#L113
The second one can be a different one than the text. If it matches, it sounds more like the speaker speaks without accent. If it is a different one, it will sound more like the speaker has an accent of that second language. You can change it with the following method:
https://github.com/DigitalPhonetics/IMS-Toucan/blob/6355e679207917b68486649bfb0065b7f556cacd/InferenceInterfaces/ToucanTTSInterface.py#L116
A friend and I noticed that for many Asian languages, the choice of the speaker also has an impact on the accent, which is an accident and we are trying to improve it.


add start typescript setup

typescript configurations
sheets.js -> sheets.ts

Ping @paulirish, any objection to this?

@samccone thanks for the review 👍  I'm merging this one, if you have any comments for improving this stuff I can make it in another PR.

Lgtm!


📝 Describe zk and bk groups

@rpelisse I would like to propose this commit to improve the documentation with the new variables to identify the zk and bk groups.

@rpelisse I don't know if you were aware about this PR ... it is a PR for your PR, but I can't see it in the main branch. I think you merged first your PR and later this one, so this commit was not already merged into the main branch.

Oh, I think I know what happened. This PR has NOTHING to do with Describe zk and bk groups, but, when I created the PR, I had not rebased, so the first commit was "Describ zk and bk groups", which became the name of the PR.
I've renamed the PR, because we generate changelog from them, so it needed to be acurate.

Ok, I know what happened (I think). i'll cherry pick this commit into the main branch.

@rmarting Done. Can you double check that other commits / PR from you has not been overlooked? Thanks!

Everything is good! Thanks for your help.


[EPIC] Camera Class - Operation

Description
Create a Class that operates the camera in realtime. Relevant Story #18

[x] #32
[x] #31
[x] #30

Epic Done


ci: Fix the method of specifying the Node.js version

Summary
Unify the Node.js version specified for running tests in CI to the one specified in .node-version file.
Until now, v14, v16 and v18 were specified. However, since we're currently developing according to the version described in .node-version file, it should be sufficient if the test succeeds with this version.
Others
The format of the CHANGELOG.md is invalid for Prettier, but since this file is generated automatically by CD ( semantic-release ), it is appropriate not to have Prettier check it.

@taigakiyokawa
Sorry, I forget to clear the reviewer list.
This is still in Draft, so I'll ask for another review once it's officially open 🙏


[otp_ctrl] Add breakout type for convenient access to HW_CFG

This adds an autogen'd struct type for convenient access to items in the HW_CFG partition.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

Addresses part of #4443.

Ok I've reused the life cycle ON/OFF type to indicate whether the vector is valid or not.


Open File doesn't do anything

Testing #226665
Testing on latest VS Code Insiders on Windows 11 Insiders
Version: 1.93.0-insider (user setup)
Commit: ff7a154d5e5e9034914f0466420f0f1407f0c95e
Date: 2024-08-27T05:04:20.235Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.25987

If I try the buttons in the top right of the outputs of the diff view:

The first one Go To Cell works, but Open File doesn't do anything.

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

Duplicate of https://github.com/microsoft/vscode/pull/226181


Add autotest case

Coverage decreased (-0.02%) to 64.17% when pulling 9667a908d9249881a48990c0024585dd30cf5f34 on sigsbee:sp31 into f0f3cac6ed41610a2509aba9294c918347d4cbac on vmware:master.

can we also update the document accordingly?


Ivy peer dependencies #refactor_for_angular9

I'm getting this message when running my angular 9 project.
ERROR in The target entry-point "ngx-image-zoom" has missing dependencies:
 - @angular/core
 - @angular/common

Duplicate of #50 - release coming shortly that adresses this with version 0.5.0


Allow To Tweak Farseer Settings

Allow developers to tweak Farseer Settings in DualityAppData. This might be really useful for certain physics-heavy games.

This issue has been superseded by issue #257


Unable to redraw matplotlib plot when using interact

Unlike ipywidgets, the following minimally reproducible example does not appear to be working for me in the Jupyter notebook or Jupyter lab environment.
import matplotlib.pyplot as plt
import numpy as np

import panel as pn

from panel.interact import interact, interactive, fixed, interact_manual
from panel import widgets

pn.extension()

# from ipywidgets import interact

%load_ext autoreload
%autoreload 2
%matplotlib inline
%config InlineBackend.figure_format = 'retina'

def plot(minimum, maximum):
    x = np.arange(minimum, maximum)
    plt.plot(x, 3*x)

interact(plot, minimum=0, maximum=10)

The widgets display, the plot also displays on load, but scrubbing through the ranges does not update the plot.
Screenshot showing that the plot renders is below:

Given the similarity to the ipywidgets API, I was expecting this minimal example to work straight out of the box. I think I might be missing something conceptually here, but I am not quite sure. Would you be kind enough to enlighten me?

Sure. Panel's interact function works with the return value of the given callback, which means that you have to ensure that whatever you want displayed is returned as the value, not displayed using whatever strange side effect %matplotlib inline uses to make the plot appear.  I'm in no way an expert or even a casual Matplotlib user, so there is probably a much simpler way to get a proper Matplotlib figure out of a pyplot plotting call, but this should work:
import matplotlib.pyplot as plt
import numpy as np
import panel as pn
pn.extension()

def plot(minimum, maximum):
    x = np.arange(minimum, maximum)
    plot = plt.figure()
    plot.add_subplot(111).plot(x,3*x)
    plt.close(plot)
    return plot

pn.interact(plot, minimum=0, maximum=10)

I think you can get the current figure with plt.gcf()

@jbednar I see! Thank you for the pointers. That makes sense, thank you for correcting my misconception!

Thanks, @philippjfr.  That makes the answer much simpler:
import matplotlib.pyplot as plt
import numpy as np
import panel as pn
pn.extension()

def plot(minimum, maximum):
    x = np.arange(minimum, maximum)
    plt.plot(x,3*x)
    return plt.gcf()

pn.interact(plot, minimum=0, maximum=10)

I.e., add return plt.gcf() to the original code to return the current figure.

I did consider trying to replicate ipywidgets interact's behavior by looking for newly created matplotlib figures, but decided against it at the time.

That would require spooky action at a distance, so I vote against it.  I've added this question to the FAQ in https://github.com/pyviz/panel/pull/161.  Thanks @ericmjl for bringing up the issue!


pre-commit python version mismatch

Running a fresh flexmeasures installation and following Developing for FlexMeasures, pre-commit didn't work for me.

An unexpected error has occurred: CalledProcessError: command: ('/home/victor/Work/Seita/flexmeasures/venv/bin/python3.10', '-mvirtualenv', '/home/victor/.cache/pre-commit/repoq6jx57b4/py_env-python3.9', '-p', 'python3.9')
return code: 1
expected return code: 0
stdout:
RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.9'

The comes from using python 3.10 to create the virtualenv and pre-commit requires python 3.9 (language_version), which is not available in my system. Pre Commit configuration is defined in .pre-commit-config.yaml.
Possible solutions:

Harmonize python versions
Remove python language_version from pre-commit configuration

I believe we can leave it away, actually.
"For each language, [hooks] default to using the system installed language" (see here)


Add dependency analysis script

Resolves #2574
Example output is here: https://bl.ocks.org/bsiegel/raw/b3390e877ec666743eb9e9b8cb0ee0a4/

@scbedd Should engineering tools also require a README? If so I can add one but it doesn't really make sense to follow the expected headers convention.

@bsiegel Nah you're fine. Add the exception and call it good.


Set null terminator in String.from_iso_array (issue #1435)

When the Array[U8] passed to String.from_iso_array has pre-allocated
space, the first pre-allocated byte (the one at the _size index) should
be set to 0 in order to be compliant with how is_null_terminated
does its check. This PR sets that byte in such a case.
Fixes #1435

One of the tests is commented out because it relies on the String.space fix, which relies on the String.is_null_terminated fix, which relies on this fix. I'll enable that test in a separate PR after/assuming all this stuff gets merged.

Apparent unrelated single test failure on Appveyor with:
The command "make CC=$CC1 CXX=$CXX1 test-ci" exited with 2.

Done. Your build exited with 1.

/Users/travis/build.sh: line 148: shell_session_update: command not found

The linker failure above that is disturbing. I'll restart the build to see if it fails again.

Thanks @Praetonus, looks like that did the trick.

@Perelandric thanks for going down this rabbit hole and taking care of these issues!  It's much appreciated!


Basket. Show confirmation screen for several operations

Show modal with several operations carousel

Status verification: ⏳
What has been verified:

Sign page
https://github.com/novasamatech/nova-spektr/issues/1883

Status verification: ✅


Deploy traction and tenant-ui alpha to e79518-dev (Digital Trust)

No CICD, manually deploy.
Smoke test the current helm charts in bcgov/traction, make changes as needed.
Provide documentation for @WadeBarnes to run the charts from his desktop to the target namespace.
Start with traction only, add tenant-ui once traction is deployed successfully.

attaching a readme + values yaml file
Traction Helm Charts Deployment.zip

Traction Helm Charts Deployment
Dependencies

Openshift oc command line tools (version 4.7)
Helm command line tools (v3.6.2)
Admin access to an Openshift namespace (ex. e79518-dev).

Helm Charts
Clone traction repository, go to traction charts and create a new file for your values.
git clone https://github.com/bcgov/traction.git
cd traction/charts/traction
cat > values-e79518-dev.yaml
<paste in contents>

values-e79518-dev.yaml contents
global:
  ingressSuffix: -dev.apps.silver.devops.gov.bc.ca
acapy:
  image:
    tag: "ghcr.io/bcgov/traction-plugins-acapy:sha-03b72fc"
    version: 'sha-03b72fc'
    buildtime: '2023-02-13T21:12:08.001Z'
    pullPolicy: Always
  secret:
    adminurl:
      generated: false
    pluginInnkeeper:
      generated: false
  pluginValues:
    tractionInnkeeper:
      printKey: true
      printToken: true
  tails:
    enabled: true
tenant_proxy:
  image:
    tag: "ghcr.io/bcgov/traction-tenant-proxy:sha-03b72fc"
    version: 'sha-03b72fc'
    buildtime: "2023-02-13T21:12:09.053Z"
    pullPolicy: Always

Deploy

In a browser, login to your Openshift namespace, copy your login command
At a command prompt, paste your login command and execute. Log in to Openshift.
Ensure you are in the correct project/namespace
Navigate to the root of your traction repository
Execute helm commands

Dry Run
It is advisable to do a dry run and examine the output to see if it works and if there are any noticeable mistakes in resource naming, url formation etc.  This is the same call as the install/upgrade but with the --dry-run flag added.
helm upgrade -f ./charts/traction/values.yaml -f ./charts/traction/values-e79518-dev.yaml traction ./charts/traction --install --wait --dry-run

Install/Upgrade
The following can be used to install (if helm project does not exists) or rollout changes (ex. new configuration, new image).
traction is the name of the helm project.
helm upgrade -f ./charts/traction/values.yaml -f ./charts/traction/values-e79518-dev.yaml traction ./charts/traction --install --wait

Remove/Delete
Removal of all the resources created is a 2 step process:

uninstall the helm project
remove secrets and storage

helm uninstall traction
oc delete secret,pvc --selector "app.kubernetes.io/instance"=traction

UPDATED: Traction Helm Charts Deployment.zip


The is no description for a switch

The switch -BccBlocked doesnt have a description

I dont know if that is intended to be like that or is actually an issue

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

ID: 73241a5f-aabf-89a3-ce35-6c790b0a10cd
Version Independent ID: 8742e952-e7d1-1bcf-73cd-c6fec3ec483b
Content: New-DistributionGroup (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/New-DistributionGroup.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@nicoride93, it's intended to acknowledge the availability of the parameter without knowing whether it's supposed to be available (yet?) or if it's fully functional. See here here for a detailed explanation about empty parameter descriptions.
Are you being directed to use that parameter from other content, or are you just pointing out the fact that it's empty?

Sorry for the delay.
Yes it answer my question
Thanks all!
On Mon, Jan 16, 2023, 11:01 Yogesh Kumar @.***> wrote:

Closed #10319
https://github.com/MicrosoftDocs/office-docs-powershell/issues/10319 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/office-docs-powershell/issues/10319#event-8236288272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJJSMZF4UO747L4XZIX223WSVIAVANCNFSM6AAAAAATW2QUB4
.
You are receiving this because you were mentioned.Message ID:
<MicrosoftDocs/office-docs-powershell/issue/10319/issue_event/8236288272@
github.com>


ng-attr-custom-directive matches with customDirective

Use case: I wanna append an attribute (directive) depending on a condition.
Using ng-attr-custom-directive to add custom-directive dynamically to an element.
Issue: customDirective matches with ng-attr-custom-directive : My directive is instantiated and 'executed'.
Expected: customDirective doesn't match with  ng-attr-custom-directive , after execution of the ng-attr directive, it append or not, depending on the evaluation of the expression, and then, if appended, 'execute' the directive.
A plnkr showing the issue :
https://plnkr.co/edit/J1XKP7XWqvSEa6feoWmR

I'm not sure if this is expected behavior or not. I tend to say it's not, because Angular usually doesn't allow you to add directives "dynamically". It also doesn't copy the value of the ng-attr-... attributes, so it's not really useful there either. There are also no tests for this behavior. Still, I assume changing this would be a BC, because the behavior has been like this for a long time (although I see no practical use case for relying on that behavior).

I don't think it should be able to dymanically add directive's using ng-attr-....
The current behavior was kind of a surprise to me tbh. But then again, it has been like this for a long time and it only affects a very uncommon usecase, so I would probably not break it.

Maybe it's not the common usecase, but the fact that the compiler matches 'ng-attr-x' with 'x' directive is not normal: As the documentation says:
The normalization process is as follows:
Strip x- and data- from the front of the element/attributes.
Convert the :, -, or _-delimited name to camelCase.

@OmarNaghmouchi, this could just be a documentation issue :)

Here is how I think about it:

ng-attr-foo provides a way to dynamically add/remore the foo property.
Compilation of each element should only happen once, thus we can't dynamically apply/un-apply directives.
If we wanted to let ng-attr-foo determine whether a [foo] directive gets applied to the element or not, the decision should be made based on the info available at compilation time.
This would mean that we would need to interpolate based on the current state of the corresponding scope (i.e. before other directives have been compiled/linked, before directive controllers have been instantiated etc).
That would lead to all sorts of confusing bugs and unexpected behavior.

Such kind of logic seems better suited for either a parent directive's template function or the foo directive's linking functions.

Afaik, ng-attr-foo doesn't really allow you to remove the foo property - it allows you to add the foo attribute with an interpolated value. If you use it without an interpolated value, it doesn't add foo, and if you have an interpolated value, then there's no way to remove the attribute completely.
So I don' think figuring out if  foo gets added is difficult. It'll be added if the attribute contains interpolation.
And @gkalpak I'm not really sure if your comment concludes that you think the current behavior is correct or that we should change it. ;)

ng-attr-foo does let you remove the attribute (if it contains interpolation and the interpolated expression evaluates to undefined).
My comment concludes that we should not change the current behavior :smiley:

You are right. This should probably be documented somewhere, because I don't find that intuitive.

My basic problem is with the normalization process. I don't think angular should matches ng-attr-foo with foo as the documentation already said.

This should probably be documented somewhere, because I don't find that intuitive.

@Narretz, tt is.
--
@OmarNaghmouchi, as I (tried to) explain in https://github.com/angular/angular.js/issues/14575#issuecomment-219388881, we can't implement it in a way that is reliable and less unexpected than the current behavior. The problem is the directives should be determined during the compile phase, but ng-attr- can evaluate its expression during the linking phase.

 As my understanding of Angular, the $compiler will first, parse the DOM checking for directive and executing their compiling function once they match( found in the DOM). So, in this stage, while it is normalizing, it match the ng-attr directive but it is not supposed to match foo directive (which is not happening, it does match the foo directive), after all foo could just be another attribute (not a custom directive)), and in the linking phase, it evaluate the expression, depending on it value, it will append or not foo .
So, i don't see very well the relation between what i am saying and the phase of compiling/linking.
Maybe, i should take a look in Angular's code, ng-attr and normalize especially, to understand what u mean, but currently i am not seeing it.

@OmarNaghmouchi, assume we have:
// A `foo` directive
myApp.directive('foo', ...)

// The following HTML

Are you saying that the foo attribute should not be applied to that
 (regardless of the value of someExpression) ? That would be a major breaking change and I don't see this new behavior as more intuitive than the current one.
The whole point of ng-attr- was to be able to use interpolation (i.e. an Angular expression) to set the value of an attribute that doesn't normally accept {{...}}. In that sense, ng-attr-foo provides another way  for adding attribute foo to an element, so it is "more expected" that the $compiler treats it as if [foo] is present.
I am not even sure if the "feature" of removing the attribute when the expression evaluates to undefined was there at the beginning or added later as a side effect of $compile.directive.Attributes handle undefined values. To offer some perspective, all ng-attr- does is call attrs.$set(...) passing the value of the expression. It is Attributes#$set that decides to remove the attribute if value is undefined.
--
In any case, since ng-attr-foo is an alternative way of adding the foo attribute, I don't think it makes sense to never apply a foo directive. Ideally, one might expect that the value of the expression would determine if the directive gets added or not, but this is unfortunately not possible (as explained above).
So this only leave's us with always applying the foo directive, as if the user had used the foo attribute directly (and not via ng-attr-foo).

Are you saying that the foo attribute should not be applied to that  (regardless of the value of someExpression) ?

Yes, i am saying that the first time the $compile run, it shouldn't match the foo directive if it found:

It should only match it when it found something like:

 or

For ng-attr-, if the passed expression evaluate to something else than undefined, it add the attribute foo (which is in our case a directive) and call $compile to recompile it container element, in case of, the added attribute foo is a directive, and only then the $compiler should initialize/compile the foo directive.

@OmarNaghmouchi, the $compiler only runs once and re-compiling already compiled elements is not supported (and leads to errors most of the time). Se, we can't do that.

@gkalpak OK, thank you for this explanation.

I know it's been a while but I faced this issue today and the only solution that I found was to evaluate the required attribute's value inside the directive link function to whether apply it or not:
Inside custom directive's link function, first line:
if ($attrs.customDirective !== 'something') {
  return;
}

Of course you could perform more advanced or complex validations. I just hope this helps someone else.


vcpkg Dependencies

Is the vcpkg package just currently unusable?  From what I can tell this package depends on vulkan@2023-12-17 which depends on<EMAIL_ADDRESS>and because of that, my project is pulling in<EMAIL_ADDRESS>which isn't compatible with this package's latest due to the namespace issue talked about in #41.
I guess there's not much maintainers of this package can do, guess I'm looking for some others' perspectives on how they manage Vulkan dependencies in their projects. Up to this point, I've been able to use vcpkg for everything, but I guess an option is to submodule all the Vulkan related deps so I can carefully control which versions are pulled in. I think I would also need to hand craft some CMake stuff to find_package() the Vulkan SDK I have installed locally.

I use vcpkg overlay for force vcpkg to use locally installed Vulkan SDK.
Folder structure:

CMakeLists.txt
overlays

vulkan

portfile.cmake
vcpkg.json

(your other project files)

portfile.cmake
set(VCPKG_POLICY_EMPTY_PACKAGE enabled)

vcpkg.json
{
  "name": "vulkan",
  "version": "1.3.283"
}

At the configuration time, I pass -DVCPKG_OVERLAY_PORTS="../overlays". Then vcpkg will not use vulkan sub-dependency for any dependencies.


Hardcoded memory in M2 wdl

These should be parameters to the entire workflow.
Perhaps the runtime block should have an extra 1GB of RAM.

I don't think this is a good idea. We'd be giving the user to many knobs to turn. 99% of users won't need to touch these parameters, and they clutter the .json and .wdl. mutect2.wdl is already so cluttered I don't wanna look at it. I think all runtime parameters should be hardcoded.

If you organize the inputs into blocks and keep all such knobs together at the end it's not too bad. A lot of our users will need to be able to tweak those settings -- and the others can ignore them.
See here for an example of how we do it: https://github.com/broadinstitute/wdl/blob/develop/scripts/broad_pipelines/PublicPairedSingleSampleWf_160927.inputs.json

So we are a slimmer version of what @vdauwera has.
@takutosato I agree with your frustrations, but then we have to hardcode to the worst case, which will be quite expensive (in the cloud), underutilized (in all backends), and have trouble dispatching (in SGE).

@vdauwera I will happily accept comments on our json template.
https://github.com/broadinstitute/gatk-protected/tree/master/scripts/mutect2_wdl

@davidbenjamin @takutosato The more I think about it, the more important I think this issue is.

Yeah we need to parameterize the heck out of all our WDLs. If anything, the example I linked to is not parameterized nearly as much as I'd like (it's derived from the prod pipeline so we're a bit constrained).
It's not that much clutter if you make those parameters task-level and organize the JSONs clearly. And it makes it waaaay easier for people to adjust what they need without touching the WDL itself. This becomes even more important once you move the WDL into a platform like FireCloud, where changing the WDL is a huge pain, whereas tweaking parameters (via a method config) is trivial.

Issue moved to broadinstitute/gatk #2949 via ZenHub


Pass tracecontext from PDF generator cookies back to app

Description
When rendering PDF, get W3C traceparent propagated by the app and pass it back to the app when making subsequent requests.
Backend PR: https://github.com/Altinn/app-lib-dotnet/pull/924
Tested locally using Application Insights both with  OTel and the classic App Insights SDK
I don't know what the best place for this code is, but at least it needs to be pretty far upstream so that the interceptor is added before any downstream requests are made.
Related Issue(s)

N/A

Verification/QA

Manual functionality testing

[x] I have tested these changes manually
[ ] Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
[ ] No testing done/necessary

Automated tests

[ ] Unit test(s) have been added/updated
[ ] Cypress E2E test(s) have been added/updated
[ ] No automatic tests are needed here (no functional changes/additions)
[ ] I want someone to help me make some tests

UU/WCAG (follow these guidelines until we have our own)

[ ] I have tested with a screen reader/keyboard navigation/automated wcag validator
[x] No testing done/necessary (no DOM/visual changes)
[ ] I want someone to help me perform accessibility testing

User documentation @ altinn-studio-docs

[ ] Has been added/updated

[x] No functionality has been changed/added, so no documentation is needed
[ ] I will do that later/have created an issue

Support in Altinn Studio

[ ] Issue(s) created for support in Studio

[x] This change/feature does not require any changes to Altinn Studio

Sprint board

[x] The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
[ ] I don't have permissions to do that, please help me out

Labels

[x] I have added a kind/* label to this PR for proper release notes grouping
[ ] I don't have permissions to add labels, please help me out

Extracted it into a separate function in a file in src/feature (but still use the same evt handler)

Extracted it into a separate function in a file in src/feature (but still use the same evt handler)

That looks great, much cleaner

@martinothamar

Are there any good ways to unit-test this stuff?
I would probably have made a Cypress test for this. Loading an existing instance with ?pdf=1, intercept all requests and assert they contain the header we want.

Have you tried running cypress locally? 😅
npx cypress open -e environment=tt02
if you set it.only(...) on the test and click the pdf test in the cypress runner you can get a little faster feedback than testing in GitHub

Yeah the normal runner doesn't work for me, probably because I'm in WSL. Managed to run it in Docker but I'm not getting the same error so I think I can't run this locally atm


CI tends to time out

The MacOS CI has a nasty tendency to time out at the Homebrew step. (Example of such a failed job.)
Possible solution: putting brew update (as I'm assuming this is the line that times out) behind travis-wait.

This hasn't happened in a while, I'm assuming it's no longer relevant.


[ip6] add ipv6 fragmentation and defragmentation support

This PR enables the fragmentation and defragmentation of IPv6 in OpenThread to receive and send larger packets than the MTU (1280 bytes).
To enable the fragmentation support use:

Switch to enable: IP6_FRAGM
Core Configuration to enable: OPENTHREAD_CONFIG_ENABLE_IP6_FRAGMENTATION

For easy testing, the CLI can be used as follows (with two nodes in the same network):

Node One

udp open
udp bind :: 1234

Node Two

udp open
udp send -s 1800 <ip_address_node1> 1234 (sends 1800 bytes over UDP)

It is now possible to transfer 1800 bytes over a network with an MTU of 1280 bytes.
Main changes

src/core/net/ip6.cpp

Ip6::HandleFragmentation() Handles the fragmentation of messages that are too large to be sent.
Ip6::HandleFragment() Handles the assembly of the received IP fragments into an complete IP packet.

@ChrIgiSta, thanks for this contribution!

@ChrIgiSta, thanks for these updates! Added a number of other comments (mostly nits).
Can you also look to adding some functional tests that can be included in our continuous integration test suite on Travis? For example, see tests/scripts/thread-cert.

@ChrIgiSta, thanks for these updates! Added a number of other comments (mostly nits).
Can you also look to adding some functional tests that can be included in our continuous integration test suite on Travis? For example, see tests/scripts/thread-cert.

@jwhui, yes, I'll take a look on it.

@jwhui, i have added a test for testing the IPv6 fragmentation. The test uses the CLI to send a large UDP packet from one node to another (like description above). Is this test sufficient for this PR or should the package factory still be implemented for fragmentation?

@ChrIgiSta, thanks for adding the CI tests!
Reviewed this PR again and provided some additional comments (mostly nits).


add riscv64

This is the only blocker for https://github.com/ipfs/kubo/issues/8398
I tested this myself on some riscv64 hardware.

Oops duplicate of #1


dev: fake static api

Motivation: experiments and automation purposes.
PoC
curl install.octolab.org/lookup | jq '.godownloader' | bash -- -b /usr/local/bin

install.octolab.org/lookup or install.octolab.org/how
support wget and curl
friendly for https://github.com/octomation/taskfiles

first step
curl -sSf https://install.octolab.org/lookup | jq '.godownloader'
{
  "stub": true
}


Docker: Could not connect to server

I run on Windows 10

docker run -it --rm -e POSTGRES_PASSWORD=postgres -p5401:5432 postgres:12.3-alpine
it works fine. I can connect to PostgreSQL, create database etc, all is ok.

But when I run the same docker on my 2nd machine Windows 7 I receive
"Could not connect to server: Connection refused (0x0000274D/10061)
Is the server running on host "localhost" (::1) and accepting TCP/IP connections on port 5401?"
What may be the reason? what to check?

Can you give all the commands you ran and any relevant files or logs for reproducing the issue
On Linux:
$ docker run -d --rm -e POSTGRES_PASSWORD=postgres -p5401:5432 postgres:12.3-alpine
0743fd4b961e2b20d30817edfe2ef5048b5b25271b983c26bdea53b370a37556

Connecting to the host on the published port in a different container:
$ docker run -it --rm postgres:12.3-alpine psql -h <IP_ADDRESS> -p 5401 -U postgres
Password for user postgres:
psql (12.3)
Type "help" for help.

postgres=#

Windows 7 would be using Docker Toolbox which is a "legacy solution" https://docs.docker.com/toolbox/overview/ while Windows 10 would be on WSL2. So maybe some difference between the platforms and how you're running it?

If you are on Windows using Docker Toolbox (docker-machine/boot2docker), you won't be able to connect directly to a container's IP address from the host or via localhost.  You would need to start the container with -p 5401:5432 and then connect to the ip of the Docker VM on port 5401. use docker-machine ip to get its address.
Similar to https://github.com/docker-library/mysql/issues/26

If you are on Windows using Docker Toolbox (docker-machine/boot2docker), you won't be able to connect directly to a container's IP address from the host or via localhost. You would need to start the container with -p 5401:5432 and then connect to the ip of the Docker VM on port 5401. use docker-machine ip to get its address.
Similar to docker-library/mysql#26

Please check the 1st msg.
As I wrote:
docker run -it --rm -e POSTGRES_PASSWORD=postgres -p5401:5432 postgres:12.3-alpine
then try to connect from a client app and receive the error.
On Windows 10 it works, on Windows 7 it does not work.
I understand the Docker behavior on W7 differs from W10.
It seems that is the reason: Docker on W7.

If the client app is running on the Windows host and not in another container, then, yes, on Windows 7  it needs to connect to the IP of the VM (at the exposed port 5401) and not localhost:5401.

If the client app is running on the Windows host and not in another container, then, yes, on Windows 7 it needs to connect to the IP of the VM (at the exposed port 5401) and not localhost:5401.

Ah, ok, now it is more clear for me, yet one question: how to retrieve the IP of the VM ?

No, that is the IP of the container itself and would only be accessible from within the linux VM. Use docker-machine ip to get the IP address of the VM.

No, that is the IP of the container itself and would only be accessible from within the linux VM. Use docker-machine ip to get the IP address of the VM.

Ok, I pull and run the existing postgres docker on my Windows host machine. Where I can find the docker-machine ip?

docker-machine ip is the command you run that will give you the IP address of your VM.
Closing, given this is not an issue with the image.  In the future (and for further assistance), these sorts of questions/requests would be more appropriately posted to the Docker Community Forums, the Docker Community Slack, or Stack Overflow.

After the  testing: the Postgres Docker does not work on Windows 7 and works fine on Windows 10.
It is because of WSL vs WSL2.
Thanx.


Issue 8

purpose
Fixes a breaking MinIO issue on AWS hosts detailed in #8. Adds a region argument to the MinIO client constructor which defaults to empty string and can be set through the MINIO_REGION Docker flag.
testing
Local: verify that default value doesn't break MinIO functionality.

Recreate app with docker-compose up -d --build --force-recreate
Go to minimo.localhost and upload new data and metadata
Confirm that upload completed successfully and that new data are visible in metadata and data browsers

Cloud:

Recreate app with  minimo-issue_21]$ MINIO_REGION=us-east-1 HOST_NAME=<lightsail public IP> docker-compose up -d --build --force-recreate
Go to minimo.localhost and upload new data and metadata
Confirm that upload completed successfully and that new data are visible in metadata and data browsers

Local testing has been completed successfully. Our cloud host is currently being used for iterating on #21, so I'm going to merge this into master and then merge it into issue_21 and verify on the cloud host that it actually fixes the problem. Will comment below with outcome.

Cloud testing completed. Closing #8.


For Drawer, does manually dragging and closing the drawer behaves different from fling?

If I drag the Drawer and close it, the Scaffold will not rebuild. But if I fling it, the scaffold will rebuild after the Drawer is dismissed. Is there a difference between manually dragging to dismiss the drawer and fling it to let it close by itself?
If I call close() before this return statement (https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/drawer.dart#L232), manually dragging will behave the same as fling.
Thanks!

@Hixie @abarth
The use case here is that we have a custom AppBar icon solution that shows both back button and menu button. Basically the build function of our Scaffold providing widget sets the leading icons as follows:
ModalRoute.of(context)?.canPop == true
        ? [new BackButton(), new DrawerButton()]
        : [new DrawerButton()];

That causes the back button to show when drawer is opened (understandably). The problem is that when the drawer is dragged to a close slowly (such that no fling is executed), the backbutton remains since Scaffold is not rebuilt.
To me, this is a framework bug because there should not be any behavioral difference between dragging drawer to close vs flinging to close. However, I would also be happy if we can treat "drawer" as a special case and never show back button (perhaps by accessing "drawerIsOpen" state?). That seems to be the behavior for normal scaffold.

It seems that Drawer is using the local history handling instead of actual navigator history. If this is a contract (and not an internal implementation detail), we can solve this by:
  ModalRoute.of(context).isFirst ? [new DrawerButton()] : [new BackButton(), new DrawerButton()];

What do you think?

This reminds me some of https://github.com/flutter/flutter/issues/12691

That's weird. Using canPop in this way should register you to be notified when the drawer closes. There must be a bug with the way we trigger that.

I had a look at the code and I don't see any obvious reasons why this wouldn't work already. Do you have a reduced test case I can start from to debug this?

Without a reduced test case there isn't much we can do here. I'm going to close the bug. Please don't hesitate to reopen if you have more information for us to act on. Thanks!


nuxt generate with dynamic routes

Hi,
I am running a fully static website, with dynamic routes. When I execute nuxt generate, on the dist/amp folder all the dynamic files are missing.
I've tried to manually add the routes into the routeAliases without success.
Is there any way to make it work with dynamic routes?
Thank you

Works normal for me.
What's the version of the nuxt and place your nuxt config here.
You are using target: 'static'?
export default {   target: 'static' // default is 'server' }


Move query execution error message into query builder result panel

Summary

Move query execution error message into query builder result panel
How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

[x]  move execution error inside data grid

https://github.com/user-attachments/assets/7b6a5479-b3e9-4a8c-be51-3f3a16759b21

[x]  check entitlement highlight

[x]  no data warning

[x]  data incompleteness warning

[x] data truncate warning

lets update PR with description of everything you have done.

lets update PR with description of everything that has been updated with regards to the result grid.

Thanks. the changeset has been updated


Ionic serve performing dissapointingly very slowly

I have recently upgraded to ionic2 rc0 but unfortunately when i do an ionic serve it takes very long (close to five minutes) before my app appears on the browser. This get even worse when i make changes to my code and the changes take very wrong again before they are reflected on the app. This sometimes leads to the computer freezing.
I had been working with ionic2@beta and never experienced this issue. Please someone help.
ionic cli -v 2.1
node -v  5.12.0
npm -v  3.10.7

Can you run:
npm install @ionic/app-scripts@latest

We have released some fixes.
Thanks,
Dan

Thank you let me give it a try
On Oct 5, 2016 11:54 PM, "Dan Bucholtz"<EMAIL_ADDRESS>wrote:

Can you run:
npm install @ionic/app-scripts@latest
We have released some fixes.
Thanks,
Dan
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/issues/8518#issuecomment-251796150,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APTFV5gqI53ZnF6nIG8u3jUpatPoEu0rks5qxA5sgaJpZM4KPM32
.

I'm going to close this issue since it's a duplicate of https://github.com/driftyco/ionic/issues/8518. Can we track this there?
Thanks,
Dan

Sure
On Oct 5, 2016 11:56 PM, "Dan Bucholtz"<EMAIL_ADDRESS>wrote:

I'm going to close this issue since it's a duplicate of #8518
https://github.com/driftyco/ionic/issues/8518. Can we track this there?
Thanks,
Dan
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/issues/8518#issuecomment-251796873,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APTFVwn5CiTvmrui55Yxupx40HH4NetKks5qxA8KgaJpZM4KPM32
.

@danbucholtz duplicate of What?


[Extractor]Parameterize serviceUrl

I can't find any examples of how to use this???  I want to change the values in between environments, how do I do this????


Add a new company using elixir for their platform

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[x] Company name
[x] Company description
[x] Company location
[x] GitHub link
[x] How Elixir is used
[x] Sorted alphabetically

Awesome!  Always great to see non-profits 👍


SCSS/CSS not being generated

❓Question
I'm facing a couple issues with SCSS while upgrading from v3 to v4, and I'm looking for some help! My project uses SCSS modules, loadable components, Tailwind, and Typescript plugins for Razzle. I use the simplified const style = require('./style.scss'); method of importing styles.

The generated server html does not contain any style links, nor does the asset.json file have .css files shown. Specifically this means that assets.client.css is blank. (This is also the case with the with-scss project in the Razzle examples folder)
HTML class names in both client and server apps are output as "undefined". For example a container div element is output as
...

The


Fixing bug/#1822 avoiding recreation of bug/#1775.

Please check carefully.
It fixes #1822 for me, but I did never experience the issue mentioned in #1775.

Just tested your changes and they look good to me, I wasn't able to reproduce bug #1733 :+1:

This PR fixes the error described in #1822 as the ACTION_DOWN event on the zoom buttons is consumed as well (by returningtrue from the isTouched method.
Is there anything to be done supporting that this fix can be merged?


ENH replace unkown parent rt error with add unknown method on listener

All this needs now is an update of the docstring:
http://niprov.readthedocs.org/en/latest/niprov.logging.html
On line:
https://github.com/ilogue/niprov/pull/97/files#diff-a1eec8702eb8ff819a77e00479ae2868L13

This fixes #96


need to increase the max number of files that can be open

i saw you encountered this problem during big shuffles. here is how i fixed it. hope it helps.
Note: this is on amazon linux.

copy os's sysctl.conf
append "fs.file-max = 100000" to sysctl.conf
copy-file bigCluster /Users/jason/projects-misc/sysctl.conf /home/ec2-user/sysctl.conf
run-command bigCluster "sudo cp /home/ec2-user/sysctl.conf /etc/sysctl.conf"
run-command bigCluster "sudo shutdown -r now"
cat < limits.conf

soft nproc 65535
hard nproc 65535
soft nofile 65535
hard no file 65535
EOF

copy-file bigCluster /Users/jason/projects-misc/limits.conf /home/ec2-user/limits.conf
8)run-command bigCluster "sudo cp /home/ec2-user/limits.conf /etc/security/limits.conf"
9)run-command bigCluster "sudo shutdown -r now"

thanks for your great work.
-jason

Hi Jason, and thank you for the kind words!
Is this in reference to this comment?

yes it's a reference to that comment
Jason Herman
Blue Elephant Capital Management
Director of Quantitative Analytics, Portfolio Manager
Jason.Herman<EMAIL_ADDRESS>914-670-8994
On 26 Sep 2016, at 03:36, Nicholas Chammas <notifications<EMAIL_ADDRESS>wrote:
Hi Jason, and thank you for the kind words!
Is this in reference to this commenthttps://github.com/nchammas/flintrock/blob/7e811104184e1d45e02bf0f4056fbc2d53ba2746/flintrock/templates/spark/conf/spark-env.sh#L18-L21?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/nchammas/flintrock/issues/148#issuecomment-249499667, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATHyPcrx9O37cV3h71pcePlYLO3JnA7bks5qt3YXgaJpZM4KAh9r.


redirectpolicy: Move the feature behind feature flag and remove CCLRP related code

Check commit description.
Fixes: #13294

test-me-please

Sorry for the confusion. Looks like I forgot to mention about it in the commit description. Sure, I'll make a separate commit to remove the CCLRP related code.

Also, I don't see the CLRP being remove from the list of generated CRDs nor being removed from the bindata.

Did you mean CCLRP? We are still discussing whether to it's okay to register the CRD - https://github.com/cilium/cilium/pull/13671#discussion_r509806503
If you meant CCLRP, then I've commented out the relevant code. Based on the other review comments, I'll remove it entirely.

Saw this under the release-blocker issue https://github.com/cilium/cilium/issues/13294 as a release-blocker so just inheriting the label to the PR for visibility.

test-me-please

test-me-please

test-me-please

test-me-please

I'll be updating the gsg with the feature flag related instructions.

test-me-please

test-gke

All the tests have passed, except Travis CI and Cilium-Ginkgo-GKE.
The failure is because the pkg/redirectpolicy unit test was earlier not marked as a privileged test. Now that we have a bpftool probe check, it needs to be marked as one. With this fix, I'm able to run the test on the dev VM.
Latest commit diff -
git diff
diff --git a/pkg/redirectpolicy/manager_test.go b/pkg/redirectpolicy/manager_test.go
index f62eee7b6..8e3d81367 100644
--- a/pkg/redirectpolicy/manager_test.go
+++ b/pkg/redirectpolicy/manager_test.go
@@ -12,7 +12,7 @@
//  See the License for the specific language governing permissions and
//  limitations under the License.
-// +build !privileged_tests
+// +build privileged_tests
package redirectpolicy

test-me-please

retest-gke

The failure is because the pkg/redirectpolicy unit test was earlier not marked as a privileged test. Now that we have a bpftool probe check, it needs to be marked as one. With this fix, I'm able to run the test on the dev VM.

Makes sense, I think there are ways around this dependency (eg by injecting an object into the LRP manager at creation time which satisfies an interface to check for socket lookup support) but this also works.

The failure is because the pkg/redirectpolicy unit test was earlier not marked as a privileged test. Now that we have a bpftool probe check, it needs to be marked as one. With this fix, I'm able to run the test on the dev VM.

Makes sense, I think there are ways around this dependency (eg by injecting an object into the LRP manager at creation time which satisfies an interface to check for socket lookup support) but this also works.

Yeah, I'm not too happy about marking the test as privileged for this particular dependency. I'll be looking into this as a follow-up. Thanks!

test-runtime

test-gke

test-4.19

GKE tests are unstable overall - https://cilium.slack.com/archives/C7PE7V806/p1603758371186700
https://cilium.slack.com/archives/C7PE7V806/p1603758403187400

test-GKE

All tests have passed. But I need to rebase my branch, and resolve merge conflicts in following files -
install/kubernetes/Makefile
pkg/k8s/apis/cilium.io/v2/client/bindata.go
pkg/k8s/apis/cilium.io/v2/register.go

test-me-please

All tests have passed. But I need to rebase my branch, and resolve merge conflicts in following files -
install/kubernetes/Makefile
pkg/k8s/apis/cilium.io/v2/client/bindata.go
pkg/k8s/apis/cilium.io/v2/register.go

We can merge the PR once all the basic tests pass.


log a bug for *arr

the following apps don't have remote pathing correct... they have a typo:
radarr
readarr
lidarr
sonarr lets the mapping work appropriately but the others need the transmission path /data/completed (which doesn't exist) to be mapped to /media/complete (even though the directory is in fact complete not completed)

bug was actually in sonarr... transmission was set to use completed and not complete. Fixed the setting in transmission and the problem went away


How to run reset_db on Heroku? Fails with: "User does not have CONNECT privilege."

I can do database operations like manage.py migrate normally, but reset_db fails with:
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/app/.heroku/python/lib/python3.5/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/app/.heroku/python/lib/python3.5/site-packages/django/core/management/__init__.py", line 345, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/app/.heroku/python/lib/python3.5/site-packages/django/core/management/base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/app/.heroku/python/lib/python3.5/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/app/.heroku/python/lib/python3.5/site-packages/django_extensions/management/utils.py", line 58, in inner
    ret = func(self, *args, **kwargs)
  File "/app/.heroku/python/lib/python3.5/site-packages/django_extensions/management/commands/reset_db.py", line 145, in handle
    connection = Database.connect(**conn_params)
  File "/app/.heroku/python/lib/python3.5/site-packages/psycopg2/__init__.py", line 164, in connect
    conn = _connect(dsn, connection_factory=connection_factory, async=async)
psycopg2.OperationalError: FATAL:  permission denied for database "template1"
DETAIL:  User does not have CONNECT privilege.

I am using dj_database_url.config as mentioned at: https://github.com/heroku/python-getting-started/blob/33fafbd40ff5532def4ce7b3d0ebc1c7e34a0be7/gettingstarted/settings.py#L115
My full project: https://github.com/cirosantilli/free-books-django-raw/tree/1fd154df716e06e02a5a3afa299a89fe5513a112
Tested on Django 1.9.6, django-extensions 1.6.7.

Since Heroku is a managed platform you probable do not have access to the "template1" database. However the command needs to connect to something (other then the database you want to reset).
I would happily accept a PR that will make connecting to the "template1" database configurable. My guess is that that still won't help you. Since you are probably not allowed to do the DROP DATABASE and CREATE DATABASE statements as well.
If you want please verify this and make a PR for the documentation so that other people can read about this in the documentation.


GEODE-8531: Fix coredump in MapSegment::remove

Fixed a coredump in MapSegment::remove whenever the the region holding the
MapSegment object had consistency checks disabled.
Added an integration test in RegisterKeysTest TS as it was were this issue
was detected.

Revision 1 uploaded:

[x] clang-format executed.

Linux tests list:

[x]  Unit tests passed.
[ ]  Old integration tests passed.
[ ]  New integration tests passed.
[ ]  Acceptance tests passed.

Windows tests list:

[ ]  Unit tests passed.
[ ]  Old integration tests passed.
[ ]  New integration tests passed.
[ ]  Acceptance tests passed.

It seems that clang compilation does not like so much my implementation of the Gmock. Will fix it and let you know once ready for review :)

Clang unused warning has been solved and I've rebased develop changes into the branch
All the tests are passing.

Codecov Report

Merging #667 into develop will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #667      +/-   ##
===========================================
- Coverage    74.03%   74.01%   -0.02%
===========================================
  Files          644      644
  Lines        51133    51135       +2
===========================================
- Hits         37854    37847       -7
- Misses       13279    13288       +9

Impacted Files
Coverage Δ

cppcache/src/MapSegment.cpp
68.06% <100.00%> (+2.25%)
:arrow_up:

...tion-test/testThinClientRemoteQueryFailoverPdx.cpp
85.48% <0.00%> (-4.04%)
:arrow_down:

cppcache/src/ClientMetadataService.cpp
62.24% <0.00%> (-3.67%)
:arrow_down:

cppcache/src/TcrEndpoint.cpp
54.27% <0.00%> (-2.11%)
:arrow_down:

cppcache/src/TXCommitMessage.cpp
74.50% <0.00%> (-1.97%)
:arrow_down:

cppcache/src/RemoteQuery.cpp
75.53% <0.00%> (-1.07%)
:arrow_down:

cppcache/src/TcrConnection.cpp
72.48% <0.00%> (-0.48%)
:arrow_down:

cppcache/src/ThinClientPoolDM.cpp
75.87% <0.00%> (-0.22%)
:arrow_down:

cppcache/src/ThinClientRedundancyManager.cpp
75.62% <0.00%> (-0.16%)
:arrow_down:

cppcache/src/ThinClientLocatorHelper.cpp
86.11% <0.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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


Add APPEND Command

[x] APPEND command implementation
[x] Unit tests
[x] Integration tests
[x] Benchmark test

Benchmark

Fixes #638

Please resolve conflicts.

Please resolve conflicts and re-request review once the PR is ready.

Negative Cases

Will merge once the final comment is resolved.