Fix #55 due to chromosome mismatch between sumstats and bed

This PR fixes #55 -- @Lipastomies Would you mind testing this?
Thanks!

testing right now, will let you know how it goes

This seems to run out of memory with relatively small endpoints as well, which is interesting.
Note: You'll need to freeze the pybedtools version to 0.8.2, latest (0.9.0) does not work on python2.7.
This does not seem to work directly, the beds are put through convert_chrpos() on line 564, and that seems to create a memoryerror (as multiplying "X" with a number creates something that we most definitely do not want). Probably would need to map the ranges back to numbers after that. And then back to strings to get the regions from the already converted summary statistic.

Got it, thanks so much for testing! @Lipastomies Apparently, I was converting chromosomes to integers for reasons... Let me try another fix
PS. We should switch from python2 to python3...!!!

@Lipastomies I just pushed an alternative fix -- we now force integer chromosome comparison in filter_sumstats as discussed #55

This seems to fix the problem.
Can you @mkanai add the version to the pybedtools in the docker image, replacing
    pip install --no-cache numpy scipy pandas pybedtools

(line 55, finemap-suite dockerfile )
with
    pip install --no-cache numpy scipy pandas pybedtools==0.8.2

Great, thanks for the confirmation! pybedtools version is now fixed in Dockerfile.


ci: Dont use .python-version in github action

Fix #22746
Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Thanks!


GeneratePdf(),  returns an empty PdfDocument

Hello i'm trying to generate a pdf via html string passed by parameter, my problem is that when i want to save the PdfDocument returned by the function, it doesn't have any pages or content (image of the debug is attached), here are the versions that i use:
Versions
HtmlRenderer.Core: 1.5.1-beta1
HtmlRenderer.PdfSharp: 1.5.1-beta1
PdfSharp: 1.51.5185-beta
Here is the code of my function:
var config = new PdfGenerateConfig();
config.PageOrientation = PageOrientation.Portrait;
config.PageSize= PageSize.A4;
PdfSharp.Pdf.PdfDocument pdf = PdfGenerator.GeneratePdf(html, config);
pdf.Save(path);

And because it has no Pages and no content, i can't save it because it throws me the exception: 'Cannot save a PDF document with no pages'
Thank you in advantage :)

you have to combine it with PdfSharp latest version and use this HtmlRendererCoreNet7.PdfSharp


Question: Will the sample Apps  work on  a  x86/x86_64 virtual device on Andriod Studio

I could debug/run the sample apps using real phone. but  could no trun /debug using  a virtual device on Android Studio. It always failed  to create the Engine no mater what I use x86 /x86_64
Anyone has experience to run the apps on virtual device, could you please share your steps?
Thanks
Ryan

They work in AVDs yes. What failure do you get in logcat?

error information from logcat:
/Filament: eglChooseConfig failed with unknown
D/AndroidRuntime: Shutting down VM
E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.google.android.filament.textured, PID: 16214
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.google.android.filament.textured/com.google.android.filament.textured.MainActivity}: java.lang.IllegalStateException: Couldn't create Engine
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3260)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3396)
at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:83)
at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2009)
at android.os.Handler.dispatchMessage(Handler.java:107)
at android.os.Looper.loop(Looper.java:214)
at android.app.ActivityThread.main(ActivityThread.java:7319)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:934)
Caused by: java.lang.IllegalStateException: Couldn't create Engine
at com.google.android.filament.Engine.create(Engine.java:46)
at com.google.android.filament.textured.MainActivity.setupFilament(MainActivity.kt:111)
at com.google.android.filament.textured.MainActivity.onCreate(MainActivity.kt:95)
at android.app.Activity.performCreate(Activity.java:7783)
at android.app.Activity.performCreate(Activity.java:7772)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1299)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3235)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3396)
at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:83)
at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2009)
at android.os.Handler.dispatchMessage(Handler.java:107)
at android.os.Looper.loop(Looper.java:214)
at android.app.ActivityThread.main(ActivityThread.java:7319)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:934)

E/Filament: eglChooseConfig failed with error code: 12288

from  Khronos Group's egl.h, 12888 = 0x3000 is suceess
#define EGL_SUCCESS                       0x3000

How is your AVD setup?

Thanks. Can you try by setting GPU emulation to hardware? And software if that doesn't work.

I had tried using hardware and software. I got same error
D/: HostComposition ext ANDROID_EMU_CHECKSUM_HELPER_v1 ANDROID_EMU_dma_v1 ANDROID_EMU_host_composition_v1 GL_OES_vertex_array_object GL_KHR_texture_compression_astc_ldr ANDROID_EMU_gles_max_version_2
E/Filament: eglChooseConfig 1 failed with error code: 0x3000
eglChooseConfig failed with unknown

Your max GLES must be 3, not 2. Filament cannot run on ES 2.0.

do you know what's configuration could let the emulator use GLES 3.0?  I only could select GLES 2.0
Thanks

You shouldn't have to do anything special but I've had issues in the past with x86_64 AVDs. Either try with x86 or with a different type of system image (with/without Google APIs, etc.). I unfortunately can't remember which variant caused me issues.

Thanks, fixed,
need put
GLESDynamicVersion = on  in the file ~/.android/advancedFeatures.ini

@RyanFordio anything you need to do after creating ~/.android/advancedFeatures.ini and setting that flag?  I've don't that restarted Android Studio, but still only get GLES20 options for the AVD:


update readme following migration to vmware-tanzu

Looks like I might need to issue a PR from a different repo in order to trigger the license request. Closing this, will issue a PR from a fork.

Looks like I might need to issue a PR from a different repo in order to trigger the license request. Closing this, will issue a PR from a fork.


[naoqi_apps] add node and launch for ALFaceCharacteristics

I followed http://doc.aldebaran.com/2-4/naoqi/peopleperception/alfacecharacteristics-api.html#alfacecharacteristics-api
and this requires ros-naoqi/naoqi_bridge_msgs#23 for msg and srvs.
% I didn't make FaceCharacteristics/PersonSmiling => [Editted] I added it
If there is any problem, please let me know.
services:

/analyze_face_characteristics
ALFaceCharacteristicsProxy::analyzeFaceCharacteristics
uses AnalyzeFaceCharacteristics.srv
/get_smiling_threshold
ALFaceCharacteristicsProxy::getSmilingThreshold
uses GetSmilingThreshold.srv
/set_smiling_threshold
ALFaceCharacteristicsProxy::setSmilingThreshold
uses SetSmilingThreshold.srv

topic:

/face_chracteristics

PeoplePerception/Person/<ID>/AgeProperties
PeoplePerception/Person/<ID>/ExpressionProperties
PeoplePerception/Person/<ID>/GenderProperties
PeoplePerception/Person/<ID>/SmileProperties
PeoplePerception/Person/<ID>/FacialPartsProperties

uses FaceCharacteristics.msg
(= AgeProperties.msg + ExpressionProperties.msg + FacialPartsProperties.msg + GenderProperties.msg + SmileProperties.msg)

This is a new feature, which may cause unexpected trouble when it is merged.
I close this and improve this in my local.
I'm sorry to leave this for a long time.


USB Joystick no seen within Injected PC app in Revive while streaming with ALVR (using Gear VR and Samsung S9)

Hi,
Thank you for putting together such a stable and efficient tool.  I am enthusiastic about VR and AR and exploring the potential of my Gear VR mobile headset while waiting for Oculus Quest to be releas3ed.  Hopefully, ALVR will be able to interface with Quest and make it a wireless Rift little brother so that the powers of PC and graphics card can be leveraged too.
Back to joystick issue:
With ALVR (version 2.3.1) I managed to get the graphics streaming to my Gear VR mobile headset fitted with a Samsung S9 over wifi.  The Revive (version <IP_ADDRESS>)  inject function works well and Soaring flight simulator Condor 2.exe (version 2.0.5)  starts apparently normally ( http://www.condorsoaring.com/ ).
My issue is that after starting Condor with Revive, the USB joysticks do not show up in Condor/Input menu, hence to operational while in flight sim).
Note that the joystick USB devices are detected by Windows and can be calibrated by Joy.cpl and listed in Steam/Controller.
Do you have any idea what needs to be done so that once Condor is started in VR mode I get the joystick (and second joystick for my rudder pedals)  recognized and operational?
Is the issue at the ALVR level or Revive level?
Thank you in advance for your help,
Donat

Thank you for your reply.   I checked again and yes the USB Force Feedback Controller is seen in Steam but not VR Steam.
Interestingly the joystick was allowed to move the cursor within the Steam / Big Screen / Setting: Controller...
I tried to modify the default controller in the Big Picture setting and no change.
Today, I noticed that I could start this game from Steam, then the game Input setting I noted that I could define the joystick's input (i.e. the original ones not showing).
From there on,  I was able to close the game, and the launch (Inject) from Revive as before and now the Game/Inputs setting detected and allowed to set it up.
It working and I am not sure what really changed.
I realize this is not related to ALVR, but is there a good resource to help me understand how the Steam (In Big Picture) controller are set?  In particular, so that the joystick is not used instead of or in conjunction with the keyboard and mouse cursor.
Thank you again for your help.


Update of value - text and add icon

@pariterre, I am waiting for Patinage Qc for their new logo and colors and the "ok" to push on the stores


Fix issue when graphql is true and include permission is WILDCARD

Why make this change?

Closes #1434

When graphql is set to true without singular type specified in runtime config file, in addition to when field auth include permission is using WILDCARD, this was giving user's access forbidden error.

What is this change?

Reason for causing this is because GraphQLSingularTypeToEntityNameMap in RuntimeConfig.cs is not adding the entity type when there is no singular type passed in, and for WILDCARD permission check, the AuthorizationResolver is retrieving all the fields columns using the GraphQLSingularTypeToEntityNameMap keys.

How was this tested?

[x] Integration Tests

I feel like #1402 will tackle this as it is more aggressive in the way that it sets the singular name, resulting in the RuntimeConfig always having a defined singular name, rather than having to rely on additional mapping data structures.


Upgrade dio package 2.1.7->3.0.7

Why is this not approved yet? And while we're at it, latest version of dio is 3.0.9 now

Why still not merged?


Rigged boxing gloves have variant TC costs.

