Fix workspace detail page is open when clicking on the 3-dots popover

Details

The click on the popover propagates to the parent if no pressable capture it, so we need to stop the propagation.
Fixed Issues

$ https://github.com/Expensify/App/issues/35681
PROPOSAL: https://github.com/Expensify/App/issues/35681#issuecomment-1925245005
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

Go to the Settings page (wrench icon)
Create any workspace if you don't have one
Press the three-dot on any workspace to open the popover/modal
Click on any empty (top/bottom) space of the popover
Verify nothing happened
Hold down the popover menu and drag it out to the overlay
Verify nothing happened

[x] 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: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb 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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[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 grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is 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 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(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[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] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/50919443/a67c7d5b-d14b-4177-85b7-a8c9246ad6de

Android: mWeb Chrome

https://github.com/Expensify/App/assets/50919443/b56cba95-1370-4002-9ee0-0c4dc7bfeb23

iOS: Native

https://github.com/Expensify/App/assets/50919443/80cd3723-6f87-4692-aefb-8065224b3cdc

iOS: mWeb Safari

https://github.com/Expensify/App/assets/50919443/0544216f-0211-4aaf-8661-2577b0e4bc88

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/50919443/c5c979d7-e4dc-4ac5-99e4-8d0146be000f

MacOS: Desktop

https://github.com/Expensify/App/assets/50919443/0267f524-8e79-4f90-8bd8-bc04ba28e67d

Hi! Just a couple quick questions:

Since we're fixing this more globally with more basic components, are we sure that there won't be any issues of this overriding propogation or a click event that we DO want somewhere else? I looked through the code and it seems okay (not that many onClicks are used) but would be good to confirm.
Why are we doing this in both of these components? I think only the BaseModal affects the three dots menu?
Thanks!

Since we're fixing this more globally with more basic components, are we sure that there won't be any issues of this overriding propogation or a click event that we DO want somewhere else?

I don't see any place where we want to propagate click in a modal and I'm pretty sure there won't be a case for that in the future.

Why are we doing this in both of these components? I think only the BaseModal affects the three dots menu?

BaseModal will be used when shouldOverlay is true, otherwise it will use PopoverWithoutOverlay, so I added it here too.

: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 https://github.com/dangrous in version: 1.4.39-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.4.39-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
failure ❌


Add xonsh and xontrib topics to the repo about settings

@jnoortheen please add
For community
⬇  Please click the 👍 reaction instead of leaving a +1 or 👍  comment

Done. Please feel free to add yourselves.


Remove spurious backtick from run command.

@sjwaight,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


wrong order while doing mean subtraction

I think there is a mistake when you subtract the mean channel-wise. In config.py, the order of the image channel-wise mean is already BGR (the author of Resnet stated here #https://github.com/KaimingHe/deep-residual-networks/issues/5). In addition, you use opencv to load image, hence it also load by BGR order (please see here). In data_generators.py, you transpose BGR -> RGB and then apply mean subtraction but this is not correct since they are already in good order (BGR).
By the way, can you share your thoughts about the std_scaling, classifier_regr_std? I really don't get where it comes from and why we need it.

I also find this error, thus
x_img = x_img[:,:, (2, 1, 0)]  # BGR -> RGB
should be commented.

In caffe, the input image is ordered as [batchsize channels height width].
but in tensorflow the order is  [batchsize height width channels ].
Can you still use the weights from the pretrained model?

@liuwei16 yes we can, this is done by separating the backend case. Keras supports 2 backend: Theano (th), and Tensorflow (tf). You can see that @yhenon deal with it by considering backend K.dim_ordering.

@vuhonganh @yhenon Also, I have some confusions about the regression part.
1、 C.std_scaling = 4.0, classifier_regr_std=[8,8,4,4], where is this consideration from? Is it the same in the caffe version?
2、I found the anchor regression is operated on the downscaled grid cells, the regression result is in the type of int, is this harmful for tiny objects with width or height less than 16 pixels?
Thank you very much

@vuhonganh so you claim that the network is using RGB image or BGR?
because as i see it now, you load BGR image -> convert to RGB->substract mean ->convert to BGR and input it to the net. what is the problem?

@giladdiv i think you should review my first comment to understand, it's clear from the first sentence: "I think there is a mistake when you subtract the mean channel-wise."

@vuhonganh
is the code all right with the mean subtraction or needs changing?
@liuwei16
did you find what C.std_scaling=4.0, classifier_reg_std=[8,8,4,4] doing? i think it is to do with zero mean and unit variance but where these values came from and why is scaling done?
Yes if object size is less than 16 pixels the model will struggle for localization and small objects more error because of the IoU implementation. It is better to resize or split your image such that the size of smallest object is greater than 16 pix... thanks

@anandkoirala @liuwei16
Any further ideas about C.std_scaling. What is its purpose? Also, for small objects less than 16x16 pixels, why would IoU implementation affects localization? If we enlarge the image, it is going to significantly increase the computation time, right?


Unbreaking on jetpacks

Was this removed? It doesn't seem to be enchant-able anymore. Used to it lowering power consumption a bit. 1.4.1

I don't recall adding anything like this. You may be thinking of Redstone Arsenal armor. I like the idea though, so I'm adding it too :)


Add support for ingesting data with tagged fields

Tagged fields are coming, see https://github.com/NordicHPC/sonar/pull/66.  The ingestion logic in sonalyze needs to be able to deal with those.
This will probably have a few different aspects:

Reading the tagged data without knowing ahead of time whether they are tagged or not.
Representing missing fields.  Some fields are not optional and it's some kind of error to be missing them; the error should (maybe) be propagated.  Some fields are optional either because that's meaningful or because they've been added "later", and "older" logs may not have them; in this case, there is either a sensible default that the log reader can fill in (this needs to be agreed between sonar and sonalyze; zero is often the right choice) or the fields becomes presented as an Option<> in the code.  Some fields are obsoleted over time and can be dropped by the reader, probably.  (I'm already not using the number-of-CPU-cores field in the log records.)

This has been implemented and pushed, but I'm awaiting a resolution on how we represent the set of GPUs, see https://github.com/NordicHPC/sonar/pull/66 before closing this.

The GPU representation has been agreed on, in line with what I already use here.


fix(tests): revive HTTP/2.0 tests

Summary
This is a revised version of the reversal PR #10599 which switched off the tests for #10595
Checklist

[X] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference
KAG-1105

@hanshuebner I am unclear on whether this (and any other related PRs- it is confusing to know which ones are correct) needs to be backported. In general we backport all test- related stuff.


Added an example for Meteor with raw documentation

See https://github.com/GoogleCloudPlatform/kubernetes/issues/7103

We signed the CLA as a corporation. The name of the company is Q42.

Hi @fabioy, I'm committed to updating the sample to the latest standards, including updating the api version, and making the tutorial platform-agnostic. I wanted to get the sample into the kubernetes repo, so I can work on it without impacting @chees any more.

LGTM (with the promise of update). Filed issue #7179 to track this. Thank you.


change contentDescription programmatically for FloatingActionMenu

Hello,
can you help me, please. How can I programmatically change contentDescription of FloatingActionMenu?
I have added android:contentDescription="my menu" into layout.xml. When I switch TalkBack on, and I tap on a FAB menu, TalkBack says "unmarked button". When I tap it again, TalkBack says "my menu". It seems like two overlayed views.
I would like change contentDescription depending on open/close state of the FAB menu. Is it possible? When I use
fabMenu.setContentDescription(getResources().getString(R.string.menu_open));
it does not work
Library version 1.5.5
Thanks
Best regards
Peter

+1 ! Any news on this one?

👍

Same problem here. If I set contentDescription for FloatingActionMenu via XML it is still read as unlabeled button by Talkback. The solution should be easy it should take android:contentDescription parameter and set the text as content description for both views ( I guess that there are two overlaying views ).

Has this been resolved? Or is there a workaround for this?


ignore git ~/.m2 for mac sourceTree

增加mac soucetree下忽略~/.m2文件夹

Pull Request Test Coverage Report for Build 329

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

Totals

Change from base Build 327:
0.0%

Covered Lines:
2314

Relevant Lines:
3202

💛  - Coveralls

@zorkeAccount  Please sign the CLA


Wrong compat bounds

Please don't have compat bounds like https://github.com/thautwarm/NameResolution.jl/blob/e0d52d57f6487faba685517ee88ff9fbbfcc707d/Project.toml#L13. You just broke my package which is three links down the depency chain with https://github.com/thautwarm/PrettyPrint.jl/releases/tag/v0.2.0. With that release, it's no longer possible to load this package and therefore also not possible to load any of the 14 packages that depend on this package.

Ref https://github.com/JuliaRegistries/General/pull/18713


Analytics: screen tracking

Events Analytics
This PR adds tracking of screens via an Answers Custom Event. I have deliberately kept the analytics integration light so that it can be easily swapped out if we choose another provider. If you load up the app in Alpha, navigate around, close the app, then re-open you should see new "Screen Event"s appear in the Fabric Events dashboard. For example on iOS Alpha:

Note: Answers sends events to the backend from the previous session at startup. So do not expect to see your events right away.
Tester check-list

[ ] Tested on multiple devices / OS versions (Test on the physical device(s) you have access to, otherwise use BrowserStack):

[ ] Galaxy S8 (Android 7.0/8.0)
[ ] Galaxy S7 (Android 6.0)
[ ] iPhone X (iOS 11.x)
[ ] iPhone 8 (11.x)
[ ] iPhone 7 (iOS 10.x)

Optional:

[ ] Google Pixel 2
[ ] Galaxy S8 Plus
[ ] Galaxy S7 Edge
[ ] Galaxy S6
[ ] iPhone 7+
[ ] iPhone 6

The app is crashing immediately when I open it on Android (Galaxy S8).
On iOS I'm able to launch the app and I'm seeing events sent through correctly.

@declanslevin apologies, I forgot to wire up the Analytics on the Java side completely. Should be good when the next alpha builds.

I'm seeing screen events successfully sent for both Android and iOS.
Ready to merge once the checks complete


#2974 restore latitude and logitude to location object

#2974
This PR adds back longitude, latitude and geoResolution to location object.
Additionally created some tests to check if geoResolution behaviour is as desired.

Codecov Report

Merging #5 (65d8473) into main (690949a) will increase coverage by 20.77%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main       #5       +/-   ##
===========================================
+ Coverage   65.02%   85.79%   +20.77%
===========================================
  Files         165       35      -130
  Lines        5595     1098     -4497
  Branches     1429      248     -1181
===========================================
- Hits         3638      942     -2696
+ Misses       1957      156     -1801

Impacted Files
Coverage Δ

data-serving/data-service/src/model/location.ts
100.00% <ø> (ø)

data-serving/data-service/src/controllers/case.ts
78.07% <100.00%> (+0.22%)
:arrow_up:

data-serving/data-service/src/util/case.ts
87.30% <100.00%> (+0.33%)
:arrow_up:

... and 130 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Automatic boot and shutdown

The server should have the possibility to automatically boot shutdown at a certain time on the wall clock.
I can imagine several ways for a shutdown e.g. a cron job.
But how does auto start work? I don’t know, if a BIOS settingwould be a good idea as this would neither be controllable for the regular user nor remotely.
If we can‘t find a reliable, good solution, there’s also the possibility to not do this inside at all but have a second unit e.g. a Raspberry Pi or an Arduino take care.
It’s a design decision.
Sent with GitHawk

This seems to work like a charm:
https://www.linux.com/learn/wake-linux-rtc-alarm-clock

Using this method I made this concept:
Since the settings server needs to get the user input and write it somewhere, it’s still iunclear to me whether this should become a shell script or something the settings server handles completely.
Here is, what I think should happen depending on the input:

Start up
Shut down
Black out
set wakealarm
start display and app
cron poweroff
cron blackout

x

x

x
x

x

x

x

x

x

x

x

x

x

x

Pseudo code
always called at boot time:
function set_auto_startup_and_shutdown
    clear old wake_alarm
    clear start_app_and_display cron
    clear shutdown cron
    clear blackout cron

    startup_time = get_next_startup_time

    if start_up_enabled:
        if blackout_enabled:
            true:
                set_start_app_and_display_cron
            false:
                set_wakealarm

    if blackout_enabled:
        set_blackout_cron

    if shutdown_enabled:
        set_poweroff_cron

function get_next_startup_time:
    get from settings server

The setting server now does all that
:-)


