handlebars runtime amd build fails to work with require js.

it simply fails to return an object (handlebars or otherwise)
it seems that if i want to use handlebars runtime in combo with requirejs / amd then i need to use the non-amd build and write a custom shim.
when i swap out the runtime version for the full version i have no such problem.

while the amd build fails to work, the handlebars.runtime file provides amd functionality via umd.
perhaps handlebars.runtime.amd should be removed from the repo.

Could you point me to the file that you were trying to reference? We've had a lot of confusion around trying to support the numerous different build/package ecosystems :(

i'm not sure what to say aside from its the handlebars.runtime.amd file available via bower. it appears that its not check into this repo.
it would be a huuuuuge help if the readme included a few words regarding amd / requirejs usage. it took me a couple of days to figure this out. part of the problem is that my project is currently on handlebars v1.3 which is broken in the same manner. its my impression that requirejs is by far the most common instance of amd usage. if it doesn't work with requirejs its broken in most amd environments.

Not using r.js myself, this is difficult for me to document. I'd be glad to
include any information you may have found from your investigation, if
you'd like to submit a PR!
On Thu, Feb 5, 2015 at 9:13 AM Matthew Kime<EMAIL_ADDRESS>wrote:

i'm not sure what to say aside from its the handlebars.runtime.amd file
available via bower. it appears that its not check into this repo.
it would be a huuuuuge help if the readme included a few words regarding
amd / requirejs usage. it took me a couple of days to figure this out. part
of the problem is that my project is currently on handlebars v1.3 which is
broken in the same manner. its my impression that requirejs is by far the
most common instance of amd usage. if it doesn't work with requirejs its
broken in most amd environments.
—
Reply to this email directly or view it on GitHub
https://github.com/wycats/handlebars.js/issues/948#issuecomment-73062001
.

I'm using full Handlebars 3.0.0 with requirejs in development mode and Handlebars runtime 3.0.0 with requirejs all compiled in one file in production mode without any issue.

@JSteunou  how could you do it? I spent a past how to just make typical paths/shim configure but Handlebars didn't work :(
It's too hard to use Handlebars with RequireJS, there are many versions but almost no document about using them at all :(

I'm using requirejs-hbs with that config
    packages: [
        {
            name: 'hbs',
            location: 'bower_components/requirejs-hbs',
            main: 'hbs'
        }
    ],
    ...

and those options in my build task
                exclude: ['handlebars-compiler'],
                paths: {
                    'handlebars': 'bower_components/handlebars/handlebars.runtime',
                    'handlebars-compiler': 'bower_components/handlebars/handlebars'
                }

just change handlebars.runtime.js or handlebars.js
(function webpackUniversalModuleDefinition(root, factory) {
if(typeof exports === 'object' && typeof module === 'object')
module.exports = factory();
else if(typeof define === 'function' && define.amd)
define(factory);
else if(typeof exports === 'object')
exports["Handlebars"] = factory();
else
root["Handlebars"] = factory();
})  ... ...
change to
(function webpackUniversalModuleDefinition(root, factory) {
root["Handlebars"] = factory();
}) ... ...
everything is ok!

The version 3.0.3 is also failing when i try to import it from ES6 file.
It would import it, but than says that Handlebars is undefined.
any clues?

@rafaelfaria please post an example showing the failure case in another bug so we can keep things clear.

Handlebars 3.0.3, either runtime or full version, does not work with RequireJS.
Failed to create a Handlebars object with either RequireJS 2.1.8 and 2.1.20
Do you need me to post an example page online?

Please.
On Thu, Aug 27, 2015 at 3:27 AM Lucas Lessa<EMAIL_ADDRESS>wrote:

Handlebars 3.0.3, either runtime or full version, does not work with
RequireJS.
Failed to create a Handlebars object with either RequireJS 2.1.8 and 2.1.20
Do you need me to post an example page online?
—
Reply to this email directly or view it on GitHub
https://github.com/wycats/handlebars.js/issues/948#issuecomment-135338895
.

My apologies for a false alarm above. Let me be clear. Handlebars does work with require.js:
// this works well
requirejs(["handlebars"], function(Handlebars) {
  // Handlebars-related templating code here
});

Passing tests:
http://lucaslessa.com/handlebars/qunit-load-handlebars-using-requirejs.html
The issue I have is when require.js is already loaded on the page, then loading handlebars.js afterwards, the Handlebars global object is nowhere to be found.
jQuery, on the other hand, still loads and makes its global objects $ and jQuery available when its script tag comes after require.js. Can Handlebars do the same?
Tests - jQuery passes, Handlebars fails:
http://lucaslessa.com/handlebars/qunit-handlebars-loaded-after-requirejs.html
I ended up patching handlebars-v3.0.3.js so it no longer tries to use define() when loading and the problem went away.
	else if(typeof define === 'function' && define.amd)
		define(factory);

Try using require-hbs plugin with the solution I posted above.

@JSteunou Thanks for the suggestion. I'm loading handlebars.runtime as part of an app service asynchronously embedded on a number of different sites, where I don't have control of the page's javascript ecosystem. That's how I ran into the problem above, but only for those sites that load require.js before my app.
My solution patching handlebars.js was too forceful. Seeing what you did above, the following (similar) workaround is better suited to my app's js, does not need a plugin, and means I can use future handlebars releases without patching:
if (typeof requirejs === 'function') {
  // define external/CDN paths - see http://requirejs.org/docs/optimization.html#empty
  requirejs.config({
      paths: {
          'handlebars/runtime': '//cdnjs.cloudflare.com/ajax/libs/handlebars.js/3.0.3/handlebars.runtime'
      }
  });

  require(['handlebars/runtime'], function (myHandlebars) {
    myAppModule.handlebars = myHandlebars;
  });
} else // regular method of loading handlebars.runtime


How to render headmatter data in custom layout? Missing docs or feature, not sure which one.

Is your feature request related to a problem? Please describe.
I would like to use headmatter data inside my custom layout but there isn't any documentation on how this can be done.
Describe the solution you'd like
If I have a cover layout. How can I render a title in the template?
example.md
---
layout: cover
title: My Title
---

/layouts/cover.vue
<template>
   <div class="slidev-layout cover"

{/* How do I render "My Title" here? */}

     <slot />

</template>

Describe alternatives you've considered
The way I currently do it is by manually styling h1 tags inside the cover css class.

Try $frontmatter.title?
Here are some docs: https://sli.dev/guide/global-context#frontmatter


Fixed #32670 -- Allowed GDALRasters to use any GDAL virtual filesystem.

Hello Everybody. Yesterday I was asking in the django-developers mail list, and today it seems I'm opening my first PR.
This PR removes the limit of django.contrib.gis.GDALRaster on which kind of paths you can instantiate. Previously this was hard limited to filesystem paths and GDAL in memory /vsimem/ paths.
GDAL has full support for virtual filesystems, where the path root specifies something about the protocol to access the file.
https://gdal.org/user/virtual_file_systems.html
With this set of changes you can pass /vsi<prefix>/ paths through django.contrib.gis.GDALRaster letting GDAL handle how to access those files.
I wonder if it would be better to provide some of those vsi as GDALRaster "drivers" like the current "MEM" driver, but for the moment this was the least resistance approach to it.
I added a test with vsizip as an example, and tried with vsis3 manually with no problems.

@kxtells Thanks for this patch :+1: All new features require an accepted ticket.

Thanks @felixxm , I just opened https://code.djangoproject.com/ticket/32670 to track/discuss this feature then.

I added some examples in the documentation, but while doing so I came up with a problem regarding is_vsi_based, so I added an extra change to handle it.
Mostly, is_vsi_based will return True for any /vsi* GDALRaster (which I think makes sense). But vsi_buffer would fail for any non memory Raster, so I added an extra check in the vsi_buffer property to return None if the raster is not actually in memory, otherwise the call to capi VSIGetMemFileBuffer will return an invalid pointer.

Thanks for the comments @felixxm , pushed a commit with an extra test and the manual adjustments suggested.


[RFC]: Add C implementation for @stdlib/math/base/special/maxabs

Description
This RFC proposes adding native C implementation for @stdlib/math/base/special/maxabs
double stdlib_base_maxabs( const double x, const double y );
Related Issues
Related issues # , # , and # .
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

@Shashankss1205 can you pleas refer to #1449 and populate description well. Thanks for opening this.  Also mark related issue as #1433.

@Pranavchiku @Planeshifter @kgryte I have understood this issue, I want to work on it. So please assign this issue to me

@Shashankss1205 Are you planning on implementing this, given that you opened the issue?

Yes I am already working on this @kgryte . Please assign it to me.

@Shashankss1205 can you please refer to #1449 and populate description well. Thanks for opening this. Also mark related issue as #1433.

I didn't understand about "populating the description", I just kept it the way other similar issues were made. Please suggest the required changes that I should make.

@Shashankss1205 I've updated the OP.

@Shashankss1205 I've updated the OP.

Thank you @kgryte for the guiding help.

Please Review the PR and suggest the needful changes @kgryte .


Create database model

As an developer
I need to create database model for recommendation API
So that the recommendation data can be retrieved correctly by users
Details and Assumptions
The database model should contain 2 tables:

Recommendation table: id of a basic product and a recommended product, and their recommendation type, e.g., cross-sell, up-sell, accessory

Acceptance Criteria
Given the table schemas
When database is created
Then querying and transactions can be performed on the database

Feedback from Prof:
Creating one table is fine.
We can directly use the Product service with the given product ID.


Align HAL heap buffers to a configuration-defined minimum alignment.

This is required to make some of the SIMD code we emit happy. Given that 95% of our buffers are suballocated in the compiler the larger-than-max_align-alignment isn't a large cost relative to the simplicity and potential perf benefits it has.
This does now require all imported buffers match the alignment. There's no way to query this today from allocators beyond using the config.h value but that's fine for now as we are only ever wrapping buffers in files that should already be aligned.
Fixes #8506.

I'm going to do a follow-up that tries changing the default alignment to 128 to measure effects. It'd be cool if it helped - free performance!


chore: increase npm retries and max timeout

CI servers fail with timeouts, or network issues quite frequently,
requiring maintainers to manually restart failed builds.
Hopefully these npm settings will help mitigating these issues.
Change-Type: patch
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

