Crashes if I add my view to UIWindow before recording snapshot

Describe the bug
If I try to run a snapshot test for a view controller but by adding it into a window myself, it causes a crash
To Reproduce
// And/or enter code that reproduces the behavior here.
import SnapshotTesting
import XCTest

class MyViewControllerTests: XCTestCase {
    func testMyViewController() {
        let vc = UIViewController()
        let window = UIWindow()
        window.rootViewController = vc
        window.isHidden = false

        assertSnapshot(matching: window, as: .image)
    }
}

Expected behavior
The window should be recorded as any other view does
Environment

swift-snapshot-testing version 1.9.0
Xcode 13.2.1
Swift 5.5
OS: iOS 15

Additional context
This is the console messages
Newly-taken snapshot does not match reference.
Test Case '-[snapshottoeventuallyTests.MyViewControllerTests testMyViewController]' failed (1.089 seconds).
Test Suite 'MyViewControllerTests' failed at 2022-03-30 15:23:21.256.
	 Executed 1 test, with 1 failure (0 unexpected) in 1.089 (1.090) seconds
Test Suite 'snapshottoeventuallyTests.xctest' failed at 2022-03-30 15:23:21.257.
	 Executed 2 tests, with 1 failure (0 unexpected) in 1.107 (1.114) seconds
Test Suite 'All tests' failed at 2022-03-30 15:23:21.258.
	 Executed 2 tests, with 1 failure (0 unexpected) in 1.107 (1.119) seconds
2022-03-30 15:23:21.262036-0400 snapshottoeventually[86546:2233788] *** -[UIWindowLayer isHidden]: message sent to deallocated instance 0x6040004b1210

in my case I have been able to solve it:
func testMyViewController() {
        let vc = UIViewController()
        let window = UIWindow()
        window.rootViewController = vc
        window.isHidden = false

        let container = UIViewController()
        container.view =  window

        assertSnapshot(matching: container, as: .image)
}

This is no longer an issue. Verified with 1.17.5

Hi @tahirmt, windows are treated specially in UIKit and I'm not sure we can come up with a full-proof way to screenshot them. At the very least you should probably invoke makeKeyAndVisible on the window, but not sure if there is more you need to do.
Since I'm not sure this is really an issue with the library I am going to convert it to a discussion. Please feel free to continue the conversation over there!


Feature/config generator

-add config generator

@sliu008 can you add a README.md how to use it?  And add this to the CHANGELOG.md too?
Thanks!


Optimize settings with 24GB VRAM and Doc Q/A

Hi, I wonder if someone could suggest settings for use with 24GB VRAM. (32GB Sys Ram, Windows System in my case).
Currently I start with:
python generate.py --base_model=TheBloke/Llama-2-13B-chat-GPTQ --load_gptq=model --use_safetensors=True --prompt_type=llama2 --langchain_mode=MyData --score_model=None --share=False --gradio_offline_level=1
This is using ~10-12 GB VRAM.
So I wonder what settings and what model (I know this changes with the release of new ones), would you suggest, to get the best possible experience by working on multiple docs, where I want to have the most possible precise and also long answers (if asked for).
I know there are other settings, that I dont specify in the line above, maybe there are also some that can be changed from default to x, to enhance the quality of work.
I also have to mention that in most cases I use german docs, so the understanding of foreign languages by the model is important.

second this, have a 4090, looking to use as much GPU as possible - have you tried with different float precision?

no I have not loaded anything different than the 4bit 128g base model, you mean like 8bit ?
I ve read that 30b models also run on 24GB. I'm not sure if doc Q/A needs extra memory, so my next test will be a 30b model; TheBloke/Wizard-Vicuna-30B-Uncensored-GPTQ


Some meaningless changes

create new tank 1. fire 3 times can get a chance to fire twice in a row; 2. if the tank continuously hit the enermy third times, it can launtch  an extra bullet


Hide "Bitcoin Contracts" balance on testnet when zero

Currently the balance shows up always, even when zero. It should show up only when non-zero

@markmhendrickson do you mind checking if this is resolved now?
I think it may have been fixed by this commit

@314159265359879 can you recreate this issue and if not, close?

This was indeed resolved with the linked commit


CI/DI: Add code to automatiically update Docker Images for Manager and Processor

Based on a similar approch from the controller:

add to GitHub actions like here: https://github.com/observ-vol-mgt/observ-vol-mgt/blob/main/.github/workflows/push.yml#L31
add relevant step in manager and processors make files like here: https://github.com/observ-vol-mgt/observ-vol-mgt/blob/main/controller/Makefile#L68
and specific
https://github.com/observ-vol-mgt/observ-vol-mgt/blob/main/controller/Makefile#L20
https://github.com/observ-vol-mgt/observ-vol-mgt/blob/main/controller/Makefile#L25

@eranra has the controller Makefile been updated since you added these instructions in the issue. I was looking at this and noticed you point to Line no 68 in that file, whereas the version I see has only 42 line.
I think I follow the approach you have outlined though, was just curious what you were pointing to.

