Multi-Job aggregation

If you use the task within a multi-job pipeline und call the task in 2 different jobs the tab is displayed und shows only the test results from the first job, that used the task.
I'm not sure how to solve this, but maybe it is a possibility to get one tab per job where the task is executed (and the tab can be named by a parameter tabName: 'Given Unit Tests" for example or, if no tabName is given, a default "JGiven Job 1" or something like this )
Or all results of all jobs can be aggregated in one tab over all jobs (that would be the preferred solution)

Solved in the initial implementation. All the results are aggregated in one tab.


0.2.8 cultures

Added pytests for new methods
Removed self.Dependencies in PyMeasure class
Fixed flake8 issues

@Daandamhuis I had to remove the self.Dependencies from the PyMeasure class. I ran through some tests with a model that had over 800 measures and it took too long to connect, because on connection the DMV query was executing for every single measure which took over a few minutes.
Obviously keeping measure.get_dependencies() (which I love btw, honestly didn't know that existed in DMV).


fs: fs.WriteStream:_write

The callback argument is always called, making it not optional.

Appears as if the bytes argument is never passed to the callback as well, so it was removed

Thanks for https://github.com/Ta-noshii who pointed this issue out.


Model Pose Landmark.

Ticket
None
Problem description
Add model Pose Landmark to trace.
This test depends on fixture manage_depedencies which was developed in PR 137. So, review PR 137 first, and then I continue to finish this PR in case of any change according to PR 137.
What's changed
Add model Pose Landmark.

@ayerofieiev-tt
We need to install Git LFS in the CI for this PR to pass.
Currently, the PR is encountering an error related to opening a file, which is specifically an LFS file.
E     RuntimeError: Unable to open file at /home/ubuntu/actions-runner/_work/pytorch2.0_ttnn/pyto\
rch2.0_ttnn/pose_landmarker_lite.task

This PR contains two LFS files.

pose_landmarker_lite.task
test_image.jpg

Installed. Running on CI to check

I found that the missing file problem involves several causes:

Not installed Git LFS. ✅ Solved as you said.
Wrongly set the file path in the test. ✅ Solved by the commits I just sent.
Not ran git lfs pull to actually get the content of LFS files. ❓

The CI still failed and I suspect it's the 3rd cause.
https://github.com/tenstorrent/pytorch2.0_ttnn/actions/runs/10858105217/job/30135655187
The error is about unrecognized file format, which I guess, is now the LFS pointer content before downloaded.
E     RuntimeError: Unable to open zip archive.

To confirm my guess, could you please check the CI server to see the file size of the two LFS files: tests/models/pose_landmark/pose_landmarker_lite.task and tests/models/pose_landmark/test_image.jpg?  The file size should not be small about a few bytes.


More data

At the moment, there is not that much data(only wave height, direction, period)
We can use more weather-related data. The weather data API documentation is here.
Some ideas: rain, temperature, windspeed, etc.

Hi, @ryansurf,  I'd like to work on this issue if available.

Thank you @ryansurf if there are not parameters to exclude between daily/hourly/current weather variables, I wanted to at least at visibility, cloud cover total and  UV Index, with potential to add more if I find relevancy.


Add vault iam auth example

This PR adds an example on how to login to Vault using the iam method.
Improved docs and tests to come on a future PR

Tests are failing after latest commit! I will investigate this, amd when they pass it should be ready to merge

Ok, most comments were addressed and the ones that weren't, the reason was that it could be addressed in a future PR, and now the CI issue has been fixed.
@brikis98 @josh-padnick do you want to give it another quick look before merge or should I go ahead?
Notify:
@infosecgithub
@bgt101


Use abbreviation in predefined DD functions

some name tweak
fix typo

I guess spin echo is fine? (although in the doc we do call it se)

@tachikoma-li I don't understand the motivation behind these changes clearly.
The scheme names should match the names of the methods, as they did previously.
"Carr-Purcell-Meiboom-Gill" => CARR_PURCELL_MEIBOOM_GILL => new_carr_purcell_meiboom_gill
It seems like you're making a breaking change to the API just to change some terminology. But the terminology is not necessarily wrong?
If you want to just eliminate the terms single axis and use some abbreviations. That's fine. But just do it completely consistently with the constants as well.
So make sure you also change this file:
https://github.com/qctrl/python-open-controls/blob/master/qctrlopencontrols/constants.py
Also these changes will break the Jupyter notebook examples, wherever they ended up landing.

@tachikoma-li if you're just migrating the documentation. I'd say just leave all the names the same, since they are currently consistent with the string names. And then just add a doc string.

@tachikoma-li I don't understand the motivation behind these changes clearly.
The scheme names should match the names of the methods, as they did previously.
"Carr-Purcell-Meiboom-Gill" => CARR_PURCELL_MEIBOOM_GILL => new_carr_purcell_meiboom_gill
It seems like you're making a breaking change to the API just to change some terminology. But the terminology is not necessarily wrong?
If you want to just eliminate the terms single axis and use some abbreviations. That's fine. But just do it completely consistently with the constants as well.
So make sure you also change this file:
https://github.com/qctrl/python-open-controls/blob/master/qctrlopencontrols/constants.py
Also these changes will break the Jupyter notebook examples, wherever they ended up landing.

This PR and following PRs to add the documentation would not break any existing APIs as they only interact with new_predefined_dds.
To add the documentation to dynamical decoupling functions, we need to make these private functions public and I think it's a good time to make the function signatures clear and consistent.
I am not sure we need to be consistent with the string constant as new_predefined_dds really is just a wrapper function. And, using a string flag is not quite ideal in this case, the long term goal (in my mind) would be removing all these strings and provide an Enum  for new_predefined_dds to select different control schemes. @charmasaur in case I missed anything here.

I thought about this more and my take is as follows.
If we want to consider the new_predefined_dds method as on a path towards deprecation (in favour of new_<foo>) then there's no harm in using different name here, since as Li says it's not actually a breaking change, and it makes the names more consistent with the ones in driven controls, so seems a worthwhile change.
If we want to keep the new_predefined_dds indefinitely then I think for me the consistency of the strings with the new function names beats consistency with driven controls, so I'd vote to keep the names the same.
So maybe we can work out what we want the future of new_predefined_dds to be. Personally I think it should be removed eventually, since I think separate functions is strictly better, but I could be convinced the other way.
Thoughts @michaelhush @tachikoma-li ?

@charmasaur if you want to depreciate new_predefined_dds, which is fine, then just do it immediately.
It doesn't have to be in this PR. But I don't think it's a good idea to make something inconsistent and leave it in an odd state.
I'd also remove the new_predefined_pulses (or whatever they are called as well).
@tachikoma-li this point is moot now, but "I am not sure we need to be consistent with the string constant as new_predefined_dds really is just a wrapper function." Doesn't fly with me. Whatever logic you use to express why you think the new name to the methods makes more sense, must also apply to the string constants. So they should all be consistent. I.e. you're arguing that single axis is not consistent with the method names. If that's true this is also the case with the string names and should be fixed too.
So I'd say there are only two paths forward:

Leave the names all the same and just make the methods public.
Change all the names and delete all the predefined method.

I don't mind which approach you take. For the record the predefined methods came from trying to match the needs of BLACK OPAL in some sense, where the names of the pulses where needed for displaying in the browser. With BLACK OPAL going, and also open controls not being used for that, this is not required anymore.
Making these breaking changes might create a problem with anything that depends on open controls. (i.e. the numerical package I think?) But as long as that's all version pinned that's fine.

I thought about this more and my take is as follows.
If we want to consider the new_predefined_dds method as on a path towards deprecation (in favour of new_<foo>) then there's no harm in using different name here, since as Li says it's not actually a breaking change, and it makes the names more consistent with the ones in driven controls, so seems a worthwhile change.
If we want to keep the new_predefined_dds indefinitely then I think for me the consistency of the strings with the new function names beats consistency with driven controls, so I'd vote to keep the names the same.
So maybe we can work out what we want the future of new_predefined_dds to be. Personally I think it should be removed eventually, since I think separate functions is strictly better, but I could be convinced the other way.
Thoughts @michaelhush @tachikoma-li ?

Thanks for the thought. To me, it is clear we should remove new_predefined_dds eventually for the following reasons: @charmasaur

it seems convenient to have a wrapper function. However, string selection is not supported by autocomplete, users still need to manually type the scheme, which is error-prone.
as the actual doc lives in the individual functions now, for users who want to use the API, it's in fact easier for them to just call the function while reading the doc. (otherwise, they would not enjoy shitf+Tab in Jupyter to see the doc without leaving it)
The annoying keyword parameters. To make the wrapper function less cumbersome, **kwargs parameter is used. This has two effects:

it's hard to doc what these kwargs are the current doc says  Additional keyword argument to create the sequence., which is not really helpful.
To use the new_predefined_dds,  users need to go through other functions signature to understand the current input parameters, which makes me feel that this wrapper function is redudent(and because of this issue, changing string selection to Enum would not fix this issue).

I'm fine with losing the predefined methods. They historically came from stuff to do with BLACK OPAL. They don't serve any important purpose in Open Controls anymore.
But I'd suggest you do that now, if you want to.

I'm fine with losing the predefined methods. They historically came from stuff to do with BLACK OPAL. They don't serve any important purpose in Open Controls anymore.
But I'd suggest you do that now, if you want to.

Thanks, I also think we should do it soon before too many notebooks are using new_predefined_dds and new_predefined_pulses. I am happy to check existing notebooks after making the changes.

@tachikoma-li @charmasaur also in situations like this, I think it's reasonable for when you make changes to the API you have to fix them in the application notes as well. I'm not sure if any of them use DDSs at the moment. But change them if they do.


feat: [CDS-86606]: add best practices for expressions usage with method invocation

Description: Adding best practices for expression usage with method invocation

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://657c5570cda85310a4cb63bc--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://657c8277fcc4910a4c0166c4--harness-developer.netlify.app


connectivity verifier in preflight_postgres_validator.rb misses one possibility

Hi,
I'm installing Chef against external PostgreSQL 9.4.5.  The connectivity_validation function in preflight_postgres_validator.rb encounters unknown message and fails:
Compiling Cookbooks...
Recipe: private-chef::default
  * directory[/etc/opscode] action create (up to date)
  * directory[/etc/opscode/logrotate.d] action create (up to date)
[2015-11-14T20:22:30+01:00] FATAL:

-----------------------------------------------------------------------
CSPG999: FATAL:  role "chef_server_conn_test" does not exist
-----------------------------------------------------------------------

The fix is simple and qualifies as obvious:
--- /opt/opscode/embedded/cookbooks/private-chef/libraries/preflight_postgres_validator.rb	2015-11-14 20:22:11.047410231 +0100
+++ /tmp/preflight_postgres_validator.rb	2015-11-14 20:30:55.443157412 +0100
@@ -95,6 +95,8 @@
         # This is also possible, depending on if they've set up pg_hba
         # by host or user or both. This is OK, since it confirms that we're
         # able to connect to the postgres instance.
+      when /.*role "chef_server_conn_test" does not exist.*/
+        # This also means we're able to connect
       else
         # This shouldn't be possible, but we still don't want to dump an
         # unhelpful stack trace on the screen. Let's at least catch it and

Thanks for reporting this and posting a fix, and sorry to hear that you ran into it.  This specific message should already be accounted for , since we don't expect that user to exist.
I will take a look Monday and see if something changed there , or if there is some difference in what gets reported out of pg 9.4 that we didn't account for .
The odd thing here is that I know there are other installations using 9.4 and they haven't run into this.
Thanks again for the detailed report, I'll post an update Monday.

It looks like this was fixed by https://github.com/chef/chef-server/pull/818
(patch is almost identical)


Handle Map in Chat workflow

You can't have a java bean with a property containing a Map , say:
Map someprop;
It kills the templater.

we can and do.  Have been testing this in the new poll bot.  Closing


Do not include .pdb files to nuget package, create separate .symbols.nupkg package instead

Consider to NuGet documentation
http://docs.nuget.org/docs/creating-packages/creating-and-publishing-a-symbol-package
Instead of including .pdb files to .nupkg directly, use -Source nuget option to generate special type of package - .symbols.nupkg package that keep pbd files and sources.
Publish main package to nuget.org and symbols package to symbolsource.org - this canbe arrived by one command.
None of popular nuget packages (newtonsoft.json for example) include pdb files.
NEST nuget package also should not contain pdb files, pdb and sources should be available at symbolsource.org

Closing this because we now ship with gitlink indexing our source.


Fix undefined name of component

Fix minor misspel, Component is React.Component now, should use name of WrapperComponent in displayName

Great catch. Thanks for the PR!


OUs - Create

Status:

Created static method CreateOrganizationUnit() in handlers.ldap.net\synapse.ldap.core\runtime\orgunit.cs and tested working.

1.1) Method throws exception while returning a new OrganizationalUnitObject. This seems to be a result of recursive lookup of SetPropertiesFromDirectoryEntry() in synapse.ldap.core\classes\directoryentry.cs till the top parent throwing "A referral was returned from the server” exception.
Need to check with Steve on this.

