Entire PC bluescreens upon connecting wiimote.

https://user-images.githubusercontent.com/62120245/226066887-cc9fb24c-731f-4a03-8a40-48850cab38ab.mp4
I have reinstalled vigembus multiple times, and restarted my pc, but the error still persists.

Well, that's not good lol
I really hope it's not some more memory corruption stuff I need to look into, that's gonna be painful to work out. I had to do that for the most recent couple versions, but I already had a suspicion on what was happening that time. Here I've got no ideas.

Here's a picture of the error it throws up before I blue screens. It was very hard to take a picture of, as it only shows up for around half a second before the PC crashes, but it might help.

That error seems rather nonsense, unfortunately. Can you upload the WiitarLog.txt file inside your WiitarThing folder? Assuming it didn't get corrupted from the crash...
I'll see if there's anything I can do to test for memory corruption. I may also have to get in touch with the ViGEmBus developer about the bluescreen, so I'll need the memory dump file Windows created:

Go to C:\Windows and copy the MEMORY.DMP file to your desktop (do not move it, you will get file permission issues otherwise)
Use 7-Zip to compress it: right-click the dump, go to 7-Zip > Add to archive..., set the compression level to Ultra, and start it. It will take a while to finish.
Once it's done, upload the compressed file to something like Google Drive and post the link here. I'll let you know when I've grabbed it so you can delete it.

Alright, downloaded it. I figured the log file would be blank, very high chance of corruption when a file gets written very shortly before a crash lol
When I get a chance I'll investigate further, thanks for posting an issue!

I finally got around to checking the crash dump, it's definitely ViGEmBus at fault here but the error code is different than what I see in the video. I definitely suspect some more memory corruption, if it were an actual bug I'd assume it would be the same code.
Aside from that though, it may be worth it to try reinstalling ViGEmBus if you haven't yet. At some point before I got those initial memory corruption issues fixed, I started getting ViGEmBus crashes of a similar nature which were fixed after a reinstall. Dunno what made it happen in the first place lol

Hey, just checking in: the blue screen also occurs when using x360ce, a separate ViGEmBus-based application, so it's definitely something related to that.

Yeah, definitely sounds like you should try reinstalling ViGEmBus.
I still haven't gotten to doing any memory corruption tests yet, but since it's occuring with other programs I'm more doubtful it's an issue on WiitarThing's side. I still want to do that though, who knows what other issues are hiding in this codebase...

i have the crash without the bluescreen when pressing sync
i created a seperate user for my pc and it doesnt present the error on there which is weird

i found out what was causing it something to do with my color profile set heres the fix https://www.pingman.com/kb/article/pingplotter-is-giving-an-error-for-systemwindowsbaml2006typeconvertermarkupextension-how-do-i-fix-it-153.html

I thought I fixed that bug already lol, I probably need to make a similar fix somewhere in the sync menu.


add bulma support

Hi!
I've added bulma support.
All tests are passed!
Bye!
Lorenzo

Awesome! Didn't even know about that one... Would you also update the README to advertise it? (and be sure to fix where it says there are three supported frameworks)

Ready to merge... Would you mind updating the readme as well?

Yes, but I did not push because the final rendering is not entirely equal to bulma template;
the "previous" button and the "next" button should be outside of the unordered list.
I'm working for fix this problem.

Ok, np. Let me know when its ready!

This branch has merge conflicts now... Did you get the issue you mentioned fixed now?

I'd like to merge this, but I don't know what kind of state it is in, and it has conflicts... If this sits for too long I plan to close this PR.

@mgwidmann / @lorenzopagano I'm happy to complete or issue a new PR; there is complexity with the variations of the position variations of the Bulma pagination, was going to opt for the centered version described here ~ http://bulma.io/documentation/components/pagination/
let me know.

If you'd like to branch off of this PR or submit a completely new one that'd be great. I didn't want to resolve the conflicts myself since I'm not too familiar with that CSS framework.

@mgwidmann I've issued another PR with the latest version of Bulma on there; take a look and let me know what you think :-)

Closing in favor of @b-murphy's PR


Fix format on IMPORTANT block.

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

Looks good; thanks!


Incorrect SQL query when using skip/take (IQueryable) functions

Hi all,
I have been fighting against the Skip/Take functions for the last couple of days. I'm using a very simple code in charge of filtering/returning entities matching a set of criteria. Everything is working fine except the Take/Skip part.
        [HttpGet]
        public async Task<List<GetBookingResponse>> GetBookings(GetBookingRequest request)
        {
            var query = Context.Bookings
                .Include(x => x.Resource)
                .AsQueryable();

            if (!string.IsNullOrWhiteSpace(request.TecTacLocationId))
                query = query.Where(x => x.ResourceId.HasValue && x.Resource.TecTacLocationId == request.TecTacLocationId);

            if (request.Start.HasValue && request.End.HasValue)
                query = query.Where(x => x.Start >= request.Start && x.End <= request.End);
            else if (request.Start.HasValue && !request.End.HasValue)
                query = query.Where(x => x.Start >= request.Start);
            else if (!request.Start.HasValue && request.End.HasValue)
                query = query.Where(x => x.End <= request.End);

            var data = await query
                .Skip(request.Offset)   // 0
                .Take(request.Limit)    // 15
                .AsNoTracking() // tried with this commented out.
                .ToListAsync();

            return Mapper.Map<List<Core.Entities.Booking>, List<GetBookingResponse>>(data);
        }

This should generate a SQL query with a top (15) statement. Instead, i'm getting the following query:
SELECT [x].[Booking_Id], [x].[ActualPrice], [x].[CanEdit], [x].[EndDate], [x].[TimeEnd], [x].[IsVcBooking], [x].[ListedPrice], [x].[Memo], [x].[Resource_Id], [x].[Date], [x].[TimeStart], [x].[BookingStatus], [x].[TecTacClientId], [x].[TecTacContactId], [x].[BookingType], [x].[Version], [x.Resource].[ResourceId], [x.Resource].[ResourceCode], [x.Resource].[ResourceDisplayName], [x.Resource].[TecTacLocationId], [b].[ResourceId], [b].[ResourceCode], [b].[ResourceDisplayName], [b].[TecTacLocationId]
FROM [Booking] AS [x]
LEFT JOIN [BookingResource] AS [x.Resource] ON [x].[Resource_Id] = [x.Resource].[ResourceId]
LEFT JOIN [BookingResource] AS [b] ON [x].[Resource_Id] = [b].[ResourceId]
WHERE [x].[Resource_Id] IS NOT NULL AND (([x.Resource].[TecTacLocationId] = @__request_TecTacLocationId_0) AND [x.Resource].[TecTacLocationId] IS NOT NULL)
ORDER BY [x].[Resource_Id]

Further technical details
EF Core version: 1.1.2
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Visual Studio 2017
Thank you,
Sebastien

Does this give correct results?

I didn't check because I have approx 841,000 bookings in my table. It takes a massive amount of times to return the record set... I have no other choice but to do the paging at the SQL Server level

There are few shortcomings or EF you are hitting in this case.
#6206 - Using include & navigation filter on same navigation causes multiple joins. Hence you get Left Join with same table.
#6647 - Left join always pulled data from both the tables.
With 2 left join like above, I believe top(15) on the generated SQL would produce incorrect results (not enough rows from first table will be selected) due to 2 left joins. Perhaps that is the reason we did not translate it to server.
Both of the above issues are fixed in 2.0.0 so it should generate much optimized query. @maumar - Can you verify generated SQL in this kind of query?
As for work-around in 1.1.2 version, @sroche-tec can you try writing query without include part and see if it translate Take to server. If it does then you can include the related data with explicit loading or a separate query to achieve result you want. In that case, you would be doing 2 queries but not pulling so much of data.

Hey, thanks for your answers. I have tried to comment out the Include part.
            var query = Context.Bookings
                //.Include(x => x.Resource)
                .AsQueryable();

            if (!string.IsNullOrWhiteSpace(request.TecTacLocationId))
                query = query.Where(x => x.ResourceId.HasValue && x.Resource.TecTacLocationId == request.TecTacLocationId);

            if (request.Start.HasValue && request.End.HasValue)
                query = query.Where(x => x.Start >= request.Start && x.End <= request.End);
            else if (request.Start.HasValue && !request.End.HasValue)
                query = query.Where(x => x.Start >= request.Start);
            else if (!request.Start.HasValue && request.End.HasValue)
                query = query.Where(x => x.End <= request.End);

            var data = await query
                .Skip(0)
                .Take(15)
                .ToListAsync();

            return Mapper.Map<List<Core.Entities.Booking>, List<GetBookingResponse>>(data);

AS a result, EF core generates the following query:
SELECT [x].[Booking_Id], [x].[ActualPrice], [x].[CanEdit], [x].[EndDate], [x].[TimeEnd], [x].[IsVcBooking], [x].[ListedPrice], [x].[Memo], [x].[Resource_Id], [x].[Date], [x].[TimeStart], [x].[BookingStatus], [x].[TecTacClientId], [x].[TecTacContactId], [x].[BookingType], [x].[Version], [x.Resource].[ResourceId], [x.Resource].[ResourceCode], [x.Resource].[ResourceDisplayName], [x.Resource].[TecTacLocationId]
FROM [Booking] AS [x]
LEFT JOIN [BookingResource] AS [x.Resource] ON [x].[Resource_Id] = [x.Resource].[ResourceId]
WHERE [x].[Resource_Id] IS NOT NULL AND (([x.Resource].[TecTacLocationId] = @__request_TecTacLocationId_0) AND [x.Resource].[TecTacLocationId] IS NOT NULL)
ORDER BY [x].[Resource_Id]

Any idea? Shall I update to 2.0.0 preview (any breaking changes coming)?

as 2.0 is major version change, there are some breaking changes for certain. You can upgrade but you may need to update code also for breaking changes.
Thanks for the generated query. Actually lets try the other way around. Keep include as is. (I am hoping that will translate the take correctly) but remove the where predicate on x.Resource.TecTacLocationId. Apply that predicate after ToListAsync part. Once you have included navigation and loaded data in memory, you can still apply the where predicate in memory without any more database query.

Interesting! I found one possible root cause. I removed all where clauses and suddenly it started to work. Actually x.Start and x.End are not database fields. They are calculated properties:
public DateTime? Start => StartDate?.Add(StartTime ?? TimeSpan.Zero);
public DateTime? End => EndDate?.Add(EndTime ?? TimeSpan.Zero);

Removing those filters solve the problems. In the end, it works with the include and even with the
where pred on x.Resource.TecTacLocationId.
Thx for your help!
Thx
Seb

