Aggregation dimension classifications

We need to add dimensions control to the aggregation API, so that, in addition to the spatial aggregation (controlled by resolution and placement) we can add additional expressions into the GROUP BY of the aggregation.
We don't want to allow arbitrary SQL expressions for the dimensions. We'd like to support categorical columns and classified continues values. For the latter, they could be numbers or times and we need to control the granularity of the classification, e.g. if the time is shown as hours, days, months... or if prices are shown in 3 intervals, etc.
We'll start by implementing the time case and validate the API by integrating its use in CARTO VL
For time aggregation, there are two interesting cases:

Grouping a time series into time units such as minutes, hours, days, months, years. A refinement of this would be two allow groups of N units, e.g. 10-minute buckets, 4-month, etc.
Grouping as before into time units, but computing them relative to (modulo) a larger time unit: hours of the day (0-23), days of the week (0-6), months of the year (1-12), ...

API parameters
We could give support to these needs with an API like this: (dimensions would be a parameter within the aggregation of the layer)
dimensions: {
  hour_of_day: {
    column: 'time_of_arrival'
    granularity: 'hour',
    multiplicity: 1, // optional: number of units per grain
    cycle: 'day',    // optional: granularity relative to this cycle
    offset: 0,       // optional: starting value for granularity
    timezone: -7200  // optional: UTC offset in seconds of the result
  }
}

Note: this nomenclature is unfortunate, because the date histogram already uses offset for the time zone offset. So we must review this. (if for example we decide not to support the offset defined here, see below, be could use  offset for the timezone offset for consistency with histograms).
The offset parameter could be useful to specify both the order of cyclic units and how they are grouped:

With multiplicity > 1, e.g. 3 for months/year, by default we would get quarters as jan-feb-mar, may-apr-jun, etc.
If we use an offset of 1, we would get feb-mar-may, ... nov-dec-jan instead.
With multiplicity = 1 it would simply define the starting point of the cycle, e.g. what day of the week is the first (0), sunday, monday?

The timezone parameter is necessary because it determines how time is broken up into days, etc. The user would specify in which timezone is the classification to be performed.
The general SQL expression for the classification would be equivalent to:
(with usec the duration in seconds of the grain units, csec the duration of the cycle in seconds, m the multiplicity, tz the timezone)

Cyclic case:
(FLOOR(date_part('epoch', timezone(tz, column)) / (usec*m)) - offset) % (csec/(m*usec))
Non-cyclic case:
FLOOR(ate_part('epoch', timezone(tz, column) / (usec*m)) - offset

This is just a conceptual example, it would work only for regular units of time, but months and larger units are irregular because their duration in seconds is not fixed.

If we're really into offsets we could add an additional cyclic_offset so that both the start of groupings of multiple units, and the order of the resulting grouping order (for cycles) can be independently defined. The resulting expression would be:
(FLOOR(date_part('epoch', timezone(tz, column)) / (usec*m)) - offset - c_offset*csec/(m*usec)) % (csec/(m*usec))
I hope we all agree we don't need that much.

Result Units
Then we need to define what units to use for the result.
To support clients through MVT it seems best to limit ourselves to numeric formats (not dates or strings). Then we have these options:

For the cyclic cases we have some common cases that users will expect:

hour of day: 0-23
day of month: 1-31

month of year 1-12
day of week: 0-6 or 1-7 ?

For the non-cyclic case I see three options:

ordinal number for the specify grain (as in the SQL expression above); e.g. if the grain is three-month periods,
then if 2018-01-01 corresponds to N, 2018-03-01 will correspond to N+1.
ordinal numbers for the specify granularity unit (we'd need to multiply the SQL expression by the multiplicity);
then if 2018-01-01 corresponds to N, 2018-03-01 will correspond to N+3 whatever the multiplicity is.
UNIX epoch number in seconds (so we generate full dates truncated to the grain precision)

For the last case the advantage is that the values could be easily formatted as dates, compared with other dates, etc, and we would have homogenous values independently of the units used.
But the value spacing would be irregular. (this irregalarity could be beneficial if we want to animate dates, but in general, e.g. for ramp-coloring, charts, having the ordinal values would be more useful and consistent with the cyclic case)
Examples
Group data by day of week, with sunday=0, etc.
dimensions: {
  dow: {
    column: 'the_date'
    granularity: 'day',
    cycle: 'week'
  }
}

Resulting SQL expression: date_part('dow', the_date)
Group data by day of week, with monday=0, etc.
dimensions: {
  dow: {
    column: 'the_date'
    granularity: 'day',
    cycle: 'week',
    offset: 1
  }
}

Resulting SQL expression: (date_part('dow', the_date) - 1) % 7
Note: we could also use the ISO convention 1=monday, etc and use date_part('isodow', the_date)
Group data by hour of day (0-23)
dimensions: {
  hour: {
    column: 'the_date'
    granularity: 'hour',
    cycle: 'day'
  }
}

Resulting SQL expression: date_part('hour', the_date)
Group data by days
dimensions: {
  interval: {
    column: 'the_date'
    granularity: 'month'
  }
}

SQL: date_part('epoch', date_trunc('day', the_date))/(24*60*60)
Group data by months
dimensions: {
  interval: {
    column: 'the_date'
    granularity: 'month'
  }
}

SQL: date_part('month', the_date) + 12*(date_part('year', the_date)-date_part('year', to_timestamp(0.0)))
Group data by 10-minute intervals
dimensions: {
  interval: {
    column: 'the_date'
    granularity: 'minute',
    multiplicity: 10
  }
}

SQL: FLOOR(date_part('epoch', date_trunc('day', the_date))/60 / 10.0),
Questions

Units of the result. I say: let's go for the first option above, i.e. ordinal number of groupings, be it 10-minute groupings or 3-month or whatever
Do we need offset?. I'd say let's do it. So maybe we could rename offset to start and timezone to offset or perhaps tz_offset.
What to do with uncommon cyclic cases.
We'll use specific expressions for the cases such as month-of-year (date_part('month', the_date)). For unexpected combinations of units
(e.g. minute/day), we could either not support it (produce error) or use a generic SQL expression such as the above. I'd leave the generic expression as a fall-back to experiment with it.
Are you happy with the parameter names? (I am OK with them).

cc @dgaubert @rochoa @davidmanzanares @Algunenano  I'd like you to take a look at this and specifically at the questions, to see if you're answers differ from mine.

TLDR: Let's start supporting what we can do natively with Postgresql functions (date_part, extract) and wait to see if supporting other subfields / cycles is really necessary.
I feel this is a super complex approach that is trying to handle too many options at once, which is not wrong, but I think we should wait to see if all those options are really needed, specially the multiplicity, cycle and offset parameter. Adding and supporting all these options will make this a big monster.

timezone: -7200  // optional: UTC offset in seconds of the result

After working in the histograms I can say that supporting this as numeric is a huge PITA and doing it correctly is literally impossible (you can't handle timezone switches with just the offset from UTC). I would strongly recommend moving to a string based timezone, supported and handled by Postgresql.

This is just a conceptual example, it would work only for regular units of time, but months and larger units are irregular because their duration in seconds is not fixed.

Not really; hours and days are also irregular, you can have leap seconds, changes in timezones (to half hours for example) and so on.

Then we need to define what units to use for the result.
To support clients through MVT it seems best to limit ourselves to numeric formats (not dates or strings). Then we have these options:

I'd avoid strings not only for MVT, but for the simple reason that they come in a certain language. Epoch is universal and the frontend can do whatever it wants with it.
Let's start with whatever is more simple using native functions and avoid adding math that comes back to bite ue:
Example for grouping by day (with start as 00:00):
Select date, min(cartodb_id) as cartodb_id from
(
    Select date_trunc('day', timezone('Europe/Madrid', time)) as date, cartodb_id FROM all_month
) _a
group by date order by date
limit 5;
        date         | cartodb_id
---------------------+------------
 2017-07-11 00:00:00 |      10492
 2017-07-12 00:00:00 |      10130
 2017-07-13 00:00:00 |       9621
 2017-07-14 00:00:00 |       9202
 2017-07-15 00:00:00 |       8796
(5 rows)

LIMIT 5;

Grouping by week of the year:
Select date, min(cartodb_id) as cartodb_id from
(
    Select date_part('week', timezone('Europe/Madrid', time)) as date, cartodb_id FROM all_month
) _a
group by date order by date;
 date | cartodb_id
------+------------
   28 |       8373
   29 |       5648
   30 |       3172
   31 |        899
   32 |          1
(5 rows)

By day of the week (0..6. Use isodow for 1..7)
Select date, min(cartodb_id) as cartodb_id from
(
    Select extract('dow' from timezone('Europe/Madrid', time)) as date, cartodb_id FROM all_month
) _a
group by date;

For the last case the advantage is that the values could be easily formatted as dates, compared with other dates, etc, and we would have homogenous values independently of the units used.
But the value spacing would be irregular. (this irregalarity could be beneficial if we want to animate dates, but in general, e.g. for ramp-coloring, charts, having the ordinal values would be more useful and consistent with the cyclic case)

You are asking for irregular formats, you get irregular ranges.

Units of the result. I say: let's go for the first option above, i.e. ordinal number of groupings, be it 10-minute groupings or 3-month or whatever

IMO, I would offer epoch output for everything and, for known ranges (hours of the day, days of the week or month, etc) I would also offer the appropriate ordinal part (0..23 for hours of the day or 0 of 6 for day of the week). Trying to use ordinals for irregular intervals where it's not handled by Postgresql will be a headache (it can be done, but expect a lot of corner cases).

Do we need offset?. I'd say let's do it. So maybe we could rename offset to start and timezone to offset or perhaps tz_offset.

I don't think it's worth it right now. I don't see anyone really needing the hours to start at minute 05 instead of 00, for example.

@Algunenano, regarding this:

After working in the histograms I can say that supporting this as numeric is a huge PITA and doing it correctly is literally impossible (you can't handle timezone switches with just the offset from UTC). I would strongly recommend moving to a string based timezone, supported and handled by Postgresql.

I saw your code to handle time zones in the histograms, but  what I don't understand is, I've been using expressions of the kind timezone('${tz} seconds', ${t})\  where tz is the offset in seconds and t is a date column and they worked for me. Isn't this deterministic? can this yield different results depending on the kind of the date column or the PG version or any other thing?

My 2 cents: I think we can remove a little bit of functionality, but I think that keeping cycle would be better (IMO, this is a common case).
Regarding the return type, I agree on returning Epoch in general, but in combination with cycle something needs to be done since Monday or 13:45 is not a particular point in time after Epoch.

I saw your code to handle time zones in the histograms, but what I don't understand is, I've been using expressions of the kind timezone(INTERVAL '${tz} seconds', ${t})\ where tz is the offset in seconds and t is a date column and they worked for me. Isn't this deterministic? can this yield different results depending on the kind of the date column or the PG version or any other thing?

Nobody needs offset +3600. They need the time in Madrid which sometimes is GMT +1 and some other times is GMT+2 so requesting 3600 is only right half the time (and force us to do funny translations for Postgresql).

@dgaubert: I intended granularity to be a single simple time unit for simplicity: second, minute, hour, day, etc. Then to support buckets such as 6 months (a semester) or 10 minutes (which could be interesting in  some data-logging situations) or 15 days, etc I added the multiplicity, which might not be a very good name, it's just the number of units.
On second thought the names are confusing, maybe we could have granularity_unitsand number_or_units or something.
To have a single granularity property it would have to accept expressions such as "6 months". I prefer to have separate properties to avoid the parsing.

Nobody needs offset +3600. They need the time in Madrid which sometimes is GMT +1 and some other times is GMT+2 so requesting 3600 is only right half the time (and force us to do funny translations for Postgresql).

Very true @Algunenano. I'll think about either allowing any name supported by PG.
By the way, regarding all your other concerns about complexity of this... My approach now is to start with this "most complicated" API, then go through some real use case examples and see how it fits and if it can be simplified. That should also give us an idea of what could be the best format for results; we initially considered epoch numbers for all, then we realised that for the cyclic case it wasn't the most convenient, then I thought that it wasn't either for the other cases... but I'd like to experiment with a full stack (client CARTO VL expressions and Maps API) before taking a decision.
And thank you very much mate for your comments and examples, you've really know this stuff :heart:

On second thought the names are confusing, maybe we could have granularity_unitsand number_or_units or something.

I feel that the naming is a bit confusing but I guess I'm not the only one. I would look for something that reads as hours of the day and avoid granularity or multiplicity names:
Maybe:
dimensions: {
  dow: {
    column: 'the_date'
    group_by_quantity: 1,
    group_by_units: 'hours',
    group_by_of: 'day'
  }
}

Or even more simple:
dimensions: {
  dow: {
    column: 'the_date'
    group_by: `hourOfDay`
  }
}

And just have a list of valid group_by names and it's translation to other units. Then, if you decide to open it more (5 minutes of the hour) you can either add an extra string 5minuteOfHour or add an extra option:
dimensions: {
  dow: {
    column: 'the_date'
    group_by: `minuteOfHour`,
    step: 5
  }
}

Or even more simple:

dimensions: {
  dow: {
    column: 'the_date'
    group_by: 'hourOfDay'
  }
}

That was what I had in my mind.

Current situation (WIP):
I'm considering some changes to the current implementation, some minor ones/easy to implement:

Review parameter names
Solve time-dimensions / dates_as_numbers dependency
Add dimension filters

And a more complex one:

Add dimension metadata

Background
Tiler API parameters
I'm using this parameters currently for the dimensions of the aggregation:
{
  column: 'the_date',
  groub_by: 'day', // 'dayOfMonth', ...
  group_by_count: 3, // optional (1)
  offset: 0, // optional (0)
  timezone: 'Europe/Madrid' // optional ('utc' == '+0000')
}

As mentioned before, offset might be confusing because it is used as a time zone offset in the histrograms.
I'm accepting in timezone both fixed TZ offsets in seconds and TZ names (including DST rules) accepted by PostgreSQL (so all IANA names are accepted).
The parameters of the internal tiler functions are slightly different:
{
    time: `to_timestamp("the_date_epoch")`,
    timezone: 'Europe/Madrid',
    granularity: 'day',
    multiplicity: 1,
    cycle: 'month',
    offset: 0
}

Dimension filtering
We should add dimension filtering to the Maps API (like aggregation columns filter); so it is applied at the proper place of the aggregation query (before/at grouping).
Dates as numbers
dates_as_numbers must be set to true for a layer to be able to use time dimensions. So we should either fail with an error otherwise or automatically set the options when time dimensions are used.
Another option is to adapt the time GROUP BY expressions depending on the value of dates_as_numbers, but we'd need to pass all the layer options to the aggregation query function.
Dimension metadata
As discussed here it could be convenient to provide metadata for the grouped dimensions. Unlike for aggregated columns stats, this is not zoom-dependent and can be computed on the base table. But we need share some information between the aggregation query generation and the stats computation.

Hey @Algunenano comments are welcome I've been using the PR #1039 with CARTO VL and I'm happy with it. As it is you it gives either numbers (counters of the grouping periods like months or 10-day spans) or ISO strings also (but with some restrictions: no multiple units nor starting time). For day of the week etc no ISO format is supported.
The parameters look like this:
{
    "dimensions": {
        "time_in_days": { // Name of the resulting dimension column
            "column": "the_time", // column discretized for the dimension
            "group": { // defines how to group the values of the column
                "units": "day", // grouping units
                "count": 1, // optional, number of grouping units per period, 1 by default
                "timezone": "Europe/Madrid", // optional (by default 'utc' == '+0000')
                "starting": "2018-01-01" // by default '0001-01-01T00:00:00', periods counted from this time
            },
            "format": "number" // optional, "iso" indicates ISO 8601 format
        }
    }
}

Quick comment:

// by default '0001-01-01T00:00:00', periods counted from this time

I think it should be 1970 as every other mayor date system

This was merged in https://github.com/CartoDB/Windshaft-cartodb/pull/1039


Add missing case for loading analysis.R, exhaustiveness checks

Fixes a bug where analysis.R files were not being loaded properly.
This also adds some uses of the typescript never type to try to ensure we won't have similar "missing case in switch" bugs again

Looks good to me

I haven't observed the original bug--am happy for you to merge without my independent verification though if you have confirmed it is now fixed.
The never trick for exhaustive switch cases is a nice one. Good addition. Might consider also adding to stancWorker.ts though we seem more likely to potentially add new message purposes/types to that one?


New project with CRNA cannot find entry file

Please make our job easier by filling this template out to completion. If you're requesting a feature instead of reporting a bug, please feel free to skip the Environment and Reproducible Demo sections.
Description
1-2 sentences describing the problem you're having or the feature you'd like to request
After installing create-react-native-app globally, I try to run create-react-native-app foo. This creates a foo project. After running cd foo to enter the foo directory, I run npm start. The terminal outputs the following:

When I try to use the Expo app to scan the barcode, the terminal outputs that there was an error compiling the JavaScript bundle.

Expected Behavior
What action did you perform, and what did you expect to happen?
I ran npm start inside of the project created by create-react-native-app. I expected that the terminal would output a qr code that I could scan with the Expo app so that I could see the project was set up successfully.
Observed Behavior
What actually happened when you performed the above actions?
When I scan the qr code in the terminal, I am provided the terminal output above.
If there's an error message, please paste the full terminal output and error message in this code block:
See above
Error text goes here!

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts: `--<EMAIL_ADDRESS>npm ls react-native: `--<EMAIL_ADDRESS>npm ls expo: --<EMAIL_ADDRESS>-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>node -v: v6.11.3
npm -v: 3.10.10
yarn --version: 'yarn' is not recognized as an internal or external command,
operable program or batch file.
watchman version: {
"version": "4.9.0",
"buildinfo": "2e41dcec654687136a4bfc8fefd8911dbed367e3 2017-08-23T18:23:28.0000000Z"
}

Also specify:

Operating system: Windows 10
Phone/emulator/simulator & version: Android (Samsung Galaxy S6 running Android 7.0)

Reproducible Demo
I've put this on github here: https://github.com/jd1388/create-react-native-app-issue
To reproduce, pull the repo, run npm install inside of the project, then run npm start. Make sure to be using the same versions of node and npm as above.

Hi! Can you try using npm 4? It seems like fbemitter isn't being flattened properly between its various dependencies.

I've updated npm to 4.6.1, but that did not fix it. However, I tried updating CRNA and using that to build a new app, and I got it to work. I updated my repo in case your curious as to what changed.

strange. well I'm glad that updating fixed it, if you run into it again feel free to make another issue


Creation of Package/ClusterPackage resource should be prevented if the manifest has the wrong scope

Describe the bug
We only support the following combinations of resource type - manifest scope:

ClusterPackage - Cluster
Package - Namespaced

Everything else might lead to undesired behavior.
To Reproduce
Create a file package.yaml with the following contents:
# This is WRONG! cert-manager is cluster scoped!
apiVersion: packages.glasskube.dev/v1alpha1
kind: Package
metadata:
  name: cert-manager
spec:
  packageInfo:
    name: cert-manager
    version: v1.15.1+1

Run kubectl apply -f package.yaml
Expected behavior
The creation should be prevented.
Screenshots
Creation is accepted.
Cluster Info (please complete the following information):

Type: Kubernetes
Version: -
Provider: -

Glasskube Info (please complete the following information):

Package Operator version: 0.12.1
CLI version:0.12.1

Additional context
This validation should be done by the validating admission webhook:
https://github.com/glasskube/glasskube/blob/f9a1dfe58b3cffa2df6f2f50b4280dc29e6e8ff0/internal/webhook/package_validator.go

I can take a look at this ticket

Hi @ankitsridhar16 how's it going with this issue? any updates or any questions?

Hi, if @ankitsridhar16 is not working on it, can I take a look into it.
Thanks.

Will a simple check between v1alpha1.PackageManifest and ctrlpkg.Package be enough? (inside the validateCreateOrUpdate function)

Please take a look at the following function:
https://github.com/glasskube/glasskube/blob/f9a1dfe58b3cffa2df6f2f50b4280dc29e6e8ff0/internal/webhook/package_validator.go#L110-L134
Here you already have access to the Package/ClusterPackage and the relevant manifest.


Changed name of file to copy in Dockerfile

The lookup-desc-template.xml file does not exist, so changed the name to the existing lookup-descriptor-template.xml file.

Thanks! The reason this is hard to keep track of is we are using our own custom dictionary internally and so I'm keeping a diff between master and an AMI on AWS with our own changes. So I definitely appreciate help keeping the master in condition that it would work for someone who wants the default setup.


KAFKA-10449: Add some important parameter desc in connect-distributed.properties

…rties.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

In WokerConfig.java we found that REST_HOST_NAME_CONFIG(rest.host.name) and REST_PORT_CONFIG(rest.port) were deprecated. And some new configuration parameters are introduced such as LISTENERS_CONFIG(listeners), REST_ADVERTISED_LISTENER_CONFIG(rest.advertised.listener),ADMIN_LISTENERS_CONFIG(admin.listeners) but not list in the sample conf file.

Thanks for the PR @tinawenqiao.
It would be nice to also mention rest.port and rest.host.name are deprecated in their descriptions. Could you add something like:
DEPRECATED: only used when listeners is not set. Use listeners instead.  to their descriptions in WorkerConfigs?
Thanks

Thanks for the PR @tinawenqiao.
It would be nice to also mention rest.port and rest.host.name are deprecated in their descriptions. Could you add something like:
DEPRECATED: only used when listeners is not set. Use listeners instead.  to their descriptions in WorkerConfigs?
Thanks

Thanks for your notice. A new patch is ready.

@tinawenqiao It looks like you made a mistake when rebasing your branch, can you ensure the PR only contains your diff? Thanks

@tinawenqiao It looks like you made a mistake when rebasing your branch, can you ensure the PR only contains your diff? Thanks

Sorry for the mistake. A clean patch is submit. Is it ok now?

Thanks for the update, it looks much better.
I should have been clearer, I meant to update the documentation text of these settings, not just the comments in WorkerConfigs.
For example in https://github.com/apache/kafka/pull/9235/files#diff-5664ddd985065623bab1870e031ce4f3fcfee0bc4951e41b6b0e1584cd32fa2dR153, we could do something like:
private static final String REST_HOST_NAME_DOC
    = "Hostname for the REST API. If this is set, it will only bind to this interface. Deprecated, only used when listeners is not set. Use listeners instead.";

Thanks for the update, it looks much better.
I should have been clearer, I meant to update the documentation text of these settings, not just the comments in WorkerConfigs.
For example in https://github.com/apache/kafka/pull/9235/files#diff-5664ddd985065623bab1870e031ce4f3fcfee0bc4951e41b6b0e1584cd32fa2dR153, we could do something like:
private static final String REST_HOST_NAME_DOC
    = "Hostname for the REST API. If this is set, it will only bind to this interface. Deprecated, only used when listeners is not set. Use listeners instead.";

Doc desc has been added.

Thanks for the update, it looks much better.
I should have been clearer, I meant to update the documentation text of these settings, not just the comments in WorkerConfigs.
For example in https://github.com/apache/kafka/pull/9235/files#diff-5664ddd985065623bab1870e031ce4f3fcfee0bc4951e41b6b0e1584cd32fa2dR153, we could do something like:
private static final String REST_HOST_NAME_DOC
    = "Hostname for the REST API. If this is set, it will only bind to this interface. Deprecated, only used when listeners is not set. Use listeners instead.";

Doc desc has been added.

@mimaison is it ok now?


Multiple adapter Error

Hi, when i am executing waterline with one adapter, everything is working well:
(inside config/model.js)
connection: 'mongoServer'

In my mongo server is creating every model and the index values.
But when i want to create one model with a custom adapter (I am developing 'sails-elasticsearch'), this model is not creating in mongo, until i insert a document:
(inside model/product.js)
connection: ['mongoServer', 'elasticsearchServer']

attributes: {...}

Is this happen because i have a mistake in the custom adapter or is because a sails problem?
Thanks

Which adapter methods does each adapter implement?  elasticsearchServer methods will probably override mongoServer methods (when present) the way this is written.

Please reopen if you feel this hasn't been fully addressed.


unauthorized: incorrect username or password

###environment
kitematic for mac
###behavior

search mysql

create mysql official

###error info
(HTTP code 401) unexpected - {"message":"Get https://registry-1.docker.io/v2/library/mysql/manifests/latest: unauthorized: incorrect username or password"}

Please try to login using your Docker ID (hub username) instead of your email. This should fix the issue.


mysqlSetupJob.password can't be set to a string

Describe the bug
I think you have a bug here: password
To Reproduce
Steps to reproduce the behavior:
# In values
mysqlSetupJob:
    password: lskdjflksjdflskdj

Expected behavior
It uses the password string
Screenshots
│ Error: template: datahub/templates/mysql-setup-job.yml:72:44: executing "datahub/templates/mysql-setup-job.yml" at <.Values.mysqlSetupJob.password>: can't evaluate field value in type string

gonna close because I only needed this for a quick and dirty POC. prob not worth supporting


Added Frames Timecode Calculator

Fixes Issue🛠
Closes #1124
Description👨💻
I have added a Frames to Timecode Calculator which convert the frames with a given frame rate to the appropriate timecode
Type of change📄

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
I have tested the working of this feature. Check the Link↗ to the video of the demo.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Also, add the option to convert Timecode to Frames in this same calculator UI, and make the new commits in this same PR!!
And then rename the folder to Frames-Timecode-Calculator and make the changes accordingly @AMS003010

Don't make separate calculators by making 2 html, css, and script files. Include both in a single index.html, style.css and script.js and should be displayed in same single UI @AMS003010


image updated

Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@JainM6 The content in the public doc is already inline with this latest image, but I missed to update this image.

@JainM6 The content in the public doc is already inline with this latest image, but I missed to update this image.

Okay. Thank you for the confirmation.


Changed std::unique_ptr to Scope in Application.h

The std::unique_ptr is changed to Scope. Nothing to explain more :)

Discard the changes to imgui.ini, as they aren't required for this pull request.

Okay Done @ThomasJowett  😄

Seems like there are still a few wrong line endings in those files... maybe try copy pasting the files from what is currently on github. If fixed, I'll merge this in.

Seems like there are still a few wrong line endings in those files... maybe try copy pasting the files from what is currently on github. If fixed, I'll merge this in.

Okay 😄

@LovelySanta Done 🙂


Feature/use spring boot di for dbpw

This PR updates hibernate-validator to the newest version to pass CVE-2020-10693.
Further, it changes the injection of the database encryption password from manual (via System.getProperty) to a Bean. This lets SpringBoot resolve the variables and allows variable substitution, which we need in order to use the secret injected by CloudFoundry.

Closing in favor of https://github.com/eu-federation-gateway-service/efgs-federation-gateway/pull/275


I have a question

In my APP Its have group chat and single chat ,when I watched old chat history in group room ,the cell is dismiss,but in single chat room no this program

is it possible to add a Video of this happening so we can more fully understand what you are describing?
You can easily record your phones screen by

Plugging it into your mac opening the application 'Quick Time Player'
Create a new Screen recording
Select your phone as the screen source by clicking the small down arrow next to the record button

If that is not possible if you run the simulator and press (Command⌘ + Shift⇧ + S) it will take screen shots of the simulator. Add those to this issue with descriptions and we will see if we can help.
That will help us help you. Thanks for your help. 😼

Thanks! 😄
Please provide all the information in the issue template so we can help!
And don't forget to read our contribution guidelines. 😊


WL-5329 SAK-24289

Can one of the admins verify this patch?

@nicholaswilson100 this is ready to go.


refactor(payments): INT-6930 Zip: remove "(Quadpay)" from checkout

What? INT-6930
Removed the string (Quadpay) from all the translation files.
Why?
A Zip team request.
Testing / Proof
Before:

Now:

@bigcommerce/apex-integrations  @bigcommerce/checkout

:green_heart: Tested!
QE details:
The task for ZIP has been validated without issue.
The following was tested:

CP01_T6930_ZIP_Auth&Capt_ZIP_SetUp_US
CP02_T6930_ZIP_Auth&Capt_Payments_US
CP03_T6930_ZIP_Auth&Capt_OrderComplete_US
CP04_T6930_ZIP_AuthOnly_Payments_US

Please see the attachment word below with the evidence:
https://drive.google.com/drive/folders/16GqGZ6LcWjVZaZma_GAEYs2cKeHsgOd3?usp=share_link
Please let me know any comment or doubt


Modified ShowNusMod behavior

Issue
Issue number - implements #248
PR type
Modify implementation

Pull Request Test Coverage Report for Build 482

0 of 0   changed or added relevant lines in 0 files are covered.
53 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.5%) to 51.008%

Files with Coverage Reduction
New Missed Lines
%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/commands/AddNusModsCommand.java
2
89.55%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/commands/AddNusModCommand.java
3
84.78%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/commands/ShowNusModCommand.java
4
54.55%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/parser/ParserUtil.java
44
50.0%

Totals

Change from base Build 478:
0.5%

Covered Lines:
3693

Relevant Lines:
7240

💛  - Coveralls

lgtm!


Error on importing

I imported this package using:
npm install react-input-colorpicker --save
and tried it using. But the following error is coming:

Please tell me how to filx this.

Also happened to me.

You can configure Babel to handle the ES6 destructuring.


fix obvious merge problem

remove the default isClose class which was removed by @lade72 already in the feature branch

Yepp, thx for the hint Captain!


Add closure for complete bulletin board dismissal

Checklist

[X] I've tested my changes.
[X] I've read the Contribution Guidelines.
[x] I've updated the documentation if necessary.

Motivation and Context
We needed a callback that fired when the bulletin board was dismissed (as opposed to individual bulletin items).
Description
Very simple addition tagged on to existing 'completeDismissal' method.

this is preferable (in our particular use case) over attaching the dismissal closure to the last bulletin itself

Thank you for your contribution @apptekstudios! We added this feature in the 2.0.0-beta.1 release.
You can update the library to try it, but I recommend to wait for the final release, as some APIs may be unstable / unfinished.


The highlight feature isn't working.

The highlight feature isn't working.
Description
When trying to use the highlight feature on the PDF viewer, an error message appears stating "Please select some text in the PDF first." Despite selecting text, the highlight feature does not activate.
Steps to Reproduce

Open the MLOrbit PDF viewer at mlorbit.netlify.app/view-pdf.
Select any text within the PDF.
Click on the 'Highlight' button.

Expected Behavior
The selected text should be highlighted in the specified color and thickness.
Actual Behavior
An error message appears: "Please select some text in the PDF first."
Screenshot

Environment

Browser: [e.g., Chrome, Firefox]
OS: [e.g., macOS 11.2, Windows 10]
URL: mlorbit.netlify.app/view-pdf

@shyama7004, can you consider using this plugin? It's free.


feat: add docker compose migration

replace/build-upon #33
add --migrate flag to the local install command (see diagram below for workflow of how this works)
add --persisted flag to local uninstall to remove persisted data
cleanup how options are passed to install

replace too many parameters with a struct

change when k8s events are logged as warnings

will only show a warning if the event has occurred at least 5 times
previously this was set to any event was shown, which resulted in unnecessary noise
all events will still be logged if verbose mode -v is enabled

TODO

[x] I need to take another pass at this code, I expect there is some strangeness in here, but want to get some other eyes on it before it goes on even longer

flowchart TD
    mStart[migration start] -->dv{does\n`airbyte_db`\nexist}
    dv -->|no| err(return error)
    dv -->|yes| conCopyStart(tmp container\nwith `airbyte_db` mounted)
    subgraph "                                              First Container"
        conCopyStart -->  conCopyAction(copy `airbyte_db` pgdata to\n`~/.airbyte/abctl/data/airbyte-volme-db`)
        conCopyAction --> conCopyEnd(tmp container removed)
    end
    conCopyEnd --> conCopySuccess{success?}
    conCopySuccess -->|no| err
    conCopySuccess -->|yes| conTransStart(tmp container with\n`~/.airbyte/abctl/data/airbyte-volume-db` mounted)
    subgraph "                                              Second Container"
        conTransStart --> conTransActionUser(add postgresql user)
        conTransActionUser --> conTransActionUserSuccess{success?}
        conTransActionUserSuccess -->|yes| conTransActionDB(rename database)
        conTransActionDB --> conTransActionDBSuccess{success?}
        conTransActionDBSuccess -->|yes| conTransActionDBEnd(tmp container removed)
    end
    conTransActionUserSuccess -->|no| err
    conTransActionDBSuccess -->|no| err
    conTransActionDBEnd --> mEnd[migration complete]
    mEnd --> install[existing installation continues...]

Few small comments, and a suggestion for the StopContainer method


NWC: create invoices via BTCPay Server

Since a couple of days, it's possible to add a lightning wallet via NWC to BTCPay Server. However when doing this with the Mutiny connect string, I get the following:

Invalid URL (No commands available or not all required commands are available (get_info, make_invoice, lookup_invoice, list_transactions))

The issue is tracked here: https://github.com/Kukks/BTCPayServerPlugins/issues/54
I'm just opening an issue with you, in case changes to your NWC interface could be needed.

We don't support list transactions and aren't planning to add it.


Editor page updated meta data

Editor page lacked def meta headers

Thank you!
What's the purpose of the highlight.js stylesheet?

It is used to create syntax highlighting as you see in the image I didn't add it I just found it in the head, it was being used for syntax highligthing on the editor page, I just made it global and it doesn't affect any thingon other page,  only on the editor page for toolkit and workshop.


Two sound machines went offline

Describe the bug
Entities in Home Assistant didn't report unavailable but no interactions worked. Wasn't running debug logs :(
To Reproduce
Unknown. Maybe my token expired?
Expected behavior
Devices continue to work without needing a HA reboot.
Screenshots
None
Additional context
Also noticed the wifi sensor stated disconnected so some sensors got created but some of the entities failed to function specifically the media player.
Did look at full log but noticed this.
Logger: homeassistant.components.media_player
Source: custom_components/ha_hatch/riot_media_entity.py:48
Integration: Media player (documentation, issues)
First occurred: January 22, 2023 at 9:04:49 PM (4 occurrences)
Last logged: January 22, 2023 at 9:09:10 PM

Error adding entities for domain media_player with platform ha_hatch
Error while setting up ha_hatch platform for media_player
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 438, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 709, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 803, in add_to_platform_finish
    await self.async_added_to_hass()
  File "/config/custom_components/ha_hatch/rest_entity.py", line 52, in async_added_to_hass
    self._update_local_state()
  File "/config/custom_components/ha_hatch/riot_media_entity.py", line 48, in _update_local_state
    _LOGGER.debug(f"updating state:{self.rest_device}")
  File "/usr/local/lib/python3.10/site-packages/hatch_rest_api/riot.py", line 120, in __str__
    return f"{self.__repr__()}"
  File "/usr/local/lib/python3.10/site-packages/hatch_rest_api/riot.py", line 113, in __repr__
    "is_clock_on": self.is_clock_on,
  File "/usr/local/lib/python3.10/site-packages/hatch_rest_api/riot.py", line 136, in is_clock_on
    return self.flags & RIOT_FLAGS_CLOCK_ON
TypeError: unsupported operand type(s) for &: 'NoneType' and 'int'

not sure if this solves all of what you are experiencing but https://github.com/dahlb/hatch_rest_api/commit/3ad05efa4524f7ac6a402e10d054f752d9e5d92f should handle what's in the logs which is caused by the mqtt partial updates nature where some attributes are not initialized at all times

the offline issue might have been related to https://github.com/dahlb/ha_hatch/issues/10

You're correct same issue today and I see the issue reported in that you linked to and I'm not seeing the same error about the flags.


i open face_mask_detection.tflite via Netron, why the dimension of this are [1, 5972, 4] and [1,5972, 2] ?

I want to use face_mask_detection.tflite port into android.
and there is a question, why label just 2 type [NoMask, Mask], but the dimension of this file are [1, 5972, 4] and [1,5972, 2] ?

If you want to use it on Android, you need to wirte the corresponding decode and nms codes. You can refer to our python decode and nms function to write the Java or C codes.
Are you Chinese? If yes, you can add my wechat: AIZOOTech. We have written an Android demo.

------------------ Original ------------------
From: "Frank Chu";
Date: 2020年4月30日(星期四) 下午3:05
To: "AIZOOTech/FaceMaskDetection";
Cc: "Subscribed";
Subject: [AIZOOTech/FaceMaskDetection] i open face_mask_detection.tflite via Netron, why the dimension of this are [1, 5972, 4] and [1,5972, 2] ? (#40)

I want to use face_mask_detection.tflite port into android.
and there is a question, why label just 2 type [NoMask, Mask], but the dimension of this file are [1, 5972, 4] and [1,5972, 2] ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

is it the android code opensource at somewhere?

there is example for android?


Since 3.20: Bar chart throws exception in case of duplicate x values

Bug report
Codepen
https://codepen.io/Crossing/pen/ZEWGBjL
Explanation

What is the behavior you expect?
The 'old' behavior as of version 3.19.3:

What is happening instead?
Subsequent labels are not rendered anymore. Exception is thrown.

What error message are you getting?

Thanks for reporting, fixed it

Wow, that was fast! Thanks a lot for the fix and the great work you are doing here 😊


Can (should?) an ImageFileReader have access to Transform* functions?

The itk::simple::Image Class has the functions:
std::vector< double > 	TransformContinuousIndexToPhysicalPoint (const std::vector< double > &index) const
std::vector< double > 	TransformIndexToPhysicalPoint (const std::vector< int64_t > &index) const
std::vector< double > 	TransformPhysicalPointToContinuousIndex (const std::vector< double > &point) const
std::vector< int64_t > 	TransformPhysicalPointToIndex (const std::vector< double > &point) const

Which as far as I understand it, do not require/operate on the actual pixel/voxel data of an image. As such, I think these functions should be exposed from an initialized ImageFileReader:
reader = sitk.ImageFileReader()
reader.SetFileName(file)
reader.ReadImageInformation()

As at this point all the necessary information is loaded.
There are a variety of cases where one might want to load the metadata from a collection of files and use this function to calculate something about physical space, without loading the pixel/voxel data ( see https://discourse.itk.org/t/finding-the-largest-physical-region-bounding-box-across-an-arbitrary-number-of-input-files-simpleitk for example)

Going down the route of adding support for all the functions of an Image to the ImageFileReader does not seem the correct route.
For this particular problem, and these particular methods, the size of the image does not matter. So a possible solution ( not too elegant ) would be to create a image of size 1, and copy the metadata from the ImageFileReader to the Image of size 1.

img = sitk.Image([1,1,1], sitk.sitkFloat32)
reader = sitk.ImageFileReader()
reader.SetFileName(file)
reader.ReadImageInformation()
#img = sitk.Image(reader.GetSize(), sitk.sitkFloat32)
img.SetSpacing(reader.GetSpacing())
img.SetOrigin(reader.GetOrigin())
img.SetDirection(reader.GetDirection())
dims = reader.GetSize()
spcs = img.GetSpacing()

Does seem to do what I wanted. Thanks!

I'm a bit surprised that TransformContinuousIndexToPhysicalPoint which I'm using on the image doesn't complain that the image is size 1, it will happily accept the numbers from the reader.GetSize() I'm using.


Subscription query hangs

The subscription query in the guide hangs
subscription AddedCommentByAuthor{
  addedCommentByAuthor(author:"Nadia"){
    id
    author
    content
  }
}

Can you share what your mutation was? AppSync requires that the filter used in the subscription be in the response of the mutation, and it can seem like it's hanging if it isn't there.


Added DDR5 Link

Added the DDR5 link..

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

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

Added the DDR5 link that was not present in the Document

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


bug (hubble): One of the hubs has different count of links as compared to other ones

What is the bug?
link count differs for 1 of the hubs despite being at 100% sync on 1.7.0
How can it be reproduced? (optional)
const { getSSLHubRpcClient } = require('@farcaster/hub-nodejs');

const testHub = async (hubRpcEndpoint) => {
  let client = getSSLHubRpcClient(hubRpcEndpoint);
  let fid = 194;

  client.$.waitForReady(Date.now() + 2000, async (e) => {
    if (e) {
      console.error(`Failed to connect to ${hubRpcEndpoint}`);
    } else {
      console.log(`Connected to ${hubRpcEndpoint}`);
      const linksResult = await client.getLinksByTarget({
        targetFid: fid,
        linkType: 'follow',
        pageSize: 10000,
     });
      console.log(`${JSON.stringify(linksResult.value.messages.length)} follows for fid ${fid} from ${hubRpcEndpoint}`);
    }
    client.close();
  });
};

// has lower count
testHub('myhub-with-lower-link-count.xyz:2283');

// have same count
testHub('nemes.farcaster.xyz:2283');
testHub('hoyt.farcaster.xyz:2283');
testHub('myhub-with-same-link-count.xyz:2283');

What are the counts you are seeing?

What are the counts you are seeing?

right now - 8042 for the hub with lowcount, 8045 for the rest

Thanks for the report. Going to close given the age and the fact that we've released many improvements to the hubs since this was opened, but if you continue to see this do open again and include the latest details. Thanks!


Global actions?

Very nice project, just asking about global actions, rather then defining an action for each password, I want to specify one globally for all my passwords

@Arrow-x global actions are out of scope, as I only made this script for myself and have no interest in maintaining it. However, if you know some bash scripting it should be relatively easy to implement yourself.


Make install dir configurable

This patch allows users to change the directory where antibody is installed

should be fixed on b06e4a8dace1bf0b95f1b041ecc8a0f6f95acb3a


Make plugin available as Github Release or on Bintray

Need to learn how to make code downloadable rather than only buildable.

Kinda done. Still need to hookup CD pipeline


[Documentation] Update RGBa parameter example to a perceptual colorspace like CIELab?

In the documentation about parameters, 4-dimensional RGBa color coordinates are visualized in 2D, and colored, to showcase how various UMAP parameters affect the final reduction. However, RGBa is not a perceptual color space, which means readers' intuitions about which points should be grouped together based on color similarity will be off.
One could instead convert the RGBa coordinates to CIELab (a perceptual colorspace). One way to do this is to go from RGBa to RGB, and then RGB to CIELab (a perceptual color space) via the color transformation functions found here: https://scikit-image.org/docs/dev/api/skimage.color.html. There might be other ways to do this conversion as well.

It's a good idea. The original RGBA example was nice just because it was a simple to explain 4D space that people could still have some basic intuition for. I would be happy to look at a PR for the same walkthrough with CIELab or similar.

So I've started working on this... Using CIELab coordinates results in UMAP reductions with more intuitive color <=> spatial differences (especially for the N=1 component image -- all of the points are nicely separated into colored bands.) On the other hand, CIELab is only a three-dimensional space, which means that the N=3 components reduction is moot.

Perhaps you could interleave the two options, explaining why CIELab makes more sense, but use the RGBa where appropriate. The different metrics section at the end is also probably easier in straight RGBa. So perhaps there is a nice balance between the two?

I think I've solved the problem another way due to the fact that CIELab is about color, alpha is about transparency. I can generate 4D coordinates where the first three are Lab coordinates, and the 4th is an alpha coordinate. Such 4d points can be run through UMap, and then for coloring I can convert the Lab coordinates to RGB and combine with the alpha... Voila, perceptual RGBa.
Colorspace purists (?) might balk, but the initial results look good.

Color space purists will definitely balk more at the pure RGBa version.


Consider adding sortInPlace to mutable collections

On Seq only, obviously.  It doesn't really have sensible performance characteristics on LinearSeq but last I checked we don't have that any more.  So just mutable.Seq or mutable.IndexedSeq.  The key is that you need to have a fast swap operation.  (Otherwise you are rebuilding two new collections: a new sorted collection, and yourself again, now sorted.)

I will give it a try.

What is the contract for this? Provided for any subclass that allows sub-linear sorting, without constructing a copy?  For example a linked list can be sorted in sublinear time using merge sort, but can it be done without creating a copy of the input?

@erikerlandson What do you mean by sublinear time for sorting?
For me, sortInPlace make sense only for collections with fast rndom access, in our case IndexedSeq. What do you think?

@kamilkloch I mis-spoke there, but basically "not greater than n-log-n cost" was what I was getting at.
@Ichoran mentioned "mutable.Seq or mutable.IndexedSeq" above - and maybe I am reading too much into it but mutable.Seq includes ListBuffer, which you could sort with merge sort in n-log-n time, but it might involve copying original nodes, like mapInPlace appears to do.
So there are possible policies. "Only available if cost is not greater than n-log-n" or "only available if objects are not being allocated to do it".  Maybe these were already documented?

Since sorting in place doesn't make sense for a lot of (most?) mutable.Seq implementations, where is the method defined? On all mutable.Seqs, and it's documented as just a best-effort thing? Or only on particular implementations for which it makes sense?
If we follow Java's strategy (copy to an array, sort the array, and copy back), it could have a reasonable default implementation for most/all mutable.Seqs. However, calling it "in place" is a bit misleading. While it doesn't allocate a new collection of its own type, it does allocate an array of its size (also, j.u.Arrays.sort may consume significant memory).

Assumption: we do not intend to reimplement sorting, but rather rely on its implementation in Java.
Then: sorting in place (in my understading = without any allocations) is possible only for mutable collecttions which are backed by scala.Array, i.e.

mutable.ArrayBuffer
ArrayDeque
ArraySeq

In all 3 cases, scala.Array is internalized in the respecitve classes.
How do we want to proceed?

I think providing it for array backed classes is a reasonable start. It's better to decide policy on other classes later, than provide them and regret it.
OT, if "in place" is "no allocations" I do wonder about mapInPlace for ListBuffer

java.util.Arrays.sort is not in-place:

Temporary storage requirements vary from a small constant for nearly sorted input arrays to n/2 object references for randomly ordered input arrays.

I think sort (similar to Java's List.sort) would be a better name, but I'm concerned about confusion between it and sorted.

Also, I think it would make sense to have it on mutable.Seq as well: we could use a merge sort on LinearSeq.

@julienrf Please take a look.


dare.py link does not work with FileImport

Fasqts can also be generated manually and imported into pipeline. The workflow in this case is FileImport but there is no workflow run id in the file path. dare.py selects the most recent workflow run based on the run id.
running the following command does not link files as expected:
python3.6 ~/dare.py link -l ~/CPTRT_datarelease_210813_A00469_0197_BH7TK5DSXY.txt -w FileImport -n CPTP -pr CPTP -r 210813_A00469_0197_BH7TK5DSXY -s fastqs

Actually the file path for the example mentioned above includes FileImport but the corresponding workflow is FileImportForAnalysis. Modified how the latest workflow run is selected by allowing worklow run to be missing from the file path. Then the function just selects the file recorded in FPR according to the parameters in the command. It assumes a single version of the same file in FPR since no worklow run id is present.


Rest API generates ANY endpoint rather than the POST - amplify api add/update

Amplify 4.19.0
After creating, it gives me ANY in API Gateway
So I did an amplify API update

But still, after amplify push there is no POST, but any

Anything I am missing here?
Does it support to create a POST endpoint?
Or I need to filter it by myself in my lambda function?
==============
Also, amplify status does not return the REST endpoint, only the graphql url

Many thanks for sharing the workaround @ykethan 👍  Looking forward to seeing a new feature being added so that we could choose any method for our api endpoints


Time Series Forecasting

Goal(s)

Predict future values based on previously observed values

Input(s)

Number

Output(s)

Number

Objective Function(s)

TBD

Initial work by @Doctor-Fu: #46

See tasks/time-series/time-series-forecasting


Pagination displaying incorrect number of rows

When pagination is enabled on a table, it displays "number + 1" in the "Showing X rows" text. See screenshot for a better explanation.

It looks like the fix should be as simple as removing the + 1 from here:
https://github.com/react-bootstrap-table/react-bootstrap-table2/blob/master/packages/react-bootstrap-table2-paginator/src/pagination-total.js#L6

I will check this out , thanks


Can't import a photo

Start a new obs by importing a photo
Choose one photo and tap "Upload 1 photo"
on Group Photos, tap "Next" to get the crash

Pretty sure the problem is https://github.com/inaturalist/iNaturalistReactNative/blob/main/src/models/Observation.js#L71, which needs to include a reference to an open realm when constructing a new ObservationPhoto... or we could make realm available everywhere with a provider and a hook as I'm suggesting in #124.

Closed in https://github.com/inaturalist/iNaturalistReactNative/pull/119


RSDK-2860 - Fix cartographer missing dependencies

Ticket: https://viam.atlassian.net/browse/RSDK-2860
Problem:

brew install cartographer-module failed to build on an arm64 Mac.

Solution:

I ran brew remove --force $(brew list) to uninstall all libraries
I added missing dependencies until the brew package was installed successfully.

Tested on:

Mac x86_64

Question: Should/Could someone with an M1 Mac repeat the steps above and test if the installation works seamlessly?


Bad libel in tab stats PID

The tab PID in stats page as a column name "Database" but it must be "PID".

Done! (commit on UI repository)


Rails5 deprecation fixes

This pull request fixes a lot of deprecation warnings generated by Rails 5.x.

1

Tried to rebase this one, but afterwards the tests were failing.

Not sure if that rebase went how it was supposed to but all the changes are there and everything is passing now.

:+1:


Support dates only starting from 1AC

Fixes https://github.com/pydantic/pydantic/issues/10967
Python has datetime.MINYEAR = 1 source. It is not possible to create datetime.date or datetime.datetime that has year smaller than 1.
This is the first time I have even looked at Rust code, thus there might be a lot of mistakes.
Thank you @changhc for providing an changelog example in #77

Thanks for the PR! Agreed with @changhc, I think given 0000 is accepted in RFC 3339 / ISO 8601 we should instead introduce a check in pydantic-core when converting to Python datetimes.

RFC 3339 / ISO 8601 is quite clear that the year should be between 0000 and 9999. Thus, I will close this PR.


Splash art

Create splash art for the game

Shanil added splash art and main menu for the game, this is done as of 211da999f6aaabf1b83279a6c98ad7cf2aa20eed


Ksql-server and StatefulSet

Would it be useful to provide a helm chart for ksql-server that use statefulset?

I think it makes sense. @HungUnicorn do you want to take a stab on this PR?

cool, I would like to

Just curious. Is the desire to make statefulset coming from the fact that it takes a longer time to regenerate the local state store in the event of failure?

@jinoobaek-qz check Gwen and Matthias talk from Kafka Summit. They were talking about Kstreams but also applicable to ksql https://www.confluent.io/kafka-summit-sf18/deploying-kafka-streams-applications.
They covered both stateful and stateless deployments.

Very helpful. Thanks!

@HungUnicorn, @jinoobaek-qz would either of you able to work on this?

Unfortunately, I do not have bandwidth to work on this at the moment.

@gAmUssA I will send a PR. Basically will use the same approach as kafka's statefulset, that the user can decide by persistence.enable


Playback and even app freezes on Samsung Tizen TV (Tizen 4.0, 2018)

Have you read the FAQ and checked for duplicate open issues?
Yes.
What version of Shaka Player are you using?
Latest (v2.5.5)
Can you reproduce the issue with our latest release version?
Yes.
Can you reproduce the issue with the latest code from master?
Yes.
Are you using the demo app or your own custom app?
Both.
If custom app, can you reproduce the issue using our demo app?
Yes.
What browser and OS are you using?
Tizen TV default browser.
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
MN : UE32N5302
PD : 04/18/2019
SN : 0EH33HAM402498W
FW : T-KTSNDEUC-1270.6
FC : SWU-OU_T-KTSNDEUC_1270_190717
MI : T-KTSNDEUC
LS : EU
DI : U7CD36PSCYHKM
MA : 24FCE5D8E1CF
VS : 1270.190717
Samsung TV TizenBrowser: <IP_ADDRESS>1293
What are the manifest and license server URIs?

Manifest: https://vst99-2.o2tv.cz/53480245581472403-samsung_tv.mpd
License: none
What did you do?

Open Internet Browser in Samsung TV
Go to https://shaka-player-demo.appspot.com/demo
Click CUSTOM CONTENT
Click plus button
Type manifest URL (see above)
Type some name
Click SAVE
Click PLAY

What did you expect to happen?
The player should play till the end of the stream.
What actually happened?

The player playback starts freezing after a while (e.g. 15 or 30 minutes), it completely stops playing after some more time (few minutes) and after some more time (e.g. 10 minutes) the whole app (Internet Browser) is completely frozen.

This is possibly related to https://github.com/google/shaka-player/issues/2059 and https://github.com/videojs/http-streaming/issues/674

Marking as duplicate of #2059.


Add UII compatible pagination to filterlib

Currently filter_lib supports pagination by request:
pagination: {
  page_num: ...,
  page_size: ...,
}

And response:
pagination: {
    min_pages_left: ...,
    total: ...,
    has_more: ...
}

We need to add additional supported format for UII components, which is required from, count and has_more
We could add additional filed to pagination request, like
pagination: {
  page_offset: ...,
  page_size: ...
}

make all fields optional, but requires page_num or page_offset

Agreed 'pagination' attribute in response for the new format is:
pagination: {
offset: ...,
limit: ...
has_more: ...
}

@nanbratan
Do we really need to change response? Is it possible to use has_more flag to continue loading on scroll?

"Search" endpoints checked:
Annotation:

[ ] [/tasks/search] - Fails with "detail": "Internal server error. Failed request to 'users' microservice: {"detail":"Not Found"}"
[x] [/jobs/{job_id}/categories/search]

https://github.com/epam/badgerdoc/pull/451


Can't symbolise symbols in graphviz macOS dylib (instruments.app can though)

https://share.firefox.dev/3Ck2ouy
Generated with:
~/projects/samply/samply/target/release/samply record --save-only --rate 100 dot ~/Downloads/dot-causes-abend.dot

I see only offsets like 0x0400 rather than function names. I'd expect the function names.
Instruments.app seems to be able to symbolize the same items:

Not sure if this is expected to work? Happy to collect more data. I've compiled graphviz with -D CMAKE_BUILD_TYPE=Debug which seems to put -g in the right spot. And I think the symbols are there because Instruments.app can see them.

If Instruments can get symbols, then samply should be able to get symbols too, i.e. this is definitely a valid bug.

Here's the libgvc.6.0.0.dylib file if that helps: https://drive.google.com/file/d/1ovCp47hPdllwvxzfIgahN4-GUrell0ah/view?usp=sharing

It'll definitely help, but it looks like your profile was captured on homebrew graphviz, not on the locally compiled graphviz. If you execute profile.libs on the web console in the profiler tab, you see paths such as /opt/homebrew/Cellar/graphviz/7.0.4/lib/libgvc.6.dylib.

Oh! Thank you. I will run with a full PATH and not assume I'm looking at /usr/local/bin/dot

Here's one I actually compiled and installed with sudo make install: https://share.firefox.dev/3vwK5OT, not the homebrew one. And instruments.app is still able to see:

Ok, not sure yet what's going wrong. The path to libgvc in the profile is /usr/local/lib/libgvc.6.0.0.dylib, is that the correct path?
I downloaded the profile, replaced both occurrences of /usr/local/lib/libgvc.6.0.0.dylib with /Users/mstange/Downloads/libgvc.6.0.0.dylib, ran samply load /Users/mstange/Downloads/dot\ 2023-01-03\ 05.38\ profile.json, and clicked Profile Info -> Re-symbolicate profile, and it symbolicated correctly: https://share.firefox.dev/3QpdHYl
I'll try to reproduce it locally.

/usr/local/lib/libgvc.6.0.0.dylib is the correct path.
I think this should reproduce with the homebrew graphviz using this command:
$ samply record --save-only --rate 100 /usr/local/bin/dot ~/Downloads/dot-causes-abend.dot

With the file from https://gitlab.com/graphviz/graphviz/uploads/b9c23983831e90a4ed20d9e94d4ce196/dot-causes-abend.dot

Oh, you're using --save-only - how are you loading the generated file? If you just drag it into profiler.firefox.com, no symbolication is expected. You need to use samply load profile.json.

Huzzah! Success:

I didn't realise they were different. I didn't realise symbolication was a separate step after-the-fact

Great! This is definitely an aspect that needs to be called out in the (eventual) documentation.

For the record, symbolication happens after-the-fact because it can take a long time (up to minutes), especially if symbol files are downloaded from a symbol server. I want the user to have an interactive UI during that wait time.

Sounds good! I see (I think, looking at the source) that e.g. Windows
symbols are downloaded from over the net, seems reasonable to have these be
two steps. Thanks for explaining
On Tue, 3 Jan 2023 at 22:15, Markus Stange @.***> wrote:

For the record, symbolication happens after-the-fact because it can take a
long time (up to minutes), especially if symbol files are downloaded from a
symbol server. I want the user to have an interactive UI during that wait
time.
—
Reply to this email directly, view it on GitHub
https://github.com/mstange/samply/issues/13#issuecomment-1369649836, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAZYOPUPJSQDI2VPXI4VEDWQQC55ANCNFSM6AAAAAATPJXUGQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Fix typos

What does this implement/fix? Explain your changes.
Fix typos in options.json.
Does this close any currently open issues?
No.
Any other comments?
No.
Checklist
Check all those that are applicable and complete.

[ ] Merged with latest master branch
[ ] Regenerate documentation with npm run docs
[ ] Add change details to CHANGELOG.md under "Next" section
[ ] Added examples for testing to examples/ directory
[ ] Travis CI passes (Mac support)
[ ] AppVeyor passes (Windows support)

Thanks for noticing and fixing these!
The src/options.json is actually a generated file which is committed because users do not need to regenerate it on install (was causing issues and users were reporting failed installations).
You will actually want to change the applicable files in https://github.com/Glavin001/atom-beautify/tree/master/src/languages
And then run npm run docs to regenerate the docs. Commit those changes.
Thanks again for contributing!

@Glavin001 I just found some typos in a couple of nginx.coffee and python.coffee in src/beautifiers. Can I directly change them or is there somewhere else I should look at?

Do you mean in src/languages? https://github.com/Glavin001/atom-beautify/blob/master/src/languages/nginx.coffee and https://github.com/Glavin001/atom-beautify/blob/master/src/languages/python.coffee ? You can make changes there, as I think you have already done with previous commits, and make sure to run npm run docs when you're finished.

My bad, I wasn't very clear about which files I was talking about.
In https://github.com/Glavin001/atom-beautify/blob/master/src/languages/nginx.coffee#L37 in src/languages/nginx.coffee, I believe dontJoinCurlyBracet is misspelled and should be dontJoinCurlyBrackets (I also pluralized it). This would affect https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/nginx-beautify.coffee#L22 in src/beautifiers/nginx-beautify.coffee. Likewise, for Python, formater should be changed to formatter and formaters to formatters in https://github.com/Glavin001/atom-beautify/blob/master/src/languages/python.coffee#L37 and  wherever appropriate in https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/pybeautifier.coffee.
For https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/nginx-beautify.coffee#L22 and https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/pybeautifier.coffee, I am guessing I can just change them directly?

Oh nooooooooo. These are actual option names meaning if they are changed and users have configured them it will suddenly and mysteriously break. Yup, I definitely need to add spell check to Pull Requests somehow since it appears I am missing all of these. This is very unfortunate.

Haha, alright then, there are no other typos in the language files as far as I can tell. Go ahead and merge whenever you are free.

@edwinksl could you instead update the title and description property values to ensure they are at least spelt correctly? Such as https://github.com/Glavin001/atom-beautify/blob/master/src/languages/python.coffee#L41

@Glavin001 Sounds good, done!

Thank you very much for contributing @edwinksl! Hopefully we can keep these typos to a minimum from here on out.

Published to v0.30.0


Eorror with type cast?

chat_gpt_sdk: ^2.1.7
[OpenAI] request body :{prompt: hello, model: text-davinci-003, temperature: 0.3, max_tokens: 200, top_p: 1.0, frequency_penalty: 0.0, presence_penalty: 0.0, stop: null}
[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: type 'List<dynamic>' is not a subtype of type 'List<Map<String, dynamic>>' in type cast
#0      new CompleteResponse.fromJson (package:chat_gpt_sdk/src/model/complete_text/response/complete_response.dart:30:26)
#1      OpenAI.onCompletion.<anonymous closure> (package:chat_gpt_sdk/src/openai.dart:119:35)
#2      OpenAIClient.post (package:chat_gpt_sdk/src/client/openai_client.dart:189:25)
<asynchronous suspension>
[OpenAI] ============= success ==================

@llorenzo thank you for issues bug

@llorenzo you using method ?

I think it was this.
openAI.onCompletion(request: request);

@llorenzo Now I have fixed the bug, so you can use the "bil" feature in the new version.

Ah that is kind. What is the "bil" feature?

@llorenzo I'm not sure what you mean.


[Suggestion] Shields use RF to stop explosions

Could you make the shield projector have a config option so that explosions cause an energy consumption spike? possibly having energy consumption scale with explosive power? whether the energy usage calculation is done on a per shield block required resistance basis or some other method

That's good idea. Will be for later though


AMQP-838: Deprecate container.setMessageConverter

JIRA: https://jira.spring.io/browse/AMQP-838
The setMessageConverter() on the listener container was only used
to convey the converter from the container factory to the listener
adapter. This is confusing to users hand-wiring a container since
there is an implication it is used by the container itself.
Deprecate the method, and use the endpoint (by default) to convey
the converter to the endpoint's adapter.
cherry-pick to 2.0.x

... and cherry-picked to 2.0.x


Added Emacs F# mode

Hi,
I have added link to Emacs F# mode. In my opinion it is quite awesome :)

Thank you!


Update Get-HostedContentFilterPolicy.md

Issue 9377 in the M365 repo.

Docs Build status updates of commit 6e35e90:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-HostedContentFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Release 1.3.0?

Just a quick question: any thoughts on the timeline for release 1.3.0?

I would like to make it somewhere this week.

@kenwdelong Feel free to test 1.3.0-SNAPSHOT in the mean time. I do not expect any changes to the API but it would be easier to fix bugs before the release.

That was my plan for today!!  :-)

I've integrated the SNAPSHOT build, and so far so good.  We'll do some more extensive testing tomorrow.
BTW, why did you comment out setReportDelay() etc in ScanSettingsBuilder?

Because that is not yet implemented in a backwards-compatible way. This is a place for pull requests from the community if anyone will need this functionality — from my experience implemented features are the most used ones.

After a day of testing, so far everything is working fine.

Good to hear that. I plan to make an official release today.

Released.

Excellent, thanks!


Travis - Update init script to use HTTPS

Update Travis initialization script to download FFMPEG library via HTTPS.

Coverage remained the same at 82.251% when pulling 919b29008094622df7e06616c7cff6d9a435dc2e on stonio:patch-6 into be6fbe834fdefa1d807da1a47f8578647094b760 on bramp:master.

Thanks!


Draco compression tries to use the fetch api to load the wasm blob off the filesystem and fails with ERR_INVALID_URL

Full error:
(node:4270) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
TypeError: Failed to parse URL from /home/ashley/node_modules/draco3d/draco_encoder.wasm
    at new Request (node:internal/deps/undici/undici:4816:19)
    at Agent2.fetch2 (node:internal/deps/undici/undici:5544:29)
    ... 10 lines matching cause stack trace ...
    at Object.tryCatcher (/home/ashley/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/home/ashley/node_modules/bluebird/js/release/promise.js:547:31)
    at Promise._settlePromise (/home/ashley/node_modules/bluebird/js/release/promise.js:604:18)
    at Promise._settlePromise0 (/home/ashley/node_modules/bluebird/js/release/promise.js:649:10) {
  [cause]: TypeError [ERR_INVALID_URL]: Invalid URL
      at new NodeError (node:internal/errors:377:5)
      at URL.onParseError (node:internal/url:563:9)
      at new URL (node:internal/url:643:5)
      at new Request (node:internal/deps/undici/undici:4814:25)
      at Agent2.fetch2 (node:internal/deps/undici/undici:5544:29)
      at Object.fetch (node:internal/deps/undici/undici:6372:20)
      at fetch (node:internal/bootstrap/pre_execution:199:25)
      at /home/ashley/node_modules/draco3d/draco_encoder_nodejs.js:36:498
      at /home/ashley/node_modules/draco3d/draco_encoder_nodejs.js:37:214
      at Object.createEncoderModule (/home/ashley/node_modules/draco3d/draco_encoder_nodejs.js:37:237)
      at compressDracoMeshes (/home/ashley/node_modules/gltf-pipeline/lib/compressDracoMeshes.js:54:52)
      at /home/ashley/node_modules/gltf-pipeline/lib/processGltf.js:67:12
      at tryCatcher (/home/ashley/node_modules/bluebird/js/release/util.js:16:23)
      at Object.gotValue (/home/ashley/node_modules/bluebird/js/release/reduce.js:166:18)
      at Object.gotAccum (/home/ashley/node_modules/bluebird/js/release/reduce.js:155:25)
      at Object.tryCatcher (/home/ashley/node_modules/bluebird/js/release/util.js:16:23) {
    input: '/home/ashley/node_modules/draco3d/draco_encoder.wasm',
    code: 'ERR_INVALID_URL'
  }
}

Reproducible:
In debugg.js script:
const draco3d = require("draco3d");
let decoderModulePromise = Promise.resolve(draco3d.createDecoderModule({}));
console.log({decoderModulePromise})

 ~ node /Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/lib/debugg.js
{ decoderModulePromise: Promise { <pending> } }
(node:81439) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
node:internal/deps/undici/undici:4816
            throw new TypeError("Failed to parse URL from " + input, { cause: err });
                  ^

TypeError: Failed to parse URL from /Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/node_modules/draco3d/draco_decoder.wasm
    at new Request (node:internal/deps/undici/undici:4816:19)
    at Agent2.fetch2 (node:internal/deps/undici/undici:5544:29)
    ... 4 lines matching cause stack trace ...
    at Object.createDecoderModule (/Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/node_modules/draco3d/draco_decoder_nodejs.js:39:247)
    at Object.<anonymous> (/Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/lib/debugg.js:4:52)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1159:10) {
  [cause]: TypeError [ERR_INVALID_URL]: Invalid URL
      at new NodeError (node:internal/errors:377:5)
      at URL.onParseError (node:internal/url:563:9)
      at new URL (node:internal/url:643:5)
      at new Request (node:internal/deps/undici/undici:4814:25)
      at Agent2.fetch2 (node:internal/deps/undici/undici:5544:29)
      at Object.fetch (node:internal/deps/undici/undici:6372:20)
      at fetch (node:internal/bootstrap/pre_execution:199:25)
      at /Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/node_modules/draco3d/draco_decoder_nodejs.js:39:1
      at /Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/node_modules/draco3d/draco_decoder_nodejs.js:39:224
      at Object.createDecoderModule (/Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/node_modules/draco3d/draco_decoder_nodejs.js:39:247) {
    input: '/Users/danvas/.nvm/versions/node/v18.1.0/lib/node_modules/gltf-pipeline/node_modules/draco3d/draco_decoder.wasm',
    code: 'ERR_INVALID_URL'
  }
}

Node.js v18.1.0

@expenses - Were you able to find a quick workaround? (For example, hardcode the path)

No, I didn't look very hard into this. An easy workaround seems like it should be possible though.

Seems to only happen in Node 18. Node 16 works fine. I tested draco 1.5.2 for both.
Assuming this problem isn't caused by gltf-pipeline in some way, could you submit to bug report to the draco repo?

Seems to only happen in Node 18. Node 16 works fine. I tested draco 1.5.2 for both.
Assuming this problem isn't caused by gltf-pipeline in some way, could you submit to bug report to the draco repo?

Done: https://github.com/google/draco/issues/888


publicIp resourceId needs to know if none is allowed

on the nic resource that consumes the publicIP, does it allow for the none scenario
createUIdef, has a publicIp address control...  this control allows for the use of new/existing/none.   The resource for that publicIp in mainTempate will have a condition property that allows for new/existing/none.
Any resourceId that references that publicIp resource, must have a condition surrounding it that accommodates for the "none" option.  This is usually don't by wraping the resourceId in an if() statement and one of the clauses is json('null').
Also need to account for the publicIpAddress property is an object variable
Also need to account for nested deployments that handle the conditions.

@bmoore-msft  triaged, understood.  Good candidate for next semester.


feat: #1022 - add onboarding hint that the cards can be tapped

What

Add one time onboarding hint that the cards can be tapped, more details in : https://github.com/openfoodfacts/smooth-app/issues/1022#issuecomment-1078813035

Screenshot

Normal
Dark

Fixes bug(s)

#1022

Part of

#561

Kinda new but I think these changes would be good to go and also the dark mode card looks kinda not so visibly readable

I agree with you, the background color is ThemeData.hintColor which changed according to theme selected, do you have an idea about the color? I feel lost in so many colors 😅

The colors as such looks good, for me it's just the opacity. Maybe there is a way to just change this

The colors as such looks good, for me it's just the opacity. Maybe there is a way to just change this

Yes, I updated the PR by increasing the opacity of hintColor to 0.9:

Normal
Dark

It looks real good to me now 🥳


fix "[Error: ENOENT: no such file or directory, open 'helpers/google/protobuf/descriptor.proto']

to solve this issue we need to have access to google/protobuf/descriptor.proto on the cli side.
Given that this file is packaged with most of the "proto" tools (here protoc, on transformer side, and protobufjs, on cli side)
I see 2 main ways:

understand and fix the dependencies in massa-proto-cli so the descriptor.proto bundled with protobufjs comes with the cli. But at this point I don't have any clue on how to do this (even if protobufjs is used by the cli it doesn't appear in node_modules)
on transformer side, add it to the list of .proto that have to be deployed with the SC so it is fetch with the .proto generated by the transformer.

So far, the second solution looks easier to me.
In the first place I see

one main drawback: descriptor.proto will get duplicated in the storage
one advantage: the descriptor.proto from protoc and protobufjs I have locally differ significantly, at some point, with the first configuration one could face some incompatibilities which is not possible with the second one.

I'm going forward with the second solution but wish to ear form you @Ben-Rey @Elli610 @0xSwapFeeder @Eitu33 @damip

to solve this issue we need to have access to google/protobuf/descriptor.proto on the cli side.
Given that this file is packaged with most of the "proto" tools (here protoc, on transformer side, and protobufjs, on cli side) I see 2 main ways:

understand and fix the dependencies in massa-proto-cli so the descriptor.proto bundled with protobufjs comes with the cli. But at this point I don't have any clue on how to do this (even if protobufjs is used by the cli it doesn't appear in node_modules)
on transformer side, add it to the list of .proto that have to be deployed with the SC so it is fetch with the .proto generated by the transformer.

So far, the second solution looks easier to me. In the first place I see

one main drawback: descriptor.proto will get duplicated in the storage
one advantage: the descriptor.proto from protoc and protobufjs I have locally differ significantly, at some point, with the first configuration one could face some incompatibilities which is not possible with the second one.

I'm going forward with the second solution but wish to ear form you @Ben-Rey @Elli610 @0xSwapFeeder @Eitu33 @damip

Why don't we add the descriptor in the cli and when it detects a custom type, the descriptor will automatically be added in the output folder

Wip in #329


[armel][cross] Added proper command line arguments to documentation.

This infomation wasn't exist in documentation, but now we can find which command line argument we should use for running RuyJIT.
@dotnet/arm32-corert-contrib PTAL

Thanks

@jkotas is there any way to dump generated code from ILC? It looks options JitDisasm=* and NgenDisasm=* don't work when I use it with --codegenopt.

NgenDisasm is the option you're looking for. We invoke RyuJIT for NGen. You need a Debug build of RyuJIT for this to work.
But I'm not sure you're using the --codegenopt argument right (I saw it in this pull request, but I forgot to comment on it because I got distracted). You need to use --codegenopt for each individual option passed to RyuJIT. So instead of:
--codegenopt "AltJitNgen=* AltJit=*"

Use
--codegenopt AltJitNgen=* --codegenopt AltJit=*

You should not need --codegenopt AltJit=* option at all.

@jkotas @MichalStrehovsky  thank you!
It looks there were some local changes which breaks logic. So for running it's enough:
./corerun ilc.dll --codegenopt AltJitNgen=* --verbose @Hello.ilc.rsp
For passing additional knobs it's neccesary to use --codegenopt anytime. For example (NgenDisasm works)
./corerun ilc.dll --codegenopt AltJitNgen=* --codegenopt NgenDisasm=* --verbose @Hello.ilc.rsp
I will add it to documntation.


bug: fix import in spark3 visitor

Import in Spark 3 visitor factory references class in Spark 2
Signed-off-by: Maciej Obuchowski<EMAIL_ADDRESS>Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)

Codecov Report

Merging #355 (98b10ce) into main (b210566) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #355   +/-   ##
=======================================
  Coverage   84.05%   84.05%
=======================================
  Files          33       33
  Lines        1951     1951
=======================================
  Hits         1640     1640
  Misses        311      311

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 b210566...98b10ce. Read the comment docs.


Add this awesome list to https://github.com/sindresorhus/awesome#readme

I think we should add this repo to https://github.com/sindresorhus/awesome#readme. I would add it under #Platforms.
Which description should we use? Something like this?

ArcGIS Platform - Platform as a Service (PaaS) for mapping applications that contain ready-to-use location services, data hosting, and content management services, but also include tools and APIs to interact and consume them.

cc: @alaframboise @jf990 @ak-kemp

Suggestion:

ArcGIS Platform - Esri's Platform as a Service (PaaS) for mapping and location analysis applications with ready-to-use location services, data hosting, APIs, tools, and content management services.

@hhkaos regarding the URL change, I didn't understand the purpose, so we should change it to this repo.


Added Blue Score the respective  folders

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

What does this PR do?
Fixes #352 .
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@SkafteNicki
Yeah, I will do the same.

I have refactored the code such that the Class implementation uses the functional implementation of the BLEU score. I have moved the tests for the functional implementation to the respective directory.
Regarding the tests for the BLEUScore class, I have doubts about using the existing MetricTester because it accepts preds and targets as torch.Tensor type but the inputs for the BLEU Score metric are an iterable of the strings.
One hack I thought we could use is to provide the iterable of strings as **mertic_args and leave preds and targets as None - but this failed all the tests.
Other option I see is to implement a separate TextMetricTester for all Text based metrics. I am not too sure about this methods.
How do I proceed with the tests for Class based Text Metrics?

I have pushed the updated changes.

@Borda the tests are failing on Github only because it says list type is not hashable. They run fine on my system.

@Borda the tests are failing on Github only because it says list type is not hashable. They run fine on my system.

and what is your system? I assume list shall not be hashable anywhere as it immutable
https://stackoverflow.com/questions/42203673/in-python-why-is-a-tuple-hashable-but-not-a-list

Yeah, it was my mistake. I was running python -m pytest -v tests but I had to run python -m pytest -v torchmetrics tests.
I have fixed the failing tests now. I had forgot to update examples in the docstring and they were failing.

I have fixed the failing tests now. I had forgot to update examples in the docstring and they were failing.

cool, mind commit your fix? :rabbit:


List in Card gets unexpected bottom margin if ListItemText's primary has spaces

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
If a List is within a Card, when ListItemText's primary has spaces in its text, the Card gets unexpected bottom margin. (only happens if it's within column directional Grid)
Expected Behavior 🤔
The Card should not get the unexpected bottom margin.
Steps to Reproduce 🕹

Steps:

have a Grid
set the Grid's direction as column
have a Card in the Grid
have a List in the Card
have a ListItemText's primary in the List
have spaces in the primary text

I have seen this a couple of time in the past. Something is wrong with how the flexbox measure space. You can set noWrap on the Typography to avoid the issue.


Uncaught ReferenceError: No locale data has been provided for this object yet.

# Check this file (and the other files in this directory) into your repository.
#
# 'meteor add' and 'meteor remove' will edit this file for you,
# but you can also edit it by hand.

meteor-base             # Packages every Meteor app needs to have
mobile-experience       # Packages for a great mobile UX
mongo                   # The database Meteor supports right now
blaze-html-templates    # Compile .html files into Meteor Blaze views
session                 # Client-side reactive dictionary for your app
jquery                  # Helpful client-side library
tracker                 # Meteor's client-side reactive programming library

standard-minifiers      # JS/CSS minifiers run for production mode
es5-shim                # ECMAScript 5 compatibility for older browsers.
ecmascript              # Enable ECMAScript2015+ syntax in app code

react
kadira:flow-router
kadira:react-layout
zimme:active-route
izzilab:material-ui
ixdi:material-design-iconic-font
fortawesome:fontawesome
accounts-password
fourseven:scss
reywood:bootstrap3-sass

#TEST:START The packages below are required for testing only
#xolvio:cucumber

#TEST:END
omega:react-intl

It is more an error from the ReactIntl package than from this wrapper.
Without seeing the code is hard to tell, could you prepare a repro project?
You need to have at least this:
language = {
    "locales": ["en-US"],
    "messages": {
        "logIn": "Log in"
    }
};

let IntlMixin = ReactIntl.IntlMixin;

LoginPage = React.createClass({
    mixins: [
        IntlMixin
    ],
    render() {
            return <div>{this.getIntlMessage('logIn')}</div>;
    }
});

and invoked with sth like that:
<LoginPage {...language}/>

Was that of any help to you?

@henyana do you still have this problem?

Closing, @henyana if you will have any problems, please open a new issue.

@wojtkowiak , I'm really appreciate your answer and sorry I've just very busy and not replied your message. Can you tell me where I can add your line of codes on my project?
language = {
    "locales": ["en-US"],
    "messages": {
        "logIn": "Log in"
    }
};

Anywhere you want, just ensure that language will be available where you want to use it.
Now 2.0-pr3 of react-intl is available so if you did not started to use it, I'd recommend on using the new version.


Capture group

Hi,
I am looking for a C source code for regular expression that supports finding matchings substrings for capture groups. I seems that tiny-regex-c doesn't support it... I wonder if you happen to know any other C libraries that support this feature. Thanks for your help.

Hi @gogo9th and thanks for checking out this library.
I don't know of other small libraries that support capture-groups, but I'm fairly certain that the PCRE library supports it.
https://en.wikipedia.org/wiki/Perl_Compatible_Regular_Expressions
https://www.pcre.org/
Otherwise, I recommend that you check out the forks of this project, as some of them really have extended the functionality. There may be someone out there who have finished capture groups.


Add Slack notification to Github workflows

Alert on failures for automatically triggered workflows. For manually
triggered workflows we rely on GitHub emails to the individual that
triggered the workflow.
See the "Workflow builder" approach in
https://github.com/slackapi/slack-github-action/.

Terraform Format and Style 🖌success
Terraform Initialization ⚙success
Terraform Validation 🤖success
Validation Output

Success! The configuration is valid.

Terraform Plan 📖success
Show Plan

undefined

Pusher: @markgoddard, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

Added some docs, resolved merge conflicts, ready for review.