Created CreateOrgUnit() in handlers.Ldap.net\Syanpse.Services.LdapApi\OrgUnit.cs.
Need to check with Steve on how to add the dynamic parameters if codes are to be run at a Synapse node.


multicode response data in of array masks...

... need to be implemented by checking type before calling the dummyfing functions.

fixed in branch i373-fixing_wrong_matrix_slicers_for_group_method.


Need clarification about command.

The following command needs clarification:
New-CsTrustedApplicationPool -id  -Registrar  -site Site:
Here, pool fqdn - can this be any name, is it supposed to be the SfB f/e pool? or Registrar?

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

ID: 5e97f81b-dfc0-6b20-fb06-4addeface54b
Version Independent ID: 830ab805-b0a2-41a4-42a6-ee248ae1bacb
Content: Deploy the SEFAUtil tool in Skype for Business 2015
Content Source: Skype/SfbServer/deploy/deploy-enterprise-voice/deploy-the-sefautil-tool.md
Product: skype-for-business-itpro
GitHub Login: @kenwith
Microsoft Alias: kenwith

Thank you for this feedback and for contributing to the wellness of the content! This is a good idea and we have added it into our backlog for triage and prioritization.

@tseward Is investigating this.

Hi @nsshommo thank you for your feedback.
Clarifications about Pool FQDN had been added after you wrote this comment https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/commit/920a35b7fcf12ee97d6dd68f89e284ca45e74168#diff-df6a3e33f95194a3833afb87b6c29867
Thank You


[BUG] Line pattern does not move with click to move

The line milling pattern does not move when using click to move. Additionally, the generated spinboxes for the start and end x,y values do not allow negative numbers.

issue fixed in import/export milling stages branch


Update sig cli release note

/assign @pwittrock @fabianofranz

@kubernetes/sig-cli-maintainers PTAL. Thanks!

@calebamiles I added a commit removing some less meaningful items and other minor adjustments.
/lgtm

@fabianofranz Thanks for cleanup!


Setup Hello Work Proyect

Added the HTML - CSS file
configured the linters for HTML and CSS
HTML was styled from CSS
Added and configured the .gitignore file

@Qoosim   In my dashboard the option to send appears, so I understand that it was approved, but the revision did not appear in the repository nor did the approval, nor did I receive any approval email


TypeScript props interface is cast to never when theme prop is not given and BaseStyledInterface is used

This line (L66) was recently added, and it causes an edge case in a very specific (but common) use case:
https://github.com/styled-components/styled-components/blob/62ae36eeb6ddf43ec2f1d5b8e0546dd8df47b15d/typings/styled-components.d.ts#L65-L68
Here we set the theme prop to be optional when P extends { theme?: T; }. But by default T is any in the exposed BaseStyledInterface.
This causes WithOptionalTheme to exhibit the bug. It then casts P to never. That means that the resulting interface is just: { theme?: any }.
The bug disappears when T is not any, meaning when theme is set to something else.
Here is an image of @JonWallsten's reproduction: http://imgur.com/a/i1GrD
The interface of ICombinedSpinButton is:
export interface ICombinedSpinButton extends IAutoplayButton {
  autoplayActive?: boolean
  autoplayRunning?: boolean
  hidden?: boolean
}

(Ignore the extend, it's irrelevant)
Note that the theme property is not cast to a type.
cc @Igorbek

@Igorbek I just hunted this bug down with @JonWallsten. It's a really specific edge case with the styled() factory, but not at all unlikely to be encountered. It is caused by the change you last made in the typings file. Mind having a look? I know how to get rid of it, but not how to properly fix WithOptionalTheme :wink:

Sure, let me take a look.

Sorry, I couldn't understand what the problem was by given screenshot. Can you please give the full repro code and point what the type expectations you had that were not met.
I tried repro code from #1070 and it did work for me.
@IIIristraM I've used the simplified repro from #1070:
import * as React from "react";
import styled from "styled-components";

interface IconProps {
    className?: string;
    icon: string;
}

class Icon extends React.PureComponent<IconProps, {}> {
    public render() {
        const { className, icon, ...others } = this.props;

        return <div />;
    }
}

const InputIcon = styled(Icon) ``;
const IconErase = styled(InputIcon) ``;

<IconErase icon="save" />; // works
<IconErase {...{ icon: "save" }} />; // works

@IIIristraM you didn't provide IconProps interface, please correct me if it differs from in what I showed.

@JonWallsten can you please paste some code so we can come up with a minimal reproduction? :)

@philpl This should probably do it:
https://www.webpackbin.com/bins/-KqwRNbXrBQ3PDkUF9Hi
Webpackbin however won't show interface related errors. So you would probably have to test it locally. I don't have a simple boilerplate, so it's probably faster you you to test it. Is that okey with you?

interface Props extends React.SVGProps<SVGElement> {
  isOpen?: boolean;
}

const DumbBox =  ({ isOpen }: Props) => (
  <div width={isOpen ? "50px" : "25px"}>
     Hello World
  </div>
);

const StyledDumbBox = styled(DumbBox)`
  color: red;
`;

This will throw an error in StyledDumbBox
Reverting to 2.1.1 fixes the build error.

Ah, I think I've got what's going on... I'll fix it shortly

I'll release a minor version today containing this fix

No rush on this since we just pinned our app to the previous version, but when is 2.1.3 due?


Middleware does not allow for other middleware

The way the the middleware is written doesn't allow for other middleware to be used. If I want to add any other middleware, that then has to be added to the individual routes.
Is there a way with the current implementation to use additional middleware with authMiddleware?

Hey @lamebear, this is unfortunately something we consider to be a limitation of Next.js. Our hope is that this is something that either Next.js or a separate library solves, because at the moment, most middleware we see either doesn't support it or supports it in limited ways.


arkime do not delete old packet file

Describe the bug
test1:
I have one partition with 137GB for arkime packet store(for test auto delete packet).with default maxFileSizeG=12,with last 10 history packet file.I have all 137GB 100% used(test failed).
test 2:
change maxFileSizeG=4,stop and clean arkime(stop arkime service and delete all packet file and wipe elasticsearch), disk usage should keep in 4*10=40GB, But I see packet store 90GB now,and still growing(so I think also failed).
I have use debug and see log"device results not deleting any files since 10 or less".
But I alreay have 36 pcap file in packet store dir.
4.1G    aio2022-220807-00000493.pcap
4.1G    aio2022-220807-00000494.pcap
4.1G    aio2022-220807-00000495.pcap
4.1G    aio2022-220807-00000496.pcap
4.0G    aio2022-220807-00000497.pcap
3.0G    aio2022-220807-00000498.pcap
867M    aio2022-220807-00000499.pcap
1.1G    aio2022-220807-00000500.pcap
734M    aio2022-220807-00000501.pcap
576M    aio2022-220807-00000503.pcap
571M    aio2022-220807-00000504.pcap
384M    aio2022-220807-00000505.pcap
963M    aio2022-220807-00000506.pcap
1.5G    aio2022-220807-00000507.pcap
4.1G    aio2022-220807-00000509.pcap
4.1G    aio2022-220807-00000510.pcap
4.1G    aio2022-220807-00000511.pcap
4.1G    aio2022-220807-00000512.pcap
4.1G    aio2022-220807-00000513.pcap
4.1G    aio2022-220807-00000514.pcap
4.1G    aio2022-220807-00000515.pcap
4.1G    aio2022-220807-00000516.pcap
4.1G    aio2022-220807-00000517.pcap
4.1G    aio2022-220807-00000518.pcap
4.1G    aio2022-220807-00000519.pcap
4.1G    aio2022-220807-00000520.pcap
3.5G    aio2022-220807-00000521.pcap
3.0G    aio2022-220807-00000522.pcap
3.3G    aio2022-220807-00000523.pcap
1.8G    aio2022-220807-00000524.pcap
993M    aio2022-220807-00000525.pcap
22M     aio2022-220807-00000527.pcap
34M     aio2022-220807-00000528.pcap
68K     aio2022-220807-00000529.pcap
24M     aio2022-220807-00000530.pcap
604K    aio2022-220807-00000531.pcap
I do not know why arkime do not delete old files. Please help.
Elasticsearch version:
7.17.2
Arkime/Moloch version:
3.4.2
OS name and version:
ubuntu 18.0.4D
How was Arkime/Moloch built/installed: (rpm, deb, easybutton, ...)
use deb
Provide logs, stack traces and steps to reproduce:
EXPIRE - device results not deleting any files since 10 or less
EXPIRE check device node: aio2022 free: 39.7987060546875 freeSpaceG: 54.69731903076172
EXPIRE check device node: akm5 free: 39.7987060546875 freeSpaceG: 54.69731903076172
EXPIRE check device node: raw free: 39.7987060546875 freeSpaceG: 54.69731903076172
EXPIRE - device [ 'aio2022', 'akm5', 'raw' ] { '/pcap140': {} } 54.69731903076172
EXPIRE - device results hits: 0
EXPIRE - device results not deleting any files since 10 or less
From logs I can see three nod[aio2022,akm5,raw],But I only have one node called aio2022.
I have default arkime segment config and new segment called[raw],I have change all segment to maxFileSizeG=4
[raw]
userNameHeader=rawpcapexport
....
I have wipe elasticsearch and still see three node.Thank you for your help.

