Time zone configuration is not used for insert causing dates to be inserted improperly

Environment
Knex version: 0.16.3
Database + version: mysql 5.7
OS: MacOS 15.1
Bug

Explain what kind of behaviour you are getting and how you think it should do

When inserting into the database, knex ignores the timezone configuration and dates/times are always assumed to be local.

Error message

N/A

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to MySQL or PostgreSQL, then single file example which initializes
needed data and demonstrates the problem.

var knex = require('knex')({
        client: "mysql",
        connection: {
            ...,
            timezone: "UTC"
        },
);

const date = new Date(2019, 11, 10, 10)

await knex('table').insert([
    { date, name: "Pete" },
    { date, name: "Jeff" },
]);

If this code runs on a computer that is not in the UTC timezone, the inserted time is incorrect. The date string that should be in the insert query should be '2019-12-10 18:00:00' for a computer that is in the -08:00 offset, however, knex instead inserts the date as '2019-12-10 10:00:00'.
I took a quick look through the source and it appears that in most/all? of the toQuery/toSQL calls (which do take a timezone as a parameter), the timezone from the configuration is not passed.

Your problem is that when you create new Date(...) it is a local time.
// it is 09:53:24 AM here right now
> new Date()
2019-12-11T07:53:24.898Z
> a = new Date()
2019-12-11T07:53:27.323Z
> a.getTimezoneOffset()
-120
>

So it doesn't matter what timezone you configure to your database driver. It affects only how data is returned from database and (maybe) how timestamps without timezone are inserted. Date object always has timezone info so by inserting new Date(2019, 11, 10, 10) you are telling DB that it should store '2019-11-10T10:00:00-08:00'. Generally Date objects are horrible to work with, you will be better off using for example Luxon for those.

Thanks for the quick reply, but I still think it's a problem. I am creating a date intentionally in local time as an example.
The knex implementation of dateToString (in the lib/query/strings.js file) which seems to be based off of the sql string library adds a line const timeZone = ctx.timeZone || 'local';. The problem is, that the timeZone is never passed down here so dates are always assumed to be local. This means that if I create a date in my local time zone (such as new Date()), it will get the hour, day, minute, for my local time. However, if I want the date to be stored in UTC, it should respect that. As the code is written there is no path to the else clause of the dateToString method.
function dateToString(date, finalEscape, ctx) {
  const timeZone = ctx.timeZone || 'local';

if (timeZone === 'local') {
    year = dt.getFullYear();
    month = dt.getMonth() + 1;
    day = dt.getDate();
    hour = dt.getHours();
    minute = dt.getMinutes();
    second = dt.getSeconds();
    millisecond = dt.getMilliseconds();
  } else {
    const tz = convertTimezone(timeZone);

    if (tz !== false && tz !== 0) {
      dt.setTime(dt.getTime() + tz * 60000);
    }

    year = dt.getUTCFullYear();
    month = dt.getUTCMonth() + 1;
    day = dt.getUTCDate();
    hour = dt.getUTCHours();
    minute = dt.getUTCMinutes();
    second = dt.getUTCSeconds();
    millisecond = dt.getUTCMilliseconds();
  }

Could you add complete (runnable single file) reproduction code to show your problem exactly. Code shown in original post does work as it should. So you should add also the code for example schema creation to see complete picture what are you doing.

Also I thing knex should not do any handling to date objects... it has been confusing for years that everything else is passed directly to the driver, but dates are handled differently... 😕

Yes, it is quite confusing! But dates are a pain, so I'm not surprised it's a bit messy.
I'll work on a fully executable example, but in the meantime will leave this for consideration.
With both mysql and knex using the exact same config, { connection: {   ..., timezone: "UTC" }, client: "mysql" }
        const knexString = knexInstance('users').insert({
            date: new Date()
        }).toString();

        const connection = mysql.createConnection((config as any).connection);
        connection.connect(() => {
            connection.query('INSERT INTO `users` (`date`) values (?)', [new Date()], (err) => {
                console.log();
            })
        })

knexString is equal to "insert into users (date) values ('2019-12-11 01:25:33.313')" and the query that is run by the direct msyql driver call is "INSERT INTO users (date) values ('2019-12-11 09:25:33.313')". You can see how mysql made the necessary conversion, and knex didn't. It seems to me that knex should mimic mysql's behavior.
Like I said, I'll work on an executable example.

Alright, I've put together some fully working code that reproduces the issue:
        const config = {
            client: 'mysql',
            connection: {
                host: '<IP_ADDRESS>',
                user: 'user',
                password: 'password',
                database: 'test',
                timezone: 'utc'
            }
        }

        const knex = require('knex')(config);

        const knexQuery = knex('users').insert({
            date: new Date()
        }).toString();

        const connection = mysql.createConnection(config.connection);
        connection.connect(() => {
            const query = connection.query('insert into `users` (`date`) values (?)', [new Date()], () => {})
            console.log("MySQL Driver Query", query.sql);
            console.log("Knex Query", knexQuery);
        });

If you run this on a computer not in the UTC timezone, you'll get something like this:
MySQL Driver Query insert into `users` (`date`) values ('2019-12-11 16:25:34.552')
Knex Query insert into `users` (`date`) values ('2019-12-11 08:25:34.499')
The dates being inserted are completely different with node mysql adjusting for the utc offset. If you remove the timezone: utc from the configuration, both dates come out exactly the same.

Thanks! Opening this until I have time to check if that is expected or not.

Looks like mysql driver does modify Date() objects according to timezone configurations 👍 So I would say that there is a bug in knex that it touches Date objects before passing them to the driver.
Here is also runkit which calls .toSQL().toNative() which is more close to the actual query that is ran https://runkit.com/mikaelle/5df12776671025001a63d3e2
Knex messes up Date object to string before passing it to the driver.  @wubzz @kibertoad any objections in changing this behaviour to leave also Date objects untouched? It might be quite significant breaking change for the people who has been using Date objects and had worked around this.

@elhigu I am fine with releasing this as a breaking change in next major version.

Upon further review, knex is not modifying the dates before they go to the DB. It appears that the toString() method is what is incorrectly showing the dates.

@jordansoltman Looks like you are right.

Here is also runkit which calls .toSQL().toNative() which is more close to the actual query that is ran https://runkit.com/mikaelle/5df12776671025001a63d3e2

I checked out again the bindings in that example and yup that date actually is an objects, I was also confused about how runkit printed out those date objects :)


When only one player is allowed, the provided entity selector allows more than one.

For example, if I have a command like /bounty <target> <amount> with the code similar to this:
    @Command("bounty")
    public void onBountyCommand(Player sender, Player target, double amount) {
    }

It still suggests @a, @e, @n, @p, @r and upon for example trying to even type in /bounty @e it sends this error in console:
[17:55:18 ERROR]: Thread Async Tab Complete Thread - #0 failed main thread check: Chunk getEntities call
java.lang.Throwable: null
        at org.spigotmc.AsyncCatcher.catchOp(AsyncCatcher.java:14) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.server.level.ServerLevel.getEntities(ServerLevel.java:2384) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.server.level.ServerLevel.getEntities(ServerLevel.java:1413) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.commands.arguments.selector.EntitySelector.addEntities(EntitySelector.java:174) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.commands.arguments.selector.EntitySelector.findEntities(EntitySelector.java:158) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at org.bukkit.craftbukkit.CraftServer.selectEntities(CraftServer.java:2927) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at org.bukkit.Bukkit.selectEntities(Bukkit.java:2558) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.parameters.PlayerParameterType.fromSelector(PlayerParameterType.java:60) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.parameters.PlayerParameterType.parse(PlayerParameterType.java:81) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.parameters.PlayerParameterType.parse(PlayerParameterType.java:50) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.node.parser.ParameterNodeImpl.parse(ParameterNodeImpl.java:122) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.autocomplete.StandardAutoCompleter.complete(StandardAutoCompleter.java:147) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.autocomplete.StandardAutoCompleter.complete(StandardAutoCompleter.java:100) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.listener.AsyncPaperTabListener.onAsyncTabComplete(AsyncPaperTabListener.java:62) ~[DufjiSMP-1.0.0-REL.jar:?]
        at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor8.execute(Unknown Source) ~[?:?]
        at org.bukkit.plugin.EventExecutor$2.execute(EventExecutor.java:77) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:80) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:1.21.1-89-1ed64f8]
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at io.papermc.paper.plugin.manager.PaperEventManager.callEvent(PaperEventManager.java:54) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.callEvent(PaperPluginManagerImpl.java:131) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:628) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.event.Event.callEvent(Event.java:45) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.handleCustomCommandSuggestions0(ServerGamePacketListenerImpl.java:822) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.lambda$handleCustomCommandSuggestions$1(ServerGamePacketListenerImpl.java:811) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144) ~[?:?]
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642) ~[?:?]
        at java.base/java.lang.Thread.run(Thread.java:1583) ~[?:?]
