32bit integer encoded vertex indices seem to be decoded as 16bit

When loading the model below vertex indices for the mesh primitive are loaded as the correct values padded with alternating zeroes causing it to be corrupted when rendered, while the model loads correctly in other GlTF viewers.
https://sketchfab.com/3d-models/reference-frame-7b41e02853134b04b460b257951cd653
P.S. Great project!

PS. Great project!

Seeing how things work at the moment, maybe the decoders:
-- | Decode vertex indices
vertexIndices :: GlTF -> Vector GltfBuffer -> AccessorIx -> Vector Word16
vertexIndices = readBufferWithGet getIndices

-- | Decode vertex positions
vertexPositions :: GlTF -> Vector GltfBuffer -> AccessorIx -> Vector (V3 Float)
vertexPositions = readBufferWithGet getPositions

-- | Decode vertex normals
vertexNormals :: GlTF -> Vector GltfBuffer -> AccessorIx -> Vector (V3 Float)
vertexNormals = readBufferWithGet getNormals

-- | Decode texture coordinates. Note that we only use the first one.
vertexTexCoords :: GlTF -> Vector GltfBuffer -> AccessorIx -> Vector (V2 Float)
vertexTexCoords = readBufferWithGet getTexCoords

-- | Decode vertex colors
vertexColors :: GlTF -> Vector GltfBuffer -> AccessorIx -> Vector (V4 Word16)
vertexColors = readBufferWithGet getColors

should use generalised getters that are aware of the byteStride from the BufferView (used in the case of interleaved, non-tightly-packed vertex attributes) and also the componentType from the Accessor for the data it's decoding from the binary chunk.

Having done more digging, the 32-bit encoding of indices is the nature of the Sketchfab automatic conversion of uploaded models.
A good work-around I've found is importing to and exporting from Blender, which re-encodes the indices as 16-bit unsigned integers.

Having done more digging, the 32-bit encoding of indices is the nature of the Sketchfab automatic conversion of uploaded models.
A good work-around I've found is importing to and exporting from Blender, which re-encodes the indices as 16-bit unsigned integers.

Unfortunately, I've only been testing on Blender, so I appreciate the report. If you can post a gltf file somewhere for reference, I can poke around

Hey!
Try downloading this file as a GlTF. That's the one I've been using. https://sketchfab.com/3d-models/reference-frame-7b41e02853134b04b460b257951cd653

I can see the difference. Sketchfab:
    {
      "bufferView": 0,
      "componentType": 5125,
      "count": 2232,
      "type": "SCALAR"
    }

vs Blender:
		{
			"bufferView":3,
			"componentType":5123,
			"count":2232,
			"type":"SCALAR"
		}

So you are right that it is using 32bit unsigned integers (vs 16 from blender)

Yeah, that's what I was seeing.
There are workarounds so I thought I would document the issue in case any tripped up on it too.

Should be fixed by https://github.com/sgillespie/haskell-gltf-loader/commit/d133575a007d524da5bad68fc492a8591834c828

All working.


Locating the DeepLab V2 VOC12 Model

Hello,
I'm attempting to get your source working; however, I'm unable to locate a copy of the "deeplab_v2_voc12.pth" pretrained model file.  I've looked on the DeepLab V2 models page (http://liangchiehchen.com/projects/DeepLab_Models.html), but I don't see a model trained on VOC.  Do you have a link to the model file used in your code?
Thanks!

under the pretrain section of their README there are links via baidu and google drive for the pretrained model file.


Update README.md

Added a paper on smpc
Resource Addition
Category

New category that I named research pubs. Could be added to the articles section too, however.
Description

It's a well cited work on secure multi-party computation.
Checklist

[x] I have followed the contribution guidelines
[x] The resource is freely accessible (or marked if paid)
[x] I have added it to the appropriate category
[x] The link is functional
[x] The description is clear and concise

yeah we're adding papers under Articles. Let's keep it that way so far :)

That makes sense @leriomaggio


Active Element after successful update operation

Keyboard Focus should be placed on the edit icon of the list/table page after Successful update operation.

Can we use default focus restoration in MatDialog Config ?
const dialogRef = this.dialog.open(DialogFromMenuExampleDialog, {restoreFocus: false});


Changing it to be adam+rating function by using LambdaLR

I have not tested it at the MultiGPULossCompute part because colab only has one gpu.
But SimpleLossCompute looks ok.https://cdn.discordapp.com/attachments/874379751748427820/879414174025912480/unknown.png
If you want to train it on multi-gpu, I suggest starting lr to be 1e-3 or try it by yourself.
It looks like it has conflict. I may have time to change it by the end of the weekend.

I went ahead and resolved the merge conflicts. There's still some things that need fixing for the notebook to be functional (tactically - the altair plots are still using NoamOpt and need to be updated), but we can discuss the details in discord. Thanks for taking this on!


ruby interop tests failing

https://source.cloud.google.com/results/invocations/7d29e2c6-51d6-4a63-9320-bcb4b2c36ef4/targets/github%2Fgrpc%2Finterop_test/tests;group=tests;test=cloud_to_cloud:aspnetcore:ruby_server:cancel_after_begin:tls;row=1
There are a large number of test failures, and they all involve ruby as either the client or the server.

Looks like this could be is because of google-protobuf 3.10 gem was just pushed.
(not good because the gem is supposed to be backward compatible).
@rafi-kamal

Interop tests are failing on master at 100% rate.