I have new elasticsearch installed and works now, so this may not a arkime issue.


Centos 7 Failure talking to yum, could not resolve host, no more mirrors to try

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 2.2.3
vboxmanage: 5.2.26r128414
ansible 2.5.0

Your OS

macOS 10.14.3

Full console output
Console output:
https://gist.github.com/jndevbc/765561d6a197d15133d5272b0b75cb12#file-02_vagrant_up
Error text:
https://gist.github.com/jndevbc/765561d6a197d15133d5272b0b75cb12#file-03_errors
default.config.yml:
https://gist.github.com/jndevbc/765561d6a197d15133d5272b0b75cb12#file-01
Summary

Downloaded drupalvm master branch
changed default.config.yml to use vagrant_box: geerlingguy/centos7
composer install
vagrant up
Gets to the TASK [geerlingguy.git : Ensure git is installed (RedHat).] task, the first part of the error: (see link to full error text above)

Failure talking to yum: failure: repodata/repomd.xml from epel: [Errno 256] No more mirrors to try.\nhttps://mirror.steadfastnet.com/epel/7/x86_64/repodata/repomd.xm

Additional Things Tried/Looked At

Opened another terminal and vagrant ssh into this same box while it is going through all those mirrors

ping <IP_ADDRESS> gets a reply

curl example-domain.com does not resolve

look at /etc/resolv.conf file

[vagrant@drupalvm-master-centos ~]$ cat /etc/resolv.conf
# Generated by NetworkManager
search test
nameserver <IP_ADDRESS>
options single-request-reopen

did try installing default drupalvm (ubuntu) and it worked fine

What am I missing, is there a network setting I need to configure? I've tried this multiple times and it won't resolve.
TIA

Continuing from above...
vagrant ssh into the box and php -v and git --version -- they aren't installed.
sudo yum install gitresults in:
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
Could not get metalink https://mirrors.fedoraproject.org/metalink?repo=epel-7&arch=x86_64 error was
14: curl#6 - "Could not resolve host: mirrors.fedoraproject.org; Unknown error"

So then, do this:
sudo vi /etc/resolv.conf

The file is managed, so it will get overwritten ... but for now, add your own nameserver(s).
Now, you can sudo yum install git and install everything else you need for your server.

continuing…

I ran into other issues, mysql wasn’t installed, etc.

So remove/clean it out:

removed all boxes vagrant box list and vagrant box remove <boxname>
looked at virtualbox GUI and removed anything left there (delete all files)
did a complete uninstall of vagrant and virtual box
for virtualbox, used the tool that comes in the *.dmg file, it runs a script and removes a lot of hidden files/folders
then I went and look through /Library and ~/Library and deleted a few other things

Fresh Install:

Fresh install of VirtualBox 6 (caveat… see below)
Fresh install of vagrant
Fresh install of centos7 vm from geerlingguy
Again: vagrant up works right up to the same spot where it “Ensures git is instsalled” and then the long error message of “Could not resolve host” for all the different mirrors that it tries

Then:

vagrant ssh into the box, sudo vi /etc/resolv.conf and added my nameservers
vagrant up —provision
And this time it went through the whole thing
And it installed mysql and everything else that I needed

Caveat: VirtualBox 6 seems to load my sites really really slowly. Much slower than when I had all my sites working and on 5.x previously.

Very strange behavior indeed! So at this point is it installing everything correctly?

If I need to do a vagrant reload --provision, then it will still have issues finding mirrors. I tried this a week ago, and had to go and edit the /etc/resolv.conf file again.

On a fresh install, what are the contents of your /etc/resolv.conf?
I don't think I've ever had to tweak mine inside VirtualBox (though I have had to mess with DNS all the time in Docker environments, go figure). In a box I just spun up I'm seeing:
$ cat /etc/resolv.conf
# Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
#     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
nameserver <IP_ADDRESS>

I believe that is a part of the bridge network VirtualBox creates for you. Is it possible you have some proxy, virus scanner, corporate network tool installed by IT, or otherwise something that could be intercepting traffic and/or causing DNS failures?

On the fresh install, I commented above, it got stuck at the same spot, and when I looked at /etc/resolv.conf (after it was stuck), the file looked like this:
[vagrant@drupalvm-master-centos ~]$ cat /etc/resolv.conf
# Generated by NetworkManager
search test
nameserver <IP_ADDRESS>
options single-request-reopen

Your default drupalvm which uses Ubuntu worked/loaded fine. But something about the Centos version…
I also see that your resolv.conf file looks different from mine.
Hmm, yes, very strange.

Changed DNS on my own machine from corporate's to another one. vagrant reload --provision now works without getting stuck.
If anyone gets "could not resolve host" and "no more mirrors to try", the first thing you can try is change your computer's DNS to something like OpenDNS. (Instructions)

Glad you could get it resolved! Thanks for posting back to here, and I'll go ahead and close out the issue.

I think the artifactory at apache.jfrog.io had a minor issue. I verified as well deb and rpm packages could be installed. Everything seems to be back up now.


Give an example for a directory with spaces.

Please add an example for pattern matching a directory with spaces. I've tried WebHelp/Online\ Help/** as an example, but it didn't seem to match anything. I'm not asking for specific help here, I'm just saying that an example like this should exist in the documentation since directories with spaces are a common use case.

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

ID: 8fd54e81-9a0c-c5e5-fb89-9534372805ff
Version Independent ID: f3d49618-32d7-0abc-1c33-8211eb7dd073
Content: File matching patterns reference - Azure Pipelines
Content Source: docs/pipelines/tasks/file-matching-patterns.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

@Eddie-Hartman Thank you for the feedback, assigning this to the author for review.
The document references the use of glob, which means spaces would be referenced using brackets, denoted as [ ].

I figured it out. In my case, I simply needed 'WebHelp\Online Help**' as spaces aren't considered special characters for these filters. The reason I used the example I gave (escaping the space character as '\ ') is that spaces are considered special characters that need escaped for some matching patterns. The documentation does not say that spaces are special characters or not. It just doesn't address them in any way.
Later I could make a pr adding a simple example, but I'm not sure if there are any "gotchas" that I should point out.

