Server 2008 R2 generation fails with bcdedit error

Server 2008 R2 has an older version of bcdedit which fails on the three instructions starting here:
https://github.com/cloudbase/windows-openstack-imaging-tools/blob/master/WinImageBuilder.psm1#L149
It doesn't contain the "locate" device. Changing it to "boot" seems to be an easy fix.
Proposed solution: using the logic just above it that determines the parameters for bcdboot.exe to determine the parameters for bcdedit as well.
Exact error message:
The device is not valid as specified.
Run "bcdedit /?" for command line assistance.
The parameter is incorrect.

Hi, I am also facing the same issue. Can you please provide the exact code that you used to fix the same.

@Parth6288 sure, here it is:
    & $bcdeditPath /store ${systemDrive}\boot\BCD /set `{bootmgr`} device boot
    if($LASTEXITCODE) { throw "BCDEdit failed: device boot1" }

    & $bcdeditPath /store ${systemDrive}\boot\BCD /set `{default`} device boot
    if($LASTEXITCODE) { throw "BCDEdit failed: device boot2" }

    & $bcdeditPath /store ${systemDrive}\boot\BCD /set `{default`} osdevice boot
    if($LASTEXITCODE) { throw "BCDEdit failed: osdevice boot" }

Notice the "boot" parameter at the end instead of "locate".

Thanks, Tom. I was able to get it working. However I am stuck with another issue. Using the -installupdate parameter the instance checks the WSUS server for available updates and accepts it. however it gets stuck at the downloading phase and proceeds with next part of the script which is to install cloudbase init. Please find attached screenshot of the same.

Below is the Windowsupdate logs.
** START **  Agent: Finding updates [CallerId = AutomaticUpdates]
2016-09-08	10:48:19:500	 848	604	Agent	*********
2016-09-08	10:48:19:500	 848	604	Agent	  * Online = Yes; Ignore download priority = No
2016-09-08	10:48:19:500	 848	604	Agent	  * Criteria = "IsInstalled=0 and DeploymentAction='Installation' or IsPresent=1 and DeploymentAction='Uninstallation' or IsInstalled=1 and DeploymentAction='Installation' and RebootRequired=1 or IsInstalled=0 and DeploymentAction='Uninstallation' and RebootRequired=1"
2016-09-08	10:48:19:500	 848	604	Agent	  * ServiceID = {3DA21691-E39D-4DA6-8A4B-B43877BCB1B7} Managed
2016-09-08	10:48:19:500	 848	604	Agent	  * Search Scope = {Machine}
2016-09-08	10:48:19:515	 848	604	Setup	Checking for agent SelfUpdate
2016-09-08	10:48:19:515	 848	604	Setup	Client version: Core: 7.6.7600.256  Aux: 7.3.7600.16385
2016-09-08	10:48:43:109	 848	604	Misc	WARNING: Send failed with hr = 80072ee2.
2016-09-08	10:48:43:109	 848	604	Misc	WARNING: SendRequest failed with hr = 80072ee2. Proxy List used: <(null)> Bypass List used : <(null)> Auth Schemes used : <>
2016-09-08	10:48:43:109	 848	604	Misc	WARNING: WinHttp: SendRequestUsingProxy failed for http://mywsusurl//selfupdate/wuident.cab. error 0x80072ee2
2016-09-08	10:48:43:109	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation MakeRequest failed. error 0x80072ee2
2016-09-08	10:48:43:109	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation failed with 0x80072ee2
2016-09-08	10:48:43:109	 848	604	Misc	WARNING: WinHttp: ShouldFileBeDownloaded failed with 0x80072ee2
2016-09-08	10:49:06:703	 848	604	Misc	WARNING: Send failed with hr = 80072ee2.
2016-09-08	10:49:06:703	 848	604	Misc	WARNING: SendRequest failed with hr = 80072ee2. Proxy List used: <(null)> Bypass List used : <(null)> Auth Schemes used : <>
2016-09-08	10:49:06:703	 848	604	Misc	WARNING: WinHttp: SendRequestUsingProxy failed for http://mywsusurl//selfupdate/wuident.cab. error 0x80072ee2
2016-09-08	10:49:06:703	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation MakeRequest failed. error 0x80072ee2
2016-09-08	10:49:06:703	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation failed with 0x80072ee2
2016-09-08	10:49:06:703	 848	604	Misc	WARNING: WinHttp: ShouldFileBeDownloaded failed with 0x80072ee2
2016-09-08	10:49:30:281	 848	604	Misc	WARNING: Send failed with hr = 80072ee2.
2016-09-08	10:49:30:281	 848	604	Misc	WARNING: SendRequest failed with hr = 80072ee2. Proxy List used: <(null)> Bypass List used : <(null)> Auth Schemes used : <>
2016-09-08	10:49:30:281	 848	604	Misc	WARNING: WinHttp: SendRequestUsingProxy failed for http://mywsusurl//selfupdate/wuident.cab. error 0x80072ee2
2016-09-08	10:49:30:281	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation MakeRequest failed. error 0x80072ee2
2016-09-08	10:49:30:281	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation failed with 0x80072ee2
2016-09-08	10:49:30:281	 848	604	Misc	WARNING: WinHttp: ShouldFileBeDownloaded failed with 0x80072ee2
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: Send failed with hr = 80072ee2.
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: SendRequest failed with hr = 80072ee2. Proxy List used: <(null)> Bypass List used : <(null)> Auth Schemes used : <>
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: WinHttp: SendRequestUsingProxy failed for http://mywsusurl//selfupdate/wuident.cab. error 0x80072ee2
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation MakeRequest failed. error 0x80072ee2
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: WinHttp: SendRequestToServerForFileInformation failed with 0x80072ee2
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: WinHttp: ShouldFileBeDownloaded failed with 0x80072ee2
2016-09-08	10:49:53:859	 848	604	Misc	WARNING: DownloadFileInternal failed for http://mywsusurl//selfupdate/wuident.cab: error 0x80072ee2
2016-09-08	10:49:53:859	 848	604	Setup	WARNING: SelfUpdate check failed to download package information, error = 0x80072EE2
2016-09-08	10:49:53:859	 848	604	Setup	FATAL: SelfUpdate check failed, err = 0x80072EE2
2016-09-08	10:49:53:859	 848	604	Agent	  * WARNING: Skipping scan, self-update check returned 0x80072EE2
2016-09-08	10:49:53:859	 848	604	Agent	  * WARNING: Exit code = 0x80072EE2
2016-09-08	10:49:53:859	 848	604	Agent	*********
2016-09-08	10:49:53:859	 848	604	Agent	**  END  **  Agent: Finding updates [CallerId = AutomaticUpdates]
2016-09-08	10:49:53:859	 848	604	Agent	*************
2016-09-08	10:49:53:859	 848	604	Agent	WARNING: WU client failed Searching for update with error 0x80072ee2

Hello Parth,
I have seen contradictory information on your issues with the updates being installed, as you stated here: http://ask.cloudbase.it/question/1428/unable-to-download-and-install-windows-patch-in-windows-2012-std/ that on 2008r2 the updates were installed correctly, but you are stating above that also 2008r2 updates' installation fails.
Can you restate on what versions of Windows do you encounter this issue?
Moreover, can you give some information on how your WSUS has been configured, in order to try to reproduce your setup?
The imaging scripts have not been tested with WSUS (although if WSUS server has been set correctly, it should be transparent for the scripts) and this requires more investigation from our part.
We should also move this conversation to another Github issues, as it does not apply to the issue described in the title.
Thank you,
Adrian Vladu

Hi Adrian,
I have created a new Github issue for this and provided complete details on the issue.
Thanks,
Parth

closing issue, as it is more than one year old and the issue is no longer present.


Inconsistent name in TOML vs API. [libraries] versus findDependencies

The names of the items in the TOML file all match the VersionsCatalog.find* functions except one, which is [libraries] versus findDependencies.
[versions] -> VersionCatalog::findVersion
[bundles] -> VersionCatalog::findBundle
[plugins] -> VersionCatalog::findPlugin
but this doesn't match:
[libraries] -> VersionCatalog::findDependency
Is it not a dependency until it is added to a configuration in a dependencies {} block, until then it is just a library in a catalog. :)
I propose renaming it to match:
[libraries] -> VersionCatalog::findLibrary

I have an alternative proposal, what if we renamed the TOML entry to dependencies instead? The term "libraries" is even technically incorrect here, you can define a dependency on anything in a repository, and that's not limited to just libraries, some plugins and applications are published there. I think changing the TOML key would make more sense, and it would also solve the inconsistency.

what if we renamed the TOML entry to dependencies instead?

Yeah, that probably does make more sense.

It's not a dependency either in the TOML file: it's a library definition: it's a model which helps building a library. It's even more obvious if you realize that the model actually has references to versions. A dependency is a reified view of a library, so it's a different beast: it's once the catalog is built that you get this. I am strongly against any rename in this area since renames were already made to make this more consistent with the model.

Should we rename [plugins] to [pluginLibraries] then? And VersionCatalog::findPluginDependency? It also uses this same mechanism where you can have references to versions, but it does not have this odd split. I don't think that the minor difference of having a "reference" is important enough to warrant this inconsistency (speaking of, do you have an issue/PR/commit where this was previously discussed? maybe I'm missing something brought up previously.).

