If wifi goes down for a moment, the jobs on workers are set to lost

If the wifi goes down (not power related), the MQTT connection between workers and the broker (on the leader) is but, and the last-will messages are sent.

Actually I'm not sure why this happens. After an ungraceful disconnect, the client will try to reconnect. I have logic that re-sets state when a reconnect occurs. I've added a small LOC self.logger.debug("Reconnecting to MQTT") to the reconnect callback to help debug the next time.


All Logins Fail As Of Today

Reset a few devices I had this afternoon and all at once they were unable to log back in... several accounts... credentials are all correct.
$ ./Shell/shell.py
Traceback (most recent call last):
File "./Shell/shell.py", line 564, in
RobinhoodShell().cmdloop()
File "./Shell/shell.py", line 37, in init
self.trader.login(username=USERNAME, password=PASSWORD)
File "/home/Z8/ETF/Shell/Robinhood.py", line 136, in login
raise RH_exception.LoginFailed()
AttributeError: 'module' object has no attribute 'LoginFailed'

Yes, Robinhood seems to have added an additional layer of security where in you need to verify the device through email/text. Will see if we can add something similar to the shell.

Reliant on this as well! Two related threads I've been following are https://github.com/westonplatter/fast_arrow/issues/85 and
https://github.com/Jamonek/Robinhood/issues/176#issuecomment-489291269
Wish I could help fix RobinhoodShell--and I've managed to induce an email auth code using https://github.com/aamazie/Robinhood/blob/master/Robinhood/Robinhood.py and my own Gmail imap realtime fetcher, but don't know where to take it from there (in code).

Hey @anilshanbhag , how can I modify RobinhoodShell's login mechanism to require, besides username and password, either a 6-digit QR code offered when 2FA gets enabled, email verification code, or even an already-authenticated token acquired through other means (from another project that solved this)?
If device_id is also required, I can also obtain that elsewhere and provide that to RobinhoodShell login inputs.


yubikey-manager: patch path of pkill binary

Motivation for this change
Fix #137263
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

[yuka@mari:~]$ nix-shell -p yubikey-manager

[nix-shell:~]$ echo $PYTHONPATH
/nix/store/5s6glfc0lmh5386fdip0p73vkawjfdmd-python3.9-yubikey-manager-4.0.5/lib/python3.9/site-packages:/nix/store/75m3xfinlzjzapyjww0l8mh149mscpm6-python3.9-click-8.0.1/lib/python3.9/site-packages:/nix/store/dqxic3j7csd4ywn94n4smmnz55p039g3-python3-3.9.6/lib/python3.9/site-packages:/nix/store/facvi78m4qsa80k836hgl4wbsx17qwwn-python3.9-packaging-20.9/lib/python3.9/site-packages:/nix/store/50ikfzzisjnk9kai7yq72likxmqr1dhl-python3.9-pyparsing-2.4.7/lib/python3.9/site-packages:/nix/store/ypvgpnbplij3riy1i9vb0s9w7ldzdbsf-python3.9-six-1.16.0/lib/python3.9/site-packages:/nix/store/p7qhdpa9bf1p538xwrplvj7pvqvbh9kg-python3.9-pycparser-2.20/lib/python3.9/site-packages:/nix/store/lxigrwzrkz3wk2j06fakslf6zavwddhc-python3.9-cffi-1.14.6/lib/python3.9/site-packages:/nix/store/yvl0gj8ky5ww9yvl5jc4iii8fq2l18ph-python3.9-cryptography-3.4.7/lib/python3.9/site-packages:/nix/store/2h96i18sa5dg9hb4ywb018m7l53dc2a0-python3.9-pyscard-2.0.1/lib/python3.9/site-packages:/nix/store/ch2a1jx449r7d3zmjmm4p7zj42m8as6h-python3.9-pyusb-1.2.1/lib/python3.9/site-packages:/nix/store/8i4l8x8z4y5kjqb2r6jyw7bglid1q025-python3.9-pyasn1-0.4.8/lib/python3.9/site-packages:/nix/store/vcpsni7lh26cgy3mrpdm1x5fljfyfiaw-python3.9-idna-3.2/lib/python3.9/site-packages:/nix/store/d5w6lr316pfq2x10g2m7l7j49amvya70-python3.9-pyopenssl-20.0.1/lib/python3.9/site-packages:/nix/store/3975hxpjb8xfws31pqfk1jy00cpjmwh6-python3.9-fido2-0.9.1/lib/python3.9/site-packages

[nix-shell:~]$ nix run -f "https://github.com/yu-re-ka/nixpkgs/archive/feature/yubikey-manager.tar.gz" yubikey-manager

[...]

# Now PYTHONPATH is still set to the first version of ykman and the new version of ykman is getting confused by that

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

4 packages built:

gen-oath-safe
yubikey-manager
yubikey-manager-qt
yubioath-desktop


[App]: regex does not match all metrics

Current Behavior
At each iteration of my training routine, I am logging 30 metrics, all starting with the same name ("signal-0", "signal-2", ..., "signal-29"). I would like to create a single lineplot with the evolution of the 30 signals.
When using regex to select the metrics starting with "signal", only 10 metrics are matched, the others are not reported in the plot:

Expected Behavior
The expected plot is the one obtained by manually selecting each signal:

(however, I would like to avoid manual selection of the signals, and regex should match this plot)
Steps To Reproduce
Reproducible Python example:
import wandb
wandb.init()
for step in range(10):
    for j in range(30):
        wandb.log({
            "step": step,
            f"signal-{j}": j
        })

Screenshots
No response
Environment
OS: MacOS
Browsers: Chrome
Version: 110.0.5481.177
Additional Context
No response

Hi @Niccolo-Ajroldi! thank you for reporting this. I have submitted this to our eng team and will be waiting for them to respond. Is it okay to follow up with you if we have any more questions?

Sure! Pls go for it.
Happy to help!
Best regards,
Niccolò
On Wed, Mar 1, 2023, 12:16 ArtsiomWB @.***> wrote:

Hi @Niccolo-Ajroldi https://github.com/Niccolo-Ajroldi! thank you for
reporting this. I have submitted this to our eng team and will be waiting
for them to respond. Is it okay to follow up with you if we have any more
questions?
—
Reply to this email directly, view it on GitHub
https://github.com/wandb/wandb/issues/5059#issuecomment-1450791016, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOR3CSYJJYCZGDTQBXI5343WZ6VA5ANCNFSM6AAAAAAVLKV3AM
.
You are receiving this because you were mentioned.Message ID:
@.***>

I encountered a similar issue. May I ask if the current version of Wandb has addressed this problem? I am currently using version 0.36 of Wandb-local.

嗨！感谢您报告此事。我已将此提交给我们的eng团队，并将等待他们的回复。如果我们还有其他问题，可以跟进您吗？

Hi @Niccolo-Ajroldi! thank you for reporting this. I have submitted this to our eng team and will be waiting for them to respond. Is it okay to follow up with you if we have any more questions?

I encountered a similar issue. May I ask if the current version of Wandb has addressed this problem? I am currently using version 0.36 of Wandb-local.

WandB Internal User commented:
Niccolo-Ajroldi commented:
Sure! Pls go for it.
Happy to help!
Best regards,
Niccolò
On Wed, Mar 1, 2023, 12:16 ArtsiomWB @.***> wrote:

Hi @Niccolo-Ajroldi https://github.com/Niccolo-Ajroldi! thank you for
reporting this. I have submitted this to our eng team and will be waiting
for them to respond. Is it okay to follow up with you if we have any more
questions?
—
Reply to this email directly, view it on GitHub
https://github.com/wandb/wandb/issues/5059#issuecomment-1450791016, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOR3CSYJJYCZGDTQBXI5343WZ6VA5ANCNFSM6AAAAAAVLKV3AM
.
You are receiving this because you were mentioned.Message ID:
@.***>

We are closing this issue due to inactivity. If you feel that it merits additional review, please provide a reproduction and add a comment explaining why. Thank you.


Search UI hosting + configuration

This opens up the example search UI to be served outside of "development mode" on a specified port and point to a specified OFAC server instance via environment variables.

Codecov Report

Merging #131 into master will decrease coverage by 1.39%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #131     +/-   ##
=========================================
- Coverage   59.35%   57.96%   -1.4%
=========================================
  Files          23       23
  Lines        2581     2581
=========================================
- Hits         1532     1496     -36
- Misses        885      926     +41
+ Partials      164      159      -5

Cool @fhwrdh! I took this and adapted it, but wanted your thoughts. Personally I don't want to run Go and node HTTP servers for this in production, so I took the build output and am having Go serve the files. It lets us serve right off of localhost:8084 and run search queries.
https://github.com/moov-io/ofac/pull/132

FYI, we talked about moving the code out of examples, so it's in webui/ on OFAC's root. I'll merge my PR today. Thanks for this change (and the original work)!
https://github.com/adamdecaf/ofac/tree/search-ui-config/webui


container.js is messy - these should be split into single files

after this has been done https://github.com/appitierre/aptr-uikit/issues/86 - could we have an examples folder inside each element?


Access violation occurs after setting mediaPlayer.AudioDevice

private void _audioDeviceComboBox_SelectionChanged(object sender, SelectionChangedEventArgs e)
{
DeviceInformation selectedDevice = (DeviceInformation)((ComboBoxItem)_audioDeviceComboBox.SelectedItem).Tag;
if (selectedDevice != null)
{
mediaPlayer.AudioDevice = selectedDevice;
}
}
Access violation occurs after setting selectedDevice in the above code. No try catch. What is the cause?

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

ID: e6e7d28c-129b-b640-7eac-f0c1807a346c
Version Independent ID: 300d1f3b-a48a-e472-e100-97b21074585f
Content: Play audio and video with MediaPlayer - Windows UWP applications
Content Source: windows-apps-src/audio-video-camera/play-audio-and-video-with-mediaplayer.md
Product: uwp
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

Thanks for taking the time to let us know about this. However, this feels not so much like feedback specifically about the documentation but rather like a support request. Here are all of the developer support resources, for completeness.

The Feedback Hub app is the right place to lobby for a change, a feature, or to make a suggestion.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

If you receive assistance via any support resource, and you believe that information would help us improve the docs, then please come back and let us know.


Allow the use inside #[test]s