What about a directory with spaces and a wild card?
so given
File Directory-12345/file1.txt
File Directory-12345/file2.txt
Something like !File Directory-*/** should work but it doesn't


Are we allowed to use this as a resource publicly?

Hi, sorry for the odd question!
I forked this (although I believe I moved repositories thus why the original fork is gone) and made some changes and edits to it, never finished with it though. I was hosting in using pages and on my own domain. Recently, someone brought it up to me while looking through my repo's that this was never under a license or anything, thus not giving me the permissions to do use it at all.
I never intended it to be used as my own site (and it wasn't even complete at the time of writing) however I would just like clarification if modification is allowed or not. I don't mind if you say I can't use it (I was using it as a resource to try and get better at non-framework front-end) so if you would like for me to delete it although that's fine with me. I've private it and took it off the domain for now.
Thanks!

Hi,
Feel free to fork & modify the code to learn from or improve upon. However, just for clarification, please do not use it as your own portfolio.


completely overpowered recipes with some other mods

for example, the end ingot from Majrusz's Progressive Difficulty can be uncrafted into a block of its kind, which is 9x the item effectively. There are lots of examples for this, even though I don't have any solution in mind.

To stop this recipes to happens, there is two methods :

Tell the mod author to send IMC to my mod to disable recipes in the uncrafter (https://github.com/Ketheroth/Uncrafter#for-mod-developers).
Add the recipes to the blacklist in the config yourself.

Unfortunately, I can't check every mods to add their overpowered recipes in the default blacklist in the config. You can create a PR to add them though.


CFE-834: Vendor to the lastest oc

Description
Vendor to the latest oc to pick up mirroring for manifest listed release payloads.
Type of change
Update oc vendor to capture this change: https://github.com/openshift/oc/pull/1423
Passed local tests

[x] make test-unit
[x] make test-e2e

Test Configuration:

Hardware: x86_64
OS: Fedora 37

/assign @lmzuccarelli @aguidirh

@jeffdyoung - thanks for this, vendoring is always a daunting task.

/lgtm

/lgtm


Option to disable system notifications simulator

Current i use scripts before launching simulators in my tests to pre approve permissions for (Location / Contacts) as there is alot of problems with XCUI Framework to interact with them.
Although because blue pill creates net new simulators Is there anyway there is a option to some how disable these?

Please move your issue to https://github.com/linkedin/bluepill


feat: configure Hub Subnet Service Endpoints (e.g Microsoft.Storage) ?

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to a problem?

There is no mention in the documentation for the module on how to configure service endpoints (e.g Microsoft.Storage) in a given hub subnet.
This means that these service endpoints must be specified outside of the module breaking the consistency of management.

Describe the solution you'd like

Provide an example of how to configure a hub subnet service endpoint in the hub
Something along the lines of (variables.tf):

      hub_networks = [
        {
          enabled = true
          config = {
            address_space                = ["<IP_ADDRESS>/20"]
            location                     = "eastasia"
            link_to_ddos_protection_plan = false
            dns_servers                  = []
            bgp_community                = "12076:20202"
            subnets = [
              {
                name                      = "hkjc-ea-hub00-sbt-shared00"
                address_prefixes          = ["<IP_ADDRESS>/24"]
                network_security_group_id = "/subscriptions/....-hub-001-nsg000"
                route_table_id            = null
                service_endpoints         = ["Microsoft.Storage"]
              },
              {
                name                      = "hkjc-ea-hub00-sbt-shared01"
                address_prefixes          = ["<IP_ADDRESS>/24"]
                network_security_group_id = "/subscriptions/....hub-001-nsg001"
                route_table_id            = null
                service_endpoints         = ["Microsoft.Storage"]
              },
              {
                name                      = "AzureBastionSubnet"
                address_prefixes          = ["<IP_ADDRESS>/24"]
                network_security_group_id = null
                route_table_id            = null
              },
.
.
.

Additional context

Using terraform module version 4.0.2 with the following  configuration produces no effect or errors:

       subnets = [
              {
                name                      = "hkjc-ea-hub00-sbt-shared00"
                address_prefixes          = ["<IP_ADDRESS>/24"]
                network_security_group_id = "/subscriptions/....-hub-001-nsg000"
                route_table_id            = null
                service_endpoints         = ["Microsoft.Storage"]
              },
              {
                name                      = "hkjc-ea-hub00-sbt-shared01"
                address_prefixes          = ["<IP_ADDRESS>/24"]
                network_security_group_id = "/subscriptions/....hub-001-nsg001"
                route_table_id            = null
                service_endpoints         = ["Microsoft.Storage"]
              },

Thanks @matt-FFFFFF  - Just to give some more context as to the general use case:

This request comes up in deployments for enterprise customers.
There is almost always a need to service endpoints to be associated to a given subnet in the hub, most often storage and keyvault since 'spoke' storage accounts  in enterprises are mandated to retrieve customer key and access credentials from keyvaults in a hub or access storage in a spoke from a hub deploy tool.
Therefore if this feature is not present a post terraform script needs to run to add the service endpoint on existing subnets - or - the subnet must be managed in another terraform module.

Making service_endpoint part of the enterprise  module would provide an immensely convenient solution to this very hacky approach to managing azure infrastructure.

Noted @archmangler
As a workaround could you create the subnet outside the module? I realise this might need a terraform state move...

Hi @matt-FFFFFF
Thanks for the suggestion. We'll need to use an alternative module for subnets. It will complicate the configuration and management process for cloud ops engineers by multiples and increase the probability of errors and failures, but I suppose there's no easier option for now.

Apologies @archmangler this is possible today with the advanced block:
See this post: https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/issues/711#issuecomment-1546602146
And here in the code: https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/blob/76a1c6dd1df2947d10ded7516cdbe1e6d0ed5294/resources.connectivity.tf#L60

@archmangler Appreciate the linked example is for a slightly different configuration, you can add the service_endpoints attribute instead and it should work

Thanks @matt-FFFFFF  - this works:
.
.
.
              lolcorp-ea-hub00-sbt-shared01 = {
                service_endpoints = [
                  "Microsoft.Storage"
                ]
              },
.
.
.


Interface for counterfactual prediction

May be out of scope for this project (or just for now), but @SamuelBrand1 points us to epidemia's interface for counterfactuals, which I think is cool. This is somewhat straightforward to implement, as we can build off of the built-in numpyro predictive functionality, but we would need to think about the interface.

I like the epidemia interface for prediction/counterfactual: https://imperialcollegelondon.github.io/epidemia/reference/posterior_predict.epimodel.html .
The tl; dr explanation is they fit an object (which contains posterior draws for the params) and then passing newdata can create forecasts (by being longer) and/or counterfactuals (by having different covariates like an earlier lockdown). Its demoed here, https://imperialcollegelondon.github.io/epidemia/articles/europe-covid.html .

Originally posted by @SamuelBrand1 in https://github.com/CDCgov/multisignal-epi-inference/issues/169#issuecomment-2191161181

I think we basically get this for free via Predictive, no?


Add key property to Channel class

Addresses suggestion in issue #267. Next step could be to use the Channel.key property in some of the ChannelContractProxy methods.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix radius slider on mobile devices

From Tamas:
On mobiles set the slider to 100% and display the radius value beneath that, centrally aligned. In it current state it looks very odd: radius value field then line break and then comes the slider.

Makes sense to me. I didn't really look at this much on mobile, didn't get that far.

Addressed by #20


Question with reference to key of a document

Is there a way to ask questions that refer to the key of a document, e.g., "What are the key results presented in 'Example2023'?"?
If not, are you planning to implement it, and do you already have a plan on how to do it?

With our new metadata update, if you use PaperQA v5+, you should be able to ask questions using papers citations (things like the title or authors), and the contextual ranking step will automatically use this information. Let me know if you have success!


Fix Bug in transform.cu

This is to fix the below error
/opt/conda/lib/python3.8/site-packages/deepspeed/ops/csrc/transformer/inference/csrc/transform.hip:688:6: error: function template partial specialization is not allowed
void launch_transform4d_0213<T>(T* out,
     ^                      ~~~

cc: @jithunnair-amd

@rraminen - curious what you ran that you came across this?

I was running the Deepspeed-Inference workloads from https://github.com/huggingface/transformers-bloom-inference.git

This error also comes up when building DeepSpeed with transformer_inference_op extension in non-JIT way


Troubleshoot Azure DevOps guide: Issues arise when there's an existing resource group in the same subscription

Problem: I configured my resource group to be mlops-stack-ado but there already exists another resource group in the same subscription.
Fix: I manually changed all resource group names and storage account names to avoid the clash, but Terraform is not happy. I eventually fixed the clashes by running terraform init -reconfigure and terraform init -migrate-state in:

databricks-config/staging
databricks-config/prod
.mlops-setup-scripts/cicd

In hindsight, it probably would have been easier to just start the project from scratch, supplying different resource group and storage account names. Hope this helps someone out there!

Closing this issue since as of https://github.com/databricks/mlops-stack/releases/tag/v0.1 we replaced Terraform.


Copy to clipboard button overlaps code

Reported by from @jess-edwards: The "copy" button in code snippets cuts off longer lines. How can we improve this UX?
cc: @kuanluo

@jseldess @jess-edwards Can you point me to where this happens? I poked around, and saw longer lines usually have this behavior:

(From https://www.cockroachlabs.com/docs/stable/upgrade-cockroach-version.html)

Yeah, as you've found, it happens wherever we've used copy to clipboard and a line is long enough to be obscured by the "Copy" text and icon. In principle, I think it's ok for long lines to continue past the right margin. The "Copy" text is just pushing the right margin further in.
A possible solution would be moving the "copy" text and icon outside of the code block, just to the right. I can't remember why we didn't go with this in the first place...

Yeah, as you've found, it happens wherever we've used copy to clipboard and a line is long enough to be obscured by the "Copy" text and icon. In principle, I think it's ok for long lines to continue past the right margin. The "Copy" text is just pushing the right margin further in.

Agreed. The screenshot you attached should have the same behavior. That might be a bug?

A possible solution would be moving the "copy" text and icon outside of the code block, just to the right. I can't remember why we didn't go with this in the first place...

Might be an overall content width issue. We can reopen the convo with Division Of and see what we can do?

@jseldess I think its okay for the copy button to cover some of it (since the user will have to scroll anyways to see the whole line.
Show the scroll bar when the user is scrolling or on hover would help clarify there is more past the right margin.
Removing the icon from the copy button will also help

Link to designs in Figma: https://www.figma.com/file/6XcxaUdad2RkjpEqnVHKAG/Education-docs-redesign?node-id=2228%3A12946


subscriptionmanager returning nil-error

The err at these locations [1,2] is always nil.
https://github.com/Axway/agent-sdk/blob/f76357d3562f60b7af16d6d36615bd674d8c1fe2/pkg/apic/subscriptionmanager.go#L140
https://github.com/Axway/agent-sdk/blob/f76357d3562f60b7af16d6d36615bd674d8c1fe2/pkg/apic/subscriptionmanager.go#L144
Fix could be like this
	if apiserverInfo.Environment.Name != sm.apicClient.cfg.GetEnvironmentName() {
		log.Debugf("Subscription '%s' skipped because associated catalog item belongs to '%s' environment and the agent is configured for managing '%s' environment", subscription.GetName(), apiserverInfo.Environment.Name, sm.apicClient.cfg.GetEnvironmentName())
                // added creation of new error to fix issue
		return errors.New("Environment of subscription is not associated with agent's environment - skipping")
	}
	if apiserverInfo.ConsumerInstance.Name == "" {
		log.Debugf("Subscription '%s' skipped because associated catalog item is not created by agent", subscription.GetName())
                // added creation of new error to fix issue
		return errors.New("Subscription skipped because associated catalog item is not created by agent")
	}

Hi @mynecker,
We discussed with the team your proposal and we are willing to integrate.
Can you please raise a  pull request and we will include that.
Regards,


feat: scoreboard page on router

Feature Request
Describe the Feature Request
Add a scoreboard page on the router to be indexed by the application to create a new functional page for the scoreboard.
Describe Preferred Solution

[x] Create a section for scoreboard
[x] Add the necessary info in the _app.tsx to be indexed by the application

Closed by #370


Failed to load plugin after 2.5.0 update

Describe the bug
I am able to load the 2.4.1 version of the plugin but the 2.5 and 2.5.1 released in the last 12 hours are failing to load.
To Reproduce
Steps to reproduce the behavior:
Load the main.js, styles.css and manifest.json into the plugin folder

Are you getting any specific error in the console? Have you attempted installing or upgrading with BRAT?

Update, I am able to repro with your installation flow and BRAT. Problem is here:
Plugin failure: obsidian-discord-share TypeError: Cannot read properties of null (reading 'version')
    at dE.initializeSettings

I'll get a patch out today.

Here are the logs from the console when I try to activate it
app.js:1 Plugin failure: obsidian-discord-share TypeError: Cannot read properties of null (reading 'version')
    at dE.initializeSettings (plugin:obsidian-discord-share:146:5532)
    at async dE.onload (plugin:obsidian-discord-share:146:2800)

I have tried loading it through BRAT as well the normal method of installation. Same error.

Update, I am able to repro with your installation flow and BRAT. Problem is here:
Plugin failure: obsidian-discord-share TypeError: Cannot read properties of null (reading 'version')
    at dE.initializeSettings

I'll get a patch out today.

Great. Thanks for the work.


Fix #723: Autoscaling Groups ingestion fails when region is denied

This PR adds another error code to the util.aws_handle_regions decorator, to handle another scenario described in #723.
I also sorted the error codes alphabetically for clarity, but happy to have it reverted if not enough value added.

Force-pushed to rebase with upstream master


Investigate Saucelabs or BrowserStack

Would be better to have tests run against all supported browsers instead of just PhantomJS as is currently the case.

:tada: awesome!


Add picard tool. It's working but redundant save method in artifact_metrics_bait model

FOR CONTRIBUTOR:

[ ] - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
[ ] - License permits unrestricted use (educational + commercial)
[ ] - This PR adds a new tool or tool collection
[ ] - This PR updates an existing tool or tool collection
[ ] - This PR does something else (explain below)

FOR REVIEWER:

[ ] .shed.yml file ok

[ ] Toolshed user iuc has access to associated toolshed repo(s)

[ ] Indentation is correct (4 spaces)
[ ] Tool version/build ok
[ ] <command/>

[ ] Text parameters, input and output files 'single quoted'
[ ] Use of <![CDATA[ ... ]]> tags
[ ] Parameters of type text or having optional="true" attribute are checked with if str($param) before being used

[ ] Data parameters have a format attribute containing datatypes recognised by Galaxy
[ ] Tests

[ ] Parameters are reasonably covered
[ ] Test files are appropriate

[ ] Help

[ ] Valid restructuredText and uses <![CDATA[ ... ]]> tags

[ ] Complies with other best practice in Best Practices Doc

I will open a new pull-request with the new version.


Enable the External vault support for HashiCorp Vault

Purpose

This is to make the HashiCorp Vault Connector retrieve the root token via a password configuration file or by prompting the user requesting the vault root token.

Goals

Root Token of the vault can be retrieved either by prompting the user via the console or by accessing the text file containing the root token.

Approach

Retrieving Token
Root token can be given by creating a text file or via the console. when using the text file approach, users can keep the text file or discard it once it is done with reading. In order to achieve this, the file name will be used as the decision parameter.

Marketing

External Vault Support for WSO2 Carbon Configurations - 4 | HashiCorp Vault

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

JDK version - 1.8
Operating System - Ubuntu 18.04

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Related: #2


docs(tutorial): french translation (_datas, pt3, pt4)

Thank you for submitting this PR...
Please check if the PR fulfills these requirements

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

Please provide the issue number?
issue #37
Did you finish translating the whole issue?

[ ] no (pr_state: WIP)
[ ] yes (pr_state: PLEASE_REVIEW)

What kind of change does this PR introduce? (check one with "x")

[ ] Translation
[x] Other... Please describe:

Other information:

@cbalit peux-tu rebase et squash stp?


File not found in version 0.3.5

After upgrade to version 0.3.5 we can not run project.
Maybe framework is missing file ChatDelegates.h

Hi,
According to the screenshot you have undeclared identifier which is not an identifier of QMServices.

These strings are defined in ChatDelegates.h

I tried import "ChatDelegates.h" but it displayed file not found.

Hi!
Is this issue about QMServices? In QMServices we don't have QMSoundManager
Maybe you are talking about Q-municate?

I copy QMBaseCallsController from Q-municate to my application.
Before I update, it can run. After update, it can not. I've tried using version 0.3.4 and it's still error.

Add this to your constants file:
static NSString *const kStopVideoChatCallStatus_OpponentDidNotAnswer = @"kStopVideoChatCallStatus_OpponentDidNotAnswer";
static NSString *const kStopVideoChatCallStatus_Manually = @"kStopVideoChatCallStatus_Manually";
static NSString *const kStopVideoChatCallStatus_Cancel = @"kStopVideoChatCallStatus_Cancel";
static NSString *const kStopVideoChatCallStatus_BadConnection = @"kStopVideoChatCallStatus_BadConnection";

These identifiers were removed from the latest Quickblox iOS SDK.

Thank @Raikerian


Fix/optimize access to builtins dictionary

The first commit is the important one (including a fix for limited API), while the second are related optimizations.

PR updated with a single commit and only touching what is needed. My attempts to uniformize coding style usually lead to objections; therefore, from now on, I'll refrain from such changes.

My attempts to uniformize coding style usually lead to objections; therefore, from now on, I'll refrain from such changes.

It's an old code base (>20 years by now), so it's usually a good thing to update code that we touch anyway to newer styles and infrastructure (such as f-strings). I consider it less helpful to do larger style updates (because they make reading the blame history harder). And occasionally, like in this case, there's a reason why things were written the way they are. If so, I should learn to leave a comment in the code more often than not. You were really just unlucky here.

@scoder This one should also be ready for merge. Note that now I'm using  PyObject_GetItem() following your advice related to avoiding borrowed refs in limited API code.

Thanks


How to get the right value from an input on load

If I want to get the value from an input and use the 'getNumber' method, it always returns "".
$("#rh1TelefoonNummer").intlTelInput("getNumber")

This only works if the page is loaded and I execute this from the Console, not when executing this from the Document Ready.
What am I forgetting?

getNumber requires loading the utilsScript, so if that hasn't finished, then it wont work. You have to wait for the done callback - see readme.


SOLR-15368: replace bad favicon logo to transparent background

https://issues.apache.org/jira/browse/SOLR-15368?jql=project %3D SOLR AND component %3D website
Description
On any modern webpage, the favicon has a transparent background, (for example - look at Github's logo as opposed to Solr's logo).
Solution
There is actually a good logo on Solr's Jira site, which I've uploaded on this very issue PR
Example (Right is the PR, left is the current icon)

Tests
The patch only replaces the favicon and left all else as is. I've run the website.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[x] I have added documentation for the Reference Guide

Agree. What's the difference between web/favicon.ico and web/img/favicon.ico? Can one be deleted?
There is also a favicon in solr/solr-ref-guide/src/images/icons/favicon.ico, can you update that as well? And I guess there's one in the solr-site git repo as well, for the website?

@janhoy whats the status of this?

See the linked Jira. I plan to commit in a day or two, barring any other comments.


Fix incorrect indicator location issue

Fixed Issue ➡ #8
This issue seems to be happen when navigation bar is initialized at first time. But this bug is occurred depends on onPageScrolled's parameters position and positionOffset. So i think we need to enforce location of the indicator at first time. WDYT? Please let me know if you have any concern about my approach @skydoves

Hi, Thank you for submitting this PR!
I have examined this issue, this issue originated from the duplicated events between onPageScrolled and onPageScrolled.
Actually, the first planning was that the indicator should move to align the scroll position. (That looks more natural when users scroll screens) But there is an issue that the indicator moves with some animations make the duplicated events (between onPageScrolled and onPageScrolled; when we select an item, the onPageScrolled and onPageScrolled are called at the same time). I think the best solution is to handle the scrolling state in the events, but I need to consider this issue more.

We can represent the above issue as below:
override fun onPageScrolled(position: Int, positionOffset: Float, positionOffsetPixels: Int) {
-          if ((selectedIndex > position && previousPosition < positionOffset)) {
-            post {
              indicator.x = itemWidth * position + itemWidth * positionOffset + indicatorPadding
              previousPosition = positionOffset
-            }
-          }
        }

The indicator will align the position of the ViewPager, but If we select a bottom navigation item, the final animation (overshoot) would be weird.

Thank you for explain details @skydoves. When i reproduced this issue, it was happened only at navigation bar is initialised. and at that time, i could see onPageSelected was not called. please let me know if i have missing points in this issue. i'll investigate this issue more 😄
감사합니다!


fixed the double mouse highlighting

whenever a controller is connected, the ui event system focuses
on the first selected obj for the event system
whenver no controller is present, the ui event system de-selects
the highlighted object so that only the mouse can highlight things
for #431

Fixed. please take another look!


vm clone fails with connection reset by peer

Cloning template template-ubuntu-12.04-25gig-tnt17 to new VM vishal-vm6.tnt17-zone1.aus1
ERROR: Errno::ECONNRESET: Connection reset by peer

Hi, this is a fairly old request and I'm sorry no one has come to help you yet. Is the problem still happening?
Thanks.

Hi, I'm going to close this to try and clean up our backlog of inactive issues. If you are still having this problem, please make sure you're running the latest version of the gem (gem install knife-vsphere) first, and then open a new issue so that it goes to the top of the list. Thanks.


Update README.md

New emoji for your readme

Awesome. Thanks, @vitoralvesp!

Awesome. Thanks, @vitoralvesp!


:seedling: Unit test for Contributors

Code coverage of 95% for contributors.

What kind of change does this PR introduce?
(Is it a bug fix, feature, docs update, something else?)

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
What is the new behavior (if this is a feature change)?**

[x] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

Codecov Report

Merging #2881 (505e541) into main (1c441f3) will decrease coverage by 4.03%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2881      +/-   ##
==========================================
- Coverage   55.18%   51.16%   -4.03%
==========================================
  Files         158      158
  Lines       13276    11992    -1284
==========================================
- Hits         7327     6136    -1191
+ Misses       5514     5488      -26
+ Partials      435      368      -67

I was pushing another PR fix to increase the coverage, so it is dropping now. Just an FYI...


Implement Embed code generation

Try to investigate embed code generation, this brings better functionality.

Investigated some, and it seems bit difficult
https://github.com/LeMisterV/Next-widget/blob/ec7ca7882e241b59a2544bd25d9907361fce4687/src/config/widgetCompiler.js
https://stackoverflow.com/questions/63248414/how-do-i-create-an-embeddable-next-js-react-component-for-third-party-websites

This material says using an iframe is easier.
We can utilize iframes as much as possible.
https://selleo.com/blog/how-to-create-embedded-react-widget

This is also using iframe, so let's just add iframe now.
https://github.com/thirdweb-dev/widgets


Reorderable Grid

Is there something like this in the pipe since the Reorderable List just landed ?
Thank you.

Sounds like there has been no plans in the last 6 months. I hope it will be added sometime soon!


The window background is not shown correctly in the designer

Describe the bug
If one designs a MetroWindow in the Visual Studios designer it is not shown correctly. It is shown correctly at runtime. This issue is more visible if one uses the dark theme
Steps to reproduce

Create a blank App with MahApps.Metro imported
Set the theme to dark
Create a new MetroWindow
You will notice that the Background is white

Expected behavior
MetroWindow should render the Background correctly. It does if one sets the Background explicitly in XAML
Sample
see https://github.com/Volmancer/OWGENC
Environment
MahApps.Metro version: v2.0.0 v2.1.0 tested
Windows build number: Win10 and Win7 tested
Visual Studio: 2019 16.6.3
Target Framework: .Net Core 3.1 (v4.5.2, v4.7.2)

Screenshots

Designer

Runtime

This seems to be fixed.
Tested with
Microsoft Visual Studio Enterprise 2019
Version 16.6.3

and
Microsoft Visual Studio Enterprise 2019 Preview
Version 16.7.0 Preview 3.1

@punker76 yes, with this line:

If you are not using this line, it does not work. But I know it seems to be a VS issue.

I'll double check this issue in the upcomming v16.7 release of VS and see if it is fixed there.

Oh yeah, I didn't noticed that the Background was set explicitly. Doesn't work in preview. But it's not a MahApps issue, it should be delegate to VS tema.

Created an issue at VS: https://developercommunity.visualstudio.com/content/problem/1106526/wpf-designer-the-window-background-is-not-shown-co.html
The more people vote for it, the better the chance that MS looks for it.

(Closed - Not a Bug) Here is the answer from Microsoft: https://developercommunity2.visualstudio.com/t/wpf-designer-the-window-background-is-not-shown-co/1106526#T-N1239787
This is a limitiation of the designer. The designer does not instantiate the real Window MainWindow. It can’t because the designer process has his own window in order to display the content. This is known XAML designer limitation, Repro class hierarchy is:
MainWindow > MetroWindow (from MashApp SDK) > Window (from WPF).
Then in XAML:
<mash:MetroWindow x:Class=‘MyApp.MainWindow’/>
Designer cannot create any Window-derived class. So we end up massaging XAML to be:
<_0:StandInWindow />
As a result we lose MetroWindow default style (see https://github.com/MahApps/MahApps.Metro/blob/develop/src/MahApps.Metro/Themes/MetroWindow.xaml).
If you’re using the XAML Designer for .NetCore, you’ll be able to see d:Background on the Window and setting a design-time background color.


Replaced deprecated UUID library references

Description
I noticed a few references to github.com/gobuffalo/uuid that crept back into the code.  That library is deprecated and we should use github.com/gofrs/uuid instead.
Setup
make server_test
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 2e15ae4bbed1f746c1cdb3c30446eb7f7311ed9a

I wonder if we should add a danger js check to flag this automatically going forward since it somehow gets added back in by accident

That would be a great idea as this has happened a few times over the last year or so.  I haven't set up checks with danger yet, but I made a story for it.


[Beta] Applied Accessibility - Test fails to check for closing  tag

Challenge Name

https://beta.freecodecamp.org/en/challenges/applied-accessibility/make-screen-reader-navigation-easier-with-the-footer-landmark
Issue Description

The test passes if the opening  tag is used and the closing  is missing.
Also, the test states the no
 tag may be used in the code but it doesn't check if the closing
 tag is still left in.
Browser Information

Browser Name, Version: Chrome, Version 63.0.3239.108 (Official Build) (64-bit)
Operating System: Windows 10 Pro
Mobile, Desktop, or Tablet: Desktop

Your Code

Training

Stealth & Agility

Combat

Weapons

Stealth & Agility Training

Climb foliage quickly using a minimum spanning tree approach

No training is NP-complete without parkour

Combat Training

Dispatch multiple enemies with multithreaded tactics

Goodbye world: 5 proven ways to knock out an opponent

Weapons Training

Swords: the best tool to literally divide and conquer

Breadth-first or depth-first in multi-weapon training?

  © 2016 Camper Cat

Screenshot

© 2016 Camper Cat


[D3D9] SWTOR: Transparent textures/decals render improperly with high AA settings

Star Wars: The Old Republic has transferred from 32-bit to 64-bit with patch 7.2, and it has apparently introduced some "pixelating" effects when moving the camera around, while looking at textures with transparency (like railings, holographic posters or lights), using "High" or "Very High" anti aliasing settings. Changing AA to Medium will turn everything normal again.
High preset - Anti Aliasing set to Very High

High preset - Anti Aliasing set to Medium

Here's a video demonstrating the issue, with movement:
https://user-images.githubusercontent.com/43139362/232287814-628c0104-67be-4e97-9514-08d8b36569ff.mp4
I've tested the game with DXVK 2.1, the latest master (https://github.com/doitsujin/dxvk/commit/0cf563d5dfcaf2f8b8c659439698f10388eddc2b), 2.0 and 1.10.3, and the issue is persistent. (1.10.3 just removes these textures alltogether). When running the game in native D3D9, there are no issues.
Please let me know if you need anything. Thanks.
Software information
Star Wars: The Old Republic - Steam Version
Using "High" graphics preset.
System information

GPU: Nvidia GTX 1070 Ti
Driver: 531.41
OS: Windows 10 Pro 22H2 (build 19045.2728)
DXVK version: 2.1

Apitrace file(s)

swtor.zip

Log files
swtor_d3d9.log (latest master)

Duplicate of #3135.
Basically, it's a Nvidia driver bug that's already fixed in the developer beta driver. Shouldn't be long until the fix makes it's way into the stable branch.

My apologies, I should've looked into that thread before opening my issue.
I've just downloaded the latest Nvidia Vulkan beta driver, and it works now. Thanks again.

No problem.


Fix crash when uploading data with skipped photo tasks

Fixes #2121
Fixes #2755

Things done:

Mark the value for "SkippedTaskData" as empty for correctness
Cleanup MediaUploadWorker to check for PhotoTaskData exactly instead of using task.toString() which led to the problem.

Verified locally by skipping a photo task and ensuring that the data was synced remotely.
@gino-m @anandwana001  PTAL?

@gino-m gentle ping!

@gino-m Gentle ping!


[Suggestion] - Autogenerated configs for mods

Rn this mod applies only one type of damage and resistance for every modded mob and weapon/tool. I suggest:

Apply more or less random values for every item, that extends default forge weapon/tool/armor class. Maybe make it based on damage/toughness/etc.
Apply same random values for every modded mob. Maybe make it based on health/resistance/bossbar/etc

This seems plausible, I'd just need to find a way to iterate through such items/mobs. I had never thought about randomly generating config values. I would've assumed people would've preferred to tweak and enter each value manually. Same for the mobs.

Will the defaults for modded mobs be pre-generated values that apply for all worlds and all playthroughs, or will you go the terrafirmacraft route of making the same modded mob type have different resistances depending on the world seed where the mob was spawned?

Well, assuming I can even do the first one, which would be easier, there's no reason I can't have both options available

Hi. Any info about this?

I was fiddling with CraftTweaker integration and decided to do that for 1.1.0 instead of the other features listed here. So this got pushed to 1.2.0. I'll be making a milestone where you can track development.
Or I could setup a Discord server for those wanting to track development more closely... not a bad idea actually.

An interesting development... When mob capabilities are added, mob attributes (health, armor, etc.) haven't been set yet. I'm not aware of a work around currently, so I'll have to generate without health and armor information.


fix: typos in bug_report.md

hacktoberfest?

hacktoberfest?

I think fixing typos doesn't count tho

They can't check for that so it would count. However, if you find typos, open an issue. Pull request are for fixes/additions to the actual code

They can't check for that so it would count.

Ah well, then why not :)

However, if you find typos, open an issue. Pull requests are for fixes/additions to the actual code

Will do it when I find more typos!

I saw the typos when trying to report a problem because I had a problem publishing a package. I later discovered that the package name "crypto" was causing the problem. Is that a reserved name? (I am using the same name on deno.land)

It is not a reserved name so it should work 🤔

It is not a reserved name so it should work 🤔

I've tried the names "crypt" and "crypto2" and both worked.
When I try the name crypto, I get the following error message:
[CRITICAL] An unexpected error occurred: "Something broke when publishing... " "Error: Something broke when publishing... \n    at Command.publishCommand [as fn] (publish.ts:351:11)\n    at async Command.execute (command.ts:796:9)\n    at async Command.parse (command.ts:731:14)\n    at async Command.parse (command.ts:688:14)\n    at async mod.ts:51:23

So, the behavior is not intended? Should I open a new issue for that?

@aykxt crypto is a reserved name - https://github.com/nestdotland/x/blob/c94add3607c9fb7b45ac916d5f0776eb0fc0aeb1/src/utils/reservedNames.ts#L25

@aykxt Thank you for these corrections 👍
Recently we wrote a specification on how to contribute but it is not yet on the master branch.
https://github.com/nestdotland/eggs/blob/dev/.github/CONTRIBUTING.md
You have to make a request to the dev branch 🙂
Please merge the latest changes from dev into this branch or create another branch from dev.


Adds an introduction to the Pipelines SDK

Added an overview of the Pipelines SDK and the various ways you can use it to build and interact with pipelines.
Replaced the DSL overview to give the full picture of authoring pipelines with SDK and DSL combined.
Moved the content about pipeline parameters into a new file: /docs/pipelines/sdk/parameters.md
Fixes https://github.com/kubeflow/website/issues/809
Fixes https://github.com/kubeflow/website/issues/810

This change is

Preview: https://deploy-preview-811--competent-brattain-de2d6d.netlify.com/docs/pipelines/sdk/sdk-overview/

Removing reviewers as this PR will be noisy until it's ready for review.
/uncc @jlewi
/uncc @kunmingg

This PR is now ready for review. I'm adding a hold to allow for multiple reviewers.
/hold

/assign @gaoning777
/cc @zanedurante
Preview:

New guide: https://deploy-preview-811--competent-brattain-de2d6d.netlify.com/docs/pipelines/sdk/sdk-overview/
Content extracted from the now-deleted DSL overview: https://deploy-preview-811--competent-brattain-de2d6d.netlify.com/docs/pipelines/sdk/parameters/

/assign @hongye-sun
/unassign @gaoning777

/lgtm

Just read the intro to the SDK -- looks great!

/retest

/hold cancel

Just read the intro to the SDK -- looks great!

Thanks @zanedurante!

/lgtm

/approve


aws_networkfirewall_firewall_policy and aws_networkfirewall_rule_group import not possible

What happened?
for the following resources no import is possible - this is a blocker for migration to official provider-aws
aws_networkfirewall_firewall_policy
aws_networkfirewall_rule_group
the import leads to the following error:
   Message:               apply failed: creating NetworkFirewall Rule Group (rulegroup): InvalidRequestException: A resource with the specified name already exists:
   Message:               apply failed: creating NetworkFirewall Firewall Policy (firewallpolicy): InvalidRequestException: A resource with the specified name already exists:
we have also the following issue:
https://github.com/upbound/provider-aws/blob/main/config/externalname.go#L2239 - this is wrong because we have a difference between stateful-rulegroup and stateless-rule group in the ARN - think its combined to the general import problem

How can we reproduce it?
kubectl create -f firewall.yaml
check that all resources created:
kubectl get managed
NAME                                                 READY   SYNCED   EXTERNAL-NAME   AGE
rulegroup.networkfirewall.aws.upbound.io/rulegroup   True    True     rulegroup       9m16s

NAME                                                           READY   SYNCED   EXTERNAL-NAME    AGE
firewallpolicy.networkfirewall.aws.upbound.io/firewallpolicy   True    True     firewallpolicy   9m16s

NAME                                                   READY   SYNCED   EXTERNAL-NAME                                                           AGE
firewall.networkfirewall.aws.upbound.io/firewallname   True    True     arn:aws:network-firewall:us-west-1:123456789101:firewall/firewallname   9m16s

NAME                                       READY   SYNCED   EXTERNAL-NAME              AGE
subnet.ec2.aws.upbound.io/sample-subnet1   True    True     subnet-07c67cb02a111a809   9m16s

NAME                                READY   SYNCED   EXTERNAL-NAME           AGE
vpc.ec2.aws.upbound.io/sample-vpc   True    True     vpc-0707128cc333d611e   9m16s

export the following resources:
kubectl get rulegroup.networkfirewall.aws.upbound.io/rulegroup -o yaml > rulegroup-orphan.yaml
kubectl get firewallpolicy.networkfirewall.aws.upbound.io/firewallpolicy -o yaml > firewallpolicy-orphan.yaml

set both resources to deletionPolicy: Orphan
kubectl apply -f rulegroup-orphan.yaml
kubectl apply -f firewallpolicy-orphan.yaml

remove both resources
kubectl delete -f rulegroup-orphan.yaml
kubectl delete -f firewallpolicy-orphan.yaml

create both resources:
kubectl apply -f rulegroup-orphan.yaml
kubectl apply -f firewallpolicy-orphan.yaml

check the status
kubectl get managed
NAME                                                           READY   SYNCED   EXTERNAL-NAME    AGE
firewallpolicy.networkfirewall.aws.upbound.io/firewallpolicy   False   True     firewallpolicy   21m

NAME                                                 READY   SYNCED   EXTERNAL-NAME   AGE
rulegroup.networkfirewall.aws.upbound.io/rulegroup   False   True     rulegroup       21m

    Message:               apply failed: creating NetworkFirewall Rule Group (rulegroup): InvalidRequestException: A resource with the specified name already exists:

    Message:               apply failed: creating NetworkFirewall Firewall Policy (firewallpolicy): InvalidRequestException: A resource with the specified name already exists:

===

What environment did it happen in?

Crossplane Version: 1.11.3
Provider Version: v0.31.0
Kubernetes Version:
Kubernetes Distribution:  EKS 1.23

i will open a PR - we fixed this already in our fork - so i will backport this
https://github.com/dkb-bank/official-provider-aws/pull/21/files


@spartan-ng/cli :ui Generates 2 src

Please provide the environment you discovered this bug in.
OS: ArcoLinux x86_64
Angular 19.0.0
NX Workspace 20.2.1
Which area/package is the issue in?
Don't know / other
Description
When I install a component npx nx g @spartan-ng/cli:ui alert it generates 2 src folders, I have to manually copy src.  This is a new project.

Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

This should be fixed by #516. It's already merged so I will test it and release a new CLI version!

just released a fix for this in 0.0.1-alpha.373


php artisan issue and workaround - problem in services.json

L5 Issue:
running php artisan optimize or subset of commands is creating a problem with services.json
So I can kind of replicate the problem. It is definitely a compile problem, but cannot pickup exactly where. It boils down to the following

php artisan is creating a problem services.json file
whereas the app services,json fixes the problem. If you get the app code to touch or write the service.json file your app will work. Example, below

if I do the following:
run php artisan optimize
it creates the following problem:
regens services.json but loads certain Service Providers in problem sequence. The ServiceProvider are failing silently.
workaround solution:

get the app code vs. php artisan code to touch services.json

Example:

create ServiceProvider
create a boot() method that loads a view composer, example, below.
if I then load or the view the app will create the proper services.json and the app will work fine.
3.1 you might need to do a composer dumpautoload
3.2 do not run run php artisan optimize or clear-compile as it will again corrupt the services.json file

If I compare a service.json file php artisan vs. app I can see that the php artisan command line is building the file incorrectly. Too much too put into this message but it is not eager loading certain ServiceProviders, but moving them to deferred which is a problem. So when you touch the file through app code it builds it the right way. In my case it loading the decorator into defer and removing the translator class so the decorator becomes the root class.
In my case it is taking my ServiceProviders and placing them in deferred and overwriting some root ServiceProvider like translator. Once the app touches the file, it moves the ServiceProvider back to eager load and then insert the root translator class again. Also, with the above change it is adding in a when condition that means certain ServiceProviders will fail silently. Again the app touch fixes the seqeunce
Example of my basic view composer boot. Don't try register() as it will fail.
 // I can then remove all of this after the touch
  public function boot()
    {
        View::composer('auth.register', TranslatorComposer::class);
    }

Could it be because you're running the app code (web) as different user/group (nginx/apache) as you would while running php artisan?

crynobone, unfortunately not, as I'm running exactly the same. In fact I can replicate it on MAMP, on the same host with full permissions. What leads me to believe this is a potential bug is that if I let the application i.e. I setup a basic ServiceProvider with a simple boot() ... example below ... to bind a view composer and then reload the browser all of my code will work fine. The reason I'm creating the ServiceProvider is to get the app to update the services.json from its own boot sequence vs artisan command line. is  I then comment out that one line and everything is back to normal. I can see when why when I do a file comparison. In my case I have a translator decorator that I can trace to see where it is writing pre and post a artisan optimize. In my case optimize is replacing Laravel's core translator with my translator decorator in the delayed loading section and then inserting a related when line which causes my decorator to fail silently.
Versus when the app writes out the services.json file i.e. vs  artisan, it places the translator core back in delayed, moves my decorator to eager and removes the when clause so the app runs fine again.
use Illuminate\Translation\TranslationServiceProvider as ServiceProvider;

class TranslatorServiceProvider extends ServiceProvider {

//    this is the code that I insert to get it to touch the services.json file and then
//    comment out the register
//    public function boot()
//    {
//        View::composer('auth.register.start', TranslatorComposer::class);
//    }

    /**
     * Registers the decorated translator
     *
     * @return void
     */
    public function register()
    {
        $this->registerLang();
        $this->registerComposer();
    }

    /**
     * Register views to composer
     */
    private function registerComposer()
    {
        $this->app->make('view')->composer('auth.register.start', TranslatorComposer::class );
        $this->app->make('view')->composer('auth.login', TranslatorComposer::class );
    }

    /**
     * Register the service provider.
     *
     * @return void
     */
    private function registerLang()
    {

        $this->registerLoader();

        $this->app->singleton('translator', function($app)
        {
            $loader = $app['translation.loader'];

            $locale = $app['config']['app.locale'];

            $trans = new Translator($loader, $locale);

            $trans->setFallback($app['config']['app.fallback_locale']);

            // TranslatorDecorator implements the translator interface and the usual i.e. implements the contract methods and has some translation and locale methods
            return new TranslatorDecorator($trans, $app['config'], $this->app );

        });

    }
   // removed the rest for brevity

