changes to help bindings

Hey, just wanted to mention that @mafintosh and I forked this recently to write some node bindings: https://github.com/maxogden/rabin/tree/master/src
We made some significant changes to the API to remove externs and make it memory safe across multiple instances, as well as moving most of the DEFINEs onto a struct so that they could be configurable at runtime. We didn't really touch the actual fingerprinting logic.
Relevant commits:

https://github.com/maxogden/rabin/commit/2bceea33322120573e403d498a3a300492fcaf39
https://github.com/maxogden/rabin/commit/8461d2f7c7685044649b2288bbcee0978d65995b
https://github.com/maxogden/rabin/commit/425ba62a8cce4ef54afc6e4925659b1d7ea751fb

If you're interested in these I'd be happy to put together a PR and we could help maintain this with you.

Hey, thanks for contacting me! I'd be delighted if you could create a PR and we can also maintain it together.
Did you measure the performance impact of the define-to-struct change?
What do you use the chunker for in node?
Please be aware that for security-critical things there exists a fingerprinting attack that could help attackers guess what data hsa been processed by the chunker just by observing the size of the resulting chunks. This was a feasibility study for my other project restic to see what throughput can be achieved with C code. For restic, I've added a generator to find custom polynomials to mitigate the fingerprinting attack. Are you also interested in that code?

Did you measure the performance impact of the define-to-struct change?

Not yet, will do

What do you use the chunker for in node?