So it was client evaluation of where predicate. I did not think of it, my mistake.
Its good all worked out in the end. 😄

I am using version 2.1 of EF and I present a similar problem with a table of 25000 records, when using Skip / Take and Include beyond the record 17000 does not return records, but if I delete the Include clauses everything works more or less well.


active-directory-conditional-access-automatic-device-registration-setup.md

There are several typos in the script for the multi-domain claims rules. I fixed them.

Hi @pirminf, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@pirminf : Thanks for your contribution to the Azure documentation! The author, @MarkusVi, has been notified to review your proposed change.

Thanks a lot for your feedback, @pirminf , this is highly appreciated.
You hit the nail on its head.
@sign-off


When referencing dll compiled with .net  standard 2, intellisense in vs code stop working.

I create a minimal test case.

The problem is in intellisense is not working when i reference a dll copiled with .net standard 2.0
The code is avalaible in https://github.com/gabomgp/test-case-bug-bakery
In the code linked, the build.cake is inside a folder with a project in .net standard 2.0. But, the problem is showed too when the build.cake is outside that folder, and inside the solution's folder, where there are other projects, some projects compiled with .net standard 2.0 and others with .net 4.6.

I think, this issue is related to https://github.com/OmniSharp/omnisharp-roslyn/issues/996
But the solution in that issue dosn't work in .cake files.

I'm getting the same thing, but with a normal .NET Framework assembly. Nothing .NET Core in the picture.
Just add #r System.IO.Compression.FileSystem and OmniSharp fails. @mholo65 should I start a new issue for this or is it connected?

@jnm2 ah, I see.. Don't think we actually handle #r directives correctly in Bakery. In Cake itself it's easier, since we use reflection and that will automatically load assemblies from GAC based on only assemblyname. With this bakery/omnisharp-thingy we expect paths to DLLs.... @patriksvensson any ideas?

I ran into the same problem today with #r System.Net.HttpClient.

@jnm2 I created a separate issue for GAC loading here #85
@gabomgp about netstandard2.0, that's not even supported in Cake itself. Why would like it to be supported via intellisense? netstandard2.0 will work eventually, but that will take a little more effort.

@gabomgp said:

But the solution in that issue dosn't works in .cake files. The solution is:

#! "netcoreapp2.0" directive at the top of your script to force the assembly resolution to assume .NET Core 2.0 mode.

Yes, we don't currently support that directive in Cake.

@mholo65 The bigger issue is that I didn't need the #r line for Cake's sake, only OmniSharp's sake. Is there an issue tracking keeping OmniSharp in sync with Cake's default assembly references?

@jnm2 oh, totally missed that. No there's no issue for that. Would you mind submitting one.

Sure. https://github.com/cake-build/bakery/issues/86

@mholo65 The reason is that i want to regenerate the whole database and populate that with entity framework core and cake as the task runner. But, the models and the context are projects compiled with .net standard 2. I must use the context and models to use code first migrations with entity framework core. to create the empy database and populate it easily in developer's machines. I cant't do it from cake because the code is in a .net standard 2 library. So, that's the reason :)

@gabomgp I'm sorry but I don't understand this. Cake does not support netstandard2.0 in any way, so why would it help to have support for it in Bakery? You simply can't reference a netstandard2.0 assembly in a cake script until we've merged the work done in Cake for supporting it.
Would you mind elaborating on why you would need Bakery support for this when it's not supported by the Cake script runner?

@gabomgp wouldn’t it make more sense to use the .NET Core EF Command Line Tools for dropping and creating your database? You can still use Cake as task runner and no need to reference DBContext in your Cake script.

It's a good alternative. Thanks for the suggestion. I'll try it.

I'm running into this issue with VS Code on macOS:

Oddly enough, the Intellisense itself is working. Also, there are no errors in the Omnisharp log and the only symbol where I'm getting this error on is the Task alias.
@mholo65 Do you have any suggestions on how to resolve this?

@ecampidoglio hmm, seems like some reference is not passed correctly to Roslyn. What version of Cake are you using? Are you using Cake, Cake.CoreCLR or Cake.Tool? What version of Cake.Bakery are you using? What version of C# for VS Code are you using? Do you use the default OmniSharp version shipped with the VS Code extension, or do specify another version? Do you have a minimal script where this can be reproduced?

Cake.CoreCLR: 0.33.0
Cake.Bakery: 0.4.1
C# for VS Code: 1.19.1 (including OmniSharp)

I'll see if I can reproduce this in a minimal script.
Btw, everything works as expected with the exact same repo on Windows.

This happens every time with the simplest of scripts:
Task("Hello")
    .Does(() =>
{
    Information("World");
})

Error message:

The type 'Action' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. (CS0012) [build.cake]

Here's the OmniSharp log:

OmniSharp server started.
Path: /Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/run
PID: 36994

    Starting OmniSharp on MacOS 10.13.6 (x64)

    DotNetPath set to dotnet

    Located 1 MSBuild instance(s)
        1: StandAlone 15.0 - "/Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin"

    MSBUILD_EXE_PATH environment variable set to '/Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin/MSBuild.dll'

    Registered MSBuild instance: StandAlone 15.0 - "/Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin"
        MSBuildExtensionsPath = /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/xbuild
        BypassFrameworkInstallChecks = true
        CscToolPath = /Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin/Roslyn
        CscToolExe = csc.exe
        MSBuildToolsPath = /Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin
        TargetFrameworkRootPath = /usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/xbuild-frameworks

    Detecting Cake files in '/Users/Enrico/Sources/Cake.Bakery.Issue81'.

    Found 1 Cake files.

    Added Cake project '/Users/Enrico/Sources/Cake.Bakery.Issue81/build.cake' to the workspace.

    Project system 'OmniSharp.DotNet.DotNetProjectSystem' is disabled in the configuration.

Hmm, might be something with the Mono version on your system (or with the one shipped with OmniSharp). Could you try playing with the omnisharp.useGlobalMono in VS Code and see if it helps?
Also, can you reproduce using Cake or Cake.Tool?
@gep13 @devlead are you able to reproduce on Mac?

Setting the useGlobalMono option to always resulted in the same error.
Here's the log:

OmniSharp server started with Mono 5.20.1.
Path: /Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/OmniSharp.exe
PID: 1379

    Starting OmniSharp on MacOS 10.13.6 (x64)

    DotNetPath set to dotnet

    Located 2 MSBuild instance(s)
        1: Mono 15.0 - "/usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/msbuild/15.0/bin"
        2: StandAlone 15.0 - "/Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin"

    Registered MSBuild instance: Mono 15.0 - "/usr/local/Cellar/mono/<IP_ADDRESS>/lib/mono/msbuild/15.0/bin"
        CscToolExe = csc.exe
        CscToolPath = /Users/Enrico/.vscode/extensions/ms-vscode.csharp-1.19.1/.omnisharp/1.32.19/omnisharp/.msbuild/Current/Bin/Roslyn

    Detecting Cake files in '/Users/Enrico/Sources/Cake.Bakery.Issue81'.

    Found 1 Cake files.

    Added Cake project '/Users/Enrico/Sources/Cake.Bakery.Issue81/build.cake' to the workspace.

    Project system 'OmniSharp.DotNet.DotNetProjectSystem' is disabled in the configuration.


rbd: callback related cleanups

The first patch is the more significant change, removing an  not-needed layer of indirection between Go and C language callbacks. Now, the "callback registry" is even more vital as all callbacks are static in C and only when returning to the Go layer, do we dispatch to a user-specified function.
The second path simply makes some naming changes that we had discussed in a review call, but I got an itchy trigger finger and merged stuff anyway. This naming was all internal stuff anyway and wouldn't have impacted any users of the library.
Checklist

[x] Added tests for features and functional changes
[x] Public functions and types are documented
[x] Standard formatting is applied to Go code

FWIW, as a learning experience I basically did the same stuff as you here, where I found out all the things that I comment here. Also I found a clean way to create fake pointers, that can be used as void* argument. That simplifies that code furthermore. I have a PR in the making that I will post later.

@ansiwen  I updated the patches. PTAL when you have a moment. Thanks!

@nixpanic PTAL


Fix "Name cannot end with a delimiter: Program Information. in Options"

add readUnicodeStringUntrimmed function
On importing a PE file, I get these exceptions in the console window:
2022-07-18 12:15:42 ERROR (ResourceDataDirectory) Invalid resource data: Name cannot end with a delimiter: Program Information. in Options  java.lang.IllegalArgumentException: Name cannot end with a delimiter: Program Information. in Options
	at ghidra.framework.options.AbstractOptions.validateOptionName(AbstractOptions.java:721)
	at ghidra.framework.options.AbstractOptions.getOption(AbstractOptions.java:215)
	at ghidra.framework.options.AbstractOptions.putObject(AbstractOptions.java:295)
	at ghidra.framework.options.AbstractOptions.setString(AbstractOptions.java:529)
	at ghidra.framework.options.SubOptions.setString(SubOptions.java:227)
	at ghidra.app.util.bin.format.pe.ResourceDataDirectory.processVersionInfo(ResourceDataDirectory.java:418)
	at ghidra.app.util.bin.format.pe.ResourceDataDirectory.markup(ResourceDataDirectory.java:358)
	at ghidra.app.util.opinion.PeLoader.load(PeLoader.java:133)
	at ghidra.app.util.opinion.AbstractLibrarySupportLoader.doLoad(AbstractLibrarySupportLoader.java:356)
	at ghidra.app.util.opinion.AbstractLibrarySupportLoader.loadProgram(AbstractLibrarySupportLoader.java:84)
	at ghidra.app.util.opinion.AbstractProgramLoader.load(AbstractProgramLoader.java:116)
	at ghidra.plugin.importer.ImporterUtilities.importSingleFile(ImporterUtilities.java:368)
	at ghidra.plugin.importer.ImporterDialog.lambda$okCallback$7(ImporterDialog.java:351)
	at ghidra.util.task.TaskBuilder$TaskBuilderTask.run(TaskBuilder.java:306)
	at ghidra.util.task.Task.monitoredRun(Task.java:134)
	at ghidra.util.task.TaskRunner.lambda$startTaskThread$0(TaskRunner.java:106)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.lang.Thread.run(Thread.java:833)

I tracked this down to readNextUnicodeString in BinaryReader.java:
public String readNextUnicodeString() throws IOException {
	String s = readUnicodeString(currentIndex);
	currentIndex += ((s.length() + 1) * 2);
	return s;
}

