Unable to deploy the resources

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful
################################################
################################################

Unreal Engine and Azure Digital Twins
integration demo

################################################
################################################
Welcome to the Unreal Engine and Azure Digital Twins (ADT) integration demo. This deployment script will help you deploy a sandbox environment in your Azure subscription. This demo leverages the ADT Link plugin, that was created along with this sample demo by WSP in collaboration with Microsoft and Epic Games, in order to demonstrate how to integrate Azure Digital Twins with the Unreal Engine.
Press Enter to continue.
Verifying your Azure CLI installation version...
Great! You are using a supported Azure CLI version.
Verifying 'azure-iot' extension version...
Great! You are using a supported version of the extension 'azure-iot'.
Retrieving your current Azure subscription...
You are currently using the Azure subscription 'Azure subscription 1'. Do you want to keep using it?
1: Yes
2: No. I want to use a different subscription

1: 1

Registering ADT resource provider in your subscription
Choose a region for your deployment from this list (using its Index):
1: Australia East
2: East US
3: East US 2
4: North Europe
5: South Central US
6: Southeast Asia
7: UK South
8: West Europe
9: West US 2

: 2

Provide a name that describes your project. This will be used to create the resource group and the deployment resources.

: demo3

Resource group 'demo3-rg' already exists in current subscription.
Collecting current user information
Creating app registration manifest
Creating app registration 'demo3-rg-8f63ae8c' in Azure Active Directory
ERROR: unrecognized arguments: --available-to-other-tenants False --reply-urls http://localhost --native-app
Examples from AI knowledge base:
az ad app create --display-name mytestapp --optional-claims @manifest.json
Create an application with optional claims
az ad app create --display-name mytestapp --identifier-uris https://mytestapp.websites.net --app-roles @manifest.json
Create an application with a role
https://docs.microsoft.com/en-US/cli/azure/ad/app#az_ad_app_create
Read more about the command in reference docs
ERROR: argument --id: expected one argument
Examples from AI knowledge base:
az ad app show --id 00000000-0000-0000-0000-000000000000
Get the details of an application. (autogenerated)
https://docs.microsoft.com/en-US/cli/azure/ad/app#az_ad_app_show
Read more about the command in reference docs
ERROR: argument --id: expected one argument
Examples from AI knowledge base:
az ad app show --id 00000000-0000-0000-0000-000000000000
Get the details of an application. (autogenerated)
https://docs.microsoft.com/en-US/cli/azure/ad/app#az_ad_app_show
Read more about the command in reference docs
ERROR: argument --id: expected one argument
Examples from AI knowledge base:
az ad app show --id 00000000-0000-0000-0000-000000000000
Get the details of an application. (autogenerated)
https://docs.microsoft.com/en-US/cli/azure/ad/app#az_ad_app_show
Read more about the command in reference docs
Creating service principal associated with the app registration
ERROR: argument --id: expected one argument
Examples from AI knowledge base:
az ad sp create --id 00000000-0000-0000-0000-000000000000
Create a service principal. (autogenerated)
https://docs.microsoft.com/en-US/cli/azure/ad/sp#az_ad_sp_create
Read more about the command in reference docs
Creating client secret for app registration
ERROR: argument --id: expected one argument
Examples from AI knowledge base:
az ad app credential reset --id 00000000-0000-0000-0000-000000000000
append or overwrite an application's password or certificate credentials (autogenerated)
az ad app credential reset --id 00000000-0000-0000-0000-000000000000 --cert<EMAIL_ADDRESS>Append a certificate to the application with the certificate file.
https://docs.microsoft.com/en-US/cli/azure/ad/app/credential#az_ad_app_credential_reset
Read more about the command in reference docs
Creating resource group deployment with id 'demo3-8f63ae8c'
WARNING: /home/azure-digital-twins-unreal-integration/deployment/azuredeploy.bicep(13,16) : Warning no-loc-expr-outside-params: Use a parameter here instead of 'resourceGroup().location'. 'resourceGroup().location' and 'deployment().location' should only be used as a default value for parameters. [https://aka.ms/bicep/linter/no-loc-expr-outside-params]
/home/azure-digital-twins-unreal-integration/deployment/azuredeploy.bicep(93,5) : Warning no-unnecessary-dependson: Remove unnecessary dependsOn entry 'identity'. [https://aka.ms/bicep/linter/no-unnecessary-dependson]
/home/azure-digital-twins-unreal-integration/deployment/azuredeploy.bicep(330,105) : Warning BCP241: The "providers" function is deprecated and will be removed in a future release of Bicep.  Found possible secret: secure parameter 'appRegPassword' [https://aka.ms/bicep/linter/outputs-should-not-contain-secrets]
ERROR: {"code": "InvalidDeploymentParameterValue", "message": "The value of deployment parameter 'tenantId' is null. Please specify the value or use the parameter reference. See https://aka.ms/resource-manager-parameter-files for details."}
Exception: /home/azure-digital-twins-unreal-integration/deployment/deploy.ps1:456
Line |
456 |          throw "Something went wrong with the resource group deploymen …
|          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Something went wrong with the resource group deployment. Ending script.

Thanks! We'll be in touch soon.

@Ron-Dong try rerunning with the ../deploy.ps1 in my pull request #14

Hey There,
I have recently made some major changes fixing up a few bugs and issues related to this repo. Feel free to try this again and see if you get the expected outputs.
If there's no reply in 1 week, I'll close off this issue. But feel free to open a new one and I'll get to it ASAP.


Use PT-Run to copy special characters to the clipboard : alt + space → *mu → μ

Description of the new feature / enhancement
Like many other users, I take daily notes on which I sometimes use scientific characters.
⁉ My problem is that I can't quickly access the special characters.
There are alternatives such as :

windows + . but you have to go to symbols and then scroll through an infinite list, which takes far too long.
Alt code, but nobody's going to memorize 50 codes to be able to quickly write a text.

💡 Proposed solution
Julia's Tab completion sequences greatly simplify the use of special characters, and it's a similar solution that I'd like to be able to use in PowerToys.
For those who don't want to go to the link, here is a short extract from the list :

Code point
Character
Tab completion sequence
Unicode name

U+003B1
α
\alpha
Greek Small Letter Alpha

U+003B2
β
\beta
Greek Small Letter Beta

U+003B4
δ
\delta
Greek Small Letter Delta

ℹ For the rest of the explanation I'll use the word keyword to refer to the equivalent of tab completion sequence in Julia.
→ Users could quickly put the character into their clipboard by typing the asterisk character * (a quick-access character that fits with the need for quick correspondence) and then the keyword of their character.

I don't know if it's possible to add it directly to my text, so that's why I'm proposing clipboard, which will still be faster than other current solutions.

⭐ Multi keywords
Using several keywords could also simplify the approach. For example, to display ÷, we could use the following keywords: div, division, divide.
I wanted to clarify that some keywords can be improved as for example (the bold keywords are those that I suggest) :

Code point
Character
Asterisk completion sequence
Unicode name

U+02260
≠
*ne, *neq, *!=
Not Equal To

U+02261
≡
*equiv, *==
Identical To

U+02263
≣
*Equiv, *===
Strictly Equivalent To

U+02264
≤
*le, *leq, *<=
Less-Than Or Equal To / Less Than Or Equal To

U+02265
≥
*ge, *geq, *>=
Greater-Than Or Equal To / Greater Than Or Equal To

U+02266
≦
*leqq, *<==
Less-Than Over Equal To / Less Than Over Equal To

Scenario when this would be used?
👉 I want to write the ≠ character quickly.
🪀 I press alt + space to open PowerToys Run and then enter the following string: *!=
✅ I press the enter key to copy the character to my clipboard

Do you know Quick Accent? It's a utility specifically for such use cases: https://learn.microsoft.com/windows/powertoys/quick-accent?WT.mc_id=studentamb_274787

Thank you for notifying me, I had forgotten this solution, I will edit my post.
I stopped using it because I can't find all the characters if I want to for example ⊂ I have no idea what to type as a base character


[ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: FormatException: Invalid date format E/flutter (22344): 01:00:00.000

I am getting this error when using this format i want to convert it to time

Could you reopen this issue with more details such as the code which produced this error?


Feature: automatically add closed issue to backlog and milestone

When an existing issue is automatically closed due to a commit, add it to the current backlog and milestone.
"Current" refers to the smallest milestone greater than the latest release.

Any idea about knowing if the issue is closed by commit?
It's discriminated on the web but there seems no relating events or data.

Do you know what keywords does GitHub use ?

close #xxx
closes #xxx
closed #xxx
fix #xxx
fixes #xxx
fixed #xxx

okay, do it tomorrow
how about merged pull request ?

push event is fine since it carries all the new commit object

Koishi uses squash merge so I don't worry about PRs. 🤣
Also I believe this feature should be optional since others may not want it. Maybe you can add a config, such as commit prefix or commit regexp.


if the user sets "Block sites from setting ant data" in preferences -> Cokies section on Opera 44, hola player doesn't load properly.

My test url is http://hola.github.io/examples/cdn/#hola_player_ima
The player try to read data from localStorage object.
Full message that given by console:
videojs5-hlsjs-source-handler.js:4Uncaught DOMException: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.
    at Object.d.88 (http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:12:637)
    at e (http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:560)
    at http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:611
    at<EMAIL_ADDRESS>(http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:823)
    at e (http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:560)
    at http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:611
    at Object.d.247../css/dvr.css (http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:25:11005)
    at e (http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:560)
    at a (http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:726)
    at http://cdn.jsdelivr.net/hola_player/1.0.24/hola_player.js?customer=demo:1:743
d.88 @ videojs5-hlsjs-source-handler.js:4
e @ _prelude.js:1
(anonymous) @ _prelude.js:1
<EMAIL_ADDRESS>@ hola_videojs_hls.js:2
e @ _prelude.js:1
(anonymous) @ _prelude.js:1
d.247../css/dvr.css @ hola_player.js:7
e @ _prelude.js:1
a @ _prelude.js:1
(anonymous) @ _prelude.js:1
(anonymous) @ _prelude.js:1
(anonymous) @ _prelude.js:1
VM64:7 Uncaught TypeError: window.hola_player is not a function
    at VM64:7

Thanks.

fixed in 1.0.26

now it works, thanks.


Rename 'apply' to 'applied', Update Ui.png

Codecov Report
Base: 70.42% // Head: 70.42% // No change to project coverage :thumbsup:

Coverage data is based on head (fdc3e85) compared to base (9be2b55).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #131   +/-   ##
=========================================
  Coverage     70.42%   70.42%
  Complexity      641      641
=========================================
  Files           100      100
  Lines          2005     2005
  Branches        222      222
=========================================
  Hits           1412     1412
  Misses          524      524
  Partials         69       69

Impacted Files
Coverage Δ

...main/java/seedu/phu/logic/commands/AddCommand.java
100.00% <ø> (ø)

...main/java/seedu/phu/model/util/SampleDataUtil.java
100.00% <ø> (ø)

src/main/java/seedu/phu/ui/StackedBarPanel.java
0.00% <0.00%> (ø)

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

...seedu/phu/model/internship/ApplicationProcess.java
47.61% <100.00%> (ø)

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

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


Reporting - generate list of 001s in MARC file

User request: Could we generate and email a list of 001s found in the MARC record before loading?

Duplicate of #636


Add to dbset but not create a table for it

Issue: a table named for my stored procedure is created because I am forced to register it in the dbset. If I add builder.ignore then I can't invoke the sp.
I have a stored procedure that I am invoking using the following syntax.
var data = dbCtx.Set<RoleAdminGridViewModel>().FromSql($"[dbo].[uspGetRoleAdminGridViewModelSp]").ToArray();

Since I have to add it to my dbCtx it creates a table called RoleAdminGridViewModel
public DbSet<RoleAdminGridViewModel> RoleAdminGridViewModel { get; set; }

public class RoleAdminGridViewModel
{
  [Key] // I am also not happy that I have to declare a primary key here. Now I always have to use
// @rownumber in my stored procs.
  int PrimaryKey
}

My work around is to drop the table and name the procedure something different.
CREATE PROCEDURE uspGetRoleAdminGridViewModelSp
SELECT 1 As Primarykey

I think you want the feature we are already tracking with https://github.com/aspnet/EntityFramework/issues/1862

I think there is a subtly different scenario from what we track in #1862 in which you actually do want a type to become an entity in the model but you know that all your queries will be sourced using some FromSql() and probably your own custom SQL or stored procedures for CUD (which at some point EF Core should support). So basically you want Migrations and EnsureCreated() to not create the table but you still want to be able to do things like change tracking.
I believe we have items tracking this, but I cannot find them at the moment :smile:

I found this one that I believe is related to that scenario:

Support for defining query #3932

I think we had another issue about just ignoring the table for Migrations and EnsureCreated() but that one I cannot find.

@divega either way I think we have this one covered. But the sample code shows a ViewModel type with no PK etc. so I don't think you would actually want this to be part of the model.
BTW the issue about ignoring tables from migration is #2725

Ok, I agree this one is probably a duplicate of #1862 then. Glad we have the other ones as well :smile:

Think we have this covered by the other issues already mentioned


Update Composer dependencies (2019-10-22-00-30)

Gathering patches for root package.
Loading composer repositories with package information
Updating dependencies
Package operations: 0 installs, 1 update, 0 removals
Gathering patches for root package.
Gathering patches for dependencies. This might take a minute.
  - Updating commerceguys/addressing (v1.0.5 => v1.0.6): Loading from cache
Writing lock file
Generating optimized autoload files
> DrupalProject\composer\ScriptHandler::createRequiredFiles

Visual regression test passed!
View the visual regression test report


Add generic environment runner

Motivation
Support arbitrary environment setups for E2E and testing, not just Docker

Codecov Report

Merging #2342 into master will decrease coverage by 9.77%.
The diff coverage is 58.97%.

@@            Coverage Diff             @@
##           master    #2342      +/-   ##
==========================================
- Coverage   85.71%   75.93%   -9.78%
==========================================
  Files         199       20     -179
  Lines       13833      561   -13272
  Branches     1525       59    -1466
==========================================
- Hits        11857      426   -11431
+ Misses       1540      110    -1430
+ Partials      436       25     -411


Crimson moon failed to load after the 1st arrival

On my 1.15.2 server the crimson moon can work as expected for the first arrival. However, starting from the second arrival, only a message saying "The crimson moon has arrived. " will be sent, and the rest of the features lose effects or perform inaccurately. The red texture of the moon, atmosphere, or rain will only maintain for an instant and then disappear lightning-fast, representing as a "red flash" on the screen. Besides, the crimson moon will arrive at any time instead of about 18:00 or later, for instance I met one at 14:30 under bright sunshine.
For the config, I reduced mobs per chunk to 2, turned on red moon texture, modify the chance to 15, and disabled usage of bed (But I'm not sure whether those changes lead to this issue, cuz if I want to test it I'll need to wait about 20 game days for the crimson moon to arrive by default settings. )
Anyway I love the concept of this mod as it adds some very attractive themes and challenges that enrich my game experience, so thx for this mod and hope to see this issue being fixed!

Met the same problem in 1.16.1, we were fed up this situation so I decided not to use it (at the state of crimson moon). However, after this, the time of our world stopped and jumped 20 days per tick. Here's the video I've recorded.
2020-08-03_23-12-56.zip

Took me a while to figure out the root cause of the issue, but it should be fixed in 1.2.0 (this is a 1.16.1 version).
I had originally assumed "day time" was 0 -> 24000, and made all my calculations around that (assuming day time is never >24000). Turns out day time doesn't have a limit, so the timing gets worse and worse as days go on. I am now calculating the actual day time from the world time, and disable the Crimson Moon while Daylight Cycle is off.
Let me know if you encounter any other issues related to this! Thanks for the reports.

I finally found out what cause the problem in my server!! It's the mod SleepVote!! I changed the gamerule /gamerule sleepPercentageRequired from 0.33 to 0 that SleepVote provided. Here's the game log in my game log:
[pool-4-thread-2/ERROR]: Couldn't get pending invite count
net.minecraft.class_4355: Could not connect to Realms: Connect timed out
at net.minecraft.class_4341.method_20998(class_4341.java:440) ~[intermediary-minecraft-1.16.1-client.jar:?]
at net.minecraft.class_4341.method_21029(class_4341.java:318) ~[intermediary-minecraft-1.16.1-client.jar:?]
at net.minecraft.class_4360$class_4362.method_21103(class_4360.java:235) [intermediary-minecraft-1.16.1-client.jar:?]
at net.minecraft.class_4360$class_4362.run(class_4360.java:228) [intermediary-minecraft-1.16.1-client.jar:?]
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) [?:?]
at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:305) [?:?]
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:305) [?:?]
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
at java.lang.Thread.run(Thread.java:832) [?:?]
When I changed the percent, then messages came one by one every 2 secs.
Ahhh I'm so happy to find out such an annoying problem!!


Why can not I set channel?

I set the channel to 9 for percussion midi, but it is not set.
i try it
new MidiWriter.NoteEvent({pitch:"C1", duration: '16', velocity:100, channel: 9});
and export midi file.

---- FILE START ----
---- FILE HEADER ----
format:        0
tracks:        1
time division: 128
---- TRACK 1 (2187 bytes) ----
delta: 0
Set Tempo: 130 BPM
delta: 0
Note On (channel 0): C1 with velocity 127

Hi @drumtj,
Thanks for your message.  It looks like that's something I haven't implemented yet, guess nobody has tried using it.  I will work on adding this as soon as I get a chance, thanks.
-Garrett

Hi @drumtj,
I just released 1.7.1 which should fix this issue.  Let me know if you have any issues,
-Garrett

Hi @drumtj,
I just released 1.7.1 which should fix this issue. Let me know if you have any issues,
-Garrett

hi
thank you for quick comment.
but i found a problem.
a decremented value of 1 is entered to channel,,  ;)