Add SPF support

https://www.linode.com/docs/email/postfix/configure-spf-and-dkim-in-postfix-on-debian-8

Hi @deacix, SPF is part of DNS configuration, its out the mail server scope it self event if closely related. What do you have in mind?

Perhaps we can leave a note in the Wiki as best practices.
Or perhaps add in setup.sh a SPF entry generator that can be copy/paste in the dns zone file.

@tomav for outgoing mails yes, but for incoming mails it would be better to have a milter that checks their SPF headers. Currently only DKIM and DMARC headers are checked.
In tutorial that @deacix has specified it's done via postfix-policyd-spf-python SPF policy agent. I have played with it earlier but without success (due to my Alpine-based images focus), but in Ubuntu-based image I think it's quite trivial to install via apt-get.
Alternatively, a tiny and simple smf-spf milter may be used.

Oh yeah, I hadn't read the entire article. Let's add it. @deacix do you want to submit a PR?

Hi @deacix it would be great if you could contribute on that topic! Let us know.
Thank you.

Hi,
i am just on the time pressure. Can some else implement it?
Sergej

Pull request incoming within the hour. Currently testing.


google.protobuf.empty is not defined

I tried with Counter example from grpc-dotnet. I am getting the error as "google.protobuf.empty" is not defined even after importing it in the top.
`syntax = "proto3";
import "google/protobuf/empty.proto";
package count;
// The counter service definition.
service Counter {
// Increment count by 1
rpc IncrementCount (google.protobuf.Empty) returns (CounterReply);
// Increment count through client stream of multiple counts
rpc AccumulateCount (stream CounterRequest) returns (CounterReply);
// Countdown with server stream
rpc Countdown (google.protobuf.Empty) returns (stream CounterReply);
}
// The request message containing the count to increment by
message CounterRequest {
int32 count = 1;
}
// The response message containing the current count
message CounterReply {
int32 count = 1;
}
`
What could be the problem? I tried in Asp.Net Core 3.1 LTS version.
Environment:
OS: Windows 10

I don't think this has anything to do with gRPC's C# implementation, which is now deprecated regardless.


German translation

lang.zip

@felixfrischeisen you shouldn't upload files like this through github as it makes version control a lot harder. I'll make sure to implement your suggestions in another commit on monday though, so they will get added to this pullrequest

@felixfrischeisen I've merged some of your proposals, however I also left out some, since they were spelled wrong or use a too abstract name (Magnify -> Magnifizierung. This is a word not commonly used. Vergrößern has the same meaning and is actively used by native speakers).
You also sometimes decided to translate some strings differently than the OS would ("Cancel" -> "Abbruch", even though Windows uses "Abbrechen", same with "Browse" and "Suchen", even though every OS uses "Durchsuchen" for such an action).
In general, your sentence structure seems to be a mix between English and German, for example you never use constructed words ("ganze Zahl" rather than "Ganzzahl"). You also had some strings you translated incorrectly (MessagecuDNNOK "cuDNN can be used" -> "cuDNN konnte nicht benutzt werden" and since the messages get displayed before cuDNN is used, you need to use the current timeform not the past).

@Electron-x thanks for the review and conversations, I gotta admit some strings were really weirdly translated. Feel free to add suggestions if you notice similar issues, I'll make sure to approve them right away so they get automatically changed in this PR


"Comminications” translation string

"Comminications"is still English but I can't find it in translation strings.

Sorry, "Communications" was missing and so was "Empty". They are both now added to the sources.


V2 beta

Thanks for submitting a pull request! Please provide enough information so that others can review your pull request:
Summary

This PR fixes/implements the following bugs/features

[ ] Bug 1
[ ] Bug 2
[ ] Feature 1
[ ] Feature 2
[ ] Breaking changes

Explain the motivation for making this change. What existing problem does the pull request solve?

Does the code pass AppVeyor?

[ ] Yes

Closing issues

Fixes #

did not mean to create that.


Error loading R projects

2/12/2017 10:18:36 AM
Recoverable
System.NullReferenceException: Object reference not set to an instance of an object.
at System.Runtime.InteropServices.Marshal.ThrowExceptionForHRInternal(Int32 errorCode, IntPtr errorInfo)
at Microsoft.VisualStudio.ErrorHandler.ThrowOnFailure(Int32 hr, Int32[] expectedHRFailure)
at Microsoft.VisualStudio.R.Package.Shell.VsAppShell.EnsurePackageLoaded(Guid guidPackage)
at Microsoft.VisualStudio.R.Package.ProjectSystem.RProjectLoadHooks.d__20.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProjectSystem.UnconfiguredProjectImpl.AutoLoadMethodStateMachine.<b__6_0>d.MoveNext()
Unable to rectify this even with uninstall/reinstall.

Hi. Sorry about that. Can you please do a
R Tools / feedback / send frown. And attach the logs so we can debug this?
Thank you
On Sun, Feb 12, 2017 at 7:28 AM Geof11061<EMAIL_ADDRESS>wrote:

2/12/2017 10:18:36 AM
Recoverable
System.NullReferenceException: Object reference not set to an instance of
an object.
at
System.Runtime.InteropServices.Marshal.ThrowExceptionForHRInternal(Int32
errorCode, IntPtr errorInfo)
at Microsoft.VisualStudio.ErrorHandler.ThrowOnFailure(Int32 hr, Int32[]
expectedHRFailure)
at
Microsoft.VisualStudio.R.Package.Shell.VsAppShell.EnsurePackageLoaded(Guid
guidPackage)
at
Microsoft.VisualStudio.R.Package.ProjectSystem.RProjectLoadHooks.d__20.MoveNext()
--- End of stack trace from previous location where exception was thrown
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task
task)
at
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task
task)
at
Microsoft.VisualStudio.ProjectSystem.UnconfiguredProjectImpl.AutoLoadMethodStateMachine.<b__6_0>d.MoveNext()
Unable to rectify this even with uninstall/reinstall.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/3181, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtUVGLv5Fwf1BjtO89fOXL--lwU1XI6ks5rbyUDgaJpZM4L-hGd
.

Hi - apologies, the functionality from the menu in Visual Studio that you mention doesn't seem to work (actually none of the options appear to do anything at all, maybe as a result of this problem).
In which directory on my machine might I find the logs that you require?
Thanks!

No problem. Do you know which version of visual studio and which version of
RTVS you are on?
On Sun, Feb 12, 2017 at 9:08 AM Geof11061<EMAIL_ADDRESS>wrote:

Hi - apologies, the functionality from the menu in Visual Studio that you
mention doesn't seem to work (actually none of the options appear to do
anything at all, maybe as a result of this problem).
In which directory on my machine might I find the logs that you require?
Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/3181#issuecomment-279232361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtUVAgPGlsEvArfpTOLcNBpdWfR_Wklks5rbzyJgaJpZM4L-hGd
.

Thanks - it is Visual Studio Community 2015, Version 14.0.25431.01 Update 3
R Tools for Visual Studio 0.5.21008.0800
Please let me know if you require anything else to help you - Thank you!

Can you please try to uninstall the extension, reset VS MEF Cache (delete %USERPROFILE%\AppData\Local\Microsoft\VisualStudio\14.0\ComponentModelCache folder) and then install it on the clean cache?

Thanks - I tried that (uninstall, delete cache folder, reinstall, start VS) but unfortunately this did not work, when trying to create a new R project after opening up VS again I get the error message "Invalid pointer (Exception from HRESULT:0x80004003 (E_POINTER)) - if I ignore this and then trying to open an existing project gives the same error message as my original post.
This extension did work fine and was able to write R code and use Variable Explorer etc. successfully for a few days but this started all of a sudden yesterday evening.

