[BUG] Tokens can be seen in exploration area despite having no vision

Module Version: v0.4.28
Describe the bug
Actors can be tracked in areas with no vision when fog exploration is on.
To Reproduce
Steps to reproduce the behavior:

Move actor to reveal area of the map
Move actor back to lose vision of the area leaving exploration vision
Move different actor through the revealed area but outside of vision

Expected behavior
Actors not in vision cannot be seen / tracked
Screenshots

Browser:

Chrome

Foundry Version: 9.255
Game System: DnD5e 1.5.7
Additional context
Not sure if this is actually intended functionality or I'm doing something wrong. No other modules aside from conditional visibility and its dependencies are active. In the screenshots above, the bottom green line is the vision range of the player character while the top green line is the edge of the exploration done by moving the character up and then down. The other token in blue can be seen as it moves in the area between the lines.

I tested it with brand new actors with no active effects on them. Also tried with the passive perception setting both on and off and received the same result. Interestingly enough, the problem only happens from a player perspective.
GM View:

Player View:

I can confirm this as well, except there's no difference between player and GM view for me. Also, I have a boatload of modules installed, so feel free to disregard this data point.

Additional problem found: the players can see GM hidden tokens (Toggle Visibility State) regardless of blocking walls.
I'll fire up an un-modded world and see if it still happens there as well.

Should be solved with 0.5.34


Romanian locale

ro.zip

Thanks, it's merged in commit https://github.com/jacwright/date.format/commit/8a5fb74dcc1466413ca51cbdcce67b332b378edb.


PP-3GB: Boot from SD even if no volume button is pressed

Hi and thank you for developing tow-boot!
Whenever the SD card is present, my PP-3GB boots from it, even if no volume button is pressed. Expected behaviour is the eMMC OS to boot, unless volume down button being pressed in the right moment, as described in [1].
I already asked on matrix room to make sure I was not misunderstanding some easy point, but it looks like some deeper investigation is needed.
I have installed latest tow-boot release (pine64-pinephoneA64-2021.10-005) following instructions in [2].
On both eMMC and SD I have installed Mobian using recent (Aug,21st) image: Mobian images since April, as far as I know [3], do not include u-boot anymore. On the SD card I also made sure to clear boot sector using instructions in [4].
[1] https://tow-boot.org/devices/pine64-pinephoneA64.html
[2] https://github.com/Tow-Boot/Tow-Boot/blob/released/boards/pine64-pinephoneA64/INSTALLING.md
[3] https://blog.mobian.org/posts/2022/03/30/universal-images/
[4] https://wiki.pine64.org/wiki/PinePhone_Installation_Instructions#Reuse_SD_card_for_data_storage_on_system_booting_from_eMMC

After some further tinkering with the SD card (at the moment, I have just one), tow-boot works as expected. Probably an old u-boot was still on the SD card (but I had wiped its first sectors as in Pine64 wiki [1]....) and it got over-written with later tests.
Before closing the issue, let me copy here the good hints I received on tow-boot matrix room: I have not found those anywhere else, then I hope that posting those here will help someone in my same situation via a simple search.
The question here is that TowBoot isn't loaded - the PinePhone will use the bootloader on the SD card first.
You have 2 choices with the OG PinePhone:
Install TowBoot on the SD as well
or
"neuter" the uboot on the SD card
Bootinfo [2] can be used both for checking active bootloaders and for wiping them.
[1] https://wiki.pine64.org/wiki/PinePhone_Installation_Instructions#Reuse_SD_card_for_data_storage_on_system_booting_from_eMMC
[2] https://sr.ht/~martijnbraam/bootinfo/


change in 3.8.3 breaks ruby 1.9

Subject of the issue
Version 3.8.3 introduces a change which doesn't work in ruby 1.9.
Your environment

Ruby version: 1.9.3-p551
rspec-expectations version: 3.8.3

Steps to reproduce
require 'delegate'
require 'pathname'
require 'rspec/autorun'

class A < DelegateClass(Pathname)
end

describe A.new('x') do
  it { should be_a(A) }
end

Expected behavior
.

Finished in 0.00333 seconds (files took 0.06674 seconds to load)
1 example, 0 failures

Actual behavior
F

Failures:

  1) x should be a kind of A
     Failure/Error: Unable to find -e to read failed line

     TypeError:
       bind argument must be an instance of Kernel
     # -e:1:in `block (2 levels) in <main>'

Finished in 0.00377 seconds (files took 0.11617 seconds to load)
1 example, 1 failure

Thanks for your report.

This PR will probably fix your issue https://github.com/rspec/rspec-expectations/pull/1112

@toy I believe this was fixed by #1112 if running against master/3.8.4 doesn't fix this for you please re-open!

@JonRowe Thanks, looks like 3.8.4 fixes the issue


Expose local BCV in verse mapping

In order for this to work, mappedCv would need to return something other than an itemGroup (or itemGroup would need to be extended to handle extra fields in all cases.)


Converted to Androidx and made this library live again

This project converted to Androidx and can be publish through jitpack.io

@tushar09  Thanks for opening this PR. I released a new version 3.4.0 that brings Fetch support to 14+. Please see the changelog for details.


Problem with destroy

