Show dialog to switch table views

User Story
As a user, I want an option to view budgets from previous months.
Description
Tables are created dynamically each month when the application is run and named Year-Month (ie. 2024-12) for the year 2024 in December. A dialog will display once a user selects "View Month", allowing the user to select other tables found in the database to view.
Acceptable Criteria

A dialog will appear when the user selects the "View Month" button
The dialog will present two dropdown menus, one for the year and one for the month
The year dropdown will only be populated with years found in the database (ie. the first 4 characters of the table name)
The month dropdown will display all 12 months
Once a user selects a year and month the corresponding table will be displayed in the table view
If the selected year does not have a table for the selected month the user will be prompted that they have no entries for a particular month of the selected year.

Testing
For testing purposes, entries will be created through SQL and tables will be manually generated to ensure previous years and months have been added correctly. The View Month option will be selected with an existing year and month to ensure the correct table is displayed to the user. A table will then be chosen with an existing year with a month where no entry has been made to ensure a warning appears to inform the user that there is no corresponding table. The dialog allowing them to select a year and month is displayed so they can choose an existing table.

Functionality was modified from the original plan. The years drop down is still populated from the years that are found in the database, however the months are populated based on what months have entries in a given year. As well, once the selected table has been loaded the totals displayed for income and expenses will change to display the corresponding totals for the specific table being viewed.
Example:
It there are several tables from 2024 and one table from 2023, after selecting 2023 only one option will appear in the month drop down. However, if there are 3 tables created in 2024 then 3 options will be in the month drop down.
This functionality was committed in commit c10ae58


Fix mei version

some files in the legacy mei2013 2.1 folder had wrong schema refereces or meiversion.num attribute values. This PR fixes that.

I see the pint and agree, nevertheless, it's what the other files still have. So in the sense of unification of the data we could pull this one and create an issue for reworking all of the schema refs

@bwbohl can I close this one?

yes, fixed with #50


Application version JSON serialization

Wanted to expose compileVersion in builds, but then I saw that we have 3 ways of serializing ApplicationVersion. This PR changes it to 1 way.
My understand is that we want to get rid of SourceRevision in favor of sourceUrl, with this PR the only users of ApplicationVersion::source are tests and ZK serialization code.
I've verified that source object, hash and buildNumber are not used anywhere except in 1 internal documentation page, which I'll update.

Are there conflicts on this branch now?

Otherwises LGTM!

All tests still pass on my machine rebased on master at least.


Extrude breadboard harness

This issue tracks the tasks needed to introduce a decent API for integrating Breadboard runtime into various applications.

[x] Stop emitting beforehandler as part of the run (diagnostics only)
[x] fully convert Probes to use report and not be EventTarget
[x] rename 'beforehandler' and 'afterhandler' to 'nodestart' and 'nodeend'
[x] teach inputs and outputs to also emit 'nodestart' and 'nodeend'
[x] write some docs on how to use harness results
[x] move handling streams from ProxyServer/ProxyClient to transport
[x] Convert WorkerRuntime to use the new remote machinery
[ ] introduce createServer API
[ ] introduce createProxyKit API
[ ] introduce createProxyKitServer API
[ ] write some docs on how to configure harness

(running notes) inside worker:

instantiate a dispatcher
if hosts a proxy client, set up proxy client and create proxy kit
if url not provided, set up init server
if hosts a run server, load board
assemble kits, including the proxy kit
let the run server serve

Declaring victory 🚢


Error in the Log

panasonic-viera.0
2021-02-23 16:58:59.184
warn
(4404) State "panasonic-viera.0.volume" has no existing object, this might lead to an error in future versions

panasonic-viera.0
2021-02-23 16:58:59.180
warn
(4404) State "panasonic-viera.0.mute" has no existing object, this might lead to an error in future versions

Sorry, Issue is already open by another user


🔨  Set tab title in .env

For now, Slidesk take the title of the first slide as the title displayed in the tab name.
Sometimes, it's too long, or contains some HTML or MD  characters making it "ugly"
It could be nice to be able to set the title in the .env file to override this behavior when necessary

Available in 2.7.6
https://github.com/slidesk/slidesk/pull/157/commits/4585ca9e6ce83d43b3ef8be1b0540d1b95ac3658


Error when imported data starts with equals sign ("=")