Actually, I face the exact same issue.
And in fact I also I tried everything to solve the problem but still persists; @Geof11061 do you by any chance have Python Tools installed in your VS?
Because I was trying to deduce what component causes this error, so I completely uninstalled VS with all its components, then I did a clean install and started all over again, first installing R and slowly adding components.
Up to the point where VS didn't contain Python Tools everything worked fine, but when I installed PT then the error occurred.
An exception was encountered while constructing the content of this frame.  This information is also logged in "C:\Users\George\AppData\Roaming\Microsoft\VisualStudio\14.0\ActivityLog.xml".
Exception details:
System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.VisualStudio.Shell.Interop.IVsShell5.LoadPackageWithContext(Guid& packageGuid, Int32 reason, Guid& context)
at Microsoft.VisualStudio.Platform.WindowManagement.WindowFrame.GetPackage()
at Microsoft.VisualStudio.Platform.WindowManagement.WindowFrame.ConstructContent()
Also here is a list of extensions:
Microsoft Visual Studio Enterprise 2015
Version 14.0.25431.01 Update 3
Microsoft .NET Framework
Version 4.6.01586
Installed Version: Enterprise
Architecture and Modeling Tools   00322-80000-00000-AA777
Visual Basic 2015   00322-80000-00000-AA777
Visual C# 2015   00322-80000-00000-AA777
Visual C++ 2015   00322-80000-00000-AA777
Application Insights Tools for Visual Studio Package   7.17.00119.9
ASP.NET and Web Tools 2015 (RC1 Update 1)   14.1.11107.0
ASP.NET Web Frameworks and Tools 2012.2   4.1.41102.0
ASP.NET Web Frameworks and Tools 2013   5.2.40314.0
Azure App Service Tools v2.7.1   14.0.11112.0
Common Azure Tools   1.8
JavaScript Language Service   2.0
JavaScript Project System   2.0
Microsoft Azure Mobile Services Tools   1.4
NuGet Package Manager   3.4.4
NVIDIA CUDA 8.0 Wizards   8.0
NVIDIA Nsight Visual Studio Edition   <IP_ADDRESS>23
PreEmptive Analytics Visualizer   1.2
Python Tools for Visual Studio   2.2.50113.00
Python Tools for Visual Studio - Django Integration   2.2.50113.00
Python Tools for Visual Studio - Profiling Support   2.2.50113.00
R Tools for Visual Studio   0.5.21008.0800
SQL Server Analysis Services   13.0.1701.8
SQL Server Data Tools   14.0.61021.0
SQL Server Integration Services
SQL Server Reporting Services   13.0.1701.8
TypeScript   <IP_ADDRESS>
Xamarin   <IP_ADDRESS> (680125b)
Xamarin.Android   <IP_ADDRESS> (ce955cc)
Xamarin.iOS   <IP_ADDRESS> (44931ae)

Yes DolanDack - thanks for your input. I do indeed have Python tools also installed.  But it does sound like my scenario is identical to yours - a conflict between the two packages of some sort?

Yep, sure it is. If you leave Python and uninstall R then is fine, if you uninstall Python and you leave R, again is fine.
By the way, I have also installed VS 2015 Enterprise on a MacBook running Windows, I have R there too which was running smoothly as far as I remember (I use it rarely). Give me 5 mins to check if our problem occurs there too. If not I need to check what components I do not have installed on that machine.