Hi @drumtj,
Sorry, I don't fully understand.  Sounds like it's not working as expected for you?
-Garrett

Hi @drumtj,
Sorry, I don't fully understand. Sounds like it's not working as expected for you?
-Garrett

hi
I set the channel to 9. However, in the midi file, I confirmed that the channel was set to 8.
---- TRACK 2 (2190 bytes) ----
delta: 0
Set Tempo: 130 BPM
delta: 0
Program Change (channel 0): Piano
delta: 0
Note On (channel 8): C1 with velocity 127
delta: 32
Note Off (channel 8): C1 with velocity 127

Hi @drumtj,
Seems to be working correctly on my end.  Can you post the code you're using?  Also what software are you using to read the file?
-Garrett

Hi @drumtj,
Seems to be working correctly on my end. Can you post the code you're using? Also what software are you using to read the file?
-Garrett

Hi
I understood it while creating a test project.
To use a percussion part in a MIDI file, you must set the channel to 10.
When I analyzed the MIDI file, I was confused because the channel value was 0-based.
Check this test link and you'll know.
My curiosity has been resolved.
Thank you for your reply.
https://drumtj.github.io/midiWriteTest/dist/index.html


Edited manage subscriptions article.

  This is the expected behavior of the build system.

  This is the expected behavior of the build system.

  This is the expected behavior of the build system.