About the PR
Changed the rigged boxing gloves to have an increased cost for passengers compared to the boxer. 8 TC for a passenger - on par with northstars - and 5 TC for boxers.
Why / Balance
This was done as a way to encourage boxers to use their job specific item more than the passengers, and to push passenger syndies away from overreliance on the item. However, through much discussion on the discord forums, passengers with boxing gloves were deemed to be iconic enough to remain as part of their syndicate job item. A compromise was made that the passengers should have a penalty compared to the boxers, but that both should keep the item.
Technical details
Simply changed the syndicate uplink YML to include two different versions of the purchase for the two different roles.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

tweak: Syndicate boxers now have an easier time acquiring rigged boxing gloves compared to passengers. 5 TC for boxers, 8 TC for passengers.

will this actually address the issues people have with it

Yeah idk, rigged gloves are powerful because:

You don't drop them
Maints are too tight and friendlies block your hits
Pairing of other syndie gear and tactics make already awkward fights one-sided (noslips, web vest and energy shield, along with slipping sec and simply being faster)

will this actually address the issues people have with it

i don't think people will be totally satisfied until the gloves are fully removed. i just felt that they should be balanced closer to the strength of the other syndicate takedown options. i felt it should be closer to nocturine, specifically.
this should, hypothetically, also make it harder for syndies to completely steamroll entire departments with this. i'm not sure exactly what the numbers add up to, but it should ensure that passenger syndies will have at least one vulnerability for sec to exploit, as opposed to the dominance of "web vest + no slips + energy shield + gloves"

Yeah idk if this'll fix it, rigged gloves are powerful because:

You don't drop them
Maints are too tight and friendlies block your hits
Pairing of other syndie gear and tactics make already awkward fights one-sided (noslips, web vest and energy shield, along with slipping sec and simply being faster)
Stun meta applies both ways but currently benefits syndies with noslips and syndie soaps/rigged gloves

i personally don't think the gloves themselves are the issue. the issue is security not communicating the fact that someone has the gloves in the first place. if you actually tried to use lethals against someone with these gloves, or just stuck to going in pairs, the syndie is almost always going to die first. not to mention the new grenade options, and the fact that a good secoff wouldn't forget their flashing options. there is genuine counterplay with the gloves. the issue is that the low TC cost makes insanely hard to counter combos much more prevalent as makes things far more hard to manage as a result.
i will say that they maybe should be around the 10 TC range for passengers and 6-8 TC for boxers. the discord thread mentioned here concluded on the numbers in the PR, but i'm open to tweaking them.

i will say that they maybe should be around the 10 TC range for passengers and 6-8 TC for boxers. the discord thread mentioned here concluded on the numbers in the PR, but i'm open to tweaking them.

Yeah idk if this'll fix it, rigged gloves are powerful because:

You don't drop them
Maints are too tight and friendlies block your hits
Pairing of other syndie gear and tactics make already awkward fights one-sided (noslips, web vest and energy shield, along with slipping sec and simply being faster)
Stun meta applies both ways but currently benefits syndies with noslips and syndie soaps/rigged gloves

i personally don't think the gloves themselves are the issue. the issue is security not communicating the fact that someone has the gloves in the first place. if you actually tried to use lethals against someone with these gloves, or just stuck to going in pairs, the syndie is almost always going to die first. not to mention the new grenade options, and the fact that a good secoff wouldn't forget their flashing options. there is genuine counterplay with the gloves. the issue is that the low TC cost makes insanely hard to counter combos much more prevalent as makes things far more hard to manage as a result. i will say that they maybe should be around the 10 TC range for passengers and 6-8 TC for boxers. the discord thread mentioned here concluded on the numbers in the PR, but i'm open to tweaking them.

I was getting at that; gloves aren't the issue themselves, they just pair waaaayy too well with other gear that makes fighting difficult
Majority of sec is unrobust and should be punished for it yeah, but coordination is honestly a lost art (and again, friendly fire is basically a hidden weapon for syndies considering how tight maints are and how quickly they get to move without slipping)


Prevent cards from loading until toggled

When porting my cards from browser mod to bubble-card, One particular feature I've found necessary is the behavior where pop-ups aren't preloaded in Lovelace until the toggle button is pressed. This functionality proved highly effective for handling cameras. Therefore, when I apply the same approach to my four cameras using popup cards, all four cards are preloaded into Lovelace. This, in turn, leads to extended loading times for the dashboard as it renders all four camera cards simultaneously. My proposed enhancement idea is to include an option that allows the popup card to be somehow prevented from loading until the pop-up is activated.
A workaround to this is to put your popups on another dashboard and you can call them like 6#camera but its a little wonky sometimes it won't open the popup card.

I agree I need this too because when I open the app I hear camera audio playing in the background now, as a workaround I've set the frigate card to mute by default but that doesn't mean it's not running in the background anymore still, might even help with performance too

I would love to be able to do that, but I don't know if it's even possible with how the pop-ups are working in Bubble Card right now.
A possible solution would be to add a conditional card in your pop-up with your camera in it, then add a new input_boolean as a condition. You can now use this same input_boolean to trigger it (with trigger_entity). As a result when you open your pop-up by toggling your input_boolean the conditional camera will be shown then hidden when the pop-up is closed.

Maybe this conditional card functionality can be built in to the card? Hopefully there's a solution soon cuz I like the idea of a popup to quickly view a camera, it just shouldn't keep the card loaded when closed.

On my dashboard I have a pop-up with two cameras and no sound, it work like charm for me. But indeed it could be cool to have such a feature, maybe by cloning the default vertical stack code to have a full control on how it work and load its content.

Are you using the frigate card for the pop-up? I turned off the sound so I don't hear it anymore, but technically doesn't mean that the card isn't active anymore though

Yes I'm using it too, and I know that some cards are lazy loaded (the frigate card is not) but it could be indeed pretty cool to have the vertical stack directly when you create a pop-up, I will take a closer look at that.

I really like the cards, but after reworking my entire dashboard to just a single view with all bubble-card popups I also found that loading times are very bad when running a large dashboard like mine. I also believe this is caused because the entire code loads when opening the frontpage (as everything is on the frontpage, since I'm only using a single view). When leaving the app open on the background and coming back to it it will simply lock up and I'll need to restart the HA app (Android).
I wonder if it isn't possible to use a dom-event instead (like browser_mod or local conditional card does)? That would certainly speed up loading times as it would load on demand instead of on startup.
But still very nice cards! Kudos!

Thanks for your feedback, have you tried v1.3.3? I've just released it and it should improve pop-ups performances a lot. But I'm aware of this issue and I'm working on it almost everyday, I really hope that this will be fixed soon 🤞

No worries, just take whatever time you need. These projects are in free time and in no way is anyone entitled to apply pressure on your project.
I will follow this thread and once in a while try out performance again.


NTR: Medication from outside source type

For new term requests, please provide the following information:
Term Label [required] Medication from outside source type
Synonyms [optional]
Definition [required except for organism name]
The definition should be understandable even for non-specialists. Include a PubMed ID or DOI to refer to any relevant article that provides additional information about the suggested term.
Collects information on which medications were provided from a source outside the study
Suggested Parent Term [required]
Please look in the ontology hierarchy in a browser such as OLS or choose one from following list:

[ ] Material (independent of other entities and can be maintained through time, such as person, cell, sample)
[ ] Process (has temporal proper parts and for some time depends on some material entity, such as life cycle, cell division, heart beating, sleeping)
[ ] Information (generically dependent on some artifact and stands in relation of aboutness to some entity, such as data, identifier, document)
[ ] Characteristic/Quality (inheres in or is borne by other entities, such as height, color, disease)
Information
Examples [optional]

Contributor [optional]
If you would like to be a contributor of the term or anyone you think that contributes to the term, such as label, definition, please provide the name and ORCID ID.
Requester [optional]
Please indicate which resource/database/project/ request the term.
ClinEpiDB

@SheenaTomko I think you're asking to define EUPATH:0060124, but I think I should define the non-multifilter version EUPATH:0021102 first.
EUPATH:0021102
Label: clinical history of medication obtained from outside study
Definition: A clinical history that specifies a medication material obtained from an organization or human being other than those designated by the investigation agents.
Parent: clinical history
Subclass of: mentions some 'medication material'

@johnwjudkins, your definition for EUPATH:0021102 looks good to me!

@SheenaTomko
EUPATH:0060123
Label: indicator of antibiotic from outside study
Definition: A clinical history of medication obtained from outside study that specifies whether an antibiotic was obtained.
Parent: clinical history of medication obtained from outside study
Subclass axiom: 'is about' some antibiotics'

@johnwjudkins looks good

Looks good to me.


Use API Extractor for TypeScript modules

Is your feature request related to a problem? Please describe.
Exposing the right API from TypeScript modules is tricky as we've discussed in https://github.com/cucumber/common/pull/1660#pullrequestreview-708109485
Describe the solution you'd like
Adopt API Extractor which solves this problem
Describe alternatives you've considered
Using a homegrown namespace as currently implemented in https://github.com/cucumber/common/pull/1660, but that doesn't really solve the "we forgot to export this file" problem.
Additional context
This is also related to @mpkorstanje's recent soapbox talks about having stable public APIs.

This will be great! I did try to add it a while ago and ran into some issues, but IIRC it was because of some peculiarities in the TypeScript emitted by protobuf, so hopefully smooth sailing this time.


fix-#392: added related blogs section in detailed blog page

Summary
added  related blogs section in detailed blog page
Description
whenever user visited individual blog page..they can see related posts based on the categories in the blog page
Issue(s) Addressed
Closes #392
Prerequisites

[ yes] Have you followed all the CONTRIBUTING GUIDELINES?

hey @krishnaacharyaa please do review it and want any changes then tell me 👍

helllo @krishnaacharyaa ??

ok sure @krishnaacharyaa on itt

hi @krishnaacharyaa
fetched all related category posts and displayed 3 posts in mobile view and 4 posts in desktop view

For mobile can we aim for this kind (style) of related posts?

small icon in the left and then the content on the right.
If you feel these are quite a few requests in a single PR, kindly let me know we can break this into subtasks, and we can raise this as another PR :) I'm totally fine with that

no its finee...i'll be on it

@krishnaacharyaa
changed naming conventions
added mobile responsive for related posts

@krishnaacharyaa  ?

