Is the Azure Storage working with .NET Core?

I'm trying to use the Azure Starage with .NET Core 1.1, but the project is getting errors of dependencies.
I can't use the "WindowsAzure.Storage": "7.2.1" dependency.
That's my project.json
{
  "dependencies": {
    "Microsoft.AspNetCore.Mvc": "1.1.0-preview1-final",
    "Microsoft.AspNetCore.Antiforgery": "1.1.0-preview1-final",
    "Microsoft.AspNetCore.Server.IISIntegration": "1.1.0-preview1-final",
    "Microsoft.AspNetCore.Server.Kestrel": "1.1.0-preview1-final",
    "Microsoft.Extensions.Configuration.EnvironmentVariables": "1.1.0-preview1-final",
    "Microsoft.Extensions.Configuration.FileExtensions": "1.1.0-preview1-final",
    "Microsoft.Extensions.Configuration.Json": "1.1.0-preview1-final",
    "Microsoft.Extensions.Logging": "1.1.0-preview1-final",
    "Microsoft.Extensions.Logging.Console": "1.1.0-preview1-final",
    "Microsoft.Extensions.Logging.Debug": "1.1.0-preview1-final",
    "Microsoft.AspNetCore.Hosting": "1.1.0-preview1-final",
    "Microsoft.EntityFrameworkCore": "1.1.0-preview1-final",
    "Microsoft.EntityFrameworkCore.Design": "1.1.0-preview1-final",
    "Microsoft.EntityFrameworkCore.Relational": "1.1.0-preview1-final",
    "Identity.Dapper.MySql": "0.3.0-alpha",
    "Microsoft.AspNetCore.Diagnostics": "1.1.0-preview1-final",
    "Identity.Dapper": "0.3.0-alpha",
    "MySql.Data": "7.0.5-IR21",
    "MySql.Data.EntityFrameworkCore": "7.0.5-IR21",
    "AspNet.Security.OAuth.Validation": "1.0.0-alpha3-0188",
    "Microsoft.AspNetCore.Authentication.JwtBearer": "1.1.0-preview1-final",
    "OpenIddict": "1.0.0-*",
    "Npgsql": "3.1.8",
    "Npgsql.EntityFrameworkCore.PostgreSQL": "1.0.2",
    "Identity.Dapper.PostgreSQL": "0.3.0-alpha",
    "Microsoft.AspNetCore.StaticFiles": "1.1.0-preview1-final",
    "Dreampper.Crosscutting": "1.0.0-*",
    "WindowsAzure.Storage": "7.2.1"
  },

  "tools": {
    "Microsoft.AspNetCore.Server.IISIntegration.Tools": {
      "version": "1.0.0-preview1-final",
      "imports": "portable-net45+win8+dnxcore50"
    },
    "Microsoft.EntityFrameworkCore.Tools.DotNet": "1.0.0-preview3-final"
  },

  "frameworks": {
    "netcoreapp1.1": {
      "dependencies": {
        "Microsoft.NETCore.App": {
          "type": "platform",
          "version": "1.1.0-preview1-001100-00"
        }
      },
      "imports": [
        "dotnet5.6",
        "dnxcore50"
      ]
    }
  },

  "buildOptions": {
    "emitEntryPoint": true,
    "preserveCompilationContext": true
  },

  "runtimeOptions": {
    "gcServer": true
  },

  "publishOptions": {
    "include": [
      "wwwroot",
      "Views",
      "appsettings.json",
      "web.config"
    ]
  },

  "scripts": {
    "postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
  }
}

@itsfelipesan Try adding an imports for ...
"portable-net45+win8"

I have the same problem, I can't use WindowsAzure.Storage": "7.2.1 with .Net standart 1.6.1

@itsfelipesan @adrianMaersk adding imports statement as @GuardRex recommended should work this.
@bleroy - do you know when this would be usable without imports? Could you chase folks as needed? :)

IIRC this was related to OData, which still required the imports. OData is still baked into Azure Storage, but they plan to remove it at some point.
The last word from @markdstafford over at OData is https://github.com/OData/odata.net/issues/568#issuecomment-230068541

We will fix this in 2.0. All the dependencies have been updated, but there are breaking changes in them so we will need to wait for 2.0 to update the version we depend on.

Will be fixed by https://github.com/aspnet/Logging/issues/562


notes_placeholder and notes_text_frame are None when a note exists

On a slide that has no notes
notes_slide: <pptx.slide.NotesSlide object at 0x0000026013225460>
background: <pptx.slide._Background object at 0x0000026013225FA0>
element: <Element {http://schemas.openxmlformats.org/presentationml/2006/main}notes at 0x26013219400>
name: ''
notes_placeholder: <pptx.shapes.placeholder.NotesSlidePlaceholder object at 0x0000026013226270>
notes_text_frame: <pptx.text.text.TextFrame object at 0x00000260132260F0>
part: <pptx.parts.slide.NotesSlidePart object at 0x0000026013225250>
placeholders: <pptx.shapes.shapetree.NotesSlidePlaceholders object at 0x0000026013226000>
shapes: <pptx.shapes.shapetree.NotesSlideShapes object at 0x0000026013225580>
part: <pptx.parts.slide.SlidePart object at 0x00000260131EE8A0>
On a slide with an existing note
notes_slide: <pptx.slide.NotesSlide object at 0x000002601470E0C0>
background: <pptx.slide._Background object at 0x000002601470D250>
element: <Element {http://schemas.openxmlformats.org/presentationml/2006/main}notes at 0x26013200aa0>
name: ''
notes_placeholder: None
notes_text_frame: None
part: <pptx.parts.slide.NotesSlidePart object at 0x00000260131EE450>
placeholders: <pptx.shapes.shapetree.NotesSlidePlaceholders object at 0x000002601470D3D0>
shapes: <pptx.shapes.shapetree.NotesSlideShapes object at 0x000002601470EFF0>
The slide note does have some maths characters  - I dont know if this is the issue or not.
I have tried to delete the notes (manually and via VBA), and this still seems to cause the issue of notes_text_frame =None.
This is looking like a bug. It is a bit of a show stopper as as I am trying to generate talking points on a slide deck of 360 slides.

As it happens I was just looking at this code last night to diagnose a different issue in a downstream system and I'm pretty sure it's working as intended.
If you can post an example .pptx file that demonstrates this behavior I'll take a look.

Closing as incomplete.


Add Find method on DbSet

In the old world, we have Find method on Dbset that is missing in the new world. It would be nice to have this in new EF world too.

Note for triage: I'm marking this for re-triage since I'm seeing more and more people asking about this and we haven't done any work on the mitigation we once talked about in the early days of EF7.

Is FindAsync also in the consideration here as well or is that a separate issue?

@jasoncavett You can consider FindAsync() part of this issue.

I've seen some good discussion and feedback on this on the ASP.NET Insiders mailing list. We should re-discuss and maybe pull it into RTM.

I would also like to add that I would like to see FindAsync() and Find() in EF7

I am also going to add that it would be nice to have Find() in EF7

P.S. I have an answer on StackOverflow showing how you could write it yourself.

That is awesome.  I am having a hard time trying to find where IAccessor is located. Any ideas?

@sirpadington In namespace Microsoft.Data.Entity.Infrastructure.

@bricelam and @sirpadington do you find reference for IAccessor? I can not find on Microsoft.Data.Entity.Infrastructure?

It was renamed to IInfrastructure<>.

What is the progress on this "Add Find method on DbSet #797"?

God we need this so much.

Man, i was hoping we would get this in Update 2 :-1:

@komengem did you have a look at: http://stackoverflow.com/questions/29030472/dbset-doesnt-have-a-find-method-in-ef7/29082410#29082410 ?

My question about implementing Find in EF7 is the efficiency. It's easy enough to take linear time to loop through the elements and find what you are looking for. But what I expect from Find is that it use the database indices to access the correct element (but without hitting the disk) in some reasonably fast manner like log n or better. If that is not what it does then I want a huge "use at your own risk" or disclaimer about the lack of efficiency. Because, the reason I would look to Find in the first place is in the expectation that an in-memory search would be faster than a search to disk.
So, while I obviously appreciate @bricelam's  code suggestions on StackOverflow, I wouldn't want the final StackOverflow solution to be the basis of the EF7 implementation.

If this is done, how do I use it.  I have 1.0.0 of EF7 and I don't see the find method.
{
  "version": "1.0.0-*",

  "dependencies": {
    "Domain": "1.0.0-*",
    "Microsoft.EntityFrameworkCore": "1.0.0",
    "Microsoft.EntityFrameworkCore.SqlServer": "1.0.0",
    "Microsoft.EntityFrameworkCore.SqlServer.Design": {
      "version": "1.0.0",
      "type": "build"
    },
    "Microsoft.EntityFrameworkCore.Tools.Core": "1.0.0-rc2-final",
    "NETStandard.Library": "1.6.0"
  },

  "frameworks": {
    "netstandard1.6": {
      "imports": ["dotnet5.6","dnxcore50", "portable-net45+win8"]
    }
  }
}

@martinRocks the milestone is 1.1.0

You can try our nightly feeds, but they are unsupported and only really intended for trying out new features (not building production applications) https://www.myget.org/Gallery/aspnetcidev

Quick question about the intended implementation, because a couple of people have posted the link to 'workarounds' on StackOverflow: will the final version maintain an identity map? Right now if you're in a loop, say, and do SingleOrDefault for the ID, adding the entity if it's not found, you will get duplicate key violations when you submit the changes to the database. Ideally, Find will be smart enough to return the new, unpersisted entity reference, instead of always hitting the database.

Yes, Find uses an identity map. Yes, find will return the new, unpersisted entity.

I would like to reopen this discussion to ask for a version of Find that only looks in memory, and does not go to the database. The point is to use the EF indices to find the entry quickly, if it is in memory.
My use case is this: I have a variety of views with viewmodels that can reference overlapping data i.e. the same entity loaded through several contexts. I want to be able to switch back and forth between the views and have them all up to date. I have no desire to implement my own identity resolution so that the different views/viewmodels share entities (that they each have loaded using different DbContexts). Neither do I want to reload everything in a given view/viewmodel just because a user edit in some other view/viewmodel has changed one entity. So it seems unavoidable that I will have several copies of a given entity in memory, coming from different DbSets. What I would like to do is, when one of the in-memory entities is changed in one DbSet then I would like to Find the other in-memory copies in the other DbSets (associated with other views / viewmodels) and update them appropriately. Obviously in this situation, having Find hit the database would be counterproductive.
I realize that the philosophy is to dispose the context (and dbSets) after the unit of work is done -- perhaps right after the queries have finished. That doesn't make this problem go away, though. If you drop the context and the dbSets then it just entails that the duplicate entities are no longer associated with dbSets (which makes them even harder to Find :-)

@sjb-sjb Covered by #7391

@ajcvickers thanks for this reference.
Do you have any general / high level comment on the overall goal of synchronizing multiple copies of an entity, which arises due to loading the entity multiple times through different contexts? I think this problem is inevitable when you have multiple views of the data, but is it really? I would like to know if I've just gotten lost in the rabbit hole. What is the usual way to deal with this?
All the best

@sjb-sjb Keep it constrained. That is, for an arbitrary number of instances, each of which can have any changes to both scalar properties or relationships, and for which there can be new entities, deleted entities, modified entities, and unchanged entities, it is very hard to define what the "correct" behavior is, and hard to come up with rules that will cover all cases. On the other hand, if there are two instances, one is known not to change, and the other only has scalar changes, then the task is pretty easy. So, as much as possible, constraint where and how changes can be made so as to keep your rules for how to merge changes as simple as possible,

Thanks for the good advice. I think scalar properties and add / remove are enough. I plan to only change relationships by setting the reference Id, so that's a scalar property. Let's say I add a new child entity and set the parent id. I don't need to also explicitly create the relationship in the traversal collection, do I?
Since the changes are being made on the ui thread and will be immediately propagated to the other entities, there is no way to have conflicting changes in two copies of an entity.
What I realized is that one doesn't want to automatically replicate the changes made in one context in all the other contexts. Creating a relationship between two collections so that one of them picks up changes made in the other, should not be automatic but is an application decision. It should be part of the view model layer.
Obviously MS doesn't have this kind of thing today, for example this goes beyond the intended design  of the Template 10 validation / view model system. (ref @JerryNixon) Is there any plan to do something like this in the future? A comprehensive view model system including cross-context synchronization primitives (and validation) would be great.

Microsoft.Data.Entity.Infrastructure namespace is not found !

@ajcvickers, a followup on the use of Find in replicating changes from one context to another. My current thinking is to constrain the problem (as you suggest) by only replicating changes when a SaveChanges occurs. That is a lot simpler than the general problem. One can just use Find and/or GetDatabaseValues directly.


Can not parse html code

This sounds very strange as this works in your demo.
I am trying to parse
```<span>Hey there...</span>```

but this output to only Hey there and span tags are removed.
This must be because angular sanitizing the generated code by marked parser.
Is there any way to handle this?

Just learned something. Below is the code generated by marked parser. It has not span tags.

Hello there...

Found. It won't work inline because you are retrieving markdown content from directive element using $elem.text() which will strip html tags. $elem.html() would work, but don't know if that will lead to any consequences. I am sticking with ng-model approach.


Fix KafkaPartitionLevelConsumerTest by using NetUtils to fetch the localhost ip

Also move NetUtils (renamed from NetUtil) to pinot-spi so that it can be accessed from the kafka package
Here are some explanations on why it works: https://stackoverflow.com/questions/9481865/getting-the-ip-address-of-the-current-machine-using-java

Codecov Report

Merging #6784 (d96966f) into master (3f93cfb) will decrease coverage by 30.42%.
The diff coverage is 45.45%.

@@              Coverage Diff              @@
##             master    #6784       +/-   ##
=============================================
- Coverage     74.14%   43.72%   -30.43%
+ Complexity       12        7        -5
=============================================
  Files          1412     1412
  Lines         68762    68763        +1
  Branches       9934     9934
=============================================
- Hits          50986    30069    -20917
- Misses        14461    36176    +21715
+ Partials       3315     2518      -797

Flag
Coverage Δ
Complexity Δ

integration
43.72% <45.45%> (-0.16%)
0.00 <0.00> (ø)

unittests
?
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Fixed in #6786


[T2A2][F14-A1]Zhang Ying

Replaced LS with LINE_SEPARATOR

@Zing1996 Some comments added. Please close the PR after reading comments.


GUI: "amount to select" isn't refreshed when moving across steps (previous <-> next)

See the screencast:
Screencast from 2022-12-08 17-04-25.webm

It's not the only occurence. A lot states do not get updated. Is it easy to fix @edouardparis?


The dependency on the "CanvasProxyInteractor" is not made clear at all

Describe the issue
We ran in to some very odd issues with the MRTK UX, to the point where null reference exceptions where appearing sporadically as the UI was being hovered over or interacted with.
On investigation, ALL of the errors were due to the absence of the "CanvasProxyInteractor" being active in the scene.
Feature area
If a component, such as the "CanvasProxyInteractor" interactor is required for features to work within a MRTK scene, then it both needs to:

Be identified by components that rely on it (e.g. a UX component on a Canvas using a "UGUI input adapter") should highlight that a dependency is missing from the scene, be it a "CanvasProxyInteractor" or another "IProxyInteractor" style class
Documentation needs to be clearer to STATE that such a component needs to be in a scene if it is a depdendency.

ISSUE MIGRATION
Issue migrated from: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/10740

Original comment by:
 keveleigh

Thanks for the call stacks! CC @Zee2

Original comment by:
 SimonDarksideJ

LMGTFY
Two exception stacks I can offer after breaking the solution again (disabling the "CanvasProxyInteractor")
NullReferenceException: Object reference not set to an instance of an object
Microsoft.MixedReality.Toolkit.UX.UGUIInputAdapter.OnPointerEnter (UnityEngine.EventSystems.PointerEventData pointerEventData) (at Library/PackageCache/com.microsoft.mrtk.uxcore@421d14e45036/Interop/UGUIInputAdapter.cs:195)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerEnterHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:22)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:262)
UnityEngine.XR.Interaction.Toolkit.UI.UIInputModule:Update() (at Library/PackageCache/com.unity.xr.interaction.toolkit@2.0.0/Runtime/UI/UIInputModule.cs:111)