@rafi-kamal  I believe there was a non-backwards-compatible change (needs to be rolled back?)
/usr/local/rvm/gems/ruby-2.5.3/gems/google-protobuf-3.10.0-x86_64-linux/lib/google/protobuf.rb:61:in `infer_package': undefined method `size' for nil:NilClass (NoMethodError)
	from /usr/local/rvm/gems/ruby-2.5.3/gems/google-protobuf-3.10.0-x86_64-linux/lib/google/protobuf.rb:109:in `fixup_descriptor'
	from /var/local/git/grpc/src/ruby/pb/src/proto/grpc/testing/test_pb.rb:9:in `add_file'
	from /var/local/git/grpc/src/ruby/pb/src/proto/grpc/testing/test_pb.rb:9:in `block in <top (required)>'
	from /var/local/git/grpc/src/ruby/pb/src/proto/grpc/testing/test_pb.rb:8:in `instance_eval'
	from /var/local/git/grpc/src/ruby/pb/src/proto/grpc/testing/test_pb.rb:8:in `build'
	from /var/local/git/grpc/src/ruby/pb/src/proto/grpc/testing/test_pb.rb:8:in `<top (required)>'
	from /usr/local/rvm/rubies/ruby-2.5.3/lib/ruby/site_ruby/2.5.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/rvm/rubies/ruby-2.5.3/lib/ruby/site_ruby/2.5.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /var/local/git/grpc/src/ruby/pb/src/proto/grpc/testing/test_services_pb.rb:23:in `<top (required)>'
	from src/ruby/pb/test/client.rb:42:in `require_relative'
	from src/ruby/pb/test/client.rb:42:in `<main>'

@haberman @teboring do you know the root cause of the failure? I'll see if 3.10.0 can be rolled back.

@jtattermusch That's not a backward-incompatible change, it just looks like a bug. How can I reproduce it?

I've yanked the gem.
➜   gem yank google-protobuf -v 3.10.0
Yanking gem from https://rubygems.org...
Successfully deleted gem: google-protobuf (3.10.0)

seems fixed on master.

@jtattermusch do you know what the root cause is or how can we reproduce the issue? We would like to release Ruby 3.10 again after fixing the bug.

I haven't verified this myself but this should be reproducable by checking out the grpc github repo and trying to load the same ruby-generated protos that this test client tried to load.
E.g. something like:
> git clone https://github.com/grpc/grpc.git
> cd grpc && ruby src/ruby/pb/src/proto/grpc/testing/test_services_pb.rb

Let me know if that doesn't work.
Thanks for addressing quickly btw.


Remove: purocean.YankNote version 3.12.9

## .\Tools\YamlCreate.ps1 -SkipPRCheck -Mode 5 -PackageIdentifier purocean.YankNote -PackageVersion 3.12.9

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

/azp run


[pure refactor] Move item-ui to separate module, move AppOptions to re_viewer_context

... both of which are towards the goal of being able to move out ViewerContext to re_viewer_context which then will allow a wave of new crates that don't depend on re_viewer, but only on re_viewer_context
Suitable for per-commit review!
Part of:

#1873

What
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct

I wanted AppOptions not to be part of the context but reconsidered then - it is nice to have experimental flags in there for the time being that we can use from all sort of places


Godot 4 script with an empty += assignment crashes the editor

Godot version
v4.0.dev.calinou [48857194b]
System information
Windows 10, Vulkan, Nvidia GTX 750 Ti
Issue description
The code that crashes the game is below. What causes the engine to crash is the direction += line.
extends CharacterBody3D

# Declare member variables here. Examples:
# var a = 2
# var b = "text"

# Called when the node enters the scene tree for the first time.
func _ready():
	pass # Replace with function body.

var movement : Vector3
var direction : Vector3
# Called every frame. 'delta' is the elapsed time since the previous frame.
func _process(delta):
#	movement = Vector3.ZERO
	if Input.is_key_pressed(KEY_SPACE):
		pass

	if Input.is_key_pressed(KEY_W):
		direction +=

	linear_velocity = movement
	up_direction = Vector3.UP
	move_and_slide()
	movement = linear_velocity
	pass

Steps to reproduce
Attach that script above to any CharacterBody3D node.
Press save.
Engine crashes without any errors.
Minimal reproduction project
GodotScriptCrashPlusEqual.zip

I think this is a duplicate of #49628.

Indeed, that looks quite similar. Closing as duplicate of #49628.


Increment version for cognitivelanguage releases

Increment package version after release of azure-ai-language-conversations

API change check
API changes are not detected in this pull request.


Fix consul permissions

[Not Tested]
https://github.com/CiscoCloud/microservices-infrastructure/issues/1061

Rebasing to trigger a CI build.

Looks like the builds are legitimately failing. Let's wait on this until those errors are fixed.

Closing because this fails with related errors.


feat: delivering partial fields via unencrypted logs

Partial fields get delivered via unencrypted logs and NoteProcessor then looks for them if it fails to find note hash.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8725  👈
#8712
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @benesjan and the rest of your teammates on  Graphite

Why does it need to hav ethe same name? Because of the note type id? If so let's leave a comment.

@nventuro that's correct. Clarified in https://github.com/AztecProtocol/aztec-packages/pull/8725/commits/bfaaaabc4736fe583d4e8ae052be2b7b1cb7c34d

Wil do that in a PR up the stack. Thanks for the reviews 🙏

Merge activity

Oct 4, 3:09 AM EDT: @benesjan started a stack merge that includes this pull request via Graphite.


Create a new web course for an intro to Argo Workflows

Summary
Katacoda is being shuttered, so the Argo Workflows Katacoda course was removed from the docs.
Proposal: Pipe Katacoda content into a new web-based course platform and then update the docs. (quick-start.md, training.md, others)
Current candidates:

https://instruqt.com/
https://killercoda.com/

Use Cases
New users would still like an easy way to a) try out Argo Workflows without setting up a K8s cluster, and b) learn the basics of Argo Workflows.
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Can we use killercoda as a solution
as we can directly migrate katacoda  projects without starting from scratch
the link attached below is the about page of their official website
https://killercoda.com/about
this link below shows an example  of kubernetes-dashboard running from killercoda
https://killercoda.com/kubernetes/scenario/kubernetes-dashboard
_Follow the steps mentioned to run the dashboard_

Im interested in working on this particular issue

Thanks, @mashriram. Yes, I have been testing Killercoda and Instruqt with a couple other contributors, and we plan to have an update posted within 4-6 weeks about a new course for the community to try out and provide feedback.

This is number 3 most popular issue. How is it going?

Going well! We should have a new course live in a couple weeks, around ArgoCon. We'll post an update here and will open a PR to update the docs.
We will need folks to take the new course and provide feedback.

Hi Everyone, @tico24 and I have updated the intro to Argo Workflows course that @alexec initially created last year. Please access it here on Killercoda (a free Katacoda replacement): https://killercoda.com/pipekit/course/argo-workflows/
We made some improvements based on prior feedback and enabled the course for the latest version of Argo Workflows, v3.4.2. Next, we're going to add additional modules to the course based on user feedback!
Please note that Killercoda is in beta as a new platform, so we expect that you may encounter some bugs. That said, we have tested the course and it is functional. It's important to use either Chrome or Safari browser to complete the course. To access the course, you will need to create a Killercoda account.
We're going to close this issue now. We're taking feedback for improving the course on the course repo here:
https://github.com/pipekit/argo-workflows-intro-course/
If you encounter bugs, please open issues on the course repo.
If you have ideas or requests for future course content, please also open issues on the course repo.
If you would like to contribute to the course, please open an issue on the course repo with the title "Contributor request - your name" and describe a) how/what you would like to contribute, and b) your experience level with Argo Workflows.
Also, once the course is stable (and Killercoda is also stable, which is a factor we can't control) we will migrate it to the Argoproj GitHub org.
Thanks for your support in improving the course! We're looking forward to your feedback.


:sparkles: Path inclusion

Bundle PR: 94

Finding a way to test this PR with bundle PR

I am investigating the failing test now, everything is working fine locally, looks like something small.


UWP .appinstaller which depends on Microsoft.UI.Xaml crashes when ran through App Installer

If the .appinstaller file specifies a Dependencies Package for Microsoft.UI.Xaml then when ran via App Installer it will crash. If ran via PowerShell's Add-AppxPackage using the -AppInstallerFile option, it does not crash.
App Installer does not have an issues board so there is nowhere else to report this except here.
I have tried with both 2.4 (which Xamarin Forms 5.0 references) and 2.5 (vlatest nuget). Same problem.

@nbevans I'm curious how is the .appinstaller file generated. Automatically?

Of course - by Visual Studio's UWP publishing process. I have tried via Azure DevOps pipeline and in VS via the UI screens. Same output.

I think I have seen similar issues where the app installer crashes when installing an msix package with a winui dependency if there is no internet connection.

@nbevans Is your app a desktop app? Because when I publish, I get an .msix as output, no .appinstaller whatsoever.
I created the .appinstaller myself by hand

For folks hitting the crash, the team has asked you to file feedback using the feedback hub, under the "Developer Platform > App deployment" area.  So:

Start feedback hub (Win+F)
Create a report of the problem using that area
Click that "record details" button
Reproduce the problem
Send the report

Thanks!

Thank you for the detailed repro!  We've identified the issue and are working on a fix to the Desktop App Installer app which processes the .appinstaller file.  I'll post an update once it's available.

Thank you for the detailed repro! We've identified the issue and are working on a fix to the Desktop App Installer app which processes the .appinstaller file. I'll post an update once it's available.

Thanks - there is also other critical bugs in App Installer which I reported some time ago:
https://github.com/MicrosoftDocs/msix-docs/issues/191

The main desktop app installer app should now be updated with this fix, go to the Microsoft Store and click "Get Updates" to grab it.  If you're opted into the flight with WinGet, that one will be updated in a few days.

Okay thanks. How long roughly until Store has rolled that update out to the majority of machines? (Or is that a silly question?)

It installed this - a version from 2020 - are you sure it has been published?

What OS are you on?  The app might still be flighting to ensure quality

Is there any news on this? Thanks!

Another new release went out?

Praise the lord - this version has the fix in it!
ms-appinstaller:?source=http://byteflagshared.blob.core.windows.net/files/panconnect.mobile.uwp.broken.appinstaller
This works now - the App Installer does not crash.
Does it contain any other fixes in this release? Could you publish a change log?

Can Microsoft remove the requirement that users "sign in" to the Windows Store in order for critical pieces of Windows infrastructure to be automatically updated? This is major impediment to rolling out App Installer bug fixes.

Sign in isn't actually required, you can close that window. It's misleading I know.

@riverar Wow, that is beyond sad. The point is, while you and I may know (and I know now that you told me), basically I lose one possible customer.
I'm thinking of adding my app to the new store when that will happen, but if I'll run into the same crap as with this one, I will give up on it forever. I'm so tired of being the unpaid alpha, beta, and gamma tester for Microsoft.

@riverar
Yeah I did try everything, including closing it. But it still didn't update to the latest App Installer. In the end I signed in using my personal "Microsoft Account" (on somebody else's machine!) and only then would it install the latest update of App Installer. I then signed out of my account on their machine.
Why does Windows Store allow apps like App Installer to get so out of date in the first place? It seems to install updates for other things fine and relatively promptly - but App Installer seems to get ignored.

Seeing this on more and more machines where App Installer won't auto-upgrade nor manual-update until you are signed in with a "Microsoft Account" in the Microsoft Store.

@nbevans shows you how much Microsoft truly cares about us

@nbevans shows you how much Microsoft truly cares about us

Please let's remain constructive as it is the only way these issues will get solved.
My feeling is that App Installer has mistakenly not been marked/signed by Microsoft as "Windows infrastructure" and so it is being treated like any other "optional" third-party Store app.
I say this because I have noticed that many other Windows components (like say "Sticky Notes", "Cortana", "Your Phone", "Xbox Game Bar" etc etc) do receive automatic updates regardless of whether the Microsoft Store app is "signed in", signed out, signed in with a non "Microsoft Account", etc.

As Rafael mentioned sign in should not be required.  I looked at this on a clean VM without an MSA attached this morning.  The device updated correctly to the retail version of the App Installer app which is version 1.4.3161.0.  What version are you trying to update to?  If I had to suspect you have used your MSA to sign up for the Windows Package Manager Insider Program.  Those users will get an update to an in-development version of the app based on the MSA signed up for the program.

@jvintzel
I was just about to post a similar conclusion. I had signed up my Microsoft Account with the Windows Package Manager Insider Program and completely forgot about it. I stumbled upon the GitHub project for that - which indeed seems to comprise active work on App Installer. In the latest release is a msixbundle for App Installer (https://github.com/microsoft/winget-cli/releases/download/v1.0.11692/Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle) which is providing a viable workaround now to get this onto end-user machines. First they install this App Installer msixbundle and then they can install our app afterward.
Is it safe to be pushing out these App Installer msixbundle's to our end-users? I assume so?


documentation

The documentation for arithemtic operators overload is missing. Also, some pages of the API Reference section seems to be merged into a single page while they should not.

It looks to me like the functions inside the batch class template aren't documented. The class itself also defines the load family of functions:

but the documentation doesn't mention that anywhere.
These functions have a different signature than the ones which are already documented.

Also, some pages of the API Reference section seems to be merged into a single page while they should not.

Can you elaborate on that point?

THe following pages are merged into a single one (meaning you can scroll down across sections while they should be on dedicated pages):

Arithmetic operators
Comparison operators
bitwise operators
Mathematical functions
Reducers
Miscellaneous

Thanks! #769 is now ready for review, if @JohanMabille and @wermos can have a look?


Shields unable to receive Basic Melee bonuses when attacking.

I was recently able to update all the shields that allow attacks as unarmed attacks to do basic damage. However, since they do attacks as unarmed attacks. Shield items will need, in some means, the ability to receive bonuses from Basic Melee attacks/damage modifiers. Such as from Specialization.

I imagine the implementation of this would similarly resolve https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/issues/661


Worked on names and features

solves #23
I've added names and distinguished features for 5 characters.
Screenshot:
This is screenshot of just 1 of the character.

This will the viewer a better understanding of the characters.
Thanks & Regards
Kindly review

@xintamosaik you'll have to merge this PR from your side


"SKIP" button in all the Fragments

Hey,
Does this library has feature to keep the "SKIP" button always active for all the fragments.
Thanks,
Vinod

Hi @vinod-jaiswal18!
Just use showSkipButton(true);. Please note that skip button is not available with second layout (AppIntro2).

Hey @PaoloRotolo ,
Ohh, I just missed it. The function 'showSkipButton()' is available.
Thanks,
Vinod


Issue referenced in Body requires space after issue number.

There must be a space after the issue number (ie "#12 " not "#12".) This is due to the way the RegEx is structured and will be resolved in a future release.
The RegEx needs to be updated so that it works even if there is no space after the issue number.
"This resolves #12 . thanks!" - this works
"This resolves #12 " - this works
"This resolves #12" - does not work.

to verify that the it is all numbers the following regex could be used /#[1-9][0-9]*/g


update to ember-cli add-on structure

update to be an ember-cli add-on.

:dancers:

We can likely now get travis running on this.


Proposal: storybook container changes

Storybook docs generation changes.
I propose to add the ability to put classes on the component container by passing containerClassNames or defaultContainerClassName to the Storybook parameters.
This allows components to remove their wrapping containers from the story (if they have them) thereby removing the clutter and use-case specific code from the code example.
For example, current example code of current components with containers:

  <Menu {...args}>
    <Menu.Item>
      Item 1
    </Menu.Item>
    <Menu.Item>
      Item 2
    </Menu.Item>
  </Menu>

Will now be cleaner and not show any use-case specific classes for the storybook example:
<Menu {...args}>
  <Menu.Item>
    Item 1
  </Menu.Item>
  <Menu.Item>
    Item 2
  </Menu.Item>
</Menu>

This:

Improves the quality of our code examples by removing clutter and use-case specific storybook containers
Removes the current used containers to have all components use the same default container (like daisyui examples also do)
Make future changes to the component preview easier since its all managed by the same container

Let me know what you think, all the naming and approach can be adjusted but I think the idea brings value.
ps. All classes removed and not replaced in this PR are redundant because they were already managed by the story component container

@Klaas058 sorry for the late reply, have had a busy couple weeks!
I 100% agree that the current classnames are too much and should reflect the examples from daisyui.com. Part of that, like you said, would mean beefing up the story-layout container and setting the default font etc.
So seems like a mix between the two-- removing redundant classnames but still showing the styles necessary for each story!

Hi @benjitrosch don't worry about it. I think I see what you're getting at.
I will update the PR to reflect this philosophy

Hi @benjitrosch, I went through the PR again trying to implement what we talked about, but it came down to just making the font-sans default.
This resulted in quite a lot of changes in stories of components that put on font-sans themselves, now all components have the same font as DaisyUI has in their docs. (for the diff see here)
Though this doesn't make the stories themselves any better.
Looking at it now I think this is because of the {...args} splitting.
This is the DaisyUI JSX:
<input type="text" placeholder="Type here" className="input w-full max-w-xs" />

This is ours:
<Input {...args} />

The className is not visible in the preview because it is 'hidden' in the {...args}, the user can see the props, but has to look them up in the props section.
I think something like this is much more informative on first glance:
<Input className='w-full max-w-xs' {...args} />

This way you have the classes both in the preview and the controls.
But it has the drawback that if you change the className in the controls, the preview doesn't update it.
It will work because it overrides the default className but it is kind of 'ugly'.
I wonder if there is a way to compute the jsx preview on runtime instead of build time.
Maybe there is a Storybook addon for that?
What do you think?


OpeNLogin BUG.

I have a nick called XavisionUPD, if another nick called XavisionUPD23 gets in when I try to log in with my account XavisionUPD it tells me that it is connected.

Fixed in 30e3fd25f999d9298f5e91935200e87ca02ee3e6
Thank you very much for the bug report.


fix(@angular-devkit/build-angular): retain CSS special comments in global styles

Prior to this change special CSS comments /*! comment */ were being removed during minification when using the esbuild based builders. This caused tools that ran post build that rely on such comments such as purgeCSS and critters not to function properly.
This will align the behaviour between the esbuild and webpack based builders.
Closes: #26432

Thanks @alan-agius4, as always, providing an outstanding turnaround !

Hi guys, just for our information, is this planned to be merged at some point soon? No rush at all, it is because this is holding us back from release the upgrade to Angular 17. Thanks!

Had a chat with @clydin earlier last week and we decided to go with an additional option. The PR has been updated to add this option.
This should be available as part of 17.1.0.


Make  { a: a } a violation

Require { a }

This can be done using the object-shorthand rule with the 'properties' setting. The same rule can be used to force this:
const foo = {
    a: function() {},
    b: function() {}
};

to be written as:
const foo = {
    a() {},
    b() {}
};

Interested in this, or just the properties?

Just the properties. The function shorthand is less readable.

As of ESLint 3.2.2, it doesn't look like it's possible to enforce the properties and prohibit functions at the same time (if anyone know of a way, a PR is welcome).

No need to prohibit the function, just don't force them.


build: print dockerfile that is being built

Open question: does this level of detail belong at the top level? Or should this live at the debug level?


Move make_array to datafusion-functions

Is your feature request related to a problem or challenge?
Part of https://github.com/apache/arrow-datafusion/issues/9285 and https://github.com/apache/arrow-datafusion/issues/8045
The goal is to extract function definitions out of the datafusion core.
Note the make_array function is somewhat special in that it has several special cases in datafusion core at the moment.
Not only do the special cases make it hard to move, it also means some datafusion functionality is not available to scalar udf functions
I think this ticket will be blocked on  removing the special case handling of MakeArrayin the expression rewriter(link) -- I will file a follow on ticket
Describe the solution you'd like
Move the make_array function to datafusion-functions-array
Describe alternatives you've considered
No response
Additional context
Note I have a draft PR with some of this code: https://github.com/apache/arrow-datafusion/pull/9222 but likely won't have time to work on it for a while

I think https://github.com/apache/arrow-datafusion/issues/9289 may be able to help us with the simplify example

@jayzhan211  I wonder if you have thoughts or time to work on this ticket?

not sure how to do yet, but I will take it.
/take


What would my domain be ?, or how could I make this work from my cpanel

What would my domain be ?, or how could I make this work from my cpanel

You're domain could be anything you want. Basically you're current server's domain. But any subdomain may be used.
This server isn't currently used with cpanel but with a ssh connection.

Port 443 in use by "Unable to open process" with PID 4!

So I have to connect the server with ssh to cpanel

This project is just a module for a Node application using the Express framework. It is intended to be used as a library in a main node application and isn't a standalone server.

Well You dont have any implementation?, I'm trying implement that but, don't work.

The documentation is very bad


Missing User Right - Obtain an impersonation token for another user in the same session

User Right Obtain an impersonation token for another user in the same session, which is available in Windows 10, Windows Server 2016/2019 is not documented.
Please document and also, if possible, please provide a table of user rights by OS version (2012-2019), including default OOB user rights assignments.
It would be good to understand how user rights and user rights assignment evolve from version to version.

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

ID: 7de1fb45-91eb-f37f-8106-81bf82a5602e
Version Independent ID: 1c765989-4c50-4b89-83fe-a8625f0f9fd5
Content: User Rights Assignment (Windows 10) - Windows security
Content Source: windows/security/threat-protection/security-policy-settings/user-rights-assignment.md
Product: w10
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Hello @ixtlanian - Thank you for your query. I have reviewed the document and found that the document has the necessary relevant information. For your specific query, please see https://social.technet.microsoft.com/Forums/windows/en-US/999dedb8-c6fa-4f2f-a5ca-e48f43bc956e/user-rights-assignment-obtain-an-impersonation-token-for-another-user-in-the-same-session?forum=ws2016
Also, I would like to suggest you open a request on a user voice channel for a separate document required as you mentioned.
@e0i  will guide you on how to open a user voice request.
Thanks.
Imran.

Imran, you did not provide any explanation on the missing user right. THe document does not contain the user right I mentioned. There cannot possibly be a separate document for this as it's clearly a mistake in this document. Please assign this issue to the owner of the document.

(ref. #8902 )

(ref. #8902 )

I would like to check that this was closed correctly.
Obtain an impersonation token for another user in the same session was indeed added to the list of User Rights.
However, the hyperlink on that item leads to the article on Impersonate a client after authentication, which is a different User Right.

This is still very much an issue and should not be closed at all.
There's a page that lists all the User Rights Assignment elements and their meanings here:
https://docs.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/user-rights-assignment
Within that page, there is an entry for:
Obtain an impersonation token for another user in the same session
The link for that entry is wrong. it is linking to:
https://docs.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/impersonate-a-client-after-authentication
..Which is wrong - This is a completely different User Right Assignment (fir impersonate a client after authentication).
The page for the right the OP has listed is missing.

This is still an issue and should not have been closed.  See comment above from Damien-Sugden for a detailed explanation of what is still incorrect in the docs.


Android image segmentation

Image segmentation for adroid lite

@googlebot I signed it!

@googlebot I fixed it.

Hi @gungoren, looks very interesting! Let me sync with some other engineers before we move forward with this, I believe we had some plans to open-source an Android port of the our existing iOS segmentation example, but I'm not sure of the status. Stay tuned!

Hi @gungoren , thanks for the pull request!
We've tried out your code and it works great, except that there seems to be a bug in cropping logic. Please see the screenshot below. We'd encourage you to fix a bug and host it in your repo, and we'd love to feature it in our upcoming gallery of community samples.

In term of accepting the pull request, it's a bit tricky because we need to make our samples build with Bazel for internal maintenance so we have to apply some restriction in the type of libraries/code can be used in the app. Unfortunately we'll have to reject the pull request at this moment, but we truly appreciate your contribution and we'll ping you here once the community sample gallery is ready so that you can add your sample there.


Bug: Can’t go back to file start with preview-page-up

RTFM Checklist

[X] I have searched exisiting issues / discussions
[X] I have read the Wiki including the Advanced section
[X] I have read man fzf / I am well versed in shell fzf

Operating system
Linux
Shell
bash
Neovim version (nvim --version)
NVIM v0.11.0-dev-93+g1c73bd7d2
Fzf version (fzf --version)
0.29
Output of :lua print(os.getenv('FZF_DEFAULT_OPTS'))
nil
Is the problem reproducible with mini.sh?

[X] My issue is reproducible with mini.sh
[ ] My issue IS NOT reproducible with mini.sh
[ ] I have not tested with mini.sh (not relevant, requires LSP, Windows, etc)

Fzf-lua configuration
require('fzf-lua').setup({
})

Describe the bug / steps to reproduce
I can’t go up to file start again after scrolling down a bit.

find any file with scrollable amount of lines with :FzfLua files
scroll down with preview-page-down
try scroll up to file start with preview-page-up

Does it scroll up a bit and just doesn’t go to the file start or the shift-up keybind doesn’t work at all?

It does go up a bit and stuck at some point. shift-up keybind is working.
Seems like it can’t go up more if there are less than half page lines left.

It does go up a bit and stuck at some point. shift-up keybind is working. Seems like it can’t go up more if there are less than half page lines left.

The preview scroll is performed by sending ctrl-f/ctrl-b for page and ctrl-d/u for half page, it’s probably related to the editor’s behavior in such cases, I’ll check what can be done (maybe scrolloff).

@boltlessengineer, in the past the code used only ctrl-u|d to scroll by half pages, recently a PR (#1150) by @toupeira changed this the default to full pages up/down ctrl-b|f, turns out ctrl-b is not like ctrl-u, try to ctrl-b at the first page of any buffer, it goes to the bottom of the page and never goes up to line 1.
This is now handled in https://github.com/ibhagwan/fzf-lua/commit/b92220ec838c195eb1c711daa69c905b1d7b8d8c and should work as expected, ty for the report!


Set proper location status indicator in settings page

Description
location status was showing incorrect in settings page, if user makes location permission off from app setting , which is fixed now.

https://github.com/tripleblindmarket/covid-safe-paths/issues/603

Thank you for your response. I have made changes in code with explainable comments. Please check it.


Rest Assured - can't POST with Parameters and Body

From<EMAIL_ADDRESS>on August 28, 2012 00:17:42
What steps will reproduce the problem? 1. Post to a URL containing a parameter with body content What is the expected output? What do you see instead? This is expected to work, testing manually works. Instead the following error is thrown:
"You can either send parameters OR body content in the POST, not both!" What version of the product are you using? On what operating system? Using Im using Rest Assured 1.1.6, which is rather old. However, looking at the code on github this still appears to be an issue Please provide any additional information below. Stack overflow post: http://stackoverflow.com/questions/12101297/rest-assured-cant-post-with-parameters-and-body
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=196

From<EMAIL_ADDRESS>on August 27, 2012 23:31:20
This has been fixed a long time ago, please update REST Assured to the latest version. As a work-around maybe you can use "given().queryParam(..)..". See answer at stackoverflow.
Status: Invalid


Use voluptuous for mysensors

Description:

Add voluptuous config validation for mysensors
Remove and clean up parts that are not needed for pymysensors 0.7.

Question:
@fabaff Do we need to add something to mysensors platforms that are not configured individually? All configuration for mysensors is done through the component.
Related issue (if applicable):
https://github.com/home-assistant/home-assistant/issues/2800
Example entry for configuration.yaml (if applicable):
mysensors:
  gateways:
    - device: '/dev/ttyUSB0'
      persistence_file: 'path/mysensors.json'
      baud_rate: 38400
    - device: '/dev/ttyACM0'
      persistence_file: 'path/mysensors2.json'
      baud_rate: 115200
    - device: '<IP_ADDRESS>'
      persistence_file: 'path/mysensors3.json'
      tcp_port: 5003
    - device: mqtt
      persistence_file: 'path/mysensors4.json'
      topic_in_prefix: 'mygateway1-out'
      topic_out_prefix: 'mygateway1-in'
  debug: true
  optimistic: false
  persistence: true
  retain: true
  version: 2.0

Checklist:
If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Do we need to add something to mysensors platforms that are not configured individually? All configuration for mysensors is done through the component.

I don't think so. PREVENT_EXTRA is the default which should help us to catch unwanted configuration entries.

Hmm, when I test to config a mysensors platform under the sensor component, I don't get any indication in the log. I can even pass random key-value pairs together with the config for the platform, without any logged errors. This might not be a big deal, in cases where discovery is used, as the platform won't setup unless setup with discovery.

T-Minus 10 hours until we release 0.27. It would be great if this could make it in time, but no rush if it can't, please don't stress :)

@robbiet480 Yes, I'll have a look at solving @teagan42 's question, but everything should be working as it is now, so should be able to get this in before time T.

I suggest we merge this as is, and I can look if it's possible to move the persistence file name creation to the config validation, later. I'll leave the decision up to y'all. I'll be at a birthday party the rest of the day, so probably AFK.

Issue created for persistence file @MartinHjelmare
https://github.com/home-assistant/home-assistant/issues/3009


Add support of link labels

Hello!
First of all, many thanks for all the effort spent on creating this awesome project!
I had to introduce the support of link labels and I felt like it would benefit the community to have this feature as well.
Right now, each labels are created using HTML defined by the app, but we could think of a way to parametrize this part if you think the effort is worth it.
Here is how you could use it

const editor = new Drawflow(el)

// Define some labels in HTML
const label1 = 'Label A'
const label2 = 'Label B'
const labels = [label1, label2]

// Labels will be automatically added if they exist
editor.addConnection(id_output, id_input, output_class, input_class, labels)

And here's what it looks like

Let me know if you see issues or if you'd like me to change something.
Cheers!

Hello! @mokaddem
Thanks for PR.
If labels is one thing missing.
There are alternatives to add the labels:

https://github.com/jerosoler/Drawflow/issues/18


Event started message should include title

Bug report: Very minor change that should be incorporated to include the event title in the 'event has started' message to be 100 percent explicit
Submitted by: Cash#8915
Server: mfDiscord

loggers already have the work done just need to include {after.name} in the ctx.send

Adding on to this ticket, we also should set permissions to allow users to create events

Resolved by adding after.name to the message and resolved the second issue by simple resetting channel permissions and then revoking the default role's access to the channel.

Re-opening issue, we should change behavior to allow everyone to connect to Shows until an event starts. Once an event starts channel should be locked to interested users in the event. After the event has concluded we can reset permissions. This will be so anyone can create an event and lock the Shows channel for their use.


Place the website content into a container div

Make it so it takes up the center of the page and has nice margins on either side.
Try to keep it looking decent on mobile too, but that's secondary and we can create a new issue to deal with that if necessary.

I can get this done asap...
Hope i can hop in

Sounds good, take all the time you need 😄


C++17 Mathematical Special Functions

These likely can be adapted from the ones in Boost.Math but we should include them for completeness:

[x] #404
[x] #445
[x] #648
[x] #463
[ ] comp_ellint_2
[ ] comp_ellint_3
[ ] #600
[ ] #601
[ ] #602
[ ] #603
[x] #462
[ ] ellint_2
[ ] ellint_3
[ ] #650
[x] #400
[x] #444
[x] #402
[x] #547
[ ] #604
[ ] sph_legendre
[ ] sph_neumann

Ok ellint_1 and 2 are preliminarily done


Enh - Show Features Enabled

For current branch, show features installed and whether they're preview or release.

No interest / disregard


improper error trapping in sizeReturn

Issue #1354 revealed the following as a problem with error trapping:
> nestedNim2 <- nimbleFunction(
  run = function(){
    returnType(double(0))
    return(nimGarbage())
  }
)
  [Note] For this nimbleFunction to compile, 'nimGarbage' must be defined as a nimbleFunction, nimbleFunctionList, or nimbleList before compilation.
> compileNimble(nestedNim2)
Compiling
  [Note] This may take a minute.
  [Note] Use 'showCompilerOutput = TRUE' to see C++ compilation details.
Error in if (typeEnv$return$type == "nimbleList" || code$args[[1]]$type ==  :
  missing value where TRUE/FALSE needed

In contrast this is error-trapped, so the issue is with processing return().
> nestedNim2 <- nimbleFunction(
  run = function(){
    returnType(double(0))
    out = nimGarbage()
    return(out)
  }
)

compileNimble(nestedNim2)
  [Note] For this nimbleFunction to compile, 'nimGarbage' must be defined as a nimbleFunction, nimbleFunctionList, or nimbleList before compilation.
Compiling
  [Note] This may take a minute.
  [Note] Use 'showCompilerOutput = TRUE' to see C++ compilation details.
Error: In sizeAssignAfterRecursing: 'nimGarbage' is not available or its output type is unknown.
 This occurred for: out = nimGarbage()
 This was part of the call:  {
  out = nimGarbage()
  return(out)
}

Note that #1356 also relates to a difference in size processing when inside return versus not.

I'm adding some error trapping in branch fix_1364.


Update redirects in Readme

Created with https://github.com/dkhamsing/frankenstein

Well how about that... :tophat:-tip

Nice, thanks!


refs #16 remove zeitwerk dependency

refs #16

Codecov Report

Merging #17 (070df24) into master (9c862f9) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           26        27    +1
  Lines         1470      1508   +38
=========================================
+ Hits          1470      1508   +38

Impacted Files
Coverage Δ

lib/calendly.rb
100.00% <100.00%> (ø)

lib/calendly/api_error.rb
100.00% <100.00%> (ø)

lib/calendly/client.rb
100.00% <100.00%> (ø)

lib/calendly/error.rb
100.00% <100.00%> (ø)

lib/calendly/loggable.rb
100.00% <100.00%> (ø)

lib/calendly/models/event.rb
100.00% <100.00%> (ø)

lib/calendly/models/event_type.rb
100.00% <100.00%> (ø)

lib/calendly/models/invitee.rb
100.00% <100.00%> (ø)

lib/calendly/models/invitee_question_and_answer.rb
100.00% <100.00%> (ø)

lib/calendly/models/invitee_tracking.rb
100.00% <100.00%> (ø)

... and 8 more

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 9c862f9...070df24. Read the comment docs.


Mobile Content Overflow

The items inside the card overlay is moving out in mobile due to overflowing text content

Some suggestions:

limit character
expand the card overlay depending on the size (this is a little bit hard to do because of the fixing of picture size)

Fixed with min height


Seems to break the rendering of code fences in the normal VSCode Previewer

I have an md file with code fences like these
```
some text
with newlines
```
and when this extension is enabled the code is rendered without the newlines and is not formatted as a code block in the previewer.
That is a showstopper for me unfortunately.
When I disable the extension the same text renders fine in the Preview.

And it breaks the mermaid preview bierner.markdown-mermaid.

Try disabling other extensions that might interfere.  I had Markdown Preview Enhanced installed and this stopped it working.


camera.snapshot is using the image_url (instead of stream_url) to make images.

Home Assistant release with the issue:

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

Description of problem:
camera.snapshot is using the image_url (instead of stream_url) to make images.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

NA

**Traceback (if applicable):**

NA
Additional information:

camera.snapshot takes a picture.  camera.record takes the video.  This is as intended.

camera.snapshot takes a picture. camera.record takes the video. This is as intended.

I understand camera.snapshot takes a picture, but of what? earlier it used to take a picture from the camera (from live feed). But now it is taking a picture of what ever is showing on the frontend (which is image_url). hope i made it clear

For generic camera, it has always used image url. If you want it from the feed, you can use ffmpeg camera, which supports stream in 0.91


[DO NOT MERGE] Make Badger link to all deployment dashboards

Makes the dashboard links for all dashboards work
Mobbed with @suzannehamilton @brenetic
https://trello.com/c/uWEg7LLm/24-add-link-to-all-dashboards-in-release-app-and-badger

Marked as DO NOT MERGE while we figure out what Grafana hostname to use - it turns out that we can't access grafana.publishing.service.gov.uk from the release app and the same is probably true of govuk-app-deployment.


Restart console-watching processes on HaaS API server reboot

If the HaaS API server reboots, we face a couple of issues with the IPMI console functionality:

We lose the contents of the log since they're kept in a temp file
The process that was recording the serial console does not get restarted.

Persisting console state to the DB will help this.
Potentially related to #416 and #417

See my comments on #417


docs(tutorial): Section about advanced theming

Subject
Tutorial
Description
Add a new section to the tutorial about the Advanced Theming (component theming) of Chakra UI.

👋 I'd like this one, please!


Distribution, New.  Add Another Item should be activated when storage location chosen, after error.

Summary
On Distribution New, if you attempt to save without picking the storage location,  and then pick a storage location, the add another item button should be enabled.
Details
Here is a failing sequence, on local, signed in as<EMAIL_ADDRESS>Steps
Navigate to new distribution form: http://localhost:3000/distributions/new
Leave all fields blank and click Save -> view will render with validation errors (eg: Storage location must exist, etc)
Fill out the form with valid values for Partner and Storage Location
Fill in the first item and quantity
Click "+ Add Another Item"
Expected: Can keep on adding more items
Actual: "+ Add Another Item" button is disabled and can't add any more
this is probably happening on donation and purchase too,  but we haven't confirmed that.
Criteria for completion

[ ] Add another item button gets enabled when storage location is chosen,  even after initial errors.
[ ] Tests to demonstrate this

Alright.  You've got it.


Bug: Nonprod role name changed

Bug Description

Seems like the role name changed that I am trying to assume sometime in the last week or two.
It was: nonprod-geospatial-data-lake-use-usersrole18D0AA3B-1GQ53EX8UMJC7
Now it is: nonprod-geospatial-data-lake-use-usersrole18D0AA3B-L1BSHG3ZPQJL
I would assume they would be constant. Is the a suffix changed during deployment somehow?
How to Reproduce**

Software and OS versions

OS:
Environment:
Relevant Software Versions: <!-- e.g. aws --version -->

Additional context

Role name is fixed now. This wouldn't happen anymore.


Hitrate based caching based on job-wise provision of all input files by a storage element

In the current implementation either all or no input files of a job are provided by a storageelement:
https://github.com/tfesenbecker/lapis/blob/0a79525ad0c3418893232fb084d9437b625696a5/lapis/connection.py#L189
This boolean treatment introduces a strong bias, which might be negligible as long as caching and cache-eviction policies are properly configured. Therefore, the more general approach would be to sample a boolean for each requested file separately, instead of one per job.

Isn't this part of the concept of the hitrate based caching scenario? I don't know exactly, but if you do, this would be a simple fix.
Could you look up the details in the thesis of Tabea?

So I checked and indeed, it seems that this is an assumption @tfesenbecker makes in her hitrate-based caching. But I think this might break as soon as one wants to study cache logics which cache blocks of files instead of the whole file.

So then please open another set of tickets for requirements of block-based caching. Closing this as the current approach is valid within the given assumptions.

Conceptually, this might also be a problem, when several tasks read different files to the cache. It is unclear, then, if in this case the cache still provides the full set of input files or none, when only a fraction of the required input files was loaded by previous jobs accessing the cache.

Sorry, I'm not following what exactly the goal is here. Yes, the current hit-rate-based-caching makes per-job decisions based on a global hit-rate. That's for Tabea's approach to model caches as black boxes, so it is per design.
That's purposely a simplification so that one does not have to worry about details like cache occupancy, cache-eviction policies and such. It's decidedly not suitable for any case where such a simplification is not desirable.
Is this ticket just to document that this is the case? Is it about extending this model? Is it about adding an alternative model?

Sorry, I'm not following what exactly the goal is here. Yes, the current hit-rate-based-caching makes per-job decisions based on a global hit-rate. That's for Tabea's approach to model caches as black boxes, so it is per design.
That's purposely a simplification so that one does not have to worry about details like cache occupancy, cache-eviction policies and such. It's decidedly not suitable for any case where such a simplification is not desirable.
Is this ticket just to document that this is the case? Is it about extending this model? Is it about adding an alternative model?

Yes, it's on one hand to document this simplification, on the other hand to extend this model, since to my understanding this extension is what hit-rate-based caching should actually be like. This is also what @tfesenbecker mentions in her thesis as a simplification. She also proposes to extend this for future studies.


Gameboy APU, 4th channel has only one waveform, with 32 samples

The text for the gameboy APU suggests that it can store 32 seperate waveforms for the wavechannel. In reality it can only store one waveform in the APU, which has 32 samples. https://gbdev.gg8.se/wiki/articles/Gameboy_sound_hardware
'fourth' in the next sentence is also incorrectly spelled 'forth'

Corrected, thanks


Update Git type for ProcessEnvironmentBlock

Previous implementation did not account for case insensitivity on Windows and made it hard to remove deprecation warnings in clients of TSC.
Also removed members that were deprecated for 3 years now, as we can be fairly certain that clients have migrated already.

@swift-ci test


Logitech G604 Lightspeed

Hello libratbag Team,
thanks a lot for the great work. Is there already any approaches for the new Logitech G604 Lightspeed? I own this mice and would like to use it in full on linux. So is there a way i can help you out?
Kind Regards

I'll see if I can do it today.

Okay, I tested it and it works fine with the current codebase. The only issue is that you won't be able to configure it when connected over bluetooth until #829 gets merged.


How to move widgets to left side

I am sorry it does not support this function.

Alright, What a pity

I found this site which has done what you wanted: http://0opslab.com/

Thank you very much @tufu9441


On ETH Main network getting lots RPC errors

@radamosch how to reproduce it?

Connect to mainnet ETH
Visit a dApp, like https://exoplanets.io or DopeRaider.com
(inspect / console) --> see this error idk what's causing


python API faults on float16 example where float32 works

The following example code works as expected when running a float32 model
import migraphx
import numpy as np

# This example using float32 works
model32 = migraphx.parse_onnx('resnet50i1.onnx')
model32.compile(migraphx.get_target('gpu'))
params32 = {}
for key,value in model32.get_parameter_shapes().items():
    params32[key] = migraphx.allocate_gpu(value)
image32 = np.zeros((1,3,224,224),dtype=np.float32)

params32['0'] = migraphx.to_gpu(migraphx.argument(image32))

result32 = migraphx.from_gpu(model32.run(params32))

Change instances of 32 to 16 and it now faults
import migraphx
import numpy as np
# The same example using float16 crashes
model16 = migraphx.parse_onnx('resnet50i1.onnx')
model16.compile(migraphx.get_target('gpu'))
params16 = {}
for key,value in model16.get_parameter_shapes().items():
    params16[key] = migraphx.allocate_gpu(value)
image16 = np.zeros((1,3,224,224),dtype=np.float16)

params16['0'] = migraphx.to_gpu(migraphx.argument(image16))

result16 = migraphx.from_gpu(model16.run(params16))

Here is the message that comes out
Traceback (most recent call last):
  File "migraphx-fp16.py", line 27, in <module>
    result16 = migraphx.from_gpu(model16.run(params16))
RuntimeError: /home/mev/source/AMDMIGraphX/src/program.cpp:360: Incorrect shape {float_type, {1, 3, 224, 224}, {0, 0, 0, 0}} for parameter: 0

It seems that np.zeros((1,3,224,224),dtype=np.float16) creates a buffer of floats and not float16. Even so, we dont handle float16 currently in our python bindings as python 2 doesnt have a character format to describe it. Python 3 does have a character format, which we should probably add support for.

Defer this to general issue of supporting python 3.* packages

@mvermeulen This issue might have been resolved with the fp16 support in python. If you still have the model, maybe I can try them to confirm the fix.

Just run a resnet50 model on both fp32 and fp16, no error was reported. so close it.


map.message.complete "payload properties" title misleading

select ToC pane click "Channels & Payloads => map.message => map.message.complete"
The map.message.complete message structure is provided and then the properties of this message are described.  However the header "Payload Properties" makes it appear as the map.message.complete.failures.failureObject.payload property structure is being described rather than the map.message.complete message itself.  Recommend changing "Payload Properties" to simply "Properties" or "Channel message properties" or similar.

https://groups.google.com/forum/?hl=en#!topic/common-map-widget-api/rvq5Wyny9I4


Investigate refactoring use of DataDecodedParamHelper

Description
When mapping ERC-20/721 transfers, we extract the values of the sender, recipient, etc. from the decodedData of a given transaction.
Each transaction that transfers tokens also has a transfers array that seemingly contains all the information of the above and using that would simply a lot of the above.
Requirements
As the mapping is only of ERC-20/721 token, the transaction types are that of MultisigTransaction | ModuleTransaction:

Investigate if both transaction types always return transfers array.
If so, add transfers array to validation schemas.
Replace DataDecodedParamHelper['getFromParam' | 'getToParam' | 'getValueParam'] with transfers value(s).
Update associated tests.

Additional information
This transfers array does not exist in the MultisigTransaction | ModuleTransaction types despite being returned by the Transaction Service. (As the above was implemented early in the project, it may not have existed then.)

This has relation to https://github.com/safe-global/safe-client-gateway/issues/1613.


Could not set 'present' on ensure: No such file or directory

The GPGTools dmg cannot be found. Error as follows:
> ./script/boxen
Boxen has a dirty tree, won't auto-update!
Notice: Compiled catalog for myhostname in environment production in 3.09 seconds
Notice: /Stage[main]/Boxen::Security/Boxen::Osx_defaults[require password at screensaver]/Exec[osx_defaults write  com.apple.screensaver:askForPassword=>true]/returns: executed successfully
Error: Could not set 'present' on ensure: No such file or directory - https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg at 7:/opt/boxen/repo/shared/gpgtools/manifests/init.pp
Error: Could not set 'present' on ensure: No such file or directory - https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg at 7:/opt/boxen/repo/shared/gpgtools/manifests/init.pp
Wrapped exception:
No such file or directory - https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg
Error: /Stage[main]/Gpgtools/Package[GPGTools]/ensure: change from absent to present failed: Could not set 'present' on ensure: No such file or directory - https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg at 7:/opt/boxen/repo/shared/gpgtools/manifests/init.pp

This is strange because pasting the URL into a browser does find the package. The package resource uses "source":
source
Where to find the actual package. This must be a local file (or on a network file system) or a URL that your specific packaging type understands; Puppet will not retrieve files for you, although you can manage packages as file resources.
The pkgdmg provider requires source with a URL that works. And "curl" exists and can get the URL. Not sure what's going wrong here.
--debug shows:
Debug: Executing '/usr/bin/curl -o /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/d20141102-94809-187pzsu/GPGTools.pkg.dmg -C - -k -L -s --fail --url https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg'
Debug: curl -o /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/d20141102-94809-187pzsu/GPGTools.pkg.dmg -C - -k -L -s --fail --url https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg did not transfer [GPGTools.pkg].  Falling back to local file.
Error: Could not set 'present' on ensure: No such file or directory - https://releases.gpgtools.org/GPG%20Suite%20-%202013.10.22.dmg at 7:/opt/boxen/repo/shared/gpgtools/manifests/init.pp

I get the same problem. I note that /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T is owned by root and only writable by root.

I've done some more investigation on this. I believe the issue is that the curl command is failing. If I run it on the command line with the -v added, I get this:
curl -o /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/d20150428-1716-1v6pi67/GPGTools.pkg.dmg -v -C - -k -L -s --fail --url https://releases.gpgtools.org/GPG_Suite-2015.03-b6.dmg
* Hostname was NOT found in DNS cache
*   Trying <IP_ADDRESS>...
* Connected to releases.gpgtools.org (<IP_ADDRESS>) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384
* Server certificate: scnr.ch (S1GdEWuSaU7sKIf2)
* Server certificate: StartCom Class 2 Primary Intermediate Server CA
* Server certificate: StartCom Certification Authority
> GET /GPG_Suite-2015.03-b6.dmg HTTP/1.1
> User-Agent: curl/7.37.1
> Host: releases.gpgtools.org
> Accept: */*
>
* SSLRead() return error -9841
* Closing connection 0

openssl s_client then reveals problems with the certificate:
openssl s_client -connect releases.gpgtools.org:443 -showcerts
CONNECTED(00000003)
depth=1 /C=IL/O=StartCom Ltd./OU=Secure Digital Certificate Signing/CN=StartCom Class 2 Primary Intermediate Server CA
verify error:num=20:unable to get local issuer certificate
verify return:0

So ways we could fix this...

Get GPGTools to use a certificate that OS X can validate
Create a boxen package that installs the necessary certificates to allow OS X to validate the cert
Use an insecure (http) connection - note that pkgdmg already puts the -k flag on the curl anyway, rendering it insecure.

Deprecated in favour of brewcask and Boxen's in built support for
homebrew/brewcask.


Add Model of the Universal Robot UR10e and UR3e

Tasks:

[x] Draft import from urdf.
[x] Edit in blender to split screw in new meshes.
[x] Creation of normal + occlusion maps.
[x] Add in samples world.
[x] Ros package.
[x] Documentation: https://www.cyberbotics.com/doc/guide/ure?version=feature-universal-robot-ure10
[x] Changelog.

For some strange reason the small changes I did on the universal robot doc page broke the robot-components of all the pages.

As discussed with @fabienrohrer, it seems on this branch the doc is loading https://cyberbotics.com/wwi/R2019b/webots.min.js but on develop it is loading https://cyberbotics.com/wwi/R2018b/webots.js, remains to find the reason of this difference.

test me

test me

test me

test me

Build Failed :confused:
https://cyberbotics2.cyberbotics.com:8443/job/Webots/591/consoleText

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

test me

test me

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

:+1:

test me

test me

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

:+1:

The branch restores now R2019a in docs/dependencies.txt to let the doc works on develop. R2019b will be restored in #435

test me

test me

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

test me

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

Build Passed :ok_hand:
You can safely merge 'feature-universal-robot-ure10' into 'develop'.

For a strange reason, doc.php is still looking for 'https://cyberbotics.com/wwi/R2019a/webots.min.js'  (version updated correctly but remain the 'min').
But the cdn is up to date<EMAIL_ADDRESS>

The CDN is probably having issues because it sometimes works and sometimes doesn't work...

:+1:

👀


Environment.updater doesn't work when set to False

my understanding of the documentation located at http://elsdoerfer.name/files/docs/webassets/environment.html#webassets.env.Environment.updater. is that if environment.updater is set to False. then assets are not built. However when this is the case, there are numerous errors that take place.

bundle._build raises a  BuildError if the output path doesn't exist.
the bundle is still attempting to build and raises the following errors in merge.py:apply_filters

filters try to access disk and raise errors if directories do not exist or the app server user is denied permission
filters try to run programs that are not necessarily installed on the app server, eg: sass.

I've found that setting updater to 'never' solves issue 1.  But so far I have not found an easy solution to number 2, personally, i would expect that if updater is set to False or never that filters never even attempt run, so there are no disk seeks or attempts to execute programs that are not installed on the app server, eg: sass.
Is there a way to ensure filters are not run in "production" mode or if updater is set to False or never?
Thanks

However, 1) has been fixed a couple days ago via 7651e59f9cd613b97fb9f29581dea23a7ffcbfcd - so try the latest head.
I think you see 2) only when using "never", so you should be fine with False, and the new version.

Thank you, I grabbed the latest and it does work as documented and as expected.  Sorry for the inconvenience, the only issues/commits i found on the subject pre-dated the 0.6 release.


Does wasmer-go support importing memory into the wasm module

Summary
I don't see any examples in the docs if wasmer-go supports importing memory from the host environment into the Wasm module? Is it possible and how?
Additional details
For example: The Wasm module (the one below) should be able to use the memory provided by the host and use the host provided functions to manage it.
Type: wasm
Size: 2.5 MB
Imports:
  Functions:
    "env"."ext_allocator_free": [I32] -> []
    "env"."ext_allocator_malloc": [I32] -> [I32]
  Memories:
    "env"."memory": not shared (20 pages..)
  Tables:
  Globals:
Exports:
  Functions:
    "intr_cust_function": [I32, I32] -> [I64]
  Memories:
  Tables:
    "__indirect_function_table": FuncRef (352..352)
  Globals:
    "__data_end": I32 (constant)
    "__heap_base": I32 (constant)

Found what I was looking for.

@radkomih it'd be useful if you posted the solution =P


remove autocorrection

Latest version.
I have disabled every think about autocorrection but the system still show the suggestion above the keyboard.
This happen then you press back to remove the tag and wstagsfield select a tag.
https://imgshare.io/image/lOEHK

@amatig Can you share your code about WSTagsField? What are your settings?

@amatig Please reopen this issue if this is still happening. Thank you


amazing !

Hi @thysultan,
Just wanted to tell you that's a great idea to optimize rendering by using canvas and the result is great. Do you will continue or this is just a proof of concept ?
Still remains a lot of work to be done but your approach is by far better than Ace or CodeMirror, the potential is just unlimited and memory and performance will be by far greater !
Good job dude, congrats :+1:

Just found another project a bit more advanced, and using the same principles : http://jdfreder.github.io/poster/
I've tested a series of copy/paste resulting in about 1,500,000 lines without freezing, done the same on ace, started to freeze after about 2000 lines :laughing:

Thanks, i haven't put much work into it recently but i do plan to continue with this.
The goals of this project where two fold, the first one was to use canvas for rendering which should yield improvements in memory and performance but more importantly implementing an efficient gap buffer data-structure in JavaScript that would yield better memory and performance overall.
The next hurdle left to overcome is a good middleware implementation that would allow implementations to extend the editor with regards to syntax highlighting and plugins.


CUDA_ERROR_OUT_OF_MEMORY

@davidsandberg Even i have set the batch_size to 1, also occur this error
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1030] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX 1060 6GB, pci bus id: 0000:01:00.0)
E tensorflow/stream_executor/cuda/cuda_driver.cc:924] failed to allocate 5.93G<PHONE_NUMBER> bytes) from device: CUDA_ERROR_OUT_OF_MEMORY

Try to change GPU memory fraction yo 0.9 or 0.8

@pavankumarkatakam thanks

@pavankumarkatakam Great! Thanks!


yarn start exits with ok in less than 4 seconds

While trying to run the development server locally, gatsby fails quietly. I was able to run the development server through Vagrant, but it fails on my native environment.
I ran yarn clean && yarn start with the verbose flag and this is the output:
aaronlynanderson@Environment-2 community-portal % yarn clean --verbose
yarn run v1.22.4
$ gatsby clean --verbose
verbose set gatsby_log_level: "verbose"
verbose set gatsby_executing_command: "clean"
verbose loading local command from: /Users/aaronlynanderson/Documents/MakerDAO/community-portal/node_modules/gatsby/dist/commands/clean.js
verbose running command: clean
info Deleting .cache, public
info Successfully deleted directories
✨  Done in 8.15s.
aaronlynanderson@Environment-2 community-portal % yarn start --verbose
yarn run v1.22.4
$ npm run acrollout && npm run develop --verbose
npm info it worked if it ends with ok
npm verb cli [
npm verb cli   '/Users/aaronlynanderson/.nvm/versions/node/v14.5.0/bin/node',
npm verb cli   '/Users/aaronlynanderson/.nvm/versions/node/v14.5.0/bin/npm',
npm verb cli   'run',
npm verb cli   'develop',
npm verb cli   '--verbose'
npm verb cli ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm verb run-script [ 'predevelop', 'develop', 'postdevelop' ]
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS>ignored because ignore-scripts is set to true<EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm verb exit [ 0, true ]
npm timing npm Completed in 346ms
npm info ok
✨  Done in 2.55s.

@MaximumCrash any ideas?

I'll changing the package.json for npm run start to use gatsby develop instead of npm run develop

Closed by #217


[4864] - blat trainee degree subjects when importing from HESA

Context
We create HESA degrees using find_or_initialize_by(:subject).
So if a trainee has two degrees with the same subject (either from UK and abroad or BSc / MSc), we only save one of them. We overwrite the first one every time.
Changes proposed in this pull request
Delete a trainees degrees upon update/create from HESA

 HESA - only creating one degree if there are multiple with the same subject


hasVersion incorrect in RDF documents

Based on observations by Steve Baskauf with respect to the hasVersion attribute of Darwin Core terms in RDF, I have made the following observations.
The hasVersion information is incorrect throughout both dwctermshistory.rdf and dwcterms.rdf. The terms all have hasVersion populated by URLs with fragment identifiers that serve to link to the right place in the HTML documentation, but do not contain an actual identifier for a version. In fact, the term version is in identifier for the term itself. There is only ever one version of a term (currently), and that is identified in the rdf:Description. For example, the following identifies a no longer current version of the Darwin Core genus term.
<rdf:Description rdf:about="http://rs.tdwg.org/dwc/terms/genus-2009-04-24">
Any functional or semantic change in the term results in the creation of a new term, which would be related to the previous one with dcterms:replaces. Thus,
<rdf:Description rdf:about="http://rs.tdwg.org/dwc/terms/genus-2009-08-24">
<dcterms:replaces rdf:resource="http://rs.tdwg.org/dwc/terms/genus-2009-04-24"/>
And since the replacement mechanism, using distinct terms each time there is a semantic change, is the versioning mechanism, the hasVersion attribute of each term would have to just reference the term itself, which seems like redundant clutter.
Since they are all incorrect, no one could have possibly used them effectively to date,  and apparently no one has even tried. I propose to remove all hasVersion tags from dwctermshistory.rdf and dwcterms.rdf. And since it is not semantically or functional relevant, I propose that this can be done as "Minor editorial errata" under 3.1 of the Darwin Core namespace policy. I'll give this a week or so for commentary here, and if there are no objections, will make the changes throughout the documentation.

I created a fix for this problem by loading the dwctermshistory.rdf file into my local triplestore and then running the following query:
# Remove the dcterms:hasVersion triples (5769 triples = 6353-584):
CONSTRUCT {?s ?p ?o} WHERE {
  ?s ?p ?o.
  MINUS {?s dcterms:hasVersion ?o}
  }

I counted the number of triples before (6353) and after (5769) and the difference (584) was the same as the number of terms that I counted (584), which should be the case if the one dcterms:hasVersion triple was removed for each term.  I visually examined the file and this seems to be a fix.
I have generated a replacement file for dwctermshistory.rdf that includes this fix as well as the fix for Issue #111 .  I can either email it to John or try to do a pull request.

OK, I've looked at the dwcterms.rdf situation now.  I'm going to propose a solution that is slightly different than what John suggested in the original issue description.  For dwctermshistory.rdf, I agree that it makes sense to remove all of the dcterms:hasVersion triples.  As I perceive the terms in dwctermshistory, they ARE all of the versions of Darwin Core terms, so it doesn't really make sense to say that the versions have versions.
However, in the case of dwcterms.rdf, I think that it is critical that some dcterms:hasVersion triples remain.  dcterms:hasVersion is the link from the current generic term (e.g. dwc:recordedBy) to it's dated version (e.g. dwc:recordedBy-2014-10-23) and we don't want that link to go away.  Yes, there is sometimes a dcterms:replaces link to the version that the current term replaces, but not every term has an earlier version.  In such a case, if all of the dwc:hasVersion links go away, there is no connection between a current term in dwcterms.rdf and any version of it in dwctermshistory.rdf .
I propose that the fix for dwcterms.rdf is to fix the dwc:hasVersion links so that they point to the correct URI of the most recent version in dwctermshistory.rdf .  Then delete all of the dwc:replaces triples from dwcterms.rdf .  They aren't needed if there is a link from the current term in dwcterms.rdf to the current version in dwctermshistory.rdf because the current version is linked to the previous version in dwctermshistory.rdf (or at least it is after applying the repairs I described earlier).
Here is the current situation (showing only the relevant triples):
In dwcterms.rdf:
dwc:recordedBy
     dcterms:hasVersion <http://rs.tdwg.org/dwc/terms/history/#recordedBy-2014-10-23>;
     dcterms:replaces dwc:recordedBy-2009-04-24.

In the repaired dwctermshistory.rdf:
dwc:recordedBy-2014-10-23  dcterms:replaces dwc:recordedBy-2009-04-24.
dwc:recordedBy-2009-04-24 dcterms:replaces <http://rs.tdwg.org/dwc/dwcore/Collector-2007-04-17>.

Here is how I am proposing it should be:
In the repaired dwcterms.rdf:
dwc:recordedBy dcterms:hasVersion dwc:recordedBy-2014-10-23.

In the repaired dwctermshistory.rdf:
dwc:recordedBy-2014-10-23  dcterms:replaces dwc:recordedBy-2009-04-24.
dwc:recordedBy-2009-04-24 dcterms:replaces <http://rs.tdwg.org/dwc/dwcore/Collector-2007-04-17>.

If the repairs are made as I have suggested, it then becomes a simple matter to construct a query to find any version of a current term.  For example, if the repaired dwcterms.rdf and dwctermshistory.rdf files are loaded into a triplestore, the SPARQL query:
SELECT DISTINCT ?version ?modDate ?status ?dfn WHERE {
  dwc:recordedBy dcterms:hasVersion ?curVersion.
  ?curVersion dcterms:replaces* ?version.
  ?version dcterms:modified ?modDate.
  ?version rdfs:comment ?dfn.
  ?version dwcattributes:status ?status.
  }
ORDER BY DESC(?modDate)

will display all of the versions of dwc:recordedBy and give their modified date, status, and definition.  Doing this sort of query would be much more difficult if all of the dcterms:hasVersion triples were removed from dwcterms.rdf.
I've loaded the repaired versions of dwcterms.rdf and dwctermshistory.rdf into our endpoint at Vanderbilt: http://rdf.library.vanderbilt.edu/sparql?view and you can try running the query above to see that it works.  Try changing the term to dwc:genus, which has a longer and more intersting history.
For reference, here are the queries I used to fix dwcterms.rdf:
I loaded the triples from dwcterms.rdf as the graph http://dwcterms
and the triples from the repaired dwctermshistory.rdf as the graph http://dwctermshistory
# List all of the terms in dwcterms.rdf to find out how many there are:
SELECT DISTINCT ?term WHERE {
GRAPH <http://dwcterms> {
  ?term dwcattributes:status ?value.
      }
  }
ORDER BY ?term
# There are 192

# For each of the 192 current terms, find the term version whose local name (after the "/") matches
# the local name (after the #) of the dcterms:hasVersion object:
SELECT DISTINCT ?s ?subjectUri WHERE {

  GRAPH <http://dwcterms> {
    ?s dcterms:hasVersion ?objectUri.
         }

  GRAPH <http://dwctermshistory> {
    ?subjectUri ?p ?o.
         }

  BIND(REPLACE(str(?subjectUri), '^.*(/)', "") AS ?subjectLocalName)
  BIND(REPLACE(str(?objectUri), '^.*(#)', "") AS ?objectLocalName)
  FILTER(?subjectLocalName=?objectLocalName)
  }
# It finds 192

# Construct the corrected dcterms:hasVersion triples:
CONSTRUCT {?s dcterms:hasVersion ?subjectUri} WHERE {
SELECT DISTINCT ?s ?subjectUri WHERE {
  GRAPH <http://dwcterms> {
    ?s dcterms:hasVersion ?objectUri.
         }
  GRAPH <http://dwctermshistory> {
    ?subjectUri ?p ?o.
         }
  BIND(REPLACE(str(?subjectUri), '^.*(/)', "") AS ?subjectLocalName)
  BIND(REPLACE(str(?objectUri), '^.*(#)', "") AS ?objectLocalName)
  FILTER(?subjectLocalName=?objectLocalName)
        }
  ORDER BY LCASE(?subjectLocalName)
  }
# For some reason, it repeats dwc:acceptedNameUsage twice and produces 193 results ???
# I deleted the extra triple

# DROP ALL, then reload only dwcterms.rdf .  Clean out all of the dcterms:hasVersion and dcterms:replaces triples:
CONSTRUCT {?s ?p ?o} WHERE {
  ?s ?p ?o.
MINUS {
  ?s1 dcterms:replaces|dcterms:hasVersion ?o.
      }
  }

# Save the output, then load the cleaned dwcterms.rdf and the repaired dcterms:hasVersion triples
# Output the repaired graph so that it can be saved in a single file (sorted by LCASE of local name):
CONSTRUCT {?s ?p ?o} WHERE {
  ?s ?p ?o.
  BIND(LCASE(REPLACE(str(?s), '^.*(/)', "")) AS ?localName)
  }
Order by ?localName

I agree that the proposed changes remove the unnecessary clutter from dwctermshistory.rdf and maintain the essential and originally intended links of the recommended terms to their histories in dwcterms.rdf.
@baskaufs Could you run your queries to create updated versions of these files based on the two rdf documents as of the latest commit and either send them to me or push them and create a pull request? Then do a separate commit for the proposed fixes in Issue #111 ?

@tucotuco Assuming I did the pull request right, I created a pull request with the dwcterms.rdf and dwctermshistory.rdf files that I modified as described in this issue and Issue #111.  I'm not sure about how to accomplish a separate commit for the two issues.  This issue involves fixing both files, and #111 applies only to dwctermshistory.rdf .  I guess I don't really understand the process of separating it into two commits, but I don't have write access to the dwc repo anyway, so maybe you can just do it.

With the new management of term versions, this issue is no longer a problem.


build: remove whitespace at eof and between tags

Fixes #1596

Not sure what the status of this PR is now- I'm just going to close it and you can open a new one if you want to go for a different approach.


Fixed console error spam at editor startup (about invalid edited scene index)

Fix #49096

Thanks!


ARCSequenceOpts: Add LoopSummary verifier

ARCSequenceOpts with loop support works on regions inside out. While processing an outer region, it uses summary from
the inner loop region to compute RefCountState transitions used for CodeMotionSafe optimization.
We do not compute the loop summary by iterating over it twice or iterating until a fixed point is reached.
Instead loop summary is just a list of refcount affecting instructions. And BottomUpRefCountState::updateForDifferentLoopInst and TopDownRefCountState::updateForDifferentLoopInst are used to compute the RefCountState transition when processing the inner loop region. These functions are very conservative and the flow sensitive nature of the summarized instructions do no matter.
This PR adds a verifying assert to confirm this.

@swift-ci test

@swift-ci test

@swift-ci Please Test Source Compatibility

Build failed
Swift Test Linux Platform
Git Sha - e1cc16eb5f3b8f2918fbae9e9100f041c2f03aa3

Build failed
Swift Test OS X Platform
Git Sha - e1cc16eb5f3b8f2918fbae9e9100f041c2f03aa3

Errors in sck are all known SILGen bugs.


minPageWidth not respected by smaller iPhone screens

iPhone 5 on latest operating system renders screen smaller than 400px in width.
To make our app fully visible on smaller screens like this, we set the minPageWidth property in ConfigProvider to 1.
Using Chrome devtools to simulate these smaller screen sizes, this worked great, and we merged the code.
However, on a real iPhone device, you still must scroll on the x-axis to view the entire page, so minPageWidth is not respected properly.

Hey Jeff, thanks for reporting this issue!

Have you been able to repro this issue on a Saucelabs emulator? (using a manual session is fine)
Do you know what the culprit is or have a hacky workaround?

Contributions are most welcome. Let me know if you want help getting started. If you are unable to, the answers to these questions will help us automate testing and fix it sooner.

I have not tried this on a saucelabs emulator, just the iPhone device itself. Do we have access to these emulators? I have never used one.
Also, now that I double check the chrome devtools, they have removed iPhone 5 as an option for aspect ratio testing. However, the 'responsive' option behaves as expected for really small screen widths.

And to the second question I do not know what the culprit is unfortunately.. i think the quickest hacky workaround I can think of off the top of my head would be altering the meta tag to scale the page down a bit. But this is not ideal obviously

My guess is that it's this issue. We could add min-device-width and max-device-width to the defaults here and make them match the max-width and min-width.
We'll also need to add maxDeviceWidth and minDeviceWidth maps here.
Do you want to try that fix?

further notes:

iphone 5 and before are 320 pixels in width.
later iphone screens render at 375px width (at smallest). the minPageWidth property does work for these screens, interestingly, since 400px is the minimum by default, so the property is not being ignored.

:tada: This issue has been resolved in version 16.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Search Input validation

Seems like search input length isn't being validated resulting in undefined.

Thanks for reporting @svnlto deploying a fix now


H 4  or V 4 (with a space after the H or V fails but its valid svg syntax

currently
/// Fetch a single float value src/scanner.dart/_fetchSingleCoordinate()
_fetchSingleCoordinate() {
_fetchFloatValue();
_skipWhitespace();
}
should be (in my not very expert opinion)
_fetchSingleCoordinate() {
_skipWhitespace();
_fetchFloatValue();
_skipWhitespace();
}

Okay will look into that.

Okay will look into that.

Closed by c068a57
Thanks for the tip.


Initial State

This is probably more of a question than an issue.
I have a SPA application which is initially server side rendered (isomorphic).
On the first SSR I want to add state so that if a user goes back to this initial server side rendered URL the SPA knows how to fetch the data based on the state of history.
What is the best practice to handle initial state with this package?
Do I have do just do something like:
import createHistory from 'history/createBrowserHistory';

....

history = createHistory();
history.replace({
  pathname: currentUrl,
  search: '?stuff=abc',
  state,
});

Or is there a cleaner way of doing this ? It seems a bit hacky that a replace is being called to replace a pathname / search when the path /search hasnt changed.
Is there a nice way of adding this state when doing the createHistory call - can it be passed in somehow ?

Yes, if you want location.state on the initial page load you have to do a replace immediately to get it. However, you don't have to pass along the current pathname and search. Just use history.replace({ state }) and you should be fine.

thanks @mjackson 👍

actually this doesnt work, if you have a server side rendered url thats http://mrdomain/stuff/?other=123 will be replaced with http://mrdomain/stuff/

Hmm, in that case you probably need history.replace({ state, ...window.location }) so you get pathname and search from the current window.location as well.
Going forward, I think it might be interesting to re-use properties from the current location in push and replace if they're not present in the given argument, but we have to explore all the edge cases first.


[NER annotation] cannot see entity labels when annotate

Only reproduce in a dev branch. Closing

Finally we can reproduce it

Again, automatically resolved by rollback


Error in installation Tensorflow , windows

Hi, I have installed Cuda and CudaDLl files and also defined paths to the respective folders. I am still getting an error on launching Tesorflow module saying DLL failed. I checked the folders. I think I have all the files in the respective folders. I think I am missing something below is the error screen on importing tensor flow.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Hi, I am having a similar problem.
OS - Windows 10 64 bit,
Python- 3.6.5 (All liberaries in path)
Tensorflow version 1.7 GPU, Pip installation
Bazel not installed
CUDA - 9.0 (All liberaries in path)
cuDNN - 7.0 (All liberaries in path)
GPU - Nvidia Titan X 12 GB
import tensorflow gives error
The error is as follows:

import tensorflow
Traceback (most recent call last):
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 18, in
pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 17, in swig_import_helper
return importlib.import_module(mname)
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_init_.py", line 24, in
from tensorflow.python import *  # pylint: disable=redefined-builtin
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 18, in
pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 17, in swig_import_helper
return importlib.import_module(mname)
File "C:\Users\Admin\AppData\Local\Programs\Python\Python36\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

Hi All,
Thank you for your time, I forced reinstalled tensorflow via conda and its working fine now. I am  not sure what the issue was. I had correct version of CUDA dll's and path. Below is the code and stack exchange lin.
pip install tensorflow --upgrade --force-reinstall
https://stackoverflow.com/questions/35953210/error-running-basic-tensorflow-example


Repos in SCM view expand after closing/opening the sidebar

Collapse a repo in the SCM view

Toggle the sidebar, now it's expanded

Similar https://github.com/microsoft/vscode/issues/101094


Fix slider bug

Fix slider bug when first selecting 0 and then sliding to other values caused the SEND button not to be enabled.
Changes

Fix slider bug
Remove Tracking Pixel

Test

To replicate the bug: in master, setup the project to make the survey show up.
Once the survey is shown, select 0 and then slide to other values (1, 2, 3 ... 10). The SEND button should still be disabled.
To test the fix checkout this branch.
Repeat Step 2. The SEND button should now be enabled whenever a score is selected.

Internal Trello Ref: link

Who's qualified to review this?

@hachibu usually @prabhatjha takes a look at the code or @marinhero who knows a little bit of Objective-c, I think 😄

@diegoserranoa cool, I assigned them to this.


Using non-default projection with OSM and Next.js

Hello,
I'm building a project that uses data drawn from WFS server on top of OpenStreetMap in Next.js using Rlayers. In the past I managed to achive what is necessary in the browser using Vite, however this project has many other, non-map features and thus relies on Next.js and React, so I need to repliacte the results here.
The problem: after converting units using proj4 the RMap component displays everyting correctly (distances, scale to surroundings, etc.), but underlying ROSM thinks I'm in the middle of the ocean... Is there a way to pass projection to OSM? I didn't find it in the docs and blindly passing projection prop does not work.
My test setup:
'use client';

import React from 'react';
import 'ol/ol.css'
import WFS from 'ol/format/WFS.js';
import proj4 from 'proj4';
import * as ol_proj4 from 'ol/proj/proj4.js';
import {RMap, ROSM,  RLayerVector, RStyle} from 'rlayers';

proj4.defs("EPSG:2180","+proj=tmerc +lat_0=0 +lon_0=19 +k=0.9993 +x_0=500000 +y_0=-5300000 +ellps=GRS80 +towgs84=0,0,0,0,0,0,0 +units=m +no_defs +type=crs +axis=neu");
ol_proj4.register(proj4);

var miejsce4326 = [18.032252,53.215649];
var miejsce = proj4('EPSG:4326', 'EPSG:2180',miejsce4326);

export default function MapElement(){
<RMap width={'100%'} height={'60vh'} initial={{center: miejsce, zoom: 14, projection: "EPSG:2180"}}>
            <ROSM
                projection={"EPSG:2180"}
            />
            <RLayerVector
                    zIndex={5}
                    format={new WFS({ version: '2.0.0'})}
                    url='https://corsproxy.io/?https://mapy.geoportal.gov.pl/wss/ext/PowiatoweBazyEwidencjiGruntow/0403?service=WFS&request=GetFeature&version=2.0.0&srsname=urn:ogc:def:crs:EPSG::2180&typename=dzialki&bbox=589920.405294056,431210.22956701985,593049.7467176472,437239.0737785711,urn:ogc:def:crs:EPSG::2180'
                    projection={"EPSG:2180"}
                >
                    <RStyle.RStyle>
                        <RStyle.RStroke color='#007bff' width={3} />
                        <RStyle.RFill color='transparent' />
                    </RStyle.RStyle>
            </RLayerVector>
        </RMap>
}

screenshot: https://imgur.com/wEyboGw
Apart from that, the console is constantly swarmed with web-worker error that is caused by Rlayers. After disabling Rlayers and map, error disappears.
./node_modules/web-worker/cjs/node.js
Critical dependency: the request of a dependency is an expression

Import trace for requested module:
./node_modules/web-worker/cjs/node.js
./node_modules/geotiff/dist-module/worker/decoder.js
./node_modules/geotiff/dist-module/pool.js
./node_modules/geotiff/dist-module/geotiff.js
./node_modules/ol/source/GeoTIFF.js
./node_modules/ol/source.js
./node_modules/rlayers/dist/layer/RLayerTile.js
./node_modules/rlayers/dist/index.js
./app/mapa/components/mapElement.js

Any help will be appreciated, I really want this to work and beside those errors this Rlayers is a brilliant piece of software and a huge timesaver.
Cheers from Poland,
Maciej

I don't know how your code works, because your component does not return the JSX.
The projection field is not part of the initial - it is an RMap parameter:
<RMap width={'100%'} height={'60vh'} projection={'EPSG:2180'} initial={{center: miejsce, zoom: 14}}>

You should not set a projection on ROSM layer - this would mean that you are including an ROSM layer in that projection - you are not - you are including an ROSM layer in its default projection. Layers reproject to the RMap projection when displayed.
I suggest you add a code linter or some other type of checker or an editor that understands types such as Visual Studio Code, it would have immediately found the invalid properties.

https://github.com/geotiffjs/geotiff.js/issues/396

Thank you, it solved my problem. I will think about type checker or even switching to typescript.


Add scale bar to maps

The maps should have a small scale bar and numeric scale shown.

It would be helpful to have the maps use a scale at a multiple of 2500 or 5000 (such as 1:2500, 1:5000, 1:7500, etc.), so we do not have unusual values.