Super @vamsidhar-914
Thank you for the efforts, really appreciated.
Can you use

For the arrow right svg
May be this is black version, make white version of it by replacing #000000 with #ffffff and make 2 svgs and kindly handle both dark and light theme
And

Can you please take care of this alignment, here the see more related blogs > should be aligned to the below card like

And kindly upload the photo of the shimmer/skeleton things aswell, if you feel its too much in a single PR we can take this shimmer loading skeletons in another PR i am fine with that :)
IF you could close the above 2 bugs we are good to merge this PR :)

fixed svg icons for light and dark mode
added loading skeletons

is this alignment ok? or u want any change to it

@krishnaacharyaa
fixed svg icons for light and dark mode
added loading skeletons
is this alignment ok? or u want any change to it

Can't we have it right side as earlier
Just the end should match
You need to use something like justify-between

Here I'm not talking about the skeletons I'm just talking about the "Related blogs" and "See all related blogs" that can we have in a single row! That would be nice
Just it should be end aligned as I marked in red line in previous comment

done it sir @krishnaacharyaa !

Thank you sir :)


Error compiling at "'CMakeFiles/dlib_.dir/src/matrix.cpp.o'

I try to compile Dlib on Raspberry Pi 3 with Ubuntu 16.04 OS but it always stop in the middle.
I run 'cmake --build . --config Release'
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-5/README.Bugs> for instructions.
CMakeFiles/dlib_.dir/build.make:86: recipe for target 'CMakeFiles/dlib_.dir/src/matrix.cpp.o' failed
make[2]: *** [CMakeFiles/dlib_.dir/src/matrix.cpp.o] Error 4
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/dlib_.dir/all' failed
make[1]: *** [CMakeFiles/dlib_.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2
trungdn@trungdn-desktop:~/src/dlib/python_examples/build$ ^C
trungdn@trungdn-desktop:~/src/dlib/python_examples/build$

c++: internal compiler error: Killed (program cc1plus)
This is possible means not enough memory. How to change spaw space

trungdn@trungdn-desktop:~/src/dlib/python_examples/build$ ^C

Why building from python_examples?
Did you tried to follow instructions from readme.txt?

COMPILING DLIB Python API
Before you can run the Python example programs you must compile dlib. Type:
python setup.py install
or type
python setup.py install --yes USE_AVX_INSTRUCTIONS
if you have a CPU that supports AVX instructions, since this makes some
things run faster.

Are you able to build and run normal examples?

No. I tried the normal way and it still stop in the middle. I will try to swap mem.

Yes. It worked when I swap mem by 2Gb.
Thank you very much.

c++: internal compiler error: Killed (program cc1plus)

This is possible means not enough memory. How to change spaw space

trungdn@trungdn-desktop:~/src/dlib/python_examples/build$ ^C

Why building from python_examples?
Did you tried to follow instructions from readme.txt?

COMPILING DLIB Python API
Before you can run the Python example programs you must compile dlib. Type:
python setup.py install
or type
python setup.py install --yes USE_AVX_INSTRUCTIONS
if you have a CPU that supports AVX instructions, since this makes some
things run faster.

Are you able to build and run normal examples?

Worked like charm. Memory was the issue. My AWS instance had only 1 gb space. Reading the problem intuition, I created a swap space of 6 more gb and it worked.
Now, I'm able to complete installation.
Thanks
Regards 🤗


SB teack-2 Remove BinaryData from azure-core-amqp(prep to release next week)

Since we need to release AMQP GA in October next week.
And BinaryData need more design work, so we are moving it into service bus temporarily to allow GA for core-amqp, after October release we will design it more and once design is approved for BinaryData, we can more it into core or core-amqp.

Tasks:

[ ] Move BinaryData to Azure.Core Experimental
[ ] Get BinaryData reviewed by the board, and move it to Azure.Core
[ ] Update Azure Core AMQP to use BinaryData from Azure.Core
[ ] Update Service Bus to use the above BinaryData in both body and amqpAnnotatedMessage


Copter falls on changing MPC_Z_VEL_MAX_UP and MPC_Z_VEL_MAX_DN in-flight

Hello.
We're controlling the copter in OFFBOARD mode using SET_POSITION_TARGET_LOCAL_NED, with position settings. Also, we want to dynamically change maximum velocities of flying.
For that, we use MPC_XY_VEL_MAX, MPC_Z_VEL_MAX_DN, MPC_Z_VEL_MAX_UP settings. Changing MPC_XY_VEL_MAX works OK, but additional changing MPC_Z_VEL_MAX_DN or MPC_Z_VEL_MAX_UP sometimes makes the copter stop the motors and fall.
We have observed this situation several times.
Here's the altitude graph of such falling:

We're taking off, set parameters, then logger hangs for a while (the straight line), then the copter stops the motors and falls.
Link to this log: https://logs.px4.io/plot_app?log=baa165fc-9aaf-484b-9073-08a1521adb5b.
Log file: https://www.dropbox.com/s/0tt1hlpjyiwt9aa/16_55_33.ulg?dl=0
I can attach some additional logs and video files, if they are needed.

What most likely was happening, was that important flight modules were blocked on reading parameters because a parameter save was in progress. This normally does not occur because parameters are not written often.
This helps: https://github.com/PX4/Firmware/pull/7950. It makes sure that the flight modules stay responsive during such operations. However it's still possible that you lose offboard control for several 100ms if a parameter write happens during a saving operation.
I suggest you set the params before the flight if that's possible.

@bkueng, thanks for the answer!
So, changing any parameters in-flight officially should not be working? Or it causes some kind of 'undefined behaviour'?
How do I fly between different points in OFFBOARD mode with different velocities, then?

So, changing any parameters in-flight officially should not be working? Or it causes some kind of 'undefined behaviour'?

It's definitely supported and we do it regularly. Just not multiple at (almost) the same time.

How do I fly between different points in OFFBOARD mode with different velocities, then?

I have not used offboard, but the SET_POSITION_TARGET_LOCAL_NED message contains velocity fields.

I have not used offboard, but the SET_POSITION_TARGET_LOCAL_NED message contains velocity fields.
Unfortunately, only position control or velocity control can be used, not both at the same time.

It's definitely supported and we do it regularly. Just not multiple at (almost) the same time.
So changing one setting is OK, but simultaneously 2-3 or more is not OK?

Unfortunately, only position control OR velocity control can be used, not both at the same time.

Isn't there another message to set the velocity then? @TSC21 might know.

So changing one setting is OK, but simultaneously 2-3 or more is not OK?

Until now, yes. https://github.com/PX4/Firmware/pull/7950 fixes that. But setting parameters for your use-case is the wrong approach. It should be possible to do that via a mavlink message.

Isn't there another message to set the velocity then? @TSC21 might know.
setting parameters for your use-case is the wrong approach

I thought exactly the same earlier. In AUTO mode there is MAV_CMD_DO_CHANGE_SPEED message. But I couldn't find the way to change the maximum speed in position control in OFFBOARD mode. The only answer I got in PX4 Forum is to use parameters.

Until now, yes
That seems like a critical issue!

We applied #7950 for testing. It doesn't stop motors anymore, but looses OFFBOARD mode almost every time we change speed with interval less than 2s.

It doesn't stop motors anymore, but looses OFFBOARD mode almost every time we change speed with interval less than 2s.

Thanks for testing. Like I wrote, it's not a good idea to constantly change parameters. The mavlink message spec of SET_POSITION_TARGET_LOCAL_NED does not exclude setting both velocity & position. But the position controller uses only one or the other (here). So we should consider extending support for that @Stifael.

The parameters MPC_XY_VEL_MAX, MPC_Z_VEL_MAX_DN, MPC_Z_VEL_MAX_UP are not meant to change during any flight. They are max-values which are respected by the controller. This means that if you set a velocity demand larger than MPC_XY_VEL_MAX, the demanded setpoint will be clipped at MPC_XY_VEL_MAX. It is kind of a safety implementation.
If you want to change velocity between waypoints, then you have to set the velocity demand accordingly (use mavros/setpoint_velocity/cmd_vel).
It is also correct that the vehicle can operate either in position control (which is a cascaded P controller with a PID velocity controller) or just the velocity controller. If you want to use position and velocity, one way to do it is by computing the velocity setpoint offboard by mimic the firmware position controller (which is just a simple P-controller).
However, we are working on supporting both by adding velocity feedforward demands. That way position and velocity demands can be sent simultaneously.

If you want to use position and velocity, one way to do it is by computing the velocity setpoint offboard by mimic the firmware position controller (which is just a simple P-controller).

What rate the firmware position controller works at? It is easy to do it offboard, but we are limited by mavlink feedback rate. It some tasks like precision landing it can be a problem.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)


doc: fix naming issues in the HierarchicalDocumentSplitter example

Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
I documented my code
I ran pre-commit hooks and fixed any issue

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.691%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
3252

Relevant Lines:
3471

💛  - Coveralls

This can also be renamed to HierarchicalDocumentSplitter:
https://github.com/deepset-ai/haystack-experimental/blob/8fb87ec5e1bdbbd70c1eeee29dd748c6ec7d7ec8/haystack_experimental/components/splitters/hierarchical_doc_splitter.py#L44


Add documentation for context

Refs GH-6

It would be really nice to have some kind of machanism to add context when a exception is sent to Sentry. I now have a middleware (that goes after the startsession middleware) to set the data since auth data is not available in a serviceprovider' register/boot method I believe (it's to early in the application).
namespace App\Http\Middleware;

use Context;
use Closure;

class SentryContext
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request $request
     * @param  \Closure                 $next
     *
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if (app()->bound('sentry')) {
            /** @var \Raven_Client $sentry */
            $sentry = app('sentry');

            // Add user context
            if (auth()->check()) {
                $sentry->user_context([...]);
            } else {
                $sentry->user_context(['id' => null]);
            }

            // Add tags context
            $sentry->tags_context([...]);
        }

        return $next($request);
    }
}

A package I used before did it with Monolog processors. This might be a nice way to go about it.

@Stayallive I actually think middleware is a super appropriate place to do this. Gonna toss your example into the docs.


Add simple and special case mapping

In this PR we add Simple_Case_Mapping and Special_Casing to the package. The new data will enable regexpu-core to correctly generate the case mapping/folding data, especially for those new characters that are not yet covered by the latest runtime.
This PR should unblock https://github.com/mathiasbynens/regexpu-core/issues/106.
To improve the package size, we enabled the compression for case folding data.

