dbgeng input, unable to terminate

I'm using Ghidra 10.1.3 and the latest debugging tools from the windows SDK (dbgeng.dll dated 9/11/2021).
One common thing I do in windbg is to edit some memory using the ed command.  It shows the current value, prompts for a new one, and then when you hit enter moves on to the next dword in memory.  When you are done, you simply hit enter without typing new data and that returns back to the normal windbg prompt, i.e. terminates the editing.
But in Ghidra, I am unable to figure out a way to break out of this input phase.  I have tried CTRL-C, CTRL-break, CTRL-Z, the pause icon on the Ghidra GUI to send an interrupt, and, "quit", "exit", of course, hitting enter.  Nothing seems to work.  Is this a bug or am I just missing the magic sequence?
In the example below I was just trying to update a single dword, so I typed my data for the first location and then did a double-enter.
0:000:x86>dd @ecx
014dbb30  0071581c 000000e0 00000001 00000000
014dbb40  00000005 014db2f0 abababab abababab
014dbb50  00000000 00000000 e4746307 0000af81
014dbb60  014c00c0 014cdbe0 feeefeee feeefeee
014dbb70  feeefeee feeefeee feeefeee feeefeee
014dbb80  feeefeee feeefeee feeefeee feeefeee
014dbb90  feeefeee feeefeee feeefeee feeefeee
014dbba0  feeefeee feeefeee feeefeee feeefeee
 0:000:x86>ed @ecx+c
014dbb3c 00000000  >>>1
>>>
>>>dd @ecx
>>>
>>>
>>>
>>>
Interrupt sent
>>>q
>>>exit
>>>
>>>
>>>

Environment (please complete the following information):

OS: Windows 10 21H1 x64
Java Version: openjdk 11.0.15 2022-04-19
Ghidra Version: 10.1.3
Ghidra Origin: official release from github

@jeffbromberger There are some issues with the Interpreter that unfortunately prevent us from making it behave exactly like the windbg console.  (We're working on them, but....)  That said, entering an empty line should work, i.e. you should see the prompt switch from ">>>" to "0:000>", and be able to enter commands again.

Ok, let me know if there's anything I can do to help but I'm thinking you should be able to repro easily.
On a side note, I'm currently trying to figure how to kill the underlying process I was debugging when it got stuck in this input loop.  Since I could never regain control in ghidra I did a disconnect and then closed the debugger.  But the .exe being debugged is still there and since it is marked as being debugged I can't kill it.  Not seeing a way to re-attach to that existing instance in a new invocation of ghidra's debugger or any other way to re-sync this connection.

OK, I think I see the issue.  I started up notepad.exe, and did the following
0:000:x86>dd @rip
00007fffe60ca2d0  78ec8348 48c98b4c abd2058b 85480010 ... 0:000> ed @rip+8  <return> 00007fffe60ca2d8 abd2058b >>>deadbeef
00007fffe60ca2dc 85480010 >>> <return> 0:000> dd @rip <return> 00007fffe60ca2d0  78ec8348 48c98b4c deadbeef 85480010
If I just put in "de", I don't get the prompt back.  Windbg interprets the "de" as the full word value, but our Interpreter does not.  You'd need to either use "eb" or repeat the other bytes in the word.  We can look into this, but I'm going to attribute this to somewhat wanky interaction between the two console flavors. We have a ticket in for an overhaul of the console, but it's going to be complicated and I don't anticipate early resolution on that front.
If you are mid-command and disconnect the debugger, your target is going to be in a weird state.      I believe you can reconnect to the target via attach if you haven't terminated the session (i.e. disconnected), but once you disconnect all bets are off.  (This is true in windbg too.)  I need to test out the derach/re-attach - am pretty sure I intended to add that functionality, but am also pretty sure I never tested it.

Thanks @d-millar for your follow-up and consideration on this issue.
I tried "eb" as you suggested but I'm not having success.  Two things - I still cannot find a way to stop input when I'm finished, but I also do not get the advance to the next byte to allow me to enter multiple items.  For example, after I type 47 to replace the 46 and hit enter, it should be showing me 4C for the "L" allowing me to change that byte.  At that point if I just hit enter it should quit, or if I give a value it should advance to the next byte.
 0:003>da 10eb890
00000000`010eb890  "FLAG{ARBITRARY-READING}........."
00000000`010eb8b0  ""
 0:003>eb 10eb890
00000000`010eb890 46  >>>47
>>>
>>>
>>>
>>>
>>>
>>>0
>>>0x47
>>>
>>>

On the other topic of the debugger connection what you describe is perfectly reasonable and not much of a practical limitation.  One small question if I may (sorry to mix two topics here, but the other doesn't seem to be an "issue" so I don't want to create something) - when I first start the debugger on my exe from the launcher it shows a DBGENG@xxxxxx entry in the Debugger Targets window.  But for the life of me I cannot figure out how to actually start debugging that one.  When I go to Debugger -> Debug foo.exe ->in dbgeng locally IN-VM then it fires up the debugger and now I have two entries DBGENG@xxxxxx and DBGENG@yyyyyy.  I'm just unclear on what this first connection is if I'm not actually able to debug with it?

@jeffbromberger Hmmm, something odd is going on, because I'm doing what I believe is exactly the same thing you're doing and getting different results.  That is to say, I start a program, break, do "eb @rip", enter "de", enter "ad", enter "be", and hit one more enter, the prompt returns, and dd returns xxbeadde @rip.  I don't have a good guess yet as to why we're getting different results.
On the launching front, there are several ways to launch a program from the debugger. The most obvious is the bug button in the tool and the associated drop-down.  This will try to run the executable associated with the program in the active tab of the Static listing using the last type of connection you ran, e.g. IN-VM dbgeng.
Alternatively, you can start a debugging session without starting the target and then run/attach/etc. one or more targets from that session.  To do this, you usually use the gear in the Targets window to start the session, and then the bug in the Objects window to launch the executable bound to the program in the Static listing.
You can have multiple sessions up, which sounds like what you did probably inadvertently.

Interesting.  So I started a whole new session of ghidra, relaunched the debugger and used your exact eb @rip sequence and guess what?  It worked :)   Then I did .kill, disconnected the debugger, and clicked the bug button to debug it again.  It attached to the debugger but this time I was seeing some extra error messages about being unable to deliver callbacks.  However, I was still able to toggle my entry point breakpoint and start the execution.  When the bp hits, I tried the exact same eb @rip command and this time it behaves like I was seeing previously.
In all debugger launches I get various errors about not being able to load extension dlls and dbghelp.dll.  I have tried running ghidrarun.bat as admin and this doesn't resolve anything.  Not sure if this is a wow64 issue or what (my exe is 32-bit and the system is 64-bit).  But the sequence seems repeatable - first time debugging the process eb works, then a subsequent time it gets the extra error messages and does not work.  Maybe that is a clue for you?
 Unable to add extension DLL:  ntsdexts
 Unable to add extension DLL:  uext
 Unable to add extension DLL:  exts
 SECURE: File not allowed to be loaded -  C:\WINDOWS\SYSTEM32\dbghelp.dll
Error code:  Win32 error 0n5
 The call to LoadLibrary( ext ) failed,  Win32 error 0n2
    " The system cannot find the file specified. "
Please check your debugger configuration and/or network access.
 Unable to add extension DLL:  wow64exts

************* Path validation summary **************
 Response                         Time (ms)     Location
 Deferred                                         srv*C:\symbols*https://msdl.microsoft.com/download/symbols
 Symbol search path is:  srv*C:\symbols*https://msdl.microsoft.com/download/symbols
 Executable search path is:
 ModLoad: 00000000`00710000 00000000`0071b000   UserAfterFree.exe
 Unable to deliver callback,  31
 0:000>g
ModLoad: 00007ff8`d7e30000 00007ff8`d8025000   ntdll.dll
 **Unable to deliver callback,  31**
 ModLoad: 00000000`774d0000 00000000`77673000   ntdll.dll
 **Unable to deliver callback,  31**
 20f8:0e68 @ 149747125 - LdrpInitializeProcess - INFO: Beginning execution of UserAfterFree.exe (C:\Users\jeffb\Documents\malware\chat\UserAfterFree.exe)
	Current directory: C:\WINDOWS
	Package directories: (null)
 20f8:0e68 @ 149747125 - LdrpLoadDllInternal - ENTER: DLL name: C:\WINDOWS\SYSTEM32\wow64.dll
 20f8:0e68 @ 149747125 - LdrpFindKnownDll - ENTER: DLL name: wow64.dll
 20f8:0e68 @ 149747125 - LdrpFindKnownDll - RETURN: Status: 0x00000000
 20f8:0e68 @ 149747125 - LdrpMinimalMapModule - ENTER: DLL name: C:\WINDOWS\System32\wow64.dll
 ModLoad: 00007ff8`d6a70000 00007ff8`d6ac9000   C:\WINDOWS\System32\wow64.dll
 **Unable to deliver callback,  31**

[SNIP]

 20f8:0e68 @ 149753484 - LdrLoadDll - RETURN: Status: 0x00000000
 20f8:0e68 @ 149753484 - LdrpGetProcedureAddress - INFO: Locating procedure "LCIDToLocaleName" by name
 20f8:0e68 @ 149753484 - LdrpGetProcedureAddress - INFO: Locating procedure "LocaleNameToLCID" by name
 Breakpoint 0 hit
 Unable to deliver callback,  31
 *** WARNING: Unable to verify checksum for  UserAfterFree.exe
 0:000:x86>db @eip
00713038  e8 2f 05 00 00 e9 4e fe-ff ff 55 8b ec 83 61 04  ./....N...U...a.
00713048  00 83 61 08 00 8b 45 08-89 41 04 8b c1 c7 01 bc  ..a...E..A......
00713058  51 71 00 5d c2 04 00 55-8b ec 56 ff 75 08 8b f1  Qq.]...U..V.u...
00713068  e8 58 00 00 00 c7 06 bc-51 71 00 8b c6 5e 5d c2  .X......Qq...^].
00713078  04 00 83 61 04 00 8b c1-83 61 08 00 c7 41 04 c4  ...a.....a...A..
00713088  51 71 00 c7 01 bc 51 71-00 c3 55 8b ec 56 ff 75  Qq....Qq..U..V.u
00713098  08 8b f1 e8 25 00 00 00-c7 06 d8 51 71 00 8b c6  ....%......Qq...
007130a8  5e 5d c2 04 00 83 61 04-00 8b c1 83 61 08 00 c7  ^]....a.....a...
 0:000:x86>eb @eip
00713038 e8  >>>90
>>>90
>>>
>>>
>>>
>>>

Ah, excellent clue - sorry I didn't get back to this today, but will take another stab tomorrow!

@jeffbromberger Well, I wasn't able to reproduce the stop-start thing, but occurs to me I'm not running exactly the same version as you are, i.e. am running in the 10.2 beta.  Your comment about the callback messages made me think of this.  (Am pretty sure that's fixed in 10.2.)  Will try 10.1.3 sometime today - with luck, reproduce the error.
The "Unable to load" messages probably don't matter.  If you want to get rid of them, copying the dbgXXX.dll's into the bin directory for your JDK will make them go away, I believe, but probably unnecessary.  Running the agent as root and connecting via GADP would also work, but that's an even bigger hassle.  The path sequence is a bit of a nightmare.  We're trying to figure out a better solution, but the fact that Java preloads dbgeng to allow it to do stack walks complicates things.

@d-millar   I'd be happy to try the 10.2 beta on my end if you'd like.  I didn't see it listed under releases, is it available somewhere or not ready yet?

10.2 is equivalent to the current master, i.e. you have to build it from source (which isnt't that hard to do, but....)

OK, wow - so completely reproducible in 10.1.3 and doesn't happen building off of master.  I can spend some time trying to figure out what and how it got fixed, but might be a bit before I get to it.  Are you interested in trying to build Ghidra from source?  That would give you the fix.  :)

@d-millar I would classify my java knowledge as "I've heard of gradle" but nevertheless I did manage to get the thing to build in a WSL2 ubuntu instance.  I then extracted the build/dist zip to my windows setup where I had been running previously (different directory of course).  First thing I noticed was there were no .bat files, only .sh.  I copied the .bats from the 10.1.3 release and it fired up.  But then when I go to disassemble notepad it reports "os/win_x86_64/decompile.exe does not exist".  Doing a recursive dir listing sure enough I don't find that file anywhere in the unzip of what I built, but it is there in the 10.1.3 tree.  Did I need some additional build steps to produce this?
If this is too much of a headache for you I can certainly keep working with 10.1.3 for now.  But if you want me to try out 10.2 I don't mind putting some more time into it...

When you build Ghidra yourself, it will only build it for the platform you are on.  Since you built in in WSL2, it built it for Linux, which is why you don't have .bat files or win_x86_64 natives files.  You'd either have to run what you built under WSL2, or try the build again on a pure Windows platform (which would require Visual Studio to build the Windows natives).

I got a windows build completed but I'm having trouble launching the debugger now.  It opens my project fine in the code browser, but when I launch the debugger I get an "error in plugin event listener" with this trace
java.lang.NullPointerException
	at ghidra.app.plugin.core.debug.gui.objects.components.ObjectTree.getSelectedObject(ObjectTree.java:190)
	at ghidra.app.plugin.core.debug.gui.objects.DebuggerObjectsProvider.getObjectFromContext(DebuggerObjectsProvider.java:322)
	at ghidra.app.plugin.core.debug.gui.objects.DebuggerObjectsProvider.isInstance(DebuggerObjectsProvider.java:866)
	at ghidra.app.plugin.core.debug.gui.objects.DebuggerObjectsProvider.lambda$createActions$19(DebuggerObjectsProvider.java:1027)
	at docking.action.DockingAction.isEnabledForContext(DockingAction.java:184)
	at docking.ComponentPlaceholder.contextChanged(ComponentPlaceholder.java:532)
	at docking.DockingWindowManager.contextChanged(DockingWindowManager.java:2232)
	at docking.AbstractDockingTool.contextChanged(AbstractDockingTool.java:208)
	at ghidra.framework.plugintool.PluginTool.contextChanged(PluginTool.java:1466)
	at docking.ComponentProvider.contextChanged(ComponentProvider.java:450)
	at ghidra.app.plugin.core.debug.gui.objects.DebuggerObjectsProvider.setProgram(DebuggerObjectsProvider.java:466)
	at ghidra.app.plugin.core.debug.gui.objects.DebuggerObjectsPlugin.processEvent(DebuggerObjectsPlugin.java:142)
	at ghidra.framework.plugintool.Plugin.eventSent(Plugin.java:329)
	at ghidra.framework.plugintool.mgr.EventManager.sendEvents(EventManager.java:286)
	at ghidra.framework.plugintool.mgr.EventManager.lambda$new$3(EventManager.java:49)
	at ghidra.util.Swing.doRun(Swing.java:292)
	at ghidra.util.Swing.runNow(Swing.java:208)
	at ghidra.util.Swing.runNow(Swing.java:163)
	at ghidra.framework.plugintool.mgr.EventManager.fireEvent(EventManager.java:216)
	at ghidra.framework.plugintool.PluginTool.firePluginEvent(PluginTool.java:475)
	at ghidra.framework.plugintool.Plugin.firePluginEvent(Plugin.java:481)
	at ghidra.app.plugin.core.progmgr.MultiProgramManager.fireOpenEvents(MultiProgramManager.java:267)
	at ghidra.app.plugin.core.progmgr.MultiProgramManager.addProgram(MultiProgramManager.java:95)
	at ghidra.app.plugin.core.progmgr.ProgramManagerPlugin.lambda$openProgram$6(ProgramManagerPlugin.java:469)
	at ghidra.util.Swing.doRun(Swing.java:292)
	at ghidra.util.Swing.runNow(Swing.java:208)
	at ghidra.util.Swing.runNow(Swing.java:163)
	at ghidra.app.plugin.core.progmgr.ProgramManagerPlugin.openProgram(ProgramManagerPlugin.java:475)
	at ghidra.app.plugin.core.progmgr.ProgramManagerPlugin.openPrograms(ProgramManagerPlugin.java:658)
	at ghidra.app.plugin.core.progmgr.ProgramManagerPlugin.acceptData(ProgramManagerPlugin.java:129)
	at ghidra.framework.plugintool.PluginManager.acceptData(PluginManager.java:48)
	at ghidra.framework.plugintool.PluginTool.acceptDomainFiles(PluginTool.java:414)
	at ghidra.framework.main.ToolButton.openDomainFiles(ToolButton.java:647)
	at ghidra.framework.main.ToolButton$2.end(ToolButton.java:625)
	at ghidra.framework.main.ZoomImageRunner$1.end(ZoomImageRunner.java:54)
	at org.jdesktop.animation.timing.Animator.end(Animator.java:755)
	at org.jdesktop.animation.timing.Animator.stop(Animator.java:660)
	at org.jdesktop.animation.timing.Animator.timingEvent(Animator.java:732)
	at org.jdesktop.animation.timing.Animator.access$200(Animator.java:75)
	at org.jdesktop.animation.timing.Animator$TimerTarget.actionPerformed(Animator.java:1041)
	at java.desktop/javax.swing.Timer.fireActionPerformed(Timer.java:317)
	at java.desktop/javax.swing.Timer$DoPostEvent.run(Timer.java:249)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

---------------------------------------------------
Build Date: 2022-May-05 0829 CDT
Ghidra Version: 10.2
Java Home: C:\Program Files\Eclipse Adoptium\jdk-<IP_ADDRESS>-hotspot
JVM Version: Eclipse Adoptium 11.0.15
OS: Windows 10 10.0 amd64
Workstation: ----

Looks like if I proceed on it will still load the debugger.  This does seem to have resolved the input issues I was originally facing.  Even ed is working.  If I type a single byte it'll still advance to the next dword as expected.  And hitting enter breaks out.  So all good there.  Let me know @d-millar if you want me to try anything else related to the above error.  Seems to happen every time I launch the debugger (the Ghidra tool, not a connection to windbg within that tool)


Added ForceMode2D.Impulse

Description and example of ForceMode2D.Impulse under "Adding Force To Bullets".

Thank you!


Unable to store form submission

Scenario: create a form and set store submissions to true.
Now we have this call stack:
\Statamic\Http\Controllers\FormController::submit
|_ \Statamic\Eloquent\Forms\Submission::save
    |_ \Statamic\Eloquent\Forms\Submission::toModel

In toModel() we have:
$model = $class::findOrNew($this->id());

The result of $this->id() is a timestamp - no wonder, just look at the code:
https://github.com/statamic/cms/blob/7cbc505da6c7f2f9568599631b607aa427d5cc62/src/Forms/Submission.php#L52
return $this->id = $id ?: str_replace(',', '.', microtime(true));

Surprise, now we have a database exception:
SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for type bigint: "1715767251.4272"
CONTEXT:  unnamed portal parameter $1 = '...' (Connection: pgsql, SQL: select * from "form_submissions" where "form_submissions"."id" =<PHONE_NUMBER>.4272 limit 1)

How is this supposed to work?

It seems to be due to Postgres being less forgiving than other engines - see the comments here: https://github.com/statamic/eloquent-driver/pull/122#issuecomment-2111965758


Polish translation

Hi
I made Polish translation for android traccar client, if it's not a problem please merge it to master. Thanks

All localization should be done through Transifex:
https://www.transifex.com/traccar/traccar/


Do not alter arguments passed to Twitter::Autolink methods

By using the 'destructive' symbolize_keys! method, the caller gets an unexpected behaviour
where the arguments passed are modified by the Autolink methods. This side-effect is better
to avoid in general.
BTW, fix the non-destructive symbolize_keys method.
All conformances tests are still OK.

Thanks for the fix. Could you make sure tests are passing in travis?

James,
The build is fixed with the latest commit I push in my repository. I need to see how I can update this pull request...

Actually, the pull request is automatically updated as far as I can see. And the travis build is successful.

Thanks for contributing.

You're welcome.


Run in web

This is intended as an replacement PR for #39 which got messed up with conflicts.
@jgclark @alkatar21 can you look at it? I added a vsix as well.

I've been trying for 30 minutes to test this in one of the web test environments, using various methods listed at https://code.visualstudio.com/api/extension-guides/web-extensions#web-extension-tests.
I think it does install OK, but I can't get further to actually test it in use.
Which method did you use to test it @elazarcoh?

vscode-test-web --browserType=chromium --extensionDevelopmentPath=$extensionLocation $testDataLocation
described in https://github.com/microsoft/vscode-test-web.
this opens a web browser with vscode, in the given folder, where I put my .vscode/settings.json for testing.

Thanks. Sounds like I'd been doing the right thing. I'd tried a variety of browserTypes as well, given the following warning about Playwright which I couldn't get to go away. I consistently get variations of this error (when running from my local GH directory with the vsix file in):
vscode-test-web --extensionDevelopmentPath=. .
@vscode/test-web: 0.0.15
Serving dev extensions from /Users/jonathan/GitHub/vscode-todo-highlight
Serving local content /Users/jonathan/GitHub/vscode-todo-highlight at /static/mount
Listening on http://localhost:3000
(node:47141) UnhandledPromiseRejectionWarning: browserType.launch: Executable doesn't exist at /Users/jonathan/Library/Caches/ms-playwright/chromium-907428/chrome-mac/Chromium.app/Contents/MacOS/Chromium
╔═════════════════════════════════════════════════════════════════════════╗
║ Looks like Playwright Test or Playwright was just installed or updated. ║
║ Please run the following command to download new browsers:              ║
║                                                                         ║
║     npx playwright install                                              ║
║                                                                         ║
║ <3 Playwright Team                                                      ║
╚═════════════════════════════════════════════════════════════════════════╝
    at openBrowser (/opt/homebrew/lib/node_modules/@vscode/test-web/out/index.js:99:59)
    at open (/opt/homebrew/lib/node_modules/@vscode/test-web/out/index.js:76:27)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:47141) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:47141) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
^C

I then remembered I'm running node v14.x (a requirement on the other project I contribute to), so I switched up to node 19.9.0, and updated everything, and then I get Chromium spinning up on localhost:3000 without errors, but it only gives a blank screen without any hint of VSCode other than its favicon. Passing different folders for test data doesn't help.

try to open the dev tools (f12) and see if there are errors that make sense


Test Hex File

testprogram.zip
I found when testing it helped to have a hex file that definitely worked when getting this running. Attached is a simple one.

I have been using bbcbasic, freshly compiled to .hex format with 35KB thus far.

I've just blasted over 35KB over bbcbasic.hex with two different ftdi adapters using Win10 / 384.000 baud. No errors

With two different Agon boards; the Agon original and the Agon light2.

And to remove all doubt regarding programs that definitely worked, or the speed I'm getting

FWIW, I upload 170kb .bin at 230400 baud/bps with my cheap FTDI adapter connected using very dodgy jumper wires. 384000 results in errors too often. I suggest that Paul check his cabling and connections, and make sure the FTDI apadter is set to 3.3V (and/or try a different FTDI adapter?).

Slower than expected serial-over-USB speeds have also been reported in programming the VDP, using the Arduino IDE or platformIO. It seems like there is some significant variation in what platforms (Mac/Linux/Windows, USB 1.0/2.0/3.0, with or without Hub) are able to achieve reliably using async serial protocol. This isn't something I'm striving to solve with this tool. It would be prudent to start with a much lower serial speed generally, say potentially as low as 57600 and then let each user experience how much faster they are able to get. Or slower even.
For example, 512Kbaud is asking too much from my setup, even though I'm using a dedicated USB3.0 port with a modern non-cloned USB-serial IC on an FTDI board, without hub. At certain speeds, even the OS driver comes into play.

To put things into perspective, there have also been some demands/requests for additional feedback during the uart transmission, which would have required the speed to significantly drop to handle the overhead of transmitting to the FTDI side during reception. Some people where willing for the tool to then even go to 9.600 baud, just to have this reporting feature in.
I was not willing to do that, but it goes to show that speed is objective.

And lastly, async serial could technically be made robust, requiring hardware handshakes during transmission. Then we would gain real speed-up and let the slowest link in the chain be responsible for the effective end-to-end speed. Unfortunately, this will immediately rule out a significant portion of FDTI adapters in the field, not to mention support from specific serial software or APIs. The common denominator is async serial without handshakes, resulting in speeds that may vary wildly between setups.
It appears that the Agon variant currently can easily, reliably, reach 384Kbaud. Mileage with other peoples setup may vary.


Fixed private props and update material dropdown

Codecov Report

Merging #237 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #237   +/-   ##
=======================================
  Coverage   65.15%   65.15%
=======================================
  Files          15       15
  Lines       33620    33620
  Branches      127      127
=======================================
  Hits        21905    21905
  Misses      11640    11640
  Partials       75       75

Impacted Files
Coverage Δ

...omponents/dropdown/tag-input-dropdown.component.ts
75.59% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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


Add Project Metadata

Added some inital project metadata. This may be updated in the development process.
@maximilian-muecke , @Noza23 , @jakobCampus : Please add the personal information you want to be displayed in the DESCRIPTION file.

Why did you change the predict method. The argument is supposed to be explicit, since all predict methods in the package should use newdata as an argument and use it for documentation. The same goes the argument name of the predict method

Why did you change the predict method. The argument is supposed to be explicit, since all predict methods in the package should use newdata as an argument and use it for documentation. The same goes the argument name of the predict method. See the the following as reference https://github.com/mlr-org/mlr/blob/main/R/RLearner_regr_lm.R

I actually got this warning when running R CMD check
* checking S3 generic/method consistency ... WARNING
predict:
  function(object, ...)
predict.ModelLinearModel:
  function(model, newdata, ...)

which suggests that we overwrite the predict generic arguments, which should not be done according to the R docs.
I see that keeping newdata is good and added that in the latest commit. However I don't see how we can overwrite the object argument. Do you know of a way to do this?

Yeah you right doesn't work


Aurora not recognizing Fortnite Profile.

What are you experiencing an issue with?

[ ] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[x] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[x] Performance problems
[ ] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[ ] Device issue
[ ] Other, (Fill in another type of issue here)

Expected behavior:
The profile should switch to the correct (fortnite) profile.
Actual behavior:
The profile remains on the desktop profile.
How often does this problem occur?
Every time.
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Creating profile with the FortniteClient-Win64-Shipping.exe file as the source.
Making the desired changes.
Opening Fortnite.

What is the version of Aurora you are experiencing this issue with?

My version is... 0.6.0b Beta
Does the previous version of Aurora have this issue?
The previous version, 0.5.1d Beta, did/did not have this issue.
Please include any logs, screenshots, or files that are related to this issue:
I can provide the log info if needed.

I'll see if I can get Fortnite downloaded to see if I can debug this.

There is a setting in Fortnite called "Peripheral Lighting"  which is turned on by default. It was switching my Logitech G810 from the custom profile I created to just a solid color as soon as I got into a game. My profile would be fine with Fortnite loaded right up until getting into a match.

How do u log into the account


Mini.files gives error and crashes on closing and reopening the Files windows

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
Mini.files
Description
When I open Mini.Files, and close the window, reopening mini.files gives an error and causes mini.files to stop working.
I have simple mini.files config, and one keymap for it.
  require("mini.files").setup({
    mappings = {
      go_in_plus = "<cr>",
      synchronize = "<c-s>",
    },
    windows = {
      max_number = 3,
      preview = true,
    },
  })

  vim.keymap.set("n", "-", function()
    MiniFiles.open(vim.api.nvim_buf_get_name(0))
    MiniFiles.reveal_cwd()
  end, { desc = "Open Mini Files" })

Some other things I observed.

After Mini.files crashes, the "-" keybind stops working.
If I navigate to some other files, Using Mini.Pick or Arrow.nvim, and press "-" again. Mini.Files opens, but closing that window and reopening it again give me the same error
If I replace the MiniFiles.Open() call with MiniFiles.open(vim.api.nvim_buf_get_name(0), false) like given in the readme, to open a "fresh" window, that makes everything work as expected, and I can close and reopen MiniFiles as many times as I want.

Neovim version
NVIM v0.10.2
Steps to reproduce

nvim -nu minimal.lua
Press "-" to open Mini.Files window
Press q to exit the window
Press "-" to open Mini.files window again

Expected behavior
Reopen the Mini.Files window again...
Actual behavior
I receive this error
Error executing vim.schedule lua callback: ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:2378: attempt to compare number with nil
stack traceback:
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:2378: in function 'is_modified_buffer'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:2550: in function 'window_update_border_hl'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:2519: in function 'window_set_view'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:1779: in function 'explorer_refresh_depth_window'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:1505: in function 'explorer_refresh'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:778: in function 'open'
        /home/sumit/.config/nvim/lua/plugins/mini.lua:61: in function </home/sumit/.config/nvim/lua/plugins/mini.lua:60>

Thanks for the issue!
I can not reproduce with only 'mini.files' enabled with config from "Description" (i.e. no other plugins or mappings). There are no errors after several -, q, -, q cycles. Would you mind trying to reproduce this locally with only 'mini.files' enabled?

Okay, I tried to make a minimal.lua file, And That doesn't seem to have this issue. So I don't know what the issue might be.
vim.loader.enable()

-- Clone 'mini.nvim' manually in a way that it gets managed by 'mini.deps'
local path_package = vim.fn.stdpath("data") .. "/site/"
local mini_path = path_package .. "pack/deps/start/mini.nvim"
if not vim.loop.fs_stat(mini_path) then
	vim.cmd('echo "Installing `mini.nvim`" | redraw')
	local clone_cmd = {
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/echasnovski/mini.nvim",
		mini_path,
	}
	vim.fn.system(clone_cmd)
	vim.cmd("packadd mini.nvim | helptags ALL")
	vim.cmd('echo "Installed `mini.nvim`" | redraw')
end

-- Set up 'mini.deps' (customize to your liking)
require("mini.deps").setup({ path = { package = path_package } })

require("mini.files").setup({
	mappings = {
		go_in_plus = "<cr>",
		synchronize = "<c-s>",
	},
	windows = {
		max_number = 3,
		preview = true,
	},
})

vim.keymap.set("n", "-", function()
	MiniFiles.open(vim.api.nvim_buf_get_name(0))
	MiniFiles.reveal_cwd()
end, { desc = "Open Mini Files" })

Actually, My MiniJump2D is also a bit broken for the past few days, I didn't bother trying to fix that(Or thought anything of it when I noticed), Since I don't use it that much. But That's also not occurring with the minimal.lua example.
In case of MiniJump2D, I have to press the "label letters" sequence twice to jump to the place I want. For example,

I press  to start MiniJump2D
Press, let's say "r' and then "w" labels
after pressing "w", the label resets to original un-filtered labels, And I have to press "r" and "w" again to jump.

So I don't know what the issue might be.

My guess would be to start with the following:

Execute :nmap q when inside 'mini.files' buffer. Maybe it gets defined to do something else.
Try disabling plugins which might interact with floating windows or key presses (like 'folke/which-key.nvim').
Fully bisect config.

Actually, My MiniJump2D is also a bit broken for the past few days, I didn't bother trying to fix that(Or thought anything of it when I noticed), Since I don't use it that much. But That's also not occurring with the minimal.lua example.

Then I am afraid it is definitely some third party plugin (or you config function/autocommand) behaving in conflicting manner.
Closing as not an issue with 'mini.nvim' per se. If you find a reproducible example which uses only it, please let me know.

Thanks, for the tips on how to pinpoint the issue, That worked wonders, I figured out the cause.
It's the plugin hardtime.nvim. I recently added it to my config.
Disabling this plugin fixes both the MiniFiles and MiniJump2D issues.
Still quite weird that hardtime somehow causes issues with mini.nvim
This was my hardtime config
local function hardtime()
  add({
    source = "m4xshen/hardtime.nvim",
    depends = {
      "MunifTanjim/nui.nvim",
    },
  })
  require("hardtime").setup()
end


UE4ShaderPluginDemoLearn

how can i display on the screen directly, not to save the result as a picture.

Oh wow, this is a lot of files. What does this do?


readonly-text (styled like edit-text) and additional styles to picker

The changes allow us to do the following
Styling the picker items like we want to:

Having a 'label' that looks like the standard edit-text but without all the edit text features

Allows us to style the item picker (especially the indicators).
Provide a readonly version of edit-text

edit-text required a bunch of additional properties/dependencies that are not needed (i.e. showing a version row


Julia 1.0 support

Codecov Report

Merging #6 (07aac40) into master (f93d259) will decrease coverage by 11.73%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master       #6       +/-   ##
===========================================
- Coverage   91.34%   79.61%   -11.74%
===========================================
  Files           7        6        -1
  Lines         104      103        -1
===========================================
- Hits           95       82       -13
- Misses          9       21       +12

Impacted Files
Coverage Δ

src/show.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/eval.jl
33.33% <0.00%> (-66.67%)
:arrow_down:

Continue to review full report at Codecov.

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


cli: migrate from urfave/cli/v1 to urfave/cli/v2

Signed-off-by: Mritunjay Sharma<EMAIL_ADDRESS>Fixes-262
This PR fixes #262
What
Migrates the Main App from urfave/cli to urfave/cli/v2
Why
Taking advantage of the new version of urfave/cli
How Tested
Locally

Thanks a lot for the PR @mritunjaysharma394 ! Quite a lot there :) Nice catch with those BoolTFlag changes too.

Thank you so much for the review and your kind words,  @kcq :-)
really means a lot, I will also propose changes to their migrate guide too so that changes like BoolTFlags, Going away of GlobalString, etc and NewStringSlice are reflected there :)

@mritunjaysharma394 Good idea! Their guide definitely needs more info. Now it'll be interesting to see if there are any unexpected gotchas :) I wish they'd still allow the flags anywhere, but in v2 they need to be first.

Yes @kcq, for now I have created a PR for them https://github.com/urfave/cli/pull/1319 and also hoping that we don’t face unexpected gotchas and even if we do, we will solve it :)
thanks for all the help!

nice PR!

Thanks a lot :)


[Documentation] Fix Broken Links

After all scripts of capi release have been migrated to capi-workspace,
CONTRIBUTING.md file is still pointing to the old now broken links.
Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

An explanation of the use cases your change solves

Links to any other associated PRs

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[not relevant] I have run all the unit tests using bundle exec rake

[not relevant] I have run CF Acceptance Tests

Thanks for fixing this @tareqmamari!


Question about more than one person on a video.

Hi, Im new to machine learning if im sorry if I asked any weird question, but how do you guys manage when there are one or more faces in a video, where FF++ fake have only one manipulated face when there are more than one faces in a scene. Thank you in advance!

how to get the pre trained model? Please help

Hi. In our cases, as explained in the paper, we separate the faces and obtain different predictions for each one of them. If one or more of the faces are classified as fake, the whole video is fake. So, for example, if you have a video with 3 frames and in each frame, you have two people, with our system you will obtain:
Frame 0:

Person 0: Fake
Person 1: Real
Frame 1:
Person 0: Fake
Person 1: Fake
Frame 2:
Person 0: Fake
Person 1: Real

Video-Level classifications:
- Person 0: Fake (since the majority are fake [2/3])
- Person 1: Real (since the majority are real [2/3])
Video is classified as fake since Person 0 is fake.
This is a simplification but you can find more information about it in the paper:
https://arxiv.org/abs/2107.02612
For the pretrained model, I will publish it as soon as possible.


Add javascript adapter as dependency

You should add javascript as dependancy in your io-package.json
"globalDependencies": [{"javascript": ">=x.x.x"}]

Thank you for the issue. I have added globalDependencies in the last update.


Merge SIAM copyedits together with recent changes

The notebooks Shallow_water and Shallow_tracer were recently returned from SIAM after copy editing. In the meantime, #205 made some changes to these notebooks.  I think I have successfully merged the two into this version, but @ketch should take a look.

Thanks for dealing with this.  Looks good.


Some security considerations - possibility to implement GPG backend?

Hi,
I just had a play around with WinCryptSSHAgent and I'm really liking it, however there seem to be some security issues with the current backend when using it with a Yubikey.
Compared to Gpg4win it uses the PIV functionality of a Yubikey instead of the GPG functionality.
When using Gpg4win, the Yubikey always needs to be plugged in and the first access to the GPG key after restarting or plugging in the Yubikey needs to be validated by entering the GPG PIN. When pulling out the Yubikey no further key access is possible anymore, so no new SSH sessions can be created.
When using WinCryptSSHAgent, it seems the Yubikey only needs to be plugged in the very first time starting it. After that I can remove the Yubikey, close and restart WinCryptSSHAgent and I can still open new SSH sessions.
While this makes authentication a lot easier, it also seems to circumvents a lot of the added security a Yubikey offers. I'm no expert on this by any means, so maybe I just got something wrong.
Would it be possible to implement something that uses the GPG functionality of a Yubikey instead of PIV to have the added security of the PIN and allow no further access as soon as the key is removed?
All other existing solutions for WSL2 only hook into a running Gpg4win instance, but don't offer a true replacement, so this would really be awesome to have.
As a sidenote, using the GPG functionality of the Yubikey would also keep the same keys compared to Gpg4win, so no new keys would have to be placed on the servers when switching from Gpg4win to WinCryptSSHAgent.

Hello,
It's weird, PIV mode requires PIN as well. After removing the Yubikey,  you will not be able to open a new SSH session. Are you sure you set the ssh agent and server correctly?

Oh wow, you are right, WinCryptSSHAgent seems to use a "MS-Organization-Access" certificate from my certificate store, but that's the only public key it shows, so it doesn't even detect the one on my yubikey.


feat: posts count on profile page

✨ Codu Pull Request 💻

👉 Please remove the below and replace with your own values, leaving the headers where they are. 👈
Pull Request details:

Amount of articles user published is displayed in their profile page

Any Breaking changes:

None

Associated Screenshots:

( Welcome file extensions include gifs/png screenshots of your feature in action )

None

Nice one @serzhan181! Thank you


Work with Bismark?

Hello!
This package/script is great considering how poor SNP calling algorithms are on BS-seq libraries. Just wondering, does this also work on Bismark generated aligned bam files? Also, I am assuming this works on both directional or non-directional bisulfite libraries.
I am currently using BCFtools after modifying the aligned BAM file (using python script) and do not end up with any alternative allele calls on bismark generated bam files? Can provide you the exact commands I am using if needed.
Cheers,
R

Hi, thanks for posting! Yes there is no reason it shouldn't work on Bismark alignment files, unless you are perhaps using the MD tags generated by default instead of redoing them with e.g. samtools calmd? Alternatively, you can give the genome fasta file and skip the MD tag usage.
Otherwise, to rule out any issue with BCFtools does the problem persist when you use alternative variant caller such as Freebayes?

I analysed bam files in two ways. First, provide reference genome fasta along with mapped bam file to your script. Second, perform samtools calmd on the aligned bam files and then use your script to modify the bam files.
I think the files which are newly sequenced (non-directional) have bad SNP calling compared to old libraries (directional).
I do not think issue is with BCFtools, as I am just doing genotype calling at provided target loci. Much simpler than standard SNP calling.

I am happy to share mapped bam files from bismark, if you want to do some diagnostics on that.

Something doesn't add up here. More modern libraries (e.g. MethylC-seq) are directional, whereas older BS-seq libraries non-directional, but in any case once they are aligned it should not make a difference unless something has gone wrong during the alignment. Are you certain everything is working correctly during this step? What do your mapping statistics look like?

Yes I think the bismark alignment was fine. Here is the alignment report
Bismark report for: H7MWJDSX3_1#188467_AATCCAATTGCATGTAGAGG_1_val_1.fq.gz and H
7MWJDSX3_1#188467_AATCCAATTGCATGTAGAGG_2_val_2.fq.gz (version: v0.23.0)
Bismark was run with Bowtie 2 against the bisulfite genome of TAIR10_wholeGenome/ with the specified options: -q --score-min L,0,0.5 --ignore-quals --no-mixed --no-discordant --dovetail --maxins 500
Option '--non_directional' specified: alignments to all strands were being performed (OT, OB, CTOT, CTOB)

Final Alignment report
======================
Sequence pairs analysed in total:	7658821
Number of paired-end alignments with a unique best hit:	2518193
Mapping efficiency:	32.9%
Sequence pairs with no alignments under any condition:	4350420
Sequence pairs did not map uniquely:	790208
Sequence pairs which were discarded because genomic sequence could not be extracted:	0

Number of sequence pairs with unique best (first) alignment came from the bowtie output:
CT/GA/CT:	651956	((converted) top strand)
GA/CT/CT:	584233	(complementary to (converted) top strand)
GA/CT/GA:	603959	(complementary to (converted) bottom strand)
CT/GA/GA:	678045	((converted) bottom strand)

Final Cytosine Methylation Report
=================================
Total number of C's analysed:	93145547

Total methylated C's in CpG context:	2561668
Total methylated C's in CHG context:	1274057
Total methylated C's in CHH context:	5095582
Total methylated C's in Unknown context:	2

Total unmethylated C's in CpG context:	10967697
Total unmethylated C's in CHG context:	12153405
Total unmethylated C's in CHH context:	61093138
Total unmethylated C's in Unknown context:	0

C methylated in CpG context:	18.9%
C methylated in CHG context:	9.5%
C methylated in CHH context:	7.7%
C methylated in unknown context (CN or CHN):	100.0%

Bismark completed in 0d 0h 15m 18s

Okay, the mapping efficiency seems quite low, but how does this compare with the directional libraries? If this is WGBS and assuming 150 bp reads then I'm estimating something like 3X coverage on Arabidopsis genome, or am I mistaken? That doesn't seem enough for what you are looking to do.

Yes, you are right! I realised this is much lower than previous runs (below is for the old library). Shall have to check that with the facility again.
But not sure, why SNP calling would be affected here though (since it should be only mapped reads that matter).
Bismark report for: HHHY7DSX2_2#169198_TACGCTGCTAGATCGC_1_val_1.fq.gz and HHHY7DSX2_2#169198_TACGCTGCTAGATCGC_2_val_2.fq.gz (version: v0.23.0)
Bismark was run with Bowtie 2 against the bisulfite genome of TAIR10_wholeGenome/ with the specified options: -q --score-min L,0,-0.5 --ignore-quals --no-mixed --no-discordant --dovetail --maxins 500
Option '--directional' specified (default mode): alignments to complementary strands (CTOT, CTOB) were ignored (i.e. not performed)

Final Alignment report
======================
Sequence pairs analysed in total:	3054311
Number of paired-end alignments with a unique best hit:	2185442
Mapping efficiency:	71.6%
Sequence pairs with no alignments under any condition:	251526
Sequence pairs did not map uniquely:	617343
Sequence pairs which were discarded because genomic sequence could not be extracted:	14

Number of sequence pairs with unique best (first) alignment came from the bowtie output:
CT/GA/CT:	1083797	((converted) top strand)
GA/CT/CT:	0	(complementary to (converted) top strand)
GA/CT/GA:	0	(complementary to (converted) bottom strand)
CT/GA/GA:	1101631	((converted) bottom strand)

Number of alignments to (merely theoretical) complementary strands being rejected in total:	0

Final Cytosine Methylation Report
=================================
Total number of C's analysed:	95072549

Total methylated C's in CpG context:	3371890
Total methylated C's in CHG context:	890011
Total methylated C's in CHH context:	1115687
Total methylated C's in Unknown context:	3628

Total unmethylated C's in CpG context:	10146802
Total unmethylated C's in CHG context:	12628506
Total unmethylated C's in CHH context:	66919653
Total unmethylated C's in Unknown context:	59830

C methylated in CpG context:	24.9%
C methylated in CHG context:	6.6%
C methylated in CHH context:	1.6%
C methylated in unknown context (CN or CHN):	5.7%

Bismark completed in 0d 0h 9m 37s

Thank you for your help though :) much appreciate

No worries, glad to help. Yes I agree it's strange, as it also doesn't seem to me like enough coverage for mapped reads in the directional libraries either.
Since you mentioned you are looking at pre-selected target loci, could you estimate the coverage in each case over those regions specifically? Perhaps the sequencing depth distribution is such that those regions just happen to be adequately covered in the directional library, whereas not in the non-directional library? If it's below the range of 10-20X then you are unlikely to get good results.

I am closing this now as I think the issue is stale. Please let me know if there is anything else to be done.


Rework coding interview prep challenges

Many of the coding interview prep challenges, specifically the rosetta code and project euler challenges I believe, ask for a specific amount to be returned. e.g. "Your function should return 12345". We should rework all of them so the function takes an argument and returns something based on the argument. More description here, and there's a PR linked in that issue with an example of a challenge that was adjusted.
If anyone adjusts one of these challenges, just reference this issue. If a large amount of challenges get reworked, we can make a list of the ones remaining at some point. I would keep PR's to a single challenge at a time.
This can be a discussion for now to make sure we can go ahead and start accepting PR's.

So I don't disagree that these would benefit from more clarity - however:
Doing so would deviate from the original challenges - is that something we're okay with? What's the ultimate intent behind incorporating these in freeCodeCamp's curriculum?

The clarity of these challenges isn't what this issue is for. To try and be more clear, this would take the existing challenges and slightly restructure them so that they can't be passed by just returning a number. So that's the intent, to make it so you can't pass them with code like:
function() {
  return 12345;
}

So instead of having (hypothetical) instructions like "return the 10th number in the sequence" - they might say "return the
nth number in the sequence". And n would be passed to the function. Then, we could write tests like:
function(5) should return 123
function(10) should return 12345

instead of
Your function should return 12345

I'm not sure if deviating from the original rosetta or euler challenges is going to be a problem. I suppose that's something to discuss. I don't mind it, and I think this may have already been done to some of them.

Sounds like a good plan.  If we do change to instructions like "return the nth number in the sequence", I think it would help to add "where n is <= <the original number>'  Reason being, it would be frustrating to run into timeouts and fail when your algorithm is correct, if not quite fast enough.

I don't mind adding something like that to the instructions. Although, it kind of seems like that might be more on us to just not write tests that exceed the number. Either way works.

Thanks for working on these @gikf & @Ne0nWinds. I had a thought as I was reviewing one of these - perhaps we should include one test on each challenge with a random input so users can't see all the values that are being tested. The text might be function(n) should return the correct value and the test would check a value different from the other tests. This would be to help deter the problem described here.

I don't really know. As much as that would make a bit harder for somebody wanting just use results to fool tests, it won't stop from reading challenge code. Looking at the other side there's also possibility of somebody sincere being affected by it, if for some reason hidden test would hit some edge case. With solution that's matching visible tests that might be rare, but still possible.

@moT01 I was actually thinking the same thing! I can add it to my PR when I get a chance

@moT01 To add on to what @gikf said, for problem 461, I had a solution that was correct for low numbers, but got less accurate with high numbers. This would've been really confusing if the hidden test case was a higher number than the visible ones. I do like this idea, though. I think it would be better if one or two test cases were slightly randomized (but still visible). I'm not sure how we'd actually implement that

Thanks for the input. I feel like we should come to a consensus on it so we can be consistent moving forward. I could go either way. The safe play is probably to just not include a test like this, but I do like the idea - so I'm torn.
As for actually randomizing the test - we could randomize the test, but the text can only be static. So there would be the same concern there.

I'm personally not a fan of "hiding" the test input/output, and here's why:

First, it makes debugging more streamlined if a camper can see the input and expected result, vs having to jump through hoops to figure that out
Second, it makes things smoother to catch if we have a bug in our test case - "Your test expects the value of 25, but really it should be 26".
Third, it keeps in line with our general practise of transparency to include the input/output in the test text.

I do understand there is a concern that I, as a camper, could write a bunch of "if parameter is X, return Y" to pass the tests, but then I'm really only hurting myself, right?

Alright, I think that settles it. Let's avoid those tests for now then @Ne0nWinds & @gikf.

There's been some discussion about how to display mathematical equations on a few of the open PR's. I think we should come to a consensus on that, as well. I looked around and most challenges don't use inline code around math - a few do. I think moving forward we should use mathJax around all math equations and variables etc - and reserve the backticks for computer code. It looks like there's some options we can use to make mathJax display a little differently if we want to at some point. What do you all think?

Keeping math to MathJax and code to backticks sounds sensible, What about a case with lone number, as a part of sequence, which refers to both code and math?

@moT01 do you know of any files that currently use the MathJax?

I found a few - 81 (code - demo), 82 and 83 are similar. There are also some that have a comment, <!-- TODO Use MathJax -->.
more: 368, 377, 431, 450, 463, 471, 473
You can right click the math and pull up some options.
Some others use inline html like 6 (code - demo)

So the downside with MathJax (vs code blocks) is that it doesn't register in Crowdin with anything we could use to hide the strings or detect translations.
@RandellDawson was poking at it, and it looks like sometimes we wrap the math in <big> tags which we could use, but that's not a consistent standard we follow.

And the upside is that it looks right.
Do you think people will try and translate them if they aren't hidden? I see some numbers in the Chinese files that just stayed as 1, 2, 3, ...

I do think it's a possibility - we could rely on proofreaders to catch it, but it's nice to have extra validation in place.

Do you think people will try and translate them if they aren't hidden? I see some numbers in the Chinese files that just stayed as 1, 2, 3, ...

We have already seen some translations of code tag content where they did not make a different translation, but they spelled the name of the function with a different case.

I do like the idea math to MathJax and code to backticks as a general rule of thumb, but I have a few thoughts:

The math symbols in mathjax and the ones in backticks are completely different, so try to stay consistent with whatever you introduced the symbol as (Look at the phi symbol in the equations vs the one in gcd)

I think backticks are better for simple math equations (add/sub/mul/div), because these things can be directly translated to code
A few things may just have to be decided on a case by case basis depending on what's more readable / makes sense.

For localization, we need a way to validate these equations don't get change or at least if they do, a QA message prompts the translator and proofreader when reviewed.  One easy way to do that currently, is to nest all of the matJax stuff in big elements.  It is best if they are in there own separate paragraph.  This creates a special big context we can use to make them untranslatable.

Below 100 of Project Euler there's few challenges left that may not be as feasible or straightforward to change. Does anybody have any ideas for these challenges?

https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-33-digit-cancelling-fractions - Checking numbers with 1 digits in numerator and denominator wouldn't make sense. 3 digits would make challenge harder than original one.
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-34-digit-factorials - Only two numbers are fulfilling requirement.
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-44-pentagon-numbers - Challenge ask for minimized value of D, but that's actually already the first such value.
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-45-triangular-pentagonal-and-hexagonal - This one is already requiring argument and there isn't much room for adding more tests, except of example and answer there's just one other case (number 1) which could be added.
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-46-goldbachs-other-conjecture - Challenge already asks for smallest number matching requirements.
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-49-prime-permutations - As challenge description explains there's no such sequences from 1, 2 or 3 digit primes, and both sequences with 4 digit primes are used in example and answer.
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-60-prime-pair-sets
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-68-magic-5-gon-ring
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-90-cube-digit-pairs
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-93-arithmetic-expressions

Those above 100, I'd start with some basic cleanup, rather than full reworking. One reason is they are in worse state - i.e. function named in pattern eulerXXX, missing images in description, or some other mangling in description. Another reason is the difficulty significantly rising at that point, which makes full rework that much harder.
As the cleanup shouldn't be as demanding, I'd go with more challenges in single PR, rather than one challenge one PR, as it is for full rework. What do you think?

Those above 100, I'd start with some basic cleanup, rather than full reworking.

I would be fine with that @gikf 👍 There's 380 challenges higher than 100, I would try to limit PR's to maybe 25 or so - maybe 50 if they are looking easy to review. Make sure that we are keeping them as closely aligned as we can with what the original Euler challenge is asking for.
I'm not sure exactly what to do with the #'s <100 in question. I don't think there's a blanket solution - they will probably need to be handled case-by-case. Maybe we come back to them later.

There are also places with listed all answers, without the need to look for them one-by-one.

Update on this: Many of those PR's that made changes to Euler just made an improvement to them that partially helps with the issue here, but doesn't make the full adjustment. So the list that hasn't been changed is still quite large.

@moT01  @gikf  can you review my PR for problem 125?


Hugo: Add figure shortcode

Add figure shortcode
Add figure examples to image example page (/examples/basic/images/)
Add figure styling
Add example image

For https://linear.app/usmedia/issue/CUE-119

@myitcv I have taken it myself :)

@myitcv yes of course, I give you my permission to use the image!

Closed via 1342263f6c9a8f8277c409db6883e84dc8782ddc


[Release Planner Test Service] SDK Review - Introduction

New SDK Review meeting has been requested.
Service Name: Release Planner Test Service
Review Created By: Wei Dong
Review Date: 12/6/2022 2:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Java,
Description:
Detailed meeting information and documents provided can be accessed here

Cancelled by: Wei Dong


[DS-2044] HTML not stripped from Discovery search form in JSPUI

This PR add "org.dspace.core.Utils.addEntities()" for each input affected by possible XSS attack as reported in https://jira.duraspace.org/browse/DS-2044 and by Khalil Shreateh page blog. I found and fix also in "subject search" JSP a lack of input escape that permits users to inject script code. In addition i tried to manage input validation for the field affected by XSS into BrowseServlet just to manage error and fix the verbose output error showed on the client.

+1 Gave this some basic testing. Seems to work fine...all previous examples now fail

Also cherry-picked to the 5.x branch


 I have no idea when I subscribed to your email list, and I also don't understand why I'm not able to unsubscribe from the email list.
Expected behavior
At least, be able to unsubscribe from the newsletter.
Screenshots

Hey @emnsen sent it to you directly as saw you were interested in Open source so I thought you'd be interested.
Also at the end of the email i wrote: if this is no interest to you I am very very sorry, just let me know 🙇♂
Apologies one again for the inconvenience.


Legacy Certification Settings Section to be implemented

Need to port over the legacy certificate section on Settings page from the production

I was poking around this issue a bit @raisedadead.  Curious, you know if there API / reducers still work from what's in production or how I'd mock the legacy data?

@benschac, the legacy cert's data is available now. We just need to check if the user has completed the challenges and conditionally render the cert button.
let me know if you are interested in pairing for this.

@ahmadabdolsaheb I can pair up for this.

I’m around
Get Outlook for iOShttps://aka.ms/o0ukef

From: Niraj Nandish<EMAIL_ADDRESS>Sent: Thursday, March 21, 2019 1:30:03 PM
To: freeCodeCamp/freeCodeCamp
Cc: Benjamin Schachter; Mention
Subject: Re: [freeCodeCamp/freeCodeCamp] Legacy Certification Settings Section to be implemented (#35410)
@ahmadabdolsahebhttps://github.com/ahmadabdolsaheb I can pair up for this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/freeCodeCamp/freeCodeCamp/issues/35410#issuecomment-475326390, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACYxIyaRGlVKdfZ0v_Qjxds_H5U1s2_Eks5vY8GbgaJpZM4bQvIf.

@benschac, @Nirajn2311 I somehow missed your messages, sorry about that. However, I would really appreciate it if you could check out the functionality and see if it works as intended.

hi testing comment api post

Me too

Test

Hello mayur :p


(feature) Adding event source field

Requirements
Introduction
Evaluation event is used for statistics purposes.
The event can be published by the proxy, or from the provider.
Feature
Enrich event by adding it source field which represents the source which publishing the message:

PROXY: set by the proxy when evaluated
PROVIDER_CACHE: set by the provider when evaluated from cache

Tasks

server-side: Support it at server-side for receiving the optional field and enrich the statistics by it.
Proxy: set PROXY source by the proxy when evaluated
OpenFeature Providers: set PROVIDER_CACHE source by the provider when evaluated from cache.
This is for providers which implemented cache, currently GO and Java.

That's a good idea to add the source of the event in the format FeatureEvent and fill it when creating a feature event.
But I would prefer to go with this name:

SERVER set when evaluated in the relay-proxy
PROVIDER_CACHE: set by the provider when evaluated from the cache.

Hello! Saw this marked as a good first issue and I’d like to give it a try :)

Hey @andrebenedetti, this is all yours!
I'll assign it to you so you are sure that no one is working on it at the same time.
If you need any tips or info please let me know.

Took some time to get familiarized with the repository (which is awesome, by the way!) and started some work here https://github.com/thomaspoignant/go-feature-flag/compare/main...andrebenedetti:go-feature-flag:feature/add-event-source-field, in case you want to take a look. Will continue tomorrow after work
This is the first Go project I work on, so I'm trying to figure out why my formatter wants to reorder imports differently than what's in the repo :thinking: any clues? Sorry for such a basic question :laughing:

Hey @andrebenedetti which IDE are you using? I am using IntelliJ so maybe it has a different policy.
Anyway don't block your self for this, this is not super important 😉


Add the new metro map component to the preview trip planner

Adding and removing markers works. When all markers are removed, the map recenters itself.

I replaced the integers with atoms if that makes it easier to read.


Fix v5.10.x builds

hmm...
https://github.com/Perl/perl5/commit/9c3c380d813ff0a7072f01b5e0109b7e21b8d0fd

Bump to 5.12.0


Add Status consts

Clone of #226
Maybe the branch name was causing the CI to work improperly.

Closing as it was a temporary issue.


[stable/nginx-ingress] TF apply error no ServiceAccount when upgrading chart version

Describe the bug
I am experiencing errors when applying an update to an nginx ingress controller, changing chart version from 1.4.0 to 1.26.2 and 1.41.2 --- I've tried both. Error:
Error: rpc error: code = Unknown desc = no ServiceAccount with the name "public-nginx-ingress-backend" found
Version of Helm and Kubernetes: Helm 2.16.9, k8s cluster v1.14.10
Which chart: stable/nginx-ingress
What happened:

Initial state: a stable and healthy nginx ingress controller deployed with chart version 1.4.0.
Bumped version to latest 1.41.2. TF plan was healthy.
Errors occurred on TF apply, including documented pre-1.26 bug spec.clusterIP: Invalid value: ""
Implemented workaround (set controller.service.omitClusterIP and defaultBackend.service.omitClusterIP to true). TF plan still healthy after change.
One error remains on TF apply, Error: rpc error: code = Unknown desc = no ServiceAccount with the name "public-nginx-ingress-backend" found
Rolled back to 1.4.0 and repeated process with upgrade to 1.26.2 - same no ServiceAccount error occurred.

Note: the not-found service account actually exists. A kubectl describe serviceaccounts public-nginx-ingress-backend showed it had been created by the chart.
What you expected to happen:
terraform apply applies successfully and updates nginx ingress chart and resources.
How to reproduce it (as minimally and precisely as possible):

apply chart version 1.4.0
attempt to apply upgrade to chart version 1.26.2 or 1.41.2

Anything else we need to know:
I have been able to upgrade successfully by setting the flag force_update = true, which I understand tears down and recreates all chart resources. However I require a seamless, downtime-free upgrade path for applying to a prod environment.
In case it's relevant, other parameters set in the chart definition include:
  name     = "public-nginx-ingress"

  set {
    name  = "controller.minAvailable"
    value = 3
  }

  set {
    name  = "controller.replicaCount"
    value = 5
  }

  values = [
    file("./redacted-dirs/nginx_values.yaml"),
  ]

  set {
    name  = "controller.service.externalTrafficPolicy"
    value = "Local"
  }

  set {
    name  = "controller.service.loadBalancerIP"
    value = google_compute_address.nginx-ip.address
  }

  set {
    name  = "controller.service.omitClusterIP"
    value = true
  }

  set {
    name  = "defaultBackend.service.omitClusterIP"
    value = true
  }

  set {
    name  = "controller.ingressClass"
    value = "nginx-public"
  }

  set {
    name  = "controller.service.enableHttp"
    value = false
  }

stable/nginx-ingress has been deprecated in favor of the upstream ingress-nginx repo. See https://github.com/helm/charts/pull/22823. Issues if still applicable can be re-opened there. Thanks 😊


nixos-option doesn't respect NIX_PATH ?

Describe the bug
NIX_PATH seems to be ignored?
To Reproduce

NIX_PATH=nixpkgs=/home/arian/Projects/nixos/nixpkgs nixos-option
terminate called after throwing an instance of 'nix::ThrownError'
  what():  file 'nixos-config' was not found in the Nix search path (add it using $NIX_PATH or -I), at /home/arian/Projects/nixos/nixpkgs/nixos/default.nix:1:60
Aborted (core dumped)

Expected behavior
Pointing to my fork of nixpkgs using NIX_PATH should work.  The application shouldn't coredump
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Duplicate of #86120

Thanks @veprbl .  It partially is. The second problem is that NIX_PATH isn't respected and the application segfaults. I'll rename the title to reflect hat

cc @chkno

nvm; I also had to add nixos-config to the NIX_PATH. this is a PEBKAC


Enforce optimizations in OpenSSL and libSRTP with cflags

Description
This PR adds compilation flags to openssl and libsrtp builds that would theoretically optimize these libraries and save CPU.
[] It needs and includes Unit Tests
Changes in Client or Server public APIs
Not needed.
[] It includes documentation for these changes in /doc.

this does not make sense anymore as we saw these flags are used by default


feat: allow mutable access to DataMut/HeadroomMut

This PR adds a contents_mut method to both of the above types
to allow in-place modification of packet buffers (i.e., parsing
existing data with another library like pnet while also modifying
packet fields). This also adds {As, Borrow, Deref}Mut to make
their use a little bit more ergonomic.
This allows for, e.g., simple mac-swap testing where the received
buffer is changed in-place over the RxQueue, before being immediately
sent out over TxQueue. This likely also enables more complex packet
processing to be done without forcing a full packet copy.

Makes sense, thanks for the PR.


Build App for Linux on MacOS

I run command npm run package-linux. Then see error on terminal:
Error: Exit code: null. Command failed: /Users/quocha/Library/Caches/electron-builder/appimage/appimage-9.0.5/darwin/appimagetool --runtime-file /Users/quocha/Library/Caches/electron-builder/appimage/appimage-9.0.5/runtime-x86_64 --no-appstream /workspace/Software/app/release/__appImage-x64 /workspace/Software/app/release/electron-react-boilerplate-1.0.0-x86_64.AppImage
dyld: Library not loaded: /usr/local/opt/gettext/lib/libintl.8.dylib
  Referenced from: /Users/quocha/Library/Caches/electron-builder/appimage/appimage-9.0.5/darwin/appimagetool
  Reason: image not found

dyld: Library not loaded: /usr/local/opt/gettext/lib/libintl.8.dylib
  Referenced from: /Users/quocha/Library/Caches/electron-builder/appimage/appimage-9.0.5/darwin/appimagetool
  Reason: image not found

Thank you for seen, Can you help me?

@amilajack I too am experiencing this issue while trying to execute npm run package-linux. Could you please help us out in approaching this issue? Thanks in advance

This is an electron builder issue. I would recommend moving this issue there


PHP-based search

I don't feel entirely comfortable leaving a navigational function as important as search in the hands of JavaScript. It's the kind of thing that can just... break.
A PHP-based search plugin and the ability to customize a search results page would be much appreciated. Again, I know this is another big ask... but as long as we're all just making feature requests... you know?

Yes, I thought the same originally, but when I created the plugin I searched for a php-search-library and there is no such library that

is lightweight
does not care if files ore database
is supported and regularly updated
is widespread

Lunr is all of this so for me it is the only choice for a project like typemill. Maybe someone has other experiences and knows a lightweight php library for it.

Excited about the improvements, if useful for all themes I will migrate to the plugin. Another idea was to open the results in a separate page, but the problem then is the styling according to the theme, so there is no really super good solution for it all yet ...

Oh, I wouldn't worry about me making "improvements". I mean, I did make a couple of accessibility changes to the HTML form, but the biggest changes came from me removing all the default styling and moving the actual CSS implementation to my theme so I could make it look exactly how I want.
While that might prove useful to some people, it's not exactly an improvement.


SEP-19 : Refactoring to remove localityManager from JobModel

This PR removes localityManager from JobModel.
There was some localityMapping creation location in JobModel that was based on localityManager being null.
That has, for now, been moved to JobModelManager.
A subsequent PR will completely get rid of that null-based logic.

@rmatharu Let's get this checked in once you have some spare time.

@rmatharu We also want to move the JobModel to samza-api (where ContainerModel and TaskModel already are).

@rmatharu do you still plan to check this in?

yes, most of the comments are addressed but i need to rebase on master and send a revision.

@rmatharu : could you please create a JIRA ticket for this change in samza open source and associate the pr with the jira (setting the subject SAMZA-XXXX:...)? It would be much easier for the release person to figure out what got into the release with a JIRA ticket. SEP shouldn't be used as the PR titile.
Please also read "Contribute using Pull Requests" in Samza contributor corner (https://cwiki.apache.org/confluence/display/SAMZA/Contributor's+Corner) to get familiar with expected Samza contributor flow. Thanks.

@rmatharu : Could we please get this rebased so it can be checked in?

This refactor along with moving JobModel to samza-api are committed as part of #1421 . Closing this PR


I can't print the full page when VuePdfComponent is mounted! (Maybe related with PDF.js)

Hi! First of all, thanks for the great package.
I've installed the version 2.1.0 via NPM and tried to put the VuePdfApp inside a Bootstrap Modal, so users can read multiple PDF files downloaded from the server.
It was working good until some user report the need to print the hole page that contains the Datatable they use to access the files. They like to make an kind of report with it.
Well, its just like this: When I trie to print the whole page once the component has no condition on it, the Print Server returns me "Warning: PDF is not fully loaded to be printed.".
And the console shows that the browser expects the print service to be initialized.
Once I started using v-if directive on the VuePdfApp component, I was able to get a window.print() or document.print() when Ctrl + P.

I am able to see the first opened document... But the next ones give me this errors and they are not shown on the component.

This is my vue component.
ModalPDFReader - Copia.txt
What do u recommend me to do?
And another thing: Can increment type="button" to the front end buttons so they can prevent page reload? Thanks!

Hi!
Any solutions to this?

Hi, in progress

@EspetoRx:

Yes, you have to use v-if directive to make non-pdf pages printable. PDFjs internally creates printContainer and css rules for printing content only inside printContainer.
<link rel="resource" type="application/l10n" :href="wLocationHost+'/css/pt_br/viewer.properties'"> must be situated inside head tag according to HTML specification. You can use relative path <link rel="resource" type="application/l10n" href="/css/pt_br/viewer.properties'">. Errors in your console usually occur when <link rel="resource" type="application/l10n" is corrupted. You have to implement entire locale file even if you don't need all buttons.  Please provide your /css/pt_br/viewer.properties.

And another thing: Can increment type="button" to the front end buttons so they can prevent page reload? Thanks!

I didn't understand what is needed. Do you need buttons of pdfjs to be of type button?
4. The most likely reason of the issue: vuePdfApp is rendered (based on enablePdfReader changed in transition enter) BEFORE a dialog is visible (which is rendered based on prop modalOpen). That is why offset parent is not set error may occur. Try to use v-if="modalOpen" for vuePdfApp. If it doesn't help (I think it will help) try to use v-if="eneblePdfReader' with watcher:
watch: {
    modalOpen: async function () {
      // to make a dialog rendered in DOM
      await this.$nextTick();
      if (this.modalOpen == true) {
        this.enablePdfReader = true;
      }
    },
  },

Hi!
Since I just needed a PDF Viewer I changed the vue-pdf-app to a HTML 5 object tag...
Thanks for the response and the great package.
Keep the greatness!

I did a CodeSandbox and it worked pretty fine for links on the internet...
https://codesandbox.io/s/bold-darkness-tfbpp?file=/src/main.js
Maybe is the way I download the PDF files from server and create the links... Their link are shown as ['blob']https://.... Maybe is this, cause the way you provided worked pretty well...

I discovered what was causing trouble with my file openings and it wasn't the blob link. Was a SWAL 2 toast... I should do it before open the modal, and not after.
Living and learning. Feel free to use the CodeSandbox showing the package opening in modal on your readme.md.
And once again thanks for the great package. It worked like a brease.

Thank you very much!

No, its me who have to thank you for the great package. Thank you so much.

Hi! I'm back again. I'm sorry to comment here after closed but it was the localization file...
I didn't realized that it was missing. When i put it back those errors started again. I wasn't able to reproduce it on CodeSandbox. It might be the way I download the files... Anyway... Is there another way I can localize the pdf reader?

Hi!

It might be the way I download the files.

It might be but it is unlikely. Remove pdf prop for vuePdfApp just to check if it works and exclude a way of file downloading resulting the error.
Move link tag with locale inside head tag.
Provide your locale file.
Provide source code of parent component for component from your initial example.
You can build your own toolbar using this example.Localization is not needed in that case


Use trusty for travis jobs.

The distro was set to "precise" in #4572 due to memory issues on trusty,
but we've been seeing performance issues on "precise" recently so let's
see how trusty is working these days.

Trying this out since I've seen a lot of CI jobs die today with timeouts on BufferHashGrouperTest. It does a lot of random access on an mmapped file, so in mem-constrained environments it can take a really long time (which precise might be? who knows?). On my laptop it takes 30s but in Travis it has taken >10 min.
I got the idea to try trusty from this message on https://travis-ci.org/druid-io/druid/jobs/272275221:

This job ran on our Precise environment, which is in the process of being decommissioned. Please read about the status of the rollout on the blog (https://blog.travis-ci.com/2017-08-31-trusty-as-default-status), and take note that you can explicitly stay on Precise by specifying dist: precise in your .travis.yml.

Maybe an environment that is being decommissioned can be expected to be more resource constrained than normal.

The build succeeded. I just restarted it to see if it'll succeed again.

LGTM. Thanks!


Update changelog for release 18.6.1

Sorry been released already
On Tue, Jun 19, 2018 at 1:18 PM Patrick Flynn<EMAIL_ADDRESS>wrote:

@patflynn commented on this pull request.
In CHANGELOG.md
https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/2180#discussion_r196509965
:

@@ -1,6 +1,13 @@

Release notes
This page documents production updates to Cloud Tools for IntelliJ. You can check this page for announcements about new or updated features, bug fixes, known issues, and deprecated functionality.
+## 18.6.1
+
+### Fixed

Fixed errors in the managed Cloud SDK cause by downgrading the version. 666

I'd be more specific about what behavior we specifically fixed. We really
want to call out to users that experienced these installation failures that
they're fixed.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/2180#discussion_r196509965,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABp8QCLT0C2kxmmBM4L_bdSAHmqCMkTgks5t-TJkgaJpZM4UttBE
.

I can be more specific of the release notes attached to the GitHub release though


Add storage live migration support to direct DirectVolumeMigration

Add prometheus progress reporting to VM Live Migration
Validate the plan if the live migration checkbox is set, ensure that the source and destination cluster have a valid kubevirt version and configuration.
Allow canceling of migration.
This includes aborting live migrations and
cleaning up rsync pods if they are still running.
Added a finalizer to the DirectVolumeMigration
which is only removed once everything is cleaned
up properly.
Properly handle rollback of direct volume migrations that include VMs. For both running and stopped VMs as well as live migration of running VMs, or quiesce and starting them after migration.

/hold
Need to update the crane-lib after a new release is made when https://github.com/migtools/crane-lib/pull/125 is merged

/hold cancel updated the library last week.


Cant get on

Sadly, I'm using bluestacks so that may be why but why is it saying "Error getting game path: Could not find a part of the path 'C:\Users\shygu\AppData\Roaming\dmmgameplayer5\dmmgame.cnf'".

Hi!
Yes, the english patch is only available for the DMM version of the game, and this installer will only wotk with that.


Added You may now proceed to the next lab

6882
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[X] All filenames are lower case (including folders, images, files, etc.)
[X] Filenames are descriptive
[X] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[X] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[X] Image references in markdown contain an alternate text

@alexandruporcescu Please check the conflict in data-catalog/harvest-object-store/harvest-object-store.md


Compiling DCMTK shows a Mac file format warning for arith.h

Description
When compiling ITK with the IODCMTK third-party module, the compiler shows a warning saying that the file
config/include/dcmtk/config/arith.h is generated with Mac file format, and that it should be converted to DOS/UNIX format.
Steps to Reproduce
Clone ITK at the specified commit (or master) and compile with Module_ITKIODCMTK set to ON.
Expected behavior
No warnings when compiling.
Actual behavior
config/include/dcmtk/config/arith.h is generated with Mac file format.
itk-Bin/Modules/ThirdParty/DCMTK/ITKDCMTK_ExtProject-build/config/include/dcmtk/config/arith.h(1,1):
warning C4335: Mac file format detected: please convert the source file to either DOS or UNIX format

Reproducibility
100%
Versions
ITK master at commit 69b7c4ef036ce36b32ace738db111d3d4720dbe4
Environment
OS: Windows 10
CMake: 3.20.3
Compiler: MSVC 2019 19.29.30133.0
Additional Information
Compiling the DCMTK library as a separate/system project at the pointed tip shows the same warnings.
I tried to find where in the upstream repository the arith.h file is generated. It looks like it is here:
https://github.com/InsightSoftwareConsortium/DCMTK/blob/master/config/Makefile.in
But I did not find why a Mac format file is being generated.
I also see that the DCMTK version used by ITK is quite behind the upstream version.

This is still relevant stale bot.


Add Torquay Academy

URL to the website: https://www.tqacademy.co.uk
This school has computer science courses at the Secondary and College level (Level 1-3 UK RQF)
All staff and students at the school have a email ending in @tqacademy.co.uk (e.g<EMAIL_ADDRESS>

@Jettford Pull request merged. Thank you!


Scripts is moving to body after save

I found a bug with moving scripts to body,
You have file - Swissup\Breeze\Model\Filter and method process
I created experiment and commented this code
      foreach ($this->domFilters as $filter) {
           $filter->process($document);
       }

I've set the HTML to a DomDocument object and then back to HTML, expecting no changes. However, after this process, the HTML structure was altered. Could you help fix this issue?

Can you please attach the HTML content that this function receives?

Hi, i can atach only part of header and a start of body where problem was observed.

To fix this bug, I need to reproduce it first. To do so, I need HTML that causes this issue. Can you replace all sensitive information with dummy placeholders and post the HTML content?


Update

I added publishers and subscribers to track the state of the current interaction and modified the content to manage multiple audio recorders. I also updated the interaction manager tests accordingly.

Merging now due to time constraints!


Documentation overlay only cares about  tags

Steps to reproduce

Write an XML doc comment with more than just <summary>
Hover over the target of the doc comment
See that only the <summary> tag is shown.

Expected  behavior
All the tags are parsed out and displayed on hover.
Actual behavior
Only the <summary> tag is cared about.
Additional context
The file that handles the documentation overlay only cares about <summary> and nothing else. This file has not been touched in just shy of three years.

Duplicate of #5063


[move-prover][paper] Submission version

Minor changes reflecting what has been submitted.

Read some references and restructured related work
Added discussion of novelity of our three transformations

Motivation
Paper
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
NA

:broken_heart: Test Failed - ci-test


Always show owner_id line if user opts to show observer preference

Avoids ambiguity/confusion about missing line

If no clear owner preference, line says "no clear preference"
If consensus = observer preference, shows that name

Other changes to more clearly state intent of option and code
"Observer ID" => "Observer Preference"
Re-order and re-name some methods and variables
I want to merge these changes before working more on this branch

Manual test script (essentially duplicated in ObserverControllerTest)

While logged in, goto Preferences, check "View Observer’s Preferred ID?", Save Edits
Click Create Observation, Fill in form with:

Where: any existing location
What: leave blank
Confidence: Could Be
Create

Observation should have Observer Preference line, which says "no clear preference"

Propose a name:  Coprinus comatus, Confidence: Could Be

Result:

Propose another name:  Agaricus, Confidence: Could Be

Result:  no clear preference (because of tie vote)

Destroy Proposed Name Agaricus;  Edit Proposed Name  Coprinus comatus Confidence: Doubtful

Result:  no clear preference (because confidence is negative)

Change Preferences to uncheck "View Observer’s Preferred ID?", Save Edits,
In browser go back to Observation and refresh
Result: no Observer Preference line

I will fix test Failure.  But first I will open an issue.  (When I test locally everything passes, but that's a bug.)

Tests passing.

Looks good to me.


[hacktoberfest] - Crop Options documentation

The library has many many options, is necessary that we can have a clear text explaining what each method does with samples when need, etc

Until next year


[Bug]: BinaryFormatter.Serialize in Copy-psObject causes exception to be thrown when running under .NET 8

What happened?
When I run Monkey365 using .NET 8, several exceptions are raised. The error message is:

[Copy-psObject] - System.Management.Automation.MethodInvocationException: Exception calling "Serialize" with "2" argument(s): "BinaryFormatter serialization and deserialization are disabled within this application. See https://aka.ms/binaryformatter for more information."

This is a breaking change in .NET 8
https://learn.microsoft.com/en-us/dotnet/core/compatibility/serialization/8.0/binaryformatter-disabled

Previous behavior
In .NET 7, the BinaryFormatter.Serialize(Stream, Object) and BinaryFormatter.Deserialize(Stream) methods were marked obsolete and raised an error at compile time. However, if your application suppressed the obsoletion, it could still call the methods and they functioned properly in most project types (excluding ASP.NET, WASM, and MAUI). For example, the APIs functioned correctly in a console app.
New behavior
Starting in .NET 8, the affected methods throw a NotSupportedException at run time across all project types except Windows Forms and WPF. The APIs continue to remain obsolete (as error) across all project types, including Windows Forms and WPF.

How to reproduce it
Make sure your instance of Powershell is using the .NET 8 runtime:
user@debian12$ pwsh
PowerShell 7.4.0
PS /home/user> [System.Runtime.InteropServices.RuntimeInformation]::FrameworkDescription
.NET 8.0.0

then import and invoke the module in the usual way. The exceptions will be logged.
Expected behavior
No exceptions
Screenshots or Logs
[14:52:21:537] - [Copy-psObject] - System.Management.Automation.MethodInvocationException: Exception calling "Serialize" with "2" argument(s): "BinaryFormatter serialization and deserialization are disabled within this application. See https://aka.ms/binaryformatter for more information."
 ---> System.NotSupportedException: BinaryFormatter serialization and deserialization are disabled within this application. See https://aka.ms/binaryformatter for more information.
   at System.Runtime.Serialization.Formatters.Binary.BinaryFormatter.Serialize(Stream serializationStream, Object graph)
   at CallSite.Target(Closure, CallSite, Object, Object, Object)
   --- End of inner exception stack trace ---
   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame). LineNumber:  - exception - debian12 -

From where are you running Monkey365?
Please, complete the following information:

Resource: Workstation
OS: Debian 12 Linux
PowerShell Version:

PS /home/user> $PsVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.0
PSEdition                      Core
GitCommitId                    7.4.0
OS                             Debian GNU/Linux 12 (bookworm)
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Monkey365 Version: Monkey365 v0.91.1-beta

Additional context
I am using the PowerShell snap for Linux - the full package has not been fully tested on Debian 12

Hi @dom34,
Good catch! And thanks for submitting this issue, I'll fix it shortly.

Fixed in main branch.
Cheers,


When loading KML documents we will spread out loading to prevent freezing

We load the first 128 features, then give up time to the render loop. This is using setTimeout with a value of 0 since we don't have a modern promise library.
Running some tests, it was slightly slower but prevented freezing. With a 51MB KMZ file the loading went from 18s to 23s, but the overall hanging went from 18s to a single 3s freeze.

Thanks for the pull request @tfili!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

:tada: :tada: :tada: :tada: :tada: :tada:

The code changes look fine to me.
Loading the KML examples in sandcastle took noticeably longer to render in this branch.  I can see that this is a win for very large KML files with lots of features, but would we have a way to only do the deferred loading once the file size or number of features is larger than some threshold? Why did you pick 128?  Maybe make that value larger?

Is this meant to replace #3970?

Loading the KML examples in sandcastle took noticeably longer to render in this branch.

I didn't see that in my testing. The times were almost the same. The GDP one was even faster. The facilities was slightly slower.

Why did you pick 128? Maybe make that value larger?

Larger values made the mouse movement more choppy and smaller values didn't really help much.

The facilities one was the one that was the most noticeable.  In master it loads almost instantly.  In this branch it took a few seconds.

With a build version of the app in incognito mode, in master facilities loaded in 800ms and in this branch it was 1200ms. The GDP kml was near identical (master was less than 100ms faster, which may be noise) and the bike race document was actually faster in this branch.
Upping the number from 128 will make them closer, but with larger documents it will increase jumpyness, but it will still be interactive to a point. The size of the file isn't a very good measure. A lot of documents have full HTML pages embedded in them that show up in the info bubble for a feature, which can make them giant but with only a few features and facilities is like 600KB with a crap ton of features. I guess it comes down to if we care about small documents being a little slower to load or not. I have some more mid-range sized files I can test with to see how changing that number effects them. I was testing with a pretty large document.

Okay, that sounds fine to me @tfili.  Thanks for taking the time to get some more numbers.
Someone else should take a quick look at this, but it's okay with me to merge =)  @mramato do you have a minute to check it ou?

Here is the loading time comparing master, branch with 128 entity limit and branch with 512 entity limit. The 512 one seems more jumpy and master freezes. This is the times for just the parsing of the KML file. Times don't include KMZ decompressing and creation of graphics resources. Times in ms.

File
master
128
512

1.8MB KMZ (13.7MB KML)
6376
7611
5541

5MB KML
2304
3146
2533

9MB KMZ (38MB KML)
2219
3351
2174

48MB KMZ (182MB KML)
14523
16714
15400

This is updated to now use time instead of the number of features since they can vary pretty widely.  This allows smallish documents to load as fast as they did before and larger documents to load in reasonable time.

Update CHANGES.md

Done

I think that's all I have @tfili while I still hold out hope for a shared time-slicing system in CesiumJS down the line, this is a decent incremental change and should be easy enough to tweak as needed if we see any unwanted behavior.  The 1 second load before slicing seems like a great idea that should work well in practice as well.
I still wish this (optionally) tied into the render loop somehow, but that could be part of the larger time-slicing system (one day).

This is supposed to supersede https://github.com/AnalyticalGraphicsInc/cesium/pull/3970 right?

Yes, thanks/.

Was this also done for GeoJSON?