Add new lidar type on server side

Hello Carla team !
I'm having a hard time adding a new lidar type
CARLA version : 0.9.13
Platform/OS : Ubuntu20.04
Problem you have experienced: I'm using a lidar that doesn't rotate around Z axis, hence cannot be represented through existing lidar types (RayCast/SemanticRayCast). I'd like to add a new lidar to the available sensors on the server side, and retrieve its data on the client side.
I thought I'd reuse existing semantic lidar sensor two tweak it to my need, so I duplicated the SemanticRayCast lidar. First, before any tweaking, I'd like the new sensor to be available "as is" (i.e. with default rotating properties), so :

In Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Sensor/

Duplicated RayCastSemanticLidar.cpp & .h, named it MyLidar.cpp /.h
Renamed the class from ARayCastSemanticLidar to AMyLidar
In AMyLidar::::GetSensorDefinition() method, changed TEXT("ray_cast_semantic") to TEXT("my_lidar")

In /home/barth/Projects/carla/karla1/LibCarla/source/carla/sensor

Added the sensor to SensorRegistry.h as detailed within the .h file.

Since the data format my lidar will be generating will look alike the SemanticLidar one, I used : std::pair<AMyLidar *, s11n::SemanticLidarSerializer>

Note : I also tried to duplicate the serializer (s11n) + data files (SemanticLidarData.h + SemanticLidarMeasurement.h) for MyLidar to have its own Serializer etc... in case it would help, but haven't been more successfull this way. (Note: in this case I also updated PythonAPI/carla/source/libcarla/SensorData.cpp so it knows about "MyLidar" measurements, detection and so on)

In Plugins/Carla/Source/Carla/Actor/ActorBlueprintFunctionLibrary.cpp

Added an else/if at the end of UActorBlueprintFunctionLibrary::MakeLidarDefinition(..) to take into account my lidar : else if (Id == "my_lidar") {..}

In the end, everything compiles when I duplicate the Serializer, Data, Measurement, and I can run the editor, but as soon as I use the PythonAPI (launch vehicle_gallery.py for instance) I get a core dump on the python client side.
What you expected to happen: Be able to spawn and retrieve data from a custom lidar sensor.
Steps to reproduce: Already explained above
Other information (documentation you consulted, workarounds you tried):

Followed tutorial for the Safe Distance Sensor  at https://carla.readthedocs.io/en/latest/tuto_D_create_sensor/#appendix-client-side-sensors : it worked
Didn't find anything related to developing lidar sensors on server side in the issues nor in the documentation