Thanks!

Available in v1.6.1

The Simple_Case_Mapping is somehow missing from the published versions:
https://www.npmjs.com/package/@unicode/unicode-16.0.0?activeTab=code

Simple_Case_Mapping

Hmm, it’s also missing from the generated output: https://github.com/node-unicode/unicode-16.0.0


bc/dc package request

I have multiple shellscripts that are dependent on using the "dc" executable.
In Gentoo, at https://www.portagefilelist.de/site/query/file/, I can place a query for "dc". After clicking the "search" button, I can look for a result that contains "/usr/bin/dc", and I find it in the "sys-devel/bc" package.
I have not been able to find a package containing either "bc" or "dc" in your stable repositories.  Can you either tell me how to specify this package with a luet command or provide this application package?
Thank you very much in advance.

Hello, right now I do not recommend updating, as it takes place a big migration to new versions and octave is not yet rebuilt. Also some things may break too.
Concerning the sys-devel/bc package, it is already is layers/X.

Hello, @msdobrescu .
Could you let me know when it is "safe" to upgrade?
Thank you very much!

Hello, we've synced the repos, you can proceed, but may get some conflicts that usually occur when atoms were moved between packages. You should uninstall the packages dependent on those if that happen and re-install them. Settings should not be lost.


Wiki: wrong example for platformsdk

It is here:

The path to the Android SDK.
"airOptions": {
	"android": {
		"listen": true
	}
}

Thanks! Fixed.


R11s-driver: Add driver policy for long-polling downgrade

Description
Allow R11s-driver consumers to disable long-polling downgrade functionality.
Breaking Changes
The type for R11sDriverPolicies is adding a new config, but the param in DocumentServiceFactory uses Partial<>, so it is not actually breaking the usability.

It would be great to add testing of both modes (long polling and websoclet) to a matrix of states we test in service tests.
I can't find where we do it (the best reference is for other types of options / policies, like runtimeOptionsMatrix / loaderOptionsMatrix), but I think @jatgarg should know where we have driver policies.

Terminology: Maybe it's fine as is, but technically it's not a downgrade, it's not allowing / not using websocket upgrade :)

Terminology: Maybe it's fine as is, but technically it's not a downgrade, it's not allowing / not using websocket upgrade :)

Actually it is a downgrade, right? Our default and first attempted connection is websocket, because we do not specify ["polling","websocket"] as the transports. Instead, we specify ["websocket"] as the only transport, which completely removes Socket.io's transport-upgrade behavior. Then, if that websocket connection fails, we are explicitly reconnecting with a "downgraded" transport (polling).
So, if "websocket" is considered an "upgrade" from "polling", then the reverse would be called a "downgrade", so this terminology would be accurate.

It would be great to add testing of both modes (long polling and websoclet) to a matrix of states we test in service tests. I can't find where we do it (the best reference is for other types of options / policies, like runtimeOptionsMatrix / loaderOptionsMatrix), but I think @jatgarg should know where we have driver policies.

I think that's a good idea. This PR still does not allow us to configure the driver to only perform polling, so reproducing a polling connection in a test might still be a challenge. Currently, to validate polling locally, I use yarn patch to change the initial transport from ["websocket"] to ["polling"].

Terminology: Maybe it's fine as is, but technically it's not a downgrade, it's not allowing / not using websocket upgrade :)

Actually it is a downgrade, right? Our default and first attempted connection is websocket, because we do not specify ["polling","websocket"] as the transports. Instead, we specify ["websocket"] as the only transport, which completely removes Socket.io's transport-upgrade behavior. Then, if that websocket connection fails, we are explicitly reconnecting with a "downgraded" transport (polling).
So, if "websocket" is considered an "upgrade" from "polling", then the reverse would be called a "downgrade", so this terminology would be accurate.

If we look at HTTP protocol, then we failed to do websocket upgrade of HTTP connection, and now trying to leverage HTTP connection as is. You are probably right RE how that is exposed in socket.io - the ideal flow would be for socket.io not to close original http socket that failed websocket upgrade and reuse it. But I think it does not provide ability to express priorities around protocols, so we only list "websocket" protocol and if http upgrade to websocket failed, then whole http socket is closed and we start over with establishing new http socket. It's a bit wasteful and slow.
You can see it even on example in https://socket.io/docs/v4/socket-io-protocol/#introduction - there is a request to upgrade to websocket:

GET /socket.io/?EIO=4&transport=polling&t=N8hyd6w
< HTTP/1.1 200 OK
< Content-Type: text/plain; charset=UTF-8
0{"sid":"lv_VI97HAXpY6yYWAAAC","upgrades":["websocket"],"pingInterval":25000,"pingTimeout":5000,"maxPayload":1000000}

Any chance you can look at it a bit more RE efficient way of using socket.io?
Per https://github.com/socketio/engine.io-protocol#transports, it suggests that there might be more efficient way to deal with connections when websocket upgrade does not go through.

By default, the client SHOULD create an HTTP long-polling connection, and then upgrade to better transports if available.

Maybe there is a way to establish first long polling connection, and then attempt to upgrade? And that sequence is as efficient to using "websocket" protocol from the starrt?

Maybe there is a way to establish first long polling connection, and then attempt to upgrade? And that sequence is as efficient to using "websocket" protocol from the starrt?

This is the default socket.io behavior, but it was explicitly overwritten to do websocket first before I had even joined the project a few years ago.
According to you, @vladsud, in #8820 discussion,

