[camera_avfoundation] Occasional crash when recording with camera

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Run test code in debug mode
It will show some message and then exits, continue to step 1 until crash happens ("Lost connection to device." is ok)

Alternatively run this in terminal, it will stop when crash happens:
while flutter run -d your_device_id_or_name --no-build; do; done

It may occasionally stop without triggering a crash with Error launching application on .... or iphone may lock screen so it would be optimal to adjust auto-lock timeout.
Reproduction of https://github.com/flutter/flutter/issues/69874.
Unfortunately this is really hard to reproduce. It may be easier on older devices like iphone 7 plus. It usually takes up to 10 runs to reproduce on my device but on newer and faster devices it could take much much more. I was not able to reproduce it by using hot restart.
Unfortunately it seems --no-build has no effect so it can take a really long time to trigger this.
As that error message says appendPixelBuffer should not be called when readyForMoreMediaData is false so it should first check readyForMoreMediaData and only when it is true call appendPixelBuffer similarly as is done with appendSampleBuffer. appendPixelBuffer probably directly calls appendSampleBuffer and will not do this for you. Also I do not see any reason why AVAssetWriterInputPixelBufferAdaptor is used there at all. I only ever used it when I appended samples which I rendered myself. It provides pixelBufferPool which can be used with CVPixelBufferPoolCreatePixelBuffer. It just provides a pool of pixel buffers into which you can render and pass to appendPixelBuffer. Here in FLTCam every sample buffer you get in captureOutput is already provided to you so there is no use for pool provided by AVAssetWriterInputPixelBufferAdaptor and no benefit for using it and you could just simply use appendSampleBuffer as well.
When I add some logging before appendPixelBuffer:
NSLog(@"%i %li %li %li", _videoWriterInput.readyForMoreMediaData, nextSampleTime.value, nextSampleTime.timescale, nextSampleTime.epoch);

1<PHONE_NUMBER>8333<PHONE_NUMBER> 0
1<PHONE_NUMBER>6125<PHONE_NUMBER> 0
0<PHONE_NUMBER>8833<PHONE_NUMBER> 0
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '*** -[AVAssetWriterInputPixelBufferAdaptor appendPixelBuffer:withPresentationTime:] A pixel buffer cannot be appended when readyForMoreMediaData is NO.'

And with this it no longer happens (still no crash even after more than 100 runs):
      if(_videoWriterInput.readyForMoreMediaData) {
        [_videoAdaptor appendPixelBuffer:nextBuffer withPresentationTime:nextSampleTime];
      }

Expected results
No crash.
Actual results
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '*** -[AVAssetWriterInputPixelBufferAdaptor appendPixelBuffer:withPresentationTime:] A pixel buffer cannot be appended when readyForMoreMediaData is NO.'
libc++abi: terminating with uncaught exception of type NSException
* thread #6, queue = 'io.flutter.camera.captureSessionQueue', stop reason = signal SIGABRT

Code sample
Code sample
import "dart:io";
import "package:flutter/material.dart";
import "package:camera/camera.dart";

void main() {
	runApp(const MyApp());
}

class MyApp extends StatelessWidget {
	const MyApp({super.key});
	@override Widget build(BuildContext context) => const MaterialApp(home: MyHomePage());
}

class MyHomePage extends StatefulWidget {
	const MyHomePage({super.key});
	@override State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
	bool loading = true;
	String where = "";
	late CameraController camera_cont;

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

	@override void initState() {
		super.initState();
		init_camera();
	}

	init_camera() async {
		final cameras = await availableCameras();
		camera_cont = CameraController(cameras.firstWhere((camera) => camera.lensDirection == CameraLensDirection.back), ResolutionPreset.ultraHigh);
		await camera_cont.initialize();
		await camera_cont.startVideoRecording();
		setState(() => loading = false);
		await Future.delayed(const Duration(milliseconds: 100));
		final file = await camera_cont.stopVideoRecording();
		await File(file.path).delete();
		setState(() => where = "Sorry, nothing this time, please try again");
		await Future.delayed(const Duration(milliseconds: 3000));
		exit(0);
	}

	@override Widget build(BuildContext context)
	{
		if(loading)return const Center(child: CircularProgressIndicator());
		else return Stack(
			alignment: Alignment.center,
			children: [
				CameraPreview(camera_cont),
				Text(where, textAlign: TextAlign.center),
			],
		);
	}
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.5 22G74 darwin-x64)

I attempted to reproduce this on my iPhone XS but I did more than 10 re-runs and I couldn't seem to reproduce the issue. I always got Lost connection to device, logs shared below.
Labeling for further investigation

logs
Launching lib/main.dart on Nexus in debug mode...
Automatically signing iOS for device deployment using specified development team in Xcode project: X8NNQ9CYL2
Xcode build done.                                           43.0s
(lldb) 2023-08-08 10:26:12.528172+0000 Runner[627:30869] [VERBOSE-2:FlutterDarwinContextMetalImpeller.mm(37)] Using the Impeller rendering backend.
Warning: Unable to create restoration in progress marker file
fopen failed for data file: errno = 2 (No such file or directory)
Errors found! Invalidating cache...
The selected imageFormatGroup is not supported by iOS. Defaulting to brga8888
[client] No error handler for XPC error: Connection invalid
Lost connection to device.

Exited.

So if I am reading this correctly, it sounds like the fix is to add this to FLTCam.m?
if(_videoWriterInput.readyForMoreMediaData) {
        [_videoAdaptor appendPixelBuffer:nextBuffer withPresentationTime:nextSampleTime];
}

I've got a similar issue with one of my applications that centers around recording video. I will try and use a custom fork with this change in my project and see if that alleviates the problem

+1 on this bug. I have an Iphone 13 Pro running iOS Version 17.2.1 and I pretty much can't get more than 3-5 seconds into a recording before it crashes the entire app when running in debug (can consistently repro).
When using release mode it seems to be working fine, so I think there is something with the way the debugger and the camera play together that is causing this problem.

@ob-juliandixon is the fix you have above working? and any reason why your going the fork route instead of creating a PR against the main repo?

@bgenidy

When using release mode it seems to be working fine, so I think there is something with the way the debugger and the camera play together that is causing this problem.

It is probably because in debug almost everything is slower so it has a harder time to append pixel buffers and there is a higher chance it will be not ready. I would not assume that it is working correctly in release, there is just much lower chance for it to happen.

is the fix you have above working? and any reason why your going the fork route instead of creating a PR against the main repo?

It is much less work (btw I am planning to do PR).

Possible related to https://github.com/flutter/flutter/issues/69874


MM-46003 Update webpack-cli to 4.10.0

This fixes an issue where webpack-dev-server suddenly decided to stop working
Ticket Link
https://mattermost.atlassian.net/browse/MM-46003
Release Note
NONE

webpack is running, however with the code on this branch, it fails with something related with the generic_modal components migration (even when running with make run).
ERROR in ./components/generic_modal.tsx 12:0-54 Module not found: Error: Can't resolve '@mattermost/components' in '/Users/pvelez/Documents/projects/mattermost/mattermost-webapp/components'

@pvev Did you try running a clean as well? That PR moved some stuff and started using a new workspace, so it’s possible that it wouldn’t work without a clean. The test server works, and this shouldn’t make anything worse for you, so I’m going to merge it now.
Also, I did see an issue where editors will fail to find the components package, but I’ll fix that separately. The build tools should still be able to find it regardless of that

@pvev Did you try running a clean as well? That PR moved some stuff and started using a new workspace, so it’s possible that it wouldn’t work without a clean. The test server works, and this shouldn’t make anything worse for you at least.
Also, I did see an issue where editors will fail to find the components package, but I’ll fix that separately. The build tools should still be able to find it regardless of that

awesome!, let me clean everything and try again. Thanks @hmhealey

the make nuke just did the trick, thanks @hmhealey!


fix: forward Ref prop on mount

fix #192

Hmm interesting fix! I guess it doesn't break anything, although I have a sneaking suspicious this will break some weird edge cases. We'll see.


Add abort trapping

Need to add abort trapping when using the I2C library to keep things from hanging if there is a problem.

Was addressed in earlier pull request cleaning code.


Unable to connect to Azure SQL Server  with Privatelink Endpoint

Hello All,
I have created a Private Link for one of the azure sql server and selected a vNET which has S2S connectivity.  I set Deny puclic access  as YES . When i try to access the DB from SSMS i am getting follwing error.  Could you please check and get me a resolution for this.
A connection was successfully established with the server, but then an error occurred during the login process. (provider: SSL Provider, error: 0 - The target principal name is incorrect.) (.Net SqlClient Data Provider)

@srisaiazure  Thank you for your feedback! Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team.
The best place for specific answers about implementations are going to be the Microsoft Q&A and Stack Overflow forums. We actively look through posts in the forums, and will do our best to get you going in the right direction!

Just checking in. Could you please provide the relevant documentation so we can route this question appropriately?

Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Defined Functions

Hello Brook,
I do not know if this is by design, but unless I have missed a step, we go from the "functions" module to the "modules" module without having introduced named functions.
It seems that how to define modules and how to define functions are concepts that are introduced simultaneously, while in the "functions" module there is only "anonymous functions" at work?
Again, this may be by design, but reading at the beginning of the module it seems that knowing how to define functions should be known at this point:

Although a bit later in the text, it is explained for the first time how to declare a named function:

Would it make sense to separate these three topics in three modules?

Anonymous Functions
Named Functions
Modules

Hope it helps.
Jon

Named functions must always be in a module so I'm not sure how to split them 🤔 maybe there's an approach but they seem like they need to be covered at the same time.

I'm going to close this issue as I don't think it's actionable, but please re-open it if you disagree!


Composite Keys don't generate ref statements

With a table defined in the schema as so:
export const warehouseOffices = pgTable(
	'warehouse_offices',
	{
		officeId: integer('office_id')
			.notNull()
			.references(() => offices.id),
		warehouseId: integer('warehouse_id')
			.notNull()
			.references(() => warehouses.id),
	},
	(table) => {
		return {
			pk: primaryKey({ columns: [table.officeId, table.warehouseId] }),
		};
	}
);

The generated .dbml doesn't include any refs for this table:
table warehouse_offices {
  office_id integer [not null]
  warehouse_id integer [not null]

  indexes {
    (office_id, warehouse_id) [pk]
  }
}

I need to add these lines to the resulting .dbml:
ref: warehouse_offices.warehouse_id - warehouses.id
ref: warehouse_offices.office_id - offices.id

What version of the ORM are you using? Do you know if this worked in a previous version?

Hey @L-Mario564  i had this issue too
Wanted it fast so i changed your index.cjs to use this
var Columns = Symbol.for("drizzle:Columns");
const pkColumns = wrapColumns(Object.values(index.table[Columns]), this.buildQueryConfig.escapeName);

