[QUESTION] Logging ExecutionContext's data

Does this library support logging the user associated with the request?
My application uses a custom @nestjs/passport strategy to handle authentication:
class MyStrategy extends PassportStrategy(BasicStrategy, 'custom') {
  async validate(username: string, password: string): Promise<IUser> {...}
}

When I need to access the user in a guard, for example, I do the following:
class PermissionGuard implements CanActivate {
  canActivate(context: ExecutionContext) {
    const args = context.switchToHttp();

    const request = args.getRequest();
    const user = request.user as IUser;
    ...
  }
}

Is there a way to automatically add the user to the log context?

that's just regular middlewares order. You need set user somehow before pino module set it's own middlewares, and configure log to add custom property. look at pino-http parameters.
I guess main problem would be calibrate modules order for proper middlewares order, because there is no guaranty in nestjs for middlewares order set on different modules


Run the windows bat but not working

I see math-with-slack.js is on disc.

MathJax has shut down

@goatandsheep  but they are using a newcdn now so that shouldn't be a problem

Can confirm it still works for me

Sorry, I'm mistaken.

Do it manually if the bat isn't working. See here and #3.


positive test case fails

A known bad jar is the newrelic java agent 7.4.0 using log4j 2.11.2
https://download.newrelic.com/newrelic/java-agent/newrelic-agent/7.4.0/
Analysing newrelic-agent-7.4.0.jar
Jar file not affected by CVE-2021-44228!

Thanks a lot for reporting @johnv-git. This has been resolved in 839460ff99013757922c83f1eb3b1055278ef10d


Avoid unnecessary include of windows.h from zbuild.h

zbuild.h is included from every .c file of zlib-ng, which forces every translation unit to parse all windows system includes only to be able to typedef ssize_t. This change removes windows.h include from zbuild.h and ssize_t is instead defined in-line with equivalent defines from windows.h

Seems clean to me, passes all relevant CI tests.


🔄 Unused exported types not being reported when consumed in the same file

Prerequisites

[X] I've read the relevant documentation
[X] I've searched for existing issues
[X] I've read the issue reproduction guide

Reproduction url
https://stackblitz.com/edit/github-ckjg6m?file=README.md
Reproduction access

[X] I've made sure the reproduction is publicly accessible

Good version
5.31.0
Bad version
5.32.0 onwards
Description of the regression
Unused exports of types that are used in the same file are no longer reported. However, other unused exports not used in the same file continue to be reported as expected.
The behaviour cannot be fixed by setting explicitly setting ignoreExportsUsedInFile: false in the config.
In the attached repro, I have created two exported functions, and three exported types. The unused internalFunction is correctly reported, however only the ThisInterfaceDoesFail interface is reported, and not the ThisInterfaceDoesNotFail interface, despite it being an unused export.
I've identified this as the first bad commit: 14d2cfc2773e1615373dd0e69c152c3edb7d1255, released in 5.32.0.

Thanks @adamrutherfordshaw! Unfortunately it's a tad less straightforward so we've needed a few more iterations to get it right. Any chance you could give this version a shot in your project(s) before we release?
npm i -D https://pkg.pr.new/knip@80537c5

Just pulled down and tested in two projects and the repro I linked, works great 👌 Thanks! 🙂

:rocket: This issue has been resolved in v5.36.7. See Release 5.36.7 for release notes.
Using Knip in a commercial project? Please consider becoming a sponsor.

Thanks for confirming!


How to change operation name in the graph？

Hello，Do you knwo how to change operation name in the graph？Which api should I use？

To change operation name use graph.rename_node, to change operation type use node.change_op_type.

Hello，I use TF1.15. And I get AttributeError: 'Graph' object has no attribute 'rename_node'

See how to use it in examples: https://github.com/CODAIT/graph_def_editor/blob/76e994726630773b8e889c26c49d552b24c71813/examples/coco_example.py#L263

@vlasenkoalexey
Thanks your reply. But I found that this example operates on frozen graph.
Do you know how to change the node name in checkpoint or savedmodel?


run failed

In file included from mediapipe/examples/desktop/demo_run_graph_main_gpu.cc:26:
./mediapipe/framework/port/opencv_video_inc.h:88:10: fatal error: opencv2/optflow.hpp: No such file or directory
88 | #include <opencv2/optflow.hpp>
|          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Target //mediapipe/examples/desktop/face_detection:face_detection_gpu failed to build
Use --verbose_failures to see the command lines of failed build steps.

Seems like you use opencv4 and hit this issue. I barely remember that opencv4 removed opencv2/optflow.hpp after 4.1.0.
Since MediaPipe recommends using opencv 3 for all platforms, we don't pay too much attention to supporting opencv 4 anymore.
You probably can rewrite line 88 of /mediapipe/framework/port/opencv_video_inc.h as the following to see if that solves the problem. Otherwise, switching back to opencv3 should just work.
#include <opencv2/video/tracking.hpp>

We are closing this issue for now due to lack of activity.

Hi I am facing a similar issue can you pls help..
In one program..
I have these includes..
#include "opencv2\face\facerec.hpp"
#include "opencv2\face.hpp"
#include "opencv2\optflow.hpp"
and these are said as not found.. can you pls help.. i have installed opencv3.1.0 so that i dont face this but am still having this issue.
thank you..

Just a quick note about OpenCV version support, coming from the above ticket that refers here.

If OpenCV 3 is the endorsed version, then the OpenCV version recommendations here must mean something else than what they may seem to mean to my small mind.

Hi, sorry to bother you. But after I commented line 89 and replaced it with #include <opencv2/video/tracking.hpp> , and build hand_tracking_gpu, it says
./mediapipe/framework/port/opencv_video_inc.h:94:10: error: 'optflow' has not been declared
   94 |   return optflow::createOptFlow_DualTVL1();

I'm on Ubuntu 20.04 and using opencv 4.1.2 (I've also tried opencv 4.2 and opencv 3.4, and none worked)
Should I build opencv 3.2 from source code?
Thank you :)

one should common those lines in ...../mediapipe/mediapipe/framework/port/opencv_video_inc.h:
 // namespace cv { // inline Ptr<DenseOpticalFlow> createOptFlow_DualTVL1() { //   return optflow::createOptFlow_DualTVL1(); // } // }  // namespace cv

thanks!

Very good, solved the problem of switching to Opencv4。

install libopencv-contrib-dev library can solve this problem


Column selection mode