[17:55:19 ERROR]: Thread Async Tab Complete Thread - #1 failed main thread check: Chunk getEntities call
java.lang.Throwable: null
        at org.spigotmc.AsyncCatcher.catchOp(AsyncCatcher.java:14) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.server.level.ServerLevel.getEntities(ServerLevel.java:2384) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.server.level.ServerLevel.getEntities(ServerLevel.java:1413) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.commands.arguments.selector.EntitySelector.addEntities(EntitySelector.java:174) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.commands.arguments.selector.EntitySelector.findEntities(EntitySelector.java:158) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at org.bukkit.craftbukkit.CraftServer.selectEntities(CraftServer.java:2927) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at org.bukkit.Bukkit.selectEntities(Bukkit.java:2558) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.parameters.PlayerParameterType.fromSelector(PlayerParameterType.java:60) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.parameters.PlayerParameterType.parse(PlayerParameterType.java:81) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.parameters.PlayerParameterType.parse(PlayerParameterType.java:50) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.node.parser.ParameterNodeImpl.parse(ParameterNodeImpl.java:122) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.autocomplete.StandardAutoCompleter.complete(StandardAutoCompleter.java:147) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.autocomplete.StandardAutoCompleter.complete(StandardAutoCompleter.java:100) ~[DufjiSMP-1.0.0-REL.jar:?]
        at DufjiSMP-1.0.0-REL.jar/revxrsal.commands.bukkit.listener.AsyncPaperTabListener.onAsyncTabComplete(AsyncPaperTabListener.java:62) ~[DufjiSMP-1.0.0-REL.jar:?]
        at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor8.execute(Unknown Source) ~[?:?]
        at org.bukkit.plugin.EventExecutor$2.execute(EventExecutor.java:77) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:80) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:1.21.1-89-1ed64f8]
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at io.papermc.paper.plugin.manager.PaperEventManager.callEvent(PaperEventManager.java:54) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.callEvent(PaperPluginManagerImpl.java:131) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:628) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.event.Event.callEvent(Event.java:45) ~[paper-mojangapi-1.21.1-R0.1-SNAPSHOT.jar:?]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.handleCustomCommandSuggestions0(ServerGamePacketListenerImpl.java:822) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.lambda$handleCustomCommandSuggestions$1(ServerGamePacketListenerImpl.java:811) ~[paper-1.21.1.jar:1.21.1-89-1ed64f8]
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144) ~[?:?]
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642) ~[?:?]
        at java.base/java.lang.Thread.run(Thread.java:1583) ~[?:?]

Then when sending the command, it sends out this error:
Only one player is allowed, but the provided selector allows more than one

Any idea on how to resolve this, or is this just a bug with Lamp?

I see. Sounds like a problem with asynchronous tab completion. You could disable it as following:
BukkitLampConfig config = BukkitLampConfig.builder(plugin).disableAsyncCompletion().build();
Lamp<BukkitCommandActor> lamp = BukkitLamp.builder(config).build();

Can you try it and let me know if it persists?

That does resolve the console error, but does not resolve the issue where it's suggesting using @a, @e, @n, @p, @r for a single player selector.

This is Brigadier's default behavior. @a and @e can be minimized using further selectors (i.e. limit=1 or d=2) to reduce to one player, and the client will display the input in red if it does not.
Do you want to remove them altogether? Either disable Brigadier through the BukkitLampConfig, or register a SuggestionProvider for the Player class. That will automatically remove the @a, @e, etc.

This is Brigadier's default behavior. @a and @e can be minimized using further selectors (i.e. limit=1 or d=2) to reduce to one player, and the client will display the input in red if it does not.
Do you want to remove them altogether? Either disable Brigadier through the BukkitLampConfig, or register a SuggestionProvider for the Player class. That will automatically remove the @a, @e, etc.

I'll do that and I'll share the code I make here, I think it's silly to suggest args that the user cannot use, but thanks for the help.

This is Brigadier's default behavior. @a and @e can be minimized using further selectors (i.e. limit=1 or d=2) to reduce to one player, and the client will display the input in red if it does not.
Do you want to remove them altogether? Either disable Brigadier through the BukkitLampConfig, or register a SuggestionProvider for the Player class. That will automatically remove the @a, @e, etc.

I'll do that and I'll share the code I make here, I think it's silly to suggest args that the user cannot use, but thanks for the help.

When you do this for your code, I would recommend adding a parameter that allows enabling/disabling them individually to make it more configurable.

Will close this as it is Brigadier's default behavior to allow this. It will show red and give an error if the selector does not reduce to a single entity/player.


Process Stoped when plot copied at 25%.

Process Stoped when plot copied at 25%....Happened many times

When the last plot copy at about 25G to the final directory , the process will stop , I tried many times , and every time will stop.

The plotting process stopped during the copying phase. The process stopped sharply at 25GB to the final directory. Tried restart the plotting process many times and this issue is still occuring.
I set ramdisk about 112G  and my memory is 128G.  I use pm2 for daemon process.

I use the main branch today.

is the drive full maybe?

内存不足，进程被杀，你可以尝试增加内存；你应该粗略的计算一下你需要多少内存，而不是直接就运行，这样你的进程会因为申请不到足够的内存而被迫中止


StreamAggregatedResources gRPC config stream closed: 13

After upgrading to isto version 1.7.2, isto proxy always reports the warning of stream aggregated resources grpc config stream closed

but The envoy request log is not printed @howardjohn

I am not sure what you mean. This exactly matches the expectations. There are no XDS request logs.


[release-0.58] device manager: externally provided mediated devices should not be removed

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

NONE

/cc @vladikr @xpivarc

@acardace Seesm the cherry-pick did not go well. Can you have a second look?

/retest-required

/retest-required

@vladikr @xpivarc can you take a look at this?
I don't really know why the network tests are failing, on my local cluster they pass just fine.

@EdDev Can you have a look on these failures, please?

/unhold
/retest

@EdDev Can you have a look on these failures, please?

What did I miss? Do you still have links to the failing jobs?

@EdDev Can you have a look on these failures, please?

What did I miss? Do you still have links to the failing jobs?

I had a look on this in the end and it was a infra issue. We will investigate why it is happening.


Reduce requried memory for inputs for MultiZarrToZarr and merge_vars

Hi,
I woudl like users to open the generated kerchunked files without using much memory. I reached the point where I could not load all single jsons anymore into memory for the MultiZarrToZarr function. So I had to create more than one, lets say combined1.parq and combined2.parq. Users then could use a catalog entry which would use code like:
import xarray as xr
xr.open_mfdataset(
        [
            "reference::/combined1.parq",
            "reference::/combined2.parq",
        ],
        engine="zarr",
        backend_kwargs=dict(
            storage_options=dict(
                lazy=True,
                remote_protocol="file",
            ),
            consolidated=False,
        ),
        parallel=True,
        data_vars="minimal",
        coords="minimal",
        compat="override",
    )

however this requires GB of memory.  Idk for what exactly but it is not applicable. Also, it creates another level of potential confusion: Why are thre multiple kerchunks of multiple kerchunks?
So the other option would be to reduce memory usage for merging with MultiZarrToZarr. However, a lot of kerchunk functions require json inputs. E.g., I also use the merge_vars function after the MultiZarrToZarr.
So what are your recommendations? Wouldnt it be nice to have an append kwarg for multizarrtozarr? Or is there already sth for that?
Best,
Fabi

OK I find that if you add this one magic little keyword chunks="auto"to the open_mfdataset command, memory requirement shrinks to a minimum:
xr.open_mfdataset(
        [
            "reference::/combined1.parq",
            "reference::/combined2.parq",
        ],
        engine="zarr",
        backend_kwargs=dict(
            storage_options=dict(
                lazy=True,
                remote_protocol="file",
            ),
            consolidated=False,
        ),
        parallel=True,
        data_vars="minimal",
        coords="minimal",
        compat="override",
        chunks="auto"
    )

that already helps a lot so... we may close it unless you have even better ideas? :)

Wouldnt it be nice to have an append kwarg for multizarrtozarr?

Yes, this is now possible as of #404 , which has not had much testing yet

Final comment here: of course kerchunk can't magically get xarray to do the right thing, and I don't know why it used so much memory in the first place. However, we should be able to combine multiple datasets and save new metadata so that it becomes unnecessary to call open_mfdataset in the general case. We can leave that as aspiration.
Closing this as nothing-to-be-done.


Not working under iOS 8

Hi, Thanks for putting time on this project and sharing it.
For me under iOS 7 it's working fine, but it doesn't sync with iCloud on iOS 8. Any idea on how to fix it?

Nothing has changed in the APIs that I'm aware of and it works in all the projects that I use it in. Try putting a breakpoint on line 126 of SDCloudUserDefaults.m and change something on another device; the breakpoint should be triggered eventually on the other device.

Thanks for your reply and I'll try your suggestion. Maybe the problem has to do with this answer in Stack Overflow: http://stackoverflow.com/questions/26428083/nsubiquitouskeyvaluestore-not-syncing-with-icloud


p5.js editor compatibility problem: using windows

I just downloaded the p5.js editor for windows and everything went smoothly but when I tried running it i got this message: "This app can't run on your pc". I'm using the Windows 8 operating system. What might be causing this to happen? Thanks in advance.

Do you happen to know if you're running windows 32-bit or 64-bit?

Hi,
Runs fine for me on Windows 8.
8.1 Enterprise
x64
i5-3230M
8G RAM
ms
On Fri, Sep 4, 2015 at 12:13 PM, Sam Lavigne<EMAIL_ADDRESS>wrote:

Do you happen to know if you're running windows 32-bit or 64-bit?
—
Reply to this email directly or view it on GitHub
https://github.com/processing/p5.js-editor/issues/143#issuecomment-137779649
.

I'm running windows 32-bit. Is that the problem?

@echo1234 that's probably the problem. I'll make you a 32-bit build tomorrow and hopefully it'll work on your system.

Thanks a lot. I would really appreciate that.

So what's the update on the 32-bit build? It's alright if you didn't have any time to make it.

hi @Echo1234 - try this, and let me know if it works: http://saaaam.s3.amazonaws.com/p5-5.1-win32.zip

Hello @antiboredom , the download seems to stop at 0.1 MB every time i try to download it. I'll reply again when i get it to work. By the way, thank you very much for taking the time to make it.

I just finished downloading it and it worked PERFECTLY! THANK YOU! You've really helped me out and man I can't thank you enough. You've really made my day mate. Thanks again.

works for me too - except for the play button... play button makes it hang up....

Run fine here, including play button.  Windows XP 32 SP3.

Run fine here, including play button.  Windows XP 32 SP3. Thanks!

Run fine here, including play button.  Windows XP 32 SP3. Thanks!

Will be there an updated build?


Testing Notification Sync

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

Hi, all
I tested the notification sync and I had the problem.
Notification sync test need two AVS devices, one of my devices is Echo Plus and the other is SampleApp
When I asked my notification by using Echo Plus, I had seen that notification state was closed. In expected result, the notification state of SampleApp was changed to off, but it did not happen.
So, I need some help to check the problem. Here is the log:
log1.TXT
log2.TXT
log3.TXT
In log1, this is SampleApp' s state before the test, the NOTIFICATION INDICATOR STATE is off.
In log2, this is SampleApp's state after creating a notification by using Echo Plus and turning SampleApp back, the NOTIFICATION INDICATOR STATE is on.
In log3, this is SampleApp's state after closing the notification state by using Echo Plus and turning SampleApp back, the NOTIFICATION INDICATOR STATE is still on.
What is the expected behavior?

Following the steps, the notification state is off.
What behavior are you observing?

The notification state did not change back to off after the steps.
Provide the steps to reproduce the issue, if applicable:

Device1: SampleApp
Device2: other AVS device
Step:

Run the device1 and check the notification state is off.
Turn off the device1 and check the device's state is offline in Alexa App.
Use device2 to create the notification.
Run the device1 and check the notification state is on.
Turn off the device1 again.
Ask notification by using device2.
Check the notification state is changed to off on device2.
Run the device1 again and check the notification state.

Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.11.0>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[x] Other - tell us more: arm-linux embedded

Tell us about your OS (Type & version):

[x] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi @elfdie00475,
Thanks for bringing this to our attention. According to the UX guidelines, the SetIndicator directive should not have been sent in log2.txt since your device was in DoNotDisturb mode. However, there was a bug in 1.11 in which the DoNotDisturb value on the device may go out of sync in certain situations. This was fixed in 1.12.
We will follow up with the cloud. In the meantime, would it be possible to upgrade to 1.12 and see if that fixes your issue?

Hi @kjkh,
Thanks your reply and tell about the DoNotDisturb behavior.
Actually, I don't handle or test the DoNotDisturb feature until now, so I can sure that DoNotDisturb is not being enabled by voice or manual.
I only test by following the steps I provided.
I still follow your suggestion to test this issue in 1.12.
Thanks.

Hi all,
I try 1.12 by following the same steps, it still happen.
Thanks~~

Hi,
I know SetIndicator and ClearIndicator directive event always send from AVS cloud to my device, so notification will use last status after re-running Sample APP if AVS cloud does not send new directive event to my device.
In time, I think AVS cloud does not send new notification directive event to my device when I re-run the Sample App.
So, I want to know  that have any functions can request the AVS cloud to update notification states in SDK?
Thanks~~

Hi all, me again
We take this issue to ask our AVS solutions architect. And this is his answer:
Looking at your log, you are right.  The cloud should send a clearIndicator directive at startup, but that doesn’t happen until line 997 (in notification_log3.TXT).  That is right in the midst of a SpeechRecognize event, so I am thinking you used a voice command to ask about notifications.
Hope for helpful.
Thanks~~

This issue is fixed in v1.15 of the SDK.


Focus change when tab key is pressed

When a user implement a custom focus change between text elements, TAB key does not respect to the new focus order.
example code: https://dartpad.dev/09bccb26e883c04d53f5525f3a721989
reproduction steps:

in the example code, when ENTER is pressed to navigate between fields focus order is correct: firstName, lastName, email, firstName
2)If TAB is used to jump between fields. Order is wrong. Focus jumps from 3rd field (email) to 4th field tel.

I observed this on Flutter for Web, yet I believe this issue will persist on Desktop and Mobile (when a keyboard is connected to the device).
flutter doctor -v
[✓] Flutter (Channel unknown, 1.19.0-5.0.pre, on Mac OS X 10.15.4 19E287, locale en-US)
    • Flutter version 1.19.0-5.0.pre at /Users/nurhan/development/flutter
    • Framework revision 28138ec5e3 (9 days ago), 2020-06-04 00:33:01 -0400
    • Engine revision 2608f2ee9f
    • Dart version 2.9.0 (build 2.9.0-13.0.dev 54adfeb93f)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/nurhan/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.46.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.11.0

[✓] Connected device (2 available)
    • Web Server • web-server • web-javascript • Flutter Tools
    • Chrome     • chrome     • web-javascript • Google Chrome 83.0.4103.97

note: example is inspired by the usage in this post.

/cc @gspencergoog
/cc @mdebbar since we recently discussed this problem

related: https://github.com/flutter/flutter/issues/57912 ?

Just adding to the knowledge base of this issue.
Tab key is ignored by onEditingComplete as well as onSubmitted. Enter key and tapping away is working correctly.

Verified this on latest master on web and desktop and was able to replicate the issue.
Upon app launch and focus on first field, tapping ENTER button from keyboard shifts the focus from first, second and to third field and back to first field.
Using TAB key if focus is on third field , shifts to fourth field instead of first.

flutter doctor -v