So you're saying you have both Illuminate\Translation\TranslationServiceProvider and App\Providers\TranslationServiceProvider in app.providers when App\Providers\TranslationServiceProvider already extending Illuminate\Translation\TranslationServiceProvider?

If I run artisan optimize this is what it writes to services.json. This fails silently so that my translator decorator and view composer are not bound to the IoC
"deferred": {
...
        // note this is my own SP that extends the Laravel Translator SP
	"translator": "App\\Administrate\\Providers\\TranslatorServiceProvider",
	"translation.loader": "App\\Administrate\\Providers\\TranslatorServiceProvider",
...
}

"when": {
	...
        //
	"App\\Administrate\\Providers\\TranslatorServiceProvider": [],
	...
}

However, if I let the provider touch the services.json this is how it writes it.  This works and the app runs fine. As you cane see it is now eager loading my SP and deferred the Laravel SP that I'm extending.
"eager"{
	...
	"App\\Administrate\\Providers\\TranslatorServiceProvider",
	...
}

"deferred": {
...
    // notice these are the Laravel Translators.
    "translator": "Illuminate\\Translation\\TranslationServiceProvider",
    "translation.loader": "Illuminate\\Translation\\TranslationServiceProvider",
...
}

I am extending the Laravel TranslationServiceProvider in my own TranslatorServiceProvider. That might be an issue.