Since we can't instrument the test binary's main function (yet), I think it would be possible to add an init_test(name: &'static str) function that users can call at the beginning of their test that uses spawn inside (usually because spawn is used inside a function imported from the library being tested). This init_test would then note the name in a global static that spawn reads from and uses as the first argument to the binary.
Furthermore we'd need to make the --mitosis-content-process-id argument be an environment variable in that case because the test binary would error due to the unknown argument.
Is there any particular reason this is not already an environment variable that is immediately cleared upon being read in init?

I'm open to making it an environment variable. I'm not quite clear on how the test stuff would work.

Basically my idea is to only annotate tests that use it. So if you have
// does some `mitosis` inside
fn foo() {}

#[test]
fn a() {}

#[test]
fn b() {
    mitosis::init_test("module::path::b");
    foo();
}

then when foo calls spawn, it fetches the "module::path::b" found in a global variable and calls the binary with that string as an argument. Thus only the test in question is run, and since it immediately finds the mitosis::init_test before doing any logic beyond the test harness, it should work just like if you added mitosis::init in the main function of a binary.
I'll cook up a PR for the env stuff, and a second one for the test stuff to show how I think it should work (+ tests showing that it does work (hopefully))

That's not going to work well, though, since it's going to first setup the
rust test harness stuff, so your process is going to end up doing more work.
On Wed, Jan 8, 2020, 3:41 PM Oliver Scherer @ Cosmian <
<EMAIL_ADDRESS>wrote:

Basically my idea is to only annotate tests that use it. So if you have
// does some mitosis insidefn foo() {}
#[test]fn a() {}
#[test]fn b() {
mitosis::init_test("module::path::b");
foo();
}
then when foo calls spawn, it fetches the "module::path::b" found in a
global variable and calls the binary with that string as an argument. Thus
only the test in question is run, and since it immediately finds the
mitosis::init_test before doing any logic beyond the test harness, it
should work just like if you added mitosis::init in the main function of
a binary.
I'll cook up a PR for the env stuff, and a second one for the test stuff
to show how I think it should work (+ tests showing that it does work
(hopefully))
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Manishearth/mitosis/issues/8?email_source=notifications&email_token=AAMK6SBASASFJAE7ZPFIDOLQ4WRDJA5CNFSM4KEFXOLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIL3YMA#issuecomment-571980848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMK6SHYHEF4OTLVFJETGXDQ4WRDJANCNFSM4KEFXOLA
.

so your process is going to end up doing more work.

yea, that's ok, we're not spawning them by the thousands. And it's better to be slow during testing than not working at all

Sounds good, feel free to make these PRs, make sure you caveat the testing
API stuff
On Wed, Jan 8, 2020, 3:46 PM Oliver Scherer @ Cosmian <
<EMAIL_ADDRESS>wrote:

so your process is going to end up doing more work.
yea, that's ok, we're not spawning them by the thousands. And it's better
to be slow during testing than not working at all
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Manishearth/mitosis/issues/8?email_source=notifications&email_token=AAMK6SDS7CZ72RI3R7GUYQTQ4WRXPA5CNFSM4KEFXOLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIL4IAI#issuecomment-571982849,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMK6SA2MDCTO5QTKRYNSJ3Q4WRXPANCNFSM4KEFXOLA
.


[docs] add documentation for FLink GCE setup using bdutil

Now that bdutil supports Flink setups, we can adjust the documentation.
Is the Google Compute Engine setup placed correctly under "Setup & Configuration" or should we put it under an extra category like "Other"? What do you think?

+1 to merge!

Will merge this...


2024.3 performance issues caused by Catppuccin

Is there an existing issue outlining your problem?

[X] I have searched the existing issues and they do not solve my problem.

Have you followed the instructions in the the pinned issue?

[X] I have followed the instructions in the pinned issue and they do not solve my problem.

Describe your problem.
When using the Catppuccin (latte) theme, the action is not as responsive as when using the default theme, and sometimes can even cause a crash.
As can be seen from the attached video, when I open a pop up to do something, the cursor experiences severe lag, whether using the keyboard arrow keys or using the mouse.
Attach screenshots.
Uploading Screen Recording 2024-12-17 at 16.26.13.mp4…
What IDE(s) are you seeing the problem on?
PhpStorm 2024.3 & IDEA 2024.3
Any additional comments?
This also happens in Intellij IDEA on the latest stable version.

Duplicate of #190


Move exception parsing to extension package

Suggestion: move exception parsing functionality to the extension package.
The Telegram API error messages are not part of the API, so they could change at any moment. That is why we cannot guaranty that code based on exceptions would work in changing circumstances. The library should throw only basic exceptions based on API response codes, such as ApiException, ForbiddenException, BadRequestException for general error codes, such as 400, '403, 429and such. To allow users handle different error codes we could provide separate packageTelegram.Bot.Exceptions`, that will return specific error code (object) that will help users handle different types of errors. This change would allow update only this package to keep up with API changes and keep core library functionality untouched.
try { bot.SendMsg() }
catch (ApiRequestException e) {
  switch (parser.Parse(e)) {
    case ParameterException:
      foo();
    case ChatBlockedException:
      bar();
  }
}

This change would mean that we will mark all minor exceptions as obsolete, and no more exception minor exceptions would be added to the library.

I suggest using Telegram.Bot.Extensions namespace prefix for this (as well as other useful library helpers). I'm already using it for KeyboardBuilders and Filters packages.

@matteocontrini The problem is not the library size. If exceptions were a part of Bot API and were properly documented then it wouldn't be a problem. But the thing is that they are changed all the time by telegram devs, they're not documented so we have to hunt them down every time therefore we can't guarantee anything stable and it's just complicate things. I understand that the proposed solution is not ideal.

@karb0f0s I agree with making a separate package for exceptions.
@tuscen It's a good idea to have that naming convention for extension packages.

@matteocontrini is right and I might have a solution for it.
Interface for exception parser in client package:
// in package Telegram.Bot
interface IExceptionParser { ApiRequestException Parse(JObject response); }

User can optionally register an exception parser
// using Telegram.Bot.Extensions.Exceptions
var client = new TelegramBotClient(token) {
  ExceptionParser = new ExceptionParser()
}

And code looks like normal
try { bot.SendMsg() }
catch (ParameterException ex) {
  foo();
}
catch (ChatBlockedException ex) {
  bar();
}

Looks good. Also, I like poulads approach.

Proposed changes to Telegram.Bot library are in #694 #695.
I'd like to change specific class exceptionNameErrorCode and exceptionNameErrorDescription to general class properties ErrorCode and ErrorCodeDescription. Than it will be possible to lookup exception like this
var exceptionTypeInfo = exceptions.FirstOrDefault(info => info.ErrorCode == apiResponse.ErrorCode)

But in this case exceptions should accept  apiResponse.Description in constructor and strip ErrorCodeDescription itself

I guess this extension point might lead to problems:
https://github.com/TelegramBots/Telegram.Bot/blob/675fc4b992b69398ddbc9d46c1efd3ea251653b1/src/Telegram.Bot/TelegramBotClient.cs#L279-L280
Here are hardcoded HTTP Errors, that we expect parser will handle:
https://github.com/TelegramBots/Telegram.Bot/blob/675fc4b992b69398ddbc9d46c1efd3ea251653b1/src/Telegram.Bot/TelegramBotClient.cs#L257-L268
But if we accept external parser, it should provide it's own HTTP Errors list, and this check might not have sense for external exception handler:
https://github.com/TelegramBots/Telegram.Bot/blob/675fc4b992b69398ddbc9d46c1efd3ea251653b1/src/Telegram.Bot/TelegramBotClient.cs#L257-L259
And it raises another question - should external parser extend only minor exceptions, or deliver all range of exceptions?

@karb0f0s I think it's okay for exception parser to return any exception type. It seems that it will need to return not only ApiRequestException  derived exception but also HttpRequestExceptions (e.g. 5XX exceptions). Therefore it will need to accept HttpResponseMessage instead of JObject.


Performance getting worse relative to other libraries

benchmarks.txt
We are many times slower than other PCRE bindings.  Worse than what's stated in the REAME.  Perhaps Text changing to UTF-8 is a cause?

Also note the benchmark isn't even finishing, at least on my machine.  (Tests are passing in CI though.)  Gotta fix that before closing this issue.


Update example in Readme

Objective
The link in the current readme is broken and references an older version of the bevy 3d/3d_scene example.
Solution
This PR updates the readme example to be based on the new bevy example and fixes the link.
There may be a better way to rotate the cylinder collider than using Collider::compound that I'm not aware of, feel free to update this PR.

Hi, I added my own version of this example along with an updated video/gif in f231f86. Closing in favor of that :)

Sorry I forgot about this PR since I opened it, thanks for updating the example. I noticed the link was still broken though: #396.


Clean up indentation in docker-storage-setup.sh

Multiple authors/contributors had left the script with
inconsistent indentation which made the code harder
to read.

LGTM. Good cleanup john. Thanks.


Adjust regex pattern based on manageiq-api-common#79

https://github.com/ManageIQ/manageiq-api-common/pull/79

This will need a regenerated client, right?

It doesn't look like this change changes anything in the client


⚠ Jenkins Plugins Artifactory has degraded performance

In 4e91030, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1141 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 748165a after 13 minutes.


Feat/sort memos by updated ts

Fixes #1261

@boojack Any updates on this ?

I think we need to sort memos in the backend.
https://github.com/usememos/memos/blob/main/store/memo.go#LL287C24-L287C24

Implemented with https://github.com/usememos/memos/pull/1760, https://github.com/usememos/memos/pull/1761


[stable/jasperreports] Add kubeapps text to charts READMEs

Signed-off-by: Carlos Rodriguez Hernandez<EMAIL_ADDRESS>Add text related to kubeapps to READMEs
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/ok-to-test
/lgtm


open link inside an article

Touching a link inside an article does nothing.
Ideally it should shows a dialog that let you check the url and open it. For instance, FairEmail does this:

Thanks for the report. This is a regression from recent changes, the fix will be available in the next release.


qspi: stm32: Add support for reset cmd on init

If the flash is used in 4-byte addressing, reading SFDP will fail after a system reset if the flash isn't power cycled or hardware reset, since Zephyr will try to use 3-byte addressing while the flash (still) expects 4-byte addressing.
This commit adds the ability to send a reset command to the flash as part of initialization, which complements the existing reset-gpio functionality, and is useful on low-pincount flashes which do not have a hardware reset.
Signed-off-by: Ole Morten Haaland<EMAIL_ADDRESS>

is it a problem to reset the flash at the init whatever the reset-cmd ?

I'm unsure, but if one uses a flash that doesn't support 4-byte addressing, it would at least not be strictly necessary, in which case it would delay boot by a few milliseconds. For most application that doesn't really matter though, but the cost of keeping it optional seems low. One could perhaps argue it should be on by default, though. I'm unsure...


Weeds 3D cool calibrators

