Add more to the About page

Add a link to the official HASS page for pathways. Look around and maybe add some more helpful info and/or make the about page easier to digest.

I have added a link to the official site, not sure if more info should be added or not. Going to leave this issue up in case someone wants to add more since right now it just has basic info.


[BUG] Scroll bar on hover in sidebar

https://user-images.githubusercontent.com/3443024/130086614-c88b7166-ff02-4964-b8a1-33bed0730d22.mp4
This happens in both Chrome and Firefox but in Firefox it's much more obvious because the scrollbar takes up width and offsets everything in the sidebar.

Just gonna remove the scrollbar within the sidebar completely for now :)

Fixed in #241


[0.18.1][Android] TypeError:expected dynamic type 'boolean' but had type 'null'...

I got red screen error after upgraded to RN 18.1 from 17.0 and this issue is occurred when clicking at any place of the app. My Development machine is Windows 7 Ultimate SP1.

@satya164 Do you have any suggestions. Is there any clean commands or steps for Windows.

Clearing the node_modules folder and executing npm install solves this issue.


Google client keys don't get picked up

We noticed that after upgrading geocoder from 1.25.0 to later versions, the Google environment variables when using a client account do not get picked up, here is the error:
<ipython-input-3-4e48b1d75176> in <module>()
----> 1 geocoder.google(address)