There were quite a few discussions, I tried to summarize all of them here: https://gradle.cloud.xwiki.com/xwiki/bin/view/Main/Design specs/Dependency management/Version catalog/ (see 2 pages below).
The TOML file is a compromise between verbosity e.g [pluginLibraries]` and semantics. I am honestly not convinced than any rename is worth the effort at this stage. Of course you can say that I'm biased and I'm ready to accept this, but given that the API is out in the wild, I would be annoyed by breaking changes: as a plugin author it makes it painful to support multiple Gradle versions. This goes off topic into the "incubating" vs "stable" API discussion of course...

Interestingly enough, the spec still says dependencies.
This got changed through #15328 and now that I look, I do not see the relationship between renaming the file - which was the right thing ™  - and renaming that block in the TOML.
Thinking some more about it, I would advocate for:

renaming libraries to dependencies in TOML. This will align the names, preserving the semantic of what is retrieved in the build. This alignment will help feature understandability. We could still allow Gradle to parse the old style but warn about it.
Moving generated dependencies accessors under a deps top level, which would reduce even more naming conflicts and make future evolution easier if needed. I understand this is a breaking change, but that's our last chance for that. Again, we could actually generate both for a while with a deprecation warning on the old style.
Modify the settings API so that you declare dependencyAlias and pluginAlias to be explicit from the start about what you define. This again could be done with deprecation / addition.

renaming libraries to dependencies in TOML. This will align the names, preserving the semantic of what is retrieved in the build. This alignment will help feature understandability. We could still allow Gradle to parse the old style but warn about it.

I must say I don't really understand why we would need changes. We moved from dependencies to libraries precisely because dependencies is not the right thing. I don't think it helps to reintroduce the confusion.

Moving generated dependencies accessors under a deps top level, which would reduce even more naming conflicts and make future evolution easier if needed.

We actually considered it during the design and I agree from the evolution standpoint, however I think it would be a big regression in usability. Most of the use cases are only about libraries. Consider that you'd have to write libs.deps. as a prefix for every single dependency. IMHO, this is not worth the change.

Modify the settings API so that you declare dependencyAlias and pluginAlias to be explicit from the start about what you define. This again could be done with deprecation / addition.

It's a library alias, not a dependency alias.

I'm creating a dependency on the library Micronaut Core. I'm not creating a dependency on the dependency Micronaut Core.

I would argue that you are creating a dependency on a dependency here. Either that, or we need a more generic term like "dependency", as "library" simply does not work in things like:
annotationProcessor libs.dagger

Dagger isn't a library, it feels weird to call it such. It is an application or annotation processor. Similarly, there are things like Checkstyle which are also not libraries.

I don't think Gradle has to break APIs, it can support both, even if they are incubating.

Was that not implied by "This again could be done with deprecation / addition."? If not, I don't understand what you mean. If changed, we would deprecate through possibly 8.0.

I would argue that you are creating a dependency on a dependency here

I'm sorry but we'll have to disagree :) I'm creating a dependency, via a declaration found in a catalog. A dependency is a realized view. Things declared in a catalog are everything but dependencies. Calling them dependencies would re-create the confusion which originally triggered the change, which is that things declared in a catalog are not added as dependencies, so the catalog file is not a central place to put all your dependencies.

Sharing here what we are looking at doing:

Add findLibrary and deprecate findDependency
Add PluginAliasBuilder plugin(String alias, String id), LibraryAliasBuilder library(String alias, String group, String name) and void library(String alias, String groupArtifactVersion) to VersionCatalogBuilder which aligns with the sections in the TOML
Deprecate VersionCatalogBuilder.alias in favor of the above

In the long run, these changes make the TOML, creation and query APIs more consistent with each other, which we believe is more important than pure semantics.

Fixed by #19369.


[Servicebus] StopProcessingAllowsCallbackToComplete failing in nightly runs

Servicebus nightly test runs are failing with:

Error message
Expected: null
But was:  <{MessageId:c629c4bb-5ad9-4e14-b6ed-4a247b20b864}>
Stack trace
at Azure.Messaging.ServiceBus.Tests.Processor.ProcessorLiveTests.StopProcessingAllowsCallbackToComplete() in /mnt/vss/_work/1/s/sdk/servicebus/Azure.Messaging.ServiceBus/tests/Processor/ProcessorLiveTests.cs:line 592
at Azure.Messaging.ServiceBus.Tests.Processor.ProcessorLiveTests.StopProcessingAllowsCallbackToComplete() in /mnt/vss/_work/1/s/sdk/servicebus/Azure.Messaging.ServiceBus/tests/Processor/ProcessorLiveTests.cs:line 592
at Azure.Messaging.ServiceBus.Tests.Processor.ProcessorLiveTests.StopProcessingAllowsCallbackToComplete() in /mnt/vss/_work/1/s/sdk/servicebus/Azure.Messaging.ServiceBus/tests/Processor/ProcessorLiveTests.cs:line 592
at Azure.Messaging.ServiceBus.Tests.Processor.ProcessorLiveTests.StopProcessingAllowsCallbackToComplete() in /mnt/vss/_work/1/s/sdk/servicebus/Azure.Messaging.ServiceBus/tests/Processor/ProcessorLiveTests.cs:line 593
at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter1.BlockUntilCompleted()    at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)    at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func1 invoke)
at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context)
at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context)
at NUnit.Framework.Internal.Commands.TimeoutCommand.<>c__DisplayClass5_0.b__0()
at System.Threading.Tasks.Task`1.InnerInvoke()

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1071995&view=results

@jsquire for notification.

Seems to have been a transient issue - passing consistently since.


Add documentation for metadata database

Add documentation and explanation for migrations.
Addresses documentation feedback from #1582

Codecov Report

Merging #1605 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #1605     +/-   ##
=========================================
+ Coverage   46.33%   46.44%   +0.1%
=========================================
  Files          26       26
  Lines        3535     3542      +7
=========================================
+ Hits         1638     1645      +7
  Misses       1519     1519
  Partials      378      378

Impacted Files
Coverage Δ

metadata/migrations.go
46.66% <ø> (ø)
:arrow_up:

metadata/db.go
55.81% <100%> (+3.91%)
: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 8558b98...e9701b9. Read the comment docs.

LGTM
A few nits, but this is perfect!


OP-18123-Added noOfDays param for Dashboard- deployment history Api

https://devopsmx.atlassian.net/browse/OP-18123

This PR  triggered this build in this execution, please wait for it  and child pipelines to succeed before merging:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GGSKJFYB73VWZ8SCT3FM26SZ

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GGSKK3VAKXMYZ19J14G689FX

SUCCESSFUL RunTests RESULT: Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 Reviewers ,  this PR did not cause the failures in  https://jenkins.opsmx.net:8181/jenkins/job/Dev-API-Automation-Test-GitPR/581/artifact/test-output/extentreport.html


Translate into Vietnamese chap [6-9] - DONE

Finish from chapter 6 to 8.
6 -  The 🤗 Tokenizers library

[x] 1.mdx
[x] 2.mdx
[x] 3.mdx
[x] 3b.mdx
[x] 4.mdx
[x] 5.mdx
[x] 6.mdx
[x] 7.mdx
[x] 8.mdx
[x] 9.mdx
[x] 10.mdx

7 - Main NLP tasks

[x] 1.mdx
[x] 2.mdx
[x] 3.mdx
[x] 4.mdx
[x] 5.mdx
[x] 6.mdx
[x] 7.mdx
[x] 8.mdx
[x] 9.mdx

8 - How to ask for help

[x] 1.mdx
[x] 2.mdx
[x] 3.mdx
[x] 4.mdx
[x] 4_tf.mdx
[x] 5.mdx
[x] 6.mdx
[x] 7.mdx

Please feel free to review it @lewtun.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

remove

@lewtun Sure, I have just updated them.
I hope my translation can contribute to the course and the community. Thanks.

remove

@lewtun Sure, I have just updated them. I hope my translation can contribute to the course and the community. Thanks.

Great, thank you! I checked the render and it looks great :)


py3.7 incompatibility: urllib.parse.quote and ~

Testing mitmproxy-master revealed a small issues with the way urllib.parse.quote escapes the full byte range:
import urllib.parse
print(urllib.parse.quote(bytes(range(125,128)).decode("utf8", "surrogateescape")))

Output:
Python 3.6: %7D%7E%7F
Python 3.7-dev: %7D~%7F

This currently causes one of our tests to fail: test/mitmproxy/net/http/test_url.py:123.
Looking through the Python bugtracker, I found:
https://bugs.python.org/issue16285
https://github.com/python/cpython/commit/21024f06622c4c55b666adb130797a4ee205d005
https://bugs.python.org/issue12910
Can we safely update our code to ignore this, or do we generate a new pitfall by (not) handling this?
We could use the safe keyword-arg to include ~, which then results in the same behaviour on py36 an dpy37.

I think we can ignore quoting of ~ in our test suite. My reading is that both of these behaviours are acceptable, but that the behaviour in 3.7 is more correct. I can't think of a reason why we need to enforce one or the other.


Generate node checksum

More info is needed to do this properly. Will look into this later.

This has been handled for CE. Need to look into H2 later since it uses a checksum for JMA.

Should be good


Add each AI parameter?

like reset in bard and Claude. System message, temperature for api mode

Thanks, that's a good idea. For now, it's useful for ordinary, fast, rush, emergency usage. I haven't stepped into such details yet. I am considering whether it's worth devoting proper effort to developing those features.

I think there aren't many projects that combine multiple AI. Aih is easier to set up and use. Maybe we should start with adding option to use gpt-4 API?

The new command .price has been added, and the new binaries have been uploaded concurrently. It appears to be an elegant solution for simplicity.

Please check it & do not hesitate to give valuable suggestions.
Thank you.

it seems bug, not switching to gpt api? maybe combine with .chatapi and prompted selection like above instead?

Thank you!
Fixed in new uploaded binaries with auto switching to .chatapi.

Yes, combining selection with .chatapi is a good idea, but it will trigger prompted selection everytime when user type .chatapi. It seems .price as a separated configuration sub-command better.
Another reason is that .chatapi will run into the exist conversation without reopening a new one with the selection.
Please testing and feel free to give valuable suggestions.

Please note that the .price command has been changed to .chatapi., but .price can still be used for compatibility purpose.
This is because .chatapi. could establish a uniform style like .chatapi. as the configuration parameter setting of .chatapi then .bing. for .bing, .bard. for .bard, etc.
Hope the setup and usage process is easy for users.
Very appreciate your valuable suggestions.
Thank you.

Hi, i have new suggestion. i wonder we can use / command like autocompletion or simplify all selection under .chat and .key . Next maybe export/save chat feature  would be good.
Chat-related:
.chat - ChatGPT Web (free)
.chatapi - ChatGPT Api (pay)
.chatapi. - Set ChatGPT API mode such as GPT3Dot5Turbo(default), GPT4, GPT432K
-gpt-3.5-turbo, $0.002/1K tokens
-gpt-4 8K Prompt, $0.03/1K tokens
-gpt-4 8K Completion, $0.06/1K tokens
-gpt-4 32K Prompt, $0.06/1K tokens
-gpt-4 32K Completion, $0.12/1K tokens
.bard - Google Bard (in chat)
.bing - Bing Chat (in chat)
.claude - Claude (via Slack) (in chat)
Key-related:
.bardkey - Reset Google Bard cookie
.bingkey - Reset Bing Chat coolie
.chatkey - Reset ChatGPT Web accessToken
.chatapikey - Reset ChatGPT Api key
.claudekey - Reset Claude Slack keys

Hi, insightful advice & should be holistically considered for the benefits of potential users who are dwelling in terminal and seek to intentionally combine the logical power of artificial intelligence.

Hello there! I have made an improvement according to your advice. Now the .bardkey,.bingkey,.chatkey,.chatapikey, .claudekey are all integrated into a single command: .key. Hope this upgrade fits for the convenience of valuable users.
Thank you for using Aih.


Owl-Bot is overwriting version updates in requirements.txt

Should we rely on templates copied by Owl-Bot or maintain these files manually in this repository?

Fixed by #5722


Widget Initialize Exception Message

The exception message when skeleton initialization fails does not mention where the template definition is. It would be very helpful to know which skeleton is actually raising the exception.

Where the issue occurred is included within the stack trace, but it would be easier to find if it were printed with the exception message.


Set BakedLightmap data to be stored in a separate file and as a binary file by default. And add a custom file extension to BakedLightmap data binary.