Also the references are missing the schema's name which you can do it like this
sourceTable: `${relation.sourceTable[Schema]}.${relation.sourceTable[TableName]}`,
foreignTable: `${relation.sourceTable[Schema]}.${relation.referencedTableName}`,

your escapeSpaces will introduce quotes to {schema}.{tableName} after adding the schema
so i changed the regex to /\s/
didn't open a PR because i just tested for my own situation but it may help you to handle this
currently using "drizzle-orm": "^0.30.4" this version for drizzle-orm


Fixed issue with Wild Events Master Data page not showing in the left menu.

Fixed the gatsby-config.js so that it includes the Master Data page in the Wild Events section.
🎉 Issues resolved:

closes #23

The left menu was not showing the Master Data page for the Wild Events section. I updated the gatsby-config.js so that it includes it in the left menu.


feat: replace hardcoded list of examples with partial Rise screen

Do not hardcode examples, instead, use default screen for our example and render it in a partial view

To do so, we need to slightly overwrite "navigate" action in that partial screen to properly navigate into Rise navigator

Remove "Share" option for default example. We always render it. Also, it would not work, because there's special handling for "example" ID anyways
That said, dropdown now has just one option for builtin connection - "Go back". Decided to make the Rise logo always go back for builtin connection (no dropdown). Saves one extra click.
When you have custom connection, the "Example" will render as always, at the end.

Questions:

How should the "Example" be named? Right now, it's just "Example".

Screenshots:

Maybe "Example" should say "Rise Example UI" without the emoji, or if we want to have an icon it should be left-aligned in the button

I think icon-less is going to be easier for now. I don't think it's worth the effort to check if label contains emoji and add padding otherwise. Most of the connections won't really have an emoji in the label, maybe we can have a special field "icon" in the future!


Make local installation easier

This PR makes the local installation easier by:

Adding infos to the README
Using sqlite outside of the production environment

Just suggestions of course, no hard feelings if it's declined or needs some changes :slightly_smiling_face:

Considering this is mostly a read-only webpage we can probably use SQLite for prod as well, wdyt @pstrzalk ? No point in paying heroku for pg :)

Considering this is mostly a read-only webpage we can probably use SQLite for prod as well, wdyt @pstrzalk ? No point in paying heroku for pg :)

Not really. Heroku restarts dynos every day at a random time and db would be gone. We can use it if we move to a VM + kamal setup (which we might, but I'd have to add proper automation on top)


pod lib lint is failing

Project for macOS builds and tests fine via Xcode and xcodebuild (as you can see on travis).
However, pod lib lint is failing.
Running pod lib lint --verbose --skip-tests
Verbose build log:
    ** BUILD FAILED **

    The following build commands failed:
    	PhaseScriptExecution [CP]\ Embed\ Pods\ Frameworks /Users/jesse/Library/Developer/Xcode/DerivedData/App-emqrrmpeufyerwcsgcwzcgzsqnpk/Build/Intermediates.noindex/App.build/Release/App.build/Script-611DC4CD8454AFE7958B747E.sh
    (1 failure)

Final output:
-> JSQCoreDataKit (8.0.0)
    - NOTE  | xcodebuild:  note: Using new build system
    - NOTE  | xcodebuild:  note: Planning build
    - NOTE  | xcodebuild:  note: Constructing build description
    - NOTE  | xcodebuild:  warning: Skipping code signing because the target does not have an Info.plist file. (in target 'App')
    - ERROR | [OSX] xcodebuild: Returned an unsuccessful exit code.

[!] JSQCoreDataKit did not pass validation, due to 1 error.
You can use the `--no-clean` option to inspect any issue.

Fixed by CocoaPods 1.6.0.beta.2


Can you tell me what's causing this?

3x GSE\API\StringFunctions.lua:134: attempt to index local 'macroversion' (a nil value)
GSE\API\StringFunctions.lua:134: in function CleanMacroVersion' GSE\API\Storage.lua:496: in function OOCUpdateSequence'
GSE\API\Events.lua:281: in function `?'
...s\Broker_Garrison\Libs\AceTimer-3.0\AceTimer-3.0-17.lua:53: in function <...s\Broker_Garrison\Libs\AceTimer-3.0\AceTimer-3.0.lua:48>
Locals:
macroversion = nil
(*temporary) =  defined @GSE\API\StringFunctions.lua:61
(*temporary) = nil
(*temporary) = "attempt to index local 'macroversion' (a nil value)"
GSE =  {
ResetButtons =  defined @GSE\API\Storage.lua:472
GUIShowDebugWindow =  defined @GSE\GUI\DebugWindow.lua:17
GetClassIcon =  defined @GSE\API\CharacterFunctions.lua:41
CloneSequence =  defined @GSE\API\Storage.lua:14
SplitCastSequence =  defined @GSE\API\StringFunctions.lua:79
CancelTimer =  defined @Broker_Garrison\Libs\AceTimer-3.0\AceTimer-3.0.lua:145
GetCurrentTalents =  defined @GSE\API\CharacterFunctions.lua:86
UnregisterComm =  defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:179
OutputQueue =  {
}
MediaPath = "Addons\GSE\Media"
GUIRecordFrame =  {
}
DebugDumpButton =  defined @GSE\API\Storage.lua:581
GetSpecNames =  defined @GSE\API\CharacterFunctions.lua:72
GUIConfirmDeleteSequence =  defined @GSE\API\GUIFunctions.lua:16
CurrentGCD = 0
GSSlash =  defined @GSE\API\Events.lua:226
inDungeon = false
UsedSequences =  {
}
PerformPrint =  defined @GSE\API\Init.lua:23
UnsavedOptions =  {
}
UpdateIcon =  defined @GSE\API\Storage.lua:921
ADDON_LOADED =  defined @GSE\API\Events.lua:91
RegisterMessage =  defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:116
UnregisterMessage =  defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:179
TranslateSpell =  defined @GSE\API\Translator.lua:191
pairsByKeys =  defined @GSE\API\StringFunctions.lua:156
SendCommMessage =  defined @AskMrRobot\Libs\AceComm-3.0\AceComm-3.0.lua:84
UpdateSequence =  defined @GSE\API\Storage.lua:484
UnregisterAllComm =  defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:200
SetEnabledState =  defined @AdiBags\libs\AceAddon-3.0\AceAddon-3.0.lua:455
isSpecIDForCurrentClass =  defined @GSE\API\CharacterFunctions.lua:60
ZONE_CHANGED_NEW_AREA =  defined @GSE\API\Events.lua:55
DebugOutput = ""
Static =  {
}
StandardAddInPacks =  {
}
GUI =  {
}
TranslatorAvailable = true
CloneMacroVersion =  defined @GSE\API\Storage.lua:30
GetSequenceNames =  defined @GSE\API\Storage.lua:798
ImportMacroCollection =  defined @GSE\API\Storage.lua:144
IsLoopSequence =  defined @GSE\API\Storage.lua:261
UnregisterAllEvents =  defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:200
GetOptionsTable =  defined @GSE\GUI\Options.lua:6
GUIDeleteSequence =  defined @GSE\API\GUIFunctions.lua:5
CreateButton =  defined @GSE\API\Storage.lua:914
PVPFlag = false
Deserialize =  defined @AskMrRobot\Libs\AceSerializer-3.0\AceSerializer-3.0.lua:246
GUIPauseDebugView =  defined @GSE\GUI\DebugWindow.lua:45
TranslatorLanguageTables =  {
}
TranslateSequenceFromTo =  defined @GSE\API\Translator.lua:50
GetName =  defined @AdiBags\libs\AceAddon-3.0\AceAddon-3.0.lua:310
FormatSequenceNames =  defined @GSE\API\Plugins.lua:45
DeleteSequence =  defined @A

You have a corrupt macro without any Macro Versions in the GSE.lua file.
Timothy Minahan
M 0406 413 680  |  E<EMAIL_ADDRESS>
On 15 May 2017, at 5:17 pm, Janet<EMAIL_ADDRESS>wrote:
3x GSE\API\StringFunctions.lua:134: attempt to index local 'macroversion' (a nil value)
GSE\API\StringFunctions.lua:134: in function CleanMacroVersion' GSE\API\Storage.lua:496: in functionOOCUpdateSequence'
GSE\API\Events.lua:281: in function `?'
...s\Broker_Garrison\Libs\AceTimer-3.0\AceTimer-3.0-17.lua:53: in function <...s\Broker_Garrison\Libs\AceTimer-3.0\AceTimer-3.0.lua:48>
Locals:
macroversion = nil
(*temporary) = defined @gse\API\StringFunctions.lua:61
(*temporary) = nil
(*temporary) = "attempt to index local 'macroversion' (a nil value)"
GSE =
{
ResetButtons = defined @gse\API\Storage.lua:472
GUIShowDebugWindow = defined @gse\GUI\DebugWindow.lua:17
GetClassIcon = defined @gse\API\CharacterFunctions.lua:41
CloneSequence = defined @gse\API\Storage.lua:14
SplitCastSequence = defined @gse\API\StringFunctions.lua:79
CancelTimer = defined @Broker_Garrison\Libs\AceTimer-3.0\AceTimer-3.0.lua:145
GetCurrentTalents = defined @gse\API\CharacterFunctions.lua:86
UnregisterComm = defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:179
OutputQueue =
{
}
MediaPath = "Addons\GSE\Media"
GUIRecordFrame =
{
}
DebugDumpButton = defined @gse\API\Storage.lua:581
GetSpecNames = defined @gse\API\CharacterFunctions.lua:72
GUIConfirmDeleteSequence = defined @gse\API\GUIFunctions.lua:16
CurrentGCD = 0
GSSlash = defined @gse\API\Events.lua:226
inDungeon = false
UsedSequences =
{
}
PerformPrint = defined @gse\API\Init.lua:23
UnsavedOptions =
{
}
UpdateIcon = defined @gse\API\Storage.lua:921
ADDON_LOADED = defined @gse\API\Events.lua:91
RegisterMessage = defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:116
UnregisterMessage = defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:179
TranslateSpell = defined @gse\API\Translator.lua:191
pairsByKeys = defined @gse\API\StringFunctions.lua:156
SendCommMessage = defined @askmrrobot\Libs\AceComm-3.0\AceComm-3.0.lua:84
UpdateSequence = defined @gse\API\Storage.lua:484
UnregisterAllComm = defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:200
SetEnabledState = defined @adibags\libs\AceAddon-3.0\AceAddon-3.0.lua:455
isSpecIDForCurrentClass = defined @gse\API\CharacterFunctions.lua:60
ZONE_CHANGED_NEW_AREA = defined @gse\API\Events.lua:55
DebugOutput = ""
Static =
{
}
StandardAddInPacks =
{
}
GUI =
{
}
TranslatorAvailable = true
CloneMacroVersion = defined @gse\API\Storage.lua:30
GetSequenceNames = defined @gse\API\Storage.lua:798
ImportMacroCollection = defined @gse\API\Storage.lua:144
IsLoopSequence = defined @gse\API\Storage.lua:261
UnregisterAllEvents = defined @CallbackHandler-1.0\CallbackHandler-1.0\CallbackHandler-1.0.lua:200
GetOptionsTable = defined @gse\GUI\Options.lua:6
GUIDeleteSequence = defined @gse\API\GUIFunctions.lua:5
CreateButton = defined @gse\API\Storage.lua:914
PVPFlag = false
Deserialize = defined @askmrrobot\Libs\AceSerializer-3.0\AceSerializer-3.0.lua:246
GUIPauseDebugView = defined @gse\GUI\DebugWindow.lua:45
TranslatorLanguageTables =
{
}
TranslateSequenceFromTo = defined @gse\API\Translator.lua:50
GetName = defined @adibags\libs\AceAddon-3.0\AceAddon-3.0.lua:310
FormatSequenceNames = defined @gse\API\Plugins.lua:45
DeleteSequence = defined @A
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I dunno why it started doing this, I had to downgrade back to the Alpha 1 to get it working again

and now I can't seem to find GS Core to even be able to get it load and I downloaded the addon from both curse and wow interface. BTW I love your addon.

Hi Janet
GSE.lua exists in C:\path\to\Wow\WTF\Account\YourAccountName\SavedVariables\GSE.lua
This location is where all WoW mods save their data. It is not part of the download.
The Gs-Core folder was turned into an adaptor for GSE1 to allow for migration from GSE1 to GSE2 update and was removed in GSE2.1.0.  In order to fix a number of bugs in 2.1.0, it started reporting corrupt macros there were present but dormant within the file.  2.1.01 will report this as a message instead of an error.
In your case you have a macro that doesn't have the sequence part (from The spec MacroVersions= etc) wishing your GSE.lua file most likely from a GSE1 macro that didn't import properly.
TimothyLuke

On 16 May 2017, at 1:44 am, Janet<EMAIL_ADDRESS>wrote:
I dunno why it started doing this, I had to downgrade back to the Alpha 1 to get it working again
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

thanks for the response. I updated the addon but I only recently decided to figure out how to use it lol
I was hoping to get this working https://mods.curse.com/addons/wow/hunter-macro-plugin-gse but it's saying GS-Core is missing so I was wondering.

That addon mod needs to be updated.  It wont with with the current GSE.
I would suggest having a look at the macros here:
https://wowlazymacros.com/forums/forum/classes/hunters/
From there look at the PVE sections and copy a macro,
Go to gse (/gs ingame)
Import
Paste
Then drag the ? icon to your menu bar.
On Tue, May 16, 2017 at 10:15 AM, Janet<EMAIL_ADDRESS>wrote:

thanks for the response. I updated the addon but I only recently decided
to figure out how to use it lol
I was hoping to get this working https://mods.curse.com/addons/
wow/hunter-macro-plugin-gse but it's saying GS-Core is missing so I was
wondering.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/311#issuecomment-301643498,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjBFbRPjNf9pN8uqCnVgqAmge9VlzCUks5r6PGigaJpZM4Naxpc
.

ok, thanks for all your help. you have a nice week.


sar_rhythmgame working weirdly in coop

the text moves twice as fast because of no alt ticks and it's impossible to get a perfect and sometimes it triggers multiple times for a single jump

also when you switch between bots and jump shortly after you get a perfect for whatever reason

correction the text moves twice as fast only as blue


feat(services/vercel): Add vercel remote cache support

close #2149

Great, thanks!

Thanks for your kind review~


feat: Add remaining 0.9.0 items for Lezer grammar

I nerd-sniped myself into finishing this...

Thanks for working on this!

Probably I should leave it and work on more important things 😀


Remove now-redundant embedded header

Pavol,
I apologize--I missed including this in the last patch. Sorry.
Dustin

thx!


Update ActiveRecord.php

Hi
I think when programmer use refresh method need to get real time what is actually in db. so it shouldn't load from cache if it is enable.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
✔/❌

Tests pass?
✔

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

I see no sense in using cache for refresh().

I'd consider it a bug. @hooman-pro would you please update changelog? Also, it is possible to make a unit test for it?

sorry really i'm busy now and unfortunately  I never write unit test yet!

I commit change log
also i rename my github username so i change my name in line 1395

@bizley I refresh underlying data in db for example user credit that can change by android application on that moment.
or one counter can change by other user requests.

Thank you.


hydrus: v520 -> v530 https://github.com/hydrusnetwork/hydrus/releases/tag/v530

Description of changes
https://github.com/hydrusnetwork/hydrus/releases/tag/v530
Breaking changes: https://github.com/hydrusnetwork/hydrus/releases/tag/v527
"first, OpenCV is updated to a new version, and this causes a dll conflict on at least one platform, necessitating a clean install
second, the program executables are renamed from 'client' and 'server' to 'hydrus_client' and 'hydrus_server', necessitating shortcut updates"
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Yes, switching to the mpv backend stops it from crashing. I agree it's very weird. But it's probably just specific gifs


add variables table

I agree we could name things better. most of these require changes in multiple repos docker,terraform& extensions.
If it is OK with everyone, I can go a head and make required changes. I think it better to change before 1.0 and keep these stable after 1.0.

Yeah, the changes will touch quite a few repos but should be simple and straightforward. My vote goes for doing this now, will be harder to change later.

I agree, better to do it now.

Made most of changes requested.
didn't update SNS_ARN as we need to publish new jar file to maven, will create new PR once that is done.


[FEAT]:  If I want to modify the format of the LLM output, which file's code should I edit? Thank you very much.

What would you like to see?
If I want to modify the format of the LLM output, which file's code should I edit? Thank you very much.

For the UI https://github.com/Mintplex-Labs/anything-llm/blob/master/frontend/src/utils/chat/markdown.js
Otherwise, it would be the system prompt dictating how the output should be listed


Would like to continue this work and publish

Hi, I contacted you via email. I'd love to continue this work, and include support for https://github.com/jaydenseric/graphql-upload as well

@hourliert can you check and grant me owner permission for https://github.com/dart-graphql
PS: I'm working with @micimize and @HofmannZ on similar GraphQL for Dart and/or Flutter as well. Hopefully we can join force and use single repo

I granted you write access to the dart-graphql organization. I have also invited @micimize and @HofmannZ.
Let me know if something looks wrong.

@hourliert I can push to non-protected branch now, but not to protected, and I can't change the repo's setting either, could you double check?


Add JSON patch support

We need an option to apply JSON patch (https://tools.ietf.org/html/rfc6902) operations on an object structure.

Implemented in 1.1.1


Character offsets don't work on a specific song

Describe your problem here.
The title describes it all. There are no errors, nothing in the command prompt, just no matter what character I use for the opponent, the offsets won't work. I have literally no idea why this is happening, and I can't check what I added that's making it do this since my character sprites back then didn't need offsets.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

yeah im having this issue too, idk how to fix it

have you checked stage offsets

have you checked stage offsets

I mean animation specific offsets


Admin Dash Board

Control on overall all applications.
a. Users Control system.
b. Control system on all user post.
c. All Statistics of application needs to be displayed.

see #24


Post Replication SQL Hook

As requested by @giggls a post replication hook is implemented: E.g. when materialised views need updating after an OSM update import has been completed. To reduce impact during catch-up, it is only applied on the most recent update.
A configuration option has been added to the JSON file, which can now carry an SQL query that is executed.
If you have any questions, let us know. We will be both at FOSSGIS in Dresden, in case there is a need for discussion IRL.

Citing from https://groups.google.com/d/msg/imposm/B8hxfcPkSN0/s3EZTF61CgAJ

I think an option to imposm run to exit after one import is a nicer solution. This way you can integrate Imposm into your script, instead of adding a "process management" to Imposm. You can do this already with the undocumented IMPOSM3_SINGLE_DIFF environment (IMPOSM3_SINGLE_DIFF=1 imposm run).

Another solution would be to make Imposm to call arbitrary SQL commands after an import/update. This makes it more complex, but the commands can run in the same SQL transaction, giving you a clean all-or-nothing.

Your patch works outside the transaction, so there is no benefit from it compared to the IMPOSM3_SINGLE_DIFF hack (which could be improved, so that is becomes an official option).

I see a benefit in so far as Sven has currently wrapper scripts that have to check from the outside whether imposm is still catching up with replication, which basically defies the purpose of the loop in imposm itself. I think we have found a general purpose solution in a central place which feels less like a hack than the SINGLE_DIFF solution.
As for the transaction I would be happy to change it to use the update tx.
The current patch does not break anything: When the user specifies an SQL command, it will be executed, otherwise nothing happens at all. I agree that running in the context of the transaction makes total sense, but already there is an operational improvement. We have tested it during the Hackweekend together, so the needed wrapper script could go away and no breakage was introduced.

Too bad we couldn't discuss this IRL, but I had to cancel FOSSGIS this year :-/

The current patch does not break anything: When the user specifies an SQL command, it will be executed, otherwise nothing happens at all. I agree that running in the context of the transaction makes total sense, but already there is an operational improvement.

But adding support for transactions will then break your current patch...
This patch needs to support for transactions from the start.  Please also update the documentation and tests.


Some code files are not provided here due to character limitations

@AntonOsika  When we run the code, if the files being created exceed the character limitation then this error pops : "Some code files are not provided here due to character limitations" and then the script just stops. I think we need to give some sort of memory to make it remember. Maybe we can use ChromaDB for the creation of embeddings. And then can use the GPT-4 to first check the embeddings in the workspace folder created and then use it or else if not present then run like normal.

Thanks @RadientBrain
I think this is a good idea, maybe you can do a POC/draft of its implementation and we can take a look?

Ok I will try and let you know

Closing in favor of keeping things clean and neat, please open your PR when ready.


python3 update-attack.py returns: No module named 'drawSvg'

I followed the setup to install the website under linux (debian). However, when I come to the task to run python3 update-attack.py, the script returns:
ModuleNotFoundError: No module named 'drawSvg'
Running pip3 install drawSvg returns:
Requirement already satisfied: drawSvg in ./env/lib/python3.11/site-packages (2.1.1)
Any idea, what is wrong?

The website relies on our mitreattack-python library (https://github.com/mitre-attack/mitreattack-python) in order to fully build. drawSvg is a dependency over there that had a major update that broke our stuff. We addressed that in mitreattack-python last month in version 2.0.5, and now the latest is 2.0.6.
Over here for the website though, the master branch hasn't been updated yet to update the dependency, and is stuck on 1.7.2 for mitreattack-python. I'm fairly certain (but not 100% certain) that just updating to 2.0.6 would be just fine, but right now we are focusing hard on the upcoming release at the end of this month, at which point the website repo here will be fully updated and working again.
Hope that helps!


Need coarse location permission together with the fine one

Fixes #6, tested on Android 12

I didn't want to go too far with the lint / readability stuff


174 2 remove relative imports from macro files in digitalmarketplace govuk frontend

trello: https://trello.com/c/aYlQvPAJ/174-2-remove-relative-imports-from-macro-files-in-digitalmarketplace-govuk-frontend

Can we add a note to the changelog explaining the change? Something like
'- You no longer need to use the govuk-frontend macros bundled with digitalmarketplace-govuk-frontend; Digital Marketplace components will now use macros under the govuk/ prefix in the template loader.'

I just realised that the ticket doesn't say anything about the relative imports in the Sass files, but we should probably change those as well. Are you okay to do that in this PR @agalamatis?

I just realised that the ticket doesn't say anything about the relative imports in the Sass files, but we should probably change those as well. Are you okay to do that in this PR @agalamatis?

Looking into them now. The rest are done.

Just tried this locally, it looks like with the changes to the Sass files this is a breaking change, apps will need to add node_modules/digitalmarketplace-govuk-frontend to the includePaths for node Sass. We should add that info to the changelog.


[5.0] Add Tests For Helper Functions

Add tests for array_where, head, last, class_basename, str_limit, str_random, str_finish functions.

:+1:


Golang client point to a 404 repo

URL for relevant page?
https://docs.influxdata.com/influxdb/v1.7/tools/api_client_libraries/
What products and version are you using?
InfluxDB v1.7
Where did you look before opening the issue?
The link in the page https://github.com/influxdata/influxdb/blob/master/client/README.md
The InfluxDB go client repo https://github.com/influxdata/influxdb1-client
The repo has been moved to influxdb1-client, the link for the golang client should be updated.

Thanks for the heads-up @Willis0826. The master branch of the InfluxDB repo was recently replaced with the v2.0 alpha InfluxDB source code. In that process, the InfluxDB (v1.x) Go client was moved into its own repo here: https://github.com/influxdata/influxdb1-client
I've updated all the links to the previous client location to point to the new location.


cron in docker image

The PM docker image doesn’t seem to have cron installed.  Is it intended to do the scheduled tasks some other way?

Use cron from the docker host 0  * * * * root docker exec peering_manager ./manage.py poll_peering_sessions --all


Stop glog from emitting false negative errors

glog prepends "ERROR: logging before flag.Parse:" to each log message
if flag.Parse(). However we can't simply call flag.Parse() since
CLI options are handled by cobra.
This PR manually forces flag.Parsed() to return true to avoid those
annoying messages.
Logic is borrowed from
https://github.com/istio/mixer/blob/master/cmd/server/cmd/root.go#L44

Jenkins job auth/presubmit passed

Jenkins job auth/presubmit passed


Update conda env packages during the creation

when the conda environment is created it first uses
the existing packages in the system - for example
the one installed from Miniconda3-py38_4.12.0-Linux-x86_64.sh,
which may become outdated and vulnerable sooner or later.
This PR makes the environment to use the latest available
necessary packages when created to fix all CVEs that are fixable
by the update of the conda environment

Signed-off-by: Janusz Lisiecki<EMAIL_ADDRESS>

CI MESSAGE: [6923317]: BUILD STARTED

CI MESSAGE: [6923317]: BUILD FAILED

CI MESSAGE: [6923317]: BUILD PASSED


Allow specification of client properties?

It would be nice if you could allow specification of the client properties in _build_start_ok_frame.  Exposing a properties field on the Connection? Pika permits this and it's nice being able to identify clients by name in the management console.

PR #142 opened for discussion?


webdash: optimize task list

closes #547

Also, closes #384 ...maybe
I added code to cache tasks in the browser, use the minimal view where possible, and stop the refresh loop when the page/tab is in the background.
It's still not ideal, but it's a substantial improvement. Previously, our task list was requesting about 500-800MB every two seconds, even when you weren't looking. Now, it's about 500 bytes when nothing has changed and nothing is running.
The page load is still slow on the first load (although faster than before). This might be improved in a number of ways: partial list rendering, server-side rendering, etc.
Our task views are kinda clunky. The basic view can still be pretty heavy, and the list doesn't need most of that weight (~500-700KB). We could improve this substantially by having server-side support for requesting specific fields, and/or being able to subscribe to an event stream.

This looks like a great improvement.  Could you apply the stoping of the background refresh loop while the page is in the background behavior to the task, node list and node pages as well?

Could you apply the stoping of the background refresh loop while the page is in the background behavior to the task, node list and node pages as well?

Done


ANY build for ESP32-s2  for board =  lolin_s2_mini fails with USBSerial not declared in scope of HardwareSerial.h (IDFGH-11650)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
framework-espidf @ 3.50102.0 (5.1.2)
Espressif SoC revision.
ESP32-S2  lolin_s2_mini
Operating System used.
Windows
How did you build your project?
Other (please specify in More Information)
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32-S2  lolin_s2_mini
Power Supply used.
USB
What is the expected behavior?
Most simple project  with CONFIG_AUTOSTART_ARDUINO=y for ESP32-S2 lolin_s2_mini should build.
What is the actual behavior?
ANY CODE built with espidf @ 3.50102.0 (5.1.2)/arduinoespressif32 @ 3.0.0 specifying  lolin_s2_mini as the board and with CONFIG_AUTOSTART_ARDUINO=y fails in
HardwareSerial.h:360:16:  with error: 'USBSerial' was not declared in this scope
Steps to reproduce.
platformio:
[env:lolin_s2_mini]
platform = espressif32
board = lolin_s2_mini
framework = espidf, arduino

Code:
#include <Arduino.h>

void setup()
{}

void loop()
{}

sdkconfig includes:
CONFIG_AUTOSTART_ARDUINO=y
Debug Logs.
In file included from C:/Users/peted/.platformio/packages/framework-arduinoespressif32-src-16f16f9b46f255eba682607449bb3231/cores/esp32/Arduino.h:196,
                 from C:/Users/peted/.platformio/packages/framework-arduinoespressif32-src-16f16f9b46f255eba682607449bb3231/cores/esp32/chip-debug-report.cpp:17:
C:/Users/peted/.platformio/packages/framework-arduinoespressif32-src-16f16f9b46f255eba682607449bb3231/cores/esp32/chip-debug-report.cpp: In function 'void printBeforeSetupInfo()':
C:/Users/peted/.platformio/packages/framework-arduinoespressif32-src-16f16f9b46f255eba682607449bb3231/cores/esp32/HardwareSerial.h:360:16: error: 'USBSerial' was not declared in this scope; did you mean 'Serial'?
  360 | #define Serial USBSerial
      |                ^~~~~~~~~
C:/Users/peted/.platformio/packages/framework-arduinoespressif32-src-16f16f9b46f255eba682607449bb3231/cores/esp32/chip-debug-report.cpp:260:3: note: in expansion of macro 'Serial'

More Information.
1: Using VS Code/Platformio (see platformio.ini above.  The esp-idf and arduinoespressif32 are the latest available as of 13:00 UTC  on 9 December 2023
2: Errors shown above occur when board = lolin-s2_mini.  However, does not occur is board = esp32-s3-devkitc-1, for example

most likely this should be sent over to https://github.com/espressif/arduino-esp32 since it looks likely to be an issue with PlatformIO and arduino dependencies.

I agree with @atanisoft, USBSerial as well as HardwareSerial.h are a part of Arduino Core. Unrelated to ESP-IDF.

Thanks for that link.  I will indeed take it up with them.  (If this goes like the rest of that release, it may be a while...


Minor doc typo in subtract.rst

An embarrassingly minor PR! Sorry! Thanks for a fantastic tool.

Thanks!


vue-loader recent issue - need update from 14.x to 15.x

This morning, I tried a clean install of the sample vueJs
An error occured about vue-loader...
To fix it , I updated this module with npm install<EMAIL_ADDRESS>--save-dev
Then, I added in webpack.shared.js line:30
const VueLoaderPlugin = require('vue-loader/lib/plugin');
And line:126, in plugins:
new VueLoaderPlugin(),

This is resolved in the latest sample app, which uses the latest release candidate of @vue/cli, which is responsible for webpack dependencies.


Add --workflow flag to kpt fn export

Use the --workflow flag to specify the workflow orchestrator. Now DIR is the first argument in consistent with other subcommands.

@Liujingfang1 @mortent @monopole @frankfarzan

/lgtm after addressing @Liujingfang1 comments.

/lgtm

/approve


When you have a lot of links, the latests do not work (I know how to fix it)

Preflight Checklist

[ x] I agree to follow the Code of Conduct that this project adheres to.
[x ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Create a text on the working area
right click > edit link
add a link to an internet page
duplicate the element  (I did it 40 times)

Expected behavior
when exporting the diagram to svg, some links are not working.
Trying to remove some links will not help.
Links work if the diagram is exported to html.
The reason why this is happening is because a missing pointer-events="all"
Adding it manually using a text editor fix the problem
Screenshots
If applicable, add screenshots to help explain your problem.
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version [e.g. 12.6.7]

Desktop (please complete the following information):

OS: [e.g. iOS] windows 10 64bits
Browser [e.g. chrome, safari] chrome and firefox
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]

OS: [e.g. iOS8.1]

Browser [e.g. stock browser, safari] chrome Version 90.0.4430.212 (Build officiel) (64 bits)  and Firefox 88.01.64 bits

Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi,
Could you attach the file so it can be tested?
Thanks,

If you wish to send us the diagram privately, you can file an issue at https://drawio.atlassian.net/servicedesk/customer/portal/16/group/17/create/65


getProperties on Shanty_Mongo_Document

Perhaps it would be nice to include something like this to Shanty_Mongo_Document. Haven't tested it much and it's just quick and dirty solution but it's good enough for me though.
/*
 * Get array of properties (recursive)
 * @param Shanty_Mongo_Document $document
 * @return array
 */
public function getProperties($document = null){
    if(is_null($document))
        $document = $this;
    $keys = $document->getPropertyKeys();
    foreach($keys as $key){
        if($document->$key instanceof Shanty_Mongo_Document){
            $rval[$key] = $document->getProperties($document->$key);
        } else {
            $rval[$key] = $document->$key;
        }
    }
    return $rval;
}

If I'm reading this right, it's the same thing I have called as import():
function import($data) {
foreach( $data as $k => $v) {
$this->setProperty($k, $v);
}
}
Which I have defined in my AppModel extending Shanty_Mongo_Document.

Not really. This is something like export. Imagine a situation when I wish to export data eg. in JSON. And that's exactly I'm trying to do.

On that subject it would be really nice to define a set of 'external' properties that will show on export() or something so there is a clean way to export data from a object without having to unset() all the fields you don't want to pass to a client.

Well that depends... I'd say it would be cleaner to have only a possibility to include internal props (such as _id or _type) or not. Any other properties IMO should be (and are) defined as a second param in find(), all() etc. methods.

I my branch i have a fix for the exclude (because you can not include and exclude fields at the same time).  Currently the exclude operation is trying to include _id and _type, but we dont need to include those fields if we are only excluding a few other fields.

I've found overriding export in my own models the cleanest way to go.

Agreed

While i agree that it may be clean in code to over ride export, you are still increasing the overhead in your application by bringing in an entire record and then scaling it back with export later.  If you dont need the data, exclude it form your query in the first place and dont transmit it from the mongo server.  By grabbin the entire record, your grabbing extra bits that need to transmit over the wire needlessly.  By excluding fields in the query you are saving milliseconds all over your application rather than trimming back an export once the time over the wire has already been wasted.

Ok for example in a set I use all the records have created/modified/wholastmodified(and more), while I need those fields in the server side code, the client never has any need to see them. So while I sometimes have to pull them back, I don't really want that information going to the client.
Its not exactly a huge deal to work around, but it would be nice if there was a 'export like' function that only exported data from a list of 'allowed' fields to subsequently encode to JSON or whatever.

I think this is possibly two issues being discussed here. You'd obviously only select what you required from Mongo to keep what's going over the wire as small as possible. But, in terms of making sure you never export something you want to hide, overriding export seems like the best way to go.

You are supposed to do that in your query to mongo:
$mongoObject->all(
	array(
		/* QUERY PARAMERTERS */
	),
	array(
		'field_to_exclude' => -1,
		'field_to_exclude_2' => -1,
	)
);

That way in your client side code, you can exclude the fields you want, and in the server side code you can exclude the fields you want.  The only problem is, in this fork of the code, you cant exclude fields because Coen has things setup to always include _id and _type, i have modified the Document.php file to allow a user to exclude fields.

I have just committed a change to my branch that will let you do the query above, and then let you properly export (skipping the requirement check)
This makes export() work and query limiting work for keeping OtW transmission minimal.


Fix assets path in releases

Closes #756
Fix unable to access /admin due to path resolving in releases. __DIR__ and other kernel forms are expanded at compile time, so they’re always returning the wrong path. This PR fixes the issue by resolving static path using Application.app_dir instead.
Please also verify that omg.server and other things are working fine.

Btw if you're running mix omg.server you probably want to move your public/ and private/:

/public -> apps/url_dispatcher/priv/static/public
/private -> apps/url_dispatcher/priv/static/private

(Moving as-is should be fine)
🙇

I'm getting these warnings when compiling:
warning: function EWallet.Helper.static_dir/1 is undefined or private
  lib/ewallet_config/storage/local.ex:28

warning: unused import Phoenix.Controller.redirect/2
  lib/url_dispatcher/plug.ex:18

mix omg.server is working well! All below works:

/admin page is accessible
Swagger works
Local uploads are working properly
Local exports are working properly

Endpoints are not served when running ewallet foreground:
_build/prod/rel/ewallet/bin/ewallet foreground
12:25:49.623 [info] AppSignal disabled.
12:25:49.816 [info] UrlDispatcher.Plug disabled.

But it can be started by manually passing SERVE_ENDPOINTS:
SERVE_ENDPOINTS=true _build/prod/rel/ewallet/bin/ewallet foreground
12:27:23.057 [info] AppSignal disabled.
12:27:23.301 [info] Running UrlDispatcher.Plug with Cowboy on port 4000.
12:27:23.301 [info] Running EWalletAPI.V1.Endpoint WebSocket endpoint at /api/client/socket.
12:27:23.301 [info] Running AdminAPI.V1.Endpoint WebSocket endpoint at /api/admin/socket.

Now when the endpoints are serving, these work!

/admin page is accessible
Swagger works
Local uploads are working properly
Local exports are working properly

I'm seeing the same warnings as @mederic-p as well.

SERVE_ENDPOINTS is intentional. In rootfs/entrypoint we default this to true if user run it via alias (e.g. docker run omisego/ewallet:latest foreground) rather than via bin/ewallet.
I’m not sure if there’s a way around this because otherwise bin/ewallet console would start the server which might not be the behavior ones want (we use that mostly for debug right?)
Otherwise thanks for double checking!

On 1 Feb 2019, at 19:32, Unnawut Leepaisalsuwanna<EMAIL_ADDRESS>wrote:
mix omg.server is working well! All below works:
/admin page is accessible
Swagger works
Local uploads are working properly
Local exports are working properly
Endpoints are not served when running ewallet foreground:
_build/prod/rel/ewallet/bin/ewallet foreground
12:25:49.623 [info] AppSignal disabled.
12:25:49.816 [info] UrlDispatcher.Plug disabled.
But it can be started by manually passing SERVE_ENDPOINTS:
SERVE_ENDPOINTS=true _build/prod/rel/ewallet/bin/ewallet foreground
12:27:23.057 [info] AppSignal disabled.
12:27:23.301 [info] Running UrlDispatcher.Plug with Cowboy on port 4000.
12:27:23.301 [info] Running EWalletAPI.V1.Endpoint WebSocket endpoint at /api/client/socket.
12:27:23.301 [info] Running AdminAPI.V1.Endpoint WebSocket endpoint at /api/admin/socket.
Now when the endpoints are serving, these work!
/admin page is accessible
Swagger works
Local uploads are working properly
Local exports are working properly
I'm seeing the same warnings as @mederic-p as well.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

Right. Since the entrypoint's code isn't changed and the SERVE_ENDPOINTS is indeed set there. I think this is good to merge.
Will regression test again once the remaining v1.1 PRs/issues are merged.

Oh also... I think EWallet is not a dependency in EWalletConfig, and intended not to be I believe.
Is it possible to move EWallet.Helper.static_dir/1 to Utils app?

Oh I didn't noticed utils app was now a thing. Will move it there.

Moving to utils going to take a bit more time (only a bit), since it depends on to_boolean which happens to be in EWallet.Helper so I need to move that too.

I've moved all relevant files, please review again. Thanks!


DOCS:  document baking via utils.webaverse.com

goto https://utils.webaverse.com/
Choose file in bake physics - get .bin file
Make a folder with your glb and .bin file
Install xrpk via npm $ npm install -g xrpk
create a manifest.json in that folder with a physics_url pointing to that .bin (https://docs.webaverse.com/docs/create/manifest-api#physics_url)
run xrpk build . in that folder
should spit out a .wbn file -- you can mint and load that into Webaverse, it's the model + baked collisions in one

added https://github.com/webaverse/docs/commit/2f72f9046a0874d39dbcd2758da81d3d19b49abf

added https://github.com/webaverse/docs/commit/2f72f9046a0874d39dbcd2758da81d3d19b49abf


luddite: 1.0.2 -> 1.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jumptrading/luddite/releases.
meta.description for luddite is: Checks for out-of-date package versions
meta.homepage for luddite is: https://github.com/jumptrading/luddite
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.3 with grep in /nix/store/v3lrqk383mvx27658xqnk9zxvcx14c6w-python3.11-luddite-1.0.3
found 1.0.3 in filename of file in /nix/store/v3lrqk383mvx27658xqnk9zxvcx14c6w-python3.11-luddite-1.0.3

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
faraday-cli
luddite
python310Packages.luddite
python311Packages.luddite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v3lrqk383mvx27658xqnk9zxvcx14c6w-python3.11-luddite-1.0.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A luddite https://github.com/r-ryantm/nixpkgs/archive/dd1ef0491bfdef6ad5bf135dec0044356147e4cd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/dd1ef0491bfdef6ad5bf135dec0044356147e4cd#luddite

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v3lrqk383mvx27658xqnk9zxvcx14c6w-python3.11-luddite-1.0.3
ls -la /nix/store/v3lrqk383mvx27658xqnk9zxvcx14c6w-python3.11-luddite-1.0.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

faraday-cli
luddite (python311Packages.luddite)
python310Packages.luddite

Maintainer pings
cc @emilytrau for testing.

Result of nixpkgs-review pr 264656 at ecdfa2f4 run on x86_64-linux 1

3 packages built successfully:

faraday-cli
luddite (python311Packages.luddite)
python310Packages.luddite


Connect Cloud Build with GitHub Enterprise Cloud with Allow IP List turned on

Connecting CloudBuild to GitHub using Google Github App won't work when the GitHub IP Allow list is turned on on the organization level. The Google GitHub App doesn't add CloudBuild IPs to the Github IP Allow List like the JIRA GitHub app does.

Thanks for this report, but this isn't the repo for app support requests.
Support requests for the Google Cloud Build app can be made from the links in their marketplace listing. https://github.com/marketplace/google-cloud-build


ENH: Enable optional BIDS entity filtering to data collection utility

related to https://github.com/poldracklab/fmriprep/pull/1770

Hello @bpinsard, Thank you for updating!

In the file niworkflows/utils/bids.py:

Line 166:9: E112 expected an indented block
Line 166:15: E999 IndentationError: expected an indented block

To test for issues locally, pip install flake8 and then run flake8 niworkflows.

It is already a dict of dict. It copies the hard-coded structure within collect data. The filter as to be per suffix (_bold, _T1w...) it is not applied to all.

The following doctest should work.
t1w in collect_data is lowercase contrary to BIDS suffix.
>>> bids_root, _ = collect_data(str(datadir / 'ds051'), '01',
    ...                             bids_validate=False, bids_filters={'t1w':{'run': 1}})
    >>> bids_root['t1w']  # doctest: +ELLIPSIS

Let's just do this and see if we need to update it.


"Logout" from wallet

Add "Clear wallet" item in the settings https://zpl.io/VQvjoQ9
Add confirmation alert screen https://zpl.io/a3dx6mW

@DenisDemyanko after talk with Vika and Sasha, we decided to rename "Clear wallet" to "Remove current wallet". Also changed alert wording here https://zpl.io/a3dx6mW
Also please let's add password confirmation after this alert https://zpl.io/bADnlvo

Crashing the wallet when you use the "Remove current wallet".
Steps to reproduce behavior:
1 Go to the "Settings" screen.
2. Tap button"Remove current wallet"
3. On the alert that appears, select "Remove wallet"
4. Enter password.
Expected: Removed current wallet.
Actual: Wallet crash.

Playable on version 4.4

Crashing the wallet when you use the "Remove current wallet".
Steps to reproduce behavior:
1 Go to the "Settings" screen.

Tap button"Remove current wallet"
On the alert that appears, select "Remove wallet"
Enter password.
Expected: Removed current wallet.
Actual: Wallet crash.

video_2019-11-13_15-14-45.zip

checked


No back navigation for secondary links in Settings

Home icon always takes you back to the main list of stories, instead of back one hierarchy level on the settings page.
Could potentially just use the material back arrow to navigate the hierarchy.

I think having the home icon makes sense, you can always use the back button to go back one level to the settings page.
what do you think @thm ?

@fgei Yeah, that was my thinking

ok, closing then


Date Field: "date" column should be removed

As @nilshoerrmann pointed out in a different thread, the Date Fields "date" column is not used anymore. So it should be removed.

I would propose this to be 2.7.0 ? Can it be 2.6.0 ?

2.6.0 has RC status, so I suggest to do it in 2.6.1 (and 2.7, of course).

Great.

Fixed in 1377797


How to use typespec.go

Hi, for the development of the new RP, we are using typespec to create the ARM API, which we need to also consume and produce in our code. For the code we use Golang. Currently we are writing the structures manually, however I see there is the package for typespec.go https://github.com/Azure/autorest.go/tree/main/packages/typespec-go present in the codebase. How do I use it?
I searched the npm, but without any luck. There is the distribution for python https://www.npmjs.com/package/@azure-tools/typespec-python but not for golang.
Thank you.

The dev releases of the typespec-go emitter aren't published to npmjs yet as it's still a work-in-progress. For now, you can grab the tarball from our github releases and install it manually. Here's the latest one.
https://github.com/Azure/autorest.go/releases/tag/v0.1.0-dev.2-20240417.1
Once installed, specify @azure-tools/typespec-go for the emitter.
If you run into any problems and/or bad codegen, please open issues here so we can get them fixed.


update to 1.2.0

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   94.72%   94.72%
=======================================
  Files           8        8
  Lines         417      417
  Branches       58       58
=======================================
  Hits          395      395
  Misses         10       10
  Partials       12       12

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 a6f5f21...9ccb543. Read the comment docs.


🧹 UI REVAMP

Hi @anubhav1206,
Hi @mrTSB,
Hi everyone,
I could use some help to style the calculator
Tasks

[ ] Create a new row for the CRL button
[ ] Update the style

If you have any recommendations for a new design, please let me know
Best regards
NH

I also do not like the current position of the CLR button. It could span a full row for now until we add trig, logarithms, or exponents to the row.

Yes, that would be nice @mrTSB 👍🏽

Thanks @mrTSB


Adds a max height to job arguments

When the arguments are e.g. ActiveRecord records with a lot of fields, the table gets really long and it's hard to scroll around and find things. Adding a max-height makes it easier to navigate.
Before:

Job arguments can be super long which is unhelpful when trying to navigate the table.

After:

Reasonable row height (10rem) so that you can easily scroll through and find things

20rem is okay too but still pretty tall

I picked a very arbitrary height, could probably make it even smaller, though I felt that being able to see a decent portion of the arguments is mildly helpful when they are long. Arguably we could make it even shorter though and you can just click through to the job details to see the full arguments.
I have no idea what if any CSS practices are used in this repo (there was hardly any CSS to reference lol) so please feel free to suggest any changes desired. Thanks!

Oh hm, I think my root issue is actually slightly different, so while this might still be a good idea for long simple arguments, there might be a better solution for my problem. My problem stems from the fact that our app is using GlobalID arguments with ActiveRecord models and also using keyword arguments.
It looks like in the JobsHelper module it tries to avoid deserializing GlobalID arguments, but if using keyword arguments, the first level argument will be a hash, and so it goes into the last case and deserializes the models, leading to a printout of all the record's fields. If it was kept to the globalID identifiers it wouldn't be so bad. (It is kind of handy to see the whole thing though, so I'm a little torn, but I feel like the table definitely isn't the place to see all of the record's fields.) So maybe a fix to handle that would be good...
Hm, I have a couple proposals to handle that.
The first recursively handles nested hashes:
https://github.com/rails/mission_control-jobs/compare/main...ibrahima:handle-keyword-arguments?expand=1
The second only specifically handles Ruby keyword arguments as indicated by the presence of the "_aj_ruby2_keywords" key.
https://github.com/rails/mission_control-jobs/compare/main...ibrahima:handle-aj-keyword-arguments?expand=1
Which would be better? My feeling is recursing might be necessary because if the reason to avoid deserializing GlobalID args is to avoid classes that don't exist in the app, you would need to apply this no matter how deeply nested a GlobalID argument is. However only handling keyword arguments is a narrower change.

@rosa Hi! Sorry to bother you, but I was wondering if you had a chance to look at this. Perhaps the max-height thing is more a matter of taste, but I think the issue with deserializing keyword globalID arguments is a bug. For now I've monkey patched it in my applications but I would love it if I didn't have to. Thanks!

Hey @ibrahima, sorry for the delay! I've been busy with other stuff and neglecting this gem, but I'm catching up now.

I think the issue with deserializing keyword globalID arguments is a bug

Yes, I agree! Good catch. Going to review this and get it merged soon.

I think to make progress here we should have a test first, with different arguments and the results we expect to see for each, instead of trying to catch those by trial and error.

Did this in #201, so I'm going to close this one. Now nested global IDs won't be deserialized, but other serialized arguments like dates will continue to be correctly deserialized.

Nice thanks!


Docs for GitLab

Docs for using GitLab.

Assumes we've published v1.3 and have the hosts option.
Also includes a tiny fix for the GitHub docs, where I stole the structure and all of the config detail from.

Shame there's no good GitLab-specific splash on Unsplash :shrug:

wrt the hosts option, I've also opened https://gitlab.com/xmpp-interop-testing/xmpp-interop-testing-gitlab-component/-/merge_requests/5 to use it more properly in the repo


Add functionality to rank grasping poses on a per object basis

This is follow up work on #4 which refactors the code to separate annotation into a separate node and to add the ability to filter ranked poses on a per-object basis.

I've added a small opencv based utility to stream image topics (which works better than image view via docker) and to manually draw rectangles. The idea is to replace manual drawing with bounding boxes generated from YOLO.

See https://github.com/acmiyaguchi/GraspKpNet/compare/demo...acmiyaguchi:GraspKpNet:detect-filter?expand=1 for differences. There's currently a bug with the latched topic that should be fairly easy to fix.


Multiindex and cufflinks

Hey
I've had some problems with a multiindex dataframe and cufflinks because I am not able to specify what I would like to be plotted as mean and what I would like to be plotted as error.
This is my dataframe:

As you can see I have the mean and standard deviation of different samples over time. I would like to display the mean with the standard deviation as continuous error (or just error bars). With cufflinks I am not sure that this is currently possible because I end up with this:

Perhaps I am just missing something.

Hi @mcrone , did you try what @mazzma12 suggested?
Unstacking the DataFrame should allow you to do this easily.
Otherwise if you can share the code with which you are generating your DataFrame

Thank you for the help @mazzma12!
I was specifying my dataframe with the code below just because of the formatting of the data from a csv:
d = df.groupby('Time [min]').agg(['mean', 'std']).T
I got around the problem with this code:
[{'x': d.xs('mean', level = 1).index.get_level_values(0),     'y': d.xs('mean', level = 1)[col],     'name': col,     'error_y':dict(         type='continuous_data',         array=d.xs('std', level = 1)[col]) }  for col in d.columns]
I will try your suggest since it is probably a cleaner way of doing it. (Sorry for the messy code, for some reason I can't seem to insert linebreaks)
Here was my final graph:

This seems to have broken again with the new plotly and cufflinks. I followed the suggestion and unstacked the columns and so I end up with this:

The only way that I seem to be able to plot it is by dropping the column with the standard deviation with: av=d.drop('std', level=1, axis=1) and then plotting this: av.iplot(kind='scatter', filename='cufflinks/simple-line') however then I am unsure of how to add in the error again.
It seems that the error suggested in the readme for version 0.6 doesn't seem to work any longer, or perhaps I just don't understand how to plot varying error for each trace.

Ok I fixed this by doing it the non cufflinks way:


use SVG icons for weapons

Created by @Algonide
WEAPON.zip

I'll start working on this issue !

@Jenfufu FYI I updated the scope of this PR:

Adding in the svg icons should complete this issue. a separate task will be made to replace the existing image with these new svgs.


Leaderboards v2

The new leaderboard documents will be stored in four separate collections:

daily_scores
weekly_scores
monthly_scores
overall_scores

Documents in these collection will include:

uid (alias user_id), reference to user._id
s (alias score), integer
eat (alias expires_at), datetime
cat (alias created_at), datetime
uat (alias updated_at), datetime

Each collection will include the indexes:

{ uid: 1 } (the order is irrelevant)
{ s: -1 }
{ eat: 1 }, { expireAfterSeconds: 0 } (to set the correct TTL for each document)

Note: overall_scores won't include neither the eat attribute nor the TTL index because overall scores don't expire.
An example query to populate the daily leaderboard page would be:
db.daily_scores.sort({ s: -1 })

Note: the query would be timezone independent because documents would expire according to their TTL, that depends on the associated user's timezone.
An alternative approach (using a single collection)
The alternative approach is to store leaderboard documents in a single collection scores.
Documents in this new collection will include:

uid (alias user_id), reference to user._id
ds (alias daily_score), integer
ws (alias weekly_score), integer
os (alias overall_score), integer
cat (alias created_at), datetime
uat (alias updated_at), datetime

An example query to populate the daily leaderboard page (EST timezone, -4:00) would be:
db.scores.find({ uat: { $gte: new ISODate("2016-11-29T04:00:00Z") } }).sort({ ds: -1 })

The issue with this approach is that it's hard to find a good index for the query. A composite index { uat: 1, ds: -1 } would include a sort stage to sort the subset of documents selected by the find condition. A composite index { ds: -1, uat: 1 } would prevent sorting the results, but it would need to scan the index to filter out documents based on the find condition. Finally a partial index would not work because the index condition should be dynamic, based on date and timezone.
Async operations
Score collections documents should be created/updated asynchronously by sucker-punch workers. There's already an open PR (https://github.com/potomak/tomatoes/pull/199) started by @bugant, that put the foundations to use workers to update records in the old leaderboard collections.
A ScoreUpdaterJob worker should take care of updating score documents. The worker takes two parameters: a user id, a score delta. It creates/updates the correct score document in each of the four collections. New documents should have a TTL according to their leaderboard period and user's timezone, i.e. daily: beginning of the next day, weekly: beginning of the next week, monthly: beginning of the next month.
Naming
The old code mixes three different concepts: leaderboard, score, and ranking.
Ranking is used in the wrong way. A user's ranking should be his/her position in a leaderboard. At the moment we're not going to show the ranking anywhere.
Score is the former value attribute of leaderboard document. Score is a more appropriate term. In Tomatoes leaderboards context, the score represents the number of tomatoes completed by a user from the beginning of a period. There are four possible periods:

daily, from the beginning of the current day
weekly, from the beginning of the current week
monthly, from the beginning of the current month
overall, from the moment the user has been created

Leaderboard is a list of score documents ordered by descending score value. There are four different leaderboards, one for each period. Note: these lists should be paginated.
TODO

[x] Create new score models (#207)

[x] DailyScore model
[x] WeeklyScore model
[x] MonthlyScore model
[x] OverallScore model

[x] ScoreUpdaterJob (#199, #208)
[x] update Tomato to perform asynchronously ScoreUpdaterJob (#199, #210)

[x] after_create (delta: +1)
[x] after_destroy (delta: -1)

[ ] Migrate score data to the new collections
[x] Create new leaderboards controllers (#211)
The old controller is called RankingsController, let's create a new controller LeaderboardsController

[x] GET /leaderboard/daily
[x] GET /leaderboard/weekly
[x] GET /leaderboard/monthly
[x] GET /leaderboard/overall

[x] API endpoints (#213)

[x] GET /api/leaderboard/daily
[x] GET /api/leaderboard/weekly
[x] GET /api/leaderboard/monthly
[x] GET /api/leaderboard/overall

Clean up tasks

[ ] Remove *Rank models
[ ] Remove *Rank controllers and views
[ ] Delete *Rank collections from DB
[ ] Delete ranking_links shared partial
[ ] Remove script to dynamically inject the you class that has been introduced by #193

References

http://openmymind.net/2011/5/8/Practical-NoSQL-Solving-a-Real-Problem-w-Mongo-Red/
https://emptysqua.re/blog/optimizing-mongodb-compound-indexes/
https://docs.mongodb.com/manual/core/index-compound/
https://docs.mongodb.com/manual/tutorial/expire-data/
https://docs.mongodb.com/v3.2/tutorial/sort-results-with-indexes/
https://docs.mongodb.com/v3.2/tutorial/analyze-query-plan/
https://docs.mongodb.com/v3.2/reference/explain-results/

I migrated users overall score by running the script:
User.all.limit(2000).skip(0).map do |user|
  begin
    tomatoes_count = user.tomatoes.count
    next unless tomatoes_count > 0
    print "User: #{user._id} (#{user.name}), score: #{tomatoes_count}... "

    user_score = OverallScore.where(uid: user._id).first
    if user_score.nil?
      puts 'Creating new score'
      OverallScore.create!(uid: user._id, score: tomatoes_count)
      next
    end

    puts 'Update existing score'
    user_score.update!(score: tomatoes_count)
  rescue => e
    puts "Error: #{e}"
  end
end; nil


Option to avoid reference cycles

Once --refGenome (#1) works, people will be able to use non-root genomes as references again.  These will generally contain paralogies which get converted into vg cycles.  As it stands, vg doesn't cope to well with cycles, especially in the reference.  Should add option to ignore paralogies in the reference, or perhaps altogether.

The important thing would be to allow there to be "duplications" between alt contigs and primary contigs, while preventing pieces of the same primary chromosome (or maybe of two different primary chromosomes) from overlapping.

OK, this is working as a blanket hal genome wide switch (off by default) for reference duplications.   It will only process alt contigs the way you want if they are in a separate hal genome.  Otherwise, will need to pass in a list of primary contig names, I guess.

--refSequenceFile added to pass a list of reference sequence names.  A file containing chr1, chr2, etc. can be used to make sure these contigs are not folded into cycles.  But everything else, ex the alts, get aligned to them and each other when possible.


Updated Validations.py script

Fixed validations.py
Fixes #1

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Rename UPBGE to BlenderGameEngine

SteamDB app page links to a few games using this
https://steamdb.info/app/757860/
https://steamdb.info/app/499460/
https://steamdb.info/app/324240/
https://steamdb.info/app/1209750/ (this one I've confirmed uses UPBGE)
Everything currently under https://steamdb.info/tech/Engine/UPBGE/
Brief explanation of the change
Edited description to include both projects which have the same signature.
BGE was removed from Blender in 2.80 in 2019, so it's likely that most of
the published titles were made with BGE and not the UPBGE fork.
Follow-up to https://github.com/SteamDatabase/FileDetectionRuleSets/pull/61#issuecomment-917361867
Note: I don't know if there's a way to actually differentiate between BGE and UPBGE titles to see if my above assumption is correct. I could be wrong since while BGE was removed from Blender in 2019, UPBGE did start as a fork in 2015 according to their README.
Note 2: Blender itself is matched even though it no longer contains the code for the BGE. So this might also match other non-game applications that bundle parts of Blender, such as Motion Studio which bundles it as an export tool.

Most common base names:

blendthumb64.dll - 80 matches
blendthumb.dll - 36 matches

Depots that matched:

324241 - I Will Escape Content
333121 - Inside The Gear Content
333311 - Inside The Gear Demo Content
350501 - Fly in the House Content
365671 - Blender Windows x86_64
365672 - Blender Windows x86_32
369624 - F-1 drive Windows32 Depot
369625 - F-1 drive Windows64 Depot
387260 - Face It Original Windows
436864 - Arctic alive Windows32 Depot
436865 - Arctic alive Windows64 Depot
499461 - Tomato Jones Content
509851 - Depot Bumper
527801 - WAR CUBE Content
531921 - Carton Content
542300 - Depot Bumper - Halloween edition (542300)
542301 - Depot Bumper - Halloween edition
548070 - Depot Bumper Halloween (548070)
576701 - Hollow Halls Content
625941 - Tomato Jones 2 Content
631921 - Somewhere on Zibylon Content
656911 - Space Hit Content
686611 - Sobreviva Content
693221 - Tomato Jones 2 Demo Content
696491 - TOTOBALL Content
708611 - Miner Ultra Shooter Content
711461 - Duel Survival Depot
721421 - FIGHTING BOX Content
731941 - Bars and Balance Content
740501 - eseapner
756351 - OutBreak: The Escape Content
757861 - Elbub Content
775791 - Rampage Ragdoll Content
782451 - Fleazer Content
782611 - Tank 51 Content
788321 - Balls and Magnets Content
788891 - Tomato Jones - Episode 3 Content
789681 - MechTroid Content
790761 - Space Chip Content
795921 - SALVATOR Content
802561 - ES Games Content win64
802562 - "ES Games"-Depot win32
803851 - Pixelpunk XL Content
804791 - Tomato Jones - Episode 3 Demo Content
805261 - PICNIC Content
811451 - Dongo Adventure Content
814591 - Caveman Stories Content
856731 - ARENA 8 Content
860681 - Smile'N'Slide Content
861511 - Sail Ships Content
896371 - Navigator Content
910701 - Space Break Content
938361 - The last soldier Content
951711 - ES Games Demo win 64
951712 - "ES Games Demo"-win 32
956371 - Coindice Simulator win64
956372 - "Coindice Simulator-win32
957871 - Parcheesi Boardgame Simulator Demo win64
957872 - "Parcheesi Boardgame Simulator Demo"-win32
963321 - Bear Vs Hunter win64
963322 - "Bear Vs Hunter"-win32
964721 - Bombman win64
964722 - "Bombman"-win32
989861 - Stop Cats Content
1025651 - Astrog Content
1062121 - Peas Adventure Content
1105481 - GGG Collection Content
1105482 - "GGG Collection"-Depot
1144521 - Air Windows 32
1144522 - Air Windows 64
1209752 - Flow State Windows
1212451 - Mineirinho Shooter DC Content
1215061 - A.V.I. Content
1246121 - NeonFlight Content
1285821 - Virus Kombat Content
1324561 - Krum - Battle Arena Content
1353131 - Hoplichess Content
1375641 - Lost Keys_Content
1382471 - Lost Keys_Demo Content
1433161 - Mechanic 8230 Demo Content
1471631 - MegaRamp Content
1500781 - Depot Champoo Club
1500782 - Champoo Club v 1.3
1500783 - Champoo Club v 1.1
1507641 - Mage Hunters Content
1534491 - Motion Studio Content
1635441 - Operation F.A.T.E. Content
1665991 - Selene's Nightmare Content
1693231 - Puzzle Soccer Content

Most common base names:

blenderplayer.exe - 20 matches

Depots that matched:

631921 - Somewhere on Zibylon Content
686611 - Sobreviva Content
731941 - Bars and Balance Content
757861 - Elbub Content
767681 - Domino Effect Content
788321 - Balls and Magnets Content
803851 - Pixelpunk XL Content
811451 - Dongo Adventure Content
814591 - Caveman Stories Content
816441 - Domino Effect Demo Depot
860681 - Smile'N'Slide Content
1375641 - Lost Keys_Content
1382471 - Lost Keys_Demo Content
1433161 - Mechanic 8230 Demo Content
1471631 - MegaRamp Content
1500781 - Depot Champoo Club
1500782 - Champoo Club v 1.3
1500783 - Champoo Club v 1.1
1507641 - Mage Hunters Content
1693231 - Puzzle Soccer Content

Thanks! I've had a quick look at some of the entries with BlendThumb*.dll and no blenderplayer.exe and they do look like BGE games.
 Those are also found in Blender itself (which makes sense, since BGE was practically just bundling a stripped down Blender to run a .blend scene)

No blenderplayer.exe, most likely the name doesn't need to be kept and users could rename it to their game name. Several depots have <game>_launcher.exe but I don't know if it's a reliable pattern suggested by BGE or just a coincidence.
All in all, there's not much to go on with BGE to be 100% sure, but indeed BlendThumb.dll and BlendThumb64.dll should be good hints as those are libraries used to make thumbnails for .blend files on Windows. They could be present in other games where Blender was used for asset creation only, but I would assume that in such projects the BlendThumb64.dll would be in the Blender installation folder in their C:\Program Files\ and not in the game export folder.

I didn't review the full list but in terms of false positives with the current heuristics, I think it's mostly just:

Blender itself - fair enough I think.
Motion Studio - which bundles Blender as an exporter, but is not made itself with BGE or UPBGE (AFAICT).

So the current state is satisfactory I think.
Also to clarify, note that this PR doesn't change the heuristics, it just renames/moves things.

I merged the rename, you can make an issue if you think the detection can be improved.


Remove src/init.py and fix many errors

Description
Major changes:

Adds type checking on a few more files.
Expr.dtype's type is no longer Optional[DataType], because that caused type havoc. Therefore, the default is changed from None to DataType.AUTO when not set explicitly. The gtir_dtype_resolver and other visitors are updated to check for DataType.AUTO instead of None.
Moves most of the code that was in eve.iterators to eve.concepts to work around a circular dependency after solving https://github.com/GridTools/gt4py/issues/565 using @egparedes's suggestion. This is a temporary workaround until we replace Eve with the new version.
gt4py.stencil_object.ArgsInfo is made non-private and used in gt4py.dace_stencil_object to fix a type issue.
Ignores typing in eve visitor contexts -- need to fix that in a follow-up PR.
npir.HorizontalMask is created as a tuple of tuples, to avoid the Optional[AxisBound] in the common.HorizontalInterval type.
Replaces gt_backend references with gt4py.backend.

Follows #978.
Resolves https://github.com/GridTools/gt4py/issues/565.
Resolves https://github.com/GridTools/gt4py/issues/927.
Resolves https://github.com/GridTools/gt4py/issues/992.
Resolves #1007.

Looks like there's a circular import problem now, how should we proceed with this?

Looks like there's a circular import problem now

I committed the code and pushed after nearly a whole day of working out these issues, only to find out about it 😆 .

how should we proceed with this?

The circular dependency should be solved, or rather avoided, by moving to the next version of eve. But in this case a bit of analysis and moving around code definitions within this version of eve could solve it.

bors try

bors try

bors try

bors try


Not able to run Butler App through IntellIJ IDE

ATTENTION! Please read and follow:

if this is a question about Error Prone, send it to<EMAIL_ADDRESS>if this is a bug or feature request, fill the form below as best as you can.

Description of the problem / feature request:

Not able to run Butler App through Intellij IDE , throwing error

Feature requests: what underlying problem are you trying to solve with this feature?

Need to setup the backend in local environment

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

Try to run the butler app in intellij IDE using run configurations

What version of Error Prone are you using?

2.4.0

Have you found anything relevant by searching the web?

Replace these lines with your answer.
Not really

Places to look:

GitHub issues: https://github.com/google/error-prone/issues
email threads on https://groups.google.com/forum/#!forum/error-prone-discuss

Does this still happen if you upgrade to Error Prone 2.5.1?
Can you share the full stack trace by copy-pasting the text, instead of a screenshot?
Can you share an example that reproduces the crash?

This is a duplicate of https://github.com/google/error-prone/issues/1991
The fix is to update to the latest versions of the IntelliJ plugin and Error Prone.


fixes Issue#427

Overview
This Pull request makes tests more robust by comparing XML documents by using canonicalization instead of doing a simple string comparison. JSON tests are also compared by using converting python dicts to sorted lists of tuples.
Related Issue / Discussion
More information at #427
Additional Information
This PR is not complete yet. I still have to merge the more recent changes in. There are also 5 tests that are failing. These are all related to the SRU suite and I believe they fail for an unrelated issue. This still needs to be clarified though.
Contributions and Licensing
(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[ ] I'd like to contribute [feature X|bugfix Y|docs|something else] to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
[x] I have already previously agreed to the pycsw Contributions and Licensing Guidelines

@tomkralidis There are currently five tests that fail, but I believe them to be unrelated to this PR. They come from the fact that the prefix used in SRU requests seems to have changed recently. The test suite is expecting an sru prefix for the http://www.loc.gov/zing/srw/ namespace, but the server is replying with a zs prefix to this namespace. Any thoughts on this?
I'll merge the recent changes to the master branch (the elapsed time stuff) into this PR and clean up a bit and it should become ready for reviewing.

Sounds good @ricardogsilva. Let's look deeper once travis is able to run.

@tomkralidis it seems that all tests are passing in Travis now, which is a bit strange as I get 5 failed tests on my dev macchine when running under python2.7 (they all pass with python3 though). What could be the cause for the differences? Something to do with the testing database?
Another thing, I've left the older code, that performs the comparison using cmplib.cmp, in the file, still available by using a Pedantic flag, as @tomkralidis suggested. Personally I'm not a big fan of this. I'd rather just replace the old implementation with this one, because I'm thinking that the pedantic flag will end up not being used much (or at all). I'll let you decide.

@ricardogsilva can you squash your commits?

@tomkralidis I can't seem to squash cleanly. Maybe I shouldn't have made the merge with master before squashing... I've tried different ways, but I keep getting conflicts. I've got to practice rebasing more ;)
Can this go through as is?

@ricardogsilva actually GitHub gave the ability to squash and merge (nice!).  Thanks for this valuable contribution!


Include / upload time to first paint screenshots

Hey, this is really neat.
It would be great to add an option that would upload the time to first paint screenshots to the artifact section for more detailed results.

Sorry, nevermind, found it.


improve safety of KernelHeaderTmpDir()

I just noticed this in utils.cpp:
    ~KernelHeaderTmpDir()
    {
      if (path.size() > 0) {
        // move_to either did not succeed or did not run, so clean up after ourselves
        exec_system(("rm -rf " + path).c_str());
      }
    }

This ticket is a reminder to add more safety checks around this. Check the path string itself to make sure it looks like the right path.

@jordalgo You removed this in bc73244963f206814ae45ec78ebe52cd389f6381 and I think it can be closed.


Roadmap creation

A roadmap for the project needs to be laid out and prioritized in the project board, https://github.com/orgs/open-telemetry/projects/105

In the August 7th US meeting, we discussed this. I'd like to propose we adopt the project management method we use in the Spec repo, and are proposing to also use in the Contributor Experience SIG. If it works for us, it would be great if we could standardize something like this across the SIGs that do this kind of work. 😺
Project management proposal

New issues come in as no status.
New issues are triaged at the beginning of every meeting.
If the issue is in scope for the project, and it seems reasonable that we might be able to accomplish it sometime this year, it is moved to accepted.

Non-accepted issues are closed.

Community members thumbs up to indicate interest.
Community members assign themselves to issues if they are willing/available to actually do work.
We choose which issues we actively work on from the list of most thumbed up issues that have a sufficient number of assignees.
These issues are moved to in progress.
We limit ourselves to four issues at a time, in order to concentrate our efforts and complete these issues as fast as possible.
Once we have finished work on an issue, it is moved to completed.

If for some reason it turns out that we cannot complete an issue, we stop working on it and it is either moved back to accepted or it is closed.


fix disable case test_leading

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


cmd: add interactive shell

https://github.com/gitql/gitql/pull/98 and https://github.com/gitql/gitql/pull/97


Function construct reporting wrong Runtime

Our app has mostly NodeJS Lambda functions using NodeJS16.X but we also have a couple of Python3.9 functions. We've recently discovered that no matter what runtime is configured on the SST function construct, all functions report the runtime as being NodeJS16.X.
I've tested changing the default function runtime to NodeJS14.X but all functions still report that the runtime is NodeJS16.X. The only way that I can get the other runtimes to report correctly is by circumventing the SST Function construct altogether and using the AWS CDK Function construct.
This can be reproduced by simply creating a new SST app from the examples/rest-api-python template and printing the runtime of the functions in the API using the following snippet:
const sstStack = stack as Stack;
sstStack.getAllFunctions().forEach((f) => {
  console.log(`Runtime: ${f.runtime}`);
});

Hey @CodyDunlap, functions is first created with placeholder runtime (ie. Nodejs 16) and placeholder handler. And at the end of the build process, all Functions are built together in parallel and gets replaced with the real runtime+handler.
I think we can make it use the real runtime. Let me prioritize this.
Btw, can you share why you need to check the runtime?

Sure! We are trying to instrument our functions with monitoring tools. We've been exploring a few different tools but they all use Lambda layers to some degree to provide metrics for the function invocations. To do this well, they provide different layers for each Lambda runtime, so we need to know the runtime of each function in our SST code so that we know which Lambda Layer to attach to the function.

I believe I'm running into this issue via getStack. I use this in unit tests to check that template output is as expected (similar to the example in the docs).
I'd like to check that every Function is using nodejs18.x, but this isn't reflected in test output.
Minimal test repro:
import { App, Function, type StackContext, getStack } from 'sst/constructs';
import { initProject } from 'sst/project';
import { Template } from 'aws-cdk-lib/assertions';
import { it } from 'vitest';

function TestStack({ stack }: StackContext): void {
  new Function(stack, 'example-function', {
    functionName: 'example-function',
    handler: './x',
    runtime: 'nodejs18.x',
  });
}

it('function runtime should be nodejs18.x', async () => {
  await initProject({});
  const app = new App({ mode: 'deploy' });
  app.stack(TestStack);
  const template = Template.fromStack(getStack(TestStack));

  template.hasResourceProperties('AWS::Lambda::Function', {
    Runtime: 'nodejs18.x',
  });
});

Are you running this against your local stage?

Are you running this against your local stage?

@jayair In the test, the stage name test-user is used, i.e.:
const name = 'app-name';
const stage = 'test-user';
await initProject({ stage });
const app = new App({ mode: 'deploy', stage, name });

app.stack(...);

This stage name doesn't correspond to any real stack in my AWS account. I'm not sure what the implications of this are.

Hmm interesting. But when deployed does this set the right runtimes?

Interesting. We might need to dig in to see why that's happening.

Interesting. I'll ask the team to take a look.

Did you make sure to do await app.finish() in your test?

Did you make sure to do await app.finish() in your test?

@thdxr Thanks for the tip, this isn't mentioned in the docs.
Adding await app.finish() causes the error: "nodejs18.x runtime is unsupported" - so it is acknowledging the runtime setting at this point, but seems to be unsuitable for unit tests.
I also tried without specifying a non-default runtime, and still get the error "nodejs16.x runtime is unsupported"

@tobz1000 just updated the doc — https://docs.sst.dev/testing#testing-infrastructure

Note that app performs several tasks asynchronously, including building the function code and container images. It then modifies the template. To ensure you are testing against the finalized resources, call await app.finish() before running any tests.

Can you share a screenshot of the error w/ more context?

@fwang Sorry to say I was testing on an old version... With sst updated to 2.29, adding await app.finish() fixes the test. Thanks for the help/docs.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Environment variables

I'd like to make some server environment variables available to CLJS, through something like environ. Does cljx enable this sort of functionality?

No. Depending on when you're compiling your ClojureScript, that's something you'll want to splice into your CLJS source via a macro, or make available in your JS runtime explicitly (e.g. always load something like "window.__environment = {…}" first, and refer to it in your ClojureScript via js/window.__environment).

Thanks. Wound up using a macro.


fix: Add allowed callers list to skill configuration settings

Description
Add allowed callers list to skill configuration settings
Task Item
Closes #4222
Screenshots

Coverage decreased (-0.01%) to 51.137% when pulling dc019e78ab9d809c5f5b0346bcfffb4bc4b7f149 on durnford/fix/allowed-callers into b90b17b6e19f4f294ca12af654c7b11c6dc6c919 on main.


gfx.h file name clash

Class name clash was resolved during lib build by forcing a local search first, thus we avoided nested namespaces, however file name clash will result in a single header placed under includes on install. That header will be the last one hit and not necessarily the one corresponding to the library you might try to build and link against (ili9341, st7735, ssd1351 all use slightly different versions of this header).
Short term solution: Rename header for each individual library using it as so: $libname_gfx.h
Long term solution: Merge header into single file and provide it as part of upm abstract infrastructure.

Header files have been renamed for now as of v0.6.1.


Password change does not require confirmation password

When a password change is requested, the New password again does not need an entry

This is against 6.0.27 version of Stroom

Fixed but not released yet