TCK PublisherVerification does not support publishers of empty streams

This code in the PublisherVerification class:
  @Override @Test
  public void required_validate_maxElementsFromPublisher() throws Exception {
    assertTrue(maxElementsFromPublisher() > 0, "maxElementsFromPublisher MUST return a number > 0");
  }

Causes an EmptyPublisher class representing an empty stream to fail conformance testing.
An empty stream may be an edge case but it is a legitimate stream, and a Reactive Streams implementation should be able to have such a Publisher. One use case would be testing that Subscribers operate correctly when subscribed to empty streams.
Therefore, TCK should allow a subclass of PublisherVerification to return zero from maxElemensFromPublisher(), and if it does TCK should test that request()-ing any elements at all from the Publisher results in only a call to the Subscriber's onComplete().

@gnp Thanks for opening this Issue, we should definitely fix this.

It is important to note that it needs to send both onSubscribe and onComplete.

Good catch @gnp, thanks for noticing this! We definitely want to support all kinds of valid publishers (recently made it possible for never-onComplete-calling publishers to be tested as well etc).
I'll draft a PR to address this soon :)

While updating the TCK to support this (that's the easy part) I found that the spec is not precise about if calling onComplete "eagerly".
Imagine this Publisher:
// EagerEmptyStream
    return new Publisher<Integer>() {
      @Override
      public void subscribe(Subscriber<? super Integer> s) {
        s.onComplete();
      }
    };

The only rules which talk more or less around this area (yet do not prohibit it) are:

1.9 "*Invoking Publisher.subscribe MUST return normally. The only legal way to signal failure (or reject a Subscriber) is via the onError method *" - it is not mentioning anything about "completing ASAP" as legal way of "rejecting". It's not really a "failure", because there may be an use case like this: "Publisher: well, I can only produce exactly-one element exactly-once, and then I act as if I'm 'empty', if the second subscriber comes in I tell it I'm completed".
rule 1.12 in fact re-states the additional sentence mentioned in 1.9: "A Publisher MAY reject calls to its subscribe method if it is unable or unwilling to serve them [1]. If rejecting it MUST do this by calling onError on the Subscriber passed to Publisher.subscribe instead of calling onSubscribe", also no rules about completing here.
2.9 "A Subscriber MUST be prepared to receive an onComplete signal with or without a preceding Subscription.request(long n) call"

So the wording of 1.9 more or less seems to imply that we can only "early onError", and not "early onComplete", although it is not stated explicitly, and since completion is not really "failing" I'd say that the onComplete being eager is not specified - and I'm not sure according to the spec if the above Publisher is legal.
Currently the TCK assumed that an onComplete coming in before an onSubscribe is NOT legal, thus I bumped into this case. I can update the TCK to be OK with this in the "empty stream" case of course, but it should be backed by some spec rule I think.
Of course a fully legal and "complete" way to signal this completion is via "complete only once demand is signalled":
    return new Publisher<Integer>() {
      @Override
      public void subscribe(final Subscriber<? super Integer> s) {
        s.onSubscribe(new Subscription() {
          @Override
          public void request(long n) {
            s.onComplete();
          }

          @Override
          public void cancel() {
            // noop
          }
        });
      }
    };

Which does not have any ambiguity problems with the spec AFAICS.
Summing up, I think we should include some wording in the spec which explicitly states if an onComplete as response to a subscription is legal or not.
// cc @viktorklang

I find that it is best illustrated in the readme just before the spec:
In response to a call to Publisher.subscribe(Subscriber) the possible invocation sequences for methods on the Subscriber are given by the following protocol:

`onError | (onSubscribe onNext* (onError | onComplete)?)`

I'll answer more in detail later :)

Yeah, I did notice that sequence-of-calls diagram, but it's not really reinforced by the spec rules - thought that it would be perhaps safer to extend wording somewhere (1.9?) to do so as well

PR submited https://github.com/reactive-streams/reactive-streams/pull/199

tracking the onComplete issue in #202


Labels on multiple source files

miette supports adding multiple labels to an error, but there seems to be a baked-in assumption that all labels refer to the same source file.
This is proving to be a limitation for a usecase of mine: highlight in the error both the usage location (i.e. where a function is called) and the definition location.
To make a practical example: report in an error a function parameter as being of the wrong type (by labelling the value passed in as parameter at the usage location) while reporting, alongside it, the expected parameter type in the definition location (once again using a label).
Is this too niche or would there be an interest in supporting it?
It would probably require a fair amount of breaking changes (e.g. Diagnostic::source_code could have to return an optional iterator, labels will have to store the associated source code, etc.).
You can see a potential implementation approach in ariadne (their multifile code example). They render them sequentially:

I tried to work around the issue by using related errors. It could work, but some adjustments would be required to the way related errors are displayed.
Using an example from my project:
Error:
  × I do not know how to handle the type (`Generic("T")`) of one of the
  │ parameters for `app::stream_file::<std::path::PathBuf>`.
   ╭─[src/lib.rs:7:1]
 7 │ pub fn blueprint() -> AppBlueprint {
 8 │     AppBlueprint::new().route(callable!(crate::stream_file::<std::path::PathBuf>), "/home")
   ·                               ─────────────────────────┬─────────────────────────
   ·                                                        ╰── The callable was registered here
 9 │ }
   ╰────

Error:
  × The definition
   ╭─[src/lib.rs:2:1]
 2 │
 3 │ ╭─▶ pub fn stream_file<T>(_inner: T) -> http::Response<hyper::body::Body> {
 4 │ │       todo!()
 5 │ ├─▶ }
   · ╰──── The callable was defined here
 6 │
   ╰────

It is unclear, from the graphical output, that the latter error is actually a sub-error of the first one. They have the same heading (Error:) and they appear at the same level. Indentation might help, perhaps?

This seems related to https://github.com/zkat/miette/pull/171! Maybe some input there would be helpful?

#171 is indeed related to the possible workaround - I left some comments, but it looks quite promising already!
What about the more architecturally-impactful approach (first-class support for label labels spanning multiple source files within the same error)?

I think labels themselves spanning multiple files is a tough sell. Part of the point of miette is also to allow all this fancy error reporting with a simple data model, and attaching that to a nice proc macro. Miette used to be a bit more flexible about this stuff, but it was way more complex, and I'm more happy where it sits now.

Rephrasing, just to make sure I understand the concern: you are worried that supporting labels across multiple files (a nicher-r usecase) will impact the ergonomics of the most common usecase (single file), therefore making this a net-negative tradeoff?

correct. Older versions of miette supported this, and it was more complicated than it was worth. I think exploring things like #171 and such is a better alternative here.

You can actually have labels in multiple files with current miette.
To make it work you need to create a SourceCode implementation for one big "virtual" file that maps different ranges to the actual files. Then the SourceSpan in your Diagnostic should be offset + len in this "virtual" file and it will just work.
For example implementations you can look at nushell or my own compiler here.
When  #324 is merged it will even show the correct file header above each context.


Android: VectorDrawable quick actions icons not rendering when installing over Play Store

Steps to Reproduce

Clone this project which uses quick_actions package with VectorDrawable (SVG-based) icons for Android's quick actions placed in android/app/src/main/res/mipmap-anydpi-v26.
Run or build locally in Android emulator or local device.

Local results
Quick actions icons get rendered normally. ✅

Results when deployed over Play Store
Quick actions icons do not get rendered, showing instead kind of a 'no icon' placeholder. ❌
The example app in the repo mentioned above has been published to Play Store so that following results can be observed.

Built using
flutter build appbundle --no-shrink

App bundle seems to be all fine, and all xml files have been copied over, no files seem to be missing (checked by decompressing generated app bundle).
Additional info
Flutter doctor (although it happens same thing regardless of platform, version, etc):
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.10.5, on macOS 12.3.1 21E258 darwin-arm, locale es-ES)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.66.2)
[✓] Connected device (1 available)
[✓] HTTP Host Availability

• No issues found!

@darshankawar I explicitly mention here that icons are missing despite setting --no-shrink, so it is not the issue of resources getting lost during shrinking; plus I provide a full reproduction of the issue.
I don't think this should be closed.

@darshankawar is there a way to create quick actions in Flutter that does not use quick_actions package?


Add alert-dialog docs

Pull Request Template
Description:
Adds alert-dialog component page and primitive page to the docs.
Fixes #57
Tested Platforms:

[x] Web
[ ] iOS
[ ] Android

Affected Apps/Packages:

[Docs]

Screenshots:

Notes:
Please go over this specific PR with a fine-toothed comb before I start contributing a bunch of these. I want to be sure I'm adding these in a way you're happy with.
I added a Button component link that goes to a 404 right now, but I will add that page sooner or later.

I'm also not sure what I should be adding to each set of "props" (sub-components?) specifically. That is... there are some other sub-components in those compound components that aren't reeeeeally used, but they definitely exist and are exposed by the framework, like Overlay. Should I include everything possible?
I have kind of the same question for what props to document. Like would I document onPress? It's exposed but if someone tries to use it, it will break the functionality of the component.

For:

Should I include everything possible?