Ok, so apparently on the Windows 10 system installed in the MacBook R Tools work fine, even alongside Python Tools. So, I am guessing that the problem might lie deeper, I have installed Tensorflow GPU (only on the desktop machine -- previous post) to work with python (that's why the NVIDIA CUDA in my previous post), have you by any chance done the same? I am trying to install tensorflow GPU + CUDA on the MacBook to see if I will experience the same problem. But generally, the whole thing looks really weird.
The list bellow is from the MacBook system:
Microsoft Visual Studio Enterprise 2015
Version 14.0.25431.01 Update 3
Microsoft .NET Framework
Version 4.6.01038
Installed Version: Enterprise
Architecture and Modeling Tools   00322-90150-00969-AA712
LightSwitch for Visual Studio 2015   00322-90150-00969-AA712
Microsoft Visual Studio Tools for Applications 2015   00322-90150-00969-AA712
Visual Basic 2015   00322-90150-00969-AA712
Visual C# 2015   00322-90150-00969-AA77
Visual C++ 2015   00322-90150-00969-AA712
Windows Phone SDK 8.0 - ENU   00322-90150-00969-AA712
Application Insights Tools for Visual Studio Package   7.0.20622.1
ASP.NET and Web Tools 2015 (RC1 Update 1)   14.1.20203.0
ASP.NET Web Frameworks and Tools 2012.2   4.1.41102.0
ASP.NET Web Frameworks and Tools 2013   5.2.40314.0
Azure App Service Tools v2.8.2   14.0.20201.0
Common Azure Tools   1.8
DataFactoryProject   1.0
JavaScript Language Service   2.0
JavaScript Project System   2.0
Microsoft Azure Data Factory Node Node   1.0
Microsoft Azure HDInsight HQL Service   2.0.2900.0
Microsoft Azure HDInsight Tools for Visual Studio   2.0.2900.0
Microsoft Azure Mobile Services Tools   1.4
Microsoft Azure Mobile Services Tools
Microsoft Azure Tools   2.8
Microsoft Azure Tools for Microsoft Visual Studio 2015 - v2.8.40114.1
NuGet Package Manager   3.4.4
Office Developer Tools for Visual Studio 2015 ENU   14.0.23928
PreEmptive Analytics Visualizer   1.2
Python Tools for Visual Studio   2.2.50113.00
Python Tools for Visual Studio - Django Integration   2.2.50113.00
Python Tools for Visual Studio - Profiling Support   2.2.50113.00
R Tools for Visual Studio   0.5.21008.0800
SQL Server Analysis Services   13.0.1701.8
SQL Server Data Tools   14.0.61021.0
SQL Server Integration Services
SQL Server Reporting Services   13.0.1701.8
TypeScript   <IP_ADDRESS>
Workflow Manager Tools 1.0   1.0

Ok I think I found were the error lies,
so R (with MS R Open) + Python = works,
Python w/o R (with MS R Open) = works,
R Tools (with MS R Open) w/o Python = works,
R (with MS R Open) + Python + CUDA = works,
R (with MS R Open) + Python + Intel MKL Tools = doesn't work
I uninstalled Intel MKL Tools and again it works fine. Unfortunately though, I like the performance increase that MKL offers and I don't feel comfortable trading it for the convenience of having R in VS, no matter how much I like it as an idea.

Thanks DolanDack!
I don't have Intel MKL Tools I must say.. Also it is worth pointing out that I did have successful installation of both Python and R within Visual Studio that was working in tandem with each other for close to a week before it started going wrong.
In the error message a posted originally, there is an assertion about a package not being loaded. Is there a debug version I can install that can give some idea as to what package is missing maybe?

@DolanDack, can you share the VS log file mentioned in the error message you posted earlier (C:\Users\George\AppData\Roaming\Microsoft\VisualStudio\14.0\ActivityLog.xml)
@Geof11061, can you also take a look and see if you have one too? It should be in the same location, with the only difference being the user folder name.
It does, indeed, appear to be a package load issue, and that log file is where detailed errors about that are written.

Hi - per Alex - can you also try 0.6?
download from here:  https://aka.ms/private-rtvs-ea-0-6
On Sun, Feb 12, 2017 at 4:31 PM, Pavel Minaev<EMAIL_ADDRESS>wrote:

@DolanDack https://github.com/DolanDack, can you share the VS log file
mentioned in the error message you posted earlier (C:\Users\George\AppData
Roaming\Microsoft\VisualStudio\14.0\ActivityLog.xml)
@Geof11061 https://github.com/Geof11061, can you also take a look and
see if you have one too? It should be in the same location, with the only
difference being the user folder name.
It does, indeed, appear to be a package load issue, and that log file is
where detailed errors about that are written.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/3181#issuecomment-279269240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtUVADQaJkLC9m96Ocp9j_Cj2UvYorQks5rb6R9gaJpZM4L-hGd
.

ActivityLog.zip.zip
Hi int19h- hopefully you can see the log file I attached?

Hm... there is nothing about RTVS in the log.
@Geof11061 , did you get the log after VS crashed? This file is created every time you start VS, so any errors from previous start are deleted.

Hi AlxanderSher - just reloaded the project and got the original error - this is the log at the time of the error
ActivityLog.zip

Hey @AlexanderSher , I have attached the log-file from my system as you instructed.
ActivityLog.zip

WoW! Guys, brilliant job @smortaz @AlexanderSher R works fine now, when I installed 0.6, with MKL too. Only a minor error, when I start VS and I have set the Data Science setting as a workspace format then I get an error (I have attached photo + log files) which is fixed if you reload the Data Science setting or bring up an interactive R window. Here is the new activity log zip in case you wish to compare them
ActivityLog.zip

Here is the stack trace from @DolanDack :
Object reference not set to an instance of an object.
   at Microsoft.R.Support.Settings.RToolsSettings.get_Current()
   at Microsoft.VisualStudio.R.Package.Repl.VsRInteractiveWorkflowProvider.CreateRInteractiveWorkflow()
   at System.Lazy`1.CreateValue()
   at System.Lazy`1.LazyInitValue()
   at System.Lazy`1.get_Value()
   at Microsoft.VisualStudio.R.Package.Repl.VsRInteractiveWorkflowProvider.GetOrCreate()
   at Microsoft.VisualStudio.R.Package.Repl.Editor.KeyProcessorProvider.GetAssociatedProcessor(IWpfTextView wpfTextView)
   at Microsoft.VisualStudio.Text.Editor.Implementation.InputController.<UpdateKeyProcessors>b__8_0(IKeyProcessorProvider p)
   at Microsoft.VisualStudio.Text.Utilities.GuardedOperations.InstantiateExtension[TExtension,TMetadata,TExtensionInstance](Object errorSource, Lazy`2 provider, Func`2 getter)

@Geof11061 , have you tried 0.6 (https://aka.ms/private-rtvs-ea-0-6)?

@AlexanderSher - Hi Alexander, I have not had a chance yet to try 0.6 but I will do so tonight and let you know. Optimistic that it will based on the above.

Awesome - can confirm that the problem does not exist in version 0.6. Thanks!!!


Mouse position broken on retina screen

OSX 10.11.1 Google Chrome. Similar problem on Firefox for Mac as well.
The x,y coordinate of where the mouse clicks is exactly halfway from the top left corner of the page, compared to the mouse's actual position. Everything on the canvas is rendered at half position.

Thanks! #9 fixes this

#9 fixed this.


Celery: Ignore loggers celery.worker.job and celery.worker.strategy for breadcrumbs

Those two loggers completely flood the breadcrumb buffer. I suspect that they "drive out" other events as well. Some people were complaining about junk breadcrumbs in celery from those loggers.
It's possible that we have a bug where we leak breadcrumbs between threads (and that's what's causing this issue), but I don't see how. If so we should see breadcrumbs from other loggers "leaked" between tasks, not just these two celery loggers.
Fix: call ignore_logger for both celery.worker.job and celery.worker.strategy in the celery integration

cc @mitsuhiko since you showed me this issue in our internal project. We have a customer with the same issue.

I am in favour of ignoring those by default.

Decided to leave out celery.worker.strategy as that one is not too verbose. It's really just celery.worker.job that spams


tour: Feature Deprecations

Context: https://go-tour-ru-ru.appspot.com/
**Hello!
Could you restore the tour in Russian, because it is not available at the moment.
App Engine feature deprecations.
The standalone legacy SDK and appcfg tooling.
Shutdown Date: August 30, 2020
Go 1.9 is no longer available.
To continue running your apps, migrate them to the Go 1.11 standard runtime or newer runtimes.**
https://tour.golang.org/welcome/2
Russian - Русский
https://go-tour-ru-ru.appspot.com/
https://cloud.google.com/appengine/docs/deprecations

Indeed, see #1022.


Enhancement : Houdini dynamic publish parameters

Changelog Description
I was exploring Houdini ideas/issues/PRs and came across allow houdini publish instances duplication.
I ~stole some code from~ was inspired by these two PRs https://github.com/ynput/OpenPype/pull/5490 and https://github.com/ynput/OpenPype/pull/5616
I was abstracting their ideas and trying imitating their functionalities by Houdini expressions.
one interesting thing that when houdini evaluates the value of a parameter it also evaluate any expressions used inside it.
so, this is actually works!
This is my first test, where I edited manually these parameters

one limitation is that family must be included in the node's name because you can have the same variant for multiple families
and that's why I can't use node's name for variant value

subset name = opname(".")
variant = strreplace(chs("subset"), chs("family"), "")
instance_node = opfullpath(".")
instance_id = opfullpath(".")
use chs('subset')

and that's a thing that you can try it yourself without pulling this branch!

Here are some tests from this branch
https://github.com/ynput/OpenPype/assets/20871534/e9d01f86-53e9-40cc-b29b-61d9d357198d
https://github.com/ynput/OpenPype/assets/20871534/be5e2758-0e6a-427f-8018-b5233847be78
Testing notes:

Create publish instances
Duplicate them as many as you want
publish

subset name = opname(".")

variant = strreplace(chs("subset"), chs("family"), "")

instance_node = opfullpath(".")

instance_id = opfullpath(".")

use chs('subset')

Let me just reply to these choices.
Skip instance_node and instance_id - these should not be 'imprinted' as attributes on the nodes as they represent the node itself. Which this PR solves: https://github.com/ynput/OpenPype/pull/5490 by removing them from as parms on the node.
Then subset_name and variant. It's up to the template system of OpenPype and the actual creator to define the subset name based on the variant - by default it includes the family, but it may not! which means we cannot rely on the variant always being the stripped subset name, etc. Because it's not always just subset = {family}{Variant} but can be different, e.g. a render usually includes the Task as well {family}{Task}{Variant} - based on templates.
Additionally, using the node name as the 'subset name' is dangerous. In practice the artist should not have full control over how to name the subset - it's based on the variant and the creator (which the creator + templates default to "{family}{Variant}). However if the node name is to represent the subset name, what happens if an artist renames the node "hello" - you're suddenly working with a subset name hello` that doesn't adhere to these rules.
I also want to remark that storing the family on the node itself to me personally makes no sense whatsoever. It's the Creator that defines what family it produces, so if the node already has the creator identifier technically we can deduce what family it should be. Having this always defined by the creator and not saved into the node will also allow us to maintain the code easier since we don't need to ensure whatever data is on the node matches what the Creator has as family (or families; which has been an occurrence with bgeo + alembic in Houdini; but also with USD families in Maya which I solved exactly like that; the Creator always defines the families and is not embedded onto the container, see this PR)
Then the output paths for the intermediate/staged paths, using chs("subset") - go for it! That makes total sense.
In short, the first step is removing ALL data from the node that isn't actually defined by what is stored in attributes:

Instance ID: Remove, it's just the node's path -> solved by https://github.com/ynput/OpenPype/pull/5490

Instance node: Remove, it's just the node's path -> solved by https://github.com/ynput/OpenPype/pull/5490

Family: Remove, the creator identifier links it to a creator which is what defines the family - can be solved with similar approach as https://github.com/ynput/OpenPype/pull/5490

Subset: Technically we can remove this, since it's always computed from creator + variant but in this case having subset defined is somewhat of a safety mechanism. It's basically saying: this was the last computation of the subset which we want to keep no matter what; unless you change the variant again. Just so that if someone were to change the template for subset names that older publishes still update into the legacy names and don't start creating new subset names midway in production.

More importantly I think this definitly tries to solve some concerns or remarks that people have had on the Houdini integration but as far as I understand are not solving it the way people like @fabiaserra requested. All of these would already be solved by a custom output HDA which doesn't even show these 'raw text fields' but e.g. has a button for "Set asset" which shows a tiny asset selector to update the asset, or a text field "Variant" when edited that will automatically compute the new "Subset" name on changes. Similarly - yes we could use chs("subset") in the path name but since the HDA would take full control over what attributes the user changes the output name can be arbitrarily updated as well if needed. Nonetheless, I would even move the output paths for the saver to the template system as well so that settings just define a default like:
rop_output_template: $HIP/pyblish/`chs("subset")`{ext}

And using the template give the rest of the control to the creator to manage the path. The custom HDA would just on attribute changes ask the Creator: "variant has changed, what should I update"? Which in practice is just updating the instance data as far as I know and triggering a save changes on the CreateContext. And there we have it, a trivial implementation of embedding the Creator/publish logic into an HDA.

I needed some time to digest your comments.
Here's a quick recap:
Let me know if I got you right.

There's no dependency between node name and  (subset name, variant name)
we should add an issue about deducing families from creator identifier in Houdini

Should that happen while creation like your PR #5616 ?

we should add an issue about using templates for file output

I imagine these templates should exist in Houdini > General settings
Here's a screenshot for visualization

There's no dependency between node name and (subset name, variant name)

There could be - but it makes no sense since:

The user can't fully control the subset name, they can choose the creator + variant name. Then the pipeline defines the subset name based on the template system.
Using the node name has issue with "uniqueness" it's totally fine to publish a variant Main and Main for two different nodes, e.g. the pointcacheMain and vdbcacheMain. The same issue exists for subsets as well actually, it's totally fine for 2x pointcacheMain and pointcacheMain subsets to be published from a single scene, for example when those are publishing into different assets. We do that a lot - single workfile, publish into multiple assets.

we should add an issue about deducing families from creator identifier in Houdini

This should happen on create AND on collect_instances, since that's when the Creator creates the instances for the CreateContext. Additionally this data should not be stored on the instance node in the scene (it should not be stored in parms on the node, it should not be serialized because the Creator defines the families).
One could argue it's actually transient_data but the publish system needs it in instance.data["families"] due to how pyblish works.

we should add an issue about using templates for file output
I imagine these templates should exist in Houdini > General settings
Here's a screenshot for visualization

Potentially - I initially imagined each Creator just had its own entry so studios could go crazy configuring it per creator if they wanted.
So you'd have:
CreateVDB/output_path and CreatePointcache/output_path etc.

Thanks a lot, I have added two issues.
https://github.com/ynput/OpenPype/issues/5712
https://github.com/ynput/OpenPype/issues/5714
Regarding the subset name, This validator I made for static mesh FBX, from my testing I found out it works well for computing the subset name and it doesn't depend on the instance name.
https://github.com/ynput/OpenPype/blob/develop/openpype/hosts/houdini/plugins/publish/validate_subset_name.py


rename productinfo to cloudinfo

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


How to use custom authentication parameters?

Hello httpyac team,
Do you have a way to pass custom authentication parameters, such as an origin header, resource, etc? IntelliJ provides this feature: https://www.jetbrains.com/help/idea/oauth-2-0-authorization.html#custom-authentication-parameters (usage sample on https://youtrack.jetbrains.com/issue/IJPL-65460/Support-Audience-and-Resource-in-HTTP-Client).
Does httpYac has something similar?
I'm asking this because I would like to authenticate to Azure EntraId using code flow with PKCE. Has anyone already achieved this using this flow type?
Thanks in advance

I have tried to support the custom request headers resource and audience that I know, as I have already used them. In other words, a configuration analogous to Intellij Http Client should be supported. For examples of how I would configure it, I would refer you to these examples using an outdated Keycloak OAuth2 Server There is also an example of how to further modify the OAuth2 request (oauth2_interceptRequtest.http)

Hi @AnWeber, I don't understand The syntax. How could I declare a custom header abc=def for example?

I did not have custom headers in the auth request in mind. So IT IS Not as easy. But it will work.
{{
  exports.oauth2_interceptRequest = function(request, context) {
    request.headers.foo='bar';
  }
}}
GET https://httpbin.org/anything
Authorization: oauth2 password

Thanks @AnWeber  :)

You were successful? :-)


Exporting trained model to ONNX without geffnet.config.set_exportable(True)?

Hi Ross,
First of all, thank you for the awesome repo, it has been wonderful to use this and pytorch-image-models so far. I just have a couple of questions about ONNX export.

I (mistakenly) trained my EfficientNet model on the models found in pytorch-image-models and then realised that to export it to ONNX I needed to have
a) used gen-efficientnet-pytorch and
b) set the geffnet.config.set_exportable(True) beforehand as you have stated in your export script.

Is there any way I could perhaps still transfer the weights from a pytorch-image-models version of EfficientNet to an equivalent gen-efficientnet-pytorch version AND at the same time make it compatible with ONNX?

Is there any chance you could also add ONNX export somehow to the pytorch-image-models versions of EfficientNet? My code base immensely prefers to work with the attributes and methods available for models in that repository compared with the models in geffnet

Seconding the suggestion in issue #32 and would love to have the option for that for pytorch-image-models as well.

Thank you

@andrew12678 the weights should be 100% compatible, there is a bug if you cannot load a checkpoint trained with pytorch-image-models in this repository, that is why I've left the features somewhat disjoint, one is focused on training, the other on minimal versions of the models for inference/export. I do plan to fix #32 and also make similar interfaces in pytorch-image-models, but still other priorities ahead of that, mostly related to EfficientDet.

