Can a go-micro stream be like a grpc stream?

go-micro stream must send then recv,but grpc not.
like this:
go func() {
        for {
	    select {
		case <-ctx.Done():
		        log.Println("close...")
			return
		default:
			log.Println("send...")
			stream.Send(&steam.Request{
				Msg: time.Now().String(),
			})
		}
	}
}()

go func() {
	for {
		req, err := stream.Recv()
		if err != nil {
                         stream.Close()
			log.Println("err:", err)
                         cancel()
		}
		log.Println("recv:", req.Data)
	}
}()

yes go-micro can do this.
example

yes go-micro can do this.
example

Sorry,I don't think so.
I think you have never read the code.

???
Go-micro bidirectional stream processing steps is that
client-->connect to server
server or client -->  rev or send data
the server side stream  function finished and  return ,  go-micro will send  EOF to client ,then the client close the stream;

???
Go-micro bidirectional stream processing steps is that
client-->connect to server
server or client --> rev or send data
the server side stream function finished and return , go-micro will send EOF to client ,then the client close the stream;

Do you know Recv() and Send() have lock?
Please read the source code.
If you receiving but nothing then you can not send anything.
That is the reason why I open this issue.

sorry i Misunderstanding it. i have read the source code.  the default stream Rev() and Send() have lock.
perhaps you can use the grpc plugin replace the default processing
it have no lock grpc client and grpc server

As I see it the question is stating that that the rpc_stream.go code has a mutex around sending and receiving data. When Send is called, it locks the mutex. When Recv is called, it locks the mutex. There for only one or the other operation can be honoured at once. This essentially limits bidirectional streaming to the ability to process requests over the connection. Such that if Recv is blocking waiting for a response from the server nothing can be sent to the server. My assumption is the server code probably looks similar.
We could potentially remove the mutex lock but we may uncover some race conditions in the process. If anyone wants to perform this optimisation feel free to take a shot at it.

sorry i Misunderstanding it. i have read the source code. the default stream Rev() and Send() have lock.
perhaps you can use the grpc plugin replace the default processing
it has no lock. grpc client and grpc server

yes,but...my project inconvenient to use grpc.

@lockp111 you have the choice of either using grpc for your requirements or submitting a PR to allow full duplex streaming in the rpc code. Feel free to do either. Otherwise, I'm closing this issue as your question is answered.


feat: misc

@Michiel87 let's merge #30 first, then I'll resolve conflicts on this branch

@Michiel87 ready for review!


Why not use use message_lite in proto message ?

I see there exists USE_LITE_PROTO  in CMakefiles.txt, but why not set  option optimize_for = LITE_RUNTIME; in proto message.

I want to deploy caffe2 to my cell phone and I want to reduce the size of the final static library.
There is a USE_LITE_PROTO to control whether to use the lib in protobuf.a, but, the caffe2.pb.cc and  caffe2.pb.h  generated from caffe2.proto use the interface offered by protobuf.a,  do I need to manually add  option optimize_for = LITE_RUNTIME; to caffe2.proto? Or there are other ways


replace DBKit with NIOKit

Relies on https://github.com/Mordil/nio-redis/pull/11

Hey @tanner0101, you just merged a pull request, have a coin!
You now have 1110 coins.


add huflit file

@builacviet First stage of verification passed, official domain is huflit.edu.vn as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@builacviet Thank you for your patience while waiting for a review and processing your request. Pull request merged.


{} block can change a program's control flow when using is used

Observe the following two programs:
{
    using x = X();
    using y = Y();
    console.log("done");
}

{
    using x = X();
    {
        using y = Y();
    }
    console.log("done");
}

function X(): Disposable {
    return { [Symbol.dispose]() {} };
}

function Y(): Disposable {
    return { [Symbol.dispose]() { throw new Error(); } };
}

Both programs would throw the same error, but the first program would print the log while the second won't.
This behavior may be surprising to many people as we are used to the fact that a {} block should not affect the control flow, but with using, {} can change your program's control flow.

Yes, that's the point of it. We were aware of this when designing the feature and consider it to be acceptable.

@bakkot
Yes, that's the point of it. We were aware of this when designing the feature and consider it to be acceptable.

I don't know how to justify this control flow implication on {} block to be acceptable. The bahvior is very implicit and subtle. Even knowing how using works, reasoning about the behavior differences in the above simple examples can still take a few seconds for me.
What should I tell my colleagues who don't know about using to understand the behavior implications in a code review? IMO this behavior creates more problems than it solves.

What should I tell my colleagues who don't know about using to understand the control flow implications in a code review?

Any using statement in a block implies code executed when exiting the block. As with any code execution, it can throw.
I believe this is the trade-off that has to be made for allowing inline RAII style. The alternative requires declaring all resources when opening the block, which has poorer ergonomics, and still implies code execution when exiting the block, but is maybe more "balanced" and thus more noticeable.

What should I tell my colleagues who don't know about using to understand the control flow implications in a code review?

You should send them a link to the MDN page on using, or any other such tutorial or overview, the same way you'd tell them about any other new syntax. All new syntax requires that people to learn about the syntax to understand what it does.

FWIW, It would have been great if we could have marked the block exit more clearly as an execution / interleaving point. Maybe someone can come up with a code editor tool that automatically adds a comment to these block exits.

I really don't see how using is better than DisposableStack + try ... finally in solving the issues in the Motivations section of the proposal.
const stack = new DisposableStack();
try {
    const x = stack.use(X());
    const y = stack.use(Y());
    console.log("done");
} finally {
    stack.dispose();
}

It solves all issues in the Motivations, with only the cost of creating a stack and adding a single try ... finally block. And it doesn't have any implication on control flow of {} blocks:
const stack = new DisposableStack();
try {
    const x = stack.use(X());
    {
        const y = stack.use(Y());
    }
    console.log("done");
} finally {
    stack.dispose();
}

In my original suggestion against adding syntax, I suggested abusing the for of construct, which didn't suffer as much from this gotcha:
for (const { using } of Disposable) {
  const resource = using(getResource());
  resource.doSomething();
  const other = using(resource.getOther());
  const stuff = other.doStuff();
  using(() => cleanUpStuff(stuff));
} // automatically cleanup, even when something throws

You can still write an adapter today on top of DisposableStack that does this.

When this proposal was first introduced, the syntax was similar to C#'s earliest version of using:
using (const x = y) {
}

Over the many years that this proposal has been discussed, there was more and more interest within the committee to switch to an RAII-style form, and the consensus was to switch entirely to the RAII form. Even C# has adopted the RAII form for using and await using in recent editions. The implications of this syntax are well known to the committee at this time.

@mhofman
In my original suggestion against adding syntax, I suggested abusing the for of construct, which didn't suffer as much from this gotcha:
for (const { using } of Disposable) {
  const resource = using(getResource());
  resource.doSomething();
  const other = using(resource.getOther());
  const stuff = other.doStuff();
  using(() => cleanUpStuff(stuff));
} // automatically cleanup, even when something throws

You can still write an adapter today on top of DisposableStack that does this.

Though I can't say if I like the idea of abusing for of contract for the effect, I think the problem for using is trying to confuse execution scope with resource scope, as an implication of adopting RAII. I've discussed this in length in #159.
Any solution that creates a seperate scope for resource binding can avoid this issue.

@rbuckton
Even C# has adopted the RAII form for using and await using in recent editions. The implications of this syntax are well known to the committee at this time.

I don't know having the committee knowing the implication is a good justification to impose this behavior for all JavaScript developers. Many JavaScript developers don't have the experience with RAII like C# or C++, and having this new syntax breaking a well established instinct on how {} block should behave is problematic for the developer community.
I think we should raise this behavior change on {} block very public and explicit and make sure we fully understand how the wider community feels about the change.


Initial Background Color Lost During PXON Export

The initial background color that is set when you make a new image is never stored to history, so it is lost during the export/import PXON process.
Steps to reproduce:

Select a nice color and click new canvas
Optional: draw something cool
Click on import/save + and then click export pxon
Copy the text to a file
Click on Choose File and select that file you just created
Observe that the background is now the checkerboard pattern that signifies transparency

Since pxon.pixels[0].originalColor already contains the displayed background color during creation, the background is inferable for any non-blank PXON image.
A related thing that happens is that that background color isn't extended to new pixels when an image is resized.
My ideal solution to both of these is that background color becomes a first-class property.


Folder check

Check that all the folders in the file path are actually folders. Sometimes IDEs etc. add random non-folder files (i.e. PyCharm) that can cause errors in the code

thx!


chore: upgrade CLI command and dynamic plugins

Hey, I just made a Pull Request!

Upgrade the Janus CLI to include the change in default module interop mode (to use the same as the backstage CLI rollup build), along with the ability to override the interop mode either globally or for a specific import (see PR https://github.com/janus-idp/backstage-plugins/pull/971).
Upgrade imported dynamic plugins to pick those built with this new CLI version.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

@tumido @kadel Any idea whether you would have time to review this PR ?
cc @janus-idp/maintainers-showcase


Update package version to the same as on PyPI

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12390119335. Examine the logs at this URL for more detail.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12390723764. Examine the logs at this URL for more detail.


Increate memory size

The default of 512 is sometimes not enough. Would be great to double it.

I've increased memory_limit up to 1024M. Affected images:
wodby/drupal-php:latest
wodby/drupal-php:7.0
wodby/drupal-php:5.6

Thanks so much, @PavelPrischepa!

@PavelPrischepa  , I tried updating the image to the latest, 7.0-2.4.9, or 7.1-2.4.9 but still it's showing 512MB memory_limit. Can you help?

@GTSLiDe in the 2.x version of d4d you should use PHP_MEMORY_LIMIT environment variable to customize memory limit.

@PavelPrischepa it worked! thank you so much!


bump version

I wouldn't normally make  a PR for this, but just wanted to double check - this should be a minor version bump because this is a new feature, right? #68

Apparently so - closing in favor of #71


Add translation engine

Add translation engine gem - as option

https://github.com/blueberryapps/blueberry_rails/pull/35


L8 Moodlight

First of all, Thanks for the massive, impressive effort. I have been trying to make use of these high-potential devices for more than a year. I managed to create several working prototypes using vanilla LVGL but no 2-way integration with HomeAssistant, and also with ESPhome (works nicely with HA but no touch!). I am beyond grateful for your generous contribution here.
Regarding the L8. Now with version 0.3.3, when I set up the backlight of L8 using moodlight  (GPIOs r:26,g:32,b:33). The resulting entity in HomeAssistant (via MQTT auto-discovery) isn't working. I tried to create a manual MQTT light using the MQTT Light Template Schema in HA which worked only for on/off but not for color.
I created 3 sliders on a page and used groupid to link them to each GPIO directly from the HASP without HomeAssistant. This seems to work fine. Kindly take a look if have an L8.
Also, in general, I think it is possible to link a color-picker object with moodlight using groupid so we can directly control it without MQTT. It needs a bit of work but will make more sense than using HA or MQTT to change the backlight color on L8.
Best Regards!

I tried to read the code parts related to moodlight and groupid in order to help and contribute to this enhancement, but because I don't have enough C++ knowledge and there is very little comments in the code, I haven't got the confidence to submit a pull request.
My simple idea with minimal code change is to assign a pre-fixed groupid to moodlight (such as 888) and then modify to value normalization function to allow for the color hex code (with a simple "if" statement to create exception for groupid 888).

Hi @mssaleh,
Thanks for the input, I'm looking into this... The moodlight uses 3 groupids, one per PWM channel.
The way groups are currently setup, each object belongs to a single groupid. A colorpicker is special in that regard...
I think for HA to recognize the RGB values, the message needs to be in {"rgb":[255,255,255]}format.
I'm a HA n00b so I need to test that part some more, but this is certainly a goal to have that working too.

OK, I got the moodlight auto-discovery working in HA now.

Thanks a lot! You are really a github treasure! I have been extremely busy at work but I am planning to contribute to your effort in March.

Looking good, still need to add brightness support.

How do you guys use this with autodiscovery in HA?
I configure everything manually - can this be done too?
Any HA config sample perhaps?

autodiscovery is not yet working in 0.3.4, but in 0.4.0-dev it should work now

But is there a way to use this with manual confuguration, not autodiscovery?

There is a moodlight command in 0.3.4

Ok my problem was that my Home Assistant's MQTT was initially configured back when autodiscovery didn't even exist. Somehow autodiscovery refused to work but re-installing the MQTT integration solved this.

But is there a way to use this with manual confuguration, not autodiscovery?

The custom component adds it manually so no more autodiscovery needed, cool.

autodiscovery was the only possibility for a while, this has been dropped now since we have a custom component that can handle it better.

yes, that's why I thought I should mention this.

I think we can close this issue in favor of the openHASP custom component?

Looking good, still need to add brightness support.

Is brightness of moodlight implemented at firmware level? Or is it just missing from the component?


why pinpoint-web-1.9.0-SNAPSHOT.war can`t  send mail

I read this configure http://naver.github.io/pinpoint/alarm.html
I cloned the latest package and compiled it，When I deploy it into the container，it has error,i only modify batch.properties file and jdbc.properties file
pinpoint web error  log：
2019-02-28 15:33:00 INFO  Job: [FlowJob: [name=alarmJob]] launched with the following parameters: [{schedule.date=1551385980005}]
2019-02-28 15:33:00 INFO  Executing step: [alarmPartitionStep]
2019-02-28 15:33:01 INFO  Job: [FlowJob: [name=alarmJob]] completed with the following parameters: [{schedule.date=1551385980005}] and the following status: [COMPLETED]
2019-02-28 15:36:00 INFO  Job: [FlowJob: [name=alarmJob]] launched with the following parameters: [{schedule.date=1551386160009}]
2019-02-28 15:36:00 INFO  Executing step: [alarmPartitionStep]
2019-02-28 15:36:01 ERROR can't send alarm email. Rule{ruleId='1', applicationId='myphp', serviceType='PHP', checkerName='TOTAL COUNT', threshold=4, userGroupId='pins', smsSend=false, emailSend=true, notes=''}
at com.navercorp.pinpoint.web.alarm.SpringSmtpMailSender.sendEmail(SpringSmtpMailSender.java:81)
at com.navercorp.pinpoint.web.alarm.DefaultAlarmMessageSender.sendEmail(DefaultAlarmMessageSender.java:41)
at com.navercorp.pinpoint.web.alarm.AlarmWriter.sendAlarmMessage(AlarmWriter.java:65)
at com.navercorp.pinpoint.web.alarm.AlarmWriter.write(AlarmWriter.java:52)
2019-02-28 15:36:01 INFO  Job: [FlowJob: [name=alarmJob]] completed with the following parameters: [{schedule.date=1551386160009}] and the following status: [COMPLETED]

my jdbc.properties file.
jdbc.driverClassName=com.mysql.jdbc.Driver
jdbc.url=jdbc:mysql://localhost:3306/pinpoint?characterEncoding=UTF-8
jdbc.username=admin
jdbc.password=admin
my  batch.proferties file
#batch enable config
batch.enable=true
#batch server ip to execute batch
batch.server.ip=<IP_ADDRESS>
batch.server.env=release
#smtp config
pinpoint.url=http://pinpoint.com:28080
alarm.mail.server.url=smtp.sina.com
alarm.mail.server.port=25
<EMAIL_ADDRESS>alarm.mail.server.password=xxxxxx
<EMAIL_ADDRESS>#flink server list
batch.flink.server=
how to solve it

Hi @suiyuewuheng
The error log above should have a "Caused by" log which tells you what caused the exception to be thrown.

@suiyuewuheng @Xylus
Same error.  The config file was wrong.
change alarm.mail.server.userName  to   alarm.mail.server.username       lowercase

@pwvictor
Thank you~^^ I will fix it.

I commited~.

there is another situation, in my environment, it doesn't need to config the username and password, just send the mail use the smtp server, how can this be done?

Hi @ipaste
Did you test?
I think that if you do not need a password username, it works fine without setting it.
Currently I am using it.


Improve the UI of Four-Player Chess Game

Hi @Navneet1206
Add a dropdown menu that allows users to select from a variety of chessboard color schemes (e.g., classic black-and-white, wood-themed, or custom color combinations). This feature will let players choose their preferred board style, enhancing their gameplay experience. This should be dynamic in nature and the player can change the color of the chess board even when playing. I’m confident I can resolve this and contribute to improving the user experience.
Thank You!

Hi @Priyanshiagarwal2006 ,
Thank you for your suggestion! Yes, this is a valid issue, and I appreciate your input. However, I’m currently focusing more on the application's functionality Like Socket.io and contract rather than the UI, as I want to ensure that the core features are smooth and user-friendly first. That said, I agree that improving the user experience through UI enhancements like the color scheme dropdown is a great idea and could be a valuable addition in the future.
Looking forward to your contributions!
Best regards,
Navneet Vishwakarma


Microchip: MEC172x: pinctrl implemenation part 1

Part 1 of a pinctrl implementation for Microchip MEC172x.
Add helper macros, defines, and DT pin definitions for use
by future pinctrl drivers.
Signed-off-by: Scott Worley<EMAIL_ADDRESS>

This is part 1 of a PINCTRL implementation based on STM32's.  This PR contains macros and pin device tree information for use by drivers in future PR's.


API Clarification: EntityEnterPortalEvent

When a survival player enters a nether portal, does EntityEnterPortalEvent fire before or after the delay?

The act of an entity being teleported is a different event. The act of an Entity physically entering a portal is the event you've stated. There's a lot of changes especially with this with regards to clarification in the refactor/event-names and the branch itself.

So, is there an event for overriding Portal behavior? (In my case, I’m working on a multiverse-type plugin and want to allow setting which worlds function as “Nethers” or “Ends” for which other worlds)

On Aug 29, 2015, at 1:43 PM, Gabriel Harris-Rouquette<EMAIL_ADDRESS>wrote:
The act of an entity being teleported is a different event. The act of an Entity physically entering a portal is the event you've stated. There's a lot of changes especially with this with regards to clarification in the refactor/event-names https://github.com/SpongePowered/SpongeAPI/blob/refactor/event-names/src/main/java/org/spongepowered/api/event/entity/EntityPortalEvent.java and the branch itself.
—
Reply to this email directly or view it on GitHub https://github.com/SpongePowered/SpongeAPI/issues/827#issuecomment-136048307.

So, is there an event for overriding Portal behavior? (In my case, I’m working on a multiverse-type plugin and want to allow setting which worlds function as “Nethers” or “Ends” for which other worlds)

Yep, DisplaceEntityEvent.Teleport. There's also the TeleporterAgent


Verify All Claims button returning error

Context
After all the necessary claims have been made, when clicking on the "Verify My Claims" Button an error is returned
https://user-images.githubusercontent.com/48899363/180697887-352bb80b-369c-424a-acbe-788d4bb23d4d.MP4

This happens when the TFN claim is included as well

This is what was returning in the terminal


Remove Update(entity) from UpdateAsync()

If I am not mistaken, the Update() method only tells EF to track the entity.
Because we already have the entity from the context (FindByIdAsync()) and then apply the changes to it, we can safely remove that line.

Furthermore doesn't the Update() method vioalte the Repository pattern?
(I still have to read up on the Repository Pattern, my gut just tells me so because a Repository should be like a List).

Hi,
Thank you for your contribution!
You're right, we don't need the  Update() here since the entity is already being tracked by EF.
Talking about the repository pattern and the Update() method, it doesn't necessarily violate the repository pattern because it's not going to save anything to the database. I'm not sure how EF Core tracks the entities when you call this method, but I believe it keeps it in a distinct collection to check later when we call SaveChanges(). It would violate the repository pattern if we call Update() and then save changes to database at the same method call.
I'm going to merge your changes, thank you!

@all-contributors please add @NoobInTraining for code


Move all settings to to JSON format

Will move most of the settings to JSON format settings file.

This is now done, we have only a few things left in the script, but that is more only for reminding but are not usage at all.

Close this ticket


Make RustPad just-enough compatible with EtherPad URLs

Allows RustPad to work as a drop-in alternative for services that have basic EtherPad (popular collaborative document editing tool) integration using an IFrame
Really, it just extracts the two key pieces of information when invoked like:
https://path/to/rustpad/SomePadName?some=param&userName=AUserName&other=param
but that is enough to make it useful in the context of the “Collaborative Editing” (=EtherPad) support in the Jitsi Meet video conferencing solution. (SomePadName will be the name of the Jitsi Meet conference and AUserName the name of the currently logged in user.)
Based on https://github.com/ekzhang/rustpad/pull/75, no real dependency through.

Interesting, I like the idea! Down to merge this if the CI works.

I just found https://docs.etherpad.org/api/embed_parameters.html which documents the EtherPad parameter list and I found that aside from userName and the Pad name there is one more parameter with semantic overlap (userColor), so hue is now derived from that if it is set.
There are now also comments about the supported and unsupported parameters that should be helpful and I restructured the code to have all the URL handling in the now renamed useFromURL.ts rather than App.tsx.
Let me know what you think!

Sorry it's been a few months @ntninja — I reevaluated my priorities and I don't think I can merge this at this time. Apologies. You might need to fork :(


ovsdb: Redesign ovsdb with simplier interface and improved performance

Before this patch, there were many redundant ovsdb functions that
causes large amounts of unnecessary ovsdb transactions. That
becomes the performance bottleneck for quilt when booting a large
cluster. This patch simplifies the ovsdb interface, and reduces
O(N) number of ovsdb transactions in terms of number of VMs to O(1).

Btw, the commit message should say "Co-Authored-By: Ethan J. Jackson<EMAIL_ADDRESS>since I wrote part of the patch.

First patch should be ovsdb: Redesign ovsdb with simpler interface and improved performance

Oh, also I think the first patch should say it's co-authored by me, since I wrote the initial starting point before handing it off.

Merged

Oops, actually merged the other one.

hmmm? I've already added co-author in the commit message. I added it as a separate line in commit message. Or should I do it in some other format?

Ah my bad, the Co-authored by looks good.

Merged


Add wechaty-puppet-puppeteer, and wechaty-puppet-padplus testing back into smoke testing

Is your feature request related to a problem? Please describe.
wechaty-puppet-puppeteer, and wechaty-puppet-padplus testing were removed from smoke testing because they might fail in git action.
Describe the solution you'd like

Please add them back, for the environments that they still work. For example, for wechaty-puppet-padplus it is under Node 10 (i.e., when TRAVIS_NODE_VERSION=10, check here).
Request coming from https://github.com/wechaty/wechaty-getting-started/issues/100

[enhancement]

Trying to fix for wechaty-puppet-padplus...
Meanwhile, why wechaty-puppet-puppeteer was removed? What environments it can run?

I have added the PADPLUS TOKEN to the GitHub Actions. However, I believe that only the master branch can get that environment variable value.
the wechaty-puppet-puppeteer was removed due to it can not pass the GitHub Actions with very strange behavior. I'll investigate it later, and please feel free to add it back if you can figure the problem out and fix it.

Thanks, I moved it back since only the master branch can get that environment variable value.
Also, I added wechaty-puppet-puppeteer back to travis. I'll leave fixing it for GitHub Actions to you, :)


respond to these folks by may 5

email from them apr 28 and may 1.
taken care off?
Hi,
Yes, please change the download for Haggis to these datasets.
Let us know if we can be of any further help with the Java GitHub Corpus.
Thanks,
Miltos
On Fri, May 1, 2015 at 11:37 AM, Open science Content<EMAIL_ADDRESS>wrote:
Hello,
With your permission, we will change the download for our haggis dataset to link to MSR13 and the Java GitHub Corpus. We have added the "Mining source code repositories at massive scale using language modeling" paper to our queue of datasets to add; we'll send another email once it's been added.
Thank you for your contributions!

I've almost finished uploading this data and will email back tonight.


⚠ ClamAV has degraded performance

In 9ed4a37, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 218 ms

Resolved: ClamAV performance has improved in f99182f.


About Tippecanoe-decode

When I use command "Tippecanoe-decode" decode a .mbtiles file, didn't find the output file.

I found how to solve this problem.


Enable values-devsecops-testing for portal and iam

Enable values-devsecops-testing for portal and iam
Secret data for pre-prod should work fine, no additional devsecops secrets needed.

@almadigabor Could you please check the version of the chart? The linter has an issue with it.

@almadigabor Could you please check the version of the chart? The linter has an issue with it.

That workflow shouldn't trigger when charts/ dir hasn't changed. I've updated the workflow on main, please merge it and see if it is gone.


ADFS 2016

Would you be able to please confirm if Scenario 1 is possible/supported on ADFS 2016? The exact requirement to be specific Prompting for Azure MFA as primary authentication and Username and password as additional authentication.
"Scenario 1
This scenario consists of two components:

Prompting for Azure MFA or an external authentication factor as primary authentication
Username and password as additional authentication in AD FS"

This doc suggests this capability is only possible with ADFS 2019.
ADFS 2016 does support part of requirement by allowing primary auth with Azure MFA as documented here:
https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/operations/configure-ad-fs-and-azure-mfa

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

ID: 404ed1e0-3d50-5f22-4721-ba79ce56589f
Version Independent ID: 7cb551c4-cdb2-d96d-4167-1bb074064453
Content: Additional authentication methods in AD FS 2019
Content Source: WindowsServerDocs/identity/ad-fs/operations/Additional-Authentication-Methods-AD-FS.md
Product: windows-server
Technology: identity-adfs
GitHub Login: @billmath
Microsoft Alias: billmath

#please-close


get rid of misc directory

the misc directory should be somewhere else, not it this project.
it might be ASU for a good place, ( or might be there already?)

But where can I put the e.g. screenshot?
Is what you meant to say, that you want to do a checkout somewhere in production and update via "git pull"? I that case, misc would be in placed in production.

i criticise all files EXCEPT the screenshot
thereshould be no need for python or git on a device with YAFS
yafs is not able to know which openwrt-versions and hardware can be build by corresponding ASU-server.
its should be the task of ASU to provide these lists on demand.
(maybe ASU has this feature already?)

i criticise all files EXCEPT the screenshot
The profiles.json files in misc are required for the demo. The python script is kind of a necessary evil unless OpenWrt creates an optimized single profiles.json file.

I also like this project to be as self contained as possible.

thereshould be no need for python or git on a device with YAFS
I agree with git, of course. But a script is needed to merge the json files.
yafs is not able to know which openwrt-versions and hardware can be build by corresponding ASU-server.
its should be the task of ASU to provide these lists on demand.
(maybe ASU has this feature already?)
ASU support is an optional to YAFS and I do not want to have it as must have feature.
Otherwise this project has no use for me.

We can't get rid of this additional folder unless we want to spam the root folder.

The web content was now moved to a www folder. Maybe that addresses your concern to some degree.


add v0.4.0 changelog

Motivation
Add v0.4.0 changelog
Modifications
Add v0.4.0 changelog
Verifying this change

[x] Make sure that the change passes the CI checks.

[x] This change is a trivial rework / code cleanup without any test coverage.

Documentation
Check the box below.
Need to update docs?

[x] no-need-doc
release

Leave my comments. PTAL

thanks a lot for the suggestions, already applied


Rebalancer restarting!

The rebalancer restarted out of the blue this morning and the telegram bot reported nicely :-)

The following entry was found in /tmp/autorebalancer.log:
runLoop error: TypeError: Cannot read properties of null (reading 'fee_base_msat')
TypeError: Cannot read properties of null (reading 'fee_base_msat')
    at /home/umbrel/lightning-jet/lnd-api/utils.js:241:43
    at Array.forEach (<anonymous>)
    at /home/umbrel/lightning-jet/lnd-api/utils.js:220:15
    at end (/home/umbrel/lightning-jet/node_modules/run-parallel/index.js:21:15)
    at done (/home/umbrel/lightning-jet/node_modules/run-parallel/index.js:25:10)
    at each (/home/umbrel/lightning-jet/node_modules/run-parallel/index.js:31:7)
    at /home/umbrel/lightning-jet/node_modules/run-parallel/index.js:46:37
    at Object.callback (/home/umbrel/lightning-jet/lnd-api/utils.js:208:18)
    at Object.onReceiveStatus (/home/umbrel/lightning-jet/node_modules/@grpc/grpc-js/build/src/client.js:177:36)
    at Object.onReceiveStatus (/home/umbrel/lightning-jet/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:365:141)
Error: async hook stack has become corrupted (actual: 540059, expected: 4110)
 1: 0xa536f4 node::AsyncWrap::PopAsyncContext(v8::FunctionCallbackInfo<v8::Value> const&) [node]
 2: 0xd2b230  [node]
 3: 0xd2c28c v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) [node]
 4: 0x154a28c  [node]

Maybe the cause is a new channel opening, as at around the same time a new incoming channel has been opened :-)

Good thing that jet is resilient to failures and was able to recover. At the same time, it should handle exceptions gracefully.  In this particular case, it is likely that a new channel was added, but the fee info was not yet available, hence the null pointer exception when trying to access the fee policy field.
Will be addressed in the next git push.


Please consider adding themes

Refer this repo：https://github.com/anuraghazra/github-readme-stats#all-inbuilt-themes--

Good idea, thx. I will consider this.

目前已支持暗黑模式：


1.7rc1 installation issue on Ubuntu 22.10

~ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 22.10
Release:	22.10
Codename:	kinetic
~ asdf plugin-add jq https://github.com/AZMCode/asdf-jq.git
~ asdf install jq 1.6
~ jq --version
jq-1.6
~ asdf install jq 1.7rc1
curl: (3) URL using bad/illegal format or missing URL
~ asdf list jq
 *1.6

I can confirm this issue continuing through the full 1.7 release

I'd like to add more information about this issue as I've been suffering it recently. It happens in Linux based machines but it works in MacOS ARM64, though.
After further investigation, it seems that the plugin isn't able to correctly parse the information on the new supported architectures in jq 1.7 that it gets from the Github repository. These are the URLs passed to curl for downloading versions 1.6 and 1.7:
1.6
curl -A curl/1.0.0 -H 'Accept: application/json' -fsS -L -o /home/user/.asdf/downloads/jq/1.6/bin/jq https://github.com/jqlang/jq/releases/download/jq-1.6/jq-linux64
1.7
curl -A curl/1.0.0 -H 'Accept: application/json' -fsS -L -o /home/user/.asdf/downloads/jq/1.7/bin/jq 'https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-amd64 https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-arm64 https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-mips64 https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-mips64el https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-mips64r6 https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-mips64r6el https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-ppc64el https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-riscv64 https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux64'


https://scratch.mit.edu/projects/331208692/

https://scratch.mit.edu/projects/331208692/
https://phosphorus.github.io/#331208692
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.132 Safari/537.36

I don't know how to transfer sb3 to apk please help me


Extend IPPool to support StatfulSet owner

Signed-off-by: Anna Khmelnitsky<EMAIL_ADDRESS>

Codecov Report

Merging #3093 (78195b3) into main (4864914) will decrease coverage by 27.29%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3093       +/-   ##
===========================================
- Coverage   58.52%   31.23%   -27.30%
===========================================
  Files         292      286        -6
  Lines       24721    23919      -802
===========================================
- Hits        14469     7472     -6997
- Misses       8701    15310     +6609
+ Partials     1551     1137      -414

Flag
Coverage Δ

kind-e2e-tests
31.23% <ø> (-10.70%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

pkg/util/sets/int32.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/util/sets/string.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/controller/egress/id_allocator.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...kg/agent/controller/networkpolicy/audit_logging.go
0.00% <0.00%> (-96.16%)
:arrow_down:

.../agent/flowexporter/priorityqueue/priorityqueue.go
0.00% <0.00%> (-92.60%)
:arrow_down:

pkg/controller/networkpolicy/endpoint_querier.go
2.85% <0.00%> (-88.58%)
:arrow_down:

pkg/agent/controller/networkpolicy/reject.go
0.00% <0.00%> (-87.91%)
:arrow_down:

pkg/controller/networkpolicy/crd_utils.go
0.00% <0.00%> (-87.24%)
:arrow_down:

pkg/controller/egress/controller.go
0.00% <0.00%> (-84.82%)
:arrow_down:

pkg/ipam/ipallocator/allocator.go
0.00% <0.00%> (-84.22%)
:arrow_down:

... and 148 more

@tnqn Could you also notify the owner of NetworkPolicy Reject feature?
The reject action is not stable enough sometimes.
https://github.com/antrea-io/antrea/runs/4435766434?check_suite_focus=true

/test-all
/test-ipv6-all
/test-ipv6-only-all
/test-windows-all
/test-flexible-ipam-e2e

/test-flexible-ipam-e2e

@tnqn @jianjuns @annakhm  We also need to support preallocation for Deployment. Do you think we need to add another Deployment *DeploymentOwner, or change StatefulSet *StatefulSetOwner to a common Controller *ControllerOwner with Kind property?

@tnqn Could you also notify the owner of NetworkPolicy Reject feature? The reject action is not stable enough sometimes. https://github.com/antrea-io/antrea/runs/4435766434?check_suite_focus=true

@GraysonWu

@tnqn @jianjuns @annakhm We also need to support preallocation for Deployment. Do you think we need to add another Deployment *DeploymentOwner, or change StatefulSet *StatefulSetOwner to a common Controller *ControllerOwner with Kind property?

Unless that can enable more code sharing, I do not feel a common Owner type is better than the current way.

@tnqn @jianjuns @annakhm We also need to support preallocation for Deployment. Do you think we need to add another Deployment *DeploymentOwner, or change StatefulSet *StatefulSetOwner to a common Controller *ControllerOwner with Kind property?

What is the rationale behind pre-allocation for Deployment? Making sure pool is not exhausted while pods are created? Or continuous IP range?

What is the rationale behind pre-allocation for Deployment? Making sure pool is not exhausted while pods are created? Or continuous IP range?

Yes, we have a requirement to assign a given range to a deployment, and to provide that it is a subset of available IPs in the IPPool.

Are we waiting for some change in this PR (deployment struct?)?

Are we waiting for some change in this PR (deployment struct?)?

I'm OK, since we don't need to add DeploymentOwner in this release.


Installation issue

Hello,
I am getting the following error in the installation:
node-pre-gyp ERR! Tried to download(403): https://mapbox-node-binary.s3.amazonaws.com/sqlite3/v3.1.11/node-v57-linux-arm.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI) (falling back to source compile with node-gyp)

After exiting it and trying to run "node gekko --ui" I am getting the following error:
TAlib is disabled
Serving Gekko UI on http://localhost:3000/

(node:1297) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Exited with code 3
(node:1297) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Also when I am trying to run "npm install sqlite3" I am getting this error:
node-pre-gyp ERR! Tried to download(403): https://mapbox-node-binary.s3.amazonaws.com/sqlite3/v3.1.13/node-v57-linux-arm.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI) (falling back to source compile with node-gyp)
make: Entering directory '/root/node_modules/sqlite3/build'
  ACTION deps_sqlite3_gyp_action_before_build_target_unpack_sqlite_dep Release/obj/gen/sqlite-autoconf-3150000/sqlite3.c
  TOUCH Release/obj.target/deps/action_before_build.stamp
  CC(target) Release/obj.target/sqlite3/gen/sqlite-autoconf-3150000/sqlite3.o

This might help as well: when I run "npm start" I get the following errors:
npm ERR! path /home/pi/package.json
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall open
npm ERR! enoent ENOENT: no such file or directory, open '/home/pi/package.json'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/pi/.npm/_logs/2017-10-17T04_36_36_317Z-debug.log

Thanks for your help!

After exiting it and trying to run "node gekko --ui" I am getting the following error:

That's not an error, it means you are installing Gekko on something without a UI (server, rasp pi) and you didn't configure the UI accordingly. See the docs.
For your other problems: Please download nodejs LTS, if that still fails manually compile sqlite, see here: https://github.com/mapbox/node-sqlite3#building-from-the-source

The problem I am facing is, that I am not able to install sqlite3. I am constantly getting the following message:
node-pre-gyp ERR! Tried to download(403): https://mapbox-node-binary.s3.amazonaws.com/sqlite3/v3.1.13/node-v48-linux-arm.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v48 ABI) (falling back to source compile with node-gyp)
make: Entering directory '/home/pi/node_modules/sqlite3/build'
  ACTION deps_sqlite3_gyp_action_before_build_target_unpack_sqlite_dep Release/obj/gen/sqlite-autoconf-3150000/sqlite3.c
  TOUCH Release/obj.target/deps/action_before_build.stamp
  CC(target) Release/obj.target/sqlite3/gen/sqlite-autoconf-3150000/sqlite3.o

Do you know what is causing that?

Yes: mapbox does not have precompiled binaries for your OS and your node version. You might have more luck with the nodejs LTS. And if that also fails just compile yourself. See the link I posted above.


Add additional colors as theme resources

https://msdn.microsoft.com/en-us/library/windows/apps/windows.ui.viewmanagement.uicolortype

I can help out on this as I am already trying to do this

Perfection. Update your code base, lots of changes.

Have you made any progress?

Some will try and post something for you soon.

Cool. Let me know.

How's it coming?

This is easy to use:
new UISettings().GetColorValue(UIColorType.Accent);

But in mobile, the method "GetColorValue" will throw a InvalidCastException.
I think use:
(Color)Application.Current.Resources["SystemAccentColor"];

It's better.


Improve a11y of docs

Add placeholder/label to input on homepage demo
Mark sections with aria-expanded

Easiest to review ignoring whitespace at https://github.com/reactjs/reactjs.org/pull/659/files?w=1.
@pauljadam Would you mind taking a look to check these changes?

Another PR added visible labels. Does it make sense to have an aria-label as well?


Support for storing the encryption keys of non-core resources (such as workspaces) in an external Key Vault

Once #4145 and #4143 are completed, we will need to enable workspace resources to access the external Key Vault for accessing the CMK.
However, since the external Key Vault is outside of TRE's network, this might not be feasible.

make sure to update the CMK section in the docs accordingly.

This one works without any changes required


Attempt to use PrettyDouble to get around Boxed/Unboxed issue.

Hi Conal,
Here's my attempt to use PrettyDouble, to get around the boxed/unboxed issue.
Please, note that I also had to update treeviz, in order to define an instance for Num explicitly, so that I could overload fromInteger.
-db

Sorry, forgot to show you the error I'm currently getting:
davids-air-2:test dbanas$ stack exec make
time hermit TreeTest.hs -v0 -opt=LambdaCCC.Monomorphize DoTree.hss resume && ./TreeTest
[starting HERMIT v<IP_ADDRESS> on TreeTest.hs]
% ghc TreeTest.hs -fforce-recomp -O2 -dcore-lint -fsimple-list-literals -fexpose-all-unfoldings -    fplugin=LambdaCCC.Monomorphize -fplugin-opt=LambdaCCC.Monomorphize:-v0 -fplugin-    opt=LambdaCCC.Monomorphize:DoTree.hss -fplugin-opt=LambdaCCC.Monomorphize:resume -fplugin-opt=LambdaCCC.Monomorphize:*: -v0

real	0m23.013s
user	0m21.251s
sys	0m1.195s
TreeTest: reifyE: not handled: GHC.Num.*
  @ (Data.Complex.Complex Data.Newtypes.PrettyDouble.PrettyDouble)
make: *** [doit] Error 1

Cheers,
-db


Unable to find write path

 >fac -v
Traceback (most recent call last):
  File "/usr/bin/fac", line 11, in <module>
    sys.exit(main())
  File "/usr/lib/python3.5/site-packages/fac/main.py", line 44, in main
    manager = ModManager(api=api, config=config)
  File "/usr/lib/python3.5/site-packages/fac/mods.py", line 296, in __init__
    self.config.mods_path,
  File "/usr/lib/python3.5/site-packages/fac/files.py", line 178, in mods_path
    return os.path.join(self.factorio_write_path, 'mods')
  File "/usr/lib/python3.5/site-packages/fac/files.py", line 143, in factorio_write_path
    self.config_file
Exception: Can not find the factorio write path.
Please set the write-path variable in /home/steam/.config/fac/config.ini

> cat ~/.config/fac/config.ini
[paths]
data-path = /home/steam/factorio/data
write-path = /home/steam/factorio

> pwd
/home/steam/factorio

Make sure Factorio was started once so that the write files are created (you should have a mods directory in ~/.factorio)
In your case you normally don't need to edit the config.ini file since ~/.factorio is in the standard path list

Been using factorio for a while. It does run fine from /home/steam/factorio once the config was gone. Just an issue when parsing the config file.

I just tried locally and had the error so there is definitely something wrong here, I'll look into it.

I just did a 0.6 release, this should fix you original issue and your paths shouldn't be ignored from the config file anymore.


Hover editor stops Tag Wrangler hover over tags

With the Tag Wrangler plugin you can create a page with an alias equal to a tag.  Then when you hover over the tag the page with that alias appears in the hover window.  However this doesn't work if the Hover Editor plugin is active.  Is there any way that Hover Editor could work with Tag Wrangler to keep this functionality.
Thanks.

I'm not able to reproduce this problem; what are you hovering over, precisely?  Source mode?  Live Preview?  Reading mode?  Legacy editor?  The tag pane?  Something else altogether?

I've also just tried to repeat the issue myself and can't.  I've no idea what I was doing wrong the first time.  Apologies for wasting your time.  Loving the plugin though.  Many thanks.


Unable to build images on arm64

I have a task definition like
platform: linux
image_resource:
  type: registry-image
  source:
    repository: concourse/oci-build-task
inputs:
  - name: repo
outputs:
  - name: my-image
run:
  path: build
params:
  CONTEXT: repo

When I run it on my M1 Mac, when it gets to the first RUN command in the Dockerfile I get the error:
#7 0.260 container_linux.go:380: starting container process caused: process_linux.go:393: copying bootstrap data to pipe caused: write init-p: broken pipe

When I run it in a linux/amd64 EC2 instance, the same task completes successfully. I'm using the docker-compose from the QuickStart, but with the concourse image changed to rdclda/concourse:7.8.2 since concourse/concourse won't run on my M1.
The full file is:
version: '3'

services:
  concourse-db:
    image: postgres
    environment:
      POSTGRES_DB: concourse
      POSTGRES_PASSWORD: concourse_pass
      POSTGRES_USER: concourse_user
      PGDATA: /database

  concourse:
    image: rdclda/concourse:7.8.2
    command: quickstart
    privileged: true
    depends_on: [ concourse-db ]
    ports: [ "8080:8080" ]
    environment:
      CONCOURSE_POSTGRES_HOST: concourse-db
      CONCOURSE_POSTGRES_USER: concourse_user
      CONCOURSE_POSTGRES_PASSWORD: concourse_pass
      CONCOURSE_POSTGRES_DATABASE: concourse
      CONCOURSE_EXTERNAL_URL: http://localhost:8080
      CONCOURSE_ADD_LOCAL_USER: test:test
      CONCOURSE_MAIN_TEAM_LOCAL_USER: test
      # instead of relying on the default "detect"
      CONCOURSE_WORKER_BAGGAGECLAIM_DRIVER: overlay
      CONCOURSE_CLIENT_SECRET: Y29uY291cnNlLXdlYgo=
      CONCOURSE_TSA_CLIENT_SECRET: Y29uY291cnNlLXdvcmtlcgo=
      CONCOURSE_X_FRAME_OPTIONS: allow
      CONCOURSE_CONTENT_SECURITY_POLICY: "*"
      CONCOURSE_CLUSTER_NAME: tutorial
      CONCOURSE_WORKER_CONTAINERD_DNS_SERVER: "<IP_ADDRESS>"
      CONCOURSE_WORKER_RUNTIME: "containerd"

Hi there @palmsey
I'm running into a similar issue, so I was wondering if you ever got this to work and kept a working config somewhere ?
Thanks in advance!

Sorry, I didn't find a solution for this and eventually switched to GH Actions.

thanks for the prompt reply @palmsey !
For What It's Worth (FWIW), I did eventually manage to get this going.
More details on https://github.com/concourse/concourse/issues/8719#issuecomment-1676821087

Curious if you can elaborate on how you fixed this issue, @gsaslis -- I am getting past initial mount issues by using a concourse arm64 image as you suggested, but when I attempt to use the oci_build_task image to build an image it fails with a similar "runc run failed: unable to start container process: can't copy bootstrap data to pipe: write init-p: broken pipe".
At the moment my hypothesis is that I also need an arm64 oci_build_task.
It's not clear if "I did eventually manage to get this going" you mean "concourse runs on an m1" OR "concourse runs on an M1 and can build images with the oci-build-task image" -- the latter is what I'm trying to do.


Reduce number of errors when can't get metadata

Closes #1136
How was it fixed? 🎯

catch the exception
implements a custom backoff retry
write a log message only when all retries were consumed
the task finish always successfully

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 10 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 87.716%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

safe_transaction_service/history/tasks.py
5
10
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
6034

Relevant Lines:
6879

💛  - Coveralls


Broken Liquid highlighting?

Describe the bug
It looks like the liquid markdown syntax is not turning the code into tokens. That is, I'd expect the following markdown to output something highlighted.
 | ```liquid
 | {% for product in all_products %}
 |   <div>{{ product.featured_image }}</div>
 | {% endfor %}
 | ```

It's working in lightshow. But not on here:
{% for product in all_products %}

{{ product.featured_image }}

{% endfor %}

Which shows up as:

{% for product in all_products %}
  <div>{{ product.featured_image }}</div>
{% endfor %}

Expected behaviour
I'd expect the syntax to be highlighted the same way it is in lightshow. Something like:

{% for product in all_products %}
  <div>{{ product.featured_image }}</div>
{% endfor %}

Ooof, this is happening because the grammar compiler is picking up the YAML format of the grammar which appears to be incomplete and doesn't highlight correctly as can be seen in lightshow
This is the result of the compiler running on that file:
{
  "name": "Liquid HTML",
  "scopeName": "text.html.liquid",
  "patterns": [
    {
      "include": "#core"
    }
  ],
  "injections": {
    "L:meta.embedded.block.js, L:meta.embedded.block.css, L:meta.embedded.block.html, L:string.quoted": {
      "patterns": [
        {
          "include": "#injection"
        }
      ]
    }
  }
}

Whilst the repo has a JSON representation of the grammar, it's not in the location expected for JSON grammar files so is being ignored:
https://github.com/github/linguist/blob/6b02d3bd769d07d1fdc661ba9e37aad0fd70e2ff/tools/grammars/compiler/loader.go#L124-L125

Ha! Yeah we're reusing the repository of rules in two files. And the repository is processed because some of the regexes are too insane to be spelled out.
I think I'll just make them .yaml files and avoid the problem. IIRC I only used that extension because of we were heavily inspired by https://github.com/microsoft/TypeScript-TmLanguage. Didn't know it was a standard :see_no_evil:

Oh right. Non-standard location.

We'll have to wait for the next release for this to get picked up, eh?

We'll have to wait for the next release for this to get picked up, eh?

Yup

Liquid highlighting is working again. I manually updated the compiled grammars used by our syntax highlighting service so we could get it working again without having to rush a new release.
I'm going to be making a 7.19.1 update to fix https://github.com/github/linguist/issues/5808 and will pull in an update to the Linguist grammar so everything is in sync across the board.


allow + render markdown between lines

currently a timediff block is rendered as just raw text
i'd like to include checkboxes/bullet lists between the items
like
`` `timediff

10:00  - 11:00
* [some-meeting](link torecording)
* [workitem 1234](jira card)

13:00 -15:00
* [ ] todo item
* [x] even more
`` `

I think the easiest way would be to reformat your makrdown as below:
` ``timediff
10:00 - 11:00
` ``
* [some-meeting](link torecording)
* [workitem 1234](jira card)

` ``timediff
13:00 - 15:00
` ``
* [ ] todo item
* [x] even more

wdyt?

Then each block would get a own total
So not fun
I wonder if there is any other way to do this

@RonnyPfannschmidt I don't think it's possible while using ```  syntax as it parses whole content inside as code. I'm open to suggestions how this could be implemented though.

@DanMundy do you have suggestion where to display it? Maybe in status bar? https://marcus.se.net/obsidian-plugin-docs/user-interface/status-bar

Closing as stale, please reopen if necessary


Add support for processing multiple paths

As of v0.3.1, only one path is supported at a time.
This should probably go in before work on #31 commences.

A few semi-random thoughts:

loop over a slice of locations?
add in support for using goroutines?

write so that one goroutine per location could be used, but lock the logic to just one goroutine by default unless overridden by configuration?

loop over a slice of locations?

This is the direction I'm currently going with this. It seems simpler and easier to implement as a "first draft".

add in support for using goroutines?

write so that one goroutine per location could be used, but lock the logic to just one goroutine by default unless overridden by configuration?

Logging would probably need to be tweaked here to carefully indicate where the messages are coming from (e.g., what directory is currently being processed for a log message).
Since the plan is to implement support for cron-like behavior, supporting multiple goroutines may not be needed; due to the semi-frequent cleanup operations and the limited age-based checks, it is doubtful (at this time) that concurrency would be needed to prune all files before the next execution schedule/event occurs.


Training COCO dataset - RuntimeError: CUDA error: unknown error

Traceback (most recent call last):
File "train_net.py", line 134, in
launch(
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/detectron2/engine/launch.py", line 63, in launch
main_func(*args)
File "train_net.py", line 128, in main
return trainer.train()
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/detectron2/engine/defaults.py", line 431, in train
super().train(self.start_iter, self.max_iter)
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/detectron2/engine/train_loop.py", line 134, in train
self.run_step()
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/detectron2/engine/defaults.py", line 441, in run_step
self._trainer.run_step()
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/detectron2/engine/train_loop.py", line 228, in run_step
loss_dict = self.model(data)
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/projects/crowd-e2e-sparse-rcnn/models/detector.py", line 147, in forward
outputs_class, outputs_coord, ctns = self.head(features, proposal_boxes, self.init_proposal_features.weight,
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/projects/crowd-e2e-sparse-rcnn/models/head.py", line 116, in forward
tmp_container = rcnn_head(features, container)
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/e/PHD/BlueberryDenseDetection/Iter-E2EDET/projects/crowd-e2e-sparse-rcnn/models/rcnn_head.py", line 120, in forward
pro_features2 = self.self_attn(pro_features, pro_features, value=pro_features)[0]
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/activation.py", line 1167, in forward
attn_output, attn_output_weights = F.multi_head_attention_forward(
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/functional.py", line 5161, in multi_head_attention_forward
attn_output_weights = softmax(attn_output_weights, dim=-1)
File "/mnt/f/Software/AnacondaWSL2/envs/pytorch/lib/python3.8/site-packages/torch/nn/functional.py", line 1841, in softmax
ret = input.softmax(dim)
RuntimeError: CUDA error: unknown error
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Any suggestion?
Thanks for help.

NUM_PROPOSALS issue, I increase this value for dense detection


Add 'fleet.search' and 'fleet.msearch' APIs

These APIs support all the parameters of search and msearch with the additional wait_for_checkpoints parameter.
Not sure how best to implement these without replication?

Closed in favor of https://github.com/elastic/elasticsearch-specification/pull/1368


Escape test names in events emitted by the JSON formatter

Converts test names to escaped strings before formatting the message to be
emitted. This prevents invalid JSON being generated when the test name contains
text that needs escaped (e.g.: the backslash in a Windows path from a doctest).

Oops, please ignore this! I've pushed a stale working tree. I'll push a working fix in a moment.


DPR2-239: Rename dms module to dms_dpr and rename dms_s3 module to dms.

I'm closing this because the double rename messes up the diff, so I will do it in two separate PRs.


cargo-cache: 0.6.2 -> 0.6.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cargo-cache/versions.
meta.description for cargo-cache is: "Manage cargo cache (${CARGO_HOME}, ~/.cargo/), print sizes of dirs and remove dirs selectively"
meta.homepage for cargo-cache is: "https://github.com/matthiaskrgr/cargo-cache"
meta.changelog for cargo-cache is: ""
Updates performed

Rust version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/yql33rjr9j968qnh1mfdvancz4qbzggi-cargo-cache-0.6.3/bin/cargo-cache passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 0.6.3 with grep in /nix/store/yql33rjr9j968qnh1mfdvancz4qbzggi-cargo-cache-0.6.3

found 0.6.3 in filename of file in /nix/store/yql33rjr9j968qnh1mfdvancz4qbzggi-cargo-cache-0.6.3

directory tree listing: https://gist.github.com/4a137199485a23155641f917bfc4c631

du listing: https://gist.github.com/9d3bb847b297384d66914c0a1e55b04a

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
cargo-cache

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yql33rjr9j968qnh1mfdvancz4qbzggi-cargo-cache-0.6.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-cache https://github.com/r-ryantm/nixpkgs/archive/f7ad37aaefc6c7bc6bdf1e601351c0369678530f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yql33rjr9j968qnh1mfdvancz4qbzggi-cargo-cache-0.6.3
ls -la /nix/store/yql33rjr9j968qnh1mfdvancz4qbzggi-cargo-cache-0.6.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-cache

Maintainer pings
cc @Br1ght0ne for testing.

Result of nixpkgs-review pr 127342 at f7ad37aa run on aarch64-linux 1

1 package built successfully:

cargo-cache

1 suggestion:

error: no-flags-array
checkFlagsArray is only intended to be used in Bash, not as a Nix attribute.
Near pkgs/development/tools/rust/cargo-cache/default.nix:18:3:
   |
18 |   checkFlagsArray = [ "offline_tests" ];
   |   ^

Result of nixpkgs-review pr 127342 run on x86_64-darwin 1

1 package built:

cargo-cache

Nothing major here, merging.
Let me know if you want me to be less forward next time.


fix(material/core): validate theme palettes

Currently if a palette is passed directly into the theme without wrapping it in a define-palette call, we end up throwing a weird null pointer error downstream.
Since we've had a couple of reports of this by now, these changes add a function that validates the palettes and throws a better error if they're invalid.
Relates to #25981.

I ended up relaxing the requirement a bit since it broke a large amount of internal targets. Will try another TGP.


fix(icon): make svg filters work in Safari/Firefox

SVG filters do not work in Safari/FF because of paths in url needing to be updated to current path
This fixes #9276

I signed it!

Fixed by #12428


why rippled sync node lost last synced ledger?

My ripple sync node completed ledger is 46435033-46455809, and then I restart rippled service, the completed ledger is 46455839-46455876. Where are these ledger? How can I restore the lost ledger while I restart rippled service at next time?

They will re-appear in complete_ledgers once you have fetched enough history to cover the gap from 46455839 to 46455809.

Is the gap amount limited? Such as no more than ledger_history value?

https://github.com/ripple/rippled/blob/fa57859477441b60914e6239382c6fba286a0c26/cfg/rippled-example.cfg#L599-L619

thank you!


About that icon...

Hi, I'm the maker of the resource pack that you took the icon from.
The pack (and, while I'm not a legal expert, I would also say the icon as a result) is licensed under LGPL, which requires attribution. If you're going to use it as your mod's icon and/or the modrinth page's icon, please at least link to my resource pack in the description of the mod and webpage.
Or just make a new icon. It's not hard, trust me; I'm not an artist and I made that icon with minimal effort. I don't even care about the icon that much, aside from the fact that it associates your project with mine when I wasn't even asked permission nor credited.

Yes buddy, i would surely link up your resource pack page!
Reply me with the link to it

+1 to this. The icon was what originally got me to look at this mod. Best to go your own route first with texturing

+1 to this. The icon was what originally got me to look at this mod. Best to go your own route first with texturing

Thanks man! Also the attribution link has been updated in the mod description

Thank you for adding attribution. I don't mean to be a jerk about it, just do keep in mind licenses when working with other people's work in the future; it's important, especially in programming. Good luck with development!


NavLink broken

For bugs, please include the following:

What is the expected behavior?
When using NavLink and activeClassName, the navbar should update with the corresponding style

What is the actual behavior?
Navbar does not update unless refreshed

What steps reproduce the behavior?
Click on nonactive item in navbar.

As a user, I should be able to see the page I am on (active navbar tab) and click on another tab to change the active tab.

Probably something silly going on

At the end, a bit of googling shows that the problem was with react-redux connect. Using a pure:false argument as the fourth option fixed it.


Normal model behavior ? or scaling ?

Hi,
I've continued training the Depth-Anything-V2-Large model serial on my custom dataset, and the predicted depth maps appear to be close to the ground truth ones. However, they appear to be scaled for some reason, which is evident from the black and white pixels(see annotations on images). I'm unsure if this is normal behavior from the model itself of if there's some scaling happening in the code.
predicted depth map:

ground truth:

Hi, I've continued training the Depth-Anything-V2-Large model serial on my custom dataset, and the predicted depth maps appear to be close to the ground truth ones. However, they appear to be scaled for some reason, which is evident from the black and white pixels(see annotations on images). I'm unsure if this is normal behavior from the model itself of if there's some scaling happening in the code.
predicted depth map:

ground truth:

Hi @YacineDeghaies ,
I'm currently trying to train metric depth model on my custom dataset. I was able to run the training code, but my predicted depth maps are worse compared to the groundtruth I have.
Could you please share the steps you followed for training on custom dataset.


Modal focus should wait after animation.

When a modal is opened focus to either the modal or the autofocus element is applied after waiting a $digest cycle using modalRender property. That is not enough as the focus happens before/during the animation in class. This could be avoided by using the promise returned from $animateCss or $animate.
I could submit a PR if you thing that is the right approach.

This sounds reasonable - feel free to file a PR for this.


Hide marker genes with marker score <0.5

We have determined that marker genes with scores <0.5 are not very useful to users. To provide more valuable genes, we'd like to implement a cut off where no marker genes with scores <0.5 are surfaced in the marker score panel.
This ticket covers the following work:

[ ] Implement cutoff
[ ] If a cell type does not have any marker genes with scores >= 0.5 Figma here
[ ] Add a line to the documentation explaining this cutoff

old ticket:
https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/gh/chanzuckerberg/single-cell-data-portal/4240

Waiting on analytics: https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/gh/chanzuckerberg/single-cell-data-portal/4259

rdev can be viewed here: https://hide-marker-genes-frontend.rdev.single-cell.czi.technology/gene-expression

@ashin-czi i'm not seeing the "Low: <1" in the marker genes panel:

@signechambers1 this is fixed now in rdev!


Trying to add my own custom_component

Version of HACS
0.2.1
Hassio version
0.93.1
Describe the bug
i read the documentation, and did every thing to be able to add to HACS my custom components. But every time i try to add i get this error:
Could not reload repo 'vitinhosessa/metro_sp' at this time, if the repo meet all requirements try again later.
Debug log
2019-05-22 20:45:07 DEBUG - hacs.frontend.views.api - GET API call for integration_url_reload with vitinhosessa/metro_sp
2019-05-22 20:45:07 DEBUG - hacs.handler.update - Started prosessing vitinhosessa/metro_sp
2019-05-22 20:45:07 DEBUG - hacs.handler.update - Loading from data from GitHub for vitinhosessa/metro_sp
2019-05-22 20:45:08 DEBUG - hacs.handler.update - metro_sp info:  ref: 'tags/0.1b3'  last_update: '21 May 2019 22:46:41'  releases: '['0.1b3', '0.1b2', '0.1b1' -'  last_release: '0.1b3'
2019-05-22 20:45:08 DEBUG - hacs.handler.update - 404 {'message': 'Not Found', 'documentation_url': 'https://developer.github.com/v3/repos/contents/#get-contents'}
2019-05-22 20:45:08 DEBUG - hacs.handler.update - Can't load manifest from vitinhosessa/metro_sp
2019-05-22 20:45:08 DEBUG - hacs.handler.update - Skipping vitinhosessa/metro_sp on next run.
2019-05-22 20:45:08 DEBUG - hacs.frontend.views.settings - Trying to serve settings

My manifest.json
{
  "domain": "metro_sp",
  "name": "Metro São Paulo",
  "documentation": "https://github.com/vitinhosessa/metro_sp/blob/master/README.md",
  "dependencies": [],
  "codeowners": ["@vitinhosessa"],
  "requirements": []
}

Additional Information
I tried to add aftership-card, and i get the same error:
Could not add repo 'https://github.com/custom-cards/aftership-card' at this time, if the repo meet all requirements try again later.
Debug Log for aftership-card
2019-05-22 20:51:07 DEBUG - hacs.frontend.views.api - GET API call for plugin_url with add
2019-05-22 20:51:07 DEBUG - hacs.frontend.views.api - Trying to add https://github.com/custom-cards/aftership-card
2019-05-22 20:51:07 DEBUG - hacs.handler.update - Started prosessing custom-cards/aftership-card
2019-05-22 20:51:07 DEBUG - hacs.handler.update - Loading from data from GitHub for custom-cards/aftership-card
2019-05-22 20:51:08 DEBUG - hacs.handler.update - aftership-card info:  ref: 'tags/1.1.0'  last_update: '04 May 2019 04:33:32'  releases: '['1.1.0', '1.0.0', '0.0.1' -'  last_release: '1.1.0'
2019-05-22 20:51:08 DEBUG - hacs.handler.update - plugin content ['rollup.config.js' -
2019-05-22 20:51:08 DEBUG - hacs.handler.update - Expected file aftership-card.js not found in ['rollup.config.js' - for custom-cards/aftership-card
2019-05-22 20:51:08 DEBUG - hacs.handler.update - Skipping custom-cards/aftership-card on next run.
2019-05-22 20:51:08 DEBUG - hacs.frontend.views.settings - Trying to serve settings

After ship is available by default

It's not showing up for me the aftership-card on store. @iantrich

Does for me. Odd

should i downgrade to 0.2.0 to see if works?

So the store is empty for you by default?

It's not empty. There is some Custom Integrations and Custom Plugins

It's this that appear on store:
CUSTOM INTEGRATIONS
HA Dockermon
Raspberry Pi Power Supply Checker
Untappd
Feedparser
Personal Capital
Trakt
Bravia TV PSK
Sonarr Upcoming Media
Radarr Upcoming Media
OWL Intuition
UniFi Gateway
ssh
Plex Recently Added
Kodi Recently Added Component
Avanza Stock
gPodder
breaking_changes
Grocy
Qbo
CUSTOM PLUGINS (LOVELACE)
text-element
circle-sensor-card
state-attribute-element
vertical-stack-in-card
tracker-card
state-element
canvas-gauge-card
beer-card
ext-weblink
username-element
list-card
rmv-card
upcoming-media-card
surveillance-card
light-entity-row
button-entity-row
flex-table-card
secondaryinfo-entity-row
bar-card
roku-card
pc-card
text-divider-row
bignumber-card
entity-attributes-card
gauge-card
group-card
home-setter
monster-card
plan-coordinates
lovelace-card-tools (i added)
compact-custom-header (i added)

For your component, the structure of your latest release (that is what hacs scan) is not correct https://github.com/vitinhosessa/metro_sp/tree/0.1b3 if you do a release now, you should be able to add it :)
The same goes for aftership-card https://github.com/custom-cards/aftership-card/tree/1.1.0, when that get a new release, reload data from the settings tab, and it will show, or just wait 500min, and it will show :)


Use of onload= causes CSP warnings if unsafe-inline is not allowed by the content-security-policy.

The code at https://github.com/trvswgnr/bs5-lightbox/blob/main/src/index.ts#L137 causes errors in the browser:

Refused to execute inline event handler because it violates the following Content Security Policy directive: "script-src 'self'". Either the 'unsafe-inline' keyword, a hash ('sha256-...'), or a nonce ('nonce-...') is required to enable inline execution. Note that hashes do not apply to event handlers, style attributes and javascript: navigations unless the 'unsafe-hashes' keyword is present.

I'm not sure of the purpose of that onload code and if it's actually needed (removing it from my local copy doesn't appear to have changed much?), but it would be better to do this onload script as an event handler on the image rather than using onload.

The same line of code also looks to cause an error (#4) if it actually runs - so I suspect it's not needed at all?

This has been resolved in v1.7.10. I believe that code was removing a loading indicator, but at this point I'm not sure. Thanks for bringing it to my attention.


User Stats and Profile Page

Create a user profile page to display game statistics and achievements.

I thought it might make sense to start with some dummy data that's in the same form as what will be got back from the server?
In the model, I have the stats set up as a subdocument.
So, it's like the following.  The stats objects won't be in any particular order, and might not include all the difficulties, because it just gets created/updated when a user completes a game
profile = {
  name: "A_Cat",
  email<EMAIL_ADDRESS>  password: "test12345",
  games: [ /* game object id's */ ],
  stats: [
    {
      difficulty: "easy",
      bestTime: 120.5,
      averageTime: 295.4,
      numSolved: 12
    },
    {
      difficulty: "medium",
      // ...
    },
    // ...
  ]
}

Do you think it makes sense to have this be the same section where the user can resume a game?  So there'd be like a list of buttons or something which show the difficulty and timeElapsed of that game.  Then the event functions for those would re-direct to the main page, and set the state for the new game.   I think the point of React is that it can redirect and set the state at the same time.  Though GameProvider needs to be moved to wrap around everything.
In that case, the profile object above would be the same, but games can populate with game objects like this: { _id: string, gameData: string, difficulty: string, elapsedTime: number }.
Also I did get distracted an I made an svg component that takes the gameData and creates a little image that shows which cells are filled in, so you could know if you're close to being complete.

 component code
I have not tested this yet
function GameSvg({ gameData , color }) {
  const gameArray = JSON.parse(gameData);

  const svgWidth = 90;
  const svgHeight = 90;

  const lines = [
    [[0, 30], [90, 30]],
    [[0, 60], [90, 60]],
    [[30, 0], [30, 90]],
    [[60, 0], [60, 90]]
  ];

  return (
    <svg width={svgWidth} height={svgHeight}>
      {/* Rectangle outlining the SVG */}
      <rect x="0" y="0" width={svgWidth} height={svgHeight} fill="none" stroke={color} />

      {/* Vertical and horizontal lines */}
      {lines.map(([[x1, y1], [x2, y2]], index) => (
        <line key={index} x1={x1} y1={y1} x2={x2} y2={y2} stroke={color} />
      ))}

      {/* Squares based on array values */}
      {gameArray.map((row, rowIndex) =>
        row.map((cell, colIndex) =>
          cell.value ? (
            <rect
              key={`${rowIndex}-${colIndex}`}
              x={10 * rowIndex}
              y={10 * colIndex}
              width="10"
              height="10"
              fill={color}
            />
          ) : null
        )
      )}
    </svg>
  );
}


Fixed 404-Error inside MessageDetail.vue

The link inside MessageDetail.vue currently leads to an 404-Error because the URL-Structure changed.

I created upstream issue — the URL shouldn't be broken.


Adding the SNe blinding example

We need to validate if this example is actually useful for Supernovae Type Ia....

⚠  Wait for #40 to be merged before! ⚠


Admin panel: add country and region filters to pending users validation view

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

The number of pending users is quite long (hundreds of users).
Persons responsible for validating are regional office IM staff, who want to view only the pending users from their region.

Describe the solution you'd like
I'd like to have a filter at the right side of the admin panel in the pending users view that allows me to filter pending users by region and NS, just like I can currently do in the list of validated users.
Both filters have value and shouldn't be seen as alternative solutions: a) region filter is useful when I want to review and approve any pending user from my region, and b) country filter is useful if, e.g., I arrange training to a specific NS, I'd like to filter by country, so that I can then easily validate all pending users from that specific country.
Filters in users view:

Filters in pending users view:

The arriving solution:

Deployed to Staging and Prod.


NPE in QueryService on attempt to query wrong state class

I changed a state of a test Process Manager and forgot to change the state class I query. It resulted in NPE in QueryService in these lines of the read() method:
        BoundedContext boundedContext = typeToContextMap.get(type);
        Stand stand = boundedContext.getStand();

There is no any boundedContext matching the type, so null is returned. We need to return an error describing the situation and steps a programmer can make to rectify the situation.

This issue is a duplicate of #687.


NH-24785 - Updated gRPC with CMAKE_CXX_STANDARD 17

liboboe updated to use c++17.
This PR is to update the artifact hash so the artifact is built using C++17.
cd build_windows_32
cmake -G "Visual Studio 17 2022" -A Win32 -DCMAKE_CXX_STANDARD=17 ../
cmake --build . --config Release -j 2
cmake --build . --config Debug -j 2

cd build_windows_64
cmake -G "Visual Studio 17 2022" -A x64 -DCMAKE_CXX_STANDARD=17 ../
cmake --build . --config Release -j 2
cmake --build . --config Debug -j 2

NH-24785

Default compiler and linker flags generated by gRPC's cmake can't be used to statically link with liboboe. As a result, they must be updated manually for all project files to match existing liboboe flags.
Building of gRPC library must be done using the Visual Studio solution and project files. The projects have been updated with compiler and linker flags required to statically link with liboboe.
To build with C++17 the following setting must be set to all gRPC project files (for all configurations 32/64 and Debug/Release):

Changes moved to PR: https://github.com/librato/grpc/pull/8


Add regression tests for removal of scalac/fastparse dependency for cached scripts

As part of 0.7.0, I ripped out the dependency on fastparse and scala.tools.nsc for scripts that are already cached; this saves us up 100s of ms of classloading time, greatly improving startup time.
Unfortunately, this improvement is not tested, and it would be easy for someone to accidentally re-introduce either fastparse or scalac into the critical path of cached scripts, regressing our timing.
However, now we have the ability to pass JAVA_OPTS to scripts, we can now pass in -verbose:class to the amm executable when running a script to see what classes it is loading. We can thus now write a regression test to verify that the first time we run an uncached script, we do load fastparse and scala.tools.nsc, and that for subsequent times, we don't. That would ensure someone adding an accidental dependency to the critical path will result in a red build

Closed by #435, thanks to @coderabhishek !


LTSA Demo credential OCA bundle added

LTSA Demo credential OCA bundle created by Northern Block.

Hi @arbordeleau — thanks for putting in the PR.  The proper flow in this repo is that the Excel file that is the source of the OCA data is included in the checkin, so that we can use that to generate/verify the contents of the OCA Bundle, and so that the identifiers in the Overlays are calculated.

Have you created an Excel file for this batch?

If so, please include it in the PR so that we can verify the bundle was properly generated.
If not, please create one as documented here

I now see that the documentation site says that the OCA Excel file is optional, but that should not be the case, and I’ll update. I apologies if that is why you didn’t create/include the Excel file.

I also see that the DCO did not get used.  This repo requires that all commits are signed with DCO.
Please see DCO - Developer Certificate of Origin - https://github.com/apps/dco. To fix, see: https://github.com/src-d/guide/blob/master/developer-community/fix-DCO.md and the “details” beside the failed check above.

I have used DCO to signed the commit. I also added the missing excel file.
Thank you!

DCO looks good, but I’m still not seeing the Excel file.
I also don’t think the location of the folder is right.  At minimum, “ltsa-demo” should be “LTSA”, but I also think it should be under the “BCGov” folder — I’m checking the @krobinsonca on that.  I don’t know what “NewCred” means, but the folder title should be named for the type of credential it will be in production.  Below that, you could have a folder “demo” that is used for this early version.
So: schema/bcgov-digital-trust/LTSA/NewCred/demo/
I’m trying to learn more from Kyle to get a better view on the title and location of the folder.

Sorry, the Excel file was supposed to be included. I have also updated the folder structure as per your recommendation, thank you! Additionally, 'NewCred Credential' is the credential name LTSA has chosen for the demo, so I understand why it may have seemed unclear.

Looking good, but not quite there yet.
First, the OCABundle.json file needs to be generated from the Excel. I generated it, and it is quite different from what you have in the PR.  To generate it, you have to have the software to run the script, which is a bit of pain.  We’re working on making this a “post-PR merge” step, but BC Wallet doesn’t yet support that, so it has to be done as part of the merge. The script is to be run from is scripts/genBundle.sh in the repo, run from the demo folder with arguments -x OCA.xlsx branding.json.  It’s a pain...
Second, another script needs to be run from the root of the repo ./scripts/gen_ocabundlesjson.sh. It generates a couple of other files in the root.
If that is not easy for you to set up, I would suggest that you remove the OCABundle.json from the demo folder, and then I’ll do a PR to put the two steps above.  Let me know — happy to do that.
Sorry this is such a pain.

Thanks for the clarification!
I'll gladly take your help with running the necessary scripts. I've removed the OCABundle.json file from the demo folder and pushed the changes.
Thanks again for assisting with this! Looking forward to your PR with the steps.

Thank you for the quick response and push! I was testing on the BC Wallet, and I noticed that the issuer logo is missing. Could you please confirm if this is something on my end or if it’s related to the recent changes? Appreciate your help in clarifying. Do i needed to add it on the excel file as well? I remember only included it in the branding.json and in png in the folder /demo.

Ah…I see.  The path in the branding.json file did not get changed when the folder location was changed.  I’ll update...


fix: keep stashBuffer when exception throw again #669

References #669
after investigate, the reason why message log is Supervisor clear stash immediately(L401) while actor restart, no matter the new actor failure again or not.
It is no issue on classic BackoffOnRestartSupervisor, because they don't have StashBuffer.
https://github.com/apache/incubator-pekko/blob/e94e7b971b74eeadc05a5b3dc28d1e17c434ad7f/actor-typed/src/main/scala/org/apache/pekko/actor/typed/internal/Supervision.scala#L397-L411

@He-Pin I am sorry for that closed this PR for no reason, this PR is not only my contribution.


Feature: Add "fromClaim" functionality to graphql upstream headers builder

Problem
I have cookie-based auth configured and use a Hasura graphql data source, and authorization in Hasura from a backend server on behalf of a user relies on a few other special headers attached to the request. This isn't an issue if Hasura is also the authentication provider, but I'm using WG for Oauth2 so Hasura doesn't have the config to parse the auth cookie.
This means that correctly accessing things in a Hasura request with authorization needs the admin secret to WG can make the request, and at minimum the x-hasura-role and x-hasura-user-id header has to be set. I have a couple of extra headers that are specific to my application permissions schema (x-hasura-realm-id, x-hasura-character-id) as well.
So, if I have a graphql operation I call from the client it fails because the request is missing the client-specific upstream headers. That can insecurely be worked around in the UI by calling client.setExtraHeaders and populating those values from the cookie custom claims and adding builder.addClientRequestHeader("x-hasura-role", "x-hasura-role"), ... but that means any client could set the request headers to whatever they want.
Suggested solution
Expand the graphql upstream header builder to include a fromClaim feature that can populate a header from a claim. Example implementation code:
builder.addClientRequestHeaderFromClaim(upstreamRequestHeaderName, claimName)

headers: (builder) => {
    builder.addStaticHeader("x-hasura-admin-secret", new EnvironmentVariable("HASURA_GRAPHQL_ADMIN_SECRET"))
    builder.addClientRequestHeaderFromClaim("x-rasura-realm-id", "realmId")
    builder.addClientRequestHeaderFromClaim("x-hasura-character-id", "charId")
    builder.addClientRequestHeaderFromClaim("x-hasura-user-Id", "userId")
    builder.addClientRequestHeaderFromClaim("x-hasura-role", "role")
    return builder
}

configureWunderGraphApplication({
  ...
      authentication: {
        customClaims: {
            role: {
                jsonPath: "hasura.role",
                type: "string"
            },
            userId: {
                jsonPath: "hasura.userId",
                type: "string"
            },
            charId: {
                jsonPath: "hasura.charId",
                type: "string"
            },
            realmId: {
                jsonPath: "hasura.realmId",
                type: "string"
            }
        }
        ...
      }
  ...
})

Alternatives
Maybe this is possible with operation hook pre-resolve to inject the headers? But that would be required for every single client UI operation and that's a lot of work.

There is a global hook where you can decrypt the cookie and add headers you want to forward to Hasura.
This example has a global hook for subscriptions but API exposes an http protocol as well
https://github.com/wundergraph/wundergraph/tree/main/examples/graphql-subscriptions-hooks

Thanks for the pointer, it turns out the onConnectionInit hook doesn't let you change the header for the WS connections (I think that's what I saw at least), changing the httpTransport.onOriginRequest hook for all operations did however.
That may work as a workaround for now, but I think adding a specific builder API would be a better solution overall.

/bounty 100

Still open.

@AesSedai I've looked into this, while I really like the idea of addHeadersFromClaim, it will be very hard to implement, because the graphql engine isn't aware of the user claims.The only way is to forward them as headers using hooks.
You can return the headers in onConnectionInit:
onConnectionInit({user}) {
   return {
      "x-user-id": user.userId
   }
}


PYIC-3078 Refactor VerifiableCredentialJwtValidator.validate for CIMT VC's

Proposed changes
Refactor VerifiableCredentialJwtValidator.validate method for CIMIT VC validation.
What changed
Refactored VerifiableCredentialJwtValidator.validate.

Why did it change

Issue tracking

PYIC-3078

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

[ ] Documented in the README
[ ] Added to deployment repository
[ ] Added to local startup repository

Other considerations

[ ] Update README with any new instructions or tasks

Removed commented code.


Issues with libcronet HTTP1.0

It seems that libcronet supports HTTP2, but in my actual testing, I found that when I set SetEnableHTTP2 to false, Wireshark detected that libcronet was unable to send requests in HTTP1.0. Do you have any solutions?
https://github.com/binarycraft007/naiveproxy/blob/master/src/components/cronet/native/generated/cronet.idl_impl_struct.cc#L271-L275
void Cronet_EngineParams_enable_http2_set(Cronet_EngineParamsPtr self,
                                          const bool enable_http2) {
  DCHECK(self);
  self->enable_http2 = enable_http2;
}

It seems that libcronet supports HTTP2, but in my actual testing, I found that when I set SetEnableHTTP2 to false, Wireshark detected that libcronet was unable to send requests in HTTP1.0. Do you have any solutions?
https://github.com/binarycraft007/naiveproxy/blob/master/src/components/cronet/native/generated/cronet.idl_impl_struct.cc#L271-L275
void Cronet_EngineParams_enable_http2_set(Cronet_EngineParamsPtr self,
                                          const bool enable_http2) {
  DCHECK(self);
  self->enable_http2 = enable_http2;
}

made an example in zig, http2 is working, but if I turn off http2, it seems that I am getting errors from cronet:
With http2:
❯ zig-out/bin/cronet https://www.google.com
info: on_stream_ready_callback
info: on_response_headers_received, negotiated_protocol=h2
info: :status: 302
info: location: https://www.google.com.hk/url?sa=p&hl=zh-CN&pref=hkredirect&pval=yes&q=https://www.google.com.hk/&ust=1678176669813955&usg=AOvVaw3UKIjLFXVbiVOM6Vwy1yJI
info: cache-control: private
info: content-type: text/html; charset=UTF-8
info: p3p: CP="This is not a P3P policy! See g.co/p3phelp for more info."
info: date: Tue, 07 Mar 2023 08:10:39 GMT
info: server: gws
info: content-length: 372
info: x-xss-protection: 0
info: x-frame-options: SAMEORIGIN
info: set-cookie: 1P_JAR=2023-03-07-08; expires=Thu, 06-Apr-2023 08:10:39 GMT; path=/; domain=.google.com; Secure
info: set-cookie: AEC=ARSKqsLPDfjc_8hvNqjv8jdQI6FFflKa6EgIDbxEvi0HW3OJT0qbIpUlcg; expires=Sun, 03-Sep-2023 08:10:39 GMT; path=/; domain=.google.com; Secure; HttpOnly; SameSite=lax
info: set-cookie: NID=511=h2Lwe0auofyl8DaKJYpf2n0AisRi55W0b37NMvX-Pr8Sas7XsZDJzVIG-VkWJfD-HWwqmeVroF0rk0mqH6DjkwceSlgp8Azxznc4E9IcAgP1cRtZiSsYvupvMn65LPJJ4ILMA3AsGP2G0Ipk34diWOCPzyc3R70VadqYRj5YQWg; expires=Wed, 06-Sep-2023 08:10:39 GMT; path=/; domain=.google.com; HttpOnly
info: alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
info: on_read_completed
info: <meta http-equiv="content-type" content="text/html;charset=utf-8">
<TITLE>302 Moved</TITLE>

302 Moved

The document has moved
here.

info: on_read_completed
info: <meta http-equiv="content-type" content="text/html;charset=utf-8">
<TITLE>302 Moved</TITLE>

302 Moved

The document has moved
here.

info: on_succeded

Without http2:
❯ zig-out/bin/cronet https://www.google.com
info: on_failed
info: net error: -2

Only supporting HTTP2 means no CDN

I see you have a cronet-zig project, unfortunately I can't zig. How to compile?

If you want to integrate cronet into zig, you may have to consider http1.0, after all ws. Of course, http2 can also request ws, but cdn is not supported.

I see you have a cronet-zig project, unfortunately I can't zig. How to compile?

download latest release from here https://ziglang.org/builds/zig-linux-x86_64-0.11.0-dev.1836+28364166e.tar.xz,
add path of zig/lib/zig.exe to $PATH, and run zig build, you will get the executable in zig-out/bin

This may require changing the code of chromium to adapt to the request of http1.0, and then compile the cronet.

I see you have a cronet-zig project, unfortunately I can't zig. How to compile?

download latest release from here https://ziglang.org/builds/zig-linux-x86_64-0.11.0-dev.1836+28364166e.tar.xz, add path of zig/lib/zig.exe to $PATH, and run zig build, you will get the executable in zig-out/bin

ok let me try

If I'm going to study it, I have to study zig. After all, my main force is C.

zig can actually export c api, and generate c obj

If I'm going to study it, I have to study zig. After all, my main force is C.

zig can actually export c api, and generate c obj

OK, I'll try it.

I have quickly built your project and it is running successfully. Same as your result, only http2 is supported, if you need to initiate http1.0, you need to change the code of chromium.
root@Ubuntu20:~/cronet-zig# ./zig-out/bin/cronet https://www.google.com
[0307/163537.198852:ERROR:cert_verify_proc_builtin.cc(603)] No net_fetcher for performing AIA chasing.
info: on_stream_ready_callback
info: on_response_headers_received, negotiated_protocol=h2
info: :status: 200
info: date: Tue, 07 Mar 2023 08:35:37 GMT
info: expires: -1
info: cache-control: private, max-age=0
info: content-type: text/html; charset=ISO-8859-1
info: p3p: CP="This is not a P3P policy! See g.co/p3phelp for more info."
info: server: gws
info: x-xss-protection: 0
info: x-frame-options: SAMEORIGIN
info: set-cookie: 1P_JAR=2023-03-07-08; expires=Thu, 06-Apr-2023 08:35:37 GMT; path=/; domain=.google.com; Secure
info: set-cookie: AEC=ARSKqsKZgGH-Pnin3gtucCo4jAg45f_zuOGBg1DQDLbEU59Yg9ADrAkAYBM; expires=Sun, 03-Sep-2023 08:35:37 GMT; path=/; domain=.google.com; Secure; HttpOnly; SameSite=lax

I see you have a cronet-zig project, unfortunately I can't zig. How to compile?

download latest release from here https://ziglang.org/builds/zig-linux-x86_64-0.11.0-dev.1836+28364166e.tar.xz, add path of zig/lib/zig.exe to $PATH, and run zig build, you will get the executable in zig-out/bin

ok let me try

ok, you may also need libcronet.so and its headers

It may be modified under net/spdy of chromium to support http1.0.

spdy

Yes, if you use libcronet, that means only http2 or quic. After all, libcronet is built under the spdy package

I see you have a cronet-zig project, unfortunately I can't zig. How to compile?

download latest release from here https://ziglang.org/builds/zig-linux-x86_64-0.11.0-dev.1836+28364166e.tar.xz, add path of zig/lib/zig.exe to $PATH, and run zig build, you will get the executable in zig-out/bin

ok let me try

ok, you may also need libcronet.so and its headers

It may be modified under net/spdy of chromium to support http1.0.

there is two direction for a tls fingerprinting resistant proxy, one is use chromium network stack, another is to modify , clienthello message of tls library. I am trying out both


Team - Manage Team - Channels - List of chanels - add screen and explanation to page

To better understand channels and how a private channel looks like it would be helpfull to add a screenshot and explain the possible values of each colmun - especially the column Type.

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

ID: ce34b12b-aa50-608b-bbfa-a1fb4bb79f8d
Version Independent ID: 9ddeb6f4-f30a-1e72-fc94-ae66def91f40
Content: Private channels in Microsoft Teams - Microsoft Teams
Content Source: Teams/private-channels.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@MarcGab Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

May be i misunderstood your reply. I am aware that i have to talk to my Admin to enable private channels. This issue here is meant to update the documentation so that it is clear that Teams does support private Channels and how to create them.
Currently you have these headlines (among others)

What you need to know about private channels
When to create a private channel

Please add a headline "How to create a private channel"