Godot version:
3.1.1 Stable
Issue description:
Wen you bake a lightmap in Godot it gets stored as text in the file scene, this is a problem wen handling complex scenes, saving the whole scene can take up to minutes, even if you do something so simple as to move a single cube.
Also if you manually save the BakedLightmap data in a separate file as a text file the same problem occurs the first time you save the lightmap. Saving the lightmap as a binary speeds this up but you have to manually do it every time you re-bake a scene.
My proposal is to have BakedLightmap data to be saved in a separate file as a binary by default, also to have a custom file extension to BakedLightmap data binary, something like *.lm.

It makes sense indeed to avoid writing binary data to text-based scene files.
CC @JFonS

Note that this issue also occurs with GIProbe (last time I checked). See also #16783.

This makes sense, I will make sure to include these changes in the new light mapper.

This should be checked again with the upcoming new lightmapper in Godot 4.0.

This was implemented in the new CPU lightmapper for 3.2.4 and the new GPU lightmapper for 4.0, closing.


cargo test with arguments doesn't work as expected, causing tests failed (with clap crate)

Problem

The arguments for cargo test are actually not belong to the binary, but they are passed to it.
It causes unexpected behavior.

See also: https://github.com/clap-rs/clap/discussions/2187
Steps

$ cat Cargo.toml
[package]
name = "a"
version = "0.0.1"
edition = "2018"
[dependencies]
clap = "2.33"

$ cat src/main.rs
fn test_a() {
    let _matches = clap::App::new("a").get_matches();
    assert!(true);
}

$ cargo test -- --nocapture
    Finished test [unoptimized + debuginfo] target(s) in 0.12s
     Running target/debug/deps/a-3805593461b4c059

running 1 test
error: Found argument '--nocapture' which wasn't expected, or isn't valid in this context

USAGE:
    a-3805593461b4c059

For more information try --help
error: test failed, to rerun pass '--bin a'

Possible Solution(s)
Not obligatory, but suggest a fix/reason for the bug, -->
Consume the cargo test specific arguments and not pass them to the binary.
or ideas how to implement the addition or change -->
Redesign the arguments for different programs:
Examples:

Use -- to seperate: cargo test test_a --nocapture -- --version --debug,

The part before -- is for the cargo-test (test_a --nocapture)
The part after -- is for the binary (--version --debug)

Something like rsync -e: cargo test test_a --nocapture --args="--version --debug"

All are for cargo-test (test_a --nocapture --args="--for-binary")
The value of --args is for the binary (--version --debug)

Notes
Output of cargo version:

Sorry, I'm a bit confused what you are trying to do here.  It is expected behavior that arguments are passed to the test executable.  If you want to write tests using clap, I would suggest not parsing the process arguments directly, but instead passing them in manually with something like get_matches_from.  You can probably grab the arguments from the process using env::args_os and skipping until a -- separator, and then pass in the result.

@ehuss Any suggestion for those questions?

How do I test my program with the args not allowed by cargo test? for example:

cargo test -- --config=local.json

How do I test my program having same names of options from cargo test?

cargo test -- --format=template.json

How do I setup clap that I can ignore all possibilities that cargo test brings into my program?

cargo test -- -h
...

It's a very unusual way to test arguments on the command-line, since they are global every test will see the same arguments, and requires manually passing in the arguments.  Tests are usually self-contained, and provide the values they need within the test.  It doesn't look like there is a way to make the libtest harness ignore the arguments.  Here are several options for you.
No harness
Disable the harness in Cargo.toml:
[[test]]
name = "foo"
harness = false

And then in tests/foo.rs:
// Run with: cargo test --test foo -- --config=abc
fn main() {
    let matches = clap::App::new("a")
        .arg_from_usage("-c --config=<FILE> 'Sets a configuration file to use'")
        .get_matches();
    println!("{:?}", matches.value_of("config"));
}

Self-contained values
Usually, in a test, you don't pass arguments in on the command-line, but instead embed them in the test like this:
#[test]
fn foo() {
    let args = ["a", "--config", "abc"];
    let matches = clap::App::new("a")
        .arg_from_usage("-c --config=<FILE> 'Sets a configuration file to use'")
        .get_matches_from(&args);
    assert_eq!(matches.value_of("config"), Some("abc"));
}

Executing a binary
Write an integration test that launches your binary (using std::process::Command), and passes the arguments to the process, and verify its behavior.  That's what Cargo itself does (see https://doc.crates.io/contrib/tests/writing.html#basic-test-structure).
Considering this is unrelated to Cargo, please follow up with any other questions on how to write tests or use clap to one of the user forums (such as urlo).

@ehuss Thanks for the solutions. Those are really helpful!
I'm gonna to close this and move to forums.


uploading hasSBOMs for guacdiff tests in guacsec/guac

These files are hasSBOM structs marshalled to JSON to aid testing of guacanalyze/guacdiff in guacsec/guac.

@nathannaveen Please review and merge when you are able to. Thanks.

Hey @arorasoham9 could you also move pkg/analyzer/test_HasSBOMs_diff.json from your PR in guacsec/guac to this test repo?

@nathannaveen I anyway pushed the last commit. You will see the pkg/analyzer/test_HasSBOMs_diff.json file removed now.


Table size smaller when exporting

When exported, the size of the table is always the same, regardless of the settings made for its size in the jupyter notebook (when viewing it in the Jupyter notebook it has the specified size, but when exporting it is minimal). Therefore, it is impossible to see the letters and what is written with many rows and few columns.

Can you attach an example here?

Of course, when I customize the table so that it looks the way I want it, it returns as expected in the Jupyter notebook. However, when I export it, it always stays the same size, regardless of the changes I make.

You can not use table selector with set_table_styles, you should use '*' if you want to select whole table.
df = pd.read_csv(
    "tests/notebooks/data/covid19.csv", parse_dates=["date"], index_col="date"
)

s1 = dict(
    selector="*",
    props=[
        ("font-family", "Source Sans Pro"),
        ("font-size", "20px"),
        ("margin", "30px auto"),
        ("border-collapse", "collapse"),
        ("border", "1px solid #eee"),
        ("border-bottom", "2px solid #00cccc"),
    ],
)

styles = [s1,dict(selector="caption", props=[("caption-side","bottom")])]

s_df = df.head(30).style.set_table_styles(styles).hide(axis='index').set_caption("test")
s_df

dfi.export(s_df, "s_df.png")


feat(upload): Accept HTTP 200 responses with empty body