Side question : is their something preventing from having two sensors using the same data and serializer ? (i.e. something like     std::pair<ARayCastSemanticLidar *, s11n::SemanticLidarSerializer>, std::pair<AMyLidar *, s11n::SemanticLidarSerializer>,
in SensorRegistry.h
Any help on this would be much appreciated !

Hi, I am facing the exact same issue. I am trying to create a livox lidar in Carla. At first, I created this new lidar but couldn't get any success but then I just copied everything from RaycastLidar sensor and changed the names as you did. I have the exact same error as you do "Segmentation fault (core dumped)". I think it is something to do with sending data. Were you able to figure this out?

So, it looks like having the same data and serializer is an issue. But that is not the only problem. I did the following tests,

I created a new sensor by copying everything from RayCastLidar sensor, however, the data and serializer were the same. In this situation, I was continuously getting this error "Segmentation fault (core dumped)".
After that, I removed the the RayCastLidar.h and RayCastLidar.cpp files and removed the corresponding entries in the SensorRegistry.h file. By just doing that, the newly added lidar started working.
So, I created the data and serializer with the new class and file names for my new lidar sensor. I brought back the RayCastLidar.h and RayCastLidar.cpp files. Also uncommented the RayCastLidar configuration in the SensorRegistry.h  This time, I got the "Segmentation fault (core dumped)" error again.
I removed the RayCastLidar.h and RayCastLidar.cpp file again and commented the corresponding configuration in the SensorRegistry.h. The new lidar sensor started working with the newly added serializer and data classes and files.

So now, I am not sure what the issue is. Why is having RayCastLidar and LivoxLidar files at the same time is causing "Segmentation fault (core dumped)" error. Let me know if you are able to find something.

Hey guys, any updates on such adding new lidar routine? I was about to do the duplication trick as well and having lots of issues in rebuilding the pythonAPI. So I was wondering if this could work or not.

Sorry, I have been working on other things lately. Hope I'll be able to come back to it. I'll post the progresses here if any 🤞

Hello guys,
I Finally managed to make it work 🥳.
Anyone still interested by this ?

Hello guys,
I Finally managed to make it work 🥳. Anyone still interested by this ?

Hi, yes.
How did you make it work?

Hello guys,
I Finally managed to make it work 🥳. Anyone still interested by this ?

Hi, I am interested too

Hello!
You can check logs of that segmentation fault at CARLA_ROOT/ Unreal/CarlaUE4/Saved/Logs
Check those logs and it may give you a hint of what is going on. Without logs I cannot help you much.
Could you supply me with those?
We will try to create a how to create a new LIDAR sensor

I am closing the issue feel free to reopen


Creators update breaks RDWrap

Updated to Windows 10 creators update and RDWrap is now broken (won't connect, I can't allow connections)
Tried re-installing RDWrap

This tells nothing. NT version? Edition?

Just to chime in here, I have the same problem.
This is RDPConf:

Windows 10 Professional x64, Version 1703, OS Build 15063.13
This is the final version, which I installed through the Microsoft Setup Assistant.

Same here. Instead, it shows it is not listening. Mine is Windows 10 Home X64. I had rdpwap installed and working fine before upgrade. I uninstalled rdpwrap. Upgraded to Creator via Windows 10 Upgrade Assistant. Downloaded Rdpwap 1.6.1 again and ran install.bat.

@foxx1337 your version is not up to date.
@rykp your issue is #194.
I'm waiting for the answer from @Recka50.

Thanks for your quick note. I see. Bad luck :( Have been using rdpwrap and love it. Look forward to a fix.

Hey all,
I came across this issue because after the creator update my install also got broken...
Tried  https://github.com/stascorp/rdpwrap/issues/45#issuecomment-182632817 and https://github.com/stascorp/rdpwrap/issues/194 but still a no go.

This is a print of the info that I get...
Os version 1703 and compilation 15063.13
Thanks for the help.

If your edition is Windows 10 Home and your termsrv build is 15004 and higher, then it WILL NOT work, because of #194 issue.

@binarymaster Is anything I can do to help fix it?
Just a side note, it was working just fine before the creators update (witch i did yesterday... dumb self)...
I know that they introduced something, and the size of the builds is different on > 10.0.15002.1001 builds, but #https://github.com/stascorp/rdpwrap/issues/194 was created on 14 Jan, where these people on the insiders program or something? How did they get > 10.0.15002.1001 this version so soon?

Yes, this problem was known before "Creators update", they are insiders indeed.

So, closing as duplicate of #194.

Hello.
Any solution for this issue.
I have just installed the creators update and It does not work
Thnx

hope master can give the solution instead of close and mention duplicate :(

Looks like the Administrator isn't listening either (lol).  Haven't heard a response

@mrpentium @MikeSam06 @JJMadrigal It doesn't help for binarymaster to talk to everyone on every issue. So, he consolidates duplicate issues into one and performs discussion on one channel, making it alot easier for everyone. Complaining that he closed your issue when there is an identical one won't help. See #194, there is discussion happening there. That is the main issue you are all experiencing (well, most of you).

Just download and copy the following file to c:\windows\system32, then reboot, hope this helps.
64-bit Windows 10: https://github.com/stascorp/rdpwrap/files/1236856/rfxvmt.zip
32-bit Windows 10: https://github.com/stascorp/rdpwrap/files/1238499/rfxvmt.zip

Thank you!!!!! That just fixed my issue with it not working and i've been waiting since updating to the creators update when it was released. When was that fix announced as I haven't seen it until now?!

Wow man this thing works . I started to explore other options just before i saw this ! Thanks man

Works great for me too - a big thank you !!!

Thank you, thank you, thank you!!!!  - did fresh install of rdp wrapper and added .dll file to system32 and now everything works.

Sweet. werked for me tooo!

Thank you Explorhk
Worked like a charm
Can you briefly describe why that works ?
I wonder why that fix isn't in the "UPDATE.bat" process ?
thanks again

@Steve-1029 This file became necessary to add in the Creators Update. @binarymaster hasn't fixed update.bat yet to do so, and it would be illegal for us to distribute this file automatically.

I can upload a 64-bit rfxvmt.dll from Windows 10 Pro 1709 Monday evening. Do I get it from system32 or SysWOW64?

Just in case if you guys have upgraded to Windows 10 Pro 1709, please find the newest file attached.
For x64
rfxvmt_x64_Build1709.zip

Question for clarification:
the FIX for version 1703 build 15063  was your dll  named rfxvmt.zip
works great.  Working on several machines
Now Microsoft has a "Targeted Version" 1709 build 16299.19
but the Update tool does not reflect that build yet.  My test Win-Home machine says it is up to date on 1703.  I guess "Targeted" is similar to Beta test version ?
So I guess sometime in the future MSFT will force that update on me
and when that happens the RDP will be broken again
and then I need to use your new rfxvmt_64_Build1709.zip
Sorry to be a little dense here but I am a Linux/MAC OSX user and I have been dragged kicking and screaming into the Win-10 world by one app that requires Windows

Pro ? I guess that should be 'Home'. Afaik 'Pro' has RDP available so
no need for RDPWrap ...
Van: "explorerhk"<EMAIL_ADDRESS>Aan: "stascorp/rdpwrap"<EMAIL_ADDRESS>Cc: "danny vandepol"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Verzonden: Donderdag 2 november 2017 17:19:50
Onderwerp: Re: [stascorp/rdpwrap] Creators update breaks RDWrap (#229)
Just in case if you guys have upgraded to Windows 10 Pro 1709, please find the newest file attached.
For x64
[ https://github.com/stascorp/rdpwrap/files/1438515/rfxvmt_x64_Build1709.zip | rfxvmt_x64_Build1709.zip ]
—
You are receiving this because you commented.
Reply to this email directly, [ https://github.com/stascorp/rdpwrap/issues/229#issuecomment-341475994 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AEl9suUX9735oiLqhe1EY428JNzywJbuks5syeumgaJpZM4M2TAh | mute the thread ] .

I just tested on both fresh install and upgrade version of Windows 10 Home 1709, it works fine.

Download RDPWrap v1.6
https://github.com/stascorp/rdpwrap/releases/download/v1.6.1/RDPWrap-v1.6.1.zip
Extract and install it.
Copy rfxvmt.dll to c:\windows\system32
Reboot

x64
https://github.com/stascorp/rdpwrap/files/1438515/rfxvmt_x64_Build1709.zip
x86
https://github.com/stascorp/rdpwrap/files/1443741/rfxvmt_x86_Build1709.zip
Hope it helps.

explorerhk wrote:

Copy rfxvmt.dll to c:\windows\system32
Reboot
x64
https://github.com/stascorp/rdpwrap/files/1438515/rfxvmt_x64_Build1709.zip
x86
https://github.com/stascorp/rdpwrap/files/1443741/rfxvmt_x86_Build1709.zip

It would be nice to have a way to get rfxvmt.dll from the original sources, though, as they can request the removal of these attachments from GitHub.  I am not sure about the latest version of the .dll file, but at least it is contained in these updates:

Cumulative update for Windows 10 Version 1511: February 9, 2016
https://support.microsoft.com/en-us/help/3135173/cumulative-update-for-windows-10-version-1511-february-9--2016

Cumulative update for Windows 10: February 9, 2016
https://support.microsoft.com/en-us/help/3135174/cumulative-update-for-windows-10-february-9--2016

The needed files are called this, I guess:

windows10.0-kb3135173-x86.msu
(SHA256) D212BC27E7623935EACE95235678053A36AEE44EE74AA73A56D193567E0C8890
windows10.0-kb3135173-x64.msu
(SHA256) E3474E1F4FF12C4BC3C7BD77760ACD2472C7D87E964AD7E1C3ECE73A089D2D0C
windows10.0-kb3135174-x86.msu
(SHA256) 10420DB8A3D7AD7C1B4195F89828129CA89DF2B94886855BEEC171DD520EC774
windows10.0-kb3135174-x64.msu
(SHA256) C179CF48BF9153A8390A789C1B1904CFE595AA5DFBA8E1E73F592E58AD572DDC

Random page that provides direct download URLs:
https://www.tenforums.com/windows-10-news/40282-cumulative-update-windows-10-version-1511-kb3135173.html
The other page tells download sizes:
http://ms-vnext.net/UpdateArchive/
http://download.windowsupdate.com/d/msdownload/update/software/secu/2016/02/windows10.0-kb3135173-x86_31f87a03e38211ca3572506c466c9e2eb0fcdb8e.msu (286.56 MB)
http://download.windowsupdate.com/c/msdownload/update/software/secu/2016/02/windows10.0-kb3135173-x64_8709ec87df2757c6077d2e7178dbf083473f138a.msu (612.44 MB)
http://download.windowsupdate.com/d/msdownload/update/software/secu/2016/02/windows10.0-kb3135174-x86_9c8184592c519d2bd1ada43d09f51e9eb01413f3.msu (270.34 MB)
http://download.windowsupdate.com/d/msdownload/update/software/secu/2016/02/windows10.0-kb3135174-x64_3c0cd8e1a7ad32c995c3ed492f20548a1e1cc4f7.msu (489.30 MB)
Maybe there are more recent version released, but I haven't found any newer for w10.
The files are quite large, and to get the needed "rfxvmt.dll" from them, the software will need to extract internal .cab file at first (using eg. extract.exe, builtin utility) and then extract the .dll from the .cab (using the same extract.exe).
This would be tedious but officially legal way to get the missing file (if the original is lost).
Maybe some older versions can be extracted from the installation files saved by Windows setup program to C: , but I have no idea about it's location since XP and the i386 directory.  Probably something useful could be extracted from SxS dir:
https://blogs.technet.microsoft.com/askcore/2008/09/17/what-is-the-winsxs-directory-in-windows-2008-and-windows-vista-and-why-is-it-so-large/
Any more elegant ideas on restoring the .dll ?

Sorry this does not work or resolve the LISTENER STATE issue in Win Pro 10 1709. I already tried it and its not working. Please advise.
Note: I only replaced rfxvmt.dll in \System32 not in \SYSWOW

Have you installed RDWrap before replacing the dll file? You also need to reboot your PC to take the changes

Yes i did, and i also rebooted... 100% doesnt work.... other versions have worked....

@mitchmenghi check the correctness of the version you copied to system32, maybe wrong architecture. Then try to reinstall RDP Wrapper.

It was working for me on 1703, but not since 1709 update, since that update I can not get it working.
I have tried all the list of steps here, but it is fully supported but still not listening.
Tried: New x64 DLL, uninstall, install, update, reboot, checked firewall ports, disabled firewall.
Using Win 10 Home x64

Thanks @explorerhk, Copying rfxvmt.dll file into Windows\System32 worked for me :)
Using Win 10 Home x86 (1709)

I have had one success
RDP Wrapper running fine on 1703
Update 1703 >> 1709
RDP broken
RECOPIED the rfxvmt.dll to System32 as the Upgrade removed it
Reboot Win-10  -  RDP Wrapper still broken
re-ran the install.bat   I just clicked the bat file
Everything working again
It has worked One time in a row for me

was working fine until yesterday
but seems not to work no more

Sickingajay-
Did your computer do a forced Upgrade to ver 1709 ?
Use to work on 1703 abut now "Upgraded" ?

@Steve-1029
No, as far as I remember.

Check these things
do  start >> run >> winver      this will tell you what release of win-10
then check c:\windows\systme32  to see if the file rfxvmt.dll is in there
On my computer when Win-10 was on version 1703 all working with the RDP wrap 1.6.1
and the rfxvmt.dll  fix
but when windows upgraded itself to 1709 the dll file was removed

@Steve-1029
windows 1709
rfxvmt.dll was removed
placed it again in system32
but still gives the same error
can't seem to find any fix to this recent upgrade

Ok
So after copying the dell reboot your machine
Then hit the install.bat script
Then the system went all Green for me

On Dec 17, 2017, at 10:56, sickingajay<EMAIL_ADDRESS>wrote:
@Steve-1029
windows 1709
rfxvmt.dll was removed
placed it again in system32
but still gives the same error
can't seem to find any fix to this recent upgrade
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Got it running again too on Win10 Home x64 after the Creator's Update - 1709:
Copied 32 bit dll to system32 and 64 bit dll to SysWOW64, ran uninstall.bat, next install.bat, reboot and all green lights. I guess uninstall/install isn't really needed but it didn't hurt ...

Tried almost everything said by everyone here
but it still says 'not listening'
Still looking for an answer
thanks

Copied 32 bit dll to system32 and 64 bit dll to SysWOW64...
64 bit to system32 and 32 bit to SysWOW64 ?

can someone provide the link for 32 bit and 64 bit dll files

Have you read the whole thread?

I'm having the same problem as @sickingajay .  Running 1709 build 16299.125 and tried multiple times all steps listed (including re-downloading all files) and still no luck.  Checked firewall also.  Any other suggestions?

I also have this issue...
Have tryed all thread hints and tips with no luck,
Stuck on the NOT LISTENING and have win 10 home (1709 build 16299.125) (x64) (DK language)
Have tryed copying and replacing dll files in all mentioned folders, also rebootet after each try. have tryed with both 32 & 64 bit versions of dll's and mixed them (just to try)
Have used admin rights on all cmds. Have tryed giving ALL users all rights to dll's and tryed disabling the firewall completely.
Result = NOT LISTENING and no response for RDP connections. Even tryed to change the RDP port = still no luck...
Hope someone has a trick for this :)
/ Merry christmas everyone ....

I'm also not being able to get this to work with build 1709 16299.125 (x64).
I tried copying both the 32-bit and 64-bit DLL to their respective folders (I've also tried to copy only the 64-bit DLL), rebooted many times, uninstalled and installed rdpwrap multiple times.

To everyone with problems with build 1709 16299.125, I was just able to solve by following the instructions from sDunkan: https://github.com/stascorp/rdpwrap/issues/194#issuecomment-339586541 .
Opening command prompt as administrator seemed to be what I was missing. Hopefully, it helps folks on this issue with the same problem.

So after restarting several time everything turned green
Just copy 64-bit dll file to system32
But the only thing that's not working is
whatever I do after taking RDP on my friends pc
He's able to see everything
unchecked the single user as well but that doesn't seem to do anything
Anyone facing any similar issues?

@raisen , had tryed that already... and I have now tryed again... It still dosen't work...
I have no clue of what to try next.... Everything mentioned in this tread have been tryed with/without admininstrator mode cmd, with/without reboots between each step, withboth x86 & x64 dll files.
With firewall on/disabled.... I'm almost giving up.....

@SpirravipDK I was almost giving up too as I was getting nowhere as well.
I used this link to download the DLL:
https://github.com/stascorp/rdpwrap/files/1236856/rfxvmt.zip
I had previously downloaded the DLL from this link: https://github.com/stascorp/rdpwrap/files/1438515/rfxvmt_x64_Build1709.zip
and it didn't work - maybe it's the reason for you too? I don't know...
I didn't have to reboot after I followed sDunkan's instructions.

i'll give it a try , thanks...


Thrust will be rebasing off of Atom Shell

Hey contributors, users, and fans. Over the next few months as I have time, I will be planning and implementing the rebasing of Thrust to Atom Shell. What does this mean. Well, we are all familiar with the problems of Atom-Shell, and those are the reasons we choose to stay with Thrust, for instance, lack of Polyglot api, retargeting C Node Modules for the specific version of node used to compile Atom-Shell, and a host of other issues that ExoBrowser as well faced prior to Thrust.
Due to the complex nature of working with the Blink API, thrust as it stands is just not feasible for a long term project, with no financial backing, nor C programmers whom are willing to commit the time to maintaining it. We already use Brightray which is the Chromium Content Library wrapper that Atom-Shell uses as is infact built by ZCBenz as well, now we will be taking it one step further and using Atom-Shell as the basis for our Thrust EcoSystem.
We are beginning to look at Thrust as a standard for Polyglot application development using Chromium Content Library, rather than as a specific implementation.
What does this look like, well we will code the same RPC abilities into Atom-Shell, and use that as the "Thrust" core module, and the *-Thrust modules will download it, consume it, and launch it the same exact way.
I promise to try to maintain 100% backwards compatibility being the author of Go-Thrust as well. My goal is to enable stability, features, and hope for those of you looking to use this technology in Products and Business settings.
As well if I cannot maintain backwards compatibility I will try to submit pull requests to the *-Thrust repositories, or at least assist in upgrading.
Thank you all for your support and love of Thrust. I hope to be able to create something stable for you.
I am just one man ( a busy one at that ) so any help, contributions, ideas, thoughts, criticisms, are very welcome.
Sincerely,
Michael Hernandez

Does this still mean that Node and chromium will be in separate processes?  My problem with atom shell has always been that I have to get my code working with the particular version of Node inside of Atom Shell. I could never get node-serialport working properly with it, but with Thrust it was a cinch.

@joshmarinacci  Yea, the point is to maintain Thrust's current system and flow, almost completely, If i can't do that, then I will not commit the change. In this new version, you will have Node available in chrome, however only in so far as a helper. There will be no APM to install and retarget native modules.
For 99% of your backend javascript you will still use a seperate node process.
Thrust will be the Bridge between Atom Shell and your Node process. With possibly its own fork of AtomShell. Heres the current Segmentation of Technologies
Blink/V8  ==> Brightray and Embedded Node ==> Atom-Shell (With injectable Shell)
Blink/V8  ==> Brigtray ==> Thrust ===> Node
What I want to do
Blink/V8  ==> Brighray and Embedded Node ==> Atom-Shell (with Thrust as Injected Shell) ==> (NodeJS, GoLang, Python, Scala, Perl, ...)

@spolu @miketheprogrammer I like the programmable browser concept very much and hope this work continues. How does this tie in with the first exobrowser concept?
Say, in theory, somebody (with a large pocket of $) would want to write a programmable browser from scratch. It seems the challenge is that browsers are not really meant to be programmable in this way, in the first place.  Certainly Google has thought of this and try to get around this with things like polymer and Dart.
A while back I tried starting off with a VM/container, so that building Chromium and complex stacks can be automated. I used Vagrant but run into some problems - VM's are not designed to display graphics.

Hi!
Just wanted to let you guys know I have uploaded a library that lets you control electron (formerly known as atom-shell) with another process via pipes and JSON messages, similar to how Thrust works. In fact, Thrust was my main inspiration for this project:
https://github.com/hoytech/valence
I agree that Thrust's approach of not dictating controller language/run-time is the right way to go, however my concern with standardising on the Thrust API is that its feature-set is very limited compared to electron's. Valence is essentially an RPC bridge so it should support everything that electron does.
Here are the perl language bindings:
https://metacpan.org/pod/Valence
Cheers,
Doug

This is awesome. I'll hack on it right away!
On Monday, April 27, 2015, Doug Hoyte<EMAIL_ADDRESS>wrote:

Hi!
Just wanted to let you guys know I have uploaded a library that lets you
control electron (formerly known as atom-shell) with another process via
pipes and JSON messages, similar to how Thrust works. In fact, Thrust was
my main inspiration for this project:
https://github.com/hoytech/valence
I agree that Thrust's approach of not dictating controller
language/run-time is the right way to go, however my concern with
standardising on the Thrust API is that its feature-set is very limited
compared to electron's. Valence is essentially an RPC bridge so it should
support everything that electron does.
Here are the perl language bindings:
https://metacpan.org/pod/Valence
Cheers,
Doug
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-96890009.

I'll probably rewrite https://github.com/spolu/node-shell on top of valence
and call it bivalence :)
The overarching goal being the support of npm install xxx and it just
works.
On Wed, Apr 29, 2015 at 8:25 AM, Stanislas Polu<EMAIL_ADDRESS>wrote:

This is awesome. I'll hack on it right away!
On Monday, April 27, 2015, Doug Hoyte<EMAIL_ADDRESS>wrote:

Hi!
Just wanted to let you guys know I have uploaded a library that lets you
control electron (formerly known as atom-shell) with another process via
pipes and JSON messages, similar to how Thrust works. In fact, Thrust was
my main inspiration for this project:
https://github.com/hoytech/valence
I agree that Thrust's approach of not dictating controller
language/run-time is the right way to go, however my concern with
standardising on the Thrust API is that its feature-set is very limited
compared to electron's. Valence is essentially an RPC bridge so it should
support everything that electron does.
Here are the perl language bindings:
https://metacpan.org/pod/Valence
Cheers,
Doug
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-96890009.

The valance equivalent being this currently:
https://github.com/spolu/node-shell/blob/master/wrapper/index.js
On Sat, May 2, 2015 at 10:36 AM, Stanislas Polu<EMAIL_ADDRESS>wrote:

I'll probably rewrite https://github.com/spolu/node-shell on top of
valence and call it bivalence :)
The overarching goal being the support of npm install xxx and it just
works.
On Wed, Apr 29, 2015 at 8:25 AM, Stanislas Polu<EMAIL_ADDRESS>wrote:

This is awesome. I'll hack on it right away!
On Monday, April 27, 2015, Doug Hoyte<EMAIL_ADDRESS>wrote:

Hi!
Just wanted to let you guys know I have uploaded a library that lets you
control electron (formerly known as atom-shell) with another process via
pipes and JSON messages, similar to how Thrust works. In fact, Thrust was
my main inspiration for this project:
https://github.com/hoytech/valence
I agree that Thrust's approach of not dictating controller
language/run-time is the right way to go, however my concern with
standardising on the Thrust API is that its feature-set is very limited
compared to electron's. Valence is essentially an RPC bridge so it should
support everything that electron does.
Here are the perl language bindings:
https://metacpan.org/pod/Valence
Cheers,
Doug
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-96890009.

Hehe bivalence -- that is a great name. :)
Thanks for the pointer to node-shell, it actually looks like a pretty similar idea to valence.
Agreed: Ideally users should just be able to npm/cpan/whatever and have everything working.

Hey all, roughly, what is the status on this?  Is there a set of TODOs laying around so other can help out?

Cancelled. Suffered an injury can't code much open source anymore
On Dec 2, 2015 2:27 PM, "Ryan Rohrer"<EMAIL_ADDRESS>wrote:

Hey all, roughly, what is the status on this? Is there a set of TODOs
laying around so other can help out?
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-161406417.

Sorry to hear that.

Hey all.  I was inspired by @hoytech's valence, but wanted something with a slightly less low level control (and that worked on windows since Electron doesn't allow stdin/stdout).
As a solution, I came up with electroncontrol and wrote bindings for go go-electroncontrol  It's significantly less flushed out than Thrust, but combined with go-bindata (to pack in dependencies and ship a single exe that auto expands) it's enough to ship my simple app (and probably others like it).
It's convenient to be able to run purely on a downloaded exe of electron.
I figured I'd share here because moving Thrust to electron was a great idea, and hopefully other people think this solution is 'good enough' as well.
/shameless self promoting

Not shameless, we all want to build great technologies. I accept this
promotion. Feel free to submit a pull request to the read me and desbribe
it as an alternative
On Dec 11, 2015 3:03 AM, "Ryan Rohrer"<EMAIL_ADDRESS>wrote:

Hey all. I was inspired by @hoytech https://github.com/hoytech's
valence, but wanted something with a slightly less low level control (and
that worked on windows since Electron doesn't allow stdin/stdout).
As a solution, I came up with electroncontrol
https://github.com/rrohrer/electroncontrol and wrote bindings for go
go-electroncontrol https://github.com/rrohrer/go-electroncontrol It's
significantly less flushed out than Thrust, but combined with go-bindata
https://github.com/jteeuwen/go-bindata/ (to pack in dependencies and
ship a single exe that auto expands) it's enough to ship my simple app (and
probably others like it).
It's convenient to be able to run purely on a downloaded exe of electron.
I figured I'd share here because moving Thrust to electron was a great
idea, and hopefully other people think this solution is 'good enough' as
well.
/shameless self promoting
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-163869884.