I create few Luminous object dynamically and then trying to destroy them in the react application on componentWillUnmount.
It seems like destroy will throw an error if Lightbox was never activated:
value: function _unbindEvents() {
      this.trigger.removeEventListener(this.settings.openTrigger, this.open, false);
      this.lightbox.el.removeEventListener(this.settings.closeTrigger, this.close, false);

      if (this.settings.closeWithEscape) {
        window.removeEventListener('keyup', this._handleKeyup, false);
      }

in the code above this.lightbox.el can be undefined when this.lighbox.elementBuilt  is false

Good catch @podviaznikov! This is fixed and released as 0.2.7: https://github.com/imgix/luminous/releases/tag/0.2.7

Perfect, thank you for the fix!


InsertMode toggle not functioning

Hey there,
during testing I've noticed that the insertMode toggle does not function.
Configuring "insertMode: true/false" works as expected, however during typing switching between those modes via the insert key (https://github.com/RobinHerbots/Inputmask#insertmode) is not toggling the mode. Tested with different Browsers on Windows 10.
A look through the open and closed issues did not yield any information. Additionally, a glance at the source code yields commented out code (https://github.com/RobinHerbots/Inputmask/blob/5e74affee5c2ad35a44774f5a855d4290e93de1d/lib/eventhandlers.js#L46).
The change seems to be have made with the commit https://github.com/RobinHerbots/Inputmask/commit/c02deef0041857a4779d4d5ca9d0f100edcc98a7.
Could you please clarify the status on the insertMode toggle?

@ganjea ,
I don't know why I commented the functionality.  I enabled it again and made some minor enhancement.

Thank you for the honesty, quick response and fix!


Add install targets.

I realized I was putting the header in the top level include & it should be in a subdirectory.  I'll fix & re-submit.


Heatmap Widget for selected features of dynamic layers

Hi, In the heatmap widget( https://github.com/tmcgee/cmv-widgets/tree/master/widgets/HeatMap ), is it possible to create heatmap for dynamic layers (such that I can create heatmap of only selected fetures of dynamic layer)? If so, how can I do that?
Thanks in advance.

@ParkShinYoo That currently isn't supported in the HeatMap widget. I have considered adding in the future support for a url or a subLayer ID in the optional layers array configuration. That would allow you to use a layer from a dynamic layer for the heatmap.
That future enhancement by itself would not use only selected features for a heatmap. You could use the draw tools to select within an area. If you are interested in selecting features via attributes, that is not currently supported. I have considered a future enhancement to connect the heatmap widget to my AttributesTable widget which might assist with that.

Thanks @tmcgee , Can you give an example (sample code) for the layer array configuration of dynamic layers.

@ParkShinYoo Unfortunately, I cannot provide an example because, as I said above, using a dynamic layer is not currently supported.

Ok @tmcgee, Thank you. Actually my requirement is to create the heatmap for the filtered results of the filter widget ( https://github.com/roemhildtg/cmv-widgets/blob/master/widgets/Filter ) either for dynamic or feature service layers. But the filter widget is supporting only dynamic layers not feature layers. Whereas I used another heatmap widget ( https://github.com/ERS-Long/HeatMap ) it supports dynamic layers but it is creating the heatmap based on layerwise, But I require the heatmap of the displayed data(filtered result). Is it possible ?

@ParkShinYoo It seems to me the same filter from the dynamic sub layer could be applied to the feature layer created here or here.

@tmcgee, I am new to the Java Script and CMV. Can you help me how to apply the Dynamic Sub layer filter to Feature layer created in the Heatmap ( https://github.com/ERS-Long/HeatMap/blob/master/HeatMap.js#L47-L52 ).
Thanks in Advance

@ParkShinYoo This is untested but it might look something like this:
var heatmapFeatureLayerOptions = {
     mode: FeatureLayer.MODE_SNAPSHOT,
     infoTemplate: infoTemplate,
     outFields: ['*']
};
heatmapFeatureLayer = new FeatureLayer(serviceURL, heatmapFeatureLayerOptions);

// get the layer definition from the dynamic layer;
var filterExpression = r.layer.layerDefinitions(r.sublayer.id);
if (filterExpression) {
    // set the definition expression for the feature layer
    heatmapFeatureLayer.setDefinitionExpression(filterExpression);
}

@tmcgee Export button(https://github.com/tmcgee/cmv-widgets/tree/master/widgets/Export) is working fine for Excel & CSV whereas as for shafefile, .kml, .kmz, .geojson it was showing "Error during Export" popup. I couldn't figure out where the issue is.
Thank you.

This type of error usually occurs when there is no geometry for the features (SHAPE field is not included in the query) or when 1 or more features has missing or invalid geometry.

On second look, I think the one line should have been brackets instead of parentheses:
var filterExpression = r.layer.layerDefinitions[r.sublayer.id];

still untested.

@tmcgee Thanks a lot. Its working now.


question: Will sdm work in macOS? Has anyone tried?

The simple answer is no.
If I (or someone else) wants to port sdm here's a list of the packages sdm relies on:
systemd-container

This package provides systemd's tools for nspawn and container/VM management:

systemd-nspawn
systemd-machined and machinectl
systemd-importd
systemd-portabled and portablectl

qemu-user-static

QEMU is a fast processor emulator: currently the package supports ARM, CRIS, i386, M68k (ColdFire), MicroBlaze, MIPS, PowerPC, SH4, SPARC and x86-64 emulation. By using dynamic translation it achieves reasonable speed while being easy to port on new host CPUs.
This package provides the user mode emulation binaries, built statically. In this mode QEMU can launch Linux processes compiled for one CPU on another CPU.
qemu-user-static package will register binary formats which the provided emulators can handle, so that it will be possible to run foreign binaries directly.

binfmt-support

Support for extra binary formats
The binfmt_misc kernel module, contained in versions 2.1.43 and later of the Linux kernel, allows system administrators to register interpreters for various binary formats based on a magic number or their file extension, and cause the appropriate interpreter to be invoked whenever a matching file is executed. Think of it as a more flexible version of the #! executable interpreter mechanism.
This package provides an 'update-binfmts' script with which package maintainers can register interpreters to be used with this module without having to worry about writing their own init.d scripts, and which sysadmins can use for a slightly higher-level interface to this module.

parted

disk partition manipulator
GNU Parted is a program that allows you to create, destroy, resize, move, and copy disk partitions. This is useful for creating space for new operating systems, reorganizing disk usage, and copying data to new hard disks.
This package contains the binary and manual page. Further documentation is available in parted-doc.
Parted currently supports DOS, Mac, Sun, BSD, GPT, MIPS, and PC98 partitioning formats, as well as a "loop" (raw disk) type which allows use on RAID/LVM. It can detect and remove ASFS/AFFS/APFS, Btrfs, ext2/3/4, FAT16/32, HFS, JFS, linux-swap, UFS, XFS, and ZFS file systems. Parted also has the ability to create and modify file systems of some of these types, but using it to perform file system operations is now deprecated.

SDM scripts use linux specific packages. Would need to be ported to work on macOS.

Apologies for the late reply, was in the wilderness with poor/no internet.
I don't know macOS well, but indeed, sdm relies on the packages you identified, with the following exceptions.
sdm can use chroot on systems that don't support systemd-container, such as Windows WSL. Q for you: Is chroot supported on macOS?
qemu may NOT be required on macOS. Q: What is the output from the command file /bin/ls ? (binfmt-support is required only for use in conjunction with qemu)
parted is required for the --extend, --expand-root , and --shrink functions.
If there's any possibility of getting sdm to run on macOS via workarounds in sdm, I would be interested in working closely with someone to test and verify sdm there.

It appears looking at the replies to this gist that with the more recent hardened runtime chroot may. not work anymore in macos: https://gist.github.com/eddieh/b3c04436eb6b28826939cd57f1b704bb

It appears looking at the replies to this gist that with the more recent hardened runtime chroot may. not work anymore in macos: https://gist.github.com/eddieh/b3c04436eb6b28826939cd57f1b704bb
If I find out otherwise and if I can answer your other questions I'll re-open this.

I'm happy to do what I can to enable sdm on MacOS, but there are no Macs here, so unless someone sends me one, I'm 100% dependent on MacOS users to sort out what changes are required.
Of the packages listed in the OP, systemd-container is optional, assuming that chroot works. parted is required for anything that mucks with partitions (extend, add, shrink, etc), I don't know how to make sdm work without qemu and binfmt, since those are required to do a cross-platform chroot, AIUI.


FieldTypes

[x] Move MVP FT functionality to Stream\Field\FieldType
[x] Finish casting (done in simple form now)
[x] Render->(view / component) - will allow for decoupling/other systems?

Field types are standalone classes and have surrounding services. Addons are too heavy to carry them around 1:1 so 1 addon can register many field types. Our core field types will/can bind to primitive types too.

This stuff is done - moving to more granular issues.


VOX to Godot errors

Godot version
4.1.1stable
System information
Godot v4.1.1.stable - Linux Mint 21.2 (Victoria) - Vulkan (Compatibility) - GeForce GT 525M (nvidia; 390.157) - Intel(R) Core(TM) i7-2630QM CPU @ 2.00GHz (8 Threads)
Issue description
Cant use VOX models in Godot, getting this errors:
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:205 - Invalid get index '0' (on base: 'Dictionary').
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:50 - Unable to iterate on object of type 'Nil'.
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-mesh.gd:62 - Invalid get index '0' (on base: 'Nil').
core/io/file_access.cpp:762 - Condition "f.is_null()" is true. Continuing.
Importing: res://stand1.vox (scale: 0.1, file version: 200, greedy mesh: true, snap to ground: false)
Cannot open file 'res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh'.
Failed loading resource: res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh. Make sure resources have been imported by opening the project in the editor at least once.
Failed loading resource: res://stand1.vox. Make sure resources have been imported by opening the project in the editor at least once.
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:205 - Invalid get index '0' (on base: 'Dictionary').
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:50 - Unable to iterate on object of type 'Nil'.
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-mesh.gd:62 - Invalid get index '0' (on base: 'Nil').
core/io/file_access.cpp:762 - Condition "f.is_null()" is true. Continuing.
When im trying to drag object to scene getting this errors:
Cannot open file 'res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh'.
Failed loading resource: res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh. Make sure resources have been imported by opening the project in the editor at least once.
I'm not sure what's wrong. I'm just a newcomer from Unity Windows to Godot Linux; everything is new for me. I tried an older version of Godot 3, and it was still the same. By the way, I'm using Wine for MagicaVoxel to create VOX models, if this says something.
Steps to reproduce
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:205 - Invalid get index '0' (on base: 'Dictionary').
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:50 - Unable to iterate on object of type 'Nil'.
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-mesh.gd:62 - Invalid get index '0' (on base: 'Nil').
core/io/file_access.cpp:762 - Condition "f.is_null()" is true. Continuing.
Importing: res://stand1.vox (scale: 0.1, file version: 200, greedy mesh: true, snap to ground: false)
Cannot open file 'res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh'.
Failed loading resource: res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh. Make sure resources have been imported by opening the project in the editor at least once.
Failed loading resource: res://stand1.vox. Make sure resources have been imported by opening the project in the editor at least once.
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:205 - Invalid get index '0' (on base: 'Dictionary').
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-common.gd:50 - Unable to iterate on object of type 'Nil'.
res://addons/MagicaVoxel_Importer_with_Extensions/vox-importer-mesh.gd:62 - Invalid get index '0' (on base: 'Nil').
core/io/file_access.cpp:762 - Condition "f.is_null()" is true. Continuing.
Cannot open file 'res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh'.
Failed loading resource: res://.godot/imported/stand1.vox-f54938a4187fbe5528391b5c7c5cb083.mesh. Make sure resources have been imported by opening the project in the editor at least once.
Minimal reproduction project
Bicicle.zip

This is an addon, not a feature, please report your issues to the addon instead


Comply to RFC Section 3.1 (content lines)

Running example/write.rs also now passes the iCalendar validator. https://icalendar.org/validator.html

Thank you for your help 👍
I have to admit, I haven't read the RFC completely, could you point out the errors you found? Is CRLF part of the standard?

Hey! I plan to help a little more when I have time, and help port it to
Rust.
The CRLF and line folding is from this page:
https://icalendar.org/iCalendar-RFC-5545/3-1-content-lines.html
I ran it with the validator and it pointed out the long lines and line
endings, but also the 2 blank content lines, so removed those too.
The line folding is to absolutely ensure data won't be truncated no matter
how it is transported, such as email.
I think without the blank lines it can be more readable with the components
rearranged, like for some very strange reason it's printing DTEND before
DTSTART sometimes.
Hope to keep making it better ^_^ and nice work with laying all the
foundations.
On 04/21, Hendrik Sollich wrote:

Thank you for your help 👍
I have to admit, I haven't read the RFC completely, could you point out
the errors you found? Is CRLF part of the standard?
—
You are receiving this because you authored the thread.
Reply to this email directly, [1]view it on GitHub, or [2]mute the thread.
Reverse link: [3]unknown
References
Visible links

https://github.com/hoodie/icalendar-rs/pull/1#issuecomment-296141725
https://github.com/notifications/unsubscribe-auth/AXol7VxRoLEBOD5r4UkPw95dFKLTOAchks5ryHaWgaJpZM4ND0lD
https://github.com/hoodie/icalendar-rs/pull/1#issuecomment-296141725

Alright, does that mean you plan to add more commits to this PR or should I merge that already?
In that case I'll try to do that this weekend.

Oh I don't plan to add any more commits to this PR, so whenever it's most convenient for you


How to change the function of increasing resolution to increasing the scale of the generated image

Hello authors, I think SRGAN is somewhat in line with my research, and I would like to modify the functionality of SRGAN so that it can generate larger-scale images instead of increasing resolution.For example, a 2m2m rock image is generated based on a 2cm2cm rock training dataset.What code should I modify to meet my needs? Thank you

@catyoyo you should read the code to change the function by yourself

I am terribly sorry, my expression was not adequate, and I fear I may have caused you to misunderstand my meaning. I deeply dislike being the kind of person who simply demands that others complete everything for me. Rather, I humbly hope that you, in your wisdom and experience, might be able to provide me with some high-level guidance and direction. As a beginner, I know I am prone to wandering down many wrong paths, and it is only through the gracious and benevolent instruction of those who have gone before that I can hope to find a clear way forward, rather than stumbling blindly in the dark.I am truly and deeply grateful for any insights or perspectives you may be willing to graciously share with me. I humbly hope that you could provide me with some guiding direction on the paths I should consider for making modifications or improvements to the code.


Add PlayStation

Our repository settings require that all commits are signed. Could you fix that?

Oh, that might be my mistake 😅
One sec and I'll remove that requirement.


Invalid compilation error for valid match guard

Description:
Works on slbeta3, but results in an error on the current master. Debugged a bit and seems like it could be related to the changes done to fix https://github.com/ballerina-platform/ballerina-lang/issues/31782.
Steps to reproduce:
enum Department {
    FINANCE,
    LEGAL,
    ENGINEERING
}

type Employee record {|
    string name;
    Department department;
|};

public function main() {
    Employee & readonly employee = {name: "Jo", department: LEGAL};

    match employee {
        {department: FINANCE} => {

        }
        {deparment: LEGAL} if !update(employee) => { // error: incompatible types: expected '(Employee & readonly)', found 'other'

        }
    }
}

isolated function update(Employee & readonly employee) returns boolean => true;

Affected Versions:
slbeta4-SNAPSHOT

@MaryamZi employee can never be matched to {deparment : LEGAL} right? In that case shouldn't it give an error (not the error currently it gives, but with proper types) ? If that's not the case, https://github.com/ballerina-platform/ballerina-lang/pull/33241 will fix the issue

Sorry, my bad! :woman_facepalming: That was a typo. Error should be "pattern will not be matched" right?

Issue is fixed as the original fix done for https://github.com/ballerina-platform/ballerina-lang/issues/31782 is reverted. Keeping open to add tests


[Bug]: UI error Could not connect to the new realtime service introduced in beta.154

Description
I updated to version v4.0.0-beta.243 and this error appears.
Maybe this other issue is related: https://github.com/coollabsio/coolify/issues/1890
Minimal Reproduction (if possible, example repository)
Go to homepage or maybe open some menu around like Keys & Tokens and after a while the error will appear.

Exception or Error
No response
Version
v4.0.0-beta.243

maybe this was the issue: https://github.com/coollabsio/coolify/issues/1927


Line thickness

Nice control. Congratulations!
I think that a great improvement could be the option of change the line thickness. Currently it's very thin for my taste.
Another thing that I noticed that when I delete a character on a specific place the cursor return to the last character. I think that it's a better idea to leave the cursor on the same place. Make sense?

Thank you @mariovillamizar
I agree with you that having the line width as part of the style would be a great addition. However, it is not a high priority task for us and might not get introduced in the short term. I encourage you to submit a PR if you have the time :)
About the caret, it is a known bug and we will hopefully fix it for the next version. Thank you!


Typescript's Vue files resolution with project folders at Nuxt's root

I want to separate my Nuxt repository into different folders, one for each product (project and product names are not real, it's just for simplicity).
I've followed this documentation (example 2): https://nuxtjs.org/docs/2.x/configuration-glossary/configuration-srcdir
It works great but I have one last problem.
In my product1/test folder, I have a test file with the line import NuxtLogo from ~/components/ui/NuxtLogo.
I get the following error from Typescript Cannot find module '~/components/ui/NuxtLogo' or its corresponding type declarations.ts(2307).
However, if I add the .vue extension, the error goes away.
Directory structure for more clarity
my-project
  - product1
    - assets
    - components
      - ui
        - NuxtLogo.vue
    - layouts
    - pages
    - plugins
    - static
    - store
    - test
      - NuxtLogo.spec.ts
    - types
      - vue-shim.d.ts
    nuxt.config.ts
    jest.config.js
    tsconfig.json
 package.json
 node_modules

product1/types/vue-shim.d.ts
declare module '*.vue' {
  import Vue from 'vue'
  export default Vue
}

My tsconfig.json file is located in my product1 folder and contains "typeRoots": ["./types"]
Why do I get still get this error?

I found out that apparently, in .spec.ts file, you'd have to use the .vue extension anyway 😄


Rates API Route

Take in rate name and state/country name as parameters.
Find the average of that rate on the respective state/country. If no state/country given then find average of rate on all.
example return for incident_rate:
{
  "incident_rate": 1107.56
}

The bar chart needs data like this:
[
    {
        rate: 'Incident Rate',
        state_country: 0,
        us_world: 2568
    },
    {
        rate: 'Mortality Rate',
        state_country: 0,
        us_world: 2.14,
    }
]

This issue needs to handle both the rates component and the bar chart. There may need to be a few different endpoints made for it.

https://github.com/bhaden94/covid-19-tracker-v2-fe/issues/29
Complete FE code change along with the new API routes.


Unable to login again

Description
Have SimpliSafe broken the login process again? I had to reboot my homebridge yesterday and now I can't login to the SimpliSafe plugin. Using the process, Homebridge just says there was an error retrieving the token.
Expected behavior
Authenticate with SimpliSafe
Steps to reproduce
Follow standard authentication process
Homebridge logs
[5/27/2023, 8:40:59 AM] Homebridge v1.6.0 (HAP v0.11.0) (Homebridge 36A3) is running on port 51949.
[5/27/2023, 8:41:08 AM] [Homebridge UI] [homebridge-simplisafe3] Failed to check registry.npmjs.org for updates: "getaddrinfo EAI_AGAIN registry.npmjs.org" - see https://homebridge.io/w/JJSz6 for help.
[5/27/2023, 8:41:08 AM] [Homebridge UI] [homebridge-config-ui-x] Failed to check registry.npmjs.org for updates: "getaddrinfo EAI_AGAIN registry.npmjs.org" - see https://homebridge.io/w/JJSz6 for help.
[5/27/2023, 8:41:10 AM] [Homebridge UI] [homebridge-simplisafe3] Incoming Request: /credentialsExist
[5/27/2023, 8:41:10 AM] [Homebridge UI] [homebridge-simplisafe3] Incoming Request: /getCodeVerifier
[5/27/2023, 8:41:10 AM] [Homebridge UI] [homebridge-simplisafe3] Incoming Request: /getSSAuthURL
[5/27/2023, 8:42:16 AM] [Homebridge UI] [homebridge-simplisafe3] Incoming Request: /getAuthCodeFromUrl
[5/27/2023, 8:42:16 AM] [Homebridge UI] [homebridge-simplisafe3] Incoming Request: /getToken
[5/27/2023, 8:42:16 AM] [Homebridge UI] [homebridge-simplisafe3] Error: getaddrinfo EAI_AGAIN auth.simplisafe.com
    at SimpliSafe3AuthenticationManager.getToken (/homebridge/node_modules/homebridge-simplisafe3/lib/authManager.js:183:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async UiServer.getToken (/homebridge/node_modules/homebridge-simplisafe3/homebridge-ui/server.js:65:12)
    at async UiServer.processRequest (/homebridge/node_modules/homebridge-simplisafe3/node_modules/@homebridge/plugin-ui-utils/dist/server.js:75:30)

Homebridge config.json contents
{
    "name": "Simplisafe",
    "cameras": true,
    "debug": true,
    "sensorRefresh": 3,
    "persistAccessories": true,
    "cameraOptions": {
        "enableHwaccelRpi": false
    },
    "excludedDevices": [
        "03f0a8f",
        "f1079f4a"
    ],
    "platform": "homebridge-simplisafe3.SimpliSafe 3"
}

Screenshots
No response
Plugin version
1.10.13
Homebridge version
1.6.0
Node.js version
18.13.0
Host OS
Docker

Error: getaddrinfo EAI_AGAIN auth.simplisafe.com means your instance cant reach the internet for some reason.
Dont think login is broken.

Oh you're right, thank you! I didn't notice that.


fix: class 'Raw' doesn't work with current pyeparse

At some point pyeparse init used to include a line Raw = RawEDF (introduced in this commit but not sure when it was removed). Anyway, since it was removed the creation of a graph at the end of the pupillometry code block no longer works. This one-liner fixes it.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #369   +/-   ##
=======================================
  Coverage   88.87%   88.87%
=======================================
  Files          44       44
  Lines        5645     5645
  Branches      924      924
=======================================
  Hits         5017     5017
  Misses        428      428
  Partials      200      200

Thanks @drammock


Mehobe

https://1drv.ms/u/s!AoqeFY6v7qUYkW9_WLpL3JOSxSCO

https://1drv.ms/u/s!AoqeFY6v7qUYkW9_WLpL3JOSxSCO

Is this it?


Auto-Paginated API Methods Can Include Duplicates If Errors Occur

Affects version 0.5.0
Similar to #557
The method GetUsersContext internally performs pagination. However, if one of the paginated API calls to Slack results in an error (like exceeding a rate limit), then the error is ignored and the results from the previous page are appended again to the results list.
The resulting behavior is that the full results list can contain many many duplicates, especially in the case of exceeding rate limits, since the rate limit backoff is not respected by the pagination code in GetUsersContext.
Suggestion: GetUsersContext should be modified so that the results list is only appended to when there is no error, and should additionally perform backoff when the error is a rate limit error.
Alternate suggestion: The UserPagination object should set its Users property to nil when there is an error.

My mitigation was to replace in my code
users, err := api.GetUsers()
if err != nil {
	return err
}

with
var p slack.UserPagination
var err error
var users []slack.User
ctx := context.Background()
for p = api.GetUsersPaginated(); err != nil; p, err = p.Next(ctx) {
	if err == nil {
		users = append(users, p.Users...)
	} else if rateLimitedError, ok := err.(*slack.RateLimitedError); ok {
		time.Sleep(rateLimitedError.RetryAfter)
		err = nil
	}
}
if !p.Done(err) {
	return err
}

which gives me error handling, retry backoff, and fixes the duplication problem.

oh no. feel free to open a PR with the changes you suggested.


How to get the raw packet size?

I'm using node_pcap to create an activity dashboard for my home WiFi network.
In order to monitor used bandwidth I need to get the full size of each packet.
I've seen packet.payload.payload.length but it's usually undefined.
If you try getting the length of raw_packet it always is the same as the snapshot size.
How would you go about doing this properly?

I figured it out: packet.pcap_header.len


Added carousel navigation buttons and removed certificate button on education card and fixed its text, also fixed mooc card in dark mode.

Related Issues or bug
Related issue #333 and the recent concern that I raised in the comments of #324.
Also, fixed issue regarding text of education cards in dark mode that was raised in comments of #263
Fixes: #333 #263
Proposed Changes

Added Carousel navigation buttons and made it optimized for mobile view. Comments of #326
Removed certificate buttons on education cards as discussed earlier  in #302 and #333
Fixed text in education cards for dark mode. Refer to #263 for the context.
Fixed mooc cards background issue whose concern I raised here
Tested all the pages for issue in dark mode and mobile view and fixed wherever was required.

Made this video to make it easy for the maintainer to review. Watch video

@smaranjitghose @anushbhatia Can you review this PR? Took me a whole day to work on carousel buttons 😄


System wide currency configuration

Expected behavior:
We need to have a standard way to manage currency formatting for the entire application. We just merged #668 which provide a number field that we can use to allow users to input numbers. We could wire a new service into that input field to ensure that input values are sanitized correctly.
Actual behavior:
All locals are using $ as currency symbol.
Let's hash out all of the requirements around this service and everything that'll need to be implemented to make it happen.
Questions:

What library can we use to provide output formatting and input sanitization?
How do we wire this into existing localization?
How do we find all of the currency fields?

/cc @jkleinsc @wolfieorama

Does this not intersect (replace?) https://github.com/HospitalRun/hospitalrun-frontend/issues/314?

@BogdanAlexandru you are correct.  This is duplicate of #314.
@taras @wolfieorama I would like to close this as a duplicate of #314.

Closing as duplicate of #314


operators [N] verticadb-operator (1.1.0)

Signed-off-by: Matt Spilchen<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

This PR overwrites an existing operator as it just contains copy-edits in the description.  Can someone add the allow/operator-version-overwrite label?

Never mind, it already has the allow/operator-recreate label, which looks to be sufficient.

/merge possible

/merge possible


Add query modifiers functionality to MySQLQueryBuilder

This allows query modifiers like HIGH_PRIORITY and SQL_CALC_FOUND_ROWS to be added to MySQL queries.

Released as v0.24.0


Better error handling

The error type needs to be enhanced to more easily specify what kind of error occurred. Specifically I would like that at least the following events to have their own value in the error enum:

Authentication has expired
Invalid authentication
Incorrect sha1 checksum

The last one should probably be in its own error type as well.

This issue has been resolved with the methods introduced on the B2Error type.


Cors Fix

You are applying for OSS program

[X] Yes my project is OSS licensed and open to public.

Repository URL to your project
URL to your website
Expected Quota of usage
<1K requests per month.
Details
- Category: Appointment Booking
- License: Open Source
- Date of release: Dec 2022

Anything else?
No response

Hi. Sorry about the bad news, but we have a new policy: Minimum required of 5 stars for OSS projects. Otherwise it is considered as personal project.
I'm sorry to tell you this, but your application does not meet the requirements


Algorithm Classification and User Interaction Improvement

Description
This project encompasses a variety of algorithms, many of which belong to broader categories such as Monte Carlo algorithms. These algorithms share numerous aspects, including concepts and terminologies. Presently, the project's components are somewhat dispersed, necessitating a concerted effort to organize them into a cohesive structure. This particular issue emphasizes the user interaction aspect – how users engage with the system. It necessitates gathering user feedback, encompassing their preferences and criticisms. Your input and suggestions, as a user, would be greatly appreciated and valuable to this endeavor.

For details see #97


Check for disambiguation page in wiki info

Especially for place names, sometimes the first result in the wikipedia drop-down is actually a disambiguation page. Is there any way to detect this and warn the user so that the wrong info doesn't get added?

@1ec5 can I assign this to you?

Yeah, although I’m not entirely clear on how to fix this issue at the moment. The Wikipedia field appears to perform a very different API action based on the input length. action=query does have a way to check whether a specific page is a disambiguation page, but I can’t get it to work with list=search. There’s no such option for action=opensearch, although this would be a good opportunity to switch to action=query&generator=prefixsearch, which can pull in Wikidata-powered article summaries in the dropdown suggestions along the lines of what the www.wikipedia.org portal does.

#5647 solves this

It appears to be not fixed. After selecting Polish Wikipedia ("polski") in WIkipedia article adding menu and typing "Kalenic" I get "Kalenica" disambiguation page as a suggestion.
And #5647 seems to be about integration with OSM Wiki WIkidata, not with Wikidata ("Takes data directly from the Wikibase data items (OSM Wiki)")

It appears to be not fixed or regressed.

Verified.

action=query does have a way to check whether a specific page is a disambiguation page, but I can’t get it to work with list=search.

That’s because prop=pageprops (and thus ppprop=disambiguation) only works when setting titles= to specific page titles, not when searching with srsearch=. Adding an srprop might be tracked in T18449.
So I think filtering out disambiguation pages would require an extra call to the MediaWiki API for each search result, which would probably not be worth the trouble. On the other hand, if there’s a way to validate a field value as soon as it’s entered (or selected from the dropdown), there are multiple ways to check an individual page’s disambiguation status, such as using ppprop=disambiguation.


feat: move search archived messages to b2c web api

Description
In order to set the correct restriction on the frontend user when searching for archived message, it should be handled in the b2c web api
Story:
https://app.zenhub.com/workspaces/mosaic-60a6105157304f00119be86e/issues/gh/energinet-datahub/opengeh-edi/219
https://app.zenhub.com/workspaces/mosaic-60a6105157304f00119be86e/issues/gh/energinet-datahub/opengeh-edi/220
References

https://github.com/Energinet-DataHub/greenforce-frontend/pull/2274

Codecov Report
Attention: 267 lines in your changes are missing coverage. Please review.

Comparison is base (21978a1) 83.07% compared to head (1d1f9e6) 71.30%.
Report is 290 commits behind head on main.

Files
Patch %
Lines

...essages/RequestAggregatedMeasureMessageReceiver.cs
0.00%
53 Missing :warning:

...ages/B2CRequestAggregatedMeasureMessageReceiver.cs
0.00%
35 Missing :warning:

.../Api/Authentication/DevMarketActorAuthenticator.cs
0.00%
29 Missing :warning:

...uration/Middleware/UnHandledExceptionMiddleware.cs
0.00%
29 Missing :warning:

source/Api/EventListeners/InboxEventListener.cs
0.00%
24 Missing :warning:

source/Api/OutgoingMessages/PeekRequestListener.cs
0.00%
17 Missing :warning:

...frastructure/ArchivedMessageEntityConfiguration.cs
0.00%
17 Missing :warning:

source/Api/Program.cs
83.05%
10 Missing :warning:

source/Api/Common/HttpRequestDataExtensions.cs
0.00%
8 Missing :warning:

...rce/ArchivedMessages.Interfaces/ArchivedMessage.cs
11.11%
8 Missing :warning:

... and 16 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #634       +/-   ##
===========================================
- Coverage   83.07%   71.30%   -11.78%
===========================================
  Files         456      369       -87
  Lines       10418     8217     -2201
===========================================
- Hits         8655     5859     -2796
- Misses       1438     2196      +758
+ Partials      325      162      -163

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


refactor(button-group): remove unnecessary marginBlockEnd property

Closes #2670
Description
The issue where Button components arranged vertically in a column had overlapping outlines has been resolved. By removing the marginBlockEnd: -1px property from the button-group.tsx file, the problem was fixed without disrupting the overall style and layout of the buttons.
Current behavior (updates)

New behavior

Is this a breaking change (Yes/No):　No

Additional Information

horizontal items have not been improved.
Perhaps the -1px approach needs to be revisited.

No outline overlap can be seen when the button groups are parallel.

No update, so we are closing.


[icu] build failure (x64-windows)

Host Environment

OS: Windows 10
Compiler: Visual Studio 2019

To Reproduce
Steps to reproduce the behavior:
./vcpkg install icu:x64-windows
Failure logs
-- Configuring x64-windows-dbg -- Configuring x64-windows-rel -- Building x64-windows-dbg -- Installing x64-windows-dbg CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:155 (message):     Command failed: C:/Code/External/CppLib/vcpkg/downloads/tools/msys2/803521c5733336c9/usr/bin/make.exe -j 9 --trace -f Makefile install DESTDIR=/C/Code/External/CppLib/vcpkg/packages/icu_x64-windows     Working Directory: C:/Code/External/CppLib/vcpkg/buildtrees/icu/x64-windows-dbg     See logs for more information:       C:\Code\External\CppLib\vcpkg\buildtrees\icu\install-x64-windows-dbg-out.log       C:\Code\External\CppLib\vcpkg\buildtrees\icu\install-x64-windows-dbg-err.log
install-x64-windows-dbg-err.log
make[1]: ./../src/c-69_1-src-86e71a956d.clean/source/install-sh: No such file or directory make[1]: *** [Makefile:87: install-library] Error 127 make: *** [Makefile:153: install-recursive] Error 2 make: *** Waiting for unfinished jobs....
Additional context
There is a src/c-69_1-src-86e71a956d.clean/source/install-sh file, but maybe the reference path of ./../ is wrong?
It was working at some point, but is not working now. It may have been because I installed Visual Studio 2022, but I have since uninstalled it. But possibly there is some environment setting that was not reverted?

Thanks for posting this issue. Please provide files install-x64-windows-dbg-out.log and install-x64-windows-dbg-err.log.

I had this error as well.
OS: Windows 11
install-x64-windows-dbg-err.log
install-x64-windows-dbg-out.log
Compiler: Visual Studio 2019 and 2022

Thanks for posting this issue. Please provide files install-x64-windows-dbg-out.log and install-x64-windows-dbg-err.log.

install-x64-windows-dbg-err.log
install-x64-windows-dbg-out.log

There is message in install-x64-windows-dbg-out.log
./../src/c-69_1-src-86e71a956d.clean/source/install-sh -c icudtd69.dll
This is correct message: /usr/bin/install -c icudt69.dll /E/Code/External/CppLib/vcpkg/packages/icu_x64-windows/E/Code/External/CppLib/vcpkg/installed/x64-windows/lib
It looks like use incorrect install.sh or executable. Did you clone new vcpkg after uninstalling Visual Studio 2022?

@LilyWangLL I just tried to clone a new vcpkg and the only commands I did were the bootstrap and the install icu
I still get the same error.
New logs are attache
install-x64-windows-dbg-err.log
install-x64-windows-dbg-out.log
d.

@n4z4m3 Could you please use the attached file to install icu again? I did some fix in portfile.cmake
icu.zip
.

@LilyWangLL  Thank you, but that did not work. I tried crossing my fingers and everything.
New log files attached
install-x64-windows-dbg-err.log
install-x64-windows-dbg-out.log
.

@n4z4m3 Due to I can't reproduce this issue locally, could you remove folder [VCPKG_ROOT]/downloads/tools/msys2 and install again? Before installing icu, please use command git pull and .\bootstrap-vcpkg.bat to update vcpkg to latest version.

@LilyWangLL
Unfortunately your suggestion did not work.
However, I was able to get it to build by moving my VCPKG_ROOT to C:\vcpkg
Prior to that, it was in a subdirectory structure which I guess you can see in the log files. So somehow it does not like the subdirectory structure or maybe something at this directory structure is cached or stored in the environment. I did look at my PATH environment variable and it doesn't appear there.
I do have msys2 installed already on my computer and that is in my PATH. But I would not expect this to be an issue.

Hey guys,
I ran into the same error earlier.
After spending hours on trying to figure out the issue, I finally found my issue caused by directory symbolic link.
In my case, my vcpkg folder is d:\codes\vcpkg, I created a symbolic link in C drive c:\codes\vcpkg.
The error happens when I try to install the package from c:\codes\vcpkg location.
Problem solved after I switched to native location( d:\codes\vcpkg).
As I noticed, most packages won't have any issue with symbolic links, but ICU and QT5 will complain.
-Jiang

Please note that vcpkg may have bugs on symbolic link.

@lijgame and @LilyWangLL It looks like the symbolic links are my problem as well. And it also explains why it just started recently for me. My C drive was getting full so I needed to move vcpkg to my D drive, but I really wanted to keep it so that it appeared like it was in the C drive so I used a symbolic link. I guess I need to find a new way to do this.
Any thoughts on the next steps? Do I close this issue and that is that? Do I rename the issue to say that icu breaks when vcpkg root is a symbolic link? Or close this issue and open a new one?

I'm glad it works for you.
You can still use symbolic link, you just need to install packages from d drive.
I have no issues linking vcpkg libraries from c drive in my projects.
-Jiang

@lijgame
Thanks. That is exactly how I am now using it. I do the package install from my D drive, but then point to the symbolic link location on the C drive in my software that utilizes the libraries.

Hi @BillyONeal, could you help take a look this issue? Thanks.

Duplicated to https://github.com/microsoft/vcpkg/issues/19087.


Which listening method should be used for the event at the end of graphics editing?

Which listening method should be used for the event at the end of graphics editing?
for example：

There is no event at cancel. You can use pm:edit to listen to events during edit but when cancelling, there is no event yet. What you can do is check the toolbar button status on click on the map or add click event listeners to those cancel buttons and then check against that.
Hope it helps.


Build failed : LicenceNotAcceptedException

Here is the entire process:
https://colab.research.google.com/drive/1kfrnprEraNMKsiwEQWJwBXHVqxTZbcl4?usp=sharing
fails both on assembleRelease and assembleDebug

Did you do step 2when you insalled the android toolchain ?

yes

turns out i had a faulty part for when calling sdkmanager for the license part , but still no luck , here is what i got so far :
ninja
# https://lindevs.com/install-ninja-build-system-on-ubuntu/
!wget -qO /usr/local/bin/ninja.gz https://github.com/ninja-build/ninja/releases/latest/download/ninja-linux.zip
!gunzip /usr/local/bin/ninja.gz
!chmod a+x /usr/local/bin/ninja
!ninja --version

java
# https://stackoverflow.com/questions/58106622/how-to-change-the-java-version-in-google-colab

!apt-get install openjdk-17-jre-headless -qq > /dev/null
import os
os.environ["JAVA_HOME"] = "/usr/lib/jvm/java-17-openjdk-amd64"
!update-alternatives --set java /usr/lib/jvm/java-17-openjdk-amd64/bin/java

android toolchain
#https://github.com/BrunoLevy/geogram.android/blob/main/DOCS/TOOLCHAIN.md
!mkdir and
%cd /content/and
# download Command line tools only  ( https://developer.android.com/studio#command-tools )
!wget https://dl.google.com/android/repository/commandlinetools-linux-11076708_latest.zip -O commandlinetools.zip
!unzip commandlinetools.zip

accept licenses
!/content/and/cmdline-tools/bin/sdkmanager --sdk_root=/content/and/cmdline-tools/ --licenses

ndk, platform-tools and path
!/content/and/cmdline-tools/bin/sdkmanager --sdk_root=/content/and/cmdline-tools/ ndk-bundle
!/content/and/cmdline-tools/bin/sdkmanager --sdk_root=/content/and/cmdline-tools/ platform-tools
!export ANDROID_SDK_ROOT=/content/and/cmdline-tools/
!export ANDROID_HOME=/content/and/cmdline-tools/

geogram
%cd /content
!git clone --recurse-submodules https://github.com/BrunoLevy/geogram.git
!git clone https://github.com/BrunoLevy/geogram.android.git
!geogram.android/Tools/app_make.sh geogram/src/bin/geobox/main.cpp

manually add sdk directory to remove missing path error when building
!echo "sdk.dir = /content/and/cmdline-tools/" > /content/geogram.android/Apps/geobox/local.properties
!echo "ndk.dir = /content/and/cmdline-tools/ndk-bundle" >> /content/geogram.android/Apps/geobox/local.properties
!cat /content/geogram.android/Apps/geobox/local.properties

replace target ndk version
!sed -i 's/25.1.8937393/22.1.7171670/g' /content/geogram.android/Apps/geobox/app/build.gradle
!cat /content/geogram.android/Apps/geobox/app/build.gradle

run
%cd /content/geogram.android/Apps/geobox
!/content/geogram.android/Apps/geobox/gradlew assembleDebug --info

error :
> Task :app:configureCMakeDebug[arm64-v8a] FAILED
:app:configureCMakeDebug[arm64-v8a] (Thread[included builds,5,main]) completed. Took 2.498 secs.
AAPT2 aapt2-7.3.1-8691043-linux Daemon #0: shutdown

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:configureCMakeDebug[arm64-v8a]'.
> [CXX1405] error when building with cmake using /content/geogram.android/Apps/geobox/app/src/main/cpp/CMakeLists.txt: -- The C compiler identification is Clang 11.0.5
  -- The CXX compiler identification is Clang 11.0.5
  -- Detecting C compiler ABI info
  -- Detecting C compiler ABI info - done
  -- Check for working C compiler: /content/and/cmdline-tools/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/bin/clang - skipped
  -- Detecting C compile features
  -- Detecting C compile features - done
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: /content/and/cmdline-tools/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- Configuring incomplete, errors occurred!
  See also "/content/geogram.android/Apps/geobox/app/.cxx/Debug/1g5e2n53/arm64-v8a/CMakeFiles/CMakeOutput.log".

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org/

BUILD FAILED in 17s
22 actionable tasks: 22 executed
Some of the file system contents retained in the virtual file system are on file systems that Gradle doesn't support watching. The relevant state was discarded to ensure changes to these locations are properly detected. You can override this by explicitly enabling file system watching.


Update different-id.md

🚀 Deployed on https://661daad33decd74b36f2495c--cozy-fox-0defec.netlify.app


Field 382 subfield 2

Field 382 (Medium of performance) has subfield 2 for Source of term.  The converter adds bf:source for terms from subfields a, b, d and p.  However, the MARC format says subfield 2 applies only to subfield a.  We agree that source should be present for all four subfields but if you want to be in conformance with MARC practice you will eliminate it for terms from subfields b, d, and p.
Examples from a test record:
Correct
[ a bf:MusicMedium ;
        rdfs:label "382 Medium of performance subfield a" ;
        bf:count "382 e",
            "382 n" ;
        bf:source [ a bf:Source ;
                rdfs:label "382subfield2" ] ],

Incorrect according to MARC standards
[ a bf:MusicMedium ;
        rdfs:label "382 b" ;
        bf:count "382 n" ;
        bf:source [ a bf:Source ;
                rdfs:label "382subfield2" ] ],

[ a bf:MusicMedium ;
        rdfs:label "382 d" ;
        bf:count "382 n" ;
        bf:source [ a bf:Source ;
                rdfs:label "382subfield2" ] ;
        bf:status [ a bf:Status ;
                rdfs:label "doubling" ] ],

[ a bf:MusicMedium ;
        rdfs:label "382 p" ;
        bf:count "382 e",
            "382 n" ;
        bf:source [ a bf:Source ;
                rdfs:label "382subfield2" ] ;
        bf:status [ a bf:Status ;
                rdfs:label "alternative" ] ],

Good news.  The MARC standard is being updated to include subfields b, d and p.  So the converter is operating in conformance with the standard.
$2 - Source of term
MARC code that identifies the source of the term in subfield $a, $b, $d, and/or $p when it is from a controlled list.
This is a minor editorial change.  The revised definition will accompany the next MARC update, which will be published soon.

Thanks - Wayne will work on verifying the $2 issue.


Support for SalesForce CodeGen

https://github.com/salesforce/CodeGen
https://huggingface.co/docs/transformers/v4.28.1/en/model_doc/codegen#transformers.CodeGenForCausalLM
Apparently one of the best for code-generation.

Can you assign this to me

GPT-J is very similar to CodeGen, but not quite: https://gist.github.com/moyix/7896575befbe1b99162ccfec8d135566 (h/t @hhamud)
Once GPT-J's merged, we can look at either adding an option to GPT-J or setting up a separate CodeGen model that shares 99% of the logic.

GPT-J is very similar to CodeGen, but not quite: https://gist.github.com/moyix/7896575befbe1b99162ccfec8d135566 (h/t @hhamud)
Once GPT-J's merged, we can look at either adding an option to GPT-J or setting up a separate CodeGen model that shares 99% of the logic.

I've noticed that others just use that script to convert codegen to be able to run on code that is based on GPT-J's model.


Add a way to organise INSTALLED extensions.

Summary
Create a way to add "flags" to installed extensions on my machine.
These "flags" should,

allow us to search for them in the extensions tab like: @installed @flag:basic-cpp17
allow us to enable all extensions that has been found in said search
be available regardless of chosen workspace (maybe even synced between computers)

This would allow to us flag extension based on which work environments we want them enabled. Because of this extensions should be allowed to have multiple flags.
Reason
As a student I have to work in many different work environments: ANSI C, C++11, C++17, Python, Web(React, Vue, Ang),Haskell, Clean.
Because of this I have 90+ extensions which my laptop is just unable to handle reasonably well. And its too much of a pain to switch between sets of extensions manually so I just deal with the abismal performance. But lately I have encountered multiple really useful extensions which conflict when using different languages and make VSCode unusable. Please save me.

You can use Profiles feature for your usecase - https://code.visualstudio.com/updates/v1_75#_profiles


Any reason for MonotomicClock to be an empty tuple struct?

Hi, I'm wondering if there's a reason that struct MonotonicClock() is defined as such rather than just struct MonotonicClock. It's extremely, terribly minor, but the second is generally considered better style AFAIK.
EDIT: typo'd second example

Thanks for raising this - always up to improve my rusty style!
Unfortunately, I am missing something here, what's the difference between the two things you mention?

Functionally? Pretty much nothing - they will compile to the same thing, a ZST. They differ slightly in how they are constructed (the first being let x = MonotonicClock(); and the second being let x = MonotonicClock;), and the second is considered better style AFAIK (it's a little cleaner and removes the unnecessary ()s).
Similarly, another tiny thing: the GCRA struct is named GCRA rather than the preferred Gcra. According to the style guide, acronyms are to be treated as one word (e.g Uuid over UUID) for readability when things are combined.
These are both super minor things but I just thought I'd put them here in an issue in case there were reasons for it. I can open a PR to change it if you'd like?

Sorry, no, I meant just textually. The first and the second string is you posted in that initial message are identical (:
I see now, though - dropping the parens is ok - I'll see about that.
With the GCRA/Gcra thing, it kinda predates my knowledge of this convention... (and I'm kiiinda grossed out by it, but yeah - it's better to follow convention!)
I'd happily accept a PR to fix both of these issues!

. The first and the second string is you posted in that initial message are identical (:
Oh, haha, my mistake. I'll edit it for clarity.

@antifuchs #20 opened PR

Sorry to bring up tiny style stuff again... but is there any reason that constructors taking a clock take &Clock and clone them instead of Clock directly? The user could just clone it before passing it in if necessary, and taking an owned clock would remove a bound AFAICT.


import os in download.py

Please add to the script download.py, import os.
Thanks.

Hi @lfdelzam,
Which version are you running? This should be fixed in the latest version. Let me know if you still run into trouble!
Best wishes,
Bastiaan


Unintuitive behavior of "save action" checkbox

When having ktlint 0.20.0-beta-4 enabled on a project and going to the "save actions", unchecking ktlint formatting there, it is disabled completely for the project. This for me is pretty unexpected / unintuitive.
If you can make the ktlint option not shown on the on-save actions, or maybe better disabled if possible so that is just shows the status, this might be better.
If this is not possible, maybe some message could be displayed that this will disable ktlint completely for the project?

When refactoring the plugin, I have removed the option to enable/disable formatting on save partly but did not refactor the on-save-actions dialog. I have reverted this behavior as I now can see the value of it. In all my personal/work projects, I have added ktlint to the build pipelines and in that case it does not make sense to disable format on save as it increases the risk that the build pipeline will fail at a later stage. But when using the plugin as an optional tool, it can make sense to disable it, so that formatting is only invoked whenever explicitly invoked.


Replace contact list with document list component

What
Replaces the contacts list markup with the document list component.
Why
This is part of work done by the govuk accessibility team to replaces bespoke markup with instances of our components gem where possible. Replacing the bespoke markup present on the feedback index with one of our components allows us to easily roll out new changes and bug fixes to all apps without the risk of tech debt building up.
Whilst not a replication of the current design, the document list is a common pattern across govuk and we in the team have assessed that impact to users will be minimal.
Card
Visual changes
Before

After

🙌🏼 🚀
Side note: the "More topics" on this page, AT reads "DisclosureTriangle" - which is interesting


roc dev ends with exit code 0 on panic

A test with helloWorldNoURL.roc where I altered roc_fx_stdoutLine in examples/cli/cli-platform/src/lib.rs to panic:
❯ cargo run --release run examples/helloWorldNoURL.roc
    Finished release [optimized] target(s) in 0.24s
     Running `target/release/roc run examples/helloWorldNoURL.roc`
🔨 Rebuilding platform...
thread 'main' panicked at 'explicit panic', src/lib.rs:310:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5
Aborted (core dumped)

roc on  main [!] is 📦 v0.0.1 via 🦀 v1.66.1 via ❄  impure (nix-shell) took 18s
❯ echo $?
134

roc on  main [!] is 📦 v0.0.1 via 🦀 v1.66.1 via ❄  impure (nix-shell)
❯ cargo run --release dev examples/helloWorldNoURL.roc
    Finished release [optimized] target(s) in 0.18s
     Running `target/release/roc dev examples/helloWorldNoURL.roc`
🔨 Rebuilding platform...
thread 'main' panicked at 'explicit panic', src/lib.rs:310:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5

roc on  main [!] is 📦 v0.0.1 via 🦀 v1.66.1 via ❄  impure (nix-shell)
❯ echo $?
0

It may be relevant that roc_dev_native uses execve after libc::fork vs the "direct" execve call in roc_run_native_fast.
This incorrect exit code hides certain failures of cli_run tests.
Do you know how we could fix this @folkertdev?

friendly ping @folkertdev


gofumports depricated

From this commit: As the v0.1.0 release notes describe, gofumports was always a bit of a hack to allow IDEs to run both gofumpt and goimports at once. Now that gopls exists and bundles both features in an efficient way, people should just use that.
It's not installing anymore with gofumpt, and therefore the goimport command does not work
Edit: I just noticed gopls has not been integrated at all yet, was that done for a reason?

Is that means to do gofumports, will need to run goimports and gofumpt separately?
The reason did not bring in gopls is I have a separate setup for all LSP.  And -remote=auto of gopls does not work so well with that. (at least a few months back). B
TW, what you think may missing for gopls with default lspconfig?
Lsplens might be something worth taking a look when neovim have better support for that.

Is that means to do gofumports, will need to run goimports and gofumpt separately?
yes

The reason did not bring in gopls is I have a separate setup for all LSP. And -remote=auto of gopls does not work so well with that. (at least a few months back). B
Ahh okay, I have not setup gopls separately (myself, I did do LspInstall go), only use this plugin so far.

Lsplens might be something worth taking a look when neovim have better support for that.
Haven't heard of that, sounds nice

I used to use both vim-go and coc-go at once, they do not play well with remove/auto option and I have to startup 2 gopls instance. It is lots of memory if you have a large go project.
I will take a look if it is any better now and allow gopls to format code. And maybe some gopls features later if they were not included in lspconfig.

That sounds amazing

I'm using lsp-compe seems to work well

lsp-compe will not startup a server.  But it worthwhile to check how it share gopls with other plugins.

If you'd like to use gopls to format the code, Please enable gopls format by:
 require('go').setup(cfg = {
  lsp_cfg = false, -- true: apply go.nvim non-default gopls setup
  lsp_gofumpt = false, -- true: set default gofmt in gopls format to gofumpt
})

And
-- Format on save
vim.api.nvim_exec([[ autocmd BufWritePre *.go :silent! lua require('go.format').gofmt() ]], false)
-- or
-- Import on save
vim.api.nvim_exec([[ autocmd BufWritePre *.go :silent! lua require('go.format').goimport() ]], false)


Flask documentation in ePub/Mobi format as well

In the future please consider to publish the flask documentation not only in PDF but also in formats easily readable by an ebook reader like Kindle or Nook. At least for me it's much easier to read longer documentation on a e-ink device rather than a monitor, and PDF are rendered quite awfully on my Kindle 3g.
I see that there's a builder for Sphinx that permits to build epub files along with their HTML versions here: http://sphinx.pocoo.org/builders.html#module-sphinx.builders.epub
ePub would be a easier format to convert, through Calibre we could have a 1:1 conversion in MOBI format too.
Thank you

I uploaded a .epub, linked to the front page of the docs:
http://flask.pocoo.org/docs/flask-docs.epub
@fmaida, could you try out this ePub and let me know if it works well for you?
Thanks,
Ron

Thank you very much. I've just downloaded and converted it into MOBI format because unfortunately my Kindle cannot handle directly the ePub format, but I have always seen that the conversion from an ePub renders a perfect copy of the original file so I guess that if the MOBI file will be OK then the original ePub file is OK too.
I've uploaded the documentation into my Kindle and I'm bringing it with me so I'll read it a little bit while I'm going to work and coming back, and tonight I'll be able to tell you if there are any problems with the format.
Meanwhile, thank you again because today you saved my eyes from reading 250+ pages on a monitor ;-)

Thanks for the screenshots, looks sharp.  Code listing do not have much room, but that's the only problem I see.  calibre provides a command-line tool which we can use to automate publishing a .mobi: ebook-convert Flask.epub Flask.mobi.  Do you have any thoughts on conversion options?  Listed here:
http://manual.calibre-ebook.com/cli/ebook-convert-2.html#epub-input-to-mobi-output
I uploaded a conversion with the defaults. Could you try it?
http://flask.pocoo.org/docs/flask-docs.mobi
If it doesn't look good, we might consider converting from another format, e.g. the .pdf.  Conversion formats:
http://manual.calibre-ebook.com/cli/ebook-convert.html

I've read the first 20 pages so far and I haven't found any issue: text is formatted properly and all the links I've tried were working properly. The only (really minor) issue is the image used as the cover for the ebook: it's actually a snapshot taken from the first page of the HTML document (index.html), it would be nice if you could use the same image used in the first page of the PDF file instead, but as I already said it's really a minor glitch. Everything else so far for what I've seen is perfect.

Updated:
http://flask.pocoo.org/docs/flask-docs.mobi
This is the output of:
ebook-convert Flask.epub Flask.mobi --cover http://flask.pocoo.org/docs/_images/logo-full.png --authors 'Armin Ronacher'

The cover doesn't look stellar, but at least it's not a screenshot of the debugger.  If this looks good enough to @fmaida, I'll automate the .mobi creation.

It's perfect!

Looks like ebook-convert is converting .epub to HTML, then to .mobi.  We can experiment with different conversions if we need improvements.  For now, it's automated via make upload-docs, running the conversion server-side to ease developer requirements.
Cover could use some love, but it works.
ebook-convert provides many formats, if we need others.

Linked to the front page of the docs:
http://flask.pocoo.org/docs/

Is there a recent epub/mobi version of the documentation?

Duplicate of pocoo/metaflask#7.

Have you guys tried the epub/mobi export at:
http://flask-.readthedocs.org/en/stable/

Yes, they are 404, too (epub and mobi).

Flask Documentation on the readthedocs.org
Flask Documentation as ePub
Flask Documentation as Mobi
Flask Documentation as zipped HTML


shouldn't need extra cons cell post-compilation

The data declaration for HandlerM could be turned into a newtype, but maintain the same semantic behavior I'm pretty sure - purescript is strict by default, and the only way to emulate laziness is with a Unit -> ... function.
So I think this change should result in better compiled output code... please let me know if I'm mistaken!

Thanks for the contribution! I don't know how newtype is implemented in PS, but since it doesn't break any tests and is more or less the way this monad would be written in Haskell, I think it's good to merge.

It's intended to be a method to creating type-level distinction for runtime code without adding any overhead :) so purescript literally doesn't do anything to make a data format for it - it's really useful for exposing a typed FFI api, where you only expose certain terms:
module Alignment (Alignment, left, center, right, justified) where
newtype Alignment = Alignment String

left = Alignment "left"
right = Alignment "right"
center = Alignment "center"
justified = Alignment "justified"

Then in some other module:
module Text where

import Alignment

foreign import text :: ReactClass {alignment :: Alignment}
```


[BUG] OPT-66B: OOM at reasonable inference sizes

Describe the bug
I am able to use DeepSpeed to perform inference at sequence lengths 128 and 256, up to batch size 16 for both.
Beyond batch size 16 and/or sequence length 256, I hit an error (which may be triggered by OOM, although it is unclear).
Here is the relevant portion of the stacktrace for seq lens 128/256 @ batch size 32:
    outputs = self.model.decoder(  File "/home/ubuntu/stdun/venv/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)  File "/home/ubuntu/stdun/venv/lib/python3.9/site-packages/transformers/models/opt/modeling_opt.py", line 697, in forward
    layer_outputs = decoder_layer(  File "/home/ubuntu/stdun/venv/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)  File "/home/ubuntu/stdun/DeepSpeed/deepspeed/model_implementations/transformers/ds_transformer.py", line 123, in forward
    self.allocate_workspace(self.config.hidden_size,RuntimeError: Workspace is null.

Using a sequence length of 512---even at batch size 1---produces a clear OOM:
  File "/home/ubuntu/stdun/DeepSpeed/deepspeed/ops/transformer/inference/ds_mlp.py", line 45, in __init__
    self.output_w = nn.Parameter(torch.empty(intm_size_per_partition,
RuntimeError: CUDA out of memory. Tried to allocate 648.00 MiB (GPU 0; 39.41 GiB total capacity; 38.29 GiB already allocated; 62.50 MiB free; 38.45 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Furthermore, the initial results from smaller sequence lengths show subpar performance compared with DeepSpeed 0.5.9. The latency is almost 2x what it was before (even with the prior bugs). This is using DeepSpeed built from source, v0.8.1+d59b5729
To Reproduce
Below is an example script infer.py. To reproduce a specific configuration failure, adjust these lines:
    for max_len in (128, 256, 512, 1024, 2048):
        for batch_size in (1, 2, 4, 8, 16, 32):

infer.py
import os
import time

import deepspeed
import torch
from transformers import pipeline, AutoModelForCausalLM, AutoTokenizer

def infer():
    n_infers = 10
    local_rank = int(os.getenv("LOCAL_RANK", "0"))
    world_size = int(os.getenv("WORLD_SIZE", "1"))
    model_org = "facebook"
    model_name = "opt-66b"
    hf_name = f"{model_org}/{model_name}"

    print("Loading model...")
    st = time.time()
    model = AutoModelForCausalLM.from_pretrained(hf_name, torch_dtype=torch.float16, low_cpu_mem_usage=True)
    tokenizer = AutoTokenizer.from_pretrained(hf_name, use_fast=False)
    print(f"Finished in {round((time.time() - st) / 60, 2)} mins")

    print("Splitting model...")
    st = time.time()
    model = deepspeed.init_inference(model,
        mp_size=world_size,
        dtype=model.dtype,
        replace_with_kernel_inject=True,
        replace_method="auto",
    )
    print(os.environ)
    generator = pipeline(task="text-generation", model=model, tokenizer=tokenizer, device=local_rank)
    print(f"Finished in {round((time.time() - st) / 60, 2)} mins")

    for max_len in (128, 256, 512, 1024, 2048):
        for batch_size in (1, 2, 4, 8, 16, 32):
            for _ in range(n_infers):
                generator(
                    "our story begins",
                    do_sample=True,
                    max_length=max_len,
                    num_return_sequences=batch_size,
                )

if __name__ == "__main__":
    with torch.inference_mode():
        infer()

To use:
deepspeed --num_gpus 8 infer.py

Expected behavior
Expected this to mostly work, failing for the largest batch sizes / sequence lengths.
ds_report output
--------------------------------------------------
DeepSpeed C++/CUDA extension op report
--------------------------------------------------
NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
 [WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
 [WARNING]  async_io: please install the libaio-dev package with apt
 [WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
async_io ............... [NO] ....... [NO]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
fused_adam ............. [NO] ....... [OKAY]
fused_lamb ............. [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
 [WARNING]  please install triton==1.0.0 if you want to use sparse attention
sparse_attn ............ [NO] ....... [NO]
/home/ubuntu/stdun/venv/lib/python3.9/site-packages/setuptools/distutils_patch.py:25: UserWarning: Distutils was imported before Setuptools. This usage is discouraged and may exhibit undesirable behaviors or errors. Please use Setuptools' objects directly or at least import Setuptools first.
  warnings.warn(
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
utils .................. [NO] ....... [OKAY]
--------------------------------------------------
DeepSpeed general environment info:
torch install path ............... ['/home/ubuntu/stdun/venv/lib/python3.9/site-packages/torch']
torch version .................... 1.12.1+cu116
deepspeed install path ........... ['/home/ubuntu/stdun/DeepSpeed/deepspeed']
deepspeed info ................... 0.8.1+d59b5729, d59b5729, master
torch cuda version ............... 11.6
torch hip version ................ None
nvcc version ..................... 11.2
deepspeed wheel compiled w. ...... torch 1.12, cuda 11.6

Screenshots
N/A
System info (please complete the following information):

OS: Ubuntu 20.04.4 LTS
GPU count and types: x8 A100s
DeepSpeed: 0.8.1+d59b5729
Transformers: 4.26.0
Accelerate: 0.15.0
Python version: 3.9.4

Docker context
N/A
Additional context
N/A

Hi @aws-stdun @mrwyattii , I've also met this error self.allocate_workspace(self.config.hidden_size,RuntimeError: Workspace is null.. What does this error mean? I checked both RAM and GPU memory and they're not full, so I guess it's not a memory issue. What else can I do to debug?

@aws-stdun Thanks for reporting this issue. I've slightly modified the code you provided to print out the memory usage at each sequence length and batch size. Could you please run again and share the output to help me debug the error you are seeing? Thanks!
import os
import time

import deepspeed
import torch
from transformers import pipeline, AutoModelForCausalLM, AutoTokenizer
from deepspeed.runtime.utils import see_memory_usage

def infer():
    n_infers = 1
    local_rank = int(os.getenv("LOCAL_RANK", "0"))
    world_size = int(os.getenv("WORLD_SIZE", "1"))
    model_org = "facebook"
    model_name = "opt-66b"
    hf_name = f"{model_org}/{model_name}"

    print("Loading model...")
    st = time.time()
    model = AutoModelForCausalLM.from_pretrained(hf_name, torch_dtype=torch.float16, low_cpu_mem_usage=True)
    tokenizer = AutoTokenizer.from_pretrained(hf_name, use_fast=False)
    print(f"Finished in {round((time.time() - st) / 60, 2)} mins")

    print("Splitting model...")
    st = time.time()
    model = deepspeed.init_inference(model,
        mp_size=world_size,
        dtype=model.dtype,
        replace_with_kernel_inject=True,
        replace_method="auto",
        max_out_tokens=2048,
    )
    print(os.environ)
    generator = pipeline(task="text-generation", model=model, tokenizer=tokenizer, device=local_rank)
    print(f"Finished in {round((time.time() - st) / 60, 2)} mins")

    for max_len in (128, 256, 512, 1024, 2048):
        for batch_size in (1, 2, 4, 8, 16, 32):
            for _ in range(n_infers):
                generator(
                    "our story begins",
                    do_sample=True,
                    max_length=max_len,
                    num_return_sequences=batch_size,
                )
            see_memory_usage(f"max_len:{max_len}, batch_size:{batch_size}", force=True)

if __name__ == "__main__":
    with torch.inference_mode():
        infer()

Hi @aws-stdun @mrwyattii , I've also met this error self.allocate_workspace(self.config.hidden_size,RuntimeError: Workspace is null.. What does this error mean? I checked both RAM and GPU memory and they're not full, so I guess it's not a memory issue. What else can I do to debug?

@Wenhan-Tan what model are you running and on what hardware?

Hi @aws-stdun @mrwyattii , I've also met this error self.allocate_workspace(self.config.hidden_size,RuntimeError: Workspace is null.. What does this error mean? I checked both RAM and GPU memory and they're not full, so I guess it's not a memory issue. What else can I do to debug?

@Wenhan-Tan what model are you running and on what hardware?

Hi @mrwyattii , I was running GPT3 on A100 40GB. BTW, the error is gone when I reduce batch size, so I believe it was limited by the GPU memory although it didn't show any CUDA OOM messages.

@mrwyattii I ran the code you provided, but I don't see any output when using deepspeed runner. Is there more to it? Btw, this issue still exists in v0.8.3 (can't even run batch size=1 with OPT-66)

@Wenhan-Tan @aws-stdun I know it has been many months, but if you are still seeing errors with the OPT model:
I just tested the code that @aws-stdun provided with the latest DeepSpeed and found that the problem is caused by num_return_sequences=batch_size. Previously, I had assumed that transformers would handle this similar to how it handles the case where we pass multiple inputs into a pipeline object, where under the hood, it is essentially running in a for loop over all inputs. But that is not the case and it would appear that the current implementation of DeepSpeed-Inference is not compatible with this option (similar to how we are not compatible with num_beams>1).

@mrwyattii Hi, I haven't tested DeepSpeed since last time I replied to this bug


[stdlib][test] Fix message test expectation.

In #32137 direct environment variable parsing was introduced, the
availability of which is branched by a preprocessor symbol (ENVIRON)
if the environment is available directly on the platform, or if getenv
must be used.
The message expectation in the unit test in the however diverged on the
non-ENVIRON branch, causing a unit test failure. This PR fixes the
expectation, but also, while we're here, OpenBSD supports the ENVIRON
branch anyway.

cc @mikeash

Looks good, thank you! I must have failed to run the test on the getenv code after changing that message.

@swift-ci please test


XCBuild output parsing

Added an initial simple implementation for parsing XCBuild output and used it to prettify the swift build --enable-xcbuild output.
This builds on top of https://github.com/apple/swift-package-manager/pull/2585, so check the second commit to see the real changes.

@aciidb0mb3r I've moved the JSON parser to TSCUtility and rebased off master.

@swift-ci smoke test


opts

Would be nice to have one object with all options in it :)

Like the cropject?

yes, like that but to have all options. Html tag would be cleaner.


Memory usage increases linearly over time

First: Thanks so much for this tool, our initial testing is showing it solving our problem quite well.
Context: We have put mmproxy in a container within a kubernetes pod as a sidecar and it works as expected (handling PROXY protocol traffic and resulting in properly spoofed traffic in the application pod). Our k8s pod is then behind an AWS ELB running the proxy protocol.
What we've seen: mmproxy's memory usage appears to be growing at a linear rate over time growing from minimal to ~3Gb over 5 days. I'm not sure if this is an indication of a problem with mmproxy or something I have configured incorrectly. I also don't know where to start in getting you useful information to help diagnose, but I'm happy to dig in.

Hurray, thanks for feedback. I guess we need to look for memory leaks.

Confirmed! Thanks!


Remove cli-platform example

Instead we're now using basic-cli everywhere inside examples/cli
Blocked on https://github.com/roc-lang/basic-cli/issues/11 because otherwise CI (which runs NixOS) fails

@rtfeldman
I looks like the cli-platform was actually removed in another PR.
So the changes here are now minor updates to the examples... not sure it's worth merging now or just closing.

Closing as this is now limited to minor changes to the examples as cli-platform was removed in another PR.