God, I find myself restarting Travis builds because of timeouts much more often than before (basically every single day). Let's hope this makes things better.

npm install timeouts or travis 20min-build timeouts?

Sorry, I mean npm install timeouts when running on Travis CI :)

Hopefully TravisCI's internet connection is more reliable than @jviotti 's Bolivian internet connection ;-)


Create hosted demo

What problem or use case are you trying to solve?
Right now one major issue that we are experiencing is that people are having issues installing OpenDevin, we would like to have a hosted demo that people could use without having to set it up on their own computers.
Describe the UX of the solution you'd like
A user could sign up, log in, and start using OpenDevin.
Do you have thoughts on the technical implementation?
This will require lots of thought including.

How to host the frontend/backend.
Where to run agents.
How to pay for it.

etc. etc.

Right now the frontend assumes it's run on the same machine as the backend. This is mostly due to the workspace directory setting; a user can currently set the workspace path to work with existing code on their machine.
If the backend isn't necessarily the user's machine, then the workspace directory is not relevant. We'll want to support uploading an existing project to the server instead. Since sessions are temporary, we'll also want to support downloading projects, version control, or some persistence method.

I'm optimistic about https://github.com/OpenDevin/OpenDevin/pull/1023 fixing the installation woes.
The biggest barrier to hosting a demo is, how to pay for it.
if there's cash, hosting options are a-plenty.

When a project is infinitely close to maturity, your idea is reasonable, but if the purpose is to solve the problem of installing OpenDevin, I think it is a more valuable option to provide better installation environment adaptation and automated judgment scripts. For example, it seems more valuable and urgent to reduce the difficulty of project deployment by providing packaged docker images and automated conditional judgment script solutions as much as possible.
This's exactly what I have observed in a project. By integrating script automation with docker images, it has received more attention and positive experience in similar projects. Project address: QAnything
This's my opinion.

Thanks @zhonggegege ! We're also working on easier local install, as @foragerr mentioned. This issue is for even easier use than that, and it's on our roadmap for June, a somewhat longer way out. So basically, we agree!

I believe the best route is to just wrap it in electron, allow people to download the app.

just wrap it in electron

Thoughts on VS Code plugin?

VSCode plugin and electron app are both great ideas! We should track those in separate issues

For the hosted demo, here's what I'm thinking:

Users log in with GitHub

this gives us access to their code, which can be easily pulled in
it also lets us limit usage on a per-user basis to keep costs down

Users must add an API key for a publicly available LLM
Everything runs in Kubernetes

Helm chart for deploying
We create a KubernetesSandbox which creates Deployments in Kubernetes

These can be stood up and torn down quickly
We use ssh to connect into the sandbox
We can limit memory and CPU usage

And here are some features we'll need to support before we have a hosted demo:

Add API keys via UI (stored in localStorage)
Log in with GitHub (maybe just using a developer token in the self-hosted version)
Create workspace from GitHub project
Multi-tenancy

need ability to run multiple agents at once, each tied to a different session

We will probably need a database, but I'm not convinced of that yet. I'd love to get away with not having one until we really need it. We can probably do some basic persistence using S3 (and local files for self-hosted).

Users must add an API key for a publicly available LLM

Would you trust a random website with your OpenAI keys?
Probably yes because OpenAI makes managing budgets and limits easy - but it's worth pondering.
I wonder if OpenAI supports OIDC similar to "login with github".

I've made a quick hosted demo with Bring-Your-Own-Key (BYOK) taking into account the suggestions in this thread. Keys are purged when unused and strict budgeting is available. Here's a quick demonstration:
https://github.com/OpenDevin/OpenDevin/assets/134402079/0cb535db-647c-4674-ac30-b313eee5aad7
I could deploy gVisor for stronger isolation between users (on top of rootless Docker) and hope to get everything up soon. Compute costs are rather insignificant at this time with BYOK, so the demo can be free, and the demo's architecture is scalable across multiple hosts.
For now, MonologueAgent and CodeActAgent have been validated working (after some modification to entrypoint, plugins etc)! The work done for this demo could be useful in getting Electron / desktop app versions up as well, which I could explore.
Any feedback is appreciated, thanks! 😀

@tlpythonuse-colab this is rad! Is the code open source?

@tlpythonuse-colab I'd love to learn more about this if you want to get in touch with me on the Slack, or via the email in my GitHub profile

It looks cool :)

This is really good.
Where can we try it ?

Any update on this issue @rbren ?  I'm also thinking recently, about how to deploy opendevin to a distributed arch. If we deploy the whole opendevin as an instance, each instance can serve to many users/sessions ( with many agents running inside). But there maybe some problems we need to solve. For example: this is a stateful service, we need to route every request of a websocket connection to the same opendevin instance.
Or if we can seperate the agent execution process, so we can start a agent_runtime for each session on something like a container?

I believe this is something folks are working on but it will take some time to get up and running. Lots to consider.

@rbren how is this going, would like to contribute to this effort.

ok we could use jslinux to run an entire os in the browser on which opendevin runs, could we not?

This is finished yay!


How to change speed from km/h to mph?

Hi, American user here; I was wondering how to convert or change the setting on the app from km/h to mph in the custom code specifically.

Wow thank you! Can't believe I missed that!


Dont able to drag outside of view in view

Hello Support team,
I am building app having collection view inside another collection view and dragging view of the innermost cell to another view top of all views. But cannot drag view outside its foremost superview.
Let me describe my view hierarchy:-
Main view controller -> Having 2 views
1) 1st view where I wanted to drop by data
2) 2nd view having collection view and inside it there is another collectionview and I have assigned class "**DragDropcollectionview**" to it. I wanted to drag view from innermost collection viewcell to @1st view i.e. Outside of all.

But not able to drag it.
How can I make it draggable outside of all its superview ?
Thanks,

Hello ZalakSA:
1、Please make sure the canvas of DragDropManager is both draggable views and droppable views' superview in the view hierarchy.
2、Make sure the collectionview that drag out of  implements  DragDropCollectionViewDelegate correctly.
It should be run.

Hello KevinZhouRafael
I tried to give the delegate to its superview as well but still not moving out.
One more thing I found today is When I try to drag droppable view named "view1" to the cell it was not moving but when once a cell is dragged to view, then view starts moving but not before than that.
I kept two views and try to drag the image in each other but it was not working.
I want to drag the image from one view to another view.
Please guide me to achieve it.
Thanks,

Hello KevinZhouRafael
For reference I have created one image showing design, I wanted to work with drag drop.

I want to drag innermost view to dragdrop view with white background kept at the top. But I am not able to drag outside of dragdrop collectionview that is inner one. I have given drag drop collectionview class to both collection view.
Please guide me for this.
Thanks,
Zalak

Please set drag drop collectionview class only to Dragdropcollectionview.
Don't set it to Dragdropcollectionview1.
2017-10-31 12:58 GMT+08:00 ZalakSA<EMAIL_ADDRESS>
Hello KevinZhouRafael
For reference I have created one image showing design, I wanted to work
with drag drop.
[image: screen shot 2017-10-31 at 10 25 22 am]
https://user-images.githubusercontent.com/14737021/32208050-01450582-be26-11e7-93c7-f99cbed3ead6.png
I want to drag innermost view to dragdrop view with white background kept
at the top. But I am not able to drag outside of dragdrop collectionview
that is inner one. I have given drag drop collectionview class to both
collection view.
Please guide me for this.
Thanks,
Zalak
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/KevinZhouRafael/DragDropiOS/issues/4#issuecomment-340660281,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APiwPhIZ_nf4Lk3mkWh-Zr0kFwDex52eks5sxqkOgaJpZM4QI5Ad
.


Bump ginkgo & gomega

What this PR does
This may or may not help with weird issues running our test suite outside of CI, where we sometimes see different set of executed tests being run on an identical setup.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/approve
/cc @dhiller @brianmcarey

/cherrypick release-1.2


cannot access com.trueaccord.lenses.Updateable in java

When I try to build the scala project to a jar for my java application.
It shows that "cannot access com.trueaccord.lenses.Updateable " in my Intellij IDE.  But there is actually no problem on compile & run time.
Did you know what is the root cause of this problem?

I'm unable to reproduce this. There's nothing special about this class. Maybe you can try to invalidate IntelliJ's cache?

Tried to invalidate the intellij cache but with no luck.
Can you help me to take a look is there any problems if I defined the class like this?
final case class BasicPortfolios(
val accountId : scala.Option[scala.Int] = { /* compiled code */ },
 val accountType : scala.Option[scala.Predef.String] = { /* compiled code */ },
val balance : scala.Option[scala.Double] = { /* compiled code */ },
val equity : scala.Option[scala.Double] = { /* compiled code */ },
val unrealizedPl : scala.Option[scala.Double] = { /* compiled code */ },
val unrealizedPlPips : scala.Option[scala.Double] = { /* compiled code */ }
) extends scala.AnyRef
with com.trueaccord.scalapb.GeneratedMessage
with com.trueaccord.scalapb.Message[com.plugin.model.protobuf.portfolio.BasicPortfolios]
with com.trueaccord.lenses.Updatable[com.plugin.model.protobuf.portfolio.BasicPortfolios]
with scala.Product with scala.Serializable
 {
  val serializedSize : scala.Int = { /* compiled code */ }
  def writeTo(output : com.google.protobuf.CodedOutputStream) : scala.Unit = { /* compiled code */ }
  def mergeFrom(__input : com.google.protobuf.CodedInputStream) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getAccountId : scala.Int = { /* compiled code */ }
  def clearAccountId : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def withAccountId(__v : scala.Int) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getAccountType : scala.Predef.String = { /* compiled code */ }
  def clearAccountType : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def withAccountType(__v : scala.Predef.String) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getBalance : scala.Double = { /* compiled code */ }
  def clearBalance : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def withBalance(__v : scala.Double) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getEquity : scala.Double = { /* compiled code */ }
  def clearEquity : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def withEquity(__v : scala.Double) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getUnrealizedPl : scala.Double = { /* compiled code */ }
  def clearUnrealizedPl : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def withUnrealizedPl(__v : scala.Double) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getUnrealizedPlPips : scala.Double = { /* compiled code */ }
  def clearUnrealizedPlPips : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def withUnrealizedPlPips(__v : scala.Double) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  def getField(__field : com.trueaccord.scalapb.Descriptors.FieldDescriptor) : scala.Any = { /* compiled code */ }
  def companion : com.plugin.model.protobuf.portfolio.BasicPortfolios.type = { /* compiled code */ }
}