Thank you @rwightman I spent some effort to make my codebase work without some of the attributes I relied on in pytorch-image-models so that I could get the gen-efficientnet-pytorch version of the model and the weights worked like a charm and I was able to export to ONNX without a problem! I even tested it on the same dummy output for the original pytorch-image-models version and the "weights transfered" gen-efficientnet-pytorchversion and got the same output so that is good news.
So for future work, since I'm assuming that the "non-exportable" nature of the pytorch-image-models version is probably there so it can be more efficient would you recommend me continuing to train with EfficientNets from pytorch-image-models (for efficiency) and then just transferring the weights or is there some underlying benefit of using gen-efficientnet-pytorch straight away with set_exportable(True) that I'm currently not aware of?

works fien with latest versions


Page: About

https://www.nuget.org/policies/About

This will require a gallery static content deployment.


Pure Acks coalescing

What is a reason do not coalesce pure ACK packets ?
The only information that is required for TCP processing is a number of arrived ACKs and cumulative acknowledgement number.
This information is enough for Reno/NewReno congestion algorithms.

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

ID: 423a1055-ed86-1e17-f29e-d35436c0ac44
Version Independent ID: 9c71623c-8217-00c8-2a7b-73bdffd8481c
Content: Rules for coalescing TCP/IP segments
Content Source: windows-driver-docs-pr/network/rules-for-coalescing-tcp-ip-packets.md
Service: unspecified
Product: windows-hardware
GitHub Login: @tedhudek
Microsoft Alias: windowsdriverdev

Hi @elenagurevich , thanks for the question. Let me investigate this for you and see what I can find out.

Hi @elenagurevich , thanks for your patience. After some discussion, the answer boils down to making sure that RSC doesn't impact Windows's TCP congestion control algorithms, both on the send and receive side. The details are internal-only, but at least we know why now. :) I'll go ahead and add a clarifying sentence to the page. Thanks for the question, and thanks for your interest in improving the docs.


Help: Aggregation-service returning metric incompatible with the number of conversions even with the scaled value

Hello everyone, everything good ?
Guys, our team today has all the necessary attribution-reporting functionalities (source, trigger, sending of aggregated reports), and to test the functioning of attribution-reporting we served a creative through Admanager, this placement generated the expected conversions (the page conversion is the same destination page as the creative), but when generating the summary report it returns a metric that is completely different from the number of reports we have in our AWS bucket (each file is equivalent to a report record)
Does anyone have any idea how we could investigate the cause of this problem and eventually find a solution?
Example of a conversion or aggregatable report (the value is scaled to mitigate noise 65534 is actually equivalent to 1, in this case one conversion):
//Histogram
[
 {
  "key": "0x9c3c00203e14ab93",
  "value": 65534
 }
]

//Body
{
   "aggregation_coordinator_origin": "https://publickeyservice.msmt.aws.privacysandboxservices.com",
   "aggregation_service_payloads": [ {
      "key_id": "1f5e5273-b0db-40d9-942f-2142a9669330",
      "payload": "z3Fvi+NpvFRTK2ecgwP2fDJZgrqilmxRbQbeawZ50mp/HVJlD5fJAmXgmx0HWSaXXNCzV0iLswzrvKwHLqTrKq8hHucraB5d74Be99EAIfcx470LMTWtCa9tg0C7V3j5EXhLH4DjqscrEtpOr1K7qWLKoXr73iKxBF+MWTr+qHKLdcRvv7vU76cKWX19cMFjgKo0X0dHv5qJWPySZGcb4Y04wV9kLr0ucJZq8h1TI7j2wd5DsEytO/hdkm7/ipZX8N/ovRXRn/Mwqvj8cRRqhIRiWScoexL/mBdI18E87ZbO0aePPx3BTSv9m4+tGRKQGLGSFGdID7QoUniwv4SPZZUzrsfojEYZSa8+LnujBXa5Dp4Q37I4xegasjZirm8wH13uzYQ8al39JTMVnETSbo9JCdKZwbLKU6ViBD1DiTsqvFHhDZiK03wYvXMEJVmUbhgl+uDUF0C6bqV9zc/sBTzz/m7MW6N2penQKWeBnPVRq3l3Aqvr5G3SUC+XXehyhZj5A1j72F8UMQHv9vnvdo9v449REKZ133IJcqAfeene8xxLZ7+6F6/QNUYbXDbID4DCxU4QvbudFn5KI/I9MsFpIrCzyif8PiUq6MD4FFVszIsCo6wlkM6f4VZWxaA4jwSZygZqROOPwyEk3PqHxkcVU+Cz2KG64mrUi6GSVMH5Wwn7fUckviWUMhkADluFPYijXxJmTuq5KDJhnJ4cdoDZ9+4Q0U5hRT3Y6/VbQiqOXhiFV3d9mTBNnpHK2iI7iEZ5Ud2Zveyai+FAv5HI8ldY+UP4UJdaie6LLIwmThNaFYFUmhCb5tIrusROxYiDnZ1ss7Kn0wZBq0pK6Hb+F0wICOwQ2G4hJNuW0dv/2hHIWVv+dmNb0S6ciDQPDmNSk7EY+ViyQ8qeCruB9L6Rftd6p+unxBLHB4VXDji8vyh4SdpMgH6Wbf2/bsE60xeTb3sj06/r2qDjaeeDvv61oS3l3G7sAU8tNuK3eDjbknt7Sg8WZNIWKqs/VI/d0Lql8KB+ZaG9Vcw4cUIe+OY49BJpORY0zrSuJ5IW"
   } ],
   "shared_info": "{\"api\":\"attribution-reporting\",\"attribution_destination\":\"https://uol.com.br\",\"report_id\":\"055ffbe8-7c43-425a-af6f-ecd3938bd6a5\",\"reporting_origin\":\"https://attribution.ads.uol.com.br\",\"scheduled_report_time\":\"1720819104\",\"source_registration_time\":\"0\",\"version\":\"0.1\"}",
   "source_debug_key": "647775351539539"
}

CreateJob  request body:
*Batch of reports are processed per day
{
    "input_data_blob_prefix": "report_2024_07_11",
    "input_data_bucket_name": "uolcsm-uolads-aggregate-reports",
    "output_data_blob_prefix": "output/summary_report.avro",
    "output_data_bucket_name": "uolcsm-uolads-aggregate-reports",
    "job_parameters": {
        "attribution_report_to": "https://attribution.ads.uol.com.br",
        "output_domain_blob_prefix": "output_domain.avro",
        "output_domain_bucket_name": "uolcsm-uolads-aggregate-reports"
    },
    "job_request_id": "meuk20240710"
}

Print with the files from the AWS bucket (each file corresponds to a aggregatable report), the files are created from the path /.well-known/attribution-reporting/report-aggregate-attribution :

Metrics from summary report file:
*The expected metric should be something close to 2621360 (65534 * 40)

If anyone else has experienced this problem, I would greatly appreciate the help, because we have already tried on multiple days but the metric returned is always far from what was expected.
Thanks !

Hi @dbrito, can you provide the getJob response as well?

hi @dbrito ,
Can you email Aggregation Service Support<EMAIL_ADDRESS>with your getJob response?
Pre-3PC deprecation, to better debug your issue, we suggest enabling debugging for Attribution Reporting API. You will receive debug enabled reports in the .well-known/attribution-reporting/debug/report-event-attribution endpoint with reports marked as "debug_enabled" in the shared_info field.
This will allow you to run Aggregation Service on debug mode by adding "debug_run": "true" in the job_parameters map. Once you run the job in debug_run, this will create 2 reports. One debug enabled and one summary report. The debug enabled report will provide you the noise that is added in the summary report.
Thanks!

Hi @maybellineboon, I will implement this debug and generate these new reports, with this evidence I will also send the email to support

@maybellineboon just one question, after activating debug (and looking here it seems that it was already activated), do I have to change anything in the way I create the .avro files for the aggregated report ?
*Currently the file consists of the payload, key_id and shared_info fields
const REPORTS_AVRO_SCHEMA = {
    "name": "AvroAggregatableReport",
    "type": "record",
    "fields": [
        { "name": "payload", "type": "bytes" },
        { "name": "key_id", "type": "string" },
        { "name": "shared_info", "type": "string" }
    ]
};

Hi @dbrito ,
The current avro schema is still the same. No changes in the aggregatable report. The only difference you'll observe will be having the debug_cleartext_payload in the aggregation_service_payloads field and having the \"debug_mode\":\"enabled\" as part of the shared_info field.
Your createJob JSON body should look like this. And you should be able to generate the summary report and a debug summary report under your S3 bucket folder output/debug/
{
  "job_request_id": "<job_request_id>",
  "input_data_blob_prefix": "<report_folder>/<report_name>.avro",
  "input_data_bucket_name": "<input_bucket_name>",
  "output_data_blob_prefix": "<output_folder>/<summary_report_prefix>",
  "output_data_bucket_name": "<output_bucket_name>",
  "job_parameters": {
    "output_domain_blob_prefix": "<output_domain_folder>/<output_domain>.avro",
    "output_domain_bucket_name": "<output_domain_bucket_name>",
    "attribution_report_to": "<reporting origin of report>"
    "debug_run": "true"
  }
}