.../geocoder/api.py in google(location, **kwargs)
    193         > elevation
    194     """
--> 195     return get(location, provider='google', **kwargs)
    196
    197

.../geocoder/api.py in get(location, **kwargs)
    159         if method not in options[provider]:
    160             raise ValueError("Invalid method")
--> 161     return options[provider][method](location, **kwargs)
    162
    163

.../geocoder/base.py in __init__(self, location, **kwargs)
    706
    707         # check validity of provider key
--> 708         provider_key = self._get_api_key(kwargs.pop('key', None))
    709
    710         # point to geocode, as a string or coordinates

.../geocoder/base.py in _get_api_key(cls, key)
    685         # raise exception if not valid key found
    686         if not key:
--> 687             raise ValueError('Provide API Key')
    688
    689         return key

ValueError: Provide API Key

It searches for an API key in MultipleResultsQuery in base.py, thus throwing an error if only a client + secret is passed.

uh.. 🤔 Good catch... not too sure what would of caused that.
@ebreton Any clue?

@avanderm Feel free to push a PR with a fix, can't pinpoint exactly what's the cause at the moment.

Will look into it when I have time, this error seems to be causing a lot of the tests to fail

I will have a loot at it too.
What would be great though, when we have fixed this, would be to have the tests merged in the base code... we are lacking a few of them :)

Ok, I think I got it:
It breaks because I have made the "traditionnal api key" mandatory 👎 .
And, correct me if I am wrong, but you do not need it when using google client / secret.
I will push a fix in a branch. @avanderm : will you be able to test from that branch and tell me if that fixes the issue. -> We will be able to merge and release a patch afterwards

Issue fixed for me, thank you very much!

Perfect! 👍


Design Pattern Implement

Design pattern implemented

Remove all of the idea files.

Closing: No reply from the author.


MDC is cleared on all SpanStoppedEvents

MDC is cleared on all SpanStoppedEvents, but that can leave it empty if a span stops in the current thread but its parent is still active (e.g. hit the /call endpoint in the sample app and see what happens).

Fixed in c0ca63ef


[Feature Request] Use Google Auth credentials for login to GoogleDriveLoader

Description:
To align with other APIs such as VertexAI and VertexModelGarden, it may be useful in some cases to pass google.auth.Credentials directly to the GoogleDriveLoader constructor instead of passing the service account file.
It should provide more flexibility. It should not change the other log in features.
Proposed solution:
Instantiate the credentials in the GoogleDriveLoader attributes. Replace the credentials calls with an attribute reference.
I would be happy to contribute if you validate this feature.
from google.auth.credentials import Credentials

class GoogleDriveLoader(BaseLoader, BaseModel):
    """Load Google Docs from `Google Drive`."""
    credentials : Credentials | None

Reference:
https://github.com/langchain-ai/langchain-google/blob/3c9efa1741431e44b0a9b061ba18a4bc51e137a9/libs/community/langchain_google_community/drive.py#L21C7-L21C24

I'm sorry, I missed this feature request. If it's not too late, of course, all contributions are welcome!

There's an older issues that used to track this, but it got closed after the split to langchain-google: https://github.com/langchain-ai/langchain/issues/15058


update all

Hi .. could we get an "update all" button in the "Aktualisierungen" window?

Why you not use the one in Admin5 Adapters?


Feature Request: Port Old Metasploit Shellcodes to IAT Based Shellcodes

Concept:
Use each binary's IAT as the source of API calls.  If it's missing an API, use LoadLibraryA/GetProcessAddress to find it.
Port over reverse_tcp_stager, meterpreter_rev_https
x64 is good for now as detection is nil and EMET does not stop x64 ROP like calls.
https://www.virustotal.com/en/file/03763c06e720aea7e17ff49f9bf36348746496858a4bdd3f21ee84077020c990/analysis/1401594994/

https://github.com/secretsquirrel/the-backdoor-factory#the-backdoor-factory-is-no-longer-maintained-or-supported


WIP: update CI

runs dart test
runs the analyzer
run dart test
build apk

#1

This should now all be covered with #210


refactor(rest): improve inspect api

Add the correct cpus, memory, devices

To avoid merge conflicts, implement devices after waiting for the result of #72.

/ok-to-test

Test Code:

For what it's worth, if you want to add unit tests for this new code, it's now easier using https://github.com/crc-org/vfkit/blob/main/test/vm_test.go ,this makes it easy to create a VM with a set of devices, start vfkit, run some tests (eg make http request to the REST API), and check that the result is the expected one.

Test Code:

For what it's worth, if you want to add unit tests for this new code, it's now easier using https://github.com/crc-org/vfkit/blob/main/test/vm_test.go ,this makes it easy to create a VM with a set of devices, start vfkit, run some tests (eg make http request to the REST API), and check that the result is the expected one.

I've started doing this in https://github.com/cfergeau/vfkit/commit/4615740c4b96d99b7491dee8481246272c01108c but not finished yet :)

I've started doing this in cfergeau@4615740 but not finished yet :)

I'm now done with the testing code/..., I've filed https://github.com/crc-org/vfkit/pull/99 , in my opinion we can move the rest of the discussions in this new PR.

Hey @cfergeau, because it's Chinese New Year now, I'm currently on vacation and will be back in about a week.

Hey @cfergeau, because it's Chinese New Year now, I'm currently on vacation and will be back in about a week.

Thanks for the message, enjoy the holidays!

I'm back :)

I'm now done with the testing code/..., I've filed https://github.com/crc-org/vfkit/pull/99 , in my opinion we can move the rest of the discussions in this new PR.

Does it look like I can close this PR?

I'm back :)

I'm now done with the testing code/..., I've filed #99 , in my opinion we can move the rest of the discussions in this new PR.

Does it look like I can close this PR?

If the json format proposed in #99 for the inspect API works for you, then yes we can close this PR.

If the json format proposed in https://github.com/crc-org/vfkit/pull/99 for the inspect API works for you

As for me, the new JSON format will make Go developers comfortable, but I cannot be certain about developers in other languages because they may not be able to determine the result of the response by inspecting the source code. Perhaps we still need to improve the relevant documentation?
I am also concerned that the new format may introduce more subtle breaking changes. For example, if I modify or delete a key but am unaware that this key is reflected in the inspect API, it could lead to breaking changes inadvertently.

Perhaps we still need to improve the relevant documentation?

Ah yes, for sure, I did not even think about documenting this yet :-/

For example, if I modify or delete a key but am unaware that this key is reflected in the inspect API, it could lead to breaking changes inadvertently.

pkg/config/json_test.go should catch many of these breaking changes, but it's not exhaustive, we could extend it to make sure it covers all the fields that are serialized to json. This way json breaking changes will be noticeable.

Perhaps we still need to improve the relevant documentation?

Ah yes, for sure, I did not even think about documenting this yet :-/

For example, if I modify or delete a key but am unaware that this key is reflected in the inspect API, it could lead to breaking changes inadvertently.

pkg/config/json_test.go should catch many of these breaking changes, but it's not exhaustive, we could extend it to make sure it covers all the fields that are serialized to json. This way json breaking changes will be noticeable.

I tried to do this in https://github.com/crc-org/vfkit/pull/103


upstream-community-operators [N] [CI] universal-crossplane (1.2.1-up.3)

Opening this PR on behalf of Upbound Inc. Upbound Universal Crossplane (UXP) is an open-source downstream distribution of Crossplane that passes the Crossplane Conformance tests and it will be maintained by Upbound Inc.
New Submissions

[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

FYI, Operator test / kiwi / Full operator test (pull_request) seems to be stuck since it's been going on for 3h 18m 3s. It took 3m 48s in the other PR, that adds the same operator.

@muvaf, please check preview, couple things are missing. Ignore missing channel, it is a bug in the preview. But fill other missing parts.

/hold

/hold cancel

/hold

/hold cancel

/test all

I was able to see the logs after timing out after 6 hours.
TASK [deploy_olm_operator_upstream_bundle : Output the catalog-operator log to a debug file] ***
changed: [localhost]

TASK [deploy_olm_operator_upstream_bundle : Get the InstallPlan debug output] ***
changed: [localhost]

TASK [deploy_olm_operator_upstream_bundle : Get the InstallPlan debug output] ***
changed: [localhost]

TASK [deploy_olm_operator_upstream_bundle : Output the InstallPlan debug output to a debug file] ***
changed: [localhost]

TASK [deploy_olm_operator_upstream_bundle : Get namespace state] ***************
changed: [localhost]

TASK [cleanup_operator_resources : Delete all processed resources in the deploy directory] ***
changed: [localhost]
Error: The operation was canceled.

By looking at the code, it tries to delete all resources in {{ olm_operator_files_path }}, which seems to be created & populated here. From what I can tell, it consists of Namespace, OperatorGroup, Subscription and CatalogSource which are roughly what's needed to test a full operator deployment. So I installed the operator following the guide here, which is trimmed down version of original (great doc btw), and then removed each of them. After a while, the command finished and all was deleted successfully.
I tried running the kiwi tests directly on my computer, but it seems it isn't made to work with darwin-amd64, so I gave up on that.

@muvaf i found that it hangs on
kubectl delete -f locks.pkg.crossplane.io.customresourcedefinition.yaml --ignore-not-found=true
customresourcedefinition.apiextensions.k8s.io "locks.pkg.crossplane.io" deleted

```

/merge possible

@mvalarh Ah thanks! Lock is an internal object that users don't really interact with and UXP manages its own CRDs in the init container so it doesn't really need all of the CRDs to be deployed. I included them in the bundle to show up as metadata. So, I removed the locks CRD from the bundle and it seems to work now.


Multiple images and custom separator

Heed your help friends!
I need to have multiple images with buttons to switch among them.
Cant find the right plugin or code for this.
Example
Exact the same function.. how to achieve?
Thank you!

ANY HELP WOULD BE APPRECIATED!

Hi, @Vyborg777, this is accomplishable now, you'd first need to build a slideshow with something like https://get.foundation/sites/docs/orbit.html
and then include the twentytwenty code of as the image in each slide:

Space, the final frontier.

AND then you'd need to attach event handlers to the slideshow buttons to retrigger twentytwenxzty on slide change.
$("#orbit .twentytwenty-container").twentytwenty();

$('.orbit-controls *, .orbit-bullets button').click(()=>{
  $("#orbit .twentytwenty-container").twentytwenty()
})

Also, you can customize any of the styles here: https://github.com/zurb/twentytwenty#build-with-scss


Update EAG language extension to 2.0.x

bumped EAG extensions to version 2.0.3 for Gitpod which is capable to start the compiler generator on file saving and show compiler generator errors on the VS Code problem view
pre-compile gamma when starting Gitpod
let the Gamma compiler generator being invoked on .eag file saving to show errors or generate and compile the compiler
makes EAG (dark) the default theme

Thanks! Now I can let base the talk on that. :smile:


FtpClient.AutoConnect() does not work after upgrading from 34.0.0 to 37.0.2 (fixed) and Session Resume failure

FTP: https://dlptest.com/ftp-test/
Computer OS: Windows
FluentFTP Version: 37.0.2
I'm having issues connecting to a specific FTP server with AutoConnect(), and I noticed my FluentFTP version (34.0.0) could be updated to 37.0.2, so I tried that. I haven't even been able to try and connect again to the specific server, because while debugging locally my application stopped connecting to the public FTP test site from DLP, which used to work just fine with AutoConnect().
Logs:

# AutoConnect()
# AutoDetect(True, False)
# Connect()
Status:   Connecting to ***:21
Response: 220 Welcome to the DLP Test FTP Server
Command:  AUTH TLS
Response: 234 Proceed with negotiation.
Command:  QUIT
Warning:  FtpClient.Disconnect(): Exception caught and discarded while closing control connection: System.Security.Authentication.AuthenticationException: The remote certificate was rejected by the provided RemoteCertificateValidationCallback.
   at System.Net.Security.SslStream.<ThrowIfExceptional>g__ThrowExceptional|137_0(ExceptionDispatchInfo e)
   at System.Net.Security.SslStream.Write(Byte[] buffer, Int32 offset, Int32 count)
   at FluentFTP.FtpSocketStream.Write(Byte[] buffer, Int32 offset, Int32 count)
   at FluentFTP.FtpSocketStream.WriteLine(Encoding encoding, String buf)
   at FluentFTP.FtpClient.Execute(String command)
   at FluentFTP.FtpClient.Disconnect()
Status:   Disposing FtpSocketStream...
# Connect()
Status:   Connecting to ***:21
Status:   There is stale data on the socket, maybe our connection timed out or you did not call GetReply(). Re-connecting...
Status:   Disposing FtpSocketStream...
Status:   Not sending QUIT because the connection has already been closed.
Status:   Disposing FtpSocketStream...

Updates
AutoConnect() worked when called after ValidateAnyCertificate = true (I tried it after analyzing the previous exception message).
But now I'm getting an exception when calling GetListing(path):
# GetListing("/test1/test2/test3", Auto)
Command:  TYPE I
Response: 200 Switching to Binary mode.
# OpenPassiveDataStream(PASV, "LIST /test1/test2/test3", 0)
Command:  PASV
Response: 227 Entering Passive Mode (35,163,228,146,4,21).
Status:   Connecting to ***:1045
Command:  LIST /test1/test2/test3
Response: 150 Here comes the directory listing.
Status:   FTPS Authentication Successful
Status:   Time to activate encryption: 0h 0m 0s.  Total Seconds: 0,4922175.
+---------------------------------------+
-----------------------------------------
Status:   Disposing FtpSocketStream...
# CloseDataStream()
Response: 522 SSL connection failed; session reuse required: see require_ssl_reuse option in vsftpd.conf man page
Status:   Disposing FtpSocketStream...

Update - March 17
I was able to connect successfully using this code:
FtpProfile autoConnectProfile = null;

try
{
    autoConnectProfile = ftpClient.AutoConnect();
}
catch (Exception)
{
    // nothing
}

if (autoConnectProfile == null)
{
    var manualFtpProfile = new FtpProfile
    {
        Host = ftpClient.Host,
        Credentials = ftpClient.Credentials,
        Encoding = Encoding.UTF8,
        Protocols = SslProtocols.None,
        DataConnection = FtpDataConnectionType.PASV,
        Encryption = FtpEncryptionMode.None,
        Timeout = 30000
    };

    ftpClient.Connect(manualFtpProfile);
}

I have followed up on this issue, and here are the results:
With FluentFTP V42+:

AutoConnect() to the above mentioned FTP test server, ftp://ftp.dlptest.com/ works fine. So that bug has been fixed in the meantime.

but in the further text of your issue:

GetListing() produces 522 SSL connection failed; session reuse required: see require_ssl_reuse option in vsftpd.conf man page. This tells us: It is a vsftpd server that requires SSL Session Resume.

Although we support SSL Session Resume in TLS 1.2 connections, there seem to be some specific versions of vsftpf ( unknown ) and a specific versions of FileZilla-Server ( = 0.9.60 ) where SSL Session Resume does not work.
Sadly, the DLP FTP Test server has remove the standard vsftpd version text from his welcome message, so we still do not know which version(s) of vsftpd exhibit this problem.
and your March 17 Update:
By setting Encryption = FtpEncryptionMode.None, you have turned off encryption, thereby using an insecure connect, since there is no SSL, there is no SSL Session Resume and thus no failure. But surely that is not what you intended?

Closing as explained by @FanDjango , this is not a client issue but rather a server issue. We have have extensive automated tests that we can use to prove we support all types of FTP/FTPS servers with reasonable success.


Added leftjoin ORM Query Type

I found myself needing to query the non-foreign-key containing side of a relationship for null on the other and innerjoin does not do that, so here is leftjoin.

Coverage decreased (-0.02%) to 86.27% when pulling 18b3540b706ac30dbe0dc2acf8b9e5f86dc89d19 on TomHAnderson:feature/leftjoin into c5c483e512020d2964e267b219a67aa4ff41f069 on zfcampus:master.

@TomHAnderson No idea why github is not updating your commits here... So weird!

Github status says there's a backlog of queue events.  I'll check back later today.

Coverage decreased (-0.02%) to 86.27% when pulling f7d911985a0babc906dfa8d3e7ccc828a2f68c2f on TomHAnderson:feature/leftjoin into c5c483e512020d2964e267b219a67aa4ff41f069 on zfcampus:master.

This PR now includes all expected commits.

@TomHAnderson Thanks !


New option : a folder to store the generated files

Sometimes, the pdf generated should be located in a specific folder, not in the same as the source files.
Here's what I set for my own use.
A configuration variable can be set to a folder at the root of the project.
If empty, the package will work like before.
If set, this folder is created (if not existing), and the generated files are saved there instead of the source directory.

The idea is good. I look over it, when I have time (in December rather less) and consider the implementation. Thank´s :)

I think it is better to ask the user for destination path. Maybe you have ideas to implement it. #44


cargo build very slow

macos

Hi, Alacritty has a lot of dependencies so it is only natural that the build process takes a while. As far as I know there is no significant slow-down caused by the Alacritty project itself.
If you're compiling Alacritty master you can get a bit of speedup by disabling lto in the Cargo.toml, other than that there aren't a lot of options.

Relevant tracking issue in Rust, rust-lang/rust#48547. This is a known area of improvement for the compiler. I doubt there's much we should be bothered to do at this point to optimize for compile times.

Closing this since it doesn't seem to be an issue of Alacritty and the lto fix I've mentioned should offer at least some improvements.


[cfe] Test failure implies missing promotion based on initialization

This is the CFE specific issue for #43647, which has the details.

language/nnbd/inference/variable_initialization_promotion_test.dart is passing on all platforms except web as of e933e91aaa29531ccb32d5adec853270eedbedae.  I believe the web problem is unrelated to flow analysis, and I'm tracking it in https://github.com/dart-lang/sdk/issues/43723.


Only focus the compose box if a modal is not open

Details
We focus the compose box initially if we aren't on a touch screen. We show the FAB Popover by default for new users who are not on a free plan, leading to the FAB being open while the compose box is focused, if it is a new user. This changes the compose box to initially be focused only if there is no modal open and we are not on a touch screen device
Fixed Issues
$ https://github.com/Expensify/App/issues/13561
Tests

Create a new user and sign in as them
Verify that the FAB is open, but the composer box does not have its focused styles

[ ] Verify that no errors appear in the JS console

Offline tests
N/a, you need to be online to create an account
QA Steps

Create a new user and sign in as them
Verify that the FAB is open, but the composer box does not have its focused styles

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome
Changes nothing on Mobile, since you can't have the FAB and chat open. Verified styles still worked

Mobile Web - Safari
Changes nothing on Mobile, since you can't have the FAB and chat open. Verified styles still worked

Desktop

iOS
Changes nothing on Mobile, since you can't have the FAB and chat open. Verified styles still worked

Android
Changes nothing on Mobile, since you can't have the FAB and chat open. Verified styles still worked

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Not able to test due to an issue with the back end, using a Desktop video instead as per this comment
https://user-images.githubusercontent.com/9059945/208951746-88229282-4222-41cd-a5a9-ae98275457c5.mov

Mobile Web - Chrome
Not able to test due to an issue with the back end, using a Desktop video instead as per this comment
https://user-images.githubusercontent.com/9059945/208951746-88229282-4222-41cd-a5a9-ae98275457c5.mov

Mobile Web - Safari
Not able to test due to an issue with the back end, using a Desktop video instead as per this comment
https://user-images.githubusercontent.com/9059945/208951746-88229282-4222-41cd-a5a9-ae98275457c5.mov

Desktop
https://user-images.githubusercontent.com/9059945/208951746-88229282-4222-41cd-a5a9-ae98275457c5.mov

iOS
Changes nothing on Mobile, since you can't have the FAB and chat open. Verified styles still worked

Android
Changes nothing on Mobile, since you can't have the FAB and chat open. Verified styles still worked

I was thinking if we could make this.shouldFocusInputOnScreenFocus into a function and move this.props.modal.isVisible && !this.props.modal.willAlertModalBecomeVisible inside of it, but there are a couple of different usages of this.shouldFocusInputOnScreenFocus, we'll probably break some edge-case, so this is indeed the best approach

I was thinking if we could make this.shouldFocusInputOnScreenFocus into a function and move this.props.modal.isVisible && !this.props.modal.willAlertModalBecomeVisible inside of it, but there are a couple of different usages of this.shouldFocusInputOnScreenFocus, we'll probably break some edge-case, so this is indeed the best approach

That was my line of thinking as well. shouldFocusInputOnScreenFocus is fairly widely used, and I wanted to avoid breaking edge case situations. Thanks for the review!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @Gonals in version: 1.2.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @chiragsalian in version: 1.2.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

I can't comment in https://github.com/Expensify/App/issues/13561, bumping here for payment

I've unlocked the issue and bumped it for payment! Thanks for your patience!


Update Temurin linux builds to use suggested Security Audit gcc compiler flags

The Adoptium security audit outlined some suggested compiler flags for enhanced security, we need to look at evaluating those, and implementing and testing their use.
As part of the Eclipse Adoptium security audit, the following gcc options were suggested as being used to enhance the security of the JDK binaries.
Code generation suggestions:

-fstack-protector-strong : Extends the basic stack protection without impacting all functions, see: https://lwn.net/Articles/584225/
--param=ssp-buffer-size=4 : param The minimum size of buffers (i.e. arrays) that receive stack smashing protection when -fstack-protector is used. This ties in with -fstack-protector-strong to better cover the stack protection.

Compiler Warning suggestions:

-Wformat : printf/scanf string checks
-D_FORTIFY_SOURCE=2 : string related compile time checks
-Wformat-security : enahnced printf/scanf string security checks
-Wshadow : warn about variable/function shadowing
-Wconversion : Warn for implicit conversions that may alter a value

Builds of jdk-22+35 (Not the GA level!) with a bunch of options. Note that while there are a lot of links in here and the builds are being retained for now this is purely as a convenience while they are being tested and they will not be retained in the future.

Architecture
Options set
AQA Result
Perf result

x64
1
TBC s/o e/o
TBC

aarch64
1
TBC s/o e/o
TBC

x64
2
TBC
TBC

aarch64
2
TBC
TBC

x64
3
TBC s/o e/o
TBC

aarch64
3
TBC s/o e/o
TBC

Initial AQA runs have been done but the openjdk set are being re-run due to them missing the testimage archive which caused a number of failures.

Noting that the -W options generate a significant amount of additional content in the logs. This is from the aarch64 builds:
$ grep -irw warning: jdk22w.none.log | cut -d: -f4-   | sort | uniq -c
      2
      3  call to 'free' declared with attribute warning: use os::free [-Wattribute-warning]
      1  warning: call to 'calloc' declared with attribute warning: use os::malloc and zero out manually [-Wattribute-warning]
      2  warning: call to '_exit' declared with attribute warning: use os::exit [-Wattribute-warning]
      2  warning: call to 'exit' declared with attribute warning: use os::exit [-Wattribute-warning]
   1520  warning: call to 'free' declared with attribute warning: use os::free [-Wattribute-warning]
    740  warning: call to 'malloc' declared with attribute warning: use os::malloc [-Wattribute-warning]
     18  warning: call to 'posix_memalign' declared with attribute warning: don't use [-Wattribute-warning]
      7  warning: call to 'realloc' declared with attribute warning: use os::realloc [-Wattribute-warning]
      4  warning: call to 'realpath' declared with attribute warning: use os::Posix::realpath [-Wattribute-warning]
     29  warning: call to 'vsnprintf' declared with attribute warning: use os::vsnprintf [-Wattribute-warning]
      1  warning: 'jfr_event_writer_flush' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'jfr_register_stack_filter' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'jfr_type_id' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'jfr_unregister_stack_filter' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'memset' writing 696 bytes into a region of size 0 overflows the destination [-Wstringop-overflow=]
      1  warning: type of 'get_lwp_regs' does not match original declaration [-Wlto-type-mismatch]
      1  warning: type of 'init_libproc' does not match original declaration [-Wlto-type-mismatch]
sxa@fedora:~/bench/x$

I'm going to ignore that earlier table for now as there were some issues with those executions.
These are all run with:

JDK_REPO=https://github.com/adoptium/jdk22u
JDK_BRANCH=jdk-22+35
USE_TESTENV_PROPERTIES=false

Architecture
Job number
Options
sanity
extended
perf result
Comment

x64
22+35-ea
None
G9768
G9776

aarch64
22+35-ea
None
G9769

x64
38
RO
G9773

aarch64
24
RO
G9770
G9777

x64
51
-cfi-vh
G9774

aarch64
34
-cfi-vh
G9771
G9778

x64
42
All
G9775

aarch64
26
All
G9772
G9779

Noting that the -W options generate a significant amount of additional content in the logs. This is from the aarch64 builds:
$ grep -irw warning: jdk22w.none.log | cut -d: -f4-   | sort | uniq -c
      2
      3  call to 'free' declared with attribute warning: use os::free [-Wattribute-warning]
      1  warning: call to 'calloc' declared with attribute warning: use os::malloc and zero out manually [-Wattribute-warning]
      2  warning: call to '_exit' declared with attribute warning: use os::exit [-Wattribute-warning]
      2  warning: call to 'exit' declared with attribute warning: use os::exit [-Wattribute-warning]
   1520  warning: call to 'free' declared with attribute warning: use os::free [-Wattribute-warning]
    740  warning: call to 'malloc' declared with attribute warning: use os::malloc [-Wattribute-warning]
     18  warning: call to 'posix_memalign' declared with attribute warning: don't use [-Wattribute-warning]
      7  warning: call to 'realloc' declared with attribute warning: use os::realloc [-Wattribute-warning]
      4  warning: call to 'realpath' declared with attribute warning: use os::Posix::realpath [-Wattribute-warning]
     29  warning: call to 'vsnprintf' declared with attribute warning: use os::vsnprintf [-Wattribute-warning]
      1  warning: 'jfr_event_writer_flush' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'jfr_register_stack_filter' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'jfr_type_id' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'jfr_unregister_stack_filter' violates the C++ One Definition Rule [-Wodr]
      1  warning: 'memset' writing 696 bytes into a region of size 0 overflows the destination [-Wstringop-overflow=]
      1  warning: type of 'get_lwp_regs' does not match original declaration [-Wlto-type-mismatch]
      1  warning: type of 'init_libproc' does not match original declaration [-Wlto-type-mismatch]
sxa@fedora:~/bench/x$

Yes, this is an awful lot of warnings!
Given it's usefulness, I won't try adding these extra -W flags

So the thoughts from talking to Andrew Haley, is the use of enhanced stack protector for Hotspot JVM stack is not beneficial as it is a private stack, and only serves to impinge performance.

Which does not leave any code options. If we feel any extra options are needed we should work with upstream to update.


Display Rewards Breakdown to users

The Vending Machine API supports a rewards breakdown, which returns detailed information on the rewards given, including when they were given, when they expire, and who gave the rewards to the user.
This information should be shown to users so tokens don't "disappear" on them when unclaimed and they expire. Here's the breakdown on VM.

Yes! This was actually a popular request from users. People want to know when they can "accumulate" and "what".
I think this is definitely something worth working on.


Virus Total

Why on the x86 release compilation is there 19/70 virus detections according to virustotal? I'm a bit confused as to what's going on.

Hello @HackerMan01010, it is a false positive detection, as already answered several times in the closed issues. Thanks for understanding.


Feat/document format updation

What

User guide section changes.

Isn't this same as https://github.com/credebl/docs/pull/29?


prefetch-dependencies: enable basic git auth

Cachi2 may fetch git tags from the source repository and/or clone other git repositories. Those may be private. Give the user a way to authenticate.
The basic-auth mechanism is copy-pasted from the git-clone task.

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

Tested in my acmiel-tenant namespace in Konflux prod with
tkn pipeline start -f pipeline.yaml \
  -p git-url=https://github.com/acmiel-rhtap/gomod-pandemonium \
  -p revision=bafb2a2dbbf28ce1abadd4564f12b1a59bdc05a0 \
  -p output-image=quay.io/acmiel-test/gomod-pandemonium:latest \
  -p hermetic=true \
  -p prefetch-input='gomod' \
  --use-param-defaults \
  -w name=workspace,volumeClaimTemplateFile=workspace-template.yaml \
  -w name=git-auth,secret=git-auth

Where pipeline.yaml is downloaded from tkn bundle list quay.io/redhat-appstudio-tekton-catalog/pipeline-docker-build:686833fcebe4f202f97434ca373edd5d5e39a088 -o yaml > pipeline.yaml and modified accordingly
And workspace-template.yaml is
spec:
 accessModes:
   - ReadWriteOnce
 resources:
   requests:
     storage: 1Gi

The prefetch task succeeded

Btw I also tested the SSH directory mechanism, but either I didn't use it right or it doesn't work. So I decided not to include it for now. The Pipeline definition doesn't include it anyway


add devtools-toolbox container build

devtools-toolbox container that ops/sre can use to run ad-hoc commands inside of a pod on an openshift cluster

Can one of the admins verify this patch?

Cancel PR. Will use https://registry.centos.org/centos/tools in favor of this.


CounterShardOperationData is not removed during delete process

Hello,
I configured all the delete process following the guide, using the provided default servlet and the relative queue.
The CounterData object and relative CounterShardData objects are correctly deleted.
But the CounterShardOperationData objects remains inside the datastore
Is this a wanted behaviour?
Thanks

Hi, thanks for reporting this.  This is an oversight, and I've labelled it as a bug to be fixed in the next release.
I'm unfortunately busy with other (paid) work though, so I don't have an ETA on a fix, but should be soon.


Wrong subnetting

Hello
it appears that the subnet calculation in step 3 of "View details of a security rule" is wrong. The page states that: "One of the prefixes in the list is <IP_ADDRESS>/8, which encompasses the <IP_ADDRESS>-<IP_ADDRESS> range".
That is wrong <IP_ADDRESS>/8 means a range of <IP_ADDRESS> to <IP_ADDRESS>.
The relevant Ip range which IP <IP_ADDRESS> is hitting is <IP_ADDRESS>/13.
Thank you for checking.
Sebastian

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

ID: 2fbd1ea4-154d-2998-203c-d77554a22d2d
Version Independent ID: 0b619241-28d5-341c-d52f-f343b3e88a75
Content: Quickstart: Diagnose a VM network traffic filter problem - Azure portal - Azure Network Watcher
Content Source: articles/network-watcher/diagnose-vm-network-traffic-filtering-problem.md
Service: network-watcher
GitHub Login: @damendo
Microsoft Alias: damendo

@shoyer089, Thanks for your feedback. We shall review this and get back to you shortly!

@shoyer089, I have assigned this thread to author for further review
@damendo, Please review this and update the IP address to match the range.

Awaiting inputs from the SME.

Hi @shoyer089,
Thank you for pointing it out. This has been fixed and the update is available at https://learn.microsoft.com/en-us/azure/network-watcher/diagnose-vm-network-traffic-filtering-problem.
Hence, closing the issue.
#please-close


Auto scaling of historical chart data

With air pressure, a fixed Y-axis of say 970-1040 hPa would be within the "norm".
However it's sort of difficult to see changes to the pressure curve with such a big scope.
A solution to this could be to set the min and max range of the Y-axis to be limited to the min max values of the associated data set.

Hi, you should be able to do this already. In the settings of the widget, where you select the dataset, there's an option "Y axis always start from zero". Un-selecting that (maybe with also setting min/max on the first display tab of the widget config) should get you there :) If that doesn't work let me know!


Change of Input value is not refelected when modification is done by third party code

Here is a demo -
http://jsfiddle.net/nsisodiya/jzYj4/
If you type anything in input box, it will be reflected back in model !
but if I modify the value of input element using JavaScript then the value is not reflected back in model , ie model is not updating.
I know, If I type
scope.price = parseInt(scope.price) + 10;

then everything will work because I am directly modify the model !
But in actual code, My third party decorator plugin do not know about model, so what should i do in such kind of situation !

Closing this as it's not in the scope of rivets to handle 3rd-party code by default. This could be addressed with a custom binder.


Injection doesn't work

Kami status always "unconnected"

You need to use custom version of RPCS3 for now: https://github.com/isjuhn/rpcs3/tree/IPC
With PCSX2 it should work on latest dev builds with IPC enabled.

Thanks for the helping. You should write in description about this nuance

Hello again, can you upload somewhere latest ipc implemented rpcs3 build?

Under artifacts here https://cirrus-ci.com/task/5704510594088960

Under artifacts here https://cirrus-ci.com/task/5704510594088960

Still "unconnected" status

Did you build newest version of KAMI?

Did you build newest version of KAMI?

No, can you publish it in "releases" there is latest v0.0.1

thanks, now it's working fine.

done

well, i had new issue. Now when i start injection mouse looking just not work. I had try it in resistance 2 and rdr.

Good morning, I apologize for my ignorance. I downloaded KAMI 0.0.6 and have RPCS3 v0.0.31, but KAMI keeps telling me "unconnected".
I have already tried starting KAMI first then RPCS3, and vice versa.
I have enabled IPC (port 25001).
I've already tried starting the game (Killzone 2 and 3) and then starting KAMI, and other variations
What can I do?
Thank you


HOTFIX: fix streams issues

RocksDBStore.putInternal should bypass logging.
StoreChangeLogger should not call context.recordCollector() when nothing to log

This is for standby tasks. In standby task, recordCollector() throws an exception. There should be nothing to log anyway.

fixed ConcurrentModificationException in StreamThread

@guozhangwang

LGTM. Thanks for the fixes! Merged to trunk.


Convert response type

Is there any way to convert response type in behaviors?
I need to create caching behavior, but I need to put in cache serialized string of response and return serialized string in order not to make serialization over and over again each time I sending the query

No, you can basically only wrap the response in an "envelope" type that
includes the cached/non-cached version.
Or do the serialization/deserialization one level up.
On Wed, Jul 10, 2019 at 6:21 AM Nikita Ilyin<EMAIL_ADDRESS>wrote:

Is there any way to convert response type in behaviors?
I need to create caching behavior, but I need to put in cache serialized
string of response and return serialized string in order not to make
serialization over and over again each time I sending the query
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jbogard/MediatR/issues/419?email_source=notifications&email_token=AAAZQMSX6B7K3V44JMW2GFTP6XA4NA5CNFSM4H7OJC22YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G6KOISA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAZQMX6S6DOB2CLN47EMDTP6XA4NANCNFSM4H7OJC2Q
.


Empty password in commandline does not work

use -E option on command line to set an empty password for database


fix: handle network not undateable error

The return value is never checked, therefore raise exception instead.
I guess we could create another exception type though.

Thanks for the fix


adding sql code to retrieve the pmids that do not have a PMC package …

…downloaded for all papers in corpus with a PMCID

Thanks so much! @azurebrd


Connect 4 in a github profile

@JonathanGin52

Looks good, thanks @alrifay for sharing 👍

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one.


Header click in the empty space to right of the label should sort

I'm submitting a ...
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here

Current behavior
A column is sortable, but you can only click on the text of the header to sort it.  Issue #536 added the ability to click on the icon also to sort the column.
Expected behavior
When you click to the right of the label, and clearly within the header cell (not on the resizer or checkbox), then the sort should also be triggered.
Reproduction of the problem
The sample grid has the same issue.  Here is a mostly working temporary workaround in my view:
  <ngx-datatable-column name="vesselStatus" sortable="true">
    <template let-column="column" let-sort="sortFn" ngx-datatable-header-template>
       <div (click)="sort()" class="sort-fullwidth">Status</div>
    </template>
    <template let-row="row" ngx-datatable-cell-template>
      {{row.vesselStatus}}
    </template>
  </ngx-datatable-column>

in my theme css:
.datatable-header{
  ...
  .datatable-header-cell{
    ...
    .sort-fullwidth{
        display:inline-block;
        width:calc(100% - 20px);
      }
  }
}

This may be better solved by a similar fix as done on issue #536 ?
What is the motivation / use case for changing the behavior?
users sometimes need a wider click area.  Smaller header text means less clickable area to trigger the sort.  Additionally, the header has a pointer icon in this extra blank space, but clicking does nothing.  This leads the user to believe the sort does not work.
Please tell us about your environment:
Windows 10, Webstorm, Angular CLI project

Table version: 0.7.x
npm package:      "@swimlane/ngx-datatable": "^6.0.2",

Angular version: 2.0.x
npm packages:
"@angular/common": "^2.3.1",
"@angular/compiler": "^2.3.1",
"@angular/core": "^2.3.1",
"@angular/forms": "^2.3.1",
"@angular/http": "^2.3.1",
"@angular/platform-browser": "^2.3.1",
"@angular/platform-browser-dynamic": "^2.3.1",
"@angular/router": "^3.3.1",

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
Chome 56

Language:
npm: "typescript": "~2.1.5",

I disagree that it should do that, maybe if you want that I would suggest implementing a header template.

I strongly agree with the original poster, that having wider clicking area than just a visible text is important from a UX standpoint. I also was trying to click on column headers and expected them to sort, but nothing happened. It gives impression that this doesn't work. And the mouse has a clicking cursor everywhere within header, which certainly doesn't help to to distinguish between genuinly clickable place, or a false one.
So at least the cursor should change to indicate non-clickable place of header, when clicking there won't change anything.

This is my hack to get around this:
.datatable-header-cell-label {
  width: calc(100% - 13px); // leave room for sort icon
  float: left;
}

it makes nearly the whole column header clickable except for the 13px that my sort icon takes up. Once you click on it once though the sort icon appears and you are able to click on that as well to sort.


Insider threat ESR statement

Perhaps, wrt the ESR and Insider Threat, we need only identify the need to expand to scope of the threat as presented for On Prem products to also include the Cloud Service.
For example, “While the Common Criteria for On Prem products addresses Insider Threats at certain levels of assurance and only with respect to the product vendor and the customer, CC in the Cloud needs to extend such attention, advice, and/or requirements to include the Cloud provider.”
This could be done without offering any solutions in the ESR and so t would simply state that it’s a probable issue that the iTC will address.

Closed as per WG discussion.


Existence of src/index.ts file causes ng test failure

Please provide us with the following information:

OS?
OS X El Capitan
Versions.
angular-cli: 1.0.0-beta.20-4
node: 5.11.1
os: darwin x64
Repro steps.

ng new something
cd something
ng test
touch src/index.ts
ng test

The log given by the failure.
ERROR in ENOTDIR: not a directory, scandir '/Users/cromed/workspace/indexts/src/index.ts'
29 11 2016 15:25:16.377:INFO [karma]: Karma v1.2.0 server started at http://localhost:9876/
29 11 2016 15:25:16.377:INFO [launcher]: Launching browser Chrome with unlimited concurrency
29 11 2016 15:25:16.436:INFO [launcher]: Starting browser Chrome
29 11 2016 15:25:17.132:INFO [Chrome 54.0.2840 (Mac OS X 10.11.6)]: Connected on socket /#tSz3xWiy9o_NWMKEAAAA with id 59087816
Chrome 54.0.2840 (Mac OS X 10.11.6): Executed 0 of 0 ERROR (0.003 secs / 0 secs)
Mention any other details that might be useful.
it doesn't seem to matter what is in the file src/index.ts just the existence of that file will cause the above error.

That's a nasty error. We'll look into it.

@dehru the default structure of the cli app does not have an index.ts file in the root of src
mac-zchapple:~ zchapple$ ng --version
angular-cli: 1.0.0-beta.24
node: 6.3.1
os: darwin x64

@zackarychapple thanks for your response.  yes I know by default there is no src/index.ts.  But we are using angular-cli as a base for building some shared components between projects at our company, and we wanted to "export" those components from src/index.ts for import into another angular2 project.  So naturally, we added src/index.ts file and told npm about it in package.json kinda like this:
src/index.ts
export * from './app/header'
export * from './app/navigation'

package.json
"main": "src/index.ts"

And that caused the issue.  The workaround is just don't name that file index.ts ( which is not as intuitive ).  we named it src/mainindex.ts and updated package.json accordingly.  But it is odd that the presence of file src/index.ts would be some kinda nasty secret no-no.

Same here, any updates on this?

This doesn't seem to happen anymore.


Translation leads to an error when the world achievement should occur

NOTE: Before you begin working on this, please assign this issue to yourself. If you do not do this, then there's the possibility that 2 people will end up doing the same task, which is not a good use of time.

Masonry and Construction do something similar.
The achievement is transferred, but you get an error.

This is not present anymore.


Skip AssetPostProcessor when file in StreamingAssets folder. #166

This change resolves #166.

I rewrote.


Mistake in accessing angas?

Is there a bug here? At both the times indicated (sunrise and sunset on 2021-04-25), the nakshatra is Hasta=13 only.. Not sure why 14, 15 are showing up.
Used:
d0_angas = daily_panchaanga.day_length_based_periods.dinamaana.get_boundary_angas(anga_type=AngaType.NAKSHATRA, ayanaamsha_id=panchaanga.ayanaamsha_id)
What am I misunderstanding?

ayanamsha should be panchaanga.computation_system.ayanaamsha_id


integration with gulp-rev

cc @nmccready

@nmccready Any update?

@NamPNQ I still don't see the point. Yes revOrigPath is not what your test expects, but it is undefined when we do a new shallow copy. So in  the end it seems doing a shallow copy should be opt in and not opt out for gulp-rev.
Are all gulp plugins supposed to make shallow copies and pass a File on? If not not I don't see the issue except with gulp-rev itself.


Question:About Backbone app

Respected Sir,
I am building a 'Secured cloud storage' project.
Details of project are:-
server side: mongodb+express+nodejs
client side: Backbonejs
The idea is to encrypt the user's file at client side on browser
itself using backbone js,
and then store that encrypted files on server on mongodb or Sever Filesystem.
As I am newbie to Js technology.
So, i searched on internet,I found "NodeCellar: Sample Application
with Backbone.js, Twitter Bootstrap, Node.js, Express, and MongoDB"
http://coenraets.org/blog/2012/10/nodecellar-sample-application-with-backbone-js-twitter-bootstrap-node-js-express-and-mongodb/
So, in order to solve this problem,I first defined my dummy database
so as i could access it.
Then,next step is to create a RESTfull api for accessing/manipulating
that data.And for doing this,i need to go with node.js and express.js.
So, i started creating a routes for my app.
And later that Need to create backbone app.
Question:I have created  for "File",And "Files"  in /public .
And for "http://localhost:3000/account/files/" route i able to get retrieve my data from db.
So, how can i create a  for the data and attached to element in the "account" template?
Would you able to help me sort this problem.
Hoping for your kind favor.

Thanks for opening an issue. I've read the details of your question a few times and I'm having trouble fully understanding the issue that is blocking you.
Did you start a project using Drywall as the starter or did you start from scratch? I'm also not seeing how the nodecellar example ties in.
It's important to understand the tools Drywall is built. Drywall is just using all these modules in concert. Maybe you could try to narrow the focus to some specific/reproducible code snippet(s).


Fixed usage example in README

The README example is invalid, I've fixed it.

Thanks man!

@serge1peshcoff thanks! @peterforgacs although on npmjs there is still the same typo - https://www.npmjs.com/package/ffmpeg-normalize

Should be fine now, thanks for the heads up!
https://www.npmjs.com/package/ffmpeg-normalize/v/1.4.0


Canon i-sensys LBP5050 is not printing in Ubuntu 16.04 LTS

The driver is correctly installed and checking in System settings -> Printers I found the Canon i-sensys LBP5050 printer, but if I try to print a test page nothing happens.
The problem isn't recognising it, the problem is sending prints to it.
What are the things to check?
4.4.0-45-generic is the kernel I'm using.
lsusb says:
Bus 001 Device 003: ID 04a9:26ba Canon, Inc.
and I don't have any other Canon device connected to this computer.
lshw doesn't show it.
dmesg have this information this if I search for Canon:
[    2.113768] usb 1-1.3: New USB device found, idVendor=04a9, idProduct=26ba
[    2.113774] usb 1-1.3: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[    2.113777] usb 1-1.3: Product: Canon CAPT USB Device
[    2.113779] usb 1-1.3: Manufacturer: Canon
[    2.113781] usb 1-1.3: SerialNumber: 0000A1B4TInJ
[    2.183513] usb 1-1.4: new low-speed USB device number 4 using ehci-pci
[    2.279252] usb 1-1.4: New USB device found, idVendor=17ef, idProduct=600e
[    2.279257] usb 1-1.4: New USB device strings: Mfr=0, Product=2, SerialNumber=0

CUPS tries to send to the program that Canon uses and that makes the driver program crash. Could be that it is built against an older library, but I'm not sure.

Asking Canon to update the driver for Linux doesn't seem to help much according to my research,

This is a Canon driver issue, unfortunately. Not sure there is an open source driver that will work... :/


Added classical section

Also added a space-themed classical music playlist

Thanks for contributing and keeping things alphabetical!


update requirements.txt

Pull Request
Description
Update version of nowcasting datamodel from verson 1.11.11 to 1.11.54
Fixes #40
How Has This Been Tested?
NOTE: I am not able to test locally by running the docker-compose command. I do not have the environment variables to run it locally. I probably missed the documentation somewhere for where I can get those, but I am looking 🕵♂ haha

[ ] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

@all-contributors please add @branberry  for code


syntax trees building

A continueation on syntax-tree with the basics there. Some minor changes to the structure but
also a new option to write out the syntax tree.

I tried running this but got more errors than there are tokens in the file, any idea what's up with that?

Basically nothing is implemented. It compiles numbers right now. Nothing more fancy than that. This is about the only valid program:
1
1.0


remove unnecessary comma

Fix SyntaxError. #33

Hi! Thanks for your vigilance ) Also, increase a patch version in package.json, please, or I won't be able to publish the updated version.

@uqee

Thanks! I'll publish in a few minutes.

THX! 😂


Fix Vulkan texture update

Fixes https://github.com/godotengine/godot/issues/78445

For some non-4 byte aligned or compressed image formats (like FORMAT_RH, FORMAT_RG8 or COMPRESS_S3TC with USED_CHANNELS_RG etc.) the staging buffer calculation was off because possible alignment adjustment offset was not taken into account. This sometimes made the last pixel in an image corrupt when uploaded to GPU. Also see RenderingDeviceVulkan::_buffer_update() which does this same calculation correctly.
edit:
This fix proably needs to be done in https://github.com/godotengine/godot/pull/80164 RenderingDeviceVulkan::_texture_update_partial() too. Interestingly, this issue is fixed in Direct3D 12 PR https://github.com/godotengine/godot/pull/70315 in RenderingDeviceD3D12::_texture_update().

Thanks!

Cherry-picked for 4.1.2.


Cargo still in 0.19.0 version

Hi there!
I ran rustup update stable:

rustc has been upgraded to 1.18;
cargo has not been upgraded, he's still in 0.19.0 rather than 0.21.0.

I reinstalled rustup but the problem is still here. Could you help me, please ?
Platform:
Kernel: 4.4.0-67-generic x86_64 GNU/Linux
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial

That's the current stable release of Cargo.

Thanks for responding!

That's the current stable release of Cargo.

Okay, so... Can you explain me why cargo build --bins don't work ? :/
--bin option is effective for 1.18, though.

Hmm, that bit of the 1.18 release announcement may have been wrong - @alexcrichton?

Is it also intended that the crate is stuck at 0.19.0 on crates.io?

The crates.io release matches the stable release.

@Songbird0 ah yes looks like the release announcement was wrong, this feature is coming in Rust 1.19, not Rust 1.18.
@mcarton we only publish stable versions of Cargo currently, which is why it's at 0.19.0.
Looks like the only issue here is with the release announcement, though, so I'm going to close.


Vertus Game issue

HI after update today. I got stuck with vertus. It's run infinity.
"UnboundLocalError: cannot access local variable 'remaining_wait_time' where it is not associated with a value"
And If I delete it, When I add again, it got error too.

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1 In the installer menu, select "gcc."

wtf The virus file. Get out of here

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Thanks, I pushed an update now that should fix it!

Thanks, I pushed an update now that should fix it!

Hi It's didn't working. I delete old game and when add new. It's stuck on seclect session althougt I selected before

Good spot, another update pushed!

Thanks for your update


Added TimeSpan? AWSConfigs.ManualClockCorrection property.

Workaround for exception "The retrieved credentials have already expired". This PR improves PR https://github.com/aws/aws-sdk-net/pull/404.
This allows using other DateTime source (NTP) instead of device time.
Usage:
private void OnApplicationReceivedReliableDate(DateTime reliableDate)
{
    AWSConfigs.ManualClockCorrection = reliableDate - DateTime.UtcNow;
}

Added public property TimeSpan? AWSConfigs.ManualClockCorrection.
AWSConfigs.ManualClockCorrection has higher priority than AWSConfigs.CorrectForClockSkew.
Added test General.TestManualClockCorrection.
Fixed test General.TestClockSkewCorrection. This test was doing nothing because it could not find services in the core assembly.
Added internal field Func<DateTime> AWSConfigs::utcNowSource. This allows tests to simulate skewed client clock.
Replaced occurrences of DateTime.UtcNow to AWSSDKUtils.CorrectedUtcNow.
Related issues: https://github.com/aws/aws-sdk-net/issues/393 https://github.com/aws/aws-sdk-net/issues/402

I'm looking into merging this into the codebase. Can you confirm you are OK with this being pulled in under the Apache 2 license?

I'm OK with the license. I rebased this branch onto the latest master.

Done. It is OK to change the hash for this commit.


Fix Tool Memory Initialization

Fixes tool memory not being loaded for pipelines and workflows.

@vasinov I think the existing unit tests cover functionality touched by this, but let me know if you'd like me to add more.


I have add background images for my slider, not sure is slick slider support to add lazy load?
https://jsfiddle.net/p7by9pwh/

No but why are you using background-image? Using  with data-lazy attribute will work as aspected.


bugfix: fix theme color lock button css

Linked Issue
Issue reported here https://github.com/skeletonlabs/skeleton/pull/1927#issuecomment-1689731301
Description
Theme color lock icon was not visible on firefox prior to this pul request.
Firefox

Changsets
Instructions: Changesets automate our changelog. If you modify files in /packages/skeleton, run pnpm changeset in the root of the monorepo, follow the prompts, then commit the markdown file. Changes that add features should be minor while chores and bugfixes should be patch. Please prefix the changeset message with feat:, bugfix: or chore:.
Checklist
Please read and apply all contribution requirements.

[x] This PR targets the dev branch (NEVER master)
[x] Documentation reflects all relevant changes
[x] Branch is prefixed with: docs/, feat/, chore/, bugfix/
[x] Ensure Svelte and Typescript linting is current - run pnpm check
[x] Ensure Prettier linting is current - run pnpm format
[x] All test cases are passing - run pnpm test
[x] Includes a changeset (if relevant; see above)

@ak4zh @niktek I think there was some breakdown in communication overnight (my time) and the previous PR related to locking was merged. Had I been online, I would not have allowed that update to be be merged.
I posted a longer message about the status of v2 and contributions on Discord earlier today. I'd recommend you read this:
https://discord.com/channels/1003691521280856084/1003699523522142399/1143936240010412133
General idea is the theme generator already had updates to support the new v2 format for themes. This feature was tested and confirmed to be in a working state for the v2 launch next week. So these new changes being implemented now undoes this, and means we would have to go through that testing process again to find new issues introduced. This PR is the perfect example of what I mean - a bug was found that was not addressed because the original PR was not fully vetted. This could have been an issue that made it to production and affected end users. Not to mention any changes to the way the theme generator means there's room for regression, which we're trying desperately to avoid right now.
We're now having to devote a portion of our time to undoing the previous PR merge, and for me to stop and explain all of this. This is time that could have been better spent getting v2 prepped and ready to ship next week.
@ak4zh I'm going to ask that you please refrain from submitting further PRs for the theme generator. You've been linked the issue for upcoming work to this feature. Put your ideas there. Any further PRs related to the theme generator will be rejected without review going forward.


Where do I place this?

Where Do I place arm-none-eabi-gcc? i get a "Command Not Found" and i have the File

Set up devkitARM properly.


C++20 concepts

@klieret it seems that the CI jobs fail to build the latex file. However I have a hard time to find why. Do you have any idea ?

I suspect latex dislikes my use of "&" as a normla character... I just committed a version replacing it with "and".

Found another thing to correct.

@sponce : i am having problems with the pre-commit installation on my ubunutu 21.04
> pre-commit install
Traceback (most recent call last):
  File "/home/chamont-l/.local/bin/pre-commit", line 5, in <module>
    from pre_commit.main import main
ModuleNotFoundError: No module named 'pre_commit'

Is it proved to work on ubunutu 21.04 ?
My installation is rotten ?

@sponce Is there some way to control the size fo fonts for the code examples ?

@sponce Is there some way to control the size fo fonts for the code examples ?

standard way, e.g. \small or \scriptsize

The default code font size seems to me excessively large.

I do not think so. In only few slides it's reduced (an essentially for code examples) but I would think twice before reducing about th amount of text on the slide. Always remember there are 2 cases when there is too much text on a slide : either it's not read and can thus be dropped with no impact, or it's read and while it's read people do not listen to you as they cannot do 2 things at the same time (at least not properly). So a few bullets is the maximum you should put on a slide, and the default font reflects that. Note that it's beamer's default, I did not change it.

@sponce : i am having problems with the pre-commit installation on my ubunutu 21.04

I must admit I did not install it so far. Just did it on my debian unstable via apt-get and pre-commit install worked just fine. Don't know for ubuntu,

I do not think so. In only few slides it's reduced (an essentially for code examples) but I would think twice before reducing about th amount of text on the slide. Always remember there are 2 cases when there is too much text on a slide : either it's not read and can thus be dropped with no impact, or it's read and while it's read people do not listen to you as they cannot do 2 things at the same time (at least not properly). So a few bullets is the maximum you should put on a slide, and the default font reflects that. Note that it's beamer's default, I did not change it.

I fully agree about text. My point is only about code examples. I found it very hard, at least for concepts, to show meaningful examples that fit in the allocated space, with the current default font size for the examples.

@sponce Working on a new version integrating various modifications you suggested. Do not review yet.

@sponce ready for a new review.

@bernhardmgruber Thanks for your previous suggestions. Most of them (if not all) now implemented.

Merged via #90


will nebula support apple silicon?

It works today via rosetta 2. I'm using it right now!
Once Go officially supports darwin/arm64 in a stable release, we'll add that build target to our official releases.

Thank you!

Go 1.16 with support for Apple Silicon is now released! 🎉


Limits yaml schema

This allows us to print:
[opc@mgmt ~]$ ./finish
ERROR: Syntax error in `limits.yaml`.

if there is a limits.yaml like:
VM.Standard2.1:
1: 1
2: 1
3: 1
VM.Standard2.2:
1: 1
2: 1
3: 1

This needs updating to support the different format we use for GCP.


Fix bug. not valid only_if of user_ulimit

Bug
default["redisio"]["default_settings"]["ulimit"] attribute set nil,  but it would have been made /etc/security/limits.d/redis_limits.conf
Cause
user_ulimit is  definition, not lwrp.
ref: https://github.com/bmhatfield/chef-ulimit/blob/master/definitions/user_ulimit.rb
Chef definition does not support nof_if/only_if.

Thanks @w1mvy... this looks good to me.


Publishing Questions

Hello
This is not technival questions but more about Titanium framework it self, which is open source but Appcaelrator is not, did you publish these app with plan or without plan? I am bit confused about that..
Thanks

I published this app using the OSS CLI and SDK. It doesn't use any cloud services (yet) so you can use both.


Can't enable module in contact form

Hi,
I'm unable to use the module in contact form.
Steps to reproduce:

Go to module settings
Configure module and enable all display options.
Hit save

Result: All options are enabled except the contact form.
I get the following error:
directory (/var/www/mystore/override/override/controllers/front) not writable
Already tried to chmod&chown.
This duplication seems incorrect ".../override/override/..." is it?
What else can i try?
Thank you.

It seems contact controller was already overridden.
Sorry to bother you.


Clean up VS Code ignores

The .vscodeignores file controls what files are packaged in the extension distribution (VSIX).
There were several files, both committed and created during a build, that should have been ignored but were not.  There were also other non applicable entries in the file from previous iterations of this repo.
This PR updates the file for our build and does some general cleanup.
Closes #30

@mbeckerle :eyes:  :+1: :-1:


Incompatibility

Crash on start with the mod "Biggerstacks 1.19.2 - 3.8.1"

a crash report would be greatly appreciated


chore: add CODEOWNERS file

This branch adds a CODEOWNERS file to automatically assign reviewers
for PRs.
Currently, I've assigned the @tokio-rs/console team to review all
changes. This team consists of @carllerche, @seanmonstar, @pnkfelix,
myself, and @zaharidichev. If any of you don't want to be automatically
assigned as reviewers for console PRs, let me know, and I can change the
settings to just assign the remaining team members individually.
We can also add people who contributed to specific parts of the project
as reviewers for certain files. I didn't actually do that in this PR,
because everyone who's made really major contributions to specific parts
of the codebase are already part of the @tokio-rs/console team...

(I'm going to wait back to hear from everyone who would be tagged as a reviewer by this change before i merge it)


[Package Issue]: TheDocumentFoundation.LibreOffice - abuse of downloadarchive NON-CDN URLs

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Microsoft Winget is abusing the bandwidth provided by The Document Foundation via the 'downloadarchive' url.
https://downloadarchive.documentfoundation.org/libreoffice/
I'm barely getting a few hundred kbps dowload speed from this when installing/updating Libreoffice via winget.
see: https://redmine.documentfoundation.org/issues/3236
quote (from year 2020, before the load on the server skyrocketed):
Yes… downloadarchive is not backed by a CDN, and shares the same upstream link, so you wouldn't get faster speed there. Commercial CDNs are expensive and definitely not worth it for just not even 10 hits per days on average. We could deploy some of these on our volunteer-driven mirror network, however that'd need to be opt in (the release downloads are mostly fixed in size, but bibisect stuff much larger and ever increasing; that's also why downloadarchive is not backed by a CDN). Most likely few mirrors would be interested in this, not just because of the extra space needs, but also because universities and ISPs prefer to mirror stuff that their users/customers use (and they're vastly more likely to have LibreOffice users than QA team members among them).
/quote
Steps to reproduce

install /update Libreoffice via winget
observe download speeds while doing so

Actual behavior
slow download speed.
Reason: this repository is NOT backed up by a CDN.
Expected behavior
solution: please use URLs on
https://download.documentfoundation.org/libreoffice/stable/

this one is served via the CDN network.

Note: downloadarchive can probably remain in use for URLs of old versions, but please do not use it as the main installation/update repository - it was obviously not designed to handle this much load.
Environment
winget --info
Windows Package Manager (Preview) v1.5.441-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.2604
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.20.441.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

User Settings: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Screenshots and Logs
No response

It looks like the latest version is from @krystofkrticka.
If I'm understanding the issue correctly, the best practice here would be to use the CDN backed URL for the latest version, and then update the URL in the n-3 or n-4 version to the one that is not CDN backed after it falls off of the CDN backed URL.

@denelon what I found out is that downloadarchive is hit or miss when it comes to speed, but when I get to my computer at home I'll look into changing the download links to download.documentfoundation.org

@denelon what I found out is that downloadarchive is hit or miss when it comes to speed, but when I get to my computer at home I'll look into changing the download links to download.documentfoundation.org

Also it looks like the winget is unable to automatically use local mirror because when I create package upgradr it asks me if I want to use my local mirror.

Also I didn't now that download archive is not backed by CDN.

BTW I used urls which were used by previous manifest versions so yes, somebody who created LibreOffice package in repo used download archive so I just kept using it as previous manifests.

it would be nice if it was possible to use local mirror automatically
just specify the URLs as found when browsing on the download server
e.g. in the folder https://download.documentfoundation.org/libreoffice/stable/7.5.1/win/x86_64/
there is the URL
https://download.documentfoundation.org/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi

simply trying to download that will get redirected automatically by the CDN to a (hopefully local) mirror.
example with wget:
---request begin---
GET /libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi HTTP/1.1
Host: download.documentfoundation.org
User-Agent: Wget/1.21.2
Accept: */*
Accept-Encoding: identity
Connection: Keep-Alive

---request end---
HTTP request sent, awaiting response...
---response begin---
HTTP/1.1 302 Found
Date: Tue, 14 Mar 2023 18:04:05 GMT
Content-Type: text/html; charset=iso-8859-1
Content-Length: 414
Connection: keep-alive
Server: Apache
X-MirrorBrain-Mirror: mirrors.up.pt
X-MirrorBrain-Realm: region
Link: <http://download.documentfoundation.org/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi.meta4>; rel=describedby; type="application/metalin                                                                        k4+xml"
Link: <http://download.documentfoundation.org/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi.asc>; rel=describedby; type="application/pgp-signa                                                                        ture"
Link: <http://download.documentfoundation.org/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi.torrent>; rel=describedby; type="application/x-bit                                                                        torrent"
Link: <https://mirrors.up.pt/tdf/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi>; rel=duplicate; pri=1; geo=pt
Link: <https://ftp.bme.hu/pub/mirrors/tdf/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi>; rel=duplicate; pri=2; geo=hu
Link: <https://tdf.mirror.liteserver.nl/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi>; rel=duplicate; pri=3; geo=nl
Link: <https://ftp.snt.utwente.nl/pub/software/tdf/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi>; rel=duplicate; pri=4; geo=nl
Link: <https://tdf.jensgutermuth.de/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi>; rel=duplicate; pri=5; geo=de
Digest: MD5=ESykXIqRmH5nx4/wut0yLw==
Digest: SHA=qyFoYiU8aOpnJU0rQhfyqIaRc6o=
Digest: SHA-256=bpALkOq2jrn5wLgV5WmE3tx7Jv/UwijS8snzLyOoLgk=
Location: https://mirrors.up.pt/tdf/libreoffice/stable/7.5.1/win/x86_64/LibreOffice_7.5.1_Win_x86-64.msi

---response end---

well downloadarchive also redirects to correct mirror but only some times so yes it'll be best if I change links to cdn version

@denelon @R-Adrian #99477

well downloadarchive also redirects to correct mirror but only some times so yes it'll be best if I change links to cdn version

TheDocumentFoundation has mirror support for files on the downloadarchive too, but due the size of the files to be mirrored only servers involved in the testing of builds are mirroring - it's basically a testing bed for the 'live' CDN.
example: if you look at the upcoming 'testing' build of LibreOffice  7.5.2 you'll see some mirrors for testing
https://downloadarchive.documentfoundation.org/libreoffice/old/<IP_ADDRESS>/win/x86_64/LibreOffice_<IP_ADDRESS>_Win_x86-64.msi.mirrorlist
but if you look at the list of mirrors for the current 'stable' version ... there are no other mirrors for downloadarchive so it has to serve that itself.
https://downloadarchive.documentfoundation.org/libreoffice/old/<IP_ADDRESS>/win/x86_64/LibreOffice_<IP_ADDRESS>_Win_x86-64.msi.mirrorlist

Now I'm thinking what should we do after this is version 7.5.1 surpassed by version 7.5.2 on CDN. Should we change links back to downloadarchive because right now it's imposible to prevent 404s without support for alternate (fallback) URL

@R-Adrian BTW I already created PR to fix this issue.

well downloadarchive also redirects to correct mirror but only some times so yes it'll be best if I change links to cdn version

TheDocumentFoundation has mirror support for files on the downloadarchive too, but due the size of the files to be mirrored only servers involved in the testing of builds are mirroring - it's basically a testing bed for the 'live' CDN.
example: if you look at the upcoming 'testing' build of LibreOffice 7.5.2 you'll see some mirrors for testing https://downloadarchive.documentfoundation.org/libreoffice/old/<IP_ADDRESS>/win/x86_64/LibreOffice_<IP_ADDRESS>_Win_x86-64.msi.mirrorlist
but if you look at the list of mirrors for the current 'stable' version ... there are no other mirrors for downloadarchive so it has to serve that itself. https://downloadarchive.documentfoundation.org/libreoffice/old/<IP_ADDRESS>/win/x86_64/LibreOffice_<IP_ADDRESS>_Win_x86-64.msi.mirrorlist

Oh thanks I didn't know this. Because when I created this manifest the mirror list still had mirrors in it.


allow to explicitly define the z-order

Objective
This allows to specify an explicit z value for ordering instead of relying on the z translation value which might already be used for actual positioning.
Solution
We add an extra optional field that can be set. The default behavior of using the z translation does not change.

Changelog
Added

Allows to specify an explicit sprite ordering value when using the 2D renderer

Related to #1275.

Status ? I don't think using Transform for this is a good idea as it could have unintended side effects. Rather, I think it should affect to Bevy Render order instead


doc test due to intended comments

Bug Report
Version
tonic 0.10 & tonic-build 0.10
Platform
Darwin Alexanders-MBP.fritz.box 22.5.0 Darwin Kernel Version 22.5.0: Thu Jun  8 22:21:34 PDT 2023; root:xnu-8796.121.3~7/RELEASE_ARM64_T8112 arm64
Description
The copying of comments can (and does) create doc tests that will then fail a cargo test command. Easiest reproduction is compile google well known types (empty & duration) which has python code in the comments, but also just the intendation is breaking it:
/// A generic empty message that you can re-use to avoid defining duplicated
/// empty messages in your APIs. A typical example is to use it as the request
/// or the response type of an API method. For instance:
///
///      service Foo {
///        rpc Bar(google.protobuf.Empty) returns (google.protobuf.Empty);
///      }
///
/// The JSON representation for `Empty` is empty JSON object `{}`.
#[allow(clippy::derive_partial_eq_without_eq)]
#[derive(Clone, PartialEq, ::prost::Message)]
pub struct Empty {}

}

this will execute service Foo as a doc test and fail

I'm aware that there is wkt types compile option, but the bug would exist either way


New version: Alibaba.AlibabaCloudCLI version 3.0.240

Created by 🥟 Dumplings in workflow run #10435.

Log
Updated: 3.0.239 → 3.0.240
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Engine Issue Tracking

[x] Use C style callback
[x] Move dispatcher code outside of lock
[x] Implement PushAsync for NaiveEngine
[x]  Isolate out the scheduling part and running policy.
- The DoPushToQueue logic can be isolated out, as subclass
- This enables multiple threading policies in for DoPushToQueue

C style callback and PushAsync is done in #63

Dispatcher is moved outside of lock scope in #67

The dispatcher bug is fixed in #69 , now things all out of lock scope.

It turns out there is another BUG in dispatcher, when things are outside lock scope. The code replied on to_delete_, which could change.
The bugfix is in https://github.com/tqchen/mxnet/commit/bd36bbc1c2a49924385489bcdbe63c71dbf8c559

Close this for now, will open new issue if new problem emerges


Provision Failed: HR Hub

Describe the issue
Unfortunately your site provisioning at least partially failed!
Expected behavior
I expected provisioning to not fail.
Details (please complete the following information):

PnP Correlation ID: 2c0843bb-e2e0-4bb4-8589-0681b5e83da8
Reference Template: HUMAN RESOURCE HUB https://lookbook.microsoft.com/details/ebb9432c-3bbd-4f64-8f74-e75a8b4f96de
Primary language of the target tenant: English
Are you using a Tenant Global Admin account to provision the template: No

Hi,
Thanks for using the Look Book site. Inpsecting the event log, we can see - based on the provided Correlation ID - that the provisioning failed while creating the target Communication Site (the exception was 403 - "Forbidden"). Could you please try to provision the solution to a different site URL or with a different user account? In case of any issue, please create another item and provide the corresponding Correlation ID for further investigation. Thanks for your cooperation.


Capybara broken with Firefox 47

FF 47 is broken when attempting to use with selenium-webdriver 2.53+ - Apparently a new release of FF should be coming next week that fixes this.  https://github.com/SeleniumHQ/selenium/issues/2110

Same problem for me.  I tried going to an older FF and then had other problems 😿

@QuotableWater7 What other problems did you have -- you should be able to rollback to FF 46, turn off updates and be good to go.  This Issue was put here as an informational item to explain the current issue -- hopefully they release 47.0.1 soon

When I'm on FF 46, here is an issue I'm seeing:
Element is not clickable at point (570, 660.75). Other element would receive the click: <button aria-expanded="false" class="chalk-btn-danger dropdown-toggle approval-btn" data-toggle="dropdown" type="button"></button>

The spec runs perfectly on our CI though (we use Solano)

@QuotableWater7 It's probably due to window size - so on your machine the default window size is different than on the CI setup and that leads to elements overlapping others (fixed header/footer, etc) - try resizing your window for that test.

how to rollback to FF 46 in Linux?

@rafiqdeen It depends on what particular flavor of Linux you're using and which package manager.

@twalpole I am using ubuntu 14.04 LTS and I tried Synaptic package manager but it shows only 28.0 version.

@rafiqdeen  http://askubuntu.com/questions/428772/how-to-install-specific-version-of-some-package/428778

Firefox 47.0.1 has been released, and appears to also have issues with selenium-webdriver -  stick with 46.0.1 for now

Firefox 47.0.1 works correctly with selenium-webdriver 2.53.4

Firefox 47.0.1 not working correctly, I am getting this
c:/ruby22/lib/ruby/2.2.0/net/http/generic_request.rb:25:in initialize': HTTP request path is empty (ArgumentError)         from c:/ruby22/lib/ruby/2.2.0/net/http/request.rb:14:in initialize'
from c:/ruby22/lib/ruby/gems/2.2.0/gems/selenium-webdriver-2.53.4/lib/selenium/webdriver/remote/http/default.rb:95:in `new'
sometimes I am getting this
c:/ruby22/lib/ruby/gems/2.2.0/gems/selenium-webdriver-2.53.4/lib/selenium/webdriver/firefox/launcher.rb:90:in connect_until_stable': unable to obtain stable fi refox connection in 60 seconds (<IP_ADDRESS>:7055) (Selenium::WebDriver::Error::WebDriverError)         from c:/ruby22/lib/ruby/gems/2.2.0/gems/selenium-webdriver-2.53.4/lib/selenium/webdriver/firefox/launcher.rb:55:in block in launch'
from c:/ruby22/lib/ruby/gems/2.2.0/gems/selenium-webdriver-2.53.4/lib/selenium/webdriver/common/socket_lock.rb:43:in `locked'

@ZigmaZigmax  The errors you are getting are coming from selenium-webdriver and nothing to do with Capybara.  Capybaras own tests run with selenium-webdriver 2.53.4 and Firefox 47.0.1 without issue, so I would suggest making sure you are actually running FF 47.0.1 and that you don't have any hung processes hanging onto ports.  If both of those are fine check out the issue reports at the selenium project on github.

Oh, I am sorry, I am new to this, But I tried capybara, I thought selenium is the only driver to launch firefox, will capybara itself can launch Firefox and do UI functional test?


WIP: UI updates for v2 theme

Codecov Report

Merging #92 (a8ae084) into prep-release (8ed74de) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           prep-release      #92   +/-   ##
=============================================
  Coverage         46.11%   46.11%
=============================================
  Files               122      122
  Lines              4233     4233
=============================================
  Hits               1952     1952
  Misses             2281     2281

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more