@crynobone  fwiw, another person have the same issue https://laracasts.com/discuss/channels/general-discussion/l5-with-psr-4-package?page=1#reply-31474

I am extending the Laravel TranslationServiceProvider in my own TranslatorServiceProvider. That might be an issue.

Yes, which you should remove Illuminate\Translation\TranslationServiceProvider since your replacement service provider already include the same feature.

This fails silently so that my translator decorator and view composer are not bound to the IoC

Fail? "translator": "App\\Administrate\\Providers\\TranslatorServiceProvider", clearly shown that it is registered as deferred (which is the expected behaviour).

@crynobone for some reason 5.0.16 fixed this issue for me. Many thanks for you help.


faster clever_t computation

Description
The current implementation of clever_t in ART computes gradients for all the classes. This is in-efficient and can be made significantly faster by only computing the gradients for the predicted and the targeted class.
Fixes # (issue)
Type of change
Please check all the relevant options.

[x] Bug fix (non-breaking)
[  ] New feature (non-breaking)
[  ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[  ] This change requires a documentation update

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

reproduced the output from clever_t by fixing the seed for NumPy and torch Cuda on CIFAR10 (~4x faster)

Test Configuration:

OS CentOS
Python version 3.7.7
ART version or commit number
TensorFlow / Keras / PyTorch 1.5.0 / MXNet

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] 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

