[ilp-core] Restrict allowed chars in headers for PSK

 Issue by adrianhopebailie
Monday Aug 07, 2017 at 14:17 GMT
Originally opened as https://github.com/interledger/java-ilp-core/issues/69

See https://github.com/interledger/rfcs/pull/257 for context

 Comment by controlsurface
Monday Oct 23, 2017 at 21:54 GMT

@sappenin , all, am planning on taking this one on next.

 Comment by sappenin
Tuesday Oct 24, 2017 at 00:41 GMT

Sounds good - although you’ll probably want to work off the Hyperledger version now - we’re moving the project under that governance structure - https://github.com/hyperledger/quilt

I would appreciate some input on this one.  I have found that while it is easy to control the characters allowed while encoding, there are some cases for decoding that are not possible to control.
An invalid private header of answer: But we control these\n machines; they don't control us! is detectable because the value portion does not contain a colon.  The '\n' results in the code creating a private header of answer -> But we control these followed by a new line of machines; they don't control us! which is detectable as an invalid header.
However, if the value does contain a colon, answer: But we control these\nmachines: they don't control us!, then we end up with two private headers answer -> But we control these and machines -> they don't control us! which ends up being valid.  Another example would be answer: But we control these\n{\"oracle\":\"candy\", \"forseen\":true}", in which case we end up with valid headers of answer -> But we control these and {\"oracle\" -> \"candy\", \"forseen\":true}"
To detect the latter example, we could add a restriction on the characters allowed in the key portion of a header (no curly braces for example).  However, from a decoding perspective, if a colon is created in the value portion, a nefarious encoder could could effectively inject a header or payload data in the value portion of a header.  Thoughts ?

@controlsurface: without providing an answer I hope the following is helpful. The PSK headers are modeled on HTTP headers so I assume there are some good examples of HTTP header decoders out there that have solved this problem.

So, I'm wondering if we're pursuing this in the wrong direction -- seems like picking out various illegal characters is not the best approach here.
Could we instead specify a modified BNF from RFC7230 that removes VCHAR as well as comments and quoted-strings? Something like:
field-name = token

token = 1*tchar

tchar = "!" / "#" / "$" / "%" / "&" / "'" / "*" / "+" / "-" / "." / "^" / "_" / "`" / "|" / "~" / DIGIT / ALPHA

If we did that, then we could use a REGEX to enforce valid characters in the header line (back-slash octet would not be allowed).

@sappenin Your suggestion makes good sense.  I will update the code and repush.

@sappenin Looking into this, it appears that we would need to expand the list of acceptable characters.  There are several unit tests elsewhere in ilp-core that do not pass with the new implementation.  A more complete list that allows all unit tests to pass include: ";" / "," / ":" / " " / "?" .  We could eliminate the question mark "?", as the tests failing due to this one do not appear to be real headers (e.g., What is the answer?).  Thoughts ?
Examples taken from Unit tests that do not pass are:
[ERROR] test(org.interledger.ipr.InterledgerPaymentRequestEndToEndTest)  Time elapsed: 5.135 s  <<< ERROR!
org.interledger.codecs.CodecException: The value of the PskMessage header 'Expires-At' (2017-11-16T07:58:20.582-08:00) contains an illegal character (The list does not contain ':')
[ERROR] test(org.interledger.ipr.InterledgerPaymentRequestEndToEndTest)  Time elapsed: 5.135 s  <<< ERROR!
org.interledger.codecs.CodecException: The value of the PskMessage header 'Encryption' (aes-256-gcm -BnF608uR7y3xY730dp9rQ) contains an illegal character This is not allowed.
at org.interledger.ipr.InterledgerPaymentRequestEndToEndTest.test(InterledgerPaymentRequestEndToEndTest.java:104) (The list does not contain spaces)
[ERROR] testWriteRead0  Time elapsed: 0.012 s  <<< ERROR!
org.interledger.codecs.CodecException: The value of the PskMessage header 'question' (What is the answer?) contains an illegal character This is not allowed.
at org.interledger.codecs.oer.psk.InterledgerPaymentWithPskCodecTests.testWriteRead(InterledgerPaymentWithPskCodecTests.java:131) (The list does not contain '?')

My read of the definition of tchar in the BNF of RFC7230, is that ? is a delimiter, and thus not allowed in the header value. If that's correct, I think we should exclude it.
However, this post is pretty informative when thinking about http header parsing, and contains some discussion about headers that contain multiple values. I wonder if we should consider this in our parser? @controlsurface Are there unit tests around this? @adrianhopebailie Do we even want to allow for this in PSK?
FWIW, there's a table at the bottom of that post that visually shows which characters are valid tchars.
@adrianhopebailie What do you think about all this? We're sort of arbitrarily inventing a standard here as a subset of RFC7230, so we should probably think about this more.
Last but not least, @controlsurface be sure to read section 3.2.3 in RFC7230 about whitespace. For example, a header line can include whitespace, like this:
header-field   = field-name ":" OWS field-value OWS

Perhaps we should use an existing library, like BasicHeaderValueParser?

PSK has been superseded by STREAM


datasets

How to generate the file "data/iu_xray/images300_array/"?

Thank you very much. However, I can't find 'MIMIC_best_weight.pth' in generate_adm.py and how can i get it?

you can download the weight from hugging face

Could you please tell me which version of this mimic_cxr is? Does this version contain the images300 file directly?

Thank you very much. However, I can't find 'MIMIC_best_weight.pth' in generate_adm.py and how can i get it?

You can download from https://huggingface.co/ShenshenBu/EKAGen, or train using the project at https://github.com/hnjzbss/medical-classification.

Could you please tell me which version of this mimic_cxr is? Does this version contain the images300 file directly?

Images300 refers to downsampling the original images to 300×300 in order to reduce preprocessing time complexity.

Could you please tell me which version of this mimic_cxr is? Does this version contain the images300 file directly?

Images300 refers to downsampling the original images to 300×300 in order to reduce preprocessing time complexity.

Thank you for the clarification regarding the Images300 downsampling. Could you please provide more details about  the updated README .

Thanks for your reply. I want to use the iu_best_weight.pth to generate the IU_xray images200 files. Could you provide it? Thanks!

Could you please tell me which version of this mimic_cxr is? Does this version contain the images300 file directly?

Images300 refers to downsampling the original images to 300×300 in order to reduce preprocessing time complexity.

Thank you for the clarification regarding the Images300 downsampling. Could you please provide more details about the updated README .

Sure! Thank you for your interest in EKAGen. We will be updating the detailed README soon.


Dependency update 2022-12-07T110226

#1662
#1661
#1659
#1660
#1658
#1657
#1656
#1655
#1654
#1653
#1652
#1651
#1650
#1649
#1648
#1647
#1646
#1645
#1644
#1643
#1642
#1641
#1638
#1639
#1640
#1636
#1637
#1635
#1633
#1634
#1632
#1631
#1630
#1628
#1629
#1627
#1625
#1626
#1622
#1623
#1624
#1621
#1620
#1619
#1618
#1617
#1616
#1615
#1614
#1613
#1612
#1611
#1610
#1609
#1608
#1607
#1606
#1605
#1603
#1604
#1602
#1601
#1599
#1600
#1597
#1598
#1595
#1596
#1594
#1593
#1592
#1591

Codecov Report
Base: 81.86% // Head: 81.48% // Decreases project coverage by -0.38% :warning:

Coverage data is based on head (b0fe80e) compared to base (0c5a004).
 Consider uploading reports for the commit ac73924 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1663      +/-   ##
==========================================
- Coverage   81.86%   81.48%   -0.39%
==========================================
  Files          63       63
  Lines        2625     2625
==========================================
- Hits         2149     2139      -10
- Misses        428      436       +8
- Partials       48       50       +2

Flag
Coverage Δ

Linux
81.48% <ø> (ø)

Windows
?

macOS
?

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

Impacted Files
Coverage Δ

...n/github.com/lib/pq/splunkpq/internal/connector.go
51.75% <0.00%> (-5.03%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix Audio Links in Sessions.json

After the change of V2 API, this will get affected we have to repeat the same procedure we have done earlier to change audio link with http URL's

Solved by #390

Done !!!


add protocol in chinese

Translate protocol spec into Chinese.

😀, it seems no maintainer know Chinese

@GrapeBaBa @yeasy can you please review and provide feedback?

Yes, I will review and provide comments.
Thanks!

@yeasy thanks!

@mqshen
After a look, I found this version is good for a standpoint.
However, there're still some typos and mistakes, so I will write u an email for the update details.
Thanks!


add more discord-specific markdown stuff

fixes #272 and #289
please force soru to fork marked and make discord-marked a new npm package XD

Closing in favour of #310


Updating gridstack dependencies via bower breaks all

Apparently the version change from lodash 3.5 to 4.0 introduces some backwards incompatible changes that makes gridstack stop working
https://github.com/lodash/lodash/wiki/Changelog
I've tested this with demo/knockout.html against lodash 4.0 and 3.10.1.
It breaks in 4:

And works on 3.10.1

:+1: ! spend 3 hours debugging this
lodash < 4.0 works fine

@jippi, me too. I've spent some hours trying to figure out what was breaking Gridstack. Thought that it was jQueryUI, but it was lodash.
Please, change this issue's name to something like: "overlapping widgets..."

#309 has the gridstack changes that will make lodash 4.0 work


[WIP] add HIDL server support

This CL implements the HIDL server on the Android platform. The HIDL
server provides IThread and IThreadSettings HIDL interfaces for the
HIDL client.
The IThread interface is implemented by the HIDL server to provides
all control operations of the Thread network.
The IThreadSettings interface is mainly called by the HIDL server to
send sensitive settings to Thread System service. The Thread System
servie stores the sensitive settings to a secure area.

Most HIDL interfaces cannot be tested by CI. Should we have untested components in GitHub?
The HIDL interfaces are used in Android. When the code is synchronized to Android, they will be tested.


[Bug]: gpt-4o editing accuracy is low

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://docs.all-hands.dev/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
whenever running any task that requires editing an existing file I got infinite loops with the agent trying to edit an existing file.
trying deploying multiple ways (docker wsl and ubuntu wsl directly), read thru the docs, also tried differeent llms.
Current OpenHands version
0.9.0 and 0.9.1

Installation and Configuration
copy paste from readme from docker, make build, make setup-config and make run for the local repo tentative

Model and Agent
default, didnt mess with any advanced settings.
Operating System
ubuntu on wsl
Reproduction Steps
No response
Logs, Errors, Screenshots, and Additional Context
No response

Just to be sure, if you didn't change the model, does that mean you are using GPT4?

i tried both gpt-4o and sonnet 3.5, sonnet wouldnt even run. gpt-4o worked, but displayed the issue.

Hey @namastex888 , we're working on editing capability and have noticed that recently gpt-4o is not working well. I changed the name of the issue so we can track this more precisely.
Sonnet-3.5 definitely runs (that's what I use all the time), but there might be an issue with the configuration. If you want to open another issue for that we can help.

Thanks for the prompt support, amazing work y'all doing. Will give it a try again shortly. 🙏


Why are transactions so expensive how to change that?

Transactions seems to be 10 times more expensive than interacting directly with raydium pool, why is that and can it be changed? I see that the library interacts with jupiter aggegator

Hi, this wrapper has been built for a bot that required very fast transaction with very few possibilities in having the trans. rejected. That's why the base fee for transactions is higher. (still very low, tho.)
If you wish to lower it, you can play with the getSwapTransaction method, in particular with prioritizationFeeLamports which is now set to auto.
If you find a good balance of not-rejected transactions and lower fees, i'll be happy to review your PR and merge your changes 😁


Support for different data types (float16, float32)

Problem:

Only float32 is currently supported which requires 2x the memory and it slower to execute as inference is memory bound.

Solution:

Add ability to export model to float16.
Add support to inference with float16.

Note:

It's quite hard to do in generically in pure C (without templates) so to avoid adding too much complexity compilation option has been chosen. Ideally that would be a run-time pick based on the value stored in the config but that requires additional complexity which I wanted to avoid but that can be still explored with a proper solution.

@krzysztof-jusiak Hey there, what about approach would be used for BF16?
thanks!

For clang, -DDTYPE=_Float16 will work.

@krzysztof-jusiak: Hi, do you think it makes sense to just move from float32 to float16? It seems float16 is always faster than float32.
Using a single data type would be simpler both in code and in numbers of modelxxx.bin (already 3 + llama 7B and counting...)

@xefoci7612 I think there is a need for different data type as they provide different accuracy and differ in sizes, etc. Ideally we would operate on max precision floating point only but that's not possible due to performance/size limitations. However, smaller models or better hardware allow to use higher precision versions. Since inference is memory bound for faster speeds usually quantization is also applied, which compress the model even more to for example 4 bits but that comes with accuracy trades off, which I don't think can be made by default for everyone.

@xefoci7612
No, float16 is not always faster than float32. If the processor does not natively support FP16 arithmetic, then it will be emulated in software
https://stackoverflow.com/questions/56697332/float16-is-much-slower-than-float32-and-float64-in-numpy

will def take a look here. one thing to be careful with is that if you want to inference in fp16 you must train in fp16 (with gradient scalers), and not in fp32 or bf16. Otherwise the range of activations can overflow.

@krzysztof-jusiak what are the benefits of fp16?

clearly checkpoints get half as small
how much faster is fp16? e.g. on your computer, or on a MacBook Air M1 or so

also my understanding is this would invalidate all our previous checkpoints because they don't contain dtype in the config. Which is on me for having chosen a dumb serialization :D

Thanks for the hint about the training, it's a really valid point.
Regarding the benefits of fp16, it's mainly performance as the weights size is smaller and since the computations are memory bound the speedup should be noticeable.
Saying that, I've not noticed a huge difference between fp16 and fp32 on my machine
fp16: 2.3tok/s
fp32: 2.0tok/s
which doesn't add up with my previous experiments with other llms, I'd expect a bit bigger difference.
I'm verifying whether the code is at memory bound stage or not yet. Maybe it's still cpu bound which would explain why the speedup isn't bigger.

also my understanding is this would invalidate all our previous checkpoints because they don't contain dtype in the config.
Backwards compatibility

will def take a look here. one thing to be careful with is that if you want to inference in fp16 you must train in fp16 (with gradient scalers), and not in fp32 or bf16. Otherwise the range of activations can overflow.

Ba

also my understanding is this would invalidate all our previous checkpoints because they don't contain dtype in the config

Backwards compatibility can be maintained by assuming that a missing dtype implies fp32.

Ok I took a look but I think the required changes are a little bit too yucky, and we're not seeing strong evidence of much better results. I do like that the files would have been smaller by ~half... Anyway I'm leaning to not include this atm but ty. If someone can demonstrate a solid higher throughput I think it's worth revisiting.


remove reference to unused s3 bucket in test file

Coverage increased (+0.2%) to 67.504% when pulling 659631b85ff2c6de2ae073b269a7aa976d95712e on remove_s3 into 2a2575a5511e3f7d21f67e48d2b7711081649942 on master.


Module Problem

i'm from Brazil and i'm trying dev an App with javaFX and TornadoFX.
After import Kmongo, i have this problem and other 99 from de same type.

Error:java: the unnamed module reads package org.litote.kmongo from both kmongo.id and kmongo.jackson.mapping

Someone help me kkkkkk
And sorry by my English!!!

It seems that you try to use kmongo with java modules enabled. This is not supported for now by kmongo. You need to set all your dependencies in the "unnamed module" - ie the classpath.
HTH

Thanks...


Navbar content is not displaying properly

Description
The navbar content is piling over one another which is hampering the UI . I would like to fix it and make some further enhancements to improve the UI of the website in terms of navbar .
Screenshots

Please add GSSoC tag and level to it.

Hello Sir,
I would like to work on this issue, please assign it to me.

Hi @Kunal0220 @sonalee21, Thank you for raising the issue. But the navbar looks good to me.

I believe it is a problem with the hosting service. I will look into it and if any changes are required, I would assign them to you. Thanks

@Kunal0220 go ahead

Aye aye captain!

@Kunal0220 i just notice that the navbar is correct in local system but its not in hosted version because some css is not rendering properly so if u have any idea about it go on it

Ok @arpit456jain , let me exercise with the code for sometime and then I'll revert back .

Hi @arpit456jain , My exams got over yesterday and I my trying to go about it . I figured out the problem by inspecting it but m not being able to make it run on my local system . Some issues with the dependencies 🥲.
Should I provide the solution so that someone can fix it ?

sure

Hi @arpit456jain , My exams got over yesterday and I my trying to go about it . I figured out the problem by inspecting it but m not being able to make it run on my local system . Some issues with the dependencies 🥲.
Should I provide the solution so that someone can fix it ?

make sure to make a virtual env

I tried all the possible techniques for my end , But i'll give one more try , if it works then fine , or else I will type down the solution here itself.

Thanx for the time @arpit456jain , really appreciated . I am unassigning myself so that someone can take this over and make it right.

I'm GSSoC22 Contributor, Please assign this issue to me.

ok @DhairyaWankhade
but let me clear a thing navbar looks perfect in local system its giving problem with hosted one

@DhairyaWankhade updates ?

Is this issue resolved..??


Using correct api to schedule a setTimeout in the runtime.

Part of the debugger work on fiber management.
Also added debug preview for images (unrelated but harmless).

You might need to bump the pxt reference in this repo.


Add OpenSSF Scorecard Score as on Helm project

I do understand that helm-unittest is not part of CNCF yet, but is a great project, and there is a benefit to add this project to OpenSSF framework.
Example scorecard report for helm https://scorecard.dev/viewer/?uri=github.com/helm/helm
How to do it https://securityscorecards.dev/
I'm not sure where or not non-maintainers could add this. Is there is an appetite to add this? Happy to contribute .
One of the example improvements, is the https://github.com/slsa-framework/slsa-github-generator
Developers can build their software using a secure process that protects against many supply chain attacks and tampering. Users of their software can verify a tamper-proof statement of the process to know how the software was created.


no https specifiers that are not using deno.land/x

For example, here is a code snippet that uses x.nest.land
https://docs.deno.com/runtime/manual/basics/modules/#remote-import
import {
  add,
  multiply,
} from<EMAIL_ADDRESS>
function totalCost(outbound: number, inbound: number, tax: number): number {
  return multiply(add(outbound, inbound), tax);
}

console.log(totalCost(19, 31, 1.2));
console.log(totalCost(45, 27, 1.15));

switch all https imports to https://deno.land/x imports so we can ensure they continue to work
(also that code sample has an annoying amount of comments in it - those should be remove too)

there are lots of imports that are currently on esm.sh which as far as I can see are modules that we don't have on deno.land/x
are these ok to leave as are?

https://github.com/denoland/deno-docs/pull/529

closed in #529


Jan 13 release says 2020 when it should say 2021

Minor typo in the releases page, a little confusing.

Thanks for reporting this! Fixed!


chore: update coverage for 2024-12-01-19-27

updates coverage metadata for 2024-12-01-19-27

@heyste, not much of a change here

/lgtm


Fixed setting cache for HTMLPurifier_Lexer to runtime folder instead of vendor by default

Overall a good change.

Thanks!


[2.0.x] Builds are not reproducible without Microsoft.AspNetCore.Internal.CoreFxLab.Sources

The 2.0.x branch of Kestrel has a package dependency on Microsoft.AspNetCore.Internal.CoreFxLab.Sources, however, there is no open-source way to figure out exactly how to generate this package.
Recommendations:
Either

Add corefxlab as a git submodule
Add scripts to kestrel to generate the package
Add scripts to aspnet/Universe to generate this package

cc @pakrym @mikeharder

Do we have the version used to build 2.0 kestrel? We can just drop it into kestrel repo as source project.

According to the TeamCity logs from the 2.0.0 rtm build, we used this package: https://dotnet.myget.org/feed/aspnetcore-release/package/nuget/Microsoft.AspNetCore.Internal.CoreFxLab.Sources/2.0.0-rtm-21470

Fixed in #2118


[BACKLOG-38435] Code changes for moving Mail Validator from core to plugin

BACKLOG-38435 Code changes for moving Mail Validator from core to plugin

:x: Build failed in 11m 33s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
assemblies/plugins,engine,plugins,plugins/mail-validator,plugins/mail-validator/assemblies,plugins/mail-validator/assemblies/plugin,plugins/mail-validator/impl,plugins/mail-validator/ui,ui

:ok_hand: All tests passed!
Tests run: 2309, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message

:x: Build failed in 3m 17s
Build command:
mvn clean verify -B -e -Daudit -amd

:ok_hand: All tests passed!
Tests run: 12, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 1h 10m 15s
Build command:
mvn clean verify -B -e -Daudit -amd

:ok_hand: All tests passed!
Tests run: 5327, Failures: 0, Skipped: 14    Test Results

:information_source:  This is an automatic message


Update/add more tests

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.
You can find it at CHANGELOG.md.

  Generated by :no_entry_sign: Danger

Incorporated in https://github.com/iov-one/iovns/pull/122.


Clean up clippy suggestions

Hello!
This patch series cleans up a number of clippy suggestions. I broke them up into a few commits to make it easier to review, and to more easily edit if you decide to reject some of the suggestions.

I believe the appveyor error is fixed in https://github.com/heartsucker/rust-tuf/pull/133. I'll rebase on top of it once it lands.

As noted in #133, this appveyor error is likely related to #131, so I'm merging anyway. I'll see if I can debug that one later.


Faster dev cycle for integration tests

Basically, we generate an app once and reuse it, instead of repeatedly creating new apps, waiting for compile, etc.

Sorry for all of the PRs! There's a few things I still want to fix too before a release.

This looks great, thanks!


Logistic Regression - Python Implementation

Implement Logistic Regression from scratch in

Python

Create new issue according to the template if you would like to implement a different algorithm (or) in a different language.

I want to work on it. Please assign it to me


Introduces EnSearch and machinery for adding quickfix items

This performs an ENSIME  PublicSymbolSearchReq and places the top 25 results into the quickfix window, creating a new list. I figured 25 was a pretty reasonable number, although the ENSME search result ordering leaves something to be desired, so its possible you won't have the symbol you're looking for.
The quickfix machinery itself should be useful for other navigation/ find usage commands once we're ready to add those as well.
To run this from NeoVim, make sure to run :UpdateRemotePlugins, otherwise :EnSearch will not appear as a command.
Addresses #174

Thanks ;)

Could we add a note to doc/ensime-vim.txt about this feature?

And thanks @ChrisCoffey for all the work you're contributing!


Using Proxy Functionality with HTML Implementation

I built a JavaScript module that I want to run with an HTML button locally. I used browserify to allow the HTML file to use modules like require, but I am receiving this error:
Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'null' is therefore not allowed access. The response had HTTP status code 400. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
I can not find a concrete answer online on how to approach this issue, but I was wondering if the proxy functionality of this module can help me out. From what I have read, this issue has been solved using the XMLhttprequest object and CORS, but if possible I would like to avoid that. If the proxy functionality is the solution, what should I put as my proxy?
If there is anything else I can inform you of, please let me know as I am very new to JavaScript/HTML. Thank you.

@nrickles3 I do not think using a proxy in this situation is the right move.  If I am understanding your problem here, I would affirm that CORS would be the right way to go here.


1.12-rc1 cherry-pick request: Add mode_override to the TPU embedding enqueue ops.

This allows the mode to be overridden at runtime allowing dynamic switching between inference and training modes. Not fully implemented yet.
PiperOrigin-RevId: 215325071

(Added @george-kurian just as an FYI)


Improve size of the touch targets for the mobile nav menu to be easier to tap.

The current touch targets are a bit small, making it hard to select the right page, we should make these a bit bigger by making text or divs bigger

fixed


1.1 Defender SmartScreen Clearance

Currently in the process of getting clearance from MS Defender for 1.1 Installation.

[x] 1.1 Admin Installer
[ ] 1.1 ETT (Standalone Portable)
[ ] 1.1 ETT-Admin (Standalone Portable)

Admin Installer has been submitted for review

1.1 Has officially passed, and will be in the latest definition update

1.1 Admin installer submitted to Windows Package Manager Repo

1.1 Admin now available in winget!
winget install --id=EliWeitzman.ETT-Admin

Now, beginning process on clearing our 1.1 standalone and portable instances

Submissions for portable variants are in with Microsoft. Awaiting analyst approval

Approved! All variants of ETT 1.1 will no longer be blocked by Defender SmartScreen!


added spaces to breaks to avoid merge conflicts

lol the fear of conflict
but you can go ahead and remove the spaces, I think all you need to do is update your code, append your name properly and push again.

Lol.. Thanks. But there are other pull requests that will still clash.

lol the fear of conflict
but you can go ahead and remove the spaces, I think all you need to do is update your code, append your name properly and push again.

Lol.. Thanks. But there are other pull requests that will still clash.

yea I get but the spaces won't look good on our base code
conflicts are meant to be resolved right? lol

yea I get but the spaces won't look good on our base code
conflicts are meant to be resolved right? lol

First few codes had spaces.. I think the author will be able to remove spaces after merging with no conflicts?

give me a sec

give me a sec

Okay

I tried resolving from my end but it just keeps loading, Network issue I suppose

Welll
On Mon, Aug 8, 2022 at 8:49 PM Success Chikezie @.***>
wrote:

I tried resolving from my end but it just keeps loading, Network issue I
suppose
—
Reply to this email directly, view it on GitHub
https://github.com/Oluwasetemi/altschool-opensource-names/pull/158#issuecomment-1208541653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7CE5OMEBQGUYGVGVQWCWLVYFQGNANCNFSM556DQAXQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe it would be seamless from your end, just try chief

Are you free for a call?
On Mon, Aug 8, 2022 at 10:03 PM Success Chikezie @.***>
wrote:

I believe it would be seamless from your end, just try chief
—
Reply to this email directly, view it on GitHub
https://github.com/Oluwasetemi/altschool-opensource-names/pull/158#issuecomment-1208607143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7CE5MH6CTLZX6DBMOKV53VYFYZPANCNFSM556DQAXQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

yea sure

https://meet.google.com/qbi-xiqu-zzw
On Mon, Aug 8, 2022 at 10:15 PM Success Chikezie @.***>
wrote:

yea sure
—
Reply to this email directly, view it on GitHub
https://github.com/Oluwasetemi/altschool-opensource-names/pull/158#issuecomment-1208617758,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7CE5I67P4IBP7MWOIEEHTVYF2GZANCNFSM556DQAXQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Modification of readme.md

Describe the bug
the open source programs this repo has been a part of has been mentioned twice in the readme.md file which isn't required.
so i'd like to remove the first mention and keep the one with the posters of the programs
Expected behaviour
the open source programs should only be mentioned once.
Screenshots

Additional context
No response
Code of Conduct

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

@barkhaaroraa there are three separate open source program mentioned as well as in the posters already.

my issue was about the first mention. so in the entire readme.md file the open source programs are mentions twice at two totally different locations. so the idea that i was proposing was that you should remove the first mention( the one with the hearts ) as all of the open source programs are already mentioned at the end with their posters

I want work for this issue .Can you please assign me ? I am particiapnt of GSSOC'23.


Setup a sandbox production environment using the new deployment architecture

Setup a sandbox environment to experiment and test the new deployment infrastructure

Setup at http://<IP_ADDRESS>


[build-windows-toolchain] Fix SKIP_TESTS

Fix SKIP_TESTS functionality in utils/build-windows-toolchain.bat script.

Resolves SR-NNNN.

@compnerd May you take some time to review this fix?

@swift-ci please test


ODS - Component - Switches

Requirements
Variants

selected / unselected
icon / no icon

Related Tasks

[x] #99
[x] #100

I've logge the following issue: #192


Add short dictionary into optimal compression

#474
optimal_compress in elements.py now considers the small dictionary when compressing strings

Thanks :)


459: enable CI

prepares an ubuntu base image with required dependencies
builds the daphne system
runs test suite
stores created artifacts

please comment on whether we should enable only for main branch - would need a small change in the lines 5&6:
  push:
    branches: [ "main" ]

@corepointer : any blocking points?

Sorry for the late reply. I'll come back to this after the deliverables are finalized.

Thx @auge for starting work on the github actions. We'll need caching (of the thirdparty dependency build step) or run this from a prebuilt container imho as one run takes 100+ minutes which will quickly deplete our quota. Same goes for the generated artifacts.
I merged this now to see how it will work out on the main branch. In case the issues become real we can disable the workflow again.


next  PR

next  PR
Originally posted by @DharmendraGITHB in https://github.com/tensorflow/tensorflow/issues/38668#issuecomment-1192153328

Hi @DharmendraGITHB,
To expedite the trouble-shooting process, could you please provide a minimal code snippet and the TensorFlow version you are using. Thank you!


Default for eslint

Is your feature request related to a problem? Please describe.
Like how you have a default for typescript, I think a default value for eslint makes sense too.
Describe the solution you'd like
{
  eslint: true
}

should be equivalent to:
{
  eslint: {
    lintCommand: 'eslint .'
  }
}

Describe alternatives you've considered
No response
Additional context
I can send a PR if interested.
Validations

[X] Read the docs.
[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that asks for the same feature to avoid creating a duplicate.

Hmm, that's an interesting proposal. Personally, I prefer using explicit configurations over implicit ones. For example, when examining the boilerplate templates generated by vite create, I suggest changing the command to eslint ./src if someone wants to lint only the configuration files. For linting other configuration files as well, the command eslint . should be used. Moreover, when creating a TypeScript project, it's advisable to modify the command to eslint . --ext .js,.ts, especially when working with a Vue project we need to include .vue. Taking all these conditions into consideration, I believe it's best to let the user decide, even though it might require a little extra time (probably less than 30 seconds 🤣). However, this approach will help prevent potential future issues that are uncertain.


first commit

Hi Guys,
please review my code, here i pushed two projects

finahub-java
Rest endpoints.

AirTableClient
Android Project.

Also Added README please check.

thank you


Debug not available

I understand debugging is a feature of the paid version. However, I selected the Free trial option, and I still don't see the debug option come up.
I don't see an option to buy the paid version

Thanks in advance for your help.

Hi Ian,

I understand debugging is a feature of the paid version. However, I selected the Free trial option, and I still don't see the debug option come up.

Could you please attach a screenshot/video showing the problem?
Does Licenses window show k6 plugin as active at your setup?

Also, for performance reasons, the plugin checks the active status no more than once every 5 minutes. So after activation of your k6 license, you should either wait for 5 minutes or restart the IDE. I will try to rework this behavior in the next plugin version.

I don't see an option to buy the paid version

You need to log in to your JetBrains account and buy a license there.
Please refer to the official guide here: https://www.jetbrains.com/help/idea/register.html#activate-license

Thanks for your quick response!
Here is a screenshot of my licenses window. As you can see, K6 shows as activated:

Here is a video of what happens when I press the "play" button. The expected behavior is I would get an option to run as normal or run in debug mode, but as you can see, I don't get that option, and it just runs K6 as normal.

Please disregard my comment about being unable to purchase. It was due to a mistake on my end.
Thanks again!

First, the same behavior is actual for the older version of plugin (before 1.1.0). Could you check the version? You can find it in the Settings -> Plugins :

Also, how does the 'Debug' icon look like?

Good call. It looks like I have version 1.0.3.

Here's my debug button. As you can see, it's disabled.

yep. Disabled debug button is expected for 1.0.3 version. Please update the the latest one.

I had to upgrade my version of Jetbrains to support the new plugin version, but it now works as expected. Thanks so much!


Javadoc and basic Readme.md

Coverage remained the same at 100.0% when pulling 497b5abfbd900898fd7cc9803b3b022bc93ac838 on 52_documentation into e590812c714930c609a1ce45653c5a8065a0910f on master.


.Net Core Web App Publish in Ubuntu

I got an error in printing pdf if my app is publish. If not, web app successfully printed pdf.
I put this on my Startup.cs:
services.AddJsReport(new LocalReporting().UseBinary(jsreport.Binary.Linux.JsReportBinary.GetBinary()).AsUtility().Create());

Only pdf is not being print? The html recipe works?
Can you try to run this on the target machine?
apt-get install -y libfontconfig1

I'm not using html recipe, only phantom pdf recipe. It's okay in development. but not worked in production.
I already installed that Sir, but no luck.
Thank you

Please try also the html recipe so we can limit the area where to search for the problem.
Can you dump the exception which you see and paste it here? It contains jsreport output log.

I've also tried the html recipe, not working. I can't find the jsreport output log.
This is my publish file:
The jsreport folder has no file/output log inside.

Ok. Thanks for trying html recipe.
At what point it crashes for you? What is the exception? Please share the part of the code where it crashes.

I don't know where it crashes coz i encounter the error in production, not in development. It works in development.

I encounter this now

Did you try to add permissions to the /tmp folder for the running process?

This is the permission for autocleanup:

This is the permission for tmp:

Hello @pofider ,
Phantom pdf recipe now works in production. I just run this command:
sudo apt-get dist-upgrade
But the size of the font is different in my windows (development platform) to ubuntu server (production platform)
Windows:

Ubuntu Server:

hi @fsfj, the size differences between Linux and Windows is expected, take a look at this for some workaround.

Closing as the main issue is solved here.


OKD support for OpenStack provider

We would like to keep this master issue to link new issues which are related to OpenStack deployment so we can keep track as well as letting community know if a specific provider deployment is supported or has known issues etc

Openstack images in origin should use RDO instead of OSP so that these could be redistributed

is general openstack support planned?
the docs only mention the redhat openstack distribution, not plain openstack.

the docs only mention the redhat openstack distribution, not plain openstack.

Any openstack distro should work, lets file a new ticket to reflect that in the docs

is general openstack support planned?
the docs only mention the redhat openstack distribution, not plain openstack.

The only proviso here is that versions prior to 4.4 used Swift, after 4.4 it's using Glance. So you would just need to make sure that your OpenStack environment meets the requirements from the Docs. Such as Swift is available ( if < 4.4 ), Neutron trunk ports, DNS, etc. Other than that, Red Hat OpenStack doesn't have anything special that allows it to work. You should be fine with any deployment of OpenStack.
Have you run into any specific issues with it, or are you just asking in general?

Available in 4.5, lets open separate issues if problems occur


Add Sky shader component

Adds sky shader from this example. source
Codesandbox

nice, thanks! its in


[BrokenLinksH2] Fixed broken links

Fixes #
Docs for Toolkit PR #
What changes to the docs does this PR provide?

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Correctly picked the right branch to base the change off (dev for new features, main for typos/improvements)
[ ] For new pages, used the provided template
[ ] For new features, added an entry in the Table of Contents
[ ] Ran against a spell and grammar checker
[ ] Contains NO breaking changes

Other information

Docs Build status updates of commit 7aab5c7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/helpers/AttachedShadows.md
:warning:Warning
View
Details

docs/high-performance/Memory2D.md
:bulb:Suggestion
View
Details

docs/parsers/Win2D_Path_Mini_Language.md
:bulb:Suggestion
View
Details

docs/helpers/DispatcherHelper.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelper.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelperRequest.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelperResponse.md
:white_check_mark:Succeeded
View

docs/helpers/RemoteDeviceHelper.md
:white_check_mark:Succeeded
View

docs/parsers/MarkdownParser.md
:white_check_mark:Succeeded
View

docs/platform-specific/PlatformSpecificAnalyzer.md
:white_check_mark:Succeeded
View

docs/platform-specific/PlatformSpecificDifferencesGenerator.md
:white_check_mark:Succeeded
View

docs/services/Facebook.md
:white_check_mark:Succeeded
View

docs/services/GraphLogin.md
:white_check_mark:Succeeded
View

docs/services/Linkedin.md
:white_check_mark:Succeeded
View

docs/services/MicrosoftGraph.md
:white_check_mark:Succeeded
View

docs/services/MicrosoftTranslator.md
:white_check_mark:Succeeded
View

docs/services/OneDrive.md
:white_check_mark:Succeeded
View

docs/services/Twitter.md
:white_check_mark:Succeeded
View

docs/services/Weibo.md
:white_check_mark:Succeeded
View

docs/helpers/AttachedShadows.md

Line 113, Column 136: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/dotnet/api/microsoft.toolkit.uwp.ui.animations.colordropshadowanimation' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 113, Column 136: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/dotnet/api/microsoft.toolkit.uwp.ui.animations.colordropshadowanimation' will be broken in isolated environments. Replace with a relative link.

docs/high-performance/Memory2D.md

Line 12, Column 481: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/communitytoolkit.highperformance.span2d-1?view=win-comm-toolkit-dotnet-7.0. Otherwise, remove the view parameter.

docs/parsers/Win2D_Path_Mini_Language.md

Line 113, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 130, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 150, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 165, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 180, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 195, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 210, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 228, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 243, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 259, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 287, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 298, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 309, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 320, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 381, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 77a078b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/high-performance/Memory2D.md
:bulb:Suggestion
View
Details

docs/parsers/Win2D_Path_Mini_Language.md
:bulb:Suggestion
View
Details

docs/helpers/AttachedShadows.md
:white_check_mark:Succeeded
View

docs/helpers/DispatcherHelper.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelper.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelperRequest.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelperResponse.md
:white_check_mark:Succeeded
View

docs/helpers/RemoteDeviceHelper.md
:white_check_mark:Succeeded
View

docs/parsers/MarkdownParser.md
:white_check_mark:Succeeded
View

docs/platform-specific/PlatformSpecificAnalyzer.md
:white_check_mark:Succeeded
View

docs/platform-specific/PlatformSpecificDifferencesGenerator.md
:white_check_mark:Succeeded
View

docs/services/Facebook.md
:white_check_mark:Succeeded
View

docs/services/GraphLogin.md
:white_check_mark:Succeeded
View

docs/services/Linkedin.md
:white_check_mark:Succeeded
View

docs/services/MicrosoftGraph.md
:white_check_mark:Succeeded
View

docs/services/MicrosoftTranslator.md
:white_check_mark:Succeeded
View

docs/services/OneDrive.md
:white_check_mark:Succeeded
View

docs/services/Twitter.md
:white_check_mark:Succeeded
View

docs/services/Weibo.md
:white_check_mark:Succeeded
View

docs/high-performance/Memory2D.md

Line 12, Column 481: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/communitytoolkit.highperformance.span2d-1?view=win-comm-toolkit-dotnet-7.0. Otherwise, remove the view parameter.

docs/parsers/Win2D_Path_Mini_Language.md

Line 113, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 130, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 150, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 165, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 180, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 195, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 210, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 228, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 243, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 259, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 287, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 298, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 309, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 320, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 381, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix broken links
@nmetulev
@michael-hawker
@Arlodotexe
The Content & Learning team is fixing broken links on docs.microsoft.com for the rest of H2. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes and does not change other content. I’d very much appreciate your timely review and feedback on the link changes I’ve suggested. Thanks!

@kturetz Please update your changes based on feedback in PR #646. Thanks.

Docs Build status updates of commit 769970b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/high-performance/Memory2D.md
:bulb:Suggestion
View
Details

docs/parsers/Win2D_Path_Mini_Language.md
:bulb:Suggestion
View
Details

docs/helpers/AttachedShadows.md
:white_check_mark:Succeeded
View

docs/helpers/DispatcherHelper.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelper.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelperRequest.md
:white_check_mark:Succeeded
View

docs/helpers/HttpHelperResponse.md
:white_check_mark:Succeeded
View

docs/helpers/RemoteDeviceHelper.md
:white_check_mark:Succeeded
View

docs/parsers/MarkdownParser.md
:white_check_mark:Succeeded
View

docs/platform-specific/PlatformSpecificAnalyzer.md
:white_check_mark:Succeeded
View

docs/platform-specific/PlatformSpecificDifferencesGenerator.md
:white_check_mark:Succeeded
View

docs/services/Facebook.md
:white_check_mark:Succeeded
View

docs/services/GraphLogin.md
:white_check_mark:Succeeded
View

docs/services/Linkedin.md
:white_check_mark:Succeeded
View

docs/services/MicrosoftGraph.md
:white_check_mark:Succeeded
View

docs/services/MicrosoftTranslator.md
:white_check_mark:Succeeded
View

docs/services/OneDrive.md
:white_check_mark:Succeeded
View

docs/services/Twitter.md
:white_check_mark:Succeeded
View

docs/services/Weibo.md
:white_check_mark:Succeeded
View

docs/high-performance/Memory2D.md

Line 12, Column 481: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /dotnet/api/communitytoolkit.highperformance.span2d-1?view=win-comm-toolkit-dotnet-7.0

docs/parsers/Win2D_Path_Mini_Language.md

Line 113, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 130, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 150, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 165, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 180, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 195, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 210, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 228, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 243, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 259, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 287, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 298, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 309, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 320, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 381, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@michael-hawker I think this is ready to merge. If not, we can close this PR and I'll have someone else from my team finish the work. Thanks!


Missing OS requirements for ubuntu-latest

What happened?
The CI of the template is currently broken: https://github.com/cookiecutter/cookiecutter-django/actions/runs/11318953105/job/31474266324?pr=5463#step:8:22
We're missing OS requirements for ubuntu-latest, most likely because the GitHub runner OS was updated
What should've happened instead?
CI should  pass.
Additional details

Host system configuration:

Version of cookiecutter CLI (get it with cookiecutter --version): latest

OS name and version: ubuntu-latest TBC what version is actually used:
On Linux, run
lsb_release -a 2> /dev/null || cat /etc/redhat-release 2> /dev/null || cat /etc/*-release 2> /dev/null || cat /etc/issue 2> /dev/null

# Insert here the OS name and version

Python version, run python3 -V: 3.12

The very first step of the job gives some details about the OS: https://github.com/cookiecutter/cookiecutter-django/actions/runs/11318953105/job/31474266324?pr=5463#step:1:8 which we can compare against the latest green run: https://github.com/cookiecutter/cookiecutter-django/actions/runs/11308996288/job/31452396112?pr=5461#step:1:8
It changed from ubuntu-22.04 to ubuntu-24.04

Does Dependabot supports updating the runs-on labels?  If so, we should pin it to ubuntu-22.04. This will trigger Dependabot to create a pull request (PR) that will fail due to the missing apt file.


Add support for AUX FREEDOM AUX-09FH

Complete set of codes. Doesn't include the fan directions though

Thanks for the PR. You need to edit 'auto' to 'heat_cool' in operationModes.

@vassilis-panos now it's heat_cool


Port 8080 in use

What version of Music Assistant has the issue?
2022.12.1
The problem
Integration does not start when Port 8080 is used by something else - it is not clear what that port is used for.
How to reproduce
Start someting that listens on Port 8080 and then Add Music Assistant integration
Relevant log output
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 372, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/mass/init.py", line 197, in async_setup_entry
raise exc
File "/config/custom_components/mass/init.py", line 190, in async_setup_entry
await mass.setup()
File "/usr/local/lib/python3.10/site-packages/music_assistant/mass.py", line 84, in setup
await self.streams.setup()
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/streams.py", line 120, in setup
await http_site.start()
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_runner.py", line 121, in start
self._server = await loop.create_server(
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 1505, in create_server
raise OSError(err.errno, 'error while attempting '
OSError: [Errno 98] error while attempting to bind on address ('<IP_ADDRESS>', 8080): address in use
Additional information
No response
What version of Home Assistant Core are your running
2022.12.5
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Linux

Did you ever manually alter something on the files of Music Assistant ?
This error simply can not happen. A port is auto selected in a range between 8095 and 8195 unless a port was hardcoded.

no I did not touch the files. I even tried deleting and reinstalling the integration. I'll check the files if I can find the one containing the port(s).

No, I did move my home assistant from docker to a hassos VM and there it works just fine, in my docker install it is still broken and I don't know why.

As this does not seem to be a MA issue then I will close this. Please feel free to report any other MA issues in the future.


Spring Cloud Dataflow samples with Pub/Sub

Is your feature request related to a problem? Please describe.
From user:

The standard apps for spring cloud data flow have implementations for apache kafka and RabbitMQ. Google PubSub is supported, but no premade apps is made with Google PubSub.

Describe the solution you'd like
A sample w/ Spring Cloud Dataflow w/ Pub/Sub
Describe alternatives you've considered
Additional context
User feedback from SpringOne

Binders for Google PubSub exists https://cloud.spring.io/spring-cloud-stream/#binder-implementations
But there are no premade versions of the standard apps https://cloud.spring.io/spring-cloud-stream-app-starters/ for google pubsub - Only kafka and rabbitmq versions exists
It would be help adoption, if it was possible add PubSub version by just adding something like https://dataflow.spring.io/kafka-docker-latest but for pubsub, perhaps https://dataflow.spring.io/pubsub-docker-latest

This is definitely a gap in our suite of samples. We should draw a bit more attention to the availability of the Binder.
Contributions are welcome!
cc/ @dmitry-s

As I understand it from https://youtu.be/AbeOdrNPfYg?t=3455 the https://github.com/spring-cloud-stream-app-starters/sftp/blob/master/spring-cloud-starter-stream-source-sftp/pom.xml just needs to add org.springframework.cloud/spring-cloud-gcp-pubsub-stream-binder and maybe
spring.cloud.stream.bindings.input.binder=pubsub
spring.cloud.stream.bindings.output.binder=pubsub

to the applications.properties
A different way could be to add a Google PubSub module in https://github.com/spring-cloud-stream-app-starters/tasklauncher-dataflow/tree/master/apps with POM like https://github.com/spring-cloud-stream-app-starters/tasklauncher-dataflow/blob/master/apps/task-launcher-dataflow-sink-kafka/pom.xml, but with the org.springframework.cloud/spring-cloud-gcp-pubsub-stream-binder instead of kafka.
I'm not sure where org.springframework.cloud.stream.app/sftp-app-starters-build is located?

You're right that the binders should be generally interchangeable. Yet, we still should test it and document it. I wasn't aware of the spring-cloud-stream-app-starters project. That might be a good place for Pub/Sub starter or maybe we would just keep it in spring-cloud-gcp. It's something to investigate further.

So you prefer to make a new project spring-cloud-gcp-stream-app-starters under the spring-cloud-gcp umbrella.
I guess, that you don't want to fork the upstream projects, as that will result in a pretty high maintains for each release?!?
So we need to some how make it depend on the upstream project and then add the org.springframework.cloud/spring-cloud-gcp-pubsub-stream-binder or how do you see it?

If we're only providing a simple example, we can copy whatever we need from the upstream project into spring-cloud-gcp and it shouldn't add too much maintenance burden. However, I would like to understand a bit more about the the whole https://github.com/spring-cloud-stream-app-starters org, and see if it makes more sense to have a Pub/Sub sample there.
cc/ @artembilan @sobychacko

No, it's not.
That project is obsolete and replaced with this: https://github.com/spring-cloud/stream-applications
See more info in Blog Post series : https://spring.io/blog/2020/07/13/introducing-java-functions-for-spring-cloud-stream-applications-part-0
So, what you are asking almost does not make sense since that project is not about "samples". It represents some function implementations and generated Spring Cloud Stream applications with Kafka and Rabbit binder bundle.
We probably can't provide Pub/Sub binder-based generation because we don't manage this GCP project, but you definitely can do something as a part of your project to provide generated artifacts with Pub/Sub binder based on those our out-of-the-box functions.
WDYT @sobychacko ?

It's not a sample, it's a standard component, the headline is misleading

I was not aware of the deprecation of https://github.com/spring-cloud-stream-app-starters

@artembilan do you see the build of of  https://github.com/spring-cloud/stream-applications with the org.springframework.cloud/spring-cloud-gcp-pubsub-stream-binder being done? should it be part of the https://github.com/spring-cloud/stream-applications, as that is intended to be a monorepo? or should it be done as part of https://github.com/spring-cloud/spring-cloud-gcp?
I sessions on Spring Cloud Dataflow @mminella have said, that the Spring team will not do a cartesian product of the steam application with all the binders in https://spring.io/projects/spring-cloud-stream, he said only premade RabbitMQ and Kafka will made

No, I don't see because Spring team just don't support that binder and its release lifecycle might not be aligned with what we release in that stream-applications project.
I think @mminella's announcement is the same what I'm telling you. So, if you would like to provide an out-of-the-box stream applications bundled with the Pub/Sub binder, you should do that in this project or a new separate one.
And that's why I'm pointing to @sobychacko . He is a lead of stream-applications project and he can advice how to proceed with the proper final artifact generation.

@sobychacko Any thoughts here?

@sobychacko how do you prefer, that the build of the out-of-the-box stream will be done, using the Google PubSub binder?

@nklausen I would suggest creating something under our spring-cloud-gcp-samples module in the new repo, and we can always move it somewhere else later if need be.


DEV: Drop BasicUserWithStatusSerializer

Merge after https://github.com/discourse/discourse/pull/18745.
This serializer is in Core now.

@AndrewPrigorshnev what do we do with this?

It's safe to merge, let me double-check, and I'll merge it today.


Use key vault along with cloud config

I would like to store all the credentials in the vault and access it through spring cloud config rather accessing it directly . Is this possible now ?

@sabareeshkkanan Thanks for filing issue. Could you give more detailed explanation about your scenario, e.g., your expected example spring cloud config properties configuration and how you want to access the property from your application.
For a similar scenario, there is an existing keyvault sample project, you can get keyvault property secrets as property in the application.

We are using spring cloud config service which is a standalone service which uses git as a backend, we are looking for other alternatives instead. Spring cloud config can use vault as backend please refer here https://cloud.spring.io/spring-cloud-static/spring-cloud-config/2.1.0.RELEASE/multi/multi__spring_cloud_config_server.html#vault-backend  but not sure it will work with Azure vault ?

Currently I working on a project that maybe fits for your scenario: https://github.com/srempfer/spring-cloud-config-azure-keyvault

I released the first version and it's available on maven central: https://search.maven.org/artifact/io.github.srempfer/spring-cloud-config-azure-keyvault-starter


Added Links for Videos and changed format of Readme for University Repo

I added links for youtube videos in the Read me
Changed the format of the Readme.md files by adding a link to each file mentioned there
Made a few changes to the format of the scripts within the Scripts/University Repo Folder

@tanmayeekamath,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Migrate update to Dialob latest v2.x

This is the version that Villu is working on to ensure we have only one version

This task and description makes no sense.


Pull access denied for microsoft/aspnetcore-build while following the exact steps in tutorial #82135

Originally posted by @Minetorpia in https://github.com/MicrosoftDocs/azure-docs/issues/82135
I followed every step in the tutorial, but sadly building the image fails. I tried Googling, but didn't find anything on how to fix this.
Step 1/20 : FROM microsoft/aspnetcore-build AS build-env
pull access denied for microsoft/aspnetcore-build, repository does not exist or may require 'docker login': denied: requested access to the resource is denied
##[error]pull access denied for microsoft/aspnetcore-build, repository does not exist or may require 'docker login': denied: requested access to the resource is denied
##[error]The process '/usr/bin/docker' failed with exit code 1
Finishing: Build and push an image to container registry

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

ID: 4cfc7aa5-d9ff-ad64-d26e-d2bd10470025
Version Independent ID: 9b4f02ea-dab9-f613-5345-496435ab2408
Content: Deploy containerized applications to Web App for Containers - Azure Pipelines
Content Source: docs/pipelines/apps/cd/deploy-docker-webapp.md
Product: devops
Technology: devops-cicd-apps
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

Any updates for this issue? I am also facing it

Today I tried to follow steps from here:
https://learn.microsoft.com/en-us/azure/devops/pipelines/apps/cd/deploy-docker-webapp?view=azure-devops&tabs=dotnet-core%2Cclassic
but ends up getting the same error as follows:
2023-04-12T08:27:27.4323436Z Step 1/20 : FROM microsoft/aspnetcore-build AS build-env
2023-04-12T08:27:28.3991274Z pull access denied for microsoft/aspnetcore-build, repository does not exist or may require 'docker login': denied: requested access to the resource is denied
2023-04-12T08:27:28.4101428Z ##[error]pull access denied for microsoft/aspnetcore-build, repository does not exist or may require 'docker login': denied: requested access to the resource is denied
2023-04-12T08:27:28.4141590Z ##[error]The process '/usr/bin/docker' failed with exit code 1
2023-04-12T08:27:28.4218355Z ##[section]Finishing: Build and push an image to container registry


What happens to data_type after unhandled exception is being run?

Does free get called on mrb_close for all GC'd objects?
I'm currently having an issue in mrusty where running into an unhandled exception leaks all the Rust objects that were lended to mruby because free doesn't seem to get called. There's a good chance that the bug might be in my code, but I haven't been able to figure it out so what I'm trying to ask is what is the best way to handle this use scenario?

All malloced regions should be deallocated by free. If not, we consider it as a bug.
Please submit an issue, preferably with bug reproducing info.

@matz I was able to track down the problem: it arises when mruby calls Rust code that calls back mruby which causes an unhandled exception.
mruby -> rust -> mruby (unhandled exception)

This breaks the first mruby context and interrupts the Rust code from finishing which should normally decrement a reference counter. Because the reference counter doesn't get decremented (it should get, even if mruby has an exception), this breaks the reference counter which will never call mrb_close which leads to the memory leak.
My current plan is to try and catch every single exception when running mruby from Rust instead of using mrb_ext_get_exc, but I'm not completely sure whether this is the best solution.

@matz Here's a minimal example:
#include <mruby.h>
#include <mruby/compile.h>

mrbc_context* ctx;

void c1(mrb_state* mrb) {
  printf("entering c1\n");

  mrb_load_nstring_cxt(mrb, "c2", 2, ctx);

  printf("exiting c1\n");
}

void c2(mrb_state* mrb, mrb_value self) {
  printf("entering c2\n");

  mrb_load_nstring_cxt(mrb, "raise 'hi'", 2, ctx);

  printf("exiting c2\n");
}

int main(void) {
  mrb_state* mrb = mrb_open();
  ctx = mrbc_context_new(mrb);
  ctx->capture_errors = TRUE;

  mrb_define_singleton_method(mrb, mrb->top_self, "c2", (mrb_func_t) c2, MRB_ARGS_NONE());

  c1(mrb);

  mrbc_context_free(mrb, ctx);
  mrb_close(mrb);

  return 0;
}

The output is:
entering c1
entering c2
exiting c1

The issue is that c2 gets interrupted. Is there a way to catch the exception?

@matz Any update on the issue? I still haven't figured out how to prevent the memory leak.

You have to put those c methods in a try/catch block and manually free the just allocated memory. Take a look at https://github.com/mruby/mruby/tree/master/mrbgems/mruby-error

@Asmod4n I'm trying to run mrb_load_nstring_cxt inside of a function that is run from mrb_protect. If I raise manually from C, state changes and it captures exceptions, but it fails to captures exceptions from mrb_load_nstring_cxt with state always being false.
mrb_value foo(mrb_state* mrb, mrb_value data) {
    mrb_load_nstring_ctx(..., "raise 'exc'");
}

...

mrb_protect(mrb, foo, ..., state); // state is ALWAYS false

I solved the issue by raising the caught exception within foo. Not the most elegant solution, but it works.


Proof of knowledge revealed message structure

Related to #2
Currently the implementations of a BBS proof of knowledge signature includes the encoding of a binary structure at the start of the proof which captures which of the messages being verified and how they correspond to the original ordering of the message set signed. Currently this structure takes the form of a bit array.
For example assume the following set of messages were originally signed
[
  "message1"
  "message2",
  "message3",
  "message4",
  "message5"
]

And a proof revealing only the 1st 4th and 5th message was generated, meaning the revealed messages supplied during the verify proof operation would appear in the following order
[
  "message1",
  "message4",
  "message5"
]

The resulting bit array in the proof would take the following form
+-----------------------------+-----------------------------+---------------------------------+
| Total messages length (16bit)| Revealed messages bit array |..... remaining proof structure |
+-----------------------------+-----------------------------+---------------------------------+

+-----------+--------------+-----------------------------------+
| 5 (integer) | 1 | 0 | 0 | 1 | 1  | ..... remaining proof structure |
+-----------+--------------+-----------------------------------+

However, this mechanism of expression does limit options around the data canonicalisation algorithm that is used on top of the cryptographic scheme. Namely it does not support a re-ordering of the revealed messages away from the order in which they were originally signed. To accomodate this we could instead elect to change the expression of this aspect of the proof from a bit array to a byte/multi-byte array, where instead the array contents is a set of un-signer integers (16bit each) where each integer corresponds to the index the message occupied in the original ordering of the signed message set
For example assume the following set of messages were originally signed
[
  "message1",
  "message2",
  "message3",
  "message4",
  "message5"
]

And a proof revealing only the 1st 4th and 5th message was generated, BUT the messages in the resulting proof are re-ordered so that they present as 4, 5, 1
[
  "message4",
  "message5",
  "message1"
]

The resulting bit array in the proof would take the following form
+-----------------------------+----------------------------------------+-------------------------------+----------------+
| Total messages length (16bit)| Revealed messages array length (16bit) | Revealed messages byte array | remaining proof |
+-----------------------------+----------------------------------------+-------------------------------+----------------+

+--+--+--------+------------------------------+
| 5  | 3 | 4 | 5 | 1 | ..... remaining proof structure |
+--+--+--------+------------------------------+

The tradeoff here is that the overhead of this structure is more than in the simpler case with the bit array.
NOTE It is probably also worth considering a more standard structured binary format like CBOR rather than a custom structure

Could someone (perhaps @tplooker or @brianorwhatever?) provide an example (the simplest possible) of a JSON-LD VC that cannot be verified using BBS+, annotating it with how the problem manifests? I'm trying to understand the scope of the issue, and especially, the scope in real world use cases.  Is it anytime that nesting is used, or is it a certain pattern of nesting?  How can a non-JSON-LD person understand the impact?
Thanks!

@swcurran I would take a look at @tplooker comment here https://github.com/mattrglobal/jsonld-signatures-bbs/issues/158#issuecomment-945238151 for the most succint description of the problem. My impression is that it is basically going to happen whenever there is nested objects (unsure how many will trigger it) within the credentialSubject that don't have an id attribute

It's helpful but not an example of the JSON-LD that triggers the issue. However, your comment in that same issue has an example.
As I understand @tplooker's explanation that you reference, the issue manifests in your credential by the existence of the recipient and vaccine items with no id below them.  Presumably, if you had just one item, or all (or all but one) had an id, life would be good. And presumably, anytime there is more than one such item anywhere in the JSON will trigger the problem.
That's going to be pretty rough for anyone that wants to use an existing structure as a VC.

@swcurran The problem is related to, but not directly caused, by the serialization algorithm (i.e., URDNA). The problem in the example is caused by the way the Holder communicated the indexes of the revealed messages to the Verifier, because it does not account for the fact that the messages received by the Verifier may not be in the same order that the Holder has them (because of URDNA).
As an example, let’s say that the Holder has messages m1, m2 and m3 and creates a proof disclosing only m2 and m3. What was proposed to let the Verifier know what messages are disclosed, is that the Holder will also send the bit array 011 indicating that they have 3 messages and that they disclosed the 2nd and the 3rd one. However, if the Verifier gets the messages as [m3, m2] (correct messages, wrong order) they will use m3 like it was the 2nd Holder’s message and m2 like it was the third. That miss-match will make the verification fail.
The solution however is not to put id's to every nested object in the VC (although that could also work). The simple solution is to communicate the indexes of the revealed messages in a way that will account for the case above. One way to do that is instead of the Holder sending the bit array 011 to send the integers list [3, 2] which solves that problem.

Thanks.  Agree that what you state is the "simple solution", but in reality that means agreeing on the exact simple solution, publishing it as a new signature scheme, updating libraries and then updating the existing applications that use those libraries so that arbitrary JSON-LD can be used. That would suggest a long delay in being able to use of BBS+ credentials for general use cases.  Thus, I agree that adding in IDs is not the correct solution, it is a practical hack that will work for the existing scheme.
I noticed, for example, that in publishing the BBS+ version of the mDL schema, that approach was used throughout.

do you have a link to the bbs+ mdl schema @swcurran? sounds interesting!

You can follow the chain to see the approach.  Start here: https://digitalbazaar.github.io/vdl-test-suite/ (Manu posted about this in the W3C CCG).  From the examples, the context for a "vDL" is https://w3id.org/vdl/v1, which resolves to a chunk of JSON that has just has an id for each attribute in the form of:
       ...
        "family_name": {
          "@id": "https://w3id.org/vdl#family_name"
        },
        "given_name": {
          "@id": "https://w3id.org/vdl#given_name"
        },
        "birth_date": {
          "@id": "https://w3id.org/vdl#birth_date"
        },
        ...

I don't know a lot about JSON-LD and perhaps those @id's per attribute are normal.  But I would image they don't contribute much to the semantic understanding (human or machine) of the attributes -- but they do provide an ID.

Cool, thanks for the link. I'm also not a JSON-LD expert but I believe an @id is more about what the node is semantically vs an identifier (id) for the node which is the issue in this instance. See this example https://www.w3.org/TR/json-ld/#example-4-context-for-the-sample-document-in-the-previous-section
There's also the "@type": "@id" line which is often seen in contexts and I don't totally understand..

What are the consequences if the signature specification didn't include any revealed message indexes or how they're encoded, and surrendered that responsibility to higher level application suites, like Linked Data and Jose processors. This neither adds nor removes any (more) leaks from the signature format, it's just makes it less opinionated about attaching information to the signature that isn't signed over anyway. This means each application will have to define how to handle index information and not be constrained by the signature format.

+1 to making the low-level signature specification (RFC dealing with the crypto specifically) not be opinionated about message indexes or how the data is encoded but instead just receive appropriately atomized / ordered data for signing. The next layer up (e.g., Data Integrity BBS+ Signature spec / JWP spec) can define how to consume / transform / canonize / order / atomize / etc. the data that is to be passed to the crypto layer.

Thanks @tmarkovski @dlongley for adding to the conversation I agree with much of the sentiment shared. In an effort to clarify further, listed below are the most relevant hard requirements BBS Signatures as a crypto scheme has that is influential in designing the contract/interface and related data structures for a derived BBS proof.

When verifying a derived BBS proof, the verifying party must know the total number of messages that were originally signed when the signature was created.
When verifying a derived BBS proof, for any revealed messages (messages that the holder or prover wishes to prove were signed by an issuer), the verifying party must know the placement each revealed message occupied in the original set of messages that were signed.

From my understanding of the opinions shared in this issue and also via DIF slack, is that most people are supportive of a proposal where the cryptographic layer has no opinion about the message indexes leaving it to upstream technologies (e.g LD-Proof or JWP) to make this definition. This means the derived BBS proof structure defined by this draft will only describe the direct cryptographic information of the proof.
Given that proposal we still need to decide how the contract for verifying a deriveProof should change, for instance the current draft defines
result = SpkVerify(spk, PK, (msg[i],...,msg[D]), nonce)

where D is the number of revealed messages makes the following invalid assumptions

spk contains information on the total number of messages originally signed
spk contains some form of mapping that maps the revealed messages (msg) to their original position in the originally signed message set.

One potential proposal would be to change the contract for verifying a derivedProof to something like
result = SpkVerify(spk, PK, revealedMessages, totalSignedMessages, nonce)

Where revealed messages is a map of the following structure
{
  1: octet_string_msg_1,
  5: octet_string_msg_5,
  9: octet_string_msg_9,
}

Another option is to retain the total signed messages as a value encoded in to spk to reduce the need to manage it externally, however still handle the ordering of the revealed messages in the way described above

Is PK already a function of totalSignedMessages (L) (via PK = DpkToPk(DPK, L) or PK = SkToPk(Sk, L))? If yes, then encoding it doesn't save much work.
I like your proposal with result = SpkVerify(spk, PK, revealedMessages, totalSignedMessages, nonce) and using a map to unambiguously describe the message input.
Implementation wise, I think application suites can choose to continue to encode the signature as it is currently, by encoding the total, indexes and proof into a single field, as long as their spec is clear this is the case. That said, we may end up with signatureValue in LD proof having different format than signature in JWT, because one chose to encode additional data into the signature. Is this a problem overall?

That said, we may end up with signatureValue in LD proof having different format than signature in JWT, because one chose to encode additional data into the signature. Is this a problem overall?

Upon further investigation, the revealed messages re-ordering issue seems to be caused by IDs that are of similar format as the blank node labels IDs. This could have a simple solution (re-sorting after transforming back to blank node labels) that will allow the LD proofs to use a bit array appended to the proof value, as originally proposed here. This can also eliminate the dangers of a correlation because of the re-ordering of the revealed messages.
That been said, IMO the bbs+ spec should remain agnostic to the way the indexes of the revealed messages are encoded. This could lead to the proofValue having different formats between implementations; however, IMO this is not a problem. Higher level specs like JWP and LD-proofs will most likely want to use different formats for the revealed message indexes either way, because of the way they work.

Is PK already a function of totalSignedMessages (L) (via PK = DpkToPk(DPK, L) or PK = SkToPk(Sk, L))?

Thanks thats a good call out, which reminded me to file #26 given the resolution to that issue I am suggesting does my proposal make more sense? e.g PK would be an ordinary public key (Note - I release this change impacts several other sections in the draft).

Implementation wise, I think application suites can choose to continue to encode the signature as it is currently, by encoding the total, indexes and proof into a single field, as long as their spec is clear this is the case.

Yes I think the point to clarify is that from this drafts perspective any data structure like a reveal indices array would be left to a higher level draft to define.

That said, we may end up with signatureValue in LD proof having different format than signature in JWT, because one chose to encode additional data into the signature. Is this a problem overall?

Given they are different digital signature representation formats I expect divergence at some level. So long as the core structure of any signature / proof and public key value remains structually the same upstream then I dont think we can prevent additional data structures being wrapped around them?

That been said, IMO the bbs+ spec should remain agnostic to the way the indexes of the revealed messages are encoded. This could lead to the proofValue having different formats between implementations; however, IMO this is not a problem. Higher level specs like JWP and LD-proofs will most likely want to use different formats for the revealed message indexes either way, because of the way they work

+1 I think much of the different at these representation layers is really due to different approaches to problems like data canonicalization which should remain entirely opaque to the crypto layer (the concern of this draft)

I think the opinion of the group has evolved sufficiently on this issue to conclude that any higher level serialization rules that accounts for the ordering of the messages required for operations like spkVerify and verify should be left to the application layer, therefore im marking this issue pending close, if others in the WG dis-agree please feel free to comment otherwise it will be closed during the next WG call.


Scripts for program and origination fees

Short TS scripts for managing new fees in staging.
Run with:
tsc example.ts
node example.js
or:
npm install -g ts-node
ts-node example.ts
Set your actual keypair/wallet if it's not at ~/.config/solana/id.json

@jgur-psyops we have a repo for helper scripts, wondering if some of this stuff would be better suited in there - https://github.com/mrgnlabs/mrgn-tools
what do you think?


WIP / DNM: Dynamic ipv6 addr assignment backup

What this PR does / why we need it:
Rebased dhcpv6 + RA daemon on a single PR.
Special notes for your reviewer:
Release note:

/uncc @vatsalparekh
/uncc @mhenriks
/cc @AlonaKaplan

RA-sender tracker in https://github.com/kubevirt/kubevirt/pull/4581
dhcpv6 tracked in #4555


git-diff: Fix typo in code

Identify the Bug
Fixes #22542.
Description of the Change
A typo inside the function subscribeToRepository was causing the mentioned TypeError because the variable this.edtior was undefined. Changing it to this.editor fixes the error.
Alternate Designs
None.
Possible Drawbacks
None.
Verification Process
After this change, subscribeToRepository no longer throws an error when called, e.g. when following the steps to reproduce the issue.
Release Notes

Fixed a typo in the git-diff package that causes repository subscriptions to fail.

Update: also fixes #22532

@sadick254 @icecream17 Here are all the related issues I found that might be fixed with this (any problems/bugs we can deal with later):
https://github.com/atom/atom/issues/22532
https://github.com/atom/atom/issues/22542
https://github.com/atom/atom/issues/22686#issue-935562185
https://github.com/atom/atom/issues/22720#issue-942225379


3rd party components replaced with standard/free ones

It took me quite some time to remove the 3rd party components and replace them with standard ones / free ones. I've opted for the WeifenLuo.WinFormsUI.Docking Docking Suite (which is free) for the dockable panes. The SandDock stuff worked fundamentally differently to the WeifenLuo ones.
I've also fixed some of the existing issues as well.
I've tested as much as I could, so I hope everything still works the same.

Wow!


[Bugfix] missing semicolon after object properties

This repairs the grunt-based tests run with "grunt". The bug occurred in
conjunction with "beautify" option, I think (I never had this bug in my
generated client, because I do not use "beautify" there but a
custom prettification via "prettier")
The other part of the test suite (the jest-based one) will be fixed in next PR.

Ouch, committed to wrong branch. Will create a new PR...


⚠ Bofrost.de has degraded performance

In b6572f6, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1015 ms

Resolved: Bofrost.de performance has improved in 26472d2.


How to make smoke particles?

I'm creating a side-scrolling game using Matter.JS, in this game one of heros is made of fire, so he needs to get some smoke, but I dont know how to make particles floating in the air, how I can make a float body in Matter.JS?
I want something like this: https://www.youtube.com/watch?v=b-QhKcae-6E
Please see my codepen here: http://codepen.io/vanuatu/pen/VeQMpp

What you're looking for is a particle system. There's nothing currently built in to the engine for this, it's more of something you'd find in a game engine so it's a bit out of scope.
You could build your own by to defining a particle object (position, velocity, lifetime etc) and then use the engine update event to update the particle positions, then render them to the same canvas as the physics objects.

thanks for the clear explanation, @liabru

No problem!

I created a fake smoke effect by using spheres and a PNG of something that resembles (black) smoke.


having trouble adding desklet to the desktop

I am new to linux.  I am running linux mint, and am trying to add the crypto coins ticker desklet.  I am adding it to my desktop, but it does not appear.  It will allow me to configure it, but it still doesn't appear.  Thank you for your help,
Jeff

Hey @jeffclifton, what version of Linux Mint Cinnamon do you have? Did you add other Desklets to the desktop to see if that works? If you add multiple instances they can be stacked so try moving them. On the newest version, you need to press the plus button from the bottom panel. If nothing works can you send me a screenshot from your Desklets Window?


J + K navigation for scrolling down / up when reviewing a PR

Vim like shortcuts for scrolling in a PR by using j for scrolling down and k for scrolling up would be super beneficial.

j/k are previous/next... in this case what are we scrolling?

Basically scrolling up and down as you'd do with the arrow keys.

Why not use the arrow keys?

Further away plus if you use the other - to come - shortcuts you would not need to move your hands. Especially if you're used to typing blindly in the 10 finger system.

Nah, if you want to scroll with j/k you can likely set that up to be global rather than just on GitHub

Just for the record. This chrome plugin works really well for this.

I'd still like navigating to next/prev file with j/k similar to navigating issues/PRs and other lists on GitHub. That with a shortcut for collapsing files would make reviews of long PRs quite easier with the keyboard.


[Improvement]: Filter out the CodecPool log in the optimizer log

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
Currently, there will be a lot of logs for compressor/decompressor such as below.

How should we improve?
We can filter out these logs by modifying the log4j property
Seems currently only the log4j property of local optimizer was managed by Amoro, so we only need to change the log4j property for local optimizer, and let the flink/spark to user(maybe we can add some docs)
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Subtasks
No response
Code of Conduct

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

@zhoujinsong @baiyangtx what do you think about this feature, if this is valid I can contribute it later.

+1


Merge R6 changes to master: Mlog10 rebase autorep colocalization

Merge coloc, autoreporting, mlog10 changes

🍾

On Nov 27, 2020, at 10:58 AM, Arto Lehisto<EMAIL_ADDRESS>wrote:
Merged #85 https://github.com/FINNGEN/pheweb/pull/85 into master.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub https://github.com/FINNGEN/pheweb/pull/85#event-4045178845, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABRO6PVTWJ3UVOLATQHSBLSR5S3RANCNFSM4TYUHNNQ.


implement Default for Rope

Small ergonomic improvement in xi-rope: The Rope type should implement Default, which would return Rope::from("").

Seems like Default is already implemented for Node<N>, see https://docs.rs/xi-rope/0.2.0/xi_rope/tree/struct.Node.html
let r = xi_rope::rope::Rope::default(); works as well and returns a length of zero as expected.

okay cool, hadn't noticed that. Thanks!


Only use the typed data interface for Ruby version >= 2.2.0

Addresses concern in https://github.com/onelogin/aead/pull/10#issuecomment-225605263
Gist of the change:

Ruby exposes a RUBY_API_VERSION_CODE constant in ruby/version.h
I asserted the versions that includes the changeset that introduces typed_data to the openssl extension ( https://github.com/ruby/ruby/commit/163cb5b43de68cef7ae4fa053d710098399d1359 )

Lourenss-Air:ruby lourens$ git tag --contains 163cb5b43de68cef7ae4fa053d710098399d1359
v2_2_0
v2_2_0_preview2
v2_2_0_rc1
v2_2_1
v2_2_2
v2_2_3
v2_2_4
v2_2_5
v2_3_0
v2_3_0_preview1
v2_3_0_preview2
v2_3_1

I think it's OK to skip v2_2_0_preview2 and v2_2_0_rc1 and standardise on v2_2_0 as the version that should start using the typed_data interface for this extension.
@pallan @csfrancis @jnormore @stonith

:shipit: I tested on a few versions of ruby from 1.9.3 through 2.3.1 and it worked correctly.