object BasicPortfolios extends scala.AnyRef with com.trueaccord.scalapb.GeneratedMessageCompanion[com.plugin.model.protobuf.portfolio.BasicPortfolios] with scala.Serializable {
  implicit def messageCompanion : com.trueaccord.scalapb.GeneratedMessageCompanion[com.plugin.model.protobuf.portfolio.BasicPortfolios] = { /* compiled code */ }
  def fromFieldsMap(fieldsMap : scala.Predef.Map[scala.Int, scala.Any]) : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  val descriptor : com.trueaccord.scalapb.Descriptors.MessageDescriptor = { /* compiled code */ }
  val defaultInstance : com.plugin.model.protobuf.portfolio.BasicPortfolios = { /* compiled code */ }
  implicit class BasicPortfoliosLens[UpperPB](_l : com.trueaccord.lenses.Lens[UpperPB, com.plugin.model.protobuf.portfolio.BasicPortfolios]) extends com.trueaccord.lenses.ObjectLens[UpperPB, com.plugin.model.protobuf.portfolio.BasicPortfolios] {
    def accountId : com.trueaccord.lenses.Lens[UpperPB, scala.Int] = { /* compiled code */ }
    def optionalAccountId : com.trueaccord.lenses.Lens[UpperPB, scala.Option[scala.Int]] = { /* compiled code */ }
    def accountType : com.trueaccord.lenses.Lens[UpperPB, scala.Predef.String] = { /* compiled code */ }
    def optionalAccountType : com.trueaccord.lenses.Lens[UpperPB, scala.Option[scala.Predef.String]] = { /* compiled code */ }
    def balance : com.trueaccord.lenses.Lens[UpperPB, scala.Double] = { /* compiled code */ }
    def optionalBalance : com.trueaccord.lenses.Lens[UpperPB, scala.Option[scala.Double]] = { /* compiled code */ }
    def equity : com.trueaccord.lenses.Lens[UpperPB, scala.Double] = { /* compiled code */ }
    def optionalEquity : com.trueaccord.lenses.Lens[UpperPB, scala.Option[scala.Double]] = { /* compiled code */ }
    def unrealizedPl : com.trueaccord.lenses.Lens[UpperPB, scala.Double] = { /* compiled code */ }
    def optionalUnrealizedPl : com.trueaccord.lenses.Lens[UpperPB, scala.Option[scala.Double]] = { /* compiled code */ }
    def unrealizedPlPips : com.trueaccord.lenses.Lens[UpperPB, scala.Double] = { /* compiled code */ }
    def optionalUnrealizedPlPips : com.trueaccord.lenses.Lens[UpperPB, scala.Option[scala.Double]] = { /* compiled code */ }
  }
  final val ACCOUNT_ID_FIELD_NUMBER : scala.Int = { /* compiled code */ }
  final val ACCOUNT_TYPE_FIELD_NUMBER : scala.Int = { /* compiled code */ }
  final val BALANCE_FIELD_NUMBER : scala.Int = { /* compiled code */ }
  final val EQUITY_FIELD_NUMBER : scala.Int = { /* compiled code */ }
  final val UNREALIZED_PL_FIELD_NUMBER : scala.Int = { /* compiled code */ }
  final val UNREALIZED_PL_PIPS_FIELD_NUMBER : scala.Int = { /* compiled code */ }
}

portfolios.getAccountId(); // intellij will prompt "cannot access com.trueaccord.lenses.Updateable"

but it can still compile and run the java application.

It sounds like the way you produce the jar doesn't make intellij fetch the sources for Lenses. Maybe you can manually add lenses as a dependency of your project?

I found that the root cause is Intellij can't access scala package object, would you have any suggestion?
i.e. can't access com.trueaccord.lenses.Updatable normal in any JAVA file.

Hi Gary,
I am unable to reproduce this. Can you share complete instructions on how
to trigger this issue? It sounds like an IntelliJ bug, not a Lenses bug.
On Wed, Jun 3, 2015 at 8:16 PM, Gary Lo<EMAIL_ADDRESS>wrote:

I found that the root cause is Intellij can't access scala package object,
would you have any suggestion?
i.e. can't access com.trueaccord.lenses.Updatable normal in any JAVA file.
—
Reply to this email directly or view it on GitHub
https://github.com/trueaccord/Lenses/issues/2#issuecomment-108701612.

--
-Nadav

Closing due to inactivity.

I have the same problem - also tested with a minimal configuration.
Intellij says: cannot access com.trueaccord.lenses.Updateable in java, but everything compiles just fine.
I'm using   "com.trueaccord.scalapb" %% "scalapb-runtime" % "0.5.42" % "protobuf"

Can you send step-by-step instructions on how to get this problem?

new scala-sbt project in Intellij (2016.2.4), with
build.sbt:
name := "CloudTest"

version := "1.0"

scalaVersion := "2.11.8"

PB.targets in Compile := Seq(
  scalapb.gen() -> (sourceManaged in Compile)(_ / "generated-proto").value
)

libraryDependencies ++= Seq(
  // For finding google/protobuf/descriptor.proto
  "com.trueaccord.scalapb" %% "scalapb-runtime" % "0.5.42" % "protobuf"
)

and scalapb.sbt:
addSbtPlugin("com.thesamet" % "sbt-protoc" % "0.99.1")

libraryDependencies += "com.trueaccord.scalapb" %% "compilerplugin" % "0.5.42"

and Person.proto:
syntax = "proto3";
package  test;
message Person {
    string name = 1;
    int32 age = 2;
}

and the java class:
import test.Person.Person;

/**
 * Created by qux on 9/26/16.
 */
public class Main {
    public static void main(String[] args) {
        Person a = new Person("7",7);
        int age = a.age();
    }
}

Thanks! Was able to reproduce this. I think it's because everything in the lenses project is inside a package object. I've just released a new version of it which moves Updatable and other classes outside the package object. Can you add this to your libraryDependencies in build.sbt?
"com.trueaccord.lenses" %% "lenses" % "0.4.7"
If this works, I'll release ScalaPB with dependency on it, so you'll get the new version of lenses transitively (and this line could be removed)

Around 8 hours ago it worked perfect with this additional dependencies.
But now I get compile-errors with the additional dependency (without it works, but intellij errors):
error: cannot access Updatable
...
class file for com.trueaccord.lenses.package$Updatable not found

Maybe it's because I'm using a map in a message:
syntax = "proto3";
package  test;
message Person {
    string name = 1;
    int32 age = 2;
    map<string, string> foobar = 3;
}

So perhaps with a new release of scalapb with the dependency on it all is fine..

Ok, remove the dependency, update ScalaPB to 0.5.43, clean in sbt, and let's see if it helps.

Works like a charm as far as I see.
No compile or Intellij errors :)
Thanks!

Awesome!


Master into beta

Description
Updates beta with master. We don't want any changes from master though, but the commits need to be added
Breaking Changes

JIRA Link

Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 15.0.0-beta.7 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 15.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(docs): Use polars instead of pandas

Fix #712, but waiting on #758

Perhaps not relevant for documentation, but please check my comment (since this would be an implementation detail)

https://github.com/probabl-ai/skore/pull/792 is merged.

PR automatically closed because we have rebased the main branch to clean the documentation commits.
Replaced by https://github.com/probabl-ai/skore/pull/820/.

Agreed, this can be closed


Loader UI and mics issues

seems like our prettier is not the same!

@mmahalwy I'm on prettier 1.7.0, yours? and can we lock it to fixed version?

@naveed-ahmad yeah we should. Mine is being weird. Yours is right

rebuild

Deployed to: http://staging.quran.com:32853

@naveed-ahmad lmk when this is ready for review

Deployed to: http://staging.quran.com:32879

Deployed to: http://staging.quran.com:32965

@mmahalwy ready for review and we need to lock prettier version.  Most of changes in this PR aren't mine, and PR looks like a huge things but its not. Very confused about what are real changes and what was result of prettifier :)

@naveed-ahmad would you be able to guide me to the files changed? I can then look at those

@mmahalwy closing this in favour of https://github.com/quran/quran.com-frontend/pull/963


Detected invalid timing cache, setup a local cache instead

docker ： nvcr.io/nvidia/pytorch:21.05-py3
Driver Version: 470.57.02
CUDA Version: 11.4
TensorRT 7.3   TensorRT 8
model ：https://github.com/JDAI-CV/fast-reid/blob/master/MODEL_ZOO.md#veri-baseline
let the VeRi  SBS  model to trt, it fail.
But use the Market1501 SBS ,it success
Both of VeRi and Market1501 are differtent architecture？
all back to non-int8 implementation for any layer consuming or producing given tensor        trt_utils.cpp:252
WARN         TRT: Detected invalid timing cache, setup a local cache instead        trt_utils.cpp:252
WARN         TRT: No implementation of layer 417 obeys the requested constraints in strict mode. No conforming implementation was found i.e. requested layer computation precision and output precision types are ignored, using the fastest implementation.        trt_utils.cpp:252
WARN         TRT: No implementation of layer 419 obeys the requested constraints in strict mode. No conforming implementation was found i.e. requested layer computation precision and output precision types are ignored, using the fastest implementation.        trt_utils.cpp:252
WARN 	TRT: Detected invalid timing cache, setup a local cache instead	trt_utils.cpp:252
WARN 	TRT: No implementation of layer 417 obeys the requested constraints in strict mode. No conforming implementation was found i.e. requested layer computation precision and output precision types are ignored, using the fastest implementation.	trt_utils.cpp:252
WARN 	TRT: No implementation of layer 419 obeys the requested constraints in strict mode. No conforming implementation was found i.e. requested layer computation precision and output precision types are ignored, using the fastest implementation.	trt_utils.cpp:252
ERROR	TRT: instanceNormalizationPlugin.cu (292) - Cudnn Error in enqueue: 3 (CUDNN_STATUS_BAD_PARAM)	trt_utils.cpp:251