[✓] Flutter (Channel master, 1.27.0-9.0.pre.128, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.27.0-9.0.pre.128 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 199d5dc086 (14 hours ago), 2021-02-28 14:36:02 -0800
    • Engine revision 34236c9837
    • Dart version 2.13.0 (build 2.13.0-81.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.53.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 88.0.4324.192

! Doctor found issues in 1 category.

Removing the platform-web label since this issue is not specific to web. It's reproducible on macOS.

This issue is reproducible on current stable and master channel
Verified environments:

Android Emulator (Pixel 4 API 29)
Chrome Version 100.0.4896.127 (Official Build) (x86_64)
Iphone Emulator (Iphone 13, iOS 15.2)

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.5, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 2.10.5 at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5464c5bac7 (9 days ago), 2022-04-18 09:55:37 -0700
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7042882/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Android Studio
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/203.7185775/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.62.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 100.0.4896.127

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 2.13.0-0.0.pre.763, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 2.13.0-0.0.pre.763 at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cf60f7659 (4 hours ago), 2022-04-27 03:19:05 -0400
    • Engine revision 189cdc7df7
    • Dart version 2.18.0 (build 2.18.0-61.0.dev)
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7042882/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[!] Android Studio
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/203.7185775/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.62.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 100.0.4896.127

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Any update ?


[website] rendering issue on implementing ops faq

Description
Scroll down till you hit the italicized section.
http://mxnet.incubator.apache.org/api/faq/add_op_in_backend
How to reproduce & test
You can run into the problem following these steps.

Run the Jekyll docs build:

ci/build.py --docker-registry mxnetci --platform ubuntu_cpu_jekyll /work/runtime_functions.sh build_python_jekyll

Find the output in /docs/static_site/build/html and deploy that to your dev web server.

#16334 fixed a similar issue with just a line break...

@aaronmarkham I missed this issue... Shall I also add this page to the PR?

@Kh4L sorry I missed your message. Up to you if you want to tackle this issue too.

@aaronmarkham  I don't see the italicized section anymore. I guess it has been fixed :+1:


Commissioners page should have larger (hi-def) commissioner images

these are available at each district's press page: http://www.miamidade.gov/district01/press-room.asp


Motivation
It's a useful feature to get a better idea about the funds on a particular address and it makes it easier to monitor a certain address for incoming or outgoing funds.
Are you planning to do it yourself in a pull request?
Yes.

This will probably have to wait for my balance refactoring.

I wanted to add a waiting label but couldn't find one.

Closing as already contained in https://github.com/iotaledger/iota-sdk/issues/81


Update PTQ config

Related to change in https://github.com/NVIDIA/NeMo/pull/9276/

@janekl to review

@janekl to review

LGTM!


[BEAM-9944] made run_pytest allow options to indicate specific test package

Make run_pytest.sh be enable to specify which tests to run.
This PR is preparation for BEAM-9943 and BEAM-9944, priotize python versions and reduce precommit tests.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tvalentyn

cc: @udim

You can already specify what to run:
tox -e envname -- <anything here is passed to pytest>

This is documented here:
https://cwiki.apache.org/confluence/display/BEAM/Python+Tips

Ah, this PR need not to be merged. Thanks, @udim


arch/arm/stm32h7: multiple fixes for stm32h7 flash interface

Summary
This PR contain few fixes for stm32h7 flash interface:

Fix cache invalidation before reading flash data after flash erase operation
Fix stm32h7_israngeerased operation to use the correct address on byte read
Fix error checking in stm32h7_israngeerased
Replace exclusive semaphore with mutex
Fix few style issues

Impact
Bug fix
Testing
Pass CI

please rebase the change to fix the ci error.

@tito97sp should we merge this patch instead yours(#7397)?

@tito97sp should we merge this patch instead yours(#7397)?

@xiaoxiang781216  Yes. This PR substitutes mine.

Could you please approve the PR if all the tests pass at your end?

@tito97sp should we merge this patch instead yours(#7397)?

@xiaoxiang781216  Yes. This PR substitutes mine.

Could you please approve the PR if all the tests pass at your end?

I'm on it

@tito97sp should we merge this patch instead yours(#7397)?

@xiaoxiang781216  Yes. This PR substitutes mine.

Could you please approve the PR if all the tests pass at your end?

I'm on it

Hi @pkarashchenko how can I move to your PR on local??


Modifier support for Custom Datatype

In PostgreSQL/PostGIS, when creating tables with geometry or geography data type, additional modifiers are allowed. In this example, we can specify geography(POINT, 4269) as data type, which indicates the column is geography point typed, and uses 4269 as its spatial reference system.
Back to sqlparser, at the moment we don't support Custom data type that comes with additional modifiers/arguments in form of (..). This patch add support for that. Since these data can be in any form like an identifier, a number or a string, here I convert all of them into String. Suggestions are welcomed.
Edit: In PostGIS this is called type modifiers.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

26 of 28 (92.86%)  changed or added relevant lines in 2 files are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.04%) to 85.949%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
22
24
91.67%

Files with Coverage Reduction
New Missed Lines
%

src/parser.rs
8
83.66%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
10900

Relevant Lines:
12682

💛  - Coveralls


How can I change the position of the response window

In the documentation a gif is shown where the response opens on the right side but for me it opens on the left side, how can I configure it?

You should be able to configure that in your neovim configuration, either globally or just for http files.
vim.opt.splitright = true

Hey @luisparedes-nomad, does this help? Can we close this issue?

it works, thx


log yaml file in config

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

Comparison is base (93f4366) 95.13% compared to head (8c7ec06) 95.13%.
 Consider uploading reports for the commit eb6586a to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #776   +/-   ##
=======================================
  Coverage   95.13%   95.13%
=======================================
  Files         127      127
  Lines        4992     4992
=======================================
  Hits         4749     4749
  Misses        243      243

Flag
Coverage Δ

unittests
95.13% <ø> (ø)

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

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


(PDB-4420) partition of resource_events

Prototype of partitioning the resource_events table. This creates a base
table (from the same schema as the existing resource_events table) and
will create 8 total partitions during migration (this ISO week +- 4)
Partitions are based on ISO 8601 weeks. Inserting into the partitions is
controlled via a trigger that directs the insert to the appropriate
partition.
Partitions are dynamically created by scanning the events to be inserted
and performing a "CREATE TABLE IF NOT EXISTS" for the partition. This is
akin to the (ensure-certname) code that already exists.
The inheritance-based implementation was chosen to maintain backwards
compatibility with PostgreSQL 9.x, since the other components of PE are
not ready for PostgreSQL 11.
TODO: Migrate existing events to partitioned tables - they are currently
dropped.
TODO: GC by dropping expired partitions.
TODO: Migration test for initial creation of partitions

CLA signed by all contributors.

I chose ISO weeks because it's the most straightforward approach. Even if we want the granularity for queries to be lower, I think the storage granularity has to stay at the week, otherwise we risk making this either too complicated, or a performance problem on the number of partitions we create (imagine 365.25 tables per year, that's a lot)

If the average report TTL is < 7 days, then partitioning by day would be better. Do we have details on the actual use of report-ttl in the field?
See: https://www.postgresql.org/docs/9.6/ddl-partitioning.html
All constraints on all partitions of the master table are examined during constraint exclusion, so large numbers of partitions are likely to increase query planning time considerably. Partitioning using these techniques will work well with up to perhaps a hundred partitions; don't try to use many thousands of partitions.
This is where the impact on the number of partitions comes into play, and one of the driving reasons for using ISO weeks.

Tests Failed \o/

Looks like Jenkins might actually be lying to us here. The acceptance tests are failing https://cinext-jenkinsmaster-enterprise-prod-1.delivery.puppetlabs.net/view/puppetdb/view/master/job/enterprise_puppetdb_integration-system-puppetdb_full-master/1947/
I think it's because only the ezbake job is nested in the PR Kickoff job. Might take a bit of cjc wizardry to fix that without duplicating jobs... I'll make a ticket


We will create another one.


Allow PEP 3107 function annotations

Allow the user to add function annotations, currently they are removed by wxGlade's codegen:
def OnButtonHelp(self, event: wx.Event) -> None:  # wxGlade: FindFilesDialog.<event_handler>
    pass

will be overwritten by
def OnButtonHelp(self, event: wx.Event):  # wxGlade: FindFilesDialog.<event_handler>
    pass

I have modified rec_event_handler to support this.
Would you be interested in generating PEP 3107 functions in the first place (should be user configurable)?

Well, personally I don't use annotations, but certainly they're useful. I prefer to write the code while the debugger is active and so full introspection is available.
I will integrate the PR once I'm done with issue #357.
When using annotations and generally it's better to use derived classes and these files won't be touched by wxGlade.
As the derived classes usually start as copy & paste from the generated code, it makes sense to generate them in first place. This is non-trivial, though as a handler can be used for several event types. With generic types like wx.Event I'm not sure whether it's too useful.


How to create stepped line chart?

Hi, there,
I am wondering if n3-charts support stepped line chart? something like below:
http://jsfiddle.net/hrabinowitz/abJLf/22/
Although above is based on d3, ne-charts is based on d3, so I think it should support. however, I can't find any document. I also tried here and there without luck. especially I tried lineMode: "step-after" without success.
Any help or sample would be greatly appreciated!
Sam

Hi @samwzm
The interpolation option for each series is your friend, try step, step-before or step-after. Here is a demo http://codepen.io/chaosmail/pen/OMvMRM
Best,
Christoph

Thank you very much for the prompt help, Christoph! it works!
by the way, are there more detailed documents about n3-charts, especially for the beginner (without d3 experience)? the doc in the website is clear, but missing lots of details.
thanks again,
Sam

thanks for the kind feedback. Yeah, we know that the docs are lacking a lot of information, and it's on our todo list to improve them. If you want to help us, feel free to add something (maybe about the interpolation) to the docs (https://github.com/n3-charts/line-chart/blob/master/docs/usage.md) and submit a PR. We are always happy about community support :)


More commands and pre-made config.json file

I added a few commands to the index.js file that include eval, suspend, and exile. I also made changes to config.json to fit changes I did to the fire command, lowest role = fired role if your wondering. I saw people wanting a suspend and exile command, so I made these, hope ya'll enjoy. Also, there's a blacklist system on there.

Hi,
Thanks for submitting your pull request. I might make a few changes to some of these features if you are okay with that, but in general this is a great head start for coding all of them.

I pretty much just used the setrank and fire command, changed some value, and boom, done, lol.

I love that you added the eval command. It'd help allow others to explore what they can do with a little bit of JavaScript.


feature: logux actions behaive as action params

https://github.com/wilddima/logux_rails/issues/8

Pull Request Test Coverage Report for Build 126

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 98.947%

Totals

Change from base Build 124:
0.003%

Covered Lines:
376

Relevant Lines:
380

💛  - Coveralls

@dsalahutdinov тесты не проходят

@wilddima принимаем PR?


Handle POSOTHER and MKTVAL in investment statements

I'm using this to pull down investment statements from TIAA-CREF, and found a few fields that weren't being parsed; specifically, positions of type POSOTHER, and the MKTVAL (Market Value) field in positions.
I've added support for both of those and unit tests for them (both using an anonymized version of a TIAA-CREF statement for POSOTHER, and by addking the MKTVAL support to positions in existing fixtures).
Thank you SOOOO much for this library!

You rock, jantman.  We are hugely encouraged when people find this library useful and take the time to tell us. :)

Thanks! It certainly was a great find. I'm using it in my biweeklybudget project, which is really rough and definitely not "end user" software, but is intended to be workable personal finance and budgeting software for people who are paid biweekly (i.e. every 14 days, or every other X-day), since nothing else out there seems to do that...

@jseutter Thank you so much for merging this. Are there any plans to get a new release up on PyPI any time soon?

Hi Jason, I'm in the middle of real life stuff and plan to have a release out afterwards. If I don't have a release out by mid-July please ping me.

On Jun 12, 2017, at 4:06 PM, Jason Antman<EMAIL_ADDRESS>wrote:
@jseutter Thank you so much for merging this. Are there any plans to get a new release up on PyPI any time soon?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@jseutter Ok, no worries, I completely understand that and was in a similar position myself a few months ago. Thanks for taking the time to update. No rush.


Logs are not shown anymore since 5.0.1

After 5.0.1 change logs do not appear. running in docker container.
docker run -i -t --name mockserver -p 443:443  jamesdbloom/mockserver:mockserver-5.0.0
the output for 5.0.0
java  -Dfile.encoding=UTF-8 -Dmockserver.logLevel=INFO -jar /opt/mockserver/mockserver-netty-jar-with-dependencies.jar -serverPort 443

2017-12-14 09:16:18,925 INFO o.m.m.MockServer MockServer started on port: 443
2017-12-14 09:17:25,186 INFO o.m.m.HttpStateHandler resetting all expectations and request logs

output for versions greater or equal to 5.0.1 (tested 5.1.1, 5.2.2)
docker run -i -t --name mockserver -p 443:443  jamesdbloom/mockserver:mockserver-5.0.1
java  -Dfile.encoding=UTF-8 -Dmockserver.logLevel=INFO -jar /opt/mockserver/mockserver-netty-jar-with-dependencies.jar -serverPort 443

but looks like mockserver works as expected though

This is now fixed.
I will release a new version tomorrow or you can use the 5.2.3-SNAPSHOT version.
In addition in the mean time you could use the /retrieve?type=logs endpoint as described at https://app.swaggerhub.com/apis/jamesdbloom/mock-server-openapi/5.2.2#/control/put_retrieve
The /retrieve?type=logs allow you to retrieve logs on a per request basis.  Using a request matcher you can retrieve all logs that are relevant to that request or if you don't provide any request matcher all logs will be returned.
I am currently working on update the documentation to include details of this and loads of other features that are not currently documented well enough.


Implement CD to update image automatically

AsIs
Currently we have CI pipelines to publish build artifacts to GHCR, but for k8s side, we have to manually run kubectl -n bennuhp rollout deploy/django for renewing container image.
For more agilities of development, we need to automate this with commit-push triggers
ToDo

Determine solutions: fluxcd / ArgoCD
Install controllers onto GKE, if needed

(Optional) Acceptance criteria
Confirm that after some changes will be merged into main:

k8s resource will be updated
image ID hash would be updated

(Optional) Additional contexts
N/A

Considering our requirements, realized that GitOps is not always needed for our deploy pipeline.
Since only application(Django) image has been built as OCI image in this repository, which should be synced with source codes in this repo, so the core requirement is not GitOps, instead automatic image rollout/update triggered by git push.
With researching lightweight and handy solution for this CIOps requirement, we adopted to use keel-hq/keel.
Refering the docs, keel is an in-clustered & a stateless solution, and will controll CD policy with annotation in application manifest.

Reopen for awaiting e2e validations.
Ref: https://github.com/hwakabh/bennu-official/pull/201#issuecomment-2266357298

Since this issue will be managed in #209, marked as wontfix.


locationFromAddress() never returns on iOS

Hi, I have a problem using locationFromAddress()
When building the following on iOS, the 1st print statment never returns. I've provided the address given in the usage example just to be sure that this wasn't the address that was failing.
There is no error message, just the app waiting for the 1st print to return. print('coordinates received'); is never reached.
Any idea of what I could have missed?
import 'package:flutter/material.dart';
import 'package:geocoding/geocoding.dart';
void main() async {
  print(await locationFromAddress("Gronausestraat 710, Enschede"));
  print('coordinates received');
  runApp(MyApp());
}

@victorfaye most likely the services are not started fully yet since your app hasn't been properly initialized yet.
I noticed that you are running the commands before you executed the runApp(MyApp()); line. I suggest you move the print statements to a location somewhere inside the MyApp class to ensure the Flutter App is fully initialized and method channels are correctly setup on the native platform.


Meta issue for Conda-Store user experience

Opening this issue to track all of the Conda-Store related user experience questions and issues. The goal is help users navigate the recent Conda-Store changes. Much of what is discussed below can be included in the docs eventually.
Questions:

[ ] 1. When should new conda environments be created via the qhub-config.yaml and when should they be created from the /conda-store endpoint? And what difference does it make?
[ ] 2. What are the steps to delete a conda environment I no longer need?
[ ] 3. What are the differences between the default and filesystem namespaces?

Known issues:

https://github.com/Quansight/qhub/issues/1158
https://github.com/Quansight/qhub/issues/1021

related to how conda-store environments are used by CDS dashboards

@iameskild , yes, I'm super confused about the new conda-store approach.
We originally added packages to our 0.4 Qhub deployment by going to the conda-store manager https://jupyter.data.science.earth/conda-store/, choosing a namespace, and dropping an environment.yml file in there.
Then we were told that if we want to have environments available for all users, we should add them in the config.yml the way we used to do and let github actions deploy them.   So I've tried that, but when I look at the conda-store manager now I see my pangeo environment with 0 bytes, and a bunch of builds not working:

Questions:

How to debug/recover from this?
Can we still shell into the conda-store pod and delete environments?

Getting back to one of my original points. 'default' and 'filesystem' are terrible names. Let's define what they are and use better names.

@costrouc any updates on what has been decided?

@costrouc does this need to remain open? since everything is happening in conda-store (repo and the such) this seems stale now

Issue: In the new UI, we need to intermittently "login" to conda-store, and we need to click on the unintuitive profile icon to do so.
Solution: Ideally, we shouldn't need to login again if we're already logged-in to Nebari. At the very least, the "login" requirement should be more obvious and intuitive.

Dharhas said: One of the biggest overall issues with Nebari is the fact that we don't have a full single-sign on, each part requires going through the auth flow again (keycloak, argo, conda-store etc).

Originally discussed on Slack

This issue has been opened for a very long time and conda-store and the conda-store-ui have changed significantly. At this point, we request all UI/UX issues be opened here https://github.com/conda-incubator/conda-store-ui .
Before closing this issue, I'll comment on the original questions:

When should new conda environments be created via the nebari-config.yaml and when should they be created from the /conda-store endpoint? And what difference does it make?
the recommendation is that all envs are built through the conda-store interface, though it is still possible that admins may want to define default envs in nebari-config.yaml.
What are the steps to delete a conda environment I no longer need?
there is now a delete button in the ui
Add a feature that auto-deletes everything but the past 5 builds per env.
We have an open issue to improve the build deletion process and options. I've added a note there


fix: [#3874] Parity issue with getContinuationActivity method missing the type property

Fixes #3874
Description
This PR fixes a parity issue in the getContinuationActivity method, adding the missing type property that its DotNet counterpart has (link). Otherwise when the ActivityHandler.run method is executed will validate and throw the Activity is missing its type error when the type property is not found in the Activity.
Specific Changes

Added the type property to be an ActivityTypes.Event in the getContinuationActivity method.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.003%) to 84.345%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
19685

Relevant Lines:
22143

💛  - Coveralls

Thanks, @sw-joelmut! Much appreciated.


VS code incorrectly matching "resourceGroup" parameter

For some reason VScode complaints about parameter name

@artisticcheese Does this still repro?  I haven't been able to find/create a deployment template that has this issue.  It's possible the schema has been corrected in the cloud.
I fixed reference so it can take three arguments.
Please let me know if you still see this issue, thanks.  See attached for (non-functional) template that was the closest I could come up with.
101-vm-with-data-management-gateway.zip

Yes, it still happens example template below which shows this

@artisticcheese Thanks!  This is a problem in the Azure schemas.  I've reported it here.

There's nothing I can do in the extension to fix this - this is handled by the built-in language service and the schemas.  We are considering work outside the extension to improve this experience, but there's nothing I can do in the extension.


@transform macro

This PR implements the @transform macro.
This macro takes as argument a data object of type Meshes.Data and a list of transformations and returns a new Meshes.Data object with data columns and new columns calculated using the transformations.
Examples:
x = [1, 1, 1, 1, 2, 2, 2, 2]
y = [1, 1, 2, 2, 3, 3, 4, 4]
table = (; x, y)
sdata = georef(table, rand(2, 8))

ndata = @transform(sdata, :z = :x + 2*:y)
values(ndata)

julia> values(ndata)
Table with 3 columns and 8 rows:
     x  y  z
   ┌─────────
 1 │ 1  1  3
 2 │ 1  1  3
 3 │ 1  2  5
 4 │ 1  2  5
 5 │ 2  3  8
 6 │ 2  3  8
 7 │ 2  4  10
 8 │ 2  4  10

Codecov Report

Merging #114 (c76e48c) into master (dc40f5e) will decrease coverage by 4.48%.
The diff coverage is 0.00%.

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

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage   94.08%   89.59%   -4.49%
==========================================
  Files          41       42       +1
  Lines         879      923      +44
==========================================
  Hits          827      827
- Misses         52       96      +44

Impacted Files
Coverage Δ

src/geoops/transform.jl
0.00% <0.00%> (ø)

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

Beautiful! 💯


Add a per chapter complete tick in map.

I think this would be helpful.

Thank you for this suggestion. We've written the code for this and it will be in a near-future release.


Fix removing a folder that contains a file is not removed from the FileSystem Dock

Fixes #94431

The issue is a side-effect from #92303. To be able to manage correctly the removing of scripts with global class names, I added all removed files in scan_actions with the action ACTION_FILE_REMOVE. But in _update_scene_groups or _update_script_documentation the method find_file is called, which calls _find_file. The method _find_file recreates the parent folder of the searched file if it does not exist in filesystem.
I added a check to prevent adding a folder in filesystem that does not exist on the disk.

Might also fix #93097

I confirm, I tested the MRP from #93097 before an after the fix and this PR solves #93097 also.

Thanks!


Failed to enable unit, unit /etc/systemd/system/xrdp.service is masked.

Hello, I get this error after the update
update-rc.d xrdp enable
Failed to enable unit, unit /etc/systemd/system/xrdp.service is masked.

you could try to unmask xrdp with systemctl unmask xrdp.service, but usually a service does not get masked "randomly", something must have done this

@dunlex - you're getting systemd errors on a system where you're using update-rc.d
What is your operating system exactly?

No further info - closing.


[Priority: Low] Show multiple receive addresses

A user that often gets paid wants to create new addresses for every payment, right now the receive address only changes if money is sent to it.
I propose either

Shuffle a couple of receive addresses and randomly show them up,
Show multiple receive addresses at once.

When we started Breeze we wanted to include an "advanced" switch in the settings, making this like that possible.
What do you think @bokobza?

yeah I think ultimately we'd like something like that. Let's leave this open and assign it to vNext :-)

The API is ready to do this feature as soon as PR https://github.com/stratisproject/StratisBitcoinFullNode/pull/506 merged.
Endpoint: addresses


DRAFT [usbdev] Support the safe retraction of IN packets

This draft PR relates to a discussion of perfective changes in #19435 (open) and #18942 (closed and subsumed).
There are two distinct cases when we may want to retract IN packets that have been presented for collection by the USB host, and which are still marked as ready for transmission ('rdy' bit set in 'configin'):

Isochronous IN traffic, when there is new data available (for Isochronous streams the more recent data is more important than older data)
Loss of ACKnowledgement of the final IN packet within the Data Stage of a Control Transfer; in this case, the missing acknowledgement is to be detected by the lack of a 'pkt_sent' interrupt for the IN Endpoint and the arrival of the zero-length OUT DATA packet that is the Status Stage. (See '<IP_ADDRESS> Error Handling on the Last Data Transaction')

this requires us to retract the final IN packet since it has indeed been sent (just not successfully), and invert the IN Data Toggle on that endpoint, to realign with the host's state.

There may be other cases, especially in handling error cases/edge conditions, that this form of IN packet retraction is required.
This PR proposes minimal changes to the IN-side hardware that involve an additional status bit in the 'configin' register. The 'sending' bit becomes set when the hardware first attempts to transmit the IN packet, and it remains set until such time as the packet is successfully transmitted (when both 'rdy' and 'sending' are cleared) or the packet is canceled by the hardware ('rdy' and 'sending' both cleared, and instead 'pend' is set).
Packet retraction operates as follows:

FW writes to the 'configin' register with the 'rdy' bit clear. The intention would be to write the 'buf' number and 'size' with their current values, but this PR incorporates logic to capture these properties of the current IN transaction so that they cannot be changed throughout the current transaction. It is therefore safe just to write '0' to the 'rdy' bit.
FW reads back from the 'configin' register to check the 'sending' and 'pend' bits.
If 'sending' is set then FW must wait until either successful transmission is communicated via the 'pkt_sent' interrupt, or the 'pend' bit becomes set because of transaction cancelation, as per normal IN packet transmission.
If 'pend' is already set, or 'sending' is clear, then the buffer may safely be reallocated and the new packet supplied immediately.

This remains a draft PR presently to encourage discussion and because, although I have reasoned through the logic, I have not yet had a chance to test the implementation properly. Basic simulations and inspection of the logic behavior in waveforms are okay, but there are obviously no new tests to exercise this functionality at this point.
@a-will

Isochronous IN traffic, when there is new data available (for Isochronous streams the more recent data is more important than older data)

For what it's worth, I'm not sure we can serve the isochronous use case perfectly without double-buffering and/or a TTL. It's really a mismatch for the functions a RoT would ordinarily serve anyway, but isochronous endpoints are only supposed to be used for data that is aligned to a particular frame number (or window of frame numbers), with a specific advertised polling rate.
The reason is all having to do with the tight timing. Isochronous transfers are performed very early in the frame, and if someone wants to implement polling for every frame, then all they have is the SOF interrupt and almost no time to swap buffers for the retraction case. The TTL would guarantee packet retraction at the particular frame, and the double-buffering would immediately replace it for the next transmission.
Or... something requiring fewer bits than a TTL would be a "retract at next SOF" bit.
Probably this is only necessary for the fastest polling rate, though. Once we get to every-other-frame or slower, there is plenty of time to retract and swap (that is, after software learns which frame numbers will have transmission scheduled -- That means it also knows which frames will not be scheduled, when it is safe to swap buffers). ...We could just specify a max polling rate and be done with it, heh.
Sorry for opening with a random aside!

I think the idea of a TTL has merit but I''m aiming here for a minimal change, given the time frame, and of course this doesn't preclude the possibility of improving Isochronous support later. It's 'push driven' in that the software can at any point switch over to the new buffer and our control software already implements double-buffering, ie. the software completes the new buffer within the packet memory before then just hitting the 'configin' register to make it current.
There's no reason, in principle, that the software cannot use the frame interrupts (1ms intervals) to set up a timer and that tracks the frame interrupts but itself fires at say 80% of the way through the bus frame. It can thus switch to the new buffer in preparation for the new frame start to implement 'pull-driven' streaming.
As you say, Isochronous support really isn't a priority, though.

I think this feature could be useful also for the UART to USB forwarding used by ChromeOS. Assuming the OT chip has received a few characters on the UART and put them in a buffer ready for an IN transaction, but the USB host has not yet retrieved them, when more characters arrive via UART. In such case, it would be desirable to "augment" the prepared IN buffer (rather than enqueue another buffer with just a few bytes), which could be done if we have this feature.

DV smoke test failure (chip_sw_spi_host_tx_rx) unrelated.


Add #[MustUseResult]

Add #[MustUseResult] attribute that can be used on methods. This enforces the result from the method call must be used.
E.g. if you has a class like this:
class Money {

  public function __construct(public readonly int $pence)
  {}

  public function add(int $pence): self
  {
     return new self($pence + $this->pence);
  }
}

You might misuse the add method like this:
$cost = new Money(5);
$cost->add(6); // this has not modified $cost

Adding the #[MustUseResult] attribute to the Money::add() method means the result from the method call must be used.
This would be reported as an error:
$cost->add(6);

And this would be OK:
$updatedCost = $cost->add(6);

Implemented by #31


Runelite freezes as soon as GPU is enabled and side bar constant sidebar glitch

Describe the bug
I can no longer load Runelite with GPU enabled, it instantly freezes while loading the log in screen.
I can load Runelite with GPU disabled and log in etc.. but as soon as I enable, it freezes.
Have uninstalled and installed RL a number of times, no luck. (If I download the 32 bit, it will instantly close rather than freeze while loading.)
I have uninstalled suggested programs that may clash such as 'Alien command center'
I have tried adding --mode=off and disabling acceleration as suggested here:
https://github.com/runelite/runelite/wiki/Disable-Hardware-Acceleration
I have played around with the 3d settings (all quite new to me) but nothing seems to make a difference.
Screenshots

Environment (please complete the following information):

OS: Windows 10
RuneLite <IP_ADDRESS> (launcher version 2.1.3)

I have a Dell G7 laptop
Nividia - GeForce GTX 1660 Ti
client.log
launcher.log
Any help would be appreciated! I've looked all over here and reddit etc and I can't find a solution! This is the only game I play and I'm eager to get back online! Thank you.

Have you followed our troubleshooting page? This entry sounds like what you're describing. https://github.com/runelite/runelite/wiki/Troubleshooting-problems-with-the-client#client-closing-when-loading

Closing since stale, and this should have been done through the support channel


Error detected while processing BufWinEnter Autocommands

Vim bootstrap version(number in first line):
" vim-bootstrap 2022-03-23 18:41:31

Operating system:
Darwin  21.4.0 Darwin Kernel Version 21.4.0: Mon Feb 21 20:34:37 PST 2022; root:xnu-8020.101.4~2/RELEASE_X86_64 x86_64
Monterey

Vim/Nvim version:
vim 8.2.4550

Languages:
go, python, rust

Description
Error detected while processing BufWinEnter Autocommands for "*"..function <SNR>79_Setup[6]..<SNR>79_IndentLinesEnable[10]..<SNR>79_SetConcealOption:

necessary tooling already been installed per language.
using asdf for go!

I get the same error and it seems related to indentLine plugin.
https://github.com/Yggdroot/indentLine/issues/369

for anyone needs quick solution,
comment this line. vim-plug smart enough to disable it.
vi .vimrc
 " Plug 'Yggdroot/indentLine'

closing this...


Task: Add option to sign a metadata without using API (local metadata)

What is the task about?
Task: Add option to sign a metadata without using API (local metadata)
Parent feature
No response
References
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

[ ] Upload pend roles locally
[ ] Save/upload the signed metadata


TSv1 use of monospace fonts is broken on Windows

TSv1 uses Menlo by default and the CSS does not fall back to a reasonable monospace fonts. Menlo isn't supported on Windows so this breaks the UI. This is particularly confusing with the error messages which rely on monospace:

There is already a PR which fixes this at #127

Fixed in #127


Add GUI tests

Added tests to binds editor (100% coverage) and midi status (80% coverage), and editing github workflow according to https://pytest-qt.readthedocs.io/en/latest/troubleshooting.html

Any progress with resolving the conflicts?


What is the recommended number of nodes for TiKV should it be even or odd?

Question
Before asking a question, make sure you have:

Searched existing Stack Overflow questions.
Yes
Googled your question.
Yes
Searched open and closed GitHub issues
Yes
Read the documentation:

TiKV Readme
TiKV Doc
TiKV-Ctl Doc
TiDB & TiKV Stack Doc

Number of nodes doesn't matter. We suggest that the number of duplicates for each region should be odd.

At least 3 TiKV nodes are recommended. It doesn't matter whether number of TiKV nodes is even or odd.

Ah, sorry, it's better to be odd.

Two conflicting opinions from @MyonKeminta and @solotzg. I am confused now 😅 . Does the number of nodes affect raft consensus algorithm? Or does it depend on number of replications of regions

The number of nodes does not affect availability directly. We recommend configuring replica count to an odd number to make the best use of raft. Meanwhile, you need to make sure the count of node is greater than replica count. Otherwise, the number of replicas cannot be satisfied.


ITV Hub - ERROR: [rtmpdump] Could not connect to RTMP server.

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.12.02. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.12.02

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
[debug] System config: []
[debug] User config: ['-k', '-o', 'D:/youtube-dl/%(title)s.%(ext)s']
[debug] Custom config: []
[debug] Command-line args: ['-f', 'best', '-u', 'PRIVATE', '-p', 'PRIVATE', 'https://www.itv.com/hub/the-x-factor/1a8647a0277', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2017.12.02
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: ffmpeg N-89369-g5a93a85fd0, ffprobe N-89369-g5a93a85fd0, rtmpdump 2.4
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (GB) as X-Forwarded-For.
[ITV] 1a8647a0277: Downloading webpage
[ITV] 1a8647a0277: Downloading XML
[ITV] 1a8647a0277: Downloading JSON metadata
[ITV] 1a8647a0277: Downloading m3u8 information
[debug] Invoking downloader on 'rtmpe://cp72511.edgefcs.net/ondemand'
[download] Destination: D:\youtube-dl\Live 11 Final.flv
[debug] rtmpdump command line: rtmpdump --verbose -r "rtmpe://cp72511.edgefcs.net/ondemand" -o "D:\youtube-dl\Live 11 Final.flv.part" --swfVfy "http://www.itv.com/mercury/Mercury_VideoPlayer.swf" --pageUrl "https://www.itv.com/hub/the-x-factor/1a8647a0277" --app "ondemand?auth=eaEd1aqbtbhdFcydcbfd5dHa2dpdddoaOdodBa8bWdxcjbJdPambJc3bjaocUa4blcK-bAjuQM-eS-7xs2LBumwtAs&aifp=v002&slist=production/" --playpath "mp4:production/priority/CATCHUP/1/8647/0277/001/1-8647-0277-001_PC011800_16X9.mp4" --resume --skip 1
[rtmpdump] RTMPDump v2.4 GIT-2015-12-14 (Compiled by KSV)
[rtmpdump] (c) 2010 Andrej Stepanchuk, Howard Chu, The Flvstreamer Team; license: GPL
[rtmpdump] DEBUG: Parsing...
[rtmpdump] DEBUG: Parsed protocol: 2
[rtmpdump] DEBUG: Parsed host    : cp72511.edgefcs.net
[rtmpdump] DEBUG: Parsed app     : ondemand
[rtmpdump] DEBUG: Number of skipped key frames for resume: 1
[rtmpdump] DEBUG: Protocol : RTMPE
[rtmpdump] DEBUG: Hostname : cp72511.edgefcs.net
[rtmpdump] DEBUG: Port     : 1935
[rtmpdump] DEBUG: Playpath : mp4:production/priority/CATCHUP/1/8647/0277/001/1-8647-0277-001_PC011800_16X9.mp4
[rtmpdump] DEBUG: tcUrl    : rtmpe://cp72511.edgefcs.net:1935/ondemand?auth=eaEd1aqbtbhdFcydcbfd5dHa2dpdddoaOdodBa8bWdxcjbJdPambJc3bjaocUa4blcK-bAjuQM-eS-7xs2LBumwtAs&aifp=v002&slist=production/
[rtmpdump] DEBUG: swfUrl   : http://www.itv.com/mercury/Mercury_VideoPlayer.swf
[rtmpdump] DEBUG: pageUrl  : https://www.itv.com/hub/the-x-factor/1a8647a0277
[rtmpdump] DEBUG: app      : ondemand?auth=eaEd1aqbtbhdFcydcbfd5dHa2dpdddoaOdodBa8bWdxcjbJdPambJc3bjaocUa4blcK-bAjuQM-eS-7xs2LBumwtAs&aifp=v002&slist=production/
[rtmpdump] DEBUG: live     : no
[rtmpdump] DEBUG: timeout  : 30 sec
[rtmpdump] DEBUG: SWFSHA256:
[rtmpdump] DEBUG: b6 c8 96 6d a3 f4 96 10 be 71 78 b0 1c a3 3d 04
[rtmpdump] DEBUG: 6b bf 91 5e 29 08 d9 da fe 11 e4 b0 42 d8 ee ea
[rtmpdump] DEBUG: SWFSize  : 990750
[rtmpdump] DEBUG: Last keyframe is first frame in stream, switching from resume to normal mode!
[rtmpdump] DEBUG: Setting buffer time to: 36000000ms
[rtmpdump] Connecting ...
[rtmpdump] ERROR: RTMP_Connect0, failed to connect socket. 10061 (Unknown error)
[rtmpdump] DEBUG: Closing connection.
ERROR: [rtmpdump] Could not connect to RTMP server.
  File "__main__.py", line 19, in <module>
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\__init__.py", line 465, in main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\__init__.py", line 455, in _real_main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 1986, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 795, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 849, in process_ie_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 1620, in process_video_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 1893, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 1832, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\downloader\common.py", line 361, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\downloader\rtmp.py", line 169, in real_download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\downloader\common.py", line 163, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 612, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpcuf0d0kb\build\youtube_dl\YoutubeDL.py", line 574, in trouble

Description of your issue, suggested solution and other information
I'm trying to download a video off ITV Hub, there is no problem when I'm downloading HLS, but when I want to get the best, which is RTMP, I get ERROR: [rtmpdump] Could not connect to RTMP server.
I'd like to note I'm outside of England, I'm using Unlocator to alter my location and I don't have anything listening on the port 1935.

Most likely a connectivity issue on your side.


pip install requests fails

`/data/user/0/org.qpython.qpy3/files/bin/qpython-android5.sh "/storage/emulated/0/qpython/scripts3/pip_console.py" && exit
d/0/qpython/scripts3/pip_console.py" && exit   <
Input pip commands, ie: pip install {module}
-->pip install requests
Downloading/unpacking requests
Running setup.py egg_info for package requests
Traceback (most recent call last):
File "", line 16, in
File "/storage/emulated/0/qpython/cache/pip-build/requests/setup.py", line 55, in
exec(f.read(), about)
File "", line 14
cake = u'\u2728 \U0001f370 \u2728'
^
SyntaxError: invalid syntax
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 16, in
File "/storage/emulated/0/qpython/cache/pip-build/requests/setup.py", line 55, in
exec(f.read(), about)

File "", line 14
__cake__ = u'\u2728 \U0001f370 \u2728'

                                     ^

SyntaxError: invalid syntax

Command python setup.py egg_info failed with error code 1 in /storage/emulated/0/qpython/cache/pip-build/requests
Storing complete log in /sdcard/qpython/.pip/pip.log
-->`

Works like a charm,
Thanks a lot!!!!

you can also switch to the python 3.6 beta version.

For all intrested in topic: I installed Beaufitul Soup 4 via Termux on Android (Python 3.6), and after this I copied them to Qpython3 (Python 3.6) - works perfectly. Maybe this solution could be implemented for other issues with package installation?

For all intrested in topic: I installed Beaufitul Soup 4 via Termux on Android (Python 3.6), and after this I copied them to Qpython3 (Python 3.6) - works perfectly. Maybe this solution could be implemented for other issues with package installation?

Can please share it


Adjust embedded image targeting for bottom border

Production:

Dev:

NOTE: I opted not to change the behavior of https://github.com/parameter1/base-cms/blob/master/packages/marko-web-theme-monorail/scss/components/_content-page.scss#L239 as that behavior is functionally/logically different than these, if that should also be changed to be targeting in the same fashion, leave a review noting it.

 When images are uploaded in Base 'extra' code is also being added.


speedup TriMesh.edgepaths by a factor 20

Based on what I did for https://github.com/python-adaptive/adaptive/commit/9f481562b0118199d4011667e857ed8eb764a2a5 I made these changes here too.
The numbers say everything :)
# with this PR
%timeit trimesh.edgepaths
4.7 ms ± 271 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

# @ master
%timeit trimesh.edgepaths
86.6 ms ± 5.65 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

Nice!!

There's a few other places we can use this trick. I'd be happy to work on those but I'll list the ones I can think of immediately:
Datashader path concatenation
Chord path concatenation

Looks good and tests are passing, merging.


Feature/opendp polars

I have other comments but maybe it is just not up to date :) Let me know when to review!

It would be great to have a complete notebook showcasing all possibilities and arguments and add it to the docs (like the other libraries)


Testing Fresnel reflections

Testing Fresnel reflections with MCML to confirm values.

Has been tested with the RayScattering branch


CLN: Use File pydantic to build file-object

Addresses #457

Merge after: https://github.com/equinor/fmu-dataio/pull/512


ClearColor and Sprite same value yields different color

Bevy version
Bevy 0.6.1
Operating system & version
Pop!_OS 21.10 x86_64
What you did
Set a sprite and the background to the same rgb value
use bevy::prelude::*;

const BG_COLOR: Color = Color::rgb(0.16, 0.16, 0.16);

fn main() {
    App::new()
        .insert_resource(ClearColor(BG_COLOR))
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup)
        .run();
}

fn setup(mut commands: Commands) {
    commands.spawn_bundle(OrthographicCameraBundle::new_2d());

    commands.spawn_bundle(SpriteBundle {
        sprite: Sprite {
            color: BG_COLOR,
            custom_size: Some(Vec2::new(50.0, 50.0)),
            ..Default::default()
        },
        ..Default::default()
    });
}

What you expected to happen
The rectangle sprite should be indistinguishable from the background.
Only one color should be visible.
What actually happened
The sprite (center) renders slightly darker than the background.

Good find. Thanks for the excellent issue and report. Under the hood, I bet this has something to do with linear vs sRGB colors.

Issue is that Color in wgpu is using f64, but we're using f32, and the difference is in the rounding

Issue is that Color in wgpu is using f64, but we're using f32 then u8, and the difference is in the rounding

Is it confirmed that this is a rounding issue? I would have expected the rgb value to be off by one in that case but the values are 38 vs 41 in the screenshot.
Even considering f32 to u8 conversion, you'd expect 0.16 to be 40.8 so either 40 or 41 which matches the background color. Compared to that the sprite is too dark. Though, maybe there are other things at play, too (desktop composition / color management / etc).

Didn't see the PR, thanks for elaborating. I tried it with a few different colors and they all seem to match up.
Though, I would prefer a different fix if I'm honest. With #4357 I'm not able to set accurate clear colors using Color::rgb_u8(). Anything below (13, 13, 13) is black for example.
I guess it's because the quantization to u8 is done using linear color, not srgb which makes it loose a ton of accuracy for darker colors.