I guess all good things come to and end :cry:
This was a great Idea that I really wanted to see rise, shame it has to end because of an injury. Get well soon!

Get well soon buddy, I read about this project a while back, but did not know that you stopped working on this. Hope you are doing well!

I am a little better, starting to maintain it again
On Wed, Feb 24, 2016 at 9:22 AM, hamza zia<EMAIL_ADDRESS>wrote:

Get well soon buddy, I read about this project a while back, but did not
know that you stopped working on this. Hope you are doing well!
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-188274328.

starting to maintain it again

:D

that doesnt mean new features in thrust, just maitenance of go-thrust
On Wed, Feb 24, 2016 at 9:55 AM, Wesley Overdijk<EMAIL_ADDRESS>wrote:

:D
—
Reply to this email directly or view it on GitHub
https://github.com/breach/thrust/issues/272#issuecomment-188289364.


Dart2js crash on local function type variables.

I received the error
Unexpected compile error.

--- Command "dart2js" (took 01.000144s):
DART_CONFIGURATION=ReleaseX64 out/ReleaseX64/dart-sdk/bin/dart compile js -Dtest_runner.configuration=dart2js-linux-chrome --sound-null-safety --test-mode --packages=.dart_tool/package_config.json tests/corelib/iterable_extension_test.dart --sound-null-safety --out=out/ReleaseX64/generated_compilations/dart2js-linux-chrome/tests_corelib_iterable_extension_test/iterable_extension_test.js

exit code:
253

stdout:
Info: Compiling with sound null safety.
The compiler crashed: Assertion failure: Local function type variables are not supported.
#0      failedAt (package:compiler/src/diagnostics/invariant.dart:39:3)
#1      JsToFrontendMapImpl.toBackendTypeVariable (package:compiler/src/js_model/js_world_builder.dart:586:9)
...