Summary
When using a third party server, we may encounter valid HTTP 200 responses that do not include a JSON body.
In the current SDK it is assumed that the response always has a JSON form and the request fails otherwise even though the response code is 200.
The failure is due to the creation of a JSONObject(responseBody) object that is passed to the function that creates the HTTP response. If that responseBody is null, empty or does not contain a json string, that call will throw an exception and the request is considered a failure.
In this change, if the response is null or empty, it is still passed to the function that creates the response, and, if it is a 200 response, it will be considered a successful response.
This behavior is equivalent to the behavior of the Amplitude Swift SDK, that accepts empty responses (https://github.com/amplitude/Amplitude-Swift/blob/35f253db41cf68d220390946c7ad7f344665d763/Sources/Amplitude/Utilities/HttpClient.swift#L35)
Checklist

[X] Does your PR title have the correct title format?
Does your PR have a breaking change?:  No

Changes included in https://github.com/amplitude/Amplitude-Kotlin/pull/185 (thanks!) closing this one.


React Server-Side Rendered App as Login UI for IdS

So my question is more about your people opinion about the following solution:
ASP .NET Core app with Identity Server with login/logout etc UI with server-side rendered React. I know that most of people will ask 'why do you want to complicate your life like that, just use MVC and Razor pages'...I don't want to, but I have very persistent people in my company that would like everything to be in React so that we can easily share react components.
Looking for solutions I've found this issue: https://github.com/IdentityServer/IdentityServer4/issues/3181 and for obvious reason we don't want to use Resource Owner Password Credentials Grant Type.
Do you think it is possible without exposing ourselves to issues with security of such solution? Would SSR react be as secure as Razor pages? Any issues that you can think of?
Would appreciate a bit of your insight.

Make your UI anything you want, as long as it triggers the right ASP.NET Core cookie authentication handler to emit the cookie so that IdentityServer can authenticate requests.


Small improvement of MediatR.Examples.Windsor

Better checking of baseRequestType and checking assumptions with Debug.Assert instead of writing assumptions as comments

Closing this pull request because I'll open another one with more important fixes.


Using Unique & Index decorators creates duplicate indexes

Describe the bug
I recently upgraded from 5.3.1 to 5.6.8 and ran into some issues when creating a schema in my test setup. SchemaGenerator.createSchema was failing because postgres was throwing a TableExistsException error due to too many create index statements in the generated SQL. The indexes in question were defined using the Index and Unique decorators. Removing those decorators in favor of using the index and unique options on the Property decorator fixed the issue.
To Reproduce
Steps to reproduce the behavior:

Create an entity using the Unique and Index decorators. An example adapted from the docs:

@Entity()
export class Author {
  @PrimaryKey()
  id: string;

  @Property()
  @Unique()
  email!: string;

  @Index()
  @Property()
  age?: number;

  @Index({ name: "born_index" })
  @Property()
  born?: Date;
}

Create the schema using the SchemaGenerator or inspect the output of SchemaGenerator.getCreateSchemaSQL:

const schemaGenerator = orm.getSchemaGenerator();
console.log(await schemaGenerator.getCreateSchemaSQL());

Notice that the generated sql contains repeated create index and alter table commands for the same indexes & constraints:

    create table "author" ("id" varchar(255) not null, "email" varchar(255) not null, "age" int not null, "born" timestamptz(0) not null, constraint "author_pkey" primary key ("id"));
    alter table "author" add constraint "author_email_unique" unique ("email");
    create index "author_age_index" on "author" ("age");
    create index "born_index" on "author" ("born");
    create index "author_age_index" on "author" ("age");
    create index "born_index" on "author" ("born");
    create index "author_age_index" on "author" ("age");
    create index "born_index" on "author" ("born");
    alter table "author" add constraint "author_email_unique" unique ("email");
    alter table "author" add constraint "author_email_unique" unique ("email");

Expected behavior
The indexes and constraints should only be included once:
    create table "author" ("id" varchar(255) not null, "email" varchar(255) not null, "age" int not null, "born" timestamptz(0) not null, constraint "author_pkey" primary key ("id"));
    alter table "author" add constraint "author_email_unique" unique ("email");
    create index "author_age_index" on "author" ("age");
    create index "born_index" on "author" ("born");

Additional context
The above expected behavior was created with the schema generator, but with using the index/unique options from Property instead:
@Entity()
export class Author {
  @PrimaryKey()
  id: string;

  @Property({ unique: true })
  email!: string;

  @Property({ index: true })
  age?: number;

  @Property({ index: "born_index" })
  born?: Date;
}

Versions

Dependency
Version

node
v14.18.1

typescript
4.6.3

mikro-orm
5.6.8

mikro-orm/postgresql
5.6.8

I will need a complete reproduction for this, there are many tests using the decorators and asserting the schema.

Are you sure this is not just a caching issue? Or maybe you have multiple core packages installed? This often happens when you forget to update some of the ORM packages, e.g. the cli that is in the dev deps.

Are you sure this is not just a caching issue?

I don't believe so -- we're not doing any explicit caching and the problem persists if I add cache: { enabled: false} to the MikroORM config. Is there something other cache that I should try to clear out?

Or maybe you have multiple core packages installed? This often happens when you forget to update some of the ORM packages, e.g. the cli that is in the dev deps.

I checked my dependencies and they're all 5.6.8, so I don't believe that's the issue.

I will need a complete reproduction for this, there are many tests using the decorators and asserting the schema.

I tried to create a repo with a reproduction, but things work as expected in a new repo. I'm not sure what I'm doing wrong in my application code, but I guess I'll keep digging. We're using the wildcard for multiple schemas, so I assume it's something there, but adding that in the minimal reproduction doesn't cause any issues 🤔
Sorry for the noise here!

I dug a bit more into this and found that our test framework (jest) is somehow importing the entity files multiple times. This is causing the decorators to be executed multiple times and the index to be registered multiple times in the metadata, which is why it shows up more than once in the generated SQL.
I'm still not certain how/why jest is importing the files multiple times, but I wonder if the decorators should check the existing metadata to not add duplicate entries to indexes/uniques?

For simple indexes it could work, but you can also have compound indexes, and indexes on the entity level, etc.

@B4nan this happens when mocking modules specifically without providing the factory param, and when those modules import entities (either directly or indirectly by importing other files that import entities)
I made a repo to reproduce it https://github.com/tduenez/mikro-index-bug


Add page where users reviews are displayed

Description

Frontend

Added a reviews tab on the sidebar (/reviews) where the currently logged in user's reviews are displayed.
Users can delete their reviews by clicking the delete button under the review they wish to delete
Reviews page is not displayed if the user is not logged in

Backend

Updated the DELETE /reviews endpoint to allow users to delete their own reviews

Screenshots
https://user-images.githubusercontent.com/22901073/222995462-ba5ceb69-82ed-44d5-8484-a5271c60f4dd.mov

Steps to verify/test this change:
Final Checks:

[ ] Verify successful deployment
[ ] Delete branch

(optional)

[ ] Write tests
[ ] Write documentation

Closes #193

Once #291 is merged into master and working, feel free to check real quick that this feature works and merge as well

Merged #291. You can go ahead and update the branch and test it out. Also don't forget to update any api calls you've added to start with /api.


504 Lambda response code when updating

Hello !
I am actually trying to deploy a Django application on AWS Lambda and I got this issue when I try to update
Calling update for stage dev..
Downloading and installing dependencies..
 - scipy==1.1.0: Using locally cached manylinux wheel
 - psycopg2==2.7.5: Using locally cached manylinux wheel
 - psycopg2-binary==2.7.5: Using locally cached manylinux wheel
 - pandas==0.23.4: Using locally cached manylinux wheel
 - numpy==1.15.0: Using locally cached manylinux wheel
 - sqlite==python36: Using precompiled lambda package
Packaging project as gzipped tarball.
Target directory /Users/ravisha/Documents/MyFleetBuy_Server/myfleetbuy/handler_venv/lib/python3.6/site-packages/zappa-0.46.2.dist-info already exists. Specify --upgrade to force replacement.
Target directory /Users/ravisha/Documents/MyFleetBuy_Server/myfleetbuy/handler_venv/lib/python3.6/site-packages/zappa already exists. Specify --upgrade to force replacement.
You are using pip version 10.0.1, however version 18.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Downloading and installing dependencies..
 - sqlite==python36: Using precompiled lambda package
Packaging project as zip.
Uploading myfleetbuy-dev-1534529408.tar.gz (151.5MiB)..
100%|█████████████████████████████████████████████████████████████████████████████████████████████| 159M/159M [00:25<00:00, 6.12MB/s]
Uploading handler_myfleetbuy-dev-1534529655.zip (10.5MiB)..
100%|███████████████████████████████████████████████████████████████████████████████████████████| 11.0M/11.0M [00:01<00:00, 5.52MB/s]
Updating Lambda function code..
Updating Lambda function configuration..
Uploading myfleetbuy-dev-template-1534529692.json (1.6KiB)..
100%|███████████████████████████████████████████████████████████████████████████████████████████| 1.62K/1.62K [00:00<00:00, 32.3KB/s]
Deploying API Gateway..
Scheduling..
Unscheduled myfleetbuy-dev-zappa-keep-warm-handler.keep_warm_callback.
Scheduled myfleetbuy-dev-zappa-keep-warm-handler.keep_warm_callback with expression rate(4 minutes)!
Error: Warning! Status check on the deployed lambda failed. A GET request to '/' yielded a 504 response code.

Also zappa tail seems to display this error but I don't know if it's linked
[1534520729301] Instancing..
[1534520729303] Failed to find library...right filename?
[1534520729337] No module named 'django': ModuleNotFoundError
Traceback (most recent call last):
  File "/var/task/handler.py", line 567, in lambda_handler
  return LambdaHandler.lambda_handler(event, context)
  File "/var/task/handler.py", line 237, in lambda_handler
  handler = cls()
  File "/var/task/handler.py", line 143, in __init__
  wsgi_app_function = get_django_wsgi(self.settings.DJANGO_SETTINGS)
  File "/var/task/zappa/ext/django_zappa.py", line 9, in get_django_wsgi
  from django.core.wsgi import get_wsgi_application
ModuleNotFoundError: No module named 'django'

I am on MacOS and my pip freeze results are
argcomplete==1.9.3
base58==1.0.0
boto3==1.7.78
botocore==1.10.78
certifi==2018.8.13
cfn-flip==1.0.3
chardet==3.0.4
click==6.7
Django==2.1
django-filter==2.0.0
django-oauth-toolkit==1.2.0
django-rest-framework==0.1.0
django-webpack-loader==0.6.0
djangorestframework==3.8.2
docutils==0.14
durationpy==0.5
future==0.16.0
hjson==3.0.1
idna==2.7
jmespath==0.9.3
kappa==0.6.0
lambda-packages==0.20.0
numpy==1.15.0
oauthlib==2.1.0
pandas==0.23.4
placebo==0.8.1
psycopg2==2.7.5
psycopg2-binary==2.7.5
python-dateutil==2.6.1
python-slugify==1.2.4
pytz==2018.5
PyYAML==3.12
requests==2.19.1
s3transfer==0.1.13
scipy==1.1.0
six==1.11.0
toml==0.9.4
tqdm==4.19.1
troposphere==2.3.1
Unidecode==1.0.22
urllib3==1.23
Werkzeug==0.14.1
wsgi-request-logger==0.4.6
zappa==0.46.2

And my zappa settings
{
    "dev": {
        "aws_region": "us-west-1",
        "django_settings": "myfleetbuy.settings",
        "profile_name": "default",
        "project_name": "myfleetbuy",
        "runtime": "python3.6",
        "s3_bucket": "zappa-3uk7ut9lh",
        "slim_handler": "true"
    }
}

Thanks !

Same proble. did you find a solution?

same problem

@3nomis @rolexsanches I'm sorry but I don't remember exactly what fixed this for us. We did however reduce the size of our deployed code which I believe helped a lot of the problems we were having.
Just in case it helps, as part of our deployment we remove nearly all of the code in our node_modules/ after we run webpack. This helped save a lot of space. We also compiled our own versions of some of our python dependencies such as opencv, numpy and lxml. We were able to compile just the parts that we needed in order to save space.

I've found out that my problem was windows. When I changed OS everything worked smoothly. This to remind me to never ever use windows for programming


[FLINK-21403][tests] Some tests expect to fail if parallelism cannot be met

See commit messages.

Looks like the RM fails slot requests with a different cause if no TM is connected.
Caused by: org.apache.flink.runtime.resourcemanager.exceptions.UnfulfillableSlotRequestException: Could not fulfill slot request f53b49e86b45419d303e2a9922c9f0b4. Requested resource profile (ResourceProfile{UNKNOWN}) is unfulfillable.
	at org.apache.flink.runtime.resourcemanager.slotmanager.SlotManagerImpl.setFailUnfulfillableRequest(SlotManagerImpl.java:651) ~[classes/:?]
	at org.apache.flink.runtime.resourcemanager.ResourceManager.setFailUnfulfillableRequest(ResourceManager.java:1371) ~[classes/:?]
	at org.apache.flink.runtime.resourcemanager.StandaloneResourceManager.lambda$startStartupPeriod$0(StandaloneResourceManager.java:123) ~[classes/:?]
	at org.apache.flink.runtime.rpc.akka.AkkaRpcActor.handleRunAsync(AkkaRpcActor.java:440) ~[classes/:?]
	at org.apache.flink.runtime.rpc.akka.AkkaRpcActor.handleRpcMessage(AkkaRpcActor.java:208) ~[classes/:?]
	... 20 more

I will fix it by reverting back to 1 TM and adjusting the job to not be runnable on a single slot.

Does it make sense to make UnfulfillableSlotRequestException a subclass of NoResourceAvailableException? Would this solve the problem?

That should also solve the problem. I think that it would make sense, but am not sure whether it is possible while maintaining the existing class structure.
UnfulfillableSlotRequestException < ResourceManagerException < FlinkException
NoResourceAvailableException < JobException

Is there a reason for why we do the exception classification via sub/parent classes, instead of interfaces?

I think the reason why UnfulfillableSlotRequestException inherits from ResourceManagerException is because it originates on the ResourceManager. I think it should be fine to change it and to make it a subclass of NoResourceAvailableException.
Another idea could be to map UnfulfillableSlotRequestException to NoResourceAvailableException on the JobMaster if we want to keep these things separated. This would then also mean that ResourceManager should not throw a NoResourceAvailableException with the argument that it is a JobException which should only originate from the JobMaster/Dispatcher.


[resotocore][fix] Add workflows command

Description
Add workflows command which also replaces the start_task command which is removed.
To-Dos

[x] Add test coverage for new or updated functionality
[x] Lint and test with tox
[ ] Document new or updated functionality (someengineering/resoto.com#XXXX)

Issues Fixed

Fixes #535

Code of Conduct
By submitting this pull request, I agree to follow the code of conduct.

Codecov Report

Merging #664 (438e002) into main (0326ca1) will increase coverage by 0.00%.
The diff coverage is 88.63%.

@@           Coverage Diff           @@
##             main     #664   +/-   ##
=======================================
  Coverage   89.01%   89.01%
=======================================
  Files          64       64
  Lines        8575     8603   +28
=======================================
+ Hits         7633     7658   +25
- Misses        942      945    +3

Flag
Coverage Δ

resotocore
89.01% <88.63%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

resotocore/core/task/__init__.py
72.00% <72.00%> (ø)

resotocore/core/cli/command.py
85.96% <93.47%> (+0.11%)
:arrow_up:

resotocore/core/__main__.py
87.61% <100.00%> (ø)

resotocore/core/cli/model.py
96.31% <100.00%> (ø)

...re/core/task/start_workflow_on_first_subscriber.py
96.87% <100.00%> (ø)

resotocore/core/task/task_handler.py
84.46% <100.00%> (+0.08%)
:arrow_up:

resotocore/core/web/api.py
65.05% <100.00%> (ø)

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 0326ca1...438e002. Read the comment docs.


fix: simplify api test dependencies

Looking to see if there are ways to optimize the testing of the API provider by reducing the dependencies given the current constraints (or lack-thereof).
By removing ingress and allocating hostport on the test pod directly we can still execute the API request.
This reduces the github action time from 4:39 to 2:45 for the E2E Testing check.

Would love feedback from @TheFutonEng and @btlghrants

I agree with this PR as well.  Reducing the cycle time at this stage of development is a clear win.


Efficient AVX512  implementation in 'InnerProductSIMD16ExtAVX512' Function

InnerProductSIMD16ExtAVX512 functions are implemented using the more efficient AVX512 instruction set

Hi @yurymalko, Can you please to review the code?

Hi @aurora327,
Thank you for the PR! I am slow to respond currently due to sickness, sorry.
I wonder, how much improvement did you see in the tests with the better implementation?

hi, @yurymalkov
Different  Size of vector1 and vector2 of the passed parameters have different performance gains, my own dataset build on 4th Generation Intel® Xeon® resulted in a 2% to 10% end-to-end improvement with 4 cores bound.
I hope you feel better soon :)

Thanks again for the PR! I've also checked the query performance, it is up to 15% for 16-dim.
I also wonder if aligned/unaligned memory makes a difference for the current architectures?

No differences were observed since the avx512 instruction used handles both aligned and unaligned data well. At the same time, the aligned buffer is highly recommended if possible.


TypeError: hiddenTextarea.setAttribute is not a function

7.1.1 gives this error:
.../node_modules/react-textarea-autosize/dist/react-textarea-autosize.cjs.js:31
hiddenTextarea.setAttribute('tab-index', '-1');
^
TypeError: hiddenTextarea.setAttribute is not a function

Please stick to 7.1.0 - im on the road, will fix it when i get to my destination. Do u see this error in the browser or while SSRing?

Yep sticking to 7.1.0. While SSR.
Piotr

On 30 Oct 2019, at 09:46, Mateusz Burzyński<EMAIL_ADDRESS>wrote:
Please stick to 7.1.0 - im on the road, will fix it when i get to my destination. Do u see this error in the browser or while SSRing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/andreypopp/react-textarea-autosize/issues/253?email_source=notifications&email_token=AAUTG37QZOST5X4AN4OC7TLQRFCUZA5CNFSM4JGVBIY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECTLDGA#issuecomment-547795352, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAUTG35QCH4SENUD2DZHY63QRFCUZANCNFSM4JGVBIYQ.

That was quick. Thank you!


[ID-34]Remove user data

Description
Remove user data
**Resolves #34
Review Time Estimate

[ ] Immediately
[ ] Within a week
[X] When possible

Type of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:

[x] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Hi @petyos, the reason for the error on the build is that i couldn't generate the mocks for some reason. I use the way that is written in the convensions but it hits the wall every time. Could you please help me? Thank you.

Hi @stefanvit ,
Please try:
cd core
mockery --name=Storage


Language Server Crash

Hello,
after updating to my local build to commit c3acda0798f9b10ac3187ad941bbd8af82fb84a1 I'm experiencing unconditional crashes of the clangd language server. I'm running on windows and building clangd with MSVC++ 16.8.3.
Here's the clangd's error information (debug build):
 #0 0x00007ff744c85b68 llvm::StringMapEntryStorage<bool>::setValue(bool const &) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\StringMapEntry.h:53:0
 #1 0x00007ff744c7415a clang::clangd::DirectoryBasedGlobalCompilationDatabase::broadcastCDB(struct clang::clangd::DirectoryBasedGlobalCompilationDatabase::CDBLookupResult) const E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\GlobalCompilationDatabase.cpp:518:0
 #2 0x00007ff744c73c69 clang::clangd::DirectoryBasedGlobalCompilationDatabase::lookupCDB(struct clang::clangd::DirectoryBasedGlobalCompilationDatabase::CDBLookupRequest) const E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\GlobalCompilationDatabase.cpp:478:0
 #3 0x00007ff744c731fc clang::clangd::DirectoryBasedGlobalCompilationDatabase::getCompileCommand(class llvm::StringRef) const E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\GlobalCompilationDatabase.cpp:377:0
 #4 0x00007ff744c74671 clang::clangd::OverlayCDB::getCompileCommand(class llvm::StringRef) const E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\GlobalCompilationDatabase.cpp:574:0
 #5 0x00007ff744b0b9b2 <lambda_79a7e5852a2b031f5703c3c9ddd8a1a8>::operator() E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\TUScheduler.cpp:623:0
 #6 0x00007ff744af66b2 llvm::detail::UniqueFunctionBase<void>::CallImpl<class <lambda_79a7e5852a2b031f5703c3c9ddd8a1a8>>(void *) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\FunctionExtras.h:196:0
 #7 0x00007ff74364549e llvm::unique_function<(void)>::operator()(void) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\FunctionExtras.h:355:0
 #8 0x00007ff744aeef77 clang::clangd::`anonymous namespace'::ASTWorker::run E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\TUScheduler.cpp:1083:0
 #9 0x00007ff744b0c3c3 <lambda_8496735dcc1912dcaf8ff2b15ebf07d0>::operator() E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\TUScheduler.cpp:579:0
#10 0x00007ff744af6702 llvm::detail::UniqueFunctionBase<void>::CallImpl<class <lambda_8496735dcc1912dcaf8ff2b15ebf07d0>>(void *) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\FunctionExtras.h:196:0
#11 0x00007ff74364549e llvm::unique_function<(void)>::operator()(void) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\FunctionExtras.h:355:0
#12 0x00007ff74510e2d3 <lambda_3b9785320f1cf733474d10195abe6ef8>::operator() E:\projects\3rdparty-build\llvm-project\clang-tools-extra\clangd\support\Threading.cpp:94:0
#13 0x00007ff74510d432 llvm::detail::UniqueFunctionBase<void>::CallImpl<class <lambda_3b9785320f1cf733474d10195abe6ef8>>(void *) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\FunctionExtras.h:196:0
#14 0x00007ff74364549e llvm::unique_function<(void)>::operator()(void) E:\projects\3rdparty-build\llvm-project\llvm\include\llvm\ADT\FunctionExtras.h:355:0
#15 0x00007ff74375c9df threadFuncAsync E:\projects\3rdparty-build\llvm-project\llvm\lib\Support\Windows\Threading.inc:35:0
#16 0x00007ff8118d542c (C:\WINDO[Error - 15:39:21] Connection to server got closed. Server will not be restarted.

If you can guide me how to debug this issue further, I will happily try to do so. The crashes happen so quickly that I cannot attach the debugger before it happens (I suppose there is a non-naive way to go about this).
Update bisect is underway

Do you get any assertions firing? Also can you disable background indexing and see if the crash still happens

No assertions, turning off Background-indexing did not prevent the crashes. I will report back once I'm done bisecting... This takes roughly 10 minutes per step (about 6 remaining) as for some reason I always get full rebuilds on every step... :(

Commit 92dd077af1ff89929f5502c6c887358f51d5afc1 introduced the crash.

If I change GlobalCompilationDatabase.c:333 to actually validate that find() did not return end() before dereferencing it, the crash is gone.

Thanks! The entry is supposed to always be in the map, I think this is a case-sensitivity error.
All FileAncestors are necessarily in DirectoryHasCDB. However we do the lookup using Dir->path() which may not be the exact FileAncestor used to look it up, as the lookup ignores case (on windows + mac).

Does llvm/llvm-project@74b3acefc7b6355e89bb9b09dc88a5948f65c342 fix this?

Yes, the crash is gone, thank you!
Now I can really try to set up a minimal repro of my template code crashing the server, before which I wanted to update the sources to trunk :D
Btw, who is in charge of the VS-Code-Plugin? There seem to be issues with the CDB-Update-Detection and language server restarting. For one, the indexing-indicator does not seem to be disposed if the current indexing run has not yet completed, leading to an endless indicator.

Yes, the crash is gone, thank you!
Now I can really try to set up a minimal repro of my template code crashing the server, before which I wanted to update the sources to trunk :D
Btw, who is in charge of the VS-Code-Plugin? There seem to be issues with the CDB-Update-Detection and language server restarting. For one, the indexing-indicator does not seem to be disposed if the current indexing run has not yet completed, leading to an endless indicator.

That bug is in apparently in vscode and pertains to all language server implementations that use progress bars not being disposed when servers are unexpectedly terminated.

@decimad That bug has now been fixed and will land next release of the vscode plugin

@decimad That bug has now been fixed and will land next release of the vscode plugin

@njames93: thanks for the good news!

@njames93: thanks for the good news!

Closing this because llvm/llvm-project@74b3ace resolved the crash from the initial report.

Closing this because llvm/llvm-project@74b3ace resolved the crash from the initial report.


re-enable doodle subproject

queued:
https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/4156/


Add reflection

This is my reflection; @linamacromer if you can merge that would be perfect.

Looks good! 👍


Automated cherry pick of #4283

Cherry pick of #4283 on release-5.18.

#4283: Promote archived channels to Beta

/cc  @jasonblais

@crspeller Can you help with a merge? Thanks!


@mzikherman - Some Artsy Primer QA

Closes a few of the issues outlined here: https://github.com/artsy/force/issues/978
Two things:

Separates the onFavorited callback into three separate methods, in order to control the order in which they appear in the favorites window.
Actually sets the notes field on the user.

Thank you for your time.

🥗


Try to fix #2296 and support to get wasm file through http protocol

Fix #2296

Thanks! Looks like the deno CI is failing though?

Thanks! Looks like the deno CI is failing though?

 Now it has been fixed and passed ci.

In deno, the usual method is to get the library directly from remote.
e.g. import {...} from'https://deno.land/x/lib/mod.ts',
But Deno.readFileSync can only read wasm files from the local file system.
So I use fetch to complete the support for getting wasm files from http.

cc @jakobhellermann mind taking a look at this?

Can this get some movement? This issue makes it almost impossible to publish a Deno library that uses wasm-bindgen without modifying the generated output.

I'll admit I don't fully understand the original issue. Given that Deno supports all the required Web APIs, doesn't --target web just work in Deno?

I'll admit I don't fully understand the original issue. Given that Deno supports all the required Web APIs, doesn't --target web just work in Deno?

It almost works with the web target. Deno's fetch doesn't support fetching local files, so it can't fetch the wasm binary.

Deno's fetch doesn't support fetching local files, so it can't fetch the wasm binary.

Oh I see, that's unfortunate.

Glad someone saw this problem.  Is there any resistance to this problem at the moment?
How can I help?

It'd be great to get some movement on this PR, this issue is a blocking one for me.

@juzi5201314 Didn't see the updates following the last round of reviews, thanks, code looks a lot better!


Duplicate run when use multiple time & Add injectServer & Remove throw error it make app crash

Duplicate when use multiple time.
Fix by using new Elysia() and name it using Elysia({ name: "elysia-ip" })
Planing add option injectServer like
https://github.com/rayriffy/elysia-rate-limit/blob/main/example/muliInstanceInjected.ts
for fix issue like https://github.com/rayriffy/elysia-rate-limit/issues/29
Remove throw error it make app crash
https://github.com/gaurishhs/elysia-ip/blob/f46952e0baa39e63aaabe19a2ada291e8ee87036/src/index.ts#L49
Fixed in Fork
https://github.com/armada45-pixel/elysia-ip/blob/main/src/index.ts

I don't see any changes in the fork

Duplicate
https://github.com/armada45-pixel/elysia-ip/blob/7822c81fe9490b4bf1c235cd29465dfcc8151a0d/example/issue.ts#L18-L42

Remove throw error
https://github.com/gaurishhs/elysia-ip/blob/f46952e0baa39e63aaabe19a2ada291e8ee87036/src/index.ts#L49

Fixed https://github.com/gaurishhs/elysia-ip/issues/14


coconut keto bread

A new recipe I made to the best recipe website.

Hey, noticed small mistakes :)
4. Pre-heat your oven to 350°F (175°C) --> missing full stop.
5. Pour the dough evenly into a Bread/Loaf form (coated with butter or parchment paper) and cook for 35min to 45mina --> same as above, and should be "35 to 45 min." (spelling/spacing)