If anything this is a "bug" there

I'm not sure this is so much a bug as a "known limitation" that should be documented... This would only be visible when using the sprite color instead of an image, which doesn't feel like the main feature.
If a user want more accurate color, it is possible with the Mesh2dBundle instead of the SpriteBundle

fn setup(
    mut commands: Commands,
    mut meshes: ResMut<Assets<Mesh>>,
    mut materials: ResMut<Assets<ColorMaterial>>,
) {
    commands.spawn_bundle(OrthographicCameraBundle::new_2d());

    commands.spawn_bundle(SpriteBundle {
        transform: Transform::from_xyz(50.0, 0.0, 0.0),
        sprite: Sprite {
            color: BG_COLOR,
            custom_size: Some(Vec2::new(100.0, 100.0)),
            ..Default::default()
        },
        ..Default::default()
    });

    commands.spawn_bundle(MaterialMesh2dBundle {
        mesh: meshes.add(Mesh::from(shape::Quad::default())).into(),
        transform: Transform::default()
            .with_scale(Vec3::splat(100.0))
            .with_translation(Vec3::new(-50.0, 0.0, 0.0)),
        material: materials.add(ColorMaterial::from(BG_COLOR)),
        ..default()
    });
}

the MaterialMesh2dBundle will have the expected color, SpriteBundle won't


Optimize table detection

Table detection (currently not yet in master, but in the tabledetection branch) takes about 2 sec / page. This needs to be sped up.
There is probably some low-hanging fruit wrt optimizing jruby_column_guesser.rb. For instance, we could probably use lower-res images for computer vision without a significant loss in accuracy. Also, the thumbnails generated in generate_thumbails.rb should be used for CV rather than generating our own. Probably other stuff too, especially the rectangle finding algorithm, could be dramatically sped up.
If there's a reliable way to determine how many cores the computer Tabula's running in has, let n be the number of cores, then we could run n-1 (because n would be mean) jruby_column_guesser threads to speed up processing, as @stucka recommends in #63.
See #63 for additional discussion.

Closed. This depends on some way old code that used JavaCV, since that has long been factored out, this is obsolete.


Cardano node provider

Hey Cardano Team!
Nice to meet you!
We're NOWNodes - the blockchain-as-a-service solution that lets users get access to full Nodes and blockbook Explorers via API.
Also, we provide nodes for Cardano.
Is it possible to make a small mention of our service in it? It can be useful for some of your users, who looking for a private blockchain node.
Thank you :)

Hello, Polina! Nice to meet you too :) Please consider adding the details to this community content section:
https://docs.cardano.org/community/community-content
As an alternative, you can create an article about NOWNodes on Essential Cardano (https://www.essentialcardano.io/), which is a community-driven resource.
Thank you kindly

Hey Olga,
thanks for info
I'll check :)


fix 🐛

What does this PR do?

FIx bugs in new CLI changes

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


http://localhost:8772

Hi - What is the purpose of this pull request? It is completely unmergable in its current format.


Migration tutorials to v1.1.1

This PR adds two tutorials for migrating to v1.1.1 from v1.0.0.
@darthsiroftardis, I tried to create the PR against "dev", but "dev" doesn't seem to be updated with the latest release changes. Please suggest the branch best suited for these files.

@gRoussac, thanks for the review! I've added a section explaining NamedKeys pre-migration. Also, I believe I addressed your comments.

Feedback addressed. Thanks @darthsiroftardis !

@darthsiroftardis, this PR needs another approval since changes were requested.


interpret, useInterpret etc should return an Actor

Description
Writing the v5 docs, and it's making me think that interpret should return an Actor-compatible API ONLY. We have a slightly strange terminology when it comes to:

Actors - in v4, we usually think of actors as something created by spawn. An actor ref is what's held in context.
Services - again, in v4 we think of them as something created using invoke.
Interpreters - something that's returned from interpret or useInterpret. To confuse things, we sometimes refer to this as a service.

Really, they're all referring to similar things - a running process that can send and receive events. We've had internal discussions to unify this language, but after an internal discussion with @Andarist he raised an interesting point:
What if we just made interpret return an Actor?
Andarist pointed out that the Interpreter type returned from interpret extends an Actor, and it doesn't really have that many other useful things on it. So if we return an Actor from interpret, we can condense the docs and teach less lingo.
Expected result
Not applicable
Actual result
Not applicable
Reproduction
Not applicable
Additional context
No response

I agree! Just got bitten by this when I was trying to type a function that took the output of useInterpret as an argument and returns some ActorRef from within it. I tried to use ActorRefFrom but then realised that wasn't the right type. It definitely feels a little convoluted and if it could be simplified that would be awesome!

Resolved by #3455


Adding Tables from Multiple Apps to Admin Panel

My piccolo FastApi project has three apps, home, accounts and flights.
Each app has a table_finder function that imports all tables from the tables.py file.
I've been unable to add all tables from all the apps to  the root app.py create_admin(
tables=
)
The tables parameter seems to only accept one app's tables.py.  Trying add tables of more than one app
as in
create_admin(
tables=[]
)
or create_admin(
tables=()
)
throws errors