i got a call from the team in Louisiana having an issue where the app kept crashing when taking videos.

I couldn't answer the question, but ended up sending them to the weeds 3D hotline. [(385)429-3337]

Any word on what the problem was.  Keeping an eye on wifi connectivity which caused us trouble in beta

@marialauracangiano has Zach been adding issues to the Weeds3D repository? We've had some questions about Weeds3D ourselves and would be interested in the answer

Hi. I don't think he has yet, we were talking about that today. Most of the
questions have been pretty simple so far so it didn't seem to be worth
adding. One some complex situation we agreed that he'll create an issue for
today. We are still getting accustomed to using issues so it's not where
our heads go first when someone comes with a problem but we'll get there.
Are there any particular questions that you had? Could you start an issue
with those so we use that as practice as well?
On Thu, May 12, 2022 at 4:01 PM Sarah Seehaver @.***>
wrote:

@marialauracangiano https://github.com/marialauracangiano has Zach been
adding issues to the Weeds3D repository? We've had some questions about
Weeds3D ourselves and would be interested in the answer
—
Reply to this email directly, view it on GitHub
https://github.com/precision-sustainable-ag/On-farm-Protocols/issues/164#issuecomment-1125373214,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK35ODHG75N7NANY36KUD3LVJVPQVANCNFSM5VSM4WXA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

--
Maria Laura Cangiano
Research Associate
USDA - ARS - SASL
NC State University
Department of Crop and Soil Sciences
Beltsville, MD 20705


Game crashes when clicking on demesne hierarchy in town menu

Describe the bug
Game crashes when clicking on any options in the Banner Kings menu.
Weird thing is that the banner king menu does not show up UNLESS I craft something.
Mod Version

Standard version
Numeric Version: <IP_ADDRESS>

Evidence

[x] The mod is present in the callstack;
[x] The issue only happens with the mod activated;
[x] You tested it with the least amount of other mods as possible.

Crash Reports
https://report.butr.link/270721.html (clicking on demesne hierarchy)
https://report.butr.link/3F5D7C.html (clicking on demesne management)
Exact moment where the crash happened:

This seems to be related to a bug in <IP_ADDRESS> that caused inconsistency with the mod data. Menu option missing indicates that. There's no around it other than starting a campaign with an updated version. It's in the changelog of <IP_ADDRESS>.


Load mesh data onto GPU via jcanephora

Needed for https://github.com/io7m/r2/issues/71

Provided by https://github.com/io7m/smfj-jcanephora to avoid a dependency on jcanephora.


Importing from d3-scale causes page load to hang forever

Describe the bug
When importing from d3-scale@4, the page never loads. No error is logged.
This happens even when disabling ssr with export const ssr = false
Reproduction
https://github.com/plmrry/svelte-kit-import-d3-bug/blob/main/src/routes/index.svelte
Logs
No response
System Info
System:
    OS: macOS 10.15.7
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 1.09 GB / 32.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 14.15.4 - ~/.nvm/versions/node/v14.15.4/bin/node
    npm: 6.14.10 - ~/.nvm/versions/node/v14.15.4/bin/npm
  Browsers:
    Chrome: 91.0.4472.114
    Firefox: 88.0.1
    Safari: 14.1.1
  npmPackages:
    @sveltejs/kit: next => 1.0.0-next.128
    svelte: ^3.34.0 => 3.38.3

Severity
blocking all usage of SvelteKit
Additional Information
No response

Note that this is fixed by downgrading from d3@7 to d3@6, so this must have something to do with the recent d3 module switch

Thanks for reporting Paul. I would expect that the bug most likely is in Vite. Perhaps https://github.com/vitejs/vite/issues/2491
A couple things you might try:

Can you reproduce it on a Vite SSR project that isn't SvelteKit? You could try adding it to https://github.com/sveltejs/vite-plugin-svelte/tree/main/packages/e2e-tests/vite-ssr
Does https://github.com/vitejs/vite/pull/3950 fix it for you?

I expect that this is now fixed with the latest version of Vite


Android Q recompile issue

Information
This is more of an FYI as I know this is brand new and not an official firmware yet.

Apktool Version (apktool -version-2.4.0) -
Operating System ( Windows) -
APK From? (Stock) -

Stacktrace/Logcat
I: Using Apktool 2.4.0
Exception in thread "main" java.lang.NumberFormatException: For input string: "Q"
        at java.lang.NumberFormatException.forInputString(Unknown Source)
        at java.lang.Integer.parseInt(Unknown Source)
        at java.lang.Integer.parseInt(Unknown Source)
        at brut.androlib.res.AndrolibResources.mapSdkShorthandToVersion(AndrolibResources.java:637)
        at brut.androlib.res.AndrolibResources.getMinSdkVersionFromAndroidCodename(AndrolibResources.java:618)
        at brut.androlib.Androlib.build(Androlib.java:290)
        at brut.androlib.Androlib.build(Androlib.java:269)
        at brut.apktool.Main.cmdBuild(Main.java:247)
        at brut.apktool.Main.main(Main.java:79)

Steps to Reproduce