based on what I read about socket.io protocols, adding long polling will result in it being used first, for some indeterminate period of time. I.e., its less of a fallback (the way it's implemented), latency of communication might be affected negatively by this change for all connections that do support websocket upgrade.
So it would be great to actually do some measurements and see if this approach is good, or it's better to first try websocket protocol, and if it fails, retry with long-polling.

I'm inclined to keep the current functionality for now, since that is how it has been for years, and it doesn't seem like external sources that you've found are consistent on which is more performant.


Missing step in the PowerShell example

On Chapter „PowerShell example“ between bulletpoint two and three their should be the adding of then Domain that should be taken over.
For Example:
new-msoldomain –name *your_domain_name*

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

ID: cd934d58-08f8-a3ca-2686-0006b0b8f74a
Version Independent ID: 84d9cab3-09b7-95ba-83ea-435ca0ccbb47
Content: Admin takeover of an unmanaged directory - Azure AD
Content Source: articles/active-directory/enterprise-users/domains-admin-takeover.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

@stephan-kueppers
Thanks for your feedback! We will investigate and update as appropriate.

Thanks @stephan-kueppers !
I made a pull request to update the steps. https://github.com/MicrosoftDocs/azure-docs-pr/pull/195688


Determine product name

Purpose of this issue is to determine what we will call the tool/product/website/etc. Need to think about how name will be used across all media and outlets (e.g. when people verbally say it, easy to remember, name gives users an idea of what the product does, looks and sounds good on website, social media, etc.).
Suggestions welcome!


breeze package not compatible with Spark 2.2.1

I have a spark application that uses ARIMA model for time series. When I am running the application using spark 2.2.1 I get this error :
Caused by: java.lang.NoSuchMethodError: breeze.linalg.DenseVector$.canSetD()Lbreeze/generic/UFunc$InPlaceImpl2; 	at com.cloudera.sparkts.models.ARIMAModel.gradientlogLikelihoodCSSARMA(ARIMA.scala:506) 	at com.cloudera.sparkts.models.ARIMA$$anon$3.value(ARIMA.scala:189)
But when I use an older version of spark 2.0.2 the application works. Apparently, the new spark is trying to call a function in the breeze package that does not exist anymore. I am not sure this issue needs to be addressed by breeze team or spark-timeseries team.

you have to use the right version of breeze with spark. Spark uses breeze and you shouldn't need to include breeze yourself when you use it from spark


ShowMenu template tag - namespace parameter seems to be ignored.

Summary
The template tag show_menu does not seem to take the parameter namespace into consideration.
Expected behaviour
The template tag should cut off all menu nodes, which do not have the namespace of the parameter namespace.
Actual behaviour
The template tag show_menu simply shows all menu nodes and does not cut off the nodes with namespace which differ from the parameter namespace passed to the template tag.
Environment

Python version: 2.7
Django version: 1.8.16
django CMS version: 3.4.2

Hello,
Thanks for reporting this.
Unfortunately this feature never implemented.
As a result we'll likely just take it out of the documentation.

FWIW, I believe the feature would be useful, and it doesn't seem super-hard to implement.

What works for me is to use a Modifier to check the namespace, for example:
from menus.base import Modifier
from menus.menu_pool import menu_pool

class MyModifier(Modifier):

    def modify(self, request, nodes, namespace, root_id, post_cut, breadcrumb):
        if namespace != 'mynamespace' or not post_cut:
            return nodes
        return [n for n in nodes if n.title in [
            'foo',
            'bar',
        ]]

@rfleschenberg Can you confirm that you are using the namespace parameter in code?
If people are using it, then we should keep it, and drop only false promised from the docs, not any of its functionality.
Agreed @czpython?

@evildmp
The functionality does not exist. The only way to "use it in code" is by creating a custom modifier like shown above or custom menu class that adds the functionality.
We will leave the attribute there until 3.5 or 3.6 but deprecate its usage because it doesn't do anything.

@evildmp I use it as shown, so @czpython is right. I believe there is a valid use-case here that djangoCMS should support. I don't know if the namespace parameter is the right way to do it.
The whole menu API seems to promise more than it does. I am not sure what the point of the whole pool / registration thing is, currently. If I understand correctly, you can effectively only have one menu, so what exactly is the purpose of registering menus with a menu pool?
I might come up with a PR to make some progress in this area, but due to time constraints, I cannot promise this. Should this discussion be taken to the ML, since it is beyond the scope of this specific issue?

Hello @rfleschenberg,
We are planning to do some major refactors of the menu system.
The goal will be to cleanup the codebase and also add new features like multiple menu support per request.
In regards to your question about the menu pool, the reason for this is that the cms allows developers to integrate their own menu class which lets the cms know about non-page nodes.
So even though all nodes are stored in one "menu" instance, the cms correctly filters them out when needed.
An example of this is breadcrumb navigation on a blog application, if you access a blog article's detail page, the cms breadcrumb navigation will not know anything about the blog article you're on, so the breadcrumb will be incomplete.
To solve this, you can create your own blog menu, register it with the cms and then attach it to the blog page, then the cms will have the nodes it needs to identify your blog detail pages. For more information, have a look at cms intro custom menus.

@czpython thanks for clearing this up! I guess I will wait for the results of the refactoring then, before I dig deeper into this.


[Task6] Display extra information from OpenDigger in the developer information hover card

Brief Information
This pull request is in the type of (more info about types):

[ ] build
[ ] ci
[ ] docs
[x] feat
[ ] fix
[ ] perf
[ ] refactor
[ ] test

Related issues: [Task 6 Display extra information from OpenDigger in the developer information hover card]#810
Details

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA

Others

I'm sorry that a previous commit was lost, leaving only the run prettier commit
I haven't completed the task yet. I want to ask through this draft PR why my code doesn't work and I can't find the error.

Hi Tenth, thanks for your work! It is good to have a draft PR to discuss.
To answer your question, why it is not working. I think it is because the element with avatar does not have id js-global-screen-reader-notice'. I am not sure what that element is. I searched that id in source code and it was at the bottom of the page, not the avatar. You can add a console.log('target:', target.id);` to see the ids of the element your mouse is hovering.
Also, the use of document.addEventListener('mouseover', is not a good way to add a feature, it could slow down the system drastically. One way I can think of is to use await elementReady to detect it when the element within the popover is ready, and then insert the info we want. Please Let me know if there is a specific reason we have to use event listener here.
Feel free to reach out if you have further questions. I would love to help. 😸

Thanks for you answer! Why i use document.addEventListener('mouseover') because I am not familiar with the event listening code. I will try to modify it with await elementReady。And In the code, I use id js-global-screen-reader-notice to find the developer information when the mouse hovers based on the following findings in the element

When mouse hover the avator the target do not return js-global-screen-reader-notice, i will try use other id or class to  find the right element

Now I set the target element of the mouse hover to an element with the data-hovercard-type attribute, but I don't know why the target does not respond when the mouse hovers over the avatar.

The mouse hover I am listening for now is to detect whether there is a data-hovercard-type when the mouse hovers over the img element or to detect whether the nearest parent element of the img element has a data-hovercard-type. Because in different scenarios, the location of data-hovercard-type is different.

Hi, sorry for the delay reply. Here are my advice on your questions. This is not a solution, feel free to discuss more. Yes, you are right. Looking for the [data-hovercard-type="user"] attribute is better than looking for the id
Code first:
  document.querySelectorAll('[data-hovercard-type="user"]').forEach((element) => {
    element.addEventListener('mouseover', async () => {
      console.log('mouseover', element)
      const $popoverContainer = 'body > div.logged-in.env-production.page-responsive > div.Popover.js-hovercard-content.position-absolute > div.Popover-message.Popover-message--large.Box.color-shadow-large.Popover-message--right-bottom > div > div'
      const popover = await elementReady($popoverContainer, {stopOnDomReady: false})
      console.log("popover", popover)
      const y = $($popoverContainer).find('a').find('img')
      y.attr('src', 'https://avatars.githubusercontent.com/u/39271899?v=4')
    });
  })

Some changes:

Instead of adding event listener to document, I added it to elements we want them to listen to
I use elementReady to select the popover container
We might need to wait til the hovercard displays

Things need to look into:

Adding event listener to the elements not working for all needed elements, not sure why
I set {stopOnDomReady: false} to make it work, but not sure this is a good way, need to look into.
In my example, I replaced src, you need to look for the element we want and add to it.

Thanks! i will try it later

Thanks for your reply again.Through your code, I have a deeper understanding of the event monitoring method. Now I can import OpenRank information, but I found that when the mouse hovers over the avatar for the first few hundred milliseconds, OpenRank information appears in js-global-screen-reader-notice, but is then overwritten by the user data generated by the GitHub script. I tried to monitor this overlay, and the result was that my monitoring code kept fighting against the GitHub script, making the hovercard unable to display normally.
That is, when the floating card appears, it is based on the user data obtained by the GitHub script, and cannot display the OpenRank information we introduced. I don't know much about GitHub scripts. Is there any way to make our information display correctly instead of being constantly overwritten? If so, I'd like to try it!

If there is no feasible method, I thought of another solution, which is to create a new container to present the openrank information instead of using js-global-screen-reader-notice

If there is no feasible method, I thought of another solution, which is to create a new container to present the openrank information instead of using js-global-screen-reader-notice

I am not sure what that is but I don't think it is a good way to change the inner content of that element becuase it is just a string and we don't know how Github parse it. I would recommend inserting data to the hovercard direclty when it pops up. We can improve the performance somehow by caching the user info to prevent constantly requesting data. Let me know what you think.

If there is no feasible method, I thought of another solution, which is to create a new container to present the openrank information instead of using js-global-screen-reader-notice

I am not sure what that is but I don't think it is a good way to change the inner content of that element becuase it is just a string and we don't know how Github parse it. I would recommend inserting data to the hovercard direclty when it pops up. We can improve the performance somehow by caching the user info to prevent constantly requesting data. Let me know what you think.

From my observation,

 is where GitHub dynamically obtains and displays user information. My previous method was to insert the openrank information before the user information, as shown in my previous reply, it has been successfully inserted. However, GitHub will re-obtain the user information and then overwrite the information I have inserted. I also tried to monitor the coverage of

and then reinsert my openrank information, but this resulted in constant overwriting between GitHub and the openrank information. So now I am trying to try another way to display the openrank information, and I will give a schematic diagram later.

I want to try this method. Do you think it's okay? It's not perfect yet. If you think it's okay, I'll continue.

I want to try this method. Do you think it's okay? It's not perfect yet. If you think it's okay, I'll continue.

If this solution is ok, I am ready to review it.

I am not sure this is what we want. From the description, I was expecting something like this:
https://github.com/user-attachments/assets/91adee0f-314d-460c-8426-7c6578e93363
Let me know if you have any concerns or troubles. We can also schedule a meeting to discuss if you want.

I think what you showed in your video is the expected result of the original issue. How did you do it?

I think what you showed in your video is the expected result of the original issue. How did you do it?

I followed the code in this comment https://github.com/hypertrons/hypertrons-crx/pull/832#issuecomment-2227042788

Based on the code you showed in #832, here is my code, but my result is as shown in the video, I added the content to the end of the card information, but it was overwritten afterwards
document.querySelectorAll('[data-hovercard-type="user"]').forEach((element) => {
    element.addEventListener('mouseover', async () => {
      console.log('mouseover', element);

      // 获取开发者名称
      const developerName = getDeveloperName(element as HTMLElement);
      if (!developerName) {
        console.error('Developer name not found');
        return;
      }

      console.log('developerName', developerName);

      // 获取悬浮卡片容器
      const $popoverContainer = 'body > div.sr-only.mt-n1';
      const popover = await elementReady($popoverContainer, { stopOnDomReady: false });
      console.log('popover', popover);

      // 获取开发者的排名信息
      const openrank = await getData(developerName);
      if (openrank === null) {
        console.error('Rank data not found');
        return;
      }

      // 将 OpenRank 信息作为红色文本直接插入到悬浮卡片容器内容的前面
      if (popover) {
        popover.innerHTML = popover.innerHTML + ` OpenRank ${openrank} `;
      }
    });
  });

https://github.com/user-attachments/assets/49be304a-ebf3-408b-a4de-6cc5ce698f8a

I used body > div.logged-in.env-production.page-responsive > div.Popover.js-hovercard-content.position-absolute > div.Popover-message.Popover-message--large.Box.color-shadow-large.Popover-message--right-bottom > div > div which is the actual hovercard, instead of the
. After we get the hovercard, we can insert anything we want.

I used body > div.logged-in.env-production.page-responsive > div.Popover.js-hovercard-content.position-absolute > div.Popover-message.Popover-message--large.Box.color-shadow-large.Popover-message--right-bottom > div > div which is the actual hovercard, instead of the
. After we get the hovercard, we can insert anything we want.

Thanks, I'll try it soon!

Thanks! It is nice to work with you, I am glad you finally made it and I also learned a lot myself during this process. However, we are not there yet, there are things we need to add before reviewing it carefully:

Add a view for this info. I know this is just one line insertion but we still need that for future maintanance. It should be easy, you can refer to other features, 99% of them are implemented this way.

Set proper Styles. After we move view to a separate file, we can handle its style. Currently, you set the color to black but it looks weird on my side cuz I am using a dark theme. You can also look for how the other features do this.

Natural insertion. For extensions, we should make it as natural as possible like it is born with the page. Let's improve it a little bit by adding a rocket icon at the begin. You can copy the icon from the repo-header-labels feature in src/pages/ContentScripts/features/repo-header-labels/view.tsx. It should be something like this (ignore the red color, we should use proper style according to the themes):

Comment properly. Please comment in Enghlish and remove the comments that are duplicate the code. here is a good article on writing comments, we don't have a convention for commeting but following best practices helps to keep our code clean and clear.

Test throughoutly. Please make sure the feature won't break when we refresh, restore, go back, etc. Just play around with it to make sure it works as expected.

BTW, Let's change the name developer-avatar-openrank to developer-hovercard-info.

Thank you for your comments, I have now completed the changes you requested and fixed the bugs I could find. And I tested the performance of the function in dark mode, now I believe you can see the openrank information clearly.

I will edit it later

Nice work! I have set this PR to ready to review. Thanks a lot!
There is something wrong with the insertion. It is not showing the info after a click(?). Could look into it?
2024-07-<IP_ADDRESS>-1.mov

It is not clear how the bug occurred. The information can still be displayed after I click it. I will find the cause of the bug later.

It is not clear how the bug occurred. The information can still be displayed after I click it. I will find the cause of the bug later.

I was testing your work. This is amazing. Thank you so much! Besides this bug, there are two chinese comments in the view index.tsx file, I think we can remove them. If you want me to help with the bug, just let me know.

Sorry I missed them due to my negligence, they have been removed now. Regarding the bug situation, it would be very helpful if you can find the possible cause.

Sorry I missed them due to my negligence, they have been removed now. Regarding the bug situation, it would be very helpful if you can find the possible cause.

I think it is the isProcessing. What are we trying to prevent by using this isProcessing variable? Adding it at the following code fix the issue.
      if (existingDeveloperName === developerName) {
        isProcessing = false;
        return;
      }

but I got an another one while trying hovering really quick. It is not working properly somehow. We need to handle this better.
https://github.com/user-attachments/assets/3f31000a-1dfc-46a9-a101-be570ba723d5

Sorry I missed them due to my negligence, they have been removed now. Regarding the bug situation, it would be very helpful if you can find the possible cause.

I think it is the isProcessing. What are we trying to prevent by using this isProcessing variable? Adding it at the following code fix the issue.
      if (existingDeveloperName === developerName) {
        isProcessing = false;
        return;
      }

but I got an another one while trying hovering really quick. It is not working properly somehow. We need to handle this better.
2024-07-<IP_ADDRESS>.mov

This variable was originally used to prevent re-addition. It has worked well for me before, but now there seems to be some problems. I will deal with it later.

I tried to move the mouse back and forth on the edge of the avatar, but the information was not repeated. However, I have added the solution you mentioned into my code.
I have used isProcessing and the developer name to prevent duplicate additions, but there is still a situation that causes duplicate additions. That is, when the mouse quickly passes over the id and avatar, the information will be added twice because it is the same name. However, in theory, the following code should prevent duplicate additions.
if (existingDeveloperName === developerName) {
        isProcessing = false;
        return;
      }
      openRankDiv?.remove();

Sometimes the information is not displayed, but I am sure that OpenRank can be found. I also found that it was successfully displayed at the beginning, but it did not display after repeated several times. I still don’t know the reason.

Sometimes the information is not displayed, but I am sure that OpenRank can be found. I also found that it was successfully displayed at the beginning, but it did not display after repeated several times. I still don’t know the reason.

It is not showing because listener event functions are fired at the same time, and the latter one is prevented by the former one while the hovercard that the former one added is closed. Isprocessing prevents duplicates additions by keeping the first one and blocking the latter ones could be problematic. I add another way to cancel previous adding and keep the latest one. The idea is very simliar but I think this one is better for this case. Let me know if this works. Here is the code:
const init = async (): Promise<void> => {
  let abortController = new AbortController()
    const hovercardSelector = '[data-hovercard-type="user"]';
  document.querySelectorAll(hovercardSelector).forEach((element) => {
    // isProcessing is used to Prevent OpenRank from adding duplicates
    element.addEventListener('mouseover', async () => {
      abortController.abort();
      abortController = new AbortController();
      const signal = abortController.signal;

      const developerName = getDeveloperName(element as HTMLElement) as string;

      // Create a unique identifier for the popover
      const popoverId = `popover-${developerName}`;

      // Get the floating card container
      const $popoverContainer =
        'body > div.logged-in.env-production.page-responsive > div.Popover.js-hovercard-content.position-absolute > div > div > div';
      const popover = await elementReady($popoverContainer, { stopOnDomReady: false });

      const openRankDiv = popover?.querySelector('.openrank-info-container');
      const existingDeveloperName = openRankDiv?.getAttribute('data-developer-name');
      if (existingDeveloperName === developerName) {
        return;
      }
      openRankDiv?.remove();

      // Set the popover's unique identifier
      // make the current OpenRank information and person match
      popover?.setAttribute('data-popover-id', popoverId);

      const openrank = await getDeveloperLatestOpenrank(developerName);

      if (!openrank) {
        return;
      }

      if (!signal.aborted && popover && popover.getAttribute('data-popover-id') === popoverId) {
        // Check if the popover is still associated with the correct developer
        renderTo(popover, developerName, openrank);
      }
    });
  });
};

I have made some changes based on your code. Thanks to your code, I have a better understanding of asynchronous operations. Now the information can be displayed normally in most cases. At least I have not encountered the situation where it is no longer displayed.


Add -Xecj_write_empty_jdeps option

@rsalvador addressed differently with https://github.com/salesforce/bazel-jdt-java-toolchain/commit/0410e2b1f1bb2cfc8a10db2c6fe21dc1a3c1a691


Does reactabular support ie9?

I checked some sample code. Looks it supports IE9, just want to double check with you. Thanks in advance.

I would say it depends on React and lodash. You may also need to shim features like Array.isArray.
Ideally we would run the test suite against IE9 to know for sure.

I checked Lodash and React version. They still support IE9. As for the Array.isArray concern, I installed es5-shim. I will report you the result.

@redguardtoo Did you check? How did it go?

Yes, it's working on IE9, Win7. I install es5-shim v4.5.9, es6-promise 4.0.5, whatwg-fetch 0.11.1 (new version of whatwg drop the IE9 support)


Encoding issue

Replace input aéroport with ae ́roport
> route de l'aéroport 64121 Artix

Result:

{
  "house_number": "64000",
  "road": "route de l'ae ́roport",
  "postcode": "64121",
  "city": "artix"
}

This had to do with Unicode normalization, and is fixed in libpostal 1.0.


Create openapi specification for factoryinsight and add "v1.0.0 preview" to the documentation

Is your feature request related to a problem? Please describe.
It is unclear how the API will look like in v1.0.0
Describe the solution you'd like
We should document our target state for the v1.0.0 architecture in the documentation as soon as possible, so that other applications can build on top of that and that we have a shared goal.
For this we need to align on the openapi specification first

Superseded by https://github.com/united-manufacturing-hub/united-manufacturing-hub/issues/1385


Control variables are clasified in a module when they appear in a view as a model variable

The control variables (TIME STEP, INITIAL TIME, FINAL TIME, SAVEPER) are classified in a module when they appear in that view as a model variable and the model translation is split by views. This provokes a crash during model loading.
We should define their values always in the main file filtering the variables.
We should also include a warning for control variables when they are defined in a view or for those regular variables that are defined in more than one view.

Solved in branch views. I will open a PR soon.


Readme modifications

@adamantivm what do you think now?

@ashyonline and I just talked about ideas for improvement:

Have someone else (not me) test this and make it work
I feel the part about libstreaming (publisher) is intermixed with the rest and doesn't need to
I would put the parts about "how to use it", specifically how to use this library to make a player, concise and all together
I suggested thinking through how someone would use it, particularly thinking that someone who wants a player may already have a publisher (or would want a pointer to a publisher) and try to sketch the content accordingly (i.e.: in which ways can they connect their publisher to this player?)

@adamantivm I've made some changes. Can you merge it so I can go forward and I will then have someone else to test this and make it work?
Thanks


Duplicates exception in doctest block

(Original issue 1068 created by mgedmin on 2014-12-16T11:16:12.970119+00:00)
Steps to reproduce:
>>> import pygments.lexers, pygments.formatters
>>> lexer = pygments.lexers.PythonConsoleLexer()
>>> formatter = pygments.formatters.HtmlFormatter(nowrap=True)
>>> print(pygments.highlight('>>> f()\nTraceback (most recent call last):\n...\nException: foo', lexer, formatter))

Expected output (what Pygments 1.6 used to produce):
>>> f()
Traceback (most recent call last):
...
Exception: foo

Actual output with Pygments 2.0.1:
>>> f()
Traceback (most recent call last):
...
Exception: foo
Traceback (most recent call last):
...
Exception: foo

(Original comment by mgedmin on 2014-12-16T20:26:29.997595+00:00)
I've bisected the breakage to https://bitbucket.org/birkenfeld/pygments-main/commits/5b840c2aca04, which was committed to fix issue #874.

(Original comment by eneepo on 2014-12-18T11:40:15.130936+00:00)
I was going to report this bug, happy to see it's been reported already. This bug is even affecting Pygments documentaion because of using Sphinx. Before I figure out the problem I reported it on stackoverflow and sphinx

(Original comment by shimizukawa on 2015-01-20T01:53:46.018465+00:00)
bump!

(Original comment by birkenfeld on 2015-01-20T06:58:16.185981+00:00)
Fix Python tracebacks getting duplicated in the console lexer (closes #1068).
→ <>

(Original comment by birkenfeld on 2015-01-20T06:58:44.037107+00:00)
Thanks for the patience. Will be released as 2.0.2 in a few minutes.

(Original comment by mgedmin on 2015-01-20T07:21:43.191683+00:00)
Thank you! ♥

(Original comment by shimizukawa on 2015-01-20T07:42:34.696467+00:00)
Confirmed. Thanks!


Git markers won't be removed properly

VSCode Version: Code - Insiders 1.13.0-insider (c8123b036b66ca6e319bae00bfd7c34bb1ad977b, 2017-06-06T21:31:11.642Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

path-intellisense
christian-kohler
1.4.2

gitlens
eamodio
3.6.1

LogFileHighlighter
emilast
1.1.1

vscode-reveal
evilz
0.0.9

php-intellisense
felixfbecker
1.2.2

auto-close-tag
formulahendry
0.4.2

smarty
imperez
0.2.0

vscode-apache
mrmlnc
1.1.1

php-docblocker
neilbrayfield
1.2.0

vscode-versionlens
pflannery
0.18.1

vscode-icons
robertohuertasm
7.9.0

vscode-todo-highlight
wayou
0.5.5

Material-theme
zhuangtongfa
2.8.5

In this steps, I disabled any extension to report it properly.

Steps to Reproduce:
First of all, you need to define a keybinding of your choice to push changes.

Make a file change:

See the marker recognizes the change
Go to the git panel.
Write a commit message.
Use a shortcut to increase productivity: Hit Ctrl+Enter and immediately your keybinding (I use Ctrl+Shift+H).
See the loading badge comes up in the sidebar
After the loading badge disappeared, you still see the git marker:

Close the editor
Open the editor again
You see the git markers are gone.

In the moment shortly before submitting, the update badge appeared, but this also reproduces in the new insiders build.

Code might take up to 10 seconds to update its state. Are you sure it doesn't update after at most 10 seconds? Note that the viewlet still reports 1 change pending, so Code still hasn't caught up with whatever you have done.

// Place your key bindings in this file to overwrite the defaults
[
    {
        "key": "ctrl+n",
        "command": "-workbench.action.files.newUntitledFile"
    },
    {
        "key": "ctrl+n",
        "command": "explorer.newFile"
    },
    {
        "key": "ctrl+shift+h",
        "command": "git.push"
    },
    {
        "key": "f6",
        "command": "workbench.action.showAllSymbols"
    },
    {
        "key": "ctrl+t",
        "command": "-workbench.action.showAllSymbols"
    },
    {
        "key": "ctrl+m",
        "command": "-editor.action.toggleTabFocusMode"
    },
    {
        "key": "ctrl+m",
        "command": "explorer.newFolder"
    },
    {
        "key": "ctrl+f",
        "command": "-settings.action.search",
        "when": "inSettingsEditor"
    }
]

Note that the viewlet still reports 1 change pending

Are you sure? For me it's the extension viewlet which reports one update is ready to install?
Another shot:

(This is a private project, and I'm not allowed to show the code, but this is really general thing.)

Got it. This is #26642


MUMUP-1427 : add in error message for noscript

Before

After

plz wait on this. turns out they wanted diff styling.

Nitpicky item of style, but I'd strongly favor "use" over "utilize". "utilize" is "use" with the connotation of "using other than for its intended purpose", which isn't what we're going for here.  (This is a subtle "voice" item that maybe belongs in the style guide.)

I think you missed the whole "plz wait on this" part @apetro . Ready for review now.

:+1:

Good use of no-script.  :+1:


Syntactical Fixes

Fixed syntactical errors for Swift 1.2

Updated let declarations
Updated instantiation of constants to not use for loop
Updated credential instantiation to print error on username/password
Updated as to as! throughout

Great work Travis! I'll try to get to this tonight or first things in the morning. I'll more than likely have you delete this PR and I'll publish a feature branch for you to PR. That way I can run the unit tests go through some debugging with you on why you are getting that error that Jeremy pointed out.
Sent from my iPhone

On Apr 12, 2015, at 1:08 PM, Travis Bogosian<EMAIL_ADDRESS>wrote:
Fixed syntactical errors for Swift 1.2
Updated let declarations
Updated instantiation of constants to not use for loop
Updated credential instantiation to print error on username/password
Updated as to as! throughout
You can view, comment on, or merge this pull request online at:
https://github.com/GraphStory/neo4j-ios/pull/4
Commit Summary
Syntactical Fixes
File Changes
M Source/Theo/Theo/Client.swift (108)
M Source/Theo/Theo/Cypher.swift (19)
M Source/Theo/Theo/Node.swift (88)
M Source/Theo/Theo/Relationship.swift (58)
M Source/Theo/Theo/Request.swift (27)
Patch Links:
https://github.com/GraphStory/neo4j-ios/pull/4.patch
https://github.com/GraphStory/neo4j-ios/pull/4.diff
—
Reply to this email directly or view it on GitHub.

@tbogosia can you re-issue the PR to https://github.com/GraphStory/neo4j-ios/tree/develop/swift-1.2 I'll be able to start regression testing on that branch and then merge to master.

@kwylez all set!


Get HTTP status code for broken (or non-broken!) images

Searched through the library (I'm using it—thanks, it's awesome), but I can't seem to find a way to get at the http status code for broken (or non-broken!) images.
I believe this is available through the onComplete listener:
...
onComplete: function() { console.log(this.status); /* "404" for example */ }
...

Do you have any idea how I could get the status? Thanks.
(Been looking at XHR stuff like this: http://stackoverflow.com/a/3915698/209803 — if that helps at all. :P )

I'm closing this feature request as it didn't get enough public interest

I'd also like to see this happening, my case is for instance a YouTube cover image that doesnt exist.
Youtube returns a functioning image (and imagesloaded correctly reports its been loaded) but the requests also returns a 404 and an image that makes my application ugly.
E.g. https://i1.ytimg.com/vi/tZeMMyHKPY0/maxresdefault.jpg


AP check run not closing voucher

The AP check run leaves vouchers unclosed.   The correct check amount is committed, but vouchers remain open for next ap checkrun.

Resolution for this issue has been committed and will be released in next patch...6.3.3.

patch 6.3.3 released.  closing.


Add dev-tools submodule

This PR adds dev-tools submodule, the symlink to .clang-format file ( 3rdparty/dev-tools/.clang-format ) and .gitignore file which excludes output dirs, temporary emacs files and temporary VScode files.

What's about GitHub Actions? Are we going to have macOS & Linux & Windows actions which build all stuff, all tests, and run all checks for correct code style?

What's about GitHub Actions? Are we going to have macOS & Linux & Windows actions which build all stuff, all tests, and run all checks for correct code style? If yes I will do it in a separate PR then. Your thoughts @benh ?

Yes, please do that in a separate PR!


Calender respects timeformat

In reference to issue #776, made changes to calendar.js to respect timeformat config option if it is used

24h format does not get displayed correctly for me. It says (language:de timeFormat:24) "Morgen um 04:00" (Tomorrow at 04:00) whilist the appointment is at 4pm (16:00).
Fixed by setting case 24 to "LT: "hh:mm" line 84 commit 3404ebb

Same here. It doesn't work for me either (language:nl timeFormat: 24. You would think at least dutch would be tested).
If I change the timeFormat in calander.js to capital HH (like ytmytm mentioed) it does work!


set background image or imagedrawable fullscreen

Hi
Thanks for the great project ! Is there any option to set background image or make imagedrawable fullscreen like this https://github.com/ihsanbal/SlidingIntoView

As the library README mentions, you can pass custom UI elements through to the slides. So...yes.


Merge to upstream flash-attention repo

I am requesting that you merge with the upstream flash-attention repo, in order to garner community engagement and improving integration and distribution.
This separation is a major blocker to AMD adoption for AI model training and inference use cases.
https://github.com/Dao-AILab/flash-attention/issues/707

I too am hoping for a merge, currently this repo is some versions behind the upstream flash-attention repo which breaks compatibilitly with other open source projects that use flash-attention. For Example exllamav2 as described below:
oobabooga#3759
It feels like we are just a step away where where ROCm could be used in a lot of projects that would help owners of consumer gpus. Flash-Attention is a big step towards that goal. I am grateful for all the contributors!

There will be a new version of Composable Kernel coming out shortly. We will redesign the FA based on that one so I think we can request to merge into the upstream for that new version.

@howiejayz Is there going to be a new version upgrade? Recently, I've been using an MI250X to train Gemma and encountered some difficulties #51. I'm not sure if it's due to the version. I noticed that the upstream version mentions "All head dimensions up to 256. Head dim > 192 backward requires A100/A800 or H100/H800. Head dim 256 backward now works on consumer GPUs (if there's no dropout) as of flash-attn 2.5.5." Currently, the branch version seems to be 2.0.4.
Do you have any suggestions regarding this issue? Any help would be greatly appreciated.

Worth mentioning here, about exllamav2 and what it works with -
turns out this version isn't really of much use - here is their reply...
https://github.com/turboderp/exllamav2/issues/397#issuecomment-2034652594

@nktice Agreed, in Transformers we had to hack something to keep compatibility with 2.0.4 https://github.com/huggingface/transformers/blob/1ab71364886010c31b20dd8c8bb0c60f8a0681ad/src/transformers/models/llama/modeling_llama.py#L418

@howiejayz Is there going to be a new version upgrade? Recently, I've been using an MI250X to train Gemma and encountered some difficulties #51. I'm not sure if it's due to the version. I noticed that the upstream version mentions "All head dimensions up to 256. Head dim > 192 backward requires A100/A800 or H100/H800. Head dim 256 backward now works on consumer GPUs (if there's no dropout) as of flash-attn 2.5.5." Currently, the branch version seems to be 2.0.4.
Do you have any suggestions regarding this issue? Any help would be greatly appreciated.

We already started development of new flash 2.5.5 internally and hd256 support will be included. ETA is still under discussion.

@howiejayz Is there going to be a new version upgrade? Recently, I've been using an MI250X to train Gemma and encountered some difficulties #51. I'm not sure if it's due to the version. I noticed that the upstream version mentions "All head dimensions up to 256. Head dim > 192 backward requires A100/A800 or H100/H800. Head dim 256 backward now works on consumer GPUs (if there's no dropout) as of flash-attn 2.5.5." Currently, the branch version seems to be 2.0.4.
Do you have any suggestions regarding this issue? Any help would be greatly appreciated.

We already started development of new flash 2.5.5 internally and hd256 support will be included. ETA is still under discussion.

Will this also be for all officially supported gpus so rx7900xtx and 7900xt included ?

Worth mentioning here, about exllamav2 and what it works with - turns out this version isn't really of much use - here is their reply... turboderp/exllamav2#397 (comment)

This is the important part from that issue:

flash-attn introduced a crucial change in 2.1.0 without which it's really kind of useless for generating text. Before this it only worked with k_len = q_len or q_len = 1, ruling out features like cache reuse, speculative decoding and chunked prefill. ExLlama used to have some workarounds, but they were problematic and mostly just ended up disabling flash-attn anyway.

So I would say supporting 2.0.4 is hard. 2.1.0 should be possible (although it checks for version 2.2.1 at the moment.)

Progressing

AOTriton has been updated in PyTorch nightly to add support for Navi31, which enables Triton-based Flash Attention in PyTorch for Navi31 out of the box:

https://github.com/pytorch/pytorch/pull/134498#issuecomment-2345330117

@evshiron , does this Triton-based Flash Attention support AMD MI60/gfx906 cards?

@Said-Akbar
I don't have these graphics cards, but in a quick search I could not find any code mentioning gfx906 in the Triton repo, so my answer will be no.


Home Screen displays calorie/nutrition intake

Home Screen After Login:

Home Screen While Inputting New Meal:

Home Screen After Inputting New Meal:

Still needs to connect to Google Fit API for accurate estimated calorie for day.

Update: Added placeholder for recommended meal

so booootiful <3


Fix the property of undefined object

My case:
Script stop if try to compile "object.length" when object=undefined. Fixed with try/catch.

Thanks!

@webarthur I tested it and your update doesn't work. I reverted the commit.
Solution for you:
{{ if object }}
<div>{{ object.length }}</div>
{{ fi }}

Thanks!

Mustache and AngularJS don't have this trouble. Use a IF for each empty object is not a good solution. I'm working in a small mvvm jquery framework, but ok... Thank you too!


event: add Stream

This implements a sequential message consumer.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#103
#102  👈
#101
#100
#99
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jespert and the rest of your teammates on  Graphite

Merge activity

Oct 1, 8:39 AM EDT: @jespert merged this pull request with Graphite.


[Develop] Add berkeley archive migration app to nix

Original PR: Add berkeley archive migration app to nix #15339
Problem: berkeley archive migration app is not specified in the nix build.
Solution: add berkeley archive migration app into flakes.
Note: building the archive migration app requires DUNE_PROFILE to be set to a specialized value.
Explain how you tested your changes:

[x] Built mina-berkeley-migration with nix
[ ] (TODO) Tested the whole migration flow with the app

Checklist:

[x] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[x] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me


[Feature Request] “Solid” variant with no panel shadow

Currently, the panels have drop shadow and KDE Plasma has no easy way remove them. It would be great if there are “solid” variants for the plasma theme with no drop shadow for panels.

Could you attach a picture as to what you mean by shadow?

Here is a screenshot with the “Lightly Plasma” Plasma style that comes with Catppuccin. Notice that there is a drop shadow between the panel and the application.

As a contrast, here is the Plasma style from “Nordic Solid”, which does not have a shadow and the panel seamlessly blends into the application.

What I am trying to achive is this seamless blend, when I use Catppuccin KDE and Konsole theme.

Thank you for the screenshot, Yeah, we'll try to fix that by moving away from lightly plasma eventually. Right now, there are some licencing issues we need to sort out first. One option is for you specifically is to change the plasma theme until we change the plasma theme we bundle with.

But also, I think most plasma styles have a shadow since it's in the default breeze so there's a chance the style we bundle with the theme will end up having it too.

I understand the shadow variant being the default option. But are you open to make a solid variant along with it? Only a few background files will be different between those.

Yup. I won't be making the plasma style since I have no experience with SVGs, but we can definitely look into it.

hey @onnyyonn, an update on this.
For now if you want you can change /home/{USER}/.local/share/plasma/desktoptheme/lightly-plasma-git/widgets/panel-background.svgz with
After you download rename it to "panel-background.svgz",
Github doesn't allow you to upload .svgz files :p.
As for switching to a different plasma theme, we're considering making modifications to lightly plasma.

This was now merged into main and there is no panel shadow, so I'll close the issue.


Reducing false positives for yaml.load

Description
The previous check allows for the use of yaml.load in the following fashion:
yaml.load(data, Loader=SafeLoader)

However, we should not penalize (and falsely alert) users for using the faster version of SafeLoader, CSafeLoader. This is an example of CSafeLoader being used.
Fix
Just whitelist it as well.

Thank you for contributing to openstack/bandit!
openstack/bandit uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


[docs] Confusing docs for assertions wait mechanism

Documentation for the assertion mechanism is confusing. It is not clear if it is required to check whether an element exists before checking its properties.
example

The January update to the Assertions guide fixed this issue
https://testcafe.io/documentation/402837/guides/basic-guides/assertions


fix fix bug for A>n, where A.max is infinity #39936

Proposed changes
pick #39936
Issue Number: close #xxx

run buildall

run buildall


Remove: calibre.calibre version 4.21.0

Removing old minor versions to reduce clutter and download issues

[X] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

this was supposed to be excluded


Fixes broken link in the Getting Started section of the README

Description
Good day, TypeScript team! 🚀   This very small PR fixes the broken link in the Getting Started section of the README.
Screenshots
Image of the broken link in README

Image of the result of the fix being applied

Thanks!


Protocol error (Input.dispatchMouseEvent): Target closed?

Environment

chrome-aws-lambda Version: 10.1.0
puppeteer / puppeteer-core Version: 10.1.0
OS: amazon-linux
Node.js Version: 14.x
Lambda / GCF Runtime: <!-- nodejs12.x

Expected Behavior
Should work properly.
Current Behavior
When running inside the lambda, in headless mode, we are seeing errors like these:
Error: Protocol error (Input.dispatchMouseEvent): Target closed.
   at /tmp/wdio/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:217:63
   at new Promise (<anonymous>)
   at CDPSession.send (/tmp/wdio/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:216:16)
   at Mouse.move (/tmp/wdio/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Input.js:343:32)
   at Mouse.click (/tmp/wdio/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Input.js:367:24)
   at ElementHandle.click (/tmp/wdio/node_modules/puppeteer-core/lib/cjs/puppeteer/common/JSHandle.js:416:32)
  at processTicksAndRejections (internal/process/task_queues.js:95:5)

We have a hard time understanding what is not working as expected and what happens. Mostly, we see these errors after navigation operation, but not only.
Not only with the mouse event, but other things as well, they all end eventually with 'Target closed'.
We guess that the chrome/chromium crashes inside the lambda, but we are having a very hard time understanding why and when.
We are running our tests with webdriover.io, using the CDP/Puppeteer protocol and using the chrome-aws-lambda.
Steps to Reproduce
Hard to explain. Running puppeteer in a lambda with some consuming/not-simple-websites (like web builders, photoshop, or sites that have animations), or sometimes even having many navigations inside single tests and running it hundreds of times a day.
It is not only getting crashed, but it is also getting the lambda stuck.
Possible Solution
Hard to say. We would like to hear ideas, perhaps someone already seen and solved/worked around the problem.
Thanks, everyone.

I also have this issue. Everything worked properly. But since yesterday I run into this issue.

I've never gotten this version to work properly for my use case. Have you tried either downgrading to 9.x or upgrading to 12.x?
https://github.com/alixaxel/chrome-aws-lambda/issues/255


WL-3325: Rewrite URLs

... that point to http[s]://[beta.]weblearn.ox.ac.uk/ on staging to https://staging.weblearn.ox.ac.uk/.
This was missed in the merge.

Hate this code, should really fix the source HTML as it would have never worked on staging-werp.bsp.ox.ac.uk


Detect NEO's with CCID disabled

CCID is disabled by default on NEO's. This can cause confusion for users of Yubico Authenticator.
Yubico Authenticator should detect that a NEO without Smart Card communication is inserted (and give the user an option to enable it). Since some users might want to just use the OTP slots (not enable CCID), there should be a "Don't show this again" checkbox in the dialog window.
See this forum topic for discussion.

At the very least, at least display a message that CCID needs to be enabled, and to advise the user to use the NEO Manager app to enable it. Right now, there is no way to know what to do (or remember that you need to do this). This might be a short term fix?


[QUESTION] Azure SDK to query Azure HCI

Library name and version
Microsoft.Azure.Management.Websites
Query/Question
Hi,
Our customer doesn't want to use Azure Public Cloud, but he has Azure HCI. We have already some automation tools that target the Public Azure Cloud using the .net Azure SDK libraries. Are these compatible with Azure HCI ( meaning, we only need to change the endpoint to the Azure HCI and everything will work just the same ), or Azure HCI has its own set of management libraries ?
Regards,
Yeska
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.  Please be aware that responses will be delayed due to the US holidays.

You can try this code
An ArmEnvironment object can let you customize the Azure cloud you want your application to run
              var options=new ArmClientOptions();
               var env = new ArmEnvironment(new Uri("your HCI endpoint"), "your audience");
               options. Environment=env;
               var client = new ArmClient(new DefaultAzureCredential(
                   new DefaultAzureCredentialOptions()
                   { VisualStudioTenantId=context.Configuration.GetValue<string>("VisualStudioTenantId") }),
                   context.Configuration.GetValue<string>("DefaultSubscriptionId"),
                   options);

Thank you @xboxeer, that's very helpful. Do you know if there is a list of Azure HCI compatibles services ? I couldn't find that anywhere. In our project we need these ones:

@YeskaNova I don't think we have this compatible services list, azure sdk is usually based on latest version of azure rest API, but you can manual overwrite the api versions that sdk is relying on
options.SetApiVersion(new Azure.Core.ResourceType("resourceType"), "apiVersion");

different api version may have breaking changes, so it is recommend that you test your code before deploy to prod


Town buildings are populated in wrong order

My Player got spawned in a building so small, that it didn't even fit Barkeep, so I started investigating.
Code in town.rs/building_factory is incorrect. It iterates over original buildings array, and uses its index to pick build_type form building_index array, which was sorted and has different indexing:
for (i, building) in buildings.iter().enumerate() {
    let build_type = &building_index[i].2;
    match build_type {

It should be changed to something like this:
for (i, _size, build_type) in building_index.iter() {
    let building = &buildings[*i];
    match build_type {

Much better, makes sense!
Thank you!


Pow(x, n)

Enter your question -
Implement pow(x, n), which calculates x raised to the power n (i.e., xn).
Enter link to the question(if question belongs to any online platform) -
https://leetcode.com/problems/powx-n/
Tags for the question(eg - Array, Basic, Stack, etc.) -
Recursion, Medium
Pull request - #44

@ishaangupta-YB,
Kindly add your solution to "leetcode" folder.
Deadline - 02/10/2021
I am assigning the issue since you have created PR already. But a gentle reminder - Abide by the guidelines

Sir please assign this issue to me
Why you should assign this to me?
i am current rated 5 star at codechef with rating of 2153 and rated expert at codeforces with rating of 1776
https://codeforces.com/profile/navjotdadwal
https://www.codechef.com/users/navjot1234
So i can handle and solve this issue

Enter your question -
Implement pow(x, n), which calculates x raised to the power n (i.e., xn).
Enter link to the question(if question belongs to any online platform) -
https://www.geeksforgeeks.org/write-a-c-program-to-calculate-powxn/
https://leetcode.com/problems/powx-n/
Tags for the question(eg - Array, Basic, Stack, etc.) -
Recursion, Medium
@SarthakKeshari  please assign this issue to me

@Anova07,
Kindly go through CONTRIBUTING.md

@ishaangupta-YB,
Deadline crossed

@SarthakKeshari is this issue assigned to me, can I work on it and provide a solution to this question ?

@Anova07,
Now since the deadline is crossed you can solve this.
Kindly add your solution to "leetcode" folder.
Deadline - 02/10/2021
This Issue will only be counted towards ranking criteria of this repository.
Gentle advice for futher Issues -
Medium and hard level questions will be given preference for Hacktoberfest tag.

@Anova07,
Now since the deadline is crossed you can solve this.
Kindly add your solution to "leetcode" folder.
Deadline - 05/10/2021