We work on an open source data sharing tool called Dat (http://dat-data.com), we're currently implementing a sort of distributed filesystem for it. The use case for this chunker for us is very straightforward, to generate smaller deltas between two versions of a file.

security-critical things

Thanks for mentioning, I noticed the polynomial generation in your Go implementation but wasn't sure why it was there, this makes sense now. We will only be working with public scientific datasets for a while so this isn't an issue for us now. However encrypted data is on our longer term roadmap.
Also, I noticed you contribute to HackRF, thanks for that. We were at CCC this summer and got the rad1o badges and had some fun writing hackrf bindings for node: https://www.npmjs.com/package/hackrf. I just got a ANT500 last week for it actually, I'm interested in messing around with WSPR and other packet radio use cases.

Hey, any news on the proposed PR?

Sorry have been trying to get a release out of our project! We got some good feedback on an API improvement recently: https://github.com/maxogden/rabin/issues/4 Let me open an issue in that repo pointing here so we are less likely to forget


Added an example of multithreaded program

Added an example of multithreaded program that uses thread local storage key.

Thanks a lot!
cc @gkoz

:+1:


fix: Passing chat metadata to Pipe functions as an extra param

Pull Request Checklist

[X] Target branch: Please verify that the pull request targets the dev branch.
[X] Description: Provide a concise description of the changes made in this pull request.
[X] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[X] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[X] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[X] Testing: Have you written and run sufficient tests for validating the changes?
[X] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[X] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description
Added a capability to pass the "metadata" object (chat_id, message_id, session_id, tools) to Pipe functions as an extra param if it wants to accept it. Metadata can be useful to functions for logging purposes. Without this change, chat metadata is not available to Pipe functions.
Added
Changed
Passing "metadata" dict to Pipe functions as an optional extra param, if the pipe() function signature has it.
Previously, this param was not passed and was not available to Pipe functions.
Deprecated
Removed
Fixed
Security
Breaking Changes
Additional Information
Not sure if this should be considered a FEAT? Seems too small of a change for a feature. And it is clearly backwards compatible.

Hi @tjbck since you closed without merging, can you provide a brief comment what's wrong with the PR? Thanks!

Hello, can we have an example of how to use it for this?


Etcd2 watcher (sometimes) keep getting old events

I have run into some hard to debug problems with etcd. A couple of times a day some of our services which are using etcd will go into a almost infinite loop.
I have traced it down to a watcher that goes wild .. as it goes wild, it will repeatedly get old events. So much that both the server and the client will eat all CPU available to them. The weird thing is that after a while the problem seems to go away.
I have been able to reproduce this on a test machine (single node etcd cluster, running on coreos 1122.3.0). Before any of our services started I ran an etcdctl watcher in the background so I was able to capture the output of the watcher.
etcdctl --debug -o json  watch --recursive --forever /ha
After all of the services were stopped again I still kept getting events from the etcdctl watch command:
[set] /ha/prometheus/services/1
{"action":"set","node":{"key":"/ha/prometheus/services/1","value":"1","nodes":null,"createdIndex":164124,"modifiedIndex":164124},"prevNode":{"key":"/ha/prometheus/services/1","value":"1","nodes":null,"createdIndex":163241,"modifiedIndex":163241}}
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=164125
[set] /ha/prometheus/services/1
{"action":"set","node":{"key":"/ha/prometheus/services/1","value":"1","nodes":null,"createdIndex":164124,"modifiedIndex":164124},"prevNode":{"key":"/ha/prometheus/services/1","value":"1","nodes":null,"createdIndex":163241,"modifiedIndex":163241}}
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=164125

This kept repeating for a long long time ..
After stopping etcdctl and restarting the command it wouldn't give me any event.
But with the after-index parameter I was able to reproduce it afterwards (only for a short time)
etcdctl --debug -o json  watch --recursive --after-index "163822" --forever /ha > watch.txt
start to sync cluster using endpoints(http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001)
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/members
got endpoints(http://<IP_ADDRESS>:2379) after sync
Cluster-Endpoints: http://<IP_ADDRESS>:2379
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=163823
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=163211
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=164125
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=163836
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=163211
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=164125
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/keys/ha?recursive=true&wait=true&waitIndex=163836

The interesting part here is the waitIndex parameter that etcdctl is suggesting to use, it is cycling between 3 values, and all of them are smaller then the specified after index parameter.
The etcd workload of the services is pretty simple, upon startup they register a key with a ttl of 30 seconds  and every 10 seconds the ttl is updated. Furthermore each service has 2 watches on directories to see if another service has come online. And upon stopping the service the key is removed.
Each service has it's own directory in etcd.
The issue can be replicated on the test machine (most of the time) by starting / stopping 3-5 of those services at the same time.
etcdctl version 2.3.2
etcd Version: 2.3.2
Git SHA: ce63f10
Go Version: go1.5.4
Go OS/Arch: linux/amd64

@ederuiter
Hi, can you provide me a simple go program that I can run against a local etcd cluster to reproduce the issue you described? It would dramatically accelerate the speed to resolve this issue.
Thanks!

@xiang90
With this simple test program I am able to trigger it pretty reliable on my test machine: https://gist.github.com/ederuiter/39ea71b16274eaae5facf41aa270fbac
The test.sh runner script accepts a single parameter specifying the number of programs to spawn, with 1 it seems almost impossible to trigger it (I have seen it once) with 2 I can reproduce is almost every time.
The test program is quite simple, it sets a single key with a default ttl of 30 seconds, and every 10 seconds key is refreshed. Besides that it has a recursive watch on the same directory as the key.
The runner script starts each test program with it's own directory (/test/test)

I noticed you used the refresh feature. This should be fixed by https://github.com/coreos/etcd/commit/3b100ad142271c37b095732a27876eafec5f8be3.
Can you upgrade your etcd cluster to 2.3.7? Then the problem should go away. Thanks.

I tested it with a normal set instead of a refresh, and with that the issue is still present. So I don't think the commit you mentioned will fix it, but when I have the time I will retest it with etcd 2.3.7

@ederuiter Did you test with a fresh etcd setup every time?

Also how long should I expect your script to reproduce the issue?

Hmm when testing with a fresh etcd setup I was unable to reproduce the issue on 2.3.2 as well as on 2.3.7 (both with and without the refresh).
On my test and production machines the issue arises directly after starting the test runner.
When I have some more time I can try to import a backup and/or running 2.3.7 on a copy of the datadir of my test machine.

@ederuiter If you use the "refresh" feature, it might make etcd into a broken state that cannot be easily fixed. That is why you can reproduce on your already broken cluster.
A lot of users and applications depend on watch features, none of them reported this issue after 2.3.3. So I am 99% sure it is the root cause. Let us know if you still find this a problem with a refresh 2.3.7 cluster.
Thanks!

Indeed, after updating the cluster to 2.3.7 I was unable to reproduce this issue.


Darwin: gdb job was killed after log exceeded 64 MB

Issue description
gdb on Darwin likes to output a lot it seems.
building of ‘/nix/store/q006jca5vxb07d9jkai6waaxsp77nf84-gdb-7.12.1’ killed after writing more than 67108864 bytes of log output

http://hydra.nixos.org/build/53302813
Nixpkgs master at 00672dec8a1367a1d559660aa7a125b31f3f6562

Might be fixed on staging http://hydra.nixos.org/eval/1362999?filter=.x86_64-darwin&compare=1362638&full=#tabs-now-succeed

Seems OK now on the very same derivation.  I suppose it was some random error that won't frequently happen.

Hmm, happened again now.  Perhaps the build is simply too chatty.


TAC meeting notes 2020-04-22

TAC meeting notes 2020-04-22
Signed-off-by: Jean-Francois Panisset<EMAIL_ADDRESS>

Thanks JF!


Consolidating samples for how-to-write-text-to-a-file

Summary
Make the code examples in https://review.docs.microsoft.com/en-us/dotnet/standard/io/how-to-write-text-to-a-file?branch=pr-en-us-9760 more usable by providing an overall sample that can be run.
Related to https://mseng.visualstudio.com/TechnicalContent/_workitems/edit/1385012.

#hold-off - Please don't merge till I can consult with content owner about process.

@BillWagner @rpetrusha - Thanks, I've put back the Snippet markers. I've changed the doc to run the whole sample so that it is runnable.


CHANGELOG.md file is severely outdated

Describe the bug
Hello!
I was trying to find any info about compatibility between different versions of this SDK and different version of Unleash server. Unfortunately, there is no such info available. The I've tried to find some implicit hints about backward compatibility through history of changes. It turned out that latest notes in CHANGELOG.md are connected with a release 6.0.1. As of now release 9.2.4 is available. Was the data about changes after 6.0.1 tracked anyhow except git commits? Is it possible to restore it?
Steps to reproduce the bug
No response
Expected behavior
CHANGELOG.md is up-to-date
Logs, error output, etc.
No response
Screenshots
No response
Additional context
No response
Unleash version
No response
Subscription type
None
Hosting type
None
SDK information (language and version)
No response

Hi,
we moved away from the changelog and into release notes directly on github. We should have updated the changelog to inform about that, sorry!
https://github.com/Unleash/unleash-client-java/releases


Turtle scaling

Hey guys, turtle won't scale the design big enough for my table.  All I can make is a little spiral maybe 6" across.
Oh, table is X max = 1280, Y max = 2060.

Sorry Barry, I saw this, and was going to respond when I got back to my computer, and I forgot.
I agree that it should at least be scalable to larger areas. I don't really like the slider bars, so maybe I'll just replace them with numbers like the shapes use. Then there will be no upper limits.

Sorry Sorry Barry, I'm going to just remove the turtle stuff for now.


Merge Sort

Add comment to get assigned to your desired language.
Submission Title
Merge Sort
Submission Type
Algorithm
Submission Description
Write Merge Sort Algorithm
Submission Directory
Algorithm => Sorting => Merge _Sort => Merge_Sort.extension
Language

[x]  C++
[x]  C
[x]  Python
[x] Java
[x] JavaScript
[ ] C#

I can do this in c++, java and python ! please assign me this issue 😄

I can do this in c++, java and python ! please assign me this issue

Ok assigning you for C++, Java, Python.

Hey @krishrahul98 Could you assign me this for C ?

I want to implement this in C++. Can you please assign me this?

Hi, i want to create program for merge sort in python,

Hi, i want to create program for merge sort in c . will you please assign me......

Can I work on this in Java?

Hi, i want to create program for merge sort in c . will you please assign me......

@rishika-singh49
merge sort in C is already assigned and added to the repository before. If you want to add a modified version of Merge sort in C then I can assign you. you can choose any other thing of your choice and if it is not in issue, you can always raise a new issue and get assigned.

Can I work on this in Java?

Assigned for Java

I can do it in JavaScript

I can do it in JavaScript

@shailshah76 Assigned for JavaScript

Hi @krishrahul98, I want to implement this in C#. Could you please assign me this?

Hi @krishrahul98, I want to implement this in C#. Could you please assign me this?

@vieee Assigned for C#


Enable AddViewControllerTests

This test was disabled because it's failing in CI and blocking the release.
This should be investigated.

Does this still require work? Interested.

Yes, the test still is disabled and not working.
I think it might be related to locale, date or or something like that.


feat(gridstyle): add a new class for one column grid layout

Alaska Airlines Pull Request
Summary:
Needed to add a new class in WCSS to support one column grid layout. This is a pre-requisite work for the AuroDocSite, in order to provide a demo page that utilizes a grid layout.

Added a new _grids.scss file that consists of grid styling to be used for a one column layout.
Added mixins to allow for updating the value of the max-width, margin and gap.

Type of change:

[X] New capability

Checklist:

[ ] My update follows the CONTRIBUTING guidelines of this project
[X] I have performed a self-review of my own update.(By importing distribution file of the new grids.scss and adding it to the AuroDocSite repo manually.)

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: marnel-mangrubang:x: chethanayasaYou have signed the CLA already but the status is still pending? Let us recheck it.

Is there a working demo somewhere that shows all these new styles in action? I feel without that all I could review is if the CSS is syntax/semantically correct without knowledge about if it seems to do the right thing.


Messaging extension - GCC Tenant - Only first command is working

Hi Team,
We tried to use messaging extension in GCC Tenant and following is the observation:
When we add app to team, messaging extension is installed. But only the first command (recents) is showing and working. All other commands are not showing in messaging extension overflow. Even if we type other commands in compose area, don't see any response.
if it is the limitation in GCC tenant, Could you please add to the documentation.

Submitted another issue here: https://github.com/MicrosoftDocs/msteams-docs/issues/2761
So, closing this duplicate.


removed drugs from workflow

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@254f3f2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage          ?   49.67%
=========================================
  Files             ?       14
  Lines             ?      155
  Branches          ?       43
=========================================
  Hits              ?       77
  Misses            ?       78
  Partials          ?        0

Impacted Files
Coverage Δ

app/cards/Card.jsx
92% <ø> (ø)

Continue to review full report at Codecov.

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


[1.12.2] Crashes when drinking milk or filling a water bucket

Minecraft Version:
1.12.2
Forge Version:
<IP_ADDRESS>23
Advanced Rocketry Version:
1.12.2-1.3.0-59
LibVulpes Version:
1.12.2-0.2.8-27
is this the latest curseforge build?
yes
#Please Pastebin crash
https://pastebin.com/43EkbZcV
Crash Report:
Crashes when drinking milk or filling a water bucket.
Explain issue in detail:
Game crashes when drinking milk or filling a bucket with water, I know it's similar to #1177 and #1178 but it still happens in 1.12.2-1.3.0-59 where commit https://github.com/zmaster587/AdvancedRocketry/commit/ee0a9b8961863d00128b9896b9da8a98eb8a429d should be included according to the changelog... :smile:

Strange, it's not crashing when fill or dump water, nor when I drink milk.  Just going to guess the raytrace is null under certain circumstances.  Take 2!

AdvancedRocketry-1.12.2-1.3.0.jar.zip
Try this one and see if it's still broken?

This version works fine. :+1:

Having the same issue here
crash-2018-02-28_14.01.43-client.txt

@Austin5003 what about the build I linked above?  That's slated to be the next release

Can confirm this was an issue but the new build seems to have fixed it

@zmaster587 That build fixed the issue.
Thanks for the quick response

Alright, good to know!  I'll close the issue then


Add Python=3.10 to Github Actions and switch to Python=3.10 for the Github Action that builds the PyPi package

Description

Adding python=3.10 to the list of tested for Python versions in the Github Actions runs; note that the GA that builds the PyPi package uses the latest Python version that is now 3.10 so this has changed to 3.10. Sister PR of https://github.com/ESMValGroup/ESMValCore/pull/1430

Closes https://github.com/ESMValGroup/ESMValCore/issues/1425

Before you get started

[x] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Tests run successfully
[ ] 🛠 All checks below this pull request were successful

yes, no, that's just my annoyance that YAML and I don't think the same way :laughing: Thanks dude, I'll remove those tomorrow, am too tired now, but if you can approve per-emptively I promise I won't make other changes :grin:

OK monsieur @zklaus - GA test branches removed, she's all your mate :beer:

many thanks Klaus, you a :star:


Add processes source

Initial try for Processes() source. I am not sure if this is good approach, but it's easiest one. I don't know how to make it protable to other platforms, it's Linux specific right now.
If it's good idea i can do something with it, if it's bad approach then let's close this PR.

Thanks for the PR! Yes, I think if we do this, we need to make it as portable as possible. That may mean it's not practical to do—but you might research a little prior art and see what other Go libraries that get process information do.

We can start with Linux-only function for example and make it compile only for linux. For other platforms we can add some placeholder with error. Then if someone wants to add unix/macos/win implementations it can be done later. Personally i wouldn't like to implement other platforms due to my lack of knowledge of those platforms.
My other concerns with this implementation is about errors: you can get error on one process because of permissions but you will still get some list.

It's possible https://github.com/mitchellh/go-ps might save us some work—but I think we need to start with an issue and a use case, to help figure out if the feature is needed at all, and if so, how it should be designed. See the contributor's guide for more about this.

Yeah, i have read contributor's guide. It's just feature from your list of ideas. Ok i will start issue about this then.


Overload map() (and transform() to support object literal for specifying the transformation

This occurred to me when reviewing https://github.com/mavoweb/vastly/pull/17
map() currently expects a callback, but the object literal syntax is a pretty convenient sugar for many use cases, and we could easily disambiguate between the two.
Same applies for transform() (see #20)

What does object literal syntax mean?

What does object literal syntax mean?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Working_with_objects#using_object_initializers
In this case, an object of node types to functions, like the current serialize.transformations.

Now that #29 is merged and transform can support object literals and arrays, what do we think the best approach is for map?

@LeaVerou Now that #29 is merged and transform can support object literals and arrays, what do we think the best approach is for map?

Check out my comment in the merged PR.
Looking at the code, I just got an idea: what if we pass node as a last argument?
Then it can be nicely simple and composable:
export default function map (node, mapping, o) {
	mapping = [
		mapping,
		(node, _, _, originalNode) => originalNode === node ? {...node} : undefined
	];
	return transform(node, mapping, o);
}

Side node: not a huge fan of so many positional arguments, but worried that turning property, parent, and originalNode into a dictionary will be bad for performance since these are called a lot.

Check out my comment in the merged PR. Looking at the code, I just got an idea: what if we pass node as a last argument?

Yeah I saw, I will do that

Then it can be nicely simple and composable:
export default function map (node, mapping, o) {
	mapping = [
		mapping,
		(node, _, _, originalNode) => originalNode === node ? {...node} : undefined
	];
	return transform(node, mapping, o);
}

Yes, this is what I had implemented in the other PR we had previously 😄

Side node: not a huge fan of so many positional arguments, but worried that turning property, parent, and originalNode into a dictionary will be bad for performance since these are called a lot.

Hmm yeah, this is certainly a tradeoff to consider. How about we keep it as is for now (positional arguments) and revisit this later?

Yes, this is what I had implemented in the other PR we had previously 😄

Huh, you’re right, I remembered it being a lot more complex, but the logic was basically the same.

Hmm yeah, this is certainly a tradeoff to consider. How about we keep it as is for now (positional arguments) and revisit this later?

Much harder to change API shape once stuff ships. I will mull it over.

Much harder to change API shape once stuff ships. I will mull it over.

Makes sense, I'll put up a PR for map, and we can decide before we merge that


Add New YNO games to discordRpcUtils

Add Oneshot, If, and Unnacomplished to discordRpcUtils

NOTE: This is not intended to be merged unless the icons for the above games are added to the Discord RPC application. Playing one of the games above will throw an annoying error about a missing asset if you close the window with them running due to the missing asset. If said Discord application is no longer accessible by the new fork owner this should probably be redone at some point...

We could package the icons for each game locally, but I'll try to see why it might've broken first. From the first glance it gets the icon for the games from yno website.

I have decided to rewrite the whole Discord RPC section, when I make a commit, I'll let you know to review it and try it.

Please review.

4053fd9 has no issues building on Windows on my end. Did not test Linux.
Games added manually via this pr now show an icon, though not a fan of the change from character face icons to doors, especially with it being off center... not too sure how the old version was pulling the icons, is there an easy way we could still do this post rpc changes?

I personally like the edit from "Disconnected" to "Going to bed...", though this gives me the idea to implement the ability to custom set this either thru a config file or in the interface somehow.
Thoughts?

As an additional thought, the character face icons are the favicon, would it be feasible to just pull that? I see the comment you threw in, I did a quick skim thru on the yno github repos and couldn't find where they are stored. Will dig a bit deeper later if needed.

Discord RPC doesn't support reading or using ico files as imageLargeKey or smallKey, before it was done by directly uploaded the icons to the discord developer portal or you could alternatively upload them to a third party host or have them locally saved. I dont' find any of these solutions good at all, so I decided to just use doors for now, but if you have any better ideas let me know.

Moot with RPC changes


How to record video?

Sorry I'm new to playwright.
Can it implement by add some params after npx playwright-cli xxx or some other usage?

To record videos you can add it to the context options as documented here: https://playwright.dev/docs/verification?_highlight=recordvideo#videos
Do you expect a functionality to add recordVideo to the code generation?

t

Yes, Can it be implemented?

To record videos you can add it to the context options as documented here: https://playwright.dev/docs/verification?_highlight=recordvideo#videos
Do you expect a functionality to add recordVideo to the code generation?

Yes, Can it be implemented?


feat: gateway query cache nodes

https://github.com/fleek-network/ursa/pull/156#issuecomment-1360277795

#156 (comment)

... can be taken up a level on client init not on every handler

Sorry, not sure what you mean here. Do you mean having a Client struct and with methods get_block, get_file, etc.?

#156 (comment)

... can be taken up a level on client init not on every handler

Sorry, not sure what you mean here. Do you mean having a Client struct and with methods get_block, get_file, etc.?

an rpc port shouldn't be passed down from every handler because of the reasons in the comment above


Rearrange the lab files

Module: 00
Lab/Demo: 00
Task: 00
Step: 00
Description of issue
The lab files are list from 01L01 through to 05L01 with both the ADDS and the AADDS next to each other at the bottom of the first lab page it shows the lab tracks, could the links the the labs be put in the same order, this has caused multiple problems with teaching as the delegates open the wrong labs.
Repro steps:

Rearrange the labs on the first page to have all the ADDS labs first then all the AADDS labs second picture

attached

Hi Paul,
unfortunately renaming the files might have some undesired downstream consequences.
We're considering alternatives
rgds
Marcin

The file don't need to be renamed just set out in the same order as you have at the bottom if the page divided into the two different streams.
Regards
Paul

From: polichtm @.>
Sent: Friday, December 3, 2021 1:39:26 PM
To: MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop @.>
Cc: Foulger, Paul @.>; Author @.>
Subject: Re: [MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual -Desktop] Rearrange the lab files (Issue #121)
Hi Paul,
unfortunately renaming the files might have some undesired downstream consequences.
We're considering alternatives
rgds
Marcin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://url4.mailanyone.net/v1/?m=1mt8n2-0006Dx-6D&i=57e1b682&c=CQjedphCXWGuQf_PVUexVVoYTiL1f2GwxliK_BUTLRcQRH4mMIAh6GQVvBVD_FZKXBgFKvRwkR2tamvHOphswDZSkCZmDBD5_9J1zOqgFZ8mIQrxab399PHA0Tu2h9N7FeUZVNkDXfbB3ANVZczQndSuVp44QI4x_p0I6ygo3acjTYsPhSxTWt2zYQGjjy3NGdg-zfRw7dhoakjRVW0AL6TXEBNAo-OirENzFqd9ivMt-RHMQjRWCcX_TchsCQK23YPLZv084c_ilpvQjKlDQT-6wFp_D9TqbN2mMF9O0wKDgBB7kUr5McJ0n6WSPhcm2PRm15rS62zImtzFyUkx7h5wYvtfY7aczoQkXCtiIDSj3cSEks3SRBQTK3H94Ylq, or unsubscribehttps://url4.mailanyone.net/v1/?m=1mt8n2-0006Dx-6D&i=57e1b682&c=0wEVsLDRcv1Q15Dro7uv42_s3X77vhiD0N0A9rmZSRmc-YG4QRM4vJhCq6ulqWhHSumdIldLrmaki2cKhljizuXkDTUSXoqwa7-ILRn1aFeMZSaPJjqr7aLUd5Tjp02FNzknV3AhahWQ68hDQaxi71tfUeGhljJp4Y6Vx5pYmYSqKW7msSi_pQd_RvlgctqwI-k0rVC5XqjdGcl1c3sakx-M8tDbiM2rWnuG3O7N2E9q8Y-UsZEAOcWAtfxrG6IxX4mQ2sPrLu--v-pQHY2dr7Mkb4lJ6wEG6PIVPLv-2f_IAusxofoEMqreNjGqqZIt.
Triage notifications on the go with GitHub Mobile for iOShttps://url4.mailanyone.net/v1/?m=1mt8n2-0006Dx-6D&i=57e1b682&c=fZgMvNB_ZW9bYnlNV48Ti3VIHbx9ULeufoe6d5g5eqDi38CiW_wd9Dmlb0zv94V0T5pI7dS8l5vfSjj8p5Ck3U8LSPDaGr77UqjlfalMBB_JN7g_V04L72CMF7oUwxq6dkF5cw9XcNzRFnTqrL9ymzSQBFfRrVT8L5rtLvZxPLx0-XhIb2wD_Vvv-WrhldQPEpgP4FtMeIjkY4IORQvN0V7Zkber2QM7ryardv86_cLIlqL1WWVuBuDz_qBpsq5XmbMWIMQB-eYbxr-Ygd60QeIIG9Qqff7IgekLDY6_NqTZN272xTU4DQJeOjejUmnz or Androidhttps://url4.mailanyone.net/v1/?m=1mt8n2-0006Dx-6D&i=57e1b682&c=51qYjD_bwVrT7SrbeWKXZlqqB9-7Hekz4R-y-Pe0M07ULSq-nPW7tzJCf6VA30eIybC6T16a_pO2CVtUS71p6fngJB-V4UiZmkQ5s_zWW1FClif9MnMe3QDV_X7oNvbWQFd2NUMkH8gNJ7UFS4kOFL2ydM5qYCQUo2XrCQy19EIJp6cz8qBGUgMVtRMPGtAl929roOqqfNlKoxefRKITcOQxmsoIxe8h0I-sd4R9BHj6MxcIvfD28Y4DK1UQJ1uZhdt9yC47ES5-Bc-tElnoHOa07GlA_IeiirTGFwqfdXIHyy4VlP8qDgiIN8V0NMuKMh-VCDarPujW2UAbhEbgRoxma4TCljJGFQmgoIaXcns45TGuEiHCaYgT53GMHox4NLqiC_UZXGkoVijESkZ8bg.

Hi Paul,
unfortunately it's not clear to me what specifically you're suggesting.
You're welcome to submit a pull request that would actually contain the suggested change
rgds
Marcin

Hi,
The lab instructions are currently list as Lab_01L01 for AADDS followed by Lab_01L01 for ADDS and each section has the AADDS labs followed by the ADDS lab, on the lab instruction page at the bottom it shows the labs separated into the two streams all the ADDS lab then the AADDS labs if the links to the instruction oages could be separated in the same way it would be easier for the delegates.
Hope this makes sense.
Regards
Paul

From: polichtm @.>
Sent: Friday, December 3, 2021 2:01:20 PM
To: MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop @.>
Cc: Foulger, Paul @.>; Author @.>
Subject: Re: [MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual -Desktop] Rearrange the lab files (Issue #121)
Hi Paul,
unfortunately it's not clear to me what specifically you're suggesting.
You're welcome to submit a pull request that would actually contain the suggested change
rgds
Marcin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://url6.mailanyone.net/v1/?m=1mt98E-0002pQ-5t&i=57e1b682&c=TkXFV-PrrpLXz7trOW5IIgg6VMU1GgqFFnn23frxzYpX_ax8YjJQxFCn3rT5vp2cTKhND3-Wg6SYeMnWuckahFl_i8sFl8CgF-21mxa8hKkLuMzRwXCiltm3_Z8f-tGG9BfwgUY4WpMd2wXAUwbgXUzLSqDW5d6_b_8tE8bP9EUMHRV0i-sLLdXnuvlRzZvW1akNBk-HORHCtnIMV9vK20IPwz_3n8u29zuPyJpnQ_Oxbqmz-ttLgWVo1kLo69O92q6TEvkxtQuKv2r81i9SLRv364lTqavJ8-mIyW47p7VyizzMeYs6bNpxFCXA21KA7Lwd7oYB6hd1kbnwEs-4L61dKtYgZXD_uhAWY975nN6XZ--FHU67Q6qlnDh2V9Qd, or unsubscribehttps://url6.mailanyone.net/v1/?m=1mt98E-0002pQ-5t&i=57e1b682&c=s-3ON1UcMtd8NFGuIN-CaYC6FGTdbG9IGhFplvgMtWMMoccGaTSVKHpxFxyzpWeqGIySR-Kgx6kvahjetejUejTD1o9VyH-KOj-TooiT52fefDQ17ywkJQodVGlAFRrzW1peo5QLb1WPk5a65HRyBGIjXrRgJiwNItzvfNmYTvOrpTvF57GUJuqYG7WaEInHLh35Rude1lkPAuWGsILlopcICZJlrRF_VQfpdfKxpD1gP6CmSJ3wk08VtvRcJsgxEquO15UTTCuUfaLOzLQd-zBrKFQ-PcYBTyZHGdHJUhey-ZOSkWdj4rZhmGjRF3bI.
Triage notifications on the go with GitHub Mobile for iOShttps://url6.mailanyone.net/v1/?m=1mt98E-0002pQ-5t&i=57e1b682&c=Su4s_ehu8ena-JpwG_5iug7QZOfq0N-GxtfSF8bs4AOIhHgeLnTYo4nXclAh6KHWfK4Ua2E12pa7FW8tSYhrIfypfdZEmbnruwTB78_mjMVTfBg7pHYLCC69Lc9InH_UIttJooiD_OzKyKTp8erEi1RkfSadDD2bHhSh7p3JNVeVQMQv5yxyi-WJLbjxKmogK3cqjHo9xDnBKAcy2qcwvnp4Lv_T-Dxh6Bdf5uKeOQgbA_q7N9wQs-68Z-AcKc5bHKqYucCNjsNGcZZdGZMa4xHxZSBWzbiNwY2bmfgW9SV3AEnriSCJn4C-_tRbytcu or Androidhttps://url6.mailanyone.net/v1/?m=1mt98E-0002pQ-5t&i=57e1b682&c=CLvPNOTuVt2XbDR0UGjZd-0-SxV3P81Vf3NNkFQAeaciYrh4gJDY_XI8PLqmECzFNkqpbvV4ZHT17-U-l0eO9a0Nx1ZYnJ_MGss3nx6dPgs9cjjpWCMtebedqWT7wTcQHokpkIpXomCnvzhS-_tJdNhmHx9W_f_wfJJow7SZ9T564OEFE878Glk1gef2bYL0wc_6GudImtsterRfKKGIpuCChpR9V7HL-XD-9YTc_wBzNMLrpFxjaZrqox_LZL167msq_TdYwfgT6HtJ-Nof2xVLAazl2MuFp4AP_5A2fwAtW5J4Q3WWrjhWxnWXtH7okszE_I3Uxsok1ktnhz8fBDX4C-3pMuSl1Nnvm-WU2MKant-dFWwR1g-h5-jdebIk1UaFW2pUKw9ARk366J4a2Q.

MOC Labs Review confirms this issue

MOC Labs Review confirms this issue

Unfortunately we cannot discount the possibility that there are downstream consequences of modifying the existing folder structure. The labs are currently clearly marked as being part of the Azure AD DS or AD DS track - any additional suggestions regarding further enhancing this approach are welcome
rgds
Marcin


CB-9219 Added filters to search, support for multiple words, localize…

… download count and other css changes

LGTM. Feel free to improve this stylistic changes I mentioned later - Merging this one in.


KAFKA-5819: Add Joined class and relevant KStream join overloads

Add the Joined class and the overloads to KStream that use it.
Deprecate existing methods that have Serde params

@guozhangwang, @mjsax, @bbejeck

retes this please

retest this please

LGTM provided jenkins success.

merged to trunk


ideviceinfo invalid param

background
This is a bug request... while trying to find a reason why my macbook air 2019 cannot see my devices I started reading the code. I noticed that the code was trying to execute ideviceinfo -l however, the param is not valid according to the online help.
https://github.com/flutter/flutter/blob/89427d672d62618461f6fa0b98dc1f9a81529722/packages/flutter_tools/lib/src/ios/mac.dart#L157

I made a mistake... idevice_id not ideviceinfo


Tor module is missing all client bridges functionality (can't use private bridges from tor project)

Issue description
Commit https://github.com/NixOS/nixpkgs/commit/0ccdd6f2b043e5123ffd1f76cd2187c39ce19b94 changed the way tor module works. While making tor config options typesafe, it also lost a way to specify custom config (services.tor.extraConfig).
In turn, it seems (from my point of view, looking at https://github.com/NixOS/nixpkgs/blob/21.11/nixos/modules/services/security/tor.nix) users cannot specify custom bridges anymore, and use client bridge functionaly to bypass censorship in their countries.
I use following torrc config:
ExcludeExitNodes {fr}

UseBridges 1

ClientTransportPlugin obfs4 exec /usr/bin/obfs4proxy

Bridge obfs4 <IP_ADDRESS>:9999 somehash iat-mode=1

And it looks like none of these options can be expressed right now, in current tor module.
@ju1m i see you are current maintainer and the author of these changes. Is there a way for me to set custom config fow now, or use bridges? Or can it be added at some point to the module?
Right now i can't connect to tor directly, as all public bridges were banned in my country, so i specify them in bridge, and i consider this functionality to be very very important for using tor at all in censored countries.
ExcludeExitNodes also allows me not to land on the same country where external resources are banned (not to say about personal security concern).

Is there a way for me to set custom config fow now

@Fedcomp, sure, services/security/tor.nix follows RFC0042 and uses Freeform modules, so you should be able to set the custom options you want in services.tor.settings as you would do for explicitly typed options, this is suggested by the migration instruction given when using extraConfig, but I agree this may not be obvious for newcomers as it is currently written.
Of course making a PR for typing more precisely the missing options would be welcome too :)

Oh sorry i didn't know about nixos-freeform modules. I though i am locked to only typed options.


[Bug] Most Invites and InviteRequests are deleted on closing VRCX

Describe the bug
I've noticed this for pretty much the entire year and a half I've been using VRCX. I love this tool, but it frustrates me that if it should freeze up (something else happening lately that might be due to needing a graphics driver update, need to try that first before I consider another bug report here) or my computer crash or something, nearly all Invite and Inviterequests from a session using VRChat and VRCX will just auto delete when the application is closed and reopened.
Part of why I use this tool is that I want to keep track of which friends have been trying to spend time with me while I'm busy; so I can remember to make time for them when I'm free. It's also nice to know who wanted to see me while I was nodding off or afk and left VRChat and VRCX open. Other types of notifications are being purged too, but I'm most concerned about those two types being logged more reliably. Like how people leaving and joining the instance or logging on and logging off seem to be.
To Reproduce
Steps to reproduce the behavior:

Open and use VRCX while in VRChat.
Continue until you have several or more notifications in the log.
Close VRCX.
Reopen VRCX.
Open notifcations tab and note that some Invites and InviteRequests have been purged arbitrarily while others are saved in the log, unless the bug is unique to me and I did something wrong during setup or something.

Expected behavior
VRCX keeps a consistently saved log of all recent Invites, InviteRequests, and maybe other such notifications the way it keeps a log of a lot of other activity like people joining and leaving your current instance or going on and offline.
What version you are running
VRCX 2024.09.02
I always let the app update though, and it's happened across every version I can remember in the past year and half or so.

The only thing the comes to mind is VRCX doesn't save self invites persistently but only for the session, as for other things not saving that does sound quite odd and has never been reported before, you could use something like sqlitebrowser to verify table entries are being saved to the database as they come in. Or open DevTools with Ctrl + Shift + I and check the console tab for notification or database related errors.

I've noticed this as well, but for some reason never really thought to report it.

I have DevTools open and I'll keep that open as I get on later this evening. So far with Group notifications I haven't noticed errors related to either of those, just some 404 not found ones that seem to be related to the VRChat api that popped up a bit ago.. Unfortunately by the time I saw you reply earlier things were petering out as far as my social circles last night and I myself was heading to bed soon.
Where exactly does VRCX save the log of VRChat notifications? I could also check if there's anything funky with the permissions for that file and see if I can compare it with a usual night's activity and see what actually made it into the file and what didn't.

Open %AppData%\VRCX\VRCX.sqlite3 with sqlitebrowser to see the table with the stored notifications, all VRCX data is saved in this file.

Thanks. Looks like the database has all the invites and requests going back to the day I first started using this application, but the application is very selective in what it loads from the database and presents to me. I will try to check later if there is a pattern, like it loads every tenth or so or something. I wouldn't expect it to load everything when the application is started up, just all of the latest couple hundred or so would be more practical for default behavior, I think.
Also, most group notifications aren't being logged. But I'm much less concerned with those, since they're also on the group pages and we have the group instance tab in game and in VRCX anyways.

RIGHT, thanks that's the exact info I needed I can see the bug now, will be fixed in next nightly. And yes group notifications not being logged is intended I never got around to implementing that.

The update worked great. I should've posted here months ago. And yeah, Group notifications aren't all that important to me. They would clutter the database with weekly meet announcements and other such unneeded data. Invites and Requests are more personal and therefore more valuable to keep track of. At least that's how I feel about it.
In fact the notification tab is the area I look at the most. Because I hate how Invites and Requests get deleted after 15 minutes. I wish VRChat would make them hang around a lot longer like group notifications, give me a chance to acknowledge I saw them at least; even if the actions available beyond dismissing them have all long since expired.


Add example with-purgecss

Add example with purgecss and tailwindcss
Purgecss is a tool to remove unused CSS. It can be used as part of your development workflow. Purgecss comes with a JavaScript API, a CLI, and plugins for popular build tools.
This tool is often used with tailwindcss. The PR adds an example of a nuxtjs configuration with purgecss and tailwindcss
Related issues
https://github.com/FullHuman/purgecss/issues/54
https://github.com/nuxt/nuxt.js/issues/892
https://github.com/nuxt/nuxt.js/issues/2810

And now, the big question: should examples in the main repo have tests?
:thinking:

@Ffloriel  any chance you fix build issues? You can see them in details, that is only thing which is preventing merge, as I see.

I have just tried to get Purgecss and Nuxt working with vuetify, it worked but some css was lost.
I followed official example.
https://github.com/FullHuman/purgecss/tree/master/examples/with-nuxt
Still trying to fix, once found will share

This is awesome! I did not know if purgecss will be able to work with Nuxt.
Will look more into it, this would be great if it could work with others UI frameworks yes. Then find a way to include it in Nuxt, I would love this feature, everyone!
I will look at this example and merge it when verified. Thank you for this PR.

Codecov Report

Merging #2942 into dev will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              dev   #2942      +/-   ##
=========================================
+ Coverage   99.91%    100%   +0.08%
=========================================
  Files          23      23
  Lines        1237    1242       +5
=========================================
+ Hits         1236    1242       +6
+ Misses          1       0       -1

Impacted Files
Coverage Δ

lib/builder/builder.js
100% <0%> (ø)
:arrow_up:

lib/common/options.js
100% <0%> (ø)
:arrow_up:

lib/core/renderer.js
100% <0%> (ø)
:arrow_up:

lib/core/middleware/error.js
100% <0%> (+2.12%)
:arrow_up:

Continue to review full report at Codecov.

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


DOC Clarify release documentation

Reference Issue
Clarify release steps
What does this implement/fix? Explain your changes.
Any other comments?
cc @agramfort @maikia

thanks @lucyleeow


Remove redundent resourceGroup scope from all paths

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

@orparnes is added to the review. #Closed

Can one of the admins verify this patch?

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6421
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6421/commits/2795b3733751019c08ab84316cf62ff92f3feacd


Update installer.sha and artifacts

Sync to latest installer release-1.3 charts.

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_operator_release-1.3
a4048281092bfe2213535cb667f33e4bfc17b38e
link
/test lint_operator_release-1.3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Error: Missing "./internal" specifier in "svelte" package

Describe the bug
Getting error when creating project:
error when starting dev server:
Error: Missing "./internal" specifier in "svelte" package
Reproduction
npm create svelte@latest my-app
cd my-app
npm install
npm run dev -- --open

And choosing Svelte 5:
◇  Which Svelte app template?
│  SvelteKit demo app
│
◇  Add type checking with TypeScript?
│  Yes, using TypeScript syntax
│
◆  Select additional options (use arrow keys/space bar)
│  ◼ Add ESLint for code linting
│  ◼ Add Prettier for code formatting
│  ◼ Add Playwright for browser testing
│  ◼ Add Vitest for unit testing
│  ◼ Try the Svelte 5 preview (unstable!)
Initially returns error.
Logs
No response
System Info
System:
    OS: macOS 14.4
    CPU: (10) arm64 Apple M1 Pro
    Memory: 778.14 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.6.1 - ~/.nvm/versions/node/v21.6.1/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 10.2.4 - ~/.nvm/versions/node/v21.6.1/bin/npm
    bun: 1.0.11 - ~/.bun/bin/bun
  Browsers:
    Chrome: 119.0.6045.123
    Safari: 17.4
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.2.0
    @sveltejs/kit: ^2.0.0 => 2.5.5
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.0.2
    svelte: ^5.0.0-next.1 => 5.0.0-next.88
    vite: ^5.0.3 => 5.2.7

Severity
serious, but I can work around it
Additional Information
No response

Whats your workaround, not using svelte 5?

@shanecmdexter I just ran into this issue, too, and was able to install next.87 instead of next.88, which is currently tagged with next in npm, and then able to run the dev server. You can install next.87 with this command: npm i -D<EMAIL_ADDRESS>or update the specific version as desired.

fixed in the latest release


make load functions compatible with pathlib

Mi @marcomusy
This PR follows issue #1169 the load functions now accept pathlib.Path as input, nothing changed in the behavior of the functions, the paths objects are converted to strings at the beginning.

Thanks!


Cyril/fix sticky sidebar

Description
This PR uses a plugin to make the sidebar sticky and to improve performance with scrolling and stable across multiple browsers
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Impacted Areas in Application
List general components of the application that this PR will affect:
Which pages?

[x] Report Page

Specific components

ReportFilter

Testing Script:
Report Page

Data loads successfully
Collapse and Uncollapse button works
Clicking on the taxonomy name gives you total reads for that taxon
Category filter works
Genus Search works
Filters work as expected
Disabling Filters works
Filtering by column works

Checklist:

[x] I have run the testing script to make sure current functionality is unchanged
[x] I have done relevant tests that prove my fix is effective or that my feature works
[x] I have spent time testing out edge cases for my feature
[x] I have updated the test script or pull request template if necessary
[x] New and existing unit tests pass locally with my changes

@cyrielo  @cdebourcy Yep - the white color changing isn't good but the sidebar worked ok for me on a few reports I tried. Not sure if I'm missing some detail.
The one big problem is that we shouldn't have white space at the end of the filter - it should go all the way to the bottom of the page. See attached screenshot


Feature/10764 fss UI checkwebsitestatus step in Pipeline

created a script to check status of Website
Added the variable in DevOps library
Added the step in Continuous Deployment Yaml to run the status check of Website for FSS UI URL

closing this PR as this is done on other brnach and merged to main


Bugfix/permission command

Modified the behavior of several commands.

@marioncottard can you review it?

Yes, I'll review it now


Image browser not working

It never worked for me , here is the log:
Command received
[2023-09-25 11:27:06] Starting Image Browser
Setting up Image Browser
[2023-09-25 11:27:06] Setting up Image Browser
[2023-09-25 11:27:06] Updating Repo /storage/sd-webui-infinite-image-browsing ...
There is no tracking information for the current branch.
Please specify which branch you want to merge with.
See git-pull(1) for details.
git pull
If you wish to set tracking information for this branch you can do so with:
git branch --set-upstream-to=origin/ master
ERROR
Error
Not running
cat: /tmp/image_browser.pid: No such file or directory
/bin/sh: 1: kill: Usage: kill [-s sigspec | -signum | -sigspec] [pid | job]... or
kill -l [exitstatus]

Is now fixed, please run cd /notebooks && git pull && cd image_browser && bash control.sh reload to update

Is there another way to open the image browser other than your website? maybe a command in the terminal?
I updated it like you said but the website https://ai.sheldonxxxx.cc/ still didn't work, so I reloaded the page but then it lost connection with my notebook .

Do I have to open the image browser through your website? Is there a way to do it from the terminal with a command?

You can either run status in terminal, and it will print all the website url, or receive using discord.

When I refreshed the page, it disconnected from my notebook.

In this case, you need to either import the perviously exported setting or enter the api key and environment Name, then click the sync button to reconnect to the notebook.

Should image browser be working now? I still can't get it to work  "Error Encountered error while start image_browser."
I get the same error as before

oh, you need to run rm -rf /storage/sd-webui-infinite-image-browsing to clean up, then run cd /notebooks && git pull && cd image_browser && bash control.sh reload to start image browser.
Next time you should be able to start image browser normally.

IT WORK !!!
Thanks man!


Download to F4, D5 and Pi0 fails

On Chrome - click to download to any of the last 3 images in the Choose Your Hardware dialog box (F4, D5, Pi0).
Nothing happens.  :-(

@shakao - FYI

might be related to https://github.com/microsoft/pxt/commit/fe1f21cccee8de698e5638e26b000eebf4288114


React Native Datepicker is undefined or type is invalid -- expected a string

I am trying to run an example but getting error https://docs.nativebase.io/Components.html#date-picker-def-headref
React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
When i console.log(DatePicker) in constructor then it says undefined so anyway DatePicker is not working

Please follow instructions mentioned in issue template
Failing which this issue will be closed

Also, probably you are searching for Date Picker with wrong version of NativeBase
Check release notes before using any new component / prop

Thanks it is working now i was using older version.


migrations: update networkpolicy_guidance_2 package name

Description
Should be m109tom110
Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
CI

Thanks @RTann. But it was already fixed here: #3784


Regression in 7.2.0 - basePath missing from URL in UI

With <script<EMAIL_ADDRESS>we see the following correct URL in the browser, and the exerciser works correctly:

Whereas with latest (or <script<EMAIL_ADDRESS>we see the URL loses the basePath and the exerciser is broken:

I'm working to isolate the code change, but as this is a potentially regression for other users I wanted to get it raised while I investigate.

The regression is caused by this change https://github.com/mrin9/RapiDoc/commit/504af8a5eaaa8f1b995e0522bf8ac155feb1dcf4
Re-instating json-refs in spec-parser.js instead of swagger-client fixes the problem.
Digging further to see what's different about the jsonParsedSpec output from the two tools.

I believe the bug is a one-liner:
https://github.com/mrin9/RapiDoc/blob/8ecfb4d778064b441660e7af51f78078eee5bc66/src/utils/spec-parser.js#L22-L26
See the bottom line overrides the output from convertedSpec.openapi back to specObj.spec in the case of a Swagger 2.0 -> OpenAPI 3.0 conversion.
If I simply put the jsonParsedSpec = specObj.spec before the if block, the problem is resolved.
    jsonParsedSpec = specObj.spec;
    if (specObj.spec.swagger) {
      convertedSpec = await converter.convertObj(specObj.spec, convertOptions);
      jsonParsedSpec = convertedSpec.openapi;
    }

I've submitted a PR - fyi @mrin9 in case you're able to review and generate a release for this.
I think this will hit anyone using the hosted latest RapiDoc since 7.2.0 was pushed, who have Swagger 2.0 definitions.

thanks a lot for spotting it early and the quick PR.
did a bug-fix release 7.2.1 and added a test case to avoid this in future


Is it possible to create our own WebTorrent and STUN Server?

yes you can, for the Tracker uWebTorrentTracker, i'm not sure if it's a good idea about the STUN server, especially if you know that the STUN server used in p2p-media-loader is provided by Google.

In order for  a peers to communicate with each other, they need to know each other's IP and port number, but in most cases devices are behind of NAT (it gives them a local IP) and firewalls or antivirus that block certain ports . so what STUN server do is to discover the device public IP and port, it's like you ask him, hey STUN server what's my public IP address and he respond with your public IP address. then you move and use that public IP and port to communicate with other peers.

There are few open source WebTorrent servers and many STUN servers available. You can run your own instances of the servers, configure P2P Media Loader to use them and create your private P2P cloud.
STUN is optional component but without it your peers most likely will not connect. It is used by browsers to do ICE.

Open source WebTorrent trackers: bittorrent-tracker, uWebTorrentTracker

Thanks for the info!


Request: Not modifying Jumplist on format

Thank you for this awesome plugin! Makes code 100x more readable.
One thing that I've noticed that trips me up is that the top of the file is added to the jump list after a format. Usually what will happen is I'll:

Autoformat the file
Hit C-o

And instead of going to where I last was (whether that is another file or another place in the current file) I'll go to the top of the file. I was wondering if it'd be possible to not modify the jump list, it's a small thing but I would appreciate it! Thank you!

I guess we could investigate under what circumstances the top of the file is added to the jumplist and then remove it from the jumplist, something like that.


Seems to work fine with newer Aeson

Built fine anyway. There are no tests right?

Would you mind having a look at the build failure? https://travis-ci.org/tibbe/ekg/jobs/67548295

@tibbe oy. I'll kick it around it.

@tibbe this error only occurred on 7.4.2 and I have no idea why Aeson broke that code only for that version of GHC. Do you need to maintain 7.4 compat going forward?
The error is about:
System/Remote/Json.hs:63:49:
    Couldn't match expected type `A.Object'
                with actual type `M.HashMap k0 v0'
    In the return type of a call of `M.insert'
    In the second argument of `($)', namely `M.insert str metric m'
    In the expression: A.Object $ M.insert str metric m

Well, fine, but why would that only be inconsistent in 7.4.2?

We can probably drop 7.4 soon (I keep compat with the latest 3 major GHC releases). I have no idea what caused the failure. I haven't had time to look into it.

@tibbe Proposed plan:

Bounce the TravisCI build (you'll need to do this, I don't think I can) and see if it passes this time (I suspect it won't based on the nature of the error)

If it does, fine. If not, I'll drop 7.4.2 from the .travis.yml and add 7.10. If it passes with those versions, we can proceed.

How does that sound?

SGTM. I just kicked the travis bot.

@tibbe Seems g2g now, I squashed the history.


Code Coverage

Increase code coverage to 100% for ContainerTest.php
Increase speed of tests by using phpunit setUp instead of creating a bunch of containers per test case

hmm.


Not working on Big Sur and later

Doesn't seem to have any effect although it reports everything is fine and status command reports the expected result (according to my usage)

Yep, same here. The program seems to have its own version of the Do Not Disturb status that is quite independent from the macOS system status.

has any one come up with an alternative solution? perhaps using applescript?

@br1ttle check out https://github.com/vitorgalvao/tiny-scripts/blob/master/calm-notifications.
brew install vitorgalvao/tiny-scripts/calm-notifications

Not working on Monterey too

Is there a working alternative for Monterey?

@ryan-efendy https://github.com/sindresorhus/do-not-disturb/issues/9#issuecomment-985931317

Hey folks, this solution worked quite nicely for me: https://github.com/sindresorhus/do-not-disturb/issues/9#issuecomment-991956241
It's this repo: https://github.com/arodik/macos-focus-mode
It uses a MacOS shortcut but has a pretty nice CLI.
I will be replacing do-not-disturb-cli with it in my Pomodoro project: https://github.com/sprzedwojski/pymodoro


switch tokenizer implementation with pure js and more compatible js-tiktoken

closes https://github.com/transitive-bullshit/chatgpt-api/issues/570

tested, works

When will this get merged?

Thanks @masterkain
It seems @dqbd has multiple version of his port, and I wasn't able to find the differences clearly documented anywhere. (@dqbd/tiktoken, js-tiktoken, and tiktoken). Langchainjs seems to be using js-tiktoken, so we'll go with that for now.
Would love for this to be clearer, though.


Tear down and initialization functions

Include an option to run initialization and tear down functions when registering services in convey
app.poll(service, channel, interval, initialization)
and
app.close(service, channel, teardown)
and then
function initialization(channel, callback) {}
function teardown(channel, callback) {}

This is a possible addition to the API that we'll consider adding

Ideally we would have an interface like so

serviceBusService = function() {
  var serviceBus = new ServiceBusClient();

  function init(queueName, done) {
    //do work to initialize the queue named 'queueName'
  }

  function teardown(queueName, callback) {
    //do work to shut down the queue named 'queueName'
  }

  function poll(queueName, dispatch) {
    //poll a queue named 'queueName'
  }

  function push(queueName, body) {
    // push data 'body' to queue named 'queueName'
  }

  return {
    init: init,
    teardown: teardown,
    poll: poll,
    send: push
  }
}

and when you declare the service it's like
app.use("serviceName", serviceBusService());

+1
On Thu, Jan 29, 2015 at 4:17 PM, kingozzy24<EMAIL_ADDRESS>wrote:

Ideally we would have an interface like so
serviceBusService = function() {
var serviceBus = new ServiceBusClient();
function init(queueName, done) {
//do work to initialize the queue named 'queueName'
}
function teardown(queueName, callback) {
//do work to shut down the queue named 'queueName'
}
function poll(queueName, dispatch) {
//poll a queue named 'queueName'
}
function push(queueName, body) {
// push data 'body' to queue named 'queueName'
}
return {
init: init,
teardown: teardown,
poll: poll,
send: push
}
}
and when you declare the service it's like
app.use("serviceName", serviceBusService());
—
Reply to this email directly or view it on GitHub
https://github.com/azuqua/convey/issues/1#issuecomment-72131419.

This is interesting. Obviously it wouldn't be app.use. I'm thinking app.set('service', serviceBusService())


Make running "Set Up CumulusCI" an option

I have created a Docker image with Salesforce DX, jq and Cumulus CI :
https://hub.docker.com/repository/docker/rupertbarrow/salesforcedx-cci
I use this to run Github Actions more rapidly : installs do not have to made for each action run.
Could we make it an option to not run the "Set Up CumulusCI" step ?  You could probably easily add an optional input with default value=true to run the task, and accept value=false to not run it.

I can contribute a solution, if this design is ok for you

Hmm - what if the Set Up CumulusCI step checked to see if the desired versions were already present in the environment and then did nothing? That would avoid having to add another option.

Did you look into this David ?


AttributeError: Can't pickle local object 'ADE20KDataset.init..'

Hi, I downloaded all the available Model Zoo, and ran the evaluation.py. However, there are some bugs in my project. Please, give me some suggestions and solutions.
Thanks.
The error is following:
Traceback (most recent call last):
  File "evaluation.py", line 279, in <module>
    sample = next(iter(val_loader))
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\site-packages\torch\utils\data\dataloader.py", line 279, in __iter__
    return _MultiProcessingDataLoaderIter(self)
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\site-packages\torch\utils\data\dataloader.py", line 719, in __init__
    w.start()
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\multiprocessing\process.py", line 112, in start
    self._popen = self._Popen(self)
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\multiprocessing\context.py", line 223, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\multiprocessing\context.py", line 322, in _Popen
    return Popen(process_obj)
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\multiprocessing\popen_spawn_win32.py", line 89, in __init__
    reduction.dump(process_obj, to_child)
  File "D:\Software\Code\Anaconda3\envs\SA-Scene-Recognition\lib\multiprocessing\reduction.py", line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object 'ADE20KDataset.__init__.<locals>.<lambda>'

@Lollipop9z If you're use Windows, please consider switching to Ubuntu.
Some strange error with pickle often occur when your OS environment is inconsistent with the author's recommand operation system.

But I still have other error on the Ubuntu with the GPU. Please, help me. ^&^
Traceback (most recent call last):
  File "evaluation.py", line 300, in <module>
    val_top1, val_top2, val_top5, val_loss, val_ClassTPDic = evaluationDataLoader(val_loader, model, set='Validation')
  File "evaluation.py", line 76, in evaluationDataLoader
    outputSceneLabel, feature_conv, outputSceneLabelRGB, outputSceneLabelSEM = model(RGB_image, semanticTensor)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/Semantic-Aware-Scene-Recognition/SASceneNet.py", line 165, in forward
    e1 = self.encoder1(x)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/container.py", line 92, in forward
    input = module(input)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torchvision/models/resnet.py", line 41, in forward
    out = self.conv1(x)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 320, in forward
    self.padding, self.dilation, self.groups)
RuntimeError: CUDA out of memory. Tried to allocate 338.50 MiB (GPU 0; 5.79 GiB total capacity; 3.59 GiB already allocated; 340.94 MiB free; 307.46 MiB cached)

But I still have other error on the Ubuntu with the GPU.
This error told me that there is no memory of the GPU, but it still have. Please, help me. ^&^
Traceback (most recent call last):
  File "evaluation.py", line 300, in <module>
    val_top1, val_top2, val_top5, val_loss, val_ClassTPDic = evaluationDataLoader(val_loader, model, set='Validation')
  File "evaluation.py", line 76, in evaluationDataLoader
    outputSceneLabel, feature_conv, outputSceneLabelRGB, outputSceneLabelSEM = model(RGB_image, semanticTensor)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/Semantic-Aware-Scene-Recognition/SASceneNet.py", line 165, in forward
    e1 = self.encoder1(x)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/container.py", line 92, in forward
    input = module(input)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torchvision/models/resnet.py", line 41, in forward
    out = self.conv1(x)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/lollipop/anaconda3/envs/SA-Scene-Recognition/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 320, in forward
    self.padding, self.dilation, self.groups)
RuntimeError: CUDA out of memory. Tried to allocate 338.50 MiB (GPU 0; 5.79 GiB total capacity; 3.59 GiB already allocated; 340.94 MiB free; 307.46 MiB cached)

The solution is modify the batch_size being small, such as 2, 4, 6, in the config file.
For example:
/Config/config_MITIndoor.yaml

VALIDATION:
    PRINT_FREQ: 10
    BATCH_SIZE:
        TRAIN: 100
        TEST: 2


build: enable consistent type imports eslint rule

@dwelle I tried running the auto fix in the codebase and it affects the whole code base ~240 files hence I have made the e changes only for example and excalidraw-app folder which shouldn't be a problem.

@dwelle I tried running the auto fix in the codebase and it affects the whole code base ~240 files hence I have made the e changes only for example and excalidraw-app folder which shouldn't be a problem. However lint check might fail due to this.

I have pushed it for entire codebase


Conflict between this doc and domain management in admin portal

When setting up a domain for GCCH in the admin portal, the docs indicate to use the office365.us endpoints. However, the domain management interface lists the commercial endpoints under Outlook.com in the admin portal.
Which set is correct?

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

ID: b1a8b9e7-32c2-9e42-5f9d-8824bd15ab8e
Version Independent ID: 4e4ba710-bb50-569d-34a6-8ef122d57aa6
Content: DNS records for Office 365 GCC High - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/dns-records-for-office-365-gcc-high.md
Service: microsoft-365-enterprise
GitHub Login: @dzazzo
Microsoft Alias: dzazzo

Hi Tom - the *.office365.us endpoints are the correct ones to use. We’ll log a bug for the portal team to update those endpoints in the admin center.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Tom Moser @.>
Sent: Monday, December 5, 2022 5:00:01 PM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: David Zazzo @.>; Mention @.>
Subject: [MicrosoftDocs/microsoft-365-docs] Conflict between this doc and domain management in admin portal (Issue #10460)
When setting up a domain for GCCH in the admin portal, the docs indicate to use the office365.us endpoints. However, the domain management interface lists the commercial endpoints under Outlook.com in the admin portal.
Which set is correct?

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

ID: b1a8b9e7-32c2-9e42-5f9d-8824bd15ab8e
Version Independent ID: 4e4ba710-bb50-569d-34a6-8ef122d57aa6
Content: DNS records for Office 365 GCC High - Microsoft 365 Enterprisehttps://learn.microsoft.com/en-us/microsoft-365/enterprise/dns-records-for-office-365-gcc-high?view=o365-worldwide
Content Source: microsoft-365/enterprise/dns-records-for-office-365-gcc-high.mdhttps://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/microsoft-365/enterprise/dns-records-for-office-365-gcc-high.md
Service: microsoft-365-enterprise
GitHub Login: @dzazzohttps://github.com/dzazzo
Microsoft Alias: dzazzo

—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/10460, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHXJ3FDTME6J4HETFAX7HF3WL2FZDANCNFSM6AAAAAASU5BUJE.
You are receiving this because you were mentioned.Message ID: @.***>


Fix typos, grammar and wrong used words.

Fixed typos and grammar. It seems that was translated by Google Translate, and was horrible.
Fixed that.

Thanks, @effexts!!


old CHAOSS handbook links REPLACED with new links

AdityaRaimec22<EMAIL_ADDRESS>Signed-off-by: AdityaRaimec22<EMAIL_ADDRESS>This PR...
Changes

Screenshots
(prefer animated gif)
Checklist

[ ] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Fixes #

Thank you for contributing @AdityaRaimec22! Unfortunately, the issue was assigned someone else.
You can find more open issues here: https://github.com/badging/badging-bot/issues.
To avoid having several people unknowingly working on the same issue, we advice that if you would like to fix an existing issue, you should leave a comment on it to indicate. Also, you didn't sign off your commits. Please read the CONTRIBUTING.MD for more information.

Here's how to fix a failing DCO check.


doc change: build.xml -> release/build.xml

The doc references a top level build.xml, which doesn't exist any more.

This is a mess, please ignore...


Keep Retrying Connection Failures

While downloading videos from YouTube I would like the program to resume when it fails to connect to the website either due to a random error (EOF) or a network change (switching WiFi connection or proxy). Don't make the program abort then and just keep trying until it can continue to download.
This is useful to reliably download everything over the night or something instead of coming back and seeing that it stopped when it shouldn't. Note that it won't be stuck in an endless loop since it's never a deterministic error. The network change issue can be reproduced easily however the EOF one is very random.
Thank you

Duplicate of #4240.


Publish github action pipeline fails as required tokens are not set

Seems that called workflows don't have access to the required variables. The caller workflow has to capture them and pass them on to the callee it seems
see
https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations
[NPM PUBLISH] Using custom version number for unstable packages: 0.5.1-unstable-20220[62](https://github.com/hyperledger/caliper/runs/7009099279?check_suite_focus=true#step:4:63)2170900
[NPM PUBLISH] Publishing<EMAIL_ADDRESS>with tag "unstable". Attempt 1/5:
NPM_TOKEN must be set when publishing to the public NPM registry.
[NPM PUBLISH] Failed to publish package<EMAIL_ADDRESS>(attempt 1/5)
[NPM PUBLISH] Error: Failed to execute "/home/runner/work/caliper/caliper/packages/caliper-publish/artifacts/npm-publish.sh" with return code 1.
[NPM PUBLISH] Publishing<EMAIL_ADDRESS>with tag "unstable". Attempt 2/5:
NPM_TOKEN must be set when publishing to the public NPM registry.
[NPM PUBLISH] Failed to publish package<EMAIL_ADDRESS>(attempt 2/5)
[NPM PUBLISH] Error: Failed to execute "/home/runner/work/caliper/caliper/packages/caliper-publish/artifacts/npm-publish.sh" with return code 1.
[NPM PUBLISH] Publishing<EMAIL_ADDRESS>with tag "unstable". Attempt 3/5:
NPM_TOKEN must be set when publishing to the public NPM registry.
[NPM PUBLISH] Failed to publish package<EMAIL_ADDRESS>(attempt 3/5)
[NPM PUBLISH] Error: Failed to execute "/home/runner/work/caliper/caliper/packages/caliper-publish/artifacts/npm-publish.sh" with return code 1.
[NPM PUBLISH] Publishing<EMAIL_ADDRESS>with tag "unstable". Attempt 4/5:
NPM_TOKEN must be set when publishing to the public NPM registry.
[NPM PUBLISH] Failed to publish package<EMAIL_ADDRESS>(attempt 4/5)
[NPM PUBLISH] Error: Failed to execute "/home/runner/work/caliper/caliper/packages/caliper-publish/artifacts/npm-publish.sh" with return code 1.
[NPM PUBLISH] Publishing<EMAIL_ADDRESS>with tag "unstable". Attempt 5/5:
NPM_TOKEN must be set when publishing to the public NPM registry.
[NPM PUBLISH] Failed to publish package<EMAIL_ADDRESS>(attempt 5/5)
[NPM PUBLISH] Error: Failed to execute "/home/runner/work/caliper/caliper/packages/caliper-publish/artifacts/npm-publish.sh" with return code 1.
[NPM PUBLISH] Aborting, could not publish package<EMAIL_ADDRESS>Error: Process completed with exit code 1.

reading the docs, the solution may be as single as just
  publish-caliper:
    uses: ./.github/workflows/publish.yml
    secrets: inherit
    needs: unit-tests

ie adding the line secrets: inherit


Confusing statement as if Load Balancer has "connection reuse" functionality

This is a suggestion for correcting a sentence in order to eliminate misreading.
Problem statement

Outbound connections can burst. A backend instance can be allocated insufficient ports. Without connection reuse enabled, the risk of SNAT port exhaustion is increased.

As per this statement in the docs, it looks like Load Balancer offers a function so called [connection reuse] for preventing SNAT port exhaustion, however, it doesn't. This statement just means that we need to care about the implementation of our applications not to overuse SNAT ports by means of, for example, connection pooling. BTW, Tips on app side are described here.
The problem here is that it looks as if Load Balancer offers connection reuse functionality, while it doesn't.
Idea
I believe the sentence should be corrected as like the below:
- Without **connection reuse** enabled, the risk of SNAT port exhaustion is increased.
+ Without any consideration on application side (e.g. connection pooling), the risk of SNAT port exhaustion is increased.

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

ID: a96406ce-2ffc-cdb3-1234-5154ccd5cc76
Version Independent ID: f840a092-e1bf-16e2-1916-4270cdfbbcdd
Content: Source Network Address Translation (SNAT) for outbound connections - Azure Load Balancer
Content Source: articles/load-balancer/load-balancer-outbound-connections.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

Thanks for the feedback! I have assigned the issue to the content author to investigate this and update the document as appropriate.

Fixed paragraph with https://github.com/MicrosoftDocs/azure-docs-pr/pull/178214
#please-close


test: verify no clear bits can be found when all bits are set

Increases test coverage for membitmap.hpp from 96.4% to 98.2%

The tests are passing now, I'll just merge this


Add system.jdbc.views table

Presto has system.jdbc.tables
but there is no equivalent for views.
Say you want to count how many views you have.. there is no way.
Also even if you want just to verify view exist.. you must do select from the view there is no metadata table to use for that.

You may want to use <catalog>.information_schema.views table.

Would the following be helpful?
SELECT * FROM system.jdbc.tables WHERE table_type = 'VIEW'

The JDBC tables are intended for use by the JDBC driver. It’s recommended to use the information_schema tables instead.

catalog.information_schema.views

awesome. wasn't aware of it.


Update publish:gitlab:merge-request action to allow commit updates and deletes

At the moment, publish:gitlab:merge-request Scaffolder action provides the ability to create merge requests that add new files to a Git repository. We have a scenario where we would like to be able to both update and delete of the existing files.
Feature Suggestion
Provide an additional input parameter for the publish:gitlab:merge-request that defaults to 'create' but could also be 'update' or 'delete'. This parameter would be passed to the creation of Git commits in the action's handler. Currently hardcoded value is 'create'
Possible Implementation
I am happy to create a PR if you agree that this would be useful.

OK, great - I'll create a PR soon

Great, we were looking for it.

we are interested in this update

I think that this has been done and shipped. Closing. Feel free to re-open or comment if there's something missing!


Stats: Add MeasureMap.recordWithTags.

Fixes https://github.com/census-instrumentation/opencensus-java/issues/1758.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4125599). Click here to learn what that means.
The diff coverage is 85.71%.

@@            Coverage Diff            @@
##             master    #1759   +/-   ##
=========================================
  Coverage          ?   82.23%
  Complexity        ?     1981
=========================================
  Files             ?      291
  Lines             ?     9181
  Branches          ?      885
=========================================
  Hits              ?     7550
  Misses            ?     1336
  Partials          ?      295

Impacted Files
Coverage Δ
Complexity Δ

...io/opencensus/implcore/stats/MeasureToViewMap.java
86.07% <ø> (ø)
26 <0> (?)

...ava/io/opencensus/implcore/stats/StatsManager.java
96.42% <100%> (ø)
8 <0> (?)

.../io/opencensus/implcore/stats/MutableViewData.java
88.31% <100%> (ø)
4 <0> (?)

...a/io/opencensus/implcore/stats/MeasureMapImpl.java
100% <100%> (ø)
14 <3> (?)

.../src/main/java/io/opencensus/stats/MeasureMap.java
42.85% <33.33%> (ø)
3 <1> (?)

Continue to review full report at Codecov.

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

Chatted with @dinooliva offline, we're wondering whether (and how much) we could benefit from the new API in terms of 1. performance and 2. readability, given that the new API pretty much does the same thing as the current record API with TagContext.
To give a comparison between the two:
Suppose we have a static TagKey:
TagKey FRONT_END_KEY = TagKey.create("my.org/front_end");
StatsRecorder recorder = Stats.getStatsRecorder();
Tagger tagger = Tags.getTagger();

Record with the current API:
TagValue value = TagValue.create("ios");
TagContext context = tagger.currentBuilder().put(KEY, value).build();
recorder.newMeasureMap().put(MEASURE, 1.0).record(context);

Record with the new API:
TagValue value = TagValue.create("ios");
List<Tag> tags = Arrays.asList(Tag.create(KEY, value));
recorder.newMeasureMap().put(MEASURE, 1.0).recordWithTags(tags);

The two code snippets above are exactly equivalent. The first one creates a new TagContext from current, while the second creates an extra Tag and a List.

My concerns with this feature are 1. I don't see a clear benefit in terms of performance and readability and 2. it introduces another way to define tags, which potentially adds confusion to our APIs.

Closing. As previously discussed, in Java particularly, adding a recordWithTags API doesn't offer us much benefits over using the existing Tag APIs. Will reconsider this if we saw a real need.


Fix background (again) for ipywidgets

For #11060
I had removed this by accident on my refactor for how widgets were rendered.
Also fixed a problem with the functional tests running with real jupyter (caused by the IInterpreterSecurity stuff). @karrtikr you fixed the functional tests when running without jupyter, but it's a little bit more tricky than that. Some of our services a registered differently when using real jupyter (Nightly Flake tests)

Going to port this to release


Add an icon for the executable

I've yet to think of ideas for it.

Done in ecc2f01210cc6ac3ae99cb3c68f2f1bda3ded737.


Inactivating only possible in compact state

when cycling through folding state in status bar, I cannot deactivate it in expanded state

last one before inactive is folded, so source code stays folded, eventhough it says inactive.
I inactivate it because I don't want to see folding, so my suggestion is expand when inactive

Thank you for your feedback. I understand that not being able to directly switch from an expanded state to inactive was an inconvenience. In response to this, I've made some updates to the Fold Ninja extension.
Now, when you click on the Fold Ninja toolbar icon, a menu will appear, offering several options including the ability to change directly from an expanded status to inactive. This should provide a more flexible and user-friendly experience.
Please try out the new version and let me know if it resolves your issue. Your feedback is valuable, and I'm happy to make further adjustments if needed. If everything is in order, please let me know so we can close this issue.


loop doesn't work when displaying three slides out of four total slides

Hi,
When the total number of slides is set to 4, slidesToShow: 3, and slidesToScroll: 1, the loop option stops working. This was mentioned in the docs, which is probably what's happening
loop
... Automatically falls back to false if slide content isn't enough to loop.

This can be reproduced in the sandbox:
https://codesandbox.io/s/embla-carousel-loop-react-forked-7200i?file=/src/js/index.js
Two questions:

What's the reasoning for this restriction? There's definitely enough slide content to loop. I was expecting to see [1,2,3] on initial render, [2,3,4] on next click, and [3,4,1] on next click.
Is there a workaround for this?

Thanks,
Patrick

Hello Patrick (@patrickhuang94),
Thank you for your question. The documentation is correct, Embla Carousel will check if there are enough slide content to make the loop effect work without any flickering (white space showing).
Now, assuming you have 4 slides, you would think that setting the CSS like this:
.embla__slide {
  min-width: 33.33333%;
}

/* ...or a nicer way to put it */

.embla__slide {
  min-width: calc(100% / 3);
}

...would result in enough content to make the loop effect work. But this isn't the case. Because the viewport is 100%. And the slide content for 3 slides (we subtract 1 slide because it's needed to flip around for the loop effect to work) will be 33.33333 * 3 = 99.99999%. Additionally, when I log the slide sizes calculated from the boundingClientRect, the browser gives 33.331691216316074. Which isn't even 33.33333%.
I hope this explanation makes sense.
Best,
David

Great, thanks for the quick response and explanation! Cheers.

Hi again Patrick (@patrickhuang94),
If you want to perform a check to know if slide content is enough to loop or not, there's an unofficial way of doing this:
embla.dangerouslyGetEngine().slideLooper.canLoop();

Just in case you want to duplicate slides based on the boolean it returns.
I hope it helps.
Best,
David

Another solution that I just thought of, instead of duplicating slides:
Currently my code looks like this

    {children.map((slide, index) => (
        <div
            key={index}
            className="embla__slide"
            style={{
                minWidth: `${100 / slidesToShow}%`,
            }}
        >

{slide}

    ))}

which, whenever slidesToShow = 3, will resolve to 33.33 * 3 = 99.99% (and thus never hit the required 100%). I can probably add some custom logic that rounds the min width to 33.34% whenever we receive a slidesToShow input of 3

Hi Patrick (@patrickhuang94),
Of course! You can solve this in different ways, but I guess the bottom line is that the library won’t do it for you 🙂. As I mentioned you can use the unofficial hint I demonstrated in this comment.
Did you solve it for your use case?
Kindly,
David

@patrickhuang94 I'm closing this issue if you have no objections?

@patrickhuang94 I’m closing this as resolved. You’re welcome to open this issue again if you don’t agree.
Best,
David

this solution doesn't work

if (notEnoughToLoop) {
  // Duplicate your slides
  embla.reInit(); // Re-init embla to pick up all the slides including the duplicated ones
}```

Hi @piotrgora,
Please refrain from just saying “This doesn’t work”.  As mentioned here:

the bottom line is that the library won’t do it for you, if you’re looking to make your carousel loop even if the slide content isn’t enough

Of course this doesn’t work  right off the bat, you have to replace the comment in that piece of code with actual code that duplicates your slides.

Hello @davidjerleke,
it was late and after this comment I understood that reInit will duplicate the slider elements itself (don't know why I thought that, but I must have misunderstood the comment above reInit )
I wrote my library using the canLopp() information and everything works fine.
Could you change the text from "Duplicate your slides" to something similar to "Write code/function here that will duplicate slider elements".
I didn't read the whole thread and missed the information you quoted.
Sorry and thanks for the quick reply!

Hi @piotrgora,

Could you change the text from "Duplicate your slides" to something similar to "Write code/function here that will duplicate slider elements" and change dangerouslyGetEngine() to internalEngine()?

Thanks for the constructive feedback. I've updated the code snippet and the comment now 🙂.

I wrote my library using the canLoop() method and everything works fine.

Glad to hear that you solved it 🎉.

I didn't read the whole thread and missed the information you quoted.

Sorry and thanks for the quick reply!

No worries! I hope I didn't come across as rude. My only intention was to convey that I was missing crucial information to understand you.
Best,
David


[CI] Exit with the error status code from xcodebuild

See https://github.com/supermarin/xcpretty#usage -- xcpretty will suppress xcodebuild's status code so we need to add && exit ${PIPESTATUS[0]}.
Test Plan: Look at Travis CI results and see that an error with the Obj-C tests is properly reported as a CI error.

Let's see the CI results

Travis tests are green because the 2nd retry of the Obj-C tests actually succeeded. Not sure if this diff truly fixes the status code, but it can't make things worse.

Shipit :)
We will monitor stability over time
On Saturday, 23 April 2016, James Ide<EMAIL_ADDRESS>wrote:

Travis tests are green because the 2nd retry of the Obj-C tests actually
succeeded. Not sure if this diff truly fixes the status code, but it can't
make things worse.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/pull/7172#issuecomment-213714716

@facebook-github-bot shipit


Make yamllint consistent with Composer

@b-chu Thanks, Brian, for creating these changes. We will hold this PR off for a week or so since there are huge changes coming into the main for the Delta Table reader and then we will re-base this on top of main after that.


Auto-redirect to /admin if accessing / - setting

Code of Conduct

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

Submission validity

[X] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Summary
I do not like to add trailing /admin to url for the admin page. Therefore I propose to update .htaccess as proposed. There an also be a setting that would enable redirect to admin page from /.
Motivation
I do not like to add trailing /admin to url for the admin page.
Describe the solution
I propose to generate .htaccess as follows:

# BEGIN YOURLS
<IfModule mod_rewrite.c>
RewriteEngine On
RewriteBase /

# for /
RewriteCond %{HTTP_HOST} ^YOUR_DOMAIN$ [NC]
RewriteCond %{QUERY_STRING} ![a-zA-Z0-9]
RewriteRule ^$ https://YOUR_DOMAIN/admin [L,R=301]

# for links
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^.*$ /yourls-loader.php [L]
</IfModule>
# END YOURLS

That's personal preference. Some people actually use the root page to display informations, or stats. Some people redirect their short domain root to their main domain. To each their own.


Complains about duplicate variable even though I use let instead of var

The following code generates a duplicate variable: something warning even though the code is perfectly fine.
if (condition) {
    let something = 1;
} else {
    let something = 2;
}

Awesome. Thanks.


Publish image hash for verification on download page

Hello everyone,
I just downloaded opnSense but i can't verify the code integrity because there is no information regards to verification hash.
Its bit difficult to for me to trust mirror websites hosting images without verification hash.
I suggest you guys to publish SHA256 or any other hash to verify the image integrity as soon as user select all 3 "Fast Download selector" option on your download page. So that user can verify the image as soon as download complete.
This way we can easily track the illegal changes / code injection.
Thanks

hashes in mails:
https://lists.opnsense.org/pipermail/announce/2017-July/000081.html
http://marc.info/?l=opnsense-announce&m=150003353632019&w=2
hashes on website:
https://opnsense.org/opnsense-17-7-rc1/
https://forum.opnsense.org/index.php?topic=5494.0
We also added SHA256 digests for the upcoming 17.7 images, which can be verified with a public RSA key we'll publish via the same channels.
Lastly, we will update the download page to also show this sort of info.
Cheers,
Franco


Does not look up from the current file's directory to find .jsbeautifyrc

Description
When the .jsbeautifyrc is in current directory - it works fine. When .jsbeautifyrc is in a parent directory - it does not apply.
Steps to Reproduce
I use the following simple config file. My global config is tab = 2 chars, in the following .jsbeautifyrc config it is 4 chars. When I do beautify (inside .js file) in the same directory where the .jsbeautifyrc is located it apply 4 chars tab as expected. But with the same js file in a subdirectory (where there is no .jsbeautifyrc and no other files at all) it beautifies with 2 chars tab (as in global settings).
{
  "indent_size": 4,
  "indent_char": " ",
  "other": " ",
  "indent_level": 0,
  "indent_with_tabs": false,
  "preserve_newlines": true,
  "max_preserve_newlines": 2,
  "jslint_happy": true
}

I believe it does not look up from the current file's directory to find .jsbeautifyrc in contrast to what is stated in the README.
Thank you for great work on this extension!

I have this same issue. The plugin will only find and use the .jsbeautifyrc file in the user home directory. If it does not exist there, an error is thrown. It does not find the project's own .jsbeautifyrc file.

I will be adding many tests to #864 to ensure that the settings are being properly extracted from both package settings and .jsbeautifyrc files. Thank you for making this issue aware to me and providing steps to reproduce.

I have to add, that actually I found that it sometimes work and some times not. For example, I have a good .jsbeautifyrc file in the project root, and it does not affect subdir. Then I it to the subdir, now it works. Till now everything as I wrote in the issue.
But then I delete all .jsbeautifyrc file from project and then copy a fresh new .jsbeautifyrc file (with identical content) into project root - and now it works even in subdirs.
So, it is not a deterministic behavious, or at least not idempotent one. I couldn't figure out the real cause.

I'd recommend that you take a look at the following functions and see if you can make any changes that fixes this experience for you:

findFile: https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/index.coffee#L563-L578
getConfig: https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/index.coffee#L632-L668

I would be happy to review and merge a pull request that resolves this issue. Thanks!

Important to everyone experiencing this issue: please provide the debugging information in a Gist as described at https://github.com/Glavin001/atom-beautify/blob/master/ISSUE_TEMPLATE.md#debug
I will be using this information to both debug and ensure thoroughness of my tests. Thank you.

I have added tests that demonstrate that this feature is working. Please provide your debugging information and I can investigate this further. Thank you.

Published to v0.29.0


remove #define RGFWDEF and make the #undefs only happen for their OS (RGFW)

While working on RGFW support, before I found out I could use RGFW with Raylib without compiling RGFW on it's own I needed to add this line #define RGFWDEF.
This line makes it so the RGFW functions are not inline'd. This could cause potential performance issues for RGFW. I also made it so the #defs and #undefs required to get RGFW with raylib only happen if they're needed for that OS. This is a very minor change but it prevents any unforeseen issues.

This should hopefully be the last PLATFORM_DESKTOP_RGFW related PR I do for a while.

@ColleagueRiley Nice! Thank you very much for the improvement! 👍


Action is not working on Netflix/servo

https://github.com/drodriguezhdez/servo/runs/429161721

Problem was the project is using a TestNG version not supported.
Upgraded TestNG version of the project to 6.4, fixed the problem