`apktool b SystemUIGoogle

Frameworks
See below
APK
https://www.androidfilehost.com/?fid=1395089523397917544
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything?
You have to at least fix the one bad .9png file before there is even a chance for this to recompile.
If you are trying to install a modified apk, did you resign it?
No
Are you using the latest apktool version?
Yes, and tried others as well

the same error,  may I ask when will apktool support android Q？

I also found an error in how array.xml was decoded.  It appears as styles instead of string arrays and without values.

The original error here is quite easy to fix and I'll do that real quick, its just a mapping between "Q" -> 10,000.
The other comments in here are not related and been hidden.

Thank you, sir!


Issues creating new user when "active" attribute is present in request

Hello,
Summary
I'm doing SCIM user provisioning tests using Postman and EntraID with SCIM Playground (https://scim.dev).
It seems "api.scim.dev" API doesn't allow creation of user when we specify the "active" attribute.
In "api.scim.dev" API Schema, it seems to indicate that the "active" attribute is "readWrite"
Encountered error message
I'm encountering error stating that we cannot write to "active" attribute:
{
  "schemas": [
    "urn:ietf:params:scim:api:messages:2.0:Error"
  ],
  "detail": "Write to \"urn:ietf:params:scim:schemas:core:2.0:User:active\" is not supported",
  "status": 500
}

Schema
{
  "name": "active",
  "description": "A Boolean value indicating the User's administrative status.",
  "type": "boolean",
  "mutability": "readWrite",
  "returned": "default",
  "required": false,
  "multiValued": false,
  "caseExact": false
}

Tests
Tested with Postman and EntraID.
When removing "active" attribute from request (Postman), then the user creation occurs successfully.
On EntraID side, it seems that the "active" attribute will always be part of requests (even if we remove attribute in Entra App schema and remove Attribute mapping).
Question
When checking SCIM schema (api.scim.dev), it seems the "active" attributes should be "readWrite", could it be a bug in solution deployed on "api.scim.dev" that may wrongly consider that attribute to be "readOnly"?
Thanks to all people working on this solution, it allows to test and play with SCIM!
Have an excellent end of Year, best regards,
Fabrice

I am also experiencing this issue around Entra ID, have not tried using postman or other methods. Also thanks for providing this platform it is invaluable for testing and understanding SCIM!

Fixed it. Thanks for reporting.
See https://scim.dev/playground/users.html#create-user
If you are still facing issues, let me know.

Thanks Arie for the correction.
I've now tested and can confirm provisioning from EntraID is now working correctly.
Thanks again for your help and for providing the scim.dev platform!

@arietimmerman How did you fix it ?
I'm not using the playground but I have the same problem when Entra updates a user.
Validation failed. Errors: [] Message: Write to "urn:ietf:params:scim:schemas:core:2.0:User:active" is not supported Body: {"schemas":["urn:ietf:params:scim:api:messages:2.0:PatchOp"],"Operation
Do I have to change something in SCIMConfig.php ?

I think I get it. Any attribute sent but not in the mapping of the SCIMConfig file results in an error.


In table/table_test.cc,  TestRandomAccess's conditions always be false because static const bool kVerbose has not been changed since initialization.
static const bool kVerbose = false;
...;
if (kVerbose) std::fprintf(stderr, "---\n");
...;
if (kVerbose) std::fprintf(stderr, "Next\n");
...;
if (kVerbose) std::fprintf(stderr, "SeekToFirst\n");
...;
if (kVerbose)
            std::fprintf(stderr, "Seek '%s'\n", EscapeString(key).c_str());
...;
if (kVerbose) std::fprintf(stderr, "Prev\n");
...;
if (kVerbose) std::fprintf(stderr, "SeekToLast\n");

Fix some typos in comments.

Thanks for the suggestion. However, I would prefer to keep the kVerbose related code around. The reason we have the "kVerbose" declaration there is so that it is easy to turn on extra logging when needed to diagnose a test failure. I would like to preserve that ability.


appease the knip overlords

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4594  👈
#4593
#4591
#4590
#4588
#4587
#4586
main

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

Join @jnsdls and the rest of your teammates on  Graphite


Enforce https if requested with protocol or in cmdline option

Isn't that intended?

I doubt it. But rather than figure that out how about this one?
This is after clicking these buttons.
1

2

3

@mlsteele this should fix the gui. waiting for nojima to get back to make sure I didn't break everything

Cool! Consider the service half approved. I think anyone on @keybase/react-hackers should be able to review this. However, CI says you have a flow error first.

Oh, also can you merge/rebase on master and fix any flow errors?

So I think adding the web case in the last few places will fix the flow errors. btw, you can do yarn test stories -u to have it update the storyshots file.

ok cool, thanks @akalin. For some reason flow was giving me no errors even though I was missing those switch cases.

Yeap, flow doesn't mandate that all cases of an option type are present in a switch

Is this ok to merge @akalin-keybase?


Application should inform about failed public key download

Feature description
The application should inform about failed public key download
Problem and motivation
Our public key API was down and we made a fresh APP install.  The user has tried to scan QR codes and all QR's was invalid. App should inform the user that public keys and/or rules are downloaded (at least for the first time) and the user can start using the application.

How is this download done? Is it "Reload" in the settings? Clicking this does not help with my Austrian vaccination certificate not being detected as valid.
I have tested app-acc-release-signed-1.1.3-RC-1.apk, and I have set "Validate with:" to "Österreich".


WIP: SAP Hana dialect

Supporting SAP HANA Dialect.
At this point only tables.
TODO:

[ ] Support functions and Views
[ ] Docs

Related to #212
Related to #20

@ariel-bentu nice work! That was fast!

On first usage, user will be requested to install it, that helps on packing because it allows us to pack for multi platforms and let node handle deps.

is there a way to advice the dependency manager that this package is?
The package is in a public npm repo maintained by SAP at https://npm.sap.com
Instructions from its site is:
npm config set @sap:registry https://npm.sap.com
npm install @sap/hana-client
In addition, why is it not finding the module, if I previouslly installed it globally (manually ran npm install -g ... )?
Thanks

On first usage, user will be requested to install it, that helps on packing because it allows us to pack for multi platforms and let node handle deps.

is there a way to advice the dependency manager that this package is?
The package is in a public npm repo maintained by SAP at https://npm.sap.com
Instructions from its site is:
npm config set @SAP:registry https://npm.sap.com
npm install @sap/hana-client
In addition, why is it not finding the module, if I previouslly installed it globally (manually ran npm install -g ... )?
Thanks

No, we don't have it for now. But I'll add in you PR, ok? @ariel-bentu

go ahead and merge.
any idea why global on npm is not recognized?

About the modules, for now it's better to install the module inside of the extension folder because in global install users usually have the latest version, and for the extension we are going to fix it in a tested version.
I'll send a review. It's 2 questions only. See below:

I'll add the registry specification and reach you back.

@ariel-bentu  https://github.com/ariel-bentu/vscode-sqltools/pull/1/files please see and test it, but I believe after this we can merge it on master

Is it fully working? @ariel-bentu

Is it fully working? @ariel-bentu

I sent you by mail, you need to change to:
args: ['--@sap:registry=https://npm.sap.com']

Regarding the -g option: using a globally installed package instead of requiring installation:
I found that npm has the command link:
npm link '@sap/hana-client'

It would be nice to first try to link, and if fails (exit code != 0) fall back to installing it. what do you think?


Current Use of EasyDMA can cause incorrect optimization

Hey, I was doing some work with short EasyDMA TWIM transactions, and noticed that certain operations failed in release (opt level 3), but not in debug (opt level 0). Furthermore, they worked in opt levels 0, 1, or 'z', but not in opt level 2 or 3.
I believe this is due to the compiler not "understanding" our use of giving buffers as .ptr()s, and assuming that certain data is never used.
I was able to fix this by placing a compiler fence (compiler_fence(Ordering::AcqRel); after the DMA transaction was completed. It is likely we will need to use this everywhere EasyDMA is used.

CC @nrf-rs/nrf52

Relevant IRC logs, including some ASM diffs of what the compiler fence changes: https://mozilla.logbot.info/rust-embedded/20181111#c15597073-c15597632

Good catch, @jamesmunns! Jorge wrote about the need for compiler fences in DMA code in his original blog post on the topic. I included that in my DMA implementation for lpc82x-hal, but unfortunately I didn't remember any of it while working on nRF52.
I think just adding the compiler fences everywhere they're needed is an acceptable solution for right now. But aside from that, I believe we shouldn't update or add any further DMA code before cleaning up what we currently have.
I think we've already talked about this on various pull requests, but I believe there should be a mid-level API (between the low-level PAC and the high-level HAL) that encapsulates things like task registers, event registers, and (most pressingly) DMA registers. Then all HAL APIs would use the mid-level API for DMA and we would have a single place to fix this issue.

Hanno,
That sounds fine to me, I have added one more method to TWIM locally, which was necessary to support the i2c accelerometer. I haven't PRd that yet, but you can see it here.
I'm happy to keep it local for now, until the easydma interface has been reworked.

@jamesmunns

That sounds fine to me, I have added one more method to TWIM locally, which was necessary to support the i2c accelerometer. I haven't PRd that yet, but you can see it here.
I'm happy to keep it local for now, until the easydma interface has been reworked.

I mean, if you want to merge that one (since it's already written), I won't veto that. I just think we need to stop piling up the duplicated DMA code at some point :)

Closed by #34


Edit add group and delete group functionality

@chrus-chong I do not currently have time to test out and debug the implementation.
However, this follows closely with the existing infrastructure in AB-3.
Please take a look at the commits and if you can, try to run and see what the issues might be.
I have begun tackling our storage issues, but the work isn't complete.
Have made this branch because I did not want to merge into the already semi-functional branch-add-group.

Codecov Report

Merging #74 (a5da476) into add-delete-group-modification (ac1f799) will decrease coverage by 4.92%.
The diff coverage is 16.66%.

@@                         Coverage Diff                         @@
##             add-delete-group-modification      #74      +/-   ##
===================================================================
- Coverage                            67.36%   62.44%   -4.93%
- Complexity                             401      408       +7
===================================================================
  Files                                   73       80       +7
  Lines                                 1330     1467     +137
  Branches                               136      157      +21
===================================================================
+ Hits                                   896      916      +20
- Misses                                 400      511     +111
- Partials                                34       40       +6

Impacted Files
Coverage Δ

...edu/address/logic/commands/CreateGroupCommand.java
0.00% <0.00%> (ø)

...edu/address/logic/commands/DeleteGroupCommand.java
0.00% <0.00%> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
86.36% <0.00%> (-4.12%)
:arrow_down:

.../seedu/address/logic/parser/CreateGroupParser.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/DeleteGroupParser.java
0.00% <0.00%> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
85.36% <0.00%> (-11.86%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
92.15% <0.00%> (-7.85%)
:arrow_down:

src/main/java/seedu/address/model/group/Group.java
0.00% <0.00%> (ø)

... and 17 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 ac1f799...a5da476. Read the comment docs.


WIP: Feat/gcs storage

Description

This MR enables using Google Cloud Storage as a variables storage (block outputs) the same way as S3 can be used.
How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

[ ] The PR is tagged with proper labels (bug, enhancement, feature, documentation)
[ ] I have performed a self-review of my own code
[ ] I have added unit tests that prove my fix is effective or that my feature works
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

cc:

@luizarvo Thank you adding the GCS storage!!! Could you also fix the code quality check errors? You can install the pre-commit hooks in your development environment: https://github.com/mage-ai/mage-ai/blob/master/README_dev.md

Sure mate =)

PTAL? @wangxiaoyou1993

@luizarvo did you test this PR?

@luizarvo did you test this PR?

You mean manually? I actually did. I am using this code version on my mage deployment. Didn't test with spark yet though (which is the main reason why I implemented it actually). I'm running a pipeline to ETL a specific table from BigQuery to a database in production with this code (and using GCS to store variables of course)


Typescript usage

For usage in Typescript I think some of the types are a bit wrong, I think:
registerFunctionComponent(callback: Function, { metaUrl: ?string, observedAttributes: ?string[] })
Should maybe be:
registerFunctionComponent(callback: Function, { metaUrl?: string, observedAttributes?: string[], name?: string })
to allow passing only some of the options?
(name is missing in the readme)

Should be fixed in 6ac1fa6 :) JSDoc doesn't support optional keys in the notation I used; but when using typedef it's possible. Changed to that.


LXD Resource Limitation Issue (SWAP)

Required information

Distribution: Ubuntu
Distribution version: 16.04.4 LTS
The output of "lxc info" or if that fails:

Kernel version: 4.4.0-116-generic
LXC version: 2.2.1
LXD version: 2.2.1
Storage backend in use: zfs

Issue description
We are running R inside of a container and are trying to LIMIT the amount of Swap space it can use on the host. Reason: When R runs very heavy loads, it uses up all the available SWAP is causes the R process to fail and slows down the entire system.
Steps to reproduce

We tried setting the swap limit variables:

    lxc.cgroup.memory.limit_in_bytes = 2G
    lxc.cgroup.memory.memsw.limit_in_bytes = 4G

We then restarted the container but obtained the following errors"

ubuntu@ubuntu:~$ lxc info --show-log R
Name: R
Remote: unix://
Architecture: x86_64
Created: 2018/04/16 08:58 UTC
Status: Stopped
Type: persistent
Profiles: default

Log:

            lxc<PHONE_NUMBER>1211.970 ERROR    lxc_cgfsng - cgroups/cgfsng.c:cgfsng_setup_limits:1971 - Permission denied - Error setting memory.memsw.limit_in_bytes to 4G for R
            lxc<PHONE_NUMBER>1211.970 ERROR    lxc_start - start.c:lxc_spawn:1205 - Failed to setup cgroup limits for container "R".
            lxc<PHONE_NUMBER>1211.983 WARN     lxc_conf - conf.c:lxc_delete_network:3103 - Failed to remove "veth7O9B7T" from host: Invalid argument.
            lxc<PHONE_NUMBER>1211.983 ERROR    lxc_start - start.c:__lxc_start:1358 - Failed to spawn container "R".
            lxc<PHONE_NUMBER>1212.529 ERROR    lxc_conf - conf.c:run_buffer:416 - Script exited with status 1.
            lxc<PHONE_NUMBER>1212.529 ERROR    lxc_start - start.c:lxc_fini:546 - Failed to run lxc.hook.post-stop for container "R".
            lxc<PHONE_NUMBER>1212.529 WARN     lxc_commands - commands.c:lxc_cmd_rsp_recv:177 - Command get_cgroup failed to receive response: Connection reset by peer.
            lxc<PHONE_NUMBER>1212.530 WARN     lxc_commands - commands.c:lxc_cmd_rsp_recv:177 - Command get_cgroup failed to receive response: Connection reset by peer.

We followed the instructions from:

https://www.safematix.com/system/linux/ubuntu-lxc-memory-swap-limit/
But still does not work.... We then found that we need to RESTART the entire server in order for it to apply..
Question: Is there a way to have these variables applied without a server reboot?
Information to attach

[ ] Any relevant kernel output (dmesg)
[x] Container log (lxc info NAME --show-log)
[x] Container configuration (lxc config show NAME --expanded)
[x] Main daemon log (at /var/log/lxd/lxd.log or /var/snap/lxd/common/lxd/logs/lxd.log)
[x] Output of the client with --debug
[x] Output of the daemon with --debug (alternatively output of lxc monitor while reproducing the issue)

lxc-monitor-debug.log
lxd.log
lxc-config-show-R--expanded.log
lxc-info-R--show.log

Did you see this: https://github.com/lxc/lxc/issues/668

@lcherone, yes that's a pre-requisite. Once you've made sure that this is enabled you should set:
lxc config set <container-name> limits.memory 2GB
lxc config set <container-name> limits.memory.swap true

Hi,
I have set that before but it did not work. After I set those variables and restarted the container, it did not limit the container to a specific amount of swap space. It caused he container to use up all the swap space on the host, which is what we dont want.

The container is set to a memory Limit of 16GB

But we dont want that to mean that the container should also use 16GB of SWAP space....
Question: What is the best way to limit a container's use of host swap space?
Thanks for your input.

That's not easily doable if I remember correctly. Swap limits are set by memsw.limit_in_bytes but that just limits memory + swap. If you limit to 16GB the container might still use all of the swap if it runs out of memory. You can try and set limits.memory.swap.priority to 10 to make it very unlikely that the container uses swap.

Hi,
Since it's not recommended that I use both "limits.memory.swap..." & "

Hi @brauner ,
I cannot see @lcherone, reply?

Ok, so your lxd.log file shows:
lvl=warn msg="CGroup memory swap accounting is disabled, swap limits will be ignored." t=2018-04-16T10:54:01+0200

To fix this you need to append swapaccount=1 to your kernel command line on your host and reboot the server.
If grub is your default bootloader you can add this option to /etc/default/grub to
GRUB_CMDLINE_LINUX, that is `GRUB_CMDLINE_LINUX="swapaccount=1" and then update grub with the appropriate command. After this these limits should work.

