Query returning {data: undefined}

I'm using apollo-angular for a fairly large project right now and it's been working beautifully. However today I built a new query, tested it in graphiql, everything was fine, however when I then ran it in my app I get {data: undefined, loading: false, networkStatus: 7, stale: true} thrown back at me.
The extra confusing part here is: Chrome network tab and Apollo tab both show that the query was successful. My Apollo Cache has the expected objects in, and the network tab shows the full response as expected (with 0 errors)
Any insight or help here would be much appreciated.
Thanks
EDIT:
I thought it may be beneficial to show how I'm querying this - just in case it's something I'm doing wrong perhaps.
this.apollo.watchQuery<FetchNotificationsQuery>({
  query: graphQLDocuments['fetch-notifications.graphql'],
  variables: {
    types: Object.keys(this.model.settings.integrations).filter(key =>this.model.settings.integrations[key]),
    limit: 20
  } as FetchNotificationsQueryVariables,
})
.subscribe((data: { data: FetchNotificationsQuery }) => {
    console.log(data);
});

Found this issue https://github.com/apollographql/apollo-client/issues/1363 so seems that I need to update apollo and look through the fix involved there.


Build a Docker image on master pushes and push it to GCR

The old repository used to build a Docker image and push it to GCR via TeamCity. Replicate this for GitHub Actions.
Merging is blocked until we got a key file from SRE.

GCR_JSON_KEY added,

GCR_JSON_KEY added,


10 final Rooosss

Link to deployed project - https://rooosss.github.io/fe-course-basic-2023/

I don't see any responsive layout at all

Thanks! Really fine project!

need to resolve conflict and I can merge it!


Undefined method confirm_billing adding an addon

heroku addons:create autobus:trip

Error:       undefined method `confirm_billing' for Heroku::Helpers:Module (NoMethodError)
Command:     heroku addons:create autobus:trip
Version:     heroku-toolbelt/3.39.5 (x86_64-darwin10.8.0) ruby/1.9.3
Error ID:    88ef2465ffbd43d5be7971b8b3e299e0

Heroku client log:
Heroku client internal error.
undefined method `confirm_billing' for Heroku::Helpers:Module
/usr/local/heroku/lib/heroku/command.rb:217:in `rescue in run'
/usr/local/heroku/lib/heroku/command.rb:268:in `run'
/usr/local/heroku/lib/heroku/cli.rb:27:in `start'
/usr/bin/heroku:24:in `<main>'

@pedro do you know what's going on here? looks like some code was removed but is still referenced: https://github.com/heroku/heroku/commit/ccb1b
https://github.com/heroku/heroku/blob/master/lib/heroku/command.rb#L216-L217

@dickeyxxx to give you some context, it turns out that the owner of the app I was developing and trying to add the addon to has not paid their bill.

that makes sense, this is just presenting the error message badly. I've added https://github.com/heroku/heroku/pull/1673 to address that


Error in Binding statement not caught.

In WPF
Background="{TemplateBinding Background"
Does not compiles and gives the error Expected '}'
OmniXAML does not see to catch this and evaluate this statement as a normal string.

Fixed!


How to send JSON object to web service?

A JAVA web service I am calling requires that I send it a JSON object. The service requires it to be a POST and it accepts "application/json".
The object looks like this:
{
    id: null,
    ticketID: 12345,
    filename: 'filename from selected file',
    desc: 'file name without extension',
    attachment: byte array of file content
}

I am trying to use the upload.http() method i see that the  the attachment is empty in the request parameter view of the browser's Network tab.
{"id":111619,"ticketID":"NewCorpActions.xlsx","desc":"NewCorpAction","attachment":{}}

    this.attachFileToTicket = function(ticket, files) {
        // Really only 1 file
        for (var i = 0; i < files.length; i++) {
            var file = files[i];

            return uploadSvc.http({
                url: '/ticket/attachFile',
                headers : {
                    'Content-Type': undefined
                },
                data: { 'id': ticket.id, att_filename: file.name, att_desc: file.name.substr(0, file.name.lastIndexOf('.') - 1), attachment: file }
            });
        }
    }

Any help would be greatly appreciated. Thanks.

You cannot have additional data with Upload.http. It just sends the file binary content in the body of the request so the only way is to specify the params in the url.

A few years late, but ASP.NET MVC can map data just fine if you submit as a form. Upload.upload submits as a form and takes extra parameters.
I have a working example that looks about like
Upload.upload({
  url: '/something',
  data: { ticketId: 5, files: files}
});

[HttpPost]
public JsonResult Something(int ticketId, IEnumerable<HttpPostedFileBase> files){
...
}


Increase privacy of vimeo player embeds

This adds the dnt parameter to Vimeo embed urls and stops Vimeo from setting third-party cookies on our site. This arose while looking at #7581.
"Setting this parameter to "true" will block the player from tracking any session data, including all cookies and analytics."
Since this will affect analytics, it would good to have @taceybadgerbrook approve this before it is merged. Do we use Vimeo analytics?

I've never used Vimeo Analytics and wasn't aware that any of our material was on Vimeo. Have you actually identified any?

Have you actually identified any?

Yes, I'm not sure how extensively they are used without a bit more digging, but there is one on the bottom of this page for example.
Do you know if anyone else is likely to be looking at the analytics Vimeo provides?

@gestchild wow that link is from an article from 2017. I don't know of anyone using Vimeo analytics or even using Vimeo. If this link is the only one you've found, I think chopping the cookies is not going to cause any problems.

I've just checked and this is the only instance of a vimeo embed, so will merge this


Share task text with carrot template engine. #372

This pull request replaces the 'manual' assembly of the shared task text with using the carrot template engine.

@dmfs I am submitting this now but there are TODOs left listed in ShareTaskText, mostly related to androidcarrot about the issues opened there. Shall I create pull requests for those?

let's wait for the next carrot release. All our changes have been merged so we don't need to use our own fork.

We don't use a fork any more here, carrot is imported directly with commit hash with jitpack.
But we use AndroidBindings from your androidcarrot extension library. Shall we copy the necessary files from that to OpenTasks?
The related pending TODOs are listed here:
https://github.com/dmfs/opentasks/blob/174c1871651f6578590cf4f0b3f5e8590cd2ee8e/opentasks/src/main/java/org/dmfs/tasks/share/ShareTaskText.java#L67-L67

Ready for review now.
There are those Lazy, Factory types added here which may be replaced with the final ones I assume coming at some point in jems.


Styled Components doesn't work after "npm run build" in a local build folder.

Hey!
I'm using the create-react-app with styled-components. I'm running index.html from the build folder(using "homepage": "." in the package.json) and it's running with styled-components but without any styles for sc classes. I've found solutions with react-app-rewired, styled-components-babel-macros and it's not help me to solve it. I don't want to use the eject for my CRA, so help me please with the solution to fix it.

https://styled-components.com/docs/tooling#babel-macro
On Tue, Feb 11, 2020 at 2:01 AM EntaltsevSN<EMAIL_ADDRESS>wrote:

Hey!
I'm using the create-react-app with styled-components. I'm running
index.html from the build folder(using "homepage": "." in the package.json)
and it's running with styled-components but without any styles for sc
classes. I've found solutions with react-app-rewired,
styled-components-babel-macros and it's not help me to solve it. I don't
want to use the eject for my CRA, so help me please with the solution to
fix it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/styled-components/styled-components/issues/3014?email_source=notifications&email_token=AAELFVTC6PIJX52WMG4RATTRCJEMBA5CNFSM4KS3KW2KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMO4IZA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAELFVWNBDYP5RRYBJHGRLTRCJEMBANCNFSM4KS3KW2A
.

@probablyup I've already test it and it's not working for me. Can you please clarify or provide me with the example of code which works after building?

@probablyup Also, I tried to deploy it to the Netlify and still has same issue.

Sorry but first and foremost please post usage questions on StackOverflow or our Spectrum as listed in our issue template.
Also as you’ve found: Babel Macros work in CRA, so you can use our Babel Macro.
Now I do want to help and would love to resolve this quickly for you, but I have nothing to go on 😔 So you’ve said it isn’t working for you, what do you mean by that? You said you think you need to use our Babel plugin I assume? But you haven’t said why you think so.

I’m running Index.html from the build folder

So you’ve built your CRA app with the build script? Does it work in development when you just start the app though or what happens instead?
The issue template still gives you a good guide on how to write an issue that helps me help you 🙌 maybe check whether you can describe what you’re seeing, what you’ve done and tried, and what you expect?

The issue is fixed. It was happened because the createGlobalStyle component was added before the app body. I've placed it in the end of the app component and it works now.


BlazorHybrid demo does not work

Copying the full repo and trying to build the BlazorHybrid demo fails with 219 errors. Well happens.
Isolating only the BlazorHybrid demo, removing reference to the project, and adding Nuget WinFormsComInterop (0.4.3) it is finally possible to build. A little bit broken CSS but not a big deal. Commented out /fetchdata is on purpose, I guess, since uncommenting that will crash with the following error:

System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
InvalidOperationException: An invalid request URI was provided. Either the request URI must be an absolute URI or BaseAddress must be set.

