Migrate images.githubusercontent.com entries in Collective table

Describe the bug
Many entries in the Collective table (401 total) point to addresses on the images.githubusercontent.com domain.
All these images are broken at the moment. We need to migrate them to a new scheme.
Fastly error: unknown domain: images.githubusercontent.com. Please check that this domain has been added to a service.

That was fixed with scripts.
https://github.com/opencollective/opencollective-api/blob/master/scripts/collective-image-check.js
https://github.com/opencollective/opencollective-api/blob/master/scripts/collective-image-update.js


refactor!: rework extensions interface

CodegenExtension is replaced with a new trait of the same name, that instead of defining extension behaviour provides a convenient interface for registering a set of extensions.
Extensions are now registered via methods on CodegenExtsBuilder, each method takes a callback that is stored in the builder. The builder is consumed by finish which returns a collection of "Callback Maps". One of the big wins here is that the "Type Callback Map" does not depend on a H: HugrView parameter.
The "root" &'c Context now lives in EmitModuleContext, as well as in TypingSession.
TypeConverter is now the "Type Callback Map". This is implemented using the new src/utils/type_map.rs, which we intend to leverage for computing "def-hooks" and "destructors" for types.
We go to some effort to ensure the callbacks are not required to have 'static lifetimes, nothing inside crate::custom assumes or requires this. However, we do in several places (e.g. TypingSession, EmitModuleContext) fix these lifetimes to 'static, in lieu of adding additional lifetime parameters to these types.
We have removed the guppy.rs test case, which was ignored, rather than rework it to use the new interface.
Otherwise, test cases are largely unchanged, and there are no snapshot changes.
All extensions have been changed to use the new interface, which is largely just moving code around. In some cases error messages are consolidated by the new tools providing more appropriate inputs to extension points. See for example CodegenExtsBuilder::custom_const.
At several points we mark TODO refactorings that will be carried out in the near future.
BREAKING CHANGE: extensions interface in src/custom.rs entirely reworked.

Codecov Report
Attention: Patch coverage is 82.92108% with 145 lines in your changes missing coverage. Please review.

Project coverage is 85.24%. Comparing base (7d91295) to head (f7355ea).

Files with missing lines
Patch %
Lines

src/custom/int.rs
52.17%
40 Missing and 4 partials :warning:

src/custom/rotation.rs
80.00%
2 Missing and 18 partials :warning:

src/custom/conversions.rs
86.55%
7 Missing and 9 partials :warning:

src/custom/logic.rs
65.95%
4 Missing and 12 partials :warning:

src/custom/prelude.rs
85.32%
1 Missing and 15 partials :warning:

src/utils/type_map.rs
80.59%
9 Missing and 4 partials :warning:

src/custom/float.rs
88.05%
2 Missing and 6 partials :warning:

src/custom/load_constant.rs
71.42%
4 Missing and 4 partials :warning:

src/custom/extension_op.rs
94.59%
1 Missing and 1 partial :warning:

src/test.rs
90.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   83.42%   85.24%   +1.81%
==========================================
  Files          27       31       +4
  Lines        4164     4187      +23
  Branches     4164     4187      +23
==========================================
+ Hits         3474     3569      +95
+ Misses        368      297      -71
+ Partials      322      321       -1

Flag
Coverage Δ