@seepgoel yes :-( --- I moved some of the functionality to a .mk folder to make things more organized. Maybe you can look for the Makefile from last week when I made that comment ( the relevant Makefile targets moved now to the ci.mk and development.mk files )


[Question] How to change scaffolding classes and register in VS

Hi. I beginner and can't understand how change existing scaffolding classes (fork or inherit) and how register them in VS 2017.
For example, need to create additional folders and classes in area scaffolding.

Closing as a dupe of this https://github.com/aspnet/Scaffolding/issues/938


Update to serde 0.8.0

Supersedes #83.

Does this serde update include any changes that would change the ordering of bytes produced by bincode?

No. There are 3 user-facing changes but all are in deserialization:

deserialize_str used to not be implemented, meaning it forwarded to deserialize and failed. Now it is implemented to deserialize a &str. This affects only Deserialize types that used to fail and now succeed.
deserialize_struct_field fails with a better message.
deserialize_ignored_any fails with a better message.

As an incentive, I suspect you will see a nice performance improvement in serialization for both structs and vectors.
I can put together a test suite this weekend before you release to prove the bytes are the same.

Sounds good.  I was only asking because I wanted to know if I'd need to bump the version number.  However, I'm pretty sure I need to bump the version anyway, simply because it's re-exporting serde which bumped to 0.8.0.
The test suite already makes sure that the rustc-serialize encoding is the same (byte for byte) as the serde encoding, so I'm not super worried.
Thanks for the contribution!

Published to crates.io as 0.6.0


fixed to check if build is a snapshot

Signed-off-by: Amit Galitzky<EMAIL_ADDRESS>Description
Fixing AD CI since build doesn't always have -SNAPSHOT.
Issues Resolved
#530
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #531 (2db4c9e) into 1.3 (cc83942) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.3     #531      +/-   ##
============================================
- Coverage     77.91%   77.75%   -0.16%
+ Complexity     4126     4117       -9
============================================
  Files           296      296
  Lines         17682    17682
  Branches       1879     1879
============================================
- Hits          13777    13749      -28
- Misses         3005     3032      +27
- Partials        900      901       +1

Flag
Coverage Δ

plugin
77.75% <ø> (-0.16%)
:arrow_down:

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

Impacted Files
Coverage Δ

...java/org/opensearch/ad/task/ADBatchTaskRunner.java
82.21% <0.00%> (-3.50%)
:arrow_down:

...ava/org/opensearch/ad/task/ADHCBatchTaskCache.java
88.88% <0.00%> (-2.47%)
:arrow_down:

...ain/java/org/opensearch/ad/model/ModelProfile.java
70.90% <0.00%> (-1.82%)
:arrow_down:

...ain/java/org/opensearch/ad/task/ADTaskManager.java
76.67% <0.00%> (-0.38%)
:arrow_down:

...port/SearchAnomalyDetectorInfoTransportAction.java
66.66% <0.00%> (+2.22%)
:arrow_up:

...ansport/handler/AnomalyResultBulkIndexHandler.java
70.96% <0.00%> (+3.22%)
:arrow_up:


Revert "make sure eldest key info is populated the same way as sibkeyinfo"

This reverts commit ac212ea109eed5e660b61a003ce6a0f30b70cd21.
That commit broke keybase id max.
r? @mmaxim @mlsteele

The offending PR: https://github.com/keybase/client/pull/8073

Uggggg what happened and why didn't tests catch it.
On Thu, Aug 10, 2017 at 11:21 AM oconnor663<EMAIL_ADDRESS>wrote:

The offending PR: #8073 https://github.com/keybase/client/pull/8073
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/pull/8081#issuecomment-321584040, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA05_3PYhU3Yi3Wk-Vk5myBLhJCNQm_vks5sWx_1gaJpZM4OzlPM
.

Whatever it is, it repros on keybase id max (with an empty cache) but not keybase id chris.


Support append stream for Spice AI data connector

🗣 Description
Support the append refresh mode without time_column using the append data stream.

The append stream is a change stream with limited functionality
When using append refresh mode, spice oss subscribe to the change data stream using the DoExchange method in the Spice AI data connector. While wrapping the data received from the flight server into the Change Envelope format.
When no append stream is presented under this refresh mode, data connector will error.\

🔨 Related Issues

https://github.com/spiceai/spiceai/issues/1868

🤔 Concerns

How to deal with dataset without primary keys

Closing until ready for review.

Closing as stale.


feat: Port test-node to most recent core revision

What :computer:

Ports test node to the most recent core revision.

Why :hand:

Several breaking changes were made in core APIs, so this is to make further maintenance of the node easier.
Note this sweet minus 1868 lines in the Cargo.lock. Nice, innit?

Evidence :camera:
You got to trust me this time.

Note this sweet minus 1868 lines in the Cargo.lock. Nice, innit?
🙌 Praise be

You got to trust me this time.
We have unit and e2e tests for changes like this 😄


EP Buffers

First of all - let me apologize up front if this issue is a bit of a mess.  I am still trying to nail down exactly what is happening and this issue should probably be split into 2-3 different issues.
After the announcement of this project on Reddit (https://www.reddit.com/r/embedded/comments/pqql3v/stm32_usb_composite_class_wrapper/) I immediately tried and ran into an issue of no enumeration (trying a composite with cdc, hid and msc).
These experiments were all done on a regular "black pill" board with stm32f411ce mcu.  According to the reference manual that MCU has got "1.25 kB" of dedicated USB FIFO buffer memory (presumably that is 1280 byte or 0x0500), so it is quite obvious why the change suggested is necessary (at least on this particular MCU).
However, even with this change there are still some issues.  trying the composite cdc, hid, msc and changing the fifo values to:
    HAL_PCDEx_SetRxFiFo(hpcd_USB_OTG_PTR, 0x80); // ALL OUT EP Buffer
    HAL_PCDEx_SetTxFiFo(hpcd_USB_OTG_PTR, 0, 0x40); // EP0 IN
    for (uint8_t i = 0; i < USBD_CDC_ACM_COUNT; i++)
    {
      HAL_PCDEx_SetTxFiFo(hpcd_USB_OTG_PTR, (CDC_IN_EP[i] & 0x7F), 0x80);
      HAL_PCDEx_SetTxFiFo(hpcd_USB_OTG_PTR, (CDC_CMD_EP[i] & 0x7F), 0x40);
    }
    HAL_PCDEx_SetTxFiFo(hpcd_USB_OTG_PTR, (CUSTOM_HID_IN_EP & 0x7F), 0x80);
    HAL_PCDEx_SetTxFiFo(hpcd_USB_OTG_PTR, (MSC_IN_EP & 0x7F), 0x80);

Enumerates like this:
[334523.568812] usb 2-4.5.4: new full-speed USB device number 122 using xhci_hcd
[334523.670669] usb 2-4.5.4: New USB device found, idVendor=0483, idProduct=52a4, bcdDevice= 2.00
[334523.670671] usb 2-4.5.4: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[334523.670672] usb 2-4.5.4: Product: STM32 COMPOSITE DEVICE
[334523.670673] usb 2-4.5.4: Manufacturer: STMicroelectronics
[334523.670673] usb 2-4.5.4: SerialNumber: 318A34583030
[334523.688054] cdc_acm 2-4.5.4:1.0: ttyACM1: USB ACM device
[334523.689281] hid-generic 0003:0483:52A4.001B: hiddev3,hidraw6: USB HID v1.11 Device [STMicroelectronics STM32 COMPOSITE DEVICE] on usb-0000:00:14.0-4.5.4/input2
[334523.689509] usb-storage 2-4.5.4:1.3: USB Mass Storage device detected
[334523.689641] scsi host7: usb-storage 2-4.5.4:1.3
[334524.777016] usb 2-4.5.4: reset full-speed USB device number 122 using xhci_hcd
[334524.878479] cdc_acm 2-4.5.4:1.0: ttyACM1: USB ACM device
[334524.973015] usb 2-4.5.4: reset full-speed USB device number 122 using xhci_hcd
[334525.074569] cdc_acm 2-4.5.4:1.0: ttyACM1: USB ACM device
[334525.173023] usb 2-4.5.4: reset full-speed USB device number 122 using xhci_hcd
[334525.274489] cdc_acm 2-4.5.4:1.0: ttyACM1: USB ACM device
[334525.373025] usb 2-4.5.4: reset full-speed USB device number 122 using xhci_hcd
[334525.474606] cdc_acm 2-4.5.4:1.0: ttyACM1: USB ACM device

This experiment is here: https://github.com/lbthomsen/blackpill/tree/master/comptest
I have not yet tested the HID, but oddities are as follows:
The CDC resets 4 times (as seen above).  This does not happen when using ST's own HAL CDC.  It does however work (at lest for transmitting).  Using:
int _write(int file, char *ptr, int len) {
    CDC_Transmit(0, (uint8_t *)ptr, len);
    return len;
}

works - serial data is received on /dev/ttyACMx.
The mass storage device however doesn't register as a storage device.  Using ST's MSC driver (example here: https://github.com/lbthomsen/blackpill/tree/master/msc), enumerates like this:
[335036.092341] usb 2-4.5.4: new full-speed USB device number 123 using xhci_hcd
[335036.202003] usb 2-4.5.4: New USB device found, idVendor=0483, idProduct=572a, bcdDevice= 2.00
[335036.202005] usb 2-4.5.4: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[335036.202006] usb 2-4.5.4: Product: STM32 Mass Storage
[335036.202007] usb 2-4.5.4: Manufacturer: STMicroelectronics
[335036.202008] usb 2-4.5.4: SerialNumber: 318A34583030
[335036.208286] usb-storage 2-4.5.4:1.0: USB Mass Storage device detected
[335036.208686] scsi host7: usb-storage 2-4.5.4:1.0
[335037.240855] scsi 7:0:0:0: Direct-Access     STM      Product          0.01 PQ: 0 ANSI: 2
[335037.241056] sd 7:0:0:0: Attached scsi generic sg4 type 0
[335037.242979] sd 7:0:0:0: [sde] 65536 512-byte logical blocks: (33.6 MB/32.0 MiB)
[335037.243191] sd 7:0:0:0: [sde] Write Protect is off
[335037.243192] sd 7:0:0:0: [sde] Mode Sense: 22 00 00 00
[335037.243403] sd 7:0:0:0: [sde] Write cache: disabled, read cache: enabled, doesn't support DPO or FUA
[335037.274280] sd 7:0:0:0: [sde] Attached SCSI removable disk

This lead me to suspect that the communication with the MSC device is not really working in the composite.

Hi,
My observation
MCU 411CE
Total Endpoints 3 (Excluding EP0)
CDC requires 2 IN EP & 1 OUT
MSC requires 1 IN & 1 OUT
at this point you have consumed all IN endpoint
Total PMA Buffer is 1.25KB (1280 Bytes)
HAL_PCDEx_SetTxFiFo takes size in WORDS, minimum is 16 words.
so total buffer consumption is
EP OUT Common (1284) is 512 bytes
EP0 IN (644) 256
CDC IN1 (1284) 512
CDC IN2 (644) 256
MSC IN (128*4)512
as above calculation suggest you have exhausted PMA buffer
my suggestion will be to partition PMA as
EP OUT Common (512 Bytes) 128 as param to HAL_PCDEx_SetTxFiFo()
EP0 IN (64 Bytes) 16
CDC IN1 (128 Bytes) 32
CDC IN2 (64 Bytes) 16
MSC IN (128 Bytes) 32

So - in the setup it is possible to add 2 CDC interfaces but how would that ever be possible - 2 IN EP's per device is 4 already + EP0 would be five.
I will try with HID + MSC only!

And yes - CDC + MSC only with the buffers you suggested works:
[356454.344290] usb 2-4.5.4: new full-speed USB device number 5 using xhci_hcd
[356454.446124] usb 2-4.5.4: New USB device found, idVendor=0483, idProduct=52a4, bcdDevice= 2.00
[356454.446126] usb 2-4.5.4: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[356454.446127] usb 2-4.5.4: Product: STM32 COMPOSITE DEVICE
[356454.446128] usb 2-4.5.4: Manufacturer: STMicroelectronics
[356454.446129] usb 2-4.5.4: SerialNumber: 318A34583030
[356454.458855] cdc_acm 2-4.5.4:1.0: ttyACM1: USB ACM device
[356454.459382] usb-storage 2-4.5.4:1.2: USB Mass Storage device detected
[356454.459566] scsi host7: usb-storage 2-4.5.4:1.2
[356455.460723] scsi 7:0:0:0: Direct-Access     STM      Product          0.01 PQ: 0 ANSI: 2
[356455.460947] sd 7:0:0:0: Attached scsi generic sg4 type 0
[356455.461339] sd 7:0:0:0: [sde] 200 512-byte logical blocks: (102 kB/100 KiB)
[356455.461589] sd 7:0:0:0: [sde] Write Protect is off
[356455.461591] sd 7:0:0:0: [sde] Mode Sense: 22 00 00 00
[356455.461819] sd 7:0:0:0: [sde] Write cache: disabled, read cache: enabled, doesn't support DPO or FUA
[356455.495957] sd 7:0:0:0: [sde] Attached SCSI removable disk


Implement resource provider API for dynamic rp

Description
This change implements the standard resource provider API for dynamic resources. This enables the dynamic-rp to serve the standard CRUDL API functionality for user-defined types.
This change does NOT add recipe support, that will come in the next change. The pattern that's supported by this change is an "inert" resource which has no backend functionality. Recipe support will be a superset of this functionality.
Type of change
Part of: #6688
Contributor checklist
Please verify that the PR meets the following requirements, where applicable:

[ ] An overview of proposed schema changes is included in a linked GitHub issue.
[ ] A design document PR is created in the design-notes repository, if new APIs are being introduced.
[ ] If applicable, design document has been reviewed and approved by Radius maintainers/approvers.
[ ] A PR for the samples repository is created, if existing samples are affected by the changes in this PR.
[ ] A PR for the documentation repository is created, if the changes in this PR affect the documentation or any user facing updates are made.
[ ] A PR for the recipes repository is created, if existing recipes are affected by the changes in this PR.

Merging this to unblock the next steps. Will address any feedback in a follow-up.


.net color

Brief description of your issue

process is .net but color doesn't changed
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
3.0.10791.6264
win7

OK

Fixed in latest nightly build 👍


fix(40806): Add missing options and possible values of DateTimeFormat

Fixes #40806
Options of Intl DateTimeFormat and possible values taken from MDN page Intl/DateTimeFormat.

interface DateTimeFormatOptions {
    dateStyle?: "full" | "long" | "medium" | "short";
    timeStyle?: "full" | "long" | "medium" | "short";
    calendar?: "buddhist" | "chinese" | " coptic" | "ethiopia" | "ethiopic" | "gregory" | " hebrew" | "indian" | "islamic" | "iso8601" | " japanese" | "persian" | "roc";
    dayPeriod?: "narrow" | "short" | " long";
    numberingSystem?: "arab" | "arabext" | " bali" | "beng" | "deva" | "fullwide" | " gujr" | "guru" | "hanidec" | "khmr" | " knda" | "laoo" | "latn" | "limb" | "mlym" | " mong" | "mymr" | "orya" | "tamldec" | " telu" | "thai" | "tibt";
    localeMatcher?: "best fit" | "lookup";
    timeZone?: string;
    hour12?: boolean;
    hourCycle?: "h11" | "h12" | "h23" | "h24";
    formatMatcher?: "best fit" | "basic";
    weekday?: "long" | "short" | "narrow";
    era?: "long" | "short" | "narrow";
    year?: "numeric" | "2-digit";
    month?: "numeric" | "2-digit" |"long" | "short" | "narrow";
    day?: "numeric" | "2-digit";
    hour?: "numeric" | "2-digit";
    minute?: "numeric" | "2-digit";
    second?: "numeric" | "2-digit";
    fractionalSecondDigits?: 0 | 1 | 2 | 3;
    timeZoneName?: "long" | "short";
}

Sorry to bother you @orta, I just created a similar PR as #40906 but with all the options I found on MDN page.
Could you tell me what would be wrong with this fix please? (First PR in TypeScript)

Alright, this looks good to me - thanks 👍🏻

I am seeing some problems compiling TypeScript code with this change. I believe the issue is that the types in lib.es5.d.ts and lib.es2020.intl.d.ts conflict:
// ES5:
interface DateTimeFormatOptions {
    localeMatcher?: string;
    weekday?: string;
    era?: string;
    year?: string;
    month?: string;
    day?: string;
    hour?: string;
    minute?: string;
    second?: string;
    timeZoneName?: string;
    formatMatcher?: string;
    hour12?: boolean;
    timeZone?: string;
}

// ES2020:
interface DateTimeFormatOptions {
    dateStyle?: "full" | "long" | "medium" | "short";
    timeStyle?: "full" | "long" | "medium" | "short";
    calendar?: "buddhist" | "chinese" | " coptic" | "ethiopia" | "ethiopic" | "gregory" | " hebrew" | "indian" | "islamic" | "iso8601" | " japanese" | "persian" | "roc";
    dayPeriod?: "narrow" | "short" | " long";
    numberingSystem?: "arab" | "arabext" | " bali" | "beng" | "deva" | "fullwide" | " gujr" | "guru" | "hanidec" | "khmr" | " knda" | "laoo" | "latn" | "limb" | "mlym" | " mong" | "mymr" | "orya" | "tamldec" | " telu" | "thai" | "tibt";
    localeMatcher?: "best fit" | "lookup";
    timeZone?: string;
    hour12?: boolean;
    hourCycle?: "h11" | "h12" | "h23" | "h24";
    formatMatcher?: "best fit" | "basic";
    weekday?: "long" | "short" | "narrow";
    era?: "long" | "short" | "narrow";
    year?: "numeric" | "2-digit";
    month?: "numeric" | "2-digit" | "long" | "short" | "narrow";
    day?: "numeric" | "2-digit";
    hour?: "numeric" | "2-digit";
    minute?: "numeric" | "2-digit";
    second?: "numeric" | "2-digit";
    fractionalSecondDigits?: 0 | 1 | 2 | 3;
    timeZoneName?: "long" | "short";
}

Fields like localMatcher have had their type narrowed, giving errors like the following:
lib.es2020.intl.d.ts(307,9): error TS2717: Subsequent property declarations must have the same type.  Property 'localeMatcher' must be of type 'string | undefined', but here has type '"lookup" | "best fit" | undefined'.

Also, several of the property values have leading spaces, e.g. " coptic". This seems like a typo.


Tag version 1.0.0 of this action

This action now works, and users can already use it by doing:
- uses: julia-actions/install-juliaup@latest
  with:
    julia-version: '1'

However, in general, we prefer to not recommend that people use @latest. So we should make a v1 tag of this action, so that users can instead do:
- uses: julia-actions/install-juliaup@v1
  with:
    julia-version: '1'

I think we're ready to make a v1 tag off of the current state of the main branch. But we should probably have people try this out for a bit first.

I think that we should document somewhere that we are allowed to change the default value of the juliaup-version in a future non-breaking (minor) release.
Then we can implement #8 in the future without having to bump to v2.

This is done now.
Releases:

https://github.com/julia-actions/install-juliaup/releases/tag/v1.0.0

Tags:

https://github.com/julia-actions/install-juliaup/releases/tag/v1.0.0
https://github.com/julia-actions/install-juliaup/releases/tag/v1


No alternate file after running a request

After running the requests in an .http file with either require('kulala').run() or require('kulala').run_all(), I get the following error when trying to switch to my previous file: E23: No alternate file

Can you provide more information about how to reproduce this?
I have a mapping for quickly switching between the current and last opened buffer, which is also broken when using Kulala, but that's my bad, because the keybind is just wrong.
I'm not bothered by it too much, so I haven't bothered fixing it in my setup. Maybe that's also happening for you.
Kulala creates new kulala://ui buffers and deletes old kulala://ui buffers every time it updates the ui.
So maybe you're trying to access an non existent buffer with that "switch to my previous file".

Sure! So in my usual workflow:

I'll be in some code file to make a change
Switch to a .http file
Run the request with require('kulala').run() (on a keymap)
After receiving the response, I'll try using CTRL-^ to switch back to the code file, and that's when I get the error message

In my case switching to the previous buffer isn't remapped, I believe CTRL-^ is the default alternate-file key mapping.
The code file's buffer is still there if I look at all the open buffers, it's just not the alternate-file anymore, and no other buffer is marked as the alternate-file either (usually indicated with a # in the buffer list)

@gorillamoe and it's Ctrl-6 (Ctrl-^ does not work on out German keyboards, at least on mine)

Thanks @gorillamoe and @Grueslayer, it's working perfectly!

@gorillamoe and it's Ctrl-6 (Ctrl-^ does not work on our German keyboards, at least on mine)

A bit off topic, but I'm [using)(https://gorilla.moe/uses) a Vortex 10 Limited Edition with US Layout.
I switched to the US Layout like 12 years ago 🤔.


Executing code from VSCode crashes motion builder on windows 11

Hello, thanks for this great extension!
Unfortunately I have a new machine with windows 11. Everytime i try to execute code from VSCode it crashes my motion builder app (2023 & 2024)

Hahahaha, my fault - i forgot that the snippet i ran had "exit()" instruction after not detecting certain thing ¯(°_o)/¯
Sorry, false alarm (‾◡◝)


database: revisit project database indexes

The initial implementation coming from https://github.com/stacklok/mediator/pull/539 chose UUIDs as indexes. This might or might not have been the right choice. Let's get back to this and research what is the appropriate choice to use so we don't crash and burn in the future. @evankanderson left some insights on the PR.

this has been addressed.


Improve OPeNDAP error message

If we try to access a non-existent URL, a seriously unhelpful error message results:
>ncdump -h https://thredds-test.unidata.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190211/OR_ABI-L1b-RadF-M3C13_G17_s20190421530382_e20190421541160_c20190421541201.nc
ncdump: https://thredds-test.unidata.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190211/OR_ABI-L1b-RadF-M3C13_G17_s20190421530382_e20190421541160_c20190421541201.nc: https://thredds-test.unidata.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190211/OR_ABI-L1b-RadF-M3C13_G17_s20190421530382_e20190421541160_c20190421541201.nc: NetCDF: Not a valid data type or _FillValue type mismatch

>curl https://thredds-test.unidata.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190211/OR_ABI-L1b-RadF-M3C13_G17_s20190421530382_e20190421541160_c20190421541201.nc
Error {
    code = 400;
    message = "Unrecognized request";
};

It's possible this is specific to hitting the 404 page from TDS. Regardless, having the library yield the OPeNDAP equivalent of "File Not Found" or "Invalid File" would be so much more helpful.

Oh, I see now it's a 400, not a 404. Either way, I'd love a better error message. (Why this particular URL is even giving an error is a bug in TDS.)

@DennisHeimbigner is there an easy way to check the error code coming back?  I could dig into this if so.  I'm not sure if DAP is parsing error codes returned or if it's just checking for an error.

You want to add .dods after the .nc, as well as some constraint. So something like:
curl https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc.dods?y[0:1:5423]
Dataset {
    Int16 y[y = 5424];
} satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc;

That said, the issue is bigger than just the error message, as ncdump on the url works in 4.6.1, but not 4.6.2.
So:
netcdf library version 4.6.2 of Dec 17 2018 19:20:47
ncdump -h https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc
ncdump: https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc: https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc: NetCDF: Not a valid data type or _FillValue type mismatch

netcdf library version 4.6.1 of Jan 27 2019 11:38:15
ncdump -h https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc | less
sarms@flip:$ ncdump -h https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc
netcdf OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201 {
dimensions:
	band = 1 ;
	num_star_looks = 24 ;
	number_of_image_bounds = 2 ;
	number_of_time_bounds = 2 ;
	x = 5424 ;
	y = 5424 ;
<snip>

So, the _FillType mismatch error is hasn't had coffee yet so recollection is hazy a fix we introduced in 4.6.2 for behavior that was documented but not previously enforced.  Let me go back and dig through the PR's/Support emails to refresh my memory.

Ok, it looks like there was some work done to work around this change when using DAP: https://github.com/Unidata/netcdf-c/pull/1155
It clearly needs to be highlighted in the documentation.

I can confirm that, for me, the following command works
$ ./ncdump -h [fillmismatch]https://thredds.ucar.edu/thredds/dodsC/satellite/goes17/GRB17/ABI/FullDisk/Channel13/20190213/OR_ABI-L1b-RadF-M3C13_G17_s20190441600381_e20190441611159_c20190441611201.nc

So perhaps this is a failure of documented DAP behavior, or at least indicates the need to highlight this new behavior.  Or, based on conversation in my office with @dopplershift just now, perhaps [fillmismatch] needs to be implicitly true when doing DAP operations.

Correct. The netcdf spec requires the type  of a _FillValue attribute to be
the same as the associated variable. We have started to enforce it.
However, if you suffix a url with '#fillmismatch', an attempt will be made
to forcibly convert the _FillValue value to the type of the variable.

Playing with the file locally, I do not get a mismatch error.  Sitting in a meeting at the moment, but surprised at the different behavior, unless I've already been told and just forgot, in which case ah, yes, I remember.

The problem may be that since DAP2 does not support signed byte, the converter
widens to int16 to store the _FillValue.

Ok, am testing fix now in the netcdf-c library.

Fixed as discussedd: see https://github.com/Unidata/netcdf-c/pull/1317

Fixed as discussedd: see https://github.com/Unidata/netcdf-c/pull/1317


refactor!: remove help package

It seems the help package wasn't a great idea. It's hard to maintain and has limitations. Removing is a better choice than not maintaining it.
Breaking Change: help targets deleted.
Depends on #49

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
13

Relevant Lines:
160

💛  - Coveralls


sink(ticdc): add limit to storage ddl sink

What problem does this PR solve?

Issue Number: ref #9602
What is changed and how it works?
Write checkpointTs every two seconds in storage sink
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`Fix the issue of gcs write limit caused by frequent metadata updates in the storage sink`.

/test verify

/retest-required

/retest

/test cdc-integration-mysql-test


Tooltips are breaking the collapse in Vue 3 + Bootstrap 5

Discussed in https://github.com/orgs/twbs/discussions/39100

Originally posted by viveknz76 August 24, 2023
I believe there is an issue with bootstrap 5 and Vue 3.
I tried to get tooltip and collapse working on a page but implementing tooltip breaks the collapse functionality. Example,
https://stackblitz.com/edit/vue-rhm9hw?file=src%2Fcomponents%2FHelloWorld.vue

I might have identified the cause here.
Next time you create a github issue which is also asked on SO, consider placing a link to the other one in each of them.
This will improve search indexing and people having the same issue will be more likely to find the answer faster.
Cheers!

Closing it since import 'bootstrap'; instead of import 'bootstrap/dist/js/bootstrap.bundle.min.js' seems to do the trick in the StackBlitz reproducible link. Feel free to add comments if that's not the case.


[ISSUE #3887 ] fix validate fail after update acl

Make sure set the target branch to develop
What is the purpose of the change
#3887
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #3888 (69194c3) into develop (b606d9c) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #3888      +/-   ##
=============================================
- Coverage      47.53%   47.47%   -0.07%
+ Complexity      4931     4923       -8
=============================================
  Files            633      633
  Lines          42497    42498       +1
  Branches        5579     5579
=============================================
- Hits           20201    20174      -27
- Misses         19798    19819      +21
- Partials        2498     2505       +7

Impacted Files
Coverage Δ

...che/rocketmq/acl/plain/PlainPermissionManager.java
71.68% <100.00%> (+1.11%)
:arrow_up:

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
66.07% <0.00%> (-5.36%)
:arrow_down:

...ava/org/apache/rocketmq/test/util/VerifyUtils.java
46.26% <0.00%> (-2.99%)
:arrow_down:

.../java/org/apache/rocketmq/logging/inner/Level.java
36.00% <0.00%> (-2.00%)
:arrow_down:

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
67.75% <0.00%> (-1.56%)
:arrow_down:

...mq/client/impl/consumer/RebalanceLitePullImpl.java
72.05% <0.00%> (-1.48%)
:arrow_down:

...java/org/apache/rocketmq/logging/inner/Logger.java
50.23% <0.00%> (-1.44%)
:arrow_down:

...etmq/client/latency/LatencyFaultToleranceImpl.java
50.00% <0.00%> (-1.29%)
:arrow_down:

...che/rocketmq/namesrv/kvconfig/KVConfigManager.java
59.18% <0.00%> (-1.03%)
:arrow_down:

.../rocketmq/broker/filter/ConsumerFilterManager.java
73.09% <0.00%> (-0.90%)
:arrow_down:

... and 7 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 b606d9c...69194c3. Read the comment docs.

Coverage increased (+0.02%) to 51.569% when pulling 69194c346c19da5ad260dbe470424fd8a33778cf on yuz10:develop3 into b606d9c638c2b9bf018d9a9968fdc9e5d6acf07f on apache:develop.

Coverage increased (+0.02%) to 51.569% when pulling 69194c346c19da5ad260dbe470424fd8a33778cf on yuz10:develop3 into b606d9c638c2b9bf018d9a9968fdc9e5d6acf07f on apache:develop.


Generate input models with dictionary type properties

Design will be specified in https://github.com/Azure/autorest.csharp/issues/2472
Input model with Collection and Dictionary properties
This issue tracks the work to implement generation of input models with dictionary type properties in the DPG generator.
Input Cadl file
TBD
Expected output
Files:
TBD
Requirements:
TBD
Test Coverage
To the extent we can, we should reuse tests from ModelShapes

CADL Map<K,V> is replaced by Record<V>. The design is in #2474

@annelo-msft I've added the initial support of dictionary support in #2657
You can refer to the following test cases for the expected output. I checked the contents which look fine to me.

general test cases: https://github.com/Azure/autorest.csharp/blob/feature/v3/test/AutoRest.TestServerLowLevel.Tests/LowLevel/Generation/DictionaryWritingTests.cs
optional dictionary property: https://github.com/Azure/autorest.csharp/blob/6761a1b7074b9c74f06790f7989b4c724f0a8234/test/AutoRest.TestServerLowLevel.Tests/LowLevel/Generation/OptionalPropertyWritingTests.cs#L55
read-only dictionary property: https://github.com/Azure/autorest.csharp/blob/6761a1b7074b9c74f06790f7989b4c724f0a8234/test/AutoRest.TestServerLowLevel.Tests/LowLevel/Generation/ReadonlyPropertyWritingTests.cs#L53-L56

@archerzz, can we close this item?

@archerzz, I don't see where the output models are - the files containing types that have dictionary properties.  Can you provide links to the Cadl input and generated output?

@annelo-msft I didn't write the Cadl file for model generation. I always start from the InputModel, assuming the emitter will generate the correct JSON output.

Here are the InputModel definition for output models: https://github.com/Azure/autorest.csharp/blob/8cfc0d7838c9736de204d4d9ab726acd8bdb3f56/test/AutoRest.TestServerLowLevel.Tests/LowLevel/Generation/DictionaryWritingTests.cs?rgh-link-date=2022-08-30T03%3A14%3A08Z#L36-L47
Here are the corresponding generated codes: https://github.com/Azure/autorest.csharp/blob/8cfc0d7838c9736de204d4d9ab726acd8bdb3f56/test/AutoRest.TestServerLowLevel.Tests/LowLevel/Generation/DictionaryWritingTests.cs?rgh-link-date=2022-08-30T03%3A14%3A08Z#L416-L560


Telus Smart Home support

Hello there!
I'm a big fan of your homeassistant module, and I use it since a long time now, but as other Quebec people with ADT, I'm now forced to move to Telus smart home (https://play.google.com/store/apps/details?id=com.alarm.alarmmobile.android.telus&hl=fr_CA&gl=US&pli=1) . Both ADT and Telus are using alarm.com accounts, but my current logins are not working anymore with your module.
Do you plan to support this migration? Could it be only a domain name to change (https://smarthome-security.telus.com/)?
Thanks a lot for your help on that!

I created another user ID and I was able to use this one and not the main account, and it's doing the trick, so I'll close that ticket, sorry for the noise!


Make New Release with Features and Bugfixes

Deploy checklist

[ ] update the history.md with a summary of new features and the bug fix

https://github.com/opendifferentialprivacy/whitenoise-core/blob/develop/History.md

[ ] update the readme, including any components that should be highlighted
[ ] test the whitenoise-samples notebooks (@ctcovington @ecowan)
[ ] update the python repo and create a pypi release (@Shoeboxam @ctcovington @ecowan). (please document)
[ ] release an updated crate (@mikephelan)

whitenoise-core-python

New branch
Test with whitenoise-core -> develop branch
Does python build and tests run?
Do notebooks still work?

Move new branch to white-core-python develop

build pypi packages


Bypass session check

This is a workaround for https://github.com/ruby/debug/issues/900
We believe there is a race condition relating to the in_subsession? check here:
https://github.com/Shopify/debug/blob/d7bf3b2e4502eb0f07a67f00e5fce5a0d661189e/lib/debug/session.rb#L2578-L2587
By making this change we ensure that evaluate always acts as if in_subsession? is true.

@vinistock I think this is a very weird case and we probably need to revisit the implementation later. But for now it's working in our testings and we have a basic test case for it too. So I'll merge this once the build runs green and then test it in Core.


doesn't work in typescript

I have installed lsp-tsserver; and used it in lualine.nvim according to the configuration; but it has no effect on ts files; it is effective in c and c++ files. I don't know how to solve it.
I execute :LspInfo

Are you attaching the lsp servers manually? if so make sure you are attaching to typescript server as well

Are you attaching the lsp servers manually? if so make sure you are attaching to typescript server as well

Thanks for the answer; but I can be sure that the tsserver is linked

Sorry; sure enough I didn't configure it correctly. Thanks for your help.

Are you attaching the lsp servers manually? if so make sure you are attaching to typescript server as well

Sorry; sure enough I didn't configure it correctly. Thanks for your help.


0kb Image Sequence

Hey Vlad!
I'm trying to render jpg sequence and am having issues with it at the moment. Hoping I can get some assistance on it. The .jpg gets rendered, but it's a single 0kb file. I tried to render .mp4 with it and it's fine. Tried using different versions but it comes out the same.
Here's my json.
{   "template": {     "src": "file:///C:\\path\\file.aep",     "composition": "compName",     "frameStart": "0",     "frameEnd": "1",     "continueOnMissing": "Boolean",     "settingsTemplate": "Best Settings",     "outputModule": "Jpg Sequence",     "outputExt": "jpg"   },   "actions": {     "postrender": [       {         "module": "@nexrender/action-copy",         "output": "C:\\file.jpg"       }     ]   } }

And... it's solved! (Magically after posting it here. It must be the magic of github. haha)
Here's the solution. Since I only needed one jpg. I could target the one frame with "00000" suffix, to copy it.
{
"module": "@nexrender/action-copy",
"input": "result_00000.jpg",
"output": "file.jpg"
}


[TEST] Testing tachyon dashboard.

This jira is created to test tachyon pr dashboard.

test comment

test

test


Add scripts for triggering build-service e2e test

JIRA
https://issues.redhat.com/browse/PLNSRVCE-185
What was changed
Added Dockerfile and script to be used by OpenShift CI on a PR basis for running e2e tests
Verification steps
Following steps will be run from inside the container we're going to build the image for

Build the image

docker build -t quay.io/psturc/build-service-openshift-ci-base-image:latest . -f .ci/openshift-ci/Dockerfile

Run the container from the image you just built, mount the current folder with the build-service repo contents and pass the env var QUAY_TOKEN with the base64 encoded dockerconfig file (make sure you're docker logged in to quay.io first)

docker run -e QUAY_TOKEN=$(base64 < ~/.docker/config.json) -v $(pwd):/go/src/github.com/openshift/origin -it quay.io/psturc/build-service-openshift-ci-base-image:latest /bin/bash

Update and export following env vars

# Please update these env var values
export GITHUB_E2E_ORGANIZATION=<your-github-org>
export QUAY_E2E_ORGANIZATION=<your-quay-org>
export GITHUB_TOKEN=<your-github-token-with-permissions-to-write-to-GITHUB_E2E_ORGANIZATION>

Install oc and login to your cluster

curl -Ls https://mirror.openshift.com/pub/openshift-v4/x86_64/clients/ocp/4.10.5/openshift-client-linux.tar.gz | tar -zx && mv oc /usr/local/bin

oc login <your cluster>

Run

# OpenShift CI specific env vars updated for the purpose of this PR
export JOB_SPEC='{"refs":{"org":"redhat-appstudio","repo":"e2e-tests","repo_link":"https://github.com/redhat-appstudio/e2e-tests","base_ref":"main","base_sha":"2dcbde3b80153f7b27a45343a1277ca3cc524f39","base_link":"https://github.com/redhat-appstudio/e2e-tests/commit/ad3d0daf2bfa1f05413cc5fdb209fe880290cf88","pulls":[{"number":57,"author":"psturc","sha":"93d48f1a75216be4004014de478c4162b65c3ed0","title":"chore: use managed-gitops CR to deploy AppStudio Components in e2e","link":"https://github.com/redhat-appstudio/e2e-tests/pull/57","commit_link":"https://github.com/redhat-appstudio/e2e-tests/pull/57/commits/5138611931e74952cf28cf6a4f9715ccd6767b86","author_link":"https://github.com/albarbaro"}]}}'
export CLONEREFS_OPTIONS='{"src_root":"/go","log":"/dev/null","git_user_name":"ci-robot","git_user_email":"ci-robot@openshift.io","refs":[{"org":"redhat-appstudio","repo":"build-service","repo_link":"https://github.com/redhat-appstudio/build-service","base_ref":"main","base_sha":"75a4c79e49ab5c1a4c15d844256d1e4419da63e3","base_link":"https://github.com/redhat-appstudio/build-service/commit/75a4c79e49ab5c1a4c15d844256d1e4419da63e3","pulls":[{"number":91,"author":"psturc","sha":"93d48f1a75216be4004014de478c4162b65c3ed0","link":"https://github.com/redhat-appstudio/application-service/pull/91","commit_link":"https://github.com/redhat-appstudio/application-service/pull/91/commits/47b9fe555e27cc65c5ebfcf51c2d26a036fab235","author_link":"https://github.com/psturc"}]}],"fail":true}'
export OPENSHIFT_CI=true
# Env vars with values set for the purpose of this PR
export BUILD_SERVICE_IMAGE=quay.io/redhat-appstudio/pull-request-builds
export BUILD_SERVICE_IMAGE_TAG=buildserv-93d48f1a75216be4004014de478c4162b65c3ed0
# References this PR number. It's required by this script: https://github.com/redhat-appstudio/e2e-tests/blob/main/scripts/install-appstudio-e2e-mode.sh
export PULL_NUMBER=13
export ARTIFACT_DIR=/tmp
# Set up github creds
git config --global credential.helper "store --file /tmp/github_creds"
echo<EMAIL_ADDRESS>> /tmp/github_creds
# Run this script
rm -rf tmp && ./.ci/oci-e2e-build-service.sh

This script should be able to deploy appstudio components from main branch on your cluster with these changes to a build-service:

the deployed image on your cluster: quay.io/redhat-appstudio/pull-request-builds:buildserv-2dcbde3b80153f7b27a45343a1277ca3cc524f39
it will use kustomize config for build-service from $JOB_SPEC env var: https://github.com/psturc/build-service/config/default?ref=2dcbde3b80153f7b27a45343a1277ca3cc524f39. This is how it should look like in the preview branch

It should be also able to run build-service test suite, run it and it should finish successfully

@trdoyle81 is looking at verifying this

Tests ran successfully
Result:
Ran 3 of 30 Specs in 181.691 seconds SUCCESS! -- 3 Passed | 0 Failed | 0 Pending | 27 Skipped PASS
PR verified @psturc


GPU rendering on Linux inside VMware eats Windows host CPU, GPU

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Performance Issue
I am running VS Code (Linux) inside VMware Workstation (Windows) - and every once in a while --type=gpu-process shows up on the command-line.
Whenever this is the case,

Linux htop flags /usr/share/code with 35-50% of CPU usage (8 cores on the VM)
Windows 10 Task Manager flags VMware Workstation consuming 25% CPU and 81%(!) GPU
VS Code "Process Explorer" has the gpu-process at 4% CPU (and 0% for the broker child)

... while doing exactly nothing.
This has the effect that consumed power is high, with the net effect that

this laptop's battery is eaten alive
fans turn on, which gets
rather annoyingly loud,
making me less productive.

Some more context: The hardware is a three year-old HP zbook 15 with Intel integrated and nvidia dedicated built-in graphics. Attached is a Thunderbolt docking station with secondary (large) external screen.
In the issue scenario, it is the Intel GPU which is used by VMware Workstation and which gets pegged. Apparently load is sensitive to screen resolution - i.e. running VMware Workstation fullscreen (with Linux -> VS Code inside) on the large screen causes greater load than running it in a window, or on the FHD integrated laptop screen.
Software context: A managed Windows 10 Enterprise, 1909, with VMware Workstation (Player) 15.6, with 3D acceleration enabled; guest: Fedora 34, X11, KDE
VS Code version: Code 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-14T22:30:16.440Z)
OS version: Linux x64 5.13.4-200.fc34.x86_64
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2903)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
31.31GB (26.25GB free)

Process Argv
--unity-launch --crash-reporter-id 80e0ae97-0bc4-4a76-8337-f435f2ec1554

Screen Reader
no

VM
100%

DESKTOP_SESSION
/usr/share/xsessions/plasmax11

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Process Info
CPU %	Mem MB	   PID	Process
    0	   160	  4365	code main
    0	    32	  4368	   zygote
    5	   128	  4397	     gpu-process
    0	     0	  4403	       broker
    0	    32	  4369	   zygote
    0	     0	  4371	     zygote
    0	    64	  7540	       window (Issue Reporter)
    0	    64	  4407	   utility
    5	   128	  4523	   shared-process
    0	    64	  4558	     ptyHost
    0	     0	  5230	       /usr/bin/bash
    0	     0	  7559	     /usr/bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    1	   257	  5170	   window (download-remote-config-files.sh (Untracked) - <masked> (Workspace) - Visual Studio Code)
    0	   128	  5196	     extensionHost
    0	    64	  5344	       /home/stefan/ManuallyInstalledSoftware/java/11/jdk-11.0.11+9/bin/java -noverify -Xmx64M -cp /home/stefan/.vscode/extensions/redhat.vscode-xml-0.17.0/server/org.eclipse.lemminx-0.17.1-uber.jar org.eclipse.lemminx.XMLServerLauncher
    0	    64	  6665	       /usr/share/code/code /usr/share/code/resources/app/extensions/json-language-features/server/dist/node/jsonServerMain --node-ipc --clientProcessId=5196
    0	   128	  5209	     watcherService
    0	    64	  5255	     searchService

Workspace Info
|  Window (download-remote-config-files.sh (Untracked) - <masked> (Workspace) - Visual Studio Code)
|    Folder (<masked>): 8759 files
|      File types: pyi(2271) py(2248) pyc(2234) so(193) json(118) txt(64)
|                  test(46) xml(43) h(40) jar(39)
|      Conf files: makefile(2) launch.json(1) settings.json(1)
|      Launch Configs: python;

Extensions (19)

Extension
Author (truncated)
Version

xml
Dot
2.5.1

gitlens
eam
11.6.0

rust-analyzer
mat
0.2.678

vscode-apache
mrm
1.2.0

vscode-docker
ms-
1.15.0

python
ms-
2021.7.1060902895

vscode-pylance
ms-
2021.7.5

jupyter
ms-
2021.8.1054968649

remote-containers
ms-
0.187.1

remote-ssh
ms-
0.65.7

remote-ssh-edit
ms-
0.65.7

remote-wsl
ms-
0.58.2

vscode-remote-extensionpack
ms-
0.21.0

powershell
ms-
2021.6.2

fabric8-analytics
red
0.3.3

vscode-commons
red
0.0.6

vscode-xml
red
0.17.0

vscode-yaml
red
0.21.1

vscode-lldb
vad
1.6.5

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
pythonvspyt700cf:30270857
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551cf:30311713
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
vscextlang:30333561
binariesv615:30325510
pythonvssor306cf:30341832
bridge0708:30335490

After haivng written the above, I noticed that the progress indicator for git operations in the status bar

was (still) turning.
I had successfully pushed to a remote ssh repo earlier - and then noticed the fans spinning. Clicking on the progress indicator causes the command box to drop down, ESC, command box disappears, Windows task manager load goes down. So right now I g

To clarify: I don't think that having a comparatively tiny progress indicator turning should eat so much CPU / GPU, even virtualized. To that extent, this is the issue that I'd like to see tracked here.


Share your models

If you trained a model you can share it here.

alphacephei.com model: https://alphacephei.com/vosk/models https://alphacephei.com/vosk/models/vosk-model-fa-0.5.zip
Thank you for your participation, dear @hrkhl .
But these are not text-to-speech models, they are speech-to-text models that can be used for ASR tasks.

@hrkhl alphacephei.com model: https://alphacephei.com/vosk/models https://alphacephei.com/vosk/models/vosk-model-fa-0.5.zip

Thank you for your participation, dear @hrkhl .
But these are not text-to-speech models, they are speech-to-text models that can be used for ASR tasks.

@hrkhl  alphacephei.com model: https://alphacephei.com/vosk/models https://alphacephei.com/vosk/models/vosk-model-fa-0.5.zip

Thank you for your participation, dear @hrkhl .
But these are not text-to-speech models, they are speech-to-text models that can be used for ASR tasks.


Add documentation on versioning

@stephpenn1 says

I'd like to have documentation on how we version a la https://jgcri.github.io/hector/articles/VersionNumbers.html

Done in 385e39b


Validation issues | 2 (BULK)

This is a bulk update.
This PR is to fix issues related to build validation messages.
If these changes are not appropriate, please let me know. I'm only trying to clean up validation issues in this repo. So I can easily close this PR or modify it in whatever way is best for the content.

Docs Build status updates of commit 52d1120:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/WINDOWSLANGUAGE-Function.md
:white_check_mark:Succeeded

dynamics-nav/WORDLAYOUT-Function.md
:white_check_mark:Succeeded

dynamics-nav/WORDXMLPART-Function.md
:white_check_mark:Succeeded

dynamics-nav/WRITE-Function--BigText-.md
:white_check_mark:Succeeded

dynamics-nav/WRITE-Function--File-.md
:white_check_mark:Succeeded

dynamics-nav/WRITEMODE-Function--File-.md
:white_check_mark:Succeeded

dynamics-nav/WRITEPERMISSION-Function--Record-.md
:white_check_mark:Succeeded

dynamics-nav/WRITEPERMISSION-Function--RecordRef-.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Designing-a-Customer-List-Report.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Designing-a-Report-from-Multiple-Tables.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Designing-a-Report-with-Images--Interactive-Sorting--and-Visibility-Toggle.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Developing-a-Sales-Rep-Role-Center-for-the-Microsoft-Dynamics-NAV-Tablet-Client.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Exporting-Data-from-Tables-to-CSV-Files.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Exporting-Data-from-Tables-to-XML-Documents.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Implementing-New-Workflow-Events-and-Responses.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Implementing-Security-Certificates-in-a-Test-Environment.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Importing-Data-from-Text-Files-to-Tables.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Inserting-Data-from-XML-Documents-to-Multiple-Tables.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Installing-the-Demo-Version.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Installing-the-Microsoft-Dynamics-NAV-Web-Server-Components-on-Three-Computers.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Installing-the-Microsoft-Dynamics-NAV-Web-Server-Components-on-Two-Computers.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Installing-the-Microsoft-Dynamics-NAV-Web-Server-Components-on-a-Single-Computer.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Installing-the-Three-Tiers-On-Two-Computers.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Installing-the-Three-Tiers-on-Three-Computers.md
:white_check_mark:Succeeded

dynamics-nav/Walkthrough--Publishing--Raising--and-Subcribing-to-an-Event-in-Microsoft-Dynamics-NAV.md
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@buck1ey
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!


Integer overflow risks, update Solidity version pragma to 0.8.x

The Challenge.sol contract uses uint256 without overflow checks, and there is some arithmetic in places that would benefit from overflow checks.
Updating the pragma to 0.8.x would include built-in solc overflow checks, easy fix that's worth doing.

Thanks for the issue 🙏
We will definitely do this.

We will soon be looking at an upgrade to the latest solidity version. But note that some overflows are a feature, not a bug, to emulate uint32 behavior of registers, where overflows are expected to happen. We'll need to be careful with introducing safe-math-by-default behavior of 0.8.x to not break expected MIPS behavior.

IIRC the specific overflow issue was in the challenge game (maybe something about being able to wrap around the binary search cursor?).
Could make sense to disable overflow protections in the VM implementation as needed, while still have it in the higher-level contracts.
But yea, worth considering these choices carefully when the time comes. :)


Use Graph Theory concepts to define positions on the BattleShip Board

Can you explain a bit more about the requirement?

Yes I will. I'll add a Document in the Wiki by tonight.

Refer to Wiki for more information regarding this!


Support multiple AutoServiceRegistrations in AutoServiceRegistrationAutoConfiguration

When we have defined multiple AutoServiceRegistration, AutoServiceRegistrationAutoConfiguration reports an error:
***************************
APPLICATION FAILED TO START
***************************

Description:

Field autoServiceRegistration in org.springframework.cloud.client.serviceregistry.AutoServiceRegistrationAutoConfiguration required a single bean, but 2 were found:
	- consulAutoServiceRegistration: defined by method 'consulAutoServiceRegistration' in class path resource [com/my/project/config/ConsulServiceAliasRegistrationConfiguration.class]
	- consulAutoServiceAliasRegistration: defined by method 'consulAutoServiceAliasRegistration' in class path resource [com/my/project/config/ConsulServiceAliasRegistrationConfiguration.class]

Action:

Consider marking one of the beans as @Primary, updating the consumer to accept multiple beans, or using @Qualifier to identify the bean that should be consumed

If it only need to check an AutoServiceRegistration exists, i think the better way is to use ObjectProvider<List<AutoServiceRegistrationAutoConfiguration>>.getIfAvailable instead of field injection.
What do you think? I'd like to create a pull request if you agree with this :)

Why would you need more than one?

@ryanjbaxter We need to register an application as multiple services in consul.

I dont think we have ever supported this.  It would need to be a separate app.

You can also manually register others manually.

@ryanjbaxter @spencergibb
In fact, AutoServiceRegistration is always supported multiple registrations as multiple spring beans.
So it's in version 1.3.x, specifically before this commit, it is working fine.
I think that if AutoServiceRegistration already supports multiple spring beans, and AutoServiceRegistrationAutoConfiguration just wants to ensure that at least one AutoServiceRegistration exists, it should not report an error.

It was never designed for multiple auto registrations. It was designed for one and then to allow the user to register more programmatically if desired.

We make assumptions in multiple places that assume a single registration.

I don’t but I am going to close this issue for now


Does not complile

The project does not compile with XCode 7.

This is true. IIRC this is due to our DTCoreText submodule, which we just switched to CocoaPods. I think switching to grabbing DTCoreText via cocoapods (pod install) plus then opening Zulip.xcworkspace  could be enough.

Here are the steps Jasdev and I took to resolve these issues:
Add this to the Podfile:
pod 'DTCoreText'
Run
pod install
git submodule update --init --recursive

Open the workspace file in XCode
Resolve the ! Errors by clicking on them and go through the "Convert to XCTest" wizard thinggy and click Cancel if it doesn't let you do anything else ... and magic... all the errors are gone-ish.
Update XCode Recommended settings
Remove the DTCoreText section from the .gitmodules file
Also remove the physical folder: rm -rf ThirdParty/DTCoreText/
If it's still in the XCode project, delete it there too: ThirdParty/DTCoreText
Clean all the things: Cmd + Option + shift + K
Remove the Zulip > iPhone target > Build Phases ... Run Script Phase that sounds like Crashlytics target

For those not so familiar with Xcode (yet), please note that by default sometimes of us open the project file instead of the workspace file.
Zulip.xcodeproj is NOT what you want.
Zulip.xcworkspace IS what you want :)


Save last viewed at state and use for replaying logs

Per https://github.com/42wim/matterircd/issues/245, we can't trust that Mattermost has updated the last viewed for channel. Instead, we save this locally and use that for replaying of logs. Useful for when experiencing flaky network causing matterircd to reconnect.
This is also part of https://github.com/42wim/matterircd/issues/313, we can dump this out to disk and read/load on start up.
With testing on my laptop, suspend, then resume. Adding print logging on line 555 (logger.Info("Using last saved for channel %s ", brchannel.ID, lastViewedAt)):
[0194] ERROR matterclient: connection not alive: NewWebSocketClient: model.websocket_client.connect_fail.app_error, read tcp <IP_ADDRESS>:42334->xxx.xxx.xxx.xxx:443: read: connection reset by peer
[0204] ERROR matterclient: got a listen error: &model.AppError{Id:"model.websocket_client.connect_fail.app_error", Message:"model.websocket_client.connect_fail.app_error", DetailedError:"read tcp <IP_ADDRESS>:42334->xxx.xxx.xxx.xxx:443: read: connection reset by peer", RequestId:"", StatusCode:500, Where:"NewWebSocketClient", IsOAuth:false, params:map[string]interface {}(nil)}
[0204]  INFO matterclient: reconnect: logout
[0204]  INFO matterclient: reconnect: login
[0205]  INFO matterclient: Found version <IP_ADDRESS>.27.0.3bd4fc983a4aa622183c3bd1e77029e0.true
[0211]  INFO matterclient: found xxx users in team myteam
[0213]  INFO matterclient: reconnect successful
INFO[2020-12-13T09:25:21+11:00] Using last saved for channel %symnb5h6ubifuiymzntyumqtgae<PHONE_NUMBER>343  module=matterircd

Then again, but with a message waiting:
|09:19 -!- Irssi: Join to #haw-test was synced in 39 secs
|09:46 <matterircd> Replaying since 2020-12-13 09:46:26
|09:46 <hloeung> [09:46] Test replay backlog  [@@zy7t6us9c3fn8yywj3cmp4mb8e]

With no old and repeated replay logs from any of the channels I'm in.

Looking good, nice simple solution 👍
Will try to do some tests myself tomorrow.


Can't sign in " auto mode or manual mode "

**In Auto Mode :
Traceback (most recent call last):
File "duce.py", line 765, in
File "duce.py", line 277, in fetch_cookies
File "browser_cookie3_init_.py", line 630, in load
File "browser_cookie3_init_.py", line 596, in chrome
File "browser_cookie3_init_.py", line 383, in init
File "browser_cookie3_init_.py", line 192, in init
File "browser_cookie3_init_.py", line 232, in _add_key_and_cookie_file
File "browser_cookie3_init.py", line 98, in crypt_unprotect_data
RuntimeError: Failed to decrypt the cipher text with DPAPI
In Manual Mode :
cloudscraper.exceptions.CloudflareChallengeError: Detected a Cloudflare version 2 Captcha challenge, This feature is not available in the opensource (free) version.
Screenshots
https://i.ibb.co/8sqDJ9c/Screenshot-255.png
https://i.ibb.co/TrJ9qg5/Screenshot-256.png
System Info:

OS: Windows 10**

I am on vacation right now. I will check on it as soon as possible😅

and this problem when run code in colab :
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/urllib3/response.py", line 362, in _error_catcher
yield
File "/usr/local/lib/python3.7/dist-packages/urllib3/response.py", line 668, in read_chunked
self._update_chunk_length()
File "/usr/local/lib/python3.7/dist-packages/urllib3/response.py", line 600, in _update_chunk_length
line = self._fp.fp.readline()
File "/usr/lib/python3.7/socket.py", line 589, in readinto
return self._sock.recv_into(b)
File "/usr/lib/python3.7/ssl.py", line 1071, in recv_into
return self.read(nbytes, buffer)
File "/usr/lib/python3.7/ssl.py", line 929, in read
return self._sslobj.read(len, buffer)
ConnectionResetError: [Errno 104] Connection reset by peer
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/requests/models.py", line 758, in generate
for chunk in self.raw.stream(chunk_size, decode_content=True):
File "/usr/local/lib/python3.7/dist-packages/urllib3/response.py", line 492, in stream
for line in self.read_chunked(amt, decode_content=decode_content):
File "/usr/local/lib/python3.7/dist-packages/urllib3/response.py", line 696, in read_chunked
self._original_response.close()
File "/usr/lib/python3.7/contextlib.py", line 130, in exit
self.gen.throw(type, value, traceback)
File "/usr/local/lib/python3.7/dist-packages/urllib3/response.py", line 380, in _error_catcher
raise ProtocolError('Connection broken: %r' % e, e)
urllib3.exceptions.ProtocolError: ("Connection broken: ConnectionResetError(104, 'Connection reset by peer')", ConnectionResetError(104, 'Connection reset by peer'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 544, in main1
auto(links_ls)
File "", line 429, in auto
course_id = get_course_id(link)
File "", line 262, in get_course_id
r = requests.get(url, allow_redirects=False)
File "/usr/local/lib/python3.7/dist-packages/requests/api.py", line 75, in get
return request('get', url, params=params, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/requests/sessions.py", line 542, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.7/dist-packages/requests/sessions.py", line 697, in send
r.content
File "/usr/local/lib/python3.7/dist-packages/requests/models.py", line 836, in content
self._content = b''.join(self.iter_content(CONTENT_CHUNK_SIZE)) or b''
File "/usr/local/lib/python3.7/dist-packages/requests/models.py", line 761, in generate
raise ChunkedEncodingError(e)
requests.exceptions.ChunkedEncodingError: ("Connection broken: ConnectionResetError(104, 'Connection reset by peer')", ConnectionResetError(104, 'Connection reset by peer'))

and this problem when run code in colab :
Can you open another issue since it's different


Get URL for search result

https://app.asana.com/0/555089885850811/1206080145852890/f
Fixes an issue with the search header where we weren't getting the referenced URL for a search result.

@anthonyshull Yeah great questions -- and I don't think we indicate that to the user, and I'm not sure we'd necessarily want to (since these point to functions that are part of the T's services to the public, even though technically they're not hosted on the mbta.com website).
However there are also other ways that could be handled anyhow. In this case the content team could create a vanity link, for example mbta.com/school-trip, which redirects to the external site.. e.g. we could have internal:/school-trip be the search result URL. They already make vanity links for all sorts of things (usually for internal URLs that're too verbose for a Tweet or signage) though I don't know if there're any constraints around external links.

though I don't know if there're any constraints around external links.

If the input does not start with http or https, then it will start with internal: or entity:.
There is nothing stopping editors from entering an internal URL as an external url, but that could be easily domain checked, unless it was hidden behind a non mbta.com redirect domain.
However, doing a quick DB audit of these destination URLs, they all follow the rule stated above.

@anthonyshull will there be breaking changes? Editors expect external destinations (beginning with http/s) to work for visitors. Such as the various payment programs which are funneled to subdomains and other orgs. These are valid and should continue working.

@anthonyshull will there be breaking changes? Editors expect external destinations (beginning with http/s) to work for visitors. Such as the various payment programs which are funneled to subdomains and other orgs. These are valid and should continue working.

It only changes the URL if internal: or entity: are found in the string.


Display timestamp with posted content

Description
As a Gateway client developer, I want to know, as accurately as possible given the available data, the time at which a given piece of content was posted to the chain.
Elaboration
We have some TODOs in the code; one of them is to come up with a reasonable timestamp to attach to content display. For now, we've been encoding the time that the content (Parquet file) was retrieved from IPFS by the SAT backend; obviously this is not acceptable by any means, as it would be different for different backends. It's also not reasonable to expect a client to convert a block number to a time, because:

Block times now with async backing are not consistent enough
Don't want to force the client to interact directly with the chain, or have to provide an endpoint to do block number → timestamp conversion. Also if the goal is to insulate web2 developers from needing to grok web3 concepts, forcing them to deal with block numbers is the opposite of that.

My initial thought is that content-watcher should get the timestamp inherent from the block when it retrieves content, and include that in the webhook payload when it sends out the content notifications. Reasons?

content-publisher is not the right place, because it would require a DSNP spec change, or at minimum a Frequency extension to the spec
Forcing the client to do it subverts the purpose of Gateway (see above)

Discussion?

We do have a published date field on content activity. https://spec.dsnp.org/ActivityContent/Types/Note.html


cbio_create() has a typo in argument

There is an argument this_seqtype used troughout the function but the forma argument accepted by the function is this_seq_type

This is now fixed in #35


Update subsystem transformer and mixed domain tests to use EAP 7.2

https://issues.jboss.org/browse/WFLY-10806
Needs https://github.com/wildfly/wildfly-core/pull/3664

PS: I would say next time just use EAP_7_2_0 for the name straightaway rather than temporary names, so we don't have to make changes across the source tree; the only change needed would be changing the container from WF to EAP.

Yeah, my approach made sense at the time (my thinking was it was a signal to update maven deps pulled in via the LegacyKernelServicesInitializer) but I've learnt the hard way now.

I guess the advantage would have been should things go very awry with WF 16 as a target, it could be disabled while more EAP 7.2 as a target tests be added... but that would be unlikely.

Retest this please


[CSDiag] Adjust assert to account for changes in `filterContextualMem…

…berList`.
Originally filterContextualMemberList would only return a limited
set of closeness kinds CC_GeneralMismatch, CC_Argument{Label, Count}Mismatch,
and unavailable/inaccessible. At some point later it also started
matching almost everything besides CC_SelfMismatch and logic in
visitUnresolvedMemberExpr needs to get adjusted to account for that.
Resolves: rdar://problem/50668864

@swift-ci please test


Include size parameter in Search documentation

No illustration of search size

Looks like we should move the section at the end of https://github.com/pelias/pelias-doc/blob/master/response.md to the /search and /reverse endpoint pages

Looks covered in the reverse topic. https://github.com/pelias/pelias-doc/blob/master/reverse.md#size
But not in the search topic.

closed with PR #44


[ROCm] Renaming GetCudaStream to GetGpuStream

This PR renames the GetCudaStream to GetGpuStream. It is a follow-up from PR#28568.

Test performed: //tensorflow/tools/pip_package:build_pip_package compiles at both develop-upstream and master branch.
@tatianashp @whchung @chsigg

@jerryyin can you please resolve conflicts ?

@jerryyin can you please resolve conflicts ?

@rthadur Will do thanks for reminding

@jerryyin, looks like more merge conflicts have shown up...please re-resolve. thanks

@deven-amd Thanks for reminding! Done

This PR seems to get stuck in the queue for a while, could you help take a second look @chsigg @whchung
In my last rebase, I accepted the code change from tensorflow/core/kernels/searchsorted_op.cc in #28479 as the latest version.

Rebasing again to cope with conflict. Looks like GetGpuStream() is already merged in this commit. This PR does not block further GetGpuStream() dependent PRs.

@jerryyin can you please rebase again , i guess new changes came in , sorry for the trouble.

@rthadur No worries, just done. I think this one that does the vertical stride collide with other PRs that deals with one operator at a time.
FYI in the rebase I accepted all upstream changes from bincount_op histogram_op and reduction_gpu_kernels.

@jerryyin  please fix merge conflicts.

@dagamayank Thanks for reminding. Done.
This is my 7th rebase followed by a conflict from somewhere else. It is a special case though because it collide with multiple in-working PRs of ours as well. A successful merge of another operator will block this one to be merged. I'd like to thank @whchung @chsigg for their repeatedly prompt review and approval. Taking that my intended update already went in from a dependent PR, it doesn't block anything at this point.

Closing this PR because it is already merged in this commit


🎉 4.0.0

ℹ About this release

Version: 4.0.0
Type: major
Last Release: 3.0.1 (5/22/2023, 9:13:38 AM) [?]
Commits to merge: 11 [?]

Build System

Deprecate node.js v14 / v17 (7a2de45)

⚡ BREAKING CHANGES

The node.js versions v14 and v17 are no longer maintained and are therefore no longer supported. See https://nodejs.dev/en/about/releases/ for more details on node.js release cycles.

📦 Development Dependencies

Update @types/node from ^18.16.3 to ^20.2.5
Update c8 from ^7.13.0 to ^7.14.0
Update eslint from ^8.39.0 to ^8.41.0

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

npm package (@latest dist-tag)
GitHub release
Docker container (sebbo2002/fitness-first-ical-proxy:4.0.0)
v4.0.0

Your semantic-release bot :package::rocket:


Validate combinations of the major incident flags

Certain combinations of the MAJOR_INCIDENT and MAJOR_INCIDENT_LITE
FlawMeta flags should never be valid, to ensure that this constraint is
enforced a validation method is added on the FlawMeta model and executed
on each save.
Closes OSIDB-329

@RedHatProductSecurity/osidb-devs I've done things like this so that we can move forward, I am not at all happy with the result but the alternatives take way more time and thinking:
I think this goes back to changing the way we handle the check on whether a Flaw is a major incident or not, currently there's a lot of shenanigans to be done because we use the same representation as bugzilla (+, -, ?, ...), the correct way IMO would be to simply have the is_major_incident flag become an Enumeration that is set to a well-defined type on ingestion from bugzilla, something like:
class MajorIncident(Enum):
    UNKNOWN                  # ( ,  )
    REJECTED                 # (-, -)
    REQUESTED                # (?, ?)
    ACCEPTED                 # (+, ) or (+, -)
    CISA                     # ( , +) or (-, +)

Then this validation doesn't even need to happen in the first place, only at ingestion from BZ, consumers who create flaws through OSIDB won't even be able to make a mistake

CI fails due to pip-audit check, see #22


Latest build broke

I detected this when trying to go to https://translations.standardforpubliccode.org/zh_Hant_TW/ , but that gave me a 404.
Looking at the workflow, it seems it failed to build.

Fixed by #54


The delete action buttons take you to the home page.

When placing an action button to delete the table element from the db, the button proceeds to delete said record, but it necessarily takes you to the main root of the website. If you do not specify any path, it takes you to the root. Is there a possibility of allowing you to remain in the same screen that you are working on?

When I press the indicated button, it deletes the card, but takes me to the main page
The ideal would be to reload the embedded view, or at least reload the entire web page.

Make it a multi-step action. 1st action delete record. 2nd action reload embedded view

Thanks! Yes, it is an alternative solution, it worked for me, if I had the possibility to just reload the page or reload the embedded view simply, that would be ideal.


server: fix panic when error occurres on peer intialisation

If error occurred on peer initialisation then p is nil which cause
panic while accessing peer variables.

Travis is borked atm, it'll be fixed in #162. Merging this now as I've verified everything checks out locally w.r.t linting, integration tests, etc.

Looking at this a bit more closely, it doesn't really avert a panic as the peer struct itself is currently always properly initialized before any operations that can error out are encountered. In spite of that, I'll still merge this as it makes the code more defensive against future modifications to the newPeer function.


Inform user if cog.yaml or predict.py are missing

cog.command.openapi_schema should inform users if cog.yaml is missing.
Currently, cog.command.openapi_schema silently returns {} if cog.yaml is missing (see here).
This is undesirable in situations where cog.yaml is in fact missing, but the user thinks that it is present. In such cases, cog build will fail on openapi schema validation, but it won't be clear why.
This just happened to me!
For various questionable reasons, cog.yaml in my project root is a symlink and the target was accidentally excluded in my .dockerignore. This led to an interesting situation where cog run python -m cog.command.openapi_schema returned the correct schema, but cog build failed at schema validation because the schema returned empty.
A warning or just a note that my cog.yaml was missing would have saved me quite a bit of detective work.

@joehoover this is a easy fix by adding a warning, should I raise a PR to fix this ?

should I raise a PR to fix this ?

@asingh9530 yes please!

@zeke missed notification, added PR for fix.


Make velocity post-processing into BDM1 the default for Two-Phase App

Mandatory Checklist
Please ensure that the following criteria are met:

[x] Title of pull request describes the changes/features
[x] Request at least 2 reviewers
[x] If new files are being added, the files are no larger than 100kB. Post the file sizes.
[x] Code coverage did not decrease. If this is a bug fix, a test should cover that bug fix. If a new feature is added, a test should be made to cover that feature.
[x] New features have appropriate documentation strings (readable by sphinx)

As a general rule of thumb, try to follow PEP8 guidelines.

@zhang-alvin I don't see any reason why this won't also work for h- adaptivity, but I haven't tested it. The post-processing happens after RANS2P runs and simply over writes the ('velocity',0) arrays in q and ebqe. This option used to be the default in most of the problems I've done. I believe we started to disable it because I hadn't updated it to include the porosity scaling, but @jhcollins and I did that. It had already been updated to work with mesh motion. The only test it broke with the comparison of the cube location in a still tank. @jhcollins and I have found that several aspects of simulations are significantly better, especially on coarse grids including numerical diffusion of water mass through the open top and non-physical variation of the VOF around  0 or 1.

@cekees I think the post-processing didn't work with adaptivity previously because post-processing relied on nodal-based partitions of the mesh. I assume that's still true?

That's right @zhang-alvin. It's still true that it requires nodal partitioning.  The algorithm naturally works on the support of each nodal DOF, so it's convenient that each "owned" node has it's support on the owning processor. We do assign element ownership in the partitioning, so there is an underlying, non-overlapping element partitioning. I wonder if we could pass that mesh to MeshAdapt. I guess the tricky thing is re-initializing the adapted mesh. The most straight-forward way to do it would be to run the same nodal partitioning scheme on the adapted mesh.

@jhcollins is doing some more checks on conservation properties, and I'm running some ALE examples on onyx before we merge this.

@tridelat the issue I found in parallel is unrelated to this PR. I'm going to merge this and do a new PR for the parallel solver issue.


Development: Fix flaky ExerciseScoresChartIntegrationTest

Checklist
General

[x] The changed server tests succeed locally and on Bamboo and on GitHub Actions.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.

Motivation and Context
The test in ExerciseScoresChartIntegrationTest was flaky. The test's setup method contains multiple awaits that regularly timed out for MySQL and Postgres because of many scheduled tasks and slower database access.
Description
The tests are fixed by setting lastScheduledRun in ParticipantScoreScheduleService. Setting the value to Instant.now() prevents the ScheduleService from calculating the scores of results unrelated to this tests (Results created in prior tests).
MySQL run without failing ExerciseScoresChartIntegrationTest:
https://github.com/ls1intum/Artemis/actions/runs/6407425948/job/17394178642
Postgres run without failing ExerciseScoresChartIntegrationTest:
https://github.com/ls1intum/Artemis/actions/runs/6407425948/job/17424739364 (Test succeeded before job was canceled [see logs])
Steps for Testing
Test changes only
Review Progress
Code Review

[x] Code Review 1
[ ] Code Review 2

lgtm
Would it make sense to merge this and the other PRs with similar fixes into one PR? This would make it easier to check the MySQL-build status and verify that all these tests now work.


[fix] RL scheduler

fix an error in rl_scheduler #421

Job PR-427-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-427/1/index.html

Job PR-427-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-427/2/index.html


Move away from using config::kCpuSpeed -> system_clock.GetFrequency()

system_clock.GetFrequency() will return the actual cpu speed. In the event that the cpu speed is changed at runtime, using the constexpr clock speed is not always a good idea. Moving drivers to use this will make sure that clock speeds are where they should be throughout the runtime of the code.

This is kind of related. I noticed that the LPC40xx's peripheral bus shares the same clock unlike the LPC17xx where each controller has its own configurable clock. I also found out that the current ADC driver changes the peripheral clock bus speed and other drivers may be unaware of this. CAN, for example, derives it's own bus clock from the peripheral bus clock. In this case, we might need a getter for peripheral bus clock frequency. However, it's still possible for a user to init drivers in a random order without knowing that one may be changing the peripheral bus clock.

Yes, I saw this too. One thing we are going to have to do is come up with a set of clear rules about designing drivers for SJSU-Dev2. Temporally, at the time in which the ADC driver was created, there was no code handling the clockspeed of the peripheral bus, so that engineer set it to what they needed. We should definitely come up with a standard about this and ways to handle different clock speeds.
I should be coming up with a PR soon that should help with this.

Should be resolved by #293


Shivansh Goel

Shivansh Goel

well done. 2.9/3. will be scaled to 1 point in the gradebook according to Mr. Yeung.
Only thing you could have done was enhance your output for the 2nd homework problem.
-Rohan :))


Removed useless env var

Codecov Report

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

@@          Coverage Diff          @@
##           master    #87   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          49     49
  Lines         758    758
=====================================
  Hits          758    758

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 2c2aaa7...78db9a5. Read the comment docs.


Podman build failure: Connection reset by peer

Description
Originally created as a https://github.com/actions/runner/issues/1994 .  It was suggested in the thread to create a bug for virtual environment, but that didn't happen, IIUC.  So, doing that now.
The issue affects multiple projects.  Basically, everyone who tries to build podman images in GHA.
Freshly failed job: https://github.com/ovn-org/ovn-fake-multinode/actions/runs/3104887855/jobs/5029867767
Quoting the original reporter @ebattat :

Starting to get the following error from 7/12 (Github actions change) when running podman build inside ubuntu-latest container:
error running container: error from /usr/bin/crun creating container for [/bin/sh -c dnf update -y --nobest]: sd-bus call: Connection reset by peer
Example for our failed build:
The full GitHub actions link
The same workflow run successfully several days ago, link
Steps to reproduce
name: Podman test
on:
workflow_dispatch:
jobs:
bundle:
runs-on: ubuntu-latest
steps:
- name: Create container
run: |
cat <<EOF | sudo podman build -t test-container -f - .
FROM fedora:36
RUN echo test
EOF
Output:
STEP 1/2: FROM fedora:36
Resolved "fedora" as an alias (/etc/containers/registries.conf.d/000-shortnames.conf)
Trying to pull registry.fedoraproject.org/fedora:36...
Getting image source signatures
Copying blob sha256:75f075168a24344c81b41a4877ffc6005723793ced6424f20b5f3ee9d6a66b57
Copying blob sha256:75f075168a24344c81b41a4877ffc6005723793ced6424f20b5f3ee9d6a66b57
Copying config sha256:3a66698e604003f7822a0c73e9da50e090fda9a99fe1f2e1e2e7fe796cc803d5
Writing manifest to image destination
Storing signatures
STEP 2/2: RUN echo test
error running container: error from /usr/bin/crun creating container for [/bin/sh -c echo test]: sd-bus call: Connection reset by peer
: exit status 1
For more detail
**Workaround for GitHub actions till fix: **
sudo -E XDG_RUNTIME_DIR= podman build -t test-container -f - .

Platforms affected

[ ] Azure DevOps
[X] GitHub Actions

Runner images affected

[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Version: 20220905.1
https://github.com/ovn-org/ovn-fake-multinode/actions/runs/3104887855/jobs/5029867767
Is it regression?
It is a regression.  Successful build from the original reporter: https://github.com/redhat-performance/benchmark-runner/runs/7229627245?check_suite_focus=true
Expected behavior
Container is able to be built.
Actual behavior
Containers are not built with:
error running container: error from /usr/bin/crun creating container for [/bin/sh -c echo test]: sd-bus call: Connection reset by peer: exit status 1
Repro steps
Run any job that builds a podman container.

Hey @igsilya.
Could you please try to set XDG_RUNTIME_DIR to emtpy value?
env:
      XDG_RUNTIME_DIR: ''

@al-cheb that workaround works, yes.  But it is a workaround.

@al-cheb that workaround works, yes. But it is a workaround.

You can reproduce the same behavior on a local vm:
user@ubnt01:~/ovn-fake-multinode$ id -a
uid=1000(user) gid=1000(user) groups=1000(user),4(adm),24(cdrom),27(sudo),30(dip),46(plugdev),110(lxd),999(docker)
user@ubnt01:~/ovn-fake-multinode$ echo $XDG_RUNTIME_DIR
/run/user/1000
user@ubnt01:~/ovn-fake-multinode$ sudo -E ./ovn_cluster.sh build
error running container: error from /usr/bin/crun creating container for [/bin/sh -c /install_pkg.sh $OS_IMAGE]: sd-bus call: Transport endpoint is not connected

sudo -E preserves a user environment.  These point to directories with UID names, which  not intended to be used by another user.

sudo -E preserves a user environment. These point to directories with UID names, which not intended to be used by another user.

Yes, but we need that environment to not pass all the other variables manually.

Hi @al-cheb,
I agreed, I think that this issue should be solved because it worked properly in the past
Actual: sudo -E XDG_RUNTIME_DIR= podman build -t test-container -f - .
Expected:  podman build -t test-container -f - .

Hi @al-cheb, @igsilya
I agreed, I think that this issue should be solved because it worked properly in the past:
Actual: sudo -E XDG_RUNTIME_DIR= podman build -t test-container -f - .
Expected:  podman build -t test-container -f - .

@ebattat, I can reproduce the same behavior on my local vm:
user@ubnt01:~$ id
uid=1000(user) gid=1000(user) groups=1000(user),4(adm),24(cdrom),27(sudo),30(dip),46(plugdev),110(lxd),999(docker)
user@ubnt01:~$ sudo -E podman build -t test-container .
STEP 1/2: FROM fedora:36
STEP 2/2: RUN echo test
error running container: error from /usr/bin/crun creating container for [/bin/sh -c echo test]: sd-bus call: Transport endpoint is not connected
: exit status 1

@igsilya
The problem is reproducible in any environment and is related to the operation of the application, not related to compatibility or version. Therefore, the issue is not related to the operation of the runner images.
I have to close the issue. If you have any other questions or concerns, feel free to contact us at any time.


Add a blacklist to forgo reloading of fonts

This makes it so that fonts from Google Web Fonts and Typekit don't get reloaded.
Haven't tested this yet.

This is interesting, but it seems to be pretty arbitrary. We don't want to just "disable" particular domains. This would make a lot more sense if you could edit the list of blacklisted domains in the settings screen. See #9 .

Closing this PR, as we need a custom black list UI, and a more sensible default list.


SA-359 - Raise a NOMIS sync event when an offender is allocated to a schedule

Codecov Report
Base: 70.76% // Head: 70.42% // Decreases project coverage by -0.34% :warning:

Coverage data is based on head (7959cbd) compared to base (a39192b).
Patch coverage: 5.26% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #182      +/-   ##
============================================
- Coverage     70.76%   70.42%   -0.34%
  Complexity      799      799
============================================
  Files           160      161       +1
  Lines          3687     3706      +19
  Branches        130      132       +2
============================================
+ Hits           2609     2610       +1
- Misses         1000     1018      +18
  Partials         78       78

Impacted Files
Coverage Δ

.../hmppsactivitiesmanagementapi/entity/Allocation.kt
96.96% <ø> (ø)

...esmanagementapi/entity/AllocationEntityListener.kt
0.00% <0.00%> (ø)

...psactivitiesmanagementapi/service/EventsService.kt
71.11% <12.50%> (-12.68%)
:arrow_down:

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

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


#435 Fix zero with scale to boolean (main)

Fixes https://github.com/ezylang/EvalEx/issues/435 in the main branch

@stevenylai I don't think this is the appropriate fix, as it doesn't really solve the: https://github.com/ezylang/EvalEx/issues/435.
The problem must be somewhere in the evaluation process since EvaluationValue already contains incorrect result.

It's a follow-up for https://github.com/ezylang/EvalEx/issues/417 where the fix is to replace .equals() with .compareTo(). We missed the int -> boolean conversion part from the previous fix.

All good. I agree with you that there is a necessity to replace .equals() with .compareTo(). I just wanted to point out that it doesn't fix the issues referenced in your PR comment. https://github.com/ezylang/EvalEx/issues/435 :)

@stevenylai I just did a bit more detailed review of the process (at first I thought that this method is not invoked during evaluation). I can see now that this is the proper fix. I apologize for the false information and thank you for providing the fix quickly. :)


[Bug Report] Many Youtube Videos no longer play that used to

Attempt to play a video, either by search or url, and the video correctly enters queue but does not play. Video will be skipped in the queue entirely. Over the course of quarantine more and more videos have become unplayable...or at least more have been noticed. My best guess is youtube made yet another change that broke lavaplayer.
To Reproduce
Steps to reproduce the behavior:
Play certain youtube videos, many of which have worked earlier in this particular bot version. I assume a youtube change has broken lavaplayer. again. ugh.
Some example videos, and yes they're almost all memes. Don't judge my discord. https://www.youtube.com/watch?v=ZZ5LpwO-An4
https://www.youtube.com/watch?v=RxgdaMZRgRo
https://www.youtube.com/watch?v=FzAb8rVqDkY
https://www.youtube.com/watch?v=HgzGwKwLmgM
https://www.youtube.com/watch?v=YOBpATa1HUY
All of these have been played successfully before at least within a few months, if not on this exact bot version. None of them play now, and plenty more.
Expected behavior
Videos to play.
Debug Info
System Properties:
  java.version = 1.8.0_251
  java.vm.name = Java HotSpot(TM) Client VM
  java.vm.specification.version = 1.8
  java.runtime.name = Java(TM) SE Runtime Environment
  java.runtime.version = 1.8.0_251-b08
  java.specification.version = 1.8
  os.arch = x86
  os.name = Windows 10

JMusicBot Information:
  Version = 0.2.8
  Owner =<PHONE_NUMBER>81880065
  Prefix = !!
  AltPrefix = null
  MaxSeconds = 0
  NPImages = false
  SongInStatus = false
  StayInChannel = true
  UseEval = false
  UpdateAlerts = true

Dependency Information:
  JDA Version = 3.8.3_464
  JDA-Utilities Version = 2.1.5
  Lavaplayer Version = 1.3.46

Runtime Information:
  Total Memory = 22
  Used Memory = 20

Discord Information:
  ID =<PHONE_NUMBER>53796874
  Guilds = 2
  Users = 12

Console Output:
[23:04:04] [INFO] [Startup]: Loaded config from C:\Users\Marc\Documents\MusicBot\config.txt
[23:04:04] [INFO] [JDA]: Login Successful!
[23:04:04] [INFO] [WebSocketClient]: Connected to WebSocket
[23:04:04] [INFO] [JDA]: Finished Loading!
[23:04:45] [ERROR] [LocalAudioTrackExecutor]: Error in playback of ZZ5LpwO-An4
com.sedmelluq.discord.lavaplayer.tools.FriendlyException: Something broke when playing the track.
	at com.sedmelluq.discord.lavaplayer.tools.ExceptionTools.wrapUnfriendlyExceptions(ExceptionTools.java:43)
	at com.sedmelluq.discord.lavaplayer.track.playback.LocalAudioTrackExecutor.execute(LocalAudioTrackExecutor.java:117)
	at com.sedmelluq.discord.lavaplayer.player.DefaultAudioPlayerManager.lambda$executeTrack$1(DefaultAudioPlayerManager.java:349)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.NullPointerException: null
	at java.net.URI$Parser.parse(Unknown Source)
	at java.net.URI.<init>(Unknown Source)
	at com.sedmelluq.discord.lavaplayer.source.youtube.YoutubeTrackFormat.getUrl(YoutubeTrackFormat.java:66)
	at com.sedmelluq.discord.lavaplayer.source.youtube.YoutubeSignatureCipherManager.resolveFormatUrl(YoutubeSignatureCipherManager.java:97)
	at com.sedmelluq.discord.lavaplayer.source.youtube.YoutubeAudioTrack.loadBestFormatWithUrl(YoutubeAudioTrack.java:87)
	at com.sedmelluq.discord.lavaplayer.source.youtube.YoutubeAudioTrack.process(YoutubeAudioTrack.java:42)
	at com.sedmelluq.discord.lavaplayer.track.playback.LocalAudioTrackExecutor.execute(LocalAudioTrackExecutor.java:105)
	... 4 common frames omitted

Duplicate of #417

Hmm, thought it was different from the start of the error but closer inspection shows them to be basically the same. My mistake.