Sample Debug Enabled Report:
{
  "aggregation_coordinator_origin": "https://publickeyservice.msmt.aws.privacysandboxservices.com",
  "aggregation_service_payloads": [
    {
      "debug_cleartext_payload": "omRkYXRhlKJldmFsdWVEAAAAgGZidWNrZXRQAAAAAAAAAAAAAAAAAAAD26JldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAKJldmFsdWVEAAAAAGZidWNrZXRQAAAAAAAAAAAAAAAAAAAAAGlvcGVyYXRpb25paGlzdG9ncmFt",
      "key_id": "23597aae-6215-4fe9-8447-cb8e2cf80de8",
      "payload": "NPng8Biaa0V9HvzuhPmdZCT7qHZ7pzX1I8R1rGWJpVWLUM5g6b4j6DM7uep28pc+5SxMGLGL2ms0JjSthi5C5u9Uc+5ibdPUWvI9lasC7D8GRXaZhkAVBydOITFvQ28P/DFuQejpLyw+/GrY2dKd57SDj++aL2LE6cnQJxsP9INRvngu32CReaUVJ1f36hw0lztiJhTK6yYAdjyMawJjjiLLM1vf5qog1YUuz1buweJFukDhJIGAAPo6+Z5fiPt5VpmNoz5WCJMpajPyKp/vXBKy57EaC7OTCGHlGdvvbVzJGtWgMwrCPL61EahTdGtIniNBR+7/mT63yLMDnR2e9lssBITe01y3GiMHbloV2Uqfi4Vg7W44YglJ2aGU8oi8lyyxPuvBO8EkZGx5OglOQF7t+ZbfNqcw+4E1EaISWMvPlhGNNDjXcAK1IqWGbblEN7v3Q+BU/nBiwCaAHIpHRi9J3ZKXYLrSxb6yHDRP+za7CLKweKFUDMYW5n8/RDHLlwhzenbkUJx5pHjH9/DsVadAt6wvuHgQmiwsFH6AvlyuNXSD0yGZ5qy0+YyFsB5rIBDGSUuGa6hU4dq0jXJtfmzo1JOu4VMU1dYcZq1bJpuN/lwfQlIr9Q/BiwLVWRi6gm1yRA3ixvjwAqr/FQh5NH5zA9mwTf9d6Kkeh8NdIaY1URMuGSuaXXbOQ3hEFmQSr26Nlb2XDIqdVyOBTZyDVf2bXIZaBmaZxvnIyahfr55tgIlPudpuZNcDzxIK6tMvbjsAbnVq5eJuAhQUbVykH2PQIanHIK/hVGc8pQ1Qn34JqinBOt6KQf3FCvvCJBicepjT+cvHu+VN7FVOiOQhLNaBcLNDJiMz0DOEXcs1CWaekUry0NTjGAbZoHPI8kxPxSz9vl9eO5eSLCKj0ztAUS6BMRBen51vIqLvwTIROHEjIxGA1cc7DC6lMcL7PS0pM3rw9GQhLxEid5qz79rYsnGtIfuxkVFPF6ZCEihFWxa/swoAMmOySTu2n5SbyfR87l+rmbfr7xKstDnZAELQtv4Ap5N07Ha0XbgK"
    }
  ],
  "debug_key": "1234",
  "shared_info": "{\"api\":\"shared-storage\",\"debug_mode\":\"enabled\",\"report_id\":\"b15d20f7-11c7-431f-9310-8d6a69018a9e\",\"reporting_origin\":\"https://privacy-sandbox-demos-dsp.dev\",\"scheduled_report_time\":\"1719355379\",\"version\":\"0.1\"}"
}

@maybellineboon Sorry for the deviation from the agenda, but after uploading some changes (more logs, debug path support) I noticed in Chrome internals that my report does not have debug enabled even though it worked perfectly locally
*I created the debug keys (debug_key and debug_reporting on source and trigger) as well as the cookie (ar_debug=1)

*debug_key has the same value on both steps (source and trigger)
Do you know if the debug has any operating limitations when it is being served via production?
Link with the implementation example:
https://partner.googleadservices.com/gampad/ads?gct=mQ8rkyFwrIsKwQGKGL0BKrABCAkQH0oAUhVmaWxlOi8vQ29tcG9zaXRlLmh0bWxYgICA4OvIyfOzAbgBk9fS8IME0gIJdGV4dC9odG1s8AKsAvgC-gGQAwHIA6wC0AP6AfADArgEst_yBcoFFmh0dHBzOi8vYWRzLnVvbC5jb20uYnKCBiwIst_yBRCAgIDg68jJ87MBGhAIABAAGAAggICA4P3pguIQIICAgOD96YLiDfoGAigBmggCEgDwCwGYDwKyAQIYAcoBAkABGL24xrQGKAAwvdT7uwY4AFgBagZfYmxhbmtwu-j1swY&iu=8804&gdfp_req=1&height=250&width=300&impl=ifr&sfv=1-0-40

Hi @dbrito ,
I was checking your link and was able to see that on trigger, you also set the cookie. However, you're only setting ar_debug=1; Path=/; HttpOnly; Secure. You're missing the "SameSite=None;". So when you check chrome://attribution-internals, you'll observe that the debug_key is getting cleared on trigger but not on source. Please update the cookie to ar_debug=1; Path=/; SameSite=None; HttpOnly; Secure. This should fix the issue and help you get debug reports.

Hi @maybellineboon Thank you very much, you are saving lifes here, I added the sameSite prop and now it is correct
Thank you one more time
Tomorrow I will generate the sample with the creative placement and with that I will follow up with the support

Hello @maybellineboon , how are you?
@maybellineboon I implemented debug and I was capturing some metrics for the test only that I noticed that not all captured reports had their debug version (debug_cleartext_payload and debug_mode property in shared_Info), would you know what scenario that would be possible ? Since debugging is enabled for everyone
*Because of this reporting scenario without debugging, the aggregation-service Job is always returning the error REPORTS_WITH_ERRORS_EXCEEDED_THRESHOLD
{
    "job_status": "FINISHED",
    "request_received_at": "2024-07-17T14:31:06.057168Z",
    "request_updated_at": "2024-07-17T14:31:03.113553793Z",
    "job_request_id": "report_2024_07_15_2",
    "input_data_blob_prefix": "2024_07_15/report_2024_07_15",
    "input_data_bucket_name": "uolcsm-uolads-aggregate-reports",
    "output_data_blob_prefix": "output/summary_report_2024_07_15.avro",
    "output_data_bucket_name": "uolcsm-uolads-aggregate-reports",
    "postback_url": "",
    "result_info": {
        "return_code": "REPORTS_WITH_ERRORS_EXCEEDED_THRESHOLD",
        "return_message": "Aggregation job failed early because the number of reports excluded from aggregation exceeded threshold.",
        "error_summary": {
            "error_counts": [
                {
                    "category": "DEBUG_NOT_ENABLED",
                    "count": 1,
                    "description": "Reports without shared_info.debug_mode enabled cannot be processed in a debug run."
                },
                {
                    "category": "NUM_REPORTS_WITH_ERRORS",
                    "count": 1,
                    "description": "Total number of reports that had an error. These reports were not considered in aggregation. See additional error messages for details on specific reasons."
                }
            ],
            "error_messages": []
        },
        "finished_at": "2024-07-17T14:31:03.110081155Z"
    },
    "job_parameters": {
        "debug_run": "true",
        "output_domain_bucket_name": "uolcsm-uolads-aggregate-reports",
        "output_domain_blob_prefix": "output_domain.avro",
        "attribution_report_to": "https://attribution.ads.uol.com.br"
    },
    "request_processing_started_at": "2024-07-17T14:31:02.933687572Z"
}

*Print of our bucket with both versions of the same report (with and without debug)

Thank you in advance for your help

Hi @dbrito ,
From the getJob error, it looks like one of the reports do not have debug_mode enabled. If some of your reports does not have the \"debug_mode\":\"enabled\" in the shared_info field, it means the report was generated in non-debug mode. You will not be able to batch these reports in "debug_run": "true" in Aggregation Service.
Please remove reports that do not have \"debug_mode\":\"enabled\" and try again.

Hi @dbrito ,
You can batch the non-debug and debug reports together without the "debug_run": "true" in the createJob request. This will produce you a summary report for both debug and non-debug reports.
However, since we're trying to debug your reports and why you're observing a significantly lower aggregate, you'll have to run the rest of the debug reports on its own after. You should be able to re-run your debug reports and get the debug summary reports.

Hi @maybellineboon , how are you?
Just updating the thread here, with debug enabled and the conditional to create report files only if they have the debug flag, the aggregation-service is working perfectly, the number of metrics is completely in accordance with the quantity of files created in the bucket
I still don't know why my api receives improper aggregate reports, but I'm going to test again without my conditional that only allows the creation of reports with debug, and with that see if the aggregation-service keeps returning the metric compatible with the number of creatives

Hi @dbrito ,
Can you help elaborate more or share what you mean by improper aggregate reports?
Attribution Reporting API's aggregate reporting will randomly send null reports aside from real aggregate reports. This is for privacy reasons. The null reports does not impact the summary reports and does not contribute to any aggregate.
So, if you're testing on live traffic, you might randomly receive null reports.

Hi @maybellineboon,
So, the inappropriate reports would be exactly those that, with debug enabled, did not have a file with the clear_text_payload version. From what you explained, I now think I understand their origin, so these are null reports sent to protect users' privacy
@maybellineboon, thank you very much for your help, without it I would still be banging my head
All the best there


[DOC] LITE Time classifier metrics

State that Keras Metrics class objects (and functions) can be used. Also link to the Keras documentation.
Closes #2435
Any other comments?
Maybe this change is also relevant for other Keras-based (tensorflow-based) estimators?
PR checklist
For all contributions

[ ] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
[x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF], [DEP] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring, deprecation or governance.

I just realized, there was a small typo: if -> If. I can create an issue for the other Keras-based classifiers.
Also, pyright (and I guess also mypy) do not understand that one can use keras.metrics.Metric objects, since metric is initialized to "accuray" which is a str. However, I didn't want to add a type hint because it seems they are not used much in aeon.

@dschrempf feel free to open an issue. We want to incorporate type hints but theres a lot to go through so they are not required for now. Always welcome though and we can help if they include optional dependencies i.e, tensorflow


Feature/update credits

also added unusualcrow's "urgency" soundtrack