NullReferenceException: Object reference not set to an instance of an object
Microsoft.MixedReality.Toolkit.UX.UGUIInputAdapter.OnDeselect (UnityEngine.EventSystems.BaseEventData eventData) (at Library/PackageCache/com.microsoft.mrtk.uxcore@421d14e45036/Interop/UGUIInputAdapter.cs:270)
Microsoft.MixedReality.Toolkit.UX.UGUIInputAdapter.OnPointerExit (UnityEngine.EventSystems.PointerEventData pointerEventData) (at Library/PackageCache/com.microsoft.mrtk.uxcore@421d14e45036/Interop/UGUIInputAdapter.cs:211)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerExitHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:29)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:262)
UnityEngine.XR.Interaction.Toolkit.UI.UIInputModule:Update() (at Library/PackageCache/com.unity.xr.interaction.toolkit@2.0.0/Runtime/UI/UIInputModule.cs:111)

Both are caused by the same effect, as the "protected IProxyInteractor ProxyInteractor" lookup fails to find any registered "IProxyInteractors" active in the scene and just returns NULL, causing ANY events requesting it to cause a NullReference check.  This would in theory affect any of the events in the "UGUIInputAdapter".
What I do not understand is what the failure is sporadic, does not always occur and will raise an exception on one "StatefulInteractable" object and not another.
But handling the null reference failure is also wise @keveleigh  :D

Original comment by:
 Zee2

@SimonDarksideJ, well aware of this; will be fixed in an upcoming update. UGUIInputAdapter will gracefully fallback if no proxy interactor is available.
In addition, we're going to be baking the proxy interactor into the default rig prefab to help avoid these points of confusion in the future!

Original comment by:
 Zee2

Fixed in #10711.

Original comment by:
 keveleigh

Fixed in #10711.

@Zee2 just to clarify, I'm not sure I see the two potential null refs mentioned above changed in that PR.
It seems like the following two sections would still null ref if ProxyInteractor is null (along with some others as I was looking through), unless your change in that PR prevents these methods from being called?
https://github.com/microsoft/MixedRealityToolkit-Unity/blob/45ea6a3b2a6bd5fef1ff3e1f7b5b8c25416850c9/com.microsoft.mrtk.uxcore/Interop/UGUIInputAdapter.cs#L206-L209
https://github.com/microsoft/MixedRealityToolkit-Unity/blob/45ea6a3b2a6bd5fef1ff3e1f7b5b8c25416850c9/com.microsoft.mrtk.uxcore/Interop/UGUIInputAdapter.cs#L285-L288

Original comment by:
 Zee2

You're right that it doesn't technically guard against those nullrefs; but, then again, they'd have very little reason to occur if the proxy interactor was baked into the rig.
I'll open this again for a future PR that'll do more safeguarding/explanation.


“nvm ls” doesn't show any installed NodeJs

I used nvm to install two version of NodeJs using Ansible.