Hi @joemathai Thank you very much for your pull request! We will review it as soon as possible.


fix: CreateRelationship did not honour the relationship identifier

Fixes a regression in PR from last year where CreateRelationship did not honour the relationship identifier
Tests added/updated

bump :)

:+1:


Recreate window after user unlocking screen

If user is locking screen it would be nice to restore state after unlock

Also, need to save crop and zoom settings

Done


Allow MDL components to be updated remotely.

Provide a reference to the MDL component, and call required methods as needed to update their state. This happens when a component is updated programmatically rather then physically via UI.

Currently, only switch and radio buttons need to provide this hook. Switches have an on() and off() method needed to trigger the update. Radio buttons use check() method.  All other current controls can be updated by simply changing their selectedValue.


Docker creates ingress network folder in /var/lib/docker even when using -g

Description
We are specifying a different docker root with -g, i.e. -g /data/docker/. It works, however docker is still recreating the /var/lib/docker folder and creates: /var/lib/docker/network/files/ingress_sbox.
Inside that folder are three files:
root@xxx:/var/lib/docker/network/files/ingress_sbox# ls -l
total 12
-rw-r--r-- 1 root root 150 Mar  7 13:18 hosts
-rw-r--r-- 1 root root  58 Mar  7 13:18 resolv.conf
-rw-r--r-- 1 root root  71 Mar  7 13:18 resolv.conf.hash