The problem is that the String returned by readUnicodeString is trimmed and its length is being used to compute indices.
The assumption being that length() characters were read, but that is incorrect if the String had whitespace at the beginning or end.
This pull requests adds a readUnicodeStringUntrimmed function that is appropriate for using length() to compute indices.
It would also be possible to make a different PR that has a function return a combination of {trimmed string, original length}.
Let me know if anything should be changed in this PR.
I looked in the issues and pull requests and could not see anything related to this.
Let me know if I should also create an issue for this.

So, while I hate that the method has a built-in trim(), and I would love any excuse to stamp it out, wouldn't this same problem be triggered by a binary having a string in the PE headers that really did end in a '.'?
I think a different approach needs to be taken for these strings when adding them to the program's info.

I think so, yes. I just fixed the case I ran into with whitespace padding but not actually ending in '.'.

@bostick by the way, the same trimmed string issue applies to BinaryReader.readAsciiString too.  You may want to update your patch...

@agatti Good point.
So this is growing legs now.
Should the immediate problem of calling length() on trimmed strings be fix here? I can also update readAsciiString in this PR.
And figure out what to do with strings that really do end in a '.' in a separate PR?
Or possibly stop using trim() altogether? Is doing trim() only a nicety for users?

I've reworked the string stuff in BinaryReader, so don't worry about doing any more changes.  If I could, I would convert this PR to an issue, but I'm not seeing that as an option.

I added readUnicodeStringUntrimmed and readAsciiStringUntrimmed. This is bare minimum change to get broken code working.

@ryanmkurtz This was closed but my changes to BinaryReader.java were not included.
The problem that I described with using trim() is still present. There are no more exceptions in the console window after your merge, but there are still other problems with program information being corrupted because of trimmed string lengths being used to set the index.
Is there any feedback about what should be changed for my changes to BinaryReader.java to get picked up?

@dev747368 can you please weigh in?

@bostick, sorry about worrying you.  I was going to add a comment to ryan's status change to address the binaryreader side of things, but didn't make it in time before you commented.
The changes to BinaryReader will be coming in another branch pretty soon.  They are bit more invasive than your pr.

Any update on the branch that is coming?

It's in internal code review.


Update scribe-slf4j to 3.2.2

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.outr", artifactId = "scribe-slf4j" } ]

labels: library-update, semver-minor

Superseded by #146.

Superseded by #146.


Fix Yarn caches on Vercel

https://github.com/orgs/vercel/discussions/222 - one of these suggestions should help.
This would speed up Vercel builds by 1.5 minutes for each package.

Fixed by #1906.


Can´t open mapbox-studio in Win 7 64bits

------ WHAT'S YOUR ENVIRONMENT?
I'm on a French Windows 7 Enterprise 64bit.
I downloaded the 64bit installer, version 0.2.7,
from https://www.mapbox.com/mapbox-studio/
------ WHAT'S YOUR DATA?

All the following steps
https://github.com/mapbox/mapbox-studio/wiki/Windows-Troubleshooting
.

How can I fix and start it?
------ LOG
Microsoft Windows [versão 6.1.7601]
Copyright (c) 2009 Microsoft Corporation. Todos os direitos reservados.
C:\Users\r337555>cd "C:\Program Files\mapbox-studio
C:\Program Files\mapbox-studio>mapbox-studio.exe
C:\Program Files\mapbox-studio>[5832:0711/073611:INFO:CONSOLE(0)] <Sat, 11 Jul 2
015 10:36:11 GMT> Mapbox Studio exited with<PHONE_NUMBER>.
[7224:0711/073611:WARNING:ipc_channel_win.cc(309)] Unable to create pipe "\.\pi
pe\chrome.5832.0.192276493" in client mode: O sistema nÒo pode encontrar o arqui
vo especificado. (0x2)
cd "C:\Program Files\mapbox-studio\resources\app"
C:\Program Files\mapbox-studio\resources\app>vendor\node index.js

@sagazluiz thanks for your detailed report.
Please try this:
Make sure via Task Manager that no mapbox-studio.exe or node.exe processes are running.
If there are, and you already have exited Mapbox Studio please kill them via right click in Task Manager.
1.) download
https://mapbox.s3.amazonaws.com/windows-builds/visual-studio-runtimes/vcredist-VS2014-CTP4/vcredist_x64-mini.7z
2.) extract vcredist_x64-mini.7z into
C:\Program Files\mapbox-studio\resources\app\vendor
3.) extract vcredist_x64-mini.7z into
C:\Program Files\mapbox-studio\resources\app\node_modules\mapnik\lib\binding\node-v11-win32-x64
If the above steps don't work, delete this folder:
C:\Users\<YOUR-USER-NAME>\AppData\Roaming\mapbox-studio
Any success?

Hi BergWerkGIS,
Thank you so much for your attention! It worked.


Try solving perf-ninja puzzles

Hello, just discovered your paper, this is great work!
Do you want to try your best model in solving perf-ninja puzzles?
https://github.com/dendibakh/perf-ninja
I'm the main author and I'm very curious to see what modern LLMs can do for perf optimizations.
-Denis

Hi Denis! Thanks for your interest! We don't have the spare funds for GPT credits to use the best model and time right now to get to this.
We've open sourced our fine-tuned codellama models here and the code for generating edits in this project. You could start with those, or you could also try to see how GPT-4 or GPT-4o works for this task. Given that we fine-tuned many of these models directly for competitive programming problems: the patterns these models learn are likely to be more high-level; whereas it seems on a quick glance perf-ninja is more low-level. As a result I'm not sure these specific "best models" will be an ideal fit. Happy to chat more if you'd like about this as well!
I'll close this issue, but feel free to drop me an email if you want to get in touch!


License

Hi!
I find the license information you provided a bit confusing. In the GitHub repo, the license file links to a Creative Commons Attribution 4.0 International License, which allows commercial use, while in the README you link to a Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International License. Could you clarify this?
Great work and thanks a lot!

Hi @timpressive,
I appreciate your interest in our work. Please note that we use LLaMA-v1 as our base LLM which requires a Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International License.
I hope it will help. Thank you.


Enable 200 or 201 for signing success.

Currently the client code rejects any response from the signing url where the status != 200. This doesn't work well with backend frameworks which use 201 for POST created success.

Thanks

Pushed v4.3.0 with this change

Excellent, thanks @seanadkinson


timer crash

panic: runtime error: index out of range [recovered]
panic: runtime error: index out of range
goroutine 2078 [running]:
gitlab.**/service/timer.(*jobItem).call.func1()
/home/gofile/src/service/timer/job.go:65 +0x52
panic(0xc306c0, 0x11eefb0)
/usr/local/go/src/runtime/panic.go:489 +0x2cf
thanks！！！

Should be caused by the application layer callback cross-border, not clock problems, check such problems, I hope you can give me some suggestions? Thank you

Thank you  for the suggestion that I have ignored the handling of the error after the interception. The new submission should meet your current needs.


Privacy setting 'none' isn't saved

When I set any of the privacy settings to 'none', it won't be saved. Higher privacy levels are saved.
If I don't want to share something, I have to manually set it to 'none' after login or UI reload.

Hi, i think i have tracked down this bug, the addon checks that the privacy ranks exists, as in the rank hasnt been removed since your last session. I didnt add a check for the nobody rank, this is fixed for the next update.


[WIP] go-orm proof of concept example

Currently fails due to issue #383
This is a proof-of-concept of constructing a Go-orm instance backed by GenjiDB.

https://github.com/paralin/genji/tree/gorm-example-mysql-dialect
Here's a variant which uses the mysql dialect instead of the sqlite one. I think AUTO_INCREMENT is not supported (right?) so that part is disabled.

Thanks @paralin ! I would love to make Genji compatible with Gorm! I think what you did is more than just an example and deserves a separate repo. I propose to create a repo in the genjidb org so then you can create a PR there, what do you think?

@asdine Sorry never saw your reply for some reason, whatever you want to do with this example / demo, let me know.

I think there are enough features to make it work; at least last time I tried it
But I'd have to spend some time digging through the code to figure out how I made it work.
If anyone really needs this urgently feel free to ping here and I'll see if I can post a repro.

@paralin There is no rush, I don't yet have time to tackle this issue, if you want to do it let me know at anytime and I'll re-open it and answer any question you may have


WinUi 3 Version 1.0 Preview 2 (1.0.0-preview2)  issue

Errors with latest preview 2
The AppWindowConfiguration class has been removed. The properties of this class is now available on the AppWindow itself, or on the Presenter classes.

Fixed


VS2013 Compile error:cpp_redis/network/tcp_client_iface.hpp(41): error C2039: “uint32_t”: not “std” member

I build the solution  for the x64 platform .
Error info:
cpp_redis/network/tcp_client_iface.hpp(41): error C2039: “uint32_t”: not a member of “std”

Hi,
Thank you for reporting that issue!
I just pushed a fix in which I included cstdint inside tcp_client_iface.hpp.
I'm closing this issue but feel free to reopen it if necessary.
Best :)


Don't override the highlight of hexokinase

No matter the priority of highlight what I set, illuminate always override the highlight of hexokinase.

Maybe add a blacklist for some specified pattern?