OPS Build status updates of commit 9816b65:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-1.2.0/manage-subscriptions-azureps.md
:white_check_mark:Succeeded
View (azps-1.2.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

  This is the expected behavior of the build system.

OPS Build status updates of commit 98ae1a5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-1.2.0/manage-subscriptions-azureps.md
:white_check_mark:Succeeded
View (azps-1.2.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Redesign tiers

We should have multiple types of tiers:

tickets: name, description, amount, quantity (the current design we have in events is fine for it, e.g. https://opencollective.com/sustainoss/events/2017)
product: name, description, fixed amount (e.g. become a sponsor for that event for $500)
donation: name, description, custom amount (choose amount and frequency). We really like what Women Who Code has on their website for that, see https://womenwhocode.com/donate

I think that would be a good start to have a constant component to handle those 3 different use cases.

Hi @xdamman, here's a first attempt at redesigning the tiers. I modified them in order to make them a same component with variants, but generally the same structure.

Tier type: Ticket
Tier type: Product
Tier type: Donation

Zeplin link: https://zpl.io/Z1EyAaz
Let me know your thoughts/feedback.

Hey @xdamman, for this you asked me to do the edit versions and that they should be inline. Can you provide the fields needed? Some are obvious but I'm not sure how many options will the user have. Thx

Hey again, I did this example for the tickets tier, based on the API that @asood123 shared with me. Let me know if this is the right path for this and if the fields are correct.


build: prepare for next release

changes

prepare for 0.2 release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
512

Relevant Lines:
584

💛  - Coveralls


Derive PartialEq for Error

This helps when wrapping this crate's Error struct in another crate. Not sure if you'd like a test to cover this too, but if so just let me know.

I don't know if comparing the string description of the underlying error is the best idea.

Is there an approach that you prefer that I can help with? I guess implemeting PartialEq is also a way forward. Right now I'm struggling to wrap this crate's error type and wondering how to move forward without resorting to converting the error to a String.

Why do you need to compare your wrapped error for equality? There are a lot of error types in the wild that cannot do that (e.g. io::Error, Box<dyn Error>, ...).

Yeah I know the problem with io::Error exists too. I want to use assert_eq! when writing library tests to demonstrate that when the lib is used in a certain way, a particular type of error occurs, not just any error. If even one of the wrapped error types can't be compared, the whole enum can't be compared and thus the whole Result<_, Err> can't be compared. The bad ergonomics bubble up through the codebase.
As I say above, ultimately there are ways for me to work around this, but since the error type in this crate is fairly simple and doesn't use Box<dyn ... etc. I thought I'd try this PR out. Your call.


Code blocks to have scrollbar

Currently our code blocks are wrapping the text where we would rather have it remain on it's line and have a scrollbar to view more.

@mrdavidlaing this page has some strange scroll issues which may be due to the docs - http://www.logsearch.io/docs/boshrelease/getting-started/deploying-logsearch.html


Add ManifoldPoint{M} scientific type

Here M is any subtype of ManifoldsBase.Manifold.
Context: https://github.com/JuliaManifolds/ManifoldML.jl/issues/3#issue-703076389f

Actually, there would be no need to constrain M at all in this package, but that is the planned use case to be implemented in MLJScientificTypes.


Removes entries from cache

removes cached SIRF, Gadgetron, ISMRMRD, STIR  URL and TAG.
uses mkdir -p rather than test.
Closes #31
addresses https://github.com/CCPPETMR/CCPPETMR_VM/pull/29#pullrequestreview-67034267

This branch is fine to merge to master.
I haven't done it though as the title of the PR doesn't fit what will be merged. (A PR is associated to a branch. any commits added to the branch will become part of the PR).
Can you therefore just merge to master on the command line and then push? Github should detect the merge and (I believe) close the PR automatically. Can you then also delete the branch from github? We won't need it anymore and the name is actually very confusing, as it doesn't correspond to 0.9.1-rc1...

OK I shall merge


fix: fixing model.transaction.conditioncheck

fix #539

Summary:
When doing transactions, the Key param had the table key values as promises and dynamo would throw the error Validation Error: Supplied AttributeValue is empty, must contain exactly one of the supported datatypes. This PR simply resolved the promises on the key values.

GitHub linked issue:

#539
Is this ready to be merged into Dynamoose? (select 1):

[x] Yes
[ ] No

Other:

[x] I have searched through the GitHub pull requests to ensure this PR has not already been submitted
[x] I have updated the Dynamoose documentation (if required) given the changes I made
[x] I have added/updated the Dynamoose test cases (if required) given the changes I made
[x] I agree that all changes made in this pull request may be distributed and are made available in accordance with the Dynamoose license
[x] All of my commits and commit messages are detailed, explain what changes were made, and are easy to follow and understand
[x] I have filled out all fields above

Pull Request Test Coverage Report for Build 1113

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 85.485%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/Model.js
1
2
50.0%

Totals

Change from base Build 1112:
0.3%

Covered Lines:
2106

Relevant Lines:
2406

💛  - Coveralls

Pull Request Test Coverage Report for Build 1113

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 85.485%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/Model.js
1
2
50.0%

Totals

Change from base Build 1112:
0.3%

Covered Lines:
2106

Relevant Lines:
2406

💛  - Coveralls


nvim-dap: set up codicons

Right now, the debugger UI looks a little sad, I mean, look at all these missing icons:

Supposedly I need codicons -- but codicons.nvim isn't in nixpkgs. Anyway, gotta do the custom vim plugin song and dance to get it, and it might not even fix the issue 🤷🏻♂

Lol my home-manager config doesn't install Windows fonts. choco install nerd-fonts-hasklig made it happy


FR useUint8Array hook

Feature request for a useUint8Array hook.

ah yes

Implemented in 239b9928ea365fecc33e8a6b3f4a8f7e4b7945bd, released in 2.5.1 ✅


The lib/main.dart template project unnecessarily contains the default useMaterial3 = true

Problem
As of version 3.16, ThemeData.useMaterial3 is set true by default.
"We may not need this anymore" - @khanhnwin at the Observable Flutter #38 Google AI Dart SDK session
Steps to reproduce

Create a Flutter project by using the default app template with the command of flutter create my_app
Open the main.dart file under the lib/ directory.
Take a look on the ThemeData of the MyApp MaterialApp() which contains the useMaterial3: true flag in its constructor.

However,
The packages/flutter_tools/templates/module/common/lib/main.dart.tmpl needs to be changed together with other codelabs:

https://github.com/flutter/flutter/issues/145361

Expected results
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // TRY THIS: Try running your application with "flutter run". You'll see
        // the application has a purple toolbar. Then, without quitting the app,
        // try changing the seedColor in the colorScheme below to Colors.green
        // and then invoke "hot reload" (save your changes or press the "hot
        // reload" button in a Flutter-supported IDE, or press "r" if you used
        // the command line to start the app).
        //
        // Notice that the counter didn't reset back to zero; the application
        // state is not lost during the reload. To reset the state, use hot
        // restart instead.
        //
        // This works for code too, not just values: Most code changes can be
        // tested with just a hot reload.
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.green),
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // TRY THIS: Try changing the color here to a specific color (to
        // Colors.amber, perhaps?) and trigger a hot reload to see the AppBar
        // change color while the other colors stay the same.
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          //
          // TRY THIS: Invoke "debug painting" (choose the "Toggle Debug Paint"
          // action in the IDE, or press "p" in the console), to see the
          // wireframe for each widget.
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Actual results
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // TRY THIS: Try running your application with "flutter run". You'll see
        // the application has a purple toolbar. Then, without quitting the app,
        // try changing the seedColor in the colorScheme below to Colors.green
        // and then invoke "hot reload" (save your changes or press the "hot
        // reload" button in a Flutter-supported IDE, or press "r" if you used
        // the command line to start the app).
        //
        // Notice that the counter didn't reset back to zero; the application
        // state is not lost during the reload. To reset the state, use hot
        // restart instead.
        //
        // This works for code too, not just values: Most code changes can be
        // tested with just a hot reload.
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.green),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // TRY THIS: Try changing the color here to a specific color (to
        // Colors.amber, perhaps?) and trigger a hot reload to see the AppBar
        // change color while the other colors stay the same.
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          //
          // TRY THIS: Invoke "debug painting" (choose the "Toggle Debug Paint"
          // action in the IDE, or press "p" in the console), to see the
          // wireframe for each widget.
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // TRY THIS: Try running your application with "flutter run". You'll see
        // the application has a purple toolbar. Then, without quitting the app,
        // try changing the seedColor in the colorScheme below to Colors.green
        // and then invoke "hot reload" (save your changes or press the "hot
        // reload" button in a Flutter-supported IDE, or press "r" if you used
        // the command line to start the app).
        //
        // Notice that the counter didn't reset back to zero; the application
        // state is not lost during the reload. To reset the state, use hot
        // restart instead.
        //
        // This works for code too, not just values: Most code changes can be
        // tested with just a hot reload.
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.green),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // TRY THIS: Try changing the color here to a specific color (to
        // Colors.amber, perhaps?) and trigger a hot reload to see the AppBar
        // change color while the other colors stay the same.
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          //
          // TRY THIS: Invoke "debug painting" (choose the "Toggle Debug Paint"
          // action in the IDE, or press "p" in the console), to see the
          // wireframe for each widget.
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.2, on macOS 12.6.2 21G320 darwin-arm64, locale en-AE)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[!] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    ✗ Flutter requires Xcode 14 or higher.
      Download the latest version or update via the Mac App Store.
    ! CocoaPods 1.11.3 out of date (1.13.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#updating-cocoapods for instructions.
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2023.3.4)
[✓] Connected device (2 available)
[✓] Network resources