https://english.stackexchange.com/questions/2794/punctuation-with-units

Thank you. Will be fixing in a bit.


[Examples] Upgrade for the new format

I have forgotten to take into account the examples in my latest PR to use getMuiTheme (https://github.com/callemall/material-ui/pull/3076).
I thought it was a good opportunity to clean our examples.
I have:

Remove the /components folder
Remove the _ for the method handlers
Fix the usage of getMuiTheme
Upgrade to ES6 classes
Use the themeDecorator to provide the muiTheme context.
I'm convinced that it's should be the only way that we should document it.

Thanks a lot @oliviertassinari :+1: :+1:

I'm going ahead :100:.

Thanks :grin:


Bugfix/navbar menu links

Description

Fixes #553

Reason for this PR

see #553

Changes in this PR

change the HTML tag to a with an href. The onClick listener then prevents the default behavior of the href event that would cause the page to reload.

Type of change (remove all that don't apply)

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Manually tested all use cases this could alter

Test Configuration:

OS: Windows

Browser: Firefox

Frontend: (remove all that don't apply)

[x] Development build

Backend: (remove all that don't apply)

[x] No backend needed to test this

Checklist: (remove all that don't apply)

[x] I have performed a self-review of my own code
[x] My changes generate no new linting warnings or console warnings
[x] I have updated the CHANGELOG.md to include any changes made in this PR (add WIP to the top, if there is none already)

Note:
Cypress does not support multiple tabs, hence I could not implement proper E2E tests, but it worked with my local setup.

@theovier in commit a2bb27e I changed the tag to router-link. That saves 1 line in the setup. I think using the router-link not like a router-link (as I explicitly prevent its defauft action that causes a page reload) is kinda strange. Feel free to revert that commit before approving :)

@tobias-wiese
I'm fine with this as I think it will always feel weird to have a link and prevent the default action to additionally call custom methods which do the routing.


在trian的时候报错：    import realesrgan.archs ModuleNotFoundError: No module named 'realesrgan'

python realesrgan/train.py -opt options/train_realesrgan_x4plus.yml --auto_resume
Traceback (most recent call last):
File "realesrgan/train.py", line 8, in
import realesrgan.archs
ModuleNotFoundError: No module named 'realesrgan'
麻烦您回复下

我也有同樣的問題 我很確定我環境都安裝了

hello,
I have that same issue.
after running
python realesrgan/train.py -opt options/train_realesrgan_x4plus.yml --auto_resume
i get
Traceback (most recent call last): File "realesrgan/train.py", line 8, in import realesrgan.archs ModuleNotFoundError: No module named 'realesrgan'
I installed all requirements as mentioned above.
system is windows with anaconda and pytorch.
anything I can do about that?

I had to run
python setup.py develop
as a single command again, then it worked.


Is there some way to download the images for Celeba-HQ

Or if you can make them available to me I can host them somewhere for people to download. I'd really like the dataset but don't particularly want to go through the process to generate it myself.

The authors released the dataset, and here is the link
https://drive.google.com/open?id=0B4qLcYyJmiz0TXY1NG02bzZVRGs
please refer to this repo .
https://github.com/tkarras/progressive_growing_of_gans

@nashory seems it is only the pretrained model, not the source images

@petergerten aren't they stored in the .dat files in the datasets directory? https://drive.google.com/drive/folders/1iGK3jDe_1-V0Lyy5468WNpb7y74u-iEp

The .dat files from google drive only contain the difference between the celebA and the celebA-HQ images. The process to create the images is actually quite complicated.
I just made my scripts  available at
https://github.com/nperraud/download-celebA-HQ
They are far from perfect but I hope it helps.

Hi @nperraud! Thanks for sharing the scripts. I am trying to download the deltas from Google Drive but it gives me an error that the "download quota exceeded for this file" for all files of course. I guess there is a limit to the number of downloads per day for Google Drive. Do you have any idea if I can find the files stored somewhere else?

I generated the files and stored it as convenient zip files. You can download from GDrive.

@suvojit-0x55aa  there should be 200k images in Celeba HQ,but your sharing are 30k of them.

@shartoo As quoted in the Progressive Growing of GANs paper

In this section we describe the process we used to create the high-quality version of the CELEBA dataset, consisting of 30000 images in 1024 × 1024 resolution.

The Celeba HQ dataset contains only 30k images.

sorry for  my careless check,you're right.

Another question,how could i use the list_attr_celaba.txt namely attribution label for celaba-hq.Does the 30k images of celeba-hq are first 30k of 200k images in celaba dataset?

Got it ,there is a image list file https://raw.githubusercontent.com/nperraud/download-celebA-HQ/master/image_list.txt which releated celaba and celaba-hq files.

@shartoo As quoted in the Progressive Growing of GANs paper

In this section we describe the process we used to create the high-quality version of the CELEBA dataset, consisting of 30000 images in 1024 × 1024 resolution.

The Celeba HQ dataset contains only 30k images.

Is It possible to share labels with the dataset? Thanks!

@Cuilie you can use this file.

@suvojit-0x55aa On your github page you said "The size of the final dataset is 89G". But in Google Drive, you have far less than 89Gb. How come?

@aciobanusebi as far as I remember that's the memory needed for raw dataset, the deltas, and the generated dataset.


calculate token name at one place

147 changed files  with 12 additions and 8,516 deletions.
Thanks for the effort @Nekle but I cannot merge that in its current state. If you are changing a few lines of code, the diff should have a few lines of code as well.

Hi @sahat . I just want to push 49e4247.
After I created pull request, I did some other changes on my fork.  I don't know this will affect the pull request before.
sorry for this.

Thanks, I merged it here: https://github.com/sahat/satellizer/commit/f02615a7f813e6504217e3462ce50b34a32506dd


New items for the inventory

Proposal:
Proposal written by PremiumPlus
What is the title of your proposal?: New items for the inventory
Is it a suggestion for the website or for the bot?: for the bot
Explain your proposal more accurately (It's best to give as much information as possible, so that we can implement the proposal better): I would add new items like the airplane ✈ (buy: $25000/ sell: $23500) oder the :trophy: trophy (buy: $20000/ sell: $18500).
Why should we add this feature?: Because i think that its not just only me who thinks that this update could be cool because u have more items to show off with some of the new items.
ReportID: 117

➤ LenoxBot commented:
Approve:
but sell-price should be less. 19'000 and 15'000
Profxy#5396

➤ LenoxBot commented:
Approve:
agree with Profxy
🎃CrEePeR28 ~~ Furkan🎃#3532

➤ LenoxBot commented:
Approve:
It‘s a good proposal but we have to change a little bit the price of the plane 😛
Monkeyyy11#0001


Allow Configuring Custom Audio Encoder

First of all, I have only just begun trying out Capture. And so far, I think it is a great front-end for screen capture with FFmpeg. Thank you very much for this great software.
I would like to be able to customize the audio format & settings when I create a custom codec in Configure -> FFmpeg -> Configure Codecs -> Custom, just like can be done in the Args field for the video encoder. Or, just be able to disable the Audio Format field so I can include the audio encoder options in the Args field.

In addition, I think it would also be good to have "PCM" & "FLAC" as options in the standard audio formats. In my experience with FFmpeg on Windows, when capturing audio to PCM, using the "copy" codec option (-c:a copy) is sufficient, rather than re-encoding to PCM (-c:a pcm_s16le) as that is the native format.
In the version of FFmpeg on my system, the encoder for FLAC audio is simply "flac" (-c:a flac).


OffscreenCanvas support?

The OffscreenCanvas API seems to allow canvas rendering in WebWorkers:
https://devnook.github.io/OffscreenCanvasDemo/index.html
https://developer.mozilla.org/en-US/docs/Web/API/OffscreenCanvas
I was wondering whether this could be used by Lightning for certain work that we're noticing is causing stutters: such as rendering large nested trees.

Hi, it could be used by Lightning but would require some fundamental changes. We have this already on our backlog but can't provide any clear timelines.


added state selector and global state functionality

Functionality to specify a state selector function, and if set, the observable stream is a stream of the required state slice. This means multiple subscribers do not each have to apply a selector function to obtain the required state slice. If not specified, then the default of null means it behaves as usual and therefore is fully backwards compatible.
Additional functionality to get a notification when there is any state change in the store. Currently each observer in a service only adds to the stream if there is a change in its particular slice of state. Therefore its subscribers will not be notified if there is a change in another slice of state being made in a separate service. The added functionality creates a global state notification so that a subscriber will get notified if there is any change in state regardless of which service changed its slice of state. The use case being that you have 'n' services each managing one slice of state, and with the globalStateChanged stream allows a new observable store value to be created when any of the 'n' services has a change of state.

Really appreciate the contribution Mickey! Oddly enough, I've been wanting to add a way to globally subscribe to store changes. You took it one step further with the slice capability. Thanks again!

As an update - just pushed 1.0.16 to npm. Thanks again for the contribution Mickey!

Hey Mickey! We had someone in the issues ask for some help on the stateSliceSelector. I gave a quick example but if you get time and have anything you want to share there related to how you're using it that'd be great. If not (since I definitely realize time is scare) that's cool too. :-)


Fix: Parser returns a wrong structure when parent suite has duplicate child suites

Fixes #1850  .
Closes #1850  .
Did you remember to?

[x] Add test case(s)
[x] Update CHANGES.txt

Hi,
I reorganized the pull request for better code review, please check #1854 .
Sorry for any inconvenience.

Thanks for your kind reminding, I will do it better next time.  Please help to review #1854 .


AvoDeduplicator clearOldEvents crash

I have several crashes in my app due to this:

I have not been able to reproduce it, but I see that it is a constant crash on Crashlytics.

Thanks for reporting!

@lu15gv please check the new version 1.2.3, should be fixed there!

@tpom6oh I'll update it, thanks!


Rename package multiconsumer to fanoutprocessor

Because

This package is under /processor/.
The returned type is already processor: https://github.com/open-telemetry/opentelemetry-service/blob/091e19dd26971dd68a45c8801591cac771f38eb2/processor/multiconsumer/multiconsumer.go#L27

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b247a56). Click here to learn what that means.
The diff coverage is 75%.

@@            Coverage Diff            @@
##             master     #250   +/-   ##
=========================================
  Coverage          ?   73.43%
=========================================
  Files             ?      112
  Lines             ?     6517
  Branches          ?        0
=========================================
  Hits              ?     4786
  Misses            ?     1489
  Partials          ?      242

Impacted Files
Coverage Δ

processor/multiprocessor/multiprocessor.go
100% <ø> (ø)

service/builder/receivers_builder.go
83.63% <100%> (ø)

service/builder/pipelines_builder.go
92.72% <50%> (ø)

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 b247a56...bddd106. Read the comment docs.

Fanout sounds a better name. Updated in https://github.com/open-telemetry/opentelemetry-service/pull/250/commits/9a54f241cdeadef5eb19fc7c532ab753deafafae.


Chtian

Pull Request Test Coverage Report for Build 75

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 90.242%

Totals

Change from base Build 72:
0.0%

Covered Lines:
2034

Relevant Lines:
2235

💛  - Coveralls


Bug:  eqMac makes small variations in key and out of tune.

Describe the bug
While paying a song in Spotify, you can notice minor and not minor tuning modifications. Could it be due to insufficiently constant signal processing?
Steps to Reproduce

Launch eqMac

Play a song in Spotify

Setup information:

Audio device used for playback: A Scarlet 2i2 with external Monitors

macOS Version: 10.15.7

eqMac Version: 0.3.6

eqMac UI Version: 1.2.2

eqMac Driver Version: 1.2

Noticed the same issue here. I thought something was wrong with my ears for a moment

Also happening for me with the same setup
audio device used for playback: headphones
macOS Version: 10.15.7
eqMac Version: 0.3.6
eqMac UI Version: 1.2.2
eqMac Driver Version: 1.2

I have the same problem...
macOs 10.15.7

i read somewhere, that someone said  EQMac is doing way more than simply EQing the input audio, and i think he said EQMac is actually synthesizing  or recoding the whole audio all over again, so it may not be doing this accurately enough.
I just read this on a forum somewhere, so those Guys may have been wrong though. I guess the original coder may tell us if this is the case,

i read somewhere, that someone said  EQMac is doing way more than simply EQing the input audio, and i think he said EQMac is actually synthesizing  or recoding the whole audio all over again, so it may not be doing this accurately enough.
I just read this on a forum somewhere, so those Guys may have been wrong though. I guess the original coder may tell us if this is the case,

I'm also experiencing this issue fairly regularly. I've noticed it when playing audio through various sources/apps including Chrome (YouTube, SoundCloud, etc.) and the VOX music player app.
Audio device used for playback: iMac headphone output
macOS Version: 10.15.7
eqMac Version: 0.3.6
eqMac UI Version: 1.2.2
eqMac Driver Version: 1.2

I'm also experiencing this issue fairly regularly. I've noticed it when playing audio through various sources/apps including Chrome (YouTube, SoundCloud, etc.) and the VOX music player app.
Audio device used for playback: iMac headphone output
macOS Version: 10.15.7
eqMac Version: 0.3.6
eqMac UI Version: 1.2.2
eqMac Driver Version: 1.2

FWIW >. tried renicing the coreaudio plugin to -20 .. doesn't appear to be load related.

I have the same problem and I confirm that the problem is not there when eqMac is not active. It happens anytime while listening to Spotify (v<IP_ADDRESS>8.g7ed9c03a). I'm on macOs 10.15.7. Thanks.

Thanks for being patient with me.
I have just released v1.0.0. You can download it here: https://github.com/bitgapp/eqMac/releases/tag/v1.0.0
Or from the website: https://eqmac.app
Could you please let me know if this issue is still present.
Thanks again.
Roman

I'm doing some cleanup of the issues.
This has probably been fixed long time ago, I encourage people to try v1.7 by download it from the website: https://eqmac.app
Closing for now, any new issues with eqMac should be submitted through the website.
Thank you!


Mondrian view should react to clicks on custom nodes with nested elements

m := GtMondrian new.
m nodes
	stencil: [:each | BrLabel new look: BrGlamorousLabelLook; text: each asString ];
	with: {42}.
m

Clicking on the 42 label should spawn the object.


Triggering user/pass prompt despite GIT_TERMINAL_PROMPT=0

root@8037ab4473da:/app# ./Run re-analyze-versions -p e6f418d6-6004-41d4-80df-1ee7162b74db
[ INFO ] Re-analyzing versions (id: E6F418D6-6004-41D4-80DF-1EE7162B74DB) ... [component: re-analyze-versions]
[ INFO ] Re-analyzing package https://github.com/amosavian/FileProvider.git ... [component: re-analyze-versions]
[ INFO ] cloning https://github.com/amosavian/FileProvider.git to /checkouts/github.com-amosavian-fileprovider [component: re-analyze-versions]
Username for 'https://github.com':
Password for 'https://github.com':

<## I hit return here to continue  ##>

[ ERROR ] Analysis failed: refreshCheckout failed: Shell command failed:
command: "git clone https://github.com/amosavian/FileProvider.git /checkouts/github.com-amosavian-fileprovider --quiet"
path:    "/checkouts"
message: "ShellOut encountered an error
Status code: 128
Message: "remote: Access to this repository has been disabled by GitHub staff due to
remote: excessive resource use. Please contact support via
remote: https://support.github.com/contact to restore access to this repository.
remote: Read about how to decrease the size of your repository:
remote:   https://docs.github.com/articles/what-is-my-disk-quota
fatal: Authentication failed for 'https://github.com/amosavian/FileProvider.git/'"
Output: """ (id: E6F418D6-6004-41D4-80DF-1EE7162B74DB) [component: re-analyze-versions]
[ INFO ] Done. [component: re-analyze-versions]

Check if GIT_SSH_COMMAND https://github.com/apple/swift-tools-support-core/blob/5446eabed71cd8baac396ea70e90ccf68449d312/Sources/TSCUtility/Git.swift#L93 helps

Haven't seen this happening since and there's nothing in the logs like a username prompt in the logs for the last 7 days. Closing.


Add issue and PR templates

Description

Add issue and PR templates to help guide people in creating useful issues.

Fixes #252
Contribution checklist
Aim to have all relevent checks ticked off before merging. See the contribution guide for more detail.

[x] Documentation has been updated to reflect change.
[x] New code has tests, and affected old tests have been updated.
[x] All tests and CI lints pass.
[ ] Added an entry to the top of docs/source/changelog.rst N/A before #467
[x] Conda lock files have been updated if dependencies changed.
[x] Marked the PR as ready to review.

Let me know what you think of the templates. Should they have more structure/guidance? Less?


Trying to access array offset on value of type null

What happened?
Trying to access array offset on value of type null
How to reproduce the bug
AdminPanelProvider.php
<?php

namespace App\Providers\Filament;

use Devlogx\FilamentPirsch\FilamentPirschPlugin;
use Filament\Panel;
use Filament\PanelProvider;

class AdminPanelProvider extends PanelProvider
{
    public function panel(Panel $panel): Panel
    {
        return $panel
            ->default()
            ->id('admin')
            ->path('admin')
            ->plugins([
                FilamentPirschPlugin::make(),
            ]);
    }
}

Dashboard.php
<?php

namespace App\Filament\Admin\Pages;

use Devlogx\FilamentPirsch\Widgets\PirschStatsWidget;
use Filament\Pages\Page;

class Dashboard extends Page
{
    protected function getHeaderWidgets(): array
    {
        return [
            PirschStatsWidget::class,
        ];
    }
}

Package Version
2
PHP Version
^8.2
Laravel Version
^11.0
Which operating systems does with happen with?
No response
Notes
No response

I had the same issue - make sue that you use the HasFilter Concern on your Page (see doc)

Exactly. the HasFilter Concern has to be present on the Dashboard or View Page. :)


update k to v6.0.55

Yes, the plan is to regenerate the decision trees. was going to do this in a separate PR but can do this tomorrow and add to the current one. Not sure who else to ask for a review this week. Maybe @ehildenb ? Everyone else I can think of is on PTO...

yes, let's add it to the current PR because every PR that changes the tar.gz file is going to add another couple megabytes to the size of the repo.
Not sure what to do about the review situation. I just know the changes to the nix files are all essentially gibberish to me and you could have done something really weird and I would have no way of knowing, so I don't really want to approve it. Is there not anyone with nix experience who could review that code that is not on vacation? I could probably review the rest of the changes if you add a bit of documentation to each file explaining what you're doing and why.

The changes seem reasonable to me overall. I'm not an expert on the Nix, but a read of it doesn't show anything super surprising or anything. @dwightguth do you want to dismiss your review so we can merge?


Make the TPS chart behave like the network I/O charts

Make the TPS chart's incoming/outgoing series asymmetric to each other like in the network I/O charts for better readability.

Done via https://github.com/gohornet/hornet/pull/51


implement more casts

not sure if this is all of #26

@oli-obk AFAICT this doesn't have unsizing.

unsizing has its own variant Unsize, it isn't part of Misc

@oli-obk Right, I meant #26 includes unsizing so this wouldn't tick all the boxes.

Unsize is handled, and I can't produce any examples that do an unsize cast and aren't correctly handled by miri.

@oli-obk But I thought vtables weren't implemented (and the box isn't ticked off in #26).

good point... so yea, the last box should be checked, but we need to wait for vtables now.


refactor: onboarding after install

Try out this version of the Stacks Wallet - download extension builds.

This PR changes the onboarding flow so that now after install the user is directed back to their app tab and it reloads the page so the extension can be detected as installed.
Issues:
https://github.com/blockstack/connect/issues/144
https://github.com/blockstack/connect/issues/98
Testing:

Point at the alpha release changes to connect linked in the PR above.
Run the test-app
Load the unpacked version into Chrome
Going thru onboarding in the full-page wallet
After you set your password, the ext should automatically redirect you back to the app tab
Click on Sign up again
Choose an account
Done

I think ultimately the goal is to extract out the concept of the Sign up button in the test-app and make it available via the connect library (@jasperjansz new designs for onboarding), but this is a good first step.
cc/ @aulneau @kyranjamie @fbwoolf

@kyranjamie one thing I noticed a little weird in testing this with the test-app is that when back at that app page the wallet is loaded but I still have to select an account before it moves past the sign up page. It seems like it should always just default to select the first indexed account to move an app past auth?

It seems like it should always just default to select the first indexed account to move an app past auth?

I think for first time registrations this makes a lot of sense

It seems like it should always just default to select the first indexed account to move an app past auth?

I think for first time registrations this makes a lot of sense

Been working on a solution for this, do either of you know why logging window to the console shows StacksProvider but if I console.log window.StacksProvider it is undefined (in the test-app)? Must be something with it being injected via the inpage script?

Bizarre. Does window === window // true?  They should resolve to the same place.

Going to close this PR, it is really outdated. Will be discussed further here: https://github.com/hirosystems/stacks-wallet-web/issues/1808


Any ideas about CRUD actions journal?

I need to journal of crud actions in admin panel. I could use after_filter method, but it isn't provided after unsuccessful create and update. Is there any way to hook after action events without modifying ex_admin source?

I suppose you could create your own resource controller with the same actions at ExAdmin.AdminResourceController and setup your router to call it. From your controller, you could all the actions on ExAdmin.AdminResourceContrller and take action before returning the conn.
Actually, you might only need to route the create and update actions to your own controller...
I would also be open to a RP that provides a hood in handle_changeset_error. Between the after filter and this new hook, you should have what you need.
There might be a third option, but I've never tried it. You might be able to create a plug that you place in your endpoint, after the call to the router plug. However, I believe that plugs stop being called once the conn is hauled, so this might now work. It would be quick to test.

@smpallen99 I suppose provides a hood means provides a hook :)

The second case
Maybe we can update ExAdmin.AdminResourceContrller with new hook this way
    case ExAdmin.Repo.insert(changeset) do
      {:error, changeset} ->
        conn
         # new hook handle_after_error_filter, maybe pattern matched handle_after_filter be better?
         |> handle_after_error_filter(conn, :create, defn, params, changeset, resource)
         |> handle_changeset_error(defn, changeset, params)
      resource ->
        {conn, _, resource} = handle_after_filter(conn, :create, defn, params, resource)
        put_flash(conn, :notice, (gettext "%{model_name} was successfully created.", model_name: (model |> base_name |> titleize) ))
        |> redirect(to: admin_resource_path(resource, :show))
    end

The third case works for plug in Endpoint
  plug App.Router
  plug App.Journal

and
defmodule App.Journal do
  @moduledoc false
  import Plug.Conn
  def init(_) do
  end
  def call(conn, _opts \\ %{}) do
    IO.inspect conn.assigns[:changeset]
    IO.inspect conn.assigns[:resource]
    IO.inspect conn.params
    conn
  end
end


Signed and unsigned word not permitted as type

I was trying to extend the _010_traffic_lights_controller_baseLang example with additional variables and noticed that FASTEN doesn't seem to accept variables of type unsigned word [N] and signed word [N]. Is this a user error or does FASTEN limit the allowed types for state machines?
  VAR {
    state variable traffic : {Green, Yellow, Red};
    pedestrian : {Walk, DontWalk};
    timer : 0..10;
    abc : signed word [5];
  }

Error message on hover:
Abstract concept instance detected. Use one of sub-concepts instead. Concept: Type
Apologies if this is a trivial mistake caused by my inexperience with FASTEN and NuSMV.

If you type "word" and start auto-completion (ctrl+space)  then FASTEN creates an "unsigned word"

Please confirm that it works like this.
If you want to change the signedness (i.e. to "signed") there is an intention (alt + enter) - unfortunately it seems that there is an MPS bug which prevents correct functioning of this intention. I have commited a fix on FASTEN side to avoid the MPS issue
https://youtrack.jetbrains.com/issue/MPS-33284
Workaround for changing the signedness - right click on the type, select "show reflective editor" and change the "signed" flag.

Thank you for the quick reply. That seems to do the trick.


docs: check documentation coverage

Summary

Depend on API Extractor to check which plugins are documented but not implemented or not properly exported.
The following plugins are documented but not implemented or not properly exported: SplitChunksPlugin

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

LGTM 👍


⚠ Fortnite has degraded performance

In a6d39fe, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1556 ms

Resolved: Fortnite performance has improved in e3cd1f5.


apt update not working

I'm trying to do a new Docker build for an existing project. The project uses phusion/baseimage:0.11. I decided to do a --no-cache build to update to the latest packages, but it keeps failing on apt-get update.
I've waited a few days to see if it was perhaps just some repository maintenance but it isn't. Below is the output of my docker build log:
Step 1/37 : FROM phusion/baseimage:0.11
 ---> 14e6c478b00a
Step 2/37 : CMD ["/sbin/my_init"]
 ---> Running in b31f613467fd
Removing intermediate container b31f613467fd
 ---> 1a2c73ae4c98
Step 3/37 : EXPOSE 80
 ---> Running in 5eae3f6b3b5d
Removing intermediate container 5eae3f6b3b5d
 ---> 4acd5a2c7a10
Step 4/37 : ENV DEBIAN_FRONTEND noninteractive
 ---> Running in 77c4e787cf19
Removing intermediate container 77c4e787cf19
 ---> a55b5fd09f8e
Step 5/37 : RUN curl http://nginx.org/keys/nginx_signing.key >> /tmp/nginx.key
 ---> Running in a1b767126928
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1561  100  1561    0     0  25590      0 --:--:-- --:--:-- --:--:-- 26016
Removing intermediate container a1b767126928
 ---> 256dd9042612
Step 6/37 : RUN apt-key add /tmp/nginx.key
 ---> Running in 9f216ed2e7fd
Warning: apt-key output should not be parsed (stdout is not a terminal)
OK
Removing intermediate container 9f216ed2e7fd
 ---> e704fa231da7
Step 7/37 : RUN apt-get update
 ---> Running in 629184e545ea
Get:1 http://archive.ubuntu.com/ubuntu bionic InRelease [242 kB]
Get:2 http://archive.ubuntu.com/ubuntu bionic-updates InRelease [88.7 kB]
Get:3 http://archive.ubuntu.com/ubuntu bionic-backports InRelease [74.6 kB]
Get:4 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
Err:1 http://archive.ubuntu.com/ubuntu bionic InRelease
  At least one invalid signature was encountered.
Err:2 http://archive.ubuntu.com/ubuntu bionic-updates InRelease
  At least one invalid signature was encountered.
Err:3 http://archive.ubuntu.com/ubuntu bionic-backports InRelease
  At least one invalid signature was encountered.
Err:4 http://security.ubuntu.com/ubuntu bionic-security InRelease
  At least one invalid signature was encountered.
Reading package lists...
W: GPG error: http://archive.ubuntu.com/ubuntu bionic InRelease: At least one invalid signature was encountered.
E: The repository 'http://archive.ubuntu.com/ubuntu bionic InRelease' is not signed.
W: GPG error: http://archive.ubuntu.com/ubuntu bionic-updates InRelease: At least one invalid signature was encountered.E: The repository 'http://archive.ubuntu.com/ubuntu bionic-updates InRelease' is not signed.
W: GPG error: http://archive.ubuntu.com/ubuntu bionic-backports InRelease: At least one invalid signature was encountered.
E: The repository 'http://archive.ubuntu.com/ubuntu bionic-backports InRelease' is not signed.
W: GPG error: http://security.ubuntu.com/ubuntu bionic-security InRelease: At least one invalid signature was encountered.
E: The repository 'http://security.ubuntu.com/ubuntu bionic-security InRelease' is not signed.
The command '/bin/sh -c apt-get update' returned a non-zero code: 100

I cannot repro this issue,
FROM phusion/baseimage:0.11
ENV DEBIAN_FRONTEND noninteractive
RUN curl http://nginx.org/keys/nginx_signing.key >> /tmp/nginx.key
RUN apt-key add /tmp/nginx.key
RUN apt-get update

I tried building an image with just the first and last lines of the dockerfile, because that key you added shouldn't have anything to do with the issue. It did apt-get update fine:
...
Get:31 http://archive.ubuntu.com/ubuntu bionic-backports/main amd64 Packages [2,496 B]
Get:32 http://archive.ubuntu.com/ubuntu bionic-backports/universe amd64 Packages [4,252 B]
Fetched 31.6 MB in 5s (6,097 kB/s)
Reading package lists...
Removing intermediate container 76494588b770
 ---> 8ebc19342543
Successfully built 8ebc19342543
Successfully tagged temp:latest

So I tried again following more closely what you did, with the inclusion of the extra step, adding a key from nginx.org, (as the Dockerfile above shows) and I still don't get any GPG errors.
Is it possible this is something unusual going on inside of your network?  (Is anyone else having this issue?)

There on no restrictions on such things on the network. To make sure I've actually tested this with and without VPN, but this didn't change the outcome. I've also untagged the image and pulled a new one, but no dice.
I've decided to try this on a second machine and I actually don't see the issue either. apt-get update runs as normal. I guess it's time to reset my Docker install. I think something may have gotten corrupted when the Linux VM ran out of diskspace (Docker on Windows).

@rene-demonsters Please go ahead and close this if it turns out to be resolved by blowing away docker.
In fact, since it sounds likely that it is leaning that way, I'm going to go ahead and close right now, but please do let us know if that wasn't the solution.

Yep, blowing away Docker did the trick. Cheers! :)

FYI, I had the same issue with phusion/baseimage:focal-1.0.0alpha1-arm.
In my case I was trying to build a new Docker image on Raspberry 4, but kept getting the same errors as above. In additon, when running date command, I saw that the time within the container was wrong (1970) which perhaps is related to the signatures being invalid.
Looking further at this post (https://bugs.launchpad.net/cloud-images/+bug/1896443/comments/12) and other similar topics, it turns out, that libseccomp2 is currently bugged on RPI 4 and needs to be updated. After installing an updated library (e.g. libseccomp2_2.5.1-1_armhf via dpkg -i on my docker-host (not in container!) and restarting Docker service, it started to work.

@Excodibur thanks for the update. Is there a way we can add this workaround to our build?

@Theaxiom Unfortunately I don't think there is much you can do in your build to avoid this (at least in my case), since the problem needed to be solved on my local server (RPI4) where I tried to build a new Docker image using yours as base-image. So to me it looks like the problem is not with the image you build at all. At some point I assume RPI4 will get a lib update and then the issue resolves by itself.

Thank you @Excodibur


[iOS] Slow observer onForegroundEvent

Hi there,
It occurs to me that when using the following snippet, the code is not immediately being run after clicking a notification (in foreground state);
notifee.onForegroundEvent(({ type, detail }) => {
      console.log('notifee.onForegroundEvent')
})

Using version 7.8.2 and RN 0.72.6.
Is there any way I can help out? It works, but ideally I would like to remove the delay.

Same issue using notifee v7.8.2 and RN v0.73.6 - the callback is executed only after a noticeable delay.

Not stale