Add "tags" as static commit aliases

What is the goal / desired outcome?
provide a human-readable way to durably reference a specific commit. The use-case is to be able to share a known-good "release" point in an ever-evolving dataset's history, and be sure that point won't change. Good description of branches vs tags here: https://stackoverflow.com/a/1457536
If there is a way to accomplish this today via workaround, what does that require?
reference a commit ID -- it's ugly, but is durable
create a "release branch" -- it floats by default, and is generally oriented towards representing the tip of a line of revisions (these are technically so similar that this isn't a terrible alternative, and likely what would be recommended today, but for some use-cases the different intention can be important: https://stackoverflow.com/a/1457536)
(Optional) What is your proposal for a feature to solve this?
Introduce a concept of "tags" that are analogous to git's tags -- pointers to commits (ie the same basic thing as a branch), but that are oriented towards durability (so no auto-update behavior ie they act identically to just typing the relevant commit ID, so once the commit is finished, you can't put-file <tag-name>
Edit to reiterate/clarify: in git, both "branches" and "tags" are just pointers to a specific commit. on a technical level, the difference between the two is only very slight: basically committing "to a branch" just creates a new commit and automatically updates the branch to point at that new commit, whereas committing "to a tag" doesn't have any meaning. because they're more difficult to edit, though, tags feel more durable.

Needless to say, this is not a difficult implementation, since this is merely a branch with some features restricted. One interesting question is if there's any need to have tag to tag provenance or something like that, or if the underlying commit provenance is good enough. I can't see a meaning for tag-to-tag provenance right now.

i think tags just being a very literal user-defined alias for a commit ID is sufficient (ie the provenance of a tag would just be the provenance of it's commit). So yea, initially I was thinking that using a simplified version of the branch implementation (or even just a "static" flag or something on branches) would make sense, but given that definition of a tag is so much simpler than a branch (particularly in pach, because of the provenance, even more so than git), i wonder if it does make sense for them to be implemented in the same way? Re #3582 I wonder if the shared part should just be a general notion of a "commit reference" that could be a commit ID, a tag name, a branch name, or any of those with various modifiers (^, ~, etc)?
The one way in which some notion of "tag provenance" could be useful would be to see the provenance of a tag, in terms of other tags (rather than opaque commit ids). But I think just being able to tell which tags point at a given commit (in addition to obviously wanting to know which commit is referenced by a given tag) would be a better way of doing that. I believe git generally shows all tags and branches that point at a commit (in parentheses) whenever a commit id is printed.


SDO Client example

I am trying to implement an SDO Client in my device and looking for an example or any relevant code. Is there any hint how to organize that? My device works well as SDO Master.

Take a look in the examples folder or take a look at canopensocket.

Thanks, canopensocket helped a lot!


Retrieve jenkins core and jenkins plugins from a custom url

When using this plugin, I'm trying to download the jenkins core and plugin dependencies through a custom repo rather than the default https://repo.jenkins-ci.org/public/. Is there a way to change the base url from which these artifacts get retrieved? I am new to gradle and haven't been able to find a way to accomplish this.

I also have not been able to achieve this. Any help would be greatly appreciated. But it seems that a code change may be required to achieve this result.

One slight hack that I did find was this task being added to the build script
// The plugin we use hard codes the public jenkins repository url.
// This overrides that url.
task replaceJenkinsPublicUrl {
  project.repositories.each { repo ->
    if ('JenkinsPublic'.equals(repo.name)) {
      repo.setUrl('https://myrepo/')
    }
  }
}

That works for me. Thanks for sharing your solution

This was an early decision to make it easier for new consumers to onboard.
Since repositories is mutable, you should be able to remove the existing JenkinsPublic repository and add your own to the repositories {} block.
repositories.removeIf { it.name == "JenkinsPublic" }

Gotcha, thanks!


Release candidate 5.1.1

Coverage remained the same at 100.0% when pulling 3fc83402dea8dc6d86dc96c5719e9a3097216dc2 on ReleaseCandidate-5.1.1 into 440c4dee53f2c1ece6a1280140fff9923f53171c on master.


nil pointer evaluating interface {}.cert

Is there an existing issue for this?

[x] I have searched the existing issues

Version
equal or higher than v1.16.4 and lower than v1.17.0
What happened?
helm upgrade cilium cilium/cilium --version 1.11.6
--namespace kube-system
--reuse-values
--set hubble.relay.enabled=true
--set hubble.ui.enabled=true
--set hubble.metrics.enabled="{dns,drop,tcp,flow,icmp,http}"
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /Users/sherwood/.kube/config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /Users/sherwood/.kube/config
Error: UPGRADE FAILED: template: cilium/templates/hubble/tls-helm/server-secret.yaml:2:10: executing "cilium/templates/hubble/tls-helm/server-secret.yaml" at <include "hubble-generate-certs.helm.setup-ca" .>: error calling include: template: cilium/templates/hubble/tls-helm/_helpers.tpl:15:18: executing "hubble-generate-certs.helm.setup-ca" at <.Values.hubble.tls.ca.cert>: nil pointer evaluating interface {}.cert
How can we reproduce the issue?
helm upgrade cilium cilium/cilium --version 1.11.6
--namespace kube-system
--reuse-values
--set hubble.relay.enabled=true
--set hubble.ui.enabled=true
--set hubble.metrics.enabled="{dns,drop,tcp,flow,icmp,http}"
Cilium Version
1.16.4
Kernel Version
Darwin Kernel Version 22.6.0:
Kubernetes Version
Client Version: v1.31.1
Kustomize Version: v5.4.2
Server Version: v1.31.2
Regression
No response
Sysdump
No response
Relevant log output

Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

[x] I agree to follow this project's Code of Conduct

@sherwoodzern did you intend to specify the version as 1.11.6 in your command lines? That version reached end of life around 18 months ago.

@joestringer That is probably the issue. I was following the instructions from the doc. What version should I be specifying?

Here's the list of actively maintained releases: https://github.com/cilium/cilium?tab=readme-ov-file#stable-releases


.next Decision SpringBoot v3 support assessment

Provide an assess for effort and complexity to introduce support for SpringBoot v3 to Drools 8 (Kogito stream)

I was able to build Spring Boot modules using Spring Boot 3.0.2 in kogito-runtimes. However the tests (and the features in general) don't work, because they need the migration to jakarta namespace as Spring Boot 3 requires Java 17. The problematic part is that the code generation is common to both Quarkus and Spring Boot. There is no separate code generation for each of the frameworks. So when code generation happens, it generates javax code, which cannot be compiled.
I saved my changes to this branch: https://github.com/baldimir/kogito-runtimes/tree/SpringBoot3. When the jakarta move is done, I will try to rebase my changes on top of the jakarta changes.
As this is blocked till jakarta migration is done, I am returning this to Backlog.


Form Validation Service: add field name to errors

ErrorList#containsField check does not works when check input data with form validation service.

I've added a new test to this pull request, and of course it passed with no errors on my local environment.


Change NewLine Character to be CRLF instead of LF for 'winutil.ps1' File

Main Issue
The CRLF New Line Character is the default one used by git on windows (even when using Git Bash), where Windows is obviously the Platform of choice when developing & debugging WinUtil, changing it to CRLF from what's currently set as, which's UNIX New Line LF (Line Feed) is for the better.
This is even shown when compiling WinUtil on GitHub Actions, indicating that the reason behind this odd switch isn't related to workflow files (this is my own guess, further investigation might be needed).

[!NOTE]
This issue wasn't present before removing winutil.ps1 from Git Tracking List, I'm guessing something happened when creating winutil.ps1 file in commit 009ab5066cc80f59d12b5385cb27ec5841796be3 ?

Why would someone care about it?
Even though PowerShell handles New Line Characters without any problems (Powershell 5 & 7).. but when viewing The Git Diff Output, because I edit all my changes through CLI, using Git & GitHub CLI Tools, as well as CLI Editors like Vim & NeoVim.. it displays warning: in the working copy of 'winutil.ps1', LF will be replaced by CRLF the next time Git touches it.. which doesn't have to be the case!
When there's common rules to follow across project's files.. then all of the files should follow said rules.
Final Thought
Although this's a minor issue to nitpick.. and it may/may not affect about 1-2% of devs who contribute to the project, and even less when considering that viewing and committing compiled files isn't needed at all.. most of the time, I still wanted to point this out to document it.. and in the hopes that someone might find a solution for it.
Thanks for taking your time reading through this Issue Ticket, and have a nice day 😄

https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings?platform=windows
Might be as simple as editing the .gitattributes file. Will check later when I'm at my desk. This will however result in quite a big initial PR because all line endings will be replaced

https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings?platform=windows
Might be as simple as editing the .gitattributes file. Will check later when I'm at my desk. This will however result in quite a big initial PR because all line endings will be replaced

Thanks for the info/link @Marterich 😄
Didn't know you could configure git settings across contributors like this.
And after reading it.. the commit will be kind of massive, every line in every text file will be changed (may be even the ones that are Already using CRLF).. I' try testing as well on my side to see how we could manage it. Thanks again for the help 👍

You're welcome @og-mrk
If this works as expected, be sure to thoroughly document your reasoning in a PR so it has any chance at all or being merged xD
IMHO, when developing a script for Windows on Windows you should adhere to the conventions and quirks that exist on the platform, so I'm all for it

@Marterich
Made a PR which tries to resolve this issue, here's a link to it.
It turns out that the generated XAML content from json files is actually causing this issue.. a bit weird, because I haven't seen this Git Warning before (may I have and didn't pay much attention).. either way, this issue should be done once that PR is merged 👍


Fix git url in package.json

Makes it easier to access the repos from the npm page

Thank you, I missed that.


Allow images to be added in cells

One of doing this could be to register a callback with the content stream so that the users of the library could place the image themselves.

Done with v0.1.0


merge from template-arlac77-github

.github/workflows/ci.yml

chore(deps): add 20.13.0 (jobs.test-node.strategy.matrix.node-version)
chore: add 20.13.0 (jobs.test-browser.steps.with.node-version)
chore: add 20.13.0 (jobs.release.steps.with.node-version)

.github/workflows/codeql_analysis.yml

chore: add 20.13.0 (jobs.analyze.steps.with.node-version)

.github/workflows/update_package_lock.yml

chore(deps): add 20.13.0 (jobs.update_package_lock.strategy.matrix.node-version)

.github/workflows/update_readme_api.yml

chore: add 20.13.0 (jobs.update_readme_api.steps.with.node-version)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
404

Relevant Lines:
412

💛  - Coveralls


Unable to resolve module crypto - @azure/notification-hubs

Is your feature request related to a problem? Please describe.
@azure/notification-hubs doesn't seem to work with React Native CLI projects
Describe the solution you'd like
Anything to be able to register a device into azure notification hub
Describe alternatives you've considered
Tried to follow a microsoft guide which tells to install some polyfill
this is what my code look like

dependencies

Additional context
After i launch npm run android this error appears

Closed as per next release


[tracking] check skipped tests for Agama 914

test_dot[1-64-128-128-4-True-True-none-tf32-int8-int8-1_0]
test_dot[1-64-128-128-4-True-True-none-tf32-int8-int8-1_1]
test_dot[1-64-128-128-4-False-True-none-tf32-int8-int8-1_0]
test_dot[1-64-128-128-4-False-True-none-tf32-int8-int8-1_1]

Errors:
FAILED language/test_core.py::test_dot[1-64-128-128-4-True-True-none-tf32-int8-int8-1_0] - AssertionError:
Not equal to tolerance rtol=0.01, atol=0.001

Mismatched elements: 1912 / 8192 (23.3%)
Max absolute difference: 24077
Max relative difference: 1280.42857143
 x: array([[  -8771,   97832,   94483, ...,   35111,   22460,   70087],
       [-124117, -115193,    1862, ...,   -2921,  -65821,  -26059],
       [ -55616,   23155,   64353, ...,  100224,   18555,   42140],...
 y: array([[ -19063,   92262,   85278, ...,   35111,   22460,   70087],
       [-121003, -114413,     854, ...,   -2921,  -65821,  -26059],
       [ -56355,   25225,   74041, ...,  100224,   18555,   42140],...
FAILED language/test_core.py::test_dot[1-64-128-128-4-True-True-none-tf32-int8-int8-1_1] - AssertionError:
Not equal to tolerance rtol=0.01, atol=0.001

Mismatched elements: 1912 / 8192 (23.3%)
Max absolute difference: 24077
Max relative difference: 1280.42857143
 x: array([[  -8771,   97832,   94483, ...,   35111,   22460,   70087],
       [-124117, -115193,    1862, ...,   -2921,  -65821,  -26059],
       [ -55616,   23155,   64353, ...,  100224,   18555,   42140],...
 y: array([[ -19063,   92262,   85278, ...,   35111,   22460,   70087],
       [-121003, -114413,     854, ...,   -2921,  -65821,  -26059],
       [ -56355,   25225,   74041, ...,  100224,   18555,   42140],...
FAILED language/test_core.py::test_dot[1-64-128-128-4-False-True-none-tf32-int8-int8-1_0] - AssertionError:
Not equal to tolerance rtol=0.01, atol=0.001

Mismatched elements: 1961 / 8192 (23.9%)
Max absolute difference: 42061
Max relative difference:<PHONE_NUMBER>6
 x: array([[ -70131,  -76608,   36342, ...,    -437,    2562,   37103],
       [ -10398,   -2496, -111479, ...,  114795,   89840,   25838],
       [  84214,  -54418, -112739, ...,  -14705,   70750,   20548],...
 y: array([[ -69119,  -78742,   62848, ...,    -437,    2562,   37103],
       [   -994,    -338,  -99921, ...,  114795,   89840,   25838],
       [  92654,  -52598, -101119, ...,  -14705,   70750,   20548],...
FAILED language/test_core.py::test_dot[1-64-128-128-4-False-True-none-tf32-int8-int8-1_1] - AssertionError:
Not equal to tolerance rtol=0.01, atol=0.001

Mismatched elements: 1961 / 8192 (23.9%)
Max absolute difference: 42061
Max relative difference:<PHONE_NUMBER>6
 x: array([[ -70131,  -76608,   36342, ...,    -437,    2562,   37103],
       [ -10398,   -2496, -111479, ...,  114795,   89840,   25838],
       [  84214,  -54418, -112739, ...,  -14705,   70750,   20548],...
 y: array([[ -69119,  -78742,   62848, ...,    -437,    2562,   37103],
       [   -994,    -338,  -99921, ...,  114795,   89840,   25838],
       [  92654,  -52598, -101119, ...,  -14705,   70750,   20548],...

See also https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/8757168039/job/24035243552#step:12:23117

It seems these 4 cases has already in the default skiplist: https://github.com/intel/intel-xpu-backend-for-triton/blob/llvm-target/scripts/skiplist/default/language.txt
So this issue is already solved, can close it.

Cases in the test-triton.sh have 4 result status:
failed, passed, skipped, xfailed, warnings
I want to know:

What is the difference between 'failed' and 'xfailed'?
Are we supposed to put 'failed' and 'skipped ' into the skiplist OR just 'skipped'?

May be this has been solved somehow

This issue is to track 4 failing test cases with Agama 821.35. These test cases passed before, so it is a regression and it needs to be investigated.

Are we supposed to put failed and skipped into the skiplist OR just skipped?

There are two options to skip a test case:

Update Python code with pytest.skip for the specific conditions.
Add this test case to the skip list.

We probably want to use the latter method, because it allows to skip tests depending on the environment. For example, we can have a skip list for PVC with the rolling driver, PVC with the LTS driver, A770 with the rolling/LTS driver, and so on. We currently use both methods as a transitional step, and the plan is to use the skip list for new failures (regressions) and gradually replace pytest.skip with adding test cases to the corresponding skip list.

@AshburnLee please verify if these tests fail with the latest rolling.

@AshburnLee please verify if these tests fail with the latest rolling.

They still fail on latest llvm-target branch with Rolling(821.35).
On latest rolling? do I need to update the driver version?

They still fail on latest llvm-target branch with the current Rolling(821.35).

Thanks. The latest rolling is 821.36, I think. It would be nice to check it as well. We want to keep this issue open until driver or tests fixed.

We want to keep this issue open until driver or tests fixed.

Oh, so we just track it, and no need to to find the commit that causes those 4 fails?
We can do that, but building Triton from very early commits needs efforts and time.

Oh, so we just track it, and no need to to find the commit that causes those 4 fails?
We can do that, but building Triton from very early commits needs efforts and time.

Right, just to track at the moment. I don't think it is a Triton commit that caused the failures, they started to fail when we updated GPU driver.

4 cases still fail on latest llvm-target branch with the current Rolling(821.35). 6/12/2024

4 cases still fail on latest llvm-target branch with Rolling 881.19. 6/17/2024

Got error while running test:
/lib/python3.10/site-packages/intel_extension_for_pytorch/lib/libintel-ext-pt-gpu.so: undefined symbol: _ZNK4sycl3_V16device32ext_oneapi_supports_cl_extensionERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEEPNS0_3ext6oneapi12experimental10cl_versionE

4 cases got PASSED on 914(914.27)

Waiting for the new agama release


Question: does youtube-dlg "update" need root permission; does it install the latest version?

Please follow the guide below
WARNING
All invalid issues will be rejected!!

[x ] I think my problem is NOT with youtube-dl
[x ] I've verified and i assure that I'm running youtube-dl-gui 0.4
[x ] I assure that i am using the latest version of youtube-dl
[x ] Searched bugtracker
[x ] I've read the FAQs file  (yes, several times)

What is the purpose of your issue?

[x] Bug report
[x ] Feature request (request for a new functionality)
[x ] Question
[x] Other

I found no real instructions or FAQ - here or in  w/ real details on using yt-dlg to update yt-dl.  I don't know if this is a bug or operator (me) error or neither.
The GUI leads you to think "update" under options will run under a user account & does show something like "downloading the latest..."  on yt-dlg's screen.
A couple days ago, I couldn't get yt-dlg to update yt-dl at all (a newer version was available).  No messages, no log output about failed to update.
I don't know if this is a yt-dlg or yt-dl issue (guessing -dlg).
I only used yt-dlg a few mo.  Before, I mostly updated yt-dl  via a PPA in the Mint repo.
There's no log entries about failed update (logging enabled), but that might only come from yt-dl.  You'd think yt-dl would log something if it just updated.
After it finished & restarting yt-dlg, checking the youtube-dl --version still showed the same version as before, though the modified date of yt-dl file had changed.  I tried updating couple more times via the GUI.  The yt-dl installed version didn't change.
A few minutes later, I updated youtube-dl in terminal using sudo, which worked fine.
Then checking the version showed the latest yt-dl - at that time - was installed.
Today, saw a new 2018.04.16 version was released, so I tried updating again from the GUI.  It said "downloading the latest youtube-dl" or similar.
It finished -successful. No mention of the version & nothing in the log.
Now the version check shows 2018.04.09 is installed.  That's checking both /usr/bin & /usr/local/bin.
But  the version on https://rg3.github.io/youtube-dl/ is 2018.04.16, not 04.09.
So I don't know what's going on.
I don't know where youtube-dlg D/L's yt-dl updates from.
Is the source yt-dlg fetches from a bit later getting the newest releases shown on  https://rg3.github.io/youtube-dl/?
Bug report
Not sure if there is one - sorry.
What operating system do you use ?
Mint 18.1, updated except for kernel.
Feature request (request for a new functionality)
Regardless, I'd suggest that youtube-dlg show the currently installed yt-dl version -  somewhere prominent - or near the menu option that probably should say, "Update youtube-dl" instead of "update". :)
To make it easier to tell if it actually updated yt-dl & to what version.

That's already in the TODO list.


Use Read Write Consistency (RWC) Token to call Microsoft Graph API

Symptom
As reported in IcM 294491919, even though az ad sp create-for-rbac has retry logic to create service principal after the application creation replication/propagation is done:
https://github.com/Azure/azure-cli/blob/f8ea47cd3330d1125b322f6c54e3da947596c1c7/src/azure-cli/azure/cli/command_modules/role/custom.py#L1451-L1469
It is still be possible that after _RETRY_TIMES (whose value is 36), the replication is not complete, leading to service principal creation failure:

The appId 'c807447b-5118-4756-a6c5-90dbdf919d22' of the service principal does not reference a valid application object.

Possible solution
AAD has a mechanism called Read Write Consistency (RWC) Token to solve this:

https://docs.microsoft.com/en-us/azure/active-directory/fundamentals/active-directory-architecture#data-consistency
https://identitydivision.visualstudio.com/IdentityWiki/_wiki/wikis/IdentityWiki.wiki/8106/Read-Write-Consistency-(RWC)-Token

Problems

AAD has a very restrictive approach towards onboarding any team on RwcToken.
Even if we use RwcToken in az ad sp create-for-rbac, it is difficult to apply RwcToken to separate command executions like az ad app create and az ad sp create. Azure CLI will have to save the RwcToken to a local file (because each execution is a separate Python process). How to save this token securely needs to be decided.

Alternative solutions
Increase _RETRY_TIMES to higher value or use exponential backoff in order to increase the max totally retry time (currently 36*5s=180s).

Use Read Write Consistency (RWC)


Upgrade to Angular 9.1

https://blog.angular.io/version-9-1-of-angular-now-available-typescript-3-8-faster-builds-and-more-eb292f989428

Done


A.params.colors is undefined - Laravel livewire?

Using alpine/livewire on loading in Kucstomer we are getting this error. Any suggestions? config file is published.
I am wondering if A.params is somehow being unset.

I'm not sure this package is compatible with alpine and livewire...
Can you check how the kustomer code is included ?


fix: #112 - default NestedKey to never

Fixes #112
Modified files:

packages/use-intl/src/useTranslations.tsx

It's just my comment with the ideas applied:

And in the conditional, I just check if NestedKey is never: [NestedKey] extends [never] ? ... : .... It seems to work fine.
You can find the fix here

I tested your fix in a larger codebase where I'm using next-intl and it seems to work flawlessly. Thank you very much for your help @kelsny!


Why are emptybot created by composer and dotnet cli different to each other ?

Describe the solution you'd like

Describe alternatives you've considered

Hi,
when we create a .netcore app with visual studio, the generated files are exactly similar to files generated by dotnet cli command. Except for .sln file.
Why the same behavior does not happen when we create an empty bot with bot composer ?
When I create an empty bot with bot composer and these cli commands :
dotnet new -i Microsoft.Bot.Framework.CSharp.EmptyBot
dotnet new emptybot

The .csproj files are different with each other
I think it would be a good idea to make them look similar to each other.
for example :
.csproj file of the bot created by BotComposer
<Project Sdk="Microsoft.NET.Sdk.Web">
  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <AspNetCoreHostingModel>OutOfProcess</AspNetCoreHostingModel>
    <UserSecretsId>{redacted}</UserSecretsId>
  </PropertyGroup>
  <ItemGroup>
    <Content Include="**/*.blu;**/*.dialog;**/*.lg;**/*.lu;**/*.onnx;**/*.qna;**/*.txt" Exclude="$(BaseOutputPath)/**;$(BaseIntermediateOutputPath)/**;wwwroot/**">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
    </Content>
  </ItemGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Mvc.NewtonsoftJson" Version="3.1.8" />
    <PackageReference Include="Microsoft.Bot.Builder.AI.Luis" Version="4.13.1" />
    <PackageReference Include="Microsoft.Bot.Builder.AI.QnA" Version="4.13.1" />
    <PackageReference Include="Microsoft.Bot.Builder.Dialogs.Adaptive.Runtime" Version="4.13.1" />
  </ItemGroup>
</Project>

.csproj file of the bot created by dotnet cli commands
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <LangVersion>latest</LangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Mvc.NewtonsoftJson" Version="3.1.1" />
    <PackageReference Include="Microsoft.Bot.Builder.Integration.AspNet.Core" Version="4.13.1" />
  </ItemGroup>

  <ItemGroup>
    <Content Update="appsettings.json">
      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
    </Content>
  </ItemGroup>

</Project>

Regards,
Parsa Lotfi
Additional context

@tracyboehrer do you know why there are differences between the VS template and the dotnet template?

Wait, sorry - I might be wrong. Do you want to use Composer to create and edit your bot?

The only insight I can give is similar to Andrews.  These are coming from different areas of our stack, and we haven't aligned them.  The Composer csproj is specific to a Composer Bot project, which would/could have it's own requirements that differ from the pre-existing CLI template.  If I had a crystal ball, I would expect some alignment, but a Composer Bot could remain different.  Maybe more along the lines of everything moving into CLI, and that can produce a Composer compatible bot.

Wait, sorry - I might be wrong. Do you want to use Composer to create and edit your bot?

I want the ability of editing bots created by CLI in Composer
Bots created by composer include so many packages like QnA or LUIS which I never use.
So I create an empty bot using CLI , but I can't edit that bot through Composer.
You know I want the goods of both sides  :grin:
The simplicity of creating bot without having to code as well as having an lightweight clean bot like the one created by cli

Is there a way to open a bot created by cli with bot composer ?

If you like you could drop references to those packages using either a package manager like VS, or by removing them from the CS proj and rebuilding. The current empty bot template in the daily build of Composer (which I think is what you are using) only takes 4 package dependencies though. Composer also would not detect that you need those packages and add them back for you if you decided to use a feature that required them (it will expect them to be present).
The template that the dotnet CLI scaffolds does not use the adaptive dialog stack, and would not be missing a number of elements that Composer expects to be present. The empty bot that Composer generates for you represents the minimum set of dependencies, and the project/folder/file scaffolding that Composer expects to be present. You can of course feel free to alter this, but some portions of Composer may not work as expected if you do so.
If you would like to use a CLI tool to create your Composer compatible bot, you can do so using the Yeoman template - it is available here https://www.npmjs.com/package/@microsoft/generator-bot-empty, and you can find instructions for working with Yeoman here: https://yeoman.io/learning/

Hello again @clearab @tracyboehrer
That yeoman couldn't help me,
I want to make an empty bot with dotnet cli
And then edit that bot with Composer.
Pleeeaaaase make that possible 🙂😁
It could be a very good feature.
Thank you ❤


🐛 Add CSS overflow/wrapping on KaTeX, tables, and code sections

(Upstreaming changes from https://github.com/nunocoracao/blowfish/pull/1129)
This fixes a few bugs where long post elements could break out of the width of the overall page and break overall formatting or views, especially on mobile devices. The corresponding edits to assets/css/compiled/main.css are from the Tailwind compiler.
Note that the scroll bars on these elements will fade in and out as needed, depending on the default behavior of the user's browser.

FWIW, the relevant issue in the Blowfish fork is https://github.com/nunocoracao/blowfish/issues/990.

Thanks for these fixes, @ragibson. I'm happy to merge these in.


fix for #37

As discussed here https://groups.google.com/forum/#!topic/coreos-dev/A1Ei5Wv-M2A i have added the changes.

👍

for reference:
https://github.com/coreos/bugs/issues/1114


#25 Continuous Testing

Added all 1.7 versions of go to test against
Added Travis badge for master branch
Travis Builds Activated

Very cool to have this automated testing.  It's already finding bugs -- I'd neglected to update the cache test after changing the cache interface.  Thanks for taking on this work!


How can I overlay bitmap/image on landmarks? I want to overlap png on iris using facelandmarks.

This template is for miscellaneous issues not covered by the other issue categories
For questions on how to work with MediaPipe, or support for problems that are not verified bugs in MediaPipe, please go to StackOverflow and Slack communities.
If you are reporting a vulnerability, please use the dedicated reporting process.

Hello @prakashsrv,
Could you please elaborate your query with complete details, exact possible solutions you are looking for from us.
Thank you

I am looking to create a color lens try-on application using the MediaPipe FaceMesh module. My goal is to overlay a PNG image, containing different color lenses, on the iris of the user's eyes, as detected by the FaceMesh module. This will allow users to virtually "try on" different color lenses and see how they look on their eyes in real-time.

Hello @prakashsrv,
Sorry for the delayed response, Could you please confirm that you are still looking for the resolution on this or the issue has been resolved from your end?  Thank you!


Set metadatalastupdated to today's date in OSC form

the metadatalastupdated date to today's date so the the user does'nt have to manually change the date every time.

Now, when you, you select an administration and a project, the date get's set to  to today's date.

#143


Create Relationship with These Sponsors

As a USIVA staff, establish a relationship with the following sponsors where USIVA staff connects them to resources like the ones below and multiple hosts like Instar Lab, a research institute, or Tao Learning, a learning institute and both are 501c3. They have their labs set up in multiple areas in Ohio and West Virginia if they asked for them but don't give them addresses until it's confirmed that we will be working with them.
Provide notes from the meeting on what their criteria are to be a host and fees to pay.
If you are interested in conducting yoga research in the U.S., you might want to consider applying for a J-1 visa under the Research Scholar or Professor category. This category allows you to participate in research projects at accredited academic institutions, research centers, museums, libraries, or similar types of organizations. You can also teach or lecture, unless disallowed by the sponsor¹.
To apply for a J-1 visa under this category, you will need to find a program sponsor who will provide you with the necessary information and support for your visa application. You will also need to meet the following requirements¹:

You must have a minimum of a bachelor's degree or equivalent in your field of specialization.
You must have sufficient English language skills to successfully participate in your program.
You must have adequate financial resources to cover your expenses and those of your dependents, if any.
You must have health insurance that meets the minimum requirements set by the U.S. Department of State.
You must not be subject to the two-year home-country physical presence requirement or have a waiver for it.

Some of the possible J-1 visa program sponsors for yoga research are:

Global Internships: This program offers customized internships and research opportunities in various fields, including health and wellness. You can work with a host organization that matches your skills and interests, and gain valuable professional experience in the U.S. You can also enjoy cultural activities and events organized by the program. You can contact them at<EMAIL_ADDRESS>or +1<PHONE_NUMBER>².

BridgeUSA: This program has a specialist category for individuals who are experts in a field of specialized knowledge or skill. You can travel to the U.S. for the purpose of exchanging knowledge and skills with American specialists by observing, consulting, or demonstrating your expertise. You can also learn from the best practices and innovations in your field. You can contact them at<EMAIL_ADDRESS>or +1<PHONE_NUMBER>³.

[Cultural Vistas]: This program provides opportunities for international professionals to enhance their skills and knowledge through training and research programs in the U.S. You can participate in a program that suits your career goals and interests, and network with other professionals from around the world. You can also explore the culture and diversity of the U.S. through various activities and events. You can contact them at<EMAIL_ADDRESS>or +1<PHONE_NUMBER>.

I hope this helps you find the best program for your needs. Good luck! 😊
Source: Conversation with Bing, 9/25/2023
(1) Programs - BridgeUSABridgeUSA - J-1 Visa. https://j1visa.state.gov/programs/.
(2) Train USA J-1 Internships and Training - Cultural Vistas. https://culturalvistas.org/programs/us/j-1-internships-training/.
(3) Train USA J-1 Internships and Training - Cultural Vistas. https://culturalvistas.org/programs/us/j-1-internships-training/.
(4) J-1 Visa Rules, Requirements, & Application | Shorelight. https://shorelight.com/student-stories/student-visa-usa-j1/.
(5) J-1 Visa Rules, Requirements, & Application | Shorelight. https://shorelight.com/student-stories/student-visa-usa-j1/.

Global Internships:
BridgeUSA: Dropped the call
Cultural Vistas: Currently in another call so will call my work number.

https://www.globalinternships.com/intrax/contact-us
Use this link for the globalinternhsips.com

Global Internships.com
Starts with the Interns, they will register in our portal, the main documents, TOR, passport, their education, etc. Then we will reach out to the host company. Host org will email training plan and agreement between spon sor and host companies, they will contact directly.
If USIVA, third party, is involved then email marketing material, projections, and the category you're in. And talk with the person in charge. Not hospitality program this time. Industry they'll be training in are they interns or trainees<EMAIL_ADDRESS>Pricing changes depending on the country, Philippines $4000, India 12 month $4000, Europe, depends where but UK $2000. India and Philippines is processed in Japan and Germany is for European applications. Canada and Americas is in North America
If small company with less than 25 or less making than $3M a year. Then will do site visits on the main location, but will do multiple site visits.

Cultural Vistas: J-1 Category
I asked about the Research Scholar but always pointed out the email that would be sent to me with the pricing list and duration. And said that would help.
They work with plenty of J-1 Categories: And it will be available in the email that Dana "Rowan" will send to email about the Research Scholar, Interns, Trainees, Specialist.
They are grateful that we reached out to them.

BridgeUSA number above and from Claude.ai was not helpful and so the best to reach them is by email.
Unfortunately I do not have a specific phone number for BridgeUSA to provide host company information. However, here are a few suggestions to get more information:

Check the BridgeUSA website or contact page to see if they list any phone numbers or other contact methods for inquiries. Their website is https://www.bridgeusa.com/.

Search online for reviews, forums, or discussions about BridgeUSA where people may have posted contact info or asked similar questions. You may be able to find a useful phone number or email address that way.

Try calling their general contact number listed on their website, even if it's not specifically for host company inquiries, and ask to be directed to the right department or person who can help. Their main number is<PHONE_NUMBER>.

Reach out to them on social media, such as Twitter or Facebook, and ask your question. Many companies monitor social media closely and respond quickly.

If you know anyone who has used BridgeUSA before, ask them if they have a contact they worked with for host companies that they can refer you to.

As a last resort, you could visit or mail a letter to their head office in San Diego and see if someone there can assist you.

I hope these suggestions help point you in the right direction to get the contact information you need from BridgeUSA! Let me know if you have any other questions.

While BridgeUSA and Cultural Vistas are renowned for their sponsorship programs, there are several other organizations that also provide J-1 Visa sponsorship. Here are a few:
Council on International Educational Exchange (CIEE): CIEE is a non-profit organization promoting international education and exchange. It sponsors J-1 visas for various categories including professors, research scholars, and students.
Academic Year in the USA (AYUSA): AYUSA is a non-profit organization that offers various exchange programs for high school students and sponsors J-1 visas.
InterExchange: InterExchange is a non-profit organization that provides a wide range of cultural exchange programs. They sponsor J-1 visas for au pairs, interns, trainees, summer work travel, and more.
American Immigration Services Council (AISC): AISC is a non-profit organization that sponsors J-1 visas for interns and trainees in various fields.
Here are the contact details for the mentioned organizations:
Council on International Educational Exchange (CIEE): You can reach them at their customer service number: +1-207-553-4000
Academic Year in the USA (AYUSA): Their contact number is +1-415-231-3871
InterExchange: You can contact them at +1-212-924-0446
American Immigration Services Council (AISC): Their contact number is +1-202-347-8050

Council on International Educational Exchange (CIEE): You can reach them at their customer service number: +1-207-553-4000
Participants start with finding an International Representative. They will send an email on a list of International Representative.
Then, participants will find the host organizations. Both participant and hosts will work to create a training plan.
After they have submitted the training plan and is reviewed by the Representative, and is accepted then it will be forwarded to the CIEE and then the visa processes begin.


pythonPackages.cntk: mark as broken

Motivation for this change
The CNTK python module seems to be broken since November 2018 (does not recognize swig).
Mark it as broken.
Needs back port to 19.03 (https://github.com/NixOS/nixpkgs/issues/56826)
CC @costrouc @abbradar
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Since https://github.com/NixOS/nixpkgs/pull/54182, swig needs to be in nativeBuildInputs.

@dotlambda thanks for the tip. It does find swig now but fails at later stage with a different error.

It fails with

unable to execute 'mpic++': No such file or directory

because openmpi isn't in nativeBuildInputs. However, I'm not sure if that one should remain in buildInputs too.

OK, that solves the mpic++ error but then we get the next error:
ImportError: cannot import name 'GenericMeta' from 'typing'

openmpi should probably remain in buildInputs since it is also a run time dependency.

I restricted the broken attribute to python 3.7
@GrahamcOfBorg build python27Packages.cntk
@GrahamcOfBorg build python36Packages.cntk

I restricted the broken attribute to python 3.7

Perfect!

Thanks for the fixes!


UX

Thinking broadly about the direction of the front end user experience.

@smarandal-telenav I wrote down some thoughts about the user experience for New MapRoulette, see above. Looking forward to talking more next week.

I asked for community feedback here - there is some valuable input there as well.

As discussed this morning - UI development is on track and I will see if @smarandal-telenav's team can help with mobile web development instead.

closing this for now


Deducing the content type for files

If we're so inclined, we could read the first bytes of the file and use the mime_sniffer crate to and pick a content type automatically.
Alternatively, we could have an extension / MIME type mapping (mime_guess crate).

The playground guesses via filename, so I'm in favor. I had trouble trying to implement this in a reusable manner, however (although I don't remember why).

I am also in favor of supporting this 👍


Update to dotnet/maui/main und enhance

more to come
working branch is here: https://github.com/lytico/maui/tree/lytico/gtksharp/linux-ongoing
Description of Change
merges commit https://github.com/dotnet/maui/tree/7d8c24e4d51c80bfdebd5ba1122015335be18979
for linux/gtk
updates api changes
Additions made

Adds PageView
Adds Controls - NavigationPageHandler (stub)
Adds Compatibility.Gtk.csproj (and renames old Compatibility.GTK to Compatibility.GTK2)
Adds SimpleSample in Controls.Samples.Linux

Is this ready to review?. Thanks for your contributions!.

thanks for caring!
no, sorry. i've to consolidate with  https://github.com/lytico/maui/tree/lytico/gtksharp/linux-ongoin
it's far more progressed.
i'm struggling  with measure of long strings (lorem ipsum in example) if there is no initial size (withcontraint, heigthcontraint are inifine)
i think i can consolidate  today, even if this problem is not solved
when finished i'll change from draft to "not draft" and give you a note

i close this in favour of https://github.com/jsuarezruiz/maui-linux/pull/11


[Feature Request] Add support for nested folders in library

I tend to organize manga based on artist / writer and other tags, so my folder structure looks like this:
Library
-Artist
--Ken Akamatsu
---Love Hina
---A.I. Love You
--Shimoku Kio
---Genshiken
---Genshiken Nidaime
-Mags
--NewType
---[Range of folders with timestamps for monthly releases]
-Other
--Sort
---[Random assortment of stuff, including pdfs]
-etc.
So I would like Mango to be able to scan the subfolders of its library, and present it as subfolders / containers / whatever you wanna call it. This is actually an issue I have with Komga, as it can read subfolders, but treats the last containing folder as the "Series" for its contents. This isnt bad, but with no metadata support it makes it hard to use for me as it combines all these last folder names into one list that is displayed to the end user, so you would see stuff like this:
Love Hina
A.I. Love You
Genshiken
Genshiken Nidaime
[NewType Timestamp]
[NewType Timestamp]
[Other Mag Timestamp]
[Other Other Mag Timestamp]
[Random Book]
[Random Book]
[Random Book]
Not at all how I would want these ebooks / manga presented to me.

I agree! It would be a nice feature to have. I will try to include this in the next release.

This feature is added in v0.2.1. Thanks for the request!


Firefox doesn't show first item in directory

Steps to reproduce: Create an FtpServer instance serving a directory with more than one file/folder. Navigate to the server in Firefox. Observe that the (alphabetically) first entry in your directory, whether it's a file or folder, is not shown by Firefox.
This might be a bug in Firefox, since all other clients I've tested (including Chrome) don't have this problem, but I wanted to document it here anyway.
Firefox's internal FTP logging shows it executing PASV followed by CWD / followed by LIST when displaying the directory listing. Unfortunately it doesn't display the data received or how it's interpreting that data, so I want to investigate this a little more.
If I can't find anything, I think this can probably be closed sooner rather than later.

Will be fixed in 3.0.0-rc.4 or 3.0.0 (whatever comes first).


Basic unit and functional tests

Hi Martin,
I added unit and functional tests (incomplete) in this fork.  I used nodeunit and sinon .. hope that's ok.
One of the functional tests is commented out because I couldn't get it to pass.  I'm probably missing something simple.

Thanks, Bryan. I've checked in the functional tests, with a fixed substitute test. I'm not ready to expose the internals of mockery, so I haven't included the unit tests.


feature/#1551 - new class on top of ColorMappingForScalar for scalar range refresh

New class:

ColorMappingForScalarContainer: Scalar container on top of any ColorMappingForScalar

Impacted classes:

ColorMappingForScalar: created a clear method
ColorMappingVariation: fixed a computing bug on method mapScalar; made method init public and fixed a side-effect when computing the slope
IConfigurationProvider: unrelated micro-refactoring

sorry out of the loop, is this for doing fancy color fading effects on lines?

@spyhunter99 The fancy color effect is already there: this time, we make it more dynamic.
Typical use case: initially you don't know the altitude range of your hike so you put default values, but as you walk and as the GPS data arrive, you get a more relevant idea of the range.
Now it's easier to refresh the colors from the actual data (like min/max), on the fly.

Could be useful to provide a sample / documentation as extra sample in the sample application. I would do that on a different PR.

Perfect!


DESTDIR is ignored

Description of problem
make install DESTDIR=${foobar} is supposed to install all artifacts in dest dir ${foobar}.
Expected result
All artifacts are installed under ${foobar}
Actual result
Only kata-collect-data.sh is installed at ${foobar}

I am always in favor of removing things if we don't need them :)

https://github.com/kata-containers/runtime/pull/402 is trying resolve this issue.


vue.config.js chunks configuration

This is  a question.

Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[x] Did you check the documentation?
[x] Did you perform a cursory search?

Description
I have a question regarding the js/chunks generated by enso's  default configuration:
JS files generated on our side are 138. We do not use (require) other yarn packages except fontawesome-pro.
I see that on your demo site (https://www.laravel-enso.com/) there are ~58 chunks loaded inside index file.
Is there different configuration used for this demo site, compared to laravel-enso/enso?
I tried playing around with configureWebpack optimization.splitChunks but had no result comparing to https://www.laravel-enso.com/. I also thought maybe my yarn is messy and started fresh with no yarn.lock file, and no node_modules, but still no improvement.
Any ideas/suggestions you have are recommended.

An interesting read about Js speed optimization - https://www.diva-portal.org/smash/get/diva2:1318692/FULLTEXT01.pdf

Difference:
https://www.laravel-enso.com/

Local

I found the reasons why this happens. I'll document this here later.

@robbykrlos thanks!

The reason why I had so many files was because on top of node_modules there were the internal (local, custom made) vue/js files that generated chunks. many of them, very small ones.
This on a test environment that had 2 CPU cores and 4GB RAM running Apache was a bit overwhelming and had a delay loading all this parallel requests.
My solution was to optimize chunks and merge some together.
Ex:
configureWebpack: {
        devtool: inProduction ? 'hidden-source-map' : 'source-map',
        optimization: {
            nodeEnv: 'production',
            splitChunks: {
                cacheGroups: {
                    vendor: {
                        test: /[\\/]node_modules[\\/]/,
                        name(module) {
                            // get the name. E.g. node_modules/packageName/not/this/part.js
                            // or node_modules/packageName
                            const packageName = module.context.match(/[\\/]node_modules[\\/](.*?)([\\/]|$)/)[1];

                            // npm package names are URL-safe, but some servers don't like @ symbols
                            return `vendor.${packageName.replace('@', '')}`;
                        },
                    },
                  ...

Which in my case generated slightly bigger chunks, which in the ends reduces the number or requests which loads faster.
I also did the same (still WIP) with local files, which can be easily merged into one small file.
My latest tests generated ~30 files, instead of 140.

Bonus:
We also found that chunk files were not cached (browser cache). This was because on Apache mod_deflate.so is interfering with the ETag and generates a bug which makes the JS files flagged as "changed" and won't let the broswer see it as "the same"
Here is the Apache issue : https://bz.apache.org/bugzilla/show_bug.cgi?id=45023#c22
Here is the workaround after mod_deflate is used:
          #This fixes some Apache issues and was recommended in several posts.
          FileETag MTime Size

          #THIS IS THE FIX
          RequestHeader edit "If-None-Match" '^"((.*)-gzip)"$' '"$1", "$2"'

          AddOutputFilterByType DEFLATE text/plain
	  AddOutputFilterByType DEFLATE text/html
	  AddOutputFilterByType DEFLATE text/xml
	  AddOutputFilterByType DEFLATE text/css
	  AddOutputFilterByType DEFLATE application/xml
	  AddOutputFilterByType DEFLATE application/xhtml+xml
	  AddOutputFilterByType DEFLATE application/rss+xml
	  AddOutputFilterByType DEFLATE application/javascript
	  AddOutputFilterByType DEFLATE application/x-javascript

          BrowserMatch ^Mozilla/4 gzip-only-text/html
          BrowserMatch ^Mozilla/4\.[0678] no-gzip
          BrowserMatch \bMSIE\s(7|8)  !no-gzip !gzip-only-text/html

          Header set Vary Accept-Encoding

The difference can be seen if you refresh (soft-refresh, not hard) the login page. Initially all js files will be loaded with a 200 OK status.
Second refresh, it will load faster, because
200 (from disk cache)
or
200 (from memory cache)


Add Marvel Avengers emojipack

I couldn't find an authoritative emoji pack for Marvel characters so I threw this together. You can see them all here: http://www.free-emoticons.com/search.php?keyword=avengers . If there is a better site to find superhero emojis I'd be fine expanding this to include more.

Tested, works, merging!

Thanks for the contribution.


Race condition on file write

I am not sure of the root cause yet, but we get the following when coupling outfile with exec
cargo run -- --outfile /tmp/some.sh --interactive --exec ghcr.io/jyotsna-penumaka/hello_sget:latest
`thread 'main' panicked at 'Execution failed: Os { code: 26, kind: ExecutableFileBusy, message: "Text file busy" }', src/main.rs:95:14`

Looks like its from https://github.com/sigstore/sget/pull/88, as when I hard code a script name to replace path on the following, it works:
https://github.com/sigstore/sget/blob/decbfd63c391ecefce3eac03bd13e7842a636455/src/utils.rs#L4-L18

@lmt-swallow might have some ideas.

I noticed in https://github.com/sigstore/sget/pull/79#issuecomment-1009526165 that before #79 the behavior of exec was subtly broken to always execute tests/test.sh, regardless of what outfile was. This seems related?
I'll see if we can get some e2e tests in place that should guard against future regressions like these.

@imjasonh I think that well may be the case.


Fix transfer of joseph_spouse_of_mary in slovakia.ts

I have just noticed that joseph_spouse_of_mary is not transfered this year to Monday 20 Mar 2023 in slovakia.ts. It works as expected in GRC.
https://github.com/romcal/romcal/blob/5d3fc9c7be367bbb679bbcf63479a4db5e461241/lib/general-calendar/proper-of-saints.ts#L218-L234
https://github.com/romcal/romcal/blob/5d3fc9c7be367bbb679bbcf63479a4db5e461241/lib/particular-calendars/slovakia.ts#L21-L23
I think adding dateExceptions to slovakia.ts would work this issue around, but I think this needs to be fixed more deeply.
I believe this issues comes from the fact that joseph_spouse_of_mary is not a solemnity in Slovakia. I think that each holy day of obligation defined in the GRC should be transfered to the same date when necessary, regardless if that celebration is a HDO in a particular calendar or not.

It looks like a bug. When the date is computed, joseph_spouse_of_mary from the Slovakia calendar should include on all properties already defined in the General Roman calendar.


Support parallel iteration over events

What problem does this solve or what need does it fill?
Parallel iteration over events would open up more performance tuning options for bevy developers, inspired by query.par_iter.
What solution would you like?
Expose the ability to perform work in parallel over events, similar to Query::par_iter.
What alternative(s) have you considered?
I tried rayon but it wasn't able to par_iter the event reader.
Another option is to collect the events into a Vec before using rayon, but it would require cloning each event.
Using a vanilla Vec in a resource as opposed to the event system.
ResMut<Events<SendServerMessageEvent>>::drain() is a possibility, but it requires use of ParallelBridge in rayon which harms performance.
Additional context
I'm creating events all over my app that need to be sent over the network. Message serialisation of these events is compute-bound and represents a blocking, non-trivial workload that takes place at the end of the schedule.
I would like to test with a parallel version of event reader for performance reasons.

I think this should be fairly straightforward to implement:  we just don't expose a method for it.


翻译production-build

翻译完成guide/production-build.md文件

webpack:master is not the intended target? :fearful:

OMG, I create a wrong PR.
I'm sorry, I will  close this one.


As a Content Editor, I want to add text content to the events page so that I can add context to the list of events.

moved from asana task
should this text be optional?

tested in qa and was able to insert a blurb above the events area:
https://test-web.cdh.princeton.edu/events/

...but it probably needs to be optional, and it isn't right now.

so we can't make the text optional, but we can detect in the template if it's only whitespace and not render the section if so. we need to do this on the events list page as well as profile list pages.

The only thing I could figure out was to check explicitly for two non-breaking spaces (which is the minimum I was able to get mezzanine to allow me to save before). Hopefully this is good enough for now.

closing since @rlskoeser got this to work in QA; despite my inability to type a non-breaking space.


MSIX: for powerrenameuwpui path

Currently we miss powerrenameuwpui in MSIX, because this commit changed its output path. It's very convenient to have MSIX working for testing stuff like toast notifications etc., and it's an easy fix.

We can close this one.


Restore winit deps

Accidentally removed in
https://github.com/jwilm/alacritty/commit/c4ece6dde3c9dcf825a44aa775535a65c0c376a6
when winit version was bumped.

Thanks!


chore: add binaries for 1.27.16, 1.28.12, 1.29.7, 1.30.3

What type of PR is this?
/kind feature

What this PR does / why we need it:
add binaries cache for kubernetes patch versions 1.27.16, 1.28.12, 1.29.7, 1.30.3
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
2566

Relevant Lines:
3613

💛  - Coveralls


Jinja syntax highlight

Problem
It's hard to read Jinja language and complex expressions inside the playbook
Solution
There was a extension with modifications to support :

Jinja language
Jinja expressions for ansible conditions (when, changed_when, failed_when, check_mode)

https://github.com/haad/vs-language-ansible
But it was discontinued, would be awesome if you have that enhancement on yours extension.
Alternatives
No response
Additional context
You can tell the difference looking at the yaml code below using the ansible-advanced and the current extension
date_range: |-
    {%- set date_range = [] -%}
    {%- set epoch_timestamp = lookup('pipe', 'date +"%s"') -%}
    {%- for item in range(retention | int, 1) -%}
    {%- set ignored = date_range.append('%Y.%m.%d' | strftime(epoch_timestamp | int + 86400 * item)) -%}
    {%- endfor -%}
    {{ date_range }}
when: hostvars[inventory_hostname].os in platform_supported

I think it's a great idea and I think we'd be happy to review a PR if somebody sends one. Note that there's a good chance that such a contribution belongs in the ansible-language-server repository and not this one.

@priyamsahoo @ganeshrn I checked this and our languageTM does recognize {{...}} blocks but not the {%...%} and {# ... #} ones, so it should not be too hard to also recognize these.
Sadly, I am clueless regarding the syntax used to define these. Does any of you know how to patch them?

This was fixed in current versions of the extension.


Add "backplateTemperature" to Nest Thermostat Devices

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
When a wireless temperature sensor is selected to use as the reference temperature to trigger heating/cooling, the temperature at the thermostat itself, as viewed in Home Assistant, is overwritten by the temperature of the wireless temp sensor. This makes tracking temperatures by room, over time, impossible in Home Assistant because the data source for the thermostat's "temperature" reading can change.
Describe the solution you'd like
Referencing Starling's API documentation, "Thermostat" devices have two temperature measurement values:
backplateTemperature: Temperature at the thermostat, in degrees C
currentTemperature: Temperature at the currently active Temperature Sensor, in degrees C. If no Temperature Sensors are present, or none is active, returns the temperature at the thermostat.
Right now, it appears "currentTemperature" is the only value available for thermostats in Home Assistant. Can you please add "backplateTemperature" to the device in Home Assistant to report the temperature at the thermostat, regardless of whether a temperature sensor is selected or not?
Describe alternatives you've considered
Other than simply not using the temperature sensors for heating/cooling purposes, there is no alternative for this problem.
Additional context
There is a "temperature" value in Home Assistant that appears to always report "null." I'm not sure whether that's just a placeholder or if this is related to the subject issue.

Does it make sense for backplateTemperature to be used as the temperature reported in the climate entity? Or just as a separate sensor value?

I don't think so, since I believe "temperature" in the climate entity is used to determine whether heating or cooling needs to be called for (and when to stop), which should be the temperature of either the thermostat itself or the value of the active temperature sensor, if one is selected. It would probably make more sense to add backplateTemperature as a separate sensor value.

No problem. This has been added. Thanks!


Dockerfile

Minimal (~11MB) Dockerfile container build
Updated README with docker badge and links

initial docker hub automated build vs my fork https://github.com/mrnetops/fastly-exporter seems to be working correctly.
docker run -p 8080:8080 mrnetops/fastly-exporter:dockerfile -token $FASTLY_API_TOKEN

Once this is merged I'll destroy and recreate mrnetops/fastly-exporter using peterbourgon/fastly-exporter as the source repo.
I have a test docker hum automated build that should hopefully trigger automatically when this PR gets merged.
https://hub.docker.com/r/mrnetops/fastly-exporter-test2/

This is great. Thank you for doing this!
Just a couple of questions:

Why are we rebuilding the binary when the release script already did it (and bit more efficiently)?
What is going to trigger the docker build and push?
Can the docker build and push to the docker hub be part of the release process, so that it's automatic?

@msuterski
for #1 the rebuild is done to ensure that the code and all dependencies -a are rebuilt with CGO_ENABLED=0 to remove libc dependencies in the first stage.  This allows us to copy just our binary and direct dependencies into the second stage based on SCRATCH.  Adding just our binary and dependencies results in a dramatically smaller docker image (~330MB -> ~11MB)
for #2 & #3 hopefully that will be accomplished via Docker Hub automated build github integration
https://docs.docker.com/docker-hub/builds/

@mrnetops thanks a lot!


types-only partially evaluation

Implement EvalAmount::Types, which should evaluate any expressions referenced by types and leave everything else unevaluated.

Also add an EvalAmount::Minimal which only evals definitions of type number.


Visibility of index names in dropdown should be based on access permissions to them

Hello there!
It would be nice if the dropdown that shows the Elasticsearch indices depended on the user permissions for each one.
For example, if user A have access to the index B, it should be shown in the dropdown. But, if user C lacks access to that index, it should be hidden from the dropdown.
This way, it would be possible to use Search Guard Kibana plugin in a SIEM-as-a-service tool because one customer would only see his own data and every other data (including the index name) wouldn't be visible for him.
Thanks,
Best regards

That's not easily possible, due to the way Kibana works. Kibana does not have any multi-tenancy capabilities built in, but uses one global user (the one you configure in kibana.yml) to issue those types of queries. That's also why all users share the same dashboards and visualizations. We're working on an Search Guard Kibana module at the moment, which will in a first step provide separation of dashboards and visualizations based on a users role(s).

Thanks @floragunncom .
I will try to be patient :wink:
BTW, do you have any public roadmap?
Best regards!

cool, i can already see the config item 'multi-tenancy' in search-guard-kibana-plugin

Not at the moment, since requirements from customers and users are changing fast. However, we think about setting up a voting tool on our website, so you can submit new feature requests and upvote existing ones.

Hello,
I think that this issue should be closed since you have published the new 5.3.0 version, which includes this enhancement (using tenants).
Thanks!!!


Having difficulty in finding Labour Data.csv in  Part-6

I have some difficulty in finding Labour Data.csv used in Part-6: Scatter Plot & Joint Plot. Could you please tell me where to find it?

Thank you for bringing it to my attention! I've now added that dataset in the Datasets folder.


merge from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 -x 'tests/**/*.mjs' --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp
chore(package): add main from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/*-test.mjs' as in template
chore(package): set $.ava.extensions='mjs' as in template
chore(package): remove unknown value for main ({{main}})

Pull Request Test Coverage Report for Build 2191

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

Totals

Change from base Build 2186:
0.0%

Covered Lines:
361

Relevant Lines:
375

💛  - Coveralls


[stable/gocd] Update ingress doc

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@GaneshSPatil  -  GoCD agents outside of the cluster connect to the GoCD server through Ingress on port 80?

Looks good to me. I'll let @GaneshSPatil approve it. I wonder if we should add a link to (or mention) the location within values.yaml that these annotations need to be added to. Maybe it will be clear that it is this?

@arvindsv - thank you! Yeah, I'll make that change. Looks like there's https://github.com/helm/charts/blob/dc4e53a219fc079074be8a2e8c65afb6608ac1f7/stable/gocd/values.yaml#L157 to add too.

@GaneshSPatil , @arvindsv  - um, one of you need to add lgtm for this to get merged.

/lgtm :)

/lgtm


Creating archive files

Hi! Thanks a lot for this library!
Do you have plans to handle creation of ar files as well?

I'm happy to accept a PR doing it (it would likely have to support a parameter for which format is to be used), but I don't have plans to implement it myself right now. Maybe in a distant future :)


Fixed compilation and instructions for debian, arch, and docker

Installation instructions seemed to be outdated. With the changes Usuba can be compiled on Debina, Arch, and Docker containers. With the updated instructions, Usuba can compile some circuits such as ace, 2darray but not AES.

Thanks a lot for this PR!
It looks fine so I may be able to merge it soon


Refuses to install despite Gcloud being installed.

The MSI installer exits with the message "Google Cloud SDK is not installed on this machine", despite running the msiexec /i CloudIap-Plugin-<IP_ADDRESS>.msi from the Google Cloud Shell.

gcloud --version
Google Cloud SDK 262.0.0
beta 2019.05.17
bq 2.0.47
core 2019.09.06
gsutil 4.42

I was able to get this working with the following steps:
-Manually extract the contents of the MSI
-Manually copy the .dll files to C:\Program Files (x86)\Microsoft\Remote Desktop Connection Manager>
-Run RDCMan.exe from the Google Cloud SDK Shell command prompt (it crashes with a generic 0xe0434352 KERNELBASE.dll .net error otherwise if you run it from anywhere else).

However, the distributed installer https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/releases/download/1.0.6/CloudIap-Plugin-<IP_ADDRESS>.msi does not work.

Win10 Pro x64 build 1803

Did you install the Cloud SDK using the installer or using the Zip file?
The installer checks for the value InstallLocation in the registry key SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Google Cloud SDK  (32 bit hive) to locate the Cloud SDK -- but this key/value is only present when using the installer.
Regarding the crash when run from a different folder. Could you check if the PATH environment variable is set? I figure the crash might be caused by `PATH´ not being set, which again could be a result of the Cloud SDK being installed using the Zip and not using the installer.
Thanks for your feedback and sorry for the late reply.

Proposed fix in https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/pull/11

I installed with the installer.
The current sdk installer does not populate that value on current versions of Windows.
Also, when you are referring to the PATH variable, what is the expected value that needs to exist in PATH? And is it the current user PATH or system PATH?
--
Zachary Hanna
On Sep 21, 2019, at 7:53 AM, J. Passing<EMAIL_ADDRESS>wrote:

Did you install the Cloud SDK using the installer or using the Zip file?
The installer checks for the value InstallLocation in the registry key SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Google Cloud SDK (32 bit hive) to locate the Cloud SDK -- but this key/value is only present when using the installer.
Regarding the crash when run from a different folder. Could you check if the PATH environment variable is set? I figure the crash might be caused by `PATH´ not being set, which again could be a result of the Cloud SDK being installed using the Zip and not using the installer.
Thanks for your feedback and sorry for the late reply.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGoogleCloudPlatform%2Fiap-windows-rdc-plugin%2Fissues%2F10%3Femail_source%3Dnotifications%26email_token%3DANEOWM4LUQW3MZDOQG4M4MTQKYROVA5CNFSM4IVLMDTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ISGQQ%23issuecomment-533799746&data=02|01|zhanna%40taos.com|06abc40bd3f8419acbe808d73e9b1ff4|3d19c14acc2741378d269d3003e15e5f|1|0|637046708295817325&sdata=jzEbHVRLZ%2Ft7XGXBQ11TxYC%2Ba6CZQzlKqNrMSHvaFdo%3D&reserved=0, or mute the threadhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANEOWM4D5BV6OAEZFSLWK2DQKYROVANCNFSM4IVLMDTA&data=02|01|zhanna%40taos.com|06abc40bd3f8419acbe808d73e9b1ff4|3d19c14acc2741378d269d3003e15e5f|1|0|637046708295827321&sdata=BD7Phv%2FX8DUKKHSNjcvrKx%2BLC1qmDSojNDa%2BvfO1V9M%3D&reserved=0.

You are right, that registry key is only populated if you do a per-machine install of the Cloud SDK, in case of a per-user install, the key is created in HKCU. The MSI's LaunchCondition currently only checks for HKLM and misses to check in HKCU.
Anyway, I removed the MSI's LaunchCondition in https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/pull/11 in order to also support Zip-based installs.
By default, the installer adds the Cloud SDK's bin directory to the PATH and the plugin currently expects that to be the case. That's a bit brittle and I fixed that in https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/pull/11 also.
I will publish a new release early next week.

Would you mind giving https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/releases/tag/1.10 a try?

Fixed in version 1.10


Fixed #367

Fixes #367.
Changes proposed in this pull request:

fixed disable/enable job/app process

Coverage decreased (-0.03%) to 91.953% when pulling 064eca0ab1804af654599bb06eae2ad6a1e59ea5 on hanahmily:master into a3db7607ea9d33beb7b94c95ac0f56b7589d2b11 on dangdangdotcom:2.1.5-SNAPSHOT.


feat: 🎸 message-format cache compiled messages

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngneat/transloco/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Message format compile is always used.
Issue Number: #358
What is the new behavior?
Message format use compile only once per message.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
none

Hi @k3nsei LGTM, but I don't think we want that as default behavior, so let's add it as an option in the plugin's configuration and make it "false" by default.

@k3nsei Hey! will you have the time to do the change requested by @itayod so we continue with this PR? 🙂

@shaharkazaz yes I will do that.

Hi @k3nsei, thanks for the update.

can you add this optimization to the plugin's config and make it false by default (as I wrote above).
if you may, you can add it to the documentation.
please don't force push it makes it hard to track the changes :)

Thanks!

@k3nsei Hey!
I really want to push this forward 🙏 (and add you as a contributor! 🥳 )
These are the fixes needed:

Make this optimization configurable as part of the config passed to the module with the default of false( we will make it true by default in the next major version, but we don't want any side effects now).
Update the docs accordingly.
Add short tests making sure it's working as expected.

We're here if you need assistance. Let's merge this thing. 💪

@itayod You can pick it up. I can't find free time to move it forward.

@itayod You can pick it up. I can't find free time to move it forward.

@itayod You can pick it up. I can't find free time to move it forward.

No rush, feel free to continue work this when you can.

@itayod You can pick it up. I can't find free time to move it forward.

No rush, feel free to continue work this when you can.

Added in v3

@shaharkazaz Hi! What is the fate of this feature request?


Use 720h instead of 30d for KCM flags

How to categorize this PR?

/area usability
/kind bug
What this PR does / why we need it:
Introduced with #5096, it seems not all Kubernetes versions can cater with 30d as value...
Error: invalid argument "30d" for "--cluster-signing-duration" flag: time: unknown unit "d" in duration "30d"

Let's use 720h instead to be on the safe side.
Release note:

NONE

/assign

Thanks! I was able to reproduce the issue with K8s 1.19.16.
From https://cs.opensource.google/go/go/+/refs/tags/go1.17.3:src/time/format.go;l=1508:

// Valid time units are "ns", "us" (or "µs"), "ms", "s", "m", "h".

@rfranzke that should probably be included in a hotfix/patch release of release v1.37?

Luckily there is no need since https://github.com/gardener/gardener/pull/5096 was only merged after v1.37 got released :)


[FLINK-33677][core] Remove flink-conf.yaml from flink dist.

What is the purpose of the change
This pr introduces breaking changes that will be included in FLINK-2.0: Remove flink-conf.yaml from flink dist.
Brief change log
Remove flink-conf.yaml from flink dist.
Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@zhuzhurk, thanks for the review, I've updated this pr accordingly, PTAL.

I've updated the PR to completely remove the standard YAML flag from the code.
Additionally, I've removed the setStandardYaml calls from RocksDBStateBackendConfigTest#testConfigureForStCompressionPerLevel and ForStStateBackendConfigTest#testConfigureForStCompressionPerLevel, as discussed with @Zakelly. Although this case is inconsistent with previous behavior, it remains meaningful.

@flinkbot run azure

I've updated the PR to completely remove the standard YAML flag from the code. Additionally, I've removed the setStandardYaml calls from RocksDBStateBackendConfigTest#testConfigureForStCompressionPerLevel and ForStStateBackendConfigTest#testConfigureForStCompressionPerLevel, as discussed with @Zakelly. Although this case is inconsistent with previous behavior, it remains meaningful.

+1 for this part.

Thanks @zhuzhurk , I've updated this pr accordingly, PTAL.


Improve readme

Edit context wording
Use when and with according to the case
Ref:

https://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/ContextWording
https://www.betterspecs.org/#contexts

Fix examples related to *_only_actions
*_only should list all the methods, so new and create (created by
the Pundit policy generator or expected to exist by default in the
policy) should be both part of the array

Nice changes. Thanks.


Support for data labeling with custom LLMs

A custom LLM allows a user to plug-in their LLM into the Autolabel library for the task of data labeling. This could be hosted on their local machine or in a cloud that can called with an API.
High level design:

A new "Custom LLM" class that extends the Autolabel BaseModel (https://github.com/refuel-ai/autolabel/blob/main/src/autolabel/models/base.py). This class needs to override two methods: label and get_cost in addition to any initialization logic.

For reference, take a look at other LLM provider classes:
-- OpenAI, Cohere, Anthropic etc internally call Langchain's LLM class (e.g. https://github.com/refuel-ai/autolabel/blob/main/src/autolabel/models/cohere.py#L37)
-- HuggingFace initializes a huggingface pipeline (e.g. https://github.com/refuel-ai/autolabel/blob/main/src/autolabel/models/hf_pipeline.py#L76)

For the custom LLM, we'd want the user to pass in a callable function with the following signature. This can be one of the parameters passed in the config Dictionary in the init call.

def callable_fn(prompt: str) -> str:
      <custom LLM logic here>
      return <completion response from LLM>

The Custom LLM class calls this function for labeling.

To use the custom LLM class, add the relevant if/else case here: https://github.com/refuel-ai/autolabel/blob/main/src/autolabel/models/init.py#L11
the provider can be "custom" or "self-hosted"

@Sardhendu to work on this

@rajasbansal for comments or suggested changes

lgtm - we should allow loading on both cpu and gpu


How to enable Google Analytics

Google analytics seems to be not working again (related https://github.com/Track3/hermit/issues/35)
This is the source code for my website:
https://github.com/moorara/milad.dev
I only added googleAnalytics to my config.toml.

Fixed!


Display lint output

Description
Display any lint output
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[X] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[X] I run go fmt ./... to check that my code is properly formatted
[X] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #1034 (39bc043) into main (0f31037) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1034      +/-   ##
==========================================
- Coverage   30.35%   30.19%   -0.16%
==========================================
  Files          46       46
  Lines        5588     5633      +45
==========================================
+ Hits         1696     1701       +5
- Misses       3655     3695      +40
  Partials      237      237

Impacted Files
Coverage Δ

pkg/services/errors.go
20.68% <0.00%> (-2.39%)
:arrow_down:

pkg/routes/thirdpartyrepo.go
18.29% <0.00%> (-1.38%)
:arrow_down:

pkg/services/consumers.go
20.22% <0.00%> (-1.21%)
:arrow_down:

cmd/migrate/main.go
0.00% <0.00%> (ø)

pkg/models/images.go
100.00% <0.00%> (ø)

pkg/models/updates.go
0.00% <0.00%> (ø)

pkg/models/devicegroups.go
100.00% <0.00%> (ø)

pkg/models/thirdpartyrepo.go
0.00% <0.00%> (ø)

pkg/services/thirdpartyrepo.go
0.00% <0.00%> (ø)

pkg/services/devices.go
59.48% <0.00%> (+0.47%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f31037...39bc043. Read the comment docs.

Test results are available in Ibutsu. The test run IDs are:
1d7249d9-ae05-4356-a0cb-7e0f62f9a686
6b5f8db5-966d-4d9d-84f9-83829ce90128

Test results are available in Ibutsu. The test run IDs are:
43dfa442-31a9-4696-a379-45deaca72419
ab10d2e6-0a23-4b77-a411-127ff7883f9c

Test results are available in Ibutsu. The test run IDs are:
4d96ec14-b995-4bdb-a5b6-4e9d58d33d50
e215600e-a7f3-4fc6-b077-16b341c54db9

Test results are available in Ibutsu. The test run IDs are:
3180b393-112e-4048-b5b4-e8783b957276
634575b0-9b37-4480-8024-9a8f4de2cfdb

Test results are available in Ibutsu. The test run IDs are:
38422616-2373-4cb3-bff0-aa6e711e7117
e11c2cb6-62b4-4d1c-9fdd-7988eef5e789

Test results are available in Ibutsu. The test run IDs are:
58e4e46d-8aa7-45b9-9cec-3a2180ca5bc9
8c1bd0b0-6316-41c9-9e4c-9447497eaa29


[enhancement]: Duplicate and Complement regions

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
clsn on discord
What should this feature add?
Regarding regional prompting and auto-negative.
Sometimes, auto-negative isn't what you really want.  You'd like to be able to say "I want this negative prompt in the complement of that region."  If I have a region that contains "an adorable long-haired cat," I'd like the complement of that region to have "cat" as its negative prompt, but not necessarily the qualifiers.
Also, at least in a video presentation someone wound up drawing two regions that were more or less the same.  So probably the simplest and most flexible way to do this is to have a "duplicate this region/layer" action, and a "take the complement of this region" switch or button, so the user can compose these things as needed.  That seems like the most intuitive and flexible way to provide this.
Alternatives
Other possibilities are more complicated, "linking" a layer to its complement (which has the advantage that editing the region automatically updates the complement, but I don't think that's really an important advantage).  The "take the complement of this layer" could be a switch (setting a flag or something) or it could be an operation (actually converting the region); either way would be okay and would probably be about the same for the user.
Additional Content
No response

My during initial testing I had auto negative as an option along with invert mask. You could choose either or - it doesn't make sense to enable both at once.
The problem is visualizing the inverted mask, I'll need to think about how to handle that. Or maybe it's not something that is visualized.

You might (or might not) want to keep the "auto-negative" feature as it is even if you implement these things, as a shorthand convenience form.
I don't really see anything wrong with "Invert region" literally recalculating the region as its inverse and thereby making potentially a huge big color rectangle over most of the image.  That's the image you're talking about after all, you might as well see it.
I also don't see anything wrong with not doing that and having that region tagged as an "inverse region" and you just have to remember what that means, like with the current auto-negative, though that is potentially more confusing.  Other options involve things like filling it with a special pattern or outline (cross-hatching, concentric circles or outlines, drawing only the outline and hoping people don't confuse it for a skinny region, a check-pattern, halftone), or reserving some section of the color-space for negatives and some for positives (that last sounds like more trouble than it's worth.)


Fix disappearance of the elevator in Endless Canyon

The elevator in Endless Canyon is disappeared.

It is recognized as an issue by the author, so it should be fixed.

It is fixed.

(Note: This elevator moves if you move the elevator in Friend's house. But, the elevator in Friend's house does not move if you move this elevator, so it is negligible now.)


Support Temporal Projection Filtering

SELECT * FROM yellow_tripdata
WHERE tpep_dropoff_datetime >= '{}'
AND tpep_dropoff_datetime < '{}'

The following query can use the underlying Count wheel to check whether there are any entries for the range, skipping execution if there aren't any.

This optimization shows the most improvement when dealing with fine-grained time ranges at second granularity.


[1.12] Backpack item has a missing texture.

Using Wearable Backpacks 1.12-3.0.3.

Someone reported this on CurseForge so I tried reproducing this with the latest Forge version yesterday, and could not. Is this an issue with a particular Forge version or another mod?
Please see if you run into the same issue if you use latest Forge, and if so, if it only happens with Wearable Backpacks enabled. If everything works fine then, it must be a conflict with another mod. I would appreciate it if you could find out which one that is.

This might be related to VanillaImmersion#12. I'm currently looking into it, but it seems to be related to Item.delegate being messed up for some reason. I'll report back when I have clear results.

I've found the culprit: You set the item model during the pre-init phase (see here. At this point, the Item hasn't been registered and hence Item.delegate does not have a valid name. Unfortunately, ModelLoader uses the delegate's name as part of the key for MRLs. I made the same mistake in my mod and hence my hammer had the model of your backpack while yours got a missing model. I assume that you were unable to reproduce this because Vanilla does not use the ModelLoader functions.
The fix for this issue is to use ModelRegistryEvent and to have all calls to setCustomModelResourceLocation there.
Whatever caused the issue in @elucent's setup probably has to change this as well.

Ah, so if I understand you correctly, @PaleoCrafter, this is due to the changes in the registry "pipeline" (unsure if that's the correct word) in Forge 1.12, and doesn't affect other versions?

Yep, this 1.12 exclusive. The registry events are fired after pre-init after the rewrite, so model registering has to happen latter (preferably in ModelRegistryEvent).


OpenCloud driver

I've written a driver for php-opencloud which I'd like to send a PR for. But I've got a couple of questions...

Looking at the Doctrine driver, if you pushMessage() to a non-existent queue it'll get created for you. But it doesn't look like SQS or IronMQ work that way (though I'm not at all familiar with the underlying API). What's right?
By default the php-opencloud API doesn't return the client's own messages when listing them, but looking at the Doctrine driver's peekQueue() I think it probably should. Is that right?
The tests I've got require a Rackspace account. I don't like mocking what I don't know, but I guess I could skip them if an account isn't available and add some mocks. What's the best approach here?

Thanks!

OK, I'm converting the tests to use Guzzle's MockPlugin to simulate the responses from the API. That way at least the php-opencloud touch points get tested.

My code is available over here. I'd do a PR right away, but I'm beginning to think this would be better in a separate repository (as maybe would the other drivers?). There are a couple of reasons:

The current stable version php-opencloud has a dependency on Guzzle 3, which creates an ugly warning in composer
Version 2 of php-opencloud is in the works, but not progressing quickly. It looks to be PHP7+, as opposed to Bernard's 5.4+
I'd like to have a go at creating a v2 version, but that would be much easier in a separate repo

@henrikbjorn - what are your thoughts? I'll do a PR against this repo if you want. Or if you think a separate repo is a good idea, create one and I'll do a PR against that. Cheers!

Hey @kynx
I created https://github.com/bernardphp/OpenCloud for you.

Awesome! PR is at https://github.com/bernardphp/OpenCloud/pull/1

Moving to bernardphp/drivers#12


output to stderr

@maxtaco

:mag_right: The commits 21aa399329f4...c634f37b838b introduced some visual changes:

Changed: Username_Email_Login-Name (before) (after)

👍


Add Peek to lru cache

Peek: returns the key's value (or nil if not found) without updating the cache.

Oh, this is only about the lru package within groupcache?
Just fork it and make your own lru. There are so many and they differ so much. I don't want to be a maintainer of a flexible lru package.
I thought this was about groupcache.

:) no worries.