I have noticed that there are problems importing data if the imported text starts with a "=". There may be a better solution than the current one.
#How I did my tests with unsatisfactory results
# Testresult: Errors
"Col1;Col2`r`ntext;=|0" | ConvertFrom-Csv -Delimiter ";" | Export-Excel test1.xlsx -AutoFilter -AutoSize
# Testresult: Displays the '-character
"Col1;Col2`r`ntext;'=|0" | ConvertFrom-Csv -Delimiter ";" | Export-Excel -AutoFilter -AutoSize
# works but with ugly interpretation as text in a formula
"Col1;Col2`r`ntext;=`"=|0`"" | ConvertFrom-Csv -Delimiter ";" | Export-Excel -AutoFilter -AutoSize

In an old issue you wrote ( https://github.com/dfinke/ImportExcel/issues/1007#issuecomment-798961673 ) that Excel interprets this as a formula, but the following test looks promising that there may be a better solution:
# Here is how I expect it to be handled (delimiter is language dependend in your test):
"Col1;Col2`r`ntext;=|0" | ConvertFrom-Csv -Delimiter ";" | Export-Csv test.csv -Delimiter ";" -NoTypeInformation
start excel test.csv

Then save as xlsx has no problem with displaying it as string and reopen the xlsx without errors. The recognition as a formula starts when you try to make changes - then you can decide what you want to do with the data.
There can be more characters than the "=" character or "'" at the beginning, wich can cause problems when importing.
It looks like in the "sharedStrings.xml" is missing "=|0" and maybe some other errors, like counter in this and maybe other files.
Maybe you can fix it so that when you import, it will not crash, but handle it like the xlsx via csv did

please could you post a repro of the issue. data, script etc

A simple example that I have come across is using the output of Compare-Object

Yes. You need to handle that in your PowerShell script. Translate == to equal for example or prefix it with a' which is how native Excel enables you to start data with an = and it does not try to execute it as a function.

Unless I'm doing something wrong, the ' solution is not the same. If I create a workbook manually with a field "Test" with the value "'=" and create a second workbook with the script
[PSCustomObject]@{Test="'="} | Export-Excel -Path "test2.xlsx"

and then import them both, I get
Get-ChildItem . | select -ExpandProperty "FullName" | Import-Excel

Test
----
=
'=

Sorry, not sure what you are attempting. I was focused on the == if you try to enter that in the Excel UI it gives a message. I believe if you use it via PS Excel will see it as corrupt. So the '= is to prevent Excel from interpreting it as a formula

Sorry what I was trying to get across was that if you type = into Excel and let Excel correct it to '= when you read that using this PowerShell module you get = as the value (ie no '). However if you set the value as '= using the PowerShell and then read it back using PowerShell you get '= (ie with a ')


Update instllation and quick start guide -cluster create-

This PR has modified files that have not been added for all versions. Please add them including the text [to be translated] so that the reviewer can easily upload or modify the translated versions. Missing folder/files:
EN - modules/ROOT/pages/installation.adoc
EN - modules/ROOT/pages/quick-start-guide.adoc

This PR has modified files that have not been added for all versions. Please add them including the text [to be translated] so that the reviewer can easily upload or modify the translated versions. Missing folder/files:
EN - modules/ROOT/pages/installation.adoc
EN - modules/ROOT/pages/quick-start-guide.adoc


Fat Jar

Need a fatJar, that contains dependencies.  Try guava at v17 then try exclude & include v18 at client site

Won't do.


(Maybe) turn off Snowpipe in favor of manual COPY INTO statements

One of the benefits of Snowpipe is that you don’t need to directly connect to Snowflake or execute any SQL commands. You just need the ability to upload files to S3. This makes it a great solution for loading very regular, stable data uploads. However, it has been less good when it comes to backfilling or incident recovery due to a couple of features in its design:
It auto-loads every file that lands in the stage
If something with the same name is loaded more than once, it only loads it the first time.
This makes it a bit less flexible than we would necessarily want, since it can really only run a single COPY INTO statement, and cannot do anything extra like deduplication or deleting bad data.
While we could continue using Snowpipe, this would be a good time to consider whether we want to stop using it all-together and include the COPY INTO statements as part of the data relay scripts. This would not be a huge increase in complexity, as the data relay scripts already do some direct queries in Snowflake, and we plan to do more (see the previous task).
Disabling the currently-running pipe would require somebody with SYSADMIN privileges in Snowflake.

Backlogging for now, per @pingpingxiu-DOT-ca-gov he does not feel this is necessary as Snowpipe is stable and not adding complexity. We can discuss later on if we want to move forward on this once other stabilization tasks are completed.


3.0 documentation fixes

Helps with #124.

Woot, much appriciated! - I've copy-pasted the changes onto my super long patch to somehow manage the commit count haha - heaps thanks! 😊

No problem :D


React | HTML Components: Adding component dependency to useImperativeHandle

Fixes #472
Updated dependencies for the useImperativeHandle hook to include the component in its dependency array. This ensures that references to the component are correctly updated whenever it changes, leading to improved synchronization and reliability in the component's behavior.

@rokotyan maybe I'm not testing this currently, but the mapRef in the leaflet-map example https://github.com/f5/unovis/blob/main/packages/shared/examples/advanced-leaflet-map/advanced-leaflet-map.tsx#L23-L32 are still always null for both events.

@lee00678 How are you testing it? I quickly tried it on my end with the dev app, and it worked.

@lee00678 How are you testing it? I quickly tried it on my end with the dev app, and it worked.

I tried it with the leaflet-map gallery example. I just added console.log in those two click functions here. Sees to me myRef.current is always null.

@lee00678 Worked on my end. Have you built @unovis/react before trying?

 @lee00678 Worked on my end. Have you built @unovis/react before trying?

That's odd. I did npm run install:clean. I also just tried to build react separately by doing npm run build:react, still no luck. The code changes are there.

@rokotyan I can't seem to get this fix working.
I added a console.log inside react/src/html-component/leaflet-map/index.tsx to ensure react was built.

 It doesn't seem to be null

You are right. I thought it's returning an empty object. The component is inside. Sorry.


Split password completions using newline

Relates to a bug #955 I raised a long time ago and did nothing about :/
I finally read up a bit more around completions, and in doing so noticed this technique was already being used for key completions (see https://github.com/gopasspw/gopass/blob/master/pkg/completion/zsh/template.go#L44). This PR applies the same technique but to password completions.
Apologies for no test, I had a look at the existing tests but couldn't see any coverage around this sort of thing, which is tricky to test anyway. Happy for suggestions / help if a test is needed.

Codecov Report

Merging #1124 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1124   +/-   ##
=======================================
  Coverage   62.57%   62.57%
=======================================
  Files         184      184
  Lines       10239    10239
=======================================
  Hits         6407     6407
  Misses       3033     3033
  Partials      799      799

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 f930c8e...7f79f01. Read the comment docs.


Upstream generation data is incorrect

Describe the issue
Hi team, I think this will be another one to send upstream to get fixed.
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://developer.data.elexon.co.uk/api-details#api=prod-insol-insights-api&operation=get-datasets-fuelhh-stream
Click on request CCGT generation for 2023-01-24 (full request below in the relevant section).
Inspect JSON output
See error

Expected behaviour
CCGTs have long ramping times and should be fairly consistent one SP to the next.
Observed behaviour
Some settlement periods have erroneous data. In the example below, we have smooth generation at around 23GW for many settlement periods, then suddenly almost 20GW of generation vanishes at 11am, but the very next settlement period we're back to 22GW. This simply isn't possible with CCGTs.
[{
    "dataset": "FUELHH",
    "publishTime": "2023-01-24T12:30:00Z",
    "startTime": "2023-01-24T12:00:00Z",
    "settlementDate": "2023-01-24",
    "settlementPeriod": 25,
    "fuelType": "CCGT",
    "generation": 23185
}, {
    "dataset": "FUELHH",
    "publishTime": "2023-01-24T12:00:00Z",
    "startTime": "2023-01-24T11:30:00Z",
    "settlementDate": "2023-01-24",
    "settlementPeriod": 24,
    "fuelType": "CCGT",
    "generation": 22841
}, {
    "dataset": "FUELHH",
    "publishTime": "2023-01-24T11:30:00Z",
    "startTime": "2023-01-24T11:00:00Z",
    "settlementDate": "2023-01-24",
    "settlementPeriod": 23,
    "fuelType": "CCGT",
    "generation": 3896
}, {
    "dataset": "FUELHH",
    "publishTime": "2023-01-24T11:00:00Z",
    "startTime": "2023-01-24T10:30:00Z",
    "settlementDate": "2023-01-24",
    "settlementPeriod": 22,
    "fuelType": "CCGT",
    "generation": 22018
}, {
    "dataset": "FUELHH",
    "publishTime": "2023-01-24T10:30:00Z",
    "startTime": "2023-01-24T10:00:00Z",
    "settlementDate": "2023-01-24",
    "settlementPeriod": 21,
    "fuelType": "CCGT",
    "generation": 22637
}]

HTTP requests/responses
GET https://data.elexon.co.uk/bmrs/api/v1/datasets/FUELHH/stream?settlementDateFrom=2023-01-24&settlementDateTo=2023-01-25&fuelType=CCGT HTTP/1.1

Cache-Control: no-cache

Additional context
Add any other context about the problem here, e.g. the browser you are using (if applicable).
Checks
Please answer the following questions for yourself before submitting an issue.

[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I'm reporting the issue to the correct repository
[x] I'm reporting a problem and not a feature request
[x] I have added the correct labels @zghanty I believe the template issue markdown needs this line removed still.
[x] I have added screenshots or other relevant evidence (if applicable)

Just to add some more info, something similar was observed in the case of NUCLEAR and WIND fuel types. There was a reported drop of around 4GW and then the very next SP, it was recovered.

Taking a look at the five-minute iterations, noticed that for SP23, only the first interval reported a non-zero value and after 11:05 there was some data outage. FUELHH reported values were evaluated with just the initial five minutes of data, 23377MW divided by 6 giving the erratic 3896 MW for CCGT.

@Samreay @trader-short : Sorry for the delay, NGESO colleague confirmed " For issue 15 related to missing generation data on 24 January 2023 we have identified this was probably caused by a metering fault, and these files are rarely updated post event"

That is frustrating to hear from ESO. Publishing data comes with a responsibility to make sure its correct and fix mistakes, surely. Its hard to automate checks on these numbers in many cases (apart from simple things like "nuclear shouldnt drop to zero randomly", so I'm worried that continuing to use this data source in the future represents a risk if there aren't any checks and monitoring, nor correcting, done on ESOs side. I'll think this over and see if any easy wins come to mind. Thanks for the update, Zaahir :)


Add template for releasing standalone components

Description
Add template for releasing standalone components
Issues Resolved
NA
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2734 (0386461) into main (925d126) will decrease coverage by 0.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2734      +/-   ##
==========================================
- Coverage   94.10%   93.77%   -0.34%
==========================================
  Files         156      156
  Lines        4208     4208
==========================================
- Hits         3960     3946      -14
- Misses        248      262      +14

Impacted Files
Coverage Δ

src/system/temporary_directory.py
64.44% <0.00%> (-28.89%)
:arrow_down:

src/system/os.py
93.75% <0.00%> (-6.25%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Hi @bbarani ,
Can you fix yaml lint issues?
Thanks!

Hi @bbarani , Can you fix yaml lint issues? Thanks!

Fixed it in https://github.com/opensearch-project/opensearch-build/pull/2737 as it was causing multiple merge conflicts.


Text of source language copied over to target language

Currently whenever I add a new string to translate I have to hunt down the new string in the translated files because they are "pre-filled" with the text of the source language. This might be a shortcoming of the translation program I use (Poedit), but the behavior seems irritating to me nonetheless.
I am taking the following steps:

Add a text in my primarily german application:

<p i18n="@@extractTest">Dies ist ein deutscher Text</p>

I then run the following commands:

ng xi18n --i18n-locale de --output-path locale
ng run "blattwerkzeug-client:xliffmerge"

The updated messages.en.xlf now contains the following node:

<trans-unit id="extractTest" datatype="html">
   <source>Dies ist ein deutscher Text</source>
   <target state="new">Dies ist ein deutscher Text</target>
   <context-group purpose="location">
      <context context-type="sourcefile">app/front/templates/about.html</context>
      <context context-type="linenumber">20</context>
   </context-group>
</trans-unit>

As yoi can see the source text was copied over to the target. I do appreciate that the state is set to new, but it seems that at least Poedit simply ignores this. When I open the merged file I am greeted by the following sight:

Note that Poedit unhelpfully says "Translated: 95 of 95 (100%)" and that it is visually hard to find the string that requires attention. I will file a bugreport with Poedit as well, but would it be possibly to introduce a flag that disables this copying behaviour in the meantime? With Poedit this moves the untranslated unit to the top and also correctly updates the statusbar.

FYI, Poedit 2.2.4 handles state="new" state correctly. I would argue that it is indeed not a great behavior to pre-fill the same string (which is never a correct translation), but as long as state="new" (or state="initial" in XLIFF 2) is used, it's a minor quibble.

Oh, sorry for that. Seems that I have gotten too much in the habit of assuming that Arch Linux always ships the most recent version. At least in this case it doesn't.
Nevertheless I would still prefer to make the "copy" behaviour configurable. I guess it has been introduced to provide a fallback translation?

There is the flag "useSourceAsTarget" that can be used to switch off this behaviour (by setting it to "false").
Originally the behaviour is there because Angular denied to aot compile an app if there were missing translations. And in this case it is better to have the original untranslated text to allow compilation.

Wow, thanks to both of you for solving my problem. I have no idea how I didn't stumble over that aptly named flag ¯_(ツ)_/¯


hfsutils: init at 3.2.6-14

Motivation for this change
HFS utilities.

init at 3.2.6-14.  "-14" is debian patchset used.
touch timestamp files to avoid autoconf
after patches, which is good because that fails.
(patches fix the generated files too so all good)
hfsutils: fix installation commands, fix build

These are especially useful as hfsprogs long since
stopped supporting HFS (as did Apple).
(ours still do, only because they're so very very ancient ;)).
I tackle updating those in 88bf051a78dd774370fade57dbb268633861a34f
(and likely a PR coming shortly after this) but I'm not entirely sure
it's worth upgrading those or not.
But will probably PR it for visibility and in case that does seem
like a good idea to anyone :).
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)

Superficially they appear okay :).

[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Closing because author did not address the review feedback in the last year.

Closing because author did not address the review feedback in the last year.


first version

References
Add references/links to any related issues or PRs. These may include:

Fixes #SPR1-2421
Requires DSpace/DSpace#SPR1-2421 (if a REST API PR is required to test this)

Description
Customizing the UI, Branding home page
Instructions for Reviewers
List of changes in this PR:
Changed the Logo
Changed the Theme
Removed Login button & Language switch on home page
Removed Statistics Menu Item
Customized fonts
Added an about us page
Moved Search bar to the content box
Include guidance for how to test or review your PR. This may include: steps to reproduce a bug, screenshots or description of a new feature, or reasons behind specific changes.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[ ] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[ ] My PR passes TSLint validation using yarn run lint
[ ] My PR doesn't introduce circular dependencies
[ ] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[ ] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[ ] If my PR includes new, third-party dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.

Please try to make better commit messages.
Recommendation is not enough explanation of what you are doing


Want to know what code is used to implement the attention visualization in the figure below?

Hi, you could refer to #78, the code is in this file.

Thanks, I already know how to run it, thank you very much


Replace VSync with PresentMode

Objective
Enable the user to specify any presentation modes (including Mailbox).
Fixes #3807
Solution
I've added a new PresentMode enum in bevy_window that mirrors the wgpu enum 1:1. Alternatively, I could add a new dependency on wgpu-types if that would be preferred.

Thanks. I agree with this approach, and the changes are perfectly fine. I'd like to try and find a more approachable naming convention ("Present" is extremely overloaded), but once that's cleaned up this LGTM.

Done

bors r+


Use different count methods depending on type of relation for batch delivery

Context
We switched to 'length' rather than 'count' because 'count' does not give us the anticipated response for collections that have been grouped. This will have a very serious impact on performance when we are sending many thousands of emails.
Changes proposed in this pull request
This PR addresses the problem by using a different count_method depending on whether the collection you are passing to BatchDelivery is grouped or not grouped.
Guidance to review
Link to Trello card
https://trello.com/c/yVWA4TlR
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault

 Fix big Performance issue for End of cycle tasks

Can we have the count being the default of batch delivery and only have the grouped batch delivery added and change in the two places that uses? Not a blocker though but I'd like to discuss before approving.

In general, I think it is best to be as explicit as possible, so to me it seems better to make sure the developer knows they need to be deliberate about what BatchDelivery class to use. With a default, it would be less clear to me. But functionally it is the same so if you feel strongly about it, I will change it.

 Fix big Performance issue for End of cycle tasks


feature(forms-submission): re-consider current implementation

Is your feature request related to a problem? Please describe.
Form supports was implemented a while back in https://github.com/netlify/cli/pull/780.
The current implementation looks at requests and tries to detect if they are form submissions (by looking at the request method and content-type), and invoke the submission-created handler accordingly.
That implementation as a few disadvantages:

It can't include any information that wasn't sent by the browser in the functions event object. See issue
It will detect any POST request with a matching content-type as a form submission and trigger the handler, where it should only track forms submissions.

Describe the solution you'd like
Ideally the CLI should parse the HTML to identify forms and link those to submissions
Describe alternatives you've considered
Keeping things as they are
Additional context
See https://github.com/netlify/cli/issues/447

Adding #1610 as another place where our CLI handling of forms is different from what we use on netlify.com
When we discussed it with the team, the solution we saw was redesigning the way we handle form submissions.


Write questions for industry

After ticket #99, we will need an interview guide for conversations with industrial operators.
Acceptance Criteria:

[x] Create a template document
[x] Write out goals of interview
[x] Create questions
[x] Share with project team

You are on a roll this sprint! Thank you @jaimiebutton !


Allow lists in query strings

There are use cases where you may want to add an array of values to a query string. I.E. ids[]=1&ids[]=2.
Previously the mergeEntries function was simply replacing any item in the search params with an identical name. This had the effect of only including the final entry in the params.
This updates the mergeEntires function to just do an append rather than a replace if the entry name contains [].

thanks @dphaener!


getDocumentSymbols returns incorrect ordered symbol list in flatten mode

monaco-editor version: 0.24.0
**Browser: Chrome 91.0.4472.77
**OS: macOS 12.0 Beta
Playground code that reproduces the issue:
https://alauda.github.io/k8s-form-in-action/demo/
Change the yaml content like the following:
containers:
  - name: nginx
    ports:
      - containerPort: 80

open devtool, execute window.DEBUG = true, click the yaml content you will get

As you can see, name symbol comes before 0 which is incorrect obviously, while containerPort comes after 0 which is correct.
What means when a symbol has multi children, the flatted result is incorrect.
If you want to run this sample locally, you can check https://github.com/alauda/k8s-form-in-action
run yarn && yarn dev and then visit http://localhost:4200
The related source code is at https://github.com/alauda/k8s-form-in-action/blob/master/src/app/demo/component.ts#L168

I'm sorry it doesn't seem to be working, but with so many requests that we get here and in the VS Code issue tracker, we just cannot provide the support we would like to offer.
That said, if you can track down your issue to a specific problem of the Monaco editor, we are more than welcoming a PR by the community!
I'll close this issue, but if you have specific follow up questions, we'll try to help as good as we can given the limited resources that we have.
Thanks for your understanding!

@hediet I don't understand what's your meaning, it is a bug of getDocumentSymbols in vscode https://github.com/microsoft/vscode/blob/main/src/vs/editor/contrib/documentSymbols/documentSymbols.ts
The non-flatten result of getDocumentSymbols(model, false, NEVER_CANCEL_TOKEN) is correct, so I can fix it on our side by using a custom util:
/**
 * fix https://github.com/microsoft/monaco-editor/issues/2517 temporarily.
 * Ideally, we should use the `flat` parameter of `getDocumentSymbols` instead.
 */
const flatSymbols = (symbols: languages.DocumentSymbol[]) => {
  const flattenSymbols: languages.DocumentSymbol[] = [];
  for (const symbol of symbols) {
    flattenSymbols.push(symbol);
    if (symbol.children?.length) {
      flattenSymbols.push(...flatSymbols(symbol.children));
    }
  }
  return flattenSymbols;
};

Is this an invalid bug? My English is not good enough to understand why you close it.

I guess this sort should use a stable sort:
https://github.com/microsoft/vscode/blob/b5d6d7e62e3c2ae4faec100a81f7ba16e1418b7c/src/vs/editor/contrib/documentSymbols/outlineModel.ts#L463
See https://github.com/microsoft/vscode/issues/125840.

@hediet Yeah, thanks for raising the new related issue in vscode.

@JounQin unfortunately the sorting operation should not be an issue, as it is stable.
Can you find out what the issue is in the vscode codebase? I'll probably wont find time this week to look into it.

Sorry, I'm not familiar with vscode codebase, but I'll try to find out when I'm free tomorrow.

Can't offer much help here, but as far as I know the Monaco editor does not provide any language intelligence for YAML out-of-the-box. So where are these symbols computed? Where does the DocumentSymbolProvider come from? I would start there and look into the ordering coming directly from that provider. That would probably also help you in providing a minimal repro if there should be a bug on Monaco's side.

@spshnke I'm using monaco-yaml.

@spahnke Here you go: https://github.com/JounQin/test/blob/monaco-editor/src/App.vue#L35
It's use monaco manually at https://github.com/JounQin/test/blob/monaco-editor/src/components/monaco-editor/setup-monaco.ts

I did some digging into this...
const value =
    `containers:
  - name: nginx
    ports:
      - containerPort: 80`;

monaco.editor.create(document.getElementById("container"), {
    value: value,
    language: "yaml"
});

const symbols = [
    {
        name: "containers",
        range: {
            startLineNumber: 1, startColumn: 1,
            endLineNumber: 4, endColumn: 26
        }
    },
    {
        name: "container 0",
        range: {
            startLineNumber: 2, startColumn: 5,
            endLineNumber: 5, endColumn: 1
        },
    },
    {
        name: "name",
        range: {
            startLineNumber: 2, startColumn: 5,
            endLineNumber: 2, endColumn: 16
        }
    },
    {
        name: "ports",
        range: {
            startLineNumber: 3, startColumn: 5,
            endLineNumber: 5, endColumn: 1
        }
    },
    {
        name: "ports 0",
        range: {
            startLineNumber: 4, startColumn: 9,
            endLineNumber: 4, endColumn: 26
        }

    },
    {
        name: "containerPort",
        range: {
            startLineNumber: 4, startColumn: 9,
            endLineNumber: 4, endColumn: 26
        }
    }
]

monaco.languages.registerDocumentSymbolProvider("yaml", {
    provideDocumentSymbols: (model, token) => {
        return symbols.map(s => {
            return {
                name: s.name,
                range: s.range,
                selectionRange: s.range,
                kind: monaco.languages.SymbolKind.Object
            }
        });
    }
});

You can reproduce this issue by running the above JavaScript code in the Monaco Editor Playground. Use F1 in the editor and use @ to jump to the symbol. In my screenshot I've added some whitespace after creating the editor so you can see both the code and the list of symbols for illustrative purposes.

const ranges = [
    {startLineNumber: 1, startColumn: 1, endLineNumber: 5, endColumn: 1}, // containers
    {startLineNumber: 2, startColumn: 5, endLineNumber: 5, endColumn: 1}, // 0-th element of containers
    {startLineNumber: 2, startColumn: 5, endLineNumber: 2, endColumn: 16}, // name
    {startLineNumber: 3, startColumn: 5, endLineNumber: 5, endColumn: 1}, // ports
    {startLineNumber: 4, startColumn: 9, endLineNumber: 4, endColumn: 26}, // 0-th element of ports
    {startLineNumber: 4, startColumn: 9, endLineNumber: 4, endColumn: 26}, // containerPort
]
ranges.sort((a, b) => {
    return monaco.Range.compareRangesUsingStarts(a, b);
});
console.log(ranges);

The reason the symbols are getting "moved around" is because of how VS Code is sorting them in compareRangesUsingStarts(*). You can see this change by running the above JavaScript code in the Monaco Editor Playground. The calling code can be found here and you can keep walking backwards from there as desired.
@JounQin I should note to you that the getDocumentSymbols(*) function is not considered API based on the contents of the vscode.d.ts file or the monaco.d.ts file so you shouldn't be using it to get an outline of the symbols from a model.

@rcjsuen thanks a lot for investigating! Did you spot the bug in the code base? Is there something wrong in compareRangesUsingStarts?

should note to you that the getDocumentSymbols(*) function is not considered API based

This is right. We are open for PRs though if they fix reproducible bugs.

Is there something wrong in compareRangesUsingStarts?

@hediet Depends who you ask. The function may be working as intended but it's the document symbol provider that should be using a different algorithm for sorting purposes.
/**
  * A function that compares ranges, useful for sorting ranges
  * It will first compare ranges on the startPosition and then on the endPosition
  */

The documentation for the function doesn't tell us whether it considers "longer" or "bigger" ranges as being "greater" or "lesser" for comparison purposes.

I should note to you that the getDocumentSymbols(*) function is not considered API based on the contents of the vscode.d.ts file or the monaco.d.ts file so you shouldn't be using it to get an outline of the symbols from a model.

@rcjsuen Thank you, I understand it, and I'm just using it for a demo example only.
But obviously, it is a bug of vscode/monaco-editor, right?

Let's track at https://github.com/microsoft/vscode/issues/138502


Slack bot for releases

It'd be great to automate our communications around releases for both Slack and emails.

This looks like it would work well: https://www.npmjs.com/package/semantic-release-slack-bot

Potentially closed via slack message.


Is there a way to give a GeoJSONLayer a name?

I am rendering some geoJson polygons in my app using the GeoJSONLayer component. I have provided some fill expressions and things are working well.
I noticed this layer is by default called geojson-1-fill and it seems that neither Layer nor the GeoJSONLayer component allow you to change this.
Is there a way to specifically give that layer a name?

Layer has an id prop and geojsonlayer has layerOptions

Ah ok. So I did layerOptions={{id: 'dskjsdjk823uj3jbkjxiugd87s87gsiuk'}} but now I am getting:
    at i.addLayer (mapbox-gl.js:19037)
    at r.addLayer (mapbox-gl.js:22876)```