when trying to run a new test.
It's only test code, not the code being tested, so I'll pre-approve the failure for now.
It's coming from js_world_builder.dart:
  TypeVariableEntity? toBackendTypeVariable(TypeVariableEntity typeVariable) {
    if (typeVariable is KLocalTypeVariable) {
      if (_closureData == null) {
        failedAt(
            typeVariable, "Local function type variables are not supported.");
      }

but if it's reachable from user code, then it's a bug.

Thanks @lrhn - seems like dart2js has never added support for type variables on local functions. While I agree that should be fixed, it would be a shame to not have test coverage in dart2js for your new changes because of this (especially since your feature doesn't relate to this issue). In the meantime, would you consider a small refactor on the test to avoid this feature? I'll comment on your CL as well.

Not sure that dart2js doesn't support generic local functions. Copying the failing test into DartPad, as
https://dartpad.dev/6cd6ac29638516e31170d33855a7e658?channel=master, seems to work, as does the tests/language/generic/local_functions_test.dart test, as far as I can see. (Did you fix it since yesterday?)
But I'll rewrite the test and see if it helps.

Sorry, you are right, I just jumped to conclusions too quickly by looking at the error message above.

mmm... I was not able to repro the crash after I patched your CL locally. Do you know if it is still failing on your CL?

Ah, never mind - I saw most closures were still around and I didn't notice that you did hoist testRec. Putting that one back in the method reintroduces the crash.

It appears the issue is directly related to records. Here is a minimal repro:
void main() {
  void f<T>(T value, (int, T) record) {
    (0, value) == record;
  }

  f("a", (0, "a"));
}

Which crashes with the following error:
The compiler crashed: Assertion failure: Local function type variables are not supported.
#0      failedAt (package:compiler/src/diagnostics/invariant.dart:39:3)
#1      JsToFrontendMapImpl.toBackendTypeVariable (package:compiler/src/js_model/js_world_builder.dart:586:9)
#2      JsToFrontendMapImpl.toBackendEntity (package:compiler/src/js_model/js_world_builder.dart:556:14)
#3      _TypeConverter.visitTypeVariableType (package:compiler/src/js_model/js_world_builder.dart:679:36)
#4      TypeVariableType.accept (package:compiler/src/elements/types.dart:499:15)
#5      _TypeConverter.visit (package:compiler/src/js_model/js_world_builder.dart:632:17)
#6      _TypeConverter.visitList (package:compiler/src/js_model/js_world_builder.dart:638:16)
#7      _TypeConverter.visitRecordType (package:compiler/src/js_model/js_world_builder.dart:672:46)
#8      RecordType.accept (package:compiler/src/elements/types.dart:440:15)
#9      _TypeConverter.visit (package:compiler/src/js_model/js_world_builder.dart:632:17)
#10     JsToFrontendMapImpl.toBackendType (package:compiler/src/js_model/js_world_builder.dart:550:16)
#11     MappedIterator.moveNext (dart:_internal/iterable.dart:393:20)
#12     JsToFrontendMap.toBackendTypeSet (package:compiler/src/js_model/js_to_frontend_map.dart:35:36)
#13     JClosedWorldBuilder.convertClosedWorld (package:compiler/src/js_model/js_world_builder.dart:150:13)
#14     JsBackendStrategy.createJClosedWorld (package:compiler/src/js_model/js_strategy.dart:188:52)
#15     Compiler.closeResolution (package:compiler/src/compiler.dart:815:25)
#16     Compiler.computeClosedWorld (package:compiler/src/compiler.dart:387:9)
#17     Compiler.produceClosedWorld (package:compiler/src/compiler.dart:629:11)
#18     Compiler.runSequentialPhases (package:compiler/src/compiler.dart:756:15)
<asynchronous suspension>
#19     Compiler.runInternal.<anonymous closure> (package:compiler/src/compiler.dart:319:7)
<asynchronous suspension>
#20     Compiler.runInternal (package:compiler/src/compiler.dart:318:5)
<asynchronous suspension>
#21     Compiler.run.<anonymous closure> (package:compiler/src/compiler.dart:239:11)
<asynchronous suspension>
#22     main (package:compiler/src/dart2js.dart:1474:3)
<asynchronous suspension>

cc @rakudrama

It appears related to the creation of the inner record. Here is an even simpler repro:
void main() {
  Object? f<T>(T value) => (0, value);
  f("a");
}

I marked this as P1 for our team, since it's a crasher, but I don't consider this a blocker for the milestone. Feel free to move it out of the milestone as we approach the deadline.


Added back some cheap logo

It's anoying to see some image not found image

why... not... use... DBH's logo...?

why... not... use... DBH's logo...?

why use DBH logo?

It isn't a svg And it never was the DBH Logo I think...

Image converters exist

https://beta.danbot.host/ legit could just use the same image this site
uses dumbass
On Tue, Feb 15, 2022 at 7:38 AM vinc3nz @.***> wrote:

It isn't a svg
And it never was the DBH Logo I think...
—
Reply to this email directly, view it on GitHub
https://github.com/DanBot-Hosting/DanBotHostingStats/pull/240#issuecomment-1040284562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK52QVLECJF7BCT6FZAJTYTU3JJLXANCNFSM5ONQACMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

I agree

I have tried every day since you wrote, but I can't reach the website to get the logo. If you want, you can send me the icon in discord (I'm called Vincenz, I'm the only one with this name in the discord)

Closing PR as website got removed


Reintegrate ScalaPB into our build

Once https://github.com/scalapb/ScalaPB/pull/458 is merged and released as part of ScalaPB, we should re-enabling automatic ScalaPB codegen. I don't mind keeping generated sources under source control.

Great news!! https://github.com/scalapb/ScalaPB/pull/458 has been merged and "will come with ScalaPB 0.8 which will be shipping soon".


Added startup message

as stated

Nice job! Please remember to implement the test cases for the method that you have added.


Add requirements.txt to dockerfile

What does this PR do?
Adds requirements.txt to dockerfile, '--break-system-packages' is needed because we are using debians image to download gnu radio but need to install the requirements.txt file
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you successfully ran tests with your changes locally? Including container validation.
[x] Have you lint your code locally prior to submission?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you labelled this PR correctly? E.G. major, milestone, breaking, breaking-change, minor, feature, enhancement, patch, fix, chore, refactor etc

ok with me, although now we should remove python3-fastapi, python3-opencv, uvicorn and have them in the requirements file instead


2 Blokes and an Armchair homebrew causes a crash with no error messages

'2 Blokes & An Armchair - N64 Remix Remix by Tesko (PD).z64' - DBCAC7AC8105089597C94153F8B0A10E6B25B61E
This homebrew causes the emulator to crash to the desktop with no error messages or notice at all.
This ROM works fine on Ares, Project64, Mupen64plus, etc.
Testing was done on the latest version of Gadolinium on Windows 10.

Thanks! Have you tried running it from the terminal and see if you get any messages? The best case scenario is a fatal log from my emulator from something unimplemented. If there are no messages in the terminal either, then that's even worse 😅 It means it's likely to straight up be a segfault...

Thanks! Have you tried running it from the terminal and see if you get any messages? The best case scenario is a fatal log from my emulator from something unimplemented. If there are no messages in the terminal either, then that's even worse 😅 It means it's likely to straight up be a segfault...

Unfortunately when I did it just did the same thing. 😖

Ok cool. Tomorrow I'll repro and keep you updated. Hopefully also can fix the bug 🤞 Thanks

Latest dev works


can training codellama?

Thank you very much for your code.
I performed train.py with the codellama 34b base.
This training went well and I confirmed that a checkpoint output of 76G, which is the same as codellama 34b, was generated.
Afterwards, when trying to load the generated model through LlamaForCausalLM, the following error occurred.
ValueError: Trying to set a tensor of shape torch.size([0]) in "weight" (which has shape torch.Size([32000, 8192])), this look incorrect.

Is there anything I missed or need to fix?

You need to remove the model.safetensors like this:
https://github.com/jzhang38/EasyContext/blob/3c68bd5602e3f37582f9bbe73ab083273bd4a1c7/train_scripts/EasyContext-1M-Llama-2-7B.sh#L22
I don't why this would happen.

You need to remove the model.safetensors like this:
https://github.com/jzhang38/EasyContext/blob/3c68bd5602e3f37582f9bbe73ab083273bd4a1c7/train_scripts/EasyContext-1M-Llama-2-7B.sh#L22
I don't why this would happen.

solved!!
thank you!!!  :)


[BUG] Progress bar not showing up when enchanting jewlery

Progress bar not showing up when enchanting jewlery
Tested on ruby ring, diamond bracelet, dragonstone bracelet
Enchant jewelry toggle is confirmed to be toggled on
progress bar confirmed to be working on other items such as crafting dhide and cleaning herbs
Steps to reproduce the behavior:

Cast respective enchant spell on diamond bracelet/dragonstone bracelet /ruby ring
Progress bar does not show up

Hi! I just tried it and it works on my end

Do you have your client.log file? Maybe there will be something in there

pics of plugin working and then testing on ruby ring

there is a warn in the client.log but unsure if is caused by action-progress plugin

2023-08-20 16:08:05 SGT [Client] WARN  n.runelite.client.eventbus.EventBus - Uncaught exception in event subscriber
java.lang.NullPointerException: null
at java.base/java.util.stream.ReferencePipeline$4$1.accept(Unknown Source)
at java.base/java.util.Spliterators$ArraySpliterator.tryAdvance(Unknown Source)
at java.base/java.util.stream.ReferencePipeline.forEachWithCancel(Unknown Source)
at java.base/java.util.stream.AbstractPipeline.copyIntoWithCancel(Unknown Source)
at java.base/java.util.stream.AbstractPipeline.copyInto(Unknown Source)
at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(Unknown Source)
at java.base/java.util.stream.MatchOps$MatchOp.evaluateSequential(Unknown Source)
at java.base/java.util.stream.MatchOps$MatchOp.evaluateSequential(Unknown Source)
at java.base/java.util.stream.AbstractPipeline.evaluate(Unknown Source)
at java.base/java.util.stream.IntPipeline.allMatch(Unknown Source)
at com.github.calebwhiting.runelite.plugins.actionprogress.Product.isMadeWith(Product.java:39)
at com.github.calebwhiting.runelite.plugins.actionprogress.detect.UseItemOnItemDetector.onMenuOptionClicked(UseItemOnItemDetector.java:67)
at net.runelite.client.eventbus.EventBus$Subscriber.invoke(EventBus.java:70)
at net.runelite.client.eventbus.EventBus.post(EventBus.java:223)
at net.runelite.client.callback.Hooks.post(Hooks.java:193)
at qq.lm(qq.java:48345)
at client.mm(client.java:12092)
at client.ov(client.java:12081)
at client.mb(client.java:3356)
at client.bs(client.java:1162)
at bm.ay(bm.java:401)
at bm.run(bm.java:380)
at java.base/java.lang.Thread.run(Unknown Source)

I do get that same exception when it works. What are you using to cast the spell? Directly fire and cosmic runes or some equipment to replace the fire runes?

I've isolated the issue to be coming from using cosmic runes inside a rune pouch.
Tried using empty hands and tome of fire + cosmic runes in inventory and the progress bar started showing up.
There is no other warning in the logs.

Ah, you are right that the rune pouch wasn't taken into account. Good catch. I'll make a fix and post an update here once it's available

Has been merged in RuneLite. Please re-open if you are still experiencing issues


Increase menu frame width during bounce

Hi, not sure if this repo is still being maintained/used and/or this is something you'd be interested in.. but this is a fix I made a while back.
The issue is this:

You open the menu
You select an item
The content view bounces to the right
The background, that is, where the content view used to be, doesn't match the menu.

The solution is to extend the menu frame during the bounce, which looks nice and certainly way better than no fix at all.

🎉
Merged in v1.5.8 !
Thank you for your contribution ;-]

Brilliant, thanks!


Plugin types must be used at most once

There is design decision in PkgTemplates that a "plugin" means "an instance of Plugin of a particular concrete type", and that a plugin can be used at most once.
I only managed to figure this out from looking at the source code though — it wasn't obvious from the documentation.
In my particular case, I was wanting to (mis)use the GitHubActions plugin to copy some other files into the .github/workflows subdirectory. So I'd written something like:
Template(;
    ...,
    plugins=[
        ...,
        GitHubActions(),  # To set up default CI
        GitHubActions(file=src_path, destination="my_workflow.yml"),  # To copy another workflow
    ]
)

and was surprised that it didn't work. And that the order in which I put them makes it not work in a different way. Ultimately it's due to this:
https://github.com/invenia/PkgTemplates.jl/blob/47aa85f0247dc5d2a695f97efcc245c37e414325/src/template.jl#L100

My first thought is a question — I assume the fact that each type of plugin can be used at most once is a feature? And expresses the fact that a plugin type shouldn't be a lower level operation like "copy this file from here to here", but rather represent the highest-level thing you could want to do.
If that's true, then I'd propose that we:

Document this design feature explicitly (both in the user guide, and in the developer guide)

Don't fail silently, but rather throw a helpful exception if the unique call above would remove any user-specified plugins.

i think we could change this behaviour https://github.com/invenia/PkgTemplates.jl/issues/266 https://github.com/invenia/PkgTemplates.jl/pull/269

@jw3126 in the end I just wrote my own template, inheriting from PkgTemplates.Plugin.
In this case it was for copying over some formatting stuff.. in case it helps, this is what I wrote (note that I haven't used this in anger in some time, so it might be out of date by now - apologies if so!) This is certainly in no way an official recommendation though, will let Nick comment on that...
# For plugin development, need access to internal functions.
using PkgTemplates: @plugin, @with_kw_noshow, Plugin
using PkgTemplates: combined_view, gen_file, render_file, tags

# FIXME This function is a bit of a hack
function default_file(path...)
    return expanduser(joinpath("~", "dotfiles", "julia_template", path...))
end

# TODO This would be nicer if templated on the CI plugin type, but then I can't use
# @plugin, and I'm lazy.
@plugin struct JuliaFormatterGitHubActions <: Plugin
    style::String="blue"
    julia_formatter_toml::String=default_file("JuliaFormatter.toml")
    julia_formatter_yml::String=default_file("julia_formatter.yml")
end

function PkgTemplates.view(p::JuliaFormatterGitHubActions, t::Template, pkg::AbstractString)
    return Dict("FORMAT_STYLE" => p.style)
end

function PkgTemplates.hook(p::JuliaFormatterGitHubActions, t::Template, pkg_dir::AbstractString)
    # Copy over the workflow file.
    outdir = joinpath(pkg_dir, ".github", "workflows")
    mkpath(outdir)
    cp(p.julia_formatter_yml, joinpath(outdir, "julia_formatter.yml"))

    # Generate files.
    pkg = basename(pkg_dir)
    formatter_toml = render_file(p.julia_formatter_toml, combined_view(p, t, pkg), tags(p))
    gen_file(joinpath(pkg_dir, ".JuliaFormatter.toml"), formatter_toml)
end

Then I just added JuliaFormatterGitHubActions() to the list of plugins.
I ended up putting this along with a couple of placeholder files in the same directory as this template

Ok thanks!

xref https://github.com/JuliaCI/PkgTemplates.jl/issues/266#issuecomment-1656757335


What to do when a dependency revision is removed?

I may be using godep wrong, but the README doesn't really explain how to work with this tool. I saved all the dependencies successfully a while ago using godep save. But as some of the repositories have been moved, I can't execute godep update or restore anymore.

ATM you need to manually prune out the olds locations and then godep save the new deps. You may be able to get away with doing a godep restore, blowing away $project/Godeps, updating the deps in your code and doing a new godep save. Or at least that's what I would try first.

It sounds very unreasonable. If you had a working tree once and you saved it, then at least on the technical level you have everything you need to deploy your project and update specific packages. Why the hassle? Also, godep doesn't tell you what dependencies are missing, I just receive random errors.


Stargazer count displaying 0

Problem
The star count for @11ty/eleventy is displaying 0 inside the<div class="elv-callout"> container within the footer, when it should be using "stargazers_count": 12238 or whatever value is in the cache (... ? "cache/" : ".cache/eleventy-fetch/").
I looked at src/_data/github.js and the logic within the try for fetching https://api.github.com/repos/11ty/eleventy with EleventyFetch doesn't seem to have any obvious issues causing an exception to be thrown so I'm curious as to why the value from cache isn't being used and why the original fetch might be failing.
Screenshot:

Side Note
Sort of unrelated, but I tried running the site locally after getting my fork up-to-date with main to see if I could pinpoint the reason for the stargazer count displaying 0 from github.js, but after doing a npm install and running npx @11ty/eleventy --serve, the build doesn't complete and the following stack trace is shown:
tannerdolby:11ty-website TannerDolby$ npx @11ty/eleventy --serve
[11ty] Problem writing Eleventy templates: (more in DEBUG output)
[11ty] Cannot read property '0' of undefined (via TypeError)
[11ty]
[11ty] Original error stack trace: TypeError: Cannot read property '0' of undefined
[11ty]     at /Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/Plugins/Pagination.js:316:61
[11ty]     at Array.map (<anonymous>)
[11ty]     at Pagination.getPageTemplates (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/Plugins/Pagination.js:316:41)
[11ty]     at Template.getTemplates (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/Template.js:668:45)
[11ty]     at TemplateMap.initDependencyMap (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/TemplateMap.js:291:41)
[11ty]     at processTicksAndRejections (node:internal/process/task_queues:96:5)
[11ty]     at async TemplateMap.cache (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/TemplateMap.js:336:5)
[11ty]     at async TemplateWriter._createTemplateMap (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/TemplateWriter.js:244:5)
[11ty]     at async TemplateWriter.generateTemplates (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/TemplateWriter.js:277:5)
[11ty]     at async TemplateWriter.write (/Users/TannerDolby/11ty-website/node_modules/@11ty/eleventy/src/TemplateWriter.js:324:23)

Every once in awhile if I do too many builds in rapid succession I hit the GitHub API limit for the unauthenticated tier 😅
We could make the code better to persist the .cache folder between builds to avoid over-hitting the API (probably the easiest) or change the code to use an authenticated request to GitHub (not too difficult either)
But it just doesn’t happen all that often 😅

Ah I see! Yeah, now that you mention it the limit for unauthenticated requests is quite low. The docs mention a rate limit of 60 requests per hour for unauthenticated requests which definitely could be reached quickly 😅

We could make the code better to persist the .cache folder between builds to avoid over-hitting the API (probably the easiest) or change the code to use an authenticated request to GitHub (not too difficult either)

Ya those are both good options, the first seems like the easiest and most fluid route. Adding OAuth or a personal access token for authenticated requests (which would bump rate limit to 5000 per hour) would also work but seems like unnecessary work dealing with auth tokens for a simple GET request, and for a situation that rarely happens like you said.


Fix logical definition of MONDO:0015627 ('multiple epiphyseal dysplasia due to collagen 9 anomaly')

Uses wrong relationship, see @gouttegd comment in #6609
Note, you cant use this relationship with a cellular component.
Also note: this will only show up when #6609 is merged, so best deal with it now.
Note 3, the relationship is the only mistake here.

To be consistent with how other diseases with relation to protein complex are modeles, I will update the relation to use:
'disease has basis in dysfunction of'


[BUG] login InstagrapiSession

Some bug when handling login response with InstagrapiSession.
[ERROR]  list index out of range
Traceback (most recent call last):
File "D:\user\prog\threads_bot\v2\main.py", line 22, in
asyncio.run(main())
File "C:\Users\Bou\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 190, in run
return runner.run(main)
^^^^^^^^^^^^^^^^
File "C:\Users\Bou\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 118, in run
return self._loop.run_until_complete(task)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\Bou\AppData\Local\Programs\Python\Python311\Lib\asyncio\base_events.py", line 653, in run_until_complete
return future.result()
^^^^^^^^^^^^^^^
File "D:\user\prog\threads_bot\v2\main.py", line 18, in main
await bot.start()
File "D:\user\prog\threads_bot\v2\threads_manager\timeline_manager.py", line 34, in start
await self.login(self.username, self.password, cached_token_path="data\.session")
File "D:\user\prog\threads_bot\v2\venv\Lib\site-packages\threads_api\src\threads_api.py", line 257, in login
token = self.instagrapi_client.private.headers['Authorization'].split("Bearer IGT:2:")[1]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range
Process finished with exit code 1

I am fixing this at the moment. Thanks for reporting.

This is fixed in https://github.com/Danie1/threads-api/releases/tag/v1.1.12
Thanks for reporting @MsLolita
Please open a new issue if you encounter any additional problems.


nipap-passwd --test-user fails for different default_backend

In nipap.conf if you set the default_backend to something other than local, then nipap-passwd fails when testing users with --test-user.  Same result when using two different sqlite auth sections.  Using the -f flag appears to have no affect.
[klandreth@localhost ~]$ nipap-passwd --test-user test -p test -f /etc/nipap/local_auth.db
The password seems to be wrong
[klandreth@localhost ~]$ ls -l /etc/nipap/local_auth.db
-rw-r--r-- 1 root root 3072 May  3 19:25 /etc/nipap/local_auth.db

This appears to only be the CLI tool nipap-passwd as the RPC and web-ui work fine despite testing failing.

Indeed, nipap-passwd doesn't do the right thing. https://github.com/SpriteLink/NIPAP/blob/master/nipap/nipap-passwd#L48 for example sets the db-file arguments (-f) to overwrite the config value for auth.backend.local.
There is a dependency on having a configuration file which we should probably try to decrease. I guess nipap-passwd would die completely if it can't find a config file at all, as per https://github.com/SpriteLink/NIPAP/blob/master/nipap/nipap-passwd#L45
Let's not try to read the config file at all if the auth_db file is specified with -f and when reading a configuration file we should probably iterate over the auth backends and try to find the local auth one.. if there is more than one we should throw an error. Instead the user would have to explicitly specify which auth_db file to work with.

Hi,
I also stumbled upon this but I have ldap as my default_auth, but have added a nipap-api user to the local_db.
When I run nipap-passwd test-user -u nipap-api@local -p kaka it says this:
The user nipap-api@local does not exist
When I run without "@" nipap-passwd test-user -u nipap-api -p kaka it throws an exception:
Traceback (most recent call last):
File "/usr/sbin/nipap-passwd", line 84, in
if not auth.authenticate():
File "/usr/lib/python2.7/dist-packages/nipap/authlib.py", line 372, in authenticate
raise AuthError('Could not connect to LDAP server')
So I went on a tour in code and found this on row 79-81 in nipap-passwd:
   if not a.get_user(args.user):
        print "The user %s does not exist" % args.user
        sys.exit(2)

If you delete this block of code you will let authlib.py "choose" auth_backend by using @auth_backend_name.
Look at the these two rows in authlib.py
164 user_authbackend = username.rsplit('@', 1)
184            auth = self._backends[backend](backend, user_authbackend[0], password, authoritative_source, auth_options)
So I commented-out row 79-81 in nipap-passwd and re-run "nipap-passwd test-user -u nipap-api@local -p kaka" and the output went happy:
Username and password seem to be correct
This is just a suggestion that could in the short-term fix this (maybe long-term also?).
You will also get the opportunity to test ldap users as well with the nipap-passwd tool.

@ping-the-one can you whip up a PR out of that? :)

@plajjan sure will do.

I think the reason for the check being there in nipap-passwd is to differentiate between an incorrect username (i.e. user doesn't exist) and an incorrect password. Perhaps authlib can be made to return that instead.
In normal auth we don't want to distinguish between missing user and incorrect password, because that makes it possible for an attacker to first brute-force the username and once found brute-force the password.

PR delivered #1119

PR merged. Closing.


my first pull-request script audiTAR.sh for Bash

What does the program do ?
unpacks .tar, prints the files of .tar and creates a log with date and name of the files unpacked
In what programming language it is written?
bash

your welcome!!!


Update semanticdb-scalac to 4.7.3

Updates org.scalameta:semanticdb-scalac from 4.7.2 to 4.7.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "semanticdb-scalac" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "semanticdb-scalac" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:semver-spec, commit-count:1

Codecov Report
Base: 87.19% // Head: 87.19% // No change to project coverage :thumbsup:

Coverage data is based on head (447696b) compared to base (bc20c14).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #724   +/-   ##
=======================================
  Coverage   87.19%   87.19%
=======================================
  Files         141      141
  Lines        1523     1523
  Branches      221      221
=======================================
  Hits         1328     1328
  Misses        195      195

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

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


Enhance early termination for combine operators

For single block combine operator, skip all remaining operators when query is satisfied
For streaming combine operator, allow early termination when enough results are streamed back
Handle EarlyTerminationException in the InstanceResponseOperator so that we won't miss the interrupt caught in CombineOperator.nextBlock()

Codecov Report

Merging #10169 (b898468) into master (f53bdf8) will decrease coverage by 56.81%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10169       +/-   ##
=============================================
- Coverage     70.42%   13.61%   -56.81%
+ Complexity     5788      176     -5612
=============================================
  Files          2006     1962       -44
  Lines        108756   106468     -2288
  Branches      16503    16241      -262
=============================================
- Hits          76593    14500    -62093
- Misses        26813    90827    +64014
+ Partials       5350     1141     -4209

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
13.61% <0.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

.../pinot/core/operator/InstanceResponseOperator.java
0.00% <0.00%> (-87.18%)
:arrow_down:

...e/operator/combine/AggregationCombineOperator.java
0.00% <0.00%> (-83.34%)
:arrow_down:

...not/core/operator/combine/BaseCombineOperator.java
0.00% <0.00%> (-85.72%)
:arrow_down:

...erator/combine/BaseSingleBlockCombineOperator.java
0.00% <0.00%> (ø)

...core/operator/combine/DistinctCombineOperator.java
0.00% <0.00%> (-90.00%)
:arrow_down:

.../core/operator/combine/GroupByCombineOperator.java
0.00% <0.00%> (-84.38%)
:arrow_down:

...nMaxValueBasedSelectionOrderByCombineOperator.java
0.00% <0.00%> (-75.18%)
:arrow_down:

...operator/combine/SelectionOnlyCombineOperator.java
0.00% <0.00%> (-62.97%)
:arrow_down:

...rator/combine/SelectionOrderByCombineOperator.java
0.00% <0.00%> (-54.55%)
:arrow_down:

.../combine/merger/AggregationResultsBlockMerger.java
0.00% <0.00%> (ø)

... and 1603 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@siddharthteotia @walterddr Added message and cause to the EarlyTerminationException and returned it in the query response


canceler: Fix goroutine leak on failure

jenkins: test this please

@tomponline @brauner should be good to go


Added tests for rfxtrx sensor

Description:
Tests for the rfxtrx sensor
Checklist:

[x] Local tests with tox run successfully.
[x] TravisCI does not fail. Your PR cannot be merged unless CI is green!
[x] [Fork is up to date][fork] and was rebased on the dev branch before creating the PR.
[x] Commits have been [squashed][squash].

Woohoo :dolphin: :dollar:


Support non ASCII attribute

String#undump is a function that does the reverse of String#dump, so we expect String#dump output.
Multibyte characters in terraform_pran results are not escaped. They differ from the results of String#dump (String#dump converts non ASCII characters to Unicode escape sequences).
example:
puts 'hello, 世界'.dump              # => "hello, \u4E16\u754C"
puts '"hello, \u4E16\u754C"'.undump # => hello, 世界
puts '"hello, 世界"'
# Traceback (most recent call last):
# 	1: from sample.rb:3:in `<main>'
# sample.rb:3:in `undump': non-ASCII character detected (RuntimeError)

Thanks for the fix, @dozen!


WIP: Added a walkthrough for additional configurations.

Added a walkthrough for additional configurations, that also follows after the initial PearAI setup walkthrough
Pretty much every QoL feature can be added here later on.
Currently, it includes:

Adding Pear to the PATH, which allows it to be opened via the terminal with the command: pear .
Opening files from the desktop in Pear via the context menu.
Deleting VSCode to replace it with Pear. This should remove context menu shortcuts for VSCode, etc.

Talked about this a while back in discord.

Do not merge yet;  the command implementation is still missing. These are a bit challenging to implement since each OS seems to have an individual way of handling certain things, such as the context menu shortcuts.
Once the command implementation is finished, this will close Issue #132 and #269

@timknappe how is this?

Still workig on it, had some environment issues lately, pretty stuffed rn. Im trying to finish it asap

okay sg! thanks


Dataloader yields wrong sequence when resuming training

System Info
- `Accelerate` version: 0.29.3
- Platform: Linux-5.15.0-101-generic-x86_64-with-glibc2.35
- `accelerate` bash location: /(...)/.venv/bin/accelerate
- Python version: 3.10.11
- Numpy version: 1.26.4
- PyTorch version (GPU?): 2.2.2+cu121 (True)
- PyTorch XPU available: False
- PyTorch NPU available: False
- PyTorch MLU available: False
- System RAM: 377.53 GB
- GPU type: Quadro RTX 8000
- `Accelerate` default config:
        Not found

Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
This is about training with a dataloader which shuffles at every epoch. For reproducibility, when resuming training, the dataloader's order should be identical to the one from the epoch where training was interrupted. However, setting train_dataloader.set_epoch(epoch) has zero effect (no change on the sequence yielded no matter the value of the epoch used).
The sequence the dataloader yields is actually the epoch n+1 if training was interrupted during epoch n.
Here is a minimal example of outputs for a DataLoader(list(range(10)), shuffle=True, batch_size=4)
Without resuming:
Epoch: 0
[6, 7, 1, 4]
[2, 0, 9, 8]
[3, 5]
Epoch: 1
[2, 4, 7, 0]
[8, 9, 5, 3]
[6, 1]
Epoch: 2
[7, 4, 5, 1]
[9, 3, 8, 2]
[0, 6]
With resuming after two steps:
Epoch: 0
[6, 7, 1, 4]
[2, 0, 9, 8]
*interuption
*resuming
[6, 1]
Epoch: 1
[7, 4, 5, 1]
[9, 3, 8, 2]
[0, 6]
Code to reproduce
import os
import re
import hydra
from omegaconf import DictConfig
import torch
from torch.utils.data import DataLoader
from accelerate import Accelerator
from accelerate.utils import ProjectConfiguration, set_seed
from collections import defaultdict

class Metrics(defaultdict):

    def __init__(self):
        super().__init__(int)

    def state_dict(self):
        return dict(self)

    def load_state_dict(self, state_dict):
        for k, v in state_dict.items():
            self[k] = v

    def log(self, accelerator: Accelerator):
        # Build the metrics to log
        metrics_log = dict()
        metrics_log["train/epochs"] = self["train/epochs"]
        metrics_log["train/steps"] = self["train/steps"]

        # Log the metrics
        accelerator.log(metrics_log)

def write_to_file(path, accelerator, obj):
    if accelerator.is_main_process:
        with open(path, "a") as f:
            f.write(obj)
            f.write("\n")

@hydra.main(version_base=None, config_path="../conf", config_name="config")
def train(cfg: DictConfig):
    # Get the last checkpoint id
    checkpoint_dir = os.path.join(cfg.trainer.dir, "checkpoints")
    iteration = 0
    if cfg.trainer.resume and os.path.exists(checkpoint_dir) and len(os.listdir(checkpoint_dir)) > 0:
        folders = os.listdir(checkpoint_dir)
        iteration = max(int(re.findall(r"[\/]?([0-9]+)(?=[^\/]*$)", folder)[0]) for folder in folders) + 1

    # Accelerator object
    project_config = ProjectConfiguration(
        cfg.trainer.dir,
        automatic_checkpoint_naming=True,
        total_limit=50,
        iteration=iteration,
    )
    accelerator = Accelerator(
        mixed_precision="no",
        gradient_accumulation_steps=1,
        project_config=project_config,
    )

    # File to log outputs
    path = "log.txt"

    # Set the seed
    set_seed(cfg.seed)

    # Local and global counters
    metrics = Metrics()
    accelerator.register_for_checkpointing(metrics)

    train_dataloader = DataLoader(list(range(10)), shuffle=True, batch_size=4)

    # Accelerate
    train_dataloader = accelerator.prepare(train_dataloader)

    # Resume from the latest checkpoint
    skipped_train_dataloader = None
    if cfg.trainer.resume and os.path.exists(checkpoint_dir) and len(os.listdir(checkpoint_dir)) > 0:
        accelerator.load_state()
        if accelerator.is_main_process:
            write_to_file(path, accelerator, "\nResuming in epoch: " + str(metrics["train/epochs"]))
        train_dataloader.set_epoch(metrics["train/epochs"])
        skipped_train_dataloader = accelerator.skip_first_batches(train_dataloader, metrics["train/batches"] % len(train_dataloader))

    while cfg.trainer.max_steps > metrics["train/steps"]:
        # Use skipped_train_dataloader the first epoch after resuming
        dataloader = train_dataloader if skipped_train_dataloader is None else skipped_train_dataloader

        write_to_file(path, accelerator, "\nEpoch: " + str(metrics["train/epochs"]))
        for batch in dataloader:
            # Update number of batches
            metrics["train/batches"] += 1

            write_to_file(path, accelerator, "\nSteps: " + str(metrics["train/steps"]))
            write_to_file(path, accelerator, str(torch.flatten(batch).tolist()))

            metrics["train/steps"] += 1
            accelerator.save_state()

            if metrics["train/steps"] >= cfg.trainer.max_steps:
                break

        # Log metrics
        metrics["train/epochs"] += 1

        # "Remove" the skipped dataloader once exhausted
        skipped_train_dataloader = None

    # Make sure that the wandb tracker finishes correctly and close the progress bar
    accelerator.end_training()

if __name__ == "__main__":
    train()

Expected behavior
The dataloader should yield identical sequences with or without resuming.

Can you try updating your accelerate version to see if we fixed it in the prior releases?

Hi! Thank you for your answer. It gives the same results with accelerate 0.30.1

Hello! I am gently uping this issue to know if you have had a chance to look into it?

Also seeing a symptom of this issue [or at least it seems very related]! I'm seeing a weird occurrence where resuming training causes the master rank to only perform grad accum for e.g., 2 passes when it should be 8 [as it is correctly on the other ranks]. Subsequent steps are performed correctly, just the first one is flawed [which eventually causes a deadlock once the dataloader is exhausted].

@muellerzr Could you check on this issue again?


Backend: Save the recorded histories on a server

Problem
As a single user, the number of files I'm generating are huge.
I had imagined that I could send the history files by email and add them to a database. However it's probably going to be more files than what I can manually process.
Solution
Backend.
Questions:

What language? rust or python.
What database? sqlite.
User management?

Yay, there is now braingridgame.com, that is ARC-Interactive with a backend for collecting the recorded interaction histories.
The backend is Django. postgresql. Allauth for Google login. Django's admin interface is good for user management.


IOUtils chain together IOException for Closeables, add overload

This pull request chains together IOExceptions thrown from closing multiple Closeable instances, which allows the method to attempt to close all of the Closeable instances rather than halting at the first IOException. An overload is added to allow closing multiple Closeable instances and passing the possible resulting IOException to an IOConsumer.

-1: This makes no sense to me as the exception are not causaly unrelated and you are forcing them to be. This might be a use case for IOExceptionList.

Hi @garydgregory , good suggestion, IOExceptionList seems like the perfect fit for this. I have revised the code to use this class.

I ended up reworking the underlying so that now the method looks like this
    public static void close(final Closeable... closeables) throws IOException {
        IOConsumer.forEach(closeables, IOUtils::close);
    }

See also forEachIndex which provides slightly different information for compatibility with other call sites.

Hi @wodencafe Thank you for your update. Now I see one new method, without any tests, and not used from anywhere.

Should the [] be changed to a ... and moved to the last parameter position? I would think so.
Can the method be used within the code base?
There are no tests.

Hello @garydgregory , my apologies, I got swamped this weekend and haven't updated this PR with tests. I also agree that the [] should be changed to ... varargs and swapped to be the last parameter, so I will implement this change, and see if there are other places in Commons IO that can take advantage of this new functionality. I will add tests for this today.
Thank you for reviewing the changes, and the contributions and feedback.

Coverage decreased (-0.5%) to 87.329% when pulling 1ccb075f79ec06e763e6b62417d0602c98524749 on wodencafe:IOUtils-closewitherrors into 5300267aa9d13ec0f800b221e3086e835be697be on apache:master.

Hi @garydgregory , I have updated the pull request with the changes you suggested.
Thank you!


Sometimes matching identifier highlight does not work

Version: 1.48.2
Commit: a0479759d6e9ea56afa657e454193f72aef85bd0
Date: 2020-08-25T19:14:08.130Z
Electron: 7.1.14
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.4.64-1-MANJARO

Steps to Reproduce:

Move the cursor with arrow keys (no modifiers) onto a word then stop
Notice after a moment it will highlight other matches that are visible on screen (or in the whole file, with the scrollbar markers)
In particular, notice that if you stop on the edge of a word, it will still highlight it.
Now, move around with ctrl+left or ctrl+right. You will only be able to stop on word edges in this case.
Notice that in this case, no highlight happens, unlike when you land on a word edge by moving only with arrows.
Additionally, if you move the cursor more than one character in other ways (f.e. you use VS Code APIs to programmatically move the cursor), no highlight happens.

As an effect of this, plugins like Neo Vim (which move the cursor in custom ways) will not show any highlight when moving more than one character (or line) at a time.

Does this issue occur when all extensions are disabled?: Yes

In the following example recording, first I move character by character with the left and right arrows keys, and when I stop moving, you see that highlight works. In the last half of the recording I move with ctrl + arrow keys, in which case no highlight happens when I stop moving:


Output aparcaseg in requested output spaces

Right now aparcaseg is being produced only for bold_space-T1w even though MNI was specified as an output space. Context: https://neurostars.org/t/segmentation-compartment-files-in-functional-space-resolution-for-after-fmriprep/2486

Ah, I see.

Hi all,
I want to create a patch for this issue but want to know if to treat it as bug or enhancement.
Short summary of the problem:
Currently if freesurfer was used, the FS label files (parcellation and segmentation) are created in the anatomical and functional folders.
In anat - label files are in T1w space (T1w grid).
In func - label files are created for each functional run in T1w space (functional grid) regardless of the requested output space (T1w, MNI or both).
Presumably all the func folder label files are just nearest neighbor resampling of the T1w space anat label files. Therefore, my suggestion is to treat this as a bug and the solution will be to always sample to the requested output spaces in the functional folder. This will give a meaningful label files that correspond to each of the output preprocessed run file that is ready to use (with no additional transformation required).
Does that sound like a reasonable path forward?


numpy 1.24 no attribute int

model-angelo version: 0.2.3
When executing the help command I get the following error. This only happens with numpy 1.24. When downgrading to numpy 1.19 another error appears. I could circumvent this error by appending a restriction of numpy to the requirements file:
echo -e "\nnumpy<1.24" >> requirements.txt

While this resolved the issue for me, I think this is rather a quick dirty fix than a sustainable solution of the problem.
Error:
$ model_angelo build --help
Traceback (most recent call last):
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/bin/model_angelo", line 33, in <module>
    sys.exit(load_entry_point('model-angelo==0.2.3', 'console_scripts', 'model_angelo')())
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/model_angelo-0.2.3-py3.9.egg/model_angelo/__main__.py", line 23, in main
    import model_angelo.apps.build
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/model_angelo-0.2.3-py3.9.egg/model_angelo/apps/build.py", line 19, in <module>
    from model_angelo.c_alpha.inference import infer as c_alpha_infer
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/model_angelo-0.2.3-py3.9.egg/model_angelo/c_alpha/inference.py", line 14, in <module>
    from model_angelo.data.dataset_preprocess import decompress_data
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/model_angelo-0.2.3-py3.9.egg/model_angelo/data/dataset_preprocess.py", line 14, in <module>
    from model_angelo.utils.fasta_utils import read_fasta
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/model_angelo-0.2.3-py3.9.egg/model_angelo/utils/fasta_utils.py", line 10, in <module>
    from model_angelo.utils.residue_constants import index_to_restype_1
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/model_angelo-0.2.3-py3.9.egg/model_angelo/utils/residue_constants.py", line 1094, in <module>
    restype_atom37_to_rigid_group = np.zeros([21, 37], dtype=np.int)
  File "/home/apps/conda/miniconda3/envs/modelangelo-0.2.3/lib/python3.9/site-packages/numpy/__init__.py", line 305, in __getattr__
    raise AttributeError(__former_attrs__[attr])
AttributeError: module 'numpy' has no attribute 'int'.
`np.int` was a deprecated alias for the builtin `int`. To avoid this error in existing code, use `int` by itself. Doing this will not modify any behavior and is safe. When replacing `np.int`, you may wish to use e.g. `np.int64` or `np.int32` to specify the precision. If you wish to review your current use, check the release note link for additional information.
The aliases was originally deprecated in NumPy 1.20; for more details and guidance see the original release note at:
    https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

Hi,
Yes, you are correct. I fixed this now.
Best,
Kiarash.

Thanks for the quick fix. I tested it and it works now!


Genesis error with cometbft v1.0.0

tower-abci works well with cometbft v0.38.16, but with v1.0.0:
2024-12-25T18:25:44.415872Z  INFO tower_abci::v038::server: ABCI server starting on tcp socket addr=<IP_ADDRESS>:26658
2024-12-25T18:25:48.350124Z  INFO tower_abci::v038::server: listening for requests
2024-12-25T18:25:48.350790Z  INFO tower_abci::v038::server: listening for requests
2024-12-25T18:25:48.350970Z  INFO tower_abci::v038::server: listening for requests
2024-12-25T18:25:48.351123Z  INFO tower_abci::v038::server: listening for requests
thread 'tokio-runtime-worker' panicked at /Users/larry/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tower-abci-0.18.0/src/v038/server.rs:173:75:
called `Result::unwrap()` on an `Err` value: missing public key

Location:
    /Users/larry/.cargo/registry/src/index.crates.io-6f17d22bba15001f/flex-error-0.4.4/src/tracer_impl/eyre.rs:10:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This happens when starting a chain fresh from genesis. If I start a chain with 0.38, kill it, then restart with 1.0.0, the chain can carry on normally.

Interesting, by default, I would expect the v038 protocol to carry over v1 nicely, but this is actually encouraging. I don't see any published domain types in either tendermint-rs or cometbft-rs, but:

https://github.com/cometbft/cometbft/tree/main/proto#readme seems to hint at a wire protocol change
https://github.com/cometbft/cometbft-rs/pull/7 hints at some work being done

However, the published version of cometbft-rs on crates.io is empty: https://docs.rs/cometbft/latest/cometbft/
At least the protos are published on buf, so I can give this a deeper look very soon and figure it out. I think we can add support soon after domain types become available. We might have to write them, but that's fine.


After updating to RN 0.62 and Update the package, Android stopped working

Bug
The package works great for me in the past, but after i was updating my RN to RN 0.62.2 this package stopped working for android.
I get all the data as in the past but i think that the data in the uri didn't save.
And i didn't get any error
-->
This is my code-
handleAddingPhoto = () => {
  ImagePicker.showImagePicker(options, async (response) => {
    if (response.didCancel) {
      console.log("User cancelled image picker");
    } else if (response.error) {
      console.log("ImagePicker Error: ", response.error);
    } else {
      ImageResizer.createResizedImage(response.uri, 400, 400, "JPEG", 80)
        .then(async (data) => {
          const data = {uri: response.uri, name: data.name};
          this.handleUploadPhoto(data);
        })
        .catch((err) => {
          console.log(err.message);
        });
    }
  });
};

Environment info
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2

Library version:
"react-native-image-picker": latest
Steps To Reproduce

create new RN 0.62.2 Project
2.add react-native-image-picker
3.try to get to the image will fail in android and will success in IOS

Describe what you expected to happen:
1.get the uri and see the image

Sorry the problem was in react-native 0.62 .
This solved my problem -
https://github.com/facebook/react-native/issues/28551#issuecomment-611085378

why did you close this issue, if it's still an issue and you only followed a workaround?


do not format a code block in documentation if it is annotated with ignore or text

In those issues, the parsing of code blocks failed fatally, exiting with an error code of 101.
close #3055
close #3054

Awesome, thank you!


Conference desk app doesn't see all tickets

The conference desk ticket search app doesn't see all tickets. Mostly, late purchases are missing, but also some earlier are at least hard to find.

Don’t close this issue, I think we could improve it for the next year

On 27 Jul 2018, at 10:49, Anders Hammarquist<EMAIL_ADDRESS>wrote:
The conference desk ticket search app doesn't see all tickets. Mostly, late purchases are missing, but also some earlier are at least hard to find.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Openproject

Project description
Web based open source project management software. It is based on rails (like Redmine or Gitlab) but has some moving parts e.g. cache, database, etc
Metadata

homepage URL: https://www.openproject.org/
source URL: https://github.com/opf/openproject
license: GNU General Public License version 3
platforms: linux, darwin

Existing work
Package/service originally requested in #164019  , but a separate issue is needed due to the size of the application.
The Gemfile of the application use features not yet handled by bundix (see: nix-community/bundix#84)
The project seems to have some nix related files in their sources: https://github.com/opf/openproject/tree/dev/nix

@b12f is this one yours? https://git.pub.solar/b12f/openproject-nix
Are there any major packaing tasks open for it?

Yes that's mine. That attempt is entirely unfinished.
I'm too inexperienced in ruby/gemfile packaging to handle the internal linking that openproject does with its gems. OpenProject recently started using Turbo next to the Angular frontend, so building the frontend is probably also a bit more involved than it used to be. There's also a bunch of tooling in the setup that is only half-reproducible, like the precompilation of assets.
I'm a former employee of OP, and the team there is really nice around contributions. There are scraps of a nix devshell left in the repo, and I know there's at least one current employee still using nix. If you're interested in packaging this, I'd be interested in helping out or tagging along.

@b12f Even though I don't have much experience with Ruby, but I would love to challenge myself to port it to NixOS and I have no clues where I should start though? I would love to receive a little help from you, is there any way to contact you easily?

I've forked your flake and based some work on that, @b12f: https://github.com/bendlas/openproject-nix
The ruby build seems to work now, now working on the angular build ...

there's at least one current employee still using nix

@b12f would you mind making an introduction? happy to have them participate here or to contact me privately.
With the packaging + module now rough cut, I'll work on implementing options + sane defaults as needed and any input on best practices would be greatly appreciated!
I also have a few questions and ideas, e.g. around:

making state directory configurable, so that I can get rid of this abomination: https://github.com/bendlas/openproject-nix/blob/6a9aab86770b8d06fb93ddfc7f84cb074e1da2b7/ruby/openproject.nix#L128-L129
decompose entry points, particularly the while true loop in what's called "cron"

see https://github.com/opf/openproject/blob/6bd908009c8b3061ac72547f794c9894f8680af6/docker/prod/cron#L11
vs https://github.com/bendlas/openproject-nix/blob/6a9aab86770b8d06fb93ddfc7f84cb074e1da2b7/module.nix#L113-L124

Thanks for your work @bendlas ! What did you use to produce gemset.nix, inscapist/ruby-nix ?

Very interested in seeing any progress on this, hope you can figure out how to finish the packaging!

@maedas they've published it here https://github.com/bendlas/openproject-nix


[Bug] Remove dependency to sun.misc.*

Search before asking

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

Paimon version
all version
Compute Engine
all
Minimal reproduce step
build with a JDK that is not provided by Oracle
What doesn't meet your expectations?
I cannot build this with another JDK that the one provided by Oracle
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Can you specify where is the dependency?

using the search is how you can see it
https://github.com/search?q=repo%3Aapache%2Fincubator-paimon+sun.misc&type=code

Hi! Thanks for reporting this issue. Which JDK and maven version are you using? What command are you using to compile the project? What error message occurs? Please provide detailed information.

Last OpenJDK 19
maven build
Error is saying that sun.misc.* is not available
No idea on how to mitigate this so far, but I think this is a general issue and perhaps Apache has a generic answer ?

Not sure what IDE you are using @innovimax, but I was able to fix this issue by unchecking this setting in IntelliJ:

Hi @ericxiao251
It doesn't fix it (since it was already checked on my Intellij IDEA)
Furthermore, it's a more general question about depending on a non public API (sun.misc.Unsafe)

@JingsongLi I don't see how this is fixed ?


Swiper not working, all slide move to left

What you did
I have intergrated swiper lib as slider in my magento 2 extension. Used swiper v4.0.3
Expected Behavior
Slide not working, i can't see any transition or move slide. All slide image move to left
When i see in chrome devtools i found swiper-slide element apply inline style
like this
width: 920px;
 opacity: 0.652174;
 transform: translate3d(-320px, 0px, 0px);

Here is screenshot

My Js  file for initial swiper . See content file here. Magento 2 js init by requireJS
How to fix this problem?. I checked everywhere and i think this problem related to swiper lib js

Looks like you are still using v3 api. Also provide a live example please

Review the 'pagination' option.

Thanks guy. I found reason my problem. My css swiper wasn't loaded so it cause this problem

Yes, it supports AMD and RequireJS


ROM shortcuts can't be deleted if there is a special character in game name

I uploaded a game ROM and gave it a name with a special character (eg. "é" or "à").
I could not delete the game later on via steam buddy.
If I select "Delete" I get a web with an error as response:
"Error: 500 Internal Server Error
Sorry, the requested URL 'http://gameros-sbc.local/shortcuts/delete' caused an error:
Internal Server Error"

Could you check out what the shortcut file is called? You can find it in ~/.local/share/steam-shortcuts/ somewhere

In steam-buddy.gbc.yaml I have (Pokémon. I tried different versions of entering the name):
`- banner: "/home/gamer/.local/share/steam-buddy/banners/gbc/Pok\xE9mon Pinball.png"
cmd: gbc
dir: '"/home/gamer/.local/share/steam-buddy/content/gbc"'
hidden: false
name: "Pok\xE9mon Pinball"
params: ""Pok\xE9mon Pinball.zip""
tags:

Game Boy Color
banner: "/home/gamer/.local/share/steam-buddy/banners/gbc/Pok\xE9mon PB.png"
cmd: gbc
dir: '"/home/gamer/.local/share/steam-buddy/content/gbc"'
hidden: false
name: "Pok\xE9mon PB"
params: ""Pok\xE9mon PB.zip""
tags:

Game Boy Color
In steam-buddy.gba.yaml I have ried with an "à" ("Tetrisà" to check for a different emulator and a different character):- banner: "/home/gamer/.local/share/steam-buddy/banners/gba/Tetris\xE0.png"
cmd: gba
dir: '"/home/gamer/.local/share/steam-buddy/content/gba"'
hidden: false
name: "Tetris\xE0"
params: ""Tetris\xE0.zip""
tags:
Game Boy Advance
`

That makes a lot of sense, thanks! I think I know how to fix this. We just need to use html.unescape in some places.

@sharkwouter  did you figure it out how to fix this?

I didn't test it, but you can probably just change this line: https://github.com/ChimeraOS/chimera/blob/202b75bc5d6537b61b0b4e502c7f025e4e924485/chimera_app/server.py#L195
To be the following:
name = html.unescape(name.strip())

That change might be needed in shortcut delete as well. Not sure, would need testing.

I think the solution is on urllib.parse.quote() function. Maybe https://github.com/ChimeraOS/chimera/pull/198 fixes this as well

Just retested. Issue still exists.


Edge browser doesn't update form sometimes

Hello,
I'm using Vuelidate 0.4.3 and found the following bug in Edge 14:
There are the same form for adding and editing items. By default it's a first item in the list with opened edit form with filled input fields. Each field has a specified list of validation rules. Also, there is a button-switcher. If edit form enabled - by default button is Add, if add form enabled - button is Cancel. After clicking Add button I clear the form and everything fine with validation and with form also - all fields is empty. But in the browser after 2-3 switching from adding to cancellation, validation stops working correctly, as a result of which the form is not cleaned. For example, we can get into the form of adding with the filled fields from the previous element. I tried to completely remove the validation, and without it everything works fine. The problem is only in the Edge-browser, in the others everything works as it should. Below is the code:
Edit.vue

{{ errorDisplay($v.obj.name) }}

    <input class="time-input" v-model="obj.name" id="id" @input="$v.obj.name.$touch()"/>
    Name:

{{ errorDisplay($v.obj.comment) }}

    <textarea v-model="obj.comment" id="id" rows="5" @input="$v.obj.comment.$touch()"></textarea>
     Comment:

validations() {
    return {
        obj: {
            name: { minimumLength, maximumNameLength, forbidden, forbiddenNames: (val, rule) => { rule = this.forbiddenNames; return forbiddenNames(val, rule) } },
            comment: { maximumCommentLength, forbidden }
        },
        validationGroup: ['obj.name', 'obj.comment']
    }
}

I don't have edge browser on any of my machines and don't own any copy of Windows. If you could provide a fiddle with reproduction, so I can run and debug it in cloud setting, it would be very very helpful.

Closed as likely not linked to vuelidate at all and no reproduction was provided.


No LICENSE.md

In order to enable open-source contributions, it would be formative to consider the specifics of a license agreement.

Also worth asking about the license of the input data, which is the entire Game of Thrones and is presumably still under copyright.

License added.


Cant install Debian on pixelbook. ChromeOS version 109

Please, help me. Troubles begin after chromos was updared on version 107...

Google Pixelbook (2017)
the permission message is crouton
the debootstrap is no file found
the script is not creating a file for some reason
you can review the debootstrap
the mount maybe modified via debootstap when the install happens
on the install
crouton ...-r -t  |tee ~/Downloads/inst.txt this will create a install log called inst.txt in Download you can look at it via chromeOS

try this was change to run in /usr/local/bin
https://drive.google.com/uc?export=download&id=1HFjX1OejjHAYbaHVL3l9krmBUkA1caZR
test2 same a crouton -r -t
this may fix the permission issue
seem to work for me

#4901 should fix this. Please try again with the latest version of crouton.


Enabling Auto Punctuation

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for discussion)

[ ] Tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Hi @ace-n , can you please review this PR? It is pretty small. Thanks.

@happyhuman These reviews are now taken care of by @googleapis/yoshi-nodejs - I added the team to the reviewers.


Update list of all submissions per steward on each submit

Currently there is no way to get a list of all submissions. One option is to publish the list via ipns and update it on each submission. This will work as long as there is only a single node, but if a steward wants to run multiple mirrored daemons all with the same public/private key pair a more sophisticated eventually distributed append log will need be used such as https://github.com/haadcode/ipfs-log.
The other issue is a bit of a catch 22 in that we need determine if there is already an existing submissions list which if there isn't takes a very long time/just doesn't return.

https://github.com/ipfs/notes/issues/108 appears to a possible solution if/when implimented.

Edited issue title now that 87ff2a7253c0b3fc600c811d770f9c1b01595a4b adds a commit list via ipns publish. But this isn't really scalable due to the slow speed of ipns.

Submission list is going to be moved over to the Ethereum block chain so closing this as won't fix.

Switched back to ipns to keep things simpler. Already includes basic public/private key for authentication.


Change connection data on runtime

Hi, I'm using a strategy with tokens to authenticate a user in a web socket session. That token will expire and I will refresh that token before its expire.
The react-stomp will disconnect because the current token already expired, and at that moment I would like that the new connection gets the new URL/header with the new refreshed token.
Is there a way to achieve it without changing the library?
I'm looking through the code and I'm not seeing how it could work.
Something that would solve my problem is a "beforeConnect" hook, with the url as a parameter or something like that, or if I would be able to set a method to the URL parameter instead of a fixed string.
For example:
<SockJsClient
url={() => AppConfig.webSocket.connectionUrl()}
...
What do you think? I could open a PR to introduce that feature.
Thanks

@lucasventurasc Ideally this should be managed by passing url as props from your parent component to react-stomp but since in react-stomp shouldComponentUpdate always returns false this won't work as expected.
How do you think about updating the shouldComponentUpdate to handle changes in url? Something like, if url in props is different from current state then close previous connection and update component.

Ok, I'm really new on React, I will make some tests with your purpose and I will give you a feedback soon.
Besides that, can you provide-me an example of that?
"passing new url as props from your parent component to react-stomp"

@lucasventurasc
I was thinking something like this:
class ExampleComponent extends React.Component {
    constructor(props) {
       super(props);
       this.state = {
           websocketUrl: "some-url"
       };
    }

    codeToUpdateWebsocketUrl = () => {
        // get new websocket url based on your logic
        this.setState({websocketUrl: newUrl});
    }

    render() {
        return (
          <div>
            <SockJsClient url={this.state.websocketUrl} topics={['/topics/all']}
                onMessage={(msg) => { console.log(msg); }}
                ref={ (client) => { this.clientRef = client }} />
          </div>
        );
    }
}

Closing due to inactivity, reopen if needed


mjml version as an optional configuration parameter

Optional "mjml_version" parameter to save time rendering the email

@ropczan Thanks for your PR. Good idea
You need to rebase, I made some fixes to the actions workflow to fix issues with php 8.

@notFloran Thank you for your prompt response :)

@ropczan I just release a new version v3.7.0 with your changes.
Thanks again for the PR.