! Doctor found issues in 1 category.

For codelab please file a separate Flutter issue as the codelabs source code is located outside this repository.

Great suggestion @TahaTesser,
Here's the extracted issue containing all of the places in the flutter/samples that need to be changed:

https://github.com/flutter/flutter/issues/145361

We need to respect the official Deprecation policy. This policy requires a grace period of one calendar year after the release on the stable channel or 4 stable releases, whichever is longer, before a feature can be removed.
Decision
Removal is scheduled for Q1 2025 with associated lint warning and dart fix.
🔗 discussion
CC: @domesticmouse


when input an error image name,  the cursor cannot move to the last error character `

In GitLab by @liyanq528 on Jun 7, 2018, 16:08
equest URL: http://localhost:4200/api/v1/imagesimage_name=library/image20180607-7
Request Method: DELETE
Status Code: 200 OK
but the image always exists;

In GitLab by @liyanq528 on Nov 8, 2019, 07:52
don't fix; the problem is from Clarity's issue

In GitLab by @luotian-github on Nov 8, 2019, 07:52
will not fix

In GitLab by @sokril on Nov 8, 2019, 08:22
closed

In GitLab by @sokril on Nov 8, 2019, 08:22
assigned to @luotian-github


Why is this not a pull request to mushroom?

Requirements

[X] I have updated Mushroom to the latest available version
[X] I did a search to see if there is a similar issue or if a pull request is open.

Is your feature request related to a problem?

Describe the solution you'd like
Just out of curiosity. Have you considered creating a pull request to Mushroom? So everyone can benefit from your improvements as well as the ones in the original repo? (You already are behing several commits)
Describe alternatives you've considered
No response
Additional context
No response

Hi @metbril,
Thanks for asking a very reasonable question.
I suggested this as a PR first (here is the discussion) but piitaya disagreed with my proposed changes, which is fine.
For everyone who finds this mod useful and wants an easy way to install it (like myself), I will keep it in HACS and try to update it from time to time.


adapter: allow skipping volume mkdir/chown

Some volumes (such as NFS mounts) return an error when they are chowned
even if the user is root. By setting this new option we assume that the
directory is already in a good state and we can just mount it into the
job.
[fixes #158544043]

@xoebus I built a dev release of bpm with this change and was able to successfully mount our NFS volume. Thanks! 🥂😊

@tcdowney Brilliant, thank you for checking!
We should be able to ship this in a release either later today or early next week.

I can't really see anyone wanting to skip_chown independently of skip_mkdir. We force you to run as vcap and skip_chown would imply that you need a directory that is owned by root.
The one use case I can imagine for skip_mkdir would be they need to edit the permissions on the directory, at which point they would probably already be managing the ownership of the directory as well.


ConcurrentTimedRotatingFileHandler Failed to delete old files

ConcurrentTimedRotatingFileHandler failed to delete old files when it reached the backupCount for the Flask API deployed in the OpenShift cluster. The same code is working fine in Windows. Are there any restrictions for the Linux OS? Can anybody help to resolve this issue?

It's hard to guess what might be wrong. Maybe permission issues? Do you have any log files with error messages or tracebacks (stack trace)? This package is fully supported on Linux.


net: UDPConn.WriteTo and UDPConn.ReadFromUDP both allocate

What version of Go are you using (go version)?

$ go version
go version devel +95ce805d14 Thu Dec 31 02:24:55 2020 +0000 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/josh/bin"
GOCACHE="/Users/josh/Library/Caches/go-build"
GOENV="/Users/josh/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/josh/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/josh"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/josh/go/tip"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/josh/go/tip/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="devel +95ce805d14 Thu Dec 31 02:24:55 2020 +0000"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/josh/go/tip/src/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/1t/n61cbvls5bl293bbb0zyypqw0000gn/T/go-build2869058686=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I'd like to be able to write a program that uses UDPConn.WriteTo and UDPConn.ReadFromUDP without allocating per-packet.
This benchmark indicates one alloc per WriteTo and two allocs per ReadFromUDP.
func BenchmarkWriteToReadFromUDP(b *testing.B) {
	conn, err := ListenUDP("udp4", new(UDPAddr))
	if err != nil {
		b.Fatal(err)
	}
	addr := conn.LocalAddr()
	buf := make([]byte, 8)
	b.ResetTimer()
	b.ReportAllocs()
	for i := 0; i < b.N; i++ {
		_, err := conn.WriteTo(buf, addr)
		if err != nil {
			b.Fatal(err)
		}
		_, _, err = conn.ReadFromUDP(buf)
		if err != nil {
			b.Fatal(err)
		}
	}
}

Two of the allocs come from constructing syscall.Sockaddrs. Maybe this is fixable, but I don't see an easy way.
The last alloc is from constructing a *UDPAddr to return from ReadFromUDP. I fear the API may make this one unavoidable.
cc @bradfitz @danderson @zx2c4

I have a vague impression of pointing this out to @FiloSottile 2 years ago but I don't remember the conclusion of our conversation. CCing in case he has a better recollection.

I found some old notes. The conclusion from last I looked into this was that the API made it unavoidable. As a result I wound up making direct syscalls on Linux but didn't port that to all platforms.
I wonder if this warrants adding a new API. ReadFromUDPWithPrealocatedSockaddr() or something...

IIUC, most sockaddrs will be re-used and they are never mutated. Given that, we could use a sync.Pool of maps to opportunistically re-use them. (This is the technique that https://github.com/josharian/intern uses for strings; it is fast and pretty good, but not perfect. There are other options, with their own trade-offs, like go4.org/intern).
I threw together CL 280934 to illustrate.

IIUC, most sockaddrs will be re-used and they are never mutated. Given that, we could use a sync.Pool of maps to opportunistically re-use them. (This is the technique that https://github.com/josharian/intern uses for strings; it is fast and pretty good, but not perfect. There are other options, with their own trade-offs, like go4.org/intern).
I threw together CL 280934 to illustrate.

Ugh. Nope, that is not safe. We end up exposing the sockaddr memory to the caller in this line (udpsock_posix.go:50):
addr = &UDPAddr{IP: sa.Addr[0:], Port: sa.Port}

Avoiding that would require making a copy of the sa.Addr bytes to serve as the new net.IP. It'd be an allocation of only 4 bytes, which is better than the 32 bytes interning the sockaddr saves, but it's still an allocation.
We might still be able to intern the sockaddrs that are destined for the kernel at least.
If we (a) switched to inet.af/netaddr's IP type and (b) returned a UDPAddr instead of a *UDPAddr, that'd eliminate the non-sockaddr allocations. But (b) requires new API, and (a) is Go 2 material.
We could do both by letting people provide their own *UDPAddr to be filled in, with a preallocated net.IP that could be overwritten...but we already have too many ways to receive a UDP packet. We don't need another one. (At least not in the standard library. Maybe we could set up some of this in golang.org/x/net somehow?)

Ugh. Nope, that is not safe. We end up exposing the sockaddr memory to the caller in this line (udpsock_posix.go:50):
addr = &UDPAddr{IP: sa.Addr[0:], Port: sa.Port}

Avoiding that would require making a copy of the sa.Addr bytes to serve as the new net.IP. It'd be an allocation of only 4 bytes, which is better than the 32 bytes interning the sockaddr saves, but it's still an allocation.
We might still be able to intern the sockaddrs that are destined for the kernel at least.
If we (a) switched to inet.af/netaddr's IP type and (b) returned a UDPAddr instead of a *UDPAddr, that'd eliminate the non-sockaddr allocations. But (b) requires new API, and (a) is Go 2 material.
We could do both by letting people provide their own *UDPAddr to be filled in, with a preallocated net.IP that could be overwritten...but we already have too many ways to receive a UDP packet. We don't need another one. (At least not in the standard library. Maybe we could set up some of this in golang.org/x/net somehow?)

Evil idea: if the buf passed in is pre-allocated memory that's larger than what's needed for the data, and can fit the sockaddr and returned udpaddr and ip, could we stuff that all in at the end of buf?

Evil idea: if the buf passed in is pre-allocated memory that's larger than what's needed for the data, and can fit the sockaddr and returned udpaddr and ip, could we stuff that all in at the end of buf?

Hyrum's Law says no. (Plus the Go standard library generally doesn't go for such evil tricks, entertaining though they be.)

Hyrum's Law says no. (Plus the Go standard library generally doesn't go for such evil tricks, entertaining though they be.)

Maybe a variant of that might be acceptable:
Right now people pass in a buffer of the maximum size of data they want:
data := make([]byte, 1472)
n, addr, err = conn.ReadFromUDP(data)
data = data[:n]

My initial idea was to place the sockaddr allocations in the region of data[n:] that doesn't wind up getting used. You cited Hyrum.
It occurred to me that other Go APIs sometimes work by taking slice to append to and then return a new slice. The reasoning goes that the caller can do the application by allocating a slice with a large capacity but a zero length, and then the appending operation is free. What if we use the same trick here:
data := make([]byte, 1472, 2000)
n, addr, err = conn.ReadFromUDP(data)
data = data[:n]

In this instance, rather than placing addr at data[n:], it's placed after the 1472 bytes by using append, which does not need to allocate for another 528 bytes. So we avoid the allocation by putting the sockaddr there.

Maybe a variant of that might be acceptable:
Right now people pass in a buffer of the maximum size of data they want:
data := make([]byte, 1472)
n, addr, err = conn.ReadFromUDP(data)
data = data[:n]

My initial idea was to place the sockaddr allocations in the region of data[n:] that doesn't wind up getting used. You cited Hyrum.
It occurred to me that other Go APIs sometimes work by taking slice to append to and then return a new slice. The reasoning goes that the caller can do the application by allocating a slice with a large capacity but a zero length, and then the appending operation is free. What if we use the same trick here:
data := make([]byte, 1472, 2000)
n, addr, err = conn.ReadFromUDP(data)
data = data[:n]

In this instance, rather than placing addr at data[n:], it's placed after the 1472 bytes by using append, which does not need to allocate for another 528 bytes. So we avoid the allocation by putting the sockaddr there.

Mmm, looks like that can still lead to unexpected problems, though perhaps these are not significant?
package main

import (
	"fmt"
)

func doTheAliasingTrick(slice []byte) *byte {
	for i := range slice {
		slice[i] = 41
	}
	return &append(slice, 42)[len(slice)]
}

func main() {
	data := make([]byte, 1472, 2000)
	x := doTheAliasingTrick(data)

	fmt.Printf("*x = %d\n", *x) // Prints 42
	_ = append(data, 43)
	fmt.Printf("*x = %d\n", *x) // Prints 43
}

Mmm, looks like that can still lead to unexpected problems, though perhaps these are not significant?
package main

import (
	"fmt"
)

func doTheAliasingTrick(slice []byte) *byte {
	for i := range slice {
		slice[i] = 41
	}
	return &append(slice, 42)[len(slice)]
}

func main() {
	data := make([]byte, 1472, 2000)
	x := doTheAliasingTrick(data)

	fmt.Printf("*x = %d\n", *x) // Prints 42
	_ = append(data, 43)
	fmt.Printf("*x = %d\n", *x) // Prints 43
}

Avoiding that would require making a copy of the sa.Addr bytes to serve as the new net.IP. It'd be an allocation of only 4 bytes, which is better than the 32 bytes interning the sockaddr saves, but it's still an allocation.

I think you can outline that. It will require some care on the caller side, but if someone needs it they can make sure they don't get in the way of the inliner.

Avoiding that would require making a copy of the sa.Addr bytes to serve as the new net.IP. It'd be an allocation of only 4 bytes, which is better than the 32 bytes interning the sockaddr saves, but it's still an allocation.

I think you can outline that. It will require some care on the caller side, but if someone needs it they can make sure they don't get in the way of the inliner.

I played with this a bit more. Results:

https://github.com/josharian/go/commit/c44a72a336c82f918cba113a8229b6e9b7d31ee9
https://github.com/josharian/go/commit/efe5967d9f99a211d6e3152b9a1749585f21f2ef

With CL 291509 and those two commits, we would have zero allocs per write and one net.IP backing array allocated per receive (4 or 16 bytes).
But those two commits involve new API and duplicating subtle code. :(
The new API is:
// WriterTo returns an io.Writer that writes UDP packets to addr.
// This is more efficient than WriteTo when many packets will be sent to the same addr.
func (c *UDPConn) WriterTo(addr Addr) (io.Writer, error)

I'm happy to propose the API and mail the other change if there's interest, but my working assumption is that they're both non-starters.

Hi Same founding here, we wrote a high concurrency dns server, and found that the bottleneck is sockaddr allocation of WriteMsgUDP
Here is our flam graph, hope it could also get a mid-stack inlining fix.

https://github.com/tailscale/go/commit/1074dae794a5acb1a9016739a2b33d2590b1fd00 remove all allocations for WriteTo.
https://github.com/tailscale/go/commit/4b4fb836e1ad39f0782b900d3cbb3ff0684d0865 reduces the size of the allocation for ReadFrom from 32 bytes down to the size of net.IP (4 bytes for IPv4, 16 for IPv6).
Both code at the cost of duplicating a bunch of code, some of it non-trivial. I'm happy to mail for 1.18, if folks aren't too horrified by that.
Remaining the last ReadFrom allocation is impossible without an API change. The key change is having a value IP type, such as netaddr.IP.
cc @neild @bradfitz

I'm not horrified by the amount of code duplication in those CLs.

I'm not horrified by the amount of code duplication in those CLs.

Great. I'll plan to mail them soon or early in the 1.18 cycle. And if I forget or don't get to it quickly enough, you (or anyone) has my explicit permission to do so.


remove old pipeline

Codecov Report

Merging #383 (d216b8f) into master (63b6645) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #383      +/-   ##
==========================================
+ Coverage   67.13%   67.22%   +0.09%
==========================================
  Files         120      120
  Lines       10591    10591
==========================================
+ Hits         7110     7120      +10
+ Misses       3481     3471      -10

Impacted Files
Coverage Δ

assemblyline/datasource/alert.py
88.46% <0.00%> (-3.85%)
:arrow_down:

assemblyline/common/heuristics.py
80.76% <0.00%> (-1.29%)
:arrow_down:

assemblyline/odm/randomizer.py
95.17% <0.00%> (-0.88%)
:arrow_down:

assemblyline/common/backupmanager.py
75.41% <0.00%> (+5.83%)
:arrow_up:

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 63b6645...d216b8f. Read the comment docs.


Lua lexer

Thank you so much for leac, its API is very elegant and easy to use.
I'm creating the new assignments for the compilers class at the college of engineering UBA (University of Buenos Aires). The students might use the following lua lexer which I want to share with you in case you find it useful. If I find the time I'll write some more tests and add an example in a PR.
import { createLexer, Rule, Rules } from "https://deno.land/x/leac/leac.ts";

const symbols = [
  ";",
  "=",
  ",",
  "::",
  ".",
  "[",
  "]",
  "...",
  "(",
  ")",
  ":",
  "{",
  "}",
];

const keywords = [
  "break",
  "goto",
  "do",
  "end",
  "while",
  "do",
  "repeat",
  "until",
  "if",
  "then",
  "elseif",
  "else",
  "for",
  "in",
  "function",
  "local",
  "return",
  "nil",
  "false",
  "true",
];

const ops = [
  "+",
  "-",
  "*",
  "/",
  "//",
  "^",
  "%",
  "&",
  "~",
  "|",
  ">>",
  "<<",
  "..",
  "<",
  "<=",
  ">",
  ">=",
  "==",
  "~=",
  "and",
  "or",
  "#",
  "not",
];
const doublequoteStringLexer = createLexer([
  {
    name: "stringContent",
    regex: /(?:\\["abfnrtv/\\nz]|\\u[a-fA-F0-9]{4}|[^"\\\n])*/,
  },
  {
    name: "LiteralStringEnd",
    str: '"',
    pop: true,
    discard: true,
  },
]);
const singlequoteStringLexer = createLexer([
  {
    name: "stringContent",
    regex: /(?:\\['abfnrtv/\\nz]|\\u[a-fA-F0-9]{4}|[^'\\\n])*/,
  },
  {
    name: "LiteralStringEnd",
    str: "'",
    pop: true,
    discard: true,
  },
]);
function smallerCaseRegexpPart(level: number) {
  if (level == 0) {
    return "";
  }
  if (level == 1) {
    return "|\\]\\]";
  }
  return `|\\]={0,${level - 1}}(?=\\])`;
}
function regexNoLongBrackets(level: number) {
  const smallerCase = smallerCaseRegexpPart(level);
  const largerCase = `|\\]={${level + 1},}(?=\\])`;
  const doesntEndWithBracketCase = `|\\]=*[^\\]=]`;
  return new RegExp(
    `([^\\]]${smallerCase}${largerCase}${doesntEndWithBracketCase})+`,
    "m",
  );
}
function longLiteralStringRule(level: number) {
  const equalSigns = Array(level).fill("=").join("");
  const lexer = createLexer([
    {
      name: "LiteralStringEnd",
      str: "]" + equalSigns + "]",
      discard: true,
      pop: true,
    },
    {
      name: "stringContent",
      regex: regexNoLongBrackets(level),
      discard: true,
    },
  ]);
  return {
    name: "LiteralStringBegin",
    str: "[" + equalSigns + "[",
    push: lexer,
    discard: true,
  };
}
const simpleRules = [...ops, ...keywords, ...symbols].map((v) => ({ name: v }));
function longCommentRule(level: number) {
  const equalSigns = Array(level).fill("=").join("");
  const lexer = createLexer([
    {
      name: "longCommentEnd",
      str: "]" + equalSigns + "]",
      discard: true,
      pop: true,
    },
    {
      name: "commentContent",
      regex: regexNoLongBrackets(level),
      discard: true,
    },
  ]);
  return {
    name: "longCommentBegin",
    str: "--[" + equalSigns + "[",
    push: lexer,
    discard: true,
  };
}
export const lex = createLexer(
  [
    {
      name: "ws",
      regex: /\s+/,
      discard: true,
    },
    ...Array(100).fill(0).map((_value, index) => longCommentRule(index)),
    {
      name: "shortComment",
      regex: /--.*\n/m,
      discard:true,
    },
    ...simpleRules,
    {
      name: "Name",
      regex: /[a-zA-Z_][a-zA-Z_0-9]*/,
    },
    {
      name: "Numeral",
      regex: /[0-9]*\.?[0-9]+/,
    },
    {
      name: "LiteralStringBegin",
      str: '"',
      push: doublequoteStringLexer,
      discard: true,
    },
    {
      name: "LiteralStringBegin",
      str: '"',
      push: singlequoteStringLexer,
      discard: true,
    },
    ...Array(100).fill(0).map((_value, index) => longLiteralStringRule(index)),
  ],
);

Wow! I'm happy this package finding it's use outside of my own projects already!

Thank you for this example.
I think it illustrates one caveat I should've paid more attention myself. I should update the docs and json example in a few days to raise the awareness.
In short, try to tokenize a code with empty strings.
It is not obvious that emitting a token for an empty regex match is tricky for a tokenizer. My solution for this is to not emit any token for empty match (otherwise it would require to mitigate infinite loops somehow).
In combination with discarded quotes this can lead to a bad stream of tokens.
In my opinion, it is only safe to discard things that won't be missed when you try to glue the tokens array back and it will still be valid code. Otherwise be ready to think about edge cases.

When composing lexers, it is safer to keep quotes. You will identify an empty string in a parser as two quote tokens in a row;
Another option might be to introduce an empty string token in an outer scope, before opening quote token;
For simple strings it might be feasible to keep the string content and quotes in a single regex, using replace property to extract the content.

In theory, it might be possible to keep track of rules that returned an empty match and return at most a single token at current offset. This will complicate the implementation and I'm not sure it's a good idea.
Related note: you produce stringContent tokens in multiple places with slightly different regexes. This means strings might contain differently escaped quotes and there is not enough information in the token to distinguish them and decide on a proper unescape function later in a parser. Using unique names might be a good idea in this case.
Another note:

const simpleRules = [...ops, ...keywords, ...symbols].map((v) => ({ name: v }));

This is ultimately a designer's choice but, if putting an effort to keep things separate to this point, I'd think whether keeping this information in rules and tokens could be beneficial for the parser as well, like this:
const opRules = ops.map((v) => ({ name: 'Operation', str: v }));
const keywordRules = keywords.map((v) => ({ name: 'Keyword', str: v }));
const symbolRules = symbols.map((v) => ({ name: 'Symbol', str: v }));

Rule names don't have to be unique. It is very similar to making a single regex rule that will match any of listed values.

For a PR, example should be self-contained, like the existing ones. Just call the lexer with minimal sample(s) to illustrate variety of supported grammar and log to console. Project tests run all examples and store the output as a snapshot.
I'm more concerned about comments. Examples must be that - examples for those who are learning how to use the package. Link(s) to official grammar is a must for an existing language. And any comments to describe the lexer design. The bigger the grammar - the more explanation it might need.
(I was quite lazy about samples and comments because my examples essentially are excerpts from peberminta examples. Big samples would also lead to unnecessarily big snapshots.)


[SR-15722] Include information about @MainActor in the symbol graph

Previous ID
SR-15722

Radar
None

Original Reporter
kap (JIRA User)

Type
Improvement

Additional Detail from JIRA

Votes
1

Component/s
Swift-DocC

Labels
Improvement

Assignee
kap (JIRA)

Priority
Medium

md5: e865b78877f6833c45e5909a5be1026c

Issue Description:
The current implementation of -emit-symbol-graph does not include any of the custom attributes, which, in turn, does not enable DocC (and the generated documentation) to document when a symbol is marked with @MainActor.
While all of the custom attributes are skipped (e.g. property wrappers, global actors), allowing @MainActor to be emitted into the symbol graph would be a nice start.
Forum thread discussing this

i opened the original issue on the swift forum (and the JIRA ticket). seems like the behaviour has since changed, and symbols marked with @MainActor are correctly represented in DocC as such. not sure how to close this issue though

Thank you @kaphacius


update warsow to 20

Should be version 2.0
You can use #{version.delete('.')} to make the URL work


Allow diff-tar to accept a target

Currently the filepath to which conda-diff-tar writes the reference.json file, and the path to which it writes the update.tar file are hardcoded. Would be nice if these could be passed as an argument.

closed by #21


Added Math for Machine Learning by Marc Peter Deisenroth

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource
For resources
Description
Mathematics for Machine Learning is a book developed by Marc Peter Deisenroth, A Aldo Faisal, and Cheng Soon Ong, with the aim of motivating people to learn mathematical concepts
Why is this valuable?
The purpose of this book is to provide the math skills needed to read books on more sophisticated machine learning topics
How do we know it's really free?
You can download the PDF of the book here. Even though the book will be published this year, the author will continue to make the PDF book available for free download after publication.
For book lists, is it a book?
Yes it's a book
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

When linking directly to a PDF, please added (PDF) at the end of the line.

Thank you, I've added (PDF) at the end of the line and all checks have passed

Nice addition! Maybe https://mml-book.github.io is a better link?

Done, I've changed the link


Withdrawing validoator comissions returns bech32 error

lens tx distribution withdraw-rewards junovaloper1qabcyl test -c
returns:
Error: decoding bech32 failed: invalid bech32 string length 7

Also noticed this with
lens query bank balances
Error: decoding bech32 failed: invalid bech32 string length 7


How to forward error to customized error controller

I created a customized ErrorFilter to suppress FilterError before sending back to clients. I can see the ErrorFilter is called, however, it does not forward to my error controller. instead of forwarding to tomcat error handling.
I even changed the error path from /error to /myservice/error, but it still ends up with default tomcat error handling. I must have missed something obviously. The dispatcher seems org.apache.catalina.core.ApplicationDispatcher, not Spring Dispatcher. Any comments and suggestions are appreciated!
RequestDispatcher dispatcher = ctx.getRequest().getRequestDispatcher(this.errorPath) 			if (dispatcher != null) { 				if (!ctx.getResponse().isCommitted()) {					 					dispatcher.forward(ctx.getRequest(), ctx.getResponse()) 				} 			}

This is not a spring project. Use spring-cloud-netfix for spring issues.

My bad, wrong board.


Feature request: more geometry options for zonal

Currently, geometry in zonal considers cells where center, the whole or part of the cell are contained within another geometry. For polygon data, it would be quite useful to calculate the fraction of overlap between cell and polygon to compute statistics.

You could do this manually in a loop over each polygon using coverage.
But yes it would be nice to automate that process. We can just call coverage instead of boolmask internally and multiple by coverage rather than masking.
Maybe with a separate coverage=true keyword? To show that it's quite different to choosing a boundary?


Functional API with multiple inputs: TypeError: unhashable type: 'dict'

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[x] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Gist: https://gist.github.com/moinnadeem/47e0755d9d3e77c77186ee15d1c801f4
What am I missing? Thanks!

Resolved: I was adding the fit parameters to initializing the model, rather than the model input/output layers!


Undefined member behaviour inconsistent

Referencing the undefined member warning in the devforum.roblox.com post:

When writing the same name twice or more across all the scripts, it will automatically ignored.

There are two problem cases (that I believe are reasonably common):

The name of the instance you are referring to has changed (and you referred to it more than once) or you made a typo and copy/pasted that typo into another script. In this case, no warning will result, despite it being an issue that needs fixing.
The name is correct, but the project hasn't imported an ignore list. If the name is only used once, a warning will occur.

Suggestions:

If the user doesn't have a datamodel.json file, either don't trigger this warning for anything, or trigger this warning for everything and include a mention about this "datamodel.json" file in the warning text (ex, "MyInstance is not a defined member of ServerScriptService in datamodel.json")
If the user does have a datamodel.json file, trigger this warning no matter how many times an instance name shows up across all scripts

The way it is implemented right now it's a problem, and needs to be changed.
Although most people will not have a .datamodel.json as it is very impractical to use and has to be constantly updated, it will be useful when it can be updated automatically by a Roblox Studio plugin that communicates with the Language Server.
So your suggestions are not the best way to fix this problem.

I don't understand your conclusion.
The only beneficial thing this warning can do for you is to tell you when you've misspelled something or forgotten to update the name of something. However, if you change the name of something in the place and forget to update it in the scripts, this warning won't trigger if 2+ scripts reference the original name. Doesn't this defeat the main benefit?
I think the best version of this feature (that could be implemented now as far as I know):

If you use datamodel.json, it could warn you if you try to access any unknown instance
If you use datamodel.json and rojo, any script that rojo would create would be considered "known", even if it wasn't listed in the datamodel.json file
If you just have a rojo file, it could warn you if you try to require an unknown script (only for areas that rojo is updating)
If you have neither, it would be silent

If it worked like that, I wouldn't have needed to disable this feature for my projects (as I don't have datamodel.json set up), as it would be rare for it to give a false positive.
All this said, I imagine that it's a ton of effort to implement for minimal benefit, so I understand if you choose to disable the feature. (I wouldn't mind it being disabled, since if I want to get someone set up to use your plugin, disabling this warning would just be another step I'd need to communicate.)

Hi, just to be clear, I'm not going to disable the undefined member warning, just ignore names that were used twice or more.
And it's because as you said, if you use the name once, you will still get a warning, but if you use the name more times but misspelled, you will not.


npm run start error code 126

Im trying to get local development going to try and add other projects but I'm getting stuck trying to start the server. This is the error I get:
[2] sh: ~/www/rarity-analyzer/node_modules/.bin/copyfiles: Argument list too long
[2] npm run start:client exited with code 126

Any thoughts on how to troubleshoot or what could be causing this?
I've had mixed results running npm run build and accessing the site with a local nginx install, sometimes it works sometimes it doesnt. My react/js skill is limited.

@seafarer I had to use the solution proposed here: https://github.com/calvinmetcalf/copyfiles/issues/14
This meant updating the package.json on publish:client & start:rarities cmds - wrapping the folder path when using globs in quotations

Thanks @0xdruu - I'll try this this weekend 🙏


[ci skip] adding user @twsl

Hi! This is the friendly automated conda-forge-webservice.
I've added user @twsl as instructed in #8.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #8

@twsl I am sorry for this long delay. I somehow missed this. Could you please restart the CI on this PR?

@conda-forge-admin, please restart ci


Improve quality of existing code

Improve the quality of existing code and add test - Happy number

Thank you for making these improvements. The naming conventions in the previous code were poor and you've definitely improved them.


Google Font weights

Hi, I'm having trouble with google fonts import. When I import Lato for example, it doesn't include all font weights, I need 100,200, etc. While it's only importing regular and above.
Any ideas why or how to fix it?
Thank you, loving Kirki so far.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/30876052-google-font-weights?utm_campaign=plugin&utm_content=tracker%2F2030945&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2030945&utm_medium=issues&utm_source=github).

Hey there!
What exactly do you mean by " it doesn't include all font weights"?
Could you please elaborate?

I put Lato as base typography, but this is what it's generating:
<link rel='stylesheet' id='kirki_google_fonts-css'  href='//fonts.googleapis.com/css?family=Lato:regular,italic,700,%7CRoboto:regular,italic,700,100,700&subset=all' type='text/css' media='all' />

It only includes regular and up, not the 100 weight for example

Are you using the typography control, or are you adding separate controls?
If you're adding separate controls did you add a font-weight control?
Take a look at https://github.com/aristath/kirki/wiki/Google-Fonts-Integration for an example...

@alfgago We're currently rebuilding the typography control.
Can you please download a new copy of the plugin ffrom this repository and check with the new implementation if you're still having this issue?
It looks like this has been fixed there.

Hi, I tried with the newest copy of the repo, but as soon as I install Kirki I get
Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Warning: Invalid argument supplied for foreach() in /home/delrosspa/public_html/dev/wp-content/themes/elias-5/inc/kirki/includes/class-kirki-field.php on line 360

Also, about your last reply, I am indeed using typography control with font-weight control, I'm using the following:
Kirki::add_field( 'elias', array(
	'type'     => 'select',
	'select2'  => true,
	'settings' => 'base_typography_font_family',
	'label'    => __( 'Font Family', 'elias' ),
	'section'  => 'base_typography',
	'default'  => 'Lato',
	'priority' => 20,
	'choices'  => Kirki_Fonts::get_font_choices(),
	'output' => array(
		array(
			'element'  => 'body',
			'property' => 'font-family',
		),
	),
) );

Kirki::add_field( 'elias', array(
	'type'     => 'slider',
	'settings' => 'base_typography_font_weight',
	'label'    => __( 'Font Weight', 'elias' ),
	'section'  => 'base_typography',
	'default'  => 300,
	'priority' => 24,
	'choices'  => array(
		'min'  => 100,
		'max'  => 900,
		'step' => 100,
	),
	'output' => array(
		array(
			'element'  => 'body',
			'property' => 'font-weight',
		),
	),
) );

Kirki::add_field( 'elias', array(
	'type'     => 'slider',
	'settings' => 'base_typography_font_size',
	'label'    => __( 'Font Size', 'elias' ),
	'section'  => 'base_typography',
	'default'  => 16,
	'priority' => 25,
	'choices'  => array(
		'min'  => 7,
		'max'  => 48,
		'step' => 1,
	),
	'output' => array(
		array(
			'element'  => 'html',
			'property' => 'font-size',
			'units'    => 'px',
		),
	),
) );

(sorry it took me this long to answer, I had moved on with the development and postponed that part, so I completely forgot).

Instead of all of the above, try something like this:
Kirki::add_field( 'elias', array(
    'type'        => 'typography',
    'settings'    => 'typography_test',
    'label'       => esc_attr__( 'Typography Control', 'test' ),
    'description' => esc_attr__( 'This is a composite typography control. It saves the data as an array, and you can define which of the controls you want shown.', 'test' ),
    'tooltip'     => esc_attr__( 'This is a tooltip', 'test' ),
    'section'     => 'kirki_typography_test',
    'priority'    => 10,
    'output'      => array(
        array( 'element' => 'body, body p' ),
    ),
    'default'     => array(
        'font-family'    => 'Roboto',
        'variant'        => 'regular',
        'font-size'      => '1.1em',
        'line-height'    => '1.5',
        'letter-spacing' => '0',
        'color'          => '#333333',
    ),
    'choices'     => array(
        'font-family'    => true,
        'font-size'      => true,
        'line-height'    => true,
        'letter-spacing' => true,
        'color'          => true,
    ),
) );

And if you can test with the latest version again... we've got daily updates on github :)

Nice, the way I was doing it worked with the latest version. But the typography type you sent me works even better.
Thank you.


New game: Chip's Challenge

Project Link
https://www.muppetlabs.com/~breadbox/software/tworld/
Project License
GPL2 or later
Steam App Ids

https://store.steampowered.com/app/346850/Chips_Challenge_1/
https://store.steampowered.com/app/348300/Chips_Challenge_2/

Additional Context
"Tile World" is an SDL re-implementation of the Chip's Challenge game engine. It supports both the Atari Lynx and the Windows releases. It's unknown if it works with the Steam release, but if that release includes chips.dat file, it should be good to go.
"Tile World 2" is a fork of the previous project, and it's a re-implementation using Qt. https://tw2.bitbusters.club/
There are even many community level packs (only playable in these re-implementations, because AFAIK the original game doesn't support custom levels): https://bitbusters.club/
Footnote: I haven't tried these projects myself. I just found out about them and I thought they would be a good addition to Luxtorpeda.

Thanks for the request! I'll take a look when I have a chance

Sadly it looks like there isn't a way to use this with the steam release. The steam release seems to use the second game's engine, and the steam release does not contain the chips.dat file.
Let me know if there's anything I'm missing there.


pow(2) fails for 2x3 tensors on GPU

NaNs show up in the result.

Turns out WebGPU pow does not handle negative inputs even when the exponent is an integer.
Put in special logic to detect this: https://github.com/praeclarum/webgpu-torch/commit/0f1c13c564af9c8b0d07c329f0f55d16c629d511


Load yolov5 failed!-20240827

--> Config model
done
--> Loading model
E Catch exception when loading pytorch model: best.onnx!
E Traceback (most recent call last):
E   File "rknn\base\RKNNlib\converter\convert_pytorch_new.py", line 3967, in rknn.base.RKNNlib.converter.convert_pytorch_new.convert_pytorch.load
E   File "rknn\base\RKNNlib\converter\pytorch_util.py", line 75, in rknn.base.RKNNlib.converter.pytorch_util.check_model_pt_version
E   File "rknn\base\RKNNlib\converter\pytorch_util.py", line 76, in rknn.base.RKNNlib.converter.pytorch_util.check_model_pt_version
E   File "D:\Anaconda3\envs\rknn\lib\zipfile.py", line 1131, in init
E     self._RealGetContents()
E   File "D:\Anaconda3\envs\rknn\lib\zipfile.py", line 1198, in _RealGetContents
E     raise BadZipFile("File is not a zip file")
E zipfile.BadZipFile: File is not a zip file
E Please feedback the detailed log file <log_feedback_to_the_rknn_toolkit_dev_team.log> to the RKNN Toolkit development team.
E You can also check github issues: https://github.com/rockchip-linux/rknn-toolkit/issues
best.onnx ********************
Load yolov5 failed!

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


[FR]: Set chunk size on upload task

Description
I may be missing something very obvious, but I see no way to set the upload chunk size from swift code for a 'FIRStorageUploadTask' created via FIRStorageReference putFile.
e.g.
let uploadTask = remoteRef.putFile(from: fileURL, metadata: metadata)

Use case: allow custom chunk sizes so resumable uploads work by default
Currently the chunk size seems hard coded to a large value
I propose allowing custom chunk sizes

In  FIRStorageUploadTask.m the chunk size seems to be set from the constant kGTMSessionUploadFetcherStandardChunkSize which is set to LLONG_MAX;
    GTMSessionUploadFetcher *uploadFetcher =
        [GTMSessionUploadFetcher uploadFetcherWithRequest:request
                                           uploadMIMEType:strongSelf->_uploadMetadata.contentType
                                                chunkSize:kGTMSessionUploadFetcherStandardChunkSize
                                           fetcherService:self.fetcherService];

Perhaps there is a way to modify this somewhere, but I can't find any documentation on it.
Thanks for any support.
API Proposal
let uploadTask = remoteRef.putFile(from: fileURL, metadata: metadata, chunkSize: myChunkSize )

Firebase Product(s)
Storage

@cclaan Thank for the feature request.  You're correct that there's currently no way to change chunkSize from LLONG_MAX
We've begun work on a Swift-only V2 API for Storage in #10161. chunkSize could work well as a default argument in the new API.

@cclaan Would you provide more description about the value of being able to set the chunkSize.  Why do you want to be able to manage the upload in smaller chunks?

My use case is uploading ~20mb files from a phone with potentially poor network connection. Using smaller chunks is required so that the upload does not restart from scratch every time the network drops. ( Previously chunkSize was LLONG_MAX )


NIM-17211: Format on Every Date Field

Overview of Changes

Added Javadocs so consumers understand the support for datePattern.
Ran formatting tool on ViewConfig

Description of Issue
The usage of the datePattern attribute was not clearly understood by consumers of the Framework.
Solution
Documentation.

You did it @lopezton-anthem!
Thank you for signing the Anthem Opensource Software CLA.


Refactor default Options

If we think it is cleaner, prefer:
A
Options DefaultOptions(Options opts) {
  return MergeOptions(std::move(opts),
      Options{}.set<Option1>(default1)
               .set<Option2>(default2)
               // etc.
               .set<OptionN>(defaultN));
}

to:
B
Options DefaultOptions(Options opts) {
  if (!opts.has<Option1>())
    opts.set<Option1>(default1);
  if (!opts.has<Option2>())
    opts.set<Option2>(default2);
  // etc.
  if (!opts.has<OptionN>())
    opts.set<OptionN>(defaultN);
  return opts;
}

(which we use in bigtable, spanner, generated libraries)

The version @dbolduc is proposing certainly does look cleaner, IMO. The only potential downsides I can think of are:

That requires computing all the default values up front, even in cases where the default is not needed. However, this cost could be avoided by using a function-local static constant "kDeafults" that's initialized by invoking a lambda. That way, we compute all the default values once, and can re-use the same defaults on every subsequent invocation.
Some defaults depend on other options, and it may be difficult/impossible to pre-compute these correctly. However, that may not be a big problem. We could still use the preferred "MergeOptions" implementation for most defaults, followed by a handful of special-case options.

I think both of these "downsides" have reasonable workarounds. I'd be happy to see a PR where someone converts one of the existing DefaultOptions() functions to use this new approach so we can see how it improves things.

Closing. If anyone feels strongly, they can reopen the issue.


refactor subtasks format

subtasks should be an array of subexercises, where each exercise is an object that contains score, duration, task and solution. This makes everything easier to manage.

structure is defined


Bad response type expectation for laravel

Hello,
Currently noticed, that at some point my responses listener stopped to working. After debugging of occuring error
Undefined property: Symfony\Component\HttpFoundation\Response::$exception noticed, that all preflight responses are using symfony response, which further is not wrapped into laravel responses toIlluminateResponse.
My question is, why not throwing an error like other middlewares does in handle method, so response then would be in laravel style
https://github.com/barryvdh/laravel-cors/blob/master/src/CorsService.php#L161
https://github.com/barryvdh/laravel-cors/blob/master/src/HandlePreflight.php#L33

Can you try again with https://github.com/barryvdh/laravel-cors/releases/tag/v0.10.1 ?
Please re-open this issue when the problem still persists.


Rat 1.4

Main improvements in this PR:

Fixes (PR 460):

fix: correct MAM03882 annotation, as reported in Mouse-GEM #21

Fixes (PR 429):

feat: add Acod1(Irg1) gene and associated reaction, as discussed in Mouse-GEM #20

Features (commit f5f78450e7c6659bef5b9cd0dd48da35b82e7226):

feat: update ortholog pairs from Alliance of Genomes to ver 5.3.0

Style (commit cee1e295795e30064ed5c310748f566d6d89ffc2):

change human-rat ortholog file from json to tsv, as discussed in Mouse-GEM #15

It would be great to note somewhere which version of Human-GEM this is based on, perhaps in the README.

It would be great to note somewhere which version of Human-GEM this is based on, perhaps in the README.

I believe you've asked this question before, would be good fix this somehow

I believe you've asked this question before, would be good fix this somehow

Indeed I have - but I still can't find where the Human-GEM dependency is listed. I'm asking about this since I know there have been quite a few releases improving Human-GEM since the previous release of Rat-GEM.

I still can't find where the Human-GEM dependency is listed

the quick answer is the dependency file is in main branch only

probably we should replace this step with an automatic approach via a centralized script in HumanGEM


Uncaught Error: Module did not self-register.

[Enter steps to reproduce:]

...
...

Atom: 1.45.0 x64
Electron: 4.2.7
OS: Unknown macOS version
Thrown From: Atom Core
Stack Trace
Uncaught Error: Module did not self-register.
At ELECTRON_ASAR.js:160

Error: Module did not self-register.
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:1:695134)
    at get_KeyboardLayoutManager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2781650)
    at get_manager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2781752)
    at Object.getCurrentKeyboardLayout (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2781878)
    at e.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1083882)
    at KeymapManager.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1251844)
    at KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1249829)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:284496)

Commands
  2x -1:11.4.0 tree-view:add-file (div.header.list-item.project-root-header)

Non-Core Packages
atom-beautify 0.33.4
busy-signal 2.0.1
emmet 2.4.3
intentions 1.1.5
linter 3.1.1
linter-htmlhint 1.6.3
linter-jshint 3.1.19
linter-ui-default 2.1.4
open-in-browser 0.5.2
open-recent 5.0.0
prettier-atom 0.60.1

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/20222. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


[Usability] Add version checker to all tutorials

Description
Following the previous PR, add version checkers to tutorials.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

cc @dmlc/gluon-nlp-team

Job PR-934/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-934/2/index.html