rust
85.24% <82.92%> (+1.81%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


v0.1.0

Fix MicroPython compatibility
Update examples:

Add titles to file names to make it clear what each example is for
Make dedicated example for single, dual, and quad relays, plus PWM example

Fixes #3
Fixes #4

@gigapod Do we also want to address #2 in this? Also considering adding toggle support.


Flaky Toyota EV detection

Describe the bug
I have had multiple CAN errors after updating to 0.8.13.
Dongle ID: 69567f65ae707320
Here is a drive with a CAN error: 69567f65ae707320|2022-02-20--19-21-49--0
What hardware does this issue affect?
comma two
Provide a route where the issue occurs
69567f65ae707320|2022-02-20--19-21-49--0
openpilot version
0.8.13
Additional info
No response

I have also been getting intermittent CAN errors after updating to 0.8.13 in my 2021 RAV4H (comma 2). It happens intermittently and usually resolves itself after restarting the car. I swapped out my OBD-C cable and checked for stock passthrough to ensure it wasn't a harness issue.
Route with the CAN Error:
5ab891a2663f10be|2022-02-19--21-29-42--0
Route without the CAN Error:
5ab891a2663f10be|2022-02-19--21-08-55--0
Additional RAV4 owners reporting the same issue after updating: https://discord.com/channels/469524606043160576/524592892627517450/945574732634402826
https://discord.com/channels/469524606043160576/524327905937850394/944968334179459122
Let me know if I can provide any additional info.

Here is another drive that didn't throw a CAN error, but a communication malfunction error:
69567f65ae707320|2022-02-21--22-59-23--9
Another drive about 15 minutes after the one above that was throwing a CAN error (I think, could've been comm malfunction). In this same drive, I pull over, restart my car, and OP operates normally and I turn it on a few times during the route:
69567f65ae707320|2022-02-21--23-22-45--0

Here is a normal drive with no errors and OP engaged: 69567f65ae707320|2022-02-22--21-01-29--0

@JamesL623 @connorlarkin1 can you guys try #23825? It seems that there's a not insignificant delay until your gas pedals start sending messages, which makes the EV detection unreliable.

Sure thing. May be able to test tonight, but if not, I will get you some routes tomorrow

Merged into master since the tests passed. We'll push this to the 0.8.13 release once you guys confirm it's reliable now.

Drive 1: OP worked perfectly
69567f65ae707320|2022-02-22--23-55-25--0
Drive 2: CAN Error
69567f65ae707320|2022-02-23--00-07-28--0
Drive 3: OP worked perfectly
69567f65ae707320|2022-02-23--00-08-58--0
Please check to make sure I have the correct changes in my interface and values files. I merged the code over myself and want to be sure there wasn't any errors

@JamesL623 If you're unable to checkout the branch manually, I recommend uninstalling and reinstalling with this URL installer.comma.ai/commaai/master-ci. I'm seeing no git diff on those routes.

Darn, forgot to compile after saving the changes to the files.
I’ll get it up and running tomorrow and send the routes to you.
On Tue, Feb 22, 2022 at 7:40 PM Adeeb Shihadeh @.***>
wrote:

@JamesL623 https://github.com/JamesL623 If you're unable to checkout
the branch manually, I recommend uninstalling and reinstalling with this
URL installer.comma.ai/commaai/master-ci. I'm seeing no git diff on those
routes.
—
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/23812#issuecomment-1048347109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVC4YFXAEVRVCNFIH4HGBS3U4QUHXANCNFSM5O6Y72IA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

The issue is resolved for me. Drove around on master-ci and restarted my car a bunch and couldn't reproduce the CAN error message.
Routes on master-ci:
5ab891a2663f10be|2022-02-22--19-26-24
5ab891a2663f10be|2022-02-22--19-38-21
5ab891a2663f10be|2022-02-22--19-47-51
5ab891a2663f10be|2022-02-22--19-51-24
Thanks @adeebshihadeh for the quick fix!

Thanks for testing @connorlarkin1! We'll get the fixed pushed out to release tomorrow after a bit more time on master.

Just go the update pushed over and will be going for a drive later. Will post routes in here when uploaded


Fix automatic generation of Zenodo DOI

After the last release of Datasets (1.8.0), the automatic generation of the Zenodo DOI failed: it appears in yellow as "Received", instead of in green as "Published".
I have contacted Zenodo support to fix this issue.
TODO:

[ ] Check with Zenodo to fix the issue
[ ] Check BibTeX entry is right

I have received a reply from Zenodo support:

We are currently investigating and fixing this issue related to GitHub releases. As soon as we have solved it we will reach back to you.

Other repo maintainers had the same problem with Zenodo.
There is an open issue on their GitHub repo: zenodo/zenodo#2181

I have received the following request from Zenodo support:

Could you send us the link to the repository as well as the release tag?

My reply:

Sure, here it is:

Link to the repository: https://github.com/huggingface/datasets
Link to the repository at the release tag: https://github.com/huggingface/datasets/releases/tag/1.8.0
Release tag: 1.8.0

Zenodo issue has been fixed. The 1.8.0 release DOI can be found here: https://zenodo.org/record/4946100#.YMd6vKj7RPY


new message in VSCode

VS Code Version: 1.84.1
OS Version: Sonoma 14.1.1

When I launch visual code, I have a message (cf picture
)
what should I do to make it go away ?
Thx

and what should I do ?

Remove those settings.

where is it ? (I am sorry I am a new user)

Open settings UI from here:

To open User settings.json do this:

To open workspace settings.json do this:

I found   "python.linting.enabled": true,
should I delete this line ?

Yes.

@Jacques2101 The Output > Python logs will have details on any other settings that we have deprecated.

@Jacques2101 No worries. We are happy to help.

User settings applies to all instances of VS Code. Workspace settings can be used to override User settings based on per project needs.


Could you add the option to play music and video?

i would like to participate.

If i am in i will do it by tomorrow.

You are in @GoldAndNitrogen

how to add file?

Please assign me, I want to do it.


Emojis

I see lines jumping when using Emojis with this theme. Nvim is then not useable.
I could not reproduce it in Gnome Terminal, but only in Kitty.
System information

OS: Linux
Neovim version: 0.7.2
onedarkpro config: minimal.lua
Terminal: Kitty

To reproduce
Steps to reproduce the behavior:

Download the Markdown file from https://codeberg.org/Mo8it/How_To_Linux/raw/commit/6aff23c48868e4512f954588a4f8e4a2571cebc6/src/day_1/terminal_basics.md
Open a Kitty terminal
Open the Markdown file with nvim --clean -u minimal.lua terminal_basics.md
Scroll through the file and see lines jumping.

This theme https://github.com/navarasu/onedark.nvim seems to have this issue, too.
This is the issue:
https://github.com/kovidgoyal/kitty/issues/3648
It seems like it is a Kitty or Neovim issue, not from the theme.


docs: Add IntelliJ documentation for generated source code

Which issue does this PR close?

N/A.
Rationale for this change

Update docs.
What changes are included in this PR?
Just docs.

How are these changes tested?

N/A.

Thanks @mbutrovich @kazuyukitanimura


update(save): add js action info context parameters to api call

Goal
Fix #139
The purpose of this PR is to make sure that the analytics context data in the spec https://docs.google.com/spreadsheets/d/15YIc9n0eqOA9soD_NleEqED49Tqy5p4S9a5IHDN8BeI/edit#gid=0 is being passed along in the js api calls.
Todos:

[x] logged_in and authed_app automatically achieved by oauth backend
[x] add
[x] delete
[x] archive

Implementation Decisions
Update api action handlers to accept the context parameters and add them in the api call.
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@collectedmind I merged this by accident without a review, so I reverted and re-opened


adding convert PhaseEncodingDirection from xyz to ijk convention

Thanks @giulia-berto!
Is it possible to add a check so that the image is in LAS when we do this conversion? If not, we need to apply different mappings, probably see https://github.com/nipreps/fmriprep/issues/2341#issuecomment-737431340
Also, please add some warning messages that we are doing this conversion.. I think the user should know about it?

I thought about it, but I couldn't find the orientation in the config.json file...should we read it from the nii files?
There's already the print("convert PhaseEncodingDirection from xyz to ijk convention") if that's what you mean.

I think we can do something like this

I made an update. I think we should check for "any A orientation" as effigies said, so if 'A' in codes.
Another problem is from which image we should read such orientation? The t1 might not be in the dataset.

I believe the affine code should come from the .json's corresponding .nii.gz (the. nii.gz of the object itself).
Also, I believe what Chris means is that, if we are converting "y" to "j", then I need to check the second item in the code (P v.s A). If we are converting "x" to "I", then we should check if the first item in the core is L or R. If we are converting "z" to "k", then we should check if the 3rd item in the code is I or S. My example code does that basically.

Thanks for the update @giulia-berto I think the code still doesn't properly handle cases where voxels are stored in odd strides like ('A', 'R', 'S'), or ('S', 'A', 'R'), etc.. Chris says that it should be very rare, so maybe we don't need to worry about it, but we need to remember this.

Yes, I also think that it's not handling correctly those cases, and I'd have done something like this to flip the polarity:
inv = pe[1:] == "-"
if codes[axis] in ('L', 'P', 'I'):
    inv = not inv
if inv:
    updated_pe += "-"

However it can be that I misunderstood all this flipping thing, so feel free to change that function.


Test sdk stage2

Testing in sdk traits and classes

Invalid due to compilation issues. Solved in #1179


Kafka spout failure config

This pull request offers plugable MessageFailureHandler for failed message.
Solution structure.
PartitionManger uses instance of class that implements IMassageFailureHandler interface for failure handling.
For instance creation it uses factory object stored in MessageFailureHandlerFactoryRepository.
This request provides 2 implementations for IMessageFailureHanlder:

DefaultFailureHandler repeats current KafkaSpout behaviour
ConfigurableRetriesFailureHandler enables possibility to configure number of retires per message.

There is also one implementation of IMessageFailureHandlerFactory: MessageFailureHandlerDefaultFactory that provides possibility to create either  DefaultFailureHandler or ConfigurableRetriesFailureHandler.
By default, if nobody filled MessageFailureHandlerFactoryRepository, it returns MessageFailureHandlerDefaultFactory object that creates DefaultFailureHandler object.

@ogorun There is work in progress to refactor kafka code at STORM-631. Can you please add your requirements/issue there so that we can fix it as part of STORM-631. Thanks.

Thank you for feedback. I'll look at it

Hi,
Thanks for pointing out your PR. This looks like it is fairly similar to https://github.com/apache/storm/pull/254, which has been merged recently.

@wurstmeister ,
Thank you for pointing me to https://github.com/apache/storm/pull/254  .
It is definitely progress in direction of failure handling functionality encapsulation. But there are two points that I'would solve another way.

I feel plugable failure handler (with reasonable default) is needed. For example, I'd rather have one point in topology (spout) that decides when to stop  reprocessing the same message (for example, after configurable number of retries) than checking all possible exceptions are handled properly in every bolt.

fail() event is divided into two parts: one part is handled in PartitionManager class and another one in FailedMsgRetryManager. I don't see cause for such division.

Hi @ogorun, Could you file a JIRA issue for this pull request here?

Hi @d2r ,
Is it better to open separate JIRA issue or this pull request is considered as part of subjects discussed in  STORM-650?

Is it better to open separate JIRA issue or this pull request is considered as part of subjects discussed in STORM-650?

@ogorun  I am sorry I did not see the notification for you comment earlier.  Yes, I think the place to discuss this is STORM-650.  We might ask on STORM-650 whether we should add a new JIRA Issue for this to the STORM-650 epic.  We might ask on STORM-650 whether we should add a new JIRA Issue for this to the STORM-650 epic.  So far, I am not sure this specific issue is addressed in STORM-650, and it seems some kafka API changes are coming as well.

@ogorun, how should we move forward with this work?  Has this issue been resolved as part of STORM-650, or does it still need to be addressed?

Closing. Feel free to reopen with updates.


How a user can change the value of a FlowFilter

You write

In FlowFilter fields, a user can enter a filter value via the user interface in the application that will affect the calculation of a FlowField.

How a user can do this in the business central web client?

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

ID: 3ea41599-f702-15a8-c9dd-13a41d6ab8df
Version Independent ID: 27f4af23-40fa-8f99-d1e8-28e81692715d
Content: FlowFilter Overview - Business Central
Content Source: dev-itpro/developer/devenv-flowfilter-overview.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for your feedback. You should be able to use the filtering described here: Entering Criteria in Filters. Thanks.

This is not an answer to my question. FlowFilters are not visible Fields that takes effect on how FlowFields are calculated.
For example the "Date Filter" in the Item table has effect on how the "Inventory" Field is calculated.
FlowFilters are not visible columns, so the normal way to filter on columns is not possible.
Can you please provide screenshots how the user can specify the "Date Filter" in the "Item List"

You are in the developer documentation, please go to the end-user documentation here: Dynamics 365 Business Central - there you will also be able to provide feedback through GitHub. Thanks.


Fixes #26

The plan is to merge this when certain experiments are done to not bother these.

Feel free to merge these, now.


Document gt4magritte

Our work on gt4magritte is sorely under-documented. It should have at least a section in the Glamorous Toolkit book.
Cheers


Update Salesforce API Version

Salesforce is retiring APIs this year and next year (link), so we're preemptively updating this legacy gem to the earliest version that continues to be supported.

Versions 7.0 through 20.0: Retiring June 1, 2022
Versions 21.0 through 30.0: Retiring June 1, 2023

Missed the environment variable. Closing.


[Automation] Generate Fluent Lite from Swagger advisor#package-2020-01

[Automation] Generate Fluent Lite from Swagger advisor#package-2020-01

API change check
API changes are not detected in this pull request.


Returning data into consistent format

Hello,
Not sure if we can call this "bug" but here is the problem:
If you have a table using "inet" and "timestamp" data types, when query via cassandra the results are returned as :
timestamp - unix timestamp in ms
inet - string, example : "<IP_ADDRESS>"
query same values via ELS:
timestamp - returned as ISO8601
inet - returned as string but appends "/" in front, example: "/127.0.01"
Any way I can do something to have consistent formatting (other than handle-ing this into the code)
Silviu

Hi,
For timestamps, you can set a date format, ISO is the default, see https://www.elastic.co/guide/en/elasticsearch/reference/6.2/mapping-date-format.html#strict-date-time https://www.elastic.co/guide/en/elasticsearch/reference/6.2/mapping-date-format.html#strict-date-time
For Ip address, it’s a bug (InetAddress.toString() instead of InetAddress.getHostAddress()).
This will be fixed in the next release available next week.
Thanks.
Vincent.

Le 25 avr. 2019 à 13:58, Silviu Caragea<EMAIL_ADDRESS>a écrit :
Hello,
Not sure if we can call this "bug" but here is the problem:
If you have a table using "inet" and "timestamp" data types, when query via cassandra the results are returned as :
timestamp - unix timestamp in ms
inet - string, example : "<IP_ADDRESS>"
query same values via ELS:
timestamp - returned as ISO8601
inet - returned as string but appends "/" in front, example: "/127.0.01"
Any way I can do something to have consistent formatting (other than handle-ing this into the code)
Silviu
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/strapdata/elassandra/issues/279, or mute the thread https://github.com/notifications/unsubscribe-auth/ACOMPGLC2IO6ANBRK7CUTWLPSGMIHANCNFSM4HIMN3RA.

Hello, I tried to set it to epoch_millis but has no effect. I created the index as :
curl -XPUT "http://<IP_ADDRESS>:9200/index_name" -H 'Content-Type: application/json' -d '
{
    "settings" : { "keyspace" : "test" },
    "mappings": {
        "users": {
            "_field_names": {"enabled": false},
            "discover": ".*",
            "properties": {
                "created_at":{
                    "index": false,
                    "format" : "epoch_millis"
                }
            }
        }
   }
}'

I'm missing something ?
Silviu

You're right, output format was no use to build the _source document.
This is fixed in the last release <IP_ADDRESS>.
Thanks for reporting the issue.

Hello the bug seems partially fixed.
The example I provided above the timestamp field is now returned in milliseconds but as string:
"created_at": "1539783087034", instead  "created_at":<PHONE_NUMBER>034
Any idea ?
Silviu

Hello @vroyer ,
any thought on this ?
Silviu

Elasticsearch store the source document as you insert it, but elassandra rebuild the source document from the cassandra table, and it use the date format to generate the date representation as string, even when the result date format is a number. You can directly access that field as a long through CQL without using the _source document.

Ok in case nothing can be done here I'll close the issue


Filled color values changing even though the data is not.

When selecting an item via table view selection, sometimes the filledColor value will change even though the data being used to create the values will not change (you can see it being printed to the console in the video).
Library setup method: CocoaPods
Version of the library: 23.0
12.3
iOS 13.0
https://user-images.githubusercontent.com/25822037/122808204-92863e80-d29a-11eb-9117-904c3d3864d8.mov

Hi @willcunane sorry don't understand the problem. Could you please explain a little more?

In the video I select a row from my tableview and sometimes (not every time) the stars in the row selected change the value of active stars. So the Phillips 66 should be 2 stars, in the video after the 2nd click, you can see that the stars go from 2 filled in stars to 5. In the console to the left I have it printing the value of what the stars should be.
The bug could very well be on my end but I tried to troubleshoot for a few hours and couldn't seem to find the issue.

I’m pretty sure the issue is the passing data .
On Jun 22, 2021, at 12:14 AM, Will Cunane @.***> wrote:

In the video I select a row from my tableview and sometimes (not every time) the stars in the row selected change the value of active stars. So the Phillips 66 should be 2 stars, in the video after the 2nd click, you can see that the stars go from 2 filled in stars to 5. In the console to the left I have it printing the value of what the stars should be.
The bug could very well be on my end but I tried to troubleshoot for a few hours and couldn't seem to find the issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/evgenyneu/Cosmos/issues/185#issuecomment-865348683, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHXPK5O3PLBV3D7UKMWFOQDTT6TUXANCNFSM47CCUD3Q.

Ok thanks, I'll check it out again tonight and close the issue if I get it figured out

After testing it multiple times the issue still persists but only on a simulator. When I run it on my device it works with no problems. I'll just leave my code below for reference on how I'm passing the data to Cosmos.
cell.cosmosLabel.rating = Double(data.rating) ?? 0

Thanks for clarifying @willcunane

So the Phillips 66 should be 2 stars, in the video after the 2nd click, you can see that the stars go from 2 filled in stars to 5.

Phillips 66 changed to five stars because you click on it (at 4 sec in the video). You can disable user interactions by
cosmosView.settings.updateOnTouch = false

Let me know if it helps


Update to conform to new patricia API

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:weswigham/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so DO NOT open a new one.

@weswigham just curious; what's the reason for closing this? Is the change incorrect, or not yet needed for the vendored version (currently 1.0.1)?

The current vendored version does not require it, nothing's actually wrong with the code.
This issue arose for me because building a client on windows does not typically use the vendored versions provided - the shortcutted build instructions circulating avoid running the (bash) build environment scripts, and as such, I attempted to install deps with a simple go get.
The root cause, I would say is that the dependency management used in docker, unlike in swarm, is not cross-platform, as godep is. (Once more bringing up #8913)
Seeing as another docker project is using godep, perhaps it's worth revisiting #8913, to reduce the number of hacks in /hack and see a more cross-platform dependency management solution?


fix @doc formatting

Summary of changes
Changed spacing so the doc formatting looks good in mix docs generation.
Checklist

[ ] New functions have typespecs, changed functions were updated
[x] Same for documentation, including moduledocs
[ ] Tests were added or updated to cover changes
[x] Commits were squashed into a single coherent commit

Also, I was wondering the reason for using __MODULE__ in many places. There are inconsistencies in the choice. I thought to change those as well but wanted to make sure that there's a reason behind it

Coverage remained the same at 64.736% when pulling 960de78d12c157a4d88e1ea7dfd229cc9aa8e537 on techgaun:docs-fix into 88a284baa8f79e1185d0130b0ebcbe82f9888deb on bitwalker:master.

I mainly use the macro in places where the module name is long, but there are still legacy instances of it around in places where I wouldn't necessarily use it today. Thanks for the PR!


158784204: Dynamic link is handled each time when installing app

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.2.1
Firebase SDK version: 5.20.2
Firebase Component: DynamicLinks
Component version: 5.20.2
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
Precondition: Application is not installed on device

Click on the dynamic link (dynamic link should redirect you to AppStore).
Install application from AppStore.
Check that dynamic link is handled.
Remove application.
Install application again.
Dynamic link is handled again, although user didn't click on it again.

Relevant Code:
func application(_ app: UIApplication, open url: URL, options: [UIApplication.OpenURLOptionsKey: Any]) -> Bool is called again after reinstalling app. Is there a way to prevent it from handling the dynamic link more than once?

@odarynakorolevych Thanks for the report. Version 5.20.2 is more than a year old and several issues have been fixed since then. Would you pod update and see if the problem is still occurring?

Hi @paulb777,
I've updated the libraries to the latest version and the issue is still reproducible.

Hi
I'm having same issue

Firebase SDK version: 6.18.0

Any solution for this

Hi @pajtimid ,
FDL SDK saves the information regarding whether the link handling is already done on UserDefaults and when you reinstall the app this value is getting cleared.
If the FDL SDK didn't find whether the link handling already done in user defaults, it would check whether there is any pending links available for this device specific to the app by checking the pasteboard and by checking on the server side. Usually this server side information regarding the link click for a specific app for a specific device is kept for 1 hour and so if you try to re-install the app after 1 hour, your app won't get the clicked link.
But Regardless of whether the app get the link or not, the above call back will get called at least once after the app is installed(As mentioned here in Step 7). So what you could do is to check for dynamicLink.url for not nil before calling your handling code.
if let dynamicLink = DynamicLinks.dynamicLinks().dynamicLink(fromCustomSchemeURL: url) {
if(dynamicLink.url != nil){
  // Your handling code here.
}

Hi @eldhosembabu,
The issue is that the link is handled another time, when it shouldn't. The user install application, we handle the deeplink, and after he/she removes the applications, reinstalls and method WITH (dynamicLink.url != nil) is handled again, although that link was already handled successfully.
So you want to say, that if the user waits one hour or more, that link won't be handled again? Why wouldn't you store that value in Keychain or on the Server side to make it not cleared after reinstall?
Thank you for looking into it!

Hi @odarynakorolevych,
The 1 hour can be looked at as the attribution window. Usually users don't install and re-install within this time period. If you're seeing it happening very frequently in app users, with varied uninstall->install time windows, please reach out to support and we can investigate further.
Also to clarify the previous comment, we are not saving anything to keychain.

Closing for staleness. Please comment or open a new issue to reopen.


Add benchmarks for inigo lib

Stack:

#72
#71
#69
#65 ⬅
#68

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #72


feat(config): support global config option for default validation trigger

Fixes #535.
I moved AureliaValidationConfiguration to a separate file and added a method defaultValidationTrigger that users can call to globally configure the default validation trigger like so:
aurelia.use
  .plugin("aurelia-validation", (config: AureliaValidationConfiguration) => {
    config.defaultValidationTrigger(validateTrigger.manual);
  });

I wasn't sure how to get the configured trigger into the ValidationController so I simply opted to register the config instance and inject it.
I considered making AureliaValidationConfiguration.getDefaultValidationTrigger() private to present a cleaner interface to users, but decided against it since this way the call in the ValidationController constructor can be type-checked.

What do you think @bigopon?

@RomkeVdMeulen Big oops, i completely forgot about this, left a couple of comments, other than that, It's looking good. Will do 2nd round of review soon

@bigopon I've renamed the config class. Any other changes you'd like?

@bigopon Anything else?

@RomkeVdMeulen let's go i guess 👍
cc @EisenbergEffect @fkleuver


Port Sameer's PAMI changes for POWER8 to PAMILRTS

Original issue: https://charm.cs.illinois.edu/redmine/issues/1664

src/arch/pami/machine.c contains the following:
#ifdef CMK_BLUEGENEQ
#define SHORT_CUTOFF   128
#define EAGER_CUTOFF   4096
#else
#define SHORT_CUTOFF   1920
#define EAGER_CUTOFF<PHONE_NUMBER>
#endif

An eager cutoff of almost 2GB seems way too large, and pamilrts/machine.c defines them as:
#define SHORT_CUTOFF   128
#define EAGER_CUTOFF   4096

Original date: 2017-08-31 19:21:12

Someone with access to one of the PAMI linux systems needs to test this: https://charm.cs.illinois.edu/gerrit/#/c/2977/

Original date: 2017-09-11 18:43:24

Some of the things in this patch probably need to be ported for PAMILRTS: ~~https://charm.cs.illinois.edu/gerrit/#/c/898/~~ https://github.com/UIUC-PPL/charm/commit/e6754908a3a8587bc7bc1f6fe0e5269924127066
Plus this patch: ~~https://charm.cs.illinois.edu/gerrit/#/c/2762/~~ https://github.com/UIUC-PPL/charm/commit/13f0463317cd6104b6ff227b4eefbcf32747e99b


Combine catalogd and operator-controller Kustomize configs

Combine the catalogd and operator-controller kustomize configuration files. e.g. https://github.com/operator-framework/catalogd/blob/main/config/base/crd/kustomization.yaml

Linked to internal Jira issue EET-4419.

#1518 can be handled as part of this card as well. @rashmigottipati if you agree then please close #1538 as a duplicate.


Prefixing the package name for ruby/setup-ruby

In package.json there is:
https://github.com/Shopify/shopify-cli-action/blob/47089d657bad6581b4a0a45216c57c6ff4b74016/package.json#L37
Could you change it to "ruby/setup-ruby": "https://github.com/ruby/setup-ruby",?
I think that would fix this on https://github.com/ruby/setup-ruby:

So then that would show the actual (many) usages:
https://github.com/ruby/setup-ruby/network/dependents?package_id=UGFja2FnZS0yOTQyNTU2NTc1

Ping


Unable to validate JWT tokens. OpenID provider spec is not correctly implemented

This issue cannot be addressed in documentation, see https://github.com/MicrosoftDocs/bot-docs/issues/2064 opened by https://github.com/nvioli.
According to the OpenID provider spec,

OpenID Providers supporting Discovery MUST make a JSON document available at the path formed by concatenating the string /.well-known/openid-configuration to the Issuer.

and in section 4.3,

The issuer value returned MUST be identical to the Issuer URL that was directly used to retrieve the configuration information. This MUST also be identical to the iss Claim value in ID Tokens issued from this Issuer.

The iss field of the JWT token provided by the Bot Connector is https://api.botframework.com, but the metadata / configuration document is located at https://login.botframework.com/v1/.well-known/openidconfiguration (and alternatively, https://api.botframework.com/v1/.well-known/openidconfiguration). Note the extra v1/ in the path.
Microsoft is not correctly implementing the OpenID provider spec, which makes JWT validation impossible using some popular services and libraries which expect strict adherence. In my case I wish to validate the JWT token with AWS API Gateway, but I am unable to do so.
Editing to add the error received to help others searching for this issue: invalid_token: the issuer in the OIDC discovery endpoint metadata does not match the configured issuer.

Can you use a custom authorizer with AWS API Gateway that lets you specify the OIDC document location?
It seems like it might be a common situation - for example, the issuer field on https://login.microsoftonline.com/contoso.com/.well-known/openid-configuration is https://sts.windows.net/6babcaad-604b-40ac-a9d7-9fd97c0b779f/.

@nvioli I had hoped that there was a workaround where you could specify the OIDC location as simple configuration (since it seems like a common need).
(I agree with your reading of the specification)
Although we might be able to add additional routes to serve the OIDC at the proper path, we won't easily be able to fix the issue in our other clouds (the "domain" of the issuer doesn't match the domain where we host the OIDC).

Issue is mitigated for public cloud. Closing for now. Please reopen if this breaks again.

@MichaelHaydenAtMicrosoft I'm not sure what that comment means. If you're using the presence of a workaround in my specific use case to argue that the issue is mitigated, I disagree. The issue is about noncompliance with a spec, and the whole point of having standards is that developers should not need to build workarounds.
@willportnoy, you mentioned that it may be possible to add routes to serve the OIDC at the proper path. Is there an issue to track that work? This issue is about the botframework's JWT implementation, so mention of other Microsoft clouds is irrelevant.

@nvioli Bot Framework also has a presence in the other Microsoft clouds, and adding additional routes in those clouds won't mitigate the issue (because the JWT issuer doesn't match the OIDC url host name).
I'm looking into adding the additional route in the public cloud to mitigate this issue.

@nvioli I've added the other route to help mitigate your issue.
https://api.botframework.com/.well-known/openid-configuration


[Task]: Implement typedesc semtype in runtime

Description
Relevant section in the spec
Describe your task(s)
No response
Related area
-> Runtime
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Added with #43279 via #43260


Enhancement Request: Customizable Time Translation in ShowTimeSelect

I've noticed an issue within the react-datepicker package on GitHub.
When using ShowTimeSelect, there seems to be no straightforward way to customize the translation of "Time."
I've successfully implemented translations throughout the react-datepicker package, except for the 'Time' translation within the ShowTimeSelect feature.
Is it possible to consider adding this feature?

In english

In spanish

You can use the property timeCaption to change the "Time" wording. You find an example here:
    <DatePicker
      ...
      timeCaption="Hora"
    />


Add cancellation tokens to IAggregateRepository methods

resolves #66

@NemrudDemir thanks for the contribution.
This implementation is incomplete however, the cancellation tokens need to be passed further down into the different repositories.


DesktopHeapType.FixGenerics - does not work when subtype arity is missing or invalid

I've noticed that FixGenericsWorker heavily relies on the subtype arity (number of type parameters) on deciding how to process the name.
ECMA-335 (I.10.7.2 Type names and arity encoding) states that the grave accent character and arity together are optional, but that is followed by the Rule 43 which states slightly different version 😕.
Using arity alone to decide whether the name is a generic type and getting count of type parameters doesn't seem right. It should be optional and so work properly even if it's not present.
Omitting the subtype arity, does not cause issues in runtime, but memory dumps will not contain the arity, therefore, given the types:
1. MyAssembly.Test`1[[System.String, mscorlib]]
2. MyAssembly.Test[[System.String, mscorlib]]
3. MyAssembly.Test`3[[System.String, mscorlib]]

ClrTypes will have the Names set to:
1. MyAssembly.Test<System.String>
2. MyAssembly.Test[[System.String, mscorlib]]
3. MyAssembly.Test<System.String,

Does the third variety ever happen in practice?  If the arity is present it should match the number of parameters right?

You're right. I unnecessarily pasted this example, it probably never happens and is invalid according to that ECMA.
The 2nd example is what bothers me and my company's software.

I've fixed the issue in the ClrMD 2.0 branch.  I'm wary of shipping it as an update to clrmd 1.1 without a lot of additional testing.  That style of manual text parsing is very error prone and every time I make changes it always seems to break in some rare corner case I didn't consider.  I'll likely make a very risk mitigating change when I port it over.
I'm keeping this issue open until I fix it in 1.1.


Remove cmake-format

CI Vulkan-Tools build queued with queue ID 33963.

CI Vulkan-Tools build # 1118 running.

CI Vulkan-Tools build # 1118 running.

CI Vulkan-Tools build # 1118 passed.


Cannot setup container with custom baseapp

Using your blog post () as a guide, I tried to run the script below, however I get a number of issues when the original apps are supposed to be reinstalled using the updated dependency to our custom baseapp. Not sure what is missing. We have to get this working as we have to use a unique id/name/publisher for our custom baseapp as part of the embed program requirements.
Script:
$containerName = 'sr365'
$imageName = 'mcr.microsoft.com/businesscentral/sandbox:15.2.39558.39771-ca'
$licenseFile = 'C:\_dev\bc.flf'

$authenticationType = 'NavUserPassword'
$userName = 'admin'
$password = ConvertTo-SecureString -String "Pass@word1" -AsPlainText -Force
$credential = New-Object -TypeName "System.Management.Automation.PSCredential" -ArgumentList $userName, $password

$app = "C:\_dev\Custom Base App\Custom Base Application_15.2.39558.0.app"

# CREATE CONTAINER
New-BCContainer -accept_eula `
                -imageName $imageName `
                -containerName $containerName `
                -auth $authenticationType `
                -Credential $credential `
                -licenseFile $licenseFile `
                -updateHosts `
                -alwaysPull `
                -assignPremiumPlan `
                -doNotExportObjectsToText `
                -memoryLimit 10G `
                -process isolation `
                -useBestContainerOS

# REPLACE DEPENDENCIES
$replaceDependencies = @{
    "437dbf0e-84ff-417a-965d-ed2bb9650972" = @{
        "id" = '<removed>'
        "name" = 'Custom Base Application'
        "publisher" = 'SR'
        "minversion" = '15.2.39558.0'
    }
}

# PUBLISH NEW BASEAPP
Publish-NewApplicationToBCContainer -containerName $containerName `
                                    -appFile $app `
                                    -credential $credential `
                                    -useCleanDatabase `
                                    -saveData `
                                    -restoreApps Yes `
                                    -doNotUseDevEndpoint `
                                    -replaceDependencies $replaceDependencies

All of the issues happen during the Publish-NewApplciationToBCContainer commandlet. In general, it looks like all of the original apps are removed, but the dependencies are not updated in all of them before they get re-published back into the database. Some work though, which I find strange.
Example of one that does not work:
Checking dependencies in C:\ProgramData\NavContainerHelper\Extensions\sr365\Extensions\Microsoft_Image Analyzer_15.2.39558.39771.app
Application has ShowMyCode set to False
at <ScriptBlock>, <No file>: line 44
WARNING: Could not publish Microsoft_Image Analyzer_15.2.39558.39771.app - Application has ShowMyCode set to False

Example of one that does work:
Checking dependencies in C:\ProgramData\NavContainerHelper\Extensions\sr365\Extensions\Microsoft_PayPal Payments Standard_15.2.39558.39771.app
Application has ShowMyCode set to False
at <ScriptBlock>, <No file>: line 44
Copy from container sr365 (C:\Applications\PayPalPaymentsStandard\Source\PayPalPaymentsStandard.app) to C:\ProgramData\NavContainerHelper\Extensions\sr365\_PayPalPaymentsStandard.app
Checking dependencies in C:\ProgramData\NavContainerHelper\Extensions\sr365\_PayPalPaymentsStandard.app
Replacing dependency from 437dbf0e-84ff-417a-965d-ed2bb9650972 to <removed>
Publishing C:\ProgramData\NavContainerHelper\Extensions\sr365\_PayPalPaymentsStandard.app
Synchronizing PayPal Payments Standard on tenant default
Installing PayPal Payments Standard on tenant default
App successfully published

A few of the apps are not available with showmycode, and these cannot have dependencies replaced.
As stated, 15.3 offers a permanent solution to the temporary problem.

In the end - very soon replace-dependencies will be obsolete (15.3 introduces a new way on dependency indirection)

As stated, 15.3 offers a permanent solution to the temporary problem.

Hi @freddydk,
What exactly will be introduced in 15.3 that will resolve this, and can we maybe find more information on that change somewhere?

I will write this up in a blog post one of the next days.


[W4.5c][W10-A1] Law Gloria

@glorialaw
Some comments added. Please ack & close the PR after reading comments.


in last windows exe gives error when trying to open table

Windows 10 Java 8

Lastest version of Flamerobin doesn't run on Windows server 2012 either

Hi, please test the latest snapshot available and let me know if it's working
https://github.com/mariuz/flamerobin/releases/tag/<IP_ADDRESS>

yes, it's fine


Placeholder is missing in the input fields.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
In all the input fields place holder is missing.
Preconditions

Admin should be logged in.

Steps to reproduce

Go to 'Admin panel'
Click on 'create/edit'
See the error

Expected Result
There should be shown placeholder in the input fields.
Actual Result
showing blank input fields. Here
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Micronaut

Description of the issue:
Micronaut service doesn't start on Google Cloud when deployed as Container in Compute Engine.
Container stall in Restarting mode.
http://micronaut.io is a microservice framework by OCI the team behing Grails / Groovy.
Expected behavior:
Service should start instantly
Steps to reproduce:
Create a new Micronaut app using the Cli
Add the JIB gradle plugin
Deploy to Container Repo and start a new instance using the Jib deployed container
Environment:
GCE
Additional Information: I guess that the generated ENTRYPOINT can't access required
classes/libs
Thank you

could this be related to #396 ?

@Solido do you have a sample we could use for testing? Are you using Java, Kotlin, or Groovy?

Not sure, Micronaut is using java as the main starter but the rest of the app is impl w/ groovy.
The shortest path to recreate is

installing micronaut using SDKMan : https://sdkman.io/ (easier than build from github)
create a app using the command mn create-app hello-world --features groovy
add JIB to Gradle

The build and the upload of the image is OK
Let me know if it's too complicated I may try to send you a blank project, just not sure about the size of the archive it will generate.

FYI I just tried this without groovy and it worked perfectly.  This may be a generic Groovy related issues similar to #396

Good news that java is working,
#396 can not even compile and he gets log about it.
Looks like all is fine and jib-docker-context contains all the compiled classes and required libs.
Here's my log from gradle jib with groovy
Retrieving registry credentials for gcr.io...
Retrieving registry credentials for gcr.io...
Building dependencies layer...
Building resources layer...
Building classes layer...
Getting base image gcr.io/distroless/java...
Finalizing...
Container entrypoint set to [java, -Xmx50m, -cp, /app/libs/*:/app/resources/:/app/classes/, awesome.server.Application]
Any idea where to look ?

I just tried running jib with the groovy hello  world and got:
`
Execution failed for task ':jib'.

Obtaining project build output files failed; make sure you have compiled your project before trying to build the image. (Did you accidentally run "gradle clean jib" instead of "gradle clean compileJava jib"?)
`

The app is working well when using a shadow jar and using a basic
CMD java -jar awesome-server-0.1-all.jar

@coollog for the Java sample I didn't even have to run compileJava,  does that error message still make sense?

Yes, Jib now doesn't require you to run compileJava explicitly since it automatically depends on the classes task. That error will show up for the cases when using languages that aren't Java itself (no src/main/java). We will have to work on adding support for other JVM languages like Kotlin and Groovy by fetching the correct source output files to build into the image. Currently, we only fetch those provided by the JavaPluginConvention, which give output files such as build/classes/java/main.

I think I get something
Parallel execution with configuration on demand is an incubating feature.
:compileJava UP-TO-DATE
:compileGroovy UP-TO-DATE
... all is working well and container deployed.
My project contains the src/main/java to awesome.server.Application
BUT jib-docker-context only contains Groovy compiled class not awesome.server.Application

I've manually moved the Applications.class to jib-docker-context and redeployed.
Inspection of the generated container show that
"Entrypoint": [                 "java",                 "-Xmx50m",                 "-cp",                 "/app/libs/*:/app/resources/:/app/classes/",                 "awesome.server.Application"             ],
Is now working.

@coollog do we have a tracking bug for adding support for Kotlin and Groovy?  If it's low effort seems like something worth prioritizing.

Up & Running
Thanks @coollog

:+1:


feat: formatter init

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #587 (d8adb8e) into tagged-release/forms-and-views (08a84bb) will decrease coverage by 0.01%.
The diff coverage is 90.21%.

@@                        Coverage Diff                         @@
##           tagged-release/forms-and-views     #587      +/-   ##
==================================================================
- Coverage                           88.74%   88.73%   -0.02%
==================================================================
  Files                                  82       82
  Lines                                2986     3001      +15
  Branches                              809      815       +6
==================================================================
+ Hits                                 2650     2663      +13
- Misses                                321      323       +2
  Partials                               15       15

Impacted Files
Coverage Δ

packages/codegen-ui/lib/utils/index.ts
100.00% <ø> (ø)

...-react/lib/react-utils-studio-template-renderer.ts
33.33% <20.00%> (ø)

packages/codegen-ui-react/lib/utils/format.ts
93.24% <93.24%> (ø)

packages/codegen-ui/lib/types/date-time.ts
100.00% <100.00%> (ø)

packages/codegen-ui/lib/types/index.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Some section headers are hyperlinked

Inexplicably, some section headers get hyperlinked. I have no idea why. This could be a Quickbook / Boostbook issue. Example:

"Synopsis", "Description BNF", and "See Also" are hyperlinked.
This happens when Quickbook renders to HTML and produces an H7 or higher heading (HTML only supports H1 through H6)

To address this issue, I have opened issue https://github.com/boostorg/boostbook/issues/12 and pull request https://github.com/boostorg/boostbook/pull/13.

whoa !


Enhance UI Starter kit test case

[ ] incorporate testing methods into starter-kit

Updated test case with new verify methods


Partially switch to cross 0.1.15

Switch to cross 0.1.15, but only use it for the targets that work. This fixes our emscripten builds.
Resolves #164

Superseded by the switch to GitHub Actions.


Syntax highlighting breaks on closing parenthesis in comment

Found another issue similar to #45
Sample code as generated by NaturalONE:
/*( imported data fields from Data Definition Module
01 X VIEW OF Y
  02 FOO (A10)
/*) imported data fields from Data Definition Module

This one is a very minor issue as the syntax highlighting is only broken on the last line with /*)... and does not affect the rest of the files as it did in #45

That is a tricky one, because /*)  is also used in type definitions for arrays, e.g. (A1/*), because why not :)
I've added a workaround that should highlight this case, but only if the comment is started at the start of line, which your case looks like.
More ambiguous things will one day be solved when the underlying language server does semantic highlighting.


Creating an options bag for optional parameters

Creating an options bag for optional parameters for the getTranslationsStatus and getDocumentsStatus APIs for Java SDK specifically

API change check
APIView has identified API level changes in this PR and created following API reviews.
preview

Check Pipeline for Testing New Definitions was run by mistake, and the failure can be ignored.


Allow to customise the numbers of spaces in tab-to-indent

Currently the option tab-to-indent use 4 spaces but it would be nice if we were allowed to customise it.
Or a least, to provide a switch between 2 and 4 spaces, as these are the most common scenarios.
I don't have a full URL to provide, this feature is used whenever we have a comment field.
Thank you.

https://github.com/sindresorhus/refined-github/issues/2960#issuecomment-606765658
3. It doesn't require options.
Anything that requires configuration most likely does not belong to Refined GitHub. It’s just extra complexity and we already have 130+ features to attend to.
Imagine you want a feature to be "per branch", someone else wants it "per organization", etc. The options’ UI would just balloon because of different needs.

Damn it, I read this thread but turns out I apparently missed or misread this point.
My apologies.

Duplicate of https://github.com/sindresorhus/refined-github/issues/2910

No worries. Here's another explaination: https://github.com/fregante/indent-textarea/issues/8#issuecomment-520009734

To clarify, tab-to-indent uses tabs, not spaces. If you'd like to display tabs as 2-character-long instead of 4, you can use https://github.com/lukechilds/github-custom-tab-size

Thank you for the links @fregante. Too bad it is only for changing the display of indentation, not when writing code in a suggestion block (the feature I really liked from your extension), but that's okay! It's better than nothing.


Is this repo still checked?

My concern being if this is enterprise ready, I'm hoping someone is checking these issues otherwise, you could lose a potential future customer that wants to pay for support and other things.
I've suggested this to try out on a small scale to see if it might fit a larger scale project but not seeing any responses is making me nervous.

Cannot get past this;
`==> /var/log/httpd-error.log <==
[Wed Aug 02 19:40:12.380152 2023] [php:warn] [pid 15220] [client <IP_ADDRESS>:47114] PHP Warning:  PHP Request Startup: function "1" not found or invalid function name in Unknown on line 0, referer: https://www.domain.com/
==> /var/log/httpd-access.log <==
<IP_ADDRESS> - - [02/Aug/2023:19:40:13 -0700] "OPTIONS * HTTP/1.0" 200 -
<IP_ADDRESS> - - [02/Aug/2023:19:40:12 -0700] "GET /asset/gzip/?t=js&d=.&f=static/js/jquery/carouFredSel.js,templates/themes/base/js/splash.js HTTP/1.1" 500 20
<IP_ADDRESS> - - [02/Aug/2023:19:42:43 -0700] "OPTIONS * HTTP/1.0" 200 -
`

Thanks very much for raising this concern @dac2020! This repo is still being checked and maintained 🙂 That's said, I am also looking for a few main contributors to speed up the upcoming new features (see current pH7Builder's Milestones and other upcoming changes.

Apologies for the mess of comments. I started out asking one question, worked my way through that so updated to a different question etc. I wish I could help but I'm not a developer.
Thank you for responding and happy to hear it's alive and well so will keep trying to get my test site up.

BTW, I emailed you but maybe it landed in spam.


validators: fix crash from bounds check

This crash was found via fuzzing, so I've also added the crashing file into our unit tests after the fix.

Codecov Report

Merging #50 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #50      +/-   ##
=========================================
- Coverage    87.2%   87.2%   -0.01%
=========================================
  Files          75      75
  Lines        5956    5954       -2
=========================================
- Hits         5194    5192       -2
  Misses        518     518
  Partials      244     244


CSS isolation stopped working in Blazor WASM project after project rename

CSS isolation "stopped" working in the Headway.Blazor.App.WebAssembly project for the ComponentTip component in the Headway.Blazor.Controls project.
To get it working add a reference to <link href="Headway.Blazor.Controls.styles.css" rel="stylesheet" /> in index.html. For consistence do the same in the Headway.Blazor.App.Server's _Host.cshtml page.


[Event Request] Codeunit::"Sales-Post" - Function SumSalesLines2 - OnBeforeSumSalesLines2

We like to request the following Event:
Object: Codeunit::"Sales-Post"
Function: SumSalesLines2
Publisher Event: OnBeforeSumSalesLines2
Reason: To implement custom business logic
Where is the event placed:
local procedure SumSalesLines2(SalesHeader: Record "Sales Header"; var NewSalesLine: Record "Sales Line"; var OldSalesLine: Record "Sales Line"; QtyType: Option General,Invoicing,Shipping; InsertSalesLine: Boolean; CalcAdCostLCY: Boolean; var TotalAdjCostLCY: Decimal; IncludePrepayments: Boolean)
var
SalesLine: Record "Sales Line";
TempVATAmountLine: Record "VAT Amount Line" temporary;
TempVATAmountLineRemainder: Record "VAT Amount Line" temporary;
SalesLineQty: Decimal;
AdjCostLCY: Decimal;
BiggestLineNo: Integer;
IsHandled: Boolean;
begin
OnBeforeSumSalesLines2(SalesHeader, NewSalesLine, OldSalesLine, QtyType, InsertSalesLine, CalcAdCostLCY, TotalAdjCostLCY, IncludePrepayments, IsHandled);
if IsHandled then
exit;
TotalAdjCostLCY := 0;
TempVATAmountLineRemainder.DeleteAll();
Event Code:
[IntegrationEvent(false, false)]
local procedure OnBeforeSumSalesLines2(SalesHeader: Record "Sales Header"; var NewSalesLine: Record "Sales Line"; var OldSalesLine: Record "Sales Line"; QtyType: Option General,Invoicing,Shipping; InsertSalesLine: Boolean; CalcAdCostLCY: Boolean; var TotalAdjCostLCY: Decimal; IncludePrepayments: Boolean; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


New Library Perspective (new authoring)

This adds the Library Perspective missing deps and fixes the build.

+1 thanks buddy.

Do we need Jenkins to test this before merge?

@manstis how starts a build from here?

jenkins retest this

Jenkins please retest this.

Merging as it needed to fix the build. @pefernan could you please fix the typo and create new PR?


Handling of ch, sh, th and ng in english

How does eudex handle sounds such as ch, sh, th and ng in english?
Or y in front as a consonant vs in the middle as a vowel?
Or c having a k sound or a s sound in certain situations?

Hi, any update on this ?


Django 4.0 Url Problem

from django.conf.urls import url (deprecated in Django 4.0)
I use re_path the urls to solve this
But also it throw an error on:  from django.utils.translation import gettext_lazy as _
ugettext_lazy (has been removed in Django 4.0)
use : from django.utils.translation import gettext_lazy as _
When you done this its giving an error:
user_recovers_password = Signal(
TypeError: init() got an unexpected keyword argument 'providing_args'

When you edit Signal file such as
user_recovers_password = Signal('request')
seems working.

Looks like someone had submitted a relevant PR: https://github.com/brutasse/django-password-reset/pull/84
It's been open since 2021 however


Update Documentation: decl_module

Discussed with @shawntabrizi, updates #8

[ ] refactor detailed explanations to be inline with code & autogenerated with rust docs
[ ] update decl_module with a more high level explanation that reads more like a tutorial

@nczhu try to gather some suggestions and feedback from @bkchr and @thiolliere for this
They are the macro gods (to me) and I think will have good suggestions how best to document this stuff.

FYI, its no longer getting renamed :)

In progress: https://hackmd.io/s/Sy6T0usOE
Still need to fact check, then style check before EOD.

PR here:
https://github.com/paritytech/substrate/pull/2147


componentWillMount() is deprecated

Hello guys, I found the following warning.How can I fix this?
YellowBox.js:80 Warning: isMounted(...) is deprecated in plain JavaScript React classes. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.
Thanks.

Hi @B4UGUYS,
We're sorry for late reply.
Looks like your issue is linked here. If you take a look at it, you'll see, that this is a core RN issue which was fixed in 0.56.0
Btw, currently we're using latest expo fork of React Native and there are no warnings like this.
Thanks for your reply!


Synchronous tasks on either end of peer program

Synchronous tasks / polled interfaces will be common for both ends of the new peer program interface.
Implement something in the base / something that works automatically such that the inheriting class only needs to implement the dispatch method.
Potentially move some of the other polled stuff into there as well?

To support development of https://github.com/vkottler/runtimepy/issues/206, we should update both ends of the "sample" peer task to have a main polled method that just polls the sample struct.


Maximum payout per hour

A limit to how much money the plugin will doll out to a player in a configurable amount of time.
Before paying out the plugin should check if the player has exceeded the maximum pay (config) allowed in X period of time (config).

maximum-pay:
money-limit: 0 #maximum amount of money that can be handed out in the time period, 0 to disable.
billing-period: 1h #defined in Minutes, or Hours, or Days (24hours), or Weeks (7 Days).

We have entire section dedicated for payment limitations already. Additionally there is /jobs limit command to indicate how much you have for current limit and when it resets. Just search for "Money gain limit" in general config file


Ctrl + H opens Zeal without requested query (Ctrl+Alt+H works)

I'm working on Visual Studio Code 1.38.1 in Ubuntu 18.04.3 LTS. Here is a screenshot of the problem:

Looks like the problem is connected with this area.

If I delete all exact: prefixes, it works fine.

So, are there any way to remove those exact: prefixes without me having to manually delete them?

Hi @Vojnuk,
Could you check the settings of Exact Docset whether it is ticked or not? To disable this, make sure that it is unticked.

Let me know if the problem still persists.
Thank you

It resolved a problem.
Thank you very much.


How to....

How? can someone tell me how to send pollMessage with quoted message?
I'll try like this
sock.sendMessage(jid, { poll: { name: "this is poll message", values: ["yep!","nope"] }}, { quoted: quotedMessage })

But the quoted don't appear!1
Is the code that I used wrong?
And onemore question! How to send quick video message?

Don't make issues for how to questions.


Fix update-openapigen.sh that can be executed from kserve dir

What this PR does / why we need it:
This helps update-openapigen.sh execute anywhere
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #https://github.com/kserve/kserve/issues/3923
Type of changes
Please delete options that are not relevant.

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

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Re-running failed tests

/rerun-all - rerun all failed workflows.
/rerun-workflow <workflow name> - rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.

/rerun-all

/rerun-all

/lgtm
/approve


Removed error thrown while rendering Multi Type chart containing Area

Data label position is being set to outEnd for Multitype charts that contain an area chart as the other chart type would enable inclusion of that line. Hence either the suggested changes can be made or the entire line checking the condition can be removed.

Hi @KrishnaTejaReddyV ,
Thanks for the submission.
Can you provide some code/screencaps showing the issue before and after the fix?

Attached the images with error before change, output before change and output after change

Thanks @KrishnaTejaReddyV !


Bind poppup to each MarkerCcluster`s marker

Hi, I have every marker added to the markerclusterer markers and I want to just bind poppup to each marker , like when you are zooming map and click markers here should poppup appear,but here clusters not working just markers without clusters
Here Loop:
var me = this;
var marker = new L.MarkerClusterGroup();
var store = Ext.getStore('Person.PersonStore');
for (var i = 0; i < store.data.items.length; i++) {
if (store.data.items[i].data.position != null) {
var lat = store.data.items[i].data.position.lat;
var lng = store.data.items[i].data.position.lng;
var position = {
lat: lat,
lng: lng
};
marker = new L.Marker(position).bindPopup('' + "sssss" + '');
map.addLayer(marker);
}
How to make them bind poppup and when you click to marker here poppup appear?
Thanks
I have read about this but they used custom marker

I have got this code which work for me. I use a .geojson file convert into js variable to get coordinates of markers :
  // We create each marker with its popup from geojson variable
  function onEachFeature(feature, layer) {
    var popupText = feature.properties.name;
    layer.bindPopup(popupText);
  }

  var geo = L.geoJson(geojson, {
    onEachFeature: onEachFeature
  });

  markers = new L.MarkerClusterGroup();
  markers.addLayer(geo);
  map.addLayer(markers);

Hi,
Please note that whenever you need help for your project, the most appropriate place would probably be the Leaflet forum.
Then if your code is fine but you know the problem comes from the library, open an issue here.
For your application, be noted that you have markers and groups. The latter can hold markers and other groups. All are considered as "layers".
MarkerClusterGroup, FeatureGroup, LayerGroup are such groups.
Create your markers (the ones that have latitude-longitude position), add them to a group (group.addLayer(marker)), and add the group to your map (map.addLayer(group)).
Hope this helps.

Thanks for the Leaflet forum.
I don't if the problem comes from the library or if I don't know how to use it. ^^
To create markers I think I did like you are saying :

Markers are "geo" variable
Group is "markers" variable (MarkerClusterGroup)

Maybe I have to create my own CSS style for cluster with two state "cluster" : blue color and "clusterError" : red color ? Is it possible to do something like that ?

Thank you for helping me but can you give me hand again,my issue is I add markers with poppup then when I click list it shows popup of marker of apprepricate marker:
cellclick: this.ListClick,
ListClick: function(grid, rowIndex, cellIndex, e) {
var me = this;
var time = new Date(e.data.position.date);
var date = new Date(time);
var mymarker = L.marker([e.data.position.lat, e.data.position.lng]).openPopup();
// var popup = L.popup().setLatLng(e.data.position.lat, e.data.position.lng).openOn(me.map);
                    me.map.panTo(mymarker.openPopup());

}
It is only move to marker but poppup doesnt work I have implememt popup on another method
Thanks in advance

Sultan,
See my answer in the forum:
https://groups.google.com/d/msg/leaflet-js/KU47UdxlpAE/ppopkObVEgAJ
Cheers


remove unused dependencies

Hi, thanks for creating this package!
This PR removes CSV, DataFrames and Statistics as dependencies, since I noticed they are not used anywhere. This allows the package to stay lightweight.

I forgot to mention, I also removed a Project.toml and Manifest.toml from the src directory. Not sure if these were checked in on purpose, but it's not very common to have this in your src directory.

Thank you for the contributions.


Initially installing the release version of an extension that has a pre-release doesn't fully update extension editor content

Testing #138086

Find a not-yet-installed extension for which a pre-release exists.
Choose to install the Release version.
The extra line in the header about there being a pre-release is missing until you close and reopen extension editor.

Also, when the extra line appears and you click on its link, the description correctly changes to that of the pre-release. But I see two problems:

No visual indicator that I'm now reading pre-release info.
No obvious way to go back to the release version's info.

Also, when the extra line appears

@JacksonKearl has created #138233 for this aspect.

I will take care of the second aspect in the issue #138233

Fixed by 112ca126f6f86387f6054e8d55544d97f72f34ef


Icons missing on App Page

Hi,
similiar to https://github.com/cptpiepmatz/great-on-deck-search/issues/20 the Icons for ProtonDB and SDHQ are not showing when Augmented Steam Extension is installed as well. I checked the settings and I can't see any toggle which would interfere with your addon tbh. Disabling Augmented Steam fixes that issue

@pchristod Augmented Steam defined a css setting to hide images for the nav buttons. Not considering other extensions, how rude. 😑
I defined this setting specifically in my css now to fix this, will be in the next update as soon the stores update the package.
Btw, this is not similar to #20, there is a spacing issue which I'm not really sure how to fix yet.

Ah I see, my bad, sorry. I thought it's similar because the extension combo is the same.
Yeah, to be fair your extension works perfectly fine, it's just the combination with Augmented that doesn't. So not really your fault. Which is why I really appreciate you fixing this when you didn't have to necessarily, thank you :)

No biggie, it just annoys me that other extension work like no other extensions are installed. The fix is uploaded to the stores will probably avaible in the next 24 hours.

@cptpiepmatz Thanks, received the update, it's working better now :) Just gotta ask: When hovering over the Icon the box will get smaller and the icon will be missing again. Is this a limitation/intended behaviour? Seems like two boxes are somehow overlapping but only if the mouse is hovering over the button/you are about to click it.

Grüße ;)

Ah shoot, did not fully test that. I see what I can do.

I got it fixed in 88f38d07040e5c091d2d6367a0486fa8355266a1.

Sounds great, thank you so much 😄


Dashboard: enable arrow key navigation on stories grid

Summary
Allows users to key into the my stories grid and use arrow keys to navigate.
Relevant Technical Choices

Follows same pattern as here: #4303 and here: #4317
Adds an invisible button to each grid item that will transfer focus to grid items according to useGridViewKeys without interrupting the grid structure and preventing nested buttons.
when new FocusableGridItem is set a useEffect hook immediately updates the focus to the actual grid item.
Adds stopProp and/or preventDefault to keyDown handlers and onClick (for native buttons) to prevent bubbling up of events that are nested inside of the grid item so that they don't cause the grid item focus to adjust
Adds some extra aria labels to card items

To-do

For some reason, opening the preview of a story in a new window (the link in the context menu) won't open on with the keyboard?
return focus to proper item after closing the preview dialog
loading more stories could be more graceful, right now you have to tab one extra time to get the loading to trigger since it's based on the loading component being visible, but think this could be done later.

User-facing changes
Now when you are on my templates and tab to the grid you can enter the grid items by using arrow keys or tab again to exit the grid and, if you are not at the top of the browser window, jump into the scroll to top button.
Updates display text to open the preview text updated to always say "preview" not "view" if story is published, per conversation with @samitron7 9/2/2020
Testing Instructions

Navigate to 'My Stories' in dashboard
Use 'Tab' to change focused element until you get to the grid. The grid should have a blue outline now.

Hit tab again to exit the entire grid (this can be tricky to see if the scroll to top button is not in view, but it should go there).
Hit the right arrow key and enter into the grid.

Now you're focused on that first template

tab again,  the 'preview' button gains focus
tab again, the 'open in editor' button gains focus
tab again, the card title link gains focus
tab again,  the context menu button gains focus
Hit enter to open the menu or the right arrow to move to the next grid item
If you open the menu, it'll gain focus. You can use the up/down arrow keys to navigate the menu.
One thing to note here is right now you have to select an item to close the menu, Mariano made an update for that ( here ) which will resolve this.

If you exit the grid and hit shift + tab, focus will be regained to whatever grid item you left off on.
Ability to interact with grid items with a cursor should persist.

Addresses #4296

@googlebot I consent.

@googlebot I consent.


docs: Update README.md

Since go get is deprecated.
I change go get command to go install.

Thank you for your contribution!
It looks good to me. So, I've merged your changes into master.


fix: don't use arguments if they are empty when running a query in the postgres driver

Does your PR solve an issue?
Kind of:

https://github.com/launchbadge/sqlx/discussions/2210
https://github.com/launchbadge/sqlx/discussions/2448

At the moment, this will throw an error with Postgres:
sqlx::query("SELECT 1; SELECT 1;")
    .execute_many(&pool)
    .await
    .inspect_err(|err| panic!("{err}"))
    .count()
    .await;

error returned from database: cannot insert multiple commands into a prepared statement

Because the query will by default set up empty arguments array and the driver treats the query as if it had arguments.
This on the other hand works fine:
pool.execute("SELECT 1; SELECT 1;").await.unwrap();

In fact, this is why the migrator impl for Postgres doesn't hit this issue.
I've patched the postgres driver to ignore arguments if they are empty, but really it's just a proposal, idk if that's the right solution.

This is not the desired behavior. The query functions and macros always use prepared statements, which are forbidden to contain more than one query to prevent SQL injection. This is a feature, not a bug.
.execute_many() only ever worked for the SQLite driver, because it uses the same code path for "raw" queries as it does for prepared queries, so it allows multiple statements in a query string by default.
IMO, .execute_many() and .fetch_many() on the Query struct should be removed, and the SQLite driver should forbid having more than one statement per query string when used with this interface.
I also need to think of a more intuitive "raw" query interface than directly invoking methods on Executor. It's too clever, which makes extremely it difficult to discover.

@abonander Ok, that makes sense, but in that case there's still probably a bug in that you can call .take_arguments() on the query which will trigger the plaintext path in the driver and you can execute multiple statements that way.


Update garbage-collection.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks, @scjane 👍


PerfKitBenchmarker roadmap

Do you have any list of planned features you are going to work on in the future (kind of a public roadmap)? What about the Docker support proposal (https://docs.google.com/document/d/1XEmo8OGd7tJIwR89yDkUXeXB13zXfVNUaczveAmWSLY/edit)? I'm trying to figure out in what direction PKB is going.
Thanks!

Mateusz,
Thanks for asking.  I'll include the roadmap and ask Carlos to discuss the
Docker Swarm proposal during the community meeting this month.
There are two parts to the roadmap;

The technical part such as declaring V1, adding more Windows Workloads,
moving up the Cloud Stack to PaaS and SaaS services, scoring, etc.
The stewardship path.  Google committed to the first year to run it.
What I'd like to do is figure out should we do it for another year or look
to move the project to its own foundation or Apache so we can open up to
more committers after year one.

We are 7 months after the first open release and the pickup and community
growth has been pretty awesome.  I'd like to keep that up because Openness
around Cloud Performance really matters.
Tony
On Thu, Oct 8, 2015 at 4:21 AM, Mateusz Blaszkowski <
<EMAIL_ADDRESS>wrote:

Do you have any list of planned features you are going to work on in the
future (kind of a public roadmap)? What about the Docker support proposal (
https://docs.google.com/document/d/1XEmo8OGd7tJIwR89yDkUXeXB13zXfVNUaczveAmWSLY/edit)?
I'm trying to figure out in what direction PKB is going.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/517.

--
Anthony F. Voellm (aka Tony)
Google Voice:<PHONE_NUMBER>
https://www.google.com/voice/b/0?pli=1#phones
Blog: http://perfguy.blogspot.com/
Benchmark like a pro... PerfKitBenchmarker
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker

I'll resolve this after https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/507 is complete.

@voellm @mateusz-blaszkowski
I will be more than happy to talk about the Docker Swarm provider proposal on the community meeting.
But in summary, the Docker Swarm proposal at the moment is just that, a proposal, which I'm requesting comments from the PerfKit community to see if there's interest in such feature. There are really 2 parts to the proposal, which I should probably split them in two proposal since they are independent.
The first part, I think is non-controversial and trivial to implement, it would just add Docker Swarm as a provider, and it would work just like the Kubernetes provider.
The second part would require several changes, but it would not change how you use PKB Today. The way I envision it is that the container features will be orthogonal to the current features, essentially, just like PKB talks to a cloud provider to build a VM, it would instead talk directly to a container 'cluster manager' (e.g. AWS ECS, Docker-swarm, Kubernetes, etc) to create/delete the containers, and manage their configuration (e.g. exposing ports, ambassador containers, attach data containers as volumes, etc). Ideally, it would use Docker images that have been pre-built from Dockerfiles, and they will try to bake-in containers' best practices where possible. (e.g. No SSH, distinguish privileged vs non-privileged, data containers, etc).

Thank you guys for answers. That clarifies some of my thoughts.
@voellm - we (Intel) have some ideas for extending PKB (especially for private cloud usage) but this may require some help from the community to achieve this. I will try to gather a list of items that we see could be valuable. Let's see if these items coincide with your needs.
@meteorfox - that would be great to see your changes. I agree that containers support should not change the way PKB is used currently. It may be delivered as a separate feature. We plan to work on Mesos&Marathon provider and we will again face the same issues. SSH is not the best candidate for containers but it is the easiest way with the current implementation of PKB. The first part you propose should be pretty straightforward, but the second part might require a lot of work in Perkit framework structure. However, I'm definitely looking forward to the changes :) Having these pre-built and configured Docker images would be awesome!

@Mateusz One thing we have talked about doing is factoring out the
transport (communications) layer to make it more extensible.  For example
where SSH is not an option or available you could tunnel commands over an
object store or filesytem in a message passing manner.  For system that can
inject messages over serial we can use that, etc.  We also have the need in
some circumstances.
Feel free to put your ideas into the up coming community deck at the end of
your Kubernetes section.
On Thu, Oct 15, 2015 at 6:51 AM, Mateusz Blaszkowski <
<EMAIL_ADDRESS>wrote:

Thank you guys for answers. That clarifies some of my thoughts.
@voellm https://github.com/voellm - we (Intel) have some ideas for
extending PKB (especially for private cloud usage) but this may require
some help from the community to achieve this. I will try to gather a list
of items that we see could be valuable. Let's see if these items coincide
with your needs.
@meteorfox https://github.com/meteorfox - that would be great to see
your changes. I agree that containers support should not change the way PKB
is used currently. It may be delivered as a separate feature. We plan to
work on Mesos&Marathon provider and we will again face the same issues. SSH
is not the best candidate for containers but it is the easiest way with the
current implementation of PKB. The first part you propose should be pretty
straightforward, but the second part might require a lot of work in Perkit
framework structure. However, I'm definitely looking forward to the changes
:) Having these pre-built and configured Docker images would be awesome!
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/517#issuecomment-148393174
.

--
Anthony F. Voellm (aka Tony)
Google Voice:<PHONE_NUMBER>
https://www.google.com/voice/b/0?pli=1#phones
Blog: http://perfguy.blogspot.com/
Benchmark like a pro... PerfKitBenchmarker
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker

As I said on the community meeting - I'm copying the list of improvements which are important for us:

Possibility to run at scale (up to 1000 nodes with thousands of instances). Current potential bottlenecks:

SSH (https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/550) - generating dozens of thousands of SSH request would be very costly (if not impossible)
logs (800 mb of logs when running redis with 100 redis client instances)
CPU consumption on the host which is running the benchmarks (Load average ~1.1 when running redis with 100 redis client instances)

Accept failures

Don't stop the run if one of the benchmarks failed (#573)
Present failure rate at the end of a run

Synchronize multiple benchmarks to be run on the same time (overlapping workloads - not sure if there's an issue for this)
Leverage scheduling policies (spreading, stacking, random, round-robin, etc.)

Misrepresented results when instances are scheduled on the same host (for i.e. SCP - when we copy the data between two instances running on the same host we get extraordinary results - much higher than the network throughput ;) )

Spawn new instances during benchmark execution

One of the interesting test scenarios would be to verify how many instances we are able to spawn without affecting other instances

Generate cluster noise (benchmarks running in a loop or for a specified period of time)
Gather only a subset of metrics
Repeat the test several times
Pluginable mechanism for storing the results

Adding another publisher should be as simple as adding a new class and changing a flag parameter. Curently we would need also changes in a core (in publusher.py)

Great feedback.
Laid out the next quarters idea here - https://docs.google.com/presentation/d/1MkMPxYN1rgIPXy0576UUVGGnUIPaJavFnY1aWfJqh14/edit
In terms of commitments #550 high on my list and we'll look to do that Jan - Mar - Q1 for Google if not before.
#573 is currently being worked by @skschneider.  He has a good model for how to handle it and working code.
Added #601 for the high CPU load you mention.
Added #602 for the log size issue.
#436 is the tracking issue for multiple benchmarks
For other issues high on your list please open GitHub issues so we can track them individually and its easier for the community to see what is needed.

In terms of tracking when things will happen we should use the "Milestone" flags on issue.  When something is being worked or planned the target release can be set.  Sound like a good idea?

Thank you @voellm for the update.
For me "Milestone" flags will be perfect as long as they are updated regularly.

Sounds good on the Milestone.  I've move bugs to target them.
To help lay things out here is an architecture diagram - https://drive.google.com/file/d/0B5iICDqrB_E9eE1pbTJDSGhiUlU/view
Here is a also a guesstimate on changes laid out on a timeline...
https://drive.google.com/file/d/0B5iICDqrB_E9OFlpYU0zaFNlT1k/view
Based on this I'm going to close the issue.


HDDS-6167. Update ozone-runner version to 20211202-1

What changes were proposed in this pull request?
Update Ozone to latest ozone-runner docker image (20211202-1 currently).
Also apply the same version to K8S tests, as these have been using hard-coded version that got outdated.
https://issues.apache.org/jira/browse/HDDS-6167
How was this patch tested?
Regular CI:
https://github.com/adoroszlai/hadoop-ozone/actions/runs/1676705266

Thanks @errose28 for the review.


Added optional timeout to ftp connection. Default set to 60 seconds.

luigi.contrib.ftp previous didn't not explicitly set connection timeout. Python docs specify timeout to be "the global default timeout setting" when no timeout is specified. This PR allows explicit timeout, set at 60 seconds by default.

LGTM


output type unknown

Found a FIXME comment on line 583 of file crates/nu-parser/src/parser.rs

Duplicate, closing it.


ANother Gap? Share endpoint alerts with Microsoft Compliance Center

I don't see Share endpoint alerts with Microsoft Compliance Center in my GCC tenant, is this another gap? It is described at https://learn.microsoft.com/en-us/microsoft-365/security/defender-endpoint/advanced-features?view=o365-worldwide#share-endpoint-alerts-with-microsoft-purview-compliance-portal. Please update the documentation if necessary

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

ID: c97d4493-75b1-312c-83b0-ad8fe72f7c0c
Version Independent ID: af86fff7-4158-6e20-683a-e9cc02a10f67
Content: Microsoft Defender for Endpoint for US Government customers
Content Source: microsoft-365/security/defender-endpoint/gov.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please help us in resolving this issue. Thanks

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Enhancement / Feature request - when running as administrator first open window should run as admin.

Steps to reproduce:

right-click on the icon
run as administrator
cmder is started
the open console is not in admin mode (no protection shield next in the tab section)

For my project I need process to run with administrator rights, so right after opening new cmder I have to open a new tab. (which is not super-practical)
I think, that once I open cmder as administator the first opened console should be in admin mode.

Check #544.


Fix render reorder causing incorrect groupings

Currently, Great Tables keeps a GT._body.body dataframe, ordered to match the final output (e.g. sorted by group). This means that what happens currently looks like this:

formatters run, put results on GT._body.body
GT._build_data() returns a new GT object, with ._body.body reordered to match final output
GT.render_as_html() -> create_body_component_h() : attempts to set any missing values in ._body.body to a string version of the original data. This is to fill in any unformatted values.

However, there are two issues with this:

We attempt to grab rows from _body.body when rendering html, as if the dataframe hasn't been sorted. (but it has).
With polars, we are accidentally filling in missing values in step (3) with the original data, in its original order.

In order to fix this, we need to decide whether _body.body should be sorted, and make sure it's filled consistently across pandas and polars.
This PR simply removes the sorting of _body.body. Since we index based on the original data (e.g. passing row=1 to a formatter refers to that row position in the original, unsorted data), it seems like we can avoid sorting ._body.body, and just grab the correctly ordered rows when rendering.
(Note that the snapshot test I added when addressing #201 was wrong 😭, and I fixed it here)

Shoot, I checked out this branch from the formatter PR one. Once that's merged I'll just get this one caught up, so it's less chaotic!

Shoot, I checked out this branch from the formatter PR one. Once that's merged I'll just get this one caught up, so it's less chaotic!

The formatter PR has been reviewed (great work!) so it can be merged.

Alright, it should now just show the diff against main (no fmt method changes)! Note that this PR makes a fairly substantial change of keeping _body.body in the original input order. This means nothing gets reordered, only that the renderer func grabs rows in final display order.
Definitely let me know if this will cause issues in the future, based on any gt for R behaviors!


Looking forward to get involved and contribute to the community 🎉

First Name
Vaibhav
Last Name
Gupta
Discord Username (if applicable)
vaibhavgupta#5585
Let us understand your motive.
I am a fourth year engineering student, from Delhi, active in the field of Machine Learning and Data Science. I want to take part in hackathons related to these fields, therefore I plan to join this community. Here please refer to my LinkedIn profile to connect to me - https://www.linkedin.com/in/vaibhav-gupta-98b5671b0/

Hey @vaibhavgupta2499 👋🏻
Thanks for requesting an invite! Welcome to SidHacks Community 🎉
Before going ahead, I would want you to complete the below To-Do list.
Your To-Do List ✔

Accept the invitation sent to you via mail.
Change the appearance of this organization to "Public" from the people list here.
Join our Discord Server here.

Accept the Rules from #📓〡rules channel.
Introduce yourself to our community in 👋〡introduction channel.

Feel free to share your excitement on socials with #SidHacksCommunity. You can also tag me @siddharth_hacks.

Close this issue once you complete all the above points.

Looking forward to interacting with you in our community! Welcome once again! 🤓


Not backing up random messages

I have a Samsung Galaxy S5 and it's not backing up some messages.  For example, I have two message threads, each with a couple of people as a group message.  It has saved random messages from each thread into my gmail, but not all of them, and it's saved all the messages in my gmail all over the place instead of keeping them all in one thread. Please help

I'm having the same issue with my g3 sadly tmobile to who was helping me was a complete lame and erased the vital message from my old phone and now it appears there is no way to retrieve this message although I advised repeatedly of the importance of this message. Can anyone please help with recovery of lost messages which did not back up for some unknown reason? Please help this is my last hope of getting this message back it was the first message my boyfriend ever sent me I' ve worked very diligently to keep this.

I'm having the same issue with my g3 sadly tmobile to who was helping me was a complete lame and erased the vital message from my old phone and now it appears there is no way to retrieve this message although I advised repeatedly of the importance of this message. Can anyone please help with recovery of lost messages which did not back up for some unknown reason? Please help this is my last hope of getting this message back it was the first message my boyfriend ever sent me I' ve worked very diligently to keep this.

that's a gmail limitation, see FAQ


PT-BR translation improvements

POEeditor is used for BD translations: https://poeditor.com/join/project?hash=nRljcnV0ET

POEeditor is used for BD translations: https://poeditor.com/join/project?hash=nRljcnV0ET

Oh, thank you. So should i close the pull request and edit through the site?


chore: Add contributors PR

Signed-off-by: Marcos Iglesias<EMAIL_ADDRESS>
Summary of Changes

Documentation

CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely, including a title prefix.
[ ] PR includes a summary of changes.
[ ] My commits follow the guidelines from "How to write a good git commit message"

@all-contributors please add @feng-tao  for bug, code, content, blog, data, doc, example, ideas, infra, maintenance, mentoring, platform, plugin, projectManagement, question, review, security, tool, test, tutorial

Having issues with this, opened a new ticket: https://github.com/all-contributors/all-contributors-bot/issues/322


RM training loss becomes NAN when finish the first training step.

I used a large model (> 170B) as my reward model. In the very beginning, loss is normal. But when training one step, the loss becomes NAN. This situation didn't happen when I used a smaller base model (e.g., 30B) to train RM. Do you have any suggestions about this?

"But when training one step, the loss becomes NAN. "
more info about your data and configuration?


Dynamic Pricing Plans and Resource Groups

Hello--,
I've noticed that the portal won't let me create more than one Dynamic Pricing Plan per resource group. Since the portal also won't let me assign a second Function App to an existing Dynamic Pricing Plan there seems to be a maximum of one dynamically priced Function App per resource group. Is this true? Can you share the rationale behind that decision? Grouping different Function Apps (each dynamically priced) with different memory sizes and requirements respectively into one resource group doesn't seem to be far off. Thanks!

Just checked: it's possible to create two dynamic pricing plans within the same resource group.
So above problem seems to be an issue with the portal. Not sure if this should be filed as a bug?

I'll talk to the Portal owners about this. It does seem like a unnecessary limitation.

Closing here since it's not part of this code, but I did talk to them about it and they'll fix.


Paket does not put .props and .targets import project on correct location in project files

When Paket installs a nuget package with {packageid}.props/{packageid}.targets in build folder, it should

Put '<import Project="{path to {packageid}.props" />' as the first child element in Project element
Put '<Import Project="{path to {packageid}.targets}}" />' as the last child element in Project element

Paket puts both imports in the middle of csproj file and by doing so, breaks compatiblity with nuget feature "Import MSBuild targets and props files into project"

Do you have a csproj file and a dependencies file (or just nuget package
name) for me to create a integration test?
On Nov 13, 2015 1:29 PM, "Thomas Heggelund"<EMAIL_ADDRESS>wrote:

When Paket installs a nuget package with
{packageid}.props/{packageid}.targets in build folder, it should

Put '' as the first
child element in Project element
Put '' as the
last child element in Project element

Paket puts both imports in the middle of csproj file and by doing so,
breaks compatiblity with nuget feature "Import MSBuild targets and props
files into project"
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1219.

https://github.com/theggelund/paket_bug

does this work for you?

It worked in the test project and in some of my real projects but not all. I will try to analyze the ones that didn't work and try to create a new test project for you.

Updated https://github.com/theggelund/paket_bug
When nuget package is sorted after MultiTarget package => nunit
the import statement comes before nunit import

Updated https://github.com/theggelund/paket_bug
Added a project called manual_project under MyCallLibrary. The initial issue is reproduced with the project. Used latest released paket 2.24.7

ok I fixed that Nunit issue

I hope it's not completely fixed.


Manual ctrl+f1 links don’t work

I should’ve probably thought this through a little more

Seems to be fine. I think I need a fallback if the sample site's not in the folder though.