This error is about instanceNorm which is not supported in tensorRT.

How can I solve it ？
Do you have the advance？

If I change the backbone without ibn, is it better

How can I solve it ？
Do you have the advance？
If I change the backbone without ibn, is it better

You can refer to projects/fastRT which has instance norm  plugin. Or you can just use backbone without ibn.


[1.12.2] Wolf equipment does not display on spirit wolves

Issue Description
Spirit wolves from Electroblob's Wizardry equipped with Wolf Armor and Storage's items do not display armor or chests on their model.
Relevant Mods
Electroblob's Wizardry
Reproduction Steps
Steps to reproduce the behavior:

Summon a Spirit Wolf, either by using a wand with the spell equipped, or a scroll.
Equip the spirit wolf with armor or chest.

Expected Behavior
Spirit Wolves equipped with armor or storage should reflect this on their model.
Screenshots / Videos

Version Information:

OS: Windows 10
Java: Java 8
Minecraft Version: 1.12.2
Mod Version: 3.7.5
Forge Version: <IP_ADDRESS>54

Additional Context
Visual only; armor and storage work as expected otherwise. Bug reproduced in a Minecraft instance with only Wizardry and Wolf Armor and Storage installed.

I have the same issue, but with TFC wolves.

I'll investigate adding compatibility layers for these mods.

I'll investigate adding compatibility layers for these mods.

Well... I got the compatibility layer working, but there is a GL blend issue somewhere which is causing the wolf armor layers to glow when Wizardry is installed. Been banging my head on it for like six hours or so tho and getting nowhere, so I will open a new issue for the glowing layers

Well... I got the compatibility layer working, but there is a GL blend issue somewhere which is causing the wolf armor layers to glow when Wizardry is installed. Been banging my head on it for like six hours or so tho and getting nowhere, so I will open a new issue for the glowing layers


Update README.md with links and how to