When I type "node -v", I got v5.5.0.
When I use "nvm ls" using "go" as user, I got: mkdir: cannot create directory `~': Permission denied node -> stable (-> N/A) (default) iojs -> N/A (default)

As you can see, I use sudo_user "go" for all the installtions, but I don't see the node I installed.
This is my .bashrc file:
export NVM_DIR="~/.nvm"
source ~/.nvm/nvm.sh
"~" here is "/ngs/app/go"

nvm is per-user - for each user account you wish to use nvm with, nvm must be installed, which will have its own node versions - and sudo should never be used.
In other words, install nvm, and use it, always as the same user, and never with sudo.

If I'm misunderstanding, then please share the output of nvm debug?

Happy to reopen with more info.


dockefile build error on source /root/.bashrc

workspace Dockerfile:
the console log show:
---> 56b18cc9b3b7
Removing intermediate container bbd2ceb35e86
Step 27 : RUN . ~/.bashrc
---> Running in 75b252000649
/bin/sh: 13: /root/.bashrc: shopt: not found
/bin/sh: 21: /root/.bashrc: shopt: not found
---> 425a7d29ecfc

I'll close this old issue now, if the error still exist we'll open it back.

Still exists for me in the latest commit (https://github.com/laradock/laradock/commit/c4aef8e901a867a57df0b1aadf1c872165e65295), @Mahmoudz . Did you ever find a fix @nickfan?

still exists.


Have gr._utils._check_tuple_needles error message report correct string if invalid cluster name supplied

Description
squidpy.gr._ligrec._check_tuple_needles has an error message when invalid cluster names are passed to it through squidpy.gr._ligrec, which reads ""Invalid cluster <Bad Cluster>.", but if the invalid one is the second object of the tuple, the error message will print the first object of the tuple. This fix should give the correct error message now.
How has this been tested?
from itertools import product
clusters = list(map(str, adata_parent.obs["leiden_coarse"].cat.categories))
clusters.append("bad one")
clusters = list(product(clusters, repeat=2))  # type: ignore[no-redef,assignment]
clusters = sorted(
            sp.gr._ligrec._check_tuple_needles(
                clusters,  # type: ignore[arg-type]
                adata_parent.obs["leiden_coarse"].cat.categories,
                msg="Invalid cluster `{0!r}`.",
                reraise=True,
            )
        )

Before: ValueError: Invalid cluster 'APC'. [ this is a real cluster in my data ]
After: ValueError: Invalid cluster 'bad one'.
This is a re-submission of a previous one in "dev" brach, hopefully it will work this time.

Looks good @dnjst , the tests are failing for other reasons (will be fixed tomorrow in #453).


Update qrcode-generator extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

What the status of this PR @theOpheliart


Bedrock Renames

Fixes #1669
Lots of changes here. A summary:

Microsoft.AspNetCore.Sockets.Http -> Microsoft.AspNetCore.Http.Connections
Microsoft.AspNetCore.Sockets.Client -> Microsoft.AspNetCore.Http.Connections.Client
Microsoft.AspNetCore.Sockets.Common -> Microsoft.AspNetCore.Http.Connections.Common
SocketsSample -> SignalRSamples
Make ServerSentEventsParser internal because a) I had to move it to the right folder path to fix the namespace and b) it's only used by tests

TODO:

[ ] Some final find-replace passes for "Sockets" (it's a little tricky because of WebSockets ;))
[ ] @davidfowl I can grab your Kestrel change tomorrow morning and yank Sockets.Abstractions.

Need a name for logging token SocketsConnectionId. cc @BrennanConroy @davidfowl

@anurse bro wrong base branch. This should be in release/2.1

Yeah, I'll try to update it before going to bed :). Autopilot...

Interesting:
        ....
        - level: Error
          message: Server returned handshake error: An unexpected error occurred during connection handshake. InvalidDataException: Missing required property 'protocol'.
        - level: Trace
          message: (LongPolling transport) polling: http://<IP_ADDRESS>:54817/testhub?id=Rs8ARL1N29TegQvMWMF4SA&_=1522428941594
        - level: Error
          message: Connection disconnected with error 'Error: Server returned handshake error: An unexpected error occurred during connection handshake. InvalidDataException: Missing required property 'protocol'.'.

Oh no! It showed up again 😨

Sockets.Abstractions is removed! This is ready to go once approved!

Build failures...

The active test run was aborted. Reason: Unhandled Exception: Unhandled Exception: System.InvalidOperationException: Another continuation was already registered.
AppVeyor failure is caused by that corefx sockets bug I think.

Merge conflicts tho...

:up: :date: merge conflicts resolved

Verbal approval from @davidfowl


Update clearm command and related test

clearm now clears the member list only instead of both the member and facility list. ClearMembersCommandTest checks that only the member list has been cleared. Fixes #100 #101.

Codecov Report

Merging #102 (e1043ed) into master (666d0ba) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #102      +/-   ##
============================================
- Coverage     72.88%   72.86%   -0.03%
- Complexity      480      482       +2
============================================
  Files            85       85
  Lines          1420     1426       +6
  Branches        144      144
============================================
+ Hits           1035     1039       +4
- Misses          346      348       +2
  Partials         39       39

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...du/address/logic/commands/ClearMembersCommand.java
100.00% <100.00%> (ø)

src/main/java/seedu/address/model/AddressBook.java
91.66% <100.00%> (+0.49%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
94.91% <100.00%> (-3.34%)
:arrow_down:

...a/seedu/address/model/person/UniquePersonList.java
93.02% <100.00%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Put "retrying" in README for SEO :-)

Some time ago I was looking for some Java library for retrying an operation waiting for the server response to contain information about finished processing of previously accepted request (so not strict "failure handling"). Unfortunately, I didn't find Failsafe as it was too low in search results for "retrying java library". Luckily a colleague of mine suggested Failsafe and I like it.
In hindsight I could search for "retry java library" with a better chance to find Failsafe (although that keyboard doesn't exist in README and/or project description on GH), but I think it would be good to cover also people looking for "retrying" in Java.

Failsafe is a lightweight, zero-dependency library for handling failures (and retrying in general) in Java 8+

?
As the original description is very consist it might be added later on in the README to do not "derate" it.

Fixed in 8b6ec6cd63039230e27de2314ded453259722fe5.


Change subtitle on github

Hi,
I would suggest changing the subtitle on the repository-page from Automatically exported from code.google.com/p/python-escpos  to Python library to manipulate ESC/POS Printers.

done


Add http basic auth based on env vars

@mbovo this is good. I am thinking about how to make it optional. I mean if we don't inject the env vars we are still requiring an empty auth

@gianarb I've added a flag to enable (default disabled) authentication, with default user/pass (orbiter/orbiter) just in case. I'm using github.com/kelseyhightower/envconfig here to read custom configuration from env, seems a good lib. As improvement in
https://github.com/mbovo/orbiter/commit/59758b2c062b1b47ccd8ca4770b2f83798aa1754 i've added a map for multiple user/pass.
I think configuration like this should be placed in global conf and propagated, instead of using repeated code. I will open another proposal/pr for this last point.

I like the mapping idea. I dont think we need library to read some envvar. I think the os package  is enough


KeyError in admin using Customer proxy model

In admin, when using a proxy model the class method "decode_session_key" raises a key error
screenshot
Here's the code:
class UserCustomerManager(models.Manager):

    def get_queryset(self):
        qs = super(UserCustomerManager, self).get_queryset()
        qs = qs.filter(customer__isnull=False)
        return qs

class CustomerInlineAdmin(CustomerInlineAdminBase):
    fieldsets = (
        (None, {'fields': ('salutation', 'get_number', 'recognized',
                           'commercial_license_no', 'legal_structure',
                           'phone', 'logo')}),
        (_("Shipping Addresses"), {'fields': ('get_shipping_addresses',)})
    )

class CustomerAdmin(CustomerAdminBase):
    add_form = UserCreationForm
    form = UserChangeForm
    fieldsets = (
        (None, {'fields': ('email', 'password')}),
        (_('Permissions'), {'fields': ('is_active', 'is_staff', 'is_superuser',
                                       'groups', 'user_permissions')}),
        (_('Important dates'), {'fields': ('last_login', 'date_joined')})
    )
    add_fieldsets = (
        (None, {'fields': ('email', 'password1', 'password2'),
                'classes': ('wide',)}),
    )
    list_display = ['get_username', 'salutation', 'recognized', 'last_access',
                    'is_unexpired']
    inlines = (CustomerInlineAdmin,)

    def get_fieldsets(self, request, obj=None):
        return super(CustomerAdminBase, self).get_fieldsets(request, obj)

class UserCustomer(get_user_model()):

    objects = UserCustomerManager()

    class Meta:
        proxy = True
        verbose_name = _("customer")
        verbose_name_plural = _("customers")

admin.site.register(UserCustomer, CustomerAdmin)```

If you want to add extra fields to a customer, then please inherit from BaseCustomer and materialize that model.

I have already done that.
Here I want to achieve something else. Instead of having the Customer section which has a dropdown with user objects, I have to show User fields and Customer fields as inlines to the User model.

showing User fields and Customer fields as inlines, while editing a User is the default configuration.
I then reorganize those fieldsets with Javascript, ie. I move the inlines on top.

Can we consider this issue solved, or is there something that needs to be done?

i found the problem, its happening when the user.username is like<EMAIL_ADDRESS>that PLUS SIGN will trigger this error on L119
username can take any characters so this is still a BUG

User.username shall never contain an email address. How does it come that someone was able to add an email address.
Btw., that field is far too short, it's max length is 30 chars, whereas email addresses can have up to 254 chars. This was the reason I added shop.email_auth to the project.

Indeed, but username field is still required. Bug traced down and fixed on our side. The developer who integrated the account part was making username to be the same as email for the sake of covering username's requireness. Still, I think there should be an option to drop the username field completely, but that's a story for another topic 😃

The developer who integrated the account part was making username to be the same as email for the sake of covering username's requireness.

Django has a well defined approach for this and you should not implement your own version of a User model, if it is not compatible with the built-in one. If you don't want to run into trouble, please use shop.email_auth which also handles email addresses for login names.
Please also consider the use-case, that some customers might buy as guests, but return later and implicitly create a new customer object with the same email address.

Username field as database column is not required by django, check AbstractBaseUser, while django shop it's using AbstractUser which is a derived class of AbstractBaseUser and some attributes of its own, like username field. Username as functionality is required and it can be completely relied on email filed.

In django-SHOP the username field is absolutely required. Anonymous customers are identified using that field, which then contains a compact version of the session key. Otherwise django-SHOP would not be able to assign a cart to a not yet logged in user.
Therefore, if you attempt to run django-SHOP without the username field, you are on your own. I then won't be able to help if you run into weird issues.

I understand. I think we can close this topic.

I believe we should document this better. I will open another issue for that.

it's all there ...
http://django-shop.readthedocs.io/en/latest/reference/customer-model.html#anonymous-users-and-visiting-customers

After reading through this issue again, I think we still have a bug here:

i found the problem, its happening when the user.username is like<EMAIL_ADDRESS>that PLUS SIGN will trigger this error on L119

I believe it is indeed a bug if we throw a KeyError because of a plus sign in the username? I will take a look at this.
I also think we should support as many custom user models as possible, and clearly document any restrictions we impose in addition to Django.

@jrief That documentation explains the design, which is nice. But I think we also need a more concise, user-focused documentation (by "user", I mean the developer who integrates django-shop as a reuseable app into their project). Maybe some kind of checklist, for example.

@jab3z I tried to reproduce your original issue, but was not able to. Having usernames like<EMAIL_ADDRESS>seems to work fine for me with code similar to what you showed above.
If you give me a reproducible demonstration of the problem (e.g. a minimal project that I can run), I can look into it further.

I fooled around a bit more, and I think I now understand the issue. You hit that error when the username field contains an "invalid" character and the recognized field is set to UNRECOGNIZED. I don't know how you ended up in that situation, maybe with a data migration or such?
However, we should take into account the fact that UNRECOGNIZED is the default value for the field. In order for django-shop to play nice with other systems, it might be a good idea to be more tolerant here. An easy fix would be to catch the exception in BaseCustomer.is_expired().

in my case i did user.username = user.email when a new customer registers
but i could just easily add a guid, its not used anywhere, i use email authentication
this seemed nicer