About the sub-components of the primitives, yes they should be included. Even though components like the Overlay "aren't reeeeeally used", they can be used, especially by users that only want the primitives. Some users might not want to use the /ui/* components.
For:

Like would I document onPress?

No, you shouldn't since it is included in the "Extends Pressable props".
For:

if someone tries to use it, it will break the functionality of the component

It should break the functionality. Did you have this issue? When a users passes an onPress to the trigger, it should maintain the original behavior in addition to the onPress prop.
 function onPress(ev: GestureResponderEvent) {
      onOpenChange(!value);
      onPressProp?.(ev);
    }

As discussed on Discord, I will add the changes and merge it when I am done.
Thank you!


[ScanDependency] Handle -Xcc options that affects module generation

Make sure the -Xcc options to the scanner are correctly considered when creating ClangImporterCC1 arguments for constructed swift interface compilation job. Under directcc1 mode, -Xcc options should be used to constructed sub-invocation but should not be added to GenericArgs for constructing interface compilation jobs.
rdar://128873665

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Remove deprecated Overlay options

Is this how it is supposed to work now? The autoPanOptions was already entirely ignored.
Or should autoPan be a boolean option and autoPanOptions be kept.

There is no autoPanOptions configuration option. It is autoPan. The true value of autoPan is deprecated. Expected values for autoPan are either false (the default), or an object consisting of animation (from the deprecated autoPanAnimation and margin (from the deprecated autoPanMargin).

I updated it according to your comment.
Though I do wonder if the default should just be undefined without allowing false.

I agree, it makes no sense to set this to explicitly. When no autoPan is configured, then no autoPan will happen.


Vertical bar chart - Prevent two bars merging as one bar when dates are the same and y values are different

Expected Behavior
2 bars not merge into 1 bar
Current Behavior
2 bars merging into 1 bar
Possible Solution

Steps to Reproduce
I have an issue where two bars merge as one bar when the dates are the same. I don't mind the x tick label merging as one date, but the actual bars I would like to keep separate.
My chart configuration:
var myChart = new Chart(ctx, {
    type: 'bar',
    data: {
      datasets: chartDatasets
    },
    options: {
      responsive: true,
      maintainAspectRatio: false,
      scales: {
        x: {
          stacked: true,
          offset: true,
          type: 'timeseries',
          time: {
            parser: 'YYYY-MM-DD',
            tooltipFormat: 'll',
            unit: 'day',
            displayFormats: {
              day: 'MMM D'
            },
          },
          ticks: {
            source: 'data',
            align: 'center',
          },
          grid: {
            drawOnChartArea: false
          },
        },
        y: {
          stacked: true,
          type: 'linear',
          ticks: {
            beginAtZero: true
          },
          title: {
            display: true,
            text: 'kg Dry Matter / ha',
            font: {
              weight: "bold"
            }
          }
        }
      }
    }
  });

I have created a JSFiddle to duplicate the problem.
jsfiddle
In the first dataset provided to the chart, there are 2 "May 18" records with different y values. Why have they merged into one bar? I am unsure if this is a bug, or if there is a config setting I need to set.
Context
2 datasets that form a stacked bar chart, x axis is timeseries and y axis is linear
Environment
v3.2.1 & 3.2.0
moment.js 2.29.1
chartjs-adapter-moment 1.0
Chrome v90.0.4430.212 (Official Build) (64-bit)

So in your use case, those 2 bars would be side by side. should the bars also be half width compared to others, since they both take a day, but cant be fitted as there is only 1 day and 2 bars?
question about the data, are those bars different things or is your date resolution just not accurate? shouldn't they be added together?

So in your use case, those 2 bars would be side by side. should the bars also be half width compared to others, since they both take a day, but cant be fitted as there is only 1 day and 2 bars?
question about the data, are those bars different things or is your date resolution just not accurate? shouldn't they be added together?

They are different data and should not be added together. It is possible it could be (n) number of bars per date so it would be good to auto calculate the width based on how many bars per date. It used to work in version 2 with the date label repeating. In v3 the date (and data merges).

The simplest solution here is to put one dataset into a different stack. by default, when stack is not specified on a dataset all of the datasets are assumed to be in the same stack. In this example, I added stack: "2" to the 2nd dataset https://jsfiddle.net/m4q9f6e1/

Another option here is to set stacked: false on both axes

The simplest solution here is to put one dataset into a different stack. by default, when stack is not specified on a dataset all of the datasets are assumed to be in the same stack. In this example, I added stack: "2" to the 2nd dataset https://jsfiddle.net/m4q9f6e1/

This is not solving my issue at all. I want the light green and dark green stacked. But there should be multiple dark green bars per day. I don't know why you closed my issue.

Another option here is to set stacked: false on both axes

This is not solving my issue at all. I want the light green and dark green stacked. But there should be multiple dark green bars per day. I don't know why you closed my issue!

I think this would be an enhancement, allowing multiple values on the same dataset for the same index value and reducing the bar width accordingly for that index value. I have no idea how much would need to be changed to make this possible.


Use unminified css and remove minified

Fixes #60

Almost! This looks good in desktop mode but I do notice that in mobile mode the responsiveness is broken. Let's fix that before merging this

(note on how I tested it: the CI step for deploy/netlify links to a preview environment where this PR gets deployed to)

I don't see the mobile issue anymore, nor the different font size I noticed previously 🤔
I've checked in the local version and the Netlify preview in Firefox and Chrome.
If someone else can verify, I think this is ready to merge.

@NickSto I still see the logo being broken in the netlify preview deployment. I did merge your other PR though

Got it. Do we need to migrate your fix from the logo-fixing PR into this PR before merging then, to make sure we don't break the logo again?


aioredis.pubsub.Receiver has no iter() method

The example code (below) references an iter() method on Receiver (here is the implementation).
There is no iter() method on Receiver. What is the correct implementation of listening on all Receiver's subscribed channels?
code:
>>> from aioredis.pubsub import Receiver
>>> from aioredis.abc import AbcChannel
>>> mpsc = Receiver(loop=loop)
>>> async def reader(mpsc):
...     async for channel, msg in mpsc.iter():
...         assert isinstance(channel, AbcChannel)
...         print("Got {!r} in channel {!r}".format(msg, channel))
>>> asyncio.ensure_future(reader(mpsc))
>>> await redis.subscribe(mpsc.channel('channel:1'),
...                       mpsc.channel('channel:3'))
...                       mpsc.channel('channel:5'))
>>> await redis.psubscribe(mpsc.pattern('hello'))
>>> # publishing 'Hello world' into 'hello-channel'
>>> # will print this message:
Got b'Hello world' in channel b'hello-channel'
>>> # when all is done:
>>> await redis.unsubscribe('channel:1', 'channel:3', 'channel:5')
>>> await redis.punsubscribe('hello')
>>> mpsc.stop()
>>> # any message received after stop() will be ignored.

This implementation does work:
class IterableReceiver(Receiver):
    def __aiter__(self):
        return self

    async def __anext__(self):
        await self.wait_message()
        return await self.get()

mpsc = IterableReceiver(loop=loop)

async def reader(mpsc):
    async for channel, msg in mpsc:
        assert isinstance(channel, AbcChannel)
        print("Got {!r} in channel {!r}".format(msg, channel))
asyncio.ensure_future(reader(mpsc))

Thanks for report.
Fixed in v0.3 branch, will make a bug-fix release soon.

Out of curiosity, when do you intend to publish an update that addresses this? It looks like the last update was in Jan. https://pypi.org/project/aioredis/

OMG, I thought it was like a month ago...
I think I can do a release in a day or two.


Training numbers

This is probably outside the scope of the "issues" but figure I'd ask.
I notice it doesn't take numbers.  Is there away to add numbers to the xml data sets so it can also do numbers?

You should be able to generate numbers like:
python generate.py --text="1 2 3 4 5 " --noinfo --bias=4.

although the quality will probably be quite bad (too little examples in dataset).
You can add your own examples in .xml format but you will have to match them to those already in dataset (content should contain tags like: <Transcription>, <Text> and <StrokeSet>, structured like in dataset).
Alternatively if you have data with consecutive points representing how to draw numbers (with labels) you could create your own dataset.
So depending on format of your dataset it might be easier or harder. :)

I'm really new to this so I'm not sure how to go about creating a dataset.  Do you have any articles or direction you can point me to?

Sorry for the delay. I get the feeling you have no data, which is problematic. Could you please elaborate a little bit more on what you are trying to achieve? :)

It's no problem, thank you for taking the time to even discuss this with
me.
I found a dataset which of numerically written numbers however it isn't
setup as the current dataset used by IAM in xml files.  What I'm trying to
accomplish is to use the handwriting but it also has to include numbers and
currently the numbers do not come out good.
On Fri, Oct 20, 2017 at 6:06 AM, Grzegorz Opoka<EMAIL_ADDRESS>wrote:

Sorry for the delay. I get the feeling you have no data, which is
problematic. Could you please elaborate a little bit more on what you are
trying to achieve? :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Grzego/handwriting-generation/issues/2#issuecomment-338200995,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQOknAGNyvv2VlG7lkOJuE9BNydaJKOks5suJrygaJpZM4P-NV6
.

Ok, is this dataset publicly available? I can look into it to see if there is a way to make it compatible with my code. :)

Awesome!  Here goes:
http://yann.lecun.com/exdb/mnist/
http://archive.ics.uci.edu/ml/machine-learning-databases/semeion/
I found these two
Sent from my iPhone

On Oct 21, 2017, at 3:05 AM, Grzegorz Opoka<EMAIL_ADDRESS>wrote:
Ok, is this dataset publicly available? I can look into it to see if there is a way to make it compatible with my code. :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Unfortunatelly, those datasets represent numbers as images. For handwriting generation you would need to have list of consecutive points showing how a digit is written. So those datasets cannot be used here.

Would this one work? This has the stroke data:
https://github.com/edwin-de-jong/mnist-digits-stroke-sequence-data/wiki/MNIST-digits-stroke-sequence-data
On Mon, Oct 23, 2017 at 2:36 PM, Grzegorz Opoka<EMAIL_ADDRESS>wrote:

Unfortunatelly, those datasets represent numbers as images. For
handwriting generation you would need to have list of consecutive points
showing how a digit is written. So those datasets cannot be used here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Grzego/handwriting-generation/issues/2#issuecomment-338804235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQOkpsMBSx4SjLVJftQ-gStOB7Yv2ZYks5svQb3gaJpZM4P-NV6
.

This one might work. :) Can you give some examples of sequences you want to generate? I just want to figure out what kind of augmentation to dataset might be needed.

about 5 digit random sequences.  In example
11445
8013
1507 etc..
On Mon, Oct 23, 2017 at 4:30 PM, Grzegorz Opoka<EMAIL_ADDRESS>wrote:

This one might work. :) Can you give some examples of sequences you want
to generate? I just want to figure out what kind of augmentation to dataset
might be needed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Grzego/handwriting-generation/issues/2#issuecomment-338826058,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQOkiB0tXseZLgH7Nry79NSXJcXQchlks5svSGRgaJpZM4P-NV6
.

Sorry for very late response. I tried this dataset and unfortunately it doesn't work well :/ The results are even worse than with original IAM dataset. If by any chance I find better dataset for this task I will post it here.

THANK YOU!!!!
On Wed, Nov 8, 2017 at 12:50 PM, Grzegorz Opoka<EMAIL_ADDRESS>wrote:

Sorry for very late response. I tried this dataset and unfortunately it
doesn't work well :/ The results are even worse than with original IAM
dataset. If by any chance I find better dataset for this task I will post
it here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Grzego/handwriting-generation/issues/2#issuecomment-342955118,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQOkiSt828fSdSpFVqBdRCh93u3PkbCks5s0hQkgaJpZM4P-NV6
.

Well it's been a while, but I was kind of interested in this problem and created MNIST handwriting dataset. If you still need to generate numbers you may find it useful. One simple solution is to just pick needed digits from this dataset and concatenate them together. :)

@Grzego THANK YOU!


Segmentation fault in scipy.sparse.linalg.svds()

Hello everyone,
I am experiencing a "Segmentation fault" error when running the following snippet:
import numpy as np

import scipy.sparse as sparse
import scipy.sparse.linalg as linalg

import concurrent.futures

def go():
    nb_rows, nb_cols, k = 100, 100, 10
    S = sparse.lil_matrix((nb_rows, nb_cols))

    rows, cols = np.random.randint(2, size=(nb_rows, nb_cols)).nonzero()
    S[rows, cols] = 1

    U, S, Vt = linalg.svds(S.tocsr(), k=k)

with concurrent.futures.ThreadPoolExecutor(max_workers=10) as executor:
    for i in range(1024):
        executor.submit(go)

Here's the output:
$ python3.5 threads.py
Segmentation fault

I don't think arpack is thread safe.  It uses a number of fortran save statements to preserve variables across calls.

Thank you @ewmoore for your answer. I've "temporarily" fixed the problem by using a ProcessPoolExecutor in place of a ThreadPoolExecutor, but I'm still deeply interested in making the code above thread-safe.

@pminervini: the short answer is that the ARPACK Fortran code cannot be run in multiple threads at the same time. It can be made thread safe with locks, but the only one thread will run at a time.

Locks added in gh-5846

Isn't it possible to ensure only one thread has "access" to the fortran interface without using locks? Lets say one wants to run other simple tasks while diagonalizing a huge matrix - this is not possible any more by means of #5848.


AlamofireObjectMapper Skips Nested Dictionaries

I have an issue with the latest release of AlamofireObjectMapper. I have an API that returns an array of JSON data and each item has 3 nested dictionaries.
I am using the ObjectMapper+Realm extension as well to map the JSON to be stored inside a realm database.
The JSON is correctly parsed, however the nested dictionaries are completely skipped despite being returned from the API correctly.
What might I be doing wrong?
This is what my Alamofire request look like:
_ = Alamofire.request(request).responseArray { (response: DataResponse<[Template]>) in { }

My JSON is structured like this:
[
    {
        "TemplateName": "Test Template",
        "TestMode": false,
        "StartDate": "2017-07-01T00:00:00+00:00",
        "EndDate": "2050-07-01T00:00:00+00:00",
        "LastModified": "2017-07-12T17:28:20.369+00:00",
        "Category": "Marketing",
        "IsActive": true,
        "CountryList": [
            "US",
            "CA"
        ],
        "TemplateImage": {
            "Width": 1024,
            "Height": 1024,
            "Url": "#",
            "ThumbnailUrl": "#"
        },
        "ExtraData": {},
        "AutoDate": {}
    }
]

My Model looks like this:
import Foundation
import ObjectMapper
import RealmSwift
import ObjectMapper_Realm

final class Template: Object, Mappable {

    @objc dynamic var templateName: String = ""
    @objc dynamic var testMode: Bool = false
    @objc dynamic var startDate: String = ""
    @objc dynamic var endDate: String = ""
    @objc dynamic var lastModified: String = ""
    @objc dynamic var isActive: Bool = false
    var templateImage: List<TemplateImage>?
    var extraData: List<ExtraData>?
    var autoDate: List<AutoDate>?

    override static func primaryKey() -> String {
        return "templateName"
    }

    required convenience init?(map: Map) {
        self.init()
    }

    func mapping(map: Map) {
        templateName <- map["TemplateName"]
        testMode <- map["TestMode"]
        startDate <- map["StartDate"]
        endDate <- map["EndDate"]
        lastModified <- map["LastModified"]
        isActive <- map["IsActive"]
        templateImage <- (map["TemplateImage"], ListTransform<TemplateImage>())
        extraData <- (map["ExtraData"], ListTransform<ExtraData>())
        autoDate <- (map["AutoDate"], ListTransform<AutoDate>())
    }

I have even tried adding the <- map["TemplateImage", nested: true] to the model mapping to no avail.

@brianramirez  Have you tried @objc dynamic var templateImage: TemplateImage? as it doesn't look to be a collection in the JSON payload.
I may be having a similar issue but need to diagnose further as the Realm model has been created but doesn't include any of the headers when looking via realm studio. Which is a bit strange.. so if you have found anything else about this it will be helpful to know.

Just started noticing this as well. Mine has nested nested Mappable Objects however.

seeing this for attributes that are Protocols instead of explicit class / struct types.

Did anyone find a solution for this?


Not sure how to get it going,

I feel ridiculous asking this but, I couldn't find an answer anywhere. I have installed the plugin on my jenkins server. I can't find any way to configure it. When I try to use it as a build step, nothing happens. No boxes open or errors. Am I misunderstanding how this works? Do I only need a jenkinsfile in my repo?
Jenkins version 2.138.2
Plugin version  0.52

I’m happy you asked! The plugin is meant to be used from a Jenkinsfile, I typically use the blue ocean plugin as well as it makes it super easy to setup pipelines!
Have a watch of the video on this post, might help you out!
https://blog.chef.io/2018/07/11/guest-post-announcing-new-jenkins-plugin-to-better-manage-habitat-packages/
https://youtu.be/dXrlqYzbUxE

This video cleared up my confusion completely. I guess, seeing it as a selectable build step with no other way to manipulate the function threw me off.

Awesome! Glad it helped, we're really trying to keep it opinionated. Otherwise we could quite quickly get lost in configuration hell. Which is a scary place!


Added some small grammar corrections.

Hi Andrea,
I just have accepted another PR for the read me so now, your have some conflicts. :S Could you please resolve them? Sorry for the inconvenience.

Done! :)

Thank you! Now you should resolve the conflicts because I can't merge your changes automatically.

Probably because I changed some of the conflicted words as well.


revert: perf: use workspace root for fs cache (#15712)

Description
Alternative to #16473
fixes https://github.com/vitejs/vite/issues/16399
refs https://github.com/vitejs/vite/pull/15712

/ecosystem-ci run

This will mean a 10% perf hit in certain projects, but I agree that the best here is to revert this until we can figure out how to properly watch the fs root.
I think there may be other perf hits for projects like vitest that init one fs cache per package, and before were all sharing the same cache. We didn't measure this at the end thouhg.
If ecosystem CI is green, I think we should merge and release a patch 👍🏼


Workspace-level notifications

Fixes #1444
@bradtaylor I reworded the tooltip, thoughts?
with an auth domain:

without an auth domain:

@zarsky-broad Do I need to add a call to POST /api/workspaces/{workspaceNamespace}/{workspaceName}/sendChangeNotification as a part of this?

This is now on hold pending further investigation.
Currently FireCloud UI has a way to trigger notifications on and off on a workspace level, for all users. However, only Data Delivery customers will get notifications because dsp-pipelines is the only codebase that calls sendChangeNotification, which sends the notification out.  This is not the desired user experience. So we are putting this on hold until we:

discover the flow of data between systems,
figure out if we can limit showing this information to only the applicable users,
go from there.

Closing the PR but I'm leaving my branch open.


MIC-338: Fixed proactive error messages appearing on reactive involvement screen

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.79%. Comparing base (88fc9b8) to head (164cb76).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
+ Coverage   80.77%   80.79%   +0.01%
==========================================
  Files         109      109
  Lines        1644     1645       +1
  Branches      274      277       +3
==========================================
+ Hits         1328     1329       +1
  Misses        304      304
  Partials       12       12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


LCD dosn't work. Black screen with white lines

Hi guys, I'm trying to install and run my own raspiblitz but I've an issue with my lcd screen.
When the raspberry starts, the LCD starts, but the screen doesn't turn blue. It remains stuck on a black screen with white lines (similar to the terminal (see screenshot below )). How can I solve?
I tryed to reboot but nothing is change. Can enyone help me?

@Lit3block your LCD seems to work well, but it has stuck with the setup process.  Did you have it set up fully previously? I would suggest restarting with a freshly burnt SDcard.

HI @openoms; First of all. thanks for your reply.

Did you have it set up fully previously?

yes I have already set it previously and it worked well.

I would suggest restarting with a freshly burnt SDcard.

Ok... so, should I start all over again (blockchain ftp transfer included?) ?
I can do it, but if there are other possibilities I would prefer.

can you access the RaspiBlitz through the terminal with ssh?
Changing only the SDcard would not require to download the blockchain again. It is the same as the update process: https://github.com/rootzoll/raspiblitz/blob/master/README.md#updating-to-new-version

Upstate: I've finished to copy and sync the blockchain. i've reboot my rasp and the screen issue is disappear. Now, everything work well! Thanks @openoms.

No problem, glad I could help, but it was you who have sorted it 👍

I think this can be closed @rootzoll


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sorry, for asking my question here!

I want to use singular table name where can I customize this parameter?
I know that Sequelize has a parameter tableName but I don't know how to do.
Do you have forum or something for asking technical question?
Sorry and thanks.

You might need https://github.com/holidayextras/jsonapi-store-relationaldb/pull/55 with that, singular names should work.

#55 has been merged and this issue is fixed in the latest 2.0.0 release.


Use Task factory methods instead of going async

The async keyword makes the compiler add a bunch of code to create the underlying state machine that drives asynchronous methods in C#.
You can see here what the compiler generated code is.
For a longer explanation of the difference, see e.g https://stackoverflow.com/questions/52146203/async-method-with-no-await-vs-task-fromresult

Released.
https://www.nuget.org/packages/FluentFTP/37.0.1

Sorry about the follow-up commit you had to made.
I cannot compile the net20, net35 and net45 locally, so I disabled those while making the changes.


Ransomeware error

arguments did not match any overloaded call:
move(self, a0: QPoint): argument 1 has unexpected type 'float'
move(self, ax: int, ay: int): argument 1 has unexpected type 'float'
Also does not grab anything

yuh i finished my own ransomware yesterday, another old dev did shit, i will try mine when im home and when it works fine i implement it in rose
and ur sure that u enabled stuff on the functions tab? i need more information on what features you enabled (if u did so), it could also be a vm, or the victim js doesn't have any saved data?

@tflsaga

The things enabled were uac, startup, screenshot, btc miner (does not work) and ransomware. I inputted all required info for ransomware. If you're updating to a new one tmrw then it's fine, thank you.
@gumbobr0t

Also the victim was on his main pc. Rose hasn't been able to grab anything either which is weird (probably a me issue I just get errors) but I'm just trying to get the ransomware to work.

buddy maybe because you did not enable to steal tokens, browser, ip, wifi, device & more, there is also in the functions tab a row to do that

only things that do not work are obfuscation, miner, ransomware & spread malware
ransomware will be fixed in the next days, same for spread malware,
dont forget to enable stealer options 😭

Ok thank you, didn't know I needed stealer for other features to work. I was just trying to reduce file size.

yuh at the top in the builder you need to go to the functions tab & enable the features you want, ofc put your webhook & a build name at the home menu in tho


[MB-9038] UpdateMTOShipment can review billable weight

Description
Update UpdateMTOShipment support editing billable_weight_cap and billable_weight_justification
Setup
Postman:

Use updateMTOShipment endpoint
Provide billable_weight_cap and billable_weight_justification
Send request
Payload should include billable_weight_cap and billable_weight_justification

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

@donaldthai Added a check for TIOs attempting to update a shipment that is not approved

Messages

:book:
:link: MB-9038

  Generated by :no_entry_sign: dangerJS against 9d4c23c30a84082bc7b24e267da99e5c5907b842


Disable/Enable clustering programmatically?

Is there a way to programmatically enable/disable clustering?

Remove the MCG layer and replace it with a FeatureGroup with the same markers.

how??


has no member named ‘eeForce’

error: ‘unitree_legged_msgs::LowState {aka struct unitree_legged_msgs::LowState_<std::allocator >}’ has no member named ‘eeForce’; did you mean ‘footForce’?
lowState.eeForce[0].x = msg.wrench.force.x;
How to solve this problem?

Hi!
I think this video might help :)

Hello,this is caused by the wrong version of unitree_legged_msgs.
There are two solutions for this.
Method 1: You can choose to put the latest unitree_legged_msgs from the unitree_ros_to_real (v3.2.1） package under ~/catkin_ws/src and recompile it.
https://github.com/unitreerobotics/unitree_ros_to_real/tree/v3.2.1
Method 2: Add the following statements to the unitree_legged_msgs/msg/LowState.msg file and recompile.
Cartesian[4] eeForceRaw
Cartesian[4] eeForce          #it's a 1-DOF force infact, but we use 3-DOF here just for visualization


Not compatible with shared_preferences: ^2.0.11

Since shared_preferences is a really critical package for many apps, I'd argue that this needs to be compatible with it.

Sure thing! A fix is long due 👍
On Thu, 30 Dec 2021 at 23:59 Philipp Bauer @.***> wrote:

Since shared_preferences is a really critical package for many apps, I'd
argue that this needs to be compatible with it.
—
Reply to this email directly, view it on GitHub
https://github.com/oakromulo/flutter_persistent_queue/issues/13, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AETPTDCQAZ4QSI2PPO6H7GTUTTI5HANCNFSM5LAL2PZQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


vendor: update ciao and virtcontainers

update ciao and virtcontainers
fixes #704
virtcontainer short log:
c286dbb device: Call addDevice after dereferencing pointer receiver.
caefa12 devices: Refactor device handling
95f3ae9 filesystem: Split function to reduce cyclomatic complexity
4640741 filesystem: Custom marshal and unmarshal device interface objects
e0fef66 qemu: add default machine accelerators
cdc77cc qemu: remove newer machine accelerators
c1291fb qemu: support for custom machine accelerators
93014e4 qemu: support custom bios
e1fa974 vendor: update ciao
ciao short log:
96bd26b openstack/block: Remove unused API for getting limits
fb02316 singlevm: Fix storage tests
dd17576 openstack/compute: Remove flavor API
5f764ed ciao-cli: Use workload API for listing workloads
fd08780 ciao-controller: Add API entrypoints for listing workloads
1a26ce2 openstack/compute: Make workload filtering use "workload"
d5cfa99 openstack/compute: put workload ID in details struct
979ab37 bat: Change references to FlavorID to WorkloadID
e89a6f7 openstack/compute: Rename Flavor to WorkloadID in create request
44223dc openstack/compute, bat: standardise on NodeID
c2ab924 openstack/compute: rename OsExtendedVolumesVolumesAttached
3d165fb openstack/compute: Simplify the way we store use IP/MAC addresses
4a30edc openstack/compute: Remove unused fields from ServerDetails struct
ad579e7 openstack/compute: Remove mostly unused Image field
c5eea77 opentstack/compute: Remove unused fields from Addresses struct
c73deac bios: add support for custom bios
19f008d ciao-deploy: Keep going if any unjoin step fails
9998583 configuration: Remove "compute_fqdn" from configuration
0b88648 ciao-controller: Use certificate common name as canonical name
b760589 storage_bat: Test that detaching from active instance fails
a524e08 singlevm: Do not access $GOPATH directly
1c4a091 singlevm: Move singlevm to use ciao-deploy setup
3d858df ciao-deploy: When an SSH command fails report stdout/stderr
9716e2d testutil: Remove unused code

lgtm


Search not working

Hi, searching in my html-doc does not work.
What happens is:

I enter a string in the search-field.
2a) if that string does not exists:  I get shown immediately, that no results are found. So this is ok. But:
2b) if that string does exist, search runs forever and does not come up with results.

I do or did this:
A) install via: $ pip install sphinx_rtd_theme
B) use sphinx 1.5.3 and have created a new doc with: sphinx-quickstart
C) add the following at the end of conf.py:
import sphinx_rtd_theme
html_theme = "sphinx_rtd_theme"
html_theme_path = [sphinx_rtd_theme.get_html_theme_path()]
Any ideas? I saw that some people have problems with this. But I could not find a solution...

This is a duplicate of https://github.com/rtfd/sphinx_rtd_theme/issues/348

sorry about this.
Thanks for letting me know.
K


Fixes

Move code to Editor folder
Update Unity to 2019.4.12
Fix nullrefs
Move settings from EditorPrefs to local .json file

Can you label this as hacktoberfest-accepted for Hacktoberfest  event?


Converting 1x1 account model to Household model in Salesforce NPSP 3.0

Hi,
We are trying to test the upgrade to NPSP 3.0 and change the account model to Household in our sandbox, however after running the utility listed here on this page none of the contacts gets converter.
https://powerofus.force.com/articles/Resource/NPSP-Convert-an-Existing-Account-Model-to-the-Household-Account-Model
The utility counts the batches and then it says e.g. 0 of 846 batches processed.
Is this a known issue and is it related to sandbox only?
Thanks

@ramtinworld The conversion tool will only look at Contacts that have related Household custom object records. If the batch ran and there's no change, that's typically because there was nothing for it to do without the Household records. If that's not the case, please post a question in the Upgrading to NPSP group in the Hub as we reserve Github issues only for actionable items for our development team. Thanks for understanding!


Multiple files from GUI

As a user, I can process multiple files in a batch from a GUI

Too much for this activity!


Tableau connector doesn't ingest tags if there is a dot in tag name

Affected module
Does it impact the UI, backend or Ingestion Framework?
Ingestion framework
Describe the bug
A clear and concise description of what the bug is.
WARNING  {Ingestion:metadata_rest:376} - Unexpected error writing tag category [name=TagName(root='TableauTags') displayName=None description=Markdown(root='Tags associates with tableau entities') mutuallyExclusive='false']: Invalid name<EMAIL_ADDRESS>To Reproduce
Ingest tags from Tableau with dots in tag names
Expected behavior
Tags will be added to OMD
Version:
OMD 0.13.0
Tableau API 3.15
Additional context
Slack https://openmetadata.slack.com/archives/C02B6955S4S/p1673452949900439

Hi @gnomolio , we are unable to reproduce the above error. Looks like there seems to be issue with antlr in 0.13.0 . Please upgrade to latest version

Dashboard

Chart

Added tags with dots(.) , colon(:) and @, seems to be ingested properly

Thanks @NiharDoshi99 for checking this out. @gnomolio we'll close this for now as we cannot reproduce it. Please feel free to reopen this issue if the issue persists on your end after upgrading.
Thanks


Data base Data

Hello there!
Looked over your script and saw that theres some pre-genned data that needs to be used. I see you are grabbing it from your DB, is there any ways you can atleast show a sample of it or where to produce this data so we can make our on data?
Thanks 👍

@BRK0014  did you get the db base data or any workaround?


RegSet debug

We've discussed this PR offline, so it's OK, but in general, try to give a meaningful title and description to help reviewers know what it's about. It will be useful later anyway.
For work-in-progress PRs, a TODO-list of what's still to be done can help too. Not just for you, also for reviewers.
You can mark explicitly your PR as work-in-progress to avoid accidental merging: https://github.blog/2019-02-14-introducing-draft-pull-requests/

closed (new pull request opened: https://github.com/nalmt/regalloc.rs/pull/3)


Documentation and errors not clear about the profile

The documentation and error do not say which profile to load. For example, the stereotype required for aspect.

Note that the profile required seems to be SysML Extensions.mdxml (this should be mdzip).


Add complete references for PIQ

Turns out these were merged into the v2-dev branch a week or two ago, so are in the schema already. Would be good to have someone review though?

Will take a look today/tomorrow!

Section 1

Add "county" field under materialFacts/address
Section 1.4 has missing properties: "Details of sinking fund", "Details of estates charges"
Add piqRef under "parkingArrangementsType"
isListed -> yesNo is missing piqRef under multiple sections

Section 2

[x] Missing piq references under properties of 2.1
[ ] Field Mapping for 2.2 missing

Section 3

[x] Missing piq references under properties of 3.1, 3.2, 3.4,3.6,3.7,3.8
[ ] Field Mapping for 3.3.x missing

Section 4

[x]  Missing piq references under YesNo properties of 4.x
[ ] This section also has a "Dont Know" option which isn't current represented in the form

Section 5

[x]  Missing piq references under  properties of 5.x
[ ] 5.2 is missing, but is very similar to 5.1 ("preventative work on dry rot" vs "Has any treatment been done on dry rot") - should we add 5.2 ?

Section 6
NA
Section 7

7.1 some missing mappings - should be okay for now
[x] Missing piq references under  properties of 7.x

Section 8, 9, 10, 11

[x] Missing refs added & Invalid refs fixed

Section B

[x] Missing refs added & Invalid refs fixed

Review complete - Back to you @edmolyneux

Reopening issue - as some of the unchecked items above still need to be added; checks items are merged in #61 above.


vfrStatementRefreshEvent not documented for vfrFormDefinition (Bugzilla Bug 3053)

This issue was created automatically with bugzilla2github
Bugzilla Bug 3053
Date: 2020-11-04T23:29:50+00:00
From: Tim Crawford <>
To:   @lgao4
CC:   @lgao4
Last updated: 2020-11-10T20:55:36+00:00

Comment 14322
Date: 2020-11-10 20:55:36 +0000
From: @lgao4
Liming takes it.

Comment 14234
Date: 2020-11-04 23:29:50 +0000
From: Tim Crawford <>

Industry Specification: ---
Target OS: ---
Bugzilla Assignee(s): @lgao4

vfrFormDefinition and vfrFormMapDefinition support vfrStatementRefreshEvent [1], but it is not documented in the VFR specification [2].
[1] https://github.com/tianocore/edk2/commit/e2a5feb9d6c468c0ddb0caed258
[2] https://edk2-docs.gitbook.io/edk-ii-vfr-specification/2_vfr_description_in_bnf/211_vfr_form_definition


Input a custom matrix A to use the function simulationGLV

I am tring to use the function simlationGLV by inputing a custom matrix A. But it doesn't seem to work well, and the generated timing data will contain nan values during the simulation. Are there any restrictions on the matrix A？

This would happen if you have a numerical explosion in one or more of the variables (i.e. the system you are trying to simulate is not stable). The solution would depend on the size and numerical range of your matrix.
I would have to see your matrix to suggest ideas to solve your specific case, but this is not a malfunctioning of miaSim but rather a feature of the GLV model.

Thank you for your reply, I have no more questions for now


Chore: Update walg option in the config file

Signed-off-by: Lakshmipathi<EMAIL_ADDRESS>

Will be closing this in favor of https://github.com/supabase/postgres/pull/514


[BUG] NPE happens on short prefix query on a field with index_prefixes

Describe the bug
( index_prefixes is not documented in OpenSearch’s documentation but AFAIK OpenSearch is compatible with ElasticSearch 7.10, which has index_prefixes parameter.)
When a prefix query run on a field with index_prefixes, if the length of characters in the query is [min_chars of the index_prefixes option] - 1, an error is responded.
The root cause seems to be NullPointerException.
To Reproduce
Steps to reproduce the behavior:

Create an index which has a field with index_prefixes option; its min_chars is implicitly 2.

PUT test
{
  "mappings": {
    "properties": {
      "t": {
        "type": "text",
        "index_prefixes": {}
      }
    }
  }
}

Search by prefix query with one character on the field.

GET test/_search
{"query":{"prefix":{"t": "a"}}}

An error is responded. It says the root cause is NullPointerException.

{
  "error" : {
    "root_cause" : [
      {
        "type" : "query_shard_exception",
        "reason" : "failed to create query: Cannot invoke \"Object.hashCode()\" because \"this.rewriteMethod\" is null",
        "index" : "test",
        "index_uuid" : "XPqfSP1xT7WJt9Sxzt65qg"
      }
    ],
    "type" : "search_phase_execution_exception",
    "reason" : "all shards failed",
    "phase" : "query",
    "grouped" : true,
    "failed_shards" : [
      {
        "shard" : 0,
        "index" : "test",
        "node" : "QA35rulpSza20Wsx6rNSEg",
        "reason" : {
          "type" : "query_shard_exception",
          "reason" : "failed to create query: Cannot invoke \"Object.hashCode()\" because \"this.rewriteMethod\" is null",
          "index" : "test",
          "index_uuid" : "XPqfSP1xT7WJt9Sxzt65qg",
          "caused_by" : {
            "type" : "null_pointer_exception",
            "reason" : "Cannot invoke \"Object.hashCode()\" because \"this.rewriteMethod\" is null"
          }
        }
      }
    ]
  },
  "status" : 400
}

simple_query_string also fails.

# fails with the same error
GET test/_search
{"query":{"simple_query_string":{"fields":["t"], "query":"a*"}}}

Expected behavior
Run without errors. I know it is not efficient.
A similar query_string runs without any errors.
GET test/_search
{"query":{"query_string":{"fields":["t"], "query":"a*"}}}

I tried some patterns (min_chars = 2, 3, 5), and it seems that the prefix query with characters the length of which is min_chars - 1 fails. When the length is less than min_chars - 1, the query succeeds.
Plugins
N/A
Screenshots
N/A
Host/Environment (please complete the following information):

Version: 1.3.1

Additional context
Add any other context about the problem here.

@woby looks like a bug - if you/someone wants to contribute a unit test for this we can fix it faster

Thanks @VachaShah for taking care of this.
@woby the changes are merged to 2.0 and will be available in the next release.
Feel free to re-open if you have follow up questions, and thanks for reporting this.


[Epic] Spring Cloud Azure 4.0 - Migration guide is correct

We should make sure all of our migration guides are correct and up to date before the 4.0 GA.

[x] https://github.com/Azure/azure-sdk-for-java/issues/27478
[x] https://github.com/Azure/azure-sdk-for-java/issues/27489
[ ] #27488
[x] #27491

Closing this issue now, the 4.0.0 has been GAed.


数据在初次加载时未能完全绘制

作者你好，我现在遇到一个问题：
我将图表设置为可左右滑动的类型，当数据传递给图表后，未滑动状态下的数据是绘制正常的，当我向右侧滑动一段距离后，新出现的 x轴的数据未能成功绘制，但实际上是有数据存在的。当我点击图表视图弹出 tooltip视图后，缺失的部分就自动绘制出来了。请问这个现象是什么原因造成的，有什么解决方式吗？
下面是截图对比和图表打印的 log，麻烦作者协助看一下。谢谢。


Prototype binding mechanism

https://github.com/w3c/webdriver-bidi/issues/157
Prototype the mechanism using long-poll. Implementation details: http://go/webdriver-binding-implementation.

Spec work seems to be in progress in https://github.com/w3c/webdriver-bidi/pull/361/files

@sadym-chromium can this be closed?


fix: class bindings

The class that was actually being added was true instead of the intended active

Please use the staging branch for your PR's in the future

Thanks for the PR!


perf(redux): don't store channel edges

Description:
Do not store details of channel edges in redux.

Motivation and Context:
We are seeing an error in the console warning of excessive memory usage. It links to https://github.com/zalmoxisus/redux-devtools-extension/blob/master/docs/Troubleshooting.md#excessive-use-of-memory-and-cpu
This is caused by us trying to store all of the edges data from the network in redux. The edges data is massive and given that we aren't using it there is no need to store it in redux.
How Has This Been Tested?

Types of changes:
Performance improvement
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 20.046% when pulling 9a0ae0a955d2c0d081cfee17ad7a7b2039dcddaf on mrfelton:fix/dont-store-edges into a2acf858b10c2589b7145930a00136d039d6ba80 on LN-Zap:master.

Coverage remained the same at 20.046% when pulling 9a0ae0a955d2c0d081cfee17ad7a7b2039dcddaf on mrfelton:fix/dont-store-edges into a2acf858b10c2589b7145930a00136d039d6ba80 on LN-Zap:master.

Coverage remained the same at 20.046% when pulling 9a0ae0a955d2c0d081cfee17ad7a7b2039dcddaf on mrfelton:fix/dont-store-edges into a2acf858b10c2589b7145930a00136d039d6ba80 on LN-Zap:master.


Linux/MacOS binary placement

We currently default to /usr/local/bin to install the binary on MacOS and Linux machines. We should consider alternative locations (e.g. user's home directory, a system-wide location available in PATH provided programmatically by the OS)
Original Comment:
Nothing blocking but longer term I wonder if this should be in some part of the user's HomeDir, as to not require sudo.  If we do that we probably want to read https://wiki.archlinux.org/title/XDG_Base_Directory and see what location we should default to (I would guess XDG_DATA_HOME).
 We should also consider delivering azd via usermode package manager like homebrew, as it both aids discovery and providers the infrastructure for no-sudo utility acquisition.

Should this be in the current milestone re. package manager support?


Update Razor TextMate grammar to latest

Fixes https://github.com/dotnet/razor/issues/6631

i will trust that this works 😆

Me too, and I wrote it!


Fix backwards incompatibility with .NET standard 2.0 by re-adding overloads with string parameters

Fixes #436
Updates the code generator to always include overloads with string parameters regardless of the target framework
Methods with ReadOnlySpan<char> overloads are also still supported
Adds a .NET Standard 2.0 test library to the sample to make sure this doesn't happen again (I don't know if it's possible to unit test this but that would probably be way better)
Removes all target directives and ReadOnlySpan<char> usages from ImGui.Manual.cs
Adds a copy of ImGui.Manual.cs (ImGui.Manual.ReadOnlySpan.cs) which includes ReadOnlySpan<char> overloads

If the tests in the sample project are not needed I'm fine with removing them. Please let me know if I've missed anything. Any feedback is appreciated.

So on .net6 users have to explicitly pass/convert string to span in order to use span version?

So on .net6 users have to explicitly pass/convert string to span in order to use span variant?

Well, you already kind of had to do that. ReadOnlySpan<char> doesn't really make a difference for string literals (e.g. ImGui.Text("Hello, world!"); regarding heap allocations, since that string will only allocate once. But it does make sense to use spans for interpolated strings, since those always result in some allocations. If you want to prevent all those allocations you already had to provide a custom InterpolatedStringHandler anyway, and a method accepting that custom InterpolatedStringHandler can just return ReadOnlySpan<char> which is then passed to ImGui.
I don't believe this makes a difference for .NET 6+ users, and even if it does it should only be very small string allocations which are far less important than the entire library just not working properly for multi-targeted libraries supporting .NET Standard 2.0.


Add full Xau support to RustConnection

This contains the necessary magic to fix #196. Some of this code is a bit ugly, but I believe that all of this actually works. No idea if anyone will ever use this library in such a way that some of the more arcane bits of this code are executed...
This PR adds a new dependency on the gethostname crate. Yay for "bypassing" forbid(unsafe_code) by just using other people's unsafe code.

Oh, come on. You got to be kidding me...
error: Variant name ends with the enum's name
  --> src/rust_connection/xauth.rs:14:5
   |
14 |     X11Family(X11Family),
   |     ^^^^^^^^^^^^^^^^^^^^
   |
   = note: `-D clippy::enum-variant-names` implied by `-D warnings`
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names


VerticalSplitDiff doesn't handle table without primary key

I get an error like this. It should probably just give a warning and proceed to the rest of the tables. Or maybe VerticalSplitClone should fail in the first place.

E1122 09:46:04.956612    5204 vtworker.go:81] diff() failed: TableScan(source) failed: Cannot read Fields for query 'SELECT col1, col2 FROM tab ORDER BY ': vttablet: rpc error: code = 2 desc = gRPCServerError: error: syntax error at position 81

You're right: We should fail if a table does not have a primary key.
In general, the diff depends on a common sort order for both the source and the destination to correctly compare them.
If you run the clone only once, we don't depend on the sort order. But if you run it again, it tries to reconcile the source and the destination. It does so in the same way as the diff and assumes that the sort order is identical on both sides.
If the table has no PK, but a UNIQUE index, MySQL uses that. But if that's not the case, the sort order would be the insertion order and that might be different on source and destination. See: https://dev.mysql.com/doc/refman/5.7/en/innodb-index-types.html
Since our schema code currently doesn't capture the available indexes, we should play it safe and fail on tables without a PK at clone time.

I think this is now addressed. I'll close this unless I hear something back.


[$150] Profile - External user can't view the time zone(Local time) changes in the main user profile

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.61-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Prerequisite
Log in with Gmail account and changes time zone (Must be Gmail account)
Steps:

As another user (or from another device) navigate to the conversation with the main user (where all changes were made)
Click on the header of the conversation
Verify the user details page display local time of the user

Expected Result:
External user can view the time zone(Local time) changes in the main user profile. (Expensifail account behavior)
Actual Result:
External user can't view the time zone(Local time) changes in the main user profile.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/097ab10c-986f-453a-87b0-71b082cab8f2

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021858726442470926380
Upwork Job ID:<PHONE_NUMBER>470926380
Last Price Increase: 2024-12-17

Issue OwnerCurrent Issue Owner: @sobitneupane

Prerequisite
Log in with Gmail account and changes time zone (Must be Gmail account)

Steps:

As another user (or from another device) navigate to the conversation with the main user (where all changes were made)
Click on the header of the conversation
Verify the user details page display local time of the user

@IuliiaHerets Is the main user account (where all changes were made) validated? If not, the time zone is not expected to display.
cc: @strepanier03

@strepanier03, I believe this is not a bug but an intentional choice, as the user account is yet to be validated.

for self-profile, user can view their own timezone without any issue

@twilight2294  don't think it matters much whether we show or hide the timezone for the self-profile.

@strepanier03 https://github.com/Expensify/App/issues/52470#issuecomment-2497895134

Issue not reproducible during KI retests. (First week)

@strepanier03 https://github.com/Expensify/App/issues/52470#issuecomment-2497895134 Good To Close

Issue not reproducible during KI retests. (Second week)

Issue not reproducible during KI retests. (Third week)


🚀 Feature: Make template source tool tip more understandable in template card

🔖 Feature description
Currently on the template card, the location tool tip of where the template is stored, only tells you what integration it uses.

This isn't really helpful information for the end user.
My proposal would be to change the tooltip to be something like this: "See template code in X". The X would be the integration type. For example: "See template code in github"

🎤 Context
Would offer more helpful information for the end user about what this link does.
✌ Possible Implementation
Not the hardest thing to implement and I'm willing to do it. Only need to add some logic here: https://github.com/backstage/backstage/blob/master/plugins/scaffolder/src/components/TemplateCard/TemplateCard.tsx#L262
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Hey :wave: So these components are actually changing a little bit in an upcoming version of the scaffolder. Here's a little sneak peek:

I think that it would make sense for us just to provide a simple component out of the box like the above, and the ability to provide your own version of the card to which you can add more things to like the Link to source.. parts. Does this make sense and we can close this ticket?

Yeah I'm good with that. Clad to hear that it will offer more customization options in the future!😄

Thanks!


feat: improve tranassctions confirmation speed on eth

I think you need to update some tests @tima-t

CAL migrated to Chainify


change icon in device handlers

new device handlers work better... but you dropped the canChangeIcon: true from this version.
Not urgent but does need to be changed at some point.

also I found out last week if you also use 'canChangeBackground: true' you can use photos instead of icons... neat trick to get better image of your light (or whatever) you are controlling!

I have so many icons in there now I thought might be better to standardize. What specific tiles would make the most sense - I will enable both I guess - but the background colors determine what color is used on the fonts - I hate ST tile logic - it is so strict - took me ages to debug yday - I am sure I will have minor updates to do soon. I released version 1.0.2 on npm since changes needed to be pushed.
I also have done a a wee bit major change on the server to account for scenarios where folks might use MQTT wildcards across multiple devices and incoming topic might map to multiples of such - I feel like it has become quite robust in terms of all kinds of MQTT u can throw at it. I am testing it in my environment right now for the next few days - once it is stable I will release it. Having to maintain legacy functionality and finding workarounds is really becoming difficult

re icons and background... try the background with change icons... it does something you dont expect... it allows you to upload a picture!!!!  Like you, I thought it was just background colour!
Its a nice trick.
Re standardisation... people like choice... for many switches that want an icon to remind them what the switch is for... eg I have festoon (string) lights in the garden and spotlights and floodlights... different icons required for each rather than just a standard switch or bulb!
re code... can you pop it up, Im testing something (and Im doing it wrong) but I might as well test it on the new code.

udpated device handlers to accommodate icon and background change. Made a fix too so refresh will be faster (do you the refresh functonality? - i find that very useful)
uploaded server to test as v2 - try it out. mines been running since yday.

testing and will close as this isnt an issue now.


shred and watch build tooling ( dgeni, gulp, jade)

New gulp files for automation
New directories for documentation examples and fragments
New doc-shredder dir ( wrapper over dgeni task to shred files from one dir into another based on #docregion tags, ( + internal test dir)
New mixin utility function for jade/harp to pull in fragments
New sample 'alternative' version of the 'setup.jade' page to show example of shredder
Updated ReadMe.md

I signed it!

A couple things that could use better docs:

[ ] Why public/doc-shredder has its own package.json, and how those deps are automatically npm installed
[ ] Why the generated fragments (and any other generated files) are checked in.

Can you add that to the readme in public/doc-shredder (or wherever you think is appropriate)?
Other than that, looks good to me! :+1:

Only the ‘test’ dir in doc-shredder has its own package.json because I wanted to isolate any test specific dependencies.  I’ll probably remove this ( the package.json) later because it doesn’t seem like I need it any longer.
The generated files are checked in so that we can avoid running the shredder over the entire _examples dir each time someone refreshes the repo ( the ‘shred-full’ gulp task).  The gulp ‘serve-and-watch’ shredder is only a ‘partial’ shredder.  It only shred’s files in directories changed during the current session.  Hope this makes sense.
Can I do this in the next version or do you want these comments put in this version?
From: Brian Ford<EMAIL_ADDRESS>Sent: Thursday, August 6, 2015 12:04 PM
To: angular/angular.io<EMAIL_ADDRESS>Cc: Jay Traband<EMAIL_ADDRESS>Subject: Re: [angular.io] shred and watch build tooling ( dgeni, gulp, jade) (#179)
A couple things that could use better docs:

[ ] Why public/doc-shredder has its own package.json, and how those deps are automatically npm installed
[ ] Why the generated fragments (and any other generated files) are checked in.

Can you add that to the readme in public/doc-shredder (or wherever you think is appropriate)?
Other than that, looks good to me! [:+1:]
—
Reply to this email directly or view it on GitHubhttps://github.com/angular/angular.io/pull/179#issuecomment-128475344.

If you could amend this commit with what you just said in your comment,
that'd be perfect.
On Thu, Aug 6, 2015, 12:17 jtraband<EMAIL_ADDRESS>wrote:

Only the ‘test’ dir in doc-shredder has its own package.json because I
wanted to isolate any test specific dependencies. I’ll probably remove this
( the package.json) later because it doesn’t seem like I need it any
longer.
The generated files are checked in so that we can avoid running the
shredder over the entire _examples dir each time someone refreshes the repo
( the ‘shred-full’ gulp task). The gulp ‘serve-and-watch’ shredder is only
a ‘partial’ shredder. It only shred’s files in directories changed during
the current session. Hope this makes sense.
Can I do this in the next version or do you want these comments put in
this version?
From: Brian Ford<EMAIL_ADDRESS>Sent: Thursday, August 6, 2015 12:04 PM
To: angular/angular.io<EMAIL_ADDRESS>Cc: Jay Traband<EMAIL_ADDRESS>Subject: Re: [angular.io] shred and watch build tooling ( dgeni, gulp,
jade) (#179)
A couple things that could use better docs:

[ ] Why public/doc-shredder has its own package.json, and how those deps
are automatically npm installed
[ ] Why the generated fragments (and any other generated files) are
checked in.

Can you add that to the readme in public/doc-shredder (or wherever you
think is appropriate)?
Other than that, looks good to me! [:+1:]
—
Reply to this email directly or view it on GitHub<
https://github.com/angular/angular.io/pull/179#issuecomment-128475344>.
—
Reply to this email directly or view it on GitHub
https://github.com/angular/angular.io/pull/179#issuecomment-128478907.

No problem.

Ok, removed Licence file and extra superflous package.json in doc-shredder test dir. Added additional readme files and comments.


Bug: Setup error - incorrect file renaming

Describe the bug and the steps to reproduce it
When creating app with npx @coinbase/build-onchain-apps@latest create, the process will contain an error: Error while renaming directories: Error: ENOENT: no such file or directory, rename '/home/.../web/src/OnchainProvidersWithRK.ts' -> '/home/.../web/src/onchainProviders.ts'
This is because the existing file is .tsx, not .ts. Furthermore, capitalization is also incorrect (should be OnchainProviders, not onchainProviders).
What's the expected behavior?
No errors
What version of the libraries are you using?
latest

This fix was merged and released with https://github.com/coinbase/build-onchain-apps/releases/tag/v0.28.1, please let me know if it's all good.

Yep it works!


Update typescript types

Updates the types for typescript.
Is required for https://github.com/zeit/now/pull/2926

Codecov Report

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

@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   90.01%   90.01%
=======================================
  Files          10       10
  Lines         991      991
=======================================
  Hits          892      892
  Misses         99       99

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 49bc51a...adc89ab. Read the comment docs.

@AndyBitz Looks good! Feel free to release (probably semver patch?) after CI passes.


Refactor code by using kube-prompt as dependency

Describe the issue you're facing
Instead of coping kube-prompt code, see if we can use it as Go dependency. This will eliminate most of the code from pkg/. It make easy to upgrade to latest kube-prompt.

/assign. I will look at it.

Thanks @sbdtu5498. Assigned to you. Let us know if you need any help.


Language on Login page isn't generic enough,  doesn't play nicely with Elemental, has unneeded options display.

Niggle 1:
On the standard login page, there is a heading: "Sign into your website."
This language is fine when it's only a site with admins/editors, but it's not appropriate wording if you are setting up a site with non-admin logins (private content, memberships, etc), specifically the word 'your'.
So I'd suggest that this label should just say 'Sign in'.
Although the login page can be overridden to change this text, this is more of an advanced activity. I would imagine there'd be lots of users that want to created protected areas, with just the Elemental theme, but wouldn't know how to do overrides.
Niggle 2:
If the Login page's theme is overridden, i.e. with :
'theme_paths'         => array(
'/login'            => 'elemental'
)
The login page doesn't look particular nice; it's got misaligned elements, the photo credits in a weird spot and the background image applies over the footer. It would be great if this page could be set to be the non-photo page, making it match the Elemental theme, without having to also make edits to the single page template itself.
Niggle 3:
The login page also shows a list of login types (I believe for signing in with Facebook, Twitters, etc). When there is only 'Standard' login available, listing this here isn't necessary and only adds to make the page more cluttered/confusing. So a suggestion would be to make this only list the login types if there are two or more.

yes. yes to all of this. 2/3 make this difficult to clean up and override too.


fix: symlinks not working in store

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 19 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 24.761%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app.ts
0
19
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
3687

Relevant Lines:
15729

💛  - Coveralls


add Neo.Runtime.Serialize/Deserialize and tests

What current issue(s) does this address, or what feature is it adding?

closes #328

How did you solve this problem?

implement Neo.Runtime.Serialize/Deserialize

How did you make sure your solution works?

added tests

Are there any special changes in the code that we should be aware of?

no

Please check the following, if applicable:

[x] Did you add any tests?
[x] Did you run make lint?
[x] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage increased (+0.3%) to 78.689% when pulling c9b21c82ed4b6150796844070ff99d9cadae49a4 on feature-runtime-serialize into 7c38ef3a567391df0f03177cb6488ce50d85d390 on development.


[stable/sonarqube] volume mount /usr/local for copy_plugins.sh replace all previous content included Open jdk installed

Describe the bug
When I deploy stable/sonarqube the /usr/local content is replace with copy_plugins.sh script, deleting all previous content. The problem is that Open jdk 1.8 is in this place, causing the sonarqube does not to start.
Which chart:
Latest.
What happened:
SonarQube cannot init because java is not longer exist. And show this message: Error running exec bin/run.sh, java is not in PATH. Or something similar.
What you expected to happen:
SonarQube start without problem.
How to reproduce it (as minimally and precisely as possible):
Always with base or custom values.
Anything else we need to know:
If volume mount for copy_plugins.sh is changed for other location, pe. /usr/src, sonarqube start ok.

I changed the deployment.yaml to put the file in /tmp/scripts in #14141, which works for me.


Determine whether there is a holiday problem

var_dump(\zjkal\ChinaHoliday::isHoliday("2025-01-01 00:00:00"));
var_dump(\zjkal\ChinaHoliday::isHoliday(1735660800));

Please make sure you use a version >= v1.1.10

It has been fixed.
The cause has been found, which may be due to the time zone difference of the PHP operating environment.
Please update to the latest v1.1.11
If you still find problems, please give us feedback in time, thank you.


[RFC] swscale avformat resize video output

Dear Alfred,
below pull request is a "draft" which works, for enabling resizing the video within avformat, using swscale & video_size in the config.
The PR is at this stage to be considered as draft. There are too many unclear questions how to handle things the way to suite baresips overall design.
Points are the following:

Currently there is only video_size in the config. The PR uses video_size for setting the "output" (SIP transfered) video_size. The same value is also used across modules for setting the video input size. How should be distinguish between sz_src (size source/input) & sz_dst (destination/output) in future? Do we want separate config values, or eventually introduce a swscale / aformat parameters? The PR uses video_size for sz_dst.
Is there a way to check, except evaluating the config file parameters, if a module was successfully loaded?

I will need to think about more about this and come back with better answers later.
but for now:

The video size used to display incoming frames, is the same size as the
incoming stream. This is a very simple logic. We might add a new config option
to force a specific resolution, in this case we also need to consider keeping
the aspect ration between incoming frame and the configured vidisp output.

One module cannot check if another module is loaded. A given module should
not have to know if another module is loaded or not. The baresip module
architecture is very simple; baresip.h defines and interface, the baresip core is using
that interface, and a number of modules are implementing that interface.

Dear Alfred,
thank you for your comments & feedback.
incoming stream. This is a very simple logic. We might add a new config option
to force a specific resolution, in this case we also need to consider keeping
the aspect ration between incoming frame and the configured vidisp output.```

I'd like to have an option to resize/rescale a video stream on the output side in order to have a possibility reducing video stream size for hardware phones. Some of them (e.g. Polycom VVX600, Grandstream GXV3140, Yealink VP530, ...) do only support CIF/QCIF or similar resolutions. My intention is not related to local display output or baresip display output, since nearly any nowadays used PC would easily handle the streams without rescaling.

For avformat that topic is not too important, since we are dealing with input formats where I have not encountered any situation needing to specify the input video size. This question came up thinking on input devices like V4L2 where it does make sense to specify the input resolution.

I'm just wondering if the aspect ratio should be our concern, or swscales and the users. Will test how the patch does when feeding values which are not the same aspect ratio. I missed that point, and think a warning could be sufficient in such case (depending how swscale does with unequal aspect ratios on input/output).

Another completely different approach would be to set the output size parameters during codec negotiation evaluating the offered h264 profile-level-id from the counterpart.

I have misunderstood and thought that you wanted to change the VIDISP resolution.
now that I understand your usecase, I can only say that it is a valid usecase and
a very nice improvement.
A similar problem would also be solved with this; for example the H.263 encoder
supports only certain resolutions. If the vidsrc cannot provide this exact resolution,
then the video encoding pipeline will fail.
An example video encoding pipeline:
.--------------.
|    videnc    |
|              |
| (avcodec.so) |
'--------------'
     /|\
      |   352x288 (a.k.a. CIF)
      |
.--------------.
|   vidfilt    |
|              |
| (swscale.so) |
'--------------'
     /|\
      |   320x240
      |
.--------------.
|    vidsrc    |
|              |
|  (e.g V4L2)  |
'--------------'

the swscale destination parameter can either be read from video_size
or we can add a new config swscale_encode_size (not sure what is best)

I've just tested 546456b which does the job nicely. :)


Migrate logging and tally data to Winston

As described in #195
Everything is compatible with the old version of the logger, except that now every line of server logs is similar to
[2021-08-31 21:53:19] info: Starting HTTP Server. or [2021-08-31 21:53:40] error: An error occurred etc

Winston options for file output:

Current options used in server loggin and in Tally data:
{
  filename: filepath,
  maxsize: 3e+7
  maxFiles: 3,
  ...
}

@JTF4 can you explain to me what is Tally data logging and why its format is
{TallyObj},  {TallyObj}, {TallyObj},?

@MatteoGheza the tally data logging is there so that we can enable edit decision list exporting in the future.

@MatteoGheza the tally data logging is there so that we can enable edit decision list exporting in the future.

@JTF4 can we change the format into
[
  {TallyObj},
  {TallyObj},
  {TallyObj}
]

or into
[
  { timestamp: current_timestamp, tally: {TallyObj} },
  { timestamp: current_timestamp, tally: {TallyObj} },
  { timestamp: current_timestamp, tally: {TallyObj} },
]

I see no reason why we can't make that change. If we come up with an EDL feature in the future, I have some specific thoughts on how I would want to see it handled anyway, so it's fine with me to make changes to how it's currently logged to a file.

The latter should work. As long as the tally data object has the device name in it with the tally1-4 it will work. In the future I'll also add a time code to the log object so it can sync with a recorder like a ki pro.

In future, when we start working on EDL and similar features, I think I can replace Winston with a custom function for storing Tally data in a better method then the one used now or then the method used before this PR.

Sounds good. I shall open a new issue sometime to describe what I was thinking for EDL.

@josephdadams @JTF4 is this ready? Do I have to revert edits to Tally Data logger?

If it works (I haven’t had time to test), it’s fine with me to merge. W


2 Suggestions c:

Hi, bad english, again xd.
• Add a cancel button
• Fix ASAP /quests a moddata (for testing quests)
• Add some method to prevent players for place and breaking, just for quests like "Farm 64 Sugar Cane" or "Break 64 Emerald Ore"

agreed

/quests a moddata has been added in v2.0.6-beta


GCP Appengine via Terraform?

Hi, so I can see AppeEngine in the https://github.com/argoproj-labs/argo-cloudops/blob/main/go.mod#L131
but I'm not sure if anyone has ever used this without AWS.
Has anyone got any pointers for how to launch a service in GCP, preferably AppEngine?

While one of the goals is to be agnostic to various clouds, the current code does not support clouds aside from AWS. To support additional clouds, there are a number of areas in the code which need a good chunk of redesign/refactoring work. We're currently planning on starting this work in 2022.


Sort table alphabetically by model name

This is definitely a matter of taste, but I think it will be easier to find the models in the table if the table is sorted alphabetically by model name rather than Org name. So, I changed "Org Name - Model Name" to "Model Name - Org Name" in the first column and sorted the table accordingly.

Yes and no.
For me personally it's a bit more difficult to read simply because not all models have a link and thus there is a "visual break" in the list.
If you add links to the three remaining models, then I will agree that this is better. At least link to their HF pages.
Btw, the first column is called "Model and usage". Shouldn't it be something like "Model and author"? Word "author" might not be the best fit, but "usage" is definitely is even worse.

If you add links to the three remaining models, then I will agree that this is better. At least link to their HF pages.

done

Btw, the first column is called "Model and usage". Shouldn't it be something like "Model and author"? Word "author" might not be the best fit, but "usage" is definitely is even worse.

I think that table column header was because it was linking to the internal download and usage docs. Let's just call it "Model" for simplicity.

Thanks Sebastian.


Update GRPC protocol to have raw tensor data at top-level of message

This allows a more performant implementation due to the way protobuf
clearing and reuse operates in conjunction with GRPC APIs.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

/assign @ellistarn
Ellis, this is the performance change to the grpc protocol that I mentioned earlier.

@rakelkar can you get ONNXRuntime in touch here?

/ok-to-test

Apologies. I just rewrote history to fix the contributors list. This PR needs a rebase before it can be reopened.


Dataflow + basis for future work

add semantic ui for future interface
fix dev performance bottlenecks (namely geojson js import)
add interactive word search, later behaviour to define
add async example between python and js
add word to lang idx

LGTM


broken links

https://azure.microsoft.com/en-us/documentation/articles/virtual-machines-extensions-configuration-samples-windows/
this article has links to non-existent MSDN pages.
@squillace

@sebastus thanks. I didn't even know we had that page.


[Question] Paypal hybrid integration automated testing with doctored payment_id/payer_id?

Description
When testing a hybrid paypal integration, I'm having trouble finding a way to automate the server-side test. I'm creating the payment on the client-side and using the checkout.js button.render() method, however the onAuthorize() step calls through to my web server to handle the sale and execute the payment together.
I need to eliminate the paypal button click and login steps as they're not relevant to my server-side test.
With Stripe for example, I've got extensive automated integration testing and am able to test against Stripe easily, as they provide test credit card tokens (see https://stripe.com/docs/testing) that have specific meanings and produce expected error messages or valid sales.
I found the PayPal negative testing page:
https://developer.paypal.com/docs/api/nt-rest/
Which would partially do what I'm after, however this seems to only cater for error conditions and involves modifying my server code to have a "test" mode where it inserts mock headers into the POST request.
The Question
Can you perhaps provide a list of payment_id and payer_id's for the sandbox which could be used to call /execute the same way as production code would, and perform expected actions? Something like:

Payment ID
Payer ID
Result

payment_id_CREDIT_CARD_CVV_CHECK_FAILED
anything
CREDIT_CARD_CVV_CHECK_FAILED error returned

payment_id_VALID
payer_id_WRONG_ID
CARD_TOKEN_PAYER_MISMATCH

payment_id_VALID
anything other than payer_id_WRONG_ID
return a successful payment with an id, etc.

Thanks!

Which would partially do what I'm after, however this seems to only cater for error conditions and involves modifying my server code to have a "test" mode where it inserts mock headers into the POST request.

Can you elaborate on why that's problematic?

@lmarkus, whose team has been doing a lot on the negative-testing front

@pnemere , My team is actively building a negative testing framework that will support input-based triggers to complement the header-based ones.
It's meant to work more or less as you described.  If you don't mind, could we talk offline? I;m sur emy team would like to run some scenarios by you.  Please shoot me an email.  lmarkus at paypal dot c0m

Thanks Markus, will email you.
@bluepnume Well it's problematic for 2 reasons:

You have to insert special test code, which you have to turn on & off, and effectively when it's running an integration test it's not testing the actual production code. Arguably it's a slight change with 1 flag but if the rest of the system doesn't need a special flag (like with Stripe)

I didn't see a way of producing a successful purchase result, which for an automated testing scenario is vital too.

Thanks for the quick replies guys!


Coding Best Practices.

Highlights :star2:

Your code is professionally written in a way it's readable and maintainable.

Suggestion :bulb:

I noticed in line 37 on index.js, you left a redundant code you might have commented on. I kidly suggest it be removed, as it makes your code clean and professional.
In your To-Do list module (from line 16 to line 32), you had element.index repeated multiple times. I kindly suggest you create a variable to store that index, as it makes your code less cumbersome.
For clarity, please rename your task.js and tasks.js to avoid confusion. The reason for this is simply to make sure when other developers want to maintain your code, they are able to understand what each module means.

Good job! :clap:

Thank you very much. could send me yours
On Mon, Feb 27, 2023 at 12:51 PM Caleb Nwaizu @.***>
wrote:

Highlights 🌟

Your code is professionally written in a way it's readable and
maintainable.

Suggestion 💡

I noticed in line 37 on index.js, you left a redundant code you
might have commented on. I kidly suggest it be removed, as it makes your
code clean and professional.
In your To-Do list module (from line 16 to line 32), you had
element.index repeated multiple times. I kindly suggest you create a
variable to store that index, as it makes your code less cumbersome.
For clarity, please rename your task.js and tasks.js to avoid
confusion. The reason for this is simply to make sure when other developers
want to maintain your code, they are able to understand what each module
means.

Good job! 👏
—
Reply to this email directly, view it on GitHub
https://github.com/brhanuhailu/to-Do-list/issues/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARSSYJE2KIMILK4DCMRF6L3WZSPORANCNFSM6AAAAAAVJLRZXM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Configuration issues

I still don’t quite understand how config.yaml is configured. Can the configuration process be more detailed? thank you very much

if u use the command the author provided:  cd workflows/general && vim config.yaml
when you enter vim, you can just enter these, and then save them
custom_openai_endpoint: "your base"
custom_openai_key: "your key"
bing_api_key: "your key"

or u can just create "config.yaml"  in the correct path and enter the info

I added sample config files, please checkout:
workflows/general/config.yml
workflows/video_understanding/config.yml


[Docs] Relationships and Policies

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 8bd7405da86f6eeae0e9f6f659713d43cdac3641 at: https://65c4c554c322751154262ac6--meshery-docs-preview.netlify.app

🚀 Preview for commit b7efcc4153a669e895b1927b6b607ec2842138ce at: https://65c50cb93838cf2f8edd184d--meshery-docs-preview.netlify.app

Thanks

🚀 Preview for commit 6b0d9cffa14cd3b540c696b745fdc12c0565c6d4 at: https://65c620f6be75aa0a208d5b05--meshery-docs-preview.netlify.app

@leecalcote looks good?

@RipulHandoo  would you like to add?

1st half of logical page enhanced

🚀 Preview for commit 07a512f7cb6fbe7f98167adb35772a2a5018c722 at: https://65cbb141db914c037b462752--meshery-docs-preview.netlify.app

🚀 Preview for commit 617f69e7cb3b051ca9d548e1edd45ada0830932b at: https://65cbd393136e022377d5358c--meshery-docs-preview.netlify.app

More work needs done here.


Manual std::hash::Hash implementation for consistency with PartialEq

This guarantees that the PartialEq for Hash would be consistent. Please let me know if anything has to be changed, modified or if it is bluntly incorrect.
This implementation is similar to the derive macro implementation.

Hi, thank you very much for the PR! This looks nearly perfect, but there is one important implementation detail to note.
It's important that two Expr::Adds or two Expr::Mul instances that would be semantically equal are considered equal under PartialEq. For example, the existing PartialEq implementation on Expr treats these as equal:
Expr::Add([1, 4, 9, 16, 25]) == Expr::Add([25, 16, 9, 4, 1])
Expr::Mul([36, x, y]) == Expr::Mul([y, 36, x])

You would need to consider this detail in your implementation of Hash. Currently, it appears that the order of terms in Expr::Add or products in Expr::Mul would influence the resulting hash. However, this shouldn't be the case to match PartialEq.

This problem is akin to the problem of hashing a multiset, where the order of elements does not matter, but the frequency in which they occur does. One idea is hashing each element on its own, and summing or XORing the resulting hashes, which would result in the same hash regardless of order, though there are a lot of interesting reads online about this topic. That could possibly give you some inspiration.

Yeah, that would a good place to start from. Found something related: CS Stack Exchange

This looks good to me! There are still a number of things I would change style-wise, but as the functionality is all there, I've decided to merge this. Thanks for your hard work!


Use webhook for bitbucket

Can I use Bitbucket webhooks for the webhook link instead of POST service?

Can you explain more in details what you mean (with links to Bitbucket documentation)? You need that PHP Censor instance call Bitbucket?
P.S.: In any case, now is exists only webhook from Bitbucket (Bitbucket docs and PHP Censor docs).

Do you ask about this? If yes then your case should work.

I confirm that it's working (at least for me)

Excellent! Thank you for testing.


Can not download  node-sass (ppc64le)

When I run npm install in Ubantu 14.04.1 LTS Linux  .There is an error says it can not find this package and then It runs postinstall to build itself. Is there a work around solution to support my ppc64le?

Node sass < 4 is very old and no longer supported. Additionally we don't
currently support ppc.
On Tue., 3 Apr. 2018, 1:30 pm QxiaoQ<EMAIL_ADDRESS>wrote:

When I run npm install in Ubantu 14.04.1 LTS Linux .There is an error
says it can not find this package and then It runs postinstall to build
itself. Is there a work around solution to support my ppc64le?
[image: image]
https://user-images.githubusercontent.com/31646170/38227576-6cd6c2d4-3731-11e8-8eb0-a0df9b4526d0.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2311, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWOoKFRM0myrPcX-8-1S3ydDP7v2Qks5tkuzYgaJpZM4TEZbE
.

You can upgrade to 4.8.3 and try running npm rebuild node-sass --force to try a local compilation of node-sass.


Transformer implementation

I'd like to refer to the attention and feedforward implementation for transformers here:
https://github.com/deepmind/neural_networks_chomsky_hierarchy/blob/main/models/transformer.py#L361-L379
I've heard that some implementations compute attention and the feedforward in parallel to speed up computation, is this the case here?
Also, the feedforward seems to be missing an initial linear transform, the relu is directly applied on h.

Hi @shawntan, thank you for your comment!
Indeed, the current code provides an implementation of the Universal Transformer (Dehghani et al., 2019). We reran our experiments with the original Transformer (Vaswani et al., 2017) and did not find a significant difference in the results. We will update the results and the code with the next revision of our paper.

I have implemented a vanilla transformer and have found the same results.
I'm curious how the current implementation implements a universal transformer though.
https://github.com/deepmind/neural_networks_chomsky_hierarchy/blob/main/models/transformer.py#L362-L379
Aren't the parameters being initialised within the for-loop, therefore having unique parameters at every layer?


Unit tests support

The Adyen_Hyva module should be supported with PHP unit tests.
All PHP files should be supported in this kind of way.
Work would be done on a separate branch checkout out from the main one, and subsequently a PR will be opened for review purposes.
Side notes:
In a weekly meet, we have discarded the need of api-functional and integration tests, deciding to add only Unit tests as a complement to the end-to-end tests.

Unit test have been added and test workflow has been activated. Closing this issue.