ionic run android --prod flag is not changing the build dev

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
After upgrading to app-script 1.0.0 ionic v2 final, prod flag is not producing the "prod" build with "ionic build android --prod"
What behavior are you expecting?
current output on terminal ;
command executed : ionic build android --prod
Actual :
ionic-app-scripts build
[01:40:53]  ionic-app-scripts 1.0.0
[01:40:54]  build dev started ...
[01:40:54]  clean started ...
[01:40:54]  clean finished in 3 ms
[01:40:54]  copy started ...
[01:40:54]  transpile started ...
[01:40:58]  transpile finished in 4.61 s
[01:40:58]  webpack started ...
[01:40:58]  copy finished in 4.92 s
[01:41:10]  webpack finished in 11.67 s
[01:41:10]  sass started ...
[01:41:12]  sass finished in 1.76 s
[01:41:12]  build dev finished in 18.07 s
Expected:
ionic-app-scripts build
[01:40:53]  ionic-app-scripts 1.0.0
[01:40:54]  build prod started ...
[01:40:54]  clean started ...
[01:40:54]  clean finished in 3 ms
[01:40:54]  copy started ...
[01:40:54]  transpile started ...
[01:40:58]  transpile finished in 4.61 s
[01:40:58]  webpack started ...
[01:40:58]  copy finished in 4.92 s
[01:41:10]  webpack finished in 11.67 s
[01:41:10]  sass started ...
[01:41:12]  sass finished in 1.76 s
[01:41:12]  build prod finished in 18.07 s
Steps to reproduce:
1.
2.
3.
main.ts file :

import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';

import { AppModule } from './app.module';
import {enableProdMode} from '@angular/core';

enableProdMode();
platformBrowserDynamic().bootstrapModule(AppModule);

tried also :

import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';

import { AppModule } from './app.module';
//import {enableProdMode} from '@angular/core';

//enableProdMode();
platformBrowserDynamic().bootstrapModule(AppModule);

Which @ionic/app-scripts version are you using?
ionic info :
Cordova CLI: 6.3.1
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.1.8
Ionic App Lib Version: 2.1.4
Ionic App Scripts Version: 1.0.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: OS X El Capitan
Node Version: v6.7.0
Xcode version: Xcode 8.2.1
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

Try it with npm run ionic:build --prod. That should do the prod build
My guess is you need to update to the latest Ionic CLI.
npm uninstall -g ionic
npm install -g ionic@latest

Sudo may or may not be required depending on your set-up.
Then ionic run android --prod should work.
Thanks,
Dan

HI Dan,
I tried as you suggested :

npm run ionic:build --prod , this did not work.

so as from my package.json structure :
it worked with , npm run build --prod instead. Am I missing any configuration? since even after doing uninstall and reinstalling of ionic@latest did not resolve this.
ionic run android --prod still not working.
package.json
{
"name": "Cuppa",
"author": "Chitrang Natu",
"homepage": "http://cuppa.com/",
"private": true,
"scripts": {
"build": "ionic-app-scripts build",
"watch": "ionic-app-scripts watch",
"serve:before": "watch",
"emulate:before": "build",
"deploy:before": "build",
"build:before": "build",
"run:before": "build"
},
"dependencies": {
"@angular/common": "2.2.1",
"@angular/compiler": "2.2.1",
"@angular/compiler-cli": "2.2.1",
"@angular/core": "2.2.1",
"@angular/forms": "2.2.1",
"@angular/http": "2.2.1",
"@angular/platform-browser": "2.2.1",
"@angular/platform-browser-dynamic": "2.2.1",
"@angular/platform-server": "2.2.1",
"@ionic/storage": "1.1.7",
"@types/lodash": "4.14.43",
"angular2-moment": "^1.0.0",
"angularfire2": "^2.0.0-beta.7.1-pre",
"chart.js": "2.3.0",
"es6-shim": "0.35.0",
"ionic": "^2.2.1",
"ionic-angular": "2.0.0",
"ionic-native": "2.2.11",
"ionic2-rating": "0.0.10",
"ionicons": "3.0.0",
"reflect-metadata": "^0.1.8",
"rxjs": "^5.0.0-beta.12",
"sw-toolbox": "3.4.0",
"systemjs": "0.19.39",
"typings": "^1.4.0",
"whatwg-fetch": "^2.0.1",
"zone.js": "^0.6.21"
},
"devDependencies": {
"@ionic/app-scripts": "1.0.0",
"@types/request": "0.0.30",
"typescript": "^2.0.3"
},
"cordovaPlugins": [
"cordova-plugin-device",
"cordova-plugin-console",
"cordova-plugin-camera",
"cordova-plugin-compat",
"cordova-plugin-crop",
"cordova-plugin-file",
"cordova-sms-plugin",
"cordova-plugin-whitelist",
"cordova-plugin-splashscreen",
"cordova-plugin-statusbar",
"ionic-plugin-keyboard",
{
"variables": {
"SENDER_ID": "XXXXXXXXXX"
},
"locator": "phonegap-plugin-push",
"id": "phonegap-plugin-push"
}
],
"cordovaPlatforms": [
"ios",
{
"platform": "ios",
"version": "",
"locator": "ios"
},
{
"platform": "android",
"version": "",
"locator": "android"
}
],
"description": "Cuppa : A business coffee APP"
}

Your package.json scripts section should look like this:
https://github.com/driftyco/ionic2-app-base/blob/master/package.json#L6-L11

@danbucholtz  thank you the command worked!
But now I am getting this error :
Uncaught Error: Cannot find module "./app.module.ngfactory"
main.ts file :
import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
import { AppModule } from './app.module';
platformBrowserDynamic().bootstrapModule(AppModule);
Please advice.

I removed node_modules and did npm install then all worked! Thank you for this amazing framework and your support.

@danbucholtz I'm having the same issue, upgrading to the latest version of ionic did not work. However, npm run ionic:build android --prod --release is working. But I can't find the .apk in the android/build/outputs folder.
[10:32:03]  ionic-app-scripts 1.1.4
[10:32:03]  build prod started ...
[10:32:03]  clean started ...
[10:32:03]  clean finished in 2 ms
[10:32:03]  copy started ...
[10:32:03]  ngc started ...
[10:32:13]  ngc finished in 9.62 s
[10:32:13]  preprocess started ...
[10:32:13]  optimization started ...
[10:32:13]  copy finished in 9.86 s
[10:32:24]  optimization finished in 11.08 s
[10:32:24]  preprocess finished in 11.08 s
[10:32:24]  webpack started ...
[10:32:44]  webpack finished in 20.18 s
[10:32:44]  sass started ...
[10:32:44]  transpile bundle started ...
[10:32:56]  transpile bundle finished in 11.47 s
[10:32:56]  uglifyjs started ...
[10:32:57]  sass finished in 12.26 s
[10:32:57]  cleancss started ...
[10:32:58]  cleancss finished in 990 ms
[10:33:15]  uglifyjs finished in 18.76 s
[10:33:15]  postprocess started ...
[10:33:15]  postprocess finished in 2 ms
[10:33:15]  lint started ...
[10:33:15]  build prod finished in 71.18 s

The issue of --prod not working even when typed into the command exists also on Ionic Package, don't know if related.

@kamok,
Sorry, I'm not sure what's going on. The @ionic/app-scripts build is working correctly. Can you provide some more info? I would think the APK would be there. Can you try updating cordova perhaps?
Thanks,
Dan

@danbucholtz I fixed it. It was a bug in ionic-cli that just required me to update ionic. npm update ionic didn't work, so I ended up removing npm and got a new npm via Homebrew. Works now.

The problem is with the "page-name.component.ts" file that is created when you use the "ionic generate page" command.
Create your pages manually and run it (ionic run android --prod) or (ionic run android --prod) again.
it worked for me.
Home they fix it soon.


[Enhancement][Entity Classifier]: Enhance Entity Classifier using words context (Presidio)

Description:  By using context words, it increases result score by matching the text coming from a specific column or a specific user input.
PR: #204

Closing this issue as this is fixed in #204 and #210


Testing Introduction topic added

Lighter weight intro to Testing

@alandefreitas Do the test matrix, and boost.config, sections work?

@alandefreitas all your suggestions included, and squashed.


Call plugin_loaded on target modules

From my investigations into https://github.com/Varriount/NimLime/pull/121 , I've found that AutomaticPackageReloader doesn't appear to call the plugin_loaded function on target modules. This function needs to be called, as several plugins perform necessary startup actions there.

This line should execuate the plugin_loaded function, afaik.

The reload_plugin function only calls plugin_loaded on the target module - not on any of the module's dependencies.
The normal sublime text plugin loading logic loops through sys.modules and calls the plugin_loaded function (if the module has such a function).

(External) dependencies are not part of the package and should not be reloaded. When you modify those, you should probably del sys.modules['dep_name'] (and probably all submodules as well) in the console.

@randy3k The modules that need plugin_loaded called are the same ones that are being reloaded. The work of finding and reloading them is already being done.
I've been able to patch AutomaticPackageReloader on my system to call plugin_loaded by using this patch:
175c176,179
<                 return module.__loader__.load_module(name)
---
>                 result = module.__loader__.load_module(name)
>                 if "plugin_loaded" in result.__dict__:
>                     result.plugin_loaded()
>                 return result

Although this code isn't quite right either - the modules loaded via reload_plugin have their plugin_loaded function being called twice.

I would argue that NimLime should not modify the python search path for this modules. You should put a file in the root directory and import all the submodules to the root. In this way, it avoids having multiple plugin_loaded definitions and calls.
See for example how it is done in Terminus: https://github.com/randy3k/Terminus/blob/master/main.py

I guess you should also refactor this part of the codebase https://github.com/Varriount/NimLime/blob/restructure-for-st3/core/commands/init.py#L31 and import the submodules relatively. Why do you need this file at the first place?

@randy3k That code loops through all submodules and exports the Sublime Text plugin classes. That way, when a new command or event listener is added to the commands directory, it is automatically picked up by Sublime Text via the star import in NimLime.py.
The import flow process looks something like this:
NimLime.py -- Star Import --> core/commands/__init__.py -- Dynamic Import --> core/commands/*.py

Is this causing some sort of problem with AutomaticPackageReloader? The dynamic import process uses the same import functions that Sublime's sublime_text.py plugin loader uses.

Is this issue live, or does #20 resolve it?


[Coverage] Use a MapVector to store coverage maps in a module, NFC

Using a MapVector allows coverage maps to be printed in a deterministic
order, and to be queried by function name. This makes an assertion
cheaper.
Depends on https://github.com/apple/swift/pull/15835.

@swift-ci Please smoke test and merge

@swift-ci Please clean and smoke test OS X platform

@swift-ci Please clean smoke test OS X platform


foreign_keys and exit options do not overwrite color as noted in README

Hello everyone,
there's a sentence in the README that describes that foreign_keys and exit configuration options actually overwrite the behavior specified by a color. At least I could not get it working.
What is my problem?
 However, since DAP stepping/continuing is asynchronous, I have to use a pink hydra. When using a pink hydra and specifying { foreign_keys = 'warn', exit = true }, the behavior of the hydra does not change, i.e. it still processes inputs that are not mapped to a head.
What would solve my problem?

option 1: I can create a pink hydra and specify foreign_keys and exit and the configuration actually changes the behavior of the hydra
option 2: I can create a amaranth hydra that works with the asynchronous events of DAP (I tried the trick with { on_key = function() vim.wait(20) end, color = 'amaranth' } that is used in the proposed git hydra, however that does not work as expected with the DAP since when i use dap.continue() it actually pastes some text into the popup created by dap.continue()

Example code:
    function cmd(command, suffix)
        suffix = suffix or ''
        return '<cmd>' .. command .. '<CR>' .. suffix
    end

    hydra({
        name = 'Debugging',
        config = { color = 'pink', foreign_keys = 'warn', exit = true },
        hint = hints.dap,
        mode = 'n',
        body = '<Leader>d',
        heads = {
            { 'b',     cmd('DapToggleBreakpoint') },
            { 'c',     cmd('DapContinue') },
            { 's',     cmd('DapStepOver') },
            { 'i',     cmd('DapStepInto') },
            { 'o',     cmd('DapStepOut') },
            { 'r',     cmd('DapToggleRepl') },
            { 'u',     function() require('dapui').toggle({ reset = true }) end },
            { 'w',     function() require('dapui').elements.watches.add() end },
            { '<Esc>', nil,                                                     { exit = true, nowait = true, desc = false }, },
        },
    })

https://github.com/benlubas/hydra.nvim/blob/eb68396f4f3185e000d64711419c71f67b23bb1d/lua/hydra/init.lua#L126
It seems like there's an inconsistency between the docs and the code here.

going back to when that code was written, the docs said the same thing. Guess it just slipped through at the time and hasn't been caught since.
However, I'm not sure fixing this will even fix the problem you're having. color and exit/foreign_keys do have to be reconciled one way or the other, so you currently can't have a Pink hydra that behaves like an Amaranth one, that would just be made into an Amaranth hydra.

ahh okay, makes sense that it does not work how I expect it to if the documentation is inconsistent.
Okay, if I understood you correctly: with the current implementation it is basically not possible to have an -- I'll use a different terminology here -- async hydra (pink) that uses foreign_keys = 'warn'?

Yeah currently impossible. I'd be open to changing it though (at the very least I will update the docs). I'm not sure how hard it would be to do though.

I mean sure, I am keen on this feature, so I'd be glad if you tried it.

I think I'm missing something. The docs don't seem to say anything about asynchrony. In my understanding, there should be no difference at all between a pink hydra with { foreign_keys = 'warn', exit = true } and an amaranth hydra.
What do colors have to do with asynchrony?
I really like this plugin as a concept, and am trying to make it work for me, but I've encountered a lot of bugs that make this really hard.
For example, red heads only trigger when i exit hyrda. That is, with the config below, to print "hmm A", I would have to pres <leader>' to enter hydra, then a (but the command won't be executed yet), and then esc -- and only then will "hmm A" be printed. Changing color to pink fixes that.
require("hydra")({
        name = "[HYDRA TEST]",
        -- hint = hint,
        config = {
          color = "pink",
          invoke_on_body = true,
          foreign_keys = "run",
          hint = {
            type = "statusline",
          },
        },
        mode = { "n" },
        body = "<leader>'",
        heads = {
          { "a", "<cmd>echo 'hmm A'<CR>", { private = true, desc = "previous" } },
          { "b", "<cmd>echo 'hmm B'<CR>", { private = true, desc = "next" } },
          { "c", "<cmd>echo 'hmm C'<CR>", { private = true, desc = "delete" } },
          { "d", "<cmd>echo 'hmm D'<CR>", { private = true, desc = "tabnew" } },
          { "<Esc>", nil, { private = true, exit = true, nowait = true, desc = "exit" } },
        },
      })

I don't see any feature described in the readme that would explain this behavior. But after seeing you talk about async colors, I'm starting to think those are not actually bugs, and I'm just missing a huge chunk of information about this plugin...


CPKAFKA-1054: Introduce batch expiry config

This is a cherry-pick from 0.11.0 to 1.0 as KIP-91 will
only land in 1.1.0 or later. We should not merge this
to trunk.
We solve the problem of expired batches not being retried
by providing the a separate config confluent.batch.expiry.ms
which controls how long to wait before batches are expired.
Before this patch, we would expire batches once
request.timeout.ms elapsed. By setting the new config
value to be very high, we ensure that batches are never expired,
which has the same effect as being retried indefinitely.

Code freeze is on Wednesday so we should merge this ASAP cc @hachikuji @apurvam.


Just one invalid latitude/longitude causes the entire map to fail to load

I had data with nan strings for some columns. This resulted in the following error:
leaflet.js:5 Uncaught (in promise) Error: Invalid LatLng object: (nan, nan)
    at new j (leaflet.js:5)
    at Object.W [as latLng] (leaflet.js:5)
    at datasette-cluster-map.js:36
    at Array.forEach (<anonymous>)
    at datasette-cluster-map.js:32

Fix released in 0.9.


Fixed generation of unnecessary phi-node in gvn pre for getelementptr

Is it a cherry-pick? Can we reflect it in the commit message?

Is it a cherry-pick? Can we reflect it in the commit message?

Added in message:
"cherry-pick from upstream with stripPointerCasts added".


Add placeholder prop

Add placeholder prop
Export component props

closes #43

Great work @drac94. I appreciate it :)


Update Packages, same version on columns

Hello,
just a little error about version compare, it does the same thing for any available package
here the example screenshot

thanks

This is a known bug which will be fixed in the next release. The patch can be seen here : https://github.com/webmin/webmin/commit/c0a037b06b0d7a684e259a980edb2d0e511851d3


Set up automatic release on PyPi

We should update the test_and_deploy.yaml to support automatic releases of the plugin through PyPi. For that, we should change the packaging inLine 115 to use
python -m build .
`´`

rather than the use of bdist (`python setup.py sdist bdist_wheel`). We need an API token of the repos owner on PyPi, though.

Thanks for adding a link to a page where the process is explained. let me just add another potentially useful resource: https://medium.com/@VersuS_/automate-pypi-releases-with-github-actions-4c5a9cfe947d

I was trying to see if I can add this functionality but it seems that I cannot look at the repository settings since I am not the owner of the repository (I'm guessing @lazigu or @haesleinhuepf is?). Can this access be changed?

@Cryaaa I have access to the repository secrets but there's no API token there. It has to be generated by someone who has access to release process on PyPi.

I made you admin @Cryaaa

Cool. I'll send a PR to update the github action for release to the latest standard and @Cryaaa can pass me the API token to be added as a repository secret. The release on github  should then trigger the automated  upload to PyPi.

Cool. I'll send a PR to update the github action for release to the latest standard and @Cryaaa can pass me the API token to be added as a repository secret. The release on github should then trigger the automated upload to PyPi.

Thanks for taking care of this! I’ll get you the API key on Monday because I’m quite busy this weekend hence my inactivity the past few days..

@jo-mueller, I have just added the PYPI API token as a secret to the repository so now most things should be in place! If you need me to take care of anything else please let me know!


feat(avm): Bytecode parsing and proof generation

Resolves #3791

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 6dac6504 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,412
179,524
716,068

l1_rollup_calldata_gas
221,644
866,776
3,448,096

l1_rollup_execution_gas
314,063
984,448
3,667,685

l2_block_processing_time_in_ms
1,591 (+14%)
5,305 (-2%)
20,935 (+1%)

note_successful_decrypting_time_in_ms
286 (-1%)
840 (-5%)
3,266 (+2%)

note_trial_decrypting_time_in_ms
96.0 (-4%)
68.2 (-39%)
140 (-5%)

l2_block_building_time_in_ms
16,114
63,924
255,846

l2_block_rollup_simulation_time_in_ms
11,734 (-1%)
46,609
186,603

l2_block_public_tx_process_time_in_ms
4,346
17,225
68,947

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,815 (-7%)
30,277 (-2%)

note_history_successful_decrypting_time_in_ms
1,950 (-3%)
3,816 (-1%)

note_history_trial_decrypting_time_in_ms
126 (-26%)
228 (-3%)

node_database_size_in_bytes
3,519,451
3,552,092

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
213 (-1%)
44,681
25,561

private-kernel-ordering
165 (-1%)
43,753
14,809

base-rollup
1,197
128,338
881

root-rollup
79.1
4,088
677

private-kernel-inner
291
71,208
25,561

public-kernel-private-input
206
31,863
25,561

public-kernel-non-first-iteration
203
31,905
25,561

merge-rollup
7.69 (+1%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,455
32,223

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
376 (+1%)
1,083

Metric
1 public data writes

tx_sequencer_processing_time_ms
536


Fix Use tableId as the credential for CCR syncer instead of tableName

Bug:
In the current version, CCR’s getBinlog and BeginTxn operations both depend on tableName, which will cause syncer to fail when the source cluster changes the table name.
Bugfix:

Add tableId in TGetBinlogRequest.
Add a tableId field in TGetBinlogRequest, and when the tableId field is set, the table field will be invalid. If the tableId field is unset, the source cluster will find the corresponding tableId based on this tableName, and this operation should only be used when initializing syncer.
change tableName to tableId in TBeginTxnRequest.

run buildall


Multiple Schedule objects from one task-file

Description
It would be very helpful to have a possibility to return multiple different Schedule objects from one task-file.
One possible usage scenario is external schedule source for Schedules.
Possible implementations are simple array or a Schedule-collection object.
Example
use Crunz\Schedule;
//...
$schedules = []; //  new ScheduleArrayObject();
forearch ($externalDefinitions as $definition) {
  $schedule = new Schedule();
  // configure $schedule
  // ...

  $schedules[] = $schedule;
}

return $schedules;

Hello, You can do it already, but with multiple events:
$commands = [
    [
        'command' => 'php -v',
        'cron' => '* * * * *',
    ],
    [
        'command' => 'php -v',
        'cron' => '* * * * *',
    ],
];

$schedule = new Schedule();
foreach ($commands as $command) {
    $task = $schedule->run($command['command']);
    $task->cron($command['cron']);
}

return $schedule;

Hello, You can do it already, but with multiple events:
$commands = [
    [
        'command' => 'php -v',
        'cron' => '* * * * *',
    ],
    [
        'command' => 'php -v',
        'cron' => '* * * * *',
    ],
];

$schedule = new Schedule();
foreach ($commands as $command) {
    $task = $schedule->run($command['command']);
    $task->cron($command['cron']);
}

return $schedule;

Hello.
This doesn't work with preventOverlapping().
Only the first event is executed.

@ivankuraev could You post some code to reproduce?

$schedule = new Schedule();

$onDemandTasks = Tasks::instance()->getActive('ondemand');
foreach ($onDemandTasks as $onDemandTask) {
    $task = $schedule->run(function() use ($onDemandTask) {
        switch ($onDemandTask->command) {
            case 'cmd1':
                ToolsOne::instance()->run();
                break;
            case 'cmd2':
                ToolsTwo::instance()->run();
                break;
            // .....
        }
    });

    $task
        ->everyMinute()
        ->description($onDemandTask->title)
        ->preventOverlapping()
        ->appendOutputTo(sprintf('logs/ondemand_%s.log', $onDemandTask->command))
        ->after(function() use ($onDemandTask) {
            Tasks::instance()->toggleActiveStatus($onDemandTask->id);
        })
    ;
}

return $schedule;

It's a little problematic to give complete working code to run it.
There are tasks that can be launched periodically.
If we have, for example, 3 tasks to run.
When using preventOverlapping(), only one task is performed.
If you don't specify preventOverlapping(), all three are executed simultaneously.

It happens because both commands generate same hash for preventOverlapping, try not using closure.


Filter out requests when popping from request storage

Requests are filtered before being added to the request storage, so as to discard irrelevant pages.
When crawling large sites, some filtering rules may be added after crawling is started. It usually involves updating the filters and updating (by restarting, or hot code reloading) the spider (assuming we're using a persistent storage backend).
In this case, some requests saved before the rules' update will be browsed anyway. This would be nice to find a way to filter them out also when popping.

So this feature request would involve purging the request storage when performing your live crawl?
I think exposing purge functionality is possible, so that one could call it manually when the spider is reloaded for example.
Let me know your thoughts.

I think it would rather involve filtering after popping a link to crawl.
Use case is:

Store millions of links
Update the rules
Relaunch the crawl process and discard rule that don't obey the new rules

But I don't now if Crawly has persistent backend store.
That said I don't think this is very much needed for now, so I'd suggest closing the issue for now and reopen if needed.


Check nix files

The syntax of nix files can be checked with nix-instantiate --parse file.nix . The output in case of failures will be something like:
error: syntax error, unexpected LET, expecting ':' or '@', at /home/skreft/code/issue127/default.nix:12:1

I'm not using nix anymore, so I will close this requirement at the moment.


lbrynet: init at 0.9.1

Motivation for this change
https://lbry.io/news/20000-illegal-college-lectures-rescued
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

This has been inactive for some time so I am closing this.


core: Expand docs on rolling upgrades

Background: https://airtable.com/tblD3oZPLJgGhCmch/viwdV1sM5KEK2L14K/recKby3c0SLGQtDj7

@mberhault, we already have basic documentation on rolling upgrades, but during testing, if you identify details/tips that should be exposed externally, please lets us know.

Also, as @sploiselle mentions in https://github.com/cockroachdb/docs/issues/1182, we may want an asciicast showing taking node down, upgrading binary, rejoining cluster, as well as docs on automating rolling upgrades.

@mberhault, this is marked as "done" in AirTable. Do you have any insights/tips that should be added to our docs?

I think somewhere between steps 3) and 4) we should mention to wait for the node to stop, either by looking at the list of running processes (eg: ps aux | grep cockroach) or looking in the log files for server drained and shutdown completed


chore(travis): configured doc upload

This will upload to the pages of my fork, but could easily be adapted
to work for the source directory.
Notes
An example of the produced result can be found here. The good thing about this setup is that pushing to master will automatically rebuild the API documentation.
This PR cannot be merged directly, as the secure environment variable needs to be adjusted to work with this repository.
Details on how to accomplish this can be found here - please skip forward to 'Giving Travis Permissions'.

Awesome. Just realised I missed the after_success section.
Thanks again!


Fix: Missing model annotation after migration

For master branch

@kengio You can close this ticket
We will rebase master on 2-6-stable soon, that's why we don't need to duplicate PRs

@kengio You can close this ticket
We will rebase master on 2-6-stable soon, that's why we don't need to duplicate PRs


ERROR:spotipy.client:HTTP Error for GET to api.spotify

I tried to download a Song but I got an Error Message.
`C:\Users\XXX\Desktop\spotify-downloader>spotdl --song https://open.spotify.com/episode/4zTF88zwxoVyXeWTGklRzp
WARNING:spotipy.client:Expected id of type track but found type episode https://open.spotify.com/episode/4zTF88zwxoVyXeWTGklRzp
ERROR:spotipy.client:HTTP Error for GET to https://api.spotify.com/v1/tracks/4zTF88zwxoVyXeWTGklRzp returned 404 due to non existing id
Traceback (most recent call last):
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\spotify_tools.py", line 34, in wrapper
assert spotify
AssertionError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotipy\client.py", line 170, in _internal_call
response.raise_for_status()
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\requests\models.py", line 940, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.spotify.com/v1/tracks/4zTF88zwxoVyXeWTGklRzp
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\runpy.py", line 193, in _run_module_as_main
return _run_code(code, main_globals, None,
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "C:\Users\XXX\AppData\Local\Programs\Python\Python38-32\Scripts\spotdl.exe_main.py", line 9, in
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\spotdl.py", line 68, in main
match_args()
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\spotdl.py", line 27, in match_args
track_dl = downloader.Downloader(raw_song=track)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\downloader.py", line 98, in init
self.content, self.meta_tags = youtube_tools.match_video_and_metadata(raw_song)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\youtube_tools.py", line 82, in match_video_and_metadata
meta_tags = spotify_tools.generate_metadata(track)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\spotify_tools.py", line 39, in wrapper
return func(*args, **kwargs)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotdl\spotify_tools.py", line 50, in generate_metadata
meta_tags = spotify.track(raw_song)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotipy\client.py", line 252, in track
return self._get("tracks/" + trid)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotipy\client.py", line 205, in _get
return self._internal_call("GET", url, payload, kwargs)
File "c:\users\XXX\appdata\local\programs\python\python38-32\lib\site-packages\spotipy\client.py", line 181, in _internal_call
raise SpotifyException(
spotipy.exceptions.SpotifyException: http status: 404, code:-1 - https://api.spotify.com/v1/tracks/4zTF88zwxoVyXeWTGklRzp:
non existing id
`

Maybe the problem is that its not a "track" but an "episode"?

Yep, that's likely it. spotdl currently does not support episodes and
podcasts.

Ok thank you for your fast response!


Updated the Output Property

Elaborated on the output property and briefly described the path module and what it is used for.
describe your changes...

[ ] Read and sign the CLA. PRs that haven't signed it won't be accepted.
[ ] Make sure your PR complies with the writer's guide.
[ ] Review the diff carefully as sometimes this can reveal issues.
Remove these instructions from your PR as they are for your eyes only.

Thanks!


stdin only works once?

Im doing something wrong here...it seems to not receive anything after the first RCVD-msg:

this is the responsible code:

As you can see, I tried to remove linebreaks, as console.log already produces a linebreak (would result in two linebreaks).
However, that's not fixing it :/
Running the script on the commandline works fine:
$ qjs app.js

solved!
turns out I had to call std.out.flush().
The C99 example revealed this :)
I made a PR so quickjs users will have an easier headstart: https://github.com/joewalnes/websocketd/pull/396

BOOM: https://github.com/coderofsalvation/WQJS


Improve insert_word and insert_identity

This PR, based on notes from PR-Nfa compose, includes:

Modifications to methods nfa::Nfa::insert_word(), nft::Nft::insert_word(), and nft::Nft::insert_word_by_parts(), which now have an optional target state (default Limits::max_state).

If the target state is not specified, a new state will be created.
These methods return the target state.

An overloaded method nft::Nft::insert_identity() that can create identity transitions for each symbol in a given vector.

We cannot use Limits::max_state. That would allocate Delta for way too many elements. We need to probably have two overloaded, functions. One with the normal parameters without target state which will inside the function default to nfa.num_of_states(), and another with explicit target state parameter (the original function).


'Killed' elements remain in the DOM

I notice that when notifications fade out, they do that via the 'killed' class which sets their opacity to 0. But I don't see anything in the code which removes those elements from the DOM. I think invisible notifications will accumulate indefinitely, bloating the DOM. Seems like there should be a periodic mechanism to cull out these old elements, removing the from the DOM entirely.

Nevermind. I see now where the element is removed.


How to link result from terminal of 'R interactive' and terminal "zsh (already run radian)"

Hi all.

This is what I am wanting to do. I ran "chunk code" and it ran on Terminal of 'R ibteactive' however I want my results to sync to terminal "zsh".
Can someone help me?
Thank you so much

If you need to send code to "active terminal" instead of the R terminal created by vscode-R, then you need to enable r.alwaysUseActiveTerminal in vscode settings so that your code in active editor will be sent to active terminal.

If you need to send code to "active terminal" instead of the R terminal created by vscode-R, then you need to enable r.alwaysUseActiveTerminal in vscode settings so that your code in active editor will be sent to active terminal.

Thank you so much.
I just did a few operations but for some reason this message is showing and I can't find the Terminal Dialog. Can you help me?

Would you like to post your r.* entries in vscode settings?

Hi renkun-ken
I would like sent to you my vscode setting
{
"editor.minimap.enabled": false,
"codegpt.apiKey": "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",
"[python]": {
"editor.formatOnType": true
},
"github.copilot.enable": {
"": true,
"plaintext": true,
"markdown": false,
"scminput": false,
"quarto": true
},
"genieai.enableConversationHistory": true,
"genieai.personalizedName": "Geni",
"CodeGPT.maxTokens": 2000,
"CodeGPT.temperature": 0.5,
"security.workspace.trust.untrustedFiles": "open",
"github.copilot.advanced": (),
"r.alwaysUseActiveTerminal": true,
"editor.inlineSuggest.enabled": true,
"git.openRepositoryInParentFolders": "never",
"terminal.integrated.defaultProfile.osx": "zsh",
"r.plot.defaults.colorTheme": "vscode",
"workbench.iconTheme": "vscode-icons",
"jupyter.askForKernelRestart": false,
"workbench.colorTheme": "Monokai Dimmed",
"r.rterm.mac": "/Users/rchai/opt/anaconda3/bin/radian",
"files.associations": {
".rmd": "markdown"
}
}
pls, Help me.
Thank you so much


Proposal Creation: center each step to the middle of the screen

Summary
As a DAO user I would like to see all each step in the middle of the screen
@evgenibir check the example, i think it visible most to the last step, i think you should have fixed length from previous steps because steps that are longer in size when moved to the next one take more space.

cc @luigicarlini1988

I think it would be better to align "current steps" at top (50px - 80px
margin top) rather then this "almost centered" way. Especially for smaller
screens (11 - 13 - 14 inches laptop display)
On Tue, 27 Dec 2022 at 22:11, arsenijesavic @.***>
wrote:

Summary
As a DAO user I would like to see all each step in the middle of the screen
https://user-images.githubusercontent.com/7712798/209722692-d3cfaa3e-bad9-439f-b60e-9d048638cd55.mov
cc @luigicarlini1988 https://github.com/luigicarlini1988
—
Reply to this email directly, view it on GitHub
https://github.com/hypha-dao/dho-web-client/issues/1918, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJPFDNHSXGFONMTVZXYP3LWPNLRBANCNFSM6AAAAAATKXAQMU
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Luigi Carlini


Add null-safe helper functions with ujson.Value#{str,num,obj,arr,bool}

e.g. .strOpt: Option[String], which would return None if the thing is ujson.Null

Using upickle 0.8.0:
My expectation was document("xyz").strOpt would now return None if the key does not exist at all in the document. Instead it blows up on me with an Unexpected exception[NoSuchElementException: key not found: xyz
Currently I have to surround any key access that could go wrong with a try/catch, wouldn't it be nice if this case was also handled safely as an Option?

@nemoo  maybe this val newStartLine = document.obj.get("xyz").map(_.str) will work around
.


Macro: When adding new nucleotides to the beginning of a row, the order of the chains changes in the Sequence mode

Steps to Reproduce

Switch to the Macro mode - Sequence mode
Add any four chains to the canvas (for example, AAA, TTT, UUU, CCC)

At the very beginning of the second chain, enter U

Actual behavior
If you add at least one new nucleotide at the very beginning of the second or any subsequent chain (but not the first or the last), the chains randomly change places (the second takes the place of the third, and the third takes the place of the fourth)
Also,
The same behavior is observed when at the very beginning of the chain we insert a copied fragment.

Expected behavior
When adding new nucleotides at the very beginning of any chain, the chains are not swapped

Desktop (please complete the following information):

OS: Windows 10 Pro (version 22H2) x64
Browser: Сhrome
Version: (122.0.6261.112) x64

Ketcher version 2.20.0-rc.1
Issue found while testing https://github.com/epam/ketcher/issues/3650
Screenshots
https://github.com/epam/ketcher/assets/139227567/7a34a874-473d-4135-a44b-de0467a74cf5

Test added to the automation table.

Test automated.

Verified.

Indigo Toolkit Version 1.21.0-dev.3.0-g6f55c664-wasm32-wasm-clang-12.0.0
Ketcher Version 2.21.0-rc.2 Build at 2024-05-20; 05:30:45
Chrome Version 124.0.6367.201 (Official Build) (64-bit)
Windows 10


CharacterBody3D process_mode reenabled while colliding Area3D leads to Error "p_elem->root is true"

Godot version
v4.0.beta.custom_build.b29bb11a4
System information
Ubuntu 20.04.3 LTS, Vulkan, TU116 [GeForce GTX 1660 Ti], nvidia 470.161.03
Issue description
If you reactivate a disabled CharacterBody3D (process_mode = DISABLED) with process_mode = INHERIT while the body is inside an area the following error appears:
ERROR: Condition "p_elem->_root" is true.
   at: add (./core/templates/self_list.h:46)

I did some (unfinished) debugging and figured out some kind of backtrace:

GodotBody3D::set_space()
GodotCollisionObject3D::_set_space()
GodotCollisionObject3D::_update_shapes()
GodotBroadPhase3DBVH::create()
BVH_Manager::create()
BVH_Manager::_check_for_collisions()
BVH_Manager::_collide()
GodotSpace3D::_broadphase_pair()
GodotAreaPair3D::GodotAreaPair3D()
p_body->set_active(true); First time adding the item to the list
GodotBody3D::set_space() Second time adding the item to the list

Steps to reproduce

Create a new project
Add a CharacterBody3D with a collision shape
Add an Area3D with a collision shape
Ensure that both objects collide
Use anything to set the CharacterBody.process_mode = Node.PROCESS_MODE_DISABLED
Use anything to set the CharacterBody.process_mode = Node.PROCESS_MODE_INHERIT

Minimal reproduction project
SelfListBugCharacterBody3D.zip

Same issue in 2D. I tried to create a minimal repro by killing some character (= make it invisible and set process mode to disable) via a hitbox and resurrecting it (= make it visible and set process mode to enabled), but the error didn't reappear.
I'll try to find the middle ground for a minimal repro that doesn't need the full animation stuff, but I'm not sure what I really need. Hopefully the 3D example will help you find how to fix it in 2D.
(alternatively, I can also provide the full project if needed, after I swap sprites for non-licensed ones)

OK, I managed to extend your bug to 2D, and also tested the case of physics process where the conditions to trigger the issues are different, and where it's easy to fix the error in one line.
I could not manage to reproduce the duplicate physics body in the MWE, only in my actual game, and even fixing the "p_elem->root is true" error doesn't avoid duplication anyway, so both seem unrelated. Therefore I'll open a new issue for the duplication.
I will rewrite repro steps for the new project I created from yours, which combines 2D and 3D scenes and scripts:
After disabling a CharacterBody2D/3D inside an Area2D/3D with a proper collision shape, if you reenable ityou may get the error "Condition "p_elem->_root" is true."

if this happens on a timeout signal, error occurs anyway
if this happens during a process/physics process, error can be avoided with await get_tree().physics_frame, await get_tree().process_frame, await get_tree().create_timer(0.1).timeout before re-enabling, or moving the node at any time in that frame

Other types of PhysicsBody2D/3D nodes don't seem to trigger the error.
Simple callstacks:
2D

E 0:00:01:0934   BaseCharacter.gd:161 @ restart(): Condition "p_elem->_root" is true.
<C++ Source>   ./core/templates/self_list.h:46 @ add()
 BaseCharacter.gd:161 @ restart()
Level.gd:7 @ _unhandled_input()

3D

E 0:00:00:0587   Test3D.gd:17 @ _on_timer_timeout(): Condition "p_elem->_root" is true.
<C++ Source>   ./core/templates/self_list.h:46 @ add()
 Test3D.gd:17 @ _on_timer_timeout()

Repro steps:

Open Test2D/3D.tscn. Make sure that Test2D/3D root node > Test With Physics Process is unchecked so it works on timer
Play the scene
Observe error in Output after 0.1s
Stop game, and this time, check Test2D/3D root node > Test With Physics Process is unchecked so it works on process
Play the scene
Observe error spam in Output
Repeat steps after uncommenting some of the lines in Test2D/3D.gd. Note how _on_timer_timeout always causes an error, while _physics_process can be fixed with any line.

2D/3D hybrid project:
Issue 70848 - SelfListBugCharacterBody3D-2D hybrid.zip


Having an issue with the installer

https://user-images.githubusercontent.com/71162577/133853819-471c7c80-eca3-4263-8564-d3d97d7a3f86.mp4
Im having an issue, so BD dissapeared off my discord, so i go and Download the latest installer, i click on the exe, loading arrow, then nothing. I look at task manager, it opens the installer in the background for 2 secs, then auto closes, how do I fix this? I tried to change permissions and run as admin and it just starts up in the bg then closes in the bg and the speed of light

7-Zip fixed the issue


Replace literals with K.epsilon()

CuDNN BN requires epsilon to be at least 1e-5, TF simply sets it so, don't know about CNTK, but Theano complains. Currently, all backends use 1e-3, leaving this value for consistency.

Review comments addressed.


Update FHIRFunctions to not hardcode name of next event

User Story

As a step in the Universal Pipeline,
I want to be able to put QueueMessages on any pipeline step instead of hardcoding the pipeline step that comes after,
so that the Universal Pipeline as a whole can support a less rigid architecture.
Description/Use Case

Presently, a UP function can generate one or more queue messages which bubble up to the FHIRFunctions.do<step> functions, which all follow this pattern:
    internal fun doRoute(
        message: String,
        dequeueCount: Int,
        fhirEngine: FHIRRouter,
        actionHistory: ActionHistory = ActionHistory(TaskAction.route),
    ) {
        val messagesToDispatch = runFhirEngine(message, dequeueCount, fhirEngine, actionHistory)
        messagesToDispatch.forEach {
            queueAccess.sendMessage(
                elrTranslationQueueName,
                it.serialize()
            )
        }
    }

The part of this pattern that is not common is the first param of the sendMessage call, which is a hardcoded queue name. In the case of the route function, every QueueMessage it generates will be sent out to the translation queue. Instead, we should somehow deduce the name of the queue from the concrete type of the QueueMessage.
This ticket was created following the update to the translation function in #13018 where it can now generate a ReportQueueMessage instead of no QueueMessages (the default case for translate is it will upload the blob and update DB and then wait for batchdecider to pick it up. When Topic.isSendOriginal is enabled, we want to skip the batch and directly call send).
Risks/Impacts/Considerations

Dev Notes

Acceptance Criteria

[ ] FHIRFunctions no longer hardcode which queue the QueueMessage gets sent to

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jack-h-wang @jalbinson @JessicaWNava @JFU-NAVA-PBC @jimmyfagan @mkalish @thetaurean

Please add your planning poker estimate with Zenhub @brick-green

notes from convo with @arnejduranovic re: clarification of this ticket

davidholiday
:monkey:  22 minutes ago
I get the ask but there are some specifics I need some clarity on
davidholiday
:monkey:  21 minutes ago
if we're not "hard coding" message routing from one pipeline segment to the next there are two alternatives that come to mind
configuration based routing
context based routing - as in - something about the report itself or what happened prior or some other constellation of factors at that time drives the logic determining where the message goes next
davidholiday
:monkey:  20 minutes ago
I'm assuming y'all are looking for (2) - but let's gain agreement on that. Is that what y'all want?
davidholiday
:monkey:  20 minutes ago
next question - what's determining where a message goes in terms of business logic ?
davidholiday
:monkey:  19 minutes ago
that is - assuming I'm right and we're talking about (2) and not (1), what conditions specifically are determining whether or not the "default" behavior of the pipeline is short circuited?
davidholiday
:monkey:  18 minutes ago
lastly - my read of this is that the ask is NOT to actually short circuit routing so much as creating the means for us to do so in future. is that correct?
davidholiday
:monkey:  18 minutes ago
ty in advance for clarifying :slightly_smiling_face: (edited)
Arnej Duranović
13 minutes ago
Good questions, David! I'll do my best to explain down below and let me know if you have additional comments or questions.
Arnej Duranović
4 minutes ago
As the ticket explains, we would like to somehow deduce the name of the queue from the concrete type of the QueueMessage., which I think falls under option TWO you have above. Take a look at doRoute in FHIRFunctions.kt. Each Universal Pipeline specific step follows this pattern and has a similar function in this file. These doX functions are essentially the entry-points for these universal pipeline steps. The first thing they do is call runFhirEngine, which does the appropriate step's business logic. In the case of the route step, this is in FHIRRouter. Notice toward the end of FHIRRouter.doWork we are returning a FHIREngineResult that contains a FhirTranslateQueueMessage! This queue message is what is going to get put up in the azure queue and what is going to trigger the Translate function to run. The goal of this ticket is to use this, or maybe other, information to deduce the first parameter of queueAccess.sendMessage( or maybe we want to eliminate the first param all-together.
Arnej Duranović
4 minutes ago
If this doesn't make sense I think it'll be easier to hop o na call and walk through it quick
davidholiday
:monkey:  2 minutes ago
no I think I get it. basically the result of processing gives us a queuemessage and the type of the message indicates where that message has to go next
davidholiday
:monkey:  1 minute ago
so the work - fundamentally - is to make changes such that pipeline step work always returns a typed queue message so the last action taken in the current pipeline stage knows where to plop the queue message for further processing (edited)

attempting to add the necessary behavior to the queue message itself. this way the message knows how to route itself. moreover the logic can be short circuited if need be. not sure if this is going to work w/o creating a mess. if it looks like it's going to be a problem I'll eject and do this the naive way by creating a method in FhirFunctions that does what I'm trying to make the queuemessage itself do

I've added behavior to all the queue message types that result in routing to have the message itself self-route. there are two failing tests. I'm not sure yet but I think the tests are at fault but I'm not sure. This is gonna take some time to sort out as all of this is a bit meticulous

of the two tests that were failing, one required a minor refactor as it was endeavoring to put a "route" typed message on the "translation" queue. The other test was failing because an update was required in my logic. PR is up.

more revision requests.
branch became stale to the point where merging master into it broke things
it became cleaner/easier to just redo the whole thing. which I did.
here's the PR
https://github.com/CDCgov/prime-reportstream/pull/15613


use AMDGPU and CUDA code tabs in GPU episode

so that material can be taught on either NVIDIA or AMD GPUs

fixed in #32


remove external_type_uuid macro

Objective

Macro external_type_uuid seems unused
https://docs.rs/bevy/latest/bevy/reflect/macro.external_type_uuid.html

Solution

Remove it and see if it was? There is a derive for the same trait that is used everywhere (#[derive(TypeUuid)]) and is a better api

bors r+


[C# generation] Format is not respected on array items

Have the following swagger spec:
"MyObject": {
  "FormatAsDate": {
    "type": "string",
    "format": "date"
  },
  "CollectionWithItemsAsDate": {
    "type": "array",
    "items": {
      "type": "string",
      "format": "date"
    }
  }
}

Expected output:
public class MyObject
{
    [JsonConverter(typeof(DateJsonConverter))]
    public System.DateTime? FormatAsDate { get; set; }

    [JsonProperty(ItemConverterType = typeof(DateJsonConverter))]
    public IList<System.DateTime?> CollectionWithItemsAsDate { get; set; }
}

Actual output:
public class MyObject
{
    [JsonConverter(typeof(DateJsonConverter))]
    public System.DateTime? FormatAsDate { get; set; }

    public IList<System.DateTime?> CollectionWithItemsAsDate { get; set; }
}

The CollectionWithItemsAsDate array is missing the DateJsonConverter json converter.

Looks like same issue as https://github.com/Azure/autorest.azureresourceschema/issues/44 - what I observed is that there's no way in autorest to access pattern/format information associated with string-type array items - the codemodel structure doesn't allow for it.


[WIP] Added NSE monitoring to NSM

Description

Added NSE monitoring to NSM. If connection to NSE dies, NSE is getting deleted from the registry.
If connection to NSE is not getting alive in 10 seconds, NSE is getting deleted from the registry.
Motivation and Context

NSMD should periodically check NSE without connections are alive and update registry since field. #836
How Has This Been Tested?

[ ] Covered by existing integration testing
[x] Added integration testing to cover
[x] Tested locally
[ ] Have not tested

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I suppose this on is WIP and will be separated to few PRs. @Bolodya1997 could you please rename it to include WIP:

@haiodo +1

This PR is based on #1146 and #1148 and so comes after.

Depends on #1189.


docs: add Immutable Tags article

Fixes #164

What type of PR is this?

Which issue does this PR fix:
#164
What does this PR do / Why do we need it:
Easier to find content without searching
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

but it is already?


multiple platform-(db)-connections, MESSAGE-scheme v1.1

Please confirm that this PR has done the following:

[x] Tests Added
[x] Documentation Added
[x] Description in RELEASE_NOTES.md Added

Description
This PR introduces the option to connect to multiple databases (multiple instances of an ixmp.Platform ), with the option to clone across platforms. It also makes logging less noisy and adds an option to set the logger-level via Python.
Furthermore, it upgrades the MESSAGE-scheme default sets and parameters incorporating the changes made during the hackathon at IIASA in August 2018.

I guess this still needs to fix the migration issue? should we remove the V1.6 or somehow rename it?

It's probably easier to change the flyway-migration files in ixmp_source, since this has only touched some test databases for the scenario explorer. The flyway-checksum issue that makes CI fail is caused by platform-dependent line endings, still need to fix that...

Ok, want to ping me when it's ready to review/bring in?

please keep on hold until we discuss how to treat "drivers"...

we're still waiting for the emissions stuff here right?

yes, @gidden, still have to test the emissions implementation...

@gidden, any ideas why the notebook execution tests would fail only on AppVeyor and only on Python 3...? All tests pass on my work PC, which would actually be closest to this CI configuration...

No idea, trying to rebuild. Sometimes fishy things can happen.

initial test in #93. basically, I pulled back to the last successful commit (fc12854) to see if it still works. Let's diagnose after that goes through.

thanks @gidden for the assist! closes #93

Ok, this seems fine to me. I believe this is our final PR for v0.2.0. Feel free to let me know if this is not the case @danielhuppmann

I think so, @gidden! Nothing I can think right now of that still needs to be done urgently.


[AnVIL DX] Document data mappings

Approach

Create AnVIL Explorer Data Cross Walk spreadsheet containing tab per entity with the columns:

UI Column Name
Azul API Respons
TDR

Complete Column Name and Azul API Response Key columns for each entity by inspecting the config and code.
Include values returned by Azul with no corresponding UI column (UI Column Name will be empty for these values).

Hi @MillenniumFalconMechanic! Please see a completed spreadsheet matching these specs!

Hi @jpaten! Looks good, thank you! Two q's:

Would you be able to sort each tab by alpha?
Is the drs_uri on the files tab a top-level field? (It looks odd!)

Complete filter tab with headers currently present on the sheet

TODO: Sort unused api responses alphabetically (do not change api responses that have headers associated)

Sheet is now sorted as specified

Thanks Jonah!


python_by_example_issue7

Hey @jstac , the 3rd hint in "Exercise 3" could be modified so that it looks more consistent with the previous context.

Thanks @shlff , good idea.


Is there a way to repeatedly filter images?

func filterimage(insertImage: UIImage) -> UIImage {
let toonFilter = SmoothToonFilter()
let testImage = insertImage
let pictureInput = PictureInput(image:testImage)
let pictureOutput = PictureOutput()
pictureOutput.imageAvailableCallback = {image in
    // Do something with image
}
pictureInput --> toonFilter --> pictureOutput
pictureInput.processImage(synchronously:true)
}

I'm using this codes to filter still image but the problem is when I filter one image it is ok.
But when I filter image repeatedly the pictureOutput has the before filtered image in it.
For example.
let filteredImage1 = filterimage(insertImage: PicOne)
let filteredImage2 = filterimage(insertImage: PicTwo)

When I do this, filteredImage2 has the same image of filteredImage1.
and when I input the third image like this,
let filteredImage3 = filterimage(insertImage: PicThree)
this filteredImage3 has the image of filteredImage2
So I tried to use removeAllTargets(), pictureOutput.removeSourceAtIndex(0) after filtering one image but it has no change.
Can anyone help me..

func filterimage(insertImage: UIImage) -> UIImage {
let toonFilter = SmoothToonFilter()
let testImage = insertImage
let pictureInput = PictureInput(image:testImage)
let pictureOutput = PictureOutput()
pictureOutput.imageAvailableCallback = {image in
    // Do something with image
}
pictureInput --> toonFilter --> pictureOutput
pictureInput.processImage(synchronously:true)
}

I'm using this codes to filter still image but the problem is when I filter one image it is ok.
But when I filter image repeatedly the pictureOutput has the before filtered image in it.
For example.
let filteredImage1 = filterimage(insertImage: PicOne)
let filteredImage2 = filterimage(insertImage: PicTwo)

When I do this, filteredImage2 has the same image of filteredImage1.
and when I input the third image like this,
let filteredImage3 = filterimage(insertImage: PicThree)
this filteredImage3 has the image of filteredImage2
So I tried to use removeAllTargets(), pictureOutput.removeSourceAtIndex(0) after filtering one image but it has no change.
Can anyone help me..

Did you find the decision of the problem?

@Semty
Yep. But it's a half answer.
I was using a live View on the RenderView and changed a little code in gpuimage to get a full size of the .Photo preset. When I use the default func to capture photo it gets 750*1000 size.
Using capturePhoto I got a Original image. and I got that image and rendered it again for the filtered image.
But the problem was. I was using
camera --> filter --> renderView
and getting a live filtered image on the view.
But I called inputPhoto --> filter --> outputPhoto.
But when I call that I think it crashes with the renderView that is in live.
So I just made another renderview in hidden and I put camera --> filter --> renderView2
than I filtered image will live video is going on in renderView1


refactor(sequence): delete plugin and move to boot

移除 sequnce plugin ，将模块移动到 api - boot

Codecov Report

Merging #30 into 2.3.x will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           2.3.x     #30   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files        299     299
  Lines       3855    3855
  Branches     370     370
=====================================
  Misses      3855    3855

Impacted Files
Coverage Δ

...ure/sequence/ApiBootSequenceAutoConfiguration.java
0.00% <ø> (ø)

...autoconfigure/sequence/ApiBootSequenceContext.java
0.00% <ø> (ø)

...g/minbox/framework/api/boot/sequence/Sequence.java
0.00% <ø> (ø)

...inbox/framework/api/boot/sequence/SystemClock.java
0.00% <ø> (ø)


Do not assert when mutex fails.

Mutex acquisition failures shouldn't be asserted while adding vcs to queues as HttpSM isn't guaranteed to be on the original thread.

Remember to remove Milestone etc. when closing a PR (without merging)


Fix flash message css issue

Purpose
Fixes styling issue of the flash messages when viewing on devices with smaller screens. The issue was discovered as part of the Delete LPA ticket
Approach
Applied styling changes to ensure the flash message text starts on the same line as the svg icon. The changes also prevents the flash message and welsh language switch from overlapping when viewed on devices with smaller screens
Checklist

[x] I have performed a self-review of my own code
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

The flash message now displays correctly on the same line on all screen sizes


Concurrency hierarchy

class (Monad m, Functor t) <= MonadFork t m | m -> t where
  fork :: forall a. m a -> m (t a)
  join :: forall a. t a -> m a

class (MonadFork t m, MonadThrow e m) <= MonadKill e t m | m -> e t where
  kill :: forall a. e -> t a -> m Unit

class (MonadKill e t m, MonadError e m) <= MonadBracket e t m | m -> e t where
  bracket :: forall a b. m a -> (a -> m Unit) -> (a -> m b) -> m b

MonadFork has a fairly obvious law of fork >=> join = id, but I'm not sure what I'd state about the others.

 Some restrictions on ma might be needed.
Another law for MonadKill might state that for all a:
(attempt do
  f <- fork (pure a)
  kill e f
  join t) == Right a

MonadBracket would take a similar approach and say that the release action is always invoked, even when the body throws an exception (law 1), and even when the thread is killed (law 2). i.e. release is always invoked, whether the body completes normally, is killed from within, or is killed from without.

I think we could just add suspend to MonadFork, because there are useful laws in terms of each other. resume and join are the same thing.
class (Monad m, Functor t) <= MonadFork t m | m -> t where
  suspend :: forall a. m a -> m (t a)
  fork :: forall a. m a -> m (t a)
  join :: forall a. t a -> m a

join <=< suspend = id
fork = fork <<< join <=< suspend

---

class (MonadFork t m, MonadThrow e m) <= MonadKill e t m | m -> e t where
  kill :: forall a. e -> t a -> m Unit

(do t <- suspend (throwError e1)
    kill e2 t
    join t)
  = throwError e2

---

data BracketResult e
  = Killed e
  | Failed e
  | Completed

class (MonadKill e t m, MonadError e m) <= MonadBracket e t m | m -> e t where
  bracket :: forall a b. m a -> (BracketResult e -> a -> m Unit) -> (a -> m b) -> m b

-- Wave hands

Yes, you're right, sorry. I didn't intend to include resume in that version.

In any case, I don't we need attempt/try to state the laws for MonadKill.

Overall, I like it. 👍
What useful things, if any, could a user do with suspend?

suspend is useful when you want to preemptively "fork" a computation, but there may not be any consumers to observe it, in which case nothing gets run.

So for example, you could build a table of forked effects, but then only observe the results as needed. This means things are only computed on demand and then memoized (at least for Aff).

So then perhaps this should be a law:
(do t <- fork a
    _ <- join t
    join t)
  = fork a >>= join

The problem with that example is you can bypass the whole suspend / join merely by sequencing the result into the parent. Instead of storing the "suspended" effects, you just store the actions, and instead of joining them, you just run them. The effect is the same.

That's not true. With the above law (multiple joins don't run the effect more than once), it's very different. If you stored Aff actions, every time you dereferenced it, an effect would run. With suspend and join, effects are only run once as needed.

Actually, that example is poor. So you might be right 😆

The lazy loading example is useful if you want to share results among multiple consumers. The sequencing an effect thing only applies with a single consumer.

Yes, that's right, for more than one consumer, suspend achieves sharing. If there's no rush, let's think it over a day & make sure this is the best law-abiding formulation we can come up with.

We can forgo algebraic laws for the time being, and just state operational laws.

@natefaubion I think so. The best I came up with was specifying its behavior in the absence of interruptibility:
nonInterruptible $ bracket aff1 f g =
  nonInterruptible $ do
    a <- aff1
    e <- try (g a)
    f a (either Failed Completed e)
    either throwError pure

This is a guarantee that, assuming nothing is interrupted, the release action is always called so long as the acquire action succeeds.
I think stating more than that — that the release action is called if the body is interrupted — cannot be stated algebraically with this formulation, at least, not without a model of Aff. So operational laws are better than nothing and we can always improve this over time.

@jdegoes In that case, MonadBracket should just contain bracket for the time being?

@natefaubion Can you think of useful laws involving never?
There's some trivial stuff:
bracket (pure a) (\_ _ -> aff) pure == (nonInterruptible aff) *> pure a

bracket never f g == never

bracket (pure a) (\_ _ -> never) pure == never

Not sure how useful it is?

I don't think those never laws are very interesting. They would require nonInterruptible anyway, since the acquisition effect can't be killed once it is initiated. So I think if we are going to add another member, nonInterruptible is a more interesting candidate.
bracket a1 (const a2) pure = nonInterruptible (a1 >>= \a -> a2 a $> a)

I also think that nonInterruptible works as a member because it has a default implementation in terms of bracket.

That makes sense.
Are you sure the above law holds? What if body is interrupted? I know it's just pure but that doesn't mean it can't be interrupted.

I think it should be OK, but it would require more laws. If you kill something that is non-interruptible, the action is run, and the exception is deferred until it completes. So in that case, you'd end up with an exception at the end for that Fiber either way. I guess you could use never then to say:
do
  t <- fork (bracket a1 (const a2) never)
  kill e t
  join t
=
  nonInterruptible (a1 >>= a2) *> throwError e

Since fork is non-deterministic, unlike suspend, a1 must be initiated.

Thinking about this, I'm not sure we can say that fork >=> join or suspend >=> join are equivalent to id in the presence of cancellation. That would require killing a join to kill the subject fiber, which I don't think is desirable behavior.

@natefaubion I think we can say fork >=> join, because this says nothing about cancelation. If you literally have fork >=> join in your code, you can always replace that with id.

OK, that's fair. I was thinking of something similar to that where:
fork (fork aff >>= join) >>= kill e

Is not necessarily the same as:
fork aff >>= kill e

But this ascribes some sort of equivalence to the fork, but the point of fork is that it's non-deterministic.


Support configuring the bot from a Python module

This enables arbitrary Python code to configure
the bot (e.g. using environment variables).

I need this to be able to run on Heroku. I haven't submitted a PR for that yet, because I'd like to get this merged in first.

That Heroku work is really nice! I'd be happy to merge it and I appreciate it.
Keep in mind the original design of Teambot isn't too great, so it's all up for negotiation. I basically just pulled in rtmbot one 4th of July to make it work, and then progressively deleted things I didn't need to make it simpler over the last couple of years. I myself find the architecture of it a bit unintuitive.
Your project seems perfectly reasonable to me, but if it meets your requirements I'd rather just have something that pulls the slack token and DEBUG straight from environment variables, and either dispenses with the config file entirely or leaves it in for backwards compatibility. That way you get a nice 12-factor app without having to load another module.  Not that your module loading code is hard to understand, but we're just pulling in constants from that module so it's just an indirection.

This PR leaves it in for backwards compatibility. That was my original intent.
Usage of a Python module is triggered by one of:

python rtmbot.py -s settings.module.path
python rtmbot.py --settings settings.module.path
TEAMBOT_SETTINGS_MODULE=settings.module.path python rtmbot.py

Additionally, TEAMBOT_SETTINGS_MODULE=settings.a python rtmbot.py --settings settings.b will use settings.b (args trump env vars).
The original README is still technically correct, but I added here additional describing how to use a module path.

Yep I understand the intent, don't worry your code is pretty clear. I am
just wondering if there is any need for this indirection though a python
module, of if you'd be just as happy if I made a commitment where teambot
read the env vars directly. In other words, is there a use case I am
missing for importing settings from code?
On Mar 24, 2017 11:46 PM, "Ryan Balfanz"<EMAIL_ADDRESS>wrote:
This PR leaves it in for backwards compatibility. That was my original
intent.
Usage of a Python module is triggered by one of:

python rtmbot.py -s settings.module.path
python rtmbot.py --settings settings.module.path
TEAMBOT_SETTINGS_MODULE=settings.module.path python rtmbot.py

Additionally, TEAMBOT_SETTINGS_MODULE=settings.a python rtmbot.py
--settings settings.b will use settings.b (args trump env vars).
The original README is still technically correct, but I added here
additional describing how to use a module path.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tdeck/teambot/pull/6#issuecomment-289193261, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACALLubbv_S09bPW4g_a4064JPEoWHh1ks5rpLhUgaJpZM4MnsOA
.

Oh ok, I guess I probably don't understand what you're proposing. I guess if env vars were supported out of the box that would satisfy my Heroku requirements. But it sounds like you're suggesting removing the rtmbot.conf completely, which is of course not backward compatible. Some people might still like to have that. Am I misunderstanding?

Also, the settings from the module need not be constants. I think you may have suggested that before.

Oh I see. That was just an aside suggestion. I'm not really planning to do
that at first, although the migration path isn't unduly onerous I would
say. I will revisit tomorrow.
On Mar 24, 2017 11:58 PM, "Ryan Balfanz"<EMAIL_ADDRESS>wrote:

Oh ok, I guess I probably don't understand what you're proposing. I guess
if env vars were supported out of the box that would satisfy my Heroku
requirements. But it sounds like you're suggesting removing the
rtmbot.conf completely, which is of course not backward compatible. Some
people might still like to have that. Am I misunderstanding?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tdeck/teambot/pull/6#issuecomment-289193777, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACALLhTX3lsJ340iVKruFxiln5L1y2-Gks5rpLsdgaJpZM4MnsOA
.

Create a settings module and set TEAMBOT_SETTINGS_MODULE in your environment to the Python path of the module. Once simple way is to cp settings.py.example settings.py, then edit settings.py as needed.

Maybe this would be more clear if I also mentioned that instead of (or in addition to) an env var, you can swap the existing -c arg with a -s and give it a Python path to a module.
Does that help any?

Sorry I guess I must not be expressing myself very clearly. To put it bluntly, I don't like this design. The goal is to be able to read the config values from env vars - that makes sense and I think we should just do that. This thing of specifying a Python module path on the command line, dynamically loading that module, and pulling constants out of it, then having a separate module that sets the constants based on environment variables, feels overly convoluted and I don't see the benefit of doing it this way.
I think instead we should just do something like I have in this working branch:
https://github.com/tdeck/teambot/commit/58f242733787f97a46cffe20930c3811d8260cbe
However, I don't mean any disrespect to your work. I just believe that if we don't need t he flexibility of being able to load an arbitrary module, we shouldn't add it.

That's fine. Though I'm not sure what you mean by "then have a separate
module". There's only one module.
Yes, my only real ask if allowing it to be configured 100% from the
environment. I designed it this way to try and match the original design
(specifically, the args usage). But to enable the command to be run without
any modification at all (my intent was to not break existing installs or
completely rewrite your project) forced needing a way to cue the code to
load a module and not a .yml file. Actually, I hoped that's by keeping the
original usage intact you'd be more likely to approve my changes and
incorporate them. :-/
I'm on mobile now and can't take a good look at your branch now but I will.
If that work does what I need that's great. What will it take to finish
that work and get it merged? And when can I roughly expect it might be
completed?
I could absolutely use this tomorrow. But I can wait too. What can I do, if
anything, to help get this over the finish line in the next few days?
On Sat, Mar 25, 2017 at 11:02 PM Troy Deck<EMAIL_ADDRESS>wrote:

Sorry I guess I must not be expressing myself very clearly. To put it
bluntly, I don't like this design. The goal is to be able to read the
config values from env vars - that makes sense and I think we should just
do that. This thing of specifying a Python module path on the command line,
dynamically loading that module, and pulling constants out of it, then
having a separate module that sets the constants based on environment
variables, feels overly convoluted and I don't see the benefit of doing it
this way.
I think instead we should just do something like I have in this working
branch:
58f2427
https://github.com/tdeck/teambot/commit/58f242733787f97a46cffe20930c3811d8260cbe
However, I don't mean any disrespect to your work. I just believe that if
we don't need t he flexibility of being able to load an arbitrary module,
we shouldn't add it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tdeck/teambot/pull/6#issuecomment-289260721, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIInrFgBqPantl0r7uC6gkGFsOaESTUks5rpf93gaJpZM4MnsOA
.

Closing this since we ended up going a different route to support env vars. Thanks for writing it

@tdeck If I pulled out the Travis stuff from this branch into a new one, do you think you'd be open to merging it? That, plus the docker stuff, will help me make additional contributions in the future.

It would be good to have CI tests; I had thought about it but hadn't done it because it's going to be a real pain to mock out the API surface, but if you're willing to write some that would be awesome. I can help if I have time, in the near future I may not but I'll review your PR.


Saddle scale

As per our discussion: saved saddledata is not transformed while plotting is done with LogNorm.

Awesome! I think this is ready to merge. One last thing (something we should start doing): could you add a short line to the changelog describing this change?

Hope this description is fine?

Looks great!


[c2][encoder] Directly use Y-tiled surface as input for encoder

case: android.mediav2.cts.CodecEncoderSurfaceTest#
testSimpleEncodeFromSurface[|Native][_c2.intel..encoder_video/_c2.intel..decoder_video/*]
Resolve cases of Encountered frames with PSNR less than configured threshold 20.0dB involving c2.intel codecs.
Tracked-On: OAM-118400

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web  for more details.

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web  for more details.

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web  for more details. For Binaries: /cactus-absp-or-local/celadon-merge/502


Remove namelist and streams from cosine bell init

They are not needed and are causing trouble because they don't get cached.

Checklist

[x] Testing comment in the PR documents testing used to verify the changes

Testing
I ran the QU and Icos versions of the cosine bell test case on Chrysalis.  Both passed and don't contain the namelist.ocean files in their init directories, as expected.
I also ran with cached init using #18 and the analysis step works correctly, whereas it was failing with missing input files before this fix.

@cbegeman, could you give this a quick glance? No need to run any tests, just best to have another pair of eyes just in case I missed something obvious.

@cbegeman, Thanks for having a look!
From what I can see, the problem is still in compass.  it looks like namelist.init and streams.init themselves are still here:
https://github.com/MPAS-Dev/compass/tree/main/compass/ocean/tests/global_convergence/cosine_bell
and the analysis test still expects a namelist file as an input:
https://github.com/MPAS-Dev/compass/blob/main/compass/ocean/tests/global_convergence/cosine_bell/analysis.py#L47-L49
So we'll probably need to fix that in compass, too.

@xylar Hmm, that was unintentional. I opened a fixup PR here https://github.com/MPAS-Dev/compass/pull/540.


New Deployments ?

Hello.
Love the project. First issue. Not necessarily code related.
I noticed there has been some activity on the master branch, but the hosted app on https://prose.io/ hasn't picked those up.
The gh-pages branch hasn't had a bump since 2018.
This probably has to do with Travis-CI being all wonky and not free now.
Would a github action file/script help ?

Hey there @thiagohersan. I've been trying to work on getting the project back into motion. The PR above will move prose over to gh-pages and restart CD. If you can give that a thumbs up hopefully the maintainers will see and merge.
In the meantime, I'm currently hosting my own fork with some bugfixes and features I've worked on @ joebabbitt.com/prosa. Since I want to avoid standing up my own gatekeeper instance and keep everything client-side, I've enabled login via github PAR token. (You'll need to create a token with the repo and user scopes to login).
Hopefully that helps!

This is wonderful.
I meant to make a github action script, but since I never heard anything from anyone here I just kind of forgot about it.
Thanks so much.

Closing this as #1142 has merged and deployments are confirmed working at prose.io :partying_face:

Wonderful. Thanks a bunch ! 🎉⚡🥂


Do not crash riak node if vnode status file is unreadable.

If for any reason the vnode status file is unreadable with
file:consult/1 the vnode would start, fail to consult, crash until the
restart limit was hit and the node would fail to start/crash.

+1 d32be75

Thanks  @mrallen1, but I want to know why the vnode status file becomes unreadable?

It interprets the vnode status file as improper Unicode since that's the default file encoding in 17+

Note, there is another PR needed for erlang 17+ that attempts to consult a pre 17 file as pre 17 encoding. @tburghart worked on it, but it's not needed until we support 17.


Are you planning to share the pretrained model weights?

Are you planning to share the pretrained model weights?
Thank you.

As the weights are huge I didn't upload it, but if you mailed me, i can share!
Thank you


Update root directory.

i have rename all the project name serially like
for 1st project => 01.project_name
and write some code for root index.html file. by main root web page i.e root index. html file , i have link all the project.
so that we can host this project by using github pages.
added style.css file on root directory to give some style to root web page.
by clicking the link main_web_page.

Great job @TechnicalAmanjeet


Fix empty batch nested updates.

What does the pull request do?
While working on #8263, we found this error: when running an empty batch update of >1 value which causes new, empty batch update to run in a PropertyChanged handler triggered from the first batch update, an AvaloniaInternalException was thrown.
Turns out that the invalid state that was being checked for by this exception was not actually invalid - it can happen on a nested empty back update, so just remove the check.

You can test this PR using the following package version. 0.10.999-cibuild0021560-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


feat: add privacy policy page

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #7089
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary
copilot:summary
copilot:poem

@floyd-li anchors not working

should work after #7112 merged

😍 Perfect, I’ve sent the payout
💵 $50 @ 4.27 USD/DOT ~ 11.71 $DOT
🧗 13rv1SWoLg9Gb3tmvHPZxb7JbVy51BtMziX7k9WQGSJ7Kp3A
🔗 0xc24b0b36e60f7a5fa9f5b8397fd6c8b1c1790be7f57f3adcc7a6fc29d8a0b7be
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


improve logging for adb execute

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.138%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
355

Relevant Lines:
427

💛  - Coveralls


Allow user specified django project

This PR fixes #206.
Description of changes

pypi.yml
Remove settings.py and wsgi.py in stave package.
READEME.md
Update stave load to stave load-samples.
stave_cli.py
Add a new configurable argument django_settings_module.
Add interactive interface for user to set up and initialize config of stave.
stave_config.py
Define default configs.
Provide a set of interfaces to access/update stave configuration.
stave_viewer.py
Update the logic of configuring django.

Possible influences of this PR

If an older version of stave was installed before, ~/.stave needs to be removed to avoid potential conflicts (e.g., KeyError).
When users choose to set django_settings_module, they need to make sure the module is a path to a valid django project and settings.py is properly configured. Otherwise there may be undefined behaviors.

Demonstration of results

Uninstall previous version of stave and remove the ~/.stave folder:

rm -r ~/.stave

Run and try different stave commands as specified in README.md. You may either
(1) Skip all the initial configs to accept default django settings; or
(2) Set django_settings_module to a settings.py file from a properly configured django project. Refer to https://github.com/asyml/stave/blob/master/simple-backend/stave_backend/settings.py for more details.

pip3 install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple stave_test

pip3 install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple stave_test

since tests-pypi is used for testing, you don't need to use a separate stave_test repo


Errors for some reason.

Traceback (most recent call last):
  File "D:\anaconda3\envs\TTS\lib\site-packages\trainer\trainer.py", line 1833, in fit
    self._fit()
  File "D:\anaconda3\envs\TTS\lib\site-packages\trainer\trainer.py", line 1785, in _fit
    self.train_epoch()
  File "D:\anaconda3\envs\TTS\lib\site-packages\trainer\trainer.py", line 1503, in train_epoch
    for cur_step, batch in enumerate(self.train_loader):
  File "D:\anaconda3\envs\TTS\lib\site-packages\torch\utils\data\dataloader.py", line 631, in __next__
    data = self._next_data()
  File "D:\anaconda3\envs\TTS\lib\site-packages\torch\utils\data\dataloader.py", line 1346, in _next_data
    return self._process_data(data)
  File "D:\anaconda3\envs\TTS\lib\site-packages\torch\utils\data\dataloader.py", line 1372, in _process_data
    data.reraise()
  File "D:\anaconda3\envs\TTS\lib\site-packages\torch\_utils.py", line 722, in reraise
    raise exception
RuntimeError: Caught RuntimeError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "D:\anaconda3\envs\TTS\lib\site-packages\torch\utils\data\_utils\worker.py", line 308, in _worker_loop
    data = fetcher.fetch(index)
  File "D:\anaconda3\envs\TTS\lib\site-packages\torch\utils\data\_utils\fetch.py", line 54, in fetch
    return self.collate_fn(data)
  File "D:\AI\Testing\TTS\TTS\tts\models\vits.py", line 343, in collate_fn
    wav_padded[i, :, : wav.size(1)] = torch.FloatTensor(wav)
RuntimeError: The expanded size of the tensor (1) must match the existing size (2) at non-singleton dimension 0.  Target sizes: [1, 44982].  Tensor sizes: [2, 44982]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\AI\Testing\TTS\train_vits.py", line 91, in <module>
    main()
  File "D:\AI\Testing\TTS\train_vits.py", line 85, in main
    trainer.fit()
  File "D:\anaconda3\envs\TTS\lib\site-packages\trainer\trainer.py", line 1860, in fit
    remove_experiment_folder(self.output_path)
  File "D:\anaconda3\envs\TTS\lib\site-packages\trainer\generic_utils.py", line 77, in remove_experiment_folder
    fs.rm(experiment_path, recursive=True)
  File "D:\anaconda3\envs\TTS\lib\site-packages\fsspec\implementations\local.py", line 185, in rm
    shutil.rmtree(p)
  File "D:\anaconda3\envs\TTS\lib\shutil.py", line 750, in rmtree
    return _rmtree_unsafe(path, onerror)
  File "D:\anaconda3\envs\TTS\lib\shutil.py", line 620, in _rmtree_unsafe
    onerror(os.unlink, fullname, sys.exc_info())
  File "D:\anaconda3\envs\TTS\lib\shutil.py", line 618, in _rmtree_unsafe
    os.unlink(fullname)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'D:/AI/Testing/TTS/vits_premier1-April-23-2024_09+49PM-dbf1a08a\\trainer_0_log.txt'

I lack sufficient Python understanding to solve this issue myself.

Oh right, the file. train_vits_win.py.

I've run into the same problem with "_The process cannot access the file because it is being used by another process" while doing some tests on Windows, but i'm not sure how i solved it. But this happens when training already crashed and the process tries cleaning up the training folder.

Did you run your command line with admin privilieges?
Is there any file open from the training directory in explorer?

The issue I'm having is primarily the RuntimeError: The expanded size of the tensor (1) must match the existing size (2) at non-singleton dimension 0.  Target sizes: [1, 44982].  Tensor sizes: [2, 44982] issue. I don't understand what this is.

Sorry for late response. Anything new here or still same error?

I close this issue, but feel free to reopen it at any time if needed.

Still same.


Admit Child - Data save failing if model is not valid

If a web user fills incorrect data in any mandatory field on any tab (ex Parent tab) and press submit, if model state is not valid, error is shown next to the field having incorrect data. User is not able to Submit form from the same tab. If user goes to next tab, say Fee and tries to submit, the application does not check for ModelState on other tabs and tries to save it. This submit call fails showing oops! message.
Suggested solution - We can check all the model states of each tab when user clicks on submit button on any tab and disallow the user to save data in case any of the model state is invalid.

@manujshanker I think this was wrongly posted here. I'm open to discussion specific to duster.net but I'll be closing this.


Suggestion: Improve bullet points

I notice that bullet points are displayed weird. Not sure why this happened. It would be great they be like in create-react-app docs

@0xGabi Do you mean that they're not indented one level (the first level is aligned with the rest of the text)? I think this might be something from docusaurus cc @delfipolito
E.g.

I styled them that way (aligned with the rest of the text) based on Adri's new design for the developer portal. But I can talk with him to change it :)

Oh I'm most likely wrong here, generally I am. I thought it was something related with rendering

@delfipolito having a bit of an indent might make it easier to read, but let's ask @owisixseven what he thinks :)


Any chance to move zip_file() rule to a standalone repository?

The zip_file() rule is extremely useful for releasing packages, and there is high demand for that. See https://github.com/bazelbuild/bazel/issues/3768, https://github.com/bazelbuild/bazel/issues/2336, and  discussions.
I'd like to use this rule in my project, but don't want to depend on the whole nomulus repository. It would be great that the rule is moved to a standalone repo.

The best place to put a zip rule would now be http://github.com/bazelbuild/rules_pkg.


How to make animated background bop to beat?

What is your question?
So I have made an animated background character, but they just constantly loop their idle and it looks horrible. Does anybody know how to make their animation play onStepHit so that they bop to the beat?

 I forgot the end
2.- He said that he want to do with the step hit
3.- Ok then if he dont want that the background play just don't put object play anim, just put the function on the code of the making background (if is possible)

Is this on stage lua code or song lua code?
If it's on stage, the first thing you need to do is remove close(), if you have it. The rest is the same.
Alright, first things first. On your addAnimationByPrefix, change the boolean at the end to false. That boolean controls whether the animation is looped or not.
Next is pretty simple, like they said.
function onBeatHit()
    playAnim('[whatever the first value of makeAnimatedLuaSprite was]', '[animation name]', true);
end


Create google.yml

Why:

What's being changed:

Check off the following:

[x] All of the tests are passing.
[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

lashaunda - Overview
lashaunda - Overview
secret_scanning


Feature/animation rigging support

TICKETID
Description

Briefly describe what this change will do

How to Test

Add steps to locally test these changes

Checklist

[ ] Tests written or updated for the changes.
[ ] Documentation is updated.
[ ] Changelog is updated.

Yo as this is a bit out of date you probably need to reupdate with the latest.
Also note that I have a PR that moves all the top toolbar/menu options under Tools/Ready Player Me sub category as per the Unity Asset store requirements


Is there a way to move the logs folder of the Portable App to a different target?

Hello.
tl;dr
Is there any way to move the SyncTrayzorPortable-x64/logs-directory to some other location (like %AppData%/SyncTrayzor/log)?
Long version:
SyncTrayzor.exe version <IP_ADDRESS>
Syncthing.exe version 0.13.10
Windows 10 version Prio N 1511 build 10586.420
I've read about the difference between the setup and portable versions of SyncTrayzor and decided for the portable (x64) version. It has been installed in directory that is synced with Microsoft OneDrive. In SyncTrayzors settings the  was changed to "C:\Users<me>\AppData\Local\Syncthing".
That way I hope(d) to retain the binary and it's settings while having all temporary data on the local machine, way outside whatever OneDrive is syncing.
Then I learned that SyncTrayzor is storing some temporary data in
SyncTrayzorPortable-x64/data/cef/cache/...
SyncTrayzorPortable-x64/logs/... as well as in
SyncTrayzorPortable-x64/debug.log.
Now these files change quite frequently, thus triggering OneDrive again and again *1. Is there a way (maybe start parameters or registry entries to make) that would allow me to shift as much of that data to some other location as well?
Thanks for programming and sharing SyncTrayzor in the first place. :)
Thanks for reading this.
Please excuse all späling errors and wöird sentence structure.
Have a nice day.
Wolf
1 Why this should be a problem? OneDrive is a sync engine and should cope with that, shouldn't it? Well, yes is should. OD has this peculiar behaviour of sinking its teeth firmly into a file that keeps changing a lot, by that ignoring all the other files that accumulate over time and would like to be synced as well. This becomes a minor annoyance as it forces me to stop SyncTrayzor now and then, restart OneDrive, wait for OD to do its thing and restart SyncTrayzor, at which point OD is completely going apesht again... There is no automatism anymore and I can never be sure the oneline version of OD is synchronized with the local version.]

Have a look at the Path configuration section of SyncTrayzor.exe.config. You can use env vars in there if you want.
Note that upgrades will overwrite this file (it's intended to be edited only be system administrators who are creating their own deployment packages for their machines).
Also be aware that syncing Syncthing's home directory (which is also stored in data/) is a very bad idea: it's device id is stored in there, as well as its index, neither of which should be the same on any two of your machines.

It might be easier to use the installed version of SyncTrayzor, and edit SyncTrayzor.exe.config to locate it's configuration file in a OD folder. The auto-upgrade mechanism is pretty smooth, so I wouldn't worry about synchronising binaries. In fact, there aren't really my options on the configuration file which you should need to change after an initial setup, so I'm not sure i'd even synchronise that.

Thanks for the quick response.
Syncthings home dir has already been moved and does not show up in .../data.
The original post should have said:

In SyncTrayzors settings < config.xml > the < SyncthingCustomHomePath > was changed to "C:\Users\AppData\Local\Syncthing".

Unfortunately, the < > were stripped with their contents. I'm sorry.

My bad, I'm in the middle of something else, and missed that.

It might be easier to use the installed version of SyncTrayzor...

I agree. My initial understanding was that the portable version would be self-contained whereas the setup version would have dependencies and such. Wrong assumption.
Since the portable version is already installed I experimented with the SyncTrayzor.exe.config as advised and it worked like a charm. Oh if only everything in life would be that easy. THANK. YOU (esp. for thinking about this configuration options in the first place.)
Just for documentations' sake, what has been changed:
<LogFilePath>%LocalAppData%\SyncTrayzor\logs</LogFilePath >
<CefCachePath>%LocalAppData%\SyncTrayzor\data\cef\cache</CefCachePath>
<SyncthingCustomHomePath>%LocalAppData%\Syncthing</SyncthingCustomHomePath>

and some other options that could've been configured in the GUI as well.
SyncTrayzor creates all necessary dirs and starts working without a hiccup.
The only file left in the binaries-dir is the debug.log, with which OD can hopefully live. It seems not to change too often.

I agree. My initial understanding was that the portable version would be self-contained whereas the setup version would have dependencies and such. Wrong assumption.

Yeah, they're both self-contained in that the installer / portable zip contains everything you need (apart from the .NET framework, which only the installer will install). The intention with the portable version is that you can installer it to e.g. a USB stick, so everything - log files, config files, etc - is stored in the one directory.


DNF_FebTest

Hi @selvabrillio, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla-ppe.dotnetfoundation.org.

TTYL, DNFBOT;

Hi @selvabrillio, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla-ppe.dotnetfoundation.org.

TTYL, DNFBOT;

@selvabrillio, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;