But the biggest issue is that it is not possible to publish as a native AOT.
When trying to do either in Visual Studio or with the dotnet command, both fails with an error:
Failed to load type 'BlazorHybrid.Counter' from assembly 'BlazorHybrid, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
2> Internal.TypeSystem.TypeSystemException+TypeLoadException: Failed to load type 'BlazorHybrid.Counter' from assembly 'BlazorHybrid, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
2>   at Internal.TypeSystem.ThrowHelper.ThrowTypeLoadException(ExceptionStringID, String, String) in /_/src/coreclr/tools/Common/TypeSystem/Common/ThrowHelper.cs:line 17
2>   at Internal.TypeSystem.ResolutionFailure.Throw() in /_/src/coreclr/tools/Common/TypeSystem/Common/ResolutionFailure.cs:line 93
2>   at Internal.TypeSystem.Ecma.EcmaModule.GetType(String, String, NotFoundBehavior) in /_/src/coreclr/tools/Common/TypeSystem/Ecma/EcmaModule.cs:line 344
2>   at Internal.TypeSystem.ModuleDesc.GetType(String, String, Boolean ) in /_/src/coreclr/tools/Common/TypeSystem/Common/ModuleDesc.cs:line 35
2>   at Internal.TypeSystem.CustomAttributeTypeNameParser.ResolveCustomAttributeTypeDefinitionName(String, ModuleDesc, Boolean) in /_/src/coreclr/tools/Common/TypeSystem/Common/Utilities/CustomAttributeTypeNameParser.cs:line 261
2>   at Internal.TypeSystem.CustomAttributeTypeNameParser.GetTypeByCustomAttributeTypeName(ModuleDesc, String, Boolean , Func`4 ) in /_/src/coreclr/tools/Common/TypeSystem/Common/Utilities/CustomAttributeTypeNameParser.cs:line 76
2>   at ILCompiler.RdXmlRootProvider.ProcessTypeDirective(IRootingServiceProvider, ModuleDesc, XElement) in /_/src/coreclr/tools/aot/ILCompiler/RdXmlRootProvider.cs:line 98
2>   at ILCompiler.RdXmlRootProvider.ProcessAssemblyDirective(IRootingServiceProvider, XElement) in /_/src/coreclr/tools/aot/ILCompiler/RdXmlRootProvider.cs:line 83
2>   at ILCompiler.RdXmlRootProvider.AddCompilationRoots(IRootingServiceProvider) in /_/src/coreclr/tools/aot/ILCompiler/RdXmlRootProvider.cs:line 47
2>   at ILCompiler.Compilation..ctor(DependencyAnalyzerBase`1, NodeFactory, IEnumerable`1, ILProvider, DebugInformationProvider, DevirtualizationManager, IInliningPolicy, Logger) in /_/src/coreclr/tools/aot/ILCompiler.Compiler/Compiler/Compilation.cs:line 64
2>   at ILCompiler.ILScannerBuilder.ToILScanner() in /_/src/coreclr/tools/aot/ILCompiler.Compiler/Compiler/ILScannerBuilder.cs:line 83
2>   at ILCompiler.Program.<Run>g__RunScanner|73_4(<>c__DisplayClass73_0&) in /_/src/coreclr/tools/aot/ILCompiler/Program.cs:line 884
2>   at ILCompiler.Program.Run(String[]) in /_/src/coreclr/tools/aot/ILCompiler/Program.cs:line 866
2>   at ILCompiler.Program.Main(String[]) in /_/src/coreclr/tools/aot/ILCompiler/Program.cs:line 1150

Am I missing something? Is there some specific order of operations that has to be followed? Any help is appreciated.

Sorry, it was my mistake. See https://github.com/kant2002/WinFormsComInterop/commit/797396d80e044095ee45a3c932eae56b697e9274
If you want fetch, I assume you need to add new type here
https://github.com/kant2002/WinFormsComInterop/blob/main/samples/BlazorHybrid/rd.xml
And maybe some additional things for HttpClient, you have to test. Unfortunately currently Blazor Hybrid require such laborious work.

Oh my... I was banging my head on the keyboard, I tried to create a new project from scratch, changing properties, versions, etc.
And it was yelling the whole time "failed to load assembly", just because it was missing one word.
Thank you for the fix :)

Don't worry. That's normal situation, this thing a bit complicated to learn from first stab.

I really like the idea of native AOT and their advantages. The only problem now is that a lot of things are not supported yet or need a lot of tweaking. Which leads me to another question - not sure if I should write here, spin another thread, or maybe discuss on another platform (Discord maybe?). Anyway, I will try.
Is it possible to deploy it as a single file? PublishSingleFile and NativeAOT properties cannot be used together.
Publishing now results in one .exe, few DLLs (found out that only webview2loader.dll is needed) and wwwroot folder.
I was able to embed the wwwroot inside the app, however this bug appeared #1333 issue, I peeked into .exe copied the manifest, fixed the error and manually forced it back in the app.
It obviously works, but it is tedious, and I cannot imagine doing all the time over when adding some new files.
Maybe I want a lot of things ;)

Found out solution for the #1333 issue.
For those who would like to embed wwwroot into the application, it is necessary to add those lines in the .csproj
<ItemGroup>
  <EmbeddedResource Update="MainForm.resx" ExcludeFromManifest="true" />
</ItemGroup>

Big help from this thread #31107
The final step is to figure out if it is possible to embed all those dlls :)

So you embed whole wwwroot, that's really nice. Regarding single file and NativeAOT. As you probably figure out that's mutually exclusive things. And what even Single file means for NativeAOT, some "magic" packer? it would not work. NativeAOT in a sense alreade single file.

I fell into the rabbit hole. Even the name PublishSingleFile is not truly a single file, and more or less dlls are there (depending on the project size, obviously). But with combination IncludeNativeLibrariesForSelfExtract publishing will result in really only one single file. And I would like to achieve the same or similar with AOT. To have only one single file. But so far, no luck.

You have to package that in single file yourself. This scenario does not supported.

Yep, I thought that. But do you have some guide or something to steer me the right way?
Should I embed it the same way as the folder? Or used something like ILmerge? Would Costura.Fody work?
What is the way to go?

I would try to look at https://github.com/MichalStrehovsky/PublishAotCompressed . I think your solution lays in unmanaged land of different kind of packers like UPX or something like that.

Will definitely try and check. Thanks a lot for your help :)


Feedback on Raycast's user experience and suggestion for direct result display

Description
Hello, I am a user of Raycast and would like to give feedback on an issue with the user experience.
As a tool that values simplicity, I really enjoy Raycast's design. However, when using plugins, I found one aspect that is not as straightforward: after entering a keyword, I need to perform a jump operation to display the results, which I find a bit cumbersome.
Raycast VS Alfred:

Therefore, I would like to ask if Raycast supports directly displaying results upon invocation? If not, are there any plans to add this feature in future updates?
Thank you for your time and patience in reading this, and I look forward to your reply.

You can add an alias to the translate command to achieve the same result


Correct missing street numbers for us/mi/ingham

@MiniCodeMonkey See OA slack - This error isn't due to your change


Survey Profile - Data Model Review

Original Profile

Link to Powerpoint for review: Survey Profile
Please use Review Tab in the powerpoint file to add NEW issues as comments (not additional boxes). Check existing comments boxes to add to the discussion on existing issues before creating new issues.Please reply to this comment with subjects of issues raised on powerpoint to track review progress.
Reviewers: @KellyVance @hauckl @Greenwoodmatthew

Comments added to ppt.   Basically currently this profile is for a seismic survey and will ned some edits to make it a geoscience survey operation.  I.e. if type == seismic then level 2 metadata is about seismic methodology/dimensionality

Comments:
Seismic Survey Lines
Replies:
BFO Dimensionality, Permit.

Seismic Sampling Method needs [1..*] cardinality to allow for multisource surveys i.e. dynamite and vibroseis
@DavidCrosswellGSQ


Run SimulatorSimulator

When REST API is done

Should run after #44

Did this right now. It works.


Add file with FaML coupling

We are very tight on space, so am I not sure if we will be able to fit all the submissions.  I will do my best.


bug: enable() immediately shows privacy layer, app unusable

Plugin version:
^4.2.0
Platform(s):
iOS/Android
Current behavior:
Upon calling PrivacyScreen.enable() the privacy layer is immediately displayed making the app unusable.
Expected behavior:
The privacy layer should only be shown when the user is trying to take a screenshot, screen recording, or viewing the task switcher.
Steps to reproduce:
Sample Repo

Clone and follow instructions to install and run for either iOS/Android.
Run on a simulator/real device.
Click the "Enable" button.
See that the privacy screen is shown and cannot be dismissed.

Related code:
See this file in the repo on line 20 where PrivacyScreen.enable() is called.
Capacitor doctor:
💊   Capacitor Doctor  💊

Latest Dependencies:

  @capacitor/cli: 5.4.1
  @capacitor/core: 5.4.1
  @capacitor/android: 5.4.1
  @capacitor/ios: 5.4.1

Installed Dependencies:

  @capacitor/cli: 5.3.0
  @capacitor/core: 5.3.0
  @capacitor/android: 5.3.0
  @capacitor/ios: 5.3.0

[success] iOS looking great! 👌

Thank you for your request.
Please share a screen recording.

Hello,
This only occurs on iOS17 with custom image added. If you remove splash than its working

I did this without splash and had the same results that the screen is blanked out.
Help me understand. When I call enable() is it supposed to show the privacy screen immediately? The documentation suggests that you use enable to protect against screenshots and view in the switcher if the user tries to take a screenshot/use the switcher. So, enable() = "wait until a screenshot is taken or the switcher is shown and then show privacy screen". Right? Because when I call enable() it IMMEDIATELY blanks my screen, which seems incorrect to me.
But, maybe I misunderstand. Do I need to add listeners and then only call enable() when the listeners are triggered?  "enable" to me != "show". It "enables" protection.
Here is a screen recording where I show it disabled first, then enabled second, all without imageName specified in the config (no splash)
https://github.com/capacitor-community/privacy-screen/assets/3846466/b882b202-72a2-4ddd-a672-721085c0c4ca

On this video you posted it is working as it should. They grey screen you get it is because you didnt add splashscreen. The privacy screen is visible only when app isnt active (in recent).

Yes, it does seem to be working for "recents"/"switcher". But it's NOT preventing screenshots. I couldn't care less about the switcher. I NEED to prevent screenshots. How do you prevent a screenshot with this plugin?

I will take a look at it in the next few days.

I close this issue as duplicate of #74.


Reduce threads to 2 to fit modern desktops PCs

Fixes #68

Coverage remained the same at 75.425% when pulling 5c5b5e4ddfd80bf11b61616c25178f6b51b3da18 on feature/reduced-threads into b484baf038f5cb51991887fd8a41658c44976dda on develop.


Incorrect template created when target close to image border.

When the target is close to the image border the template created for the correlation filter and the histogram model is off, causing the tracker to drift after a few frames. This is due to the lefttopLimit and rightbottomLimit of STAPLE_TRACKER::getSubwindow, which clip subWindow and causing the aforementioned issue.
A quick solution is to pad subWindow accordingly by replicating the border pixels.
I managed to solve the problem like this, however I am not allowed to submit a PR, could I be given permission in order to share my solution?
Thanks in advance

Welcome to create PR for this project. It's first time for me :) Seems I already allow PR in the setting page. If you know how to make PR allow, please tell me.

In my experience, I always fork project first before create a PR. But I didn't see you do the fork, I think you just clone the project. Can you try fork first?

You are absolutely right, I should have forked it instead of cloning. Submitted the PR, let me know if it makes sense. Thank you!

Done!


problem when trying to compile the obfuscated script using pyinstaller

SyntaxError: invalid non-printable character U+0017
both pyinstaller and python latest

@yo-22 use nuitka instead of pyinstaller


Add census data

Two things.

primary purpose - add yml templates and python scripts for dcp-provided census data for population and housing. There are two source files - one by blocks/borough/city, and one by ntas/community districts/council districts/borough/city.
some maintenance - a while back I edited a bunch of the scripts and made it so that path was specified in yml, not just hard-coded in a script (in the case of script-sourced dataset that are still pulling from just one endpoint). I missed bpl_libraries back then so added it now (and in the one test yml file that uses it)

Open to other naming possibilities for dataset.
This is used by EDDE and DevDB for housing/pop census data

Oh and also just added some paths to ymls following up on some stuff I had done a while back

@alexrichey this is pretty limited in scope but is an example of adding new data sources via data library - two new yaml files that specify that the dataset "source" is from the added scripts

I think I get the gist of the changes. Looks good to me, though maybe we want consider a linter/formatter to flag unused imports and enforce style?

We use black often for linting/formatting but could definitely stand to incorporate it with a tad more rigor/norms (commit hooks or something along those lines).
I don't think it complains about unused imports (or removes them)

(started typing this before I saw Finn's reply lol)
@alexrichey agree on the value of linters/formatters. we haven't taken a shot at implementing them as a CI check on all mono repo PRs, but would love to here sometime soon
black has been our go to for python formatting (it may flag unused imports though) and here's where it was being used by some datasets that are now in the mono repo
I had some fun exploring the use of trunk to lint many languages at once, but the overhead of the VS Code extension has me turning away from it


Enhance the Walking Controller Logger

Currently, walking logger logs mainly the locomotion data, while to debug the problems,  we may need other information of the controller such as:

all joint angles & velocities
end effector pose & velocities

cc @fjandrad @GiulioRomualdi  @DanielePucci

I am getting convinced that having a powerful logger is very important if we want to increase the overall performances of the algorithms we are running on the robot. Maybe we can take this a step further and start developing a standalone application able to log all the relevant data, like internal states of the controllers, commands sent and feedbacks.

I completely agree with that.

To add what you said, it can be useful also adding a Matlab script, with options to enable, for visualizing the data. Because, in this case it is not recommended logging all the data in one single file, since it can become easily large size.

I don't think the file size is an issue. Right now hard disks are pretty cheap, and we could use a dedicated machine for logging. For the data visualization, a link that appeared a couple of times in some previous discussions is https://github.com/facontidavide/PlotJuggler (I never tried it). I think the possibility to drag and drop the data is pretty useful. For logging data, we could exploit https://github.com/ADVRHumanoids/MatLogger2

We should develop something like coredumpctl

What do you mean? :thinking:

Suppose that you are running something in the terminal. At the certain point a segmentation fault occurs. Using coredumpctl gdb you can understand the reason of the problem without running again the application.

Interesting, but how do you think to use it in practice to log data?

Ehm  I don't know

Suppose that you are running something in the terminal. At the certain point a segmentation fault occurs. Using coredumpctl gdb you can understand the reason of the problem without running again the application.

With @prashanthr05 we are working to get an offline environment in which you can replay data logged with data dumper and run your application on that data as if the robot was working. Not sure if it makes sense, but we could start from something like and build up to something more powerful for logging/debugging and testing.

This morning I started to work on a new version of the Logger. The logger is based on MatLogger2.
You can find the code here: https://github.com/GiulioRomualdi/YarpMatLogger

In the last days, I worked on the logger:  https://github.com/GiulioRomualdi/YarpMatLogger.
Since the logger does not depend upon the walking and it can be used also by different modules, probably we should finf a proper repo where store the code.

I would consider the logger enhanced. Also the title is a bit generic suited for an epic xD. We might consider closing this issue and then open specific issues for where to put the logger and individual features we want to add to it.

@GiulioRomualdi please decide if you want to substitute the current logger but let's not put too much effort on this. Then close this issue.

I will move the new logger in the new walking-repo https://github.com/dic-iit/bipedal-locomotion-controllers

Closing after https://github.com/robotology/walking-controllers/pull/97


Use a translated bitmap to enable more efficient marking

Since the mark-and-compact GC is currently less performant than the copying one, it makes sense to tweak its performance. Here I take the idea from @ulan and run with it. Basically, we now pass the object's absolute word number to g/set_bit and thus avoid some arithmetic and passing the heap_base along.
The marking was sped up by 4 instructions. This is compounded by the fact that now we don't pass the heap base to the marking routine mark_object any more:

mark_object: 120 now vs. old 123 instructions
every callsite to mark_object 1 instruction less.

Also speeds up the iteration over the bitmap after marking (BitmapIter::next has now 92 instructions compared to formerly 110). In particular, the inner loop is now more compact.
Fixes #2892.
Benchmarks will be included in #2927 when available.
Further optimisation opportunities

[ ] show that due to 2-word object allocation the bitmap can be halved (not clear how)
[ ] pass tag (and first word?) to object allocator
[ ] when marking, use the skewed pointer and use pointer shifting, buffer padding tricks
[ ] inner loop in BitmapIter::next can be eliminated.

As far as I understand, the idea here is to move bitmap address back so that when we compute the bit byte index of an object with object_addr / WORD_SIZE and add that to bitmap address, we will have a byte in the bitmap.
This way we don't want to allocate bits for the static heap, as suggested in #2892. It's faster, but doesn't come with space overhead. I like it.

Currently I am fighting with the heap's alignment when doing the tests. Here is the principal problem:
When heap_base is not divisible by 32 then adding heap_base % WORD_SIZE (effectively) to the mark location (set_mark's argument) can cause two effects

marking can happen beyond the allocated mark blob
mark bits appear in shifted positions when iterated over in 64-bit chunks.
The latter will cause the sweep phase to reclaim the wrong objects (or reach into the middle of them, looking for heap tags). (The former could be compensated easily by bumping the mark blob's size by one.)

Aligning the heap_base on the IC is pretty much trivial, but in the test environment it is trickier, since a Vec[u8]'s payload can end up being allocated at any address that is divisible by 8.

Looking good! Thanks for grasping the nettle!

The PR looks good to me, but I think I'll need to defer to someone who actually speaks rust.

The PR looks good to me, but I think I'll need to defer to someone who actually speaks rust.

@ulan can you have a look?

LGTM, but would be good to see benchmarks. Marking should be more efficient now, but I can't tell if the new bitmap iterator will be faster or slower, and what will be the effect of it to the overall GC.

Yes, there can be surprises, although unlikely. Benchmarks will be added to the MR.

It would be helpful to see the changes in the PR description. For example, number of instructions in marking or one iteration of bitmap iterator, or even better, actual benchmark results.

Added instruction balances.

What happens to TODOs in the PR description? If they're for another PR should we remove them? They will be in the commit message when this is merged.

I nuked the TODOs as these were meant to track my progress, and are now irrelevant.

As promised, here come the performance counters for the change

baseline is release 0.6.16
compared to with a7868cd1f1378f0b2c7f30c2626b46ac765e9651 reversed

[nix-shell:~/motoko]$ tail compacting-baseline_perf.csv compacting-a7868cd1f13_perf.csv
==> compacting_perf.csv <==
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29548225,222,54,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29572634,222,54,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29318871,224,37,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29305196,223,36,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29326984,223,36,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29186706,223,26,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29178490,225,23,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29159337,224,22,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,29173610,224,22,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,32425028,224,200,16

==> compacting-a7868cd1f13_perf.csv <==
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30624396,222,54,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30649781,222,54,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30397508,224,37,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30384817,223,36,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30407620,223,36,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30268366,223,26,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30261620,225,23,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30243489,224,22,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,30258762,224,22,16
rwlgt-iiaaa-aaaaa-aaaaa-cai,canister_update createProfile,33511297,224,200,16

A few quick percentages:
[nix-shell:~/motoko]$ ghc -e '33511297/32425028*100'
103.35009425435192

[nix-shell:~/motoko]$ ghc -e '30258762/29173610*100'
103.71963565702016

[nix-shell:~/motoko]$ ghc -e '30624396/29548225*100'
103.6420834077174

We would regress more that 3% (on performing pure GC) with this PR reverted!


Wrong output by :file after moving current file using :!mv

Steps to reproduce

:!mv /path/to/current-file-name  /path/to/newfilename
:file  or :echo @%

OR

:!mv current-filename new-filename *
:file or :echo @%

Expected behaviour
After renaming current file using :!mv or :!rename commands, I expect that :file to show new name of the file. But :file still shows the previous name of the file.
Version of Vim
9.0
Environment
Operating system: archlinux-2021.09.01
Terminal: xfce4-terminal 1.0.4
Value of $TERM: xterm-256color
Shell: bash 5.1.16
Logs and stack traces
No response

When I execute :!mv ... like this:
$ mkdir -p /tmp/path/to
$ echo 'some text' >/tmp/path/to/current-file-name
$ vim -Nu NONE /tmp/path/to/current-file-name
:!mv /tmp/path/to/current-file-name /tmp/path/to/newfilename

I get this error:
E211: File "/tmp/path/to/current-file-name" no longer available

Vim complains that the file from which the current buffer was read no longer exists.  Which is why reloading the buffer with :edit makes it empty; because from Vim's point of view, there is no file to read anymore.
:! is a Vim command which can be used to execute an arbitrary shell command, which can move the file wherever it wants.  I don't think Vim can know where it was moved.

You might be interested in the :Rename command provided by the vim-eunuch plugin.

Thank You @lacygoill


[sql] Can't save binary data with execute (SQLite)

First of all, thanks for your work on Tauri and essential plugins like this one!
I'm having trouble inserting Uint8Array data into BLOB columns in SQLite. Instead of being inserted as binary data, it's inserted as a stringified JSON object.
My main use case is a uuid column storing the 128-bit binary representation of a UUID.
CREATE TABLE node (
    uuid BLOB PRIMARY KEY
    -- ...
)

const uuid = new Uint8Array(16)  // this would be a generated 128-bit UUID
const result = await db.execute('INSERT INTO node (uuid) VALUES ($1)', [uuid])

What's inserted into node.uuid:
'{"0":0,"1":0,"10":0,"11":0,"12":0,"13":0,"14":0,"15":0,"2":0,"3":0,"4":0,"5":0,"6":0,"7":0,"8":0,"9":0}'

If I instead save the ArrayBuffer itself, I get an empty object:
const uuid = new Uint8Array(16).buffer  // its underlying ArrayBuffer
const result = await db.execute('INSERT INTO node (uuid) VALUES ($1)', [uuid])

'{}'

Am I doing something wrong, or does the binary data simply not survive serialization when sent from JS to Rust?
If it's the latter, maybe serde_binary can be of help?

uint8array is not an actual array but just a plain JS object which the ipc converts to a json object. what's been necessary so far (and used for the core writeBinaryFile api for examples, is to convert it to an actual array before sending it back -> Array.from(uuid)
so i guess we should at that to the plugin too

uint8array is not an actual array but just a plain JS object

Yes, but I think the API should support TypedArrays too for representing binary data.
As long it's saved as binary data and returned as binary data by sqlx, Array.from sounds good :)

I have this working using something like:
      Object.values(JSON.parse(snapshotTextFromDb))

but I wonder if we should not use JSON to send data to Rust, if I understood correct the new IPC system in Tauri 2 supports sending binary data, should we switch to that?
My issue at the moment is that I'm sending a ~200kb binary document and it's quite slow (and it's also hanging the main thread)


Allow for input:scroll_factor configuration

Describe your PR, what does it fix/add?
This PR will allow for a scroll_factor configuration within an input block.  The purpose is to control the scroll factor of external mice.
Closes #2574.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
This is my first PR to this repo and I am not a C++ expert.  But, it's a small one, and it works locally.
Is it ready for merging, or does it need work?
I tested it locally and it's ready to merge.

@vaxerski here ya go! https://github.com/hyprwm/hyprland-wiki/pull/506

hehe style failed

@vaxerski done


Add new feature EU Committee of Regions Members and alternates

What is EU Committee of Regions Members ?
The European Committee of the Regions (CoR) has 329 members representing local and regional authorities from all 27 EU Member States.
Members must be democratically elected and/or hold a political mandate in their home country. Each national government proposes its regional and local representatives (members and alternates) – national delegations – who must be approved by the Council of the EU.
CoR members are grouped according to their political affiliation – EPP, PES, Renew Europe, ECR, EA, The Greens – and areas of interest – commissions.
Why it's important to fetch this list ?

It consolidates every members and alternates in the European committee of regions.

Changes :

Add eu_cor_m_a.py to crawlers/
Add eu_cor_m_a.yml to config/
Update docs/_data/source.yml

See also : Members' portal, European Committee of the Regions

Waaaa that seems like a nasty web site to crawl :)

Waaaa that seems like a nasty web site to crawl :)

Waaaa that seems like a nasty web site to crawl :)

Yes, that wasn't easy to figure out how to catch every members and alternates. 😅

Waaaa that seems like a nasty web site to crawl :)

Yes, that wasn't easy to figure out how to catch every members and alternates. 😅


Mica support on Win32 platform

Adds support for a window to utilize darkmode and Windows 11-style taskbars by defining SA_Win11 before inclusion of Slimapp. Requires that the app links to dwmapi.lib

Note that the changes require Windows 11 Insider Preview 22523 or higher to work for Mica. Before 22523, the only easy way to set Mica was through an undocumented attribute.
Not entirely sure if this is in scope, but was fun to add and figured I might as well PR it in case. Cheers!

Hey, thanks for the contribution :)
Generally, I would want to avoid any additional specific dependencies, or operating system version requirements, as the whole point of the 'Slim' in the name there is to minimize dependencies and setup effort.
Though, as long as it's all technically 'optional' and doesn't impose any additional requirement (code would still compile in much older version of Window), then I don't mind accepting it.

The code should still compile in older windows versions just fine. Anything added in my PR is wrapped in an #ifdef SA_Win11 block, so without that definition it imposes no extra requirements.


sorting index based on integer value

 var db = new Dexie('name'); 	db.version(4) 	.stores({       sentence: "&sentenceId, [sentence__authorId+sentenceChapter+sentenceNo], sentenceContent, headingContent, sentenceStarts, sentenceEnds"       )};
and I need to get the result sorted by sentenceNo
`db.sentence.where('[sentence_authorId+sentenceChapter+sentenceNo]')
.between([articleId, chapter_selected.toString(), -Infinity], [articleId, chapter_selected.toString(), '\uffff'])
.each (function (sentence) {
//............
});`

but the result I get from the above query is like sorting an integer column in string format
eg: 1,11,12,13,14...
how to sort sentenceNo in integer
eg : 1,2,3,4,...

From http://stackoverflow.com/questions/40880150/indexeddb-sorting-index-based-on-integer-value/40885499#40885499:
Are you sure that sentanceNo are not stored as strings instead of numbers? IndexedDB enumerate your index in respect to its types.
If you're certain that typeof sentanceNo === 'number', the only thing I could think of would be if you're using a Safari < 10 with IndexedDBShim - theoretically, the shim could have this kind of bug, but that is just a theory.

sorry the problem isn't dexie it's caused by json (used it for bulk add) request, and the json generated from a mysql database result and mysql gives all result in string that caused the error.


Rename lessons to en in order to clarify lesson structure/location

The first time I cloned and jumped in after looking over some issues and wanting to poke around the project, I actually couldn't find the English lessons until I realized it was lessons and not the 2-character code for english (en) like the other languages.
This is not only slightly confusing but treats English as if it is a higher priority over other languages. I thought it made sense to just simply treat all languages equally and consider them equal first class citizens.
Note
There may be jekyll configuration changes needed to handle the name change. I poked around but didn't see anything obvious after grepping/checking general config. Let me know if there would be more needed changes.

Thank you @ybur-yug, I concur this is a good idea.  I'll give this a whirl locally to check the configuration.

Thanks 👯 👯 👯

We could probably move all the translated lessons into lessons along with the new en language folder. What do you think about this?

I like that @nscyclone, keeps the root folder clean.
One issue I mentioned to @ybur-yug is that this will generate new URLs and the previously crawled ones will now 404, so we need to handle that.

@doomspork @nscyclone I'm for doing all of them at once into a lessons dir. Gets it done and there is very little chance we'll have to deal with the change again.

Let's do that than @ybur-yug

@doomspork do you have a timeline on when you'd like to do this? Happy to help out with the grunt work involved for crawlers etc

@ybur-yug no timeline, just need to make sure this PR covers our bases which it does not at the moment.  Can't merge and auto-deploy without a solution in place to address the massive number of 404 and broken links.
Feel free to investigate and try some solutions out.

I've been swamped lately with work so I haven't had a chance to further investigate how to handle all these redirects, @ybur-yug have you had looked into this any further?

@doomspork I've been busy as all hell lately too. I briefly looked into it but haven't gotten anywhere noteworthy.

I'm probably going to have a hard time contributing much of anything of significance for a few weeks with the travel/work schedule I've got going right now, honestly. So if someone else wants to investigate further that would be awesome, or I can pick it up around the 20th when things slow down for me.

I suppose that the solution is just to set some simple 301 redirect rules.
If I am not mistaken, we use Cloudflare's nginx as the production server. I am not very familiar with it but this article might be helpful.
# Translations
http://elixirschool.com/([\w]{2})/lessons/(.*) → http://elixirschool.com/lessons/$1/$2
# Original lessons
http://elixirschool.com/lessons/([\w]{3,})/([\w]+) → http://elixirschool.com/lessons/en/$1/$2

@ybur-yug / @nscyclone — as nice as this change would be, I think the logistics to pull it off are posing to be quite significant as-is.  Can we close this and evaluate it at a later date?

So many updates have happened I'm just going to close this. Would be easier to do from scratch when we have a better plan.


Improved Docs

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

please don't refuse my pull request
i'm beginner in gitthub
i have to grow up


Changes from Hackathon

Let me know if you have any questions!

In addition to the above comments, it looks like this commit is missing the mayhem.yml workflow, however I see it in another branch of your fork (mayhem). Did you mean to commit a different branch?

@andrewwchong Please make your changes to this PR by May 14th, otherwise we will have to close it.

Sorry, I pull requested using the wrong branch. Please refer to the other pull request.


Refactor: simplify req.acceptsEncodings with spread syntax

Replaced use of arguments in req.acceptsEncodings with the spread syntax (...encodings) to improve readability and reduce code complexity, while maintaining the same functionality.

already added #6086

Hello @Abdel-Monaam-Aouini Thanks for your input on this! I went with a slightly different approach to keep things concise by calling accepts(this).encodings(...encodings) directly, avoiding the intermediate accept variable. This makes the code a bit more streamlined and skips a minor, unnecessary allocation. I thought this might be more consistent with our style, but I’m happy to discuss if you see other benefits to your approach!


Make Vcpkg Experimental detector

Context
As we do Vcpkg rollout, we want to make sure we have good metrics about its performance and unexpected failures across the board by having it running as an experimental detector

Is there any information how it can be used and what is supported?


Immutable header on service-worker.js

Describe the bug
Sapper applied the following cache-control header to service-worker.js: no-cache, no-store, must-revalidate
SK in the node-adapter currently applies the immutable header: public,max-age=31536000,immutable
https://github.com/sveltejs/kit/blob/f92c3e212dbb3247904fd5c3229c040a9650db41/packages/adapter-node/src/handler.js#L82-L89
We either need to hash the SW somehow or remove the immutable header. Otherwise it is impossible to update the SW,
Reproduction
Can provide if needed.
Logs
No response
System Info
"@sveltejs/adapter-node": "1.0.0-next.61"
"@sveltejs/kit": "1.0.0-next.230"

Severity
serious, but I can work around it
Additional Information
No response

This issue was resolved by only adding the immutable header to the _app/immutable folder


Add remote addr (ip) example

Hiya!
Made a quick example for using the remote addr field that was added before. While not particularly interesting from a code perspective it is useful since if you are hosting that particular example you can use it as a way to check what your external IP is. I just kind of think it is neat. :P
Name of the example is ip but that may be a bit too generic? Idk.

Beautiful, thank you!


Contribution Guidelines Update 2022

The additions to the Contribution Guidelines

Please don't keep the default commit name that github makes. Change it, to describe very briefly what the commit does. @webdev03 'Update auto.yml' is no good_.
Do not fight in PRs or issues, that just slows things down.

OK, you should add this in a CONTRIBUTING.md file. It will prompt everyone who wants to contribute too.
Sorry for the default commit names. We could enforce semantic naming with Commitizen...

OK, you should add this in a CONTRIBUTING.md file. It will prompt everyone who wants to contribute too. Sorry for the default commit names. We could enforce semantic naming with Github Actions...

Already did that.

I edited it.

@webdev03 i dont like what you did.

@webdev03 fixed it.

K

This isn't an issue to comment on, it's a notice. I will delete all the comments.

@webdev03 update the post when updating the file.

Closing because this isn't a real issue. This will still be pinned.

Since nobody will be contributing I don't need this pinned.


Unscheduling an item in pyglet.clock doesn't really seem to unschedule it

So I encountered this situation when switching a sprite image from an animation to a single image on the go.  Then the sprite seems to try to render the animation anyway, even though it's not there any longer.  Trying to track it but no success yet:
Traceback (most recent call last):
File "/Users/wojtek/PycharmProjects/lands/client/main.py", line 13, in
pyglet.app.run()
File "/Users/wojtek/PycharmProjects/lands/venv/lib/python3.7/site-packages/pyglet/app/init.py", line 144, in run
event_loop.run()
File "/Users/wojtek/PycharmProjects/lands/venv/lib/python3.7/site-packages/pyglet/app/base.py", line 175, in run
self._run()
File "/Users/wojtek/PycharmProjects/lands/venv/lib/python3.7/site-packages/pyglet/app/base.py", line 187, in _run
timeout = self.idle()
File "/Users/wojtek/PycharmProjects/lands/venv/lib/python3.7/site-packages/pyglet/app/base.py", line 308, in idle
redraw_all = self.clock.call_scheduled_functions(dt)
File "/Users/wojtek/PycharmProjects/lands/venv/lib/python3.7/site-packages/pyglet/clock.py", line 333, in call_scheduled_functions
item.func(now - item.last_ts, *item.args, **item.kwargs)
File "/Users/wojtek/PycharmProjects/lands/venv/lib/python3.7/site-packages/pyglet/sprite.py", line 281, in _animate
frame = self._animation.frames[self._frame_index]
AttributeError: 'NoneType' object has no attribute 'frames'

Thanks for opening the ticket. Are you able to show a short code example that illustrates the problem? I can't replicate this on my end.

Tried to replicate outside of my code and didn't manage to do it.
It could have something to do with the fact that I'm running a loop which runs object updates (movement, combat, etc.) using pyglet.clock.schedule_interval, and then I was iterating over a copy of the set of all in game sprites, while modifying that set in the meantime.  And then on_draw was called trying to draw them and perhaps some of them could have been deleted in the meantime because it also made a copy of the sprite set before starting iteration?

In general, modifying something while you're iterating over it is a bit dangerous. Instead of deleting things during the iteration, how about making a separate list for "dead sprites", and then adding to that list instead of deleting them. You can then deleting all of them at the end, or beginning of your next iteration.
Just in case you're not, you should also make sure that you're calling Sprite.delete() on the Sprites, instead of just del sprite.

Yup, made a sprite deletion queue and some other tweaks and it seems to be working.

Great! I'm glad that fixed it for you.


VS Code file paste button always disabled.

VS Code insider 1.48
I have copied a file and now want to paste on another folder.
I can't paste it because paste button is always disabled.

/duplicate of #100797


Card 6548 update docs

CARD-6548
https://peachworks.leankit.com/card/639265513

Merged https://github.com/peachworks/ngPeachBootstrap/commit/51fbf42722aa22d143bb9cc6dd7bccf524c476f5

v1.2.3


Verify that files passed in as values to exec processes operate as expected on cloud executors.

Description of the bug
In the locidex select module, we pass in the manifest file as a file that is resolved later. Similarly this occurs when using Bakta. As the desired behaviour on cloud executors is to stage copy the file into the place of execution it needs to be verified that this step does work.
Command used and terminal output
No response
Relevant files
No response
System information
No response

Fixed in: https://github.com/phac-nml/mikrokondo/pull/96


Support partial goal progress in criteria

PR includes

[x] Feature

Related issues
Resolve #2778
Related issues https://github.com/taskany-inc/bricks/issues/1137
QA Instructions, Screenshots, Recordings

Also added accounting of partial score from linked goal by criterion


Fix broken links as per LinkChecker output.

Fixes #550.
URL        `rope_science/intro.md'
Name       `Rope Science'
Parent URL https://google.github.io/xi-editor/docs/crdt-details.html, line 596, col 78
Real URL   https://google.github.io/xi-editor/docs/rope_science/intro.md
Check time 3.344 seconds
Size       5KB
Result     Error: 404 Not Found
 5 threads active,     0 links queued,  129 links in 134 URLs checked, runtime 16 seconds

URL        `../crdt.md'
Name       `doc/crdt.md'
Parent URL https://google.github.io/xi-editor/docs/rope_science_09.html, line 502, col 51
Real URL   https://google.github.io/xi-editor/crdt.md
Check time 3.801 seconds
Size       5KB
Result     Error: 404 Not Found

URL        `../update.md'
Name       `update.md'
Parent URL https://google.github.io/xi-editor/docs/rope_science_12.html, line 604, col 82
Real URL   https://google.github.io/xi-editor/update.md
Check time 3.828 seconds
Size       5KB
Result     Error: 404 Not Found

NOTE: I updated the update.md link, but then realized I don't know where it is. Any pointers?

Turns out LinkChecker only crawls a single page, so I'll be using it to cover the entire documentation and updating as I go.
It might be useful to include a call to LinkChecker (or equivalent software) to prevent link rot for pull requests and changes in the future.

@LiHRaM Awesome, thanks for this. I believe 'update.md' is now a section in frontend.md, although @ArazAbishov can probably clarify. If you could ping myself or araz when this is ready to go that would be great. Thanks!

@cmyr I'm pretty happy with the result. (I managed to find the appropritate update.md section etc.)
Ready to merge! :+1:

Awesome, thanks for this! I'm also very open to suggestions around adding something like LinkChecker to the CI, haven't looked into that before.

@LiHRaM After deployment two links are still not working on the frontend page; the second two links in the additional resources section. The first looks like a capitalization issue, not sure about the second.
I guess to test that properly we'd have to use jekyll to build the site locally? Not totally sure. 😕

Ugh. Sorry. LinkChecker seems to ignore 404 errors on references to github. (Crawling recursively with wget also ignores those.)
I'm sorry, I didn't realize capitalizing a section heading would ruin the reference. :( I'll fix those and make another pull request. I'm also running a copy on lihram.github.io/xi-editor/, so you won't have to pull it before you can see what it looks like deployed.
Here's what the script looks like:
$ wget --spider -r -l3 -nv https://lihram.github.io/xi-editor/ 2>&1 | grep -A100 "Found \(.*\) broken links\?."
Found no broken links.


resetPasswordForEmail does not return error it throws

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
when calling the resetPasswordForEmail with an invalid captchaToken
      const resetPasswordForEmailResponse =
        await supabase.auth.resetPasswordForEmail(email, {
          redirectTo: emailRedirectTo.toString(),
          captchaToken,
        });

instead of returning the AuthApiError in the { error } of the response it throws somewhere in the call stack which seems to be uncatchable from client code.
To Reproduce
callresetPasswordForEmail with invalid captchaToken to see it throw instead of be returned in the response.
Expected behavior
either return the error in the response, or at the bare minimum allow for callers to wrap in a try/catch to stop from throwing.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
Add any other context about the problem here.

hi @lifeiscontent, i can't seem to reproduce the error but it would be great if you can provide the code you use for creating the supabase client.


dots windowcontrols disappear

Im really appreciate this extension for steam, but in new version, dots style windowcontrols has gonne, and old dots style doesn't work for new steam client,
so, is it possible to update dots style work for new steam client?
best regard !

v3.0 adds support for macOS and Windows-like window controls.


coverage is shitty and broke the build

I am on it.

https://github.com/bcoe/nyc/issues/130
I will try to switch to ava because I think the issue above is related to the fact that tape is not using multiple process so I will handle babel 6 upgrade right now.

Closed by switching to ava.


chore: Add conditional compile for VRCSDK API

Fixed some compile errors I've found when using AvatarOptimizer in non-VRChat project.

I don't want to release 1.6.x in the future unless I have fatal bugs so could you work on 1.7.x branch? When I have to release 1.6.x, I'll backport this fix to 1.6.x.

All right!


How do I instantiate a model directly from a string containing json?

Hi, sorry to ask such a dumb question, but I can't figure out how to do a simple decode from a string containing my json to my model object:
struct doh: Codable {
  var homer = ""
}
extension doh : ArrowParsable {
  public mutating func deserialize(_ json: JSON) {
    homer <-- json["homer"]
  }
}

now to create one:
let myJson = "{ \"homer\": \"simpson\"}"
var aSimpson = doh()
aSimpson.deserialize(JSON(myJson)!)

this doesn't work! How do I create a doh object from the myJson string?
Thanks.

Hi @ntkon,
You need to go through native JSONSerialization in order for it to work.
(Actually this could be a great future improvement to support JSON strings)
Here is the working version:
let myJson = "{ \"homer\": \"simpson\"}"

guard let jsonData = myJson.data(using: .utf8),
    let jsonObject = try? JSONSerialization.jsonObject(with: jsonData, options: .mutableContainers),// as? NSDictionary,
    let json = JSON(jsonObject) else {
    return
}

var aSimpson = doh()
aSimpson.deserialize(json)
print(aSimpson)

Also the aSimpson.deserialize(json) part can be written like:
aSimpson <-- json

Hope this helps :)

Thanks @s4cha, much appreciated. I'm liking using arrow, ws and stevia - nice tools.

Sorry, I forgot to do that. Closed :)


[Patch] Update regex to parse hook names from module identities

This allows the hook names to be correctly parsed from modules published to npm under a user's namespace/scope - e.g. @user/sails-hook-name
Follows discussion in #3502

I updated my comment on the issue, the regex can be reduced further to the below:
/^(?:@.+\/)?(sails-hook-)/

See: https://regex101.com/r/lC8sM4/2
Sorry! :)

@lennym nice- this was the intended behavior (unless an identity is specified in the hook def, in which case that's used). Do you think you could tweak this pr to support using that if it's not undefined? Happy to help if I can-- this is what we need to be able to properly override core hooks via npm installed hooks (eg If you npm install an experimental version of sails-hook-orm-mongoose in your app that is fully compatible with blueprints and pubsub, then that hook needs to expose its identity as  'orm' ). @sgress454 and I can help with tests and docs on this if you don't have time (we were wanting to get this in anyway)

And @salakar, thanks for helping review!

Working on this now, adding some tests and should have it merged soon.


Consolidate UPath, test fsspec local

The primary goal here is to consolidate UniversalPath and UPath.
This helps with some down stream usage.
This also adds testing for fsspec using file:// uri, which it turns out seem to be failing on Windows currently, so skipping those for now.
To my knowledge, there shouldn't be any significant functionality differences, other than no longer needing the UniversalPath class.
If this approach is generally acceptable, the examples notebook will need to be updated as well.

This is great @brl0! I didn't realize you could get around circular imports like this. That's why I initially split UniversalPath from UPath. Could you run nox from the universal_pathlib directory and commit/push up the changes to get the formatting conformed to the rest of the code?

Thanks @andrewfulton9, I appreciate the feedback!  I figured that was the reason for the split, it was definitely tricky to work though the circular imports, and I was close to giving up.  As usual, all credit goes to stack overflow for guiding the way.  :)
I haven't updated the notebook outputs yet.  I am thinking that might be better to do in another PR, in part because this one is pretty large already, and in part because I am running into a couple of issues when trying to do so.  The issues I noticed were that the UPath untested default warning appears, which we may want to filter or suppress, since this notebook is the primary user documentation.  Also, the readme.exists() cell is now returning False, so it looks like something somewhere may have broken, although I don't think it is in this PR, I tested on main and the issue still happens, so I will create an issue to address that.
Since this is already so large, I am holding off on any other changes, clean up, etc for another PR.
I think this PR is ready for your review.  Let me know if you have any questions, concerns, etc.

Merged! Thanks @brl0


[ic-card] updating card text color to the correct spelling fails accessibility

Summary of the bug
Currently in the card css we have the text color set to color: var(--ic-architechtural-black) !important; but this is an incorrect spelling and should be color: var(--ic-architectural-black) !important;.
When updating to the correct spelling of the variable name, the card only shows black text and therefore when slotted into hero, doesn't take on the var(--ic-theme-text) like it used to.
Tried adding !important to the theme text css line in hero but it has no affect.
🪜 How to reproduce
Tell us the steps to reproduce the problem:

Update the css variable to the correct spelling in ic-card.css
Run storybook
Go on a hero story with a card in the right content slot
See the error

🧐 Expected behaviour
The behaviour should remain as it is now where the text turns white. Updating the variable name to the correct spelling shouldn't impact the current behaviour.

spelling is wrong, but color issue of text may be fixed

@GCHQ-Developer-741 to look into as part of dark mode


Update ubuntu1604.json

Updated ubuntu-16.04.2-server-amd64.iso to ubuntu-16.04.3-server-amd64.iso. Notice the "2" to "3." The old URL is no longer valid.

Never mind. @splashx covered this in #33 - closing.


Fix bugs for compute of Bias; Add support labels 0 and 1; Increase minimum out iterations

/intelci: run


[2.3.0-pre6] (found cause) nil pointer dereference on hub checkout

I was running git checkout https://github.com/robbyrussell/oh-my-zsh/pull/3007, where PR 3007
is from an unavailable fork. I found the issue on lines 98 and 99 of commands/checkout.go:
https://github.com/github/hub/blob/4777bd3/commands/checkout.go#L98-L99
where you're using pullRequest.Head.Ref as well as remote.Name which are probably nil in
this instance (in 2.2.8 there's a check for that).
The thing is, rather than checking if they are null and throwing an error if that's the case,
we can just use the following command from this GitHub guide:
$ git fetch origin pull/ID/head:BRANCHNAME

Where ID is the PR number and branchname can be anything. If the source fork is not available,
branchname could be pr-ID or just fetch it onto FETCH_HEAD and checkout that afterwards.
Below is the crash report from hub:

Error (runtime.errorString): runtime error: invalid memory address or nil pointer dereference
Stack:
goroutine 1 [running]:
panic(0x8d8460, 0xc820010150)
	/home/travis/.gimme/versions/go1.6.3.linux.amd64/src/runtime/panic.go:443 +0x4e9
github.com/github/hub/commands.transformCheckoutArgs(0xc8200c02c0, 0x0, 0x0)
	/tmp/go/src/github.com/github/hub/commands/checkout.go:94 +0x11e3
github.com/github/hub/commands.checkout(0xc07e20, 0xc8200c02c0)
	/tmp/go/src/github.com/github/hub/commands/checkout.go:34 +0x30
github.com/github/hub/commands.(*Command).Call(0xc07e20, 0xc8200c02c0, 0x0, 0x0)
	/tmp/go/src/github.com/github/hub/commands/commands.go:45 +0x188
github.com/github/hub/commands.(*Runner).Call(0xc820011700, 0xc07e20, 0xc8200c02c0, 0x0, 0x0, 0x0, 0x0)
	/tmp/go/src/github.com/github/hub/commands/runner.go:109 +0x5e
github.com/github/hub/commands.(*Runner).Execute(0xc820011700, 0x0, 0x0, 0x0, 0x0)
	/tmp/go/src/github.com/github/hub/commands/runner.go:97 +0x221
main.main()
	/home/travis/gopath/src/github.com/github/hub/main.go:14 +0x50

Runtime:
GOOS: linux
GOARCH: amd64

Version:
git version 2.9.3
hub version 2.3.0-pre6

Thanks for reporting! This is indeed something that I have overlooked when reimplementing the commmand for the 2.3.x series.

Awesome! Ping me if you need help testing. I'll read the contribution guide and try to solve it myself once I have a moment; I've been looking for an excuse to try the Go language :smile:

@mcornella Thanks for the interest, but no need right now! I've already made a fix. Will push soon.

Wow you're fast! Thanks! Closing then, you've got this under control :wink:


Performance issue

library which works fine while loading for the first time but from the subsequent calls, it take around 30-40 seconds.
I tried using all 3 loaders but everything takes a long time to load. I have records with 3000 entries. Could you please check this performance issue?
Thanks,
Murthy

Can you try running OpenDbExample in the module examples? Also can you provide a link to your code?
thanks

I executed OpenDbExample, both Simple & jaxb works fine locally but Dom loader was taking forever time.
But when i used either simple or jaxb loader and deploy in Cloud Foundry, it works only for first load and take 30sec to 5min to load for subsequent calls. You could try pushing your code into cloud foundry here https://pivotal.io/platform
you could create a trial account and try.

I had a similar problem on my laptop, and it turns out being related to /dev/random blocking, so getting a new random seed would take forever. It impacted other programs as well (randomly!).
On the pivotal website there is an explanation on how to switch to dev/urandom (this should not block):
https://discuss.pivotal.io/hc/en-us/articles/230141007-Java-Web-Applications-Slow-Startup-or-Failing

That's certainly interesting - thanks for pointing out.
@murthychandrappa can you confirm that this helps with your problem?

Hi!
It seems that this performance issue actual in 2021.
This is my test on the default JVM setting in Linux:

This is normal behavior:

The instruction from Oracle that was linked before is out of date for JDK 11 and later. You should correct the settings file in your $/usr/lib/jam//conf/security/java.security file.
According to this 12 years old issue () to avoid config setting in Linux we can refactor code from
   public KdbxHeader() {
        **SecureRandom random = new SecureRandom();**
        cipherUuid = AES_CIPHER;
        compressionFlags = CompressionFlags.GZIP;
        masterSeed = random.generateSeed(32);
        transformSeed = random.generateSeed(32);
        transformRounds = 6000;
        encryptionIv = random.generateSeed(16);
        protectedStreamKey = random.generateSeed(32);
        streamStartBytes = new byte[32];
        protectedStreamAlgorithm = ProtectedStreamAlgorithm.SALSA_20;
    }

To:
   public KdbxHeader() {
        **SecureRandom random = SecureRandom.getInstance("SHA1PRNG");**
        cipherUuid = AES_CIPHER;
        compressionFlags = CompressionFlags.GZIP;
        masterSeed = random.generateSeed(32);
        transformSeed = random.generateSeed(32);
        transformRounds = 6000;
        encryptionIv = random.generateSeed(16);
        protectedStreamKey = random.generateSeed(32);
        streamStartBytes = new byte[32];
        protectedStreamAlgorithm = ProtectedStreamAlgorithm.SALSA_20;
    }

I think I could test it and make a PR with a fix if you don't undermind.

Fixed in 2.2.1

Hi. Sorry for bumping the closed issue but, this is still unresolved for us. For each next call of the tool, it takes longer to execute. I've used KeePassJava2 2.1.4 and then 2.2.1 and was still able to reproduce the error.
Anything else that I am missing?
Using Java17 to compile and start the lib.

Hi
Which OS are you using?
Did you try the test from Oracle?
Have you tried using a different SecureRandom algorithm?
Thanks

Hi Jorabin and thank you for commenting,... I will try everthing you've suggested soon and give you a feedback asap.

Hi @jorabin,
I've checked the Oracle test and I am not affected by "waiting time", nevertheless, I've also tested execution of JAR with -Djava.security.egd=file:/dev/./urandom parameter. It didn't changed time execution.
OS: Windows 10, Java 17 on WSL2, Unbutu:
java -version
openjdk version "17.0.12" 2024-07-16
OpenJDK Runtime Environment (build 17.0.12+7-Ubuntu-1ubuntu222.04)
OpenJDK 64-Bit Server VM (build 17.0.12+7-Ubuntu-1ubuntu222.04, mixed mode, sharing)

I am using DomDatabaseWrapper.load to load the kdbx Files.
Also, our KDBX Files are not small, but not that huge either. Each KDBX has 6 Groups, each having 25 Entries. About half entries have more "String Fields".
And, I haven't used different SecureRandom algorithm.
Also, this Problem occurs when opening different KDBX Files one after another and not when reopening already opened KDBX. So the use-case to reproduce is:

open kdbx1, takes 10 seconds
open kdbx2, takes 20 seconds
open kdbx3, takes 30 seconds

Please try moving to 2.2.2 and use the Jackson implementation.

Also, it would be interesting to know what happens when you run OpenDbExample https://github.com/jorabin/KeePassJava2/blob/2c47bd7f1513353f1594b95534a91d24a3078088/example/src/main/java/org/linguafranca/pwdb/kdbx/OpenDbExample.java#L72

Hi @jorabin, I've upgraded to 2.2.2 and switched to JaxbDatabase. All databases are opening under 5 seconds, no matter the size or how many times I execute the commands. I think the issue can now be closed. Also, I am not sure how many people have several KDBX Files to get the exact use-case out of it, but, it would be maybe wise to update the readme that the DomWrapper has been showing some Performance issues.
Thank you for your support!

Hi @lordgreg
Thanks for the update.
There are significant warnings in the README that the DomWrapper is slow ...

A DOM based implementation of KDBX. Being DOM based it is rather slow, but messes less with existing content than the other implementations. Known to work on Android.

Suggest that you use Jackson, not Jaxb, as 2.2.3 supports improved management of secrets in memory for Jackson only and I will drop support for Jaxb (as well as Dom and Simple) altogether in version 3.
Would be good to hear how that goes for you and will close this issue (again) having heard back.
thanks

Would be good to hear how that goes for you and will close this issue (again) having heard back.

Give me until Monday, I will provide the feedback :)
Kind regards.

Hi @jorabin - I've switched to JacksonDatabase and currently see no time difference, everything works exceptional as it should. No hiccups even after opening the 4th, 5th or 6th Database. From my point of view, the issue can be closed.
Thank you for your awesome support and work!


RE-622 Fix GitHub URLs ending with .git in report

This change strips ".git" from GitHub URLs used in the report.
The report produced by osa-differ/rpc-differ includes links to the
individual commits on GitHub. If a URL supplied on the command-line or
defined in an Ansible role file ended with ".git", the links generated
for the report would fail with a 404 without this change.
Without this change:
Valid commit links only generated from a URL not ending in ".git":
https://github.com/major/osa_differ/commit/SHA
Invalid commit links generated from a URL ending in ".git":
https://github.com/major/osa_differ.git/commit/SHA

Codecov Report

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

@@           Coverage Diff           @@
##           master      #15   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           2        2
  Lines         270      270
  Branches       35       35
=======================================
  Hits          225      225
  Misses         37       37
  Partials        8        8

Impacted Files
Coverage Δ

osa_differ/osa_differ.py
82.88% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


MySQL dialect reading column name from undefined

15:15:55 🚨 Cannot read property 'Column_name' of undefined
TypeError: Cannot read property 'Column_name' of undefined
    at MySQL.<anonymous> (/var/www/directus_v9/node_modules/knex-schema-inspector/dist/dialects/mysql.js:275:60)
    at step (/var/www/directus_v9/node_modules/knex-schema-inspector/dist/dialects/mysql.js:33:23)
    at Object.next (/var/www/directus_v9/node_modules/knex-schema-inspector/dist/dialects/mysql.js:14:53)
    at fulfilled (/var/www/directus_v9/node_modules/knex-schema-inspector/dist/dialects/mysql.js:5:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Smells like #23 to me, which was fixed with #19, a simple null check, I would assume this affects all dialects (haven't looked at the code)

Yeah definitely, this one over here in the primary() method:
https://github.com/knex/knex-schema-inspector/blob/e14f525d5302ff1d83f1467a3a1f2873f9d906ef/lib/dialects/mysql.ts#L278-L280

I wrongly assumed that every table would have a primary key returned here

Hi, is there a plan to fix this issue?


Convert NAD83 XCoord/YCoord to NAD83 (or WGS84) Lon/Lat

Preferably in the cleaning script so it's transparent what transformations took place.
NAD83 or WGS84 should be alright for the purposes of this dataset since they are relatively close (1~1.5m off).

Script that will convert is done!

Converted. I am going to keep this issue open to see if everything works when we put the data on a map.

Closing.  If there's an issue, we open a new issue! :)


Client crash on start up due to Exception from crafttweaker for 1.12

What happens:
When attempting to start Minecraft the game crashes with an error report:
11:42:37 | game | UCH	minecraft{1.12} [Minecraft] (minecraft.jar)
11:42:37 | game | UCH	mcp{9.19} [Minecraft Coder Pack] (minecraft.jar)
11:42:37 | game | UCH	FML{<IP_ADDRESS>} [Forge Mod Loader] (forge-1.12-<IP_ADDRESS>7.jar)
11:42:37 | game | UCH	forge{<IP_ADDRESS>7} [Minecraft Forge] (forge-1.12-<IP_ADDRESS>7.jar)
11:42:37 | game | UCE	crafttweaker{4.0.2} [CraftTweaker2] (CraftTweaker2-1.12-4.0.2.jar)
11:42:37 | game | UCE	ctgui{1.0.0} [CT-GUI] (CraftTweaker2-1.12-4.0.2.jar)
11:42:37 | game | [10:42:37] [main/FATAL] [FML]: The following problems were captured during this phase
11:42:37 | game | [10:42:37] [main/ERROR] [FML]: Caught exception from crafttweaker (java.lang.NoClassDefFoundError: net/minecraftforge/event/furnace/FurnaceFuelBurnTimeEvent)
11:42:37 | game | [10:42:37] [Forge Version Check/INFO] [ForgeVersionCheck]: [forge] Found status: UP_TO_DATE Target: null
What you expected to happen: Minecraft should load.
Script used (Please Pastebin or gist your script, posting an unpasted or ungist'd script will automatically close this issue): No script.
Minetweaker.log file (Please Pastebin or gist your file, posting an unpasted or ungist'd file will automatically close this issue): Log file is empty as client doesn't start
Affected Versions (Do not use "latest"):

Minecraft: 1.12
Forge: 1.12-<IP_ADDRESS>7
Crafttweaker: both CraftTweaker2-1.12-4.0.3 and CraftTweaker2-1.12-4.0.2

Your most recent log file where the issue was present:  Crash report above

update forge.


Readability

Added colour to highlight vulnerable API keys

Thanks for the commit! For the maps_api_scanner.py, I already changed the code for showing all the vulnerable API list again in the end however it was not existing in the python3 code as well, so this should work.


make functions and packages public

Since this is a framework for testing, we should attempt to make all functions (within reason) public.

ex: PrivValFileContent and OverwritePrivValFile in v7.

this may also be useful with the internal/ repo. For example, the local-interchain lense clone could use docker internal functions, but it can not be accessed outside.

Was just going to create an issue to make some of the functions in the internal package public, specifically the FileRetriever type and associated functions, fetching files (such as private keys) from validator nodes can be extremely useful in some more complex test scenarios.

still have to do v7

Compelted


New Snap Assist shortcut

Description of the new feature / enhancement
Add shortcut for the new Snap Assist in time for Windows 11 launch.
Win + Z Open Snap Assist

Scenario when this would be used?
Used to open Snap Assist using keyboard allows for it to work when Mouse Hover doesn't (several legacy apps and some new).
Supporting information
No response

I think this is a good idea. But this is nothing the PowerToys team can do. Please fill a feedback report in Feedback Hub. Thank you.

Sorry.
I thought the Shortcut Guide Power Toy was about all Windows shortcuts.
Maybe the Ctrl + D shortcut could change into:
Ctrl + D Add a virtual desktop (< or > to switch)
This way the guide would free a row to insert the new shortcut.

This is tracked in #179


How did you created the xcframework of SPM library

Hello @sjavora and thank you for providing this.
OOC I am interested to know how you created an xcframework from an SPM package.
I am trying to do this on pointfree snapshot testing but without success.
Thank you in advance

Hi, check out https://github.com/sjavora/swift-syntax-xcframeworks/blob/main/.github/bootstrap.sh, which is used to produce the binaries.

Hi @sjavora, this is interesting approach.
Recently our team came across to create swift-syntax's xcframework to reduce the compiler time on iOS platform.
I have few questions regarding this PoC since I notice that while building the wrapper in the script is using:
swift build --package-path $SWIFT_SYNTAX_NAME --arch $ARCH -c $CONFIGURATION -Xswiftc -enable-library-evolution -Xswiftc -emit-module-interface

Do you have an example of consuming this in a package that’s meant to be used in an iOS project, or consuming it within an iOS projcect directly? We notice that with the swift build it only build the macOS's lib file. I also not pretty sure whether there is a configuration on swift build can change the arch to iOS.

Have you considered using xcodebuild instead of swift build? What have you found to be the difference?
Since some people suggest using the xcodebuild which available on Xcode 11 onward. I wonder is there any limitation not to use it?

For us we tried with using the xcodebuild but the target package can not recognize the library inside the swift-sytax-wrapper package. By using the xcodebuild  we need to make all the library as dynamic. We notice that need to make it static after archiving. The ${Framework}/Products/usr/local/lib/  won't have the framework file.
With dynamic's library, we can generate the framework file and export the xcframework. But the target package won't recognize the library inside the wrapper package.
Thanks for your time for reading and provide this interesting PoC again.

@carterTsai95 macros should not need to run on anything but macOS. There shouldn't be a reason to build them for iOS... except that's not true in practice in if you put the macros directly in the same package (or project I guess?) as the code that uses them. Since we put our macros in a separate package, this problem doesn't apply to our app (see https://forums.swift.org/t/macros-and-xcframeworks/68122/37 and really the whole thread - this is the most info I think you can get on the topic).
Dynamic libraries are a no go as per this post.
As for xcodebuild, I did actually end up using it internally for some reason, but those changes haven't made it here yet. The problem was that you need to create the .a library file manually out of object files, because there is no way (that I know of) to have xcodebuild do that (info in the thread again).

@carterTsai95 macros should not need to run on anything but macOS. There shouldn't be a reason to build them for iOS... except that's not true in practice in if you put the macros directly in the same package (or project I guess?) as the code that uses them. Since we put our macros in a separate package, this problem doesn't apply to our app (see https://forums.swift.org/t/macros-and-xcframeworks/68122/37 and really the whole thread - this is the most info I think you can get on the topic).
Dynamic libraries are a no go as per this post.
As for xcodebuild, I did actually end up using it internally for some reason, but those changes haven't made it here yet. The problem was that you need to create the .a library file manually out of object files, because there is no way (that I know of) to have xcodebuild do that (info in the thread again).

Thanks @sjavora for your input, it makes sense that separate out the macro package. I appreciate you take time to answer my question. 🙏


Deeper integration for package:test running on device

Relates to #26080
Right now you can run package:test tests on a device using Flutter driver but there's no easy way to get reports of failed tests and notify the host about them. In the meantime we can check the logs output for "Some tests failed."
Support for getting more fine-grained test reports could probably be added with some effort, either with Flutter driver or native testing frameworks (Espresso/XCUITest).
Creating as a tracking issue for pull requests relating to notifying the test runner about on-device test results.

Is there a reason to run this through driver? Or would it be better to have a flutter test that ran on device?


[Feature] XYZ grid or just X function to run all models

Hello,
It would be interesting to have an XYZ grid that let you run PEZ through different parameters values.
Especially being able to run through all the models.
Thanks.
(By the way, what is the difference between all models, and how to know which one to choose?)

Hi.
Currently there are no any plans to implement XYZ grids. Actually this extension is just a port of official Hard Prompts Made Easy / PEZ Dispenser implementation with minimal revisions.
All models are pretrained OpenCLIP models.
The author of Hard Prompts Made Easy implementation says:

clip_model: the name of the CLiP model for use with . "ViT-H-14" is the model used in SD 2.0 and Midjourney. "ViT-L-14" is the model used in SD 1.5. This should ideally match your target generator.
clip_pretrain: the name of the pretrained model for open_clip. For "ViT-H-14" use "laion2b_s32b_b79k". For "ViT-L-14" use "openai".


Ddt 994 transportdocument summaries

Looks like this is dependant on some changes from mp (moving some fields from cargo item it looks like)


Review add-to-app Android usage page

Extracting from https://github.com/flutter/flutter/issues/32863

[x] Make sure every instruction has both languages per platform
[ ] Describe DartEntrypoint pathToBundle, dartEntrypointFunctionName, DartCallback
[ ] Explain the need for vm:entry-point
[ ] Talk about pushing platform channel messages to Flutter before showing the view vs launching an entrypoint vs launching a route
[ ] Document the implications of FlutterEngine ownership by the app vs by the activity, onSaveInstanceState etc
[ ] Point existing add-to-app users to flutter.dev/go/android-project-migration for Android embedding migration.
[x] Remember to mention that if you don't keep your own FlutterEngine, you lose the VM service connection from flutter attach after you kill your activity/VC #24813
[ ] Specify why to create a FlutterEngine earlier
[ ] Review all classes with redundant names like FlutterActivity and make sure the full import path is specified
[ ] Describe how to specify different entrypoint/route with FlutterEngine

(triage) @xster Can you please assign this bug to somebody who is working on it or remove it from the march milestone. All bugs in milestones with a month should be assigned to people actively working on it.

Closing this issue for now. Track individual documentation issues as needed.


Non-Integer Pixel Coordinates Causing TypeError in tutorials_mnmst_histology.ipynb

Issue Description:
In the tutorials/tutorials_mnmst_histology.ipynb notebook within the MNMST project, there is a step titled "Set coordinates" where x_pixel and y_pixel are adjusted using a scale_fact. The code snippet involved is:
adata.obs["x_array"] = adata.obs["x2"]
adata.obs["y_array"] = adata.obs["x3"]
adata.obs["x_pixel"] = adata.obs["x4"] * scale_fact
adata.obs["y_pixel"] = adata.obs["x5"] * scale_fact

Unlike another tutorial (SpaGCN/tutorial/tutorial.ipynb) in the SpaGCN project, it appears that x_pixel and y_pixel are not adjusted using scale_fact in a consistent manner. Furthermore, since scale_fact may not be an integer, x_pixel and y_pixel become non-integers, leading to a TypeError when used as slice indices in image processing:
nbs = image[
    max(0, x_pixel[i] - beta_half) : min(max_x, x_pixel[i] + beta_half + 1),
    max(0, y_pixel[i] - beta_half) : min(max_y, y_pixel[i] + beta_half + 1)
]

This error occurs because Python requires slice indices to be integers. If x_pixel[i] or y_pixel[i] are non-integers, even with an integer beta_half, the resulting indices might still be non-integers.
Suggested Fix:
To resolve this, I suggest ensuring that all pixel values are explicitly converted to integers before being used as indices. This can be done using list comprehensions as shown below:
adata.obs["x_pixel"] = [int(x) for x in adata.obs["x_pixel"]]
adata.obs["y_pixel"] = [int(y) for y in adata.obs["y_pixel"]]

This tweak will ensure that all pixel indices remain integers, thus preventing the TypeError.
Impact:
This issue, if unresolved, can disrupt image processing tasks in automated pipelines, particularly those involving spatial coordinates in histology images. Addressing it will enhance the robustness and reliability of the tutorial scripts.
Request:
I look forward to any feedback or alternative solutions you might suggest. Thank you for your attention to this matter!

Hi, dear @KaishinShaw , I am sorry for the late reply.
The integration of morphological information was not well addressed in this paper, and we provide detailed tutorials on this in our other paper MuCST, which can be accessed at bioRxiv, where the full code for morphological information extraction can be found. I appreciate you pointing out this error, and I will update that part of the tutorials later.

Hi, @KaishinShaw
Thank you for bringing this issue to my attention. I have resolved the bug you reported. If you have any other issues or if the problem persists, please feel free to reopen this issue.
Best regards, Yu Wang


[Fix] compile error in source tutorial 3 C#.

body.ToArray() got called twice. This is a small  change.

@Skripd Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Thank you!


Add factory method for creating encrypted MessagePublisher

Adds factory method in QueueServiceSender to create an encrypted version of a MessagePublisher.
Reason for adding is that all of the support is already in place for encrypting messages when sending through a MessagePublisher, similar to what is available when using a MessageQueueProducer<T>.
Long-term plan with this addition is to deprecate MessageQueueProducer<T> and instead only use MessagePublisher, since they are so similar and share almost the same responsibility. It is not clear for all callers what the difference between these two classes are, so having just one would simplify for callers.

@xiaodong-izettle

Yes, totally agree... :-)
Nice one, @oskarwiksten. 👍


Fix OS 4721262: FailFast when JsAddRef fails in CallbackMessage ctor.

Allowing the code to continue after a failed JsAddRef will cause corruption of the datastructure which will cause issues later, notable during the dtor when it looks for the Ref which should have been added and should exist.
When OOM occurs and this call fails, we should terminate the process because the OOM is not recoverable in this case. Assert in debug builds and raise the FailFast error in any case.

This change is

@curtisman @Yongqu review please?

LGTM


DRAFT: UFT: start pipeline

resolves #134

Currently uploading test version, need --publish flag if correct.


SSO: Rebuild the server url without query string and/or hash

Summary
There have been reports of people unable to login using GitLab SSO. For SSO login attempts we were removing any potential query strings from the final url so they are not included as part of the server base url used for API calls.
Based on a small investigation it could be that GitLab not only appends query strings but also a hash as part of the redirect_uri, this hash in case it is present is not being explicitly removed.
With this PR, we attempt to fix this by reconstructing the base url using the base origin and the pathname in case that the mattermost server is hosted on a subpath
Ticket Link
Potentially Resolves: https://github.com/mattermost/mattermost-mobile/issues/4712

Building app in separate branch.

@amyblais once the build completes, maybe ask one of the reporters to confirm this indeed fixes it for them. (They will only be able to test it on Android)

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9758
https://circleci.com/gh/mattermost/mattermost-mobile/9756

Artifact links:
https://9758-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://9756-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk

Beta build is confirmed to work properly and allow SSO login.
Also retested latest google play version and confirmed the issue persists on latest google play release.

/cherry-pick release-1.34


Fix Rendering Issues and Update DBML Parsing for Latest Syntax

This pull request addresses the following updates and improvements:
•	Rendering Fixes:
•	Resolved issues with the live SVG rendering that previously caused errors with complex DBML schemas.
•	Updated the parser to handle larger DBML inputs efficiently.
•	DBML Syntax Update:
•	Aligned with the latest DBML syntax standards from dbdiagram.io.
•	Supported new syntax such as enum, project definition, relationships, and table groups.
•	Code Refactoring:
•	Cleaned up code for better readability and maintainability.
•	Added missing functionality in parser.ts to process updated DBML structures.
•	Bug Fixes:
•	Fixed issues with single and double quotes in enum values.
•	Improved error handling for invalid or incomplete DBML inputs.
•	Testing:
•	Tested with various DBML files, including complex and large schemas.
•	Ensured backward compatibility with older DBML syntax.

Thank you for this job, let me check on it in the next days and release a new plugin version

Your welcome check it out and let me know


Time Integration page in the docs is empty

page in the docs
source code

@efaulhaber Do you have time to create a page? I think you wanted to do that anyway when you created the stepsize callback.

I will update the page later when I add Kick-Drift-Kick time integration.


Assets folder

Icons for streaming services
Popcorn image for submit button
Background image
Popcorn container to hold watchlist

Updated popcorn piece for SUBMIT button

Created/added background image

Created/added popcorn bucket

Created/added streaming service icons
Uploaded Mockup


node: The chown_paths may be too large to chown

The number of files to chown could be really large such that the argument list becomes to long.

Codecov Report

Merging #132 (69a2f3b) into master (c0cc73c) will increase coverage by 0.95%.
Report is 129 commits behind head on master.
The diff coverage is 86.44%.

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   87.02%   87.98%   +0.95%
==========================================
  Files         122      135      +13
  Lines        7648     8771    +1123
==========================================
+ Hits         6656     7717    +1061
- Misses        992     1054      +62

Files Changed
Coverage Δ

astacus/server.py
0.00% <0.00%> (ø)

setup.py
0.00% <ø> (ø)

tests/system/test_astacus.py
100.00% <ø> (ø)

tests/unit/coordinator/test_cleanup.py
94.28% <ø> (ø)

version_from_git.py
0.00% <0.00%> (ø)

...tion/coordinator/plugins/clickhouse/test_plugin.py
29.37% <20.96%> (-10.82%)
:arrow_down:

...tion/coordinator/plugins/clickhouse/test_client.py
36.36% <36.36%> (-4.07%)
:arrow_down:

astacus/coordinator/cluster.py
89.60% <50.00%> (ø)

astacus/coordinator/plugins/m3db.py
87.03% <50.00%> (-2.69%)
:arrow_down:

...ation/coordinator/plugins/clickhouse/test_steps.py
40.00% <50.00%> (-11.73%)
:arrow_down:

... and 87 more

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


User access

We can declare the maximum size if array also and ask the user to enter the numbers he/she wishes.

Got it cleared.


Intellisense in Python REPL (console)

From @eliavv on September 7, 2017 17:41
I wish the extension would include Python REPL autocomplete feature like PTVS has.
Copied from original issue: DonJayamanne/pythonVSCode#1196

Your best option is to use IPython as your REPL as it provides autocomplete. Otherwise we have no plans to re-implement the REPL in the extension.

Thanks.
Is there any explanation on how to config the IPython as the default terminal?

@eliavv I don't think there is a way as IPython isn't a terminal per-se, but a Python application. IOW you would launch the terminal and then run ipython manually.


very slow

very slow sometime. It even cant work in some project but the origin Tslint extension work normal.

Terribly slow.
Please find the attached gif.

Same issue here.  Never had issue before the switch.

Please provide more info about which version of VS Code you are on, your project setup, which other extensions you have installed,  and what is slow

Extensions: Partial Diff, Rainbow Brackets, vscode-typescript-tslint-plugin
VSCode Version: <IP_ADDRESS>
webpack rule applied to .ts files:
{
test: /.tsx?$/,
use: [{ loader: "ts-loader", options: { transpileOnly: true, experimentalWatchApi: true } }],
exclude: /node_modules/,
},
usersettings.json:
{
"window.zoomLevel": 1,
"php.validate.executablePath": "C:/Program Files/PHP/php.exe",
"git.autofetch": true,
"explorer.confirmDelete": false,
"files.trimTrailingWhitespace": true,
// "tslint.autoFixOnSave": true,
"editor.formatOnSave": true,
"editor.codeActionsOnSave": {
"source.fixAll.tslint": true
},
"breadcrumbs.enabled": true,
"tslint.ignoreDefinitionFiles": true,
"editor.fontFamily": "Consolas, 'Courier New', monospace, Operator Mono",
}

What is slow is the auto-fix on save functionality, which was almost instant with the old extension, and now takes 4-5 seconds

After disabling auto-save, the plugin is still unusably slow, errs out constantly, and causes me to have to restart Code every 10 minutes.  Decision now is ESLint with Typescript config, or back to the deprecated TSLint extension

Same here. Unusably slow. Large files take ages to update, leading to race conditions with auto-fix on save where a change was made, the file is saved and TSLint having not caught up breaks the code.

Same issue here. It's so slow I can't believe it.

Same here. Seems to be needing a lot of resources.
Especially slow while running/watching ng serve app

Today, the RC for TypeScript 3.5 was released talking about speed improvements in regard to a regression introduced in 3.4:

Unfortunately, as part of a bug fix in TypeScript 3.4 we accidentally introduced a regression that could lead to an explosion in how much work the type-checker did, and in turn, type-checking time.

As VSCode currently uses 3.4 and this extension makes heavy use of the TypeScript server, it would explain the slowness. If this is true, we should see improvements with the next VSCode TypeScript dependency upgrade.

Does this reproduce in the latest VS Code insiders build?

Having the same issue with this.  Very slow, that everything turns to a crawl.  Sometimes, when I save, it doesn't recognize the changes fast enough and mangles the entire page.
Disabling the plugin resolves it.
Plugin Version: 1.2.3
Version: 1.43.0 (system setup)
Commit: 78a4c91400152c0f27ba4d363eb56d2835f9903a
Date: 2020-03-09T19:47:57.235Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.17134

Closing as TSLint has been deprecated in favor of ESLint and this extension is no longer being maintained
Please look into migrating to use ESLint and the ESLint VS Code extension


Process all messages from a connection

This updates to latest revault_tx (https://github.com/revault/revault_tx/pull/107) and fixes #24

Arg the fix for nightly isn't patched in hoggfuzz-rs...


Location simulation

Hi @adamhrz ! Thanks for the PR! I'm really excited to update this project with WatchKit. A few comments:

When I try to Build, Xcode notifies me that 1.25mileSampleGPS.json is missing. Could you either include the file or remove the dependency?

The matt-buckley.png image files snuck in, please remove :]

I noticed that the location manager is named WJRLocationManager-- for consistency could you remove the prefix?

There is some commented-out code in RunNotifier.m that should be removed

Raizlabs is great :] but I don't want to change the BundleIDs to com.raizlabs

I may have more comments after this round of edits. Thanks again for contributing!


lack of clarity in index.qnd about R3T

Feedback from Ian:
(also not sure what R3T is. Am I out of the loop? google returned a zip code in Manitoba)

fixex 03658ee54d98a01586681795ec7ec3b79b67816e