Unfortunately this cannot be changed based on the highlight priority since hexokinase uses nvim_buf_add_highlight to place highlights on the buffer which does not utilize the priority mechanism that is used with matchadd and matchaddpos. In terms of a black list, this also would not work because illuminate does not place highlights down manually, it does so using matchadd which matches a single pattern so any blacklist would have to be combined with the main pattern with backreferences (and even then I'm not sure if it would work).
That being said, to fix the example in your screenshot, you can use let g:Illuminate_highlightUnderCursor = 0.

I used https://github.com/neoclide/coc-highlight before, which contains the feature of illuminate and hexokinase, also use rpc to call nvim_buf_add_highlight to highlight, but it can handle the override color.
I think illuminate and hexokinase are written by the same author, they should get along with each other.

They do get a long with each other, they each perform their function, it just so happens the function of vim-illuminate is to [highlight] other uses of the word under the cursor which can sometimes be colour values. While you say coc-highlight handles the override, I would argue it is backwards to disable the curword matching for the colour highlighting, just look at vim-matchup (vim-matchup would take precedence). vim-illuminate is highlighting temporary locations in your code while vim-hexokinase is building on top of the syntax highlight. On top of that, coc-highlight uses nvim_buf_add_highlight for both whereas vim-illuminate uses matchadd and was written before nvim_buf_add_highlight was implemented.
I think if coc-highlight does both illuminate and hexokinase, then you should just use it since it's clearly more suited to your needs.


[webview_flutter] Keyboard language can't change on Android

My flutter project use webview_flutter v0.3.14+1 and there is a problem when I try to text with Chinese or other keyboard language, the original keyboard always override the correct keyboard language. As you can see below, the chinese keyboard will show and disappear quickly, and you can't even change the language from the keyboard.
This is what I get in my application:

Does anyone have any idea or also struggle with the same issue here?
Thanks in advance for your help!
Logs
[✓] Flutter (Channel unknown, v1.7.8+hotfix.4, on Mac OS X 10.14.5 18F132, locale zh-Hant-TW)
    • Flutter version 1.7.8+hotfix.4 at /Users/bigstarpie/development/flutter
    • Framework revision 20e59316b8 (9 weeks ago), 2019-07-18 20:04:33 -0700
    • Engine revision fee001c93f
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/bigstarpie/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.7.5

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.38.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.4.1

[✓] Connected device (3 available)
    • Android SDK built for x86 • emulator-5554 • android-x86    • Android 9 (API 28) (emulator)
    • macOS                     • macOS         • darwin-x64     • Mac OS X 10.14.5 18F132
    • web                       • web           • web-javascript • Google Chrome 76.0.3809.132

• No issues found!

It looks you are on an older Flutter version. Could you switch to stable and do a flutter upgrade?

Hello @escamoteur , I have updated the latest stable version, but still have this problem.
[✓] Flutter (Channel stable, v1.9.1+hotfix.2, on Mac OS X 10.14.5 18F132, locale zh-Hant-TW)
    • Flutter version 1.9.1+hotfix.2 at /Users/bigstarpie/development/flutter
    • Framework revision 2d2a1ffec9 (2 weeks ago), 2019-09-06 18:39:49 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/bigstarpie/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.7.5

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.38.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.4.1

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

does this only happen on iOS?
Could you also give it a try on latest master please?

No, this only happen on Android, iOS works perfect.
I switch to v1.10.6-pre.63, but the problem still exists.
[✓] Flutter (Channel master, v1.10.6-pre.63, on Mac OS X 10.14.5 18F132, locale zh-Hant-TW)
    • Flutter version 1.10.6-pre.63 at /Users/bigstarpie/development/flutter
    • Framework revision cc3ca9a916 (23 minutes ago), 2019-09-25 10:57:58 -0400
    • Engine revision 63949eb0fd
    • Dart version 2.6.0 (build 2.6.0-dev.0.0 69b5681546)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/bigstarpie/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.7.5

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.38.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.4.1

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

It also happens with Japanese keyboards.
@mklim
Is this the same problem you know below?
https://github.com/flutter/engine/blob/6ac4f9fdf6c0fc5de53a9032a5d81ba0fad59fe6/shell/platform/android/io/flutter/plugin/platform/SingleViewPresentation.java#L285-L289

Thanks for trying, I'm sorry but we have to wait for the Flutter team now.

I found a trick to change a language in that situation. When tapping outside the text field, it loses its focus but the keyboard does not disappear🐛. Then tapping inside the text field again, it gets its focus and now the keyboard works well.

is any update on this? I have same problem 😭

I have the same problem too. Initially, the keyboard works, but after the application collapses and reopens, the keyboard stops working. Test on Samsung s10e, google pixel 3xl, Redmi note 4

Also, TextField does not update when tapped on the keyboard after the application was collapsed and reopened

Couldn't reproduce this with Cyrillic or Arabic keyboards. webview_flutter v0.3.19, flutter v1.15.3, Gboard, Android 10

@bparrishMines this is one of four webview issues marked as critical.  How's your bandwidth for being able to look at these?

Note that webview keyboard support is not ready for production use on Android, and this issue is just one of the reasons.
With the current allocation webview keyboard is probably not going to be considered stable before Q3 2020.
I think the current state of this issue reflects reality (a far away milestone, even though it's customer critical), though maybe we want to consider removing the customer: critical just so it stops surfacing on weekly issue triages?

Same issue #54648

can someone provide a URL that reproduces this issue?

can someone provide a URL that reproduces this issue?

https://github.com/flutter/plugins/tree/master/packages/webview_flutter/example project can reproduce this issue.

Install Gboard app, and add other languages in it, such as Chinese(“简体中文”)
Run the above official example project
Click the search box in the left drawer of the flutter site, then the keyboard will show, and CANNOT change languages.

We're aware that this is a high priority bug but we are unable to work on it at this time. We continue to look at this bug every week and will update here as soon as we are able to work on it. Sorry for the delay.

(Assigning per conversation with @blasten; @bparrishMines please let me know if I was wrong to assign this to you!)

This has been fixed in Flutter 1.22/ WebView 1.0 - use hybrid composition in the Flutter WebView plugin.
https://github.com/flutter/plugins/tree/master/packages/webview_flutter#using-hybrid-composition


[BUG] [WCAG] All pages - Page title needs to be more descriptive.

Describe the bug
Currently, the page titles are all "Reach4Help". Users who interact using a reader will not know what the page they've navigated to until the reader gets through the top part of the page (including the top-nav menu).
In order to be WCAG compliant, each page needs to have a unique title that describe the topic or purpose.
To Reproduce
Steps to reproduce the behavior:

Navigate to the Reach4Help website.
Navigate through different pages of the website.
Notice all page titles are "Reach4Help"

Expected behavior
Each page has a unique title that appears visually in the browser's window/tab.
Screenshots

Desktop (please complete the following information):
Windows 10 - Chrome - Incognito.
Additional context
https://www.w3.org/WAI/WCAG21/quickref/#page-titled

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

@isma-osy @sabind @ashwinkjoseph
I think I can do this if i just map the current route to a title string, add the title string to en.json, and add that as a subtitle

I would qualify this as a change request because its changing the app's behavior
This is post-MVP and needs UX team lookibng into it

@isma-osy I need to clear up a merge error but this is fixed

Fixed and deployed on https://app.reach4help.org/


WaveMixerStream32 in Universal (UWP)?

WaveMixerStream32 is not present in the NAudio Universal assembly.
Is there a way to emulate its functionality with the available types for UWP?

I think it was just that I didn't want to turn on the "allow unsafe code" flag for the Universal project and WaveMixerStream32 requires that.  You could just copy the code verbatim into your own project - no need for it to be part of NAudio. WaveMixerStream32 is really a legacy class now that I don't intend to support going forward - I'd recommend using MixingSampleProvider where possible instead.

No
Le 4 sept. 2017 19:22, "Mark Heath"<EMAIL_ADDRESS>a écrit :

I think it was just that I didn't want to turn on the "allow unsafe code"
flag for the Universal project and WaveMixerStream32 requires that. You
could just copy the code verbatim into your own project - no need for it to
be part of NAudio. WaveMixerStream32 is really a legacy class now that I
don't intend to support going forward - I'd recommend using
MixingSampleProvider where possible instead.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/naudio/NAudio/issues/232#issuecomment-327005092, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHQRwb7bmsbqwu5bp63eo-l6p9c9hu2fks5sfDG7gaJpZM4PLpbo
.

Noted. Thanks.


Build system changes to allow compiler toolchain override

E.g., works with Clang as
make clean && make debug.linux CC=clang-3.5 CXX=clang++-3.5 AR=/usr/bin/ar RANLIB=/usr/bin/ranlib \
  EXTRA_CFLAGS="-Wno-unknown-warning-option -Wno-sign-conversion -Wno-sign-compare -Wno-nested-anon-types -Wno-c99-extensions -Wno-float-conversion -Wno-null-conversion -Wno-invalid-noreturn" \
  LTO=OFF

(Works on x86_64/linux at least.)

Also enables the compilation with afl-fuzzer's wrapped gcc/g++ ( #333 ) but link fails because generated code depends on functions available in the compiler-default libc only.
Blocked by #424

FYI: missing sign-off dco.

@akiss77 which commonly accepted flags from the list below do you like to support and what behavior do you expect?

AS
CC
CXX
CFLAGS
CXXFLAGS
 LDFLAGS
ASFLAGS

CC, CFLAGS, CXX, CXXFLAGS, and LDFLAGS would certainly be good, while the definition of AR and RANLIB seems also be needed sometimes (although quite some documentation discourages their definition saying that CMake should figure that out by itself). AS and ASFLAGS are also nice to have (for the sake of completeness), however, I haven't seen a case where they would have been needed yet.
As for the behaviour: use the specified compilers and archive tools, as well as use the specified compiler/linker options instead of the defaults. (Same for the assembler if implemented.)

@akiss77 great! We see this the same way. Please update this PR with support of CC, CFLAGS, CXX, CXXFLAGS, and LDFLAGS - I agree that other flags are not needed yet. Provided options should override defaults.

Can you please elaborate why we need to be able to specify the path to the compiler binary (and the other build tools) via environment variables?
With CMake-based projects you usually just set the path with -DCMAKE_C_COMPILER=... when you configure the project.

@akiss77 do you have any plans to update this PR?

@egavrin I had an offline discussion about this with Akos a while ago. The conclusion was that this can be closed.

Closed after offline discussion.


Fix for sort bug when switching sort column sort is descending

Included tests. Tried not to mess up the existing tests too much...

Thanks for following my conventions, really nice work!

fixed https://github.com/iamisti/md-data-table/issues/39


Services: first build: abs added

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_9eyrtt88zv/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps and --no-build-isolation.
ERROR: /tmp/abs_9eyrtt88zv/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_9eyrtt88zv/clone/recipe/meta.yaml:43: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_9eyrtt88zv/clone/recipe/meta.yaml:43: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_9eyrtt88zv/clone/recipe/meta.yaml:43: missing_description: The recipe is missing a description
ERROR: /tmp/abs_9eyrtt88zv/clone/recipe/meta.yaml:43: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_1btor3t5ke/clone/recipe/meta.yaml:44: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_1btor3t5ke/clone/recipe/meta.yaml:44: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_1btor3t5ke/clone/recipe/meta.yaml:44: missing_description: The recipe is missing a description
ERROR: /tmp/abs_1btor3t5ke/clone/recipe/meta.yaml:44: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Can't create negated condition (AND NOT)

Describe the bug

Can't create a negated condition.
No option to create a negated condition with 1 rule.
Also, it looks like a regular AND condition when I create it via API calls.

To Reproduce

Try to create a segment rule, there is no option to negate it.
When you create a regular segment rule, I am able to convert it by updating its type in database. It looks like a normal condition with one rule, but in reality, that condition should be negated.

Expected behavior
We should be able to add AND NOT conditions.
Screenshots
https://github.com/Flagsmith/flagsmith/assets/7023385/7030326c-a353-480c-b8f7-dbb228fb81c4
How are you running Flagsmith?

[x] Self Hosted with Docker
[ ] Self Hosted with Kubernetes
[x] SaaS at flagsmith.com
[ ] Some other way

Additional context

Turns out the frontend made an incorrect assumption about the naming of the NOT operator.
Closing in favour of the initial implementation ticket  https://github.com/Flagsmith/flagsmith/issues/382
CC @kyle-ssg


C3 crashes the browser with large data points

I am creating six  line charts where i need to append data to c3 charts, I am using React and Flux after some time my browser crashes is it just because of C3 or too much number of DOM elements

duplicate with #1591


Motion planning panel

Added motion planning panel (under teleop panel).
the new panel includes

end-effector planning
walking planning
fixed-feet collision free motion planning

Currently, walking planner calls the default foot step planner. End-effector and collision free planners call either matlab IKserver or exotica to solve IK and planning problems.
Some of the parameters are not passed to exotica, one still needs to setup the parameters and options via exotica xml config file.
@MarcoCar

cool!  Would love to see some screenshots or video demo :)

I am not in the lab right now, I will create some screenshots and videos tomorrow morning.

Hi Pat @patmarion ,
Here is the video link https://youtu.be/RpnrVSp_VJk
There are 3 sub panels under motion planning,

end-pose planning;
walking planning
fixed-feet motion planning.

The action sequence should be: first plan an end-pose (goal configuration and stance pose), then use walking planner to walk to the stance pose, and finally use fixed-feet motion planning algorithms to generate a collision free trajectory.

The director needs to be launched with -exo (exotica option) to use the collision-free end-pose planner (iDRM, inverse Dynamic Reachability Map) and Exotica-OMPL motion planning algorithms.

If run director without -exo, the default algorithms will be used, e.g. IKServer and RRTConnect.

looks great!  I made a few inline comments.  I think this is ready to merge after those are resolved.

@yimingyanged : could I get a demo before we merge this? I want to make sure this panel can be shared with Marco and to see how it might be improved to get it running on Val
It seems great though.
This panel is closer to supporting proper collision free motion planning than the telep panel but I see planning a something that is called from within the task sequencing so the task sequencing needs to talk to elements and setting in this panel

I was copying stuff from footstepsdriver to make things work without looking into the details. Now I removed the unnecessary codes:

removed interactive component (since we don't manually set walking goal)
removed uneven terrain stuff (for now, we only consider flat floor in motion planning)

Looks good.  @mauricefallon merge when ready.


Pr example

1. Add/Update Component [{component}.tsx]
2. Add/Update Component Demo [{component}-{variant}-example.tsx]
3. Update Sidebar (Only Applies To New Additions)
4. Create Docs File
5. Update Registry UI
6. Update Registry Examples

Closing as this is only for example to view the files needing cahnged


Frontend app throws 400 error after updating the additionalLoginParams in resource explorer

I am following the exact same steps for our front-end and back-end apps but after updating the additionalLoginParams with backend client id front end app starts throwing 400 errors. Let me know if there is any new steps that needs to be added for it to work correctly. Using linux app service.
Please see below logs.
019-08-20T14:42:57.603134952Z info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
2019-08-20T14:42:57.603186753Z       Request starting HTTP/1.1 POST http://my-ui.azurewebsites.net/.auth/login/aad/callback application/x-www-form-urlencoded 2388
2019-08-20T14:42:57.869451789Z info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
2019-08-20T14:42:57.869486889Z       Request finished in 266.0319ms 302
2019-08-20T14:42:58.156264205Z info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
2019-08-20T14:42:58.156338807Z       Request starting HTTP/1.1 GET http://my-ui.azurewebsites.net/
2019-08-20T14:42:58.165491922Z info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
2019-08-20T14:42:58.165550023Z       Request finished in 9.1053ms 400

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

ID: 8ff46628-b5dc-8b02-02b6-8f510d8059b0
Version Independent ID: 24d09b70-d144-d152-ac5a-c49219a5d8e8
Content: Authenticate and authorize users end-to-end - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-auth-aad.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

So we figured out the issue, in Frontend application configuration setting there is below property that needs to be set to true.
WEBSITE_AUTH_DISABLE_IDENTITY_FLOW: true

Please update the documentation with the above property so people know how to configure this as there is some kind of issue with the linux app easyAuth flow.

@heetvachhani
Thanks for the feedback! We are currently investigating and will update you shortly.

@cephalin can we get the doc updated with screen grab of WEBSITE_AUTH_DISABLE_IDENTITY_FLOW set to true in the Application Settings blade as mentioned by @heetvachhani.

@mattchenderson Can you confirm the requirement on Linux for front-end applications to have WEBSITE_AUTH_DISABLE_IDENTITY_FLOW set to true?

I updated the doc this week and did not reproduce this requirement to set the app setting. No action. #please-close


There's no sounds?

Im using this but is that there's no sound or it just still not implemented yet?
When i tried mirroring only my phone has Sounds but not with on My Laptop
Phone : Redmi 9

https://github.com/Genymobile/scrcpy/blob/master/README.md#audio-forwarding

https://github.com/Genymobile/scrcpy/blob/master/README.md#audio-forwarding


Investigate interruption of FulgoraGraphComputer

Hi Dan, Can you please provide more of a description for this ?  Is this still applicable even with the new GraphComputers in addition to the HadoopGraph processing ?


fix SNOMED CT Code for "Patient" in FHIR R4 Export

This is to fix https://github.com/synthetichealth/synthea/issues/726

Thank you for the pull request!


Fix some test failures for bpf_nat_test.c

This PR fixes two test failures on the latest master.
BPF Unit Test failure for bpf_nat_test.c
https://github.com/cilium/cilium/pull/24212 introduces a known bug of Coverbee into the code base (https://github.com/cilium/coverbee/issues/7). This only becomes a problem when we run the test with the affected code (the code includes nat.h or conntrack_map.h). We introduced the feature to disable coverage report per file to prevent CI from failing. The latest exclusion pattern is also introduced in https://github.com/cilium/cilium/pull/24212 (https://github.com/cilium/cilium/pull/24212/commits/9a274dbe7f07a97b2a1749f067c75e91eecd4276).
Yesterday, we merged the PR that introduces a new test file affected by the bug https://github.com/cilium/cilium/pull/18414 (https://github.com/cilium/cilium/pull/18414/commits/07edf550cab7f865ef83522061256183a803109e), but we couldn't notice that because the BPF Unit test CI ran without the change introduced in https://github.com/cilium/cilium/pull/24212.
Coccicheck failure for bpf_nat_test.c
https://github.com/cilium/cilium/pull/24392 "fixed" the Coccicheck, which was not running correctly for a while. This was merged on May 21st. https://github.com/cilium/cilium/pull/18414 introduced the code which fails with Coccicheck but ran the check on May 20th, which was before the fix merged. That's why we couldn't notice the failure before the merge.
Fix some test failures for bpf_nat_test.c

BPF Unit Test and Cocchicheck recovered. Once reviews are in, I'll revert the preview commit.

Modified tests were passed, and reviews are in. I don't think it does make sense to run e2e tests since we only have changes to BPF Check related code.


Add tests for buffer overflows

In exercises where the solution is written to a buffer, there should be a test for buffer overflows. Buffer overflows are a a common error source in C so we should help our users to avoid them.

Thanks!


Fix r and s size to 32-bytes

This pull request resolves #281.
The AWS KMS signs and responses with a signature formatted by ASN.1. And ASN.1 format serialize integer with \x00 byte as prefix if it can be confused with negative integer. It serializes with only least bytes. So the integers, r and s, can be 33-bytes and 31-bytes.
The parameter signature of secp256k1.ecdsaRecover requires 64-bytes consists of 32-bytes r and 32-bytes s. So it is checking the condition.
https://github.com/odanado/aws-kms-provider/blob/ca5d7da4bfbf382cd2c14002d5efb0ba62b8ad25/src/crypto.ts#L33-L38
But the original code didn't care 31-bytes case. So I tried to fix the bug.

The E2E test seems failed by problem about credentials.. 😢  Could you check it? @odanado

The reason why the E2E test is failing is that the secrets isn't being passed to the workflow.

Note: With the exception of GITHUB_TOKEN, secrets are not passed to the runner when a workflow is triggered from a forked repository.
https://docs.github.com/en/actions/reference/encrypted-secrets#using-encrypted-secrets-in-a-workflow

@moreal
Could you merge the upstream master branch? E2E tests should then be running on GitHub Actions.

@moreal
Could you merge the upstream master branch? E2E tests should then be running on GitHub Actions.

I rebased this pull request on the master branch. Could you check it? 🙏🏻

:)

Released: https://github.com/odanado/aws-kms-provider/releases/tag/v0.3.4


VSCode server does not respect my default shell setting

Bug description

I adopted option 1 to enable vscode server in wsl. But when I open the terminal in vscode, it goes to sh instead of bash which I actually set as default.
Inside wsl, $SHELL is /run/current-system/sw/bin/bash. However, in vscode, $SHELL becomes /nix/store/sh4d0lnbrapnqjsn1jwb4r07lmwirc8c-sh/bin/sh.
To Reproduce
Steps to reproduce the behavior:
Here is what I set in modules:
nixos-wsl.nixosModules.default
({ pkgs, ...}: {
  system = {
    stateVersion = "24.05";
  };

  programs.nix-ld = {
      enable = true;
      package = pkgs.nix-ld-rs;
  };

  wsl = {
    enable = true;
    useWindowsDriver = true;
    defaultUser = "${specialArgs.username}";
  };
})

and this is how I set my default shell:
environment.shells = with pkgs; [
  bash
];
users.defaultUserShell = pkgs.bash;

Logs
I don't know if it's related but I printed the $PATH in vscode's integrated terminal anyway.
sh-5.2$ echo $PATH
/home/yang/.vscode-server/bin/384ff7382de624fb94dbaf6da11977bba1ecd427/bin/remote-cli:/usr/lib/wsl/lib/:/home/yang/bin:/home/yang/.local/bin:/run/wrappers/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/usr/lib/wsl/lib:/home/yang/.nix-profile/bin:/nix/profile/bin:/home/yang/.local/state/nix/profile/bin:/etc/profiles/per-user/yang/bin:/nix/var/nix/profiles/default/bin:/run/current-system/sw/bin:/mnt/c/Windows/system32:/mnt/c/Windows:/mnt/c/Windows/System32/Wbem:/mnt/c/Windows/System32/WindowsPowerShell/v1.0/:/mnt/c/Windows/System32/OpenSSH/:/mnt/c/Program Files (x86)/NVIDIA Corporation/PhysX/Common:/mnt/c/Program Files/NVIDIA Corporation/NVIDIA NvDLISR:/mnt/c/WINDOWS/system32:/mnt/c/WINDOWS:/mnt/c/WINDOWS/System32/Wbem:/mnt/c/WINDOWS/System32/WindowsPowerShell/v1.0/:/mnt/c/WINDOWS/System32/OpenSSH/:/mnt/c/Program Files/Git/cmd:/mnt/c/Program Files/Microsoft SQL Server/150/Tools/Binn/:/mnt/c/Program Files/Microsoft SQL Server/Client SDK/ODBC/170/Tools/Binn/:/mnt/c/Program Files/dotnet/:/mnt/c/Program Files/Docker/Docker/resources/bin:/mnt/c/Users/fyang/AppData/Local/Microsoft/WindowsApps:/mnt/c/Users/fyang/AppData/Local/Programs/Microsoft VS Code/bin:/mnt/c/Users/fyang/.dotnet/tools:/nix/store/nswmyag3qi9ars0mxw5lp8zm0wv5zxld-systemd-255.9/bin:/nix/store/3jc4m5gwimj4mbm01ijgkm2di8hiy5l3-gnugrep-3.11/bin:/nix/store/nswmyag3qi9ars0mxw5lp8zm0wv5zxld-systemd-255.9/bin:/nix/store/3jc4m5gwimj4mbm01ijgkm2di8hiy5l3-gnugrep-3.11/bin:/nix/store/nswmyag3qi9ars0mxw5lp8zm0wv5zxld-systemd-255.9/bin:/nix/store/3jc4m5gwimj4mbm01ijgkm2di8hiy5l3-gnugrep-3.11/bin

WSL version

WSL Version： <IP_ADDRESS>
Kernel version： <IP_ADDRESS>-2
WSLg version： 1.0.59
MSRDC version： 1.2.5620
Direct3D version： 1.611.1-81528511
DXCore version： 10.0.26100.1-240331-1435.ge-release
Windows version： 10.0.22631.4317

I found a related issue here. It seems that enabling the wsl.warpBinSh option may have caused some problems and needs further attention.

I remember wsl.warpBinSh is working fine before. I think commit 5a7c64a broke it but I am not sure.


The qrCode of DingTalk Group has expired in README.md

Please check it.

Thanks, I' will update it.

Fixed in #1000


npm run deploy fails

npm run deploy fails due to following:
Syntax error: Missing : after property ID. line: 11 file: webpack.config
Files to push were:
└─ appsscript.json
└─ dist/code.js
└─ dist/index.html
└─ node_modules/ajv/.tonic_example.js
└─ node_modules/gas-entry-generator/.eslintrc.js
└─ node_modules/gas-webpack-plugin/.eslintrc.js
└─ node_modules/hash.js/.eslintrc.js
└─ node_modules/npm/node_modules/ajv/.tonic_example.js
└─ webpack.config.js

Not sure why but started from scratch and it is working now.  I did have to npx clasp create first.  If this is necessary please update docs.  Otherwise thank you so much for creating the project.  It now makes apps script dev and deploy a pleasant experience!!

@russellkt , please look at Getting Started Did you do what is said in point 3?

Replace the "scriptId" value in the .clasp.json file.


TOML Inline Tables?

Is it possible to serialize to TOML with inline tables?
Currently serialization will emit a section such as:
[tool.uv.sources.myapp]
path = "./src"

Where I'd prefer:
[tool.uv.sources]
myapp = { path = "./src" }

I've tried with both toml::to_string() and toml::to_string_pretty()
Thanks!

See https://github.com/toml-rs/toml/issues/592#issuecomment-1659101635


GetScanStatus API results in 404

Describe the bug
Wanted to try this new API and see how it goes. However, i am getting a 404 on request. Even from the "try it" on the page itself. Postman/Powershell report 404.
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/rest/api/power-bi/admin/workspaceinfo_getscanstatus
Click on "Try it"
Try the request.
Get 404 error on API. Either the URL is wrong or api is not functioning.

Expected behavior
A working API endpoint
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 87.0.4280.88

Smartphone (please complete the following information):
Additional context

@eligr Eli can you take a look?

I get this same thing for this and GetScanResult. Is this used at all with Azure Purview by any chance to gather data for scan results? If it is I'm wondering if that's partially why my scans are all failing.

Looping in the correct team.

This is the blog describing the scanner API flow: blog describing the scanner API flow: https://powerbi.microsoft.com/en-us/blog/announcing-new-admin-apis-and-service-principal-authentication-to-make-for-better-tenant-metadata-scanning/
In regards to the scan status API, which is part of this flow, we just released a patch, hopefully, it'll resolve your issue.

@eisuissa It now seems to work after the patch for the scanstatus and scanresults with no 404! 👍

Closing the issue since the 404 is resolved.

Closing the issue since the 404 is resolved.


fix accessibility warnings

Found in Ghost Inspector test

Document should have one main landmark
Page should contain a level-one heading
All page content should be contained by landmarks

Would need to shadow/extend/overwrite the theme's index file, since this would require adding a wrapping element to the root of each page.

Opened PR instead of shadowing: https://github.com/LekoArts/gatsby-themes/pull/796


MAIN - pr preview

Happy holidays @VivyTeam/web !
I thought that gifs are cool but previewing our PRs is even ✨✨cooler✨✨.
I am fun of surge these days and wanted to try it out. Its super cool for deploying static websites.
When I say "super cool" I mean easy to use 😎
Have a look and let me know what you think!
Also since I didnt communicate this with anybody its also fine if you say we that this is not something we would like to have. I will then close the PR and no big problem 👍

Warnings

:warning:
Missing reviewer - Looks like this pull request is missing a reviewer. That's OK as long as this PR is still a 'work in progress', if that is the case please add 'WIP' in the PR title or corresponding label.

Messages

:book:
✅  Get a 🏆 for adding description on this PR. - Celebrate on every step!

  Generated by :no_entry_sign: dangerJS against 91311054bc87c80eb452c20907c73bb2540e8f28

@lanwen @Weetbix please have another look! Now the url that is being generated will be ${CIRCLE_BUILD_NUM}-${CIRCLE_PROJECT_REPONAME}-${CIRCLE_PROJECT_USERNAME}.surge.sh

CIRCLE_BUILD_NUM : The number of the CircleCI build.
CIRCLE_PROJECT_REPONAME: The name of the repository of the current project.
CIRCLE_PROJECT_USERNAME: The organisation name.

For instance 1828-vivy-components-vivyteam.surge.sh
As we discussed with Kirill in private the idea was to use branch names as part of the url. However branch names can cause the pr-preview surge command to fail when / is part of the branch name. I couldnt find a way that in yml files we can replace that part of the string. if one of you can find one I am happy to implement it.
Please let me know your thoughts and feedback on this!


Implement more formatting traits

It'd be nice if every struct implemented LowerExp, UpperExp, Binary, Octal, LowerHex, and UpperHex.
These would make it a lot nicer to print things like BigInt and Ratio, which currently only have Debug and Display.

See also: rust-lang/rust#39479.

num-complex now has all of these.
num-bigint has all but LowerExp and UpperExp, which could actually be interesting for large numbers, especially if we can figure out how to do that quickly, without full digit-by-digit division.
num-rational still needs them all.

Closing in favor of separate issues, rust-num/num-bigint#6 and rust-num/num-rational#1.


Make NamedView conform to View

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.66%. Comparing base (4a83a91) to head (16c1eba).

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   96.66%   96.66%
=======================================
  Files           1        1
  Lines         120      120
=======================================
  Hits          116      116
  Misses          4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Release/1.0.0 - solution-pack-fortimanager-ztp-feature-examples

Please review and merge for release.
This does require the pending pull request of the new FortiManager ZTP Flow Solution Pack ( https://github.com/fortinet-fortisoar/solution-pack-fortimanager-ztp-framework/pull/2 ).

@fortijames Since this solution pack only includes demo records, it's recommended to use a scenario to create the same. [Refer to https://fortisoar.contenthub.fortinet.com//detail.html?entity=socSimulator&version=1.0.2&type=solutionpack]

Please exclude the "modules" folder from the solution pack, as the "FortiManager ZTP Flow" pack already provides these modules.

@fortijames Since this solution pack only includes demo records, it's recommended to use a scenario to create the same. [Refer to https://fortisoar.contenthub.fortinet.com//detail.html?entity=socSimulator&version=1.0.2&type=solutionpack]

I don't know if this will work very well for this framework. These records are not SOC Simulations. I might need some help going through what this would look like.


Can't find class MyRestApi - docs are incorrect - nothing works

Just the first example works.  #2 and #3 don't work, can't find the classes no matter what I try.
Would like to use this but documentation is lacking way too much and I can see syntax errors in your examples.   Do better.

Why you close without fixing your docs?


Fullpage Screenshot Enhancement

I'm struggling to get full page screenshots working again after firefox finally started taking only screenshots of the visible area.  Realistically I've disabled updates on firefox and am not updating canopy until I can get this working properly.  I'm hoping someone else has had this issue and knows of a reliable way to get the old functionality back.  I think it would be a pretty nice to have enhancement if it could be done.

Full page meaning the entire length of the page even if it requires scrolling down?

Hmm some googling shows that the Selenium w3 spec says to only take a screenshot of the viewport.
https://github.com/seleniumhq/selenium/issues/1141
I have no clue how to implement this, I simply wrap the selenium call and if it fails, fall back to windows take a screenshot of the entire desktop + open apps.

@whiteleyj @lefthandedgoat Any update found? Share with us..

I've been exploring some methods but they all seem to have issues.  So far I'm working on a vertical panorama stitching, but it's using image magick stuff which would be a weird dependency to add to the project.  If I get something working properly I'll try to put it out as it's own thing at this point.

looking forward to new support

@whiteleyj @lefthandedgoat @timsheng @KevM @charlesj
I have tried aShot. It worked like a charm.
Check this post. Hope it helps.


Update the Naked Domain information for App Services Managed Certificates

[Enter feedback here]
please remove the "Public Preview" information for the App Service Managed Certificates as it is now GA
Also on the remark 12, please remove the "Naked domains aren't supported" as it is current is supported. This is the full sentence:
12 Naked domains aren't supported. Only issuing standard certificates (wildcard certificates aren't available). Limited to only one free certificate per custom domain.

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

ID: 36f49a06-90b1-cbe1-3853-2c38277b79f8
Version Independent ID: 02ff6f6c-3ef6-b48f-060d-b290419e0aba
Content: Azure subscription limits and quotas - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/azure-subscription-service-limits.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@coelho5br Thanks a lot for bringing this to our notice and helping us improve our Azure documentation. I have raised a PR to update the document based on the feedback shared. Once the PR is reviewed and approved , changes would go live.
At this point it appears that there's no further edits or updates needed to the documentation, other than what has been listed above. We will go ahead and close this issue for now, however if you have any further feedback on the document please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


Could we ignore widget forward-char to accept a suggestion?

In some cases, I typed forward-char to step ahead char-by-char to accept partial suggestion.
For example, I first type cd and then the suggesstion shown up is cd ~/.zsh/zsh-autosuggestions. In this case, if the directory I really want to change to is ~/.zsh, only thing I can do is  type every chars instead of using forward-char.
Could we ignore widget forward-char to accept a suggestion?
ZSH_AUTOSUGGEST_ACCEPT_WIDGETS=(
-   forward-char
    end-of-line
-   vi-forward-char
    vi-end-of-line
)

See my comment here to configure the forward-char widget to partially accept the suggestion. I'd like to keep forward-char accepting the suggestion by default as it does in the fish shell.

Understood, thanks.


More project info

Will description, BOM and diagrams be available?

Please see the kicad subdirectory

Are there plans to improve the circuit design of the board or is this the final version?

I have had the PCBs made and completed building a few pieces of this adapter. They are working pretty well, so that would be a yes. I cannot rule out slight changes in the future though, especially the PCB layout.

It is not.

It is not. The USB-C is now reserved to programming and debugging purposes.

It is not. The USB-C port is now reserved for programming and debugging purposes.

https://github.com/Franticware/usb-to-ps1-mouse-pro/tree/main/wiring


Migrate ads state to MST

Child of wp-pwa/wp-pwa#72

[x] reducers
[x] actions
[x] selectors
[x] components

Blocked until #285

Done at 0926db472571854fb05edd1a48a1384748113aeb.


Repeated should implement From et. al

Repeated is just a thin wrapper over vec. Instead of exposing just method, should implement from.
This way people can call into() and not worry about importing the crate just to call from_vec
Since it can’t fail this seems even more appealing.
Usecase: I only use protobuf as client. Importing protobuf crate just to call from_vec seems unnecessary.

Please reopen if this issue is still relevant.


Error while handling response code 422

It seems that gidgethub is not able to handle a part of the 422 response. This happened when the app tried to make a review comment on a line that was not part of the pull request diff. I didn't know that was not possible but this mistake revealed this bug in the code.
{
  "message": "Unprocessable Entity",
  "errors": [
    "Line must be part of the diff"
  ],
  "documentation_url": "https://docs.github.com/rest/reference/pulls#create-a-review-for-a-pull-request",
}

The code that triggered the error, taken from gidgethub.sansio.decipher_response:
                exc_type = InvalidField
                if errors:
                    if any(
       ------>          e["code"]
                        in ["missing", "missing_field", "invalid", "already_exists"]
                        for e in errors
                    ):
                        error_context = ", ".join(repr(e.get("field")) for e in errors)
                        message = f"{message} for {error_context}"

Logged traceback:
2020-12-10T14:57:43.757626+00:00 app[web.1]:     await gh.post(
2020-12-10T14:57:43.757626+00:00 app[web.1]:   File "/app/.heroku/python/lib/python3.8/site-packages/gidgethub/abc.py", line 175, in post
2020-12-10T14:57:43.757626+00:00 app[web.1]:     data, _ = await self._make_request(
2020-12-10T14:57:43.757626+00:00 app[web.1]:   File "/app/.heroku/python/lib/python3.8/site-packages/gidgethub/abc.py", line 114, in _make_request
2020-12-10T14:57:43.757627+00:00 app[web.1]:     data, self.rate_limit, more = sansio.decipher_response(*response)
2020-12-10T14:57:43.757627+00:00 app[web.1]:   File "/app/.heroku/python/lib/python3.8/site-packages/gidgethub/sansio.py", line 340, in decipher_response
2020-12-10T14:57:43.757628+00:00 app[web.1]:     if any(
2020-12-10T14:57:43.757628+00:00 app[web.1]:   File "/app/.heroku/python/lib/python3.8/site-packages/gidgethub/sansio.py", line 341, in <genexpr>
2020-12-10T14:57:43.757628+00:00 app[web.1]:     e["code"]
2020-12-10T14:57:43.757628+00:00 app[web.1]: TypeError: string indices must be integers

Ugh, the one thing about error cases that seemed to be consistent isn't consistent.

A quick fix would be to check whether the key code exists in the errors object, otherwise accumulate whatever is in the errors object and combine it with the message:
error_context = ", ".join(repr(e) for e in errors)
message = f"{message}: {error_context}"

Feels like it's a bug with with GitHub, since the behavior described is not documented at all.
https://docs.github.com/en/rest/overview/resources-in-the-rest-api#client-errors
Maybe worth reporting the problem with GitHub, so they can either fix the doc or the response.


email contact: set up and  install new email address

currently we use a gmail adress for contacting people (confirmation mail, password reset, individual contact of speakerinnen)
is it possible to have an<EMAIL_ADDRESS>address?
if not, Anne will create<EMAIL_ADDRESS>and we have to implement it.

@tyranja haven't we talked about that recently?

Ja, we did not decide the name of the mails. I keep forgetting that. I wrote a mail.

So that is done now. The only thing that we have to do is to redirect every conversation to team@ instead of<EMAIL_ADDRESS>right!? @tyranja (when you are back)

done!


feat: made entire terms and services page accessible 🚀

Related Issue
Closes #1047
Description

JavaScript Changes:

Added role="main" to the main content container to indicate the main content of the page for better accessibility.
Changed the heading element from <h2> to <h1> for improved semantic structure and added an id attribute for association with section element which contains multiple p element in it. This helps us the accessibility devices to properly announce the content to the end user.
Added aria-labelledby attribute to the content section to establish a relationship with the heading for better screen reader navigation.
Added role="list" to the unordered list and role="listitem" to list items to convey the list structure to assistive technologies.
Improved the aria-live and aria-describedby attributes for the contact email link, ensuring screen readers can announce updates and providing a description for better understanding.

CSS Changes:

Simplified the list item icons by removing inline SVGs and using the ::before pseudo-element with custom icons added through CSS. This enhances accessibility and reduces code redundancy.
Removed the pulse animation from the heading "Terms and Service" to improve accessibility and create a more straightforward and concise design.
Enhanced the overall UI with refined color choices, well declared font weights, and better visual organization, ensuring a visually appealing and accessible page.

These changes significantly improve the accessibility of the Terms and Service page, making it more user-friendly for all individuals, including those using assistive technologies. The updated CSS also enhances the visual appeal of the page while maintaining accessibility standards.

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Hey there @crocmons, @rohansx 👋🏻
Please make sure to review this PR.
This PR comes under level3 change because this is one of the core contribution to this project which enhances overall accessibility for the entire Terms and Services Page of Informatician Website with significant number of code changes and refactoring the codebase to make entire accessible site.
You can further refer this manual here for more details about adding level labels to PR.
Thank you.


How to use in window?

怎么在window系统上使用? 小白提问

I can not find the installation of the options, there is no exe installation files, and thank you for your answer

it's right there https://github.com/cesarferreira/dryrun#installation


Dry run

Emit the information about the tables you would have backed up and/or restored, but dont actually perform them

Fixed by https://github.com/schireson/databudgie/pull/89


Refactor Columns and add ColumnNames

Address feedback around the Column APIs from the first API review

    public void InsertColumn(int columnIndex, DataFrameColumn column)

Should we continue to have these, or instead would it be more natural to make the .Columns property a mutable collection (of some sort)?

Refers to: src/Microsoft.Data/DataFrame.cs:54 in 23bc683. [](commit_id = 23bc6837fb745194ace20bcbee1ba310fee305b5, deletion_comment = False)

Should we continue to have these, or instead would it be more natural to make the .Columns property a mutable collection (of some sort)?

How do you feel about delaying this until we absolutely need it? We already have an OnColumnsChanged "event", but other than that, I don't see a strong reason to make a new type?

How do you feel about delaying this until we absolutely need it?

Ask yourself a different question. Why would we have an IReadOnlyList of Columns, but then have InsertColumn, RemoveColumn, etc. methods on the DataFrame class? What would be a more natural API? Modifying the collection itself, or asking the owner of the collection to modify it?

I don't see a strong reason to make a new type

It's fairly simple to make this type. See all the derived types from https://docs.microsoft.com/en-us/dotnet/api/system.collections.objectmodel.collection-1. Specifically the ones that are in corefx:

System.Net.Mail

AttachmentCollection
LinkedResourceCollection
MailAddressCollection

CngPropertyCollection
SyndicationElementExtensionCollection

Doing so also follows our Guidelines for Collections


Fix the registry kwarg

I think we should change the kwarg in line 255 (above) from ~dest_registry~ to registry as well.

And have two registry kwargs?


Update README.md NEWS section

reference end of support for 9.1 LTS

I propose that we skip the azure test for this PR. Please like 👍 if you agree


More Groqd Playground features

Some more improvements to the playground:

Loading indicator on fetch
Cmd + Enter in editor will trigger a fetch in the playground
twoslash-like type hints on runQuery to automatically show return type.

Yeah, the code I borrowed intentionally stripped all newlines to keep it to one line – but I agree that it gets a bit hard to read 🤔 I'll noodle some more on that.


This library is not working for  Andriod 12 + version

this library not woking for  for Andriod 12 + version , all support  configure file and permission any one idea please help us

try pulling PR #7 locally, this should fix android 12 build

this library not woking for for Andriod 12 + version , all support configure file and permission any one idea please help us

#7 PR merged. Please try again.


Mirroring the refresh icon to work with Tailwind's animate-spin

Thanks for the amazing work!
Currently the direction of the arrows on the refresh icon points anti-clockwise, so when combined with Tailwind's animate-spin utility, it appears to be spinning backwards.
If this was mirrored (which is admittedly easy enough to do ad-hoc) it would fix the issue.
I'm not too sure how to PR this with the Figma file, optimized directory, etc.
Thanks!

Yes! This is hurting my head

Hey!
Totally a workaround, but you can wrap your SVG in an element, and have the animate-spin class on that element.
Then, on the SVG itself, set a scale-x-[-1] class (which will basically flip the icon horizontally.
https://play.tailwindcss.com/AqpZaEVw0E
The reason you need the wrapping element is that if you apply animate spin on the SVG directly (alongside the "flip" class), the transform properties in both those utilities will clash.
Hope it helps!
I'm not saying there will never be an "inverted" refresh icon in Heroicons – but at least you've got a working solution in the meantime 👍
Will move this issue to our GitHub discussions, as some other folks may find this trick useful!


Conditional Logic in the templates

Hello Mathias,
Do you consider adding if/else or any other conditional logic to bake? While using it, that is one of the things I find missing.
Thanks.

M

Hi @musdy
There is some sort of if functionality in it as of right now. Described in the README right here
Can you describe a scenario you'd like to see?
There is no else task so far. one would have to do a if-not
<html>
    <body>
        <!--(bake includes/container.html _if="name")-->
        <!--(bake includes/login-panel.html _if="!name")-->
    </body>
</html>

Hey @MathiasPaumgarten ,
I am using Grunt bake in another project and realized that the thing that's missing is more logic on if:

Custom homepage text

Any way of doing something similar to this without polluting the bake file?

M

Hi @musdy
thanks for the interest in grunt-bake. This is an interesting feature. I don't know how much time I have on my plate, but I try to squeeze it in, if I can find a nice clean way without breaking the API as it is. There might be a nice way in parsing the value within _if="value" where value represents a logic expression.
Let me get backs to you about this. And of course, feel free to take a stab at a PR if you want to.

Hey @musdy, I've added your feature request in version 0.3.16. See this PR: #49

Thanks a lot man. This is great!


Batch v1 vs Batch v2 sending raw audio data

Hi there!
I noticed that in Batch v1 it is possible to send raw audio bytes for transcription, which is preferable for my use case. However, I noticed that in Batch v2, it appears that URI's are used, which based on my understanding, means that sending raw audio bytes is not supported in Batch v2. Is being able to send raw audio bytes via Batch v2 planned in the future? Or is it currently possible and I am missing something? I was hoping just to get some clarification regarding this as I want to use the newest API version, but not if it means a critical use case for me is missing.

Hi! Thanks for asking. You are correct, the v1 API does support raw bytes input, and that feature is missing from v2. The immediate reason is that we wanted the v2.ProcessAudioSource  to be more specific and thus understandable, whereas the v1.ProcessAudio handled all use cases in the same method.
We do have the raw byte stream input support on the backlog for the v2 also, but the release schedule for that is not clear at the moment. We will keep v1 alive until the streaming support exists in v2 also, so you can continue using the v1 raw byte input for now.


systemd service start timeout configuration

Some machine are too busy or slow and Docker service is taking longer than default 90 seconds on CentOS 7 to start. System then just kill the process. Add an optional configuration to set the start time out.
Didn't find equivalent with upstart/init .

Yes a timeout is necessary.
Side track but we'll need a new CI system as well to test these. But not Travis.
Considering doing this out of my pocket.
Thoughts?
-Jason

On Aug 27, 2015, at 5:50 PM, Benoit Carriere<EMAIL_ADDRESS>wrote:
Some machine are too busy or slow and Docker service is taking longer than default 90 seconds on CentOS 7 to start. System then just kill the process. Add an optional configuration to set the start time out.
Didn't find equivalent with upstart/init .
You can view, comment on, or merge this pull request online at:
https://github.com/AnsibleShipyard/ansible-docker/pull/5
Commit Summary
systemd service start timeout configuration.
File Changes
M defaults/main.yml (1)
M templates/docker-service.j2 (3)
Patch Links:
https://github.com/AnsibleShipyard/ansible-docker/pull/5.patch
https://github.com/AnsibleShipyard/ansible-docker/pull/5.diff
—
Reply to this email directly or view it on GitHub.

I'm testing CircleCI right now. Got the whole build running but I got a error: read unix /var/run/docker.sock: connection reset by peer. Are you trying to connect to a TLS-enabled daemon without TLS?
Most likely a permission issue or the machine is using kernel < 3.10. Checking...

Kernel is fine, still don't know why I have the error. But it seems docker already installed on these build machine. Might be the issue...
ubuntu@box322:~$ uname -r
3.13.0-61-generic

ubuntu@box322:~$ cat /etc/*-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=12.04
DISTRIB_CODENAME=precise
DISTRIB_DESCRIPTION="Ubuntu 12.04.5 LTS"
NAME="Ubuntu"
VERSION="12.04.5 LTS, Precise Pangolin"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu precise (12.04.5 LTS)"
VERSION_ID="12.04"

They may not let you access it, like Travis.
-Jason

On Aug 28, 2015, at 10:44 AM, Benoit Carriere<EMAIL_ADDRESS>wrote:
Kernel is fine, still don't know why I have the error. But it seems docker already installed on these build machine. Might be the issue...
ubuntu@box322:~$ uname -r
3.13.0-61-generic
ubuntu@box322:~$ cat /etc/*-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=12.04
DISTRIB_CODENAME=precise
DISTRIB_DESCRIPTION="Ubuntu 12.04.5 LTS"
NAME="Ubuntu"
VERSION="12.04.5 LTS, Precise Pangolin"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu precise (12.04.5 LTS)"
VERSION_ID="12.04"
—
Reply to this email directly or view it on GitHub.

This repo will have to have travis disabled as there is no way any of the current CI systems will let you touch their docker daemons as they too are running jobs in Docker. It would require privileged mode which is an inherit root required security risk.

This is one of those repos where we'll rely on vagrant to run the CI test and tell us if it went well.

Travis is disabled. I'm thinking of setting up a Jenkins box for testing. Though I'm on a prod issue atm with someone's playbook. I think a sufficient "things passed" or "things didnt' passed" with vagrant is enough for me. If the vagrant setup doesn't work let me know and I'll re-prioritize.

I added a Vagrantfile, tested on Ubuntu and CentOS 7 and works fine.


feat(buck2): add support/buck2:sync-cargo-deps target

This change introduces 2 new targets to our system to help with synchronizing Rust crate depdendencies from the root Cargo.toml into Buck2's third-party/rust/Cargo.toml which is used by the Reindeer tool.
Now when adding or altering the root Cargo.toml, you can run:
buck2 run support/buck2:sync-cargo-deps

This will copy over all the [workspace.depdencies] from the Cargo workspace over to the [workspaces] section in
third-party/rust/Cargo.toml as well as the [path.crates-io] sections for temporary forks. Next, it will run reindeer --third-party-dir third-party/rust buckify to update both the third-party/rust/Cargo.lock and third-party/rust/BUCK files. The
program will report if any files have changed and advise the user to commit these changes to a Git commit.
Additionally, there is a check version of this target which will passively check whether or not the sync task should be run. You can invoke this with:
buck2 run support/buck2:check-cargo-deps

Future work may tie the support/buck2:check-cargo-deps target into CI as a check when reviewing pull requests, making it easier to detect if the Cargo and Buck2 worlds are out of sync.
To recap, a reasonable way to approach adding a new Rust into the system if it isn't already present in the root Cargo.toml is to add it (please keep the dependencies in sorted lexical order to help with tooling and diffs) and run buck2 run support/buck2:sync-cargo-deps to create the Buck2 crate dependency targets.

bors try

bors merge

bors merge


Extends filterSelected event to allow multiple frameworks

Extends filterSelected event to allow multiple frameworks
This work was done to meet the acceptance criteria:
It should send the page view to Google Analytics in the following format: /{framework family}/{lot}/filters/{group}/{key}/{value}/vpv
Ticket: https://trello.com/c/g0h1Ibq6/137-how-to-measure-impact-of-adding-filters-to-dos-opportunities-page

@CrystalPea - comments have been addressed


Path template configuration with regular expressions

Hello,
I went through the documentation page on routing and didn't notice any mention for using regular expressions in path template properties.
After haphazardly experimenting with different expressions for UpstreamPathTemplate, I noticed a simple expression like (.+) appeared to be able to match against input, but I couldn't figure out whether the captured group could be used in DownstreamPathTemplate. Or perhaps this was purely coincidental behaviour.
I just wanted to ask whether or not there was support for using regular expressions to capture fragments of a route. If there's any documentation I missed regarding this, please point me in the right direction.
Currently using version 12.0.1.
Thanks in advance.

Would this allow routing restrictions, so that if certain values appear in the upstream route they will go to destination 1, if others appear go to destination 2?
Defining 2 ReRoutes is fine, but I want to have something like:
myurl/customer/(Bob|Andy)/accounts go to destination 1,
myurl/customer/(Other|Any|None)/accounts go to destination 2
Thanks.

Something new?

@philproctor commented on Jan 16, 2019:

This will be resolved as part of my work on https://github.com/ThreeMammals/Ocelot/issues/737

Aha! The #737 issue is closed by me. Reason: No communication.

@miqh commented on Jan 16, 2019:

I went through the documentation page on routing and didn't notice any mention for using regular expressions in path template properties.

Regular expressions in templates are not supported/implemented!

After haphazardly experimenting with different expressions for UpstreamPathTemplate, I noticed a simple expression like (.+) appeared to be able to match against input, but I couldn't figure out whether the captured group could be used in DownstreamPathTemplate. Or perhaps this was purely coincidental behavior.

coincidental behavior

I just wanted to ask whether or not there was support for using regular expressions to capture fragments of a route. If there's any documentation I missed regarding this, please point me in the right direction.

No documentation at all! Not implemented!


Update pytests

This PR adds the suggestions from #2 that were not included when that PR was merged.
To provide clarity on this comment from #2: when diameter is set to None, cellpose runs much quicker than if it is set to a specific value

To provide clarity on this comment from #2: when diameter is set to None, cellpose runs much quicker than if it is set to a specific value

Thanks @kushalbakshi.  It makes sense to set diameter=None.  After further review of the documentation, the diameter is automatically estimated when it is set to None.

Thanks @kushalbakshi. All changes look good. Were you able to successfully run these tests locally?

I have not run the tests locally yet. With the tutorial-db offline, I currently don't have access to an account that has permissions to create schemas.

Thanks @kushalbakshi. All changes look good. Were you able to successfully run these tests locally?

I have not run the tests locally yet. With the tutorial-db offline, I currently don't have access to an account that has permissions to create schemas.

Let's hold off on merging until these tests have been run locally.  Where you able to get permissions to create schemas on db.datajoint.com?

Thanks @kushalbakshi. All changes look good. Were you able to successfully run these tests locally?

I have not run the tests locally yet. With the tutorial-db offline, I currently don't have access to an account that has permissions to create schemas.

Let's hold off on merging until these tests have been run locally. Were you able to get permission to create schemas on db.datajoint.com?

I'm in the process of setting one up. Instead of <username_> as the database prefix, in order to maintain consistency with the internal _ conventions, we're setting up a test workflow for the sciops team that will retain this structure and have permissions to create schemas

Here is the output of the pytest after the most recent changes: