0.8.0 Configuration Normalization info

Regarding the "breaking change" of the "Configuration Normalization" in version 0.8.0  I'm just wondering,
as it is written in those releasenotes ("This is a breaking change but MS will continue to operate without any changes to your config.") it could be understood as there currently being some kind of compatibility mode phase where both the old and new formats are working concurrently.
Is that the case?
If so, is there somewhere we would get some info/warning regarding configuration that must be migrated before that compatibility phase ends?
I could not see any such in my MS logs output and scrobbling remains fully functional as it was before re-pulling docker compose using "image: foxxmd/multi-scrobbler".
Does that mean no migration is needed?

There is no deprecation phase for the config changes and there is no way to get warnings about these changes.
However, MS will not fail to operate because the changes have not been made, it simply will not see the old config properties anymore (until they have been moved).
I do plan to add actual config validation checks (and deprecation warnings) back in in the future but they currently do not run. The config schema is solely for viewing documentation though you can also use this site to check your config validity by copy-pasting your config into the right-hand pane.
Sorry for the confusion with my wording in the release. Below is a list of the properties that have been moved and how to migrate them. I'll mention this in the release.
To Migrate Configs
All configs already had a  data property but now also have an (optional) options property at top-level.
[
  {
    "name": "Deezer",
    "data": {
		// ...
    },
    "options": {
       // ...
	}
  }
]

If this property does not exist you must create it before moving any of the listed properties. If you do not have any of the listed properties in your configs you do not need to do anything (including creating options).
For both Source and Clients

scrobbleThresholds
maxPollRetries
maxRequestRetries
retryMultiplier

Should be moved from data to options.
For Clients only
There was a data.options property like:
{
// ...
  "data": {
   // ...
   "options": { ... }
   }
}

This should be moved to be a sibling of data like:
{
// ...
  "data": {
   // ...
   },
   "options": { ... }
}

Additionally you can reference:

config example
kitchen sink example
options schema for Sources
options schema for Clients

to verify your config is correct.

Thank you for the ample clarification.
As there "is no deprecation phase for the config changes",
would it then be correct to assume that if everything appears to remain working as as desired after the upgrade,
that means these changes require no further action in such cases?

Yes that is a fair assumption.
Part of the motivation for moving these settings to options was to differentiate between "required" or major-behavioral settings in data IE credentials, filters for not scrobble certain users in jellyfin, blacklisting apps in chromecast...
vs. "optional" settings or settings used for fine-tuning MS experience that don't primarily effect scrobbling/monitoring IE polling interval, number of request retries, more verbose logging...
All of the settings that were migrated to options in this change are the latter.
In 95% of cases users won't have these settings listed anyways. In the remaining 5% the change may result in some un-optimal behavior IE source stopping early due to network issues that were compensated for by higher # of allowed retries. In 0% of cases should these changes break MS, though.

Very good.
I hope my questions and your answers will also benefit other users who might have had similar questions in mind.
With that, I have no further questions on this and am closing the issue.


TcpSocketListener stops receive after first time

Hi, TcpSocketListener stops receive after first time.
public TcpSocketListener listener;
var listenPort = 8912;
listener = new TcpSocketListener();
listener.ConnectionReceived += DO;
AsyncHelpers.RunSync(async () => await listener.StartListeningAsync(listenPort));
I try to reinit this. Do not working:
listener.StopListeningAsync();
listener.Dispose();
listener = new TcpSocketListener();
listener.ConnectionReceived += DO;
AsyncHelpers.RunSync(async () => await listener.StartListeningAsync(8912));
listener.ConnectionReceived += DO;
I dont know, why

For me too


Tag in own table

I'd like to put the tags in their own table and hence add anoterh photo<->tag-table. That way autocompletion for example would be much simpler and searching for photos with a specific tag is possible without full text search.
I'd like to discuss this change bevor I start working on it.

Adding a new table for the tags is a good idea. I can add an update-script which converts all existing tags to the new format (When needed).

Suggestion by @qligier from #513:

Tags should be in their own table, lychee_photo_tags, with columns photo_id (unsigned mediumint, index) and tag (varchar 30 should be enough). If so, column lychee_photos.tags should be dropped.

This issue will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community. Important issues have been migrated. Feel free to re-add your issue(s) when necessary.


Add InstanceSnapshot status validation

This PR adds a kubebuilder validation marker for InstanceSnapshot CRD

@QcFe done! A week later, though...

/merge


Write migration guide for azure-spring-cloud-starter-cache

Parent issue: #24351

[x] azure-spring-cloud-starter-cache, investigate whether this starter can be deprecated

Closing this issue now for it's already in the migration guide.


Pre-OCA Styling -- convert the word "bc" to "BC"

I ran through the ISED Business Banking Initiative issuers and verifiers, and there are several credentials issued by "bc_registries". In the BC Wallet, those are displayed as "Bc Registries".  Although this will be dealt with when we are able to use OCA, it would be good (assuming it is easy) to enhance the prettifying of identifiers to display such if the if the word "Bc" could be changed to "BC".
Only do this if it is extremely easy as we will (I hope!!) have OCA soon enough.

Example screenshot:

I believe BC Wallet displays the credential base on the schema name, not the credential definition name. We'd have to issue a new schema to the ledger and update some configurations. I could do it in an afternoon but there might be some interruptions to the ISED demo as I update though. I'd rate the change as about a medium

On a side note BC Wallet should probably be using the credential definition as the title rather than the schema id. I can create a ticket for this change if you'd like

No -- we don't want to change the objects on the ledger -- don't do that.
In the wallet (somewhere) there is code that converts schema type strings like "bc_person_schema", "bcPersonSchema", and "bc-person-schema" (and perhaps some other standard conventions) into capitalized words: "Bc Person Schema".  What I'm suggesting is adding after that conversion, anything that "Bc" to "BC". No change should be made to the objects themselves.

Not sure we should change which string we are using (schema_id, cred_def_id).  I think that "schema_id" is used because a ton of implementations use "default" in the CredDefId, which looks bad.  OCA will fix all of this, making those strings irrelevant.  This use of those strings just a fallback because in many cases (all cases, right now) -- it is all we have.

@swcurran , we are not doing any string processing. The best way is to use the tag part of credential definition  id and using a fallback to the schema name. That logic has being implemented already, do we have our credential definition IDs properly tagged?


Server Shutdown

Server Stoping
After world start loading server exit

Log

Please use the discord for support questions.
These issues are for actual bugs/issues within the plugin.
Anyway you should use paper instead of spigot.
https://papermc.io
All spigot plugins will keep working perfectly fine, but your server will perform better
Terra also requires paper to run

Thank
Dne ne 17. čvc 2022 18:38 uživatel Roan-V @.***> napsal:

Please use the discord for support questions.
These issues are for actual bugs/issues within the plugin.
Anyway you should use paper instead of spigot.
https://papermc.io
All spigot plugins will keep working perfectly fine, but your server will
perform better
Terra also requires paper to run
—
Reply to this email directly, view it on GitHub
https://github.com/PolyhedralDev/Terra/issues/349#issuecomment-1186563872,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJISPF6WA7VXFQ3EFWV4IRDVUQZG3ANCNFSM53Z6ZUBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


add schema validation to user input

Add schema validation
having an input in YAML allows the user to provide in a single file all the options. This has the advantage of avoiding passing every option via the command line.
Also the schema package takes care of validating the input.
Now the command line can be called like:
 xtp_gradient -i tests/files/example.yml

where the example.yml file contains all the user keywords that the user wants to supply. The only mandatory keyword is the path to the molecule. For example the following is a valid input:
molecule: tests/files/ethylene.xyz

I'm not sure if I understand when this works. You show the example of when we have a pre-defined tool such as the xtp_gradient. That I can follow.
However, the votca wrapper is freely importable into any custom code, like in the run_python.py in the newly created examples folder. That one will not know about command line options and yaml?

I'm not sure if I understand when this works. You show the example of when we have a pre-defined tool such as the xtp_gradient. That I can follow.
However, the votca wrapper is freely importable into any custom code, like in the run_python.py in the newly created examples folder. That one will not know about command line options and yaml?

You are right there is a schema for each command line. But for the wrapper is not necessary because we assume that the user know what she/he is doing

You are right there is a schema for each command line. But for the wrapper is not necessary because we assume that the user know what she/he is doing


How to make sure the downloader go through socks5 or http/https as proxy when using shadowsocks/v2ray in PAC mode?

Put this into "启动参数" of N_m3u8DL-CLI-SimpleG.exe, would that work?
--proxy socks5://<IP_ADDRESS>:1080
or is threre a update version with new option of this down below would be needed?
socket = "socks5://<IP_ADDRESS>:1080"

Can't customize proxy, the program will use the system's proxy(HTTP) settings.

Future versions will try to add custom proxy functions.

Future versions will try to add custom proxy functions.

Also looking forward to it!
Much thx for your great work!


⚠ Bofrost.at has degraded performance

In b249c2a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1551 ms

Resolved: Bofrost.at performance has improved in e3699ed.


New term request - "seed" oil content

The current best fit is TO:0000491 fat and essential oil composition related trait which is not 'grain' specific.
Rationale for new term - oil content (amount) and composition are very important harvest traits for crops, and not just for seed, but also fruits - eg olives. These lipids are storage lipids/triacylglycerols, not other kinds of lipids used in membranes.
Related parts of the tree include: seed composition based quality trait
Suggested definition:
Per cent of grain dry weight accounted for by oil or triacyl glycerol.
Reported by: mlschaeffer
Original Ticket: obo/plant-trait-ontology-to-requests/277