Type: Feature Request
Please bring back the previous hybrid mode of selection related to column mode selection.
VS Code version: Code 1.74.3 (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T16:59:02.252Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: No

"hybrid mode"?  What was that?

It was a problem with my keyboard input to VSCode not working properly not to VSCode. Closing.


Update deployment.md

To ease the reading; created hiccups to me while reading.

@MIhirMishra Thank you for your contribution. Please sign the CLA so we can accept your PR for review.

@MIhirMishra Thank you for your contribution. Please sign the CLA so we can accept your PR for review.

Page preview:
https://deploy-preview-25936--kubernetes-io-master-staging.netlify.app/docs/concepts/workloads/controllers/deployment/#rollover-aka-multiple-updates-in-flight

Page preview:
https://deploy-preview-25936--kubernetes-io-master-staging.netlify.app/docs/concepts/workloads/controllers/deployment/#rollover-aka-multiple-updates-in-flight

Assigning @kbhawkey since she is already engaged on this. Thanks @kbhawkey!!!
/assign @kbhawkey

Assigning @kbhawkey since she is already engaged on this. Thanks @kbhawkey!!!
/assign @kbhawkey

/unassign

I logged issue #26923.


need one more option Exactly the same as on Gif

I find only left to right and right to left sliding
but no option as on gif
I mean sliding numbers(not their positions)

Same problem here

Update: in order to let it behave correctly, it turns out I have to do the following every time before initiating a "startCounting()":
numberView.font = .systemFont(ofSize: 20)
Where 20 is your desired font size of course.


Is this possible to run agent from docker-container

Gentoo Linux (the only true server linux distro) is not supported by Docker Cloud currently. There's also no dockercloud-agent in repos (both official and docker layman repo). I've done several manual attempts to build it from sources, but seems there's some dependency mess.
I guess it should be relatively easy to run it from docker container, but since I believe there's hard interaction with host machine docker itself would anybody help me running it this way? passing -v /var/run/docker.sock:/var/lib/docker.sock won't help?

@gutsal-arsen
It is possible to run the agent in a container, but it won't be supported officially. We are looking into    moving the current implementation of the cloud to a docker native solution, and the agent will be deprecated soon.
I have done some POC on running the agent in a container in tutum cloud, and it should work in Docker Cloud as well with some changes of the code. If you are interested, you can see the code here as a reference: https://github.com/tutumcloud/tutum-agent/tree/containerization

@tifayuki thank you. Will apparently check it now.


Improve Self referencing loop exceptions on properties by including a recommendation

The solution to a self-referencing loop on most Properties is to add an ignore reference loop attribute, so the Exception may as well be there to tell novices how to most easily resolve it. Most devs running into this for the first time are likely to be surprised at the depth of Json.Net in having ways of handling this, and it may lead them to explore JsonProperty(ReferenceLoopHandling more deeply.

I wholeheartedly endorse this, I ran into this problem before and was scratching my head as to what could've been circular.
And adding more information is always helpful.


Spanish translation

Hi,
I'm interested in helping with the spanish translation.

We area a group of Scala users in Mexico. We will be doing it at https://github.com/scalamx/scala_school.

Hey, let us know when you finish your translation via a pull request and we can see if it needs any changes.

@poguez Do you think your translation is in a good state? We would love to have your contributions :)


jnp.sum signature doesn't have promote_integers argument

Description
As noted from the jax.numpy.sum function here in the documentation
The function should receive the promote_integers parameter but the binding of the function doesn't have it.
import jax.numpy as jnp
from inspect import signature as sig

x = sig(jnp.sum)
print(str(x)) # (a, axis: Union[int, Tuple[int, ...], NoneType] = None, dtype=None, out=None, keepdims=None, initial=None, where=None)

Also calling the function with this parameter will always return type error.
import jax.numpy as jnp
from inspect import signature as sig

x = jnp.array([[1, 2, 3], [4, 5, 6]])
promote_integers = False

y = jnp.sum(x, promote_integers=promote_integers)
# y = jnp.sum(x, promote_integers=promote_integers)
# TypeError: sum() got an unexpected keyword argument 'promote_integers''

Not sure if this parameter can be added to the function's binding or be removed from the documentation.
Thanks
What jax/jaxlib version are you using?
0.3.14
Which accelerator(s) are you using?
CPU
Additional system info
Linux
NVIDIA GPU info
No response

The promote_integers argument was added in https://github.com/google/jax/pull/12382, which was part of the JAX v0.3.20 release. If you're using JAX v0.3.14, the argument will not be supported.

Thanks for clarifying


Maelstrom problem

Is it possible for you to fix this script so that the auto repair and collect starts working again?
It is broken in the 15.3 update that just came,

i'm sorry but I don't have time to work on this, will try to fix it when I have spare time

updated


Crash within dispatchBlock:(dispatch_block_t)block forModule:(NSNumber *)moduleID in  RCTBridge.m

Crashing while generating a payload on line 1330. Interestingly all the variables are pointing to 0x0 or nil.

Looks like another text layout crash (look at the stack in be bottom right corner).

Closing this out since https://github.com/facebook/react-native/commit/0689c0790edcc9cb8494e3f15300e2743556f442 hopefully fixes the source of the text crashes.


Azure-ad Discovery keys must be reset after sometime

I have used open-id from azure-ad for my app and I am using this library, but after some months I am unable to log in, and a I have to perform a GET request to the URL for the discovery keys, copy them exactly as they appear, but also noticed that they haven't changed and are the same but I have to do this everytime this happens.
Have someone experienced something similar and how did you solve this problem? There is no much documentation in the official docs from azure.
Library version: 3.1.4
My app is in Angular 5.

I'm experiencing this as well, although I did not notice if the keys are the same or not.  I'll check for this the next time it happens.  To resolve, I have top copy / paste keys as well.

Yes, this is a pitty. Normally, the API would just read the dicsovery document and the key document it points to. But Azure AD does not allow this as it does not support CORS.
I plan a solution for this where you can point to another CORS-enabled service that returns this data.
For the time being, you have to copy/paste the key or you can also disable Token validation on the client side by using NullValidationHandler instead of JwksValidationHandler.
I guess, im most cases, security relys on the server.


Allow access to peer X.509 certificate subject field CN and certain non-DNS SANs

Extend XCM to allow access to peer X.509 certificate subject field and non-DNS Subject Alternative Names (SANs) on established TLS sessions within a TLS, BTLS, or UTLS XCM connection.
In particular, it should be possible to retrieve

the CN part of the subject field
All SANs of the following types: directoryName and rfc822Name

The new API should be a natural extension to the already-existing API to access dNSName SANs (i.e., the tls.peer_names attribute).
Consider if a new list type socket attribute value type should be introduced. Such a socket attribute value type may be a good idea. Even if not strictly required to fit these X.509 SANs, it will be useful and likely also required to have for future (e.g., binary element-type) lists. This would deviate from the colon-separated string approach to list encoding used by tls.peer_names.
See RFC 5280 for more on the SAN types.

One could introduce a new level "tls.peer_cert", where all the certificate-related information is available. Also, the subject could be a new level.
"tls.peer_cert.subject.cn"
"tls.peer_cert.names"
"tls.peer_cert.emails"
"tls.peer_cert.directories"
For list-type nodes, it would be useful if the elements could be access directly, with something like
xcm_attr_get(conn, "tls.peer_cert.email[0]");
If "list" is added a value type, one could also ask if a "map" (or "directory") value type should also be added. In that case, what is this map's relationship with the xcm_attr_map.h type?

Implemented in a series of commits, the last being 5a04ab947d901feb0c65f64d10685a2e1c450bea.


solution for problem statement 5

Created pr for issue #24

Hello.. your solution is correct but will you please upload this file in the solutions folder as instructed previously? Then we can merge your PR.

Your pull request is merged. Happy Contributing


Undefined is not an object (evaluating 'this.hex')

Error :  TypeError: undefined is not an object (evaluating 'this.hex')
Usage:
import Pie from 'react-native-pie'

<Pie
                        radius={80}
                        sections={[
                            {
                                percentage: 10,
                                color: '#C70039',
                            },
                            {
                                percentage: 20,
                                color: '#44CD40',
                            },
                            {
                                percentage: 30,
                                color: '#404FCD',
                            },
                            {
                                percentage: 40,
                                color: '#EBD22F',
                            },
                        ]}
                        dividerSize={6}
                        strokeCap={'butt'}
                    />

RN version: 0.63.4
react-native-pie: 1.1.2
Device: Android  10 (emulator)

Killing the app and opnening it again seems to fix the issue


Migrating MigraDoc to .NET Standard 2.0 based on this fork of PDFSharp?

I've not studied this in-depth but it seems to me that the only thing preventing MigraDoc from targeting .NET Standard 2.0 is the lack of a PDFSharp port due to it still referencing System.Drawing. Would a fork of MigraDoc referencing this PDFSharp fork and targeting .NET Standard 2.0 be feasible (with only a subset of the features ofc)?

Definitely. See the discussion in #16 on where this fork is (hopefully) heading.

I've actually just finished getting this integrated and building, just need to write up some tests... looks like everything should be there minus the desktop preview, native windows printing, and charting support. Seems like charting could be added back and probably in a LOT more robust fashion

Here's what I've been working on: https://github.com/tudorserbanconnatix/MigraDoc It's a very basic change to MigraDoc that involved retargeting all projects to .NET Standard and referencing this fork of PDFSharp instead of the original, plus a couple of fixes.

 Didstopia fork for me currently just has a Migradoc folder within. Seems to me like having raw pdf functionality is valuable, but often times (at least for me) I've always just referenced migradoc anyway.
Ultimately I think the migradoc could stand to have quite a bit of refactoring to make it more pleasant to deal with, but first things first.
Maybe @Dids can chime in with how to move forward with migradoc. I just tested the migradoc functionality, and it works, but was going to just roll up a large example document in migradoc and figure out a way to legitimately test it.

If I understood the question correctly (if there was one, hehe), I'd rather keep them as two completely separate projects, instead of bundling them together.

Any news on this?

I guess #16 is also related to this?

@Dids : Not sure if you are addressing me or @ronnyek. For my own needs what I have here is good enough: https://github.com/tudorserbanconnatix/MigraDoc

Thanks! In that case I'll close this one for now.


Convert nodelink to typescript

Closes #27
TODO:

[ ] Open an issue to add more type safety, this is still most plain js at the moment
[ ] Tests need to be fixed now or with another issue, won't pass CI as it stands (looks like a path error for the types.ts file)

Thanks for the help @waxlamp and @subdavis!


Proble installing composer libraries

I need some help i cant pass instaling from here
I'm on ubuntu server 21.4
php 7.4

@KaiserSKL Hello, I think you are running composer install in the wrong way.
You need to run "composer install" in the terminal.
After that, you need to check if the "vendor" folder is inside the application folder. This means that the composer successfully installed all the dependencies of the project.

thnx a lot i had some issuel so I reconfig the lock dependencies and all went smoothly


Input parameter X in Diffusor.transform

Hi @davisidarta,
Great work! I think this package will  really help expand how we are using the KNN graph structures of single-cell datasets.
I was taking a closer into the Diffusor class and found that you don't actually  use the parameter X in Diffusor.transform. Does this mean that the data can only be self-transformed? Or probably I'm missing something?
Best,
Parashar

Hi @parashardhapola ! Thank you for your interest in TopOMetry and for your kind words! :)

I think this package will really help expand how we are using the KNN graph structures of single-cell datasets.

So I hope! It stores graphs and decomposes them into new dimensionality reduced bases within the TopOGraph object. I'm doing this entirely on my own while on hospital rounds, so I'm sorry the fuzzy and cknn transformers are not so well documented, although you'll find extensive docstrings in the code.

I was taking a closer into the Diffusor class and found that you don't actually use the parameter X in Diffusor.transform. Does this mean that the data can only be self-transformed? Or probably I'm missing something?

That's a consequence of scikit-learn transformers used as a base, but I can actually change it to be optional. The transform() step basically calls an adaptive eigendecomposition of the kernel or the transition matrix, so it only needs these graphs to operate, and not the full data. As you've noticed, this means transformation requires a fitted affinity graph. Although Nystrom's out-of-sample extension is possible, it is not yet implemented, as I'm still considering possible extension methods using landmarks.
TopOMetry is in its very early days, and I'm considering adding new features that others would find useful. Would an extension of the graph-learning process to out-of-sample data be what you're thinking about, so as to do this iteratively in memory batches within Scarf?


HyperBian SSH not working

[x] I confirm that this is an issue rather than a question.

Bug report
SSH doesn't work after a clean install of HyperBian.
Steps to reproduce

install HyperBian
Add ssh file to boot
ssh pi@host
enter password 'raspberry'

What is expected?
Login should work
What is actually happening?
Please note that SSH may not work until a valid user has been set up.

See http://rptl.io/newuser for details.
<EMAIL_ADDRESS>password:

after entering password:
debug1: Authentications that can continue: publickey,password
Permission denied, please try again.

System

Hyperion Server:

Build:             (HEAD detached at 2.0.13) (GitHub-dfd9438/5dc696b-1653218333)
Build time:        May 22 2022 12:55:01
Git Remote:        https://github.com/hyperion-project/hyperion.ng
Version:           2.0.13
UI Lang:           en (BrowserLang: en-US)
UI Access:         expert
Avail Screen Cap.: dispmanx,framebuffer,qt
Avail Video  Cap.: v4l2
Avail Services:    boblight,cec,effectengine,forwarder,flatbuffer,protobuffer,mDNS,SSDP,borderdetection
Config path:       /home/pi/.hyperion
Database:          read/write

Hyperion Server OS:

Distribution:      HyperBian GNU/Linux 11 (bullseye)
Architecture:      arm
CPU Model:         ARMv7 Processor rev 3 (v7l)
CPU Type:          Raspberry Pi 4 Model B Rev 1.5
CPU Revision:      c03115
CPU Hardware:      BCM2711
Kernel:            linux (5.15.32-v7l+ (WS: 32))
Root/Admin:        false
Qt Version:        5.15.2
Python Version:    3.9.2
Browser:           Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.5 Safari/605.1.15

Did you consider that the keyboard layout is English by default?
That often creates problems that the password is invalid as the ‚y‘ character is at a different Position (e.g. on a German keyboard, you would need to type ‚z‘)

Yes sure. I also tried with copying the password.
the ssh file is removed after looking at the SD card again. Normal?

Just create a "userconf"-file on the boot partition with containing this line:
"pi:$6$ZGEEFHP4I1jNqzvS$2pKzPMEVvX85M3gq2q6P5sLroqDu1cfpBCObiRmC.rzQuT.KZ.llYh3/ZBii98DUHLGpYC9bU7cCwZQe7bCiD1"
and the "ssh"-file too
then it should work

Yesss that worked :) thx

https://github.com/hyperion-project/HyperBian/pull/8

It didn't work for me.
Adding a "userconf" file with the content @LeLunZ said above and an empty "ssh" file, when I try to ssh using the "pi" user and "raspberry" as password, I got permission denied.

did you try the method from my last post?

Yes, exactly what you posted and still can't ssh :(

which error do you get?

After three attempts to log in, I get the "Permission denied" error and I was kicked out of the ssh session :shrug:

can confirm tp modding's fix works on a new install.
For the next guy, would it be possible to link to this issue from https://docs.hyperion-project.org/en/user/HyperBian.html#hyperbian-ssh please

A new HyperBian image was just released with default user and password changed to align with Raspberry Pi OS security measures.
Furthermore, the respective HyperBian installation documentation was updated, too.


make plugins easier

it would be helpful if you made creating plugins easier/ made a readmy specifically for it
thanks!

Added to the todo list:
https://github.com/Athlon1600/php-proxy-app


Fixed Learn to play translation error.

The text showed garbled with no spaces in my game.
It looks like A15 uses "Tutorial" as the translate key, not "LearnToPlay"
Changed LabelKey in the learn to play menu entry's def file to "Tutorial."
From the decompiled code.
string label = "Tutorial".CanTranslate() ? "Tutorial".Translate() : "LearnToPlay".Translate();
optList.Add(new ListableOption(label, (Action) (() => MainMenuDrawer.InitLearnToPlay()), (string) null));
optList.Add(new ListableOption("NewColony".Translate(), (Action) (() => Find.WindowStack.Add((Window) new Page_SelectScenario())), (string) null));

Oh, but it breaks other languages. Must be something else going on. Closing PR will check into it.


Remove title bar

Need to move the following before this can be done:

Account chooser button (move to empty spot at top of sidebar)
Refresh button (move to issue list)
Search bar
New saved search button
New issue button (remove, since there's one at the top of the issue list)


Update es-419.coffee

Changed to formal language for teachers survey.
$120 dollars? D: OMG perhaps a little expensive for third worlders like me.
Is it finally done? :D

Almost there (line 258)


feature: support GPT4ALL-J

GPT4ALL-J is Apache-licensed, which makes it very appealing as could be re-distributed freely, however, they seem to have hardforked ggml/llama.cpp, so it's not compatible with the llama.cpp backend

go-gpt4all-j is here: https://github.com/go-skynet/go-gpt4all-j.cpp

this is getting really close, but needs refactoring on the API code


Fix - Add permit if receive nack command.

Motivation
Add permit if receive nack command.
The availablePermits will reduce when resend message to consumer.
Modifications
Verifying this change
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc

Please take a look. @gaoran10 @mingyifei

Please take a look. @gaoran10 @mingyifei

availablePermits are processed at the time of consumption.

availablePermits are processed at the time of consumption.

CreditManager done not work.
You can try it. When consumer client send nack command, broker will resend message and decrease availablePermits, after 1000 times sended, will blocked. @mingyifei

ok, you can test the multi-bundle implementation, the previous version will be deprecated in the future.

availablePermits are processed at the time of consumption.

CreditManager done not work.
You can try it. When consumer client send nack command, broker will resend message and decrease availablePermits, after 1000 times sended, will blocked. @mingyifei

AmqpMultiBundlesChannel


sample: properly set LRO type

Fixes #2457.

Codecov Report

Merging #2463 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2463      +/-   ##
============================================
+ Coverage     86.55%   86.56%   +<.01%
- Complexity     5291     5292       +1
============================================
  Files           458      458
  Lines         21078    21080       +2
  Branches       2300     2301       +1
============================================
+ Hits          18245    18247       +2
  Misses         2009     2009
  Partials        824      824

Impacted Files
Coverage Δ
Complexity Δ

...gle/api/codegen/transformer/OutputTransformer.java
89.45% <100%> (+0.08%)
36 <0> (+1)
: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 4061233...4af412b. Read the comment docs.


All slides content going to the middle

I'm using
hieroglyph==0.7.1
and this is the talk I'm preparing:
https://github.com/AndreaCrotti/unix
So the problem is that all the content go down in the middle instead of having the header on top and the rest in the middle.
I think it's something to do with style but I tried to rebuild some of my old presentations with the same structure and it doesn't happen.
Any idea about what it could be?
To reproduce just checkout that and run "make slides"..
thanks a lot,
Andrea

I also tried with both Python2 and Python3 but same issue, it must be something silly I'm doing
https://raw.githubusercontent.com/AndreaCrotti/unix/master/README.org
but can't really see what yet..

As a short answer/workaround
I fixed that in 'mine' version, which isn't pull-requested, due a trip. But I guess you can use it
--albert
An iPhone is great, but for typing mail

Op 24 aug. 2015 om 15:11 heeft Andrea Crotti<EMAIL_ADDRESS>het volgende geschreven:
I'm using
hieroglyph==0.7.1
and this is the talk I'm preparing:
https://github.com/AndreaCrotti/unix
So the problem is that all the content go down in the middle instead of having the header on top and the rest in the middle.
I think it's something to do with style but I tried to rebuild some of my old presentations with the same structure and it doesn't happen.
Any idea about what it could be?
To reproduce just checkout that and run "make slides"..
thanks a lot,
Andrea
—
Reply to this email directly or view it on GitHub.

Awesome @AlbertMietus it works!
Would be good to understand why as well but for now it's fine I can use this.

Hi Andrea
Will try to go into details later. But can remember it was quite trivial (but it often is, after solving😀)
Better still, it will be merged into mainstream (I guess/assume).
For now, good luck with your presentation
--albert
An iPhone is great, but for typing mail

Op 24 aug. 2015 om 15:59 heeft  Crotti<EMAIL_ADDRESS>het volgende geschreven:
Awesome @AlbertMietus it works!
Would be good to understand why as well but for now it's fine I can use this.
—
Reply to this email directly or view it on GitHub.

Hi @AlbertMietus thanks again..
The only problem now is that the spacing is not enough, see this picture for example

There should be a bit more space between the header and the body of each slide, it's a bit too squashed now..

Sorry no time/computer to look into this.
But fiddle a bit with the ccs, probably the H1 and H2 bottom margin or padding.
Foot luck
--albert
An iPhone is great, but for typing mail

Op 25 aug. 2015 om 17:29 heeft Andrea Crotti<EMAIL_ADDRESS>het volgende geschreven:
Hi @AlbertMietus thanks again..
The only problem now is that the spacing is not enough, see this picture for example
There should be a bit more space between the header and the body of each slide, it's a bit too squashed now..
—
Reply to this email directly or view it on GitHub.

I sort of suspect the issue has to do with not having a theme set. I don't really recall what Hieroglyph does in that case, but I sort of suspect it'd use the "base" theme that all the others build off of. Haven't looked at @AlbertMietus 's fork recently enough to know if that's consistent with the changes he made there.

+1 I'm also experiencing this problem. I used the hieroglyph quickstart and threw some random text into a couple slides and the text on all the slides starts from the middle of the page down.

I also expirience this problem

I found that this is because of margin-top: 465px added in https://github.com/nyergler/hieroglyph/commit/d2ecf96c73005103a16dbf1ab73ac1a3936c1c74
I have workaround this by adding
h2 {
  margin-top: 0;
}

to the http://docs.hieroglyph.io/en/latest/styling.html#custom-css


Silence replicator spec puts

Introduced in #15990
This no longer outputs messages into the test output, and puts the messages under test
before:
...............................Replicating from server id=97000000000140, path=k1/k2 to 1 servers
Settings: {:k1=>{:k2=>{:k3=>"v3"}}}
Done
.....

after:
....................................

updated to use rspec's `expect {}.to output(message).to_stdout

I hadn't seen #16023 - which attempted to hide these messages as well.

Looks like all suggestions have been applied.  Merging.


Connect the donation.html to button with anchor tag

I connect the donation button with donation.html file
you can check and merge it

Thanks 👍

Fixes : #408

Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Premium viewtime script

Coverage decreased (-0.01%) to 56.604% when pulling d809f311c8ae0c77776f49a1079ea02ff81f0b37 on premium-viewtime-script into 1c51fc59605374d7b3cbeb89556299ae6b540faa on master.

Coverage decreased (-0.1%) to 56.621% when pulling 9130bbec292b40db46acc80e4139664f475c2d1d on premium-viewtime-script into 5b4322ed286dfe42e810c865e1c460438d4264f8 on master.

Coverage decreased (-0.1%) to 56.621% when pulling 9130bbec292b40db46acc80e4139664f475c2d1d on premium-viewtime-script into 5b4322ed286dfe42e810c865e1c460438d4264f8 on master.


Automatically initialize Git repositories for new projects - Fails

Did an upgrade to the 1.1.4 . Everything seems to be working fine, except for initializing git repos for new projects.
I get this: [ERROR] Error while getting Gitolite infos, check your SSH keys (path, permissions) or your Git user.
(nothing else useful)
But, it doesn't make any sense... The SSH keys are there and accessible. Pretty much as they were before the upgrade.
Even more, the already existing repos function without an issue.
I did the standard test that people are doing mentioned throughout the issues, everything seems fine.
After 4-5 days trying to figure this out, I'm getting desperate. Any help would be appreciated.

Ok, the SSH keys issue is solved, and the Config Test passes successfully. Everything seems to be in order, except that creating a new repo still fails.
Regardless if it's creating a new project, or a new repo to an existing project, it fails.
git_hosting has nothing, it's empty.

Can you please try with the latest release? (1.2.0)


Taxonomy: Add new categories to blog posts

Tim: send Marie csv of all blog posts, so she can begin adding categories via spreadsheet.
Marie: mark each blog post on csv with new categories available at https://docs.google.com/spreadsheets/d/1QGla_Yw0q92fubPiL9m9j1G8kTq7t01LWWCcw59G7uk/edit?usp=sharing.

I should be able to get started on this next week! When the time comes, if it's possible to get a CSV with title & link for each post, that would be hugely helpful when I start adding categories.
Also, @aparkening, have we made any decisions on how to categorize things like link posts and OOR/10 Years Ago in ALA, etc.? That may be a question for Sara, but I don't remember if those kinds of things really fit in the new taxonomy.
Not sure if this is a question for @aparkening or @murtaugh, but definitely let me know the best way to include multiple categories (separate columns?) and how to indicate which is the primary category (i'm guessing the first one listed would be primary?) Thanks!

Here's the report: http://alistapart.com/reports/all-blog-posts
If you want to add columns for categories, that's cool. Someone will be entering these through EE, there's no way to automate assigning these.

@marieconnelly I think link posts maybe don't need a category (and we aren't really writing new ones of those anyway), but OOR can probably go to Web Industry?

Awesome - thanks, @sarawb!

Priorities have shifted away from taxonomy work.  Closing to revisit another day.


Strict null check

@piotrWichlinskiSoftwaremind @mateusz-garlacz @dagl I've merged Piotrs PR with some fixes to make the CI pass (I've had to disable testing for now, we will have to implement that later). Please rebase all other PRs.


services.btrfs.autoScrub: running scrub prevents system suspend

Describe the bug
When the filesystem scrubbing service configured via services.btrfs.autoScrub is running, the system cannot suspend until the scrubbing is completed.
To Reproduce
Steps to reproduce the behavior:

enable the autoscrub service on some Btrfs file system, e.g. by

services.btrfs.autoScrub = {enable = true; fileSystems = [ "/" ];};

start the resulting service, e.g. systemctl start btrfs-scrub-.service
suspend-to-ram the system: systemctl suspend
the system fails to suspend and a trace appropriating this failure tu btrfs can be found in the system journal

Expected behavior
The scrub should be paused properly when the system suspends.
More context
It's likely that this is an upstream bug. I'll ask around on the btrfs mailing list and will update this issue accordingly.
Metadata