My piccolo FastApi project has three apps, home, accounts and flights. Each app has a table_finder function that imports all tables from the tables.py file.
I've been unable to add all tables from all the apps to the root app.py create_admin( tables= ) The tables parameter seems to only accept one app's tables.py. Trying add tables of more than one app as in

I can't reproduce your problem. You have to import the table from app like this
from home.tables import Task
from flight.tables import Flight
from accounts.tables import User

app = FastAPI(
    routes=[
        Mount(
            "/admin/",
            create_admin(
                tables=[Task, Flight, User],
            ),
        ),

and everything works as it should.

create_admin( tables=[] )
or create_admin( tables=() )
throws errors

create_admin argument tables is list type not tuple and that's the way you get error.
Hope this helps.

Alright thanks a lot. I guess I've to bypass the table_finder helper function to be able to do this. Because the table finder_finder returns a list of tables so when it's from multiple apps it becomes a list of lists as in , tables=[]

No problem, can you please close the issue.

Rather than import the tables manually what I've now done is to unpack each app list this way
tables=[
*app_a.table_classes, *app_b.table_classes, *app_c.table_classes
]
Thanks for your help.


Migrate to go mod AND use go 1.14.1

dupe of https://github.com/justwatchcom/elasticsearch_exporter/pull/350 ?

Yep. Dup.