@rfleschenberg gabor is working on this project. I don't have more things to provide in order to reproduce this issue. It was weird, because it not occurred all the time, only sporadically.

and this happens only in django-admin listing page without any filtering
list_display = ['get_username', 'salutation', 'recognized', 'last_access', 'is_unexpired']
get_username method will trigger the exception

That would be a different issue than the one I analyzed then. In my case, it was is_unexpired that triggered the problem. Can you please show us the full traceback?

Environment:

Request Method: GET
Request URL: http://<IP_ADDRESS>:8011/en/madmin/jumla_shop/usercustomer/

Django Version: 1.9.7
Python Version: 2.7.6
Installed Applications:
[u'modeltranslation',
 u'base.account',
 u'djangocms_admin_style',
 u'django.contrib.admin',
 u'django.contrib.auth',
 u'polymorphic',
 u'django.contrib.contenttypes',
 u'django.contrib.sessions',
 u'django.contrib.messages',
 u'django.contrib.staticfiles',
 u'django.contrib.sites',
 u'reversion',
 u'cmsplugin_cascade',
 u'cmsplugin_cascade.clipboard',
 u'cmsplugin_cascade.extra_fields',
 u'cmsplugin_cascade.sharable',
 u'cmsplugin_cascade.segmentation',
 u'djangocms_text_ckeditor',
 u'cmsplugin_filer_file',
 u'cmsplugin_filer_folder',
 u'cmsplugin_filer_image',
 u'cmsplugin_filer_teaser',
 u'cmsplugin_filer_video',
 u'cms',
 u'treebeard',
 u'menus',
 u'sekizai',
 u'rest_framework',
 u'rest_framework.authtoken',
 u'rest_auth',
 u'shop',
 u'easy_thumbnails',
 u'easy_thumbnails.optimize',
 u'filer',
 u'post_office',
 u'adminsortable2',
 u'django_extensions',
 u'crispy_forms',
 u'el_pagination',
 u'django_filters',
 u'solo',
 u'djng',
 u'newsletter',
 u'ckeditor',
 u'ckeditor_uploader',
 u'nested_inline',
 u'fsm_admin',
 u'captcha',
 u'jumla_shop',
 u'base.account_activation',
 u'base.address',
 u'base.contact_us',
 u'base.cms_slider',
 u'base.common',
 u'base.cms_newsletter',
 u'base.forgot_password',
 u'base.gallery',
 u'base.homepage',
 u'base.social_network',
 u'base.footer_link',
 u'base.search',
 u'base.page_thumbnail',
 u'base.cms_plain_text',
 u'vendor',
 u'discussion',
 u'service_charge',
 u'campaign',
 u'reorder']
Installed Middleware:
[u'djng.middleware.AngularUrlMiddleware',
 u'cms.middleware.utils.ApphookReloadMiddleware',
 u'django.middleware.security.SecurityMiddleware',
 u'django.contrib.sessions.middleware.SessionMiddleware',
 u'django.middleware.locale.LocaleMiddleware',
 u'django.middleware.common.CommonMiddleware',
 u'django.middleware.csrf.CsrfViewMiddleware',
 u'django.contrib.auth.middleware.AuthenticationMiddleware',
 u'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
 u'vendor.middleware.VendorMiddleware',
 u'django.contrib.messages.middleware.MessageMiddleware',
 u'django.middleware.clickjacking.XFrameOptionsMiddleware',
 u'cms.middleware.user.CurrentUserMiddleware',
 u'cms.middleware.page.CurrentPageMiddleware',
 u'cms.middleware.toolbar.ToolbarMiddleware',
 u'cms.middleware.toolbar.ToolbarMiddleware',
 u'cms.middleware.language.LanguageCookieMiddleware']

Template error:
In template /home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/contrib/admin/templates/admin/change_list.html, error at line 91
   +   81 :         {% endif %}
   82 :       {% endblock %}
   83 :
   84 :       <form id="changelist-form" method="post"{% if cl.formset.is_multipart %} enctype="multipart/form-data"{% endif %} novalidate>{% csrf_token %}
   85 :       {% if cl.formset %}
   86 :         <div>{{ cl.formset.management_form }}</div>
   87 :       {% endif %}
   88 :
   89 :       {% block result_list %}
   90 :           {% if action_form and actions_on_top and cl.show_admin_actions %}{% admin_actions %}{% endif %}
   91 :            {% result_list cl %}
   92 :           {% if action_form and actions_on_bottom and cl.show_admin_actions %}{% admin_actions %}{% endif %}
   93 :       {% endblock %}
   94 :       {% block pagination %}{% pagination cl %}{% endblock %}
   95 :       </form>
   96 :     </div>
   97 :   </div>
   98 : {% endblock %}
   99 :

Traceback:

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
  174.                     response = self.process_exception_by_middleware(e, request)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
  172.                     response = response.render()

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/response.py" in render
  160.             self.content = self.rendered_content

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/response.py" in rendered_content
  137.         content = template.render(context, self._request)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/backends/django.py" in render
  95.             return self.template.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render
  206.                     return self._render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/test/utils.py" in instrumented_test_render
  92.     return self.nodelist.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render
  992.                 bit = node.render_annotated(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render_annotated
  959.             return self.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/loader_tags.py" in render
  173.         return compiled_parent._render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/test/utils.py" in instrumented_test_render
  92.     return self.nodelist.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render
  992.                 bit = node.render_annotated(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render_annotated
  959.             return self.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/loader_tags.py" in render
  173.         return compiled_parent._render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/test/utils.py" in instrumented_test_render
  92.     return self.nodelist.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render
  992.                 bit = node.render_annotated(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render_annotated
  959.             return self.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/loader_tags.py" in render
  69.                 result = block.nodelist.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render
  992.                 bit = node.render_annotated(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render_annotated
  959.             return self.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/loader_tags.py" in render
  69.                 result = block.nodelist.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render
  992.                 bit = node.render_annotated(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/base.py" in render_annotated
  959.             return self.render(context)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/template/library.py" in render
  223.         _dict = self.func(*resolved_args, **resolved_kwargs)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/contrib/admin/templatetags/admin_list.py" in result_list
  326.             'results': list(results(cl))}

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/contrib/admin/templatetags/admin_list.py" in results
  302.             yield ResultList(None, items_for_result(cl, res, None))

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/contrib/admin/templatetags/admin_list.py" in __init__
  293.         super(ResultList, self).__init__(*items)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/contrib/admin/templatetags/admin_list.py" in items_for_result
  202.             f, attr, value = lookup_field(field_name, result, cl.model_admin)

File "/home/gabor/.virtualenvs/myjumla/local/lib/python2.7/site-packages/django/contrib/admin/utils.py" in lookup_field
  286.             value = attr(obj)

File "/home/gabor/.virtualenvs/myjumla/src/djngo-shop/shop/admin/customer.py" in is_unexpired
  142.             return not user.customer.is_expired()

File "/home/gabor/.virtualenvs/myjumla/src/djngo-shop/shop/models/customer.py" in is_expired
  285.             session_key = CustomerManager.decode_session_key(self.user.username)

File "/home/gabor/.virtualenvs/myjumla/src/djngo-shop/shop/models/customer.py" in decode_session_key
  81.             n = n * base_length + cls.REVERSE_ALPHABET[c]

Exception Type: KeyError at /en/madmin/jumla_shop/usercustomer/
Exception Value: u'+'

http://jsbin.com/duqeqik/edit?output

https://github.com/awesto/django-shop/blob/master/shop/models/customer.py#L96

That would fix your specific issue, but leave a lot of similar issues open. I am thinking about something roughly like this, maybe:
In shop.models.customer.BaseCustomer:
    def is_expired(self):
        """
        Return true if the session of an unrecognized customer expired.
        Registered customers never expire.
        Guest customers only expire, if they failed fulfilling the purchase (currently not implemented).
        """
        if self.recognized is CustomerState.UNRECOGNIZED:
           try:
               session_key = CustomerManager.decode_session_key(self.user.username)
           except KeyError:
               log.warning('Session key decoding failed')
               return False
           return not SessionStore.exists(session_key)
        return False

But I am not sure if there are other problems with usernames that are not valid session keys.

you are totally right, besides to modify ALPHABET is not a solution, maybe the problem is with the ENCODE-DECODE logic, it should work two way on ANY characters not just a subset, any good codec will not limit the input

My idea was to limit the scope of the change as much as possible, but feel free to send a PR with a (backwards-compatible) improvement to the encoding logic.

I agree that in some edge cases this can cause problems. Didn't think about in the first place.
Just created PR #475 which is pending.


autodoc always showing special members

Details

Project URL: http://geni-lib.readthedocs.io/en/latest/index.html
Build URL (if applicable):
Read the Docs username (if applicable): barnstorm

Expected Result
No special members in doc output
Actual Result
The last few builds, regardless of settings (have tried setting autodoc defaults to no-special-members in conf.py, also setting it on individual pages), if you have undoc-members set, special python methods will also always be included, cluttering up the docs with entries for things like __getitem__ and __delattr__, etc.
You can see an example at:
http://geni-lib.readthedocs.io/en/latest/api/geniaggregate/vts.html

Actually I think the problem might be more directly related to setting inherited-members, since apparently object and Exception have documented special members in the python doc, and it follows the inheritance tree all the way down.  Still, we don't have special-members set, so I don't know why these show up at all (documented or not).

This sounds like an issue with Sphinx, not Read the Docs specifically. The actual functionality of the builds is based on how Sphinx works, and likely this is a bug that should be filed on their tracker.

This is an issue with the specific setup of sphinx at RTD, it seems, as I do not have this problem making sphinx builds locally.


[web-ui] Make app dir to work as intended

Thanks to @sudarshanmg and #116

Love if you can take a look here! Ask any questions if you want to @sudarshanmg


With toObject(), ObjectIDs aren't stringified.

This might be a documentation fix or potentially a code update to check if id is an ObjectID and stringify it here: https://github.com/feathersjs/feathers-mongoose/blob/master/src/service.js#L90

This is only an issue with inter-service communication, internally.

@marshallswain is this still an issue? I've never run into this issue with the latest version, although I remember in the feathers-mongoose < 3 having to do some ObjectId casting.

I don't think it is.  Let's close it.  I can always reopen if needed.


Update release packaging to include Swift frameworks built with Xcode 9.3

This is in preparation for the release of Xcode 9.3, and should not be merged until Xcode 9.3 is out of beta, and the GM release of it has been installed on lvhost1.
You can see the test run of the pipeline here which appears to show this working as expected.
I've not added Xcode 9.3 to the PR matrix jobs as we only have a single worker running Xcode 9.3 and we'd likely run into throughput issues with it.

Hey, can you merge it?

Might this solve #5709 ?

How can I use it with xcode 9.3?
I get always this error:
*** Downloading realm-cocoa.framework binary at "v3.3.2" *** Skipped installing realm-cocoa.framework binary due to the error: 	"Incompatible Swift version - framework was built with 4.0.3 (swiftlang-9<IP_ADDRESS> clang-9<IP_ADDRESS>) and the local version is 4.1 (swiftlang-902.0.48 clang-9<IP_ADDRESS>)."


Close issue #19

resolve #19. Clicking on Sun button preserves lock on the selected asteroid and doesn't remove its orbit. Add 'Reset' button to deselect asteroid and reset view (can also be helpful to restore view from weird camera angles and scales). Earth selection behaviour is intentionally left unchanged as asteroid tracking can be a bit complicated from the Earth-locked view.

Thanks Vadym for this excellent change :+1:
Could you help me understand the difference between clearLock and dropLock in the code?

dropLock clears selection even if locked_object == null, while original method clearLock skip this action due to this line of code at the top: if (!locked_object) return. I implemented dropLock in order to force clearing selection from resetView method, which emulates absence of lock through declaration locked_object = null at the top. This declaration is needed to reset camera. A bit confusing - I know, but this is how I made it :)
Please do not merge yet for I've just found a little bug in my ResetView method. I hope I'll fix it next week.


Teamnado 3919 to prod-stable

Codecov Report
Base: 89.68% // Head: 90.10% // Increases project coverage by +0.41% :tada:

Coverage data is based on head (91580d7) compared to base (762fc41).
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff               @@
##           prod-stable      #93      +/-   ##
===============================================
+ Coverage        89.68%   90.10%   +0.41%
===============================================
  Files               42       42
  Lines              659      697      +38
  Branches           121      132      +11
===============================================
+ Hits               591      628      +37
- Misses              68       69       +1

Impacted Files
Coverage Δ

...es/SatelliteManifestPage/SatelliteManifestPage.tsx
100.00% <ø> (ø)

...lliteManifestPanel/satelliteManifestPanelUtils.tsx
97.97% <0.00%> (-0.39%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Unnecessary field name string copy?

Since string literals have static storage duration, is the copy here required? It should be enough o simply initialize a string view with pointer and size.
https://github.com/getml/reflect-cpp/blob/8f7c84954c7ce541ea75e1b820d74d29ac587f04/include/rfl/internal/StringLiteral.hpp#L17

I'm pretty sure it is. It's not only static, it is constexpr static.
It is necessary to support this syntax:
rfl::Field<std::string, "firstName">

So at compile time, the string "firstName" is copied into the class itself. I do not think that "firstName" would be persisted so I am pretty sure you couldn't just create a view.
Also, it is very important that this is all happening at compile time. So there really is no runtime overhead either way.

Looks like https://godbolt.org/ tricked me - string_view used to work once, but not any more.
Maybe in the future C++ versions it will work.
After translation phase 6, a string-literal that does not begin with an encoding-prefix is an ordinary string
literal. An ordinary string literal has type “array of n const char” where n is the size of the string as defined
below, has static storage duration (6.7.5), and is initialized with the given characters

Yes, I gave a try myself and couldn't make it work either.


Add placeholder to empty application list

Before

After

@Hyperkid123 @terezanovotna

Codecov Report

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

@@           Coverage Diff           @@
##           master     #114   +/-   ##
=======================================
  Coverage   74.82%   74.82%
=======================================
  Files          36       36
  Lines         433      433
  Branches       61       62    +1
=======================================
  Hits          324      324
  Misses         87       87
  Partials       22       22

Impacted Files
Coverage Δ

src/components/SourcesSimpleView/formatters.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Specify dependency on aws-crt-cpp >=0.19.8

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


Magic constants

https://github.com/Adivius/Graphite/blob/a7cadffdfb35faf635afbeab56db2b92f60b12f2/Core/Screen.java#L36
https://github.com/Adivius/Graphite/blob/a7cadffdfb35faf635afbeab56db2b92f60b12f2/Core/Screen.java#L70

Closed in 9f1ffd13824d0b9b7babf26fd2f9d2723257f202


Notebook removal

Removing notebooks that are now hosted elsewhere.

UNION ALL
TopN

We'll remove these as part of removing all the notebooks in 29.


select_autoescape default value for default_for_string should be False

According to jinja's select_autoescape documentation

The select_autoescape() function returns a function that works roughly like this:

def autoescape(template_name):
if template_name is None:
return False

Furthermore, the docs makes it clear that jinja's default autoescape is still False. So, If I setup autoescape like this:
from jinja2 import Environment, select_autoescape
env = Environment(autoescape=select_autoescape())

I would expect autoescape to end up as False
However,  in the code here when template_name is None, it returns the default value of default_for_string, which at this time is True.
Should default_for_string not be False, just like default?
Environment:

Python version:
Jinja version: 3.0.x

I don't plan to change the param defaults at this time. Both are configurable, to whatever behavior the application needs.


A script to run Gemini models

Adding a script to run Gemini models with a text prompt.
Steps:
python -m venv .venv
pip install -r requirements.txt
gcloud auth login && gcloud auth application-default login && gcloud auth application-default set-quota-project oss-fuzz
python -m experimental.manual.prompter -p prompt.txt -l vertex_ai_gemini-1-5
ls `./responses/`

See the name attribute for more supported model names.

/gcbrun skip

TBR.

Thanks so much. How do you also provide a project source code, along with the prompt, or do you just include it inside the prompt.


This symbol is required by 'method org.http4s.EntityEncoderInstances.entityEncoderContravariant'.

Hi all
I have following code, that do not compiled:
import _root_.io.circe._
import org.http4s.dsl.io._
import org.apache.kafka.streams.state.{ReadOnlyKeyValueStore, StreamsMetadata}
import cats.effect._
import org.apache.kafka.streams.KafkaStreams
import org.http4s._
import org.http4s.dsl.impl.Root
import org.http4s.server.Server
import org.http4s.server.blaze.BlazeBuilder
import io.circe._
import io.circe.syntax._
import io.circe.generic.auto._

import scala.collection.JavaConverters._

final case class Instances(host: String, port: Int, stores: Set[String])

object PathRest {

  def server
  : KafkaStreams => ReadOnlyKeyValueStore[String, String] => String => Int => IO[Server[IO]]
  = stream => store => hostname => port => {

    BlazeBuilder[IO].bindHttp(port, hostname)
      .mountService(storeService(store), "/store")
      .mountService(streamService(stream), "/stream")
      .start
  }

  private def streamService
  : KafkaStreams => HttpService[IO]
  = stream =>
    HttpService[IO] {
      case GET -> Root / "instances" =>
        Ok()
    }

  private def storeService
  : ReadOnlyKeyValueStore[String, String] => HttpService[IO]
  = store => {
    HttpService[IO] {
      case GET -> Root / "value" =>
        read(store)
      case _ =>
        IO(Response(Status.Ok))
    }
  }

  private def read
  : ReadOnlyKeyValueStore[String, String] => IO[Response[IO]]
  = store => {
    val v = store.get("PATH-STREAM-DATA")
    if (v.isEmpty)
      toJson(NotFound())
    else
      toJson(Ok(v))
  }

  private def instances
  : KafkaStreams => IO[Response[IO]]
  = stream => {
    val metas: List[Instances] = stream
      .allMetadata().asScala.toList
      .map(m => Instances(m.host(), m.port(), m.stateStoreNames().asScala.toSet))
    Ok(metas.asJson.noSpaces)
  }

  private def toJson
  : IO[Response[IO]] => IO[Response[IO]]
  = a =>
    a.map(res => res.withContentType(Some(MediaType.`application/json`)))

}

The compiler complains:
[IJ]sbt:path-stream> compile
[info] Updating {file:/home/developer/Desktop/microservices/path-stream/}path-stream...
[info] Done updating.
[warn] Found version conflict(s) in library dependencies; some are suspected to be binary incompatible:
[warn] 	* org.log4s:log4s_2.12:1.4.0 is selected over 1.3.3
[warn] 	    +- org.http4s:http4s-core_2.12:0.18.0-M2              (depends on 1.4.0)
[warn] 	    +- org.http4s:blaze-core_2.12:0.12.9                  (depends on 1.3.3)
[warn] 	    +- org.http4s:blaze-http_2.12:0.12.9                  (depends on 1.3.3)
[warn] 	* org.typelevel:cats-core_2.12:1.0.0-RC1 is selected over 1.0.0-MF
[warn] 	    +- io.circe:circe-core_2.12:0.9.0-M2 ()               (depends on 1.0.0-RC1)
[warn] 	    +- org.typelevel:cats-effect_2.12:0.4                 (depends on 1.0.0-MF)
[warn] Run 'evicted' to see detailed eviction warnings
[info] Compiling 3 Scala sources and 4 Java sources to /home/developer/Desktop/microservices/path-stream/target/scala-2.12/classes ...
[error] /home/developer/Desktop/microservices/path-stream/src/main/scala/io/khinkali/PathRest.scala:27:18: Symbol 'type cats.functor.Contravariant' is missing from the classpath.
[error] This symbol is required by 'method org.http4s.EntityEncoderInstances.entityEncoderContravariant'.
[error] Make sure that type Contravariant is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'EntityEncoderInstances.class' was compiled against an incompatible version of cats.functor.
[error]     BlazeBuilder[IO].bindHttp(port, hostname)
[error]                  ^
[error] one error found
[error] (compile:compileIncremental) Compilation failed
[error] Total time: 3 s, completed Nov 29, 2017, 4:36:27 PM

Something wrong the with libraries?
Thanks

The error message is telling you that you have both cats 1.0-MF and 1.0-RC in your dependencies.
You need cats-effect-0.5 to avoid the dependency on cats-1.0-MF.


Want to add dynamic images from a list inside a Carousel

Hey,
I am using carousel_pro.dart library and I have a task to populate it with the images from my database. I have already populated the imaged from the database to a list but now when I am trying to populate those images inside a carousel I am not able to I am getting an error
Getting data from the database :

getBannerImage() async {
final response = await http.get(myDatabaseAPI);
setState(() {
var datauser = json.decode(response.body);
listcount = datauser['acf']['banner_custom_field'].length;
banner_image_list.clear();
for (var i = 1; i <= listcount; i++) {
if (datauser['acf']['banner_custom_field']['mobile_banner$i'] !=false) {
banner_count++;
banner_image_list.add(datauser['acf']['banner_custom_field']['mobile_banner_$i']['url']);
}
}
});
}

Carousel code inside the build method

Widget banner_images = new Container(
height: MediaQuery.of(context).size.height * 0.26,
child: new Carousel(
boxFit: BoxFit.fitWidth,
images: [
Image.network(
banner_image_list[2],
),
],
autoplay: false,
dotSize: 4.0,
indicatorBgPadding: 4.0,
dotBgColor: Colors.transparent,
dotColor: Colors.black,
),
);

Hi @TheParthShah
This platform is not met for assistance on personal code.
Please see https://flutter.dev/community for resources for asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Feature question: linear constraints

Hi. Thanks again for this great package.
I am wondering whether you have any plans to implement linear constraints in SPGBox. How to do it is way beyond anything I know, but as I understand it, I think this is possible with spectral projection gradient methods. For example, the R package BB apparently allows it --see this, p.13.
Many thanks.
Don

Actually I have a plan to support general non-linear constraints, but it is not something I plan to do in any near future. I'm not sure either if that makes sense within this package or if it will be another independent implementation with this one as a dependency.

This is only my two cents. But I am not aware of someone using spectral methods to deal with general nonlinear constraints successfully. I know that Mario (Martínez) and Ernesto (Birgin) tried to use a spectral method in an Augmented Lagrangian framework, and it did not perform well. It was worse than Quacan, the method they use to solve the subproblems of the augmented Lagrangian in their Algencan implementation. As a matter of fact, if you want to use Algencan in Julia you can use the NLPModelsAlgencan.jl package (https://github.com/pjssilva/NLPModelsAlgencan.jl). There is also the very good Ipopt solver that has a package to use it in Julia.

Great to hear that! It will make the package (or independent implementation) even better.

Ops... I forgot to mention. An intermediate alternative is to make the method receive the projection onto the feasible set as a function, instead of hard-coding a specific set (a box). This is an easy change to make Leandro, and then the user that wants to deal with different constraints should be responsible to give the function that implements the projection. We can give some pre-defined projections like Box, Ball, Simplex (I have a good method for that). Unfortunately, projection onto a general polyhedron is not that easy. It is easy to implement (using a general quadratic programming solver), but the projection can be quite expensive and this would slow down the method.

Indeed, in fact JMM passed me a routine with the AL-SPG implementation, and that is what I was planning to port here. But I don't know how well it performs.


Documentation and example on bare bytes API

I checked out the bare-bytes API for use with FFI. The API is easy enough for people who knows a bit about Elliptic Curves and Diffie-Hellman, but can be pretty confusing for people with limited knowledge.
Without any modification to the code itself, I suggest to add a usage example to the x25519() function in the docs.
I can write the example, test it and make a PR, but I just wanted to know your input about it before doing it!

Please do it, that would be awesome

Thanks for this @zer0x64!


Installation issue (dependency error).

Hello CellScopes team,
I have encountered a dependency error while installing the CellScope package – Unsatisfiable requirements for package ImageTransformations. Here is the traceback:
ERROR: Unsatisfiable requirements detected for package ImageTransformations [02fcd773]:
 ImageTransformations [02fcd773] log:
 ├─possible versions are: 0.2.0-0.10.0 or uninstalled
 ├─restricted by compatibility requirements with Colors [5ae59095] to versions: 0.8.1-0.10.0 or uninstalled
 │ └─Colors [5ae59095] log:
 │   ├─possible versions are: 0.9.0-0.12.10 or uninstalled
 │   └─restricted to versions 0.12 by Baysor [cc9f9468], leaving only versions: 0.12.0-0.12.10
 │     └─Baysor [cc9f9468] log:
 │       ├─possible versions are: 0.6.2 or uninstalled
 │       └─Baysor [cc9f9468] is fixed to version 0.6.2
 ├─restricted by compatibility requirements with Augmentor [02898b10] to versions: 0.5.0-0.8.13, leaving only versions: 0.8.1-0.8.13
 │ └─Augmentor [02898b10] log:
 │   ├─possible versions are: 0.6.0-0.6.6 or uninstalled
 │   └─restricted to versions * by CellScopes [708d50d9], leaving only versions: 0.6.0-0.6.6
 │     └─CellScopes [708d50d9] log:
 │       ├─possible versions are: 0.1.0 or uninstalled
 │       └─CellScopes [708d50d9] is fixed to version 0.1.0
 └─restricted by compatibility requirements with Images [916415d5] to versions: 0.9.0-0.9.5 — no versions left
   └─Images [916415d5] log:
     ├─possible versions are: 0.17.3-0.26.0 or uninstalled
     ├─restricted to versions * by CellScopes [708d50d9], leaving only versions: 0.17.3-0.26.0
     │ └─CellScopes [708d50d9] log: see above
     ├─restricted by compatibility requirements with Colors [5ae59095] to versions: 0.19.3-0.26.0 or uninstalled, leaving only versions: 0.19.3-0.26.0
     │ └─Colors [5ae59095] log: see above
     └─restricted by compatibility requirements with ImageMorphology [787d08f9] to versions: 0.25.0-0.25.3 or uninstalled, leaving only versions: 0.25.0-0.25.3
       └─ImageMorphology [787d08f9] log:
         ├─possible versions are: 0.1.0-0.4.5 or uninstalled
         └─restricted to versions 0.3 by Baysor [cc9f9468], leaving only versions: 0.3.0-0.3.2
           └─Baysor [cc9f9468] log: see above

I am only starting with Julia and I couldn't figure where these requirements are stated explicitly, nor how to fix the issue. I am running Julia v1.9.3. and installing the package using instructions from readme.
Best regards,
Vasily

@tsvvas Thanks for your interest in our package. We have not tested on Julia v1.9.3. Maybe there are some dependency issues. I am using Julia V1.7.3 currently. I also tested CellScopes.jl on Julia V1.8.+ as well and it worked. You may give it a try on V1.7.3+ or V1.8 and let me know if there is any issue. We will test it on V1.9.+ on the next phase of the package development.

I had the same problem. There was a compatibility issue with several packages which were installed as dependency for Gadfly in my case. In your case, try to remove "ImageTransformations", "Colors", "Augmentor", and/or "Images". Did it for me.

I will test it on a newer version of Julia later. The issues you came across are most likely the dependencies compatibility issue. That happened to me when I used other Julia packages. I will fix it later. Stay tuned.

Hi @tsvvas @MaximilianNuber , I made a new branch for Julia 1.9 to fix the dependency issue. Please re-install the package using the commands below:
using Pkg
Pkg.add(url="https://github.com/bicycle1885/Leiden.jl")
Pkg.add(url="https://github.com/HaojiaWu/CellScopes.jl#julia_v1_9")

Let me know if you find any issues.

Running into the same problem here. Tried it with Julia 1.7.3/1.8.0/1.9.3 and with the fix you proposed. As soon as I try to install Baysor and CellScopes is already installed this error persists.
ERROR: Unsatisfiable requirements detected for package ImageTransformations [02fcd773]:
 ImageTransformations [02fcd773] log:
 ├─possible versions are: 0.2.0-0.10.0 or uninstalled
 ├─restricted by compatibility requirements with Colors [5ae59095] to versions: 0.8.1-0.10.0 or uninstalled
 │ └─Colors [5ae59095] log:
 │   ├─possible versions are: 0.9.0-0.12.10 or uninstalled
 │   └─restricted to versions 0.12 by Baysor [cc9f9468], leaving only versions 0.12.0-0.12.10
 │     └─Baysor [cc9f9468] log:
 │       ├─possible versions are: 0.6.2 or uninstalled
 │       └─Baysor [cc9f9468] is fixed to version 0.6.2
 ├─restricted by compatibility requirements with Images [916415d5] to versions: 0.9.0-0.9.5
 │ └─Images [916415d5] log:
 │   ├─possible versions are: 0.17.3-0.26.0 or uninstalled
 │   ├─restricted to versions * by CellScopes [708d50d9], leaving only versions 0.17.3-0.26.0
 │   │ └─CellScopes [708d50d9] log:
 │   │   ├─possible versions are: 0.1.0 or uninstalled
 │   │   └─CellScopes [708d50d9] is fixed to version 0.1.0
 │   └─restricted by compatibility requirements with ImageMorphology [787d08f9] to versions: 0.25.0-0.25.3 or uninstalled, leaving only versions: 0.25.0-0.25.3
 │     └─ImageMorphology [787d08f9] log:
 │       ├─possible versions are: 0.1.0-0.4.5 or uninstalled
 │       └─restricted to versions 0.3 by Baysor [cc9f9468], leaving only versions 0.3.0-0.3.2
 │         └─Baysor [cc9f9468] log: see above
 └─restricted by compatibility requirements with Augmentor [02898b10] to versions: 0.5.0-0.8.13 — no versions left
   └─Augmentor [02898b10] log:
     ├─possible versions are: 0.6.0-0.6.6 or uninstalled
     └─restricted to versions * by CellScopes [708d50d9], leaving only versions 0.6.0-0.6.6
       └─CellScopes [708d50d9] log: see above

As I am not that familiar with Julia I'm not sure how to get it to work
Hope you can help

Running into the same problem here. Tried it with Julia 1.7.3/1.8.0/1.9.3 and with the fix you proposed. As soon as I try to install Baysor and CellScopes is already installed this error persists.
ERROR: Unsatisfiable requirements detected for package ImageTransformations [02fcd773]:
 ImageTransformations [02fcd773] log:
 ├─possible versions are: 0.2.0-0.10.0 or uninstalled
 ├─restricted by compatibility requirements with Colors [5ae59095] to versions: 0.8.1-0.10.0 or uninstalled
 │ └─Colors [5ae59095] log:
 │   ├─possible versions are: 0.9.0-0.12.10 or uninstalled
 │   └─restricted to versions 0.12 by Baysor [cc9f9468], leaving only versions 0.12.0-0.12.10
 │     └─Baysor [cc9f9468] log:
 │       ├─possible versions are: 0.6.2 or uninstalled
 │       └─Baysor [cc9f9468] is fixed to version 0.6.2
 ├─restricted by compatibility requirements with Images [916415d5] to versions: 0.9.0-0.9.5
 │ └─Images [916415d5] log:
 │   ├─possible versions are: 0.17.3-0.26.0 or uninstalled
 │   ├─restricted to versions * by CellScopes [708d50d9], leaving only versions 0.17.3-0.26.0
 │   │ └─CellScopes [708d50d9] log:
 │   │   ├─possible versions are: 0.1.0 or uninstalled
 │   │   └─CellScopes [708d50d9] is fixed to version 0.1.0
 │   └─restricted by compatibility requirements with ImageMorphology [787d08f9] to versions: 0.25.0-0.25.3 or uninstalled, leaving only versions: 0.25.0-0.25.3
 │     └─ImageMorphology [787d08f9] log:
 │       ├─possible versions are: 0.1.0-0.4.5 or uninstalled
 │       └─restricted to versions 0.3 by Baysor [cc9f9468], leaving only versions 0.3.0-0.3.2
 │         └─Baysor [cc9f9468] log: see above
 └─restricted by compatibility requirements with Augmentor [02898b10] to versions: 0.5.0-0.8.13 — no versions left
   └─Augmentor [02898b10] log:
     ├─possible versions are: 0.6.0-0.6.6 or uninstalled
     └─restricted to versions * by CellScopes [708d50d9], leaving only versions 0.6.0-0.6.6
       └─CellScopes [708d50d9] log: see above

As I am not that familiar with Julia I'm not sure how to get it to work
Hope you can help

Hi @ddcham, I'm curious which operating system were you installing Julia and CellScopes on? Linux, Windows, or Mac? And also, did you start from a brand new Julia environment? Or your Julia environment has other packages installed previously?

Hey @BuilderPie,
I am running Linux on a dedicated server and this was a fresh Julia installation

I did exactly as you did in your notes. When I am then trying to install Baysor through:
using Pkg;Pkg.add(PackageSpec(url="https://github.com/kharchenkolab/Baysor.git")); Pkg.build()
it gives me the Error outlined above.
However I dont even get to the point where I need baysor in the xenium analysis anymore as it throws another error while doing
@time brain = cs.read_xenium(xenium_dir; min_gene = 0, min_cell = 0, prefix = "brain")
Here I get a never ending string of:
 SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}, SparseArrays.SparseVector{Float64, Int64}).
This might be caused by recursion over very long tuples or argument lists.

I get the same error if installing Baysor first before CellScopes, and also the other way around.
julia> Pkg.add(url="https://github.com/TheHumphreysLab/CellScopes.jl")
Updating git-repo https://github.com/TheHumphreysLab/CellScopes.jl
Resolving package versions...
ERROR: Unsatisfiable requirements detected for package ImageTransformations [02fcd773]:
ImageTransformations [02fcd773] log:
├─possible versions are: 0.2.0-0.10.1 or uninstalled
├─restricted by compatibility requirements with Compat [34da2185] to versions: 0.5.0-0.10.1 or uninstalled
etc..


Pre-configured image link doesn't work

Good morning. The link for the pre-configured image doesn't work anymore.
Can you guys re-upload the image on another host?

Hi @gringoh , we are working on a new Test Environment. As you say, the link you have mentioned, it doesn't work. However, if you want, we can talk through Slack and I can give you virtual machine so you can work with it.

@cmoralesdiego that would be great! I'll be contacting you through Slack

Hello all, I think the link still points to my old mega.nz account that was closed a few months ago.
I've managed to recover a copy of the VM from an old backup and I've uploaded to a new mega account.
https://mega.nz/file/AQckxYwZ#ykMYRL6Yo2ctr11W0KTs14wZQEGJ0iSKEz56uolGRfI
It's a copy of the original VM from the broken link.
Could you please download the image and placed in other host so you don't depend on my own link?
Once you have it, I would deleted from my side as is 10Gb.
Thanks.
Hope it helps.

Thanks @netmanito , this afternoon, I will download it and you will be able to remove it. I will do it at the my 22 hours. (Spanish time)

thank you very much @netmanito. I will let you know as soon I finish downloading it

@netmanito I have finished downloading the image. Thank you very much!

@netmanito , you can remove the link if you want.

Hi @cmoralesdiego and @gringoh,
Have you tried if it works?
The user and passwd is the one already documented in the repo.
If everything is fine and you don't need it anymore, I'll remove the link, but I can wait a few days if you need so.
thanks

@netmanito hi, I have tried to run the ova, and it doesn't work when I am trying to import it. It fails.

@netmanito I have just made run the virtual machine. Thnx

As for today, the prebuilt virtual machine link is not working.

Thanks for the feedback: i'll search for a solution


Documentation feedback/best practices

I've been straight up pushing directly to the documentation branch instead of submitting pull requests, will be submitting PRs from now on. Just wanted to put stuff on the board. Can we come up with a sort of standard for handling documentation?
Here's what we have on the site so far:
https://use-stripe-cart.netlify.app/usage/addItem()
Can I get some feedback on the example code being shown? And also anything else on this site so far.
@Developerayo @ChrisBrownie55 @doingandlearning

I'd say the best way to go about adding documentation is to give a complete description, more like a paragraph as opposed to a short "To do  use <method()>." Then, provide a few small examples without surrounding code (i.e. no imports or other components) just how to use it. And then when that's all done, I'd recommend adding in a full project example with a link to something like a CodeSandbox and/or a CodePen (could be embedded for easily editing from the documentation).

@ChrisBrownie55 okay, completely agree. I'll start submitting PRs for documentation for your feedback


Add the new Facebook Built-in NLP

Hello,
This PR add the new Facebook Built-in NLP to Botkit to understand and extract meaningful information out of the messages people send.
Enjoy

I merged in these changes with some mild changes.
The API is now at controller.api.nlp

Sound good 👍


Update README to instruct which version of Swinject to use

@jakubvano, I updated README to tell which version of Swinject (v1 or v2) to use. Do you think my update is clear enough?
If it's ok, merge this PR please? If not, your advice is appreciated😃

👍 I think it is understandable. Especially liked the #Uncoment .. - it is nice and succinct way of highlighting SwinjectStoryboard usage.

Thanks😃


NameError (uninitialized constant Sync::Actions):

Hi!
I added the include Sync::Actions in my form object class expecting to be able to use the Sync::Model.enable and saving my record. But I got this error: NameError (uninitialized constant Sync::Actions).
Any ideia what could it be?
Thanks, Paulo

Change Sync:Model to RenderSync

@paulofabiano Hey, could you get this work? For me it worked.
Maybe would be nice to sent a PR fixing the old method for this new one 😄


Modal backdrop no longer shows in 2.3.0

Describe the bug
I upgrades from 2.0.0 to 2.3.0. The opaque backdrop div is no longer rendered with modals; previously it was inserted as the last child in the body (although there was a bug where several could be added).
To Reproduce
I've tried both static and dynamic modals. Static certainly adds a backdrop of some kind in that background elements are not clickable. However, the opaque layer is not shown.
I tried setting backdropClasses as an attribute, but that doesn't appear to work.

Hey @reubenfirmin,
Please update to Tailwind CSS v3.0+. The modal backdrop is still there, but the gradient classes are now changed from the deprecated ones to the new ones: bg-gray-900/50 dark:bg-gray-900/80 fixed inset-0 z-40.
Cheers,
Zoltan

@zoltanszogyenyi I do see the modal-backdrop div in your samples, but it's no longer appearing in mine. I.e. there's literally no modal-backdrop element present.
Here's my body, vs yours.

(I was on tailwind 3.3.5, upgraded to 3.4.0 now but no diff)

Can you double-check the installation? Try clearing the cache. If it doesn't work, can you send the code over some environment so that I can reproduce it? Thanks!

@zoltanszogyenyi OK - did a clean build, flushed my cache, and also tried another browser.
You'll have to squint a bit, since this is kotlin, but hopefully interpretable:
class Modal(private val modalId: String, consumer: TagConsumer<*>):
    DIV(mapOf("class" to "hidden overflow-y-auto overflow-x-hidden fixed top-0 right-0 left-0 z-50 justify-center " +
            "items-center w-full md:inset-0 h-[calc(100%-1rem)] max-h-full"), consumer) {

    fun render(block: FlowContent.() -> Unit) {
        id = modalId
        tabIndex = "-1"
        attributes["aria-hidden"] = "true"
        attributes["data-modal-backdrop"] = "static"
        div("relative p-4 w-full max-w-2xl max-h-full") {
            div("relative bg-white rounded-lg shadow dark:bg-gray-700") {
                block()
            }
        }
    }
}

@zoltanszogyenyi Hey, since this is closed, did you lose track of it? Still an issue.

For some reason, I've got the same issue
I find that manually adding this to the complied CSS file fixed my problem
.inset-0{     inset: 0px;   }

Hey everyone,
Just released v2.4.1 with an update to the modals.
Is this still happening to you alll?
If it does, feel free to re-open the issue.
Cheers,
Zotlan

Still happens to me, no backdrop at all.

Hi @zoltanszogyenyi  It still happens for me. No modal-backdrop element, same as OP. Rendering with turbo stream. Tried with CDN tags and code from example as follows. All console logs are firing. Thanks
<script<EMAIL_ADDRESS>
`import { Controller } from "@hotwired/stimulus"

import { initFlowbite } from "flowbite"
import { Modal } from 'flowbite';
export default class extends Controller {
connect() {
console.log('Remote modal controller connected');
const options = {
placement: 'center-center',
backdrop: 'dynamic',
backdropClasses:
'bg-gray-900/50 dark:bg-gray-900/80 fixed inset-0 z-40',
closable: true,
onHide: () => {
console.log('modal is hidden');
},
onShow: () => {
console.log('modal is shown');
},
onToggle: () => {
console.log('modal has been toggled');
},
};
this.modal = new Modal(this.element, options);
this.modal.show();
initFlowbite();
}
disconnect() {
this.modal.destroy()
}
}`

I had the same problem, what I had to do was to set the inset: 0 and opacity: 0.5 (any value) manually. I don't know if it's an error from Flowbite because, to me, it looks like the classes from Tailwind were not loading properly.
BTW I'm using the tailwindcss-rails gem.

Hey! Any update with this issue?

The issue seems to be with Tailwind not recognising that the classes are used and shaking them out of the build.
I couldn't get the backdrop to render even on the latest version of Flowbite, but adding the classes used by the overlay to the safelist in the Tailwind config fixes the issue.
{
  safelist: ['fixed', 'bg-gray-900/50', 'dark:bg-gray-900/80', 'inset-0', 'z-30'],
}

Obviously will break again if these classes change, but solves the issue.

Guys, found and solved the issue. Tailwind does not add a component in css if it is not in the static files. The model adds class dynamically to the page and the css is not built because of it. So just open the inspector copy the backdrop classes paste it in a hidden div and it will auto build and work. Hope it works, worked for me!

@zoltanszogyenyi do something about it

@joshiabir you're wrong a bit. I've researched this issue and the problem is related to flowbite/plugin.js It does contain safelist mentioned earlier and because of tailwind to purge CSS of the unused classes, but something is wrong with plugin.css which contains safelist - there is an issue with backdrop. I've described this problem here https://github.com/themesberg/flowbite/issues/998

I actually used the safeList method and it seems to work for me! Though if the classes are changed in future might not work again!
{
safelist: ['fixed', 'bg-gray-900/50', 'dark:bg-gray-900/80', 'inset-0', 'z-30'],
}
Just by putting the above in my tailwind.config.js the modal overlay became visible!


Rename functions-array to functions-nested

Which issue does this PR close?

Closes #11598 .
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @alamb and @jayzhan211 !


feat: optional shieldPath

Description
Currently withShield generates a boilerplate configuration using allow for all procedures, and overwrites it on each generation.
This PR adds a new option shieldPath that allows your to provide a path to your own tRPC Shield to use as the permission layer.
Alternatively, we could extend withShield to allow either a boolean (existing functionality) or a string path. I decided on shieldPath to be consistent with the other options contextPath and trpcOptionsPath.
References
Example input:

shieldPath: https://github.com/lottamus/prisma-trpc-generator/blob/1c5695135815b904efe59d166e818e42d05d3a6e/prisma/schema.prisma#L11
custom shield config: https://github.com/lottamus/prisma-trpc-generator/blob/1c5695135815b904efe59d166e818e42d05d3a6e/prisma/shield.ts#L8

Example output:
prisma/generated/routers/helpers/createRouter.ts

When I set these options:
  withShield        = true
  shieldPath        = "./shield/my-shield"

The outputted shield is not in the correct path, nor the import is correct inside createRouter

Hey, can we merge the options instead?
Same field withShield. This will make it accept either a boolean to enable/disable generation or a string path, to use an existing shield.

I resolved the conflicts and addressed @omar-dulaimi comments here: https://github.com/omar-dulaimi/prisma-trpc-generator/pull/78
Tysm 🙏🏽

Closing in favor of https://github.com/omar-dulaimi/prisma-trpc-generator/pull/78
Thank you for your work regardless though :)


Update to Gradle 6.0

Update to Gradle 6.0 by executing the following:
./gradlew wrapper --gradle-version=6.0 --distribution-type=all --gradle-distribution-sha256-sum=a1eb4439c0a85bc7e64a22658d862e43b7d0ddfbf69a7abf6256e0b7514295df

However, we cannot currently do this until the bintray-release plugin supports it, see https://github.com/novoda/bintray-release/issues/298.

The issue in question has been hanging around for a long time now, maybe replacing the plugin is the better long term solution here?

Done in #89


fix/monitor: AutoScalingNotifier.ShouldNotify should call NotifierBase.ShouldNotify

这个 PR 实现什么功能/修复什么问题:

是否需要 backport 到之前的 release 分支:

release/3.2

/area monitor
/cc @zexi @zhaoxiangchun

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6147   +/-   ##
=======================================
  Coverage    7.03%    7.03%
=======================================
  Files         646      646
  Lines      110826   110826
=======================================
  Hits         7801     7801
  Misses     102424   102424
  Partials      601      601

Flag
Coverage Δ

#aFlag
7.03% <ø> (ø)

Continue to review full report at Codecov.

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


added logseq-powertags-plugin

@xyhp915 appreciate your help that this plugin needs to be whitelisted as it interacts with the DOM directly through a mutation observer.
Submit a new Plugin to Marketplace
Plugin Github repo URL: https://github.com/hkgnp/logseq-powertags-plugin
Github releases checklist

[x] a legal package.json file.
[x] a valid CI workflow build action for Github releases. (Optional for theme plugin only).
[x] a release which includes a release zip pkg from a successful build.
[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

Plugin Github repo URL: https://github.com/hkgnp/logseq-powertags-plugin

Looks like you renamed the repo?
https://github.com/hkgnp/logseq-power-plugin

Nope, different plugin!
On Fri, Dec 16, 2022 at 1:01 AM Bad3r @.***> wrote:

Plugin Github repo URL: https://github.com/hkgnp/logseq-powertags-plugin
Looks like you renamed the repo?
https://github.com/hkgnp/logseq-power-plugin
—
Reply to this email directly, view it on GitHub
https://github.com/logseq/marketplace/pull/299#issuecomment-1353405551,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALNXKGMPWJ65NTPXP4DDLITWNNFFTANCNFSM6AAAAAAS745OYI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Activating /check in Github Action

Seems like Microsoft does not quite know what makes it into which SDK release
Seems --check is not in .NET Preview 7


Cache invalidation

Investigate Progressive Web App offline capability and options for updating the app to the latest version without the user having to clear the cache.
Questions:

Why is a hard refresh required in some cases?
What are the options for triggering the update?

A potential solution for this problem would be to show a notification to the user informing them that a new version is available and a refresh button (https://dev.to/daniellycosta/showing-new-version-available-notification-on-create-react-app-pwas-3jc9).

Problem
Users have to do a hard refresh or refresh and then close and reopen the tab of the app to see a new version. This is because the caching strategy provided by create-react-app is offline-first and because browser can't switch to the new service worker until all tabs using it are closed.
As part of this spike, we investigated whether it's possible to display a new version on a simple refresh (as opposed to hard refresh). We've created a branch called cache-busting-explorations, in which we use the service worker API to check whether a new version is deployed and activating the new service worker programatically.
It appears that browsers on iOS don't support the onupdatefound event. It also seems that browsers on iOS deviate from the service worker specification as follows: They load the new service worker, activate and immediately switch to the new service worker on page refresh.

OS
Browser
type
Current behavior
Behavior in cache-busting-explorations

Android
Chrome
browser
not investigated
Refresh causes "update available" button to be shown. Clicking the button (or refreshing again) shows new version.

Android
Chrome
PWA
not investigated
Refresh causes "update available" button to be shown. Clicking the button (or refreshing again) shows new version.

Android
Firefox
browser
same as Chrome
same as Chrome , except pull-to-refresh not available

iOS
Safari
browser
not investigated
single refresh is sufficient (but it shows multiple tabs error until refresh)

iOS
Safari
PWA
not investigated
pull-to-refresh isn't available. Same as in browser, except kill app instead of refreshing.

iOS
Chrome
browser
not investigated
same as in Safari

iOS
Chrome
PWA
not available as PWA
not available as PWA

Resources

article that explains our current problem
potential solutions

Other ideas
It may be useful to show an app version somewhere in the app. This might be helpful to debug whether a user is on the newest version.
It may be possible to hand-roll an "update available" event: We could store the newest deployed version number inside a firestore document and auto-increment it on every deploy. The app would then check its own version number and check it against firebase. If it's not equal, then a "reload app" button could be displayed.


Add ability to exclude required extra directories from cruise data transfers

Currently transferring OpenVDM created files (Data Dashboard, Transfer Logs, etc) is an all or nothing option.  These are saving internally as required extra_directory configurations.  It would be easy/cleaner to simply list the required extra directories in with the current list of non-required extra directory configurations.

Released in v2.7.0


Fix: CUS-1729 GitHub actions for publishing

JIRA ticket link/s

CUS-1729

Summary of changes

Introducing an app that has permission to write to this repo and the app have been set to be allowed to push a commit to main since it's changing the version in the package.json.
Trialing the ability to sign the commits in github actions
Always run build and test on every commit

Additional Info
Used the bypass reviewers on this PR as some of these changes can only be tested in the main branch

Used Merge without waiting for requirements to be met (bypass branch protections) option for this PR as the changes can only be tested in the main branch


Can toml-rs support tuple with different type?

#[derive(RustcEncodable, RustcDecodable, Debug)]
struct AA {
	a: Vec<(i32,i32)>,
}

#[derive(RustcEncodable, RustcDecodable, Debug)]
struct BB {
	b: Vec<(String, i32)>,
}

fn main() {
	let a = AA { a: vec![(1, 2)] };
	let a1 = toml::encode_str(&a);
	println!("{:?}", a1);
	let a2 = toml::decode_str::<AA>(&a1);
	println!("{:?}", a2);

	let b = BB { b: vec![("1".to_owned(), 2)] };
	let b1 = toml::encode_str(&b);
	println!("{:?}", b1);
	let b2 = toml::decode_str::<BB>(&b1);
	println!("{:?}", b2);
}

This print:
"a = [[1, 2]]
"
Some(AA { a: [(1, 2)] })
"b = [["1", 2]]
"
None

Unfortunately this just isn't a case where toml can round-trip all types in Rust. Tuples are encoded as arrays and arrays are only allowed to contain a single type according to the specification, so there's not much that can be done here. I'd recommend perhaps using an intermediate struct to introduce some tables though!


Properly handle deferred futures

Currently, most of our facilities (wait_xxx, when_xxx, dataflow) do not properly handle deferred futures.

I think we need to execute the function associated with a deferred future, if the future is about to be destroyed and the value has not been retrieved. If this is not done, the associated function might never be executed.

The current implementation reflects what the standard mandates. This enables lazy evaluation, where unnecessary computations are not evaluated at all. We may want to differ from the standard here (we already do for launch::async, and the rationale for introducing launch::deferred does not apply to us either), but I think that deserves it's own issue.

Yah, by letting a future returned from async block if needed the standard implies this behavior for deferred futures as well. There was simply no need to add a special clause for launch::deferred as the general clause already covers the required behavior for deferred futures.
In our case however, we need special handling for deferred futures as otherwise the function associated with those might never execute.

That's not what I said, the blocking from futures applies to launch::async only. See: http://melpon.org/wandbox/permlink/cpMhTfbFu7lAs3FX

Well, let's close this issue and discuss it separately.


[AIRFLOW-2773] Validate Dataflow Job Name

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2773
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
DataFlowPythonOperator generates a job name that does not get accepted by Dataflow API when task_id contains underscores. This PR replcaes _ (underscore) with a - (dash) and also checks if the name is valid

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Added assertion with regex

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

PTAL @fenglu-g @tswast @Fokko

Codecov Report

Merging #3623 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3623      +/-   ##
==========================================
- Coverage   77.14%   77.13%   -0.01%
==========================================
  Files         204      204
  Lines       15733    15733
==========================================
- Hits        12137    12136       -1
- Misses       3596     3597       +1

Impacted Files
Coverage Δ

airflow/models.py
88.52% <0%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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

@Fokko Agreed. I have added the tests.