Include links to GitHub repo and Docker Hub image (right now it's not very easy to get to the Docker Hub page from the GitHub repo).
Also a simple "How to use" section with some copy-paste-able image name / command.

I'm no longer working on this. I'm currently cleaning up my forked repos, I just wanted to leave this note here before removing this branch and my fork. :nerd_face: :coffee:


[Android] App freeze with multiple issues Video attached

Ledger Android 2.9 version Android multiple issues
Device redmi note 8 pro
Issues
App freeze on startup
After typing password for login no response
Delay in finger print popup
Even if Fingerprint popup it freezes not recognize the finger
Video upload below
https://mega.nz/file/jZlxySAJ#D41-FZ65qvemAtMuVAIFwUvqpEnP26jeQxZ205dQeEU

@gre please have a look


More plugin cleanup

The pack/extra_syntax_files directory just needs to be merged into plugins. There's too blurry of a line between a "syntax" plugin, and a "feature" plugin, and I can never remember which directory things are in.

Done with https://github.com/evanthegrayt/vimfiles/commit/c0e838786a912e3d6cf6b5fd6f2e9eb4f67934e7


GenericBioengineApp: error with

Summary
when trying out different models, I found that fearless-crab will throw an error when used with this example data (courtesy of the Gerlich lab). I'm not sure whether the data is wrong, or something else got wrong. It showed a pretty long traceback...
Details
Traceback

Failed to run the model, error: Traceback (most recent call last):
  File "/models/bioengine-model-runner/1/model.py", line 236, in start_model_worker
    output_tensors = pred_pipeline(*input_tensors)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_prediction_pipeline.py", line 111, in __call__
    return self.forward(*input_tensors)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_prediction_pipeline.py", line 165, in forward
    prediction_tensors = self.predict(*list(input_sample.values()))
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_prediction_pipeline.py", line 135, in predict
    return self._model.forward(*input_tensors)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_model_adapters/_model_adapter.py", line 56, in forward
    self.load()
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_model_adapters/_model_adapter.py", line 41, in load
    self._load(devices=devices or self.default_devices)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_model_adapters/_tensorflow_model_adapter.py", line 66, in _load
    self._model = self._load_model(weight_file)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/bioimageio/core/prediction_pipeline/_model_adapters/_tensorflow_model_adapter.py", line 32, in _load_model
    return tf.keras.models.load_model(weight_file)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/tensorflow/python/keras/saving/save.py", line 212, in load_model
    return saved_model_load.load(filepath, compile, options)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/tensorflow/python/keras/saving/saved_model/load.py", line 130, in load
    _read_legacy_metadata(object_graph_def, metadata)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/tensorflow/python/keras/saving/saved_model/load.py", line 179, in _read_legacy_metadata
    node_paths = _generate_object_paths(object_graph_def)
  File "/tmp/python_env_qT5KJf/0/lib/python3.8/site-packages/tensorflow/python/keras/saving/saved_model/load.py", line 204, in _generate_object_paths
    for reference in object_graph_def.nodes[current_node].children:
IndexError: list index (0) out of range

Screenshot

Thanks for the report, this is because we only has tensorflow 2 installed in the BioEngine, but the model need tensorflow 1.15.
I will need to disable BioEngine for this model for now: https://github.com/bioimage-io/bioengine-model-runner/commit/57fc0f1409d742d3baef4fed1056ca7680cf41c3

k, thanks. Are any models in bioimage model zoo with tensorflow 2.x? It is my understanding that this is not the case. Why not put the proper tensorflow version on the server?


Ruby on Rails capstone project - Budget app

Hi there 👋,
In this particular project, I :
Design

- Followed these [design guidelines](https://www.behance.net/gallery/19759151/Snapscan-iOs-design-and-branding?tracking_source=) including:
  - Colors.
  - Typography: font face, size and weight.
  - Layout: composition and space between elements.

NOTE: In these design guidelines there are several UIs that I won't need for this exercise; also, some pages are not given a design and I create them following the design guidelines of the other pages.

Original design idea by Gregoire Vella on Behance.
I gave an appropriate credit to the author in the README of this project.
Interactions

Splash screen

A simple page with the name of budget app, and links to the sign up and log in pages.

Sign up and log in pages

The user should be able to register in the app with full name, email and password (all mandatory).
The user can log into the app using email and password.
If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

Home page (categories page)

When the user logs in, they are presented with the categories page.
For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that category.
When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.
There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

Transactions page

For a given category, the list of transactions is presented, ordered by the most recent.
At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions in that category).
There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new category" page

The user fills out a form to create a new category, indicating their name and icon (both mandatory).
The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new transaction" page

The user fills out a form to create a new transaction with:

name (mandatory)
amount (mandatory)
categories (mandatory at least one)

The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that category.
When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.

Testing requirements

I create unit and integration tests for all the most important components of my RoR application.

Technical requirements

I use Postgres as your database.
I use devise for authentication.
I validate all user input to make sure that anyone with bad intentions cannot compromise the app.
I use (.erb) as a view template engine.
The project is deployed and accessible online.
My database schema reflect the following structure:

NOTE: I do not change column names visible in the diagram. I change the "Entity"  name according to the theme I have chosen for my project (please note: "Transaction" is a name already used by ActiveRecord, so using it as a name for my model and table will result in an error).

Live Demo Link
Live Demo Link
Video for explanation

@HAMMAS-SALEEM Thank you very much for all the corrections you have made that have helped me improve my project. Thank you immensely.


FR: allow-license:xxx if check-license presented

Maybe a whitelist makes more sense. Otherwise I have to constantly monitor the used licenses and include the ones in the disallow-license list. Default would be allow anything, once the setting is enabled only licenses that are part of the defined list are allowed.

@shochdoerfer : sound great

Implemented!

That was faster than I could have a look and I like it! :D
But also a bit sad about composer that things are not nested like
"white-list:not this but..."
"white-list": [
  "this",
  "yay",
]

Yep, json is not allowing to do
{
    "string-setting", /* neds key, and `"setting": "setting"` is silly */
    "array-setting": [ ... ]
}


php code syntax highlighting does not work in markdown files

VSCode Version: 1.20.0

OS Version: OSX 10.13.3

Steps to Reproduce:

open empty editor
set file type to markdown
add php code snippet

observe php syntax highlighting does not work
add additional code snippets for other languages

observe syntax highlighting works

Does this issue occur when all extensions are disabled?: Yes/No
Yes.
c.f.: twitter thread with VSCode support

This issue was moved to Microsoft/vscode-markdown-tm-grammar#9


Group the resources by type

What do you think about group the resources by type?
e.g.
### Videos
...

### Articles
...

### Resource
...

Before I built it with markdown tables and one of the columns was the type of content (video/article/paper/etc). It started to get difficult to manage the rows so I replace them with plain links.
Now, this repo has the problem of not knowing if the resource/link is a video, article, or any type of content.
Although your idea is super valid, I also group them by a more granular topic like in the core-web-vitals, where I separate the links by topic types: CLS, LCP, etc. In measuring-performance, I also have granular topics like tools, benchmark, runtime perf, more general resources, etc.
So, I started to think if it would be easy to understand the type of content by an icon, like in the sustainability category. I also thought of adding a [topic] at the beginning of each link too, like [Article] Website performance and the planet.
I'm open to discussing it.


HDDS-6965. Increase timeout for basic check

What changes were proposed in this pull request?
basic (unit) check takes 45+ minutes, and sometimes even times out after 60 minutes.  It looks like Github runners have variable performance.  Since we have more tests now, I think it is reasonable to increase the timeout.
https://issues.apache.org/jira/browse/HDDS-6965
How was this patch tested?
Regular CI:
https://github.com/adoroszlai/hadoop-ozone/runs/7087566467
basic (unit) check succeeded in 1h 1m 44s, so it would have failed prior to this change.

Thanks @ayushtkn and @kerneltime for the review.


Version v0.2.0-beta.3 causes Home Assistant loading issues

Upon configuring version v0.2.0-beta.3 successfully, when the integration is running within HA (not HACS), Lovelace will become stuck on a loading page upon refreshing the web browser or HA app. Disabling the alarmdotcom integration fixes and resolves the loading issue. Re-enabling causes the issue to return.

This has now been resolved in v0.2.0-beta.4.


Beerpay goes to blank page for me

I think the link to the beer page is broken....:(

Actually, I wanted to stop asking for support, I can't manage to find time or motivation to work on this project since many months. I don't think this is really honest to ask for support while working on this.
But thank you for your help. 🙂


feat(Combinatorics/SimpleGraph): add miscellaneous lemmas

I noticed that many of these lemmas are in #15711, so I think I'll mark it as a dependency

Actually, #15711 is very big and unfocused, so I would rather mark it as depending on your PR, rather than the other way around.

Alright. @meow-sister, are you OK with that?

Alright. @meow-sister, are you OK with that?

Sorry for reply lately. My PR would changes a lot in coding style and naming convention before merged.

Does that mean you are happy with the current PR (#16294) being merged before yours (#15711)?

Does that mean you are happy with the current PR (#16294) being merged before yours (#15711)?

In my opinion, the merging order is not so important.

Okay. I have turned the dependency around

The rest looks good
maintainer merge

This seems to have slipped off the queue
bors r+


Add to Cart

Add to Cart functionality throws an internal server error.could you please Look into this?

Now fixed the issue.successfully added the product in cart.

Thank you for fixing this issue. It is now working fine.


Use $lib/server instead of $lib for db and auth

I am new to SvelteKit so I am not entirely familiar with the consequences, but shouldn't the db and auth modules be in $lib/server instead of just $lib?
https://kit.svelte.dev/docs/modules#$lib-$lib-server
The Lucia docs do it that way too:
https://lucia-auth.com/getting-started/sveltekit/

Seems like a good idea to move them!


fix(mitreLimit): Rename wrongly named mitre limit param

This was wrongly called mitre_level. It is currently called mitre_limit in the PostGIS documentation.
See https://postgis.net/docs/ST_OffsetCurve.html

[!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

#114  👈 (View in Graphite)
#113
#112
main

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


Added command to generate random hex color

Added command to generate a random hex color, formatted "#abc123"

Thanks again! It would be great if you could update the README as well

No problem. I've made the requisite changes. Let me know if this works.

@irwinj Great, Thank you!


feat(default-theme): added custom form styles file

Changes
Added new common file with default forms styles
Moved styles from other components with forms to the new one
closes #984

Checklist

[x] I followed contributing guidelines

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: patzick:x: MalwurfYou have signed the CLA already but the status is still pending? Let us recheck it.


AZ Pull Request 10

https://twitter.com/EverdreamValley/status/1712423541763625391

As a reminder Vibrant Venture would need to support both domain names


Vite js server keep running after stopping or restarting app

I'm having an issue with the package usage.
When I start my ASP .NET Core MVC project by setting the middleware to autorun vite dev server, the middleware works fine.
But when I stop my project or restart it, the vite dev server instance is not stopped.
So I end up with several instances of the vite server.
Would it be possible when stopping the asp core server that the vite server stops.
Thank you for your work on this package, it allows to integrate quickly and easily to mvc projects

Hello friend.
Hmm. That's strange. The vite command is supposed to be closed since it's part of the same instance.
Are you using the Ctrl + 5 and F5 commands together while using Visual Studio IDE? Both processes run separately, so all other terminals that Visual Studio opens must be closed in order for the vite dev server instance to close.
If this is not your case, please provide more details to try to reproduce it.

Hello @TanguyPa. It seems that this problem is something common when starting node processes.
It may not be able to be fixed soon. Anyway, I'll keep checking it to see if I find a solution.
In the meantime, avoid using IIS or start Dev Server externally, without AutoRun option.

My guess is the AppPool in IIS never truly "shuts down" so the parent process is maintaining a connection to the Vite Server and keeping it alive.
I think if this project wants to support IIS, it might need to hook into the IIS lifecycle events.

I get a similar issue when running from the command line with dotnet watch run. If I restart the app with Ctrl+R a time or two the Vite dev server is not always closed. Sometimes it is, sometimes it isn't. Have yet to see a pattern, but it might not be directly related to IIS hosting.

Also seeing this issue on Windows and also with dotnet watch.

How are you folks running dotnet watch ?
Try running it in dotnet watch --no-hot-reload as that will close the process down and then restart it. I'll give it a try and see if I can recreate the issue.

👋 Hi, folks. I found this line in Oleksii Holub's (@Tyrrrz) CliWrap project.
I can try setting CreateNoWindow to true to see if that helps. I would guess that the child process might be attached to the host console (Hot Reload), and thus it doesn't exit.
It's worth testing out, but not sure if it will solve the Windows issues.
https://github.com/Tyrrrz/CliWrap/blob/3eb76065612d38543b19aae37cb0e2b85bd7d8dc/CliWrap/Command.Execution.cs#LL80C1-L85C34

I just submitted a PR and tested it on Windows.
I was able to recreate the issue with IIS Express, but could not track down the issue.
As an interim fix, the code now checks to see if the port is occupied, if it is, it will close the old process and start a new one. This should work in general, including in scenarios with dotnet watch.

The main problem is that the vite script keeps running until you close the console. It happens when the project is run using IIS, dotnet watch and generally from the Visual Studio IDE.
I just tested The PR #18 from @khalidabuhakmeh. His solution might work provisionally, but I have my doubts about killing the port because what if that port is used by a process other than Vite? Wouldn't it be a trouble if we just kill it?

The React template for SPA projects runs the React development server in a second console, and that process does stop if the project does too. I haven't figured out how it works yet but I think the full solution might be close that way.

@Eptagone I agree with your points, you didn't have to merge my PR. 😅
But I want to address your points from another perspective.

What if that port is used by a process other than Vite?

Another process might use the port, but it's the port the user has in their configuration/setup. We can assume that they picked it with intentionality.

Wouldn't it be trouble if we just kill it?

It had it coming. THIS IS OUR PORT! (just kidding). The user can probably restart those processes, and knowing there is a conflict, they can change their environment. This is development, after all.
We may want to add in the README that IIS Express and .NET Watch might work in a way that doesn't kill the parent process, so folks using those environments might want to consider not using the auto-start functionality or know that we'll restart those processes for them.

Ups. Well, anyway, I think it may be a temporary solution.
I'll add an option like KillUsedPort so the user can choose if he wants to kill the busy port.

Thanks @austins. This is finally fixed with the 5bb0a5aff1904fe57c3bca54828fedd6ec8ebdf4 commit.


[GH-43213] Fix duplicate comments printed in quick info section

Fixes #43213
Only push values to the comments array if it's unique (similar to how we do it for declartation comments in         let doc = JsDoc.getJsDocCommentsFromDeclarations)

@sandersn
I've addressed your comments.

I removed the forEachUnique since it wasn't really doing much in this case and replaced it with a normal for each loop.


Un-elide App component details in example

Closes https://github.com/ReactTraining/react-router/issues/3958

Again, you shouldn't be relying on this one example. This is covered elsewhere in the docs.

@timdorr what's the purpose of eliding the details of App anyway?

Keeps the example brief. Just rendering children doesn't show off the router at all.

@timdorr You can choose to keep it, I believe I'm not the only one to have been confused by this.

You shouldn't be relying on this one example. This is covered elsewhere in the docs.

One reason that I did rely on this example is that the documentation for this project is, in my opinion, very poor. I could not find a short document describing how to use react-router. Here is where I tried to find it.
On the readme, the first two links look like they could be what I'm looking for.

The tutorial is not what I'm looking for, because it assumes I will clone it and run some code, and it is not short. It waits until lesson 2 for any useful information

On to the second link. This goes to another repo with a readme.

The first link is to the tutorial.
The second link starts by describing something that is not react-router, making it unnecessarily long (for my purpose). It only starts describing react-router in the comments of the code example here (https://github.com/ReactTraining/react-router/blob/master/docs/Introduction.md#with-react-router). There are not many comments and it assumes I read the non-react-router part of the document.

the imprecise phrase "figure out the children" is used. The reader is left to infer (perhaps from the non-react-router example) that this.props.children is set to the matching nested route.
Onto the next link.

This at least has some useful information, but it only vaguely defines what a route config is.
Compare this to the ember router docs, at https://guides.emberjs.com/v2.8.0/routing/
It very clearly defines what Ember router is - it maps urls to route handlers, and defines route handlers.

The next page clearly states how to declare routers, by calling the map function

etc. The documentation is about as short as it can be.


RBF default epsilon too large 2020

This PR was prompted by issue #4523 describing instability in RBF interpolation with noisy data and a user-chosen value of epsilon which was too large for the data interpolated, at least it reminded me that I had posted a bug report #2020 some years ago, and since it was never addressed I thought I should fix it.
The code change is in the calculation of the default epsilon for RBF interpolation which seem to have been in error due to a misinterpretation of the documentation on which it was based.  New unit tests illustrate the behavior described in #4523 (while being deterministic), fail with the previous implementation and pass with the new.

Thanks for the contrubution!
I've left a few minor style comments inline. One other style request is to squash your commits.
A bigger question is --- in #2020 you refer to some documentation, can you give a more specific reference please? I did not manage to find a ppt on the matlabcentral site.
I had only a cursory look at the rbf.py code and it seems it could use a bit of work. Pauli left some specific suggestions in #4523, would you be interested in looking into it? IMO breaking backwards compat is better be done in one go (not necessarily one PR, just one release)

The ppt is in the zipfile download at Mathworks, top-right link (I would have attached it here, but GH does not support such).
Re the rework of the whole file: agreed, it does need some work, and probably more than the suggested changes.  I'll look at raising a PR after this one (clearly those changes are outside the scope of this PR).

Requested changes implemented.

I don't have any significant comment but I noticed that
ximax = np.amax(self.xi, axis=1)
ximin = np.amin(self.xi, axis=1)
ximax - ximin

could be changed to
np.ptp(self.xi, axis=1)

A bigger question is --- in #2020 you refer to some documentation, can you give a more specific reference please? I did not manage to find a ppt on the matlabcentral site.

@ev-br The reasoning in #2020 seems pretty intuitive to me.  If you want RBF to approximate a function using something like a blend of its values at 'nearby' points, then it makes sense to care more about the expected distances from points to their neighbors than from points to random points, for the purpose of heuristically determining a default value of 'nearby'.

OK, here's a rebased branch: https://github.com/ev-br/scipy/tree/pr/4643
There are a couple of style tweaks (commit message prefix, docstrings in test cases), and I've added a test case from the original bug report with a random seed so that it's deterministic.
Also used xlogy in the thin-plate kernels to avoid RuntimeWarnings in 0*log(0).
@jjgreen can you take over https://github.com/ev-br/scipy/commit/aaa70fc2f0704641a944090d23925357d1789cd8. Then this is PR can be merged with the green button.

And could you also take over 3847fe3825f0185a028f9c22dd9a9090e98756f1 which fixes  https://github.com/scipy/scipy/issues/1885
(These RuntimeWarnings were coming from 0*log0 and are fixed by using special.xlogy, it looks like)

Could you please let me know when you've finished adding stuff, I just spent a hour futzing around working out how to merge you changes into my branch, now to find out that you have just obsoleted that work.

The push after commit 443c2d8 is purely to test the clhanges (due to a recent change in the version of Cython required for a build I can no longer test locally).  A rebased commit will follow a sucessful test-run.

Whoa. I was trying to be helpful, and I'm surprised I've obsoleted your work.
Anyway, 443c2d8 is not necessary, having both tests is fine. You can either keep it or remove it. My preference would be to keep both, but it's up to you.
I tested the branch locally, as soon as TravisCI tests are done, it's ready to go in IMO.
Cython related failures can be removed by git rebase upstream/master.
And here's a how to stuff into your branch: https://github.com/scipy/scipy/pull/4500#issuecomment-73437775
(thanks Pauli)

Apologies for the irritability, I had 2 attempts at merging from pr/4634 with merge conflicts, then the email about ev-br@3847fe3,  before I found out I should have been cherry-picking,

BTW, my cython issues are that I have cython 0.21, and there are no debs for 0.22 available yet.

(off-topic for this PR) If you have suggestions for https://github.com/scipy/scipy/issues/4334, it'd be welcome.
I'm nearly sure something along the lines of pip install cython --upgrade worked for me.

That should work, Cython is pretty easy to install with pip. At least I've never seen it go wrong (apart from connection timeouts).

[off topic]
Cython is pretty easy to install via pip if you like living dangerously, sudo pip install .. downloads stuff off the internet and then immediately runs a script that you have no opportunity to inspect as root.  Maybe that will work out OK :-).  To do it safely you need a --user and then the result is in ~/.local/bin/ which you add to your path or symlink the executable to user/local/bin and make sure that you keep an eye on the buglists for the package in case something comes up and ....
Repeat with `gem

@jjgreen true, I also wait for Ubuntu packagers in most cases. But once I'm building Scipy, then pretty much anything else in Python land is easy by comparison - so then I tend to avoid OS packaging completely.

Okay, Travisbot builds all passed, let's put this in. Thanks @jjgreen! Looking forward for your next contributions!
For future PRs (no action asked for at the moment), its better to try to preserve commit authorship, and there is no requirement to squash everything into a single commit :-).


Prabhakar dataset (ICCP19)

Hello, I have a question about Prabhakar dataset (ICCP19).
I've downloaded Prabhakar dataset from the link you provided,
but there were several sets without ground truth HDR image and some of the directories were not aligned. (GT file or txt file exists in different folders in each set.)
I wonder if you could check the dataset.
(I'm writing here since I couldn't find the github for your ICCP19 paper.)
Thank you!

Thank you for your interest in our work!
We have provided a script to prepare and structure the downloaded ICCP19 dataset in the "dataset" folder. Please refer to the "Dataset Preparation" section of the README for instructions on preparing the downloaded dataset using the script.

Hi, the link of  Prabhakar dataset is not valid. Could you please to provide a new web link to download ?

Updated the link in the README.


Can't purchase domain on test API

Hey when I try to purchase a domain via the testing url endpoint, I get an error; "missing bill_to_email". Has anyone experienced this?
Array[DEBUG] HTTP Response body ~BEGIN~
{"code":"-1073456591","message":"[P3OWGDCOMM002] {gdBasketPurchase.PurchaseBasketXML.1}BasketPurchase has invalid/missing bill_to_email ","name":"ApiError"}
~END~

getting same error :disappointed:

Same here.

Just so you guys know, this is an api error on godaddy's side of things.

yes , @anderson-arlen  is right, i tried to talk with support team, and they were unable to resolve it ,
they wasted two weeks , i had to scrap my project, !

That's super weird, isn't this one of their CORE functionalities? How are they overlooking this and decline to put any resources to deal with the issue?

I actually moved on from Godaddy after an attempt at contacting support and getting nothing.

Turns out the API keys generated on developer.godaddy.com are no good for buying domains. If you generate a key on reseller.godaddy.com, it works. Or you can use the same test key that the live documentation uses: sso-key UzQxLikm_46KxDFnbjN7cQjmw6wocia:46L26ydpkwMaKZV6uVdDWe

Hm, I can't get the available() method to work with this key-secret pair
you provided. Is this just for the purchase() method?
On Mon, Nov 13, 2017 at 7:19 PM, Arlen Anderson<EMAIL_ADDRESS>wrote:

Turns out the API keys generated on developer.godaddy.com are no good for
buying domains. If you generate a key on reseller.godaddy.com, it works.
Or you can use the same test key that the live documentation uses: sso-key
UzQxLikm_46KxDFnbjN7cQjmw6wocia:46L26ydpkwMaKZV6uVdDWe
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gellu/godaddy-api-client/issues/2#issuecomment-343991240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGqklpWfdAKG2GA_QZS6sHC8ZsXbeUq0ks5s2Ho5gaJpZM4OJZXl
.

sorry, that's the complete auth header. They key pair is:
client id: UzQxLikm_46KxDFnbjN7cQjmw6wocia
secret: 46L26ydpkwMaKZV6uVdDWe

Yeah, I got that, I just get 401 Unauthorized when I use those client id
and secret. Did you actually get it to work with those?
On Tue, Nov 14, 2017 at 12:09 AM, Arlen Anderson<EMAIL_ADDRESS>wrote:

sorry, that's the complete auth header. They key pair is:
client id: UzQxLikm_46KxDFnbjN7cQjmw6wocia
secret: 46L26ydpkwMaKZV6uVdDWe
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gellu/godaddy-api-client/issues/2#issuecomment-344075592,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGqklpNxi0Qj68eJJJZagfT7b_RQEjOMks5s2L4RgaJpZM4OJZXl
.

Getting same Error


screen share wrong resolution. triple monitor setup with pivot

Hi all, there is an issue with screen sharing. I have 3 monitors in the following setup:

1: 2560x1080; 2: 2560x1080; 3: 1080x1920
I usually put the edumeet on the monitor 3 - portrait orientation.
When I click  - on monitor 3 - to the share my screen button ( I want to share monitor 1 or 2). the resolution drops down to 1080x456 resolution.
So if I want to share it in native resolution, I need to move the chrome window to monitor 1 or 2 and click on the share button there, and the resolution will be to the proper 2560x1080. ( after that I can move back the chrome window to monitor 3 and the resolution stays 2560x1080)
Google Chrome | 87.0.4280.88 (Official Build) (64-bit) (cohort: Stable) (there is an error in normal, and also in private mode)
Microsoft Windows [Version 10.0.19041.685]
letsmeet.hu but the error is also on letsmeet.no
There is no error if I use firefox.
From curiosity, I tested it in Ms Edge, and the problem appears, same as Chrome  - not so surprising because of the same engine - Microsoft Edge | 87.0.664.60 (Official build) (64-bit)

@nullaegy
This is the chromium engine's bug and there is no official solution yet.
See: https://bugs.chromium.org/p/chromium/issues/detail?id=1056311
I tried to fix this bug with different methods, but I couldn't find a solution.

Thank you for your effort! I'll subscribe to that bug and we'll see what happens. I'll close this issue.

I think we should keep it open until it will be fixed somehow.
I think we should also check what can we do if we detect resolution, and set advanced constraints according it.
Maybe we can add constraint to getDisplayMedia and this way make a workaround.


[changelog] add possibility to overwrite previous tag Resolves #382

Resolves #382
Context

Checklist

[ ] Review Contribution Guidelines.
[ ] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Update documentation when applicable.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you @andreabettich. You found most places to make the required changes, only missing updates for the Maven plugin. The following types must be updated as well:
https://github.com/jreleaser/jreleaser/blob/ba1ed9b50d2921789b9b95cdb4b583321d1a5504/plugins/jreleaser-maven-plugin/src/main/java/org/jreleaser/maven/plugin/GitService.java
https://github.com/jreleaser/jreleaser/blob/ba1ed9b50d2921789b9b95cdb4b583321d1a5504/plugins/jreleaser-maven-plugin/src/main/java/org/jreleaser/maven/plugin/JReleaserAutoConfigReleaseMojo.java
https://github.com/jreleaser/jreleaser/blob/ba1ed9b50d2921789b9b95cdb4b583321d1a5504/plugins/jreleaser-maven-plugin/src/main/java/org/jreleaser/maven/plugin/internal/JReleaserModelConverter.java

I fixed the formatting and added the missing parts. Sorry for already amending and force-pushing onto the branch. makes it less transparent what I changed.

Thank you. Force push is OK. I'll do one full review again and then we'll be good to go.
If you feel like doing so, you could also send a PR to jreleaser/jreleaser.github.io for updating configuration snippets. If not that's OK, I can do that later 😄

Added some documentation https://github.com/jreleaser/jreleaser.github.io/pull/16

Thank you for your contribution! 🎉
You don't have to wait til v0.8.0 is released to test out this feature. You can grab the latest early-access release and invoke the CLI directly. Alternatively the jreleaser/jreleaser-slim:early-access Docket image should work with GitLab CI as well
https://hub.docker.com/r/jreleaser/jreleaser-slim/tags
Just have tom wait til the current early-access build finishes to grab the binaries.

awesome, thanks a lot!


Fixed typescript definitions

Summary
Corrected onPageScrollStateChanged type definition.

Test Plan

What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Checklist

[ ] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

@renjfk could you try to resolve conflict ?


fix xpc_darwin.go avoid cgoCheckPointer

change type
unsafe.Pointer => C.uintptr_t(uintptr(unsafe.Pointer))
ref https://github.com/golang/go/issues/12416
and fix test.

:+1:

:100:


3.2 version no config.py file

previous versions had config.py with contained Configuration class.  This version does not have that file.

Saw notes on its removal.  SOURCES,txt still has file listed.


[WiP] Use python-urlify instead of slugify

To be compatible with urlify.js

Github gone mad
Closed in favor of #4272


Technical House-keeping on Gauss-Electric

Looking good! These are just a couple minor technical house-keeping items:

[x] after each new environment is finished (ie .. math::, after a table or an itemized list) a new line needs to be included (it signals the end of the environment and makes the build more stable)
->>fixed (Dom)

[x] need to attribute / cite the gif (unless it is something you made??)
->>New figure from us (Dom)

[x] for equations you can make the brackets look better when there are large symbols using \left( \right) (ie. equation after 7)
->>Fixed (Dom)

[x] divergence should be \mathbf{\nabla} b/c it is a vector operator
->> Is it me or \mathbf{\nable} doesn't do anything? I did a \boldsymbol{\nabla} instead  (Dom)

@fourndo , @lacmajedrez , @focus-shift, @DanielBild-Enkin

ah, sorry you use \boldsymbol for nabla, nice catch

One more spacing thing: after the math on line 88 there needs to be a new line


Config file for repo settings

Feature Request
When using template, there are now many places that need to be immediately updated to work. For example, package-name, python-version, {username/repo}. Is there a way to write all these only once and propagate the changes?

More info, look at this conditional github workflow action for renaming things:
https://simonwillison.net/2021/Aug/28/dynamic-github-repository-templates/

Look at how cookiecutter works. I think it's designed to replace all {placeholder.names} on repo create.

https://github.com/rochacbruno/python-project-template


Add update mechanism

Commit https://github.com/OCR-D/format-converters/commit/5b9568fd2b6dbfe891ef81826b7fffea7d21d814 was missing in our installation (fixed now). I noticed that just running make all or make install does not update any of the existing cloned code from external git repositories. I think we should start using git submodules to get explicit dependencies.

Originally posted by @stweil in https://github.com/UB-Mannheim/ocr-fileformat/pull/106#issuecomment-570098548

I am not sure that we need git submodules with which a lot of people also struggle to use. Moreover, this would only bee a partial update mechanism because we also directly install from a release (e.g. JPageConverter) or download from a website (e.g. Saxon).
Thus, I think we need a proper update mechanism in the Makefile. This then should check first for new code of ocr-fileformat and taking into account that there are git-installations as well as installations for the latest release. Afterwards possible updated packages from the vendor should be loaded. This maybe can been done with make realclean and then make install again. Maybe there are smarter ways to do it as well w/o downloading potentially the same files again.

Thus, I think we need a proper update mechanism in the Makefile.

Easiest solution: clone the repository depth-1 to a tempdirectory, uninstall, reinstall. It's less efficient, but the cleanest.

I am not sure that we need git submodules with which a lot of people also struggle to use. Moreover, this would only bee a partial update mechanism because we also directly install from a release (e.g. JPageConverter) or download from a website (e.g. Saxon).

Yes, but both can be managed via make. And that's exactly what since has been done.

Thus, I think we need a proper update mechanism in the Makefile. This then should check first for new code of ocr-fileformat and taking into account that there are git-installations as well as installations for the latest release. Afterwards possible updated packages from the vendor should be loaded. This maybe can been done with make realclean and then make install again. Maybe there are smarter ways to do it as well w/o downloading potentially the same files again.

The process for updating is now as simple as:
git pull
make install # will also git submodule update

Can be closed IIUC.


For multi-release modular jars, use the newest module-info available …

…when detecting module name.
This should solve the problem with log4j. Should be cherry-picked to release100, if approved/accepted.
There's one thing that is not completely correct: the problematic method does not know the source level, so it cannot decide which versioned module-info it should use. So it always uses the module-info for the newest platform version, which may not be always correct. But given the method only needs the module name, and changing the module name using multi-release jar feels like a bad idea, this may be fine in almost all cases.

This needs to be cherry picked for release100 branch.


eslint-plugin-tsdoc: set docs url to eslint-plugin/README.md

Just a small quality of life improvement – when a user selects Show documentation for tsdoc/syntax in a supporting editor, this takes them to the GitHub-formatted readme for eslint-plugin-tsdoc rather than the top-level of this GitHub repo. (Same strategy as several other eslint plugins.)

This fix was released with tsdoc version 0.12.19


ARROW-9896: [Go] Rename ipc tool from 'arrow-json-intergration-test' to 'arrow-json'

Minor issue but suggested

Seems the name "arrow-json-intergration-test" is used in many languages. So I close this.


Task 152021892

From fa07094 onwards the goal was to show species in orthologs and to put some order in the old BioentityPropertyService. The code there and surrounding classes was a bit messy.

Build passes tests:

http://bar:8085/browse/ATLAS-BASECIDEVTASKS0-1
http://bar:8085/browse/ATLAS-GXACIDEVTASKS14-1


golang regexp style does not work

Not sure if I understand it correctly, but from the doco - I should be able to use golang regex syntax. Thus these code snippet below should work, however it does not.
local re = require("re")

-- quote
assert(re.quote("^$.?a") == [[\^\$\.\?a]])

local data = "Today is 21/10/2022"

local ptn = "[\d]+\/[\d]+\/[\d]+"
i, j = re.find(data, ptn)
print(i, j)

-- why output nil? Should return match start and end i, j?

-- this case, string.find return correct but not re ? buggy?
print( string.find(data, "%d%d/%d%d/%d%d%d%d") )

print( re.find(data, "%d%d/%d%d/%d%d%d%d") )

the output of the above is
nil	nil
10	19
nil

What am I doing wrong or it is a bug?
Thanks heap for the work.

Realize that lua string - need to escape the \. And this modules wont work with lua regex syntax.
However there is a bug in index - I push a MR to fix it.


How to deal with one time action?

For example, when ACTION_UPDATE dispatched, update method should be runned.
{type:ACTION_UPDATE} -> {shouldUpdate:true} -> update()
That's correct.
But next time when other action dispatched, shouldUpdate remains true, and will trigger update method again.
How to deal with this kind of one time action?
I supposed to do it like this:
{type:ACTION_UPDATE} -> {shouldUpdate:true} -> update() -> {type:ACTION_UPDATE_DONE} ->  {shouldUpdate:false}
But will it be a little complicated? Is there any easier way?

Perhaps dispatch some action in whatever looks for that state to "reset" the variable back?
It sounds more like you need to restructure your state and application code, as temporal state doesn't really work well in redux or React. Ryan Florence gave a good talk about removing time from your state at React Rally: https://www.youtube.com/watch?v=kp-NOggyz54

Thank you, I will take a look at that:)


ASEv2 Zone deployment -- clarification?

Two clarification questions about this page:

If you do not do the steps on this page (eg you deploy an ASE with an ILB through the portal), is the ASE deployed across multiple zones?  If not, what redundancy does it have?  Are the scale units deployed instead across multiple availability sets / fault domains / update domains?

Where the document says " If you pin your ILB ASE to a specific AZ, the resources used by a ILB ASE will either be pinned to the specified AZ, or deployed in a zone redundant manner", could you clarify the last phrase: "...or deployed in a zone redundant manner".
When would an ASEv2 with an ILB be deployed in a zone redundant manner?  Do you mean by following the steps later in this article?

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

ID: d05795f9-5027-e37d-7862-29c72f5c4edb
Version Independent ID: 7ccc48be-64dd-10ed-a0bf-e0c90db04bed
Content: Availability Zone support for App Service Environments - Azure App Service Environment
Content Source: articles/app-service/environment/zone-redundancy.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

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

Hi @larryclaman, I did not notice the steps you were referring to, but I did want to clarify if you mean deploying an ASE through an ARM template or through the CLI? When it comes to how appropriately setup your ASE for high availability, https://docs.microsoft.com/en-us/azure/architecture/reference-architectures/enterprise-integration/ase-high-availability-deployment is a great reference doc but no, ASEs aren't deployed across AZs by default. Furthermore, scale units aren't deployed across availability sets.
With regards to ...or deployed in a zone redundant manner, I'll confer witht the content author for clarification.

Hi @RyanHill-MSFT, When I referred to 'not following the steps on this page', I indeed meant "if you deploy an ASE via the portal or cli".
Per your comment "ASEs aren't deployed across AZs by default. Furthermore, scale units aren't deployed across availability sets." -> Yes, this is exactly the type of clarification that I would like to see added to the ASE documentation.
For example, you referred to the page https://docs.microsoft.com/en-us/azure/architecture/reference-architectures/enterprise-integration/ase-high-availability-deployment, where it says "App Services Environments can be deployed to multiple availability zones, only in internal or ILB mode. "    Again, there's a lack of clarity, specifically around the use of the word "can".  They way it's currently written could be interpreted as "an ASE can be optionally deployed across multiple availability zones", which is incorrect.  It should be re-written as "App Service Environments are deployed into a single AZ, but here is a reference architecture that enables you to deploy two ASEs, each of which will be pinned to a distinct availability zone".

Hi @larryclaman you cannot deploy ASEv2 in a zone redundant manner. You can either deploy it normally (where Azure choses a zone) or you can zone pin it. This way, you can deploy two ASE instances in different zones and achieve zone resiliency.
With ASEv3 you have zone redundancy built in, where the platform balances infrastructure and worker across three availability zones.
#please-close


removing kubectl gcloud component

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: RagniGarg97
To complete the pull request process, please assign mkumatag after the PR has been reviewed.
You can assign the PR to them by writing /assign @mkumatag in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @mkumatag

@ragni Can you please update the description with the reason for removal with proper error messages?
It will be useful for tracking purposes.

@ragni Can you please update the description with the reason for removal with proper error messages?
It will be useful for tracking purposes.

Done

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag, RagniGarg97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[WIP] add --no-default-build flag

Enable users to disable the default build for when there are no special requirements found. This allows users to take other actions if no environment is specified.

This is a nice PR! Could you explain a bit more what you want to achieve and what the motivation is for this change?
As a "user" I am confused about what --no-default-build is meant to do or when it would be useful so maybe we can expand the help text or give it a different name? In particular how it is different from --no-build. It seems related to buildpacks?

Also, I missed the [WIP] tag, so if things are in progress and will become clearer later I'll wait :)

Thanks for doing a review! Up for suggestions on the on the name.
I wanted to make it so that when there are no "special" requirements needed for a project, the user can take other actions as needed for their scenarios, for e.g. reduce the time to container by using a pre-built image for example in binderhub scenarios.  It differs from --no-build because it will still build when there are configuration files detected in the repo. Just won't do a build when nothing is found. Let me know if there are any other questions.

I made a local directory that contains some files, none of them trigger a build pack, checked out this PR and ran it:
$ ls /tmp/some-repo
$ repo2docker --no-default-build /tmp/some-repo
Picked Local content provider.
Using local repo /tmp/some-repo.
$

and there is no new docker image.
If I use master instead on the same repo it does build an image with all the basics needed to run a jupyter notebook server but not (much) more.
I like the idea of faster builds, but I am not sure I understand how this is meant to work :-/

It differs from --no-build because it will still build

Sorry I wasn't clear enough, the similarity to --no-build is in the naming, which is one reason I think we should try and come up with a different name. Because as you say they aren't related.

Ah. Thanks, yeah, a new name makes sense. Let me think about something that's more clear.
Yes, with this PR, no image will be generated when no build packs are picked. The idea is that the user can do something else after running repo2docker when no image is generated, like use another image that they already have access to for example (instead of building a new one even if it's basic). Or throw an error just as another basic example.

new name suggestions, let me know if something sounds clearer based on my explanations and i'll update the PR to reflect it: --skip-default-image or --skip-default-build-pack

Yes, with this PR, no image will be generated when no build packs are picked.

Ok, why use repo2docker only to fetch a repo and do nothing else with it? Especially as all the things which have been fetched will be deleted as soon as repo2docker exits. Which means you couldn't use it in another image. Could you give your concrete use case as an example maybe?

Basically, my scenario is:

Have a private registry to store my images.
Given a github repo, check if there are special configuration files to run the notebooks.
If yes, build the image and push it to my private registry using --push
If no, don't build and push an image, just use a previously built default image.

Since repo2docker already does a great job detecting if there are any configuration files, i don't want to replicate this outside of repo2docker to determine if I should even call repo2docker. But i don't want it to build another image if there are no special requirements as well, which is what it does currently without my change.
With --skip-default-image, the idea is that if no configuration files are detected, no further action is taken. As a result, no image will be pushed to the registry, and i can detect that and just fall back to using my default image. Basically use the same image for all repositories that don't need a specific environment, e.g. just have a generic python image.

This is a very useful feature! From my understanding, the idea is to really check if a repo (or directory) will trigger any particular build pack we have, or if it'll just trigger the default. Is that right?
My recomendation in this case would be to have a flag that'll just output the name of the buildpack that's triggered, saying 'default' if it's the default buildpack. This will be more generally useful for the use case of 'detect what kinda stuff this repo has', rather than specifically not doing anything. What do you think?

How about making it a second entrypoint called repo2docker-detect that returns the name of the build pack detected? And/or putting as much as possible inside the "library" part of repo2docker so that you could do something like import repo2docker; print("Build pack is:", repo2docker.detect()).
Would that open the door to it not even downloading the source (where possible) as a future optimisation?

Thanks! More than just detecting, I'd like to take action after the detection; if it's not default, then build an image; otherwise don't do anything so the caller can make a call as to what to do.
To understand the suggestion better, is the flow being recommended a two-step process?

run repo2docker with something like --detect-build-pack or repo2docker-detect
if the answer is anything but default, call repo2docker again to generate the build image?

@shibbas yep, that's the workflow I'd like. Since you have something calling repo2docker already, it should be able to do this. It'll also help other workflows that might wanna do something similar.

Hi @shibbas :sunny:
A group of repo2docker maintainers are meeting this weekend and are triaging PRs. It looks like this WIP PR left off with a conclusion that a two-step process would be preferred.
We've opened a new issue (#783) that gives a place to discuss more specifics and get input from the community.
Would you like to keep this WIP PR open for future work? If not or if it would be easier work from a new PR, please let us know and we would be happy to close this PR.

Closing based on last comment


Every sentence in a single line

Following our approach to have, for an easier review process, every sentence (ending with a period) in it's own line.
Hi Sally, I was in a talk with @quaid yesterday, by accident we talked about this PR for the blog.
Here is my offer to format your blog post.
I didn't change any text or content.
If you want to, you can use it, otherwise just ignore it.

/CC @quaid

hey, I missed that memo :) thank you!


[Adaptive] do we have a while loop action?

Use this query to search for the most popular feature requests.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
It seems we only have ForEach for loop.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Put this into backlog before we have a process to take new actions

Current alternative: use ForEach on a non-empty array and change index to -1 in the loop.

Closing as duplicate of https://github.com/microsoft/botbuilder-dotnet/issues/4862


Warning about the start of the .rodata section doesn't seem disableable

https://github.com/ethteck/splat/blob/main/src/splat/segtypes/common/data.py#L146

Should we add an option to turn off the warning as a whole or just per segment? Or both?
Another idea besides being able to turn off this warning is to detect if the jumptable is used by a hasm file and ignore the suggestion if that's the case

I think any warning emitted should be configurable at some level, even if it's not per-warning type. but yeah I think global + per segment is grood

gotcha, makes sense. yeh I agree with the idea I think


Update constructor docs

The docs suggest the following call signature (https://juliamatrices.github.io/BlockBandedMatrices.jl/latest/):
BlockBandedMatrix{T}(undef, (rows, cols), (l, u))

but this is incorrect. The correct signature is:
BlockBandedMatrix{T}(undef, rows, cols, (l, u))

Please update the docs.

Fixed in #79.


[occm] Should internal-network-name have an impact on the internalIPs picked up for LoadBalancer Members?

[occm] internal-network-name for LoadBalancers
Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
When a Loadbalancer is created with the OCCM. The OCCM chose the default InternalIP of a worker node, but It would be nice to have the same behavior as internal-network-name because a worker node can have multiple interfaces, in my case, the loadbalancer is not in the same subnet as my main InternalIP, and I cannot connect my InteralIP subnet to a router.
Environment:

OCCM version: latest
OpenStack version: 2024.2
Others:

@pabclsn is this related to https://github.com/kubernetes/cloud-provider-openstack/issues/2718 ?


Invalid JVM target?

I've configured Detekt to use JDK 17 as the JVM target...
In the project's build.gradle file:
ext {
    java_version = JavaVersion.VERSION_17
}

In the module's build.gradle file:
def buildConfig = rootProject.extensions.getByName("ext")

tasks.withType(Detekt).configureEach {
    jvmTarget = buildConfig.java_version
}

So in theory this should work, yet when I try to run a task I get this error:
Execution failed for task ':app:detektBaseline'.
> Invalid value (19) passed to --jvm-target, must be one of [1.6, 1.8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18]

Why is it trying to use version 19 if I've specified version 17?

Hmm, actually, if I run the task from Android Studio it works...but it doesn't work if I use gradlew.

I do have this in my module's build.gradle:
compileOptions {
    sourceCompatibility buildConfig.java_version
    targetCompatibility buildConfig.java_version
}

kotlinOptions {
    jvmTarget = buildConfig.java_version.toString()
}

Your example configures the KotlinCompile tasks, so it's a bit different, but is there any reason this shouldn't work?

kotlinOptions {
    jvmTarget = buildConfig.java_version.toString()
}

will not work for kapt. This behaviour got changed recently in Kotlin where now you need to configure jvmTarget for each kotlin tasks

OK well, I just tried your suggestion and it didn't work. I got the same error.

Oh. I had to actually run compileDebugKotlin too.
It works now, thanks! 👍


Unable to playback the recorded video on iOS

Been struggling using the library with iOS. I am able to record my video correctly but the playback is only showing the first frame of the video.
I have validated that the video is successfully recorded as I send it to a server without any issue, the only problem is the playback.
The issue seems to be with the blob URL, specifically, the url created is in format blob://example and there doesn't seem to be a way to play that back, or at least, I haven't found how.
Anyone else running into this issue?

I was having a similar issue.  Try including playsinline muted autoplay properties on your video element.


Exports a IConfetti type

Maybe is very interesting to can use the isolate objects config of JSX and still with ts types  :)

Hey thanks @JoaoVictor6, this was on my list of todos ~ now ConfettiProps interface is exported, which extends React div props, see here.


Binding a class to LMap makes custom classes disappear

Description
No error thrown, but when adding a class via Vue's class bindings to LMap, the map loses a bunch of classes. In my case it makes the tiles white. In the provided demo it doesn't do that exactly, but when inspected, the map is missing a bunch of leaflet- prefixed CSS classes after the toggle has been clicked.
Live Demo
https://jsfiddle.net/wishinghand/5noybxh2/10/
Steps to Reproduce
Setup a button to toggle a Boolean in Vue's data object, defaulted to false. When true set the cursor to a custom icon.
Expected Results
Cursor gets changed to a custom icon and the map acts as normal.
Actual Results
The cursor is changed, but the map goes white and disappears, though any polygons/lines and popups stay.
The maps starts with these classes:
vue2leaflet-map
leaflet-container
leaflet-touch
leaflet-retina
leaflet-fade-anim
leaflet-grab
leaflet-touch-drag
leaflet-touch-zoom

and only has vue2leaflet-map plus the extra class I bound.
Browsers Affected

[ x] Chrome
[x ] Firefox
[ ] Edge
[ x] Safari 9
[ ] Safari 8
[ ] IE 11

Versions

Leaflet: whatever Vue2Leaflet 1.0.2 uses
Vue: v2.5.16
Vue2Leaflet: v1.0.2

@wishinghand Sorry for not answering here, this is due to the fact that we bind the leaflet map directly in the component $el
now this is not ideal, but to change this we would need to bind it inside another element inside the component which could be a breaking issue.
I am not sure how to proceed here, what do you think ? Either way I want to update the docs to specify this quirk.

I don't think I'm skilled enough to weigh in. I don't know what the alternative to binding to $el would be.

Closing this and making a new issue to update the docs


BREAKING CHANGE: change ckb decimal to 18

We should also change SUDTTransfer#amount to Uint256, since the layer2 is using decimal 18. The burn / mint interface should also be update.