Sorry I removed my comment as I saw the link https://www.safematix.com/system/linux/ubuntu-lxc-memory-swap-limit so presumed you had already set it.
The issue I linked to was https://github.com/lxc/lxc/issues/668

Hi,
I have done this and it required a server reboot, not just a container reboot to work.
Kind Regards

@elbrinke, but that's not the point. You have to reboot the server once to enable this feature. This has nothing to do with LXD or containers in general. It's just that your kernel that is shared between containers wasn't properly setup to handle swap limits. So you should reboot the server once and then you can do swap limits. Currently no-one on your system can because it's simply not enabled.

I see, thanks @brauner


the ability to work with Schematics pragmatically via API, instead of cli

please provide  documentations for schematics API to work with schematics without the cli.

Hi @eng-dibo, this seems to be a duplicate of https://github.com/angular/angular-cli/issues/15779


⚠ ClamAV has degraded performance

In 654eff8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 55 ms

Resolved: ClamAV performance has improved in 5ca4451.


SyntaxError

Thanks for the script. However, after following your instructions, I get the following error:
2020-12-30 21:06:23 ERROR (MainThread) [frontend.js.latest.202012120] http://192.168.XXX.XXX:8123/local/zwavegraph3.js:7:0 SyntaxError: expected expression, got '<'
I put the .js file in /config/www/

This repository is archived as this exact graph was included as part of the Zwavejs2mqtt.
If you are using the ZwaveJS, please upgrade to full Zwavejs2mqtt to get this functionality back.


View recompilation appears to be broken

Describe the bug
Views used to recompile once you make changes to cshtml - that does not appear to be working anymore with 3.0
To Reproduce
Steps to reproduce the behavior:

Create new netcore3.0 mvc web app
start debugging
edit Index page, save, reload page

Expected behavior
changes should be visible, but they are not.
Reproduction

checkout https://github.com/taori/Sandbox/tree/netcore3-recompilefails
Debug ComposableWebApplication
Change Home/Index.cshtml in main project
Reload
And then i get the error.
MyFeature/Page1 should also work if changing content.

Screenshots

After digging through the comments in this: https://github.com/aspnet/Blazor/issues/1428 i stumbled into a possible reason. As far as i recall file updates used to work even with debugger attached a while back. Could it be that in order not to break blazor client change updates, mvc auto reload during debugging got cleaved by the changes for blazor?
Speaking about the comment i have seen here: https://github.com/aspnet/Blazor/issues/1428#issuecomment-422459641

It is not a bug, you have to install Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation package. After that you can just call AddRazorRuntimeCompilation() in the ConfigureServices method in the Startup class.
For more details you can check here: https://docs.microsoft.com/en-us/aspnet/core/mvc/views/view-compilation?view=aspnetcore-3.0#runtime-compilation

@u-ashikov Thanks for pointing it out. Looks like i missed some patch notes.
Currently getting "Cannot find compilation library location for package"-messages. I'll see if i can get rid of them somehow. I hope this isn't the same issue as with 2.x because i came around to trying 3.0 in hopes that this is fixed now.

Nope. Still not working - is this just broken on my machine? can someone confirm? Adding repro on issue itself.

@javiercn Yes. Even without debugging i am having this issue.

@mkArtakMSFT Can you forward the attention to #14359 ?
I've noticed that with a fresh standalone created project with no external RCL the feature works - even when i use the debugger. For the Reproduction i posted it is not working in either case.
Technically this should be looked at in the scope of #14359 since this appears to be broken by loading plugins but works in principle. If you agree, close this ticket - i'll add the repro sample on the other ticket.

@taori looks like the other issue was resolved. For your original question @u-ashikov's recommendation is the way to go to get parity with 2.2's compilation.

@pranavkm It wasn't resolved. He just closed it dismissing the actual issue - changing content breaks the application. I would really appreciate your help with this, since it evidently isn't working

@pranavkm @javiercn suggested this might be related to missing reference info so i figured i'd look into your repos for alternate ways to add the application parts in a way that works - i found one which has a similar approach to what i am trying in my repro guide. I managed to reproduce the same issue with your application ported to 3.0
Just thought i'd share that it doesnt seem to be related to the missing *.deps.json or *.runtimeconfig.json in my version since it happens with your ported application as well.

@pranavkm Correct me if i am wrong, but what i figured from reading sources and the stacktrace is:

We pass additional ressources to the RazorReferenceManager using the options parameter, code wise this looks like it should be working. this happens here

The problem is what happens prior to that and makes the recompilation fail, is that earlier in that file an attempt is made to resolve references using AssemblyPartExtensions here which in the end uses this to attempt to resolve the given references.

I suppose this is where a fix would have to be applied in order for this to work since dependent libraries do not have a compositional programming style and will therefore not be capable of handling non system assemblies without major architecture changes.

You probably have more insight in this, but this is what it looks like to me. At least this explains why the application crashs and burns even when a file that isn't even referenced is changed.

Ok - So now i have a solution where razor recompile works while debugging for assemblies loaded through AssemblyLoadContext. See https://github.com/taori/Sandbox/tree/netcore3-razorFixedRecompilePOC
Fortunately because ICompilationReferencesProvider is checked first i was able to forward the reference resolution attempt to my custom application parts instead of letting the CompilationLibrary crash the application.

@pranavkm Sorry if uncalled for PR's are considered rude here. Just wanted to see if it would build. I don't mind if this PR is declined. As mentioned in the PR there are some obvious flaws with it atm + you guys probably prefer to handle delicate parts of the code yourself.

@taori you generally don't want to copy the deps file to your output directory. MVC will attempt to load references from a deps file if it's present next to an assembly that's added as an AssemblyPart. The default reference search path is fairly limited to (https://natemcmaster.com/blog/2017/12/21/netcore-primitives/#depsjson). Not having these files - https://github.com/taori/Sandbox/blob/8ac6378378636fb815245d78f6bea04b7b8bb5df/src/ComposableWebApplication/Plugins/netstandard2.0/ComposableWebApplication.Plugin1.deps.json should resolve your lookup issues.
If you do need them to be present, using the workaround you have and implementing a ICompilationReferencesProvider would be the way to go.

@pranavkm I verified your suggestion - you are correct. Removing the deps files does fix the lookup issues. Are there some negative sideeffects for removing those files? If not i would just have to look if there is a Property which can omit those deps files from the output folders or set up post build tasks to delete them prior to debugging.
Thanks for the article. I will check it out to find out what exactly is happening. In that case i suspect you are inclinded to decline the PR and just document this behavior? Perhaps the exception should specify this as a potential reason for the exception though, so those who run into the issue immediately know how to fix it without digging in documentation.

@pranavkm After reading all 3 pages of the article you shared it appears to me like removing deps.json to make it work seems like an unwise decision. And wrapping AssemblyPart just in order to make it work with plugin seems a bit like code smell, no?

Are there some negative sideeffects for removing those files?

Not particularly. If it's not the entry point, it generally shouldn't mater.

wrapping AssemblyPart just in order to make it work with plugin seems a bit like code smell, no?

We do something like this n the framework, https://github.com/aspnet/AspNetCore/blob/release/3.0/src/Mvc/Mvc/src/MvcServiceCollectionExtensions.cs#L342-L351. It's the intended design for that interface.
A couple more questions to mull about:

You're running in to issues with the deps file because your app uses runtime compilation. Do you require runtime compilation to be enabled in your application?
If you simply referenced your plugin projects in the web application rather than attempt to load them at runtime, these wouldn't be issues. Is there a reason you chose to load them at runtime?


Use utils

What this PR does / why we need it:
this patch finally removes the set hack we had been relying on in favor of the annotation and utils provided by CAPI
Release note:
NONE

depends: #191

/lgtm


TypeError: Cannot read properties of undefined (reading 'push')

Describe the bug
Clicking "now" to change date of post is not working. Same issue occurs if I try to update the date using a different method.
To Reproduce
Visit collection 'legal', document 'User Agreement' and click 'Now'. This crashes the site and shows me this error:
"TypeError: Cannot read properties of undefined (reading 'push') at i.value (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:172:53116) at i.value (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:172:52079) at Fa (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:69876) at Na (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:69671) at gs (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:105470) at ul (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:96717) at sl (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:96642) at Js (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:93672) at https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:188:45314 at t.unstable_runWithPriority (https://unpkg.com/netlify-cms@^2.0.0/dist/netlify-cms.js:196:3844)"
Expected behavior
Date changes to "Now"
Screenshots
Applicable Versions:

Netlify CMS version<EMAIL_ADDRESS>Git provider: git-gateway
Browser version: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/101.0.4951.41 Safari/537.36

CMS configuration
backend:
  name: git-gateway
  repo: jakesing/gatsby-swapstack
  branch: main
media_folder: static/img
public_folder: /img
publish_mode: editorial_workflow
collections:
  - name: blog
    label: Blog
    folder: content/blog
    create: true
    slug: index
    media_folder: ""
    public_folder: ""
    path: "{{title}}/index"
    editor:
      preview: false
    fields:
      - label: Title
        name: title
        widget: string
      - label: Author
        name: author
        widget: relation
        collection: authors
        value_field: email
        display_fields:
          - name
        search_fields:
          - name
          - email
      - label: Publish Date
        name: date
        widget: datetime
      - label: Description
        name: description
        widget: string
      - label: Body
        name: body
        widget: markdown
      - label: Featured Image
        name: thumbnail
        widget: image
    publish: true
    type: folder_based_collection
    sortable_fields:
      - commit_date
      - title
      - date
      - commit_author
      - description
    view_filters: []
    view_groups: []
  - name: authors
    label: Authors
    folder: src/pages/blog/_authors
    media_folder: ""
    public_folder: ""
    format: json
    create: true
    slug: "{{name}}"
    path: "{{name}}/index"
    fields:
      - label: Name
        name: name
        widget: string
      - label: Title
        name: title
        widget: string
      - label: Email
        name: email
        widget: string
      - label: Shortbio
        name: shortbio
        widget: text
      - label: Image
        name: authorimage
        widget: image
      - label: URL
        name: url
        widget: string
    publish: true
    type: folder_based_collection
    sortable_fields:
      - commit_date
      - name
      - name
    view_filters: []
    view_groups: []
  - name: legal
    label: Legal
    folder: content/legal
    media_folder: ""
    public_folder: ""
    create: true
    slug: "{{fields.slug}}"
    fields:
      - label: Title
        name: title
        widget: string
      - label: Publish Date
        name: date
        widget: datetime
      - label: Description
        name: description
        widget: string
      - label: Body
        name: body
        widget: markdown
      - label: Slug
        name: slug
        widget: string
    publish: true
    type: folder_based_collection
    sortable_fields:
      - commit_date
      - title
      - date
      - commit_author
      - description
    view_filters: []
    view_groups: []
slug:
  encoding: unicode
  clean_accents: false
  sanitize_replacement: "-"
isFetching: false
error: null

Additional context

Closing as stale


 For that reason, I'm using your tool to give me a notification pop-up when tests are done, regardless of the results.
The command looks like this: unit-test: npm-run-all actual-unit-test notify.sh --continue-on-error.
This works fine, but sometimes I'm annoyed that I have to check to see if they pass or failed. What I'd like to do is run a chain of commands, and at the end of it change the notification depending on the exit code (e.g. a green notification if it passed, a red on failure). That way, no matter what I'm doing on my computer, I can see at a glance whether I can continue coding or if I need to read what failed.
Because --continue-on-error always returns an exit code of 0, I don't know how to "tell" notify.sh if the tests passed. I'm not a commandline expert, per se, but based on my knowledge, it seems like I can either use your tool so I get notifications regardless of the outcome, OR I stop using npm-run-all and chain my npm scripts with || notify.sh at the end.
This could be done with a bash script, but considering I have npm scripts for acceptance tests that call npm scripts for unit tests -- and some AI driven tests -- I'd rather not have to write that bash script myself.
Full disclosure: I'm extremely exhausted right now. I hope this request makes sense, and I wouldn't be surprised if there's an obvious solution to this problem I'm not seeing. Thanks!

The --continue-on-error documentation suggests it already does what you want:

Set the flag to continue executing
other/subsequent tasks even if a task threw an
error. 'npm-run-all' itself will exit with
non-zero code if one or more tasks threw error(s)

However, I’m seeing a zero exit code even when one of the bundled commands fails, so I’m puzzled 🤔


feat: Add local proxy section, and first lab s3-replication

Add section in labs for local-proxy with the first lab being S3 replication.
Completes FLI-625

@markphelps Yeah we could do that. I was a bit unsure how to go about it however.
Should we provide a script to seed data into Flipt as well, or just let people play around with port-forwarding and getting data into Flipt that way?


mercure: 0.16.3 -> 0.17.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mercure is: Open, easy, fast, reliable and battery-efficient solution for real-time communications
meta.homepage for mercure is: https://github.com/dunglas/mercure
meta.changelog for mercure is: https://github.com/dunglas/mercure/releases/tag/v0.17.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.17.1 with grep in /nix/store/xgqahka97axi2ydc0dr774mjpgl8xns0-mercure-0.17.1
found 0.17.1 in filename of file in /nix/store/xgqahka97axi2ydc0dr774mjpgl8xns0-mercure-0.17.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

mercure

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xgqahka97axi2ydc0dr774mjpgl8xns0-mercure-0.17.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mercure https://github.com/r-ryantm/nixpkgs/archive/403bcc3f2db9cb063ce4fd978e7f273b0e99f5a0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/403bcc3f2db9cb063ce4fd978e7f273b0e99f5a0#mercure

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xgqahka97axi2ydc0dr774mjpgl8xns0-mercure-0.17.1
ls -la /nix/store/xgqahka97axi2ydc0dr774mjpgl8xns0-mercure-0.17.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

mercure

Maintainer pings
cc @gaelreyrol for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


Vectorize the covariance calculation

I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Hi @kemchenj , did you mean to close this PR? Looks like a useful simplification.

Hi.. I'm happy to accept a new PR that fixes it :)
Thanks!


Design a logo

Create a simple and minimalist logo for the website

assign me pls @ANISH0309

Hey @ANISH0309 ,
I would love to design a logo, please assign this issue to me.

I would also like to work on the logo design. Pls assign to me too.

I would like to be assigned as well :)

assign me pls @ANISH0309

Hey, assigning you this issue.

I would like to work on this issue, please do assign me

Hey @ANISH0309 , I would love to design a logo, please assign this issue to me.

Hey @bhaveshgoyal182, are you interested in making the logo?


Prefer to use URI.open and File.open instead of Kernel.open

👋  CodeQL detects "Use of Kernel.open or IO.read with a non-constant value" warnings in https://github.com/ruby/ruby. I hope to suppress them.

I fixed the URI.open ones similarly at #787. Not yet attended unfortunately!

Oh! Thanks to share it.

Should we enable Security/Open in RuboCop configuration file too?

👍 or added CodeQL actions.

That too, just thought of RuboCop since it's already used.

@rafaelfranca done 🚀

Thanks @rafaelfranca ❤


Focus space not working

I've followed the guide to disable system integrity protection and still cannot focus between spaces.
Everything in that file besides focusing different spaces work.

I'm using an M1 Pro 14.2.1 (23C71)
My skhdrc:
# Changing Window Focus

# change window focus within space
cmd - left : yabai -m window --focus west
cmd - right : yabai -m window --focus east
cmd - up : yabai -m window --focus north
cmd - down : yabai -m window --focus south

# change focus between external displays (left and right)
# right (-)
cmd - 0x2C : yabai -m display --focus east
# left (.)
cmd - 0x2F : yabai -m display --focus west

# Modifying The layout

# rotate layout clockwise
shift + cmd - r : yabai -m space --rotate 270

# flip along y-axis
shift + cmd - y : yabai -m space --mirror y-axis

# flip along x-axis
shift + cmd - x : yabai -m space --mirror x-axis

# toggle window float
shift + cmd - f : yabai -m window --toggle float --grid 4:4:1:1:2:2

# Modifying Window Size

# maximize a window
shift + cmd - w : yabai -m window --toggle zoom-fullscreen

# balance out tree of windows (resize to occupy same area)
shift + cmd - e : yabai -m space --balance

# Moving Windows Around

# swap windows
shift + cmd - down  : yabai -m window --swap south
shift + cmd - up    : yabai -m window --swap north
shift + cmd - left  : yabai -m window --swap west
shift + cmd - right : yabai -m window --swap east

ctrl + cmd - down   : yabai -m window --warp south
ctrl + cmd - up     : yabai -m window --warp north
ctrl + cmd - left   : yabai -m window --warp west
ctrl + cmd - right  : yabai -m window --warp east

# move window to display left and right
#right (-)
shift + cmd - 0x2C : yabai -m window --display east; yabai -m display --focus east;
# left (.)
shift + cmd - 0x2F : yabai -m window --display west; yabai -m display --focus west;

# move window to a different space (workspace)
shift + cmd - 1 : yabai -m window --space 1;
shift + cmd - 2 : yabai -m window --space 2;
shift + cmd - 3 : yabai -m window --space 3;
shift + cmd - 4 : yabai -m window --space 4;
shift + cmd - 5 : yabai -m window --space 5;
shift + cmd - 6 : yabai -m window --space 6;
shift + cmd - 7 : yabai -m window --space 7;
shift + cmd - 8 : yabai -m window --space 8;
shift + cmd - 9 : yabai -m window --space 9;

# focus different space (workspace)
cmd - 1 : yabai -m space --focus 1
cmd - 2 : yabai -m space --focus 2
cmd - 3 : yabai -m space --focus 3
cmd - 4 : yabai -m space --focus 4
cmd - 5 : yabai -m space --focus 5
cmd - 6 : yabai -m space --focus 6
cmd - 7 : yabai -m space --focus 7
cmd - 8 : yabai -m space --focus 8
cmd - 9 : yabai -m space --focus 9

# Start/Stopping Yabai

# -- start/stop yabai --
ctrl + shift + cmd - r : yabai --restart-service

Terminal output:
emilpriv@EmilsMBM1Pro ~ % yabai -m space --focus 1
cannot focus space due to an error with the scripting-addition.

Hey i was having a similar issue today
Follow this guide
https://github.com/koekeishiya/yabai/wiki/Installing-yabai-(latest-release)#configure-scripting-addition
also make sure that you uncomment the last line of your sudoers file, because if not then it won't include the new yabai sudoers your create.
it should read something like this
#includedir /private/etc/sudoers.d
after this i restarted my laptop and then it it began to work i can now focus and switch panes like i wanted to.

Hey @0xAquaWolf, thanks. I followed your steps and can confirm yabai is working correctly for me. The last step is important:
    yabai -m signal --add event=dock_did_restart action="sudo yabai --load-sa"
    sudo yabai --load-sa

So far, everything is working perfectly. 👍🏻

@axeII That is awesome to hear, i'm glad i could pay it forward. Yabai is awesome but sometimes its a bit tricky to configure. Cheers!


Use map from c3nav

c3nav is cool and we should use its map on the 34C3. What we know:

c3nav uses Leaflet, so we should be able to use those tiles with the same frontend stuff as we already have for OSM ✓
we will have building levels which probably are map layers - Leaflet can do this and we need to change our building level detection logic for drop points (this probably will get easier as it does not depend on polygon stuff anymore ✓
room and POI data is GeoJSON ✓

Open points:

[x] use the same coordinate system as c3nav so we can use their tiles
[x] create different map layers for levels -1/0/1/2
[ ] filter drop points on map by current level
[x] assign the building level of a drop point according to the current map layer on creation
[ ] changing the map layer should do magic in the drop point creation/edit form
[ ] changing the level in the form should change the map accordingly
[ ] read rooms information dynamically from GeoJSON so we can use rooms from the c3nav API for location autodetection
[ ] export of drop point data as GeoJSON or at least structured display of drop points that makes it easy to add them to c3nav (perfect would be filtering based on last change)
[ ] poll the c3nav API in our API timeout as well to keep the user's c3nav session alive
[x] change web server security header configuration on c3bottl.es to allow tiles from c3nav

[ ] creating in level -1 does not work
[ ] map link in dp modal does not work as it does not include the level

[ ] see what we can or need to do for 35C3 before we close this

Tiles from the 35C3 staging server seem to work just fine, maybe we should have the tile server configurable and feed it into the map via the HTML, so that we can change it on the fly and/or make it configurable by admins.


can't all be hidden

Why can't all be hidden on the WeChat official account page？but chat pages can do  it

Fixed in Commit d5c34b1


e_*_day data from inverter can be misleading close to midnight

Due to the fact that the luxpower inverter time-of-day is inaccurate, if the inverter is fast, it may reset the e_*_day counters before real midnight. This means that lxp-bridge stores 0.0 for the e_*_day data as the last data for a given day.  This is problematic if you're using an influxql  query of the form
select last(e_pv_day) as "Solar Yield" from inputs group by time(1d)

to get a daily summary of the solar yield of the array.
However the more I think about this the more difficult it is to work round properly. It would be much better if the inverter were capable of time-syncing (to something ...) properly ! About the best I can think of is the following pseudo code ...
if ((time is between 23:55:00 and midnight) AND (all e_*_day data is zero))
    set all e_*_day data to NULL

but suspect this may still cause problems at daylight savings switchover. It also assumes that Luxpower HQ will continue to use a 5 mins data collection schedule.
Anyway just wanted you to be aware of this as a potential issue.
-- Chris

Hi!
Thanks for the heads up, yeah this looks like it could be annoying. I have experimented with time syncing to the inverter with some success, but I'd suggest not relying on the _day registers at all; how about something like this instead?
select spread(e_pv_all) from inputs group by time(1d);

This should have the same result I think but without relying on the inverter knowing the time.

Thanks.
Yes that query gets round the problem. I'm new to influx and grafana, so I'm still learning :)
-- Chris


[$250] Adding some text just after the closing backtick ` doesn't quote the text inside the backticks - reported by @adeel0202

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any chat
In the chat input, enter some text inside the backticks to quote it
Enter some text right after the closing backtick (without any space)
Send the message

Expected Result:
The text inside the backticks should be quoted
Actual Result:
The text inside the backticks is not quoted
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.43-2
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/159847891-150d3885-4008-4783-9ca7-0b8f960d8b1b.mov
Expensify/Expensify Issue URL:
Issue reported by: @adeel0202
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1647273215995929
View all open jobs on GitHub

Got some feedback in the thread above, it doesn't appear there's a real reason we shouldn't do this so I'm moving it along

Applied for the reporting bonus.

Proposal
The bug is coming from this regex:
https://github.com/Expensify/expensify-common/blob/f77bb4710c13d01153716df7fb087b637ba3b8bd/lib/ExpensiMark.js#L52
We need to modify it a bit for it to match  &#x60 even when &#x60 is immediately followed by a letter.
So instead of (\B|_) we can have (\B|_|).

  regex: /(\B|_)`(.*?)`(\B|_|)(?![^<]*<\/pre>)/g,

https://user-images.githubusercontent.com/4520893/160849405-d998fee9-1ce1-45cd-9f83-50d573cc467f.mov

Thanks, @brianmuks your proposal looks good!
But entering,
- `a`a  works
-  a`b` doesn't work ( I think we should get this too fixed here)

Can you send an updated proposal to address the above case too? Thanks!
cc: @Beamanator

Thanks, @brianmuks your proposal looks good!
But entering,
- `a`a  works
-  a`b` doesn't work ( I think we should get this too fixed here)

Can you send an updated proposal to address the above case too? Thanks! cc: @Beamanator

Thanks @Santhosh-Sellavel . Let address the  issue .

Updated Proposal
   regex: /(\B|_|)`(.*?)`(\B|_|)(?![^<]*<\/pre>)/g,
https://user-images.githubusercontent.com/4520893/160922589-a6f7fbc9-2ae7-44c9-b86e-dc6404a41222.mov

@Beamanator
https://github.com/Expensify/App/issues/8288#issuecomment-1083162233
https://github.com/Expensify/App/issues/8288#issuecomment-1083580658
@brianmuks  proposal looks good to me.
🎀👀🎀 C+ reviewed

Looks good to me, thanks for the reviews @Santhosh-Sellavel and for the proposals @brianmuks 👍

📣 @brianmuks You have been assigned to this job by @Beamanator!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Hired @adeel0202 as bug reporter
@Santhosh-Sellavel and @brianmuks can you please apply here https://www.upwork.com/jobs/~015a0a72eaba588d69 Thanks

@mallenexpensify Pr is ready :
https://github.com/Expensify/expensify-common/pull/437

Hired @brianmuks in Upwork. @Beamanator is there a reason @Santhosh-Sellavel wasn't assigned to the PR to review?

@mallenexpensify yeah, @Santhosh-Sellavel wasn't automatically assigned b/c we don't have all of the automation set up in the expensify-common repo yet, and also I couldn't manually assign him b/c currently C+ can only be manually assigned in E/App repo, unless they comment on issues / PRs in those other repos. I made an internal issue to work on that / think about it here: https://github.com/Expensify/Expensify/issues/205269

oh... wow, that's new to me! Thanks for raising internally, sounds like something we should try to fix.
Can @Santhosh-Sellavel help in/on the PR at all? I'm planning to compensate them the full C+ price so feel free to ask for help (if it's needed/wanted)

I think it would be great for @Santhosh-Sellavel to check it out, I still can't assign him to review until he posts in the PR though :D
Also @Santhosh-Sellavel can help test the next PR in E/App which needs to be made when the expensify-common PR gets merged 👍

@brianmuks Any update on the Next PR
cc: @Beamanator

@brianmuks just to make sure we're all on the same page, we're expecting you to also create a PR in Expensify/App which updates the version of expensify-common so Expensify/App picks up the changes you made. Let us know if you have questions how to do that

Yes I need help on what is expected in the Expensify/App PR.

@brianmuks In E/App, update package.json to have the expensify-common package point to the latest commit (which has your changes)

Your PR will look like this: https://github.com/Expensify/App/pull/7138/files
The commit hash you will put in package.json (and, after npm install, the commit hash will appear in package-lock.json) can be found here (the most recent commit): https://github.com/Expensify/expensify-common/commits/main

Thanks I am on it!

@mallenexpensify just so you know, @brianmuks 's fix was merged & deployed to staging. It's a bit of a weird situation b/c his fix to expensify-common was merged about a week ago, then while I was having trouble testing, it was merged into E/App in this PR: https://github.com/Expensify/App/pull/8743
As you can see on that PR, it just got deployed to staging yesterday (here: https://github.com/Expensify/App/pull/8743#issuecomment-1109144387)
FYI some of our automation may not work so well b/c I closed brian's latest PR because his changes to E/App were already made in another issue, but he still deserves full payment whenever the changes make it to production 👍

So.... @Beamanator tbc... we're waiting for https://github.com/Expensify/App/pull/8743 to hit production, then waiting 7 days after than, right?

Yes exactly 👨🍳 👍 💯

https://github.com/Expensify/App/pull/8743 on staging...

Hit staging 16 days ago, not sure what's taking so long ¯_(ツ)_/¯

PR is on production, updated title to pay on 5/20 if no regressions

@mallenexpensify, a friendly reminder 😄

@adeel0202 paid $250 for reporting.  Thanks for the ping, not sure how i missed this on Friday.
@Beamanator can you confirm that @brianmuks is due $250?   Assuming @Santhosh-Sellavel is the same amount for C+ as Brian too

Looks like @brianmuks was already paid $250, let me know if you're not seeing Brian.
Hired @Santhosh-Sellavel , please let me know once you've accepted
https://www.upwork.com/jobs/~015a0a72eaba588d69

@mallenexpensify
Thanks, Accepted!

Paid @Santhosh-Sellavel $250, everyone should be paid now! Comment if needed.


sv.json

Added a swedish translation to txadmin

@kaviwi555 Can you do this changes for me please?

Closed due to inactivity.


add: tdrop-git

Progress

[x] Edit packagelist
[x] Add initial pacscript
[x] Contact devs
[x] Add maintainer to pacscript

Applogies for the force push - LazyGit just started doing it before I could respond. Anyway thats the commas removed.

Great! Can you pull from master so the CI will start working?

Is that what you wanted ?

I meant from the GitHub interface 👀

Do you want me throw this out and start again ? This branches history is now pretty messed up. I can squash it all down ?

Closing this due to some git errors on my part - will open another pr to keep things clean.


Move the icon style color to its own example

Per discussion on #4457, move the example of the icon colour to its own example and restore the previous icon example

@alexbrault Can you please rebase your code on master?

And I propose to remove all the popup code in this example

Thanks @alexbrault


Does it contain profanity?

Can we ensure that the generated string is SFW?

I can't make any hard guarantees since I didn't build the word lists myself (they come from NLTK), but there have been many pull requests to remove NSFW words and phrases, erring on the side of caution.
As a point of confidence, this library has been in use by vmpooler from Puppet for some time. They've contributed the most to cleaning up the word lists.
If you want to be extra sure, you can find a list of NSFW words and compare it against the plaintext word lists that this library uses: https://github.com/schmich/spicy-proton/tree/master/corpus


link to docs and incorporate roman/gerard feedback

Incorporate feedback from 2022-10-31 Slack discussion:

Link to GitHub Discussions in README to make it easier for people who aren't in the Locally Optimistic Slack to get in touch.
Link to GitHub Issues in README to make it easier for people to contribute.

Update links from readthedocs to Datafold Docs in cases where the content is available in Datafold Docs and not autogenerated from python docstrings.

Note: we should remove the same content from readthedocs so that it doesn't have to be maintained in two locations.

Clean up the formatting and wording a bit throughout the README.

thank you @erezsh 🙌


intel-media-driver: 21.3.5 -> 21.4.1

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

1 package built:

intel-media-driver


DNS binding

Hi,
I'm trying to run this service but i see that the DNS that will start into the container, it's binded to his internal ip address, so, it's not possible to have a DNS service exposed to the network.
With this limitation, it's not possible to configure a container/virtualization host on network with services published via domain names.
As you can see from netstat, service is binded to internal container ip address:
udp        0      0 $_container_ip_:53            <IP_ADDRESS>:*                           1654/conmon

that is not reachable from network, also with publish costraint
I've also tried to not specify binding address, but i get error:
Error: cannot listen on the UDP port: listen udp4 :53: bind: address already in use

because there are other services running that are related to libvirt

This is not intended to be used with --net host. So at you said, the internal dnsmasq is binding port 53 on it's own IP, then you need to expose port 53 using --publish.
But if you already have a DNS service running on your host, the --publish may fail because port is already in use by another DNS service (like the dnsmasq started by libvirt or NetworkManager)

What is your OS ?
Do you have NetworkManager running ?
If you run that command on your host, could you paste the output here ? sudo netstat -plunt | grep -w 53

Hi,

I'm running it on Centos 8 with libvirt and podman
yes, NetworkManager is running and is mounted as ro volume on container
this is output of command:

tcp        0      0 $_virbr0_address_:53        <IP_ADDRESS>:*               LISTEN      1090/dnsmasq
udp        0      0 $_dalidock_container_ip_:53            <IP_ADDRESS>:*                           1199/podman
udp        0      0 $_virbr0_address_:53        <IP_ADDRESS>:*                           1090/dnsmasq

I don't understand why podman is listenning on $_dalidock_container_ip_, it should be the IP address of your network bridge. Maybe this is specific to podman.

it's the same configuration about docker, also on those container management system, you will get an ip address relative to docker subnet.
If i run
podman run \
  --name dalidock \
  --net host \
  --cap-add NET_ADMIN \
  --publish $_LIBVIRT_HOST_:53:53/udp \
  --publish 80:80 \
  --env DNS_DOMAIN=my.local.env \
  --env LB_DOMAIN=my.local.env \
  --volume /run/NetworkManager:/run/NetworkManager:ro \
  --volume /var/run/libvirt:/var/run/libvirt:ro \
  lionelnicolas/dalidock

and i disable the internal dnmasq used for NAT subnet I can see virtual machine that were detected.
dalidock[14]: [INFO]  wait for domain test QEMU guest agent to reply
dalidock[14]: [INFO]  name=test       hostname=test       ip=None            net=br0        domain=my.local.env use_wildcard=False

At this point, it will be already reachable with $_hostname_.$_domain:, In this case test.my.local.env? Metadata are necessary always or just for setup custom DNS/LB entry?

Yes, metadata and labels are only needed for custom DNS/LB.
So in that case, all these commands should return the correct IP:
# using `host` from `bind-utils` package on RedHat-like or `bind9-host` on Debian-like

host test ${LIBVIRT_HOST}

host test.my.local.env ${LIBVIRT_HOST}

Or using dig :
dig @${LIBVIRT_HOST} test.my.local.env

I see that you have defined a qemu guest agent in your VM config (wait for domain test QEMU guest agent to reply). If the guest agent is not running inside the VM, dalidock will timeout when trying to get the IP address. If it timeouts, then no DNS record will be created as there is no known IP to associate. You can customize the timeout by adding :
    --env LIBVIRT_IP_TIMEOUT=120

If you want to make dalidock use libvirt DHCP leases instead of the guest agent, you'll need to remove the org.qemu.guest_agent.0 channel from the VM config.

Yes, I always use qemu guest agent, but virtual machine was a simple test that was booting, so, after that I need to configure it and so on. I think that could be fail while I work to configure it, after a reboot, it will works properly, but timeout option can be very useful.
I will try to configure and running some tests, also changing domain and using LB domain.

Ok !
I'm thinking about adding a fallback to DHCP lease discovery if the guest agent fails, that would help in your case.


Unable to use hashHistory with fractal structure

Hi,
I did a build for my project and now I am trying to get it to run as a standalone file( not served ).
By standalone I mean expecting it to work with fileURL like file:///Users/username/project/dist/index.html.
This is not working, I get a blank page.
The solution to this problem seems like using hashHistory instead of browserHistory.
I have tried replacing all borswerHistory to hashHistory but that doesnt seem to work. Please help me use hashHistory with this kit

I could be wrong (without testing it first) but this could have to do with the way most browsers handle file:/// URLs. Have you tried using something like http-server by running http-server dist/ and see if hashHistory works then?
Or is your use case required to serve up via file:///?

My setup requires the files to be served via file:///
replacing browserHistory with hashHistory actually worked, but the problem was the since my public path was / , the js and css files were loaded as file://scqeefs4532334.js as opposed to file:///<path>/<hash>.js
This is solved.
Solution:

Replace all browserHistory to hashHistory
change publicPath in config file from / to ./


Unable to send SNS with MessageAttribute in InvokingBindingAsync()

We need to send sns notification with message attribute also on daprClient.InvokingBindingAsync(), to filter sns message based on message attribute filter policy subscription. could you please help.

Could you please confirm message attribute added on daprClient.InvokingBindingAsync() method Request. Like this
Request:
request.MessageAttributes.Add(
"SNSType",
new MessageAttributeValue()
{
Type = "String",
Value = "SNS"
});
Response:
For the Subscription filter policy. Like this on AWS
{
"SNSType": [
"Type ":"String"
"Value":" SNS"
]
}
Could you please update asap. Thanks

The InvokeBinding call just takes your provided object/data (serialized to json if an object) and sends it to the component. So if you're encoding that into your request object, it should be fine.
If you're asking if the BindingRequest object has a MessageAttributes field. It does not. Bindings are generic and we don't have anything tech specific in the object.

@SPrabaga - Do you need anything else here?

I'm going to close this issue, feel free to reopen if you need more help.


Exception in thread "main" java.lang.ClassCastException: Cannot cast net.minecraftforge.fml.loading.moddiscovery.AbstractModProvider$DefaultModFileInfo to net.minecraftforge.fml.loading.moddiscovery.ModFileInfo

latest.log

that doesnt give any good info

try updating mods and modloader since its not pointing anywhere about unify

i am on the latest version of forge and the issue only starts when unify is updated to the release version

could you provide more info please

unfortunately i cant give anything more other than the mod list as the game refuses to load past that point with the latest version of the mod
modlist.txt

try it with unify only first and if it works then its a conflict with a mod and you gotta find the conflicted mod


Tests

Ping @Evelf

We now have 100% coverage including:

98% coverage from unit and integration (postgres) test
93% coverage from acceptance test

That's what I'd call a good start !


MGMT-15284: Update the must-gather image

Stop collecting Job objects.
Use a better method of listing namespaces of interest.
Do not collect unnecessary data in KMM-Hub deployments.

/lgtm


enable_http2 does not produce target_groups using http2

Versions

Terraform: v0.14.6

Provider(s):

+ provider registry.terraform.io/hashicorp/aws v3.38.0
+ provider registry.terraform.io/hashicorp/template v2.2.0
+ provider registry.terraform.io/microsoft/azuredevops v0.1.4

Module: terraform-aws-alb

Reproduction

Steps to reproduce the behavior: Use enable_http2 parameter and set it to true (the default)

Code Snippet to Reproduce

  source = "terraform-aws-modules/alb/aws"
  name   = "example-stage-api"

  load_balancer_type = "application"

  vpc_id          = data.terraform_remote_state.example_stage_networking.outputs.vpc_id
  subnets         = data.terraform_remote_state.example_stage_networking.outputs.public_subnets
  security_groups = [aws_security_group.example_stage_api.id]

  access_logs = {
    bucket = aws_s3_bucket.api_logs.id
  }
  enable_http2      = true
  target_groups = [
    {
      name_prefix      = "ex-"
      backend_protocol = "HTTP"
      backend_port     = 5000
      target_type      = "instance"
    }
  ]

  https_listeners = [
    {
      port            = 443
      protocol        = "HTTPS"
      certificate_arn = data.aws_acm_certificate.wildcard.arn
    }
  ]

  http_tcp_listeners = [
    {
      port        = 80
      protocol    = "HTTP"
      action_type = "redirect"
      redirect = {
        port        = "443"
        protocol    = "HTTPS"
        status_code = "HTTP_301"
      }
    }
  ]

  tags = {
    project   = "example"
    lifecycle = var.life_cycle
  }
}

Expected behavior

target groups created with HTTP2 as protocol. docs
Actual behavior

Note the resulting target_group has protocol_version                   = "HTTP1"
$ terraform state show module.alb.aws_lb_target_group.main[0]
# module.alb.aws_lb_target_group.main[0]:
resource "aws_lb_target_group" "main" {
    arn                                = "arn:aws:elasticloadbalancing:us-east-1:<>:targetgroup/ex-20210428194122674900000001/3a9ab6d7cca82a25"
    arn_suffix                         = "targetgroup/ex-20210428194122674900000001/3a9ab6d7cca82a25"
    deregistration_delay               = 300
    id                                 = "arn:aws:elasticloadbalancing:us-east-1:<>:targetgroup/ex-20210428194122674900000001/3a9ab6d7cca82a25"
    lambda_multi_value_headers_enabled = false
    load_balancing_algorithm_type      = "round_robin"
    name                               = "ex-20210428194122674900000001"
    name_prefix                        = "ex-"
    port                               = 5000
    protocol                           = "HTTP"
    protocol_version                   = "HTTP1"
    proxy_protocol_v2                  = false
    slow_start                         = 0
    tags                               = {
        "Name"      = "ex-"
        "lifecycle" = "stage"
        "project"   = "example"
    }
    target_type                        = "instance"
    vpc_id                             = "vpc-<>"

    health_check {
        enabled             = true
        healthy_threshold   = 2
        interval            = 5
        matcher             = "200"
        path                = "/"
        port                = "traffic-port"
        protocol            = "HTTP"
        timeout             = 4
        unhealthy_threshold = 10
    }

    stickiness {
        cookie_duration = 86400
        enabled         = false
        type            = "lb_cookie"
    }
}

Terminal Output Screenshot(s)

Additional context

This behavior makes sense because this variable isn't used in the code from my observation. (Screenshot of search below)

I am using the below code for adding  HTTP2 protocol version still its taking default HTTP1.
tried with alb module version 5.9.0/5.13.0 provider 3.7.0 .
Terraform: v0.12.4
required output is same as mentioned by @tmegow
@antonbabenko  didn't work for me. Thanks.
target_groups             = [
{
name_prefix           = "xx-x"
backend_protocol      = "HTTP"
backend_port          = 15600
target_type           = "ip"
protocol_version = "HTTP2"
health_check = {
enabled             = true
interval            = 300
path                = "/"
port                = "traffic-port"
healthy_threshold   = 5
unhealthy_threshold = 5
timeout             = 25
protocol            = "HTTP"
matcher             = "200-399"
}
stickiness = {
type = "lb_cookie"
cookie_duration = 300
enabled = true
}
}


Fixed returning only one obj when put

https://nanopay.atlassian.net/browse/CPF-142
When put array Dig data, only one data returned as a returning data.
Fixed this issue by adding Object[] type on json/Outputter to parse.

it is on wrong branch