Two related terms:
fat and essential oil composition related trait (TO:0000491): is a seed quality trait (currently has no def'n.)


Add boundary time and asset property publishers

Pull request
Description
This PR adds support for boundary time and asset property publishers. These will be immensely helpful to implement other player features and complete the reactive backbone structure of the player.
Changes made

Add boundary time publishers.
Add asset property publishers.
Improve test helpers.
Integrate Timelane to visualize player internal events.
Remove player properties and seek target (not necessary anymore).

Checklist

[x] Your branch has been rebased onto the main branch.
[x] APIs have been properly documented (if relevant).
[x] The documentation has been updated (if relevant).
[x] New unit tests have been written (if relevant).
[x] The demo has been updated (if relevant).

🌙 Available in TestFlight nightlies
The most recent state can be tested in the following builds:

Platform
Version

iOS
-

tvOS
1.0.0 (53)


Update index.js to solve Issue 92.

This fix worked for me and a few others.

@googlebot I signed it!


Lobby screen: XSS protection for message_area widgit

Currently message_area takes raw HTML and inserts it into the Lobby Screen page. This opens the system to XSS attack by Lobby Screen managers, and to general layout disaster.
Either we have to stop using HTML (but note that Markdown is no better because it can include raw HTML) or someone has to sanitise the message.

I'd try to avoid to reinvent the wheel and use one of the WYSIWYG editors out there, that may already have solved this.

I've been doing some research about this and I though that bleach from Mozilla was the best option, see: https://github.com/mozilla/bleach
But then while I was implementing it I tough it was a bad idea as I was including a adding quite a lot of exceptions in de backend for only 1 particular widget, so it felt I wasn't doing it right, and it doesn't keep the framework and the widgets independent.
After thinking about alternatives, I think that the best way is doing it inside the widget and use a sanitise javascript library before loading the content into the widget div. One good options seems to be: https://github.com/punkave/sanitize-html
@jw35 ?


parameter  of az ml command is wrong

the command below does not work.
>az ml workspace share -n my_workspace -g my_resource_group --role "Data Scientist" --user<EMAIL_ADDRESS>I changed param "-n" into "-w"  then works.
>az ml workspace share -w my_workspace -g my_resource_group --role "Data Scientist" --user<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0ca3ea12-4f8c-72e4-4703-ea2ef9dcee0f
Version Independent ID: a4b7c496-22f9-0fd6-cd7d-0fd613f1a1ec
Content: Manage roles in an Azure Machine Learning workspace - Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-assign-roles.md
Service: machine-learning
Sub-service: core
GitHub Login: @Blackmist
Microsoft Alias: larryfr

@tamitarai Thank you for your feedback! We will review and provide an update as appropriate.

@tamitarai Thanks for pointing this out. We have raised a PR 80372 to fix the documentation. The change should be updated in the next 24 hours.

@tamitarai We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.


chore: codescan updates

:tada: This PR is included in version 9.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Hystrix Dashboard Breaking after update Steeltoe 3.2 version with DotNet6

Describe the bug
After updating Steeltoe 3.2 version with DotNet6 sometime Circuit section is not loading in dashboard  and sometimes only few circuits display not all.
but older version "Steeltoe.CircuitBreaker.Hystrix.MetricsEventsCore" Version="2.1.1" with netstadared2.0 is working fine with same API
Steps to reproduce
Steps to reproduce the behavior:

DotNet6 + Steeltoe 3.2
Start the Load and Open Hystrix Dashboard
Circuit sections will not display sometimes and sometime few Circuits is showing

Expected behavior
sometime Circuit section is not loading in dashboard  and sometimes only few circuits display not all.
Environment (please complete the following information):

Steeltoe 3.2 version
DotNet6

Screenshots
If applicable, addHytrix Issues with Dotnet6.docxscreenshots to help explain your problem.
Additional context or links
Add any other context about the problem here.
Hytrix Issues with Dotnet6.docx

@deepakapj :
Thanks ... I'm getting ready to look at this... and was wondering if you had any more detail on your environment/setup ... are you using Docker (.e. g. app running in Docker,   dashboard in Docker, everything running locally????) .
Also, wondering if you have a sample that I should use to start with to repo the problem?
Thanks
Dave

Closing due to lack of response for additional information. If there is still an issue here, please provide more information and ideally a sample app that reproduces the problem.


Benchmark compilation issues on lassen / pecan with cpp17 + cuda11

@wrtobin, I just saw your commit message, I love it.


proposal: for index, rune, size = range for byte slices and strings

While thinking about #27148 "optimize ranges over string(byteSlice)" I got the idea that instead making the compiler work hard to prove that not doing the string conversion with an allocation and copy is safe (which often will not succeed) a new range type that interprets a byte slice as utf8 encoded runes could be introduced instead. This gives programmers a simple syntax with an existing keyword to iterate over runes in a byteslice while the byteslice is allowed to change.
The proposal is to allow a new 3 value range for strings and byte slices:
for index, rune, size = range string {
   ...
}

for index, rune, size = range byteslice {
   ...
}

where index and rune are the same as in for index, rune = range string  and size  is the number of bytes needed to encode the rune in UTF8. In case a rune could not be decoded from the string or byteslice due to an invalid UTF8 byte sequence: size is set to 1 and rune to utf8.RuneError.
Updates to the byteslice while executing the loop are taken into account only when decoding byteslice[i+size:].

This is backwards compatible with existing range syntax and Go1 since 3 value ranges do not exist yet.
Allows iterating over the decoded runes in a byteslice without allocation which can happen for for i, r = range string(byteslice) and allows for easier symmetry with equivalent string based functions.
additionally gives the size of the decoded rune which is useful for e.g. strings.Map and other functions where the size is used to determine if a real RuneError rune was decoded or how much space is needed to write the rune to a byte slice.
there is no performance overhead vs existing for i, r = range string loop for the 3 value loop since internally the loop already computes the size of the rune to correctly advance the index on the next iteration. This will however allow user supplied code in the loop to use this hidden variable.

For example current strings.Map first range loop:
for i, c := range s {
	r := mapping(c)
	if r == c && c != utf8.RuneError {
	   continue
	}

	var width int
	if c == utf8.RuneError {
		c, width = utf8.DecodeRuneInString(s[i:])
		if width != 1 && r == c {
			continue
		}
	} else {
		width = utf8.RuneLen(c)
	}

could become:
for i, width, c := range s {
   r := mapping(c)
   if r == c && !(c == utf8.RuneError && width == 1) {
       continue
   }

and the start of the bytes.Map range loop could be rewritten from:
for i := 0; i < len(s); {
	wid := 1
	r := rune(s[i])
	if r >= utf8.RuneSelf {
		r, wid = utf8.DecodeRune(s[i:])
	}

to:
for i, wid, r := range s {

Note: the fastpath for 1-byte encoded runes would still be applied in the new bytes.Map loop similar to how the gc compiler currently adds this optimization automatically for 2 value string range loops.

Would't be the better if the size is computed explicitly as its fixed and does not change? Also, I don't think range is the ideal keyword to perform this extension as if you think about it a range should iterate over a starting - ending point only and having size would be extraneous.
A more suitable 3rd parameter for range that makes sense is a step parameter which would be the difference between each number in the sequence.

size here is the size of the rune that was decoded, its not fixed and changes from rune to rune.
Its the step between two indexes.

I'd rather fix this by doing the optimization in the compiler.
Changing the language to support this optimization seems overkill.

This should definitely not be done for []byte. Bytes do not have any concept of a "rune" in any way, and shouldn't.

This should definitely not be done for []byte. Bytes do not have any concept of a "rune" in any way, and shouldn't.
I would not think this is that of an absolute given the existence of utf8.DecodeRune and other utf8 function operating on byte slices and the general symmetry of byteslices and string in various packages like utf8.

edit - To add on to this, this would mean that doing i, e, _ := range byteslice would be entirely different from i, e := range byteslice
fair point that will indeed be confusing.

I would not think this is that of an absolute counter point given the existence of utf8.DecodeRune and other utf8 function operating on byte slices and the general symmetry of byteslices and string in various packages like utf8 showing byte slices frequently being interpreted as utf8 encoded runes.

That's because utf8 is an encoding. When one does utf8.DecodeRune(byteslice), they are saying "take this byte slice and look at it like it's encoded in utf8. Whats the first rune?". The byteslice has no knowledge of its encoding, and it rightfully shouldn't.
And of course the utf8 package is going to assume utf8 encoded bytes. I could say the same about the utf16 package but say that we should be using utf16 instead.
The point I'm trying to make is, bytes aren't special. They are just a uint8, and should be treated as such. If we want to introduce a concept of runes for your byte slice, either use a package that specifies how to convert from bytes to runes (like utf8), or convert it to a string.

Maybe something like i, r = range(rune) byteslice would better signify that range interprets the byteslice as runes and byteslice does not have an inherent rune concept.

Or i, r = range string(byteslice) :wink:

That's because utf8 is an encoding. When one does utf8.DecodeRune(byteslice), they are saying "take this byte slice and look at it like it's encoded in utf8. Whats the first rune?". The byteslice has no knowledge of its encoding, and it rightfully shouldn't.

The same is true for strings. https://golang.org/ref/spec#String_types "A string type represents the set of string values. A string value is a (possibly empty) sequence of bytes." but currently range UTF8 decodes runes from the bytes of a string.

Or i, r = range string(byteslice) 😉
Which is not the same semantic since changes to the byteslice should not be taken into account during iteration.

The same is true for strings. https://golang.org/ref/spec#String_types "A string type represents the set of string values. A string value is a (possibly empty) sequence of bytes."

I thought this would be brought up. I wasn't going to do it, but I was thinking that if you did I would mention that both ranging over a string and converting it to []rune assume utf8 encoding, as you have mentioned already.

The range keyword introduces the way the string or byteslice is decoded even if both have no knowledge of this on their own.

There's not a single thing built-in to the language that assumes that a []byte is utf8 encoded IIRC.

Which is not the same semantic since changes to the byteslice are not taken into account during iteration and therefore often a copy is needed.

This is a fair assessment, although I feel like in the case where you want it to be available to be changed, the utf8 package (as verbose as it would be) should be used.
import "utf8"

func main() {
	byteslice := []byte("안녕!")

	for i := 0; i < len(byteslice); {
		r, rlen := utf8.DecodeRune(byteslice[i:])

		// code here

		i += rlen
	}
}

I think it's much less safe to assume that a []byte is utf8 encoded

updated the proposal to the strings version only.

Or if we do the bug where rune32 and int32 are separate types (#29012), then we could define len(a rune32) to be the UTF-8 length of the rune.
But in general we're moving away from hard-coding UTF-8 assumptions from the language.

If performance of this kind of operation is essential, we should teach the compiler to treat the relevant functions from the utf8 package as intrinsics.  Ranging over a string is already somewhat non-orthogonal.  Let's not make it even more different.


Mobile network toggle view could be invisible for API 17 and above?

Mobile network toggle doesn't work on my device. I was checking the code and it's commented that it works only for api 16 and below. So, would it be better to not show it for api 17 and plus in the first place? Small issue but could save people using this library some time scratching their head. :) Please let me know if I could contribute.

Fix: 8e895ff6294cdddfcb965c44fbdd0dd156ee9659


feat: [] Toolkit package

Added the new react-apps-toolkit package.
Mostly reused the things we have in the other packages as well.
I had to add the build command, as the hooks will be written in TS.
Also somehow the bootstrap command of lerna didn't install the packages for this project. Might be because of: https://github.com/lerna/lerna/issues/1457
that's why I changed it to the solution proposed in the issue.

Do we want to merge this into canary? Or should we just merge onto master and release 0.0.x releases until we are happy?

Do we want to merge this into canary? Or should we just merge onto master and release 0.0.x releases until we are happy?

I'd say it is fine to merge this into master to not conflict with the cca changes

Will be merged to master with https://github.com/contentful/create-contentful-app/pull/881


FF 97: Tab heights

On FF97, the tab bar used to be Compact Size bar , but now it's back to the default after the most recent update.
Cannot use the Compact Mode with Material Fox's CSS, as it default to the largest tab.
In about:config settings browser.compactmode.show set to true do not help either.

Remove that “Firefox View” icon might help


Update OddEVEN.c

What the program do ?

In what programming language it is written?

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Extract Window class for cleaner code.

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

Project coverage is 20.87%.

Files
Patch %
Lines

src/VulkanChess/Window.cpp
0.00%
47 Missing :warning:

src/VulkanChess/main.cpp
0.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   27.72%   20.87%   -6.85%
==========================================
  Files           4        4
  Lines         101       91      -10
  Branches       48        0      -48
==========================================
- Hits           28       19       -9
- Misses         62       72      +10
+ Partials       11        0      -11

Flag
Coverage Δ

Linux
?

Windows
20.87% <0.00%> (-0.47%)
:arrow_down:

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

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


Systoliccleanup

Coverage decreased (-21.3%) to 48.121% when pulling 7d7d6531eddc81f41f525a12faaa6b59af3ac644 on systoliccleanup into 9f2568330c7ca8f2160a1b4723c64332ab20f841 on master.


Table structure from the Summary description

Hi
I use the following in the swagger description
` |Field | Description|
|-----------|------------|

|defaultMessage|Human readable message explaining the reason why the field
value is erroneous.|

|field|The name of the JSON field that contains an erroneous value.|

|rejectedValue|The value that was submitted but rejected.|`

** Which produces the following the swagger editor screen  **

However when I run bootprint openapi , final html page is not having table structure as above. Could you please advise ?
After bootprint openapi transformation

Hi and thanks for using bootprint-swagger. This could be fixed for you with a custom lesscss-file. Since this is the output of the marked markdown-renderer there is no partial to be adapted.
Your problem looks worth fixing though. Maybe marked can be configured (in the bootprint-base-package) to render tables with a custom css class to include borders.
But I can't get to it until next Wednesday (probably). A pull-request would be welcome. Ideally it would include a test-case to verify the resulting output (have a look at the tests in this module)

thank you for the reply, I am not sure I could start this change before Wednesday. However I will start to go through documentation and see what i can do. If you are able to fix it before me, please give me a shout.
thank for your contribution
cheers
Rajesh

Please do not hesitate to create issues for missing or bad documentation or to ask for specific developer questions. This is almost as important to me as the code itself.

I am now working on it.

thanks mate, it works as expected. I am in process of testing all of our swagger docs using this tool. I will definitely reach you if have any queries or issues.
cheers
Rajesh


interactionCreate eventhandler missing typings

Use
const payload = data as any;
as a temporary workaround

I'll fix this in types branch when I come to this function.

I'll fix this in types branch when I come to this function.


Remove package attributes from AndroidManifest + added JVM compatibility

I removed package attributes in AndroidManifest to prevent
Setting the namespace via the package attribute in the source AndroidManifest.xml is no longer supported.

error during build phase. Which only happen when you upgrade Android Gradle Plugin to version 8.0+.
I also added JVM toolchain compatibility.
Tested on my app using OPPO Reno 5 Pro (API 33) and Android API 34 emulator.

I will add it and make an update. Thanks a lot. I missed the PR

merged! new package version


Add missing linking libs for iOS

When creating a new iOS app in Xcode with Tensorflow, we need to add these references to fix linking errors.

signed it!  16.03.2018, 11:58, "googlebot" notifications@github.com:Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).📝 Please visit https://cla.developers.google.com/ to sign.Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.What to do if you already signed the CLAIndividual signersIt's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.Corporate signersYour company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the project maintainer to go/cla#troubleshoot.The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.The email used to register you as an authorized contributor must also be attached to your GitHub account.—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


Add GitHub Wiki Page link for exception if WebDriverConfig.properties not present

Title - Add GitHub wiki page link for how to use property file in log Error while loading property WebDriverConfig.properties file. Refer - http://........
Description:
If WebDriverConfig.properties file not present in test suite we provide error message in log, but develop might not understand how to do it. Add GitHub Wiki page URL
Please check content of property file and try again.
11:02:52.587 [main] ERROR org.iqa.suite.commons.listeners.SeleniumMethodInvocationListener - Error while loading property WebDriverConfig.properties file.
Please check content of property file and try again.
java.io.FileNotFoundException: src/test/resources/properties/framework/WebDriverConfig.properties (No such file or directory)
at java.io.FileInputStream.open0(Native Method)

@zodgevaibhav - README.md file has been updated with required details. Please can you check and close the issue if it looks good. Thanks !


WIP: polish be-docker-plain

Enhance be-docker-plain having a valid service running.

Top.. thanks @tbugfinder

@tbugfinder some feedback:

Do we really need to have a full NGINX installation in a plain docker quickstarter template? Users would have to clean up before they can start using it. I'd rather use your example in a README.md or a blog post so others can understand what to modify to arrive at an NGINX Docker image.
Using ENTRYPOINT won't let you override the container process or pass in additional options. CMD is usually preferred (see my comments above). Also, I'd rather not have an extra entrypoint.sh file to carry around unless really needed (which it is not for such simple examples). You should use the ENTRYPOINT or CMD's exec form to run multiple commands in succession directly and without the file indirection like so:

CMD ["sh", "-c", "cmd1 && cmd2"]

What about running just 'nc' and echoing hellow.. done

An approach that's often seen is to simply sleep in a loop. Here is a fully working example:
FROM alpine:latest

CMD ["/bin/sh", "-c", "while true; do sleep 1d; done"]

Here, the CMD directive uses the exec form, which is the Docker recommended way. The exec form though does not execute a shell, but we need one to execute the loop.

I would really consider this https://github.com/opendevstack/ods-project-quickstarters/issues/260#issuecomment-510881603

Status?

@gerardcl I think that a security workflow could easily over-complicate things. Especially at the beginning of a project where you are still in an exploratory phase. Do you think that would be compatible? Also, personally, a security workflow decision doesn't have to block this issue moving forward.
@tbugfinder I prefer @clemensutschig's approach with nc since it overall:

shows how to install an artifact (nc) inside the Dockerfile
shows how to run a command with arguments in a (CMD please) exec form
lets the containerized process respond with a message so the user can verify that the deployment was successful using a simple HTTP request

@metmajer I fully agree on what you say and I believe it indeed follows the quickstarter purposes. My comment was more on the direction on implementing right workflows internally (and maybe expose them from an ODS point of view) so to keep that comment in mind...I also commented in the past that we could (since being only using openshift) stick to official provided images by RH, but I think that a DTR path is much more a better track! I see this as a self-managed artifact/dependency management tool (e.g.: nexus) with an enhanced "check" layer, isn't?
Following the proposals for keeping the docker alive and solve this issue we are in I agree on what you proposed. We could even load a message saying "Hello world" hehe

Hi again,
I was thinking and following your comments now, what about a Dockerfile like the following:
FROM alpine:latest

RUN apk add python3

RUN echo "Hello World" > index.html

EXPOSE 8080

USER 1001

CMD ["python3", "-m", "http.server", "8080"]

?

For me still too complex... just 'nc' should do
gerardcl<EMAIL_ADDRESS>schrieb am Sa., 20. Juli 2019, 17:30:

Hi again,
I was thinking and following your comments now, what about a Dockerfile
like the following:
FROM alpine:latest
RUN apk add python3
RUN echo "Hello World" > index.html
EXPOSE 8080
USER 1001
CMD ["python3", "-m", "http.server", "8080"]
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opendevstack/ods-project-quickstarters/pull/264?email_source=notifications&email_token=AJV7CSHLHWC32CWRCXGPDOLQAMVS5A5CNFSM4ICOZ3D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NQSDY#issuecomment-513476879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJV7CSADV6NHJUWDHCZLNRLQAMVS5ANCNFSM4ICOZ3DQ
.

yep, I am also fine, indeed the goal is that the user just rewrites everything so even better if he/she does not need to also "remove" hehe
yep!

I agree with @metmajer and @clemensutschig, please just use the simplest possible, and then let's get this merged.

@tbugfinder note that an echo "Hello World!" | nc -l 8080 will terminate after it has sent the response once. There should be an option -k, depending on the target OS, that keeps nc listening. This resource may help: https://superuser.com/questions/1008348/netcat-keep-listening-for-connection-in-debian.

@metmajer right! see https://github.com/opendevstack/ods-project-quickstarters/pull/264#issuecomment-511149132
then there is consensus on the solution to provide for this Dockerfile:
# This is an example dummy Dockerfile
FROM alpine:latest

CMD ["nc", "-k", "-l", "<IP_ADDRESS>", "8080"]

Hey @gerardcl and thanks for chiming in! Currently, your implementation wouldn't echo anything. For nc to emit Hello World!, we'd need the following in plain Bash (the <IP_ADDRESS> is optional):
echo Hello World! | nc -k -l 8080

In exec form, this would need to be written as CMD ["/bin/sh",  "-c", "echo Hello World! | nc -k -l 8080"] because of the pipe. What do you think?

@gerardcl Because it's @tbugfinder's first contribution, I'd like him to finish this off. Hurry up, @tbugfinder and let me know if you need support :)

sure @metmajer :) but please test it before since I think it has an issue (not killing) when trying to kill the docker if doing the pipe

Updated the CMD as requested.

@clemensutschig I'm slightly confused where to edit CHANGELOG.md. Should a comment be added into ## [Unreleased] ?

Should ODS make use of https://github.com/teohm/git-chlog ?

@tbugfinder correct

This was tested successfully, indeed.

removed WIP flag.

@tbugfinder Cool! Thanks for the link to the changelog tool, did not know it. It would be really nice to have something like this. Right now though, our changelogs are based on issues, not pull requests ...


feature_request(anchors): capital letters in headers support

1. Summary
If capital letters an anchor names:
it would be nice, if remark-validate-links will not mark it as warnings.
2. Argumentation
Capital letters in URLs is valid: URLs will be open i browsers.
I use capital letters for more readable URLs; see answer for details.
3. Data

Kira_remark-valdate-links__capital_letters_in_headers--non-valid_demo.md

1. [Kira](#Kira)
1. [isGoddess](#isGoddess)
1. [Кира](#Кира)
1. [этоБогиня](#этоБогиня)

# Kira

# isGoddess

# Кира

# этоБогиня

4. Expected behavior
No warnings.
5. Actual behavior
D:\SashaDebugging>remark Kira_remark-valdate-links__capital_letters_in_headers--non-valid_demo.md --use remark-validate-links --no-stdout
Kira_remark-valdate-links__capital_letters_in_headers--non-valid_demo.md
  3:4-3:17  warning  Link to unknown heading: `Kira`. Did you mean `kira`            missing-heading                  remark-validate-links
  4:4-4:27  warning  Link to unknown heading: `isGoddess`. Did you mean `isgoddess`  missing-heading                  remark-validate-links
  5:4-5:17  warning  Link to unknown heading: `Кира`. Did you mean `кира`            missing-heading                  remark-validate-links
  6:4-6:27  warning  Link to unknown heading: `этоБогиня`. Did you mean `этобогиня`  missing-heading                  remark-validate-links

‼ 5 warnings

Thanks.

Ids are processed with GitHub Slugger, which lowercases content by default
https://github.com/remarkjs/remark-validate-links/blob/872b653b57dfa785fe6d84b05291f6d1f8668132/index.js#L189-L191
Slugger has a maintainCase option which may resolve the issue you are seeing. https://github.com/Flet/github-slugger/blob/master/index.js#L17-L23
Try out adding maintainCase option support to remark-validate-links, and if it works feel free to open a PR!

The problem here is slightly different I think. remark-validate-links expects GitHub-style slugging. But your example has hardcoded HTML references.
Say you had:

# Kira

...remark-validate-links doesn’t pick up on the ids: it only sees kira from the heading.

Related to https://github.com/remarkjs/remark-validate-links/issues/1

I’m closing this as full-on HTML parsing is not in the scope of this project. However, as mentioned in GH-1 you can integrate with this plugin by parsing the HTML yourself, looking for ids and names, and exposing them on node.data.id (where node is the HTML node).
This plugin then picks up them!


Temp files not cleaned up

Description
I have a microservice implemented with Nancy and OWIN.   The service's responsibility is to accept files and write them to blob storage.  When the request is over a certain size, Nancy writes the file to disk in a temp directory.  Occasionally these .tmp files fail to get cleaned up.
@grumpydev mentioned on the stack link below that this issue hasn't been seen in awhile but we are on clinteastwood so it looks like the issue hasn't been completly resolved.
https://stackoverflow.com/questions/44847409/nancy-owin-service-fabric-microservice-writes-requests-to-temp-file/44847845#44847845
Steps to Reproduce
I have not been able to reproduce this issue on demand.
System Configuration
We are seeing this problem in our Service Fabric clusters which host Nancy via OWIN

Nancy version:
2.0.0-clinteastwood

Nancy host

[ ] ASP.NET
[X] OWIN
[ ] Self-Hosted
[ ] Other:

Environment (Operating system, version and so on):
Windows Server

.NET Framework version:
4.5.2

Additional information:

These are the related issues around the temp file cleanup that was added to 2.0 #2159, #2430, and #2512.

Those 3 PRs would have been merged in either clinteastwood, barneyrubble or the alpha releases.  I can confirm we've been using clinteastwood since March and are still seeing the behavior.

I've since moved to a new job so I'm not suffering from the temp file issue any more, and I didn't have any luck resolving the issue. We were on OWIN so we could swap RequestStream for RecyclableMemoryStream.

Any additional info on how we could (reliably?) reproduce this?


About Twind's auto-completion when developing Deno's Fresh in VScode

How can I get Twind auto-completion to work when developing Deno's Fresh in VScode?
Is Twind Intellisense not possible and Tailwind CSS IntelliSense needs to be set to work?
// environment

$ deno --version
deno 1.35.0 (release, aarch64-apple-darwin)
v8 <IP_ADDRESS>
typescript 5.1.6

// deno.json

{
  "lock": false,
  "tasks": {
    "start": "deno run -A --watch=static/,routes/ dev.ts",
    "update": "deno run -A -r https://fresh.deno.dev/update ."
  },
  "imports": {
    "$fresh/"<EMAIL_ADDRESS>    "preact"<EMAIL_ADDRESS>    "preact/"<EMAIL_ADDRESS>    "preact-render-to-string"<EMAIL_ADDRESS>    "@preact/signals": "https://esm.sh/*<EMAIL_ADDRESS>    "@preact/signals-core": "https://esm.sh/*<EMAIL_ADDRESS>    "twind"<EMAIL_ADDRESS>    "twind/"<EMAIL_ADDRESS>    "$std/"<EMAIL_ADDRESS>  },
  "compilerOptions": {
    "jsx": "react-jsx",
    "jsxImportSource": "preact"
  }
}

I also have the same doubt, the twind autocomplete does not work for me in vscode

See this issues for alternative solution https://github.com/denoland/fresh/issues/1519

This is a separate VSCode Twind intellisense extension that I developed, which only supports v1. It is still a work in progress. If you're interested, feel free to check it out.


Fix more HDF4

NDG groups
null terminated strings

Fixes #505 , cc @abarciauskas-bgse

This of course reinforces that we will really need some HDF4 tests; but it's less clear what the "ground truth" should be!

Thank you @martindurant ! I tried this with MCD12Q1 (after pip install git+https://github.com/martindurant/fsspec-reference-maker.git@hdf4_2) it is working! 🙏🏽


refactoring to use env path, use token with get requests

Build log
Preview build with this PR (commit): https://output.circle-artifacts.com/output/job/d15f3835-827c-45b0-80cf-b3e863d8dc81/artifacts/0/index.html

Build log
Preview build with this PR (commit): https://output.circle-artifacts.com/output/job/b5d38a0c-4bb9-49fc-82c7-a8bb92bb8e79/artifacts/0/index.html


[controller _manager] Expected Behaviour

Controller Manager should:

Read frequency parameters, and use them for timers. For that, python launcher must be upgrade.
Declare and read all parameters availables when initialize the node and send them to plugins. Have a callback for a parameter change, and send it to plugins.

Done


Shutdown k8s

Description
Just E2E tests.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

Found the available test cluster - LinuxWin for linux. Please check the build status.

End-to-end tests cancelled on linux. Please check the build logs

/ok-to-test

Codecov Report

Merging #38 into master will decrease coverage by 0.86%.
The diff coverage is 40.89%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   54.18%   53.31%   -0.87%
==========================================
  Files          70       77       +7
  Lines        6120     6516     +396
==========================================
+ Hits         3316     3474     +158
- Misses       2545     2771     +226
- Partials      259      271      +12

Impacted Files
Coverage Δ

pkg/injector/injector.go
19.20% <0.00%> (+0.59%)
:arrow_up:

pkg/injector/pod_patch.go
65.33% <ø> (ø)

pkg/placement/raft/logger.go
0.00% <0.00%> (ø)

pkg/placement/raft/server.go
0.00% <0.00%> (ø)

pkg/placement/raft/snapshot.go
22.22% <22.22%> (ø)

pkg/actors/actors.go
47.80% <25.00%> (ø)

pkg/config/configuration.go
73.02% <50.00%> (+6.04%)
:arrow_up:

pkg/http/api.go
75.42% <51.56%> (-1.10%)
:arrow_down:

pkg/grpc/api.go
61.21% <76.92%> (-0.28%)
:arrow_down:

pkg/placement/raft/fsm.go
78.57% <78.57%> (ø)

... and 11 more

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 d558367...f6a6f5f. Read the comment docs.

Found the available test cluster - LinuxWin2 for linux. Please check the build status.

End-to-end tests cancelled on linux. Please check the build logs

/ok-to-test

Found the available test cluster - LinuxWin for linux. Please check the build status.

Found the available test cluster - LinuxWin2 for windows. Please check the build status.

End-to-end tests failed on linux. Please check the build logs

End-to-end tests cancelled on windows. Please check the build logs

/ok-to-test

Found the available test cluster - LinuxWin2 for linux. Please check the build status.

End-to-end tests cancelled on linux. Please check the build logs

/ok-to-test

Found the available test cluster - LinuxWin for linux. Please check the build status.

Found the available test cluster - LinuxWin2 for windows. Please check the build status.

End-to-end tests failed on linux. Please check the build logs

End-to-end tests cancelled on windows. Please check the build logs

/ok-to-test

/ok-to-test

Found the available test cluster - LinuxWin for linux. Please check the build status.

Found the available test cluster - LinuxWin2 for windows. Please check the build status.

End-to-end tests failed on windows. Please check the build logs

End-to-end tests cancelled on linux. Please check the build logs


ref: #1646
@charliermarsh the different violations have different severities. Would you like me to add a setting that lets users only see errors above a certain severity?

@colin99d - Na we need to tackle severities more holistically, I think. We could consider adding a severity level for all of bandit...?

Since AFAIK all Bandit rules have some notion of severity.

I think adding severity into this rule, and then eventually to all the other ones slowly makes a lot of sense. Then we can just make a bandit-wide (and maybe eventually have this work with other rule sets) rule.

@charliermarsh, ready!

Do we inherit this name from flake8? If not, then I recommend renaming the rule to denylist_calls to be more inclusive.

Agreed. Even if it's inconsistent with Bandit, we should still change it. (I believe we've done that in the past.)


Phase 1 of

Implements a new delete extension that allows to avoid the possible conflict of the extra subscript extension with the new strike-through extension. This issue is discussed in https://github.com/gohugoio/hugo/issues/12597.
Also adopts two changes in the original Goldmark code for strike-through.

@bowman2001 Thank you. I will try to take this for a spin either later today or early tomorrow.

@bowman2001 This looks good. I carefully examined the test cases you removed, and I'm fine with the results. I'll will take care of the Hugo side of things and update the documentation.

 And thanks for all your good work. I've learned a lot, again.


Chisel Pipelined ALU

A pipelined implementation of the Chisel "TensorAlu" module as well as more flexible scratchpad read/write interfaces are added. Chisel unit tests are added for the modified ALU. Around ~4x speedup is seen in the cycle count reported by the "ALU Unit test" portion of "tvm/vta/tests/python/integration/test_benchmark_gemm.py", while more modest speedups are seen in all other tests located in the "tvm/vta/tests/python/integration" and "tvm/vta/tests/python/unittest" directories.
Code contributions to this PR were made by the following individuals (in alphabetical order): @suvadeep89, @stevenmburns, @pasqoc, @adavare, @sjain12intel, @aasorokiin, and @zhenkuny.

@adavare I'm not completely following what you are doing here but if you aren't changing the scratchpad interface from the original design, you'll need to make one modification in LoadUop (delay the LSB by the same amount as the synchronous read).
  when(io.vme_rd.data.fire()) {
    mem.write(waddr, wdata, wmask)
  }

  // read-from-sram
  io.uop.data.valid := RegNext(io.uop.idx.valid)

  // delay LSB of idx by a cycle because of the one-cycle memory read latency
  val sIdx = RegNext(io.uop.idx.bits % numUop.U)
  val rIdx = io.uop.idx.bits >> log2Ceil(numUop)
  val memRead = mem.read(rIdx, io.uop.idx.valid)
  val sWord = memRead.asUInt.asTypeOf(wdata)
  val sUop = sWord(sIdx).asTypeOf(io.uop.data.bits)

  io.uop.data.bits <> sUop

If you are inserting our new TensorLoad implementation in this PR, then we corrected that in there.

@stevenmburns: You're right, that bugfix was not previously included in this PR, but I've just added it.

@tmoreau89, @vegaluisjose: I see that "tests/scripts/task_python_vta_tsim.sh" is being skipped in the pr-merge CI. Do you know why this is happening?
The 2 pytest commands in this test run successfully for me in ~12 min or so. I was considering extending the tsim test with Chisel unit tests ("sbt test"), but perhaps it is being skipped since the runtime is already too high?

Nice @adavare, could you please rebase to incorporate #28? we added linting to unittest (was disable before). After this rebase, you could try make lint to see if everything goes fine and we can merge this. Thank you!

@vegaluisjose: I've pulled in #28 and fixed a few linting errors that came up.

@tmoreau89, @vegaluisjose: I see that "tests/scripts/task_python_vta_tsim.sh" is being skipped in the pr-merge CI. Do you know why this is happening?
The 2 pytest commands in this test run successfully for me in ~12 min or so. I was considering extending the tsim test with Chisel unit tests ("sbt test"), but perhaps it is being skipped since the runtime is already too high?

Thank you for the extensive PR @adavare and team. On the testing, I think it would be good to run some more extensive unit tests in this repo (TVM submodule) but we'll want to limit the overall runtime of the Chisel tests in the mainline repo (CI testing is already getting very long).
I'm not too worried about making the CI testing time long on the tvm-vta repo given that PRs don't get submitted too often. Therefore I'm in favor of running longer, more extensive tests.

Let's get this one PR merged in first, then we can enable the unit tests in a separate PR. Thank you @vegaluisjose for your help in reviewing the PR and ensuring the unit tests are passing

Thank you @adavare and the team (@suvadeep89, @stevenmburns, @pasqoc, @adavare, @sjain12intel, @aasorokiin, and @zhenkuny) for these improvements on the VTA chisel design, and @vegaluisjose for reviewing the PR. The PR has been merged.


cli: given a layer hash, return a list of images that contain that layer

Currently, we support listing/searching images within a repo, but we have found it useful to also retrieve a list of images given a layer hash.

@rchincha can you confirm the OCI dist spec doesn't have a query for this, and this needs to be implemented client side?


SolverRadialView should include an option for min / max x axis clamps

Since we frequently use SolverRadialView for billboarded menus we should include an option to clamp the x axis. Currently if the player is looking at the ceiling or floor when a SolverRadialView is enabled it can place menus in strange-looking positions - an x angle clamp would give devs a simple way to prevent this without the additional restrictions that come with SolverRectView.

@Railboy can you describe a bit more what you're seeing?  I can add a gravity aligned if you'd like which would make it not do vertical orientations.

Updated sync and let me know if you have any issues.  Here are the specifics:
		public enum ReferenceDirectionEnum 		{             /// <summary>             /// Orient towards head including roll, pitch and yaw             /// </summary> 			HeadOriented,             /// <summary>             /// Orient toward head but ignore roll             /// </summary> 			HeadFacingWorldUp,             /// <summary>             /// Orient torwards the head movement direction found in veil singleton             /// </summary> 			HeadMoveDirection,             /// <summary>             /// Orient towards head but remain vertical or gravity aligned             /// </summary>             HeadGravityAligned 		}

That should work - will test today.

This is a useful addition but not what we were aiming for.
Here's an image to clarify::

Now I understand more what you're looking at but need to think of a better way of doing it.  The solver system is setup to have multiple solvers that can work independently on an object.  This would allow for a radial view solver with a height or pitch limiter.  I would hate to overload the radial view solver with a number of special case elements.  I'll look into it a bit more and propose something or find a more elegant way of adding it.

Had a thought about this today - what about a constraint system similar to the solvers? Ie, multiple constraint components that are gathered and executed after the solvers.

You actually don't need anything in addition to solvers you just need a constraint solver :).  As in we can just add a constraint solver to the mix and it will work.  I'll do a quick first pass for you tomorrow and get a solver that works as a constraint solver.


SHA1 hash value of box.phar file to verify

TL;DR
What are the SHA1 file hash of the released Box3 (box.phar) files?
TS;DR
Thanks for the effort of the humbug/box contributors to maintain the inherited wonderful 'box'.
I don't know if it's the right place to ask but I would like to know what the SHA1 file hash values of the released box.phar files are.
Since I use 'box.phar' a lot, I customized the conventional (box2's) installer and made an  box3 installer of the latest release for my ease.
Though, the only left part was the checksum comapreson between the remote and downloaded files.
I searched the repo but couldn't find the hash to compare.
So, where/how can I get the file hash value of the released box.phar file?
It doesn't have to be a SHA1 hash, if it's something that can be verifiable.
P.S. I will close this issue if it's not a proper place to ask.
Thanks.

Question
Answer

Box version
3.0.0-alpha.3

PHP version
PHP 7.1.14 (cli)

Platform with version
macOS 10.13.4

Github Repo
https://github.com/KEINOS/Phar_Box3_installer

Hi!
I'm not an expert on the subject, so this is my understanding and might be wrong.
The idea of the SHA1 signature is that when you create a PHAR, a SHA1 code of the PHAR content stored in .phar/signature.bin is generated. When executing the PHAR, before executing any content, the SHA1 of the PHAR will be retrieved and compared against the stored one. If they don't match, i.e. the content of the PHAR has been tempered with, the PHAR won't be executed.
Note however that it's a very simple signing mechanism and I don't think it's a very secure one. Also right now the current releases of Box are not signed (I plan to fix that before the stable release though).

Now of if you permit it I'll deviate a bit on the goal of your project. First thanks for it, I really miss an installer!
I initially planned to contact @theseer and @MacFJA about this but your issue may be the good opportunity for this :)
I really think the security with PHARs is an issue at the moment: it's a pain in the ass to deal with and I would like to see if we could improve that in any way. So here are the following points I would like to discuss with you guys.
Simplify the installer process
Here's a list of some installers: Composer, Box2, Puli.
To me the whole installer process looks very tedious, brittle and error prone. It is hard to get things right and easy to mess up and overall a very laborious work.
What I really would love to see is either a generic installer for PHARs or a way to generate the installer for each specific project. I.e. there would be a kind of generic installer that you need to configure and it would dump the project specific installer.
 IMO the greatest pitfalls are:

You need to generate keys, register a GPG key eventually
Once you are signing your PHAR, it's harder for your contributors to contribute as they can no longer generate the PHAR. I did something there in PHP-Scoper via the Makefile, but I would like to ship a "dev" mode with Box: it skips the signing but would trigger a message when executed that it's a PHAR for dev purposes and not for prod.
It is harder to deploy & distribute
It is much less convenient to use as you need to keep the public key (acme.phar.pubkey or acme.phar.asc) next to the PHAR

Maybe there is something that could be simplified here, like a la Composer with the CA Bundle... I admit I couldn't dig much into it yet...
Simplify the PHAR tracking
So maybe [phar.io](phar io) can help with the installer & tracking PHARs in your project. To me the biggest drawback I find to it right now is that it requires to install a dedicated too where I would much prefer a Composer plugin. For example this is what I do most of the time to not deal with PHARs right now:
composer require --dev bamarni/composer-bin-plugin
composer bin box require --dev humbug/box:^3.0@dev

This is done by leveraging bamarni/composer-bin-plugin and helps to install those utility tools without polluting your project dependencies whilst making it easy to discover and keep track of.
With PHARs... There is nothing except phive, but it's not on a project basis and requires to install a dedicated too. Instead maybe we could achieve the same experience:
composer require --dev phar-io/phive-composer-plugin
composer phive install phpunit

WDYT?

@theofidry
Wow! Thanks for the quick reply! 😃

The idea of the SHA1 signature is that when you create a PHAR, a SHA1 code of the PHAR content stored in .phar/signature.bin is generated.

I re-read my post and realized that I was asking wrong! I wasn't asking about the generated .phar file by the "box.phar" but the "box.phar" it selt. Sorry for my writing...
I just wanted to know the hash of the "box.phar" itself ($hash = sha1_file('box.phar');), so that I can check if the downloaded "box.phar" file from the releases page is not broken or corrupted,  as a checksum. 😣
A checksum like the typical download links, you know. NOT the SHA1 signature of the generated Phar files by the "box.phar".
I looked up the assets in the release API of the repo though. But it seems that GitHub only provides filesize and timestamps of the file and doesn't provide the SHA1 hash like the way commit files does.

Simplify the installer process

In my opinion, the current install process is basic enough. Which is:

Downloading the "box3" ("box.phar") from your release page.
Place it to one's project.

But the reason I made the Box3 installer, was that many people that I know, asked me that the "box.phar" doesn't work (doesn't archive), and most of the reasons were whether the "php.ini" settings were wrong or PHP extension missing.
So, I forked the Box2's installer which checked the environment before downloading the "box.phar".
I also feel a little tedious about using composer.
Since the downloaded "box.phar" archiver works perfectly for me, so far,  and I'm not skilled enough to use it in an advanced way, I do not have the need to keep the whole thing under vendor/ directory. That's another reason why I made the Box3 installer.

Simplify the PHAR signing

I wasn't thinking enough about distributing my ".phar" files generated by the "box.phar". Like signing or implementing the self-update feature to my PHP archives. All I had in my mind was to replace the archive manually to the new ones.
It would be my next level to think about since I'm pretty new with archiving as phar and using "Box".
Though, I understood the importance and the hardness of implementing these features to the archiver (Box3).
Sorry that I can't tell much. Wish had more skills to talk about inside the "box.phar". 😭

@theofidry
I updated (TL;DR part of ) the question. Please consider and I'm sorry for your inconvenience but thank you for your cooperation. 🙇
@c33s
I agree to use $checksum = hash_file('sha256','box.phar');  rather than using $checksum = sha1_file('box.phar');.
As long as I can verify the hashes between the provided one from the releases pages and calculated from the downloaded file, it'll cover my needs. The secure the better so why not, right?

A checksum like the typical download links, you know. NOT the SHA1 signature of the generated Phar files by the "box.phar".

Oh that one, sorry I don't know :/

In my opinion, the current install process is basic enough

I'm not sure I consider a 1KLoC of code simple :) Also it "works fine" because it works for you, but you're lucky! This script is under-tested to say the least and can break in many different ways. For example:

link1: there is no test for that version, the requirement checker of Box is unless tested up down to 5.3.29 IIRC
link2: this will break if the PHAR extension does not exists
link3 this and like most other FS related functions can just fail

I will review that code later once you consider it ready, I would be happy to merge it to Box. But my point is it is very error prone and hard to get right.
I also suspect that this script was heavily based on the Composer one at that time, so maybe we could re-use the one from Composer as well.
Another thing that an installer should do is checking the integrity of setup script itself:
php -r "copy('https://getcomposer.org/installer', 'composer-setup.php');"
php -r "if (hash_file('SHA384', 'composer-setup.php') === '544e09ee996cdf60ece3804abc52599c22b1f40f4323403c44d44fdfdd586475ca9813a858088ffbc1f233e9b180f061') { echo 'Installer verified'; } else { echo 'Installer corrupt'; unlink('composer-setup.php'); } echo PHP_EOL;"
php composer-setup.php
php -r "unlink('composer-setup.php');"

And the script itself should check the integrity of the PHAR.

I also feel a little tedious about using composer.

That's entirely fair. I don't want to remove the ability to easily install a PHAR without Composer, that's what the installers are for, this is just about allowing one to do so.
One issue if you install a PHAR like that and use it for a specific project, someone starting with that project has no idea there is a dependency to that PHAR and even if this can be documented, he/she still has to install it. Ideally you should have the possibility to "declare" that dependency in some way.

It would be my next level to think about since I'm pretty new with archiving as phar and using "Box".

No problem :) I'm was pretty new to PHARs a couple of month ago as well 😄

Hi,
since I got mentioned, I figured I should add a few lines here ;)
Quotes In random order from previous posts:

With PHARs... There is nothing except phive, but it's not on a project basis and requires to install a dedicated too.

I'm not sure I understand that. Phive very much works on a per project basis. It even keeps track of the installed version, version constraint (e.g. when run with one like phive install<EMAIL_ADDRESS>and target name for you in a file that can and should be added to the project repository. A phive install (or, when you have the composer plugin installed composer phive install) will recreate the persisted state.

Simplify the PHAR signing

There is only so much that can be simplified here. One could theoretically wrap the gpg calls as well as the uploading of a newly generated key to the keyservers into a shellscript or even small cli application. But I'm not convinced that's worth the effort, as many "edge" cases need to be considered for key handling (import existing, first run, expire/update/... key) and whether calling magic-helper sign -with-key release produciton-key is any more simple than gpg -u<EMAIL_ADDRESS>--detach-sign --output foo.phar.asc foo.phar?

Phar Installers

To me, all those phar installers are a serious pita. Almost all installers suggest Piping curl or wget call output directly into php (or sh, for other tools and languages). That's a security nightmare and should be forbidden. It also is not reproducible as of course the webserver output will change.
I'm not a big fan of self-updating phars either. Handling all the security implications is way out of scope for every tool except for phive or composer -- when composer finally starts to implement security. Merely checking a sha256 or other hash is useless - from a security perspective. We, as in phar.io's phive, have a pending ticket to provide a library that can make use of phive's infrastructure by roughly mapping a self-update call to a phive update.
An installer also externalizes dependency information.  Once you have the phar in hand, like after a manual download, there is no way of finding out what is required other than running it - and wait for it to fail. A imho better way is to use a manifest within the phar - like https://github.com/phar-io/manifest. Phive checks the php version specified as well as required extensions and let's the user decide whether or not the installation should continue in case of a mismatch. Generic code in the phar could be written to re-test the environment. Maybe we'll extract this from phive into a reusable library.

Thanks for the reply @theseer :)

when you have the composer plugin installed

I think I totally missed the composer plugin, I'll have to check it out!
Regarding the PHAR signing, I don't necessarily mean as the author. Proper documentation (and there's already a couple of articles about it) may be enough. The big negative part I see however (for users) is having to deal with the .pubkey/.asc.

To me, all those phar installers are a serious pita.

That's fair. But I also think one a la Composer is "safe enough". Again I'm not say this should be the de-facto way to install it, I'm saying if could still be an alternative as for some users it's good enough even if it's not the most secure way.

I'm not a big fan of self-updating phars either. Handling all the security implications is way out of scope for every tool

I agree, plus the versioning is really tedious to deal with. I would honestly prefer to make PHIVE more accessible & convenient to use than investing efforts in this.

An installer also externalizes dependency information. Once you have the phar in hand, like after a manual download, there is no way of finding out what is required other than running it - and wait for it to fail. A imho better way is to use a manifest within the phar - like https://github.com/phar-io/manifest. Phive checks the php version specified as well as required extensions and let's the user decide whether or not the installation should continue in case of a mismatch. Generic code in the phar could be written to re-test the environment. Maybe we'll extract this from phive into a reusable library.

I think there's something that could be done there. I like the manifest idea: it's really neat that phive can check before that it can install the PHAR, this is in fact the main reason behind installers. I however really don't like the idea you need to maintain a manifest for it. At all.
But I think we can greatly improve this. Maybe in another issue, but I managed to get a requirement checker in box working down to PHP 5.3. So unless you are in a lower version, as long as the app requirements were properly listed in the composer.json, i.e. you added ext-intl: * if you need the intl extension, you will have a graceful failure listing what is wrong. Maybe there would be a way to generate a manifest file from it that could be published along side the PHAR which phive could read when present...

aren't we mixing to many topics in one issue? shouldn't we split it up in different issues?
we come from a simple "please provide a hash signature for your downloads/releases" to installer and gpg signing :)

To add the little knowledge I have on this subject, there are several kind of Phar signature:

An embed signature (link 1, link 2)
A side file signature (often with the extension .pubkey or .asc)
A crypted Phar (similar to the creation of a side file, but everything is encrypted)

As noted by others, the main inconvenient with a strong signature is, it implied an asymmetry encryption and require a trusty tier server to store the key to decrypt/validate a signature (GPG and OpenSSL).
And you have to retreive this key to ensure that the phar is valid.

aren't we mixing to many topics in one issue? shouldn't we split it up in different issues?

To @c33s, yes and no :smiley:
Adding a hash/signature is simple, but the necessity to have a good process for futur use is not. If tomorrow I need/want to add a hash (on its way) I prefer to have something that everyone can understand and refer to and don't recreate the same thing every time.

I'm sorry if @KEINOS is a bit spammed as indeed it was not the original question :P
I'm however happy to have a bit of a brainstorm here and we can open well defined & scoped issues from this later.
@MacFJA isn't there also a way with the Mozilla CA-Bundle as done in Composer and Puli?

From what I understand of composer/ca-bundle and its usage, it's for getting the path to global certification file (mainly (only?) for HTTPS)
In composer it's only used to retreive content from HTTPS url (except if I missed something)

@theofidry

I'm sorry if @KEINOS is a bit spammed as indeed it was not the original question :P

No worry. Thanks. I felt more like a Padawan or an Initiate summoned to Box Masters' Council :p
I think I need to read the source more.

it "works fine" because it works for you, but you're lucky!

Indeed. I'll challenge to make the CI tests that at least I can do.
Or will it be possible to implement the requirent-checker to the installer?

please provide a hash signature for your downloads/releases

Since now I know that there's no hash providing at this time, for a temporary treatment, I included the hash values of each 'box.phar' releases into the Box3 installer's manifest.json, which were calculated in 3 different environments. (Work, Home, Friend's house)
I will switch to the /humbug/box's hash as soon as it provides or find another better way.

I will review that code later once you consider it ready, I would be happy to merge it to Box. But my point is it is very error prone and hard to get right.

I will be humbled if that happens! Though, I'm not sure if I can make the original installer better. But I'll try my best.
For those who read this and the Box3 installer's source, and had any suggestions please feel free to PR and anneal it !!

Or will it be possible to implement the requirent-checker to the installer?

I think we could sort of take the same idea and generate an installer. The idea is to:

Generate an installer
Generate the hash
Generate the instructions (as does Composer)

The biggest challenge I see in the installer is that the code cannot rely on any autoloading of some kind. The requirement checker actually does.
I'm wondering however, do you guys think that using a PHAR for an installer would be ok? We still check against the hash before executing it and a big advantage of using a PHAR is that you could rely on the autoloading.
I appreciate that an installer is not best way to installer a PHAR, but again this is about having an alternative :)

Closing in favour of #8. The goal is to have 4 possible ways to install a PHAR:

Composer (so you install the library not the PHAR per se)
phive
the installer (#8)
downloading the PHAR over HTTPS (GitHub releases) -> this does not require a hash check since it's provided by HTTPS itself

downloading the PHAR over HTTPS (GitHub releases) -> this does not require a hash check since it's provided by HTTPS itself
i don't agree with you at this poin, hashes are not only for security but also for integrity checks but the main part is that github hosts its binary files on a different webpage (currently amazon)

https://github.com/humbug/box/releases/download/3.0.0/box.phar redirects to something like this
https://github-production-release-asset-2e65be.s3.amazonaws.com/109533757/5e856880-b507-11e8-96f6-34c84e87a2d6?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20180917%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20180917T133907Z&X-Amz-Expires=300&X-Amz-Signature=9a8041d4bb8dd3e9a3b6e5903d75eabe70105ec3fa5db53e79996ece16003697&X-Amz-SignedHeaders=host&actor_id=649209&response-content-disposition=attachment%3B%20filename%3Dbox.phar&response-content-type=application%2Foctet-stream
of course all is https but it would improve security if you provide hashes. a file downloaded from a hacked amazon storage could still be verified. additional signing the phar with gpg would even more improve the security.

Yes I agree, I’ll GPG sign it at some point
On Mon 17 Sep 2018 at 15:42, Julian<EMAIL_ADDRESS>wrote:

downloading the PHAR over HTTPS (GitHub releases) -> this does not require
a hash check since it's provided by HTTPS itself
i don't agree with you at this poin, hashes are not only for security but
also for integrity checks but the main part is that github hosts its binary
files on a different webpage (currently amazon)
https://github.com/humbug/box/releases/download/3.0.0/box.phar redirects
to something like this
https://github-production-release-asset-2e65be.s3.amazonaws.com/109533757/5e856880-b507-11e8-96f6-34c84e87a2d6?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20180917%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20180917T133907Z&X-Amz-Expires=300&X-Amz-Signature=9a8041d4bb8dd3e9a3b6e5903d75eabe70105ec3fa5db53e79996ece16003697&X-Amz-SignedHeaders=host&actor_id=649209&response-content-disposition=attachment%3B filename%3Dbox.phar&response-content-type=application%2Foctet-stream
of course all is https but it would improve security if you provide
hashes. a file downloaded from a hacked amazon storage could still be
verified. additional signing the phar with gpg would even more improve the
security.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/humbug/box/issues/165#issuecomment-422021334, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AE76gUOiGq1m97lTnI8tU55Rf3txkoF5ks5ub6bMgaJpZM4TlFk0
.

a hash, especially a sha256 also help. with such a hash it is easy to check the file at virustotal
https://www.virustotal.com/#/file/397F012CAE0E7124CC2F949DAA45CEF594BFE2784C700FE33724DEA43CE89777

IMO ensuring the download is done via HTTPS is enough here: it will do the checksum check and what it lacks is if Amazon, GitHub or the repo are compromised. But in any of those cases, providing a hash is not sufficient anyway.
So I think for that solution it's good enough, for better security having the installer and/or installing it via phive.io (with a GPG check) is the way to go


MudDataGrid add documentation and CancellationToken

Feature request type
Enhance component
Component name
MudDataGrid
Is your feature request related to a problem?
MudDataGrid doesn't have a documentation.
Describe the solution you'd like
I want to have documentation for MudDataGrid and adding CancellationToken just like MudTable.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

MudDataGrid doesn't have documentation.

Do you mean XML documentation?
It was done for v7: https://github.com/MudBlazor/MudBlazor/pull/9000
I think it was also @jperson2000 who added CancellationToken for MudTable recently.

MudDataGrid doesn't have documentation.

Do you mean XML documentation? It was done for v7: #9000 I think it was also @jperson2000 who added CancellationToken for MudTable recently.

no I meant example in mudblazor doc.  I checked dev.mudblazor I couldn't see any example for it and the CancellationToken not added for it till v<IP_ADDRESS>-preview 4.
I wanted to test v7.0.0-rc.1 but it has an error and I couldn't run it.
I get this error :
Unhandled exception. System.IO.FileNotFoundException: Could not load file or assembly 'MudBlazor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.

File name: 'MudBlazor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'

Process finished with exit code 134.

@sardar97  I have the same error as you, do you have any way to fix it?


Small cleanups about opensslconf.h

Checklist

[X] compilation tested !

There two duplicate inclusions,
And no need to generate_buildtest.pl on opensslconf.h as it is the 1st include made in it.

Ping @levitte : I open a .d file to see its content, and found these nits .. at the end..

Merged.
c7e10755fac1d8f4421aa60646d311ab2068752d opensslconf.h inclusion cleanup No need to buildtest on opensslconf.h


Additional Custom API Activities

There are more Activities in Teams now that are not listed in the presence state list on the Configure Custom API tab.
Seemingly missing:

InAConferenceCall
Inactive
OffWork
OutOfOffice
PresenceUnknown
UrgentInterruptionsOnly

List of current states on https://docs.microsoft.com/en-us/graph/api/resources/presence?view=graph-rest-beta
If these could be added for the next release, that would be great.

If this should be a feature request, apologies in advance.

Two different things here Availability vs Activity. PresenceLight at this time does not change lights based on Activities. I would welcome a PR to add such functionality, but would need to ensure that users had the option of using one or the other and can control light color accordingly.

Ah, got it. Thanks.


py3 compat

this works for me (python3.4)

Thanks for the PR. I've committed with a few fixes for Python 2.7.


promote group all to templated variable

Required by ansible >= 2.2
Fixes: #1960

This change is

@wattsteve

@eparis this is needed anyway with ansible >= 2.2. Checked that with Vagrant. If no nits it can be merged right away.

@redhat-k8s-bot test please


LIGHTS | Fade in / out

Hello,
would it be possible to implement some kind on fade in / out on turning off and on lights?

Hi @TonnyCS
Sorry it's taken a long time to get back to you! I own an L1 model now and noticed that when changing brightness or when turning on, it fades already? It's just when turning off that it doesn't fade?
Do your light strips do the same?


the user journey

should be done in figma, and covering the flows for user, admin, supervisor

need feedback from Mr. Nayer

implemented!


Remove the QueryList type

I think this is safe to remove now. On jvm we have CopyOnWriteArrayList, native we use stately, js we use a normal mutable list and it should accomplish everything we had before.

my biggest question is if CopyOnWriteArrayList is safe to use an iterator over.
ie is this safe:
copyOnWriteListOne + copyOnWriteListTwo

internally this does
  val result = ArrayList<T>(this.size + elements.size)
  result.addAll(this)
  result.addAll(elements)
  return result


AQCU-1352: missing qualifiers

include qualifiers on dates that are exactly matching the start/end times for the period

Coverage remained the same at 98.047% when pulling 7523baef5d457f04a78d0ebbf78d5f9cac640170 on mhines-usgs:AQCU-1352 into 1fe3ea85205083b3dfb125f116e5b43802430cfa on USGS-R:master.


Know memory layout / size of a class for debug purpose?

Background
When working on https://github.com/flutter/flutter/pull/113927, I find it weird:
// case A
class AnimatedController {
  String? debugLabel;
  String toString() => ...debugLabel...;
}
void main() { ... use AnimatedController and toString ... }

// case B
class AnimatedController {
  // no debugLabel field at all
}

In both cases, my (draft) experiment surprisingly says a profile flutter build (dart aot) yields the same memory usage! I originally thought removing one field that is used must reduce memory usage.
Problem
May I know memory layout / size of a class for debug purpose? For example, if I can see what is the memory layout of AnimatedController, then I can very clearly know what happens and why memory is not reduced.

You can use --print-object-layout-to=file.json for gen_snapshot. This will dump class layouts in JSON format to a file. In Flutter tooling you can pass this flag through --extra-gen-snapshot-options=....
Dart objects are currently aligned by two word sizes, so removing a field does not always reduce the size of the object.

@mraleph Hmm my experiment shows different result about this:

the field is also effectively unused if it always contains the same constant value

The result is:

v1: 2.51GB
v2: 4.00GB

But we know secondField is always null (always contains the same constant null)
run using: dart compile exe a.dart && ./a.exe. On a 64bit mac.
class C {
  int? mainField;
  int? secondField;

  int get hashCode => mainField.hashCode; // v1
// int get hashCode => mainField.hashCode ^ secondField.hashCode; // v2
}

Future<void> main() async {
  print('start');
  final arr = <C>[];
  for (var i = 0; i < 100000000; ++i) {
    arr.add(C()..mainField = 10);
  }
  print('hash=${Object.hashAll(arr)}');

  print('sleep...');
  await Future<void>.delayed(const Duration(seconds: 10000));
}

Or, another almost equivalent experiment:
v1 2.51GB, v2 4.00GB
class AnimationController {
  int? mainThing;

  // v1
  String toString() => '$mainThing'; // and no field "debuglabel" at all

// v2
// String? debugLabel;
// String toString() => '$mainThing $debugLabel';
}

Future<void> main() async {
  print('start');
  final arr = <AnimationController>[];
  for (var i = 0; i < 100000000; ++i) {
    arr.add(AnimationController()..mainThing = 10);
  }
  print('hash=${Object.hashAll(arr)}');
  print('tostring=${arr.first.toString()}');

  print('sleep...');
  await Future<void>.delayed(const Duration(seconds: 10000));
}

Btw thank you and I find out how to make that PR experiment work (add two dummy fields to make it cross 16B alignment)


8.15.6 - Bring back Aggregation LocalMetadata

Elastic.Clients.Elasticsearch version: 8.15.6
Elasticsearch version: 8.15.1
.NET runtime version: 8.x
Operating system version: Any
Description of the problem including expected versus actual behavior:
We initially had long discussion about adding support for LocalMetadata and did a PR for this feature here https://github.com/elastic/elasticsearch-net/pull/2320 .  LocalMetadata allowed libraries like Foundatio.Parsers / users to pass along extra data locally on the aggregation for things like runtime type, timezone information and more. This allowed us to parse a lucene like (dsl) string and convert it to your agg model, consume the returned aggregation data and convert it to our own dsl / models which supports multiple tech stacks like sql server.
Expected behavior
Bring back LocalMetadata so we can store data on the aggregation for round tripping.
Reference: https://github.com/FoundatioFx/Foundatio.Parsers/pull/84

Hi @niemyjski,
This sounds like a very specialized use-case that might not be required for 99% of the users. Isn't that something that could easily be implemented by the user with a ConditionalWeakTable?

Yes, great suggestion and I was kind of thinking along those lines of having a lookup table. I'm going to try this approach.


"Better" daily tab

I'm playing around with a new Daily tab which has more informations but also requires a bit more user interaction, not sure if it's better this way or not :/.

One huge advantage is that I can now include "Tomorrows" dailies and also extra categories like Fractals or special stuff like Wintersday/Halloween/FestivalX dailies.

A small video of that feature: http://take.ms/vx4nG
With e8d94c77ec836cf468d52f6978e90c39d842878e it's in the repository but not live yet, still haven't decided if I like the new one more ;).


Problem in "Configuring the Proxy Server" documents

Description:
I tried to run a standalone deployment behind a reverse proxy server. The configuration and tutorial in the provided documents are not complete since I did exactly like what it said but there are some problems.
Here is the devportal view:

When I click on the Sign in button, the hostname which should be myDomain name, changes to the localhost
This problem is the same for the publisher portal too.

Affected Product Version:
WSO2 API Manager 3.0.0
OS, DB, other environment details and versions:
Ubuntu 16.04 and Nginx as the reverse proxy server.
Related Issues:
https://github.com/wso2/product-apim/issues/6808

just to confirm, was this the doc you followed? https://apim.docs.wso2.com/en/latest/InstallAndSetup/DeployingWSO2APIManager/configuring-the-proxy-server-and-the-load-balancer/


Issue Overriding a Mounted Instance

Version
3.0.2
Reproduction link
https://jsfiddle.net/geLtaor2/
Steps to reproduce
Select option "A" and then select option "B" to receive the exception.
However if you select option "B" first you don't receive the exception.
What is expected?
The number 3 should be displayed.
What is actually happening?
The exception "Cannot read property 'nextSibling' of null" is thrown.

You can't manually manipulate the HTML at the same time you make Vue control it. To avoid this kind of problem, wrap vanilla JS inside a component but you can't use both at the same time

@posva I've simplified my example but what I actually do is return back the HTML from an ajax call to the server and then try to wire up any Vue components in the returned string. It works the first time, but if the element has already been mounted via Vue then it doesn't work.

Ah, I see: you cannot mount an app inside of another app, you have to use components instead

After further research I believe this scenario should be supported. I've just tested it in Vue 2 and it works fine. https://jsfiddle.net/cydnhpu6/, therefore it is a regression.


Multiplatform ViaVersion - Sponge / BungeeCord / Bukkit

Please report any issues you have with the modular build in the comments.
We've been working on developing ViaVersion for Sponge and Bungee, this comes in a multiplatform jar which should work on: Sponge, Bukkit, BungeeCord.
This has been tested to work on SpongeVanilla & SpongeForge. Just drop it in your mods folder and go.
It has also been tested on BungeeCord.
This supports the same versions as normal ViaVersion
There are a few missing features but otherwise it should work fine.
You might have noticed that there is a 'modular' branch.
Please know that this builds might be unstable and are not meant for production use.
Downloads: BuildServer
Sponge Thread
https://forums.spongepowered.org/t/wip-viaversion/14928
If you run a Sponge server it would be appreciated if you could help us test ViaVersion for Sponge to ensure there are no issues.
TODO:

[x] See if CraftBukkit 1.9 is fixable (yay!)
[x] Implement the handItem cache for Sponge
[x] Test CommandBlock system & remove old system from listeners
[x] Bungeecord:

[x] Handle unsupported protocols correctly
[x] Handle unregistered or failed to ping servers
[x] Default hand is always left
[x] Look into this exception: https://paste.matsv.nl/mefozohepe.avrasm
[x] Look into why 16w39a is readTimeOuting (309 is changed to 210 before bungee)
[x] Config Option for Server Protocols
[x] Config Option for Searching Interval (also allows disabling, if disabled search server command says so)
[x] Fix magic random order @ config
[x] Profile / Look to reduce reflection
[x] Add info about server protocols to dump

Is ViaVersion ready for 1.11 ? :D

@TheIntelloBox On the edge of, providing no major changes tomorrow.

Closing this issue as this has been merged with master, will be released for 1.0.0. (Hopefully today if mojang releases 1.11)


Fix gtest dependency

This has two fixes:

it fixes the CI to install the gtest development files
it fixes the Meson build to allow gtest to fall back to a wrap

Yeah, fallbacks with optional dependencies have lots of weird interactions in Meson. I don't understand them all, obviously.


Add quotes

completed captions in csv
referencing issue #2

Looks good! Happy Hacktoberfest.

@RushiVachhani I had to revert these captions as they didn't pass the csv validators that I checked with later on. But you can validate and submit them once again. would love to have them in our sheet.

Sure


Bitwise shift

It seem the "Bitwise shift" test for 32bits system only.
use "PHP_INT_SIZE * 8" to get the real value for the system. or ask for system architecture to be tested in commandline parameters.

Target system's integer size can now be specified using the --integer-size command line option


Fix email bug: wrong flow status for past executions

Pull Request Test Coverage Report for Build 3900

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 35.867%

Files with Coverage Reduction
New Missed Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/ConditionalWorkflowUtils.java
1
92.5%

azkaban-common/src/main/java/azkaban/jobExecutor/AbstractProcessJob.java
2
64.29%

azkaban-exec-server/src/main/java/azkaban/execapp/JobRunner.java
2
76.67%

Totals

Change from base Build 3897:
-0.02%

Covered Lines:
10319

Relevant Lines:
28770

💛  - Coveralls


Proposal to add AdminStatePac to ConnectivityServiceEndPoint and ServiceInterfacePoint

TAPI and MEF NRP State Machine Alignment.docx

This issue was discussed in the TAPI call on 06/27 and it was agreed to enhance the state attributes for the SIP and CSEP as proposed. Separate related issue is #195 that proposes SIP/CSEP update operations.


Modified generator to generate default value for parameters in description

Description
Modified generator to generate default value for parameters in description
Issues Resolved
Related to #477
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.

I think we should have a code coverage goal that includes generated code. Try to come up with corresponding auto-generated tests?


Bug Fix: Crash if load font regular and solid style

FontAwesome was crashed if load font both regular and solid styles.

Crash Source Code
let starImage = UIImage.fontAwesomeIcon(
    name: .star,
    style: .solid,
    textColor: UIColor.yellow,
    size: CGSize(width: 60, height: 60)
)
let starOImage = UIImage.fontAwesomeIcon(
    name: .star,
    style: FontAwesomeStyle.regular,
    textColor: UIColor.yellow,
    size: CGSize(width: 60, height: 60)
) // crash

Cause of this bug
In my research, checking loaded font logic is wrong:
public class func loadFontAwesome(ofStyle style: FontAwesomeStyle) {
    if !UIFont.fontNames(forFamilyName: style.fontFamilyName()).isEmpty {
        return
    }
    ...
}

Style solid and regular has same fontFamilyName() but different fontName().
First load solid, and then 'regular', FontAwesome expect "Font-Awesome-5-Free-Regular-400.otf" file was already loaded, but actually it is not loaded.

@bobbyleary I agree it’s more reasonable. Mind submitting a PR?

@bobbyleary Your code makes sense! Thank you for your comment. My PR will close and hope yours.


[Community] [WFLY-19715] Add documentaion for the new attributes added to control resource constraints for the HTTP management interface.

This PR depends on the following feature being merged to WildFly Core and WildFly upgrading to the corresponding version - https://github.com/wildfly/wildfly-core/pull/6166
https://issues.redhat.com/browse/WFLY-19715

@bstansberry I have applied the suggestions and removed the examples.

Thanks @darranl


Add CustomPDFReader or additional params for the VerticalPDFReader/HorizontalPDFReader

Hi!
Thanks for the lib.
Can you add the ability to create a custom PDFReader or add some parameters to modify the inner VerticalPDFReader/HorizontalPDFReader LazyColumn?
Example:
I need to add paddings for the pdf items.
Can I do it differently?
// VerticalPDFReader
state.pdfRender?.let { pdf ->
            LazyColumn(
                modifier = Modifier
                    .fillMaxSize()
                    .tapToZoomVertical(state, constraints),
                horizontalAlignment = Alignment.CenterHorizontally,

                verticalArrangement = Arrangement.spacedBy(space = 24.dp),
                contentPadding = PaddingValues(24.dp),

                state = lazyState
            )
...

Thanks!

@zabochen
The implementation can be more complex than it may seem because it could interfere with the zoom and pan logic, I add the feature in the list of things to add in future releases.


Update data.json

Herrie must be dead because the site redirects to some dodgy looking garbage.

Thanks for noticing this, but it doesn't need to be deleted. URL can instead be updated to https://github.com/EdSchouten/herrie and a new comment can say that the project was abandoned by its author.

Thanks.


Added function for top writers

Added function for top writers and also     Update for Followers Count, in response to issue #12 opened by jarifibrahim

Merging this not sure if this will work now, but have to make all this up again. Thanks!


Update Level 2 Scene

Implemented new fbx with the level 2 updated from artist.
Test it playing lvl 2 scene, we can make some changes as you like in timing for platforms or lasers etc...
Hope you like it C:

Is the link of the large files in README updated? In order to download everything needed for lvl2
Thanks!

Is the link of the large files in README updated? In order to download everything needed for lvl2 Thanks!

Nope, I will update it sorry!


Remove height 32 from RLN

Remove tree_height 32 resources and associated circuit loading within

Part of https://github.com/vacp2p/zerokit/issues/237

Please also add some description to your PR, and link back to the issue in which this was assigned to you

Done


Add missing iam_policy proto

Fixes #100

[ x] Tests and linter pass
[ ] ~Code coverage does not decrease (if any source code was changed)~
[ ] ~Appropriate docs were updated (if necessary)~

I signed it


Blueprint Collections / sharedModels - allow for choosing a bundle

It would be nice to allow choosing a bundle where we want to create a collection / sharedModel. For example:
const Fields = collection({
  id: "Fields",

  fields: [ ... ],

  relations: [ ... ],

  bundle: "Fields" // if not specified, it should be "App"
})

The usefulness of this is that I can separate, from the beginning, my app-related logic (which might be very specific, hence not very reusable) from stuff that can be built having reusability in mind.

I think that we should also keep in mind the way some files are imported, e.g. instead of having
import { something } from "../../collections/...";

we should do:
import { something } from<EMAIL_ADDRESS>