system: "x86_64-linux"
host os: Linux 4.19.98, NixOS, 19.09.1970.6d9a4a615ee (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.2
channels(root): "home-manager-19.09, nixos-19.09.1970.6d9a4a615ee, nixos-unstable-20.03pre210639.e59dcf8ccec"
channels(user): "home-manager-19.09"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
- tasks/filesystems/btrfs.nix

The service probably needs Conflicts=suspend.target sleep.target and the other suspend related targets.

The issue still appears with a 5.4 kernel.
system journal


Issue #884 - the two manpages packages can not be found.

remove dependencies that cant be found.
These packages can not be found when creating the container.

#884

Not required, already fixed just had not made into master.


Remove Go 1.21 polyfills

We can drop these now that the main module requires Go 1.21.
See:

#111

Only the coverage check is red. Merging.


Editing release tag from command line always results as "untagged-XXXXX"

So basically I'm trying to automate my release process by creating a draft "proxy" release and I would like to edit its tag version when I'm about to actually publish the release.
The thing is, I realized that:
1_ When I try to edit anything regarding the release, the tag ends up being something like "untagged-1421c803c66244c71c64"
2_ Can we even change just the tag for a draft release?
STEPS

So first I apply this (which works):

gabu$ hub release create --draft -m "Proxy release" -t "release/e2e-qa" v0.0.1-proxy
https://github.com/<my_team>/<my_project>/releases/tag/untagged-eb4323448616e25da835
gabu$

✅

✅

But when I try to edit something (let's say just the title):

gabu$ hub release edit -m "This is such an amazing title" -t "release/e2e-qa" v0.0.1-proxy
gabu$

✅

❌

Which ends up being the original name of the draft release tag that is untagged-eb4323448616e25da835
And when I try to hub release edit again with the latter tag name, it "works" even though it didn't work at the very first edit attempt.
That said, I tried somehow to edit the tag by just adding an additional argument in the end (just trying here...) but again it would result in no changes except for the renaming of the tag to untagged-eb4323448616e25da835.
Which makes me wonder: is this the expected behavior?
And most importantly... can we actually change the tag of a draft release by command line?
I feel like I'm missing something trivial here...
Thanks in advance!

Thank you for reporting! This seems like it's a bug with GitHub API. I'll update this thread once it's resolved!

can we actually change the tag of a draft release by command line?

No, I don't think this will be possible even with the bug in question fixed. But, you will be able to edit the tag in the web interface. As for the command-line, you could always delete the draft release and create a new one with an updated tag name, containing the original release's title and body.

Running into this as well :(

@mislav Is there a plan to fix this issue? We are running into this issue as part of our release automation effort too (https://github.com/konveyor/move2kube/blob/3edb173611cb806351770003b8ad17eed13aa207/.github/workflows/release.yml#L257).

Sorry that this is biting you all! This issue was reported to the responsible team at GitHub, but I don't know when it will be solved.

Sorry that this is biting you all! This issue was reported to the responsible team at GitHub, but I don't know when it will be solved.

Experiencing same issue even today. Any updates on this bug, seems it originally reported 3 years ago and still open?

As this is still not fixed, can the upstream bug be followed somehow (is it tracked publicly)?


Add option to raise error when rate limited

The motivation of this came from us using Zendesk API as part of background jobs. If we are rate limited, we do not want the job to be hogging the background worker, and instead raise an error and be requeued immediately. This will allow us to handle the error and delay the job for a later time. And also for other jobs to be processed asap.

Thank you for your contribution @tommi-lew . We regret the long delay on merging it.

I just noticed that this is merged! Thanks @ggrossman , better late than never :)


⚠ ClamAV has degraded performance

In 45edf90, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 86 ms

Resolved: ClamAV performance has improved in 295585f.


NPCs are not visible after sometime

After i create npcs and make them work, after sometime they automatically disappear and after sometime they appear again.
This is happening again and again . pleaase help

Happens to me too


Alternate definition of locally compact

This suggestion came in through email

I was looking at the definition of “locally compact” property at $\pi$-base. In my version of Willard’s “General Topology”:
18.1 Definition. A space $X$ is locally compact iff each point in $X$ has a nhood base consisting of compact sets.
And the current $\pi$-base definition is
A space $X$ is locally compact if each point of $X$ has a compact neighbourhood.
These definitions match for Hausdorff spaces, but not even for locally Hausdorff ones! See
https://math.stackexchange.com/a/337657/58818
The $\pi$-base definition coincides with Munkres’ (Sec 29), Kelley (p. 146), and Counterexamples’, though... But Willard’s definition seems overall more in vogue nowadays at least for Operator Algebras.

It's worth considering if this is a point where we want to make another break with counterexamples sim. the T_ns, or just want to add another property (and maybe links between them).

We should definitely list both properties, just as there are two notions of monotone paracompactness depending on author (although I believe we list neither currently). I wonder if "locally" is more commonly used to mean "each point has a relevant neighborhood" or "each point has a relative neighborhood base"; often this is equivalent (e.g. locally metrizable).

This also illustrates why I think mathoverflow and math.SE should be supported citations for the pi-base. This space should be added to the pi-base as the counter-example to "locally compact (Munkres) => locally compact (Willard)"!

Hi. I was the one who made the suggestion by e-mail. Here are another 2 cents.
I would say that "Every point has a nhood basis with such property" is what we end up using in practice for local properties, whereas the weaker version is used when the property pass to subspaces; e.g. Hausdorff and metrizability; Local (path-)connectedness and contrability are also quite useful, and we need the stronger version.
Local normability is funny, since normability passes only to closed subspaces;  so we can only guarantee that a regular space s.t. every point has a normal nhood is locally normal (in the strong sense).
Moreover, even the regular property may be seen as "local property" in the stronger sense: A space is regular iff every point has a nhood basis of closed sets. (and the weaker version is trivial).
Thanks for considering my comment. I really appreciate you work at the $\pi$-base.

@luizgcordeiro That was were my mind was headed as well. I think the convention "locally P" means "has a nbhd base of sets satisfying P" is reasonable, and clarify otherwise. So https://topology.jdabbs.com/properties/P000023 could be renamed "Locally compact by a single neighborhood" and we should add a new "Locally compact", along with some relevant theorems.

I've written this up (along with some other conventions) here: https://github.com/pi-base/contributing/blob/master/CONVENTIONS.md

Looking around, I see that "weakly locally compact" is often used for the single-neighborhood case, vs. "locally compact" for a neighborhood base. So in an upcoming PR I'll adjust the conventions to use "locally P" and "weakly locally P" consistently (while updating https://topology.jdabbs.com/properties/P000023 ).

Steen & Seebach also use the terminology "strongly locally compact" = Every point has a closed compact nbhd.
= properties (2),(2'),(2'') in https://en.wikipedia.org/wiki/Locally_compact_space. = Property P000024 in pi-base.
I understand the desire for consistency in pi-base's terminology.  pi-base currently does not have a term for property (3), which would be useful to introduce.   But if we switch the meaning of "locally compact" and introduce "weakly locally compact" (= (1) in wikipedia page) as proposed, we would have the situation that "locally compact" (= (3) in wikipedia page) would imply "strongly locally compact" and not the reverse, which can be confusing.
What is more common in the literature?

Oh thanks for catching that @prabau; I'd completely blanked that P000024 is already in the pi-base. And this made me realize that https://topology.jdabbs.com/theorems/T000006 is incorrect - it should imply (weakly) locally compact. https://topology.jdabbs.com/spaces/S000029 should be a counterexample.
From what I've seen, "locally compact" is defined as convenient for the subject at hand, and often the spaces considered are Hausdorff so there's no ambiguity anyway.
So perhaps we do this: "strongly locally P" means there's a basis of P neighborhoods, "weakly locally P" means there's a single P neighborhood, and "locally P" is used exactly when those notions are equivalent.

I only have the 1st page access at the moment but https://www.jstor.org/stable/24340250 seems relevant to this discussion.

And now I see that Steen/Seebach's strongly locally compact is not what I'm suggesting either - I was thinking about the definition in https://www.jstor.org/stable/24340250  (-:
After re-reading the March 2019 discussion above though, I think it's just another break we should make from Steen/Seebach for consistency. It just means we also need to rename P24 (and note in its definition the original terminology).

With the current terminology for "strongly locally compact" (= property (2) in wikipedia = every point has a closed compact nbhd), https://topology.jdabbs.com/theorems/T000006 is correct (since the whole space is a closed compact nbhd of any point).
"strongly locally compact" in https://www.jstor.org/stable/24340250 that you mentioned is a different notion, I agree.  Although that reference is not really classical.  (Engelking assumes locally compact spaces are Hausdorff so avoids all the issues)

I lean toward consistency over classical, particularly when (as you say) it's all negligible assuming Hausdorff. But if you'd prefer different adverbs for "weakly/strongly locally P" since "strongly locally compact" is not isolated in the literature in referring to wikipedia's (2) I'm open to ideas.

Well, the weakest condition is wikipedia (1), which we could rename as "weakly locally compact".  Wikipedia (3) is what is suggested to rename to just "locally compact" (which agrees with Willard), so that seems ok.
But then, as you suggest, it would be good to rename P000024 to something else than "strongly locally compact".
From https://en.wikipedia.org/wiki/Relatively_compact_subspace, a "relatively compact subset" is a subset with compact closure.
Wikipedia (2'): Every point has a relatively compact nbhd.  Wikipedia (2''): Every point has a local base of relatively compact nbhds.  So we could call that property (2) "locally relatively compact" maybe?
It matches https://projecteuclid.org/download/pdf_1/euclid.pjm/1102720477 p. 134.  It's only one reference though.
Wikipedia already uses the terminology "strongly locally compact" with Steen/Seebach's meaning in https://en.wikipedia.org/wiki/Particular_point_topology for example.  So we would deviate from that.  Maybe ok.

One more observation: "relatively compact" is a hereditary property.  Every subset of a relatively compact set is relatively compact.  So if there is one such nbhd, there is a local base of such nbhd.  So no need to distinguish between "locally relatively compact" and "weakly locally relatively compact".  (That's the equivalence between (2') and (2'') in wikipedia.)
To confuse things even further, nlab mentions still a fourth notion of local compactness:
"local compactness via compact closures inside neighbourhoods" (def. 2.2 in https://ncatlab.org/nlab/show/locally+compact+topological+space) which is not listed in wikipedia.
I don't think we need to introduce that right now in pi-base.

Trying to summarize all the various names so far and how they fit with the wikipedia article (and nlab),
assuming your proposed convention "weakly locally P" = each point has a nhbd with property P, and "strongly locally P" = each point has a local base of nbhds with property P (and "locally P" when weak and strong are the same).  I think I would prefer a simpler convention, but going with this for the below:
[the numbers below refer to the various definitions in https://en.wikipedia.org/wiki/Locally_compact_space
(1) = "weakly locally compact"
(2) = "locally relatively compact" (same for weak and strong)
(3) = "strongly locally compact"
Wikipedia does not have the corresponding "strongly locally closed-and-compact" (in your proposed terminology).  That would correspond to def. 2.2 in https://ncatlab.org/nlab/show/locally+compact+topological+space.  Maybe it should be added to wikipedia?
The corresponding "weakly locally closed-and-compact" is identical to wikipedia (2) = "locally relatively compact", since if there is a nbhd whose closure is compact, there is one which is closed and compact.
Terminologywise, wouldn't it be simpler to revert to the current  convention that "locally P" means there is a nbhd base satisfying P for each member of the base?  And just introduce "weakly locally P" for the weak case in case it is needed, i.e., in case they are different?
Then:
(1) = "weakly locally compact"
(2) = "locally relatively compact"
(3) = "locally compact"
(new) = "locally closed-and-compact"  (Def 2.2 in nlab)  (maybe introduce this?)

I read the Pi Mu Epsilon article you mentioned and am starting to like your proposed terminology "locally closed-and-compact".  There are two dimensions that can vary: weak vs. strong, and compact vs. closed and compact.  That gives 4 possibilities.
Personally I think it would be simpler to revert to the current convention that "locally P" means there is a nbhd base satisfying P for each member of the base.  And just introduce "weakly locally P" for the weak case in case it is needed, i.e., in case they are different.  What do you think?
So the 4 possibilities would be:

"weakly locally compact" = wikipedia (1)
"locally compact" = wikipedia (3)
"weakly locally closed-and-compact" = wikipedia (2) (which would be the same as "locally relatively compact")
"locally closed-and-compact" (missing from wikipedia, but same as def 2.2 in  https://ncatlab.org/nlab/show/locally+compact+topological+space)

How would that look?

Dropping "strongly" avoids the name collision with Steen/Seebach, so I'm happy with that.
I won't add locally closed-and-compact in #63 but a new PR that includes a counterexample distinguishing it from locally relatively compact would be nice.


Update eksGoRelease readme guide

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


CI research and implementation

Research and implement a proper CI configuration for this project.

Hint: look into how seaport does this.

Passing this to @fkrause98


Sema: Fix crash with SpecializeExpr around something that's not a type

Unfortunately the diagnostic is terrible.
Fixes https://github.com/swiftlang/swift/issues/74858

@swift-ci Please smoke test

null check seems reasonable by I cannot reproduce the crash with example you checked in because DS doesn't exist, if I add DS it doesn't crash with the main compiler from yesterday.

I messed up the test because I first put it in test/decl/typealias/generic.swift. Oops, hang on

@swift-ci Please smoke test

No worries!


great job

hi,ol3-contextmenu is good,thangs for your work.but it can not create second level menu，like File->Open。if it can create second level menu or more,it will be more great.

Hi! This is already in my plans. Keep an eye on it.

Hi! Thanks for you response。Do you hava a schedule on the submenu?

I'm terrible busy trying to release a project of my own, so I can't tell right now about schedule. But I'll dedicate some time as soon as possible.
Have a good day.

@beyondlh I guess it's ready now, please take a look at #19.


Update links with latest patch bundle (.rvp extension)

ReVanced Team (& some other patch bundles) updated their patches & merged integration (API v4).
But patch bundle url is still updating per old integration.
Could you please Update it?

Sure! Do you by chance know the format to use in the patch-bundle.json files for API v4?

UPDATE:
The Compose Manager currently is still using API v19 (the old API) meaning there is not file format for the RVP patch bundles (API v20). So as of right not I'll keep everything how it is until that manager gets updated to v20. oSumAtrIX is who told me this too.

Made the decision to revert .rvp (API v20) patch bundle support until the Compose Manager adds support for it. Until then all the patch bundle sources in this repository will be set to use the latest .jar (API v19) patch bundle type.

Some of the patch bundle sources had to be set to "N/A" due to the their repositories deleting the releases from the .jar (API v19) patch bundles.

There is now support for API v4.


release: v1.0.0a8

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>--
It should solve a failing tests in https://github.com/inveniosoftware/invenio-collections/pull/70

LGTM

Thanks @krzysztof! sudo make release 😉


UI shows orange turn on-off switch when connected to the VPN

When I connected to the VPN, the main UI still shows the orange switch which should only be showed when the user doesn't connect to the VPN. This issue can be seen in the screen recording below:
https://user-images.githubusercontent.com/1403194/146995853-7090f27a-9b18-4931-ab51-f5f7c8b4012f.mp4
My device info:

Lenovo Tab P11, TB-J606L
Android 11
PIA version: 3.12.4 (571) - playstore

hello @kreaninw apologies for the very late reply! Is this something you can still reproduce on version 3.19.2?

hello @kreaninw apologies for the very late reply! Is this something you can still reproduce on version 3.19.2?

Thanks! Sorry, I can't test this issue at the moment because I moved to another service 🙏

I understand. Thank you for taking the time to reply and report it in the first place!


Display codec in download table

have another column displaying codec , this would be beneficial to know the quality difference apart from bitrates.

have another column displaying codec , this would be beneficial to know the quality difference apart from bitrates.

Is this really needed?? Becouse, all the audio files will be downloaded as .mp3 and all videos as .mp4 !

If the file is not being transcoded then it is needed because there is a respectable difference between mp3, opus and m4a codecs. Some people prefer one over the other.


The abortController from the loader function is not working

Describe the bug
The "abortController" from the "loader" function is not working because the "abort" call was removed from the "cancelMatch" function on this commit 3b559fc.

Working on v1.18.3: https://stackblitz.com/edit/github-gowxsm
Not working on v1.18.4: https://stackblitz.com/edit/github-kgyq1m
Your Example Website or App
https://stackblitz.com/edit/github-kgyq1m
Steps to Reproduce the Bug or Issue

Click on the "About" link
Quickly (before the loader is resolved, after 5000ms) click on the "Home" link
Wait 5000ms to see that the loader is resolved, because the "abort" event wasn't triggered

Expected behavior
We expect the signal from the abortController to be aborted, and the "abort" event to be triggered.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: 1.20.4

Additional context
No response

I also found another issue in this working example (https://stackblitz.com/edit/github-gowxsm) with the version 1.18.3. If you click on the "About" link twice quickly, the loader is aborted and is not called anymore, which makes the page not load.


Build fails on FreeBSD: undefined: syscall.Mprotect

While building trufflehog-3.78.1 - these errors occur:
# github.com/wasilibs/go-re2/internal/alloc
vendor/github.com/wasilibs/go-re2/internal/alloc/alloc.go:58:16: undefined: syscall.Mprotect
vendor/github.com/wasilibs/go-re2/internal/alloc/alloc.go:80:18: undefined: syscall.Mprotect

Go-1.22
FreeBSD 14.1

@yurivict Thanks for the report and sorry for the trouble!
I think I know the issue and while I don't know if I'll be able to actually run tests on FreeBSD on CI, I'll add a check to at least compile the code for it.

Thanks again for the report @yurivict - I have released 1.6.0 including the fix for this issue. Sorry for the trouble.

@anuraaga
Thank you for fixing it so quick!


refactor: factor gateway data fetch and solve into a function

A follow-up on https://github.com/prefix-dev/pixi/pull/1767

Progress report is not the best, as fetching repodata for environment will be displayed a fair few times. Is there a way to make this better?

Progress report is not the best, as messages such as fetching repodata for environment can end up being displayed a fair few times. Is there a way to make this better?

Yes but it requires implementing a Reporter.

@baszalmstra I am not sure what this entails. But it does not sound straightforward 🥲 . Is it worth it? Or should we pull the messaging out of the function and wrap it when / it needed where it is used?

Yeah that sounds fair! 👍

Yeah that sounds fair! 👍

Actually that does not work. Even with the messaging outside of the loop, the iterator is still there waiting... Reporter it is then :-)

Rattler-build has a custom implementation here: https://github.com/prefix-dev/rattler-build/blob/a28628bc6d29e227dffc1a64c27fa5f0b50f7dc9/src/render/solver.rs#L136
Its not great but might serve as inspiration!

Hi guys! What is the status of this PR @olivier-lacroix @baszalmstra?


v4 FluentFilteredExtension: Add styles for Site Tree

Another one for me to look at (please feel free to assign this to me).
Add some additional styles so that it is clear which Pages are and are not currently visible as you browse the CMS in any particular Locale.
Something like this (where the "About us" page isn't visible on this current Locale):

I'd also like to look into how easily I could add similar visibility to things like SiteTree dropdown fields, but that may end up being a further improvement later down the line.

I thought we implemented this with the original PR https://github.com/tractorcow/silverstripe-fluent/pull/304

That PR only contained the styles for the "Inherited" and "Locale Drafted" flags. This is a new style to try and illustrate that a Page is not visible on the current Locale.

@tractorcow I am wondering though:
This Extension is not tied to SiteTree, it could be applied to any DataObject. With that in mind, it feels slightly wrong that I would implement the updateStatusFlags() method in this Extension.
I would be happy with either of these options:

I implement the updateStatusFlags() method in this Extension for any time it is applied to SiteTree (probably quite often?).
We leave it to devs to write their own Extension which can make all of the correct assumptions about the DataObject that this Extension has been applied to.

I think 1. is ok; Just document that updateStatusFlags is only intended for cases where this extension is added to SiteTree, and that it can be ignored all other times.


Better large file handling

Large files fail with RangeError (in Chrome)

Haven't seen RangeError in a while, but large downloads seem to stall part-way through (around when the tab hits 2GB ram?)

You should check to see if you are hitting your machine's paging limit.

For a 105MB file, the page uses 540MB of memory. By commenting out bits of the code at a time, I found this breakdown:

90MB just for loading instant.io in chrome (incl. my browser extensions)
210MB from drag-drop/buffer
105MB from create-torrent in webtorrent.seed
105MB from file.getBlobURL in onTorrent on instant.io

We're creating lots of buffers that don't get released, it seems. Ideally, we only need to hang onto one copy of the buffer.

There are still some leaks, but it doesn't look like we're losing any full copies of the file anymore! Progress! There's only one copy of the file in memory.
For a 360MB file:

90 MB for base instant.io page
360 MB for file
160 MB for webtorrent.seed (and leaks?)

cc @jhiesey

We're now running into a bug in Chrome where blobs can't be larger than 500MB. https://code.google.com/p/chromium/issues/detail?id=375297 This looks like it's really close to being fixed though!
I was able to seed a 1GB file in Firefox and stream to Chrome!

This issue was moved to https://github.com/feross/webtorrent/issues/319.


Revert "Revert "Revert "dispatch.summaries by default when possible"""

Reverts vespa-engine/vespa#5568
Down to 23 failing ...

👍


Migrate to COGs for offline map imagery

[x] Show live overlay in offline area selector map.
[x] Use downloader to download images to local file system.
[x] Show images in local filesystem if available.
[ ] #1820
[ ] #1788
[x] Delete remnants of mbtiles downloader and viewer.

For this phase we can reuse the basecmap url specified in the survey

Closing in favor of fine-grained tracking issues #1820 and #1788


Error 'Site' object has no attribute 'settings' after integrating into existing Django project

I followed integration docs from here
http://docs.wagtail.io/en/v2.2.2/getting_started/integrating_into_django.html
and integrated wagtail into saleor project.
But when I try to access home page I get following error

Request Method:
GET

http://localhost:8000/en/

2.0.8

AttributeError

'Site' object has no attribute 'settings'

/Users/synerzip/tutorial/saleor/saleor/core/views.py in home, line 17

/Users/synerzip/tutorial/saleor/env/bin/python3

3.6.3

['/Users/synerzip/tutorial/saleor',  '/Users/synerzip/tutorial/saleor/env/lib/python36.zip',  '/Users/synerzip/tutorial/saleor/env/lib/python3.6',  '/Users/synerzip/tutorial/saleor/env/lib/python3.6/lib-dynload',  '/anaconda3/lib/python3.6',  '/Users/synerzip/tutorial/saleor/env/lib/python3.6/site-packages',  '/Users/synerzip/tutorial/saleor']

Fri, 28 Sep 2018 00:28:13 -0500

Hi @yogesh-kamble - thanks for the report.
It looks like something in your project is setting request.site, most likely because you have django.contrib.sites.middleware.CurrentSiteMiddleware enabled, and this is conflicting with Wagtail's request.site variable. There's a plan to eliminate Wagtail's request.site detailed at https://github.com/wagtail/wagtail/issues/3834#issuecomment-340009510 - see also #4748.


Footer Updated

In this PR, I changed the color of headings in the footer, so through this, we clearly distinguish between headings and Links (https://user-images.githubusercontent.com/103263909/233885554-74895c5a-bd1a-4764-8aa8-09dd06d9b25f.png)

@Saurav-Pant These are some merge conflicts in this PR, please sync the fork repo to the main repo.

Conflict resolved @Mr-Sunglasses

Thx @Saurav-Pant


Fix race in MapKey

Fix race in ManualClock
Fix race in MapKey
MapKey was previously exported in https://github.com/Netflix/spectator-go/pull/42
FWIW, the current MapKey behavior could done more safely by introducing a lock rather than precomputing during construction, as introduced in this change. I chose to avoid the lock on the assumption that users are not depending on the JIT computation behavior.
Make MapKey() thread-safe by precomputing the value during construction.
This does not change any documented behavior of Id, which is immutable
(apart from one case mentioned below). However, the lack of map clone in
Tags() means that previously a caller could construct a new Id, calls
Tags(), modify the resulting map, and then call MapKey(). This would
result in a different output after this change, which captures the tags
at construction.
I believe this is more aligned with the intent of this type. The
documented behavior mentions the lack of clone to warn the user of
modifying the map from Tags(), but does not prevent it.
Interestingly, in the previous implementation, this was likely hidden by the Registry calling
MapKey(), which uses a mutex for registration via a blocking call to NewTimer() or friends. I suspect
that the most common use of Id has been to construct and immediately
register. Since the value for MapKey() was previously precomputed
lazily, subsequent calls after registration were threadsafe.
The use-case that led to this discovery is relying on MapKey() to populate a local cache of Id values for use by PersistentTimers. The cache was introduced to avoid the relatively expensive construction of a PersistentTimer (and its underlying side effects) that may already have been created.
Race example for ManualClock
DEBUG: 2021/03/01 12:54:54 posting data to http://<IP_ADDRESS>:38093, payload 2 bytes
DEBUG: 2021/03/01 12:54:54 response HTTP 200:
DEBUG: 2021/03/01 12:54:54 posting data to http://<IP_ADDRESS>:36927, payload 2 bytes
INFO: 2021/03/01 12:54:54 Timed out attempting to POST to http://<IP_ADDRESS>:36927
==================
WARNING: DATA RACE
Read at 0x00c000280330 by goroutine 67:
  github.com/Netflix/spectator-go.(*ManualClock).Nanos()
      /home/r2/code/ext/spectator-go/clock.go:41 +0x3a
  github.com/Netflix/spectator-go.(*LogEntry).Log()
      /home/r2/code/ext/spectator-go/log_entry.go:54 +0x6c
  github.com/Netflix/spectator-go.(*HttpClient).doHttpPost()
      /home/r2/code/ext/spectator-go/http_client.go:135 +0x74c
  github.com/Netflix/spectator-go.(*HttpClient).PostJson()
      /home/r2/code/ext/spectator-go/http_client.go:143 +0x145
  github.com/Netflix/spectator-go.TestHttpClient_PostJsonTimeout()
      /home/r2/code/ext/spectator-go/http_client_test.go:124 +0xa6e
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1194 +0x202

Previous write at 0x00c000280330 by goroutine 73:
  github.com/Netflix/spectator-go.(*ManualClock).SetFromDuration()
      /home/r2/code/ext/spectator-go/clock.go:48 +0x4d
  github.com/Netflix/spectator-go.TestHttpClient_PostJsonTimeout.func1()
      /home/r2/code/ext/spectator-go/http_client_test.go:108 +0x44
  net/http.HandlerFunc.ServeHTTP()
      /usr/local/go/src/net/http/server.go:2069 +0x51
  net/http.serverHandler.ServeHTTP()
      /usr/local/go/src/net/http/server.go:2887 +0xca
  net/http.(*conn).serve()
      /usr/local/go/src/net/http/server.go:1952 +0x87d

Goroutine 67 (running) created at:
  testing.(*T).Run()
      /usr/local/go/src/testing/testing.go:1239 +0x5d7
  testing.runTests.func1()
      /usr/local/go/src/testing/testing.go:1512 +0xa6
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1194 +0x202
  testing.runTests()
      /usr/local/go/src/testing/testing.go:1510 +0x612
  testing.(*M).Run()
      /usr/local/go/src/testing/testing.go:1418 +0x3b3
  main.main()
      _testmain.go:135 +0x236

Goroutine 73 (running) created at:
  net/http.(*Server).Serve()
      /usr/local/go/src/net/http/server.go:3013 +0x644
  net/http/httptest.(*Server).goServe.func1()
      /usr/local/go/src/net/http/httptest/server.go:308 +0xd8
==================
--- FAIL: TestHttpClient_PostJsonTimeout (0.00s)
    testing.go:1093: race detected during execution of test
DEBUG: 2021/03/01 12:54:54 posting data to http://<IP_ADDRESS>:34955, payload 2 bytes
DEBUG: 2021/03/01 12:54:54 response HTTP 503:
DEBUG: 2021/03/01 12:54:54 posting data to http://<IP_ADDRESS>:34955, payload 2 bytes
DEBUG: 2021/03/01 12:54:54 response HTTP 503:
DEBUG: 2021/03/01 12:54:54 posting data to http://<IP_ADDRESS>:34955, payload 2 bytes
DEBUG: 2021/03/01 12:54:54 response HTTP 503:
INFO: 2021/03/01 12:54:55 registry config has no uri. Ignoring Start request
DEBUG: 2021/03/01 12:54:55 Got 1 measurements
DEBUG: 2021/03/01 12:54:55 Sending 1 measurements to http://<IP_ADDRESS>:38663
DEBUG: 2021/03/01 12:54:55 posting data to http://<IP_ADDRESS>:38663, payload 161 bytes
DEBUG: 2021/03/01 12:54:55 response HTTP 200:
DEBUG: 2021/03/01 12:54:55 Got 3 measurements
DEBUG: 2021/03/01 12:54:55 Sending 3 measurements to http://<IP_ADDRESS>:39485
DEBUG: 2021/03/01 12:54:55 posting data to http://<IP_ADDRESS>:39485, payload 248 bytes
DEBUG: 2021/03/01 12:54:55 response HTTP 202:
INFO: 2021/03/01 12:54:55 1 measurement(s) dropped due to validation errors: key too short: [o] (1 < 2)
DEBUG: 2021/03/01 12:54:55 Got 1 measurements
DEBUG: 2021/03/01 12:54:55 Sending 1 measurements to http://<IP_ADDRESS>:33923
DEBUG: 2021/03/01 12:54:55 posting data to http://<IP_ADDRESS>:33923, payload 161 bytes
DEBUG: 2021/03/01 12:54:55 response HTTP 200:
DEBUG: 2021/03/01 12:54:55 Got 6 measurements
DEBUG: 2021/03/01 12:54:55 Sending 6 measurements to http://<IP_ADDRESS>:33923
DEBUG: 2021/03/01 12:54:55 posting data to http://<IP_ADDRESS>:33923, payload 850 bytes
DEBUG: 2021/03/01 12:54:55 response HTTP 200:
DEBUG: 2021/03/01 12:54:55 Got 6 measurements
DEBUG: 2021/03/01 12:54:55 Got 5 measurements
DEBUG: 2021/03/01 12:54:55 Sending 5 measurements to http://<IP_ADDRESS>:33923
DEBUG: 2021/03/01 12:54:55 posting data to http://<IP_ADDRESS>:33923, payload 724 bytes
DEBUG: 2021/03/01 12:54:55 response HTTP 200:
FAIL
FAIL    github.com/Netflix/spectator-go 1.053s
ok      github.com/Netflix/spectator-go/histogram       0.159s
FAIL

Race example for MapKey() -- this is the test added in this changeset, but without the change to MapKey(). The scope of the test run is limited due to an unrelated test failure in TestHttpClient_PostJsonTimeout that is seemingly only triggered with the race detector enabled. i.e. this failure occurs on master.
➜ go test -race ./...  -run TestId
==================
WARNING: DATA RACE
Write at 0x00c00007efd8 by goroutine 9:
  github.com/Netflix/spectator-go.(*Id).MapKey()
      /home/r2/code/ext/spectator-go/id.go:55 +0x55a
  github.com/Netflix/spectator-go.TestId_mapKeyConcurrent.func1()
      /home/r2/code/ext/spectator-go/id_test.go:31 +0x38

Previous read at 0x00c00007efd8 by goroutine 10:
  github.com/Netflix/spectator-go.(*Id).MapKey()
      /home/r2/code/ext/spectator-go/id.go:20 +0x64
  github.com/Netflix/spectator-go.TestId_mapKeyConcurrent.func2()
      /home/r2/code/ext/spectator-go/id_test.go:35 +0x38

Goroutine 9 (running) created at:
  github.com/Netflix/spectator-go.TestId_mapKeyConcurrent()
      /home/r2/code/ext/spectator-go/id_test.go:30 +0xd8
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1194 +0x202

Goroutine 10 (finished) created at:
  github.com/Netflix/spectator-go.TestId_mapKeyConcurrent()
      /home/r2/code/ext/spectator-go/id_test.go:34 +0x104
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1194 +0x202
==================
--- FAIL: TestId_mapKeyConcurrent (0.00s)
    testing.go:1093: race detected during execution of test
FAIL
FAIL    github.com/Netflix/spectator-go 0.007s
ok      github.com/Netflix/spectator-go/histogram       (cached) [no tests to run]
FAIL

@dmuino Thanks for the additional context on the lazy compute.
Do you think a lock or a sync.Once would be better than the approach I've taken? I'm happy to amend to either.

Do you think a lock or a sync.Once would be better than the approach I've taken? I'm happy to amend to either.

I think sync.Once is exactly what we need here to make it thread-safe. This way we do not need to worry about the temporary Ids.
It might be nice to actually cache the Ids use for Measure anyway to get a small performance improvement anyway but that is unrelated to this PR.

@dmuino I've updated the implementation to use sync.Once. Please note there is a slight change in behavior from the previous lock-free implementation. See the commit message for more detail.
I've also added a commit to fix a couple of incorrect error messages that I discovered when running the tests locally.

though as pointed out in the commit comment the key value for an error would be a bit odd.

@brharrington I may have been unclear. I did not introduce the string-as-error behavior in this change. I wanted to call out that in the previous implementation, an error encountered in MapKey() would have returned an ERR string and subsequent calls would have attempted to recompute. This change caches the ERR string and will not reattempt.
It's possible to avoid the caching, but it will add additional complexity to this method and/or the use of a mutex.

Given the long term plan to delegate most of the logic to spectatord, I do wonder how worthwhile it is to have users investing in performance improvements and related bug fixes plus our time reviewing them for an implementation that should be short lived.

Understood, though I would argue that this particular method will still have utility when spectatord becomes the backend for this library as long as it retains a type for Id.

@dmuino Thanks all for the review. Would it be possible to tag this as 1.1.2 after merge?


Ambari DB password change not supported

added a warning that states that changing Ambari DB's password is not supported as per Vijay Srirangam

@MeranNasser : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MeranNasser : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thank you @JasonWHowell. your suggestion makes sense and I commited the changes

We've handled the request internally. Sorry for the duplicate effort. It should go out this afternoon I think.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/190621

#please-close


add RetroShare

Do you have more information you can add on retroshare? My understanding is that it leaks metadata to an external network observer because it uses the direct IPs for each "friend". (unless also combined with something like Tor)


Return docker container id on driver creation.

Hi,
While debugging some problems, I run into an issue of getting the container id while starting the driver. Is there a way to get the container id, besides the session id when starting the driver?
Thank you

@addrians yes, this is possible. Take a look at /status API, e.g. the following HTTP request (JSON was beautified for readability):
$ curl -s http://selenoid-host.example.com:4444/status
{
    "total": 80,
    "used": 10,
    "queued": 0,
    "pending": 1,
    "browsers": {
      "firefox": {
        "46.0": {
          "user1": {
            "count":1,
            "sessions":[
                {
                    "id": "a7a2b801-21db-4dae-a99b-4cbc0b81de96",
                    "container": "20c0f303dec60943c5779ad3f3af2b2b9cbe832b4a9c8d70745cf1533b848640",
                    "vnc": false,
                    "screen": "1920x1080x24",
                    //...
                 }
            ]
          },
         //....
}

Here container means Docker container ID.

Awesome. I was looking everywhere but the status page. Thank you.


Make the final pdf file visible on the browser

They should be able to view the pdf file on the browser without downloading.

I am using github pages action to publist the pdf on osbm.github.io


Squashing bugs in documentation generation

Updated the API reference
Added a markdown-to-restructuredtext conversion for docstrings via pypandoc, because our docstrings are in a markdown style
Fixed the documentation version number not coming from cirq/_version.py
Fixed the documentation configuration not ensuring cirq is included in the sys path when finding docstrings
Filled in some empty configuration strings
Marked backslash-using doc strings in common_channels.py as r-strings
Fixed several 'duplicate entries' errors related to attribute docstrings
Clarified docstring of PauliString.pass_operations_over
Escaped some modifier characters such as | and ** that were confusing the parser
Deleted some unused documentation files
The build_docs script now clears out old docs instead of complaining that they're present
Tweaked landing page and dev docs

Fixes https://github.com/quantumlib/Cirq/issues/789
Fixes https://github.com/quantumlib/Cirq/issues/857
Fixes https://github.com/quantumlib/Cirq/issues/1107

@tswast you have to give an Approve review for the LGTM to count.


Adding Freetube to Video section

I noticed, Freetube was missing from any of the media categories. This pull request would see it being added to the video category.
Project URL
The project can be viewed here.
Category
It is a YouTube front end, so I would suggest it be applied to the Video category.
Description
Here is a direct quote from the Free Tube site:
"FreeTube is a YouTube client for Windows, Mac, and Linux built around using YouTube more privately. You can enjoy your favorite content and creators without your habits being tracked. All of your user data is stored locally and never sent or published to the internet. FreeTube grabs data by scraping the information it needs (with either local methods or by optionally utilizing the Invidious API). With many features similar to YouTube, FreeTube has become one of the best methods to watch YouTube privately on desktop."

Why it should be included to Awesome macOS open source applications  (optional)
It is not only an open source app, it is also privacy focused which is a massive plus.
Checklist

[ ] Edit applications.json instead of README.md.
[ x] Only one project/change is in this pull request
[x ] Screenshots(s) added if any
[ x] Has a commit from less than 2 years ago
[ x] Has a clear README in English

@josamontiel Could you please update your request. I'm not sure why but I'm seeing a lots of changes.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1663 ms

Resolved: Bofrost.at performance has improved in 37ed57b.


Bug: Getting "already installed" when extension isn't installed

Not sure how to debug this properly for you or give any more context - this is all the output given:

installing extension: vsx:yzhang.markdown-all-in-one
already installed

Not just this extension, it happens seemingly at random. Something seems to be amiss with the state management

For anyone getting similar issues, deleting the ~/.vscode[-oss]/extensions directory and starting again resolves these inconsistencies. Not sure what the root issue is though

Could you reproduce it and give steps to do so?

I think the v0.5.0 should have fixed the issue.


Language Issue

Please change language to English so every one can understand.Thanks for your effort

why? Look at the code yourself,OK?


Possible to forward all broadcast traffic instead of only that of wc3?

Would it be possible? i would love to use this on other games that also use broadcast traffic to find lan clients.
If not, understandable :)

Unfortunately, no.
This program does not just forward traffic but sends its packets (according to Warcraft 3 LAN game network protocol) to find active games and extract information about them. Moreover, this program modifies the network port in some packets for correct operation.
Therefore, this program is created specifically for Warcraft 3 and cannot be used for other games.
If you want to play over a local network with remote players, then I can recommend ZeroTier to create a virtual local network. If there is a problem with a specific game over ZeroTier, then here is a possible solution to this problem.

Thank you so much for your reply, i appreciate it very much :)
I see, that makes a lot of sense. We did try Tailscale, but it does NOT do broadcast traffic. Ill have to look into that article more, thanks for sharing it!


Setting mix compile warning on test

What happened
Solved https://github.com/nimblehq/elixir-templates/issues/61
Insight
1/ Adding the deps.unlock --check-unused to the codebase alias
2/ Adding Code.put_compiler_option(:warnings_as_errors, true) into the test_helper.exs
Proof Of Work
1/ The test is passed on CI
2/ Tested on my local

Minor addition to this PR would be to link to the original article you shared on Slack 💭

Minor addition to this PR would be to link to the original article you shared on Slack 💭

Actually, I added that link in the https://github.com/nimblehq/elixir-templates/issues/61, I mentioned that issue in the PR's description already.


nixos/k3s: fix k3s multi node test

Things done
Seems like the whole problem with the multi-node test was that it set the k3s package to use for the server node, but not for the server2 and agent nodes. This deployed a kubelet newer than the apiserver, thus violating the k8s version skew policy. I think this has just worked until v1.31 but then led to incompatibilities. There is also an issue in k8s where people experienced the same thing with v1.31.
The simple fix was to set the same k3s version for all  nodes in the test. Additionally, I refactored the test a little in a separate commit, mainly to parallelize preliminary tasks to speed up test execution.
However, I don't know why the test worked sometimes anyway with v1.31, but maybe this was caused by another race condition.

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Fixes https://github.com/NixOS/nixpkgs/issues/354211
@NixOS/k3s
Add a :+1: reaction to pull requests you find important.

@ofborg test nixosTests.k3s.multi-node

I could reproduce the error I got when bumping K3s to 1.31.2+k3s1 when running the test in RPi4 in this PR.
Meaning, it's unrelated to 1.31.2+k3s1 bump indeed (we already knew that). But also, that this PR doesn't solve that (etcd) problem. So it's something else.

Ok I removed the commit that fixes the test and put this on draft for now. I will reactivate it once https://github.com/NixOS/nixpkgs/pull/355964 s merged.

@ofborg test k3s.multi-node.k3s_1_28
@ofborg test k3s.multi-node.k3s_1_29
@ofborg test k3s.multi-node.k3s_1_30
@ofborg test k3s.multi-node.k3s_1_31

@ofborg build k3s_1_31.passthru.tests.multi-node k3s_1_30.passthru.tests.multi-node k3s_1_29.passthru.tests.multi-node k3s_1_28.passthru.tests.multi-node

Tests seems good here... I guess, we should go ahead then.

CC @marcusramberg

CC @Mic92 @marcusramberg merge?


Add @lbernick to pipeline approvers list

Changes
Lee was already added as a Pipelines maintainer in https://github.com/tektoncd/community/pull/694
This PR updates the Pipelines OWNERS file to reflect that
/kind misc

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
NONE

/cc @tektoncd/core-maintainers

/retest
/approve

/test pull-tekton-pipeline-integration-tests


Flutter Web - How to control or remove visibility icon added to TextField by browser

How to control or remove visibility icon added to TextField by browser?
When I set obscuretext to true, the browser adds a visibility icon to my TextField, but it's impossible to control or remove it.

Even when I add a suffixIcon manually, it still appears, and still useless

I did the test on the Flutter website to prove that an updated and stable version is being used
Code:
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  static const String _title = 'Flutter Code Sample';

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: _title,
      home: MyStatefulWidget(),
    );
  }
}

class MyStatefulWidget extends StatefulWidget {
  const MyStatefulWidget({Key? key}) : super(key: key);

  @override
  State<MyStatefulWidget> createState() => _MyStatefulWidgetState();
}

class _MyStatefulWidgetState extends State<MyStatefulWidget> {
  late TextEditingController _controller;

  @override
  void initState() {
    super.initState();
    _controller = TextEditingController();
  }

  @override
  void dispose() {
    _controller.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: TextField(
          obscureText: true,
          controller: _controller,
          decoration: const InputDecoration(
            suffixIcon: Icon(Icons.add)
          ),
        ),
      ),
    );
  }
}

Hi @andre799, This is a known issue for Edge Browsers and is being tracked here https://github.com/flutter/flutter/issues/83695, You may follow up there for further updates closing this issue as a duplicate if you disagree feel free to write in the comments and I will reopen it.
Thank you.

Hi @andre799, This is a known issue for Edge Browsers and is being tracked here #83695, You may follow up there for further updates closing this issue as a duplicate if you disagree feel free to write in the comments and I will reopen it.
Thank you.

Okay, I will follow, thank you very much!


Issue-Add Delete Click Handler

Automated Review Comments
Issues in src/tests/snapshots/BitcoinTracker.test.js.snap:

(18, 'Hardcoded string found')
(19, 'Hardcoded string found')
(20, 'Hardcoded string found')
(21, 'Hardcoded string found')
(22, 'Hardcoded string found')
(24, 'Hardcoded string found')
(28, 'Hardcoded string found')
(29, 'Hardcoded string found')
(31, 'Hardcoded string found')
(32, 'Hardcoded string found')
(50, 'Hardcoded string found')
(51, 'Hardcoded string found')
(52, 'Hardcoded string found')
(53, 'Hardcoded string found')
(54, 'Hardcoded string found')
(56, 'Hardcoded string found')
(60, 'Hardcoded string found')
(61, 'Hardcoded string found')
(63, 'Hardcoded string found')
(64, 'Hardcoded string found')
Issues in src/tests/snapshots/CoinList.test.js.snap:
(8, 'Hardcoded string found')
(10, 'Hardcoded string found')
(12, 'Hardcoded string found')
(13, 'Hardcoded string found')
(15, 'Hardcoded string found')
(16, 'Hardcoded string found')
(24, 'Hardcoded string found')
(26, 'Hardcoded string found')
(28, 'Hardcoded string found')
(29, 'Hardcoded string found')
(31, 'Hardcoded string found')
(32, 'Hardcoded string found')
Issues in src/components/Coin.js:
(2, 'Hardcoded string found')
(6, 'Hardcoded string found')
(13, 'Line exceeds 80 characters')
Issues in src/components/CoinList.js:
(2, 'Hardcoded string found')
(9, 'Line exceeds 80 characters')
(10, 'Line exceeds 80 characters')

I got it.


[Bug]: How to hide these logs?

Hi, thanks for making this components. I have a question, can we hide the logs on the browser console?
Tremor Version
3.18.0
Steps to reproduce
Using any tremor npm components
What is expected?
hide the logs even on the development mode
What is actually happening?
too much logs, hard to see my debug log.

What browsers are you seeing the problem on?
Chrome

i found it, i can uncheck the info from Custom levels


Documents IWDG peripheral across all STM32 families

I had to rename registers on stm32fhx3 to match. Complete and generalize the previous IWDG PR.

Looks good, just need to resolve the merge conflict.

Thanks!


Enable localization/modification of "Recent" string

Thanks for a great theme!
Super small PR for a thing i had to modify when attempting localization. Not sure of the name but it's in line with the read_more_copy in summary-with-image.html

Thanks, @naile


TASK: Add target="_blank" to links in PageList partial

I found it to be quite disorienting to leave the backend entirely when clicking on a preview link., so in order to improve UX, I'd like to suggest to open the page preview links in a new tab.
Thanks for this great module!

Thank you for your PR! :)
You are right regarding the UX, it's much better to not leave the backend when one is clicking on a preview.


postgresql-11.11 and pgbadger 11.8 produces empty html report

Hi !
Pgbadger (11.8 version) produces empty html report from logs of our postgres (11.11) cluster.
Our postgres.conf file has following log line prefix:
log_line_prefix: 'timestamp: %m user: %u host: %h db: %d app: [%a] '
We execute pgbadger as:
pgbadger /data/pg_log/postgresql-Fri.log -o report.html --prefix='timestamp: %m user: %u host: %h db: %d app: [%a] ' -f stderr
Here is example of logs in file:
timestamp: 2022-06-23 00:00:15.639 MSK user: sentry host: [local] db: sentry app: [[unknown]] STATEMENT:  INSERT INTO "sentry_eventuser" ("project_id", "hash", "ident", "email", "username", "name", "ip_address", "date_added") VALUES (152, 'a30d004660658a2b2e14cd489a21f480', NULL, NULL, NULL, NULL, '<IP_ADDRESS>', '2022-06-22 21:00:15.636023+00:00') RETURNING "sentry_eventuser"."id"
timestamp: 2022-06-23 00:00:15.681 MSK user: sentry host: [local] db: sentry app: [[unknown]] ERROR:  duplicate key value violates unique constraint "sentry_environmentproject_project_id_29250c1307d3722b_uniq"
timestamp: 2022-06-23 00:00:15.681 MSK user: sentry host: [local] db: sentry app: [[unknown]] DETAIL:  Key (project_id, environment_id)=(152, 2) already exists.
timestamp: 2022-06-23 00:00:15.681 MSK user: sentry host: [local] db: sentry app: [[unknown]] STATEMENT:  INSERT INTO "sentry_environmentproject" ("project_id", "environment_id", "is_hidden") VALUES (152, 2, NULL) RETURNING "sentry_environmentproject"."id"
timestamp: 2022-06-23 00:00:19.051 MSK user:  host:  db:  app: [] LOG:  automatic analyze of table "esx_site_backend_api.public.shedlock" system usage: CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.02 s
timestamp: 2022-06-23 00:00:19.440 MSK user:  host:  db:  app: [] LOG:  automatic vacuum of table "esx_site_backend_api.public.registered_client": index scans: 1
pages: 0 removed, 1790 remain, 0 skipped due to pins, 271 skipped frozen
tuples: 15158 removed, 19250 remain, 0 are dead but not yet removable, oldest xmin: 650513400
buffer usage: 3777 hits, 0 misses, 0 dirtied
avg read rate: 0.000 MB/s, avg write rate: 0.000 MB/s
system usage: CPU: user: 0.01 s, system: 0.00 s, elapsed: 0.37 s
timestamp: 2022-06-23 00:00:19.833 MSK user:  host:  db:  app: [] LOG:  automatic analyze of table "esx_site_backend_api.public.registered_client" system usage: CPU: user: 0.17 s, system: 0.00 s, elapsed: 0.39 s
Here is result of pgbadger's execution:
[========================>] Parsed 125439235 bytes of 125439235 (100.00%), queries: 0, events: 1
LOG: Ok, generating html report...
And finally it is empty inside after opening in browser. By the way overall size of .html file is 5.5 Mb.
We've tried to run verbose and found following messages:
DEBUG: Start parsing postgresql log at offset 0 of file "/data/pg_log/postgresql-Fri.log" to 119253976
DEBUG: Unknown stderr log line format:  pages: 0 removed, 174 remain, 0 skipped due to pins, 103 skipped frozen
DEBUG: Unknown stderr log line format:  tuples: 180 removed, 96 remain, 0 are dead but not yet removable, oldest xmin: 656129919
DEBUG: Unknown stderr log line format:  buffer usage: 244 hits, 3 misses, 0 dirtied
DEBUG: Unknown stderr log line format:  avg read rate: 0.495 MB/s, avg write rate: 0.000 MB/s
DEBUG: Unknown stderr log line format:  system usage: CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.04 s
DEBUG: Unknown stderr log line format:  pages: 0 removed, 1 remain, 0 skipped due to pins, 0 skipped frozen
DEBUG: Unknown stderr log line format:  tuples: 56 removed, 1 remain, 0 are dead but not yet removable, oldest xmin: 656129964
...
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
DEBUG: Unknown stderr log line format:  RETURNING *
Seems we are experiencing some troubles with prefix, but can't understand how to fix it.

The documentation says:
pgBadger supports any custom format set into the log_line_prefix directive of
your postgresql.conf file as long as it at least specify the %t and %p patterns.

the pid is not present in your log_line_prefix

Thanks ! My bad.
May be you could add a line about it in '--help' ? Or throw an according exception if supplied --prefix doesn't contain mandatory values ?

Or read the doc :-)


WIP: [FIX] java.lang.IllegalArgumentException IAB

Worked on iOS.. waiting for remove [WIP]

Worked on iOS.. waiting for remove [WIP]


Add a Shutdown() method to TracerProvider

See https://github.com/open-telemetry/opentelemetry-specification/pull/1074

Ironically, it turns out we've had this "forever": https://github.com/open-telemetry/opentelemetry-ruby/blob/dd805ece5b6c77c8920e0d476fab7f424f4804c9/sdk/lib/opentelemetry/sdk/trace/tracer_provider.rb#L44-L61
It looks like we need minor documentation tweaks - I think this was moved from Tracer to TracerProvider at some point.


release v0.0.1

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


nodezoo-github unable to authenticate

Just a PSA to say that if you run into this issue, and have set your token according to the docs in system.env, make sure you pull the latest version of Fuge
I blew away my setup to QA test Dean's changes, and everything looks good

Good spot,
@shanel262 we should document how to add this and also the minimum version of fuge required to run. This will be your next task.

@shanel262 You lucky duck! @Georgette has done the hard work for you, you just need to add minimum fuge version.

@mcdonnelldean It's a lucky Monday! Thanks @Georgette! I'll get on this :)


pre-grouped data rework

Allow pre-grouped syntax to work in conjunction with the data syntax. Ideally one should be able to use either the dims() context or the data(df) context, both or none.
Requires:

decoupling processing of different contexts
allowing df to contain arrays rather than vectors if pregrouped data has a more complex shape
ensuring that all helpers (esp. figuring out categorical conversions) work in all conditions, at the moment verbatim seems a bit problematic with pre-grouped data. The following errors

using CairoMakie, AlgebraOfGraphics, DataFrames
n = 5
df = DataFrame(g = 1:n, xy = [[Tuple(rand(2)) for _ in 1:2] for _ in 1:n])
fig = draw(mapping(df.xy => verbatim, layout=df.g => nonnumeric) * visual(Lines))

See https://github.com/JuliaPlots/AlgebraOfGraphics.jl/discussions/345 for a proposal.


Typo in verbiage

Following line:
If you want to see the request unit usage by collection, select Apply splitting and choose the collection name as a filter. You will see a chat like the following with a choice of collections within the dashboard. You can then select a specific collection name to view more details:
should read:
If you want to see the request unit usage by collection, select Apply splitting and choose the collection name as a filter. You will see a chart like the following with a choice of collections within the dashboard. You can then select a specific collection name to view more details:
Please change word chat to chart

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

ID: 9ac8cba1-4317-a0b7-395e-c880cdbc7dbd
Version Independent ID: e46d7e04-d5b1-57f8-ee04-a91feee04e59
Content: Monitor the throughput usage of an operation in Azure Cosmos DB
Content Source: articles/cosmos-db/monitor-request-unit-usage.md
Service: cosmos-db
GitHub Login: @StefArroyo
Microsoft Alias: esarroyo

@stypich Thank you for your feedback. We are reviewing this and will get back shortly.


React to runtime perf changes

@davidfowl @pranavkm @NTaylorMullen
Reacting to https://github.com/aspnet/XRE/commit/678d2c7ab8d3b54fcea902493d8f6e2228425992. Fixing broken Mvc repo on CI.

Yeah, some Mvc tests are resolving project with appbase, instead of app name. Working on more reactions. Sending PR soon.

Why is using base path a problem? Was that ability removed?

Yes, it was removed