Is it somehow adding this layer over and over again?

there might be a bug when it rerenders the GeoJSON... are you having the component rerender?

Yes, I do. That is I am rendering something that is a changing collection of GeoJson entities. So this error is throughout the app if I do assign a layer name ... 🤔
Is there a way around it? I guess all I need is to set a specific layer so that I can use queryRenderedFeatures and restrict the search to a specific layer...

any update?


[BUG]: Disallowing 0-dimensional embeddings

Refs: #1698
Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Added validation for 0-dimensional embeddings

Test plan
How are these changes tested?

[x] Tests pass locally with pytest for python, yarn test for js

Documentation Changes
N/A

@HammadB, this one is good to go, only cluster tests fail.


验证在 6/7/8/lts/node 下的功能正确性

Pull Request Test Coverage Report for Build 91

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

Totals

Change from base Build 87:
0.0%

Covered Lines:
1504

Relevant Lines:
1504

💛  - Coveralls


Exchange fee provider

introduce a exchangeFee interface to manage the calculation of the exchange fee

closed with v0.2.4, commit 517752fca39652982626b1ea1ec7fedf7e735776


DataTarget.ClrVersions crashes when reading live process v0.8.31

The exact same code, parameters and process work using v0.8.30. So it seems like a new problem.
DataTarget dataTarget = DataTarget.AttachToProcess(processId, timeoutMillis, attachFlag);
ProcessId is process to a running .NET program.
timeoutMillis = 5000
attachFlag = Passive
Then accessing dataTarget.ClrVersions crashes on this line:
if (ReadMemory(moduleBase + 0x3c, buffer, buffer.Length, out read) && read == buffer.Length)
Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
  at Microsoft.Diagnostics.Runtime.LiveDataReader.GetFileProperties(UInt64 moduleBase, UInt32& filesize, UInt32& timestamp) in c:\Microsoft\clrmd\Microsoft.Diagnostics.Runtime\datatarget.cs:line 2522    at Microsoft.Diagnostics.Runtime.LiveDataReader.EnumerateModules() in c:\Microsoft\clrmd\Microsoft.Diagnostics.Runtime\datatarget.cs:line 2378    at Microsoft.Diagnostics.Runtime.DataTargetImpl.InitModules() in c:\Microsoft\clrmd\Microsoft.Diagnostics.Runtime\datatarget.cs:line 1372    at Microsoft.Diagnostics.Runtime.DataTargetImpl.EnumerateModules() in c:\Microsoft\clrmd\Microsoft.Diagnostics.Runtime\datatarget.cs:line 1350    at Microsoft.Diagnostics.Runtime.DataTargetImpl.get_ClrVersions() in c:\Microsoft\clrmd\Microsoft.Diagnostics.Runtime\datatarget.cs:line 1220
Let me know if you need more information, or would like to see the error in action via a join.me meeting. I can host the meeting.

Note: This error occurs when using the source code zip you provided in your latest post. However, it seems the error does not occur when using the DLL from nuget.

Thank you for the bug report!  I will not be able to take a look until this weekend, but I'll be sure it's fixed asap, and certainly before publishing a NuGet package.

Hello!  This issue is being closed due to its age.  The codebase has moved significantly since this was filed.  Please feel free to open a new issue for ClrMD about this issue if it still reproduces on ClrMD 1.0.  (Also, I apologize for having not looked at or resolved this earlier and for resolving it now.  In order for me to be able to make some progress on the growing issue count in ClrMD I need to clear the backlog down to a manageable set of issues, but do feel free to file a new one if this is still a problem.)


CORS error when intercepted browser request with url override gets a redirection response

Steps to reproduce
Tell us about your environment:

Puppeteer version: v10.0.0
Platform / OS version: Debian 9 stretch (Docker container)
URLs (if applicable): -
Node.js version: v14.17.0

What steps will reproduce the problem?
When intercepting browser's requests and overriding the original url (as described here https://github.com/puppeteer/puppeteer/blob/v10.0.0/docs/api.md#httprequestcontinueoverrides) changing the domain of the original request, if the called service returns a redirection (to the same  overridden domain) I get the following CORS error.
Access to fetch at 'http://overridden-domain/path (redirected from 'http://original-domain/path') from origin 'http://origin' has been blocked by CORS policy: The 'Access-Control-Allow-Origin' header has a value 'http://origin' that is not equal to the supplied origin. Have the server send the header with a valid value, or, if an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

CORS headers are correctly set and, if I remove the url override everything works fine.
It seems that the issue is caused by the browser seeing a redirection from the original domain to the overridden domain which is not permitted on CORS requests.
My goal is to override my SPA calls to the api calling the internal Docker api name instead of the public one (to speed up the page rendering). Everything works fine until I don't get any redirection from the api.
In my codebase I do something like the following
function getPageHtml(page, url) {
  await page.setRequestInterception(true)
  page.on('request', req => req.continue({ url:  req.url().replace(/^http:\/\/original-domain/, 'http://overridden-domain') })

  await page.goto(url, {
    waitUntil: 'networkidle0',
    timeout: 30000,
  })

  return await page.content()
}

What is the expected result?
The Puppeteer's documentation states If set changes the request URL. This is not a redirect. The request will be silently forwarded to the new URL. For example, the address bar will show the original URL. so I'm expecting that when changing the original request domain, I wouldn't get a CORS error if a redirect response is returned to the same overridden domain.
I understand why the issue is happening and I understand that it's the correct browser behavior as, indeed, it's seen as CORS request redirection to a different domain but, is there a way to get around it and still be able to use the internal domain name to make requests to the api?
Thank you in advance :slightly_smiling_face:

I have the same problem, request failed when override request domain.

I'm still experiencing this today


[WIP] add subnets precompile

Description
Related Issue(s)

Closes #[1075]

Type of Change

[ ] 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)
[ ] Documentation update
[ ] Other (please describe):

Breaking Change
If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.
Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have run cargo fmt and cargo clippy to ensure my code is formatted and linted correctly
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)
Please include any relevant screenshots or GIFs that demonstrate the changes made.
Additional Notes
Please provide any additional information or context that may be helpful for reviewers.

https://github.com/opentensor/subtensor-js-tests/blob/main/src/evm/eth.subnets.test.js e2e test case added.


chore: remove date-fns

This PR removes date-fns completely. Orphaned package react-dates is also moved to react-admin-date-picker. Unmet peer-dependency react-with-direction is added.
Breaking Changes:

useLocaleName(); is removed and should be replaced with const intl = useIntl(); const localeName = intl.locale;
LocaleContextProvider is removed and should be replaced with IntlProvider
FormatLocalized is removed and should be replaced with https://formatjs.io/docs/react-intl/components#formatteddate and https://formatjs.io/docs/react-intl/components#formattedtime

The "breaking-changes" statements from the PR-description should be committed to code within this PR (into packages/react/admin/changelog.md)
I think it's a good idea to create the changelog.md now.


try_spdm_challenge() restricts SPDM responder to having one slot populated

try_spdm_challenge() unnecessarily restricts an SPDM responder to having one slot populated.  The last check against param2 only allows 1 bit to be set in the below code snippet.
    if (spdm_response.header.spdm_version == SPDM_MESSAGE_VERSION_11 && slot_id == 0xFF) {
        if (auth_attribute.slot_id != 0xF) {
            return RETURN_DEVICE_ERROR;
        }
        if (spdm_response.header.param2 != 0) {
            return RETURN_DEVICE_ERROR;
        }
    } else {
        if ((spdm_response.header.spdm_version == SPDM_MESSAGE_VERSION_11 && auth_attribute.slot_id != slot_id) ||
            (spdm_response.header.spdm_version == SPDM_MESSAGE_VERSION_10 && *(uint8 *)&auth_attribute != slot_id)) {
            return RETURN_DEVICE_ERROR;
        }
        if (spdm_response.header.param2 != (1 << slot_id)) {
            return RETURN_DEVICE_ERROR;
        }
    }

@bhenning10 and I talked, the SPDM spec doesn't restrict param2 to having only one bit set; otherwise it shouldn't be a mask.  In reality, param2/slot_mask needs to be passed back to the libspdm client as a new parameter to spdm_challenge() to implement the proper policy, unless the expected slot ID bit is not properly set. which is a device error.  A client needs to decide what to do if new slots show up, and if slots disappear.

Slot mask. The bit in position K of this byte shall be set to 1b if and only if slot
number K contains a certificate chain for the protocol version in the SPDMVersion
field. Bit 0 is the least significant bit of the byte. This field is reserved if the public
key of the Responder was provisioned to the Requester in a trusted environment.

11/15/2021 - approved.

11/22/2021 - approved.


Systemd wiki example error(?)

I've just created a new install of PowerDNS-Admin on a new centos7 server (fully updated to the current version).
When I create the systemd service as shown in the wiki article it fails to start with permission errors unles the owner:group is change on the following files:
chown -R powerdnsadmin:powerdnsadmin  /opt/web/powerdns-admin/app
Is this a bug or have I missed something?

powerdnsadmin is just an example username for systemd config. You have to make sure that user has write permission to the PowerDNS-Admin directory to write log files.

So the answer was yes, I missed something!  My mistake and sorry for the noise. :)


fix: upgrade wait-for-expect to 1.0.0

What:
Dep upgrade

Why:
I noticed I got both 0.4.0 and 1.0.0 in my lockfile when installing react-testing-library

How:
GitHub UI

Checklist:

[ ] Documentation
[ ] Tests
[x] Ready to be merged
[ ] Added myself to contributors table

If you rerun the build, it's gone, IIRC. You retrigger that specific build, you don't generate a new one

Are there breaking changes in the library?

Good question. If it works it works by passing the tests, it works - the only change is in the way we export the function (so you no longer have to use .default in some cases).
Since we import (not require) it doesn’t matter.
It is breaking - hence the change of the version, but not in our (dom-testing-library) case.

Super 👍 thanks friends!

:tada: This PR is included in version 3.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adjust Docs on Select

Adjust documentation on StreamExt select combinator to indicate behavior when
one of the stream ends early.
Squashed, updated version of https://github.com/rust-lang-nursery/futures-rs/pull/1183 after comments.
@MajorBreakfast @cramertj

Thanks!


pythonPackages.pywinrm: 0.1.1 -> 0.2.2

Motivation for this change
Bump version to latest upstream release and remove the already broken/optional Kerberos support.
Things done

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

[x] NixOS
[x] macOS
[x] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The latest version of pywinrm has a new dependency on requests_ntlm which requires the lower-level ntlm-auth. I've removed the optional kerberos dependency, as it was already broken, now requires additional dependencies to function, and I couldn't figure out a clean way of manipulating the LD_LIBRARY_PATH (or patch things with the absolute path) to fix the original error:

error: libgssapi_krb5.so: cannot open shared object file: No such file or directory

The other changed dependencies were just making things accurate with the upstream requirements. Here's the pipdeptree output, if you're curious:
pywinrm==0.2.2
  - requests [required: >=2.9.1, installed: 2.14.2]
  - requests-ntlm [required: >=0.3.0, installed: 1.0.0]
    - ntlm-auth [required: >=1.0.2, installed: 1.0.3]
      - six [required: Any, installed: 1.10.0]
    - requests [required: >=2.0.0, installed: 2.14.2]
  - six [required: Any, installed: 1.10.0]
  - xmltodict [required: Any, installed: 0.11.0]

The tests I disabled require a local IIS server and network access.

The latest version of pywinrm has a new dependency on requests_ntlm which requires the lower-level ntlm-auth. I've removed the optional kerberos dependency, as it was already broken, now requires additional dependencies to function, and I couldn't figure out a clean way of manipulating the LD_LIBRARY_PATH (or patch things with the absolute path) to fix the original error:

Typically it tries to load the library somewhere with ctypes or cffi. In that function, don't let it use the name of the library but instead pass in the absolute path to the library.

Thank you for the feedback! I force pushed the branch after added the following changes:

moved the three libraries into dedicated files under pkgs/development/python-modules/
switched to using the proper checkInputs
instead of fetchurl, refactored to use fetchpypi when possible and fetchFromGitHub for ntlm-auth since they strip out the tests directory in the PyPI tarball

@FRidh were you wanting any other changes here? Everything is working properly for me when using it locally.
I don't have any real experience with the Kerberos ecosystem, and I did try to trigger the runtime error with the old pywinrm package version (which is why it got labeled as broken) by following the basic instructions laid out in https://github.com/diyan/pywinrm/issues/21, but couldn't reproduce the problem. If you want, I can try to add back in the kerberos support as an optional install and package up the new requests-kerberos dependency, but again, I'm not confident that I can test it properly to know if it's working.


Update k8s.io/ja/docs/setup/independent/control-plane-flags/

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
Outdated content content/ja/docs/setup/independent/control-plane-flags.md.
content/en/docs/setup/independent/control-plane-flags.md 2 insertion(+)

Proposed Solution:
Update content content/ja/docs/setup/independent/control-plane-flags.md.
For details, please execute the following command.
git diff 551489f 7b07d19 -- content/en/docs/setup/independent/control-plane-flags.md

Page to Update:
https://kubernetes.io/ja/docs/setup/independent/control-plane-flags/

やります

/close


Update to netstandard 2.0

Updated WebApiContrib.Core.Formatter.Yaml to 2.0

thanks!
I merged the other PR and now this one has conflicts, do you mind rebasing? thanks

Done :)

☀


Backwards compatible way of importing reverse

CC: https://github.com/jp74/django-model-publisher/pull/35/files

Coverage remained the same at ?% when pulling f1e4a0bfb1cd1fc99ca51a806b71676b301fb640 on maszhin:master into 3d90821b1c781c0741b92aec48bd51b9fa7c6478 on wearehoods:master.

Thanks for contribution.


Incorrect Unnecessary Parentheses on lambda

results = resultsArray <#> (\r -> (un ShapeID r.id /\ r)) # Object.fromFoldable

Somehow the opposite is also true:

In this one they are actually unnecessary.


x/tools/cmd/godoc: Path mapping of module name is incorrect

What version of Go are you using (go version)?

go version go1.14 linux/amd64

What version of godoc are you using?
Latest version. Installed with:

GO111MODULE=off go get -u golang.org/x/tools/cmd/godoc

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/starptech/.cache/go-build"
GOENV="/home/starptech/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/starptech/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/mnt/ssd/repositories/golang-dynaday/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build252071029=/tmp/go-build -gno-record-gcc-switches"

What did you do?
In my organization, we host on Gitlab and projects are namespaced with their "groups". My module and VCS name are git.$organization.com/$group/my-project.
When I run godoc -http=:6060 in root and click on the package git.$organization I see the following error in the terminal:
using module mode; GOMOD=/mnt/ssd/repositories/golang-dynaday/go.mod
2020/03/07 00:03:31 cannot find package "." in:
        /src/$organization

This results in a broken UI and thus my packages aren't listed correctly.
go list -m all list correctly all packages and modules but godoc doesn't display them.
The workaround is to remove $group name from the module name and update all places accordingly. I'm not aware of such a constraint.
go.mod
module git.$org.com/internal/golang-dynaday

go 1.14

replace git.$org.com/internal/golang-dynaday/submodule => ./submodule

require (
	git.$org.com/internal/golang-dynaday/submodule v0.0.0-00010101000000-000000000000
)

What did you expect to see?
I would expect to see all my packages and modules as listed with go list -m all
What did you see instead?
Only one entry with the name of my organization git.$organization.com can be seen in "Third party". It looks like the current implementation can't handle arbitrary module names especially when they can't be mapped to the local filesystem.
According to @dmitshur this can be related to https://golang.org/cl/205661.
Related: #26827

@dmitshur I tested your patch but it didn't work. Only the error is gone but the packages are still not recognized.

Thanks for the detailed report and testing out that CL.
To clarify, when you write $organization or $group, you mean for those to be substituted by an arbitrary string, such as example or aaa. There isn't a $ character in the import paths, is there?

To clarify, when you write $organization or $group, you mean for those to be substituted by an arbitrary string, such as example or aaa. There isn't a $ character in the import paths, is there?

Yes, arbitrary strings.

godoc was deprecated in CL 349051


Add mdx-driven mli documentation comments

Closes https://github.com/mt-caret/polars-ocaml/issues/5.
Beautiful docs:

I've uploaded the current version of the docs here: https://mt-caret.github.io/polars-ocaml/
Also has the following related changes:

fix bytecode binding generation in polars-ocaml-macros (more details here: https://github.com/tizoc/ocaml-interop/issues/52)
set requisite environment variables so that expect tests in both Rust and OCaml are more stable

I think odoc only generates documentation for things specified in opam files, which currently is grossly underspecified. I fixed this in https://github.com/mt-caret/polars-ocaml/pull/32 but I'm not super eager to rebase through a million PRs so I'll just wait until the PRs are merged before doing more work here.

While this is nowhere near complete, I'm opening this to review for merging since:

there's also a fair amount of functionality in this PR
I want to nudge folks when adding new things to also add docs, so I'm not stuck with the Sisyphean task of adding docs for a continuously growing list of things!

I'll divy up the work of adding more docs into more PRs.


[dart fix] dart fix keeps adding dependencies even when not requested

When you have any transitive dependdencies (hosted or git or path), it adds any dependency as:
 xxxx: any
even for non pub.dev hosted packages and even when MISSING_DEPENDENCY is not specified
To reproduce (tested on dart 3.4.4 on ubuntu 22.04):

Create a simple dart project: dart create dartfix_exp
Modify the dartfix_exp_test.dart with the following:

import 'package:path/path.dart';
import 'package:test/test.dart';
import 'package:yaml/yaml.dart';

void main() {
  test('path', () {
    expect(url.join('a', 'b'), 'a/b');
  });
}

run dart fix --dry-run

It should properly report the undeclared dependencies (but present as transitive through the test package)
To fix an individual diagnostic, run one of:
  dart fix --apply --code=MISSING_DEPENDENCY
  dart fix --apply --code=unused_import

If I run
dart fix --apply --code=unused_import
it properly removes the unused package:yaml/yaml.dart but is also adds the dev dependency path: any although I did not ask for. And as I said it even does that for non hosted dependency which is wrong and requires deleting the modification.

@keertip


CLOUDSTACK-8615 :skip all tests in test_ps_limits.py for lxc

issue
All test cases are failing due to unsupported storage type
solution
skip test if supported storage type(for lxc hyp) is not present

Test assign Vm to different account ... SKIP: unsupported storage type
Test assign Vm to different account ... SKIP: unsupported storage type
Stop attach and detach volume from VM ... SKIP: unsupported storage type
Stop attach and detach volume from VM ... SKIP: unsupported storage type
Test create multiple volumes ... SKIP: unsupported storage type
Test create multiple volumes ... SKIP: unsupported storage type
Test create snapshot and templates from volume ... SKIP: unsupported storage type
Test create snapshot and templates from volume ... SKIP: unsupported storage type
Test Deploy multiple VMs with & verify the usage ... SKIP: unsupported storage type
Test Deploy multiple VMs with & verify the usage ... SKIP: unsupported storage type
Test delete and recover instance ... SKIP: skip
Test delete and recover instance ... SKIP: skip
Test Deploy VM with 5 GB volume & verify the usage ... SKIP: unsupported storage type
Test Deploy VM with 5 GB volume & verify the usage ... SKIP: unsupported storage type

Ran 14 tests in 2.833s
OK (SKIP=14)

LGTM

please wait for 2 manual reviews before pushing to master.

The changes Look Good to me


Rename sync-state command

For usability and consistency, we should rename the sync-state command to just sync. Here are some other things to modify as part of this work:

Currently, the sync is performed by doing miden-client sync-state -s. The -s flag should be removed in order for the sync to be performed when miden-client sync is ran. There are some flags that we can keep (such as being able to see which block we are synced up to).
The repo's README should be updated to reflect this (and other docs, in case there are any).
There are naming inconsistencies across the codebase. For instance, we have src/cli/sync_state.rs, src/client/sync_state.rs but src/store/state_sync.rs. These should be fixed, in case the codebase contains things like comments and names (like the SQL table name) that are inconsistent with the naming.

I noticed that also tag-related commands are currently under sync. We should consider putting them under their own top-level command. To get info about the current state we could use something like:
miden-client sync -i

But also, could have a more general "info" command on the top level that would give a summary of the client state (e.g., current sync height, number of accounts, number of notes ready to be consumed, maybe something else).
Separately, but related, I'd rename transaction command to just tx to make it easier to type.


Edit sessions are not same

Testing #161860

I have opened VS Code repo in VS Code Desktop and insiders.vscode.dev
Stored some edit sessions
🐛 Second edit session (from insiders.vscode.dev) is not showing the changes in Desktop

Likely also duplicate of https://github.com/microsoft/vscode/issues/161939


Remove the overrides to grant more resources to the fake intake

What does this PR do?
Removes the overrides to grant more resources to the fake intake.
Which scenarios this will impact?

aws/eks
aws/kind

In the past, it has been necessary to bump the amount of memory granted to the fake intake because of the huge volume of data generated on the EKS and kind scenarios.
Since DataDog/datadog-agent#24064, the data are now kept on disk instead of memory.
Motivation
Save money.
Additional Notes

This graph shows the memory usage and the memory limit of the fake intake on a kind scenario:

A change in the color of the graph indicates a container restart.
We can see that the default 1GiB memory limit is too low and the fake intake restarts in loop.
This is also because GOMEMLIMIT is set 600MiB lower than the memory limit: https://github.com/DataDog/test-infra-definitions/blob/a62ab931e64875504b7d7a839d311d2f94fbb211/scenarios/aws/fakeintake/fakeintake.go#L60

/merge

/merge


docs: few typos and rewordings in README

Update README with a few typos and rewordings
Checklist:

[X] Docs updated
[ ] Tests written
[ ] Check for breaking changes in Resource
[ ] Version bumped

@gruzewski Nice thanks!


add a little info about the naming conventions for config parameters

/cc @achambers
unless I missed it we didn't clarify that the config for a plugin is scoped under the plugin name
feel free to kill this if it's unneeded

@ghedamat Would this be more appropriate in the configuration section of the docs?

In fact, let's just put it there altogether. I'll close this now and we can add it there

@achambers it used to be in the configuration section and @lukemelia suggested to move it here :p
I'm easy either way, was just trying to clarify how the naming conventions are set up


GIFS API work

Giphy API it's working on the same page (index.html) just below the about us. Using a different form which I've created for testing.
We still need to add the code for the main search bar and generate the data on the results.html

Fixed


Friendship.add() doesn't work when bot and user are in the same chat group

Bot and user are not friends, they only share the same chatgroup.
Logs below:
15:42:25 VERB Friendship static add(zhao_cq, Please tell me if this works)
15:42:25 VERB PuppetPadpro friendshipAdd(zhao_cq, Please tell me if this works)
15:42:25 SILL PadproGrpc GrpcSearchContact(zhao_cq)
15:42:25 SILL DedupeApi dedupeApi(GrpcSearchContact, {"Username":"zhao_cq"})15:42:25 SILL DedupeApi dedupeApi(GrpcSearchContact) deduped api call missed, call the external service.15:42:25 SILL WechatGateway sendLong() reqSeq:<PHONE_NUMBER>
15:42:25 SILL WechatGateway sendLong() receive back package size: 549
15:42:25 SILL GrpcGateway parse(GrpcSearchContact) get response: {"MsgType":2,"UserName":"v1_f4cee0e00f0e409994bd2fcb7342c08eb13f30a80d20c46afa7f589ec249907c@stranger","NickName":"H.kt","Signature":"What I cannot create, I do not understand","SmallHeadImgUrl":"http
15:42:25 SILL DedupeApi dedupeApi(GrpcSearchContact) got results from external service, resolve 0 duplicate api calls.
15:42:25 WARN PuppetPadpro friendshipAdd zhao_cq has been friend with bot, no need to send friend request!

Fixed the issue in the log with 555ba2ffc58f0a602ce0b6898de2bfc63d466175, but still can not add friend, this is a underlying issue, and we are trying to fix this.

This bug should be fixed by v0.0.54, please test it with that version. @lhr0909

This is still happening in version 0.0.56

Friendship.add() doesn't work when user id start with wxid_xxx

@jhmax720 This is a limitation that wechat has. Currently wechaty-puppet-padpro is doing a search with the wechat id to get the contact, then add it as friend, because the contact start with wxid can not be searched anymore, so it doesn't work on wechaty-puppet-padpro.
We will try to find a way to do this, but the fix time is unknown.

@windmemory Thank you for clarifying this, is there a way to get group members' real weixin id in a group chat? room.memberAll() currently only return objects with Id like wxid_xxx. There's no way to use Friendship.add() in a group chat. Many thanks

I know the limitation for that. Currently there is no way to get weixin id in a group chat. Although you can get it when you use a real phone or iPad, but the data is loaded in an async way, that means there is another api to get those info, and currently that api hasn't been implemented. Sorry about that.

Close this issue since no more comments for more than a month.

Can someone help me recover my wechat account. It suddenly logged off and said that it was blocked. Please I need to recover it badly. Thanks in advance.

This is not the right place to send such request. @jusicorpuz


Split run_mtress into load_config and run_mtress

When changing some of the parameters externally, it would be handy to be able to load the configuration just once. In particular, reading columns from csv files is time consuming.
https://github.com/mtress/mtress/blob/7b6ade0b9feca69c6237faa4bc0f02c46f132aac/mtress/run_mtress.py#L37

Implemented in #15


⚠ Samsung has degraded performance

In 667e874, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2668 ms

Resolved: Samsung performance has improved in bb552bb.


IpChecksumFailed should be set if checksums fail validation

IpChecksumFailed should be set if checksums fail validation, the text says incorrectly that IpChecksumSucceeded should be set.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for the update @pravb, the change should be live soon.


scripts:Update unique error enums as of 12/19/16

Six updates and some new valid usages.
Cmd to do the update was:
python spec.py -update -remap 127-132:128-141:1465-1263:2168-1646,2:2373-1965

@mark-lunarg good idea, opened up #1285 to track this.

cherry-picked and pushed to master


[hdf5] Build error

Package: hdf5[core,cpp,szip,zlib]:x64-linux -> 1.12.1#4
Host Environment

Host: x64-linux
Compiler: GNU 8.3.1
vcpkg-tool version: 2022-11-10-5fdee72bc1fceca198fb1ab7589837206a8b81ba
vcpkg-scripts version: 7d9775a3c 2022-11-17 (3 hours ago)

To Reproduce
vcpkg install
Failure logs
-- Downloading https://github.com/HDFGroup/hdf5/archive/hdf5-1_12_1.tar.gz -> HDFGroup-hdf5-hdf5-1_12_1.tar.gz...
-- Extracting source /home/vsm/vcpkg/downloads/HDFGroup-hdf5-hdf5-1_12_1.tar.gz
-- Applying patch hdf5_config.patch
-- Applying patch szip.patch
-- Applying patch pkgconfig-requires.patch
-- Applying patch pkgconfig-link-order.patch
-- Using source at /home/vsm/vcpkg/buildtrees/hdf5/src/df5-1_12_1-859faf3d27.clean
-- Found external ninja('1.10.2').
-- Configuring x64-linux-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:96 (message):
    Command failed: /home/vsm/vcpkg/downloads/tools/cmake-3.24.0-linux/cmake-3.24.0-linux-x86_64/bin/cmake /home/vsm/vcpkg/buildtrees/hdf5/src/df5-1_12_1-859faf3d27.clean -DHDF5_ENABLE_PARALLEL=OFF -DHDF5_BUILD_TOOLS=OFF -DHDF5_BUILD_CPP_LIB=ON -DHDF5_ENABLE_SZIP_SUPPORT=ON -DHDF5_ENABLE_SZIP_ENCODING=ON -DHDF5_ENABLE_Z_LIB_SUPPORT=ON -DHDF5_BUILD_FORTRAN=OFF -DHDF5_ENABLE_THREADSAFE=OFF -DHDF5_BUILD_UTILS=OFF -DBUILD_TESTING=OFF -DHDF5_BUILD_EXAMPLES=OFF -DHDF5_INSTALL_DATA_DIR=share/hdf5/data -DHDF5_INSTALL_CMAKE_DIR=share -DHDF_PACKAGE_NAMESPACE:STRING=hdf5:: -DHDF5_MSVC_NAMING_CONVENTION=OFF -DSZIP_USE_EXTERNAL=ON -DALLOW_UNSUPPORTED=OFF -DCMAKE_MAKE_PROGRAM=/usr/bin/ninja -DCMAKE_SYSTEM_NAME=Linux -DBUILD_SHARED_LIBS=OFF -DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=/home/vsm/vcpkg/scripts/toolchains/linux.cmake -DVCPKG_TARGET_TRIPLET=x64-linux -DVCPKG_SET_CHARSET_FLAG=ON -DVCPKG_PLATFORM_TOOLSET=external -DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON -DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON -DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON -DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE -DCMAKE_VERBOSE_MAKEFILE=ON -DVCPKG_APPLOCAL_DEPS=OFF -DCMAKE_TOOLCHAIN_FILE=/home/vsm/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON -DVCPKG_CXX_FLAGS= -DVCPKG_CXX_FLAGS_RELEASE= -DVCPKG_CXX_FLAGS_DEBUG= -DVCPKG_C_FLAGS= -DVCPKG_C_FLAGS_RELEASE= -DVCPKG_C_FLAGS_DEBUG= -DVCPKG_CRT_LINKAGE=dynamic -DVCPKG_LINKER_FLAGS= -DVCPKG_LINKER_FLAGS_RELEASE= -DVCPKG_LINKER_FLAGS_DEBUG= -DVCPKG_TARGET_ARCHITECTURE=x64 -DCMAKE_INSTALL_LIBDIR:STRING=lib -DCMAKE_INSTALL_BINDIR:STRING=bin -D_VCPKG_ROOT_DIR=/home/vsm/vcpkg -D_VCPKG_INSTALLED_DIR=/home/vsm/vsm/again/vcpkg_installed -DVCPKG_MANIFEST_INSTALL=OFF -G Ninja -DCMAKE_BUILD_TYPE=Debug -DCMAKE_INSTALL_PREFIX=/home/vsm/vcpkg/packages/hdf5_x64-linux/debug
    Working Directory: /home/vsm/vcpkg/buildtrees/hdf5/x64-linux-dbg
    Error code: 1
    See logs for more information:
      /home/vsm/vcpkg/buildtrees/hdf5/config-x64-linux-dbg-out.log
      /home/vsm/vcpkg/buildtrees/hdf5/config-x64-linux-dbg-err.log

Call Stack (most recent call first):
  /home/vsm/vsm/again/vcpkg_installed/x64-linux/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:426 (vcpkg_execute_required_process)
  buildtrees/versioning_/versions/hdf5/1000a70aefebfdd856715d265c3421c80157b773/portfile.cmake:60 (vcpkg_cmake_configure)
  scripts/ports.cmake:147 (include)

/home/vsm/vcpkg/buildtrees/hdf5/config-x64-linux-dbg-out.log
-- The C compiler identification is GNU 8.3.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /opt/rh/devtoolset-8/root/usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Looking for include file sys/file.h
-- Looking for include file sys/file.h - found
-- Looking for include files sys/file.h, sys/ioctl.h
-- Looking for include files sys/file.h, sys/ioctl.h - found
-- Looking for 3 include files sys/file.h, ..., sys/resource.h
-- Looking for 3 include files sys/file.h, ..., sys/resource.h - found
-- Looking for 4 include files sys/file.h, ..., sys/socket.h
-- Looking for 4 include files sys/file.h, ..., sys/socket.h - found
-- Looking for 5 include files sys/file.h, ..., sys/stat.h
-- Looking for 5 include files sys/file.h, ..., sys/stat.h - found
-- Looking for 6 include files sys/file.h, ..., sys/time.h
-- Looking for 6 include files sys/file.h, ..., sys/time.h - found
-- Looking for 7 include files sys/file.h, ..., sys/types.h
-- Looking for 7 include files sys/file.h, ..., sys/types.h - found
-- Looking for 8 include files sys/file.h, ..., features.h
-- Looking for 8 include files sys/file.h, ..., features.h - found
-- Looking for 9 include files sys/file.h, ..., dirent.h
-- Looking for 9 include files sys/file.h, ..., dirent.h - found
-- Looking for 10 include files sys/file.h, ..., setjmp.h
-- Looking for 10 include files sys/file.h, ..., setjmp.h - found
-- Looking for 11 include files sys/file.h, ..., stddef.h
-- Looking for 11 include files sys/file.h, ..., stddef.h - found
-- Looking for 12 include files sys/file.h, ..., stdint.h
-- Looking for 12 include files sys/file.h, ..., stdint.h - found
-- Looking for 13 include files sys/file.h, ..., unistd.h
-- Looking for 13 include files sys/file.h, ..., unistd.h - found
-- Looking for 14 include files sys/file.h, ..., io.h
-- Looking for 14 include files sys/file.h, ..., io.h - not found
-- Looking for 14 include files sys/file.h, ..., winsock2.h
-- Looking for 14 include files sys/file.h, ..., winsock2.h - not found
-- Looking for 14 include files sys/file.h, ..., globus/common.h
-- Looking for 14 include files sys/file.h, ..., globus/common.h - not found
-- Looking for 14 include files sys/file.h, ..., pdb.h
-- Looking for 14 include files sys/file.h, ..., pdb.h - not found
-- Looking for 14 include files sys/file.h, ..., pthread.h
-- Looking for 14 include files sys/file.h, ..., pthread.h - found
-- Looking for 15 include files sys/file.h, ..., srbclient.h
-- Looking for 15 include files sys/file.h, ..., srbclient.h - not found
-- Looking for 15 include files sys/file.h, ..., string.h
-- Looking for 15 include files sys/file.h, ..., string.h - found
-- Looking for 16 include files sys/file.h, ..., strings.h
-- Looking for 16 include files sys/file.h, ..., strings.h - found
-- Looking for 17 include files sys/file.h, ..., stdlib.h
-- Looking for 17 include files sys/file.h, ..., stdlib.h - found
-- Looking for 18 include files sys/file.h, ..., memory.h
-- Looking for 18 include files sys/file.h, ..., memory.h - found
-- Looking for 19 include files sys/file.h, ..., dlfcn.h
-- Looking for 19 include files sys/file.h, ..., dlfcn.h - found
-- Looking for 20 include files sys/file.h, ..., inttypes.h
-- Looking for 20 include files sys/file.h, ..., inttypes.h - found
-- Looking for 21 include files sys/file.h, ..., netinet/in.h
-- Looking for 21 include files sys/file.h, ..., netinet/in.h - found
-- Looking for 22 include files sys/file.h, ..., netdb.h
-- Looking for 22 include files sys/file.h, ..., netdb.h - found
-- Looking for 23 include files sys/file.h, ..., arpa/inet.h
-- Looking for 23 include files sys/file.h, ..., arpa/inet.h - found
-- Looking for 24 include files sys/file.h, ..., stdbool.h
-- Looking for 24 include files sys/file.h, ..., stdbool.h - found
-- Looking for include file quadmath.h
-- Looking for include file quadmath.h - found
-- Looking for ceil in m;
-- Looking for ceil in m; - found
-- Looking for dlopen in dl;m
-- Looking for dlopen in dl;m - found
-- Looking for WSAStartup in ws2_32;m;dl
-- Looking for WSAStartup in ws2_32;m;dl - not found
-- Looking for gethostbyname in wsock32;m;dl
-- Looking for gethostbyname in wsock32;m;dl - not found
-- Looking for gethostname in ucb;m;dl
-- Looking for gethostname in ucb;m;dl - not found
-- Looking for lseek64
-- Looking for lseek64 - found
-- Looking for fseeko
-- Looking for fseeko - found
-- Performing Test HAVE_STAT64_STRUCT
-- Performing Test HAVE_STAT64_STRUCT - Success
-- Looking for stat64
-- Looking for stat64 - found
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of char
-- Check size of char - done
-- Check size of short
-- Check size of short - done
-- Check size of int
-- Check size of int - done
-- Check size of unsigned
-- Check size of unsigned - done
-- Check size of long
-- Check size of long - done
-- Check size of long long
-- Check size of long long - done
-- Check size of __int64
-- Check size of __int64 - failed
-- Check size of float
-- Check size of float - done
-- Check size of double
-- Check size of double - done
-- Check size of long double
-- Check size of long double - done
-- Check size of int8_t
-- Check size of int8_t - done
-- Check size of uint8_t
-- Check size of uint8_t - done
-- Check size of int_least8_t
-- Check size of int_least8_t - done
-- Check size of uint_least8_t
-- Check size of uint_least8_t - done
-- Check size of int_fast8_t
-- Check size of int_fast8_t - done
-- Check size of uint_fast8_t
-- Check size of uint_fast8_t - done
-- Check size of int16_t
-- Check size of int16_t - done
-- Check size of uint16_t
-- Check size of uint16_t - done
-- Check size of int_least16_t
-- Check size of int_least16_t - done
-- Check size of uint_least16_t
-- Check size of uint_least16_t - done
-- Check size of int_fast16_t
-- Check size of int_fast16_t - done
-- Check size of uint_fast16_t
-- Check size of uint_fast16_t - done
-- Check size of int32_t
-- Check size of int32_t - done
-- Check size of uint32_t
-- Check size of uint32_t - done
-- Check size of int_least32_t
-- Check size of int_least32_t - done
-- Check size of uint_least32_t
-- Check size of uint_least32_t - done
-- Check size of int_fast32_t
-- Check size of int_fast32_t - done
-- Check size of uint_fast32_t
-- Check size of uint_fast32_t - done
-- Check size of int64_t
-- Check size of int64_t - done
-- Check size of uint64_t
-- Check size of uint64_t - done
-- Check size of int_least64_t
-- Check size of int_least64_t - done
-- Check size of uint_least64_t
-- Check size of uint_least64_t - done
-- Check size of int_fast64_t
-- Check size of int_fast64_t - done
-- Check size of uint_fast64_t
-- Check size of uint_fast64_t - done
-- Check size of size_t
-- Check size of size_t - done
-- Check size of ssize_t
-- Check size of ssize_t - done
-- Check size of ptrdiff_t
-- Check size of ptrdiff_t - done
-- Check size of off_t
-- Check size of off_t - done
-- Check size of off64_t
-- Check size of off64_t - done
-- Check size of time_t
-- Check size of time_t - done
-- Check size of _Bool
-- Check size of _Bool - done
-- Looking for CLOCK_MONOTONIC
-- Looking for CLOCK_MONOTONIC - not found
-- Performing Test H5_HAVE_TM_GMTOFF
-- Performing Test H5_HAVE_TM_GMTOFF - Success
-- Performing Test H5_HAVE___TM_GMTOFF
-- Performing Test H5_HAVE___TM_GMTOFF - Failed
-- Performing Test H5_TIME_WITH_SYS_TIME
-- Performing Test H5_TIME_WITH_SYS_TIME - Success
-- Performing Test H5_HAVE_STRUCT_TIMEZONE
-- Performing Test H5_HAVE_STRUCT_TIMEZONE - Failed
-- Looking for gettimeofday
-- Looking for gettimeofday - found
-- Performing Test H5_HAVE_STAT_ST_BLOCKS
-- Performing Test H5_HAVE_STAT_ST_BLOCKS - Success
-- Looking for ioctl
-- Looking for ioctl - found
-- Performing Test H5_HAVE_STRUCT_VIDEOCONFIG
-- Performing Test H5_HAVE_STRUCT_VIDEOCONFIG - Failed
-- Performing Test H5_HAVE_STRUCT_TEXT_INFO
-- Performing Test H5_HAVE_STRUCT_TEXT_INFO - Failed
-- Looking for _getvideoconfig
-- Looking for _getvideoconfig - not found
-- Looking for gettextinfo
-- Looking for gettextinfo - not found
-- Looking for _scrsize
-- Looking for _scrsize - not found
-- Looking for GetConsoleScreenBufferInfo
-- Looking for GetConsoleScreenBufferInfo - not found
-- Looking for TIOCGWINSZ
-- Looking for TIOCGWINSZ - found
-- Looking for TIOCGETD
-- Looking for TIOCGETD - found
-- Looking for getpwuid
-- Looking for getpwuid - found
-- Looking for alarm
-- Looking for alarm - found
-- Looking for fcntl
-- Looking for fcntl - found
-- Looking for flock
-- Looking for flock - found
-- Looking for fork
-- Looking for fork - found
-- Looking for frexpf
-- Looking for frexpf - found
-- Looking for frexpl
-- Looking for frexpl - found
-- Looking for gethostname
-- Looking for gethostname - found
-- Looking for getrusage
-- Looking for getrusage - found
-- Looking for llround
-- Looking for llround - not found
-- Looking for llroundf
-- Looking for llroundf - not found
-- Looking for lround
-- Looking for lround - not found
-- Looking for lroundf
-- Looking for lroundf - not found
-- Looking for lstat
-- Looking for lstat - found
-- Looking for pread
-- Looking for pread - found
-- Looking for pwrite
-- Looking for pwrite - found
-- Looking for rand_r
-- Looking for rand_r - found
-- Looking for random
-- Looking for random - found
-- Looking for round
-- Looking for round - not found
-- Looking for roundf
-- Looking for roundf - not found
-- Looking for setsysinfo
-- Looking for setsysinfo - not found
-- Looking for signal
-- Looking for signal - found
-- Looking for longjmp
-- Looking for longjmp - found
-- Looking for setjmp
-- Looking for setjmp - found
-- Looking for siglongjmp
-- Looking for siglongjmp - found
-- Looking for sigsetjmp
-- Looking for sigsetjmp - not found
-- Looking for sigprocmask
-- Looking for sigprocmask - found
-- Looking for snprintf
-- Looking for snprintf - found
-- Looking for srandom
-- Looking for srandom - found
-- Looking for strdup
-- Looking for strdup - found
-- Looking for strtoll
-- Looking for strtoll - found
-- Looking for strtoull
-- Looking for strtoull - found
-- Looking for symlink
-- Looking for symlink - found
-- Looking for system
-- Looking for system - found
-- Looking for tmpfile
-- Looking for tmpfile - found
-- Looking for asprintf
-- Looking for asprintf - found
-- Looking for vasprintf
-- Looking for vasprintf - found
-- Looking for waitpid
-- Looking for waitpid - found
-- Looking for vsnprintf
-- Looking for vsnprintf - found
-- Looking for sigsetjmp
-- Looking for sigsetjmp - found
-- Looking for difftime
-- Looking for difftime - found
-- Looking for clock_gettime
-- Looking for clock_gettime - found
-- Looking for clock_gettime in rt
-- Looking for clock_gettime in rt - found
-- Looking for clock_gettime in posix4
-- Looking for clock_gettime in posix4 - not found
-- Check size of __float128
-- Check size of __float128 - done
-- Check size of _Quad
-- Check size of _Quad - failed
-- Testing maximum decimal precision for C - 21;33;
-- maximum decimal precision for C var - 33
-- Found Perl: /usr/bin/perl (found version "5.16.3")
-- Configuring incomplete, errors occurred!
See also "/home/vsm/vcpkg/buildtrees/hdf5/x64-linux-dbg/CMakeFiles/CMakeOutput.log".
See also "/home/vsm/vcpkg/buildtrees/hdf5/x64-linux-dbg/CMakeFiles/CMakeError.log".

/home/vsm/vcpkg/buildtrees/hdf5/config-x64-linux-dbg-err.log
CMake Error at /home/vsm/vcpkg/scripts/buildsystems/vcpkg.cmake:841 (message):
  [vcpkg]: find_package ended with z_vcpkg_find_package_backup_id being less
  than 0! This is a logical error and should never happen.  Please provide a
  cmake trace log via cmake cmd line option '--trace-expand'!
Call Stack (most recent call first):
  CMakeLists.txt:512 (find_package)

Additional context
vcpkg.json
{
  "name": "vsm",
  "version": "4",
  "dependencies": [
    "boost-filesystem",
    "boost-locale",
    "boost-system",
    "boost-thread",
    "boost-program-options",
    "boost-timer",
    "boost-uuid",
    "boost-foreach",
    "boost-format",
    "boost-nowide",
    "boost-circular-buffer",
    "boost-date-time",
    "boost-property-tree",
    "boost-log",
    "boost-lockfree",
    "boost-math",
    "boost-dll",
    "protobuf",
    "eigen3",
    "cryptopp",
    "catch2",
    "fmt",
    "cli",
    "nlohmann-json",
    "pugixml",
    {
      "name": "minizip-ng",
      "features": [
        "zlib"
      ]
    },
    {
      "name": "hdf5",
      "features": [
        "cpp"
      ]
    }
  ],
  "builtin-baseline": "01d6f6ff1e5332b926099f0c23bda996940ad4e8"
}

[vcpkg]: find_package ended with z_vcpkg_find_package_backup_id being less
than 0! This is a logical error and should never happen.  Please provide a
cmake trace log via cmake cmd line option '--trace-expand'!

CC @Neumann-A Given the macro nature of vcpkg's find_package, do we perhaps run into trouble with bad behaving Find modules or wrappers (i.e. calling return or find_dependency while being in the caller's scope)?

@keinkoenig Can you try to put quotes around rhs 0 here:
https://github.com/microsoft/vcpkg/blob/4cb4a5c5ddcb9de0c83c85837ee6974c8333f032/scripts/buildsystems/vcpkg.cmake#L840
... just to exclude some unexpected set(0 1).
Background (happened before):
cmake_minimum_required (VERSION 3.18)
set(0 1)
if ("0" LESS "0")
	message(STATUS A)
endif()
if("0" LESS 0)
	message(STATUS B)
endif()

Also why is this 1.12.1. HDF5 in vcpkg is at 1.12.2 (and keeping hdf5 up to date is preferred unless they terribly broke something again.)

1.12.1#4

It might even be fixed with 1.12.1#5, https://github.com/microsoft/vcpkg/pull/25036. Note the changes to the set(${<var>} 1 ...).

I verified that the bug occurs in 1.12.1#4 and is already fixed in 1.12.1#5.

i fixed it by purging vcpkg and upgrading the manifest baseline to latest.

This issue has been resolved, please reopen it if there are still problems.


Reconnect Internet?

Hello, in your other repo you write, follow these steps:

Close Steam client if open.
Disconnect from the internet.
Run Steam as Administrator.
Inject VAC-Bypass.dll into Steam.exe process.
After successful injection messagebox saying Initialization was successful! should appear.
6. Reconnect to the internet.
Press Retry in steam window saying 'Could not connect to Steam servers'.

I didnt compiled or tested anything atm. But i can't see the part of "Reconnect to the Internet" in the bypass-loader repo / code. If the loader does not take this step, is this step really necessary at all?

You have to disconnect from the internet to be sure VAC hasn't been loaded before you inject bypass. Bypass loader handles process creation and injection for you.


Linux (Manjaro) - Gimp 2.10.32 - Stable Horde - Exception: Input payload validation failed

Hi !
I'm having an issue with the stable-horde plugin.
When I try to use text2image, I get the following error message :
Une erreur est survenue durant l’exécution de python-fu-stable-horde
Exception: Input payload validation failed

Traceback (most recent call last):
  File "/usr/lib/gimp/2.0/python/gimpfu.py", line 740, in response
    dialog.res = run_script(params)
  File "/usr/lib/gimp/2.0/python/gimpfu.py", line 361, in run_script
    return apply(function, params)
  File "/usr/lib/gimp/2.0/plug-ins/gimp-stable-diffusion-horde.py", line 212, in generate
    raise Exception(message)
Exception: Input payload validation failed

Thanks for your help !
Adrien

I just tried it and couldn’t reproduce the problem. Which plugin version do you use? You find the version at top of the py file. It should be 1.3.4.

The py script is indeed in version 1.3.4.
I made a few tests and it seems this issue appears with float values for the "Prompt strength" parameter. With integer values it works just fine.

Thanks for the update. I will check it.

The problem is fixed in version 1.3.5. Now only integer values can be selected for the prompt strength.


Footer: Add banner

We need to add powered by Alchemy to the footer.
Preview

File

Specs
194x42, center :)

There's some distinct styling differences between ur image and the current footer state, am I just approximating from this and adding: powered by alchemy and good hunting! ?

for ref:

yeah, looks like the footer was never updated to the designs from Gluzman.
If you can do the changed to the footer that would be good. If not, then we can just approximately place the Powered by Alchemy image under Logo/Tally Ho! and we can update footer later

Here is a link to the footer, just in case.
Sorry for the mix-up, didn't notice it wasn't updated

awesome! will do it all!


Support analysis of buildSrc folder in Gradle projects

I noticed that by default detekt ignores Gradle buildSrc folder.
    detekt {
        config = files("${project.rootDir}/detekt.yml")
        input = files("${project.rootDir}/buildSrc/src/main/kotlin")
    }

The problem is that using input config overrides "all paths where detect looks typically for Kotlin files", so now buildSrc folder is the only folder that is checked.
How can I add buildSrc and keep existing folders?

I think we haven't considered buildSrc in our gradle plugin.
What you can do it write a custome detekt check just for buildSrc and add dependsOn Check.
val detektBuildSrc by tasks.registering(Detekt::class) {
    description = "Runs over buildSrc"
    buildUponDefaultConfig = true
    setSource(files([buildSrcDir]))
    ....
}

Thx for code - I will play with it.
BTW I think plugin should also check buildSrc folder by default (or at least have an easy way to add it)

I think that this issue could be fixed just by adding the detekt plugin to the buildSrc module, right? Could we close this?

BTW I think plugin should also check buildSrc folder by default (or at least have an easy way to add it)

As @BraisGabin mentioned, you can apply the Detekt Gradle plugin to the buildSrc project and it will work out of the box.


feat(validators): bigger and smaller than validators

Thanks for that - I think it makes sence to match common naming, for example in input its called min
https://www.w3schools.com/tags/att_input_min.asp
Another option is greaterThan
I would also add range


[Blazor] Event handling - ParentChild2.razor without Task.Yield()

I believe the await Task.Yield() in the ParentChild2.razor example no longer makes sense. It probably remained from an earlier era of Blazor when these in-line lambdas had to be asynchronous.
cc @guardrex

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/components/event-handling.md
ASP.NET Core Blazor event handling

Let's have one or both of them look at this as well. I'll add this to my email.

@danroth27 ... I pinged Steve (email), but my message may have been lost in the shuffle.
Do you recall why we had this demo code set up this way? ...
<Child2 OnClickCallback="@(async (value) => { await Task.Yield(); messageText = value; })" />

IIRC, it was because we were trying to demo async in the lambda. Task.Yield() was likely chosen purely for demo purposes. Lambdas never had to be async AFAIK ... it was just that we wanted to put something here async that's runnable. I'm 🤔 that the example should be changed.
The existing Child2 component stays the same ...

Child2 Component

<button @onclick="TriggerEvent">Click Me</button>

@code {
    [Parameter]
    public EventCallback<string> OnClickCallback { get; set; }

    private async Task TriggerEvent()
    {
        await OnClickCallback.InvokeAsync("Blaze It!");
    }
}

The ParentChild2 component is changed to the following. Note that we don't specify a render mode (e.g., @rendermode InteractiveServer) because we assume global render modes for our examples.
@page "/parent-child-2"

<PageTitle>Parent Child 2</PageTitle>

Parent Child 2 Example

    <Child2 OnClickCallback="@(async (value) => { await Task.Yield(); messageText1 = value; })" />
    @messageText1

    <Child2 OnClickCallback="@((value) => { messageText2 = value; })" />
    @messageText2

@code {
    private string messageText1 = string.Empty;
    private string messageText2 = string.Empty;
}

Do you recall why we had this demo code set up this way?
<Child2 OnClickCallback="@(async (value) => { await Task.Yield(); messageText = value; })" />

Task.Yield() will force the method to return asynchronously, so I assume this is trying to show how an async event handler that doesn't complete synchronously can still handled. Making the change to show both he sync and async versions makes sense to me.
Note that syntactically I don't think you need the lambda expressions to be explicit Razor expressions. You're already in a C# context so the wrapping @() can be dropped.

Thanks @danroth27.

don't think you need the lambda expressions to be explicit Razor expressions

... that aspect might be due to an ancient limitation (i.e., experimental days of ancient Blazor times 👴). I just confirmed that all is well without them 🎉.
@hakenr ... Can you use my recommended replacement?

I still think the Task.Yield() is very rarely seen in today's codebases and most of the developers won't understand the "just to force the method to run asynchronously" purpose of it in this example. From my POV something like await Task.Delay(2000) works better (and we can add explanation "This version is asynchronous and the new @messageText2 value will be assigned and rendered after two seconds").
So I would go for:
@page "/parent-child-2"

<PageTitle>Parent Child 2</PageTitle>

Parent Child 2 Example

    <Child2 OnClickCallback="(value) => { messageText1 = value; }" />
    @messageText1

    <Child2 OnClickCallback="async (value) => { await Task.Delay(2000); messageText2 = value; }" />
    @messageText2

@code {
    private string messageText1 = string.Empty;
    private string messageText2 = string.Empty;
}

(I also moved the asynchronous version to the second occurrence, but it is not that important.)

That sounds good to me.

We're going to have a problem with the line length. We need to keep that long line at <= 85 characters.


daemon: Deprecate EnableRemoteNodeIdentity

Remote Node Identity is needed for various other features to work or work fully, including EgressGateway and Policies.
There is no benefit to having it turned off.
daemon: The option "EnableRemoteNodeIdentity" is now deprecated and will be permanently set to "true" in the v1.16 release.

/test


[Improvement]: Cards must be in center

Issue: Cards are in left side
Expectation: The cards must be placed in the center of the screen

I can work on it.. can you please assign it to me under GSSOC 2023?