Steps to reproduce the issue:

Create a swarm
Create a service

Describe the results you received:
Docker is creating the /var/lib/docker folder.
Describe the results you expected:
Docker shouldn't create the /var/lib/docker folder.
Output of docker version:
Client:
 Version:      17.03.0-ce
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   3a232c8
 Built:        Tue Feb 28 08:01:32 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.03.0-ce
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   3a232c8
 Built:        Tue Feb 28 08:01:32 2017
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 20
 Running: 6
 Paused: 0
 Stopped: 14
Images: 20
Server Version: 17.03.0-ce
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: active
 NodeID: 9xet8ik8dju4y8786a2b3aemd
 Is Manager: true
 ClusterID: c935qq94yavddxpuc6292zbxu
 Managers: 3
 Nodes: 5
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
 Manager Addresses:
  <IP_ADDRESS>:2377
  <IP_ADDRESS>:2377
  <IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 977c511eda0925a723debdc94d09459af49d082a
runc version: a01dafd48bc1c7cc12bdb01206f9fea7dd6feb70
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-64-generic
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.952 GiB
Name: ip-172-31-70-70
ID: FMBW:VTZX:TIKD:2IDU:P23J:LAOZ:5PAX:ZUOT:WUIY:NTES:LX5W:VR76
Docker Root Dir: /data/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details (AWS, VirtualBox, physical, etc.):
Swarm is hosted in AWS, we are specifying a second EBS volume (which is encrypted) with -g.

ping @aboch

Interested in picking this up as a first bug contribution to Docker, could anybody potentially mentor me through this?

@imjacobclark This is most likely going to need to be fixed in github.com/docker/libnetwork.

@imjacobclark - let me know if you  will work on this as otherwise I will give it a try.

I found the problem so will submit the pull request soon.

I'm using docker 19.03.5, and this issue still exist:
`
docker info
Client:
Debug Mode: false
Server:
Containers: 10
Running: 8
Paused: 0
Stopped: 2
Images: 14
Server Version: 19.03.5
Storage Driver: devicemapper
Pool Name: docker-252:0-20642-pool
Pool Blocksize: 65.54kB
Base Device Size: 10.74GB
Backing Filesystem: ext4
Udev Sync Supported: true
Data file: /dev/loop0
Metadata file: /dev/loop1
Data loop file: /opt/docker/lib/296608.296608/devicemapper/devicemapper/data
Metadata loop file: /opt/docker/lib/296608.296608/devicemapper/devicemapper/metadata
Data Space Used: 276.9MB
Data Space Total: 107.4GB
Data Space Available: 19.48GB
Metadata Space Used: 18MB
Metadata Space Total: 2.147GB
Metadata Space Available: 2.129GB
Thin Pool Minimum Free Space: 10.74GB
Deferred Removal Enabled: true
Deferred Deletion Enabled: true
Deferred Deleted Device Count: 0
Library Version: 1.02.137 (2016-11-30)
Logging Driver: syslog
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: active
NodeID: fs9icy5ohanizw2r6mxz7d7ta
Is Manager: true
ClusterID: vm4dibc0fc5chmspf3pupf7ol
Managers: 1
Nodes: 1
Default Address Pool: <IP_ADDRESS>/8
SubnetSize: 24
Data Path Port: 4789
Orchestration:
Task History Retention Limit: 5
Raft:
Snapshot Interval: 10000
Number of Old Snapshots to Retain: 0
Heartbeat Tick: 1
Election Tick: 10
Dispatcher:
Heartbeat Period: 5 seconds
CA Configuration:
Expiry Duration: 3 months
Force Rotate: 0
Autolock Managers: false
Root Rotation In Progress: false
Node Address: <IP_ADDRESS>
Manager Addresses:
<IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 894b81a4b802e4eb2a91d1ce216b8817763c29fb
runc version: 425e105d5a03fabd737a126ad93d62a9eeede87f
init version: fec3683
Security Options:
seccomp
Profile: default
userns
Kernel Version: 4.19.111generic
Operating System: Debian GNU/Linux 9 (stretch)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.775GiB
Name: esa
ID: LTB5:OI2M:RCSI:7QWY:E3QB:M75B:B7R5:ZX35:Z3SN:IZNG:ULVG:TPZ7
Docker Root Dir: /opt/docker/lib/296608.296608
Debug Mode: false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false
WARNING: the devicemapper storage-driver is deprecated, and will be removed in a future release.
WARNING: devicemapper: usage of loopback devices is strongly discouraged for production use.
Use --storage-opt dm.thinpooldev to specify a custom block storage device.
`


Add support for Velas mainnet & testnet / eth_gasPrice

Website
Please update the deployment configuration to allow the deploy of the contracts with the following gas price:
eth_gasPrice
ChainId
chainId : 106
https://docs.velas.com/clusters#mainnet
The errors that rise this issue are commented here:
https://github.com/gnosis/safe-singleton-factory/issues/17

It is not possible to rely on a dynamic gas price as access to the private key is required to adjust the gas price before deployment and we will not expose the private key


ColorPicker doesn't resize appropriately

when you toggle on / off. it does not resize the window until it finds a different color name.
Based on 0.29 code base with the Color Name PR:

Originally posted by @crutkas in https://github.com/microsoft/PowerToys/issues/8553#issuecomment-744691609

Already fixed


Windows as Entities

Objective
Windows as Entities
Solution
What solution would you like?

[x] WindowId is now Entity
[x] Extend Commands with a window method returns WindowCommands similar to how the entity method returns EntityCommands,
[x] WindowCommandQueued will implement Command and on write send itself as an event that window backends like bevy_winit will process.
[x] Window should be separated into different components:

[x]  WindowsCommandQueue
[x]  WindowCursor
[x] WindowCursorPosition
[x] WindowHandle
[x] WindowPosition
[x] WindowResizeConstraints
[x] WindowResolution
[x] WindowTitle
[x] WindowPresentation
[x] WindowModeComponent
[x] WindowCurrentlyFocused (marker) (WindowFocused is taken by a window event)
[x] WindowDecorated (marker)
[x] WindowResizable (marker)
[ ] A world query struct containing all these components and markers should be publicly exposed

[x] Windows should be spawned like an entity in Commands with a WindowDescriptor component
[ ] Window backends should query Added and create the windows for them
[ ] while also applying a Window marker confirming they have been created, query Added<Window> or use a WindowCreated event to get newly created windows.
[ ] Despawning windows should be done just as entities (as they are now entities), window backends should close/destroy windows their internal component has been dropped, e.g. WinitWindow (wrapper around winit's Window that destroys itself on drop)
[ ] A resource called PrimaryWindow contains the entity id of initial window created by WindowPlugin, this replaces Windows::primary
[ ] Insert PrimaryWindow resource when primary window has been created
[ ] PrimaryWindow when add_primary_window is false?
[x] WindowPlugin should change exit_on_close into a enum called ExitCondition with the values OnPrimaryClosed (when primary window is closed), OnAllClosed (when all windows are closed), DontExit (keep app headless)

Changelog

WindowId has been replaced by Entity
Window has been split into multiple components:

Window (marker)
WindowCursor
WindowCursorPosition
WindowHandle
WindowPresentation
WindowModeComponent
WindowPosition
WindowResolution
WindowTitle
WindowCanvas
WindowDecoraded  (marker)
WindowCurrentlyFocused (marker)
WindowResizable (marker)
WindowTransparent (marker)
WindowMinimized (marker)
WindowMaximized (marker)

Created WindowCommandsExtensions so that we can commands.window(entity) to get WindowCommands

Similar to how ÈntityCommands work

Split enum WindowCommand into separate structs that all impl Command
Most methods previously existing on Window is now ei
Removed Windows as this should now be accomplished by querying for entities with the Window marker component: Query<Entity, With<Window>>

Migration Guide

[ ] bevy_render
[ ] bevy_window
[ ] bevy_text
[ ] bevy_ui
[ ] bevy_winit
[ ] examples

Process flow:

WindowPlugin sends window_event(CreateWindow { windowDescriptor })
WinitPlugin creates the runner and runs
handle_create_window_events()

What is responsible for flushing the commands?
Should all WindowComponents be read-only?
User request changes happening through WindowCommands
Window backend consumes the command queue and updates the components accordingly

Updates the WinitWindow
Updates the corresponding Component

The changed components can now be queried by the user

Or

User changes the WindowComponent
Window backend listens for Changed events, when updates the WinitWindow accordinly?

Current way the user interacts with windows:

User uses WindowCommands to queue operations to be performed on the window
Backend resolves the commands and updates the corresponding window-components
The user can look at the window-components in a read-only fashion

Type inconsitencies:
Cursor position

WindowCursorPosition.position: Option<DVec2>
SetCursorPositionCommand.position: Vec2
Winit has PhysicalPosition<i32> and LogicalPosition<f64>


Add support for auto-replies using sieve

Originally created by Antoine Nguyen on 2012-08-31T20:20:36Z
Add support for auto-replies using the sieve protocol. Find a way to present this new feature (maybe by using the same screen as the one used for postfix?).
The sievelib module must first implement the vacation extension.

This issue was moved to modoboa/modoboa-sievefilters#6


feat(RELEASE-982): add the catalog (RHEC) url to a results file

This commit adds the catalog (RHEC) url to a results file A task generates a results/publish-pyxis-repository-results.json file with all the catalog url generated.

Fixing the README format, I got some redundant code while resolving the conflict and rebasing simultaneously. I will post a comment once it is ready for review.

Ready for review!

test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite