What's New: SignalR supports trimming and AOT: Edit and move to include

Contributes to #33031
Do an edit/writing pass on content/code provided by eerhardt here:
https://github.com/dotnet/AspNetCore.Docs/issues/33031#issuecomment-2219013346
..and move to an include in the What's New for .NET 9 Preview 7 for https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/release-notes/aspnetcore-9.0.md

Associated WorkItem - 284277

@wadepickett When using AspNetCore.SignalR.Client to write a client application with AOT (Ahead Of Time) compilation, issues may arise such as not being able to receive messages from the server. As for when AspNetCore.SignalR.Client.Core will support this, it would be best to keep an eye on official announcements from the ASP.NET Core team or check their GitHub repository for any updates on feature support and compatibility. You can also reach out to the community or file an issue on their GitHub page to get more specific information or potential workarounds.
I'm using .NET 9, and the code is as follows:
`  _connection = new HubConnectionBuilder()
.WithUrl("http://localhost:5000/UpgradeHub")
.Build();
        _connection.On<string, string>("11", (user, message) =>
        {
            Console.WriteLine($"{user}: {message}");
        });
        await _connection.StartAsync();`

@wadepickett When using AspNetCore.SignalR.Client to write a client application, I encountered an issue where AOT publishing prevents receiving messages from the server.

I confirm this report as it happened to me. A simple working signalr code stops working when published by AOT flag.

@JusterZhu

When using AspNetCore.SignalR.Client to write a client application, I encountered an issue where AOT publishing prevents receiving messages from the server.

You can solve this with these three steps for classes that are being processed by SignalR (they are either an input type or an output type):

Write a JsonConverter for them:

Assume this class:

[JsonConverter(typeof(PointDataDtoConverter))]
public class PointDataDto
{
    public byte[] PointData { get; set; }
}

Which is used as output type of a signalr server function:
public class ServerHub : Hub
{
    public async Task<PointDataDto> GetPointData() => await PointDataDtoGenerator();
}

Write a JsonConverter for it:

using System;
using System.Text.Json;
using System.Text.Json.Serialization;

public class PointDataDtoConverter : JsonConverter<PointDataDto>
{
    public override PointDataDto Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
    {
        if(reader.TokenType != JsonTokenType.StartObject)
        {
            throw new JsonException();
        }

        var pointDataDto = new PointDataDto();

        while(reader.Read())
        {
            if(reader.TokenType == JsonTokenType.EndObject)
            {
                return pointDataDto;
            }

            if(reader.TokenType == JsonTokenType.PropertyName)
            {
                string propertyName = reader.GetString();
                reader.Read();

                if(propertyName == "PointData")
                {
                    string base64String = reader.GetString();
                    pointDataDto.PointData = Convert.FromBase64String(base64String);
                }
                else
                {
                    throw new JsonException();
                }
            }
        }

        throw new JsonException();
    }

    public override void Write(Utf8JsonWriter writer, PointDataDto value, JsonSerializerOptions options)
    {
        writer.WriteStartObject();

        writer.WritePropertyName("PointData");
        string base64String = Convert.ToBase64String(value.PointData);
        writer.WriteStringValue(base64String);

        writer.WriteEndObject();
    }
}

Write a JsonSerializerContext for them:
It is an empty class based on JsonSerializerContext. It must have JsonSerializable attributes of type of the class itself and its members.

namespace System.Text.Json.Serialization;
[JsonSerializable(typeof(PointDataDto))]
[JsonSerializable(typeof(byte[]))]
public partial class PointDataDtoContext : JsonSerializerContext { }

Register the JsonSerializerContext:

builder.Services.Configure<JsonHubProtocolOptions>(o =>
{
    o.PayloadSerializerOptions.TypeInfoResolverChain.Insert(1, PointDataDtoContext.Default);
});

Works as it should when compiling with aot enabled in dotnet 9.


Cannot login with YubiKey (on Vaultwarden)

I'm trying to use this with vaultwarden, which should be pretty compatible. Every time I try to login though, it asks for my password and then fails with: Login failed: Could not login: could not obtain two-factor auth token: no second factor available and no other error.
When I setup TOTP, everything works.

I also tried to setup the external daemon thing, but it seems to be overwritten by the flatpak I think and I haven't found any option to just completely disable starting the daemon in the sandbox, other than just killing it manually after the start, maybe that should any option and documented on the respective wiki page.

I think external daemon docs should probably be removed at this point as it is untested. The following is also untested, but will probably work: flatpak run --device=all com.quexten.Goldwarden in order to give access to the yubikey / fido2 key.

The trick why I had all the problems was that the settings of flatseal don't work on autostart and you have to edit .config/autostart/com.quexten.Goldwarden.desktop with for the flatpak to work. Maybe that's worth documenting.
When using normal flatpack run even without the parameter it seemed to work using flatseal. Weird that there's this distinction.


Refined class selection UI

Description of new feature
The user needs a fast, easy-to-use UI for specifying the classes in their expression data.
Motivation for new feature
The classes won't be specified in the data the user gets from the facility.

Could we log the typical number of samples in this issue?  Do you remember the typical number, @mikekucera?
If it’s low (e.g. a dozen), then we should just stick with tap for this UI.  Swiping is interesting for a large number of samples, but it’s only really useful for large numbers.

It really depends on the sample data. We should ask users.
I have some old sample expression files that have 130 columns. It seems like the current UI won't scale for that many samples. However, in this particular case the first half of the samples are A and the second half are B, so the way the UI defaults to half A vs half B would actually work well.
Some options...

The "swipe" UI that we discussed.
We could actually support dropping a class file onto the UI. I don't know how common it is for users to have a class file though.
We could somehow save the state of the class selector, and if we detect they are using the same expression file again we could auto-fill to how it was the previous time.

Let's test with what we have now

Let’s close this, as the number of samples is typically low.


Change expression widget's sameForm method to use .tex()

In the Khan Academy web app, we have two exercises that require the answer given by the user to be in a specific form. One of those exercises is working. The other is not.
I'm not totally clear on the specifics of how KaTex works, but for the exercise that is not working, the parsed form looks the same no matter what format the answer is in. That means that no matter what form the answer is in, it's always passing the form comparison. The sameForm method is currently taking the parsed expression and calling .strip(). Changing it to use .tex() makes the form comparison work for both exercises. (If you'd like to hear more about the format each method gives for questions for the two exercises, I'm happy to dump a bunch of that here for your perusal.)
This change should not affect exercises that don't care what form the answer is in. Only these two exercises are known to use that option.
Test plan:
I made this change in the webapp repo in third_party/javascript-khansrc/kas,
navigate to both /math/algebra2/x2ec2f6f830c9fb89:exp/x2ec2f6f830c9fb89:rational-exp/e/understanding-fractional-exponents and /math/algebra2/x2ec2f6f830c9fb89:exp/x2ec2f6f830c9fb89:rational-exp/e/exponents_3,
did a bunch of questions where I first copied the question format then used the format they were looking for,
and verified that the question format was considered incorrect and the expected format was considered correct.

If you're wondering why you're added as a reviewer, I just grabbed a bunch of people who seemed to have worked on this.

I'm going to recommend that @kevinbarabash take a look?  The code looks fine to me but I don't have enough concept to know if this will break anything else.

This looks great, I just did code review with Diedra


New comment.

Dear human,
Here's a new entry for your approval. :tada:
Merge the pull request to accept it, or close it to send it away.
:heart: Your friend Staticman :muscle:

Field
Content

message
It works as a cms, but offers you lots of treating what the c blog

will look like and just how it will function. By recognizing this, you are able to

tweak posts or pages and perhaps give visitors another way

to take. Parents could get the animated or children's

movies at will, as well as the kids could be quiet for 2 hours.

name
Lashawn

email
443d3166392f49e955c53e7b56437e12

url
https://fanzlive.com

hidden

date
2017-09-30T01:03:49.792Z

No idea what you are talking about here.


does not support async methods

Using the airbnb preset with:
import ham from 'ham';

class Foo {
  async blah() {}
}

I get:
Expected token type "Punctuator" but "Identifier" found at test/webdriver.js :
     1 |import ham from 'ham';
--------^
     2 |
     3 |class Foo {

It works fine with:
import ham from 'ham';

async function blah() {};

class Foo {
  blah() { return this::blah(); }
}

Fixed in cst repo

How do I use the cst repo?
On 13 Jun 2016 23:16, "Oleg Gaidarenko"<EMAIL_ADDRESS>wrote:

Fixed in cst repo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jscs-dev/node-jscs/issues/2270#issuecomment-225725274,
or mute the thread
https://github.com/notifications/unsubscribe/AAZQTB2yE_LB63hP-qff-qMgq0nJFNLhks5qLdaqgaJpZM4I0arw
.

Wait until cst is updated in jscs, hence this ticket is still open

I still get this issue. See related issue filed with AtomLinter https://github.com/AtomLinter/linter-jscs/issues/282 for more info.
Sample Code:
import shortid from 'shortid';

const rootResolvers = () => {
  return {
    Query: {
      async test(root, args, context) {
        return test;
      },
    },
  };
};

export default rootResolvers;

JSCS CLI Export:
jscs resolvers.js -c airbnb.json
Expected token type "Punctuator" but "Identifier" found at resolvers.js :
     1 |import shortid from 'shortid';
--------^
     2 |
     3 |const rootResolvers = () => {

1 code style error found.

jscs --version
3.0.7

Those are different issues - you are talking about object method whereas original issue is about class method, created - https://github.com/cst/cst/issues/133


add cn

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Abner Zeng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Feature: Unstoppable Domain integration

cc:
@JohnnyJumper
@merenkoff

I have to handle the translation but in the meantime you can start the review

👀

Thanks @mosadialiou 👀 means km looking

Thanks @mosadialiou 👀 means I'm looking

Yeah sure. Take your time

@kcw-grunt You didn't translate the text "Lookup" on iOS.
What do you suggest for its translation?

@kcw-grunt You didn't translate the text "Lookup" on iOS.
What do you suggest for its translation?

Yeah.. that was an oversight and will push up the fix soon....I'll send you the translations directly

https://github.com/litecoin-foundation/loafwallet-ios/tree/checkout-the-lookup

@kcw-grunt You didn't translate the text "Lookup" on iOS.
What do you suggest for its translation?

Yeah.. that was an oversight and will push up the fix soon...
Here is a branch with all the translations: https://github.com/litecoin-foundation/loafwallet-ios/tree/checkout-the-lookup

Cool! I will pick up just what I need.

Ok thanks. see my Telegram notes regarding the GIF.


Deprecate approxEqual.

Meanwhile all occurences of approxEqual in phobos, druntime and dmd are removed.
In changelog for release 2.091 it has already been announced that approxEqual will eventually be deprecated.

It seems like there are still uses of approxEqual in public documentation tests.

It seems like there are still uses of approxEqual in public documentation tests.

Semicolon after the deprecation attribute

Thanks for detecting this. I should have tested, that the deprecation message really fires. I had to remove all the unittests now too (but I guess this is intended, isn't it), because they produce errors with that message. Or is there a better way?

Semicolon after the deprecation attribute

Thanks for detecting this. I should have tested, that the deprecation message really fires. I had to remove all the unittests now too (but I guess this is intended, isn't it), because they produce errors with that message. Or is there a better way?

cc @atilaneves

cc @atilaneves

This depreciation should be better documented. The behavior difference between isClose and approxEqual is not simply:

better default values and is symmetric in its arguments

as described in the release notes. The change I am noticing is that behavior near zero has changed. By default, 0.0 does not satisfy isClose for any number, where as for approxEqual it did. Whether this is more correct or not is not material. When functions are replaced, users need to know the more important ramifications without having to research it.
Also problematic is that the documentation for approxEqual is no longer available, so it is not easy to compare the behavior of the two functions without looking at the code. For this reason, it would make sense to include a note about migrating from approxEqual to isClose in the isClose documentation. It's not easy to find release notes unless one already knows where to look.

The change I am noticing is that behavior near zero has changed. By default, 0.0 does not satisfy isClose for any number, where as for approxEqual it did.

You are right. The reason for this is, that it strongly depends on the use case, which value is best here, so the users should always specifiy this value, if they want to compare to 0.0. (I took that from pythons implementation of isClose.) I'll file a PR to add this information.

Also problematic is that the documentation for approxEqual is no longer available, so it is not easy to compare the behavior of the two functions without looking at the code.

It is: https://dlang.org/phobos-prerelease/std_math_operations.html#.approxEqual

!!!
As mentioned by @dkorpel this is an extremely commonly used function in existing D programs. A message cautioning people not to use approxEqual in new code is good. Deprecation annotations are good. The Phobos policy that as a deprecated function approxEqual will be outright removed after X months is catastrophic.

The Phobos policy that as a deprecated function approxEqual will be outright removed after X months is catastrophic.

Nice piece to read, indeed. I think, here is not the right place to discuss this - this applies to much more than just approxEqual. And I'm not sure if I'm willing to do this discussion, but I will accept, whatever it will lead to.

!!!
As mentioned by @dkorpel this is an extremely commonly used function in existing D programs. A message cautioning people not to use approxEqual in new code is good. Deprecation annotations are good. The Phobos policy that as a deprecated function approxEqual will be outright removed after X months is catastrophic.

https://github.com/dlang/DIPs/blob/master/DIPs/accepted/DIP1013.md

Another non-backward compatible case: float to double conversion:
File: text.d
import std.stdio;
import std.math: approxEqual, isClose;

void main()
{
    double q = 31.79;
    float rf = 31.79f;
    double r = rf;

    writefln("q: %g; r: %g", q, r);
    writeln("approxEqual(q, r): ", approxEqual(q, r));
    writeln("isClose(q, r)    : ", isClose(q, r));
}

Run:
$ dmd test.d
$ ./test
q: 31.79; r: 31.79
approxEqual(q, r): true
isClose(q, r)    : false

This is obviously the type of case that's a nuisance because it requires code maintainers to debug underlying causes. One could argue that the comments in the release notes describing how to change the default arguments to be closer to approxEqual should suffice. But that leaves existing code authors with quandary. The introduction of isClose is advertised largely as a bug fix for poor behavior in approxEqual. Does making the change suggested in the release notes reintroduce the prior poor behavior? There is not enough info in the release notes, one has to do research to decide what to do.
As an aside, I'm reluctant to be overly critical of work that addresses a long standing issue (Issue 15881 was reported 5 years ago.). And I'll admit that I wouldn't have bothered looking into this if depreciation notices didn't start appearing during compiles.
But still, I had to spend time debugging the above case. And, I still need to research how to approach it. If I'm having to do this, it's likely other developers will need to do this also.
Personally, I think float to double conversion should fall into the default isClose definition of being close. This maybe a case where D and Python should have slightly different behavior. But regardless, this is another case that is useful to include in the isClose documentation.

Another non-backward compatible case: float to double conversion:

There are two things, that catch my eye in your example:

writefln("q: %g; r: %g", q, r); uses default precision of %g which in my opinion is already a bad choice for floats, but even worse for doubles. See #8015 were we discussed this briefly. Replace it with writefln("q: %.16g; r: %.16g", q, r); and you'll get immediately the source of the problem: q: 31.79; r: 31.79000091552734. (If you count them, there are 257_698_037 double values between these two, while when using floats, they are identical.)
As you wrote below: The release note says to get almost identical behavior, you should replace approxEqual(q, r) with isClose(q, r, 1e-2, 1e-2). In that case you'll get true in both cases. isClose with defaults just shows you, that there is something odd that you should investigate.

This is the type of case that's a nuisance because it requires code maintainers to debug underlying causes. One could argue that the comments in the release notes describing how to change the default arguments to be closer to approxEqual should suffice. But that leaves existing code authors with a quandary. The introduction of isClose is advertised largely as a bug fix for poor behavior in approxEqual. Does making the change suggested in the release notes reintroduce the prior poor behavior?

Yes it does; in my opinion, this is obvious. (But it still removes the asymmetry, which is a minor thing.) In this case, I'd say, your fallacy is to think in terms of precision of floats while you use doubles (the same trap, that the %g defaults trapped into).

There is not enough info in the release notes to tell, one has to do research to decide what to do.

What additionally should have been written there? I think, we cannot list any fallacy, people do with floating point numbers. That's beyond such a release note.


How to make the git branch completion work?

I found _git-branch function in the git plugin, but I don't know how to make it work.
I tried to type following in the terminal:
_git-branch
and hit the tab button, but only the files and folders appear, not branches. any idea?

That's not how it's used. The _git_branch function is itself just the completion definition. It defines completions for the git branch subcommand and is sourced automatically. To use those completions, type git branch  in your terminal (not _git-branch) and press tab.

I found where the problem that lies
in my .zshrc, the line:
alias git=hub
which hub suggested
it makes my terminal painfully slow both for ls and 'git status' in my mac(macbook air 2010), and tab completion for branch doesn't work.
delete this line on my .zshrc file make the tab completion for branch works


Add support for CocoaPods test spec

There is a CocoaPods blog post detailing how to do so.

This is stalled while I work on issue CocoaPods/CocoaPods#7040, which will fix the issue occurring now, where no output is logged while tests run during CocoaPods validation, causing Travis builds to timeout.

This has been merged, slated for v1.5.0

It's in the first beta of 1.5.0. Try it out!

Done. Finally!!!!!!


convert NamedTuples to @dataclass

Hi, I'm creating a frontend wrapper for this project and noticed that NamedTuple and @dataclasses are used almost interchangeably in model.py. I was wondering why that's the case and if it would be possible to just change all NamedTuples to @dataclasses. This would make the model more consistent. I tried changing it for a couple classes and didn't notice anything breaking.

If I recall correctly, we favor NamedTuple whenever possible so that the underlying data structure is guaranteed to be immutable by default. We use @dataclasses when mutability is required.
Replacing NamedTuple by @dataclasses may not alter the program behavior though it may prevent the type-checker from detecting potential bugs caused by mutations.
That said if that causes problems for the wrapper you are writing, we could change this convention. I see that @dataclasses were introduced in python 3.7 and that they also support immutability via the frozen flag, so I wonder if it was meant to fully replace NamedTuple though the doc does not clarify: https://docs.python.org/3/library/dataclasses.html

Ahh, that's great reasoning. A big part of the project I'm working on is letting users modify generated models such as adding nodes, updating vulnerability descriptions, etc. This would be easier (albeit riskier) to do with mutable @dataclasses but perhaps frozen @dataclasses would strike a good balance.
Also, @dataclasses have been a lot easier for me to serialize into JSON as opposed to NamedTuples, which did not yield me good results. It might be that they were not designed with that functionality in mind. On my end, I had to change a couple NamedTuples into @dataclasses to be able to send data over to the frontend. (I tried using the YAML serializer but that brought me a few problems, unfortunately)

@jonabox Have you tried the orjson serialization package (https://github.com/ijl/orjson/) mentioned in one of the answers to the stackoverflow article you cited? According to the doc it handles NameTuples correctly.

Not quite, I ended up changing NamedTuples into @dataclasses(), some of them frozen. I then added an encode method to these classes to fine-control the json.dumps() output. For example:
@dataclass(frozen=True)
class CachedCredential():
    """Encodes a machine-port-credential triplet"""
    node: NodeID
    port: PortName
    credential: CredentialID

    def encode(self):
        return dataclasses.asdict(self)

cachedCredential = CachedCredential("client", "TCP",  "TCP-credential" )
json.dumps(cachedCredential, default=lambda x: x.encode())
# returns {"node": "client", "port": "TCP",  "credential": "TCP-credential"}

I might still try orjson, and get back with any findings, but those slight modifications have worked really well for me so far.


Lat/Lon asset/event association

@DanielJMaher this will grab the lat lon from the events when available.

Is this ithe matching Backend PR for the front end that is in?


Add Support for Pods (Marathon 1.4)

Marathon 1.4 added pods. They're special groups of containers that share networking and storage resources on a single Mesos agent.
Pods only support the Mesos containerizer and can be interacted with via the /v2/pods endpoints. From the Marathon documentation, sample POST/GET/DELETE:
$ curl -X POST -H "Content-type: application/json"<EMAIL_ADDRESS>http://<ip>:<port>/v2/pods <<EOF
 {
    "id": "/simplepod",
    "scaling": { "kind": "fixed", "instances": 1 },
    "containers": [
      {
        "name": "sleep1",
        "exec": { "command": { "shell": "sleep 1000" } },
        "resources": { "cpus": 0.1, "mem": 32 }
      }
    ],
    "networks": [ {"mode": "host"} ]
 }
 EOF

 curl GET http://<ip>:<port>/v2/pods/simplepod::status
curl -X DELETE http://<ip>:<port>/v2/pods/simplepod

This has been addressed by now.


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for meuble-salon-canape-moderne-tunisie processing.
🔨 Explore the source changes: 6a8092fc700b5a1ae957230e62f3ad9fd49085c0
🔍 Inspect the deploy log: https://app.netlify.com/sites/meuble-salon-canape-moderne-tunisie/deploys/61ddd15a0147f10008c7aa0b


Shader Insanity with task list

Integrate shader compilatiuon with task list.
Shader declares use of a task list resource.
Task list will inferr that use and add the corresponding task.
No more need to pass task resources around, they simply become a global in the shader after usage declaration.
Syntax proposal:

DAXA_DECLARE_TASK_IMAGE_USE(task_image_debug_name, access, slice);

void main()
{
  daxa_Image2Df32 image = task_image_debug_name;
}

It would be nice to have a way to make the slice run-time configurable.

I'm not convinced I like it yet, and would like to see it implemented first

Merged changes that allow for task use declaration in inl files.


Add Getters and Setters source action failing

command 'nbls.java.generate.getter.setter' not found

Sorry, just noticed that it is a duplicate of #214

Closing this issue since it is a duplicate.
Please check #214 for tracking this issue.


Logout on one device prevents all other devices from getting new jwt

Discussed in https://github.com/supabase/supabase/discussions/3637

Originally posted by Vinzent03 October 24, 2021
I am using the dart SDK, but I think that doesn't matter.
Let's say I'm signed in on device A and B. Now I log out on Device A. This revokes all refresh tokens in auth.refresh_tokens. Now my jwt expires on device B and I want to get a new jwt by calling the token endpoint. This requires a refresh token, this is saves on the client, but because of the log out on device A, it's revoked and doesn't work. Which causes device B to be logged out too.
So my question is, is this intended or is my workflow wrong?

Any news???

Hey @wizzymotion,
Thanks for your patience so far! We've added this into our task board and we are hoping to address this within the month. Let me know if you have any further questions/concerns.
Thanks!

I know this is intended behavior for GoTrue. I know supabase does their own thing with GoTrue, but on the netlify/gotrue repository readme (here), it clearly says

POST /logout
Logout a user (Requires authentication).
This will revoke all refresh tokens for the user. Remember that the JWT tokens
will still be valid for stateless auth until they expire.

I think we should consider the security implications of changing this behavior before we go forward with changing it.
Currently, if a refresh token gets stolen from the user, users can logout and ensure that any stolen tokens are revoked. If this behavior is changed, there needs to be a way to ensure that all refresh tokens are revoked (I am ok if it is different from the current behavior, we just need to ensure that we still provide this functionality).
Additionally, logging out a user currently does not revoke the access token associated with that user (I believe from my reading of the source). This behavior needs to be clarified in the documentation or changed to meet expectations.
I think to meet all needs, it would be appropriate for /logout to revoke the access token used to authorize the logout, but we need seperate endpoints for /revokeRefreshToken and /revokeAllRefreshTokens to be able to ensure users can secure their api's properly.

I edited my comment above for clarity, sorry to confuse the person who already read and thumbs-upped it.

Hey all,
The behaviour described is indeed intended -- logout should revoke all refresh tokens and log out all devices. It seems like the docs have been updated to reflect this as well. We will transitioning moving to use database sessions which might make it easier to allow for management of different sessions on different devices.
Going to close the issue for now but feel free to comment or request to re-open if there are still further issues.


fixes https://bugs.launchpad.net/jio/+bug/1467502

takes care of truncated files on local machine

Please submit this upstream. They're much better equipped to review it.

this code is present on upstream.

On Thursday 09 July 2015 01:17 PM, Puneet Goyal wrote:

this code is present on upstream.
—
Reply to this email directly or view it on GitHub
https://github.com/JioCloud/python-glanceclient/pull/1#issuecomment-119859655.

So, do we want the repository rebased to latest upstream changes, or
just apply this one patch ?
--
Ritesh Raj Sarraf
RESEARCHUT - http://www.researchut.com
"Necessity is the mother of invention."

I recommend rebasing.

That'll also pull in new dependencies. I know Soren was holding it off to
the IceHouse release for a while. Please let's wait for Soren to confirm
first.
On Thu, Jul 9, 2015 at 1:53 PM, Puneet Goyal<EMAIL_ADDRESS>wrote:

I recommend rebasing.
—
Reply to this email directly or view it on GitHub
https://github.com/JioCloud/python-glanceclient/pull/1#issuecomment-119871601
.

--
Ritesh Raj Sarraf
RESEARCHUT - http://www.researchut.com
"Necessity is the mother of invention."

No worries. What happens when we rebase after merging some of the code that is already present on upstream?

Well.... IF you rebase to the latest, you do pull in your fix, but also
pull in all other changes up till that revision.
On Thu, Jul 9, 2015 at 2:03 PM, Puneet Goyal<EMAIL_ADDRESS>wrote:

No worries. What happens when we rebase after merging some of the code
that is already present on upstream?
—
Reply to this email directly or view it on GitHub
https://github.com/JioCloud/python-glanceclient/pull/1#issuecomment-119873960
.

--
Ritesh Raj Sarraf
RESEARCHUT - http://www.researchut.com
"Necessity is the mother of invention."

I know that, what I wanted to ask was, what if we merge this PR and at a later moment of time we decide to rebase...will it merge automatically? or there will be a conflict due to existing code?

Which new dependencies will be introduced by rebasing glanceclient?

If we end up not rebasing and this already exists upstream, please submit the cherry-pick. Don't copy/paste code like this.

Depends. If it is already in the upstream repo and you cherry-pick, it
should merge/rebase fine. Otherwise, you may get conflicts.
On Thu, Jul 9, 2015 at 2:25 PM, Puneet Goyal<EMAIL_ADDRESS>wrote:

I know that, what I wanted to ask was, what if we merge this PR and at a
later moment of time we decide to rebase...will it merge automatically? or
there will be a conflict due to existing code?
—
Reply to this email directly or view it on GitHub
https://github.com/JioCloud/python-glanceclient/pull/1#issuecomment-119879754
.

--
Ritesh Raj Sarraf
RESEARCHUT - http://www.researchut.com
"Necessity is the mother of invention."

I was referring to usual bump in versioned dependencies, in requirements.txt
On Thu, Jul 9, 2015 at 2:27 PM, Soren Hansen<EMAIL_ADDRESS>wrote:

Which new dependencies will be introduced by rebasing glanceclient?
—
Reply to this email directly or view it on GitHub
https://github.com/JioCloud/python-glanceclient/pull/1#issuecomment-119880080
.

--
Ritesh Raj Sarraf
RESEARCHUT - http://www.researchut.com
"Necessity is the mother of invention."

I fixed this first and later realized this exists upstream. How do I get the cherry pick commit where this change was made? does git support reverse code change mapping against the commit id's.
Let me check the dependencies change in requirements.txt.

@rickysarraf Yes, but what are they?

@punituee "git reset --hard HEAD^; git cherry-pick commit_id_of_upstream_commit ; git push -f name_of_remote name_of_branch"
git reset --hard HEAD^    <---- Basically removes the topmost commit (the one you added).

@sorenh For glanceclient, nothing has been bumped. The requirements.txt was last touched 9 months ago.

Great. Just rebase then.

Soren, thanks. I think you misread what I wrote. I really don't have the commit id. Is there a way to figure out in what commit the code change was pushed without going through the history manually?
And yes we should re-base.

@punituee "git blame filename.py" shows the origin of every line in the file.

@sorenh thanks again. It shows the latest commit, which has overwritten the previous one. Latest one can not be cherry picked because it edits the code due to other commits in between.
Oh lets just rebase it. If it breaks we can catch it in the gate and revert back to current version and merge just the fix.
Hope it doesn't come to that.

@punituee The repo is rebased now to the latest, i.e. June 30th, 2015. The fix you've proposed is now already pulled in. Please verify and then you may close this PR.

Yes it is present. Lets test this further in build.

@rickysarraf why would you rebase an entire repo without testing it? It broke the build environment. We need to reset it back to the previous commit.

I had to go through the code manually but I found the commit id for my change upstream
https://github.com/openstack/python-glanceclient/commit/e3600ad7beb7cb4a067038d750e530a64fd1e93a
Since the build environment broke because we rebased, this single commit can be cherry picked.


WIP: Blockfrost ledger client

I'm kinda blocked because I don't understand addresses well enough lol.


Use python3.9 and debian buster

Codecov Report

Merging #309 (abfef74) into master (5fb7f7b) will increase coverage by 0.10%.
The diff coverage is n/a.

:exclamation: Current head abfef74 differs from pull request most recent head 589ed50. Consider uploading reports for the commit 589ed50 to get more accurate results

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
+ Coverage   67.10%   67.20%   +0.10%
==========================================
  Files         120      120
  Lines       10572    10561      -11
==========================================
+ Hits         7094     7098       +4
+ Misses       3478     3463      -15

Impacted Files
Coverage Δ

assemblyline/odm/randomizer.py
94.73% <0.00%> (-1.32%)
:arrow_down:

assemblyline/remote/datatypes/__init__.py
69.49% <0.00%> (-0.51%)
:arrow_down:

assemblyline/remote/datatypes/queues/comms.py
77.77% <0.00%> (-0.49%)
:arrow_down:

assemblyline/remote/datatypes/hash.py
82.52% <0.00%> (-0.17%)
:arrow_down:

assemblyline/datastore/stores/es_store.py
59.73% <0.00%> (-0.14%)
:arrow_down:

assemblyline/run/pubsub_reader.py
0.00% <0.00%> (ø)

assemblyline/datastore/helper.py
32.31% <0.00%> (+0.04%)
:arrow_up:

assemblyline/run/cli.py
60.32% <0.00%> (+0.08%)
:arrow_up:

assemblyline/common/caching.py
31.86% <0.00%> (+0.34%)
:arrow_up:

assemblyline/common/backupmanager.py
75.41% <0.00%> (+5.70%)
: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 5fb7f7b...589ed50. Read the comment docs.


Add new Google analytics code.

Did you test this locally in some way?

Yes I ran the blog on my local machine.
From: Martin Laporte<EMAIL_ADDRESS>Reply-To: "coveo/source.coveo.com"<EMAIL_ADDRESS>Date: Friday, November 17, 2017 at 3:21 PM
To: "coveo/source.coveo.com"<EMAIL_ADDRESS>Cc: Ronald Vincent<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [coveo/source.coveo.com] Add new Google analytics code. (#150)
Did you test this locally in some way?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/coveo/source.coveo.com/pull/150#issuecomment-345355631, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAToE77h6yL5DFzfIw5INEudUc91WNscpks5s3erBgaJpZM4QidHR&data=02|01|rvincent%40coveo.com|dce4c004ed704f4bac9508d52df8c548|f1735c1d9ea143b89aea2e71110c26d5|0|0|636465468847789282&sdata=YcBvOszSMwZKj6TCVRDEQRiK7F7%2BM2QbyzqYCXhVCIk%3D&reserved=0.
Coveo Solutions Inc | 3175 Chemin des Quatre-Bourgeois, Suite 200 Quebec City, QC G1W 2K7 Canada
Coveo Software Corp | 620 Davis St San Francisco, CA 94111 United States
Coveo B.V. | Beech Avenue 54-80 1119 PW Schiphol-Rijk The Netherlands
Unsubscribehttp://pages.coveo.com/unsubscribe-page.html | Privacy http://www.coveo.com/en/privacy | Contact ushttp://www.coveo.com/en/company/contact-us


Note Limit up to<PHONE_NUMBER>

If you make a number larger than<PHONE_NUMBER>, it won't convert backwards.

Thanks for your report!
What language or implementation are you referring to?

In Java, you are obviously limited by what the compiler allows for int, i.e. 2,147,483,647.
In JavaScript, everything should work without any problems up to Number.MAX_SAFE_INTEGER, i.e. 9,007,199,254,740,991.
In PHP, everything should work safely up to PHP_INT_MAX, i.e. 2,147,483,647 or 9,223,372,036,854,775,807, depending on your system.

So there shouldn't be any problems with 1,999,999,999 or 2,000,000,000.
Does this help? Otherwise, can you give a specific example? Thanks!

You were right. Closing this ticket out. I did a variation on your routine but didn't count the length of my string properly and so my BASE constant was off by one, which created the problem.


File ignore list / ignore git folder

Since a relatively frequent use case is probably to have a git repo with dot file templates, the SRC_DIR might also contain eg. a .git directory, which I don't want in my home folder. Therefore, some kind of ignore list would be helpful, eg. with a parameter:
dot-templater --ignore .git .dotconfig ~/dotfiles ~

An easy fix would be to just ignore .git altogether, this would be a breaking change though. Alternatively, a --ignore-git would probably suffice in most cases too.

Fixed with #18


deploy:migrate fails if db/schema.db is missing.

Steps to reproduce
Remove db folder, or put it in .gitignore
then do 'cap production deploy'
Expected behavior
Deployment
Actual behavior
Tasks: TOP => deploy:migrate
(See full trace by running task with --trace)
The deploy has failed with an error: Exception while executing as user@server: Exception while executing as user@server: rake exit status: 1
rake stdout: Nothing written
rake stderr: rake aborted!
Errno::ENOENT: No such file or directory @ rb_sysopen - /var/www/App/releases/20180624112354/db/schema.rb

System configuration
https://gist.github.com/acesuares/a5cb99884c528cee72223a328558579b

This is as expected. If the Capistrano rails migration task is called, and is fails, it is expected that the deployment fails. This isn't Capistrano behavior, this is how the rails migration take task behaves. If you don't want to run migrations, there are instructions for omitting that part of the deployment here: https://github.com/capistrano/rails

Thanks! Didn't know that.


chore(EMI-2025): Add tracking for Limited-Time Offer signals

This PR resolves EMI-2025
Description
This PR adds tracking for Limited-Time Offer signal tracking
⚠ It also updates some of the deprecated tracking functions for artwork grid item
Please let me know if you have any concerns or questions about my code, naming, or ways of adding fields :)

PR Checklist

[ ] I have tested my changes on iOS and Android.
[X] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[X] I have added tests, or my changes don't require any.
[X] I added an app state migration, or my changes do not require one.
[X] I have documented any follow-up work that this PR will require, or it does not require any.
[X] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Update tracking for artwork taps + add tracking for Limited-Time Offer signals - mrsltun

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (Update tracking for artwork taps + add tracking for Limited-Time Offer signals - mrsltun)

  Generated by :no_entry_sign: dangerJS against 95b2f88e3d39aa7ff8a4e1f7b490e4b0453b7a61


Mas i1817 overflowrtq

Expand on use of riak_kv_overflow_queue so that it is used by the riak_kv_replrtq_src, as well as riak_kv_reaper and riak_kv_eraser.
This means that larger queue sizes can be supported for riak_kv_replrtq_src without having to worry about compromising the memory of the node.  This should allow for repl_keys_range AAE folds to generate very large replication sets, without clogging the node worker pool by pausing so that real-time replication can keep up.
the overflow queues are deleted on shutdown (if there is a queue on disk).  The feature is to allow for larger queues without memory exhaustion, persistence is not used to persist queues across restarts.

https://github.com/basho/riak_kv/issues/1817

Overflow Queues extended to include a 'reader' queue which may be sued for read_repairs.  Currently this queue is only used for the repair_keys_range query and the read-repair triggered by key amnesia.


Android Crash when Searching

Bug description
When searching with any tag, the app crashes.
App does not crash when searching without tags.
Steps to reproduce

Open App
Input anything into search tab
Click either the search icon or enter button on keyboard

Context
Unsure of where to find main.log on Android, will provide if I can find it or be told where to look, whichever comes first.
Screenshots
Video attached showcases the error in action.
https://github.com/Bionus/imgbrd-grabber/assets/80283974/2a397585-2593-4d1f-ab05-df1b91e2b2a9
System information

OS: Android 14
Grabber version: 7.12.2

Additional context
Android automatically offered to clear the Cache, which did not fix the issue, clearing app data did not fix the issue either.
Search function worked as of 3 days ago of this report being made.

Weird, it's searching fine on Android + Gelbooru for me. Is that the only source it happens on? 🤔

Search function worked as of 3 days ago of this report being made.

Did you update the app over that period?

Did you update the app over that period?
Not that I can recall, but I think I can close this issue as the problem has seemingly solved itself, without any user input.

It must've just been some System issue that has since self-corrected, thank you for taking the time to look into this though!


Refactor Module to remove Harness Structure - Fixes #169

Pull Request (PR) description
This PR refactors the module structure to remove the need for a test harness.
It also updates the module to submit examples to the PS Gallery.
This Pull Request (PR) fixes the following issues

Fixes #169

Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[x] Resource documentation added/updated in README.md in the resource folder.
[x] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[x] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[x] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[x] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

@johlju - would you mind reviewing at some point? It is a bit of a large one, but is mainly just moving files around.

This change is

Codecov Report

Merging #182 into dev will decrease coverage by <1%.
The diff coverage is n/a.

@@        Coverage Diff         @@
##           dev   #182   +/-   ##
==================================
- Coverage   94%    94%   -1%
==================================
  Files        6      8    +2
  Lines      799    873   +74
==================================
+ Hits       758    821   +63
- Misses      41     52   +11

Thank you so much @johlju!


[WIP] T4LysozymeTestSystem

Initial commit -- right now T4LysozymeTestSystem is MybTestSystem copied and pasted with the name changed so it obviously isn't going to do anything.

Why is this here?
    def __init__(self):
        super(AblImatinibTestSystem, self).__init__()
        solvents = ['vacuum', 'explicit'] # TODO: Add 'implicit' once GBSA parameterization for small molecules is working
        solvents = ['vacuum'] # DEBUG

Specifically, does the problem with allowing explicit solvent actually have to do with explicit solvent itself, or with small molecule parameterization?  gaff.xml does not contain residue information and won't be able to construct templates.

(When I try to run the T4 lysozyme test system it fails when adding solvent, but because of the ligand)

(When I try to run the T4 lysozyme test system it fails when adding solvent, but because of the ligand)

I thought I had added that DEBUG statement just to speed up testing, but something else seems to be going wrong when you remove that.  I'll stop by!

@juliebehr : The problem seems to be that your PDB file (181L.pdb) has a ligand in it that doesn't have any protons.  Remember that ForceField will try to parameterize any molecules without parameters, but these ligands need to have all hydrogens added since it has to infer the bond orders, aromaticity, and charge state from the connectivity in Topology and element names.
You will need to strip out the small molecule, process it with OpenEye to add protons, and add it back.  It may be easier to just create the molecule from IUPAC name with OpenEye tools and do a shape overlay, stripping the original small molecule out.  We should probably automate that process at some point---@andrrizzi may have already done so for YANK.
To make it easier to test/debug, I've added some code that lets you just run this test directly with
python perses/tests/testsystems.py

Note also that we don't have GBSA for small molecules working yet, so I have commented that out.
You probably also want to strip the crystallographic waters out of the PDB file, since when we do get implicit solvent working, those won't be wanted there.

In yank I added back protons through leap, but this will likely change atom types. Also, I have never played with shape overlay in OpenEye yet. Sorry I can't be of much help.

I don't believe leap can add protons to the small molecule, which is what @juliebehr needs to do here.  (pdbfixer already takes care of the protein residues.)

This will run, after taking a very long time, and then I get:
================================================================================
SAMS sampler iteration     0
--------------------------------------------------------------------------------
Expanded Ensemble sampler iteration        0
................................................................................
MCMC sampler iteration 0
Taking 50 steps of Langevin dynamics...
potential  =<PHONE_NUMBER>90017.875 kT
kinetic    =<PHONE_NUMBER>52.777 kT
force norm = 912290.656 kT/A/dof
Final energy is          nan kT
................................................................................

OH

Added minimize step, still problems:
Minimizing 'explicit-complex'...
Initial energy is<PHONE_NUMBER>88949.062 kcal/mol
Final energy is     701937.835 kcal/mol
Minimizing 'explicit-receptor'...
Initial energy is   -41005.549 kcal/mol
Final energy is     -80619.295 kcal/mol
Minimizing 'vacuum-complex'...
Initial energy is<PHONE_NUMBER>48427.000 kcal/mol
Final energy is      -4384.998 kcal/mol
Minimizing 'vacuum-receptor'...
Initial energy is    12345.586 kcal/mol
Final energy is      -4762.848 kcal/mol
================================================================================
SAMS sampler iteration     0
--------------------------------------------------------------------------------
Expanded Ensemble sampler iteration        0
................................................................................
MCMC sampler iteration 0
Taking 50 steps of Langevin dynamics...
potential  = 1177427.981 kT
kinetic    = 112537.615 kT
force norm = 356.626 kT/A/dof
Final energy is          nan kT
................................................................................

I'm guessing the benzene is still messed up, currently investigating

Pretty sure this is just a unit error in assigning coordinates to the benzene OEMol, confirming and fixing if so...

Added this code to write out a PDB file after you add the benzene:
        # DEBUG: Write initial coordinates
        outfile = open('initial.pdb','w')
        PDBFile.writeFile(modeller.getTopology(), modeller.getPositions(), outfile)
        outfile.close()

Looks like there are a few problems:

The benzene is way outside the pocket
It's really tiny. Did you forget to convert from Angstroms to nanometers?

Pretty sure this is just a unit error in assigning coordinates to the benzene OEMol, confirming and fixing if so...

Yes!

Current status:
Minimizing 'explicit-complex'...
Initial energy is   -20739.364 kcal/mol
Final energy is     -20741.473 kcal/mol
Minimizing 'explicit-receptor'...
Initial energy is   -24380.706 kcal/mol
Final energy is     -81211.251 kcal/mol
Minimizing 'vacuum-complex'...
Initial energy is    15589.085 kcal/mol
Final energy is      -4442.742 kcal/mol
Minimizing 'vacuum-receptor'...
Initial energy is    11959.039 kcal/mol
Final energy is      -4767.055 kcal/mol
================================================================================
SAMS sampler iteration     0
--------------------------------------------------------------------------------
Expanded Ensemble sampler iteration        0
................................................................................
MCMC sampler iteration 0
Taking 50 steps of Langevin dynamics...
potential  = -34791.692 kT
kinetic    =<PHONE_NUMBER>434.055 kT
force norm = 1591685.782 kT/A/dof
Final energy is          nan kT
................................................................................

Somehow, your hydrogens ended up directly on top of your carbons:
ATOM   2638  C1      1   1      25.978   5.327   4.779  1.00  0.00           C
...
ATOM   2644  H1      1   1      25.978   5.327   4.779  1.00  0.00           H

Oh, weird! Ok, looking through openeye docs now.

I couldn't figure out what was going on there either.

Any luck?

Not yet

Giving up on openeye and using geometry engine to propose hydrogen locations.

seems good to merge?

It's fine with me!

Merging.


The document  of  "send!"  is  inaccurate

The current the document of send! said: "The method must take &mut self as the receiver."
I tried the method likes this async fn get_count(&self) -> ActorResult<usize>, it pass the compler.
Then I found the following words at https://old.reddit.com/r/rust/comments/hu10wh/actzero_asyncawait_compatible_actor_system/

Actor methods can have &self, &mut self or self: Addr<Local> receiver types, and this controls whether these methods can execute concurrently or not.

How about putting these words into the document of send!?

I did further reading.  The design of act-zero is very beautiful, thank you @Diggsey
Both call! and send! internal will call  send_mut, and always passed the &mut the_actor to MutItem , no matter the actor method is &self or &mut self, like the name imply  "mutex". So the method as "&self" also can pass the compile.
I love the idea of:

Actor methods can have &self, &mut self or self: Addr receiver types, and this controls whether these methods can execute concurrently or not.

However call!  and send! is not mean to it for v0.4.  So how can I call the methods concurrently?

Yeah, at the moment only &mut self is usable with send! and/or call!. To run things concurrently you should use send_fut_with or send_fut.
There's a more complicated example here: https://github.com/Diggsey/raft-zero
Originally the idea was that &self methods could run concurrently with each other, but it complicated the implementation quite a bit, and I found they were not that useful in practice.
The current idea is that you run whatever you like concurrently using send_fut, and then when you are ready to update the actor state, you can call! or send! a method on the actor. send_fut_with is a convenience that makes it easier to pass the actor address into the future.
I'd be happy to accept PRs to fix issues in the documentation :)

I have tried to restructure a project with act-zero.  I found the majority of the function just gets data, more precisely, gets the part of the actor state.  It's expensive to archive it with the exclusive call!.   send_fut_with  can not access the actor state (or the fields of the struct).    call! nested in the  send_fut_with "  is expensive even more.


Pasting larger icon into smaller location does not resize it

Firstly, thanks for the app!
In Apple's Icon Composer I can copy the 512px size icon and paste it into the 256 slot and it will automatically be resized to fit. This is a quick and easy way of creating all size versions of an icon.
IconComposer2x does not do this resizing, it just crops the image.

@gingerbeardman Sorry, currently no time to improve IconComposer2x... I'm open to pull requests though!

Here's part of the work: https://github.com/lemonmojo/IconComposer2x/pull/2
This sets image view scaling to proportionally down, this is a bit of a cheat.
The save code needs to check that the image is of the correct size and if not really resize it as it saves.

This isnt implemented into the lastes version 1.1 right? Seems handy if this is able to downsscale the main image to fill in all the other formats needed

It's not complete, so nothing to merge. Sorry.
I now use a MAS app Icon Set Creator to generate my icns files.

ahh okay thanks for the link.


Can't redeploy to Heroku

So after a lot of work, I finally got this deployed to Heroku.
However, now when I try to re-deploy after making some changes, I get the following error:
± git push production master
Counting objects: 29, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (19/19), done.
Writing objects: 100% (19/19), 1.61 KiB | 0 bytes/s, done.
Total 19 (delta 15), reused 0 (delta 0)
remote: Compressing source files... done.
remote: Building source:
remote:
remote: -----> Fetching set buildpack https://github.com/ddollar/heroku-buildpack-multi.git... done
remote: -----> Multipack app detected
remote: =====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-nodejs.git
remote: =====> Detected Framework: Node.js
remote:
remote: -----> Creating runtime environment
remote:
remote:        NPM_CONFIG_LOGLEVEL=error
remote:        NPM_CONFIG_PRODUCTION=true
remote:        NODE_ENV=production
remote:        NODE_MODULES_CACHE=true
remote:
remote: -----> Installing binaries
remote:        engines.node (package.json):  0.12.7
remote:        engines.npm (package.json):   2.14.4
remote:
remote:        Downloading and installing node 0.12.7...
remote:        Downloading and installing npm 2.14.4 (replacing version 2.11.3)...
remote:
remote: -----> Restoring cache
remote:        Loading 2 from cacheDirectories (default):
remote:        - node_modules
remote:        - bower_components (not cached - skipping)
remote:
remote: -----> Building dependencies
remote:        Pruning any extraneous modules
remote:        Installing node modules (package.json)
remote:
remote:        > gulp-rails@ postinstall /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c
remote:        > gulp build
remote:
remote:        [06:10:31] Using gulpfile /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/gulpfile.js
remote:        [06:10:31] Starting 'build'...
remote:        [06:10:31] Starting 'clean'...
remote:        [06:10:31] Finished 'clean' after 6.91 ms
remote:        [06:10:31] Starting 'iconFont'...
remote:        [06:10:31] Starting 'images'...
remote:        [06:10:31] gulp-svgicons2svgfont: Font created
remote:        [06:10:32] Finished 'iconFont' after 432 ms
remote:        [06:10:32] gulp-imagemin: Minified 9 images (saved 25.43 kB - 8.9%)
remote:        [06:10:32] Finished 'images' after 1.01 s
remote:        [06:10:32] Starting 'sass'...
remote:        [06:10:32] Starting 'browserify'...
remote:        [06:10:32] Bundling global.js...
remote:        [06:10:32] Finished 'sass' after 285 ms
remote:        [06:10:36] Bundled global.js in 3.81 s
remote:        [06:10:36] Finished 'browserify' after 3.83 s
remote:        [06:10:36] Starting 'rev-assets'...
remote:        [06:10:36] Finished 'rev-assets' after 40 ms
remote:        [06:10:36] Starting 'rev-font-workaround'...
remote:        [06:10:36] 'rev-font-workaround' errored after 640 μs
remote:        [06:10:36] TypeError: Cannot read property 'split' of undefined
remote:        at /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/gulp/tasks/rev/rev-font-workaround.js:30:42
remote:        at forOwn (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/lodash/dist/lodash.js:2106:15)
remote:        at Function.forEach (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/lodash/dist/lodash.js:3303:9)
remote:        at Gulp.<anonymous> (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/gulp/tasks/rev/rev-font-workaround.js:25:5)
remote:        at module.exports (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:34:7)
remote:        at Gulp.Orchestrator._runTask (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/index.js:273:3)
remote:        at Gulp.Orchestrator._runStep (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/index.js:214:10)
remote:        at /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/index.js:279:18
remote:        at finish (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:21:8)
remote:        at /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:52:4
remote:        at f (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/node_modules/end-of-stream/node_modules/once/once.js:17:25)
remote:        at DestroyableTransform.onend (/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/orchestrator/node_modules/end-of-stream/index.js:31:18)
remote:        at DestroyableTransform.emit (events.js:129:20)
remote:        at /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2/node_modules/readable-stream/lib/_stream_readable.js:965:16
remote:        at process._tickCallback (node.js:355:11)
remote:
remote:
remote:        npm ERR! Linux 3.13.0-66-generic
remote:        npm ERR! argv "node" "/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/.heroku/node/bin/npm" "install" "--unsafe-perm" "--userconfig" "/tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/.npmrc"
remote:        npm ERR! node v0.12.7
remote:        npm ERR! npm  v2.14.4
remote:        npm ERR! code ELIFECYCLE
remote:        npm ERR! gulp-rails@ postinstall: `gulp build`
remote:        npm ERR! Exit status 1
remote:        npm ERR!
remote:        npm ERR! Failed at the gulp-rails@ postinstall script 'gulp build'.
remote:        npm ERR! This is most likely a problem with the gulp-rails package,
remote:        npm ERR! not with npm itself.
remote:        npm ERR! Tell the author that this fails on your system:
remote:        npm ERR!     gulp build
remote:        npm ERR! You can get their info via:
remote:        npm ERR!     npm owner ls gulp-rails
remote:        npm ERR! There is likely additional logging output above.
remote:
remote:        npm ERR! Please include the following file with any support request:
remote:        npm ERR!     /tmp/build_a04984ffdd9d47cbfa3be2e3dd43541c/npm-debug.log
remote:
remote: -----> Build failed
remote:
remote:        We're sorry this build is failing! You can troubleshoot common issues here:
remote:        https://devcenter.heroku.com/articles/troubleshooting-node-deploys
remote:
remote:        If you're stuck, please submit a ticket so we can help:
remote:        https://help.heroku.com/
remote:
remote:        Love,
remote:        Heroku
remote:
remote:
remote:  !     Push rejected, failed to compile Multipack app
remote:
remote: Verifying deploy...
remote:
remote: !	Push rejected to seedfeed-production.
remote:
To git@heroku.com:seedfeed-production.git
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'git@heroku.com:seedfeed-production.git'

Anyone know what's going on?

Hm... seems like config.iconFont.dest is undefined. If you're not using an iconFont, you can remove 'rev-font-workaround' from the rev tasks in gulp/tasks/rev.
This repo is getting a little behind... if you want to try and integrate the latest version of gulp-starter, which this was based on, I just merged in some updated rails helpers and a README here: https://github.com/vigetlabs/gulp-starter/tree/master/gulpfile.js/extras/rails
It's on my to-do list to get this repo up to date.


PLT-110: Expose V1/V2 cost model param names as fixed enumeration

For each api version, we create an enumeration datatype that attaches a "tag/label" to the otherwise
untyped array of costmodel params coming from the ledger , i.e. [Integer].
Each tag/label is a nullary constructor in the enumeration datatype and the order of its appearance matters.
Also some "dt-generic" code to derive a human-readable/JSON key name for such nullary constructors (tags).
This will hopefully allow us to be more flexible in future versions (V3) , so that we can define our own costmodel param ordering and not rely/assume/expect an alphabetical order of cost model parameter names.
Pre-submit checklist:

Branch

[x] Tests are provided (if possible)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[ ] Formatting, PNG optimization, etc. are updated

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Are you gonna change ApiCommon right after this PR? I wanted to steal mkDynEvaluationContext into plutus-core.

I am not planning to change ApiCommon further, I am only planning to move the module's name as per: https://github.com/input-output-hk/plutus/pull/4530/files#diff-c09f0252ea88dbf83eca3a54a89b9b8cb0647795c8b1de0b8df18068ab62d723

/benchmark plutus-benchmark:validation

^ just out of paranoia


Update canonical urls

These are the updated canonical urls.

Deploy preview for calico ready!
Built with commit c30a1fc0bfd3b77a125675223fbd1f6f8f70c0af
https://deploy-preview-2865--calico.netlify.com

Thanks @rene-dekker !


Implement of Contract model

We can reference the design of https://github.com/ckb-js/kuai/issues/3

https://github.com/ckb-js/kuai/pull/69


Bug: NCICTCAEv5 grading incorrect for "INR increased"

What happened?
Roche user reported that admiral was incorrectly using "x ULN" in criteria. For example grade 1 is:
">1.2 - 1.5; >1 - 1.5 x baseline" however admiral has: ">1.2 - 1.5 xUN; >1 - 1.5 x baseline"
Session Information
No response
Reproducible Example
No response

thanks @millerg23 - @bms63 @manciniedoardo let's try and prioritise this one for the Dec release. i wouldn't say its worth a hotfix release.


Add rspec binstub

Rspec test suite requires being run in context of bundle. This provides the bundler binstub necessary to do so.
Added bundler config because the default binstub location is presently used for bins provided by this gem itself, not for binstubs used by its developers.
Partially addresses #44

Just had a conversation with some other devs and apparently, there's a slightly more conventional way to solve this...
Rather than having brew-gem's bin under bin/, it should be moved to exe/. This leaves bin/ open for bundler binstubs which then eliminates the need for the .bundle/config file being committed.
Thoughts on this?

Updated PR to reflect current convention (exe for bins distributed by the gem, bin for binstubs)

bump


Does this lib support when we have canvas inside div ..

I have canvas diagram on page and some info below it in html elements ... I tried but it show only html no canvas comes ...

Tried data-html2canvas-ignore attribute?
Duplicate from https://github.com/niklasvh/html2canvas/issues/464


Add a simple analogy to git reset

translated and addapted some text from the book progit- section Reset Demystified to help understand the git reset
added some images as well

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Delete prisma-ecosystem in README

deleted 'prisma-ecosystem' which is deleted in docs/

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

Thanks!


is the attached image normal? i'm expecting nothing should be blocking (means at least within 120ms max)

is the below normal? i'm expecting nothing should be blocking (means at least within 120ms max)
acting as a proxy, it waits for the backend to response. is selectgo the total for each block or 1 block that is a few hours?
i'm using the block profiling

This seems to be a problem with the use of pprof

Perhaps if you look at the pprof documentation, it will help you understand more.

@li-jin-gou ok thx.
i gave the wrong pprof information. it's supposed to be another one. i'll check docs first and revert.


Module not found: Error: Can't resolve - when adding components

Version
[ "nuxt": "^2.0.0",](https://github.com/nuxt.js/releases/tag/ "nuxt": "^2.0.0",)
Reproduction link
https://github.com/MasterMindLegion/Nuxt-problem
Steps to reproduce
Greetings, I got a problem when I want to add any component. For an example I have included Test.vue in @components/Test.vue, which is being used in @components/tags/TagsBlock.vue - !!!be aware there is an import with a small letter of  test.vue - but I got nevertheless the same problem with any component. !!!
When I add a component I get always the same type error
./components/tags/TagsBlock.vue?vue&type=script&lang=js& (./node_modules/babel-loader/lib??ref--2-0!./node_modules/vue-loader/lib??vue-loader-options!./components/tags/TagsBlock.vue?vue&type=script&lang=js&)
Module not found: Error: Can't resolve<EMAIL_ADDRESS>in '/Users/martinchalupa/Web/projects/PravoPress/pravoPress/components/tags'

What is expected ?
I expect a component to be added properly.
What is actually happening?
This is always the problem...
failed to compile with 1 errors               friendly-errors 10:56:18

This dependency was not found:                        friendly-errors 10:56:18
                                                      friendly-errors 10:56:18
* @components/Test.vue in ./node_modules/babel-loader/lib??ref--2-0!./node_modules/vue-loader/lib??vue-loader-options!./components/tags/TagsBlock.vue?vue&type=script&lang=js&

This bug report is available on Nuxt community (#c10723)

Guys, for some reason vue-loader has problem parsing string as '', so use " " form importing components. This solution solved components import problem.


Setup basic list and scanner

Setup some basic list and scanning. It retrieves the food information from the open food facts api and adds the name to the list.

No conflicts. Approved PR


Use 7za instead of unzip because of dos codepages

7za does not mangle the filepaths. It is already a somewhat of a dependency anyway.
unzip on my machine: 6.0-21
unzip on lilo: 6.0-20ubuntu1
So my guess is that ubuntu has patched unzip to have this behaviour.
Even with the modified locale unzip still mojibakes

That is unfortunate. Using 7za is fine with me. But perhaps this solution of fiddling with the locales is too clever and we should simply rely on the unmojibake macro again? (removed in 033620d)

Fiddling with locales is okay if we can guarantee that is always that particular codepage. It will probably work until someone with characters in his name from a different encoding will be a student (e.g. japanese, cp866 is made Cyrillic).

Well, it seems as if the character names in the zip are actually in UTF-8. It's just that for some reason unzip thinks (probably because it gets told so by Brightspace) that they are in CP866.


globalSummon and multipleActions should be able to be used at the same time

I'm not sure if this is currently supported so apologies if I filed this incorrectly.
Description of the new feature/enhancement
In settings.json, allow actions in "actions" list to be globalSummon and have multiple actions.

It seems like both "globalSummon" and "multipleActions" go into the "action" field of the "command" object so you can't use both at the same time. But it seems natural to users (also I have a scenario where I would like this to work) that they would want to do globalSummon and then run some actions.
This would allow terminal to take market from autohotkey where users want to register global hotkeys as well but terminal can do something hotkey actions better because terminal can display the output and provide better user interface.
Proposed technical implementation details (optional)
Make globalSummon its own field in "command" that changes how the action keybinding is bound and let multipleActions behave the as it used to.

Hmm, I feel like I've heard this one before...
It's not #10279 or #12135... It isn't in the spec, maybe I'm making this up.
<silly> Maybe we could add a then property to the globalSummon, that you could put an action (including a multipleActions action) into. So you could summon and do some stuff with a single keybinding.
That's not even that silly of an idea 🤔


The views are freezing then arise some error

Arising some error,
onVideoError: NSConcreteNotification 0x16fc1a50 {name = AVCaptureSessionRuntimeErrorNotification; object = <AVCaptureSession: 0x180a35a0 [AVCaptureSessionPresetInputPriority]>
<AVCaptureDeviceInput: 0x16dd1ba0 [Front Camera]> -> <AVCaptureVideoDataOutput: 0x180d8a30>; userInfo = {
AVCaptureSessionErrorKey = "Error Domain=AVFoundationErrorDomain Code=-11800 "The operation could not be completed" UserInfo=0x16fb5230 {NSLocalizedDescription=The operation could not be completed, NSUnderlyingError=0x16d0ae60 "The operation couldn\U2019t be completed. (OSStatus error -12780.)", NSLocalizedFailureReason=An unknown error occurred (-12780)}";
}}
Than: WARNING: Renegotiation needed but unimplemented.
Than the App is freezing
What should I do if delegate calls a peerConnectionOnRenegotiationNeeded method?
I disabled an execution of device rotation.
Any ideas ?
Thanks!

What device model and iOS version are you using? Does this error occur
every time? In what situation do you get the error? Can you list the steps?
What AppRTC Server are you connecting to and is the remote device a web
browser or another iOS device?
--kelly
On Thu, Jul 30, 2015 at 6:46 AM, Kos<EMAIL_ADDRESS>wrote:

Arising some error,
onVideoError: NSConcreteNotification 0x16fc1a50 {name =
AVCaptureSessionRuntimeErrorNotification; object =
-> ; userInfo = {
AVCaptureSessionErrorKey = "Error Domain=AVFoundationErrorDomain
Code=-11800 "The operation could not be completed" UserInfo=0x16fb5230
{NSLocalizedDescription=The operation could not be completed,
NSUnderlyingError=0x16d0ae60 "The operation couldn\U2019t be completed.
(OSStatus error -12780.)", NSLocalizedFailureReason=An unknown error
occurred (-12780)}";
}}
Than: WARNING: Renegotiation needed but unimplemented.
Than the App freezing
What should I do if delegate calls a peerConnectionOnRenegotiationNeeded
method?
I disabled an execution of device rotation.
Any ideas ?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/ISBX/apprtc-ios/issues/5.

So, I have a iPhone5, ios 8.4
Steps: I set connection up, similar to example, then a connection is established, everything's fine, View and audio exchanging is running, then rotate the device at 180grad ( tentatively ) then local video view is freezing and in this moment  the error arises. The freezing is going until I rotate the device again by the same manner  and the freezing is gone, a behavior of running is very weird,  following log:
onVideoError: NSConcreteNotification 0x16fc1a50 {name = AVCaptureSessionRuntimeErrorNotification; object = <AVCaptureSession: 0x180a35a0 [AVCaptureSessionPresetInputPriority]>
<AVCaptureDeviceInput: 0x16dd1ba0 [Front Camera]> -> <AVCaptureVideoDataOutput: 0x180d8a30>; userInfo = {
AVCaptureSessionErrorKey = "Error Domain=AVFoundationErrorDomain Code=-11800 "The operation could not be completed" UserInfo=0x16fb5230 {NSLocalizedDescription=The operation could not be completed, NSUnderlyingError=0x16d0ae60 "The operation couldn\U2019t be completed. (OSStatus error -12780.)", NSLocalizedFailureReason=An unknown error occurred (-12780)}";
}}
WARNING: Renegotiation needed but unimplemented.
WARNING: Renegotiation needed but unimplemented.
Exchange happens between browser and device.
I use default google stun servers.
I think, a server side is not guilty,  it depends on some video capture system. Presumably, the webrtc lib consists of a lot of wrappers of native classes ( ObjC ) and probably inside of some class call a rotation method of device.
What should i do when delegate call a peerConnectionOnRenegotiationNeeded method?
What are cases a Renegotiation occurs ??
Because the "onVideoError" error occurs in the same time with  the Renegotiation WARNING.
My take on it, this 2 events are connected with each other.
Thanks

Yes, rotation seems to cause a problem and lead to crashing. What needs to
happen is before rotation the app will need to destroy the views and
recreate them after rotation in the correct orientation.
On Thu, Jul 30, 2015 at 7:59 AM, Kos<EMAIL_ADDRESS>wrote:

So, I have a iPhone5, ios 8.4
Steps: I set connection up, similar to example, then a connection is
established, everything's fine, View and audio exchanging is running, then
rotate the device at 180grad ( tentatively ) then local video view is
freezing and in this moment the error arises. The freezing is going until I
rotate the device again by the same manner and the freezing is gone, a
behavior of running is very weird, following log:
onVideoError: NSConcreteNotification 0x16fc1a50 {name =
AVCaptureSessionRuntimeErrorNotification; object =
-> ; userInfo = {
AVCaptureSessionErrorKey = "Error Domain=AVFoundationErrorDomain
Code=-11800 "The operation could not be completed" UserInfo=0x16fb5230
{NSLocalizedDescription=The operation could not be completed,
NSUnderlyingError=0x16d0ae60 "The operation couldn\U2019t be completed.
(OSStatus error -12780.)", NSLocalizedFailureReason=An unknown error
occurred (-12780)}";
}}
WARNING: Renegotiation needed but unimplemented.
WARNING: Renegotiation needed but unimplemented.
Exchange happens between browser and device.
I use default google stun servers.
I think, a server side is not guilty, it depends on some video capture
system. Presumably, the webrtc lib consists of a lot of wrappers of native
classes ( ObjC ) and probably inside of some class call a rotation method
of device.
What should i do when delegate call a peerConnectionOnRenegotiationNeeded
method?
What are cases a Renegotiation occurs ??
Because the "onVideoError" error occurs in the same time with the
Renegotiation WARNING.
My take on it, this 2 events are connected with each other.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/ISBX/apprtc-ios/issues/5#issuecomment-126359777.

Must I do it within peerConnectionOnRenegotiationNeeded method?

Last time i looked into the issue it was related to
ARTCVideoChatViewController's videoView:didChangeVideoSize: method that is
called when orientation change. Looks like changing the frame size for
the RTCEAGLVideoView for the local and remote views is what leads to the
error.
-k
On Fri, Jul 31, 2015 at 10:50 AM, Kos<EMAIL_ADDRESS>wrote:

Must I do it within peerConnectionOnRenegotiationNeeded method?
—
Reply to this email directly or view it on GitHub
https://github.com/ISBX/apprtc-ios/issues/5#issuecomment-126770035.

Hello,
Look, Presumably I found a point that causes a issue.
Acoording to ticket: https://code.google.com/p/webrtc/issues/detail?id=3162&can=1&q=onVideoError&colspec=ID Pri Mstone ReleaseBlock Area Status Owner Summary
There are discussing about stop video session on a background thread, so it's fine, next a following code of ARDAppClient:
- (void)orientationChanged:(NSNotification *)notification {
    UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
    if (UIDeviceOrientationIsLandscape(orientation) || UIDeviceOrientationIsPortrait(orientation)) {
        //Remove current video track
        RTCMediaStream *localStream = _peerConnection.localStreams[0];
        [localStream removeVideoTrack:localStream.videoTracks[0]];

        RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack];
        if (localVideoTrack) {
            [localStream addVideoTrack:localVideoTrack];
            [_delegate appClient:self didReceiveLocalVideoTrack:localVideoTrack];
        }
        [_peerConnection removeStream:localStream];
        [_peerConnection addStream:localStream];
    }
}

A [localStream removeVideoTrack:localStream.videoTracks[0]] and RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack] are called from different  threads and probably the system tries to call removeVideoTrack and createLocalVideoTrack simultaneously. I made a test, I inserted a pause (sth like sleep() ) between removeVideoTrack and createLocalVideoTrack, for convincing that  a removeVideoTrack was executed and createLocalVideoTrack will be executed after, the problem is gone.
Look at my stack while my test was running:

The didChangeVideoSizes is selector that handles an error onVideoError ( when I talked about in the beginning).
What do you think about it? Would you agree that handling of two methods of video Capture is not good simultaneously?

Thats great to hear you solved the problem! Do you want to issue a pull
request and I can merge it into the code repository?
thanks!
On Thu, Aug 6, 2015 at 7:18 AM, Kos<EMAIL_ADDRESS>wrote:

Hello,
Look, Presumably I found a point that causes a issue.
Acoording to ticket:
https://code.google.com/p/webrtc/issues/detail?id=3162&can=1&q=onVideoError&colspec=ID Pri Mstone ReleaseBlock Area Status Owner Summary
There are discussing about stop video session on a background thread, so
it's fine, next a following code of ARDAppClient:

(void)orientationChanged:(NSNotification *)notification {
UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
if (UIDeviceOrientationIsLandscape(orientation) || UIDeviceOrientationIsPortrait(orientation)) {
//Remove current video track
RTCMediaStream *localStream = _peerConnection.localStreams[0];
[localStream removeVideoTrack:localStream.videoTracks[0]];
  RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack];
  if (localVideoTrack) {
      [localStream addVideoTrack:localVideoTrack];
      [_delegate appClient:self didReceiveLocalVideoTrack:localVideoTrack];
  }
  [_peerConnection removeStream:localStream];
  [_peerConnection addStream:localStream];

}
}

A [localStream removeVideoTrack:localStream.videoTracks[0]] and
RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack] are called
from different threads and probably the system tries to call
removeVideoTrack and createLocalVideoTrack simultaneously. I made a test, I
inserted a pause (sth like sleep() ) between removeVideoTrack and
createLocalVideoTrack, for convincing that a removeVideoTrack was executed
and createLocalVideoTrack will be executed after, the problem is gone.
Look at my stack while my test was running:
[image: screen shot 2015-08-06 at 16 48 51]
https://cloud.githubusercontent.com/assets/7373566/9113418/27ef23f4-3c5d-11e5-81f8-f5f57d73a7ea.png
The didChangeVideoSizes is selector that handles an error onVideoError (
when I talked about in the beginning).
What do you think about it? Would you agree that handling of two methods
of video Capture is not good simultaneously?
—
Reply to this email directly or view it on GitHub
https://github.com/ISBX/apprtc-ios/issues/5#issuecomment-128385684.

Hi, I'm having the exact problem. For me, this error is occurring only when I add mandatoryConstraints for maxWidth and maxHeight. Without them, everything seems fine. By trial and error method I managed to find out that inserting 1 second pause anywhere between [localStream removeVideoTrack:localStream.videoTracks[0]]; and [_delegate appClient:self didReceiveLocalVideoTrack:localVideoTrack]; fixes the problem. Anything below 1 second doesn't work for me. Since pausing for 1 second is creating a significant delay I was wondering if there is a more elegant solution for this problem?

Guys look at my pull request. There are improvements regarding the issue

Yep, works like a charm now. Thanks!
One more thing, when I used the default way to detect orientation changes using UIDeviceOrientation, it would register FaceUp and FaceDown orientation change and cause local view to be reloaded unnecessarily so I switched to UIInterfaceOrientation and that seems to work better for me.

I'm using @kos9kus 's fix, and generally it works - rotates fine, with one exception - front camera, when orientation is set to 4 (UIDeviceOrientationLandscapeRight).  Any ideas?

Hello guys! I Maybe fix it. This is my solution :
In ARDAppClient class

(void)orientationChanged:(NSNotification *)notification {
UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
if (UIDeviceOrientationIsLandscape(orientation) || UIDeviceOrientationIsPortrait(orientation)) {
//Remove current video track
RTCMediaStream *localStream = _peerConnection.localStreams[0];
RTCVideoTrack *localVideoTrack = localStream.videoTracks[0];
[localStream removeVideoTrack:localStream.videoTracks[0]];
  //RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack];
  if (localVideoTrack) {
      [localStream addVideoTrack:localVideoTrack];
      [_delegate appClient:self didReceiveLocalVideoTrack:localVideoTrack];
  }
  [_peerConnection removeStream:localStream];
  [_peerConnection addStream:localStream];

}
}
Don't recreate RTCVideoTrack when orientation changed

Hello guys! I Maybe fix it. This is my solution :
In ARDAppClient class

(void)orientationChanged:(NSNotification *)notification {
UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
if (UIDeviceOrientationIsLandscape(orientation) || UIDeviceOrientationIsPortrait(orientation)) {
//Remove current video track
RTCMediaStream *localStream = _peerConnection.localStreams[0];
RTCVideoTrack *localVideoTrack = localStream.videoTracks[0];
[localStream removeVideoTrack:localVideoTrack];
  //RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack];
  if (localVideoTrack) {
      [localStream addVideoTrack:localVideoTrack];
      [_delegate appClient:self didReceiveLocalVideoTrack:localVideoTrack];
  }
  [_peerConnection removeStream:localStream];
  [_peerConnection addStream:localStream];

}
}
Don't recreate RTCVideoTrack when orientation changed

Hello guys! I Maybe fix it. This is my solution :
In ARDAppClient class

(void)orientationChanged:(NSNotification *)notification
{
UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
if (UIDeviceOrientationIsLandscape(orientation) || UIDeviceOrientationIsPortrait(orientation))
{
//Remove current video track
RTCMediaStream *localStream = _peerConnection.localStreams[0];
RTCVideoTrack *localVideoTrack = localStream.videoTracks[0];
[localStream removeVideoTrack:localVideoTrack];
  //RTCVideoTrack *localVideoTrack = [self createLocalVideoTrack];
  if (localVideoTrack)
  {
      [localStream addVideoTrack:localVideoTrack];
      [_delegate appClient:self didReceiveLocalVideoTrack:localVideoTrack];
  }
  [_peerConnection removeStream:localStream];
  [_peerConnection addStream:localStream];

}
}
Don't recreate RTCVideoTrack when orientation changed


Dark Mode with Switcher

Requesting for Dark Mode with Switcher.

Hey 👋 We ask that you please do not use GitHub for support or feature requests 😄. We use GitHub solely for bug-tracking and on-going feature development.
Many questions can be answered by reviewing our documentation. If you can't find an answer then our forum is a great place to get community support, plus it helps create a central location for searching problems/solutions.
Thanks!


Export/backup station list

Please add the option to export (and import) the station lists.
In a sense the option is already there, if the end user knows where to look (~/Library/Application Support/com.github.SokoloffA.Radiola/bookmarks.opml), but it'd be nice to have a menu option for this process too.

Yeah, I'll do it. No problem with exporting. But for import, I need to think about how to merge existing and new stations.

I added this to the beta version of Radiola 6.99.1.
If you are interested, you can test it by downloading Radiola-6.99.1.dmg

I have released the second beta, if you are interested you can download it at the link.

In release 7.0.0


Winding direction is wrong

glTF 2.0 spec defines the front face using a CCW winding direction.
The exporter fails to export correctly a simple cube. Normals computed using the CCW winding direction point towards the inside of the cube.
As by default materials are marked as doubleSided, this normally doesn't matter as both faces are going to be rendered, but in some cases it can give weird results. For example, exporting the default scene enabling the lights will show artifacts and a very different lighting setup in babylon.js

I attach a scene I have been playing with to understand this issue
Scene.tar.gz

@julienduroure @emackey

Can you include a .blend file for this?
My first suspicion is that Blender's normals are flipped in your blend file.  Blender 2.80 doesn't visually show this as well as 2.79 used to.  But you can enable the normal vector display at the bottom of the "viewport overlays" pulldown when in Edit Mode.

@emackey

Start with an empty file.
Add a cube.
Export as .glb with default settings.
Import that .glb back in.
Compare normals.

You can tell they're backwards from the shading, but I also cut away some tris so you can see the normals.

@scurest I'm shocked.  Also I can't reproduce that on my current setup.  What OS, Blender version, addon version/branch are you using to reproduce?

Blender: (2, 80, 75)
Add-on: 1f35003 (current master)
More tests show this doesn't happen if you triangulate the mesh first. Therefore possibly this issue? We use tessellate_polygon to triangulate n-gons and my Blender is from August, so it wouldn't have that fix.

I am using Blender 2.81 alpha (2019-10-06 14:05) and for the exporter current master. As scurest mentioned, it doesn't happen if I triangulate the mesh first.
Normals looks okay. No need to provide a .blend file, just export the default scene (cube, light and camera)

This fixes it
diff --git a/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py b/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py
index 08270b3..a31616d 100644
--- a/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py
+++ b/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py
@@ -617,6 +617,11 @@ def extract_primitives(glTF, blender_mesh, blender_object, blender_vertex_groups
             triangles = tessellate_polygon((polyline,))

             for triangle in triangles:
+                # Triangle winding-order is reversed in old versions of Blender
+                # See https://developer.blender.org/T70594
+                if bpy.app.version < ?????:
+                    triangle = (triangle[1], triangle[0], triangle[2])
+
                 for triangle_index in triangle:
                     loop_index_list.append(blender_polygon.loop_indices[triangle_index])
         else:

I just don't know how to find out what the first Blender version the fix went into is.

Latest blender 2.81 (2019-10-16 16.56) works well.

Yeah, I was finally able to reproduce with an older Blender build from Oct 3rd.
I'm not sure we need a code change here yet though.  Still investigating.

This :
https://developer.blender.org/rBA64d638b2adda6cc5581913ad7af3627764ca57db
You can't use a current master addon with old blender version, as order changed

Yes, but this affects Blender 2.79.  So I think we do need a version-check block around that code change from 2fb64552718.

Poking around on a Github mirror of the Blender source code...

https://github.com/sobotka/blender/commit/0905af68ecd446210b5938a1374245fe3555aecd (13 days ago) Blender version bump (281, 13) -> (281, 14).
DOES NOT have the fix for tessellate_polygon: https://github.com/sobotka/blender/blob/0905af68ecd446210b5938a1374245fe3555aecd/source/blender/blenlib/intern/scanfill.c#L910

https://github.com/sobotka/blender/commit/4b570f5b575e5a004326b8a6aa724bb0d26906a3 (7 days ago) Blender version bump (281, 14) -> (281, 15).
DOES have the fix for tessellate_polygon: https://github.com/sobotka/blender/blob/4b570f5b575e5a004326b8a6aa724bb0d26906a3/source/blender/blenlib/intern/scanfill.c#L910

Conclusion: Triangles need to be reversed for versions < (2, 81, 15).


Change default Lyrics fields

Currently it is -
For Unsynced - UNSYNCEDLYRICS
For Synced - LYRICS
Change it to -
For Unsynced - LYRICS
For Synced -  SYNCEDLYRICS
https://wiki.hydrogenaud.io/index.php?title=Tag_Mapping#:~:text=Script-,Synced Lyrics,LYRICS,-Additional Information

Has this changed recently? See #84 for a discussion where I specifically changed everything to use "UNSYNCED LYRICS" by default because that's what fb2k would internally translate to store in the USLT ID3 frame.
Can you confirm that fb2k stores "LYRICS" tags in USLT but not "UNSYNCED LYRICS" on the latest version?

for Vorbis as I have set
it stores in SYNCEDLYRICS for Synced Lyrics
it stores in LYRICS for UnSynced Lyrics

I checked. fb2k does indeed (in v1.6.7) store the "UNSYNCED LYRICS" tag in the USLT frame for mp3 files. In formats such as ogg and flac which use Vorbis comments there is no special handling or structure for tags, they're just key-value pairs of utf8 data (https://www.xiph.org/vorbis/doc/v-comment.html) so no particular values are better or worse than any others there.
In short: there is good reason to keep the current defaults (as openlyrics's settings UI actually explains) and there is no reason to change them for Vorbis files.


smtp/mime: update test to match new behavior

Ticket
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/3487
The suricata PR modifies the behavior, so both suricata and SV PR should be merged at the same time...

Continued in https://github.com/OISF/suricata-verify/pull/1851


Include link to Help Thirsty Koalas Fund

Include link to Help Thirsty Koalas Fund

Codecov Report

Merging #1082 into master will decrease coverage by 2.65%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1082      +/-   ##
==========================================
- Coverage    95.2%   92.54%   -2.66%
==========================================
  Files          34       34
  Lines        6883     6883
==========================================
- Hits         6553     6370     -183
- Misses        330      513     +183

Impacted Files
Coverage Δ

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.72% <0%> (-8.52%)
:arrow_down:

databricks/conftest.py
92% <0%> (-6%)
:arrow_down:

databricks/koalas/frame.py
93.43% <0%> (-3.31%)
:arrow_down:

databricks/koalas/utils.py
95.12% <0%> (-3.05%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

databricks/koalas/namespace.py
85.8% <0%> (-1.55%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

databricks/koalas/testing/utils.py
78.51% <0%> (-0.75%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

Thanks! merging.

I do love this commit.


Loading an expired federated user's token from cache leave the authenticator stuck in an invalid state

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Tested with google:  If the cache provide the user with an expired token he will be stuck in an invalid state preventing the signout method from working.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have dependencies other than AWS Amplify.
Log in with google, wait for the token to expire, reload the page. You get an expired token which won't refresh or go away without clearing your browser cache.
What is the expected behavior?
The cache should not return an expired token if it has no means to refresh it.
Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
0.2.12. I don't think it worked in previous versions.
You can turn on the debug mode to provide more info for us by setting window.LOG_LEVEL = 'DEBUG'; in your app.

The token refresh fail because googleAuth.currentUser.get() return null while trying to refresh the token.
I assume it is impossible to refresh an expired token when loading back the app without prompting the user or a refresh token.  The currentuser is always loaded from cache, even while the app is running, which makes it a bit harder to distinguish if the refresh was caused by the app reloading.
I think simply clearing the cache when token refresh fail should be enough to fix this.

@Brulogaz Good catch! We didn't catch this error in the signOut method. I have created a pr to fix this issue. As you suggested, the idea is to clear out the cache so that users will be logged out on reloading the page once they cannot get the federation credentials from cognito. You can also catch this case when you call Auths.currentCredentials() and handle this case on your need.

Closing the issue as it should have been fixed in the latest version.


Typing Alt+F4 raises two AppWindow.Closing events

Describe the bug
That's assuming you cancel the first event so that you can prompt the user to save the window contents via a content dialog. My code assumed that a second close event came for the user clicking the window close button i.e. "just close the window, I really don't want to save". But of course the second close event after typing Alt+F4 scuppered that scheme.
In the winow's WindowSubclass you can see two WM_SYSCOMMAND messages with wParam = SC_CLOSE are generated in responce to Alt+F4. This isn't reproducable in a WPF app, so I assume that it's a WinAppSdk problem.
Steps to reproduce the bug
Add the following code to a window class, then type Alt+F4
    AppWindow.Closing += (s, args) =>
    {
        args.Cancel = true;
        Debug.WriteLine("received AppWindow.Closing event");
    };

Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.4.1: 1.4.230913002
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

Woops, seems to be a side effect of some other code I'd written. Not a bug at all...


[Button] onTab state

In order to improve accessibility, we use the active "onTab" state style for buttons. It means when you discover page elements with the "Tab" button each interactable component should be "highlighted".
We have ready designs in Figma and need to update the current styles we have in the web3uikit.
Please find 'onTab' in Figma:

https://www.figma.com/file/lENHKVkCjdfZXfFLv5T2Yn/Moralis-UI-Kit?node-id=18%3A15330
https://www.figma.com/file/lENHKVkCjdfZXfFLv5T2Yn/Moralis-UI-Kit?node-id=206%3A9391

Try to add them to all themes, not only to those mentioned above
If there is no onTab style, use 2px solid pale blue border
As we use border already, let's use the box-shadow trick @BillyG83 used

nice job making this issue @Y0moo I will dev this ASAP for you

@BillyG83  are we going with box-shadow or border?

@akinolu52 because button is already using border, its best to use box-shadow, this is already present, you can see it on every button
:focus {
        box-shadow: 0px 0px 0px 2px ${color.redDark};
    }

this task is to align with the design files as when i did this i picked whatever color i thought was best, design did not agree =]

hey @akinolu52 i hope you dont mind but i will reassign this to @Darrellbor as you are busy on Admin and he needs a task to start tomorrow. TY dude