When/ Why are the tests failing? :(


doc: improve issue template

Checklist

[x] documentation is changed or added
[x] commit message follows commit guidelines

Description of change
Improve the issue template:

suggest using --verbose npm switch when providing logs
better direct users to use backticks correctly.

I don't have write access here, can someone land this please?

CI: https://ci.nodejs.org/view/All/job/nodegyp-test-pull-request/113/ :heavy_check_mark:
Landed in https://github.com/nodejs/node-gyp/commit/53b607426f9e8780e403a82dc5944456f5823336


[4052] Added a new translation and conditional logic to display contributor adherence

Ticket: [4052]
Feature flag: n/a

Purpose
Updates requested by Mark to publicly show contributors take ownership.
Summary of Changes
Add a new translation string. Then updated the logic differentiate between write permission and visibility
Side Effects
None
QA Notes
Verify that a user with write permissions sees the add icon, question mark and instructions on how to link a DOI and the adherence text.
Verify that a user without write permissions only sees the adherence text.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5567

Relevant Lines:
7532

💛  - Coveralls


Create creature models.

Need to ensure Templates, Monsters and Races are all accounted for in the Scema


batshit depends on batshit-devtools

Is it necessary for batshit to depend on the devtools package?

It just imports a type from it so it wont increase bundle size or anything if that is what you are worried about. Since its a monorepo I could probably make it so its not a dependency, but I dont think its really an issue.


java-linkedlist

Created linkedlist in java

Following naming guidelines are important to maintain consistency.😊
Thanks for contributing


Emoji in Ubuntu shell are not deleted correctly

Environment
Windows build number: 10.0.18362.0
Windows Terminal version (if applicable): 1.0.1401.0

Any other software?

Steps to reproduce

Open an Ubuntu/bash window
Note the position of the cursor.
Type the word "git"
Open the Windows emoji picker with Win + period and choose an emoji. For example, the "smiling face with smiling eyes" emoji (😊) or any other emoji that isn't part of the original windows black-and-white emoji set. For contrast, the recycle (♻) emoji does not show this problem because it's part of that set.
Press backspace once to delete the emoji. Notice that the emoji is still visible but the cursor is in the middle of it.
Press backspace again to "finish" deleting the emoji. The original text "git" will still be visible.
Press Enter.

Expected behavior
The git program runs outputting information about subcommands.
If I delete an emoji character that takes up two blocks of space, I expect the cursor to go backward two spaces, and I expect that pressing Enter will execute the command that is displayed.
Actual behavior
The command prompt complains "gi: command not found", even though the line above still says "git"

Deleting double-width emoji will move the cursor back only one space, and the displayed command is be the command that will be executed.

Hey lemme real quick cross link this with discussion over in #6555, which is seeing some similar disagreement in character width

@guilhermetod Heh, you're right, it seems the Windows Terminal came with a powerline out of the box, and it DOES have some issues... but that's a red herring :) I only intended to show you the command itself in line 1 and the output in line 2.

It looks like we root-caused this in /dup #6555, so I'm gonna close this one out. Thanks everyone for reporting!


Improving our Build/CI times

There's a pretty good blog post about improving Rust build times https://matklad.github.io/2021/09/04/fast-rust-builds.html. After the work in #2340, I figured it was worth finding and taking a look at again to see if there's anything about improving CI times.
The answer: Yes, it does. I've summarized some of the easier parts in the post, as well as some bits that it doesn't cover, but that I'm familiar with due to my time trying to optimize the build of various projects (which I've done a lot)
Low Hanging Fruit
Here are some of the easiest. See the blog I linked  which ah CI focused things, which are very actionable. Most of these are focused on making the GitHub Actions Cache more effective, since by default it's is not great for Rust.

Prior to uploading the cache, delete artifacts which cannot be meaningfully cached from the target folder. Specifically: all of the builds of our own crates, because CI will just rebuild anyway. This is the trickiest thing, but also pretty important.
The blog post referenced https://github.com/Swatinem/rust-cache (although I'm unsure we want to add a GHA tool), and https://github.com/rust-analyzer/rust-analyzer/blob/94d9fc2a28ea5d97e3a9293b9dac05bdb00304cc/xtask/src/pre_cache.rs#L30-L53, but note that we need to modify the approach to work in our repo[^modify].

Disable debug info in CI, because it's enormous (which bloats the GHA cache), and not useful in CI. We should just set RUSTFLAGS="-Cdebuginfo=0" in CI (and only in CI).

Disable incremental builds in CI. That is, set CARGO_INCREMENTAL=0 in the CI environment — incremental builds are faster if you have a full build and only a single changed file, but slower for full rebuilds. Also, incremental builds prevent the use of something like sccache (more on it below)

[^modify]: For example: ather than "./crates/" we would need to check "./implementations/rust/ockam/".
Beyond that, there's a lot of good information about how to cut up your dependency tree, and making sure dependencies aren't  bottlenecks by using cargo +nightly build -Ztimings, which is a build-time profiler[^ztimings].
This is definitely something we need to be using eventually, even if not for CI. That is, at some point we need to be checking what the build times for the code that uses ockam is. (This can be measured by using -Ztimings on some of the examples)
[^ztimings]: This requires nightly, but is built into cargo. It spits out a pretty HTML graph — I've spent a lot of time with this, so I can probably answer questions anybody has about it.
Some additional notes
Largely speaking this is not stuff from that blog, and only the first of them is focused on CI.

We use a docker image, so we should probably set up sccache on it if we have not already, as the largest downside to using sccache is that cargo install sccache is about as slow as you expect.
This can help a lot, but requires we have our caching set up for the sccache directory (~/.caches/sccache on linux, I think?), and a few env vars (RUSTC_WRAPPER=sccache and CARGO_INCREMENTAL=0, although step 3 above covers this). Often you want to set SCCACHE_CACHE_SIZE to something custom too, but I don't know what a good value for GHA (or ockam) is.
This can alleviate the need to cache the target directory too, and vice versa, although it's worth noting that sccache is a more robust cache — cargo will sometimes discard build artifacts which are sill needed.

The same author has a separate post on being careful about not making your examples and integration tests eat the entire build time: https://matklad.github.io/2021/02/27/delete-cargo-integration-tests.html.
This is something that is very easy to do accidentally — I ended up writing a guide on avoiding it at a previous job: https://github.com/mozilla/application-services/blob/main/docs/design/test-faster.md... It's a bit of a footgun. I don't think we have any cases that are too bad here, but haven't really looked.

Investigate a "workspace-hack" (For example rustc's, firefox's...), which can help build times out enormously.
I've never seen one of these on a project that is published to crates.io (because you don't want to have foo-workspace-hack in users dependency graph, but supposedly https://crates.io/crates/cargo-hakari solves this issue, and allows them to be used for a case like ours. This would be great, but needs investigation.

https://github.com/ockam-network/ockam/pull/2410 disables debuginfo and incremental builds in CI

More good info on improving the build: https://fasterthanli.me/articles/why-is-my-rust-build-so-slow
I'll... need some time to go through that one. fasterthanlime's blogs are always full of great info, but quite long.


[Delete Event] The administrator can delete not all past events

Environment:
Windows 10 Pro, Chrome. Версия 109.0.5414.120 (64 бит)
Reproducible:
sometimes
Build found:
31.01.2023
Steps to reproduce

Go to https://www.testgreencity.ga/GreenCityClient/#/
The Admin is logged (Login<EMAIL_ADDRESS>Password<EMAIL_ADDRESS>Go to 'Events'
Find the past event.
Click  'More' button, which is placed on your event.
Click 'Delete'button
Click on 'Yes' button in pop-up

Actual result
The "Delete" button is not in every past event
Expected result
All data about the past event are delete
User story and test case links
E.g.: "User story #3196
[Test case] https://jira.softserve.academy/secure/enav/#/49224
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type ("UI, "Functional"), "API" (for back-end bugs).

cannot reproduce


Fix of a bug in bind() method of XWalkChannel

crosswalk.js is injected only when the thread argument is nil.

@jondong , PTAL

@halton, please grant @jondong push permission, thanks.

invitation to @jondong sent when this repo is setup, please accept.


Add requirements to cargo_test.

This extends the #[cargo_test] attribute to support some additional requirements to control whether or not a test can run. The motivation for this is:

Can more clearly see when a test is disabled when it prints "ignored"
Can more easily scan for disabled tests when I do version bumps to see which ones should be enabled on stable (to pass on CI).

The form is a comma separated list of requirements, and if they don't pass the test is ignored.  The requirements can be:

nightly — The test only runs on nightly.
>=1.55 — The test only runs on rustc with the given version or newer.
requires_git — Requires the given command to be installed.  Can be things like requires_rustfmt or requires_hg, etc.

This also enforces that the author must write a reason why it is ignored (for some of the requirements) so that when I look for tests to update, I know why it is disabled.
This also removes the CARGO_TEST_DISABLE_GIT_CLI option, which appears to no longer be necessary since we have migrated to GitHub Actions.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

I'm opening this up as a draft to get some feedback to see if this is a worthwhile direction to go. This is something I've wanted for a while, as it would help understand which tests are disabled. One issue is that there is some code duplication in cargo-test-macro that is hard to share (would need to create yet another crate to hold that shared code and keep build times down).
Ideally there would be a better way to have runtime test skipping (as described in https://internals.rust-lang.org/t/pre-rfc-skippable-tests/14611).

I think this is a fine direction to go in. If possible I'd ideally like to leverage code generation to share code where the checks are in the generated code rather than the macro itself (so the macro doesn't have to do things like spawn rustc and such). This is generating a literal #[ignore] but is that necessary? Presumably you'd search for ignored tests with a regex rather than running tests with --ignored?

:umbrella: The latest upstream changes (presumably #9889) made this pull request unmergeable. Please resolve the merge conflicts.

As @ehuss said, the generation of #[ignored] can help us be more confident in which tests have been run and which haven't. I like the explicit, especially when tracing old logs (though I doubt the frequency of this scenario).
Yet, is it a problem that the tests and #[cargo_test] macro generation are run under different arch?

@epage or @weihanglo, I'd like to move forward with this. Would either of you be willing to review this?
I updated it for the latest master.  There are also some docs in the contrib guide which explain what has changed.

I would take a look then. Thank you for updating this!

@bors r+

:pushpin: Commit 8e35e2f044a8a042f9f2eedc35cb6db4649533c9 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 8e35e2f044a8a042f9f2eedc35cb6db4649533c9 with merge 2e35678c3967f500ff885df54ceff0cd439e2687...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 2e35678c3967f500ff885df54ceff0cd439e2687 to master...


[BUG] (.NET tool flashes many windows when checking for updates)

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 tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
When the .NET Tool is enabled in the preferences. Every time WingetUI checks for updates it flashes four command windows with the title "%localappdata%\Fork\Update.exe". This only happens when fork is installed.
Stopping WingetUI or disabling the .NET Tool stops this behavior.
Steps to reproduce the issue

Install fork
Start WingetUI with the .NET Tool enabled.
Wait for an update check
See flashing windows.

WingetUI Log
https://gist.github.com/miffe/9c8250db81b5e1f76dad973f09b3247e

Package Managers Logs
https://gist.github.com/miffe/8634566f4c650b2ac09e756fbef631e7

Relevant information
WingetUI 2.2.0
Fork <IP_ADDRESS>
Fork issue: fork-dev/TrackerWin#2134
Screenshots and videos

Is fork's update.exe added to the path?
Then the issue is actually with the PowerShell module calling the command update to refresh its sources.

It is a known bug that has been fixed in source code, but that has not been released yet.

Yes, forks update.exe is in path.

Please try temporarily removing it from path. The issue should not be reproducable then


Fix header and footer links

Overview
A few header/footer links are broken. This PR fixes them:

Technology
Terms of Service
Donate

Connects #2288
Notes
The submenu button-type link was switched from react-router <Link>. If a menu button is added that should be an internal link, this will need to be updated, A possible solution is pass down an external boolean like how the main header buttons work.
Testing Instructions

localhost:6543
Header > How It Works > The Technology > Overview
Footer > Terms of Service
Header & Footer > Donate

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
[ ] This PR is targeted at the correct branch (develop vs. ogr/develop)
[ ] If this PR applies to both OAR and OGR a companion PR has been created

I see the donate links currently working in OS Hub staging and production. If that was the only motivation for changing from Link to a then we may not need that change.

I see the donate links currently working in OS Hub staging and production. If that was the only motivation for changing from Link to a then we may not need that change.

See #2290

See above note about the 'technology' link - otherwise, this is working well!

b786fad
Also, I noticed that sometimes the social media links wouldn't be clickable. It turns out this was from the footer shape blocking them. 89f3cd1 sets pointer-events: none; on the footer shape.


Content Suggestion: Bitcoin Takeover Podcast

Link to Content: https://bitcoin-takeover.com/
Briefly Describe Content: "Listen to Season 4 of The Bitcoin Takeover Podcast: 10 interviews with hardware wallet makers (Ledger, KeepKey, BitBox, Trezor) & their breakers."
Suggested Section: https://bitcoin-only.com/#podcasts

Closing this suggestion because there's one issue opened already for Bitcoin Takeover Podcast https://github.com/bitcoin-only/bitcoin-only/issues/110


Makefile: fix warning

Overriding the /promu target causes a warning during build.
This updates the build to fix the warning and first build promu by
using the go compiler directly, and then using the just built promu
to rebuild itself.

Makefile:17: warning: overriding recipe for target '/home/pgier/go/bin/promu'
Makefile.common:236: warning: ignoring old recipe for target '/home/pgier/go/bin/promu'

Using the promu target from Makefile.common would use the downloaded promu to run promu build.  I thought it would be better to use the locally built promu to rebuild itself.
Maybe Makefile.common should have a common-promu target to allow overriding?

On second though, this PR is fine as it is. :+1:


Tag release for 4.0.0

It would be useful to have an official 4.0.0 tag for the data as well.

done


chore(main): release 1.6.0

:robot: I have created a release beep boop
1.6.0 (2024-06-12)
Features

style: add tailwind css and some good styles to get started (#36) (978870b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aimeerivers/watchthis-user-service/releases/tag/v1.6.0 :sunflower:


[Develop] Split mina_graphql.ml to a set of files

Merge to develop of #13956

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


feat(RepresentationTheory/GroupCohomology/LowDegree): Identify groupCohomology A nwith Hn A for n = 0, 1, 2

This PR gives isomorphisms between groupCohomology A n (defined as the nth cohomology of the complex of inhomogeneous cochains of A, whose nth object is (Fin n → G) → A) and Hn A (defined more minimally and explicitly) for n = 0, 1, 2. We do the same for cocycles.

I think that the API of homological complexes has to be developed a little bit more in order to ease the constructions here. I have started experimenting on another branch https://github.com/leanprover-community/mathlib4/tree/HnIsoGroupCohomology_jriou ; see commit https://github.com/leanprover-community/mathlib4/commit/e76c697f92964c85244bb14f98c48efaf91fd4b7 ) I will try to see if this API addition is exactly what we need to improve this computation (and future ones!), and I will PR the missing API as soon as possible...

In my branch HnIsoGroupCohomology_jriou (see diff https://github.com/leanprover-community/mathlib4/compare/HnIsoGroupCohomology...HnIsoGroupCohomology_jriou ) I have added the suitable homology API. In order to use the API, the design suggestion I make is that the comparison iso between the complexes, e.g. shortComplexH1Iso should be (inhomogeneousCochains A).sc' 0 1 2 ≅ shortComplexH1 A (with nice definitional properties) rather than (inhomogeneousCochains A).sc 1 ≅ shortComplexH1 A (which involves the ComplexShape.prev/next which is more opaque). Then, with the appropriate general API which relates the homology/cycles/opcycles with that of HomologicalComplex.sc' 0 1 2, most of proofs can be done by simp.
Similarly, for zero cocycles, I am not sure we need to introduce an extra definition of zeroCocycles because this identifies to H1 (i.e. invariants). Then, in order to have a quite similar design as above, as there is only a kernel in degree 0, I suggest that the iso of short complexes which was necessary in degrees > 0 be replaced by just an Arrow isomorphism. The equational lemmas are easily obtained.
(Before this experiment with group cohomology, I was not completely sure about the exact API that might be needed in order to handle this smoothly. I think that this is the right design.)
I will PR the necessary general homological complex API in a separate PR, so that this one can depend on it.

Thank you so much! I've merged that branch into mine, I hope that's wise? I'm still not confident with git.

Thanks!
bors merge


Dockerfile and Readme update

Pull Request Review
Checklist

[x] Issue: #142
[x] List of related PRs. : None
[x] Project builds without any errors.
[x] Unit Tests updated (or created where needed) and all pass.
[x] You have ran manual functional “smoke” test (does product as a whole still work?).
[x] User Story meets the acceptance criteria - and passes. acceptance criteria should be understood at outset.
[x] There is a PR for update of internal documentation for the affected Project(s) - README etc.
[x]  Description of the changes:  #142

Hi @chowarth123 @dvaccarosenna , done the suggested changes, could you please approve this PR.
Thank you,
Loka

@lokanalla @manojn97 the issue and PR title and description are a bit unclear to me. What is this PR trying to achieve? "Dockerfile and Readme update" could be for any number of reasons. Can we rename the issue to highlight the feature being implemented here. Is it "Support IPv6"?

@dvaccarosenna I have made the above suggested changes to the PR name


1349 Added Care Evolution sender and schema

This PR adds Care Evolution as a sender.
Changes

Added Care Evolution as a sender
Create schema for Care Evolution

Checklist
Testing

[x] Tested locally?
[ ] Ran quickTest all?
[ ] Ran ./prime test against local Docker ReportStream container?
[ ] Downloaded a file from http://localhost:7071/api/download?
[ ] Added tests?

Security

[ ] Did you check for sensitive data, and remove any?
[ ] Does logging contain sensitive data?
[ ] Are additional approvals needed for this change?
[ ] Are there potential vulnerabilities or licensing issues with any new dependencies introduced?

Process

[ ] Includes a summary of what a code reviewer should verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Security
List potential security threats and mitigations in the PR
Fixes
List GitHub issues this PR fixes

#issue

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Carlos,
Are you able to create fake data using the new schema?
Also, note my change needed for organizations-test.yml

Yes, I can create fake data with no issues.  The extends works if that is what you are asking.


CentOS 7, line 15, information, command not found.

LSB Version:	:core-4.1-amd64:core-4.1-noarch
Distributor ID:	CentOS
Description:	CentOS Linux release 7.9.2009 (Core)
Release:	7.9.2009
Codename:	Core
$ sudo wget https://raw.githubusercontent.com/rubo77/log4j_checker_beta/main/log4j_checker_beta.sh -q -O - |bash
bash: line 15: information: command not found

@rubo77 it uses information through-out the script, I would expect this to be echo or a function definition in the beginning of the shell script ?

So what is the problem?

@rubo77 thanks!

Many thanks


[BUG] heal

txAdmin/FXServer versions:
v4.14.2 atop FXServer 5652
Describe the bug
Healing yourself or the whole Server isn’t possible as the Heal Option on the Ingame Menu does nothing
To Reproduce
Steps to reproduce the behavior:
Use the In-Game Menu and try healing yourself with the heal option
Expected behavior
You get full health
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here, like for example if it's a server issue, which OS is fxserver hosted on.

Do you have any more information?
Works for everybody else... you might be using s framework or anticheat that is not dealing well with the txAdmin heal event.


Auto-buyer removed?

Hi there,
Why was auto buyer removed? Are we expecting a new version?
Using the old one, with the improvement suggested in the PR, I got some nice hits yesterday, but none since (a few hours running, with smaller delay - 3-5 seconds each).
Is it broken?
Thanks for all your efforts.
Is there any place we can chat?

Here is a part of the log:

It has been moved to a separate branch because, after some thought, an autobuyer isn't the main purpose I wanted to pursue with this project. I just wanted to show how easy it is to create, so EA can fix it for the future, so the game is more enjoyable for everyone.
The goal of the project is to enhance the FUT web app experience to make it more enjoyable and user friendly.
Chatting is possible with the gitter button on the readme.

The link to install the script is down

That's correct. You can still test the code by viewing the raw file here on github.
I've removed the script from Openuserjs.com for the same reason as stated above.

Hi Mardaneus86,
Where can I find this raw file that you mentioned? Thank you

Check the branch list ;-) It's fairly obvious. Remember that you might get banned fairly easily by using it.

@Mardaneus86 Thanks. I get that. Just had an idea and wanted to look at how you were doing things :)
What about other features of the scripts...has anybody reported being banned for using futbin prices script or so?

No mentions of it yet. I'm incorporating pinEvents for most scripts as well. Will be finished with that soon; so there is no way to tell if it's automated or not.

That would be great. Looking forward to further updates :+1:

@Mardaneus86 Good job here, this is exactly what I was looking for. A question: I was trying to find the raw file of the Autobuyer script, but I can't find it even in the related branch. Is it just me or it has been removed?

It wasn't you, it's available again :-) made a merge mistake


dbeaver/pro#2336 Move MSSQL Kerberos to PRO

Verified

For the benefit of other users who find this MR wanting to use Kerberos, the MSSQL JDBC driver directly supports Kerberos auth. See https://learn.microsoft.com/en-us/sql/connect/jdbc/using-kerberos-integrated-authentication-to-connect-to-sql-server?view=sql-server-ver16
I can confirm this works in dbeaver 24.0.3.


Using deprecated listeners in 4.0.0

Calling removeListener on DeviceEventEmitter causes warnings similar to #412.
Additionally, DeviceEventEmitter is already deprecated and it might be removed at any point of time.
This is happening when calling release on the engine.

This is not happening when using registerEventHandler.