Somehow widgets seem to interfere with each other

If more than 1 WidgetLoader-based widget is loaded, the following error sometimes appears on either one of them:
Call Script.setWidget() to set content of the widget

I though that rolling-up the components with iife would ensure that no global state is shared. But I get the feeling it does..
Not sure how to debug this also :)

Seems to happen after I've called widget.presentSmall() while it's also a widget on the homescreen.
Steps to reproduce:

visit https://scriptable-ts-boilerplate.vercel.app/ / (or local)

Add Stickynote widget to Scriptable and to home screen

Add COVID note widget to Scriptable and to home screen

Run Stickynote from within Scriptable
Result: Scriptable shows preview in app

Go to home screen
Result: The widget now shows the above error.

Must have something to do then with the following code:
https://github.com/jasperhartong/scriptable-ts-boilerplate/blob/eaf977cdbe7923a7df3fb4a9702feca09c4149ae/scriptable-api/public/compiled-widgets/widgetLoader.js#L71-L76
Tried:

removing the await
making it an If ... else (so either calling presentSmall OR setWidget)

https://github.com/jasperhartong/scriptable-ts-boilerplate/commit/ab5da494377f3c91f2707ceb318eedbf98b0e182 fixes this (should've created a PR :P )

Nope.. still there 🤯 So not the root issue.


Proposal to add Cognito as an IAM service

Cognito is not only used for mobile services. It can also be used for other application such as web applications. Cognito can handle unlimited number of identities and give access to AWS resources.

Let's add this mapping..


treewide: sha256 -> hash for fetchCrate

Only causes a rebuild of rure and its dependents, as that's the only package that actually gets updated here.
Part of #325892
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Could you split rure change into another commit?

Split out, and merge conflicts are fixed.


Worms Armageddon Problems

Worms Armageddon using ddrawcompat 3.2 works fine minus having to run the games at the minimum resolution of 640x480
versions 3.2 above, the game runs at highest resolution, but now some menus and the map editor are broken (just displays background and nothing else)
the game behaves the same from 4.0 to the latest, so the only difference happens changing from 3.2 to 4.0+
On latest Windows 11, Adrenaline 24.8.1 (9.1 breaks 3.2 completely), RX 6600
The logs between the versions are identical, aside from the added information from in newer versions, so I'm just adding the Newest version Log, though I don't expect the log to be all that useful, as more useful would be knowing what was changed on the rendering side from 3.2 vs the next version
EDIT: I've figured out that I can get the Debug log into a zip to make it shippable on Github and that's the one that may actually be useful as it was above 25mb uncompressed (at like 95MB)
log.zip

It's the DisplayResolution setting that broke it. Use DisplayResolution=app until I can figure out why.
The GOG version doesn't have this issue, so it might also be fixable by one of the (semi-)official patches. Have you checked PCGamingWiki?

I'm using a CD copy (european, not sure which version though) but I haven't tried any yet patches.
DisplayResolution=app works, it just breaks full res gameplay (crashes) so basically the old 3.2 functionality, but without the 3.2 driver problem (which for me tbh is functional, just not optimal)
log.zip
However I have further information
Setting DisplayResolution=1024x768 (max supported res) results in no crash in gameplay, and it also shows why I was not seeing the UI in some menus, such as weapons and game options menus in singleplayer/multiplayer game configurator (top right option)
The UI was moved Down Left, and was oversized, indicating that the UI is attempting to scale to display resolution and not window resolution
setting DisplayResolution=640x480 fixes the menus, and allows for 1024x768 gameplay without crashing, it's just a bit blurry (functionally it's better as you're able to see more)
I've also toyed around with 640x480 windows compat and it just made a small window on the top left of the screen.


Error: Compilation of µWebSockets has failed and there is no pre-compiled binary available for your system. Please install a supported C++11 compiler and reinstall the module 'uws'

Steps leading to the issue
npm i -g socketcluster
socketcluster create myApp && cd myApp (npm installs uws without errors)
node server.js (it errors here, on require)

Full stacktrace
 [Error] Error: Compilation of µWebSockets has failed and there is no pre-compiled binary available for your system. Please install a supported C++11 compiler and reinstall the module 'uws'.
    at native (C:\code\wsApp\node_modules\socketcluster-server\node_modules\uws\uws.js:25:19)
    at Object.<anonymous> (C:\code\wsApp\node_modules\socketcluster-server\node_modules\uws\uws.js:29:3)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at SCServer (C:\code\wsApp\node_modules\socketcluster-server\scserver.js:83:18)

Additional details

Windows 10, Node.js 7.2.0, npm 3.10.9, socketcluster v5.1.3, node-gyp v3.4.0
I already have MinGW g++ (x86_64-posix-seh-rev1, Built by MinGW-W64 project) 6.2.0 Copyright (C) 2016 Free Software Foundation, Inc. This is free software; see the source for copying conditions.  There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
I also have Visual Studio 2015 Community Edition installed, along with Visual C++ 2015 build tools installed
I tried going into the myApp directory and running npm install uws again, but it does nothing.

Output from socketcluster create myApp
λ socketcluster create myApp && cd myApp
Creating app structure...
Installing app dependencies using npm. This could take a while...
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead

><EMAIL_ADDRESS>install C:\code\myApp\node_modules\uws
> node-gyp rebuild > build_log.txt 2>&1 || exit 0

><EMAIL_ADDRESS>postinstall C:\code\myApp\node_modules\spawn-sync
> node postinstall

<EMAIL_ADDRESS>C:\code\myApp
+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | +--<EMAIL_ADDRESS>|   |   | | `--<EMAIL_ADDRESS>|   |   | |   +--<EMAIL_ADDRESS>|   |   | |   +--<EMAIL_ADDRESS>|   |   | |   +--<EMAIL_ADDRESS>|   |   | |   `--<EMAIL_ADDRESS>|   |   | +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | +--<EMAIL_ADDRESS>|   |   | | `--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>|   |   +--<EMAIL_ADDRESS>|   |   | +--<EMAIL_ADDRESS>|   |   | `--<EMAIL_ADDRESS>|   |   `--<EMAIL_ADDRESS>|   |     +--<EMAIL_ADDRESS>|   |     `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   `--<EMAIL_ADDRESS>|     +--<EMAIL_ADDRESS>|     | `--<EMAIL_ADDRESS>|     |   +--<EMAIL_ADDRESS>|     |   `--<EMAIL_ADDRESS>|     +--<EMAIL_ADDRESS>|     | +--<EMAIL_ADDRESS>|     | +--<EMAIL_ADDRESS>|     | +--<EMAIL_ADDRESS>|     | +--<EMAIL_ADDRESS>|     | +--<EMAIL_ADDRESS>|     | `--<EMAIL_ADDRESS>|     `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   | `--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   +--<EMAIL_ADDRESS>|   `--<EMAIL_ADDRESS>|     `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| |     +--<EMAIL_ADDRESS>| |     +--<EMAIL_ADDRESS>| |     | `--<EMAIL_ADDRESS>| |     `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | |   +--<EMAIL_ADDRESS>| | |   `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | | +--<EMAIL_ADDRESS>| | | | | +--<EMAIL_ADDRESS>| | | | | `--<EMAIL_ADDRESS>| | | | `--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | |   `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | |   `--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| |   +--<EMAIL_ADDRESS>| |   | +--<EMAIL_ADDRESS>| |   | +--<EMAIL_ADDRESS>| |   | | +--<EMAIL_ADDRESS>| |   | | `--<EMAIL_ADDRESS>| |   | `--<EMAIL_ADDRESS>| |   `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | +--<EMAIL_ADDRESS>| | | `--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>    +--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
[Success] SocketCluster sample 'C:\code\myApp' was setup successfully.

Contents of node_modules/uws directory
λ ls node_modules/uws
LICENSE    binding.gyp    package.json  uws.js              uws_darwin_47.node  uws_linux_46.node  uws_linux_48.node
README.md  build_log.txt  src/          uws_darwin_46.node  uws_darwin_48.node  uws_linux_47.node  uws_win32_48.node*

I took a look at node_modules/uws/build_log.txt, it seems node-gyp errored due to Python 3. I ran npm config set python C:\Python27\python.exe and it works now.

same error, but my uws/build_log.txt says D:\XodoV3\app\node_modules\uws>if not defined npm_config_node_gyp (node "C:\Users\kk\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )  gyp WARN install got an error, rolling back install gyp ERR! configure error  gyp ERR! stack Error: unable to verify the first certificate gyp ERR! stack     at TLSSocket.<anonymous> (_tls_wrap.js:1098:38) gyp ERR! stack     at emitNone (events.js:105:13) gyp ERR! stack     at TLSSocket.emit (events.js:207:7) gyp ERR! stack     at TLSSocket._finishInit (_tls_wrap.js:628:8) gyp ERR! stack     at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:458:38) gyp ERR!  What is your SocketCluster server version (npm ls socketcluster-server)? What is your uws version (npm ls uws)?
Maybe try upgrading everything to the latest version.

my node version is v8.1.4 (latest one),
it seems my project doesn't have soketcluster-server, (npm ls
socketcluster-server) returns empty.
uws version i 8.14.0
On 13 July 2017 at 15:01, Jonathan Gros-Dubois<EMAIL_ADDRESS>wrote:

@liuqimin323 https://github.com/liuqimin323 What is your Node.js
version? What is your SocketCluster server version (npm ls
socketcluster-server)? What is your uws version (npm ls uws)?
Maybe try upgrading everything to the latest version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SocketCluster/socketcluster/issues/254#issuecomment-315214374,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APppgdfMwEPwMXtIMhvJrN702CI7k9gvks5sNpOngaJpZM4LQDFG
.

I have it solved by downgrade my node js to 6.11.1, thanks!

on windows 10 you need C++ windows build tools
https://www.npmjs.com/package/windows-build-tools

Installing Latest nodejs and npm on Ubuntu(16.04) with PPA, solved this problem.
Earlier version
node - v6.14.3
npm - 3.10.10

Latest version which solved the problem-
node - v10.6.0
npm - 6.1.0

Refer - How to Install Latest Node.js and NPM on Ubuntu with PPA

I had to remove uws and then install uws to fix this on my "New" laptop.

I have aliases for g++ and gcc pointing to clang (I guess all Macs do), so why can't it find a C++ compiler? 🤔


Unnecessary "new Curl()" in Readme.

On Readme.md:97, there is a  new Curl() being passed to the Botman class; the Botman Constructor does not know what to do with the extra argument. Is the Curl Handler something that is required in a future version, or is it something that is no longer necessary? Either way, the documentation should be updated to reflect the current version.

The Curl dependency is not for BotMan but for the DriverManager - see https://github.com/mpociot/botman/blob/master/src/Mpociot/BotMan/DriverManager.php#L35


Interfaces with all members optional is almost the same as 'any'

TypeScript Version:  1.8.0 / nightly (2.0.0-dev.201xxxxx)
Code
interface AllMembersOptional {
    first?: string;
    second?: number;
}

let a: AllMembersOptional = 7;
let b: AllMembersOptional = () => 7;

Expected behavior:
compiler error '7' is not type AllMembersOptional -- let a: AllMembersOptional = 7;
compiler error 'function: number' is not type AllMembersOptional -- let b: AllMembersOptional = () => 7;
Actual behavior:
no compiler errors or warnings and the following javascript is generated:
var a = 7;
var b = function () { return 7; };

Actual example
Referencing angular.d.ts and converting an angular directive
angular.module('test').directive('custom', function () {
  return {
    controller: 'testCtrl',
    template: '

,
    //...
  };
});

carelessly into a component
angular.module('test').component('custom', function () {
  return {
    controller: 'testCtrl',
    template: '

,
    //...
  };
});

generates no typescript compiler error messages and will fail at run-time because the options parameter needs to be an object
angular.module('test').component('custom', {
    controller: 'testCtrl',
    template: '

,
    //...
  }
);

options parameter cannot a function (or number or string etc).
Just about anything passed in for a parameter of an interface type with all optional parameters will be allowed by typescript

See this FAQ answer. An interface with all optional properties is similar to an empty interface in this case.

It is similar to an empty (i.e. useless) interface but is it the same?
Per FAQ:
In general, you should never find yourself declaring an interface with no properties.
If never is a good idea, would it make sense to have a compiler flag to issue warnings/errors?

Take a look at #7485 and #3842 -- #7845 is an proposal to fix this based on code from the closed PR #3842.

Looked at both. #3842 definitely discusses the issue and issues with implementing. this helps a lot. thanks


Song audio not loading

Describe your bug here.
The song audio isn't loading. I remembered to put "Inst.ogg" and "Voices.ogg". Instead of loading the audio, it is silent while just repeating the first 0.1 seconds of the chart.

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

Broken files, re-export them

Ok, i'll try

It works! Thanks.


Exported lein2deps dependency is outdated

The exported lein2deps dependency is hardcoded (here) an now outdated
This was observed here https://github.com/clj-commons/aleph/pull/674#issuecomment-1439092407

This is only important for Java compilation.


Build out ReadTheDocs documentation

Including:

Overview Section
BISG from scratch
Autodoc

Probably needs some copy editing, but basically done.


Why the num of outputs is not equal to length of dataset?

I try to train the retinanet with val dataset for testing. The length of dataset is 4952. And I set the batch_size  to 18. After training 1 epoch, the total iterations are 276. The length of dataset cannot divide the batch_size, so the size of the last batch should be 2. Therefore length of the total outputs of an epoch should be the same as the length of dataset. But I get 4968(=276*18). I get the outputs of each batch through anchor_head.get_bboxes and bbox_results = [bbox2result(....).....]. Then concat them to get the total outpus. Why does this happen?

@Icecream-blue-sky This is because drop_last is true by default.

@Icecream-blue-sky This is because drop_last is true by default.

I have checked the code and find that drop_last is False by default. Even if the drop_last is true, since the length of dataset is 4952, the length of outputs should be 4950 not 4968.

This is because GroupSampler will perform additional operations.
class GroupSampler(Sampler):

    def __init__(self, dataset, samples_per_gpu=1):
        assert hasattr(dataset, 'flag')
        self.dataset = dataset

        for i, size in enumerate(self.group_sizes):
            self.num_samples += int(np.ceil(
                size / self.samples_per_gpu)) * self.samples_per_gpu

@Icecream-blue-sky Training process or testing process?

@Icecream-blue-sky Training process or testing process?

At training process

@Icecream-blue-sky Training process or testing process?

At training process

It seems unnecessary.


Apply access check when opening a previously pinned program

@shankarseal fyi

Looks like the functions to convert SDDL -> SECURITY_DESCRIPTOR aren't in the DDK. Are we ok with passing a self-relative SECURITY_DESCIPTOR instead of an SDDL?

The EbpfIoDevice device object is restricted to:
//
// SDDL_DEVOBJ_SYS_ALL_ADM_ALL allows the kernel, system, and admin complete
// control over the device. No other users may access the device
//

So adding an additional access check here is redundant.

Closing this as an additional access check is pointless given that this API can only be called by admin, kernel or local system.
If we ever loosen the restrictions on the device object, revisit this.


Repo sync for protected CLA branch

The pull request is created from master637364164523233141 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change. @evanwindom


Infinity scroll pagination question

What I get with this version of redis-oplog is that when I scroll to the next page and reactively update limit parameter subscription is stopped and resubscribed which causes all records to be removed and re-added instead of just adding records for new page. Is it expected?

How does your pub look like?

I don't have lazy load in my app

Simplified version like this:
Meteor.publish("posts", function (limit) {
    return posts.find({}, {sort: {'date' : -1}, limit: limit});
})

this.subscribe("posts", Session.get('limit'))

When limit variable is increased in client normal behavior is to unsubscribe from old subscription and subscribe to a new one, but meteor just sends out new docs as old ones are already in minimongo.

@ignl
The fact that your code worked before but not now, seems more like a quirk than intended design of Meteor. We may execute the unsub faster than original Meteor.

Do you unsub from the old then resub? This should fix things, and prevent server-side pub from resending docs. You are benefiting from the mergebox
I think the right and clean approach is to send a skip option to the pub. Keep subscribing to new pubs as you need to, which only sends new docs. When the view is closed, unsub from all of them (store their handlers in an array)

Meteor.publish("posts", function (limit, skip) {
    return posts.find({}, {sort: {'date' : -1}, limit, skip});
})

const limit = Session.get('limit')
this.subscribe("posts", limit, limit ? limit - COUNT : 0)

[Note that I edited the two comments above, pls view online]

@ignl
The fact that your code worked before but not now, seems more like a quirk than intended design of Meteor. We may execute the unsub faster than original Meteor.
1. Do you sub to the new then unsub? This should fix things, and prevent server-side pub from resending docs. You are benefiting from the mergebox

2. I think the right and clean approach is to send a `skip` option to the pub. Keep subscribing to new pubs as you need to, which only sends new docs. When the view is closed, unsub from all of them (store their handlers in an array)

@ramezrafla It was certainly not a quirk and is documented meteor behavior (link here). So this is a common pattern in meteor and as documented the way meteor works is not stopping old subscription until new one is fully ready. Does redis-oplog work different in such case?

@ignl
That pattern is common, I am referring to the fact that the data persisted a bit AFTER you unsubscribed, which gave you a time to resubscribe again without losing data. If you notice in the doc your are mentioning, there is no unsub. You keep subscribing with a higher limit.
My points above are still valid. You should have no time where you are not unsubscribed so you don't get flicker.

OK, so I checked other changes I did after your comments and found major one I forgot about - doing this.unblock() on publications after update to meteor 2.3, so that was the culprit. Sorry for the false alarm :)


Less secure

isn't xchacha20poly1305 less secure than AES

Sorry for a not issue

isn't xchacha20poly1305 less secure than AES

No. XChaCha with Poly1305 has a larger security margin than AES-GCM.


fix Fill nodes to just be a separate pass

Codecov Report
Patch coverage: 81.63% and project coverage change: +2.67 :tada:

Comparison is base (261737d) 75.23% compared to head (808c514) 77.90%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #220      +/-   ##
==========================================
+ Coverage   75.23%   77.90%   +2.67%
==========================================
  Files          59       58       -1
  Lines        5245     5015     -230
==========================================
- Hits         3946     3907      -39
+ Misses       1299     1108     -191

Impacted Files
Coverage Δ

src/Finch.jl
63.63% <ø> (-36.37%)
:arrow_down:

src/symbolic/simplify_program.jl
79.16% <50.00%> (-7.79%)
:arrow_down:

src/looplets/fills.jl
89.47% <88.23%> (+1.97%)
:arrow_up:

src/base/broadcast.jl
75.00% <91.66%> (+0.25%)
:arrow_up:

src/base/mapreduce.jl
70.00% <100.00%> (ø)

src/looplets/nulls.jl
100.00% <100.00%> (+12.50%)
:arrow_up:

src/looplets/shifts.jl
58.62% <100.00%> (+3.06%)
:arrow_up:

src/tensors/levels/sparsetrianglelevels.jl
89.47% <100.00%> (ø)

... and 3 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update sbt-github-actions to 0.13.0

Updates com.codecommit:sbt-github-actions from 0.12.0 to 0.13.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.codecommit", artifactId = "sbt-github-actions" } ]

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #136 (17a32ef) into master (c40775f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #136   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines            3         3
=========================================
  Hits             3         3

Continue to review full report at Codecov.

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


Writing to BlobProperty

Hello,
we are trying to write to a BlobProperty via the gcloud datastore api.
From what I have read, I understand that if I send str using python 2.7, data is stored as blob_type in datastore. To store a string, I need to convert it to unicode.
Now I have very large string, which crosses the 1500 bytes limit, as a BlobProperty.
Would be great if you can let me know how I can do that?
Cheers...

@thinrhino As you pointed out, just make sure it's type if bytes, then it'll be stored as a blob.
Maybe I'm misunderstanding though. Could you provide a code snippet of what you tried and how it fails?

@thinrhino Please feel free to reopen if @dhermes reply didn't answer your question.


Update azure-stack-deploy

Small PowerShell Typo.
Thanks,

@CharbelNemnom : Thanks for your contribution to the Azure documentation! The author, @ErikjeMS, has been notified to review your proposed change.

@CharbelNemnom : Thanks for fixing that! I'll approve it.

#sign-off


Resume table syntax

This patch updates the handwritten wat/wast programs to use the new resume table syntax.

This fails currently because wasm-merge does not support the new syntax, yet. I will create a PR with binaryen this week to change the syntax there.

The new syntax has landed in binaryen now, and I've updated the harness to use a more recent binaryen, wasmtime, and reference interpreter.
There's also some weirdness about the preprocessor inserting lines starting with # to mark where it did something. In the past they only appeared in the beginning of imports.wat.pp, but now they appear elsewhere. I've switched to just sed-ing all of these lines away. We probably need to do the same in other Makefiles, too.


GET /api/v1/feeds/{feed_id}/posts/ response / schema incorrect

Currently returns
    {
      "id": "44d51254-c7c0-5643-adb5-6e488b7defd9",
      "datetime_added": "2024-12-05T14:24:14Z",
      "datetime_updated": "2024-12-05T14:24:18Z",
      "title": "monday evening post",
      "description": "

\n\n\n
\nmonday evening post\n
\n\n\n
\n<IP_ADDRESS>\n\ngoogle.com\n\nfile.ext\n
\n\n
\n
",
      "link": "https://throw-away123456.blogspot.com/2024/12/monday-evening-post.html",
      "pubdate": "2024-12-02T19:09:00Z",
      "author": "Throwawy",
      "is_full_text": true,
      "content_type": "text/html; charset=utf-8",
      "added_manually": false,
      "categories": []
    },

Response is missing profile_id
Also schema should not have markdown_file (schema shown below)
{
  "page_size": 50,
  "page_number": 3,
  "page_results_count": 50,
  "total_results_count": 3,
  "posts": [
    {
      "profile_id": "3fa85f64-5717-4562-b3fc-2c963f66afa6",
      "id": "3fa85f64-5717-4562-b3fc-2c963f66afa6",
      "datetime_added": "2024-12-06T06:28:26.016Z",
      "datetime_updated": "2024-12-06T06:28:26.016Z",
      "title": "string",
      "description": "string",
      "link": "string",
      "pubdate": "2024-12-06T06:28:26.016Z",
      "author": "string",
      "categories": [
        "string"
      ],
      "is_full_text": false,
      "content_type": "plain/text",
      "added_manually": false,
      "markdown_file": "string"
    }
  ]

Response is missing profile_id

this looks like a sync issue, the post has been processed on h4f but it hadnt been added to obstracts so it doesn't have a profile_id yet.


fix(semantic): ? on varaible declaration type annotations syntax error

Closes #5955

#5956  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


Flav/resource string ids

Description

See context here and here.
This PR introduces a new way to generate string Id for resources. Currently, most tables already include a sId column. The main issue with the current algorithm is that it requires a database roundtrip to exchange an sId for a ModelId. This happens frequently as we use sId in various places outside our infrastructure, such as storing files in GCS using sId.
To avoid this issue, we propose an algorithm that encodes a ModelId into a string ID and decodes a string ID back into a ModelId. This PR implements this new algorithm. We want to leverage these string IDs more and include additional data. The new string ID represents the following:
[<shard_key>, <workspace_id>, <resource_id>]

The goal is to use these IDs as we scale.
Constraints for this algorithm:

Security is not a concern. The primary goal is to expose a more user-friendly ID and embed extra data.
We want a minimum length of 10 characters, similar to the current sId.

To achieve this without reinventing the wheel, we use https://sqids.org/, which generates short and unique identifiers.
⚠ Sqids guarantees that encoding different numbers will produce unique IDs. However, due to the algorithm's design, decoding random IDs can sometimes produce the same numbers. For example (fake data, just to illustrate):
[1, 2, 3] -> abc
abc -> [1, 2, 3]
def -> [1, 2, 3]

This is not an issue for us.
To illustrate how we plan to use this, this PR also updates the recently added FileResource to use this approach. Since it is not yet used in production, we can safely drop the legacy sId column in favor of public IDs.
Regarding the current sId, we will need to continue using it for existing links and references. However, we will start exposing public IDs for older resources. These can be easily differentiated as public IDs will have a resource prefix.
Risk

Safe to rollback until we start using the file upload API.
Deploy Plan
Once in production, run the migration to drop the sId column on the files table.
psql $FRONT_DATABASE_URI -f ./migrations/db/migration_35.sql

Make sure to test this more than you would normally since this is a big big big trapdoor if it's broken :)

Thoroughly tested with the files 💪 !


Upgrade to Kie 6.1.0.Final

Upgrade project to a new API (Kie 6.0.1.Final).
For this reason I added the bom version 6.1.0.Final and I modified loading KnowledgeSession to use a new API.

Unfortunately due to the lack of time I am not able to continue developing and maintaining junit-drools project. Thank you for your contributions and I hope you will find one of the forks still maintained and helpful for you!


docs(TrajectoryPoint):  add comment for acceleration_mps2 to clarify its usage

Description
Part of https://github.com/autowarefoundation/autoware.universe/issues/5987

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@youtalk @isamu-takagi Neither @mitsudome-r or I have write access to this repository, could any of you approve this PR so that we can merge it? Thanks.


Embedded app updating isn't reliable enough

Perhaps we should check the version before opening the embedded app, that way we know it's the newest one. Only do this check once every 30 mins when the app is open (so save a timestamp in memory, not on disc) to avoid unnecessary requests as much as possible
That way we can also ditch the current 'push every device when there is an update' strategy, saving some processing power

I think this is mostly fixed now that the url for each version of an embedded app is different


gh-1487 - Update behaviour of ToVerticesHandler

Codecov Report

Merging #1488 into develop will decrease coverage by 0.62%.
The diff coverage is 30%.

@@              Coverage Diff              @@
##             develop    #1488      +/-   ##
=============================================
- Coverage      66.66%   66.04%   -0.63%
  Complexity      1744     1744
=============================================
  Files            724      659      -65
  Lines          23685    20868    -2817
  Branches        2997     2482     -515
=============================================
- Hits           15790    13782    -2008
+ Misses          6681     6062     -619
+ Partials        1214     1024     -190

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/uk/gov/gchq/gaffer/data/element/Edge.java
75.89% <ø> (-0.43%)
0 <0> (ø)

...ava/uk/gov/gchq/gaffer/data/element/id/EdgeId.java
16.66% <100%> (+5.95%)
0 <0> (ø)
:arrow_down:

...re/operation/handler/output/ToVerticesHandler.java
71.42% <12.5%> (-11.91%)
0 <0> (ø)

.../operation/addelements/impl/WriteUnsortedData.java

...tstore/io/reader/converter/PrimitiveConverter.java

...tore/serialisation/impl/LongParquetSerialiser.java

...ents/impl/rdd/CalculateSplitPointsFromJavaRDD.java

...gov/gchq/gaffer/parquetstore/index/GraphIndex.java

.../uk/gov/gchq/gaffer/parquetstore/ParquetStore.java

...ion/getelements/handler/GetAllElementsHandler.java

... and 59 more

Continue to review full report at Codecov.

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


add solar sense project and can messages

can messages + project folder

ids are placeholders

LGTM. Are the CAN messages just from fw14 or have they been reconsidered for fw15 design?

these are fw15 CAN msgs.


Pause at TOD

This should be an easy task. However, the most straightforward way to pause via SimConnect is not bound to a key in the simulator by default. This leads to a situation where it is easy to programmatically pause pause the sim, but difficult for the user to unpause.
An alternative solution could be to use one of the toggle pause inputs, but that could lead to situations where the sim unexpectedly pauses or unpauses because there's not a way to query the pause status.
This being the case, I'm putting this issue on the back burner.

I've gone ahead and implemented this, with a strongly worded comment that users need to be sure they bind the unpause key.


Adds ANCM external dependencies

@jkotalik,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Migration rp v5

New models, namespaces, splitting requests/responses. Supports only RPv5.

Thx. Some tests are failing - it's ok for now.


⚠ Bofrost.lu has degraded performance

In ecb11d1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1227 ms

Resolved: Bofrost.lu performance has improved in 7a9565a after 6 minutes.


Playback Speed

Would be really nice to be able to control playback speed on local videos. If anyone has an ideas on how to approach this, please tell, I'm willing to give it a try!

I can't think of any feasible solution. Opening a video in Helium is equivalent to opening it in Safari, so really you'd need some browser extension (that supports that feature) that takes over local video playback. I know VLC has a Safari extension, but I haven't tested it with Helium and last I heard, their extension wasn't working properly in the most recent versions of OS X.

Yeah bummer :/
I might just create a script that converts video playback speed locally then. Should this be closed do you think? It is a pipe dream?

Yeah, I think that's the best solution, unfortunately.

@horak, @JadenGeller -
Forgive replying to an old topic (especially if you have a better solution to this already), but have just found helium (thanks, it's ace!) and this was exactly my problem and I couldn't find a decent answer. Want to watch local videos but faster. Best solution I came up with is below (forgive the unnecessary mess of code - it's taken from this gist: https://gist.github.com/mengguoru/1c04b21ca048a670d84a - with some very quick styles added). Right now you have to edit the file to play videos, but it would only take ten minutes to make add a video path via a query string or something.
<!DOCTYPE html>
<html>
<head>
	<meta charset="UTF-8">
	<title>Html5 Video Player</title>
	<link href="http://vjs.zencdn.net/5.0.2/video-js.css" rel="stylesheet">
    <script src="http://vjs.zencdn.net/5.0.2/video.js"></script>
    <style type="text/css">
	body, div, dl, dt, dd, ul, ol, li, h1, h2, h3, h4, h5, h6, pre,
	form, fieldset, input, p, blockquote, table, th, td, embed,
	object, menu, nav, aside, section, article {
		padding: 0;
		margin: 0;
	}

	body {
		font-size: 62.5%;
		font-family: Arial, Helvetica, sans-serif;
		/*height: 100vh;*/
		min-height: 100vh;
		color: #fff;
		background-color: #000;
	}

	::selection {
	}
	::-moz-selection {
	}

	a {
		color: #00f;
	}
		a:hover {
		}
	.videofull {
		height: 100vh;
		float: left;
		width: 100%;
	}
		.videofull > div {
			height: 100%;
			width: 100%;
		}
    </style>
</head>
<body>
	<div class="videofull">
	<video class="video-js vjs-default-skin" controls preload="auto" data-setup='{ "playbackRates":[0.25,0.5,1,1.25,1.5,2]}'>
  <!-- let src point to your video's File URL on your computer-->
        <source src="yourvideo.mp4" type='video/mp4' />
        {# <source src="xxx.webm" type='video/webm' />
        <source src="xxx.ogv" type='video/ogg' /> #}
  </video>
  </div>
 </body>
</html>


Lazysizes does not load the first images you see on the screen.

Lazysizes does not load the first images. They are the first photos that the user sees as soon as they land on the page.
Right after the blank photos, lazysizes works perfectly. But even if you scroll up or down lazysizes never load the first rows of images.

Every photo that does not load has the lazyloading class, but never changes its class to lazyloaded
I'm using VueJs ^2.5.17. I've tried this with Google Chrome and Safari.
It looks a lot like this problem: https://github.com/aFarkas/lazysizes/issues/311

I've solved this by putting in this directive for lazysizes and vuejs:
https://www.npmjs.com/package/vue-lazysizes
In my case I have a Laravel and vuejs project compiled all with webpackjs.
Since lazysizes is called within the id=app of all my web, it seems that there was a conflict between lazysizes and vuejs.
With this I solved it, I hope it helps people.

@amoralesdesign
Thanks for your information. My question would have been wether you use the attr change plugin and if yes and still have problems, wether you can simply re-add the lazyload class inside of vue lifecycle hooks yourself. (For my personal info: Did you use the attr change plugin?)

@amoralesdesign
Thanks for your information. My question would have been wether you use the attr change plugin and if yes and still have problems, wether you can simply re-add the lazyload class inside of vue lifecycle hooks yourself. (For my personal info: Did you use the attr change plugin?)

No, I didn't know about the existence of that plugin inside lazySizes.
I'll try it out later. Thank you.

I'm running into this with a laravel vue project as well, adding that plugin didn't seem to help


user-select has no effect on before/after pseudo elements

computed value of user-select on before/after pseudo elements is none, this rule does nothing
https://developer.mozilla.org/en-US/docs/Web/CSS/user-select

I've just tested this and it looks like Edge and IE don't respect these specs. Text of pseudo elements is selectable (see https://codepen.io/MartijnCuppens/pen/EeGprm and https://s.codepen.io/MartijnCuppens/debug/EeGprm/ZoABaKQWDYyr for IE)
But since there is no text in this pseudo element, there is nothing that can be selected so this can indeed be removed. I need some confirmation on this though.


support PostgreSQL

Ⅰ. Describe what this PR did
support PostgreSQL
Ⅱ. Does this pull request fix one issue?
#1972
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2174 into postgresql will increase coverage by 0.05%.
The diff coverage is 48.63%.

@@               Coverage Diff                @@
##             postgresql    #2174      +/-   ##
================================================
+ Coverage         53.31%   53.36%   +0.05%
- Complexity         2474     2485      +11
================================================
  Files               458      458
  Lines             15353    15234     -119
  Branches           1833     1803      -30
================================================
- Hits               8185     8130      -55
+ Misses             6388     6343      -45
+ Partials            780      761      -19

Impacted Files
Coverage Δ
Complexity Δ

...tobuf/convertor/GlobalReportResponseConvertor.java
3.33% <ø> (ø)
1 <0> (?)

...rotocol/transaction/GlobalStatusResponseCodec.java
50% <ø> (ø)
1 <0> (?)

...obuf/convertor/BranchRegisterRequestConvertor.java
100% <ø> (ø)
3 <0> (?)

...rotocol/transaction/UndoLogDeleteRequestCodec.java
62.5% <ø> (ø)
3 <0> (?)

...erializer/seata/protocol/AbstractMessageCodec.java
23.07% <ø> (ø)
2 <0> (?)

...alizer/seata/protocol/MergeResultMessageCodec.java
79.48% <ø> (ø)
6 <0> (?)

...rotobuf/convertor/RegisterRMResponseConvertor.java
82.75% <ø> (ø)
3 <0> (?)

...rotocol/transaction/GlobalCommitResponseCodec.java
50% <ø> (ø)
1 <0> (?)

...protobuf/convertor/MergedWarpMessageConvertor.java
87.5% <ø> (ø)
6 <0> (?)

...buf/convertor/GlobalLockQueryRequestConvertor.java
95.83% <ø> (ø)
3 <0> (?)

... and 132 more


Crash on Android when emulate_mouse_from_touch = false

Godot version
0ddd9c3e8f9088fa139e25b2903289d727073e12
System information
arch linux, google pixel 6
Issue description
I've found that Android builds of projects are crashing when a touch release event occurs. This issue is only present when the emulate_mouse_from_touch setting is set to false in the project's settings, and a control node is present in the scene.
Expected result:
The touch release event should be registered without any crashes.
Actual result:
The android build crashes when a touch release event occurs.
Steps to reproduce

Create a new project
Set emulate_mouse_from_touch setting to false
Add a button to the main scene
Build the project for android
Run the project on android device
Touch and release on the screen

Minimal reproduction project
android-touch-release-crash.zip

This may not be android specific, I was able to reproduce this on a laptop with touch screens.


Fix oauth-advanced sample-proxy sample

This PR is a quick fix for the oauth-advanced sample-proxy sample.
There are two changes:

Fix the validator package version to 2.0.0 because new versions are not backwards compatible. A better solution could be update the logic to the newest validator version.

Add the basePath variable to ejs render calls on login and registration when an error or validation issue occurs.

Regards!

I signed it!


Hashed userpass authentication?

I’d like to not store my userpass/basicauth password in plaintext, so I am wondering if there’s any interest in a hasheduserpass authenticator?
I’d envision it would look something like this:
{
  "//": "...",
  "auth": "hasheduserpass:<username>:<sha224’d password>-<hex salt>[:vivify=<sha224’d vivify password>-<hex salt>][:+localhost]"
}

The password would still be transferred from the browser in plaintext and would be hashed on the server-side. In combination with HTTPS, this should be fine and avoid storing the password in plaintext.
I’d love to implement this, but wanted to check first that there’s interest and I’m not missing something.

We should probably use an algorithm designed for password hashing (probably Argon2?) instead of sha224.
...But that gets a bit weird, since currently the basic auth gets used for every request, whereas normally with a password scheme you'd use it once to get a session token and thereafter use that. Early on in my career I made the mistake of using a good password hash directly in an API and woke up to CI taking 45 minutes to run. So it seems like there's kindof two bad ways to do this:

Use a not-great password hash, giving users a somewhat-false sense of security (it's not nothing, but not "best practices")
Use a best-practice password hash, introducing a performance footgun for users. We'd at a minimum want to clearly document the pitfall.

An auth option that allowed for some sort of token exchange e.g. with a cookie would be more ideal, but also obviously much more work to implement.
What do others think? (Since you're specifically fishing for maintainers' thoughts, I should point out that I am some rando whose involvement is somewhat limited these days).

@zenhack  How about implementing an API first in Perkeep then? Would there be any interest for this feature?


N814 false positive: from decimal import Decimal as D

N814 (camelcase-imported-as-constant) triggers even if the alias is a single uppercase letter in code like
from decimal import Decimal as D
from fractions import Fraction as F

but perhaps it shouldn’t?

I tend to agree that we shouldn't flag this.

It’s also an interesting question whether from collections.abc import AsyncGenerator as AG is acceptable style for a module that defines a lot of them.

Did you mean N817 instead of N814? These won't be flagged by the latter as it'll be considered as an acronym for the imported name. See https://play.ruff.rs/9326f4aa-a8bf-47c0-a45f-5b3d04b902c7.

You can also use the extend-aliases option here:
[tool.ruff.lint.flake8-import-conventions.extend-aliases]
"collections.abc.AsyncGenerator" = "AG"
"decimal.Decimal" = "D"
"fractions.Fraction" = "F"

I think I'd suggest to use the configuration instead of adding special casing for certain imports. What do you think? @rdaysky


Level 2 Finished

Finished level 2 and added it to level selection, the player is now also prevented from replaying levels and must complete them in order?

LGTM


[SPARK-2165] spark on yarn: add support for setting maxAppAttempts in the ApplicationSubmissionContext

Looks like Kyle @knusbaum is busy, I will take this over if nobody has problem.
https://github.com/apache/spark/pull/3878


Experimental setting of EVE w/ stage1 in Table 5

Hi, thanks for your great work!
I'm curious about the specific experimental setting of EVE w/ stage1 in Table 5, shown in the figure below.

Would you please provide the amount of training data in each stage of this model? Appreciate your time and help in advance!

Hi, thanks for your great work!
I'm curious about the specific experimental setting of EVE w/ stage1 in Table 5, shown in the figure below.
Would you please provide the amount of training data in each stage of this model? Appreciate your time and help in advance!

We use 8M of 33M pretraining data to train PEL and PAL solely, and then directly utilize LLava-mix-665K to finetune all modules. We also tried 16/33M and didn't see much difference during this setting.


refactor the processNotifications method

A structure of notification is described here:
https://docs.adyen.com/development-resources/webhooks/understand-notifications#notification-structure

As we are not supporting SOAP on the notification module, we don't need to iterate notification request items as below method, because it will only be one request item.
https://github.com/commercetools/commercetools-adyen-integration/blob/18c39ed19c44a69a9df1b1281e16389b1776472c/notification/src/handler/notification/notification.handler.js#L8-L12

As discussed for now it is not an issue and can be closed


added CONTRIBUTING.md

Hey there!
First, if you are reading this, you are amazing!
I'm Armaan and actively taking part in Hacktoberfest and HackSquad, I was contributing to various repos recently and found out this repo (had a lot of contributions.
As always, I jumped and tried to find out "CONTRIBUTING.md" which all the beginners find first to start contributing but unlike other amazing repos, I couldn't find any, "CONTRIBUTING.md" contains a set of guidelines and steps on how to contribute to the project!
This change (separate file on steps and guidelines) will help increase contributors to contribute even more especially beginners, and to make it fun, emojified and engaging for users!
Who doesn't like fun? : D
So, I went ahead and made a separate "CONTRIBUTING.md" file with steps and guidelines, I hope this will help beginners and potentially increase contributors to this amazing repo!
Since I'm not the owner/maintainer of this repo, I do not have full data about the project, so there might be slight mistakes, and I deeply apologise for that.
This PR is free for any modifications/edits.
If this change was not really required, you are free to close this PR or merge it if you want, I'm sorry.
I am hoping for a positive and quick merge!
Thanks and regards,
Armaan.

Thanks, but I don't think a generic contributing guide is helpful for this repo.
There are lots of guides online that can help you learn how to contribute to open source in a meaningful way.
What would be helpful is:

An overview of the project, how it fits together, what tests are expected, and how templ is tested - e.g. how the parser is tested, how generation is tested etc., and the expectations around how those tests are written (they're all named as assertions).

Explanation of how to raise:

New features:

Discuss the design of the feature you want to add before you start coding so you don't waste your time on a design that's not going to work.
Where relevant, include examples of example input templ files, and what you expect to see in the output HTML.

Issues:

It really helps if you provide a reproduction repo, or can create a failing unit test to describe the behaviour.
Follow the LSP troubleshooting guide in the online docs.

More useful stuff could include.

An explanation of how to use the xc task runner to run tasks including unit tests (and ideally an update to the flake.nix to include a dev shell to build and test the software).
It's clear that the repo uses conventional commit syntax, link to that.
The coding style is never nesting. Comments are proper sentences, and end with a full stop. Reduce needless line breaks (line breaks are used to separate "paragraphs" of code).
There is a linter in the CI process, you can (and should) run it locally with xc prior to PR.
If you change the generator, you will need to run xc generate to regenerate all files. This is done automatically by the xc test command.

No need for emojis. Simple, clear language is better.
If you think you can do that, go ahead! If not, I'll close this in a day or two.

hello there, I'm sorry for the change that was unnecessary, I've updated the CONTRIBUTING.md file a bit, and made it better ( added the guidelines which were written in the top comment ), I hope that'd be helpful.
I'm sorry if this doesn't help as well.
Thanks and regards,
Armaan.


Show CPU and RAM for VMs

I have a request from a large customer to show CPU and RAM data for VMs. Don't know if this would be a overkill. Told them Azure Advisor already tells them when a resource is being underutilized.

We're not planning to include the RAM or CPU for the VMs because this information is already included showing the family of the VM.

This dashboard is not a real time monitoring tool, as you say, with the advisor recommendations it's enough to detect which machines are being underutilized. But if your customer still wants to see this, you can request this data creating a custom extension where you can obtain this data from a log analytics workspace API request. (We're using something similar in the AKS CCO Dashboard)


Add a podcast/interview segment at the end of PSPowerHour

At the end of a PSPowerHour session, especially one where there's only 3-5 presenters, it would be cool to have a chatting segment like the one at the end of the Dec 18, 2018 stream. That way you could feel good about running a session that might have fewer presenters, but still give viewers an hour (or something) of content.
My recommendation would be just to chat with the presenters about the things they presented. If there's nothing else to say, then there's probably some current tech news. Wouldn't have to be long. 10-20 min?
cc @vexx32 @michaeltlombardi

agreed, that was enjoyable

I would even consider perhaps dropping the "standard" down to 6 presenters and just more or less always planning in the 10-20mins of chit chat and Q&A time?

"PSPowerHour B-Sides"
I think it's a good idea. Also gives you a chance for any follow up Q/A or to mention something you may have forgotten during the main presentation.

@devblackops totally. A 10 min window doesn't leave much time for Q & A either. Plus it'll take a bit of pressure off of the organizers to find so many presenters.

I like the idea!  As long as @michaeltlombardi is up for it I'm in.  Particularly if we move to some cadence that leaves us with fewer speakers more often than not.  I'll help out more than last time (i.e. not sitting silently) if we do : P

Closing this out - is now pretty much standard, awesome idea!

🎉


Bump CI

Update CI in this repository to the newest methodology.

@bsousi5 @nategraff-sifive Just an FYI, we cannot merge this PR. It's restricted to repository code owners. If you approve, please merge.


Just release binaries to github

Thanks for creating versioned releases of dep! This will be helpful as changes are introduced to the file format & tool over time. I have a suggest to make them easier to use.
Current process to install a released verison of dep (inside a container or other):
apt-get update
apt-get install -y unzip
curl -fsSL -o dep.zip https://github.com/golang/dep/releases/download/v0.3.0/dep-linux-amd64.zip
unzip dep.zip
rm dep.zip
mv dep /usr/local/bin/dep
chmod +x /usr/local/bin/dep

Instead, if you simply published the binary to github (do not zip it):
curl -fsSL -o /usr/local/bin/dep https://github.com/golang/dep/releases/download/v0.3.0/dep-linux-amd64
chmod +x /usr/local/bin/dep

Many other go projects do this (publish the binaries directly to the github release), it would be awesome if dep can too.

Thanks for creating versioned releases of dep!

💯 💯 👍

Instead, if you simply published the binary to github (do not zip it):

this seems fair, we don't actually have multiple files to zip up there.
are there any drawbacks to doing this, other than the minor increase in file size?

are there any drawbacks to doing this, other than the minor increase in file size?

Not that I know of. gosu uses this approach, and I've used it for personal projects.
I was hoping the increase in file size would be moot with http gzip encoding, but it seems github static files (served from s3) do not support this. Either way, we're talking about a relatively small amount of data.


windows cpp server crashes on server streaming RPCs

error given:
c:\local\grpc\src\server\server.cc:316] assertion failed: GRPC_CALL_OK == grpc_call_start_batch(call->call(), ops nops, buf)
here is a related issue, which did fix this or something similar on linux:
https://github.com/grpc/grpc/issues/999
I get this when running route_guide_server from grpc-common.  the offending RPCs are listfeatures and routechat (one is a server streaming RPC, one is bidirectional).  The other RPCs seem to work fine.  The windows client works fine with a linux server.  see my grpc-common fork here:
https://github.com/larsonmpdx/grpc-common/tree/master/cpp/route_guide
I added a vsproject folder and manually imported the protobuf generated files so it should be easy to replicate.  see this file for build steps/ how to configure libraries for your machine:
https://github.com/larsonmpdx/grpc-common/tree/master/cpp/route_guide/vsproject
this is a low priority fix for me; I don't ever plan to run a windows server.  just an fyi, with a build howto

I recompiled today with the newest everything - doesn't happen anymore


guid episode tag should be unique and not change

As per https://itunespartner.apple.com/en/podcasts/faq/Updating your show_Basics the guid should be a unique id, not a URL etc which might change. An easy way to allow this and to not break existing deployments would be to allow this to be specified as an attribute.
Expected behavior
guid is unique and independent of media URL and can be specified manually.
Actual behavior
guid is the URL of the media file.
Reproduction Steps
N/A
Operating System and Version
N/A
Web Browser and Version
N/A
Hugo Version
N/A

Fixed in #210


Issue templates.

I don't have access to manage issue templates, but it would be helpful to have templates for:

Unsupported search syntax.
Failed deserialization.
New formats, games, currencies, etc.

This could be one template or several.

I don't know if I can give you access. Can't find the button for it. But I'm going to create them. You can latter roast me :smile:

Isn't the new formats, games, currencies part the same as failed deserialization?

#25


Add the endpoint to the warden wrapper

Fix #137
Without the endpoint the warden client panics

@matteosuppo Looks like you beat me to this by a few hours :) I did basically exactly the same thing, except I didn't add warden.AllowedHandlerPath.  Looking at https://github.com/ory-am/hydra/blob/1e48e1819672fada95d8b24251d12cd883ba61fc/warden/warden_http.go#L72, it seems the path is basically overridden every time, so I'm not sure what the best practice is here.  In any event though, your fix works for me too :)

Yeah that actually doesn't make sense - I overlooked that in review. I think you can remove all paths in the SDK and just do
		Endpoint: c.clusterURL,

could you just replace all Endpoint: pkg... with Endpoint: c.clusterURL,, test if everything works ok and push this PR? I would merge it then :)

This is an oversight on my part, I saw what @boyvinall mentioned and didn't bother adding the Endpoint.
Overlooked this part:
https://github.com/ory-am/hydra/blob/1e48e1819672fada95d8b24251d12cd883ba61fc/warden/warden_http.go#L71
AFAIK Everything else requires correct Endpoint be set.

 Thanks for chipping in!


Not able to access page on browser using IP only need to append port as well.

File: get-started/part4.md
As we using same docker-compose.yml file which is created on part3 and there is host port mapping is 4000:80 so will not able to access page after deploy app on swarm manger using IP like <IP_ADDRESS>. Either we need to add 4000 port after IP like <IP_ADDRESS>:4000 or need update port mapping '80:80 instead of '4000:80 ''on docker-compose.yml file before deploy app on swarm manager.

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Missing Screenshots

I took some screenshots to show the views of this URCap.

One thing to add, I did notice that the keyboard does not take input from my keyboard when it is displayed on the Program Node. I am posting this hear to avoid opening another issue.

One thing to add, I did notice that the keyboard does not take input from my keyboard when it is displayed on the Program Node. I am posting this hear to avoid opening another issue.


DRAFT: Handle invite links

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
DRAFT - handle invite links
Did a bunch of tests and studying and I think this will be best:
Create a separate "processing invite..." screen where we can show the progress in the handling of invites, then this can shown from the QE reader, the text entry, and the invite link handler
Process invite takes a mnemonic, and has the following states to display

Processing (checking invite on chain)
Success - shows who invited you and with how many Seeds, moves on automatically to "enter name" signup screen after a second or two.
Error - on chain invite is not available - user can close the screen with a checkmark

✅ Checklist

[ ] Github issue details are up to date for people to QA.
[ ] I have tested all my changes.

🕵♂ Notes for Code Reviewer
🙈 Screenshots
👯♀ Paired with
nobody

No longer needed.
https://github.com/JoinSEEDS/seeds_light_wallet/pull/937


Added skwp theme - minimalist theme showing relative home dir, git branhc, and rvm status on right when active

Hi :) Any chance we can merge my theme in? I am developing a big dotfile project (github.com/skwp/dotfiles) and it would be nice to be able to point people to the official robbyrussel oh-my-zsh instead of my fork. Thanks!

pretty please? with a cherry on top? :) users of YADR (http://github.com/skwp/dotfiles) would really benefit from this one. thanks.


hash_tree_root of merkleization.nim cannot be compiled

import
  unittest2,
  stew/byteutils,
  ../ssz_serialization/merkleization

type
  Eth2Digest = object
    data: array[32, byte]

  ExecutionAddress = object
    data: array[20, byte]

  BloomLogs = object
    data: array[256, byte]

  UInt256 = object
    data: array[32, byte]

  ExecutionPayload* = object
    # Execution block header fields
    parent_hash*: Eth2Digest
    fee_recipient*: ExecutionAddress
      ## 'beneficiary' in the yellow paper
    state_root*: Eth2Digest
    receipts_root*: Eth2Digest
    logs_bloom*: BloomLogs
    prev_randao*: Eth2Digest
      ## 'difficulty' in the yellow paper
    block_number*: uint64
      ## 'number' in the yellow paper
    gas_limit*: uint64
    gas_used*: uint64
    timestamp*: uint64
    extra_data*: List[byte, 32]
    base_fee_per_gas*: UInt256

    # Extra payload fields
    block_hash*: Eth2Digest
      ## Hash of execution block

proc main() =
  var h: ExecutionPayload
  let z = hash_tree_root(h)

main()

error message:
F:\projects\nim-ssz-serialization\tests\test_hast_tree_root.nim(43, 25) template/generic instantiation of `hash_tree_root` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(1294, 22) template/generic instantiation of `hashTreeRootAux` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(804, 20) template/generic instantiation of `merkleizeFields` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(805, 8) template/generic instantiation of `enumerateSubFields` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(565, 17) template/generic instantiation of `addChunkDirect` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(566, 27) template/generic instantiation of `hash_tree_root` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(1294, 22) template/generic instantiation of `hashTreeRootAux` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(804, 20) template/generic instantiation of `merkleizeFields` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(805, 8) template/generic instantiation of `enumerateSubFields` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(565, 17) template/generic instantiation of `addChunkDirect` from here
F:\projects\nim-ssz-serialization\ssz_serialization\merkleization.nim(201, 15) Error: attempting to call undeclared routine: 'combineChunks'

maybe missing mixin or export marker.

Can't reproduce based on what you describe. Tried three things:

rm ~/.nimble/ -r && nimble install ssz_serialization [to pull in all the Nimble dependencies; probably there's a better way] using Nim 1.6.14, and modified only the import line of your code to import ... ssz_serialization/merkleization instead of import ... ../ssz_serialization/merkleization. This compiled fine.

Then, made a fresh git clone of nim-ssz-serialization at commit d24c719ebfd53f5c606177ac760446692abbb056 and put verbatim the code you posted into nim-ssz-serialization where the paths lined up correctly. This also compiled fine.

I know this isn't what you're saying happens here, but I've seen cases where there was some quirk of nimbus-eth2's build environment creates build failures which don't exist outside of that. So I did basically the second thing, but within the vendored copy of nim-ssz-serialization in nimbus-eth2/vendor/nim-ssz-serialization. Also builds fine.

Hmm, it's a Nim 1.6.12 problem. Looks like Nim 1.6.14 does not suffer from this bug.
I found this when try to compile fluffy with recent ssz-serialization.
nimbus-eth1 still not migrate to Nim 1.6.14 because of nim vm regression in 1.6.14.
I think we should wait for 1.6.16 or later

In which case it's not a nim-ssz-serialization bug. Reasonable to close this issue?

In which case it's not a nim-ssz-serialization bug. Reasonable to close this issue?

if there's a simple / trivial workaround, I'd go for that

In which case it's not a nim-ssz-serialization bug. Reasonable to close this issue?

if there's a simple / trivial workaround, I'd go for that

Well, as many of
either

the "simple / trivial workaround" doesn't reduce the code quality or create strange-looking code which has to be explained and wondered about/around indefinitely, i.e. if the workaround is just better, or at least neutral, and creates negligible risk; or

explicit "TODO" for removal, linking to this issue, and mentioning in the codebase that it's a Nim 1.6.12 issue that Nim 1.6.14 doesn't have, so it is easy to find and remove when (not if) appropriate. For example, when nimbus-eth1 updates to Nim 1.6.16.

Nimbus and associated libraries already have too many workarounds. Adding more creates maintenance costs for a language version which is only going to become less common (how many people are able to use 1.6.12 but not 1.6.14 yet?). These costs might be worthwhile, e.g., to facilitate nimbus-eth1 development right now.
For example, the workaround removed in https://github.com/status-im/nimbus-eth2/pull/5219 years after it was introduced had been, in fact, fixed in upstream Nim and incorporated into Nimbus's vendored version within weeks after that, but because none of this was done in that case, it took disproportionately long to discover, reverse-engineer what precise failing scenario, test the removal, and remove. That's a failure mode I'd like to avoid, in general.

I think you're overthinking it a bit - ie fixing a problem like this usually boils down to something like a mixin, bind or simply reordering the code a little so that the compiler can find the function during instantiation - there is 0 maintenance cost once you've introduced the fix and the only reason you consider it a "workaround" is the path that brought us here, ie a bug report - this can also be seen in that the removal of most workarounds tends to be underwhelming it terms of what it achieves in terms of code real estate or hacks: they change some minor detail in a some minor function but rarely have any effect beyond that, unless the issue is truly egregious.
Also, the objective of a bug report is to solve the issue and there might be multiple ways of doing that: fixing the library, upgrading the compiler etc - since 1.6.14 has other problems / bugs, it becomes a tradeoff between the various bugs and sometimes, the fix is really simple on this side of the problem meaning we should go for it.


Question: is it possible to have multiple strategies per technology?

I think it's now possible to give a behavior in the technologies table. But does that result in a single strategy or how sould I interpret this?

The behaviour maps the InstalledTechnology to the Network class used to calculate its load. It's not quite the same thing as a strategy.
It's a one-to-one map (each technology has one behaviour and maps to one Network class), except in the case of base loads where the two load profile components – "flexible" and "inflexible" – result in the base load splitting into two technologies. In that case, the behavior column in the technology_component_behaviors table defines how each part works.
The Network class may do different things depending on the strategies chosen by the user; for example, the ElectricVehicle cares about both the buffering_electric_car and solar_storage strategies, and will behave slightly differently depending on which of those are enabled.
Out of interest, what prompted this question? Something to do with the hybrid heat-pump, or...?

Out of interest, what prompted this question? Something to do with the hybrid heat-pump

Yes. :+1:  We (@ChaelKruip and I) wanted to know what the default behavior would be.

I'm adding new Network classes to model the hybrid heat-pump, so the two Technology records (one for gas, one for electricity) will have to have a new behaviour. Maybe "hhp_electricity" and "hhp_gas"?


Auto-update to new versions of postgresql?

Hi, there is already postgresql v12.3:
https://anaconda.org/conda-forge/postgresq
However, psqlodbc is still v11.
Would it be possible to keep this repository in sync with the postgresql package?

I'm running into the same issue. Postgresql is now 13.1 but this package is still on 12.3.

Please make a PR if you want a new version.

Hi, there's a a new version https://github.com/conda-forge/psqlodbc-feedstock/pull/2


why error on broadcast: too long mempool chain

data: {
  success: false,
  code: 1,
  message: 'the transaction was rejected by network rules.\n' +
    '\n' +
    'too-long-mempool-chain, too many descendants for tx aabbccafb8ef1196a5acda9447eb006180da169b4d2c5a3e42e9d0ba5c427e02 [limit: 25]\n' +
    '[010000000001016903a3d047b7bdcaf32e325591a3afd42554ce5e817290eeaca4b883cfccbbaa010000000048cf35f302ce810000000000002251205eee7699f1178895007732da1057ca234eccc5a84f7f68e48e409859996219b84a55010000000000225120d57cbafdc296699d0e839cae23c69ccf32cae3a3c95f2dea9357e14a076c15f00140ffb81b46ad1258a96ed7d10a1ca18bb4175fb0a0baa2d5d4f8163c11c78bb42b03dc6bd517c87a1c47f62bc041c79a1c1dfcf8a915dbc2b8c7a8b81062ce244d00000000]'
}

It's been broadcasting but it's been failing, it's been failing all night

https://bitcoin.stackexchange.com/questions/99168/clarifying-descendants-ancestors-in-error-cases-for-too-long-mempool-chain


Pasting V1 code into converter sometimes gets hijacked by buildPtn_MSTR()

... and this regex gets pasted in the window instead ...
(*UCP)(?s)(?<var>[_a-z]\w*)\h*=(?>\s*(?:(?:(?:\h*;|(?<=\h);).*)|(?:#TAG★\w+★#))*)*\R+(?<blk>\((?<guts>(?>.+?)+?)\R+\h*\))

Are you running latest version? This should've been fixed here https://github.com/mmikeww/AHK-v2-script-converter/commit/95a54b795d01105c2299451eff6effd41859a069

That commit was made on July 04, I've re-synced my fork several times since then.

All OK since re-syncing.


Added test cases for Check and Person classes

Added new test cases

Codecov Report

Merging #125 into master will increase coverage by 2.47%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master     #125      +/-   ##
============================================
+ Coverage     30.86%   33.33%   +2.47%
- Complexity      189      207      +18
============================================
  Files            61       61
  Lines          1565     1569       +4
  Branches        172      173       +1
============================================
+ Hits            483      523      +40
+ Misses         1056     1018      -38
- Partials         26       28       +2

Impacted Files
Coverage Δ
Complexity Δ

src/seedu/addressbook/commands/Command.java
87.5% <ø> (+16.66%)
10 <0> (+1)
:arrow_up:

src/seedu/addressbook/data/person/Person.java
57.77% <ø> (+2.22%)
17 <0> (+1)
:arrow_up:

src/seedu/addressbook/ui/MainWindow.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

src/seedu/addressbook/autocorrect/Dictionary.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

src/seedu/addressbook/parser/Parser.java
82.2% <100%> (+1.86%)
31 <0> (+3)
:arrow_up:

src/seedu/addressbook/data/AddressBook.java
70.17% <0%> (+19.29%)
14% <0%> (+3%)
:arrow_up:

src/seedu/addressbook/ui/Formatter.java
45.83% <0%> (+45.83%)
6% <0%> (+6%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Margin Trade - Liquidation price not updating

Margin position with loanID: 0x12f35517f86a89904ca73d67bf09789931bc9deb315ad3e1302fbb526f5e0365 has sucesfully done top-ups to the position, but the UI don't show a lower liquidation price.
User reported this to a CM via DM. This is a 2x LONG position (XUSD/BPro) created with 0.01 BPro as collateral. So initially the position should be of 0.02 BPro. Opening price: 42,167.86; today the position has 0.05987 BPro and a margin of 406.98% (so very hard to liquidate), but the liquidation price shown by the UI is still the same.
Potentially related to #1389

Putting this on hold since the trading abi changed and all trading transactions will fail until we merge https://github.com/DistributedCollective/Sovryn-frontend/pull/1559 .


Why laravel not returning null?

I try to use chartjs in my app but when i called query its not returing null results in array my command
my year_of_addmission=[''2005','2006','2007','2008','2009','2010','2011','2012','2013','2014','2015']
App\Student::select(DB::raw('count(*) as male_count'))->where('gender', '=', 2)->groupBy(DB::raw('(year_of_admission)'))->orderBy('year_of_admission')->get()->toArray();

result

"[
  [
    "male_count" => 6,
  ],
  [
    "male_count" => 4,
  ],
  [
    "male_count" => 1,
  ],
  [
    "male_count" => 4,
  ],
  [
    "male_count" => 1,
  ],
  [
    "male_count" => 3,
  ],
  [
    "male_count" => 2,
  ],
  [
    "male_count" => 3,
  ],
  [
    "male_count" => 2,
  ],
  [
    "male_count" => 1,

but i want like this if student not count its should be return null like this

"[
 [
    "male_count" => null,
  ],
  [
    "male_count" => 6,
  ],
  [
    "male_count" => 4,
  ],
  [
    "male_count" => 1,
  ],
  [
    "male_count" => 4,
  ],
  [
    "male_count" => 1,
  ],
  [
    "male_count" => 3,
  ],
  [
    "male_count" => 2,
  ],
  [
    "male_count" => 3,
  ],
  [
    "male_count" => 2,
  ],
  [
    "male_count" => 1,
 because in 2005 no male(gender=2=male)  student count so how to get this please help thanks

Please ask on the forums. :)


Support java 7 with scala 2.11

I have had a compilation error trying to use Monocle 1.3.2 in a Java 7 environment. Please build monocle for Java <8 with Scala 2.11 if you can do so without sacrificing functionality.
Yes, Oracle Java 7 is EOL, I know, but I am working on several projects which target the IBM JVM, where 7 is still very much the norm.

I don't remember why did I change the config: https://github.com/julien-truffaut/Monocle/commit/230a9a3d27b3c829cd94b90548207a112f756fa0
Maybe it was just a try that wasn't supposed to be released or we used it when we experimented with java8 date.
Anyway, we should aim to quickly release 1.4.0-M2 supporting java 7 for scala 2.11

I will soon release 1.4.0-M2 with this fix

@gwils 1.4.0-M2 is released, can you please give it a try?


Running without CUDA/Nvidia?

I tried running this on Linux per the instructions, but with the cpu-only version of Torch. That still gives me the NVIDIA error:
RuntimeError: Found no NVIDIA driver on your system. Please check that you have an NVIDIA GPU and installed a driver from http://www.nvidia.com/Download/index.aspx
Is it possible to test this without CUDA/NVIDIA?

I think if sufficient if you remove all the .cuda() in the code.


can't deploy services with external volume driver without restarting kontena-agent

I'm using the volume driver https://portworx.com/ for volume persistence.
There are two problems in using this custom driver with kontena:
1 - The driver itself is registered by a portworx container. When that container is not started, kontena-agent do not recognize pxd driver. All containers that declare volumes with this driver are not deployed, and if for some reason the driver is unregistred even for a moment, kontena stop the containers (I'm not sure if that is the case but I'm noticing instability and services are dropping, when that happen the command kontena node show does not show the pxd volume driver). The driver container need to start before kontena-agent, but here comes the second problem
2 - portworx use etcd or consul to orquestrate a cluster of persistent storage and replication volumes. I don't want to maintain two etcd installs, and use kontena etcd, which works great, but portworx driver needs to start after kontena-agent which is Chicken and egg problem :-)
What I do now:
1 - Start the node
2 - Wait for pxd service
3 - kontena node show will not recognize the driver
4 - Restart kontena-agent. Wait a few moments
5 - kontena node show will recognize the driver
6 -  Manual redeploy stack with kontena stack deploy
What I whould like to do:
Use a volume with grid or stack scope and disable the check for the volume driver in order to force the deployment (not so good solution, but I think is necessary to control this behavior in problems like this). I'm using a affinity filter, it should not have problems.
Or, I would like to let the check for the driver on, but kontena needs to be way more tolerant in order to work with not static drivers like this. (maybe checking more often on volume driver information, and having a delay before removing/stopping/rescheduling container service)
Whould be great to run portworx as a stack with kontena, it was my first try, but I'm running as a systemd service to see if improves stability

Whould be great to run portworx as a stack with kontena, it was my first try, but I'm running as a systemd service to see if improves stability

Yes, trying to run Docker drivers as Docker containers often does not really work, and tends to lead to a multitude of issues... The answer is to either use systemd services with Docker 1.12, or upgrade to Docker 1.13 and use docker plugin containers.

2 - portworx use etcd or consul to orquestrate a cluster of persistent storage and replication volumes. I don't want to maintain two etcd installs, and use kontena etcd, which works great, but portworx driver needs to start after kontena-agent which is Chicken and egg problem :-)

I think the only solution to this problem in general is for the driver to be able to start up without etcd being available yet, and have operations block or fail until etcd becomes available. Then the kontena-agent would need to be able to wait for weave/etcd to be up before querying/retrying those volume driver operations.

Or, I would like to let the check for the driver on, but kontena needs to be way more tolerant in order to work with not static drivers like this. (maybe checking more often on volume driver information, and having a delay before removing/stopping/rescheduling container service)

The agent will currently refresh volume drivers from docker info every 60s. Those are updated to the server, and immediately taken into account for service scheduling decisions...

and if for some reason the driver is unregistred even for a moment, kontena stop the containers (I'm not sure if that is the case but I'm noticing instability and services are dropping, when that happen the command kontena node show does not show the pxd volume driver).

When this happens, what does kontena service events show? Are you seeing service ... has wrong number of instances (or they are not distributed correctly) events that lead to re-scheduling/re-deployments of the service?

Update: I managed to create a more stable installation, unfortunately I can't install as a plugin. This is what I did:

kontena-agent and portworx driver are systemd services
kontena-agent wants portworx.service (weak dependency, in case node has no persistence)
Installed consul as a systemd service
portworx.service depends on consul

Caveat: Even with portworx service working good, is necessary to have at least one volume with pxd driver, in order to docker info command recognize the driver.
Problem: Even with docker info command showing the pxd driver. Kontena agent need a restart in order to pick up the driver.
I will keep a eye and see if this setup resist to host restarts. But I think, kontena agent should pick up the driver after 60s, it isn't.
Information:
agent version: 1.3.4
  docker version: 17.06.0-ce
  connected: yes
  last connect: 2017-08-05T01:15:17.001+00:00
  last seen: 2017-08-05T01:15:28.373Z
  public ip: x.x.x.x.x
  private ip: x.x.x.x
  overlay ip: <IP_ADDRESS>
  os: Ubuntu 16.04.3 LTS
  kernel: 4.4.0-89-generic
  drivers:
    storage: overlay2
    volume: local,pxd

Problem: Even with docker info command showing the pxd driver. Kontena agent need a restart in order to pick up the driver.

Yes, my earlier comment about the agent refreshing the node info from docker info every 60s was wrong, it's actually memoized at startup: https://github.com/kontena/kontena/blob/v1.3.4/agent/lib/kontena/workers/node_info_worker.rb#L139
This probably counts as a bug.

Note that this only applies to classic plugins that Docker picks up from /var/lib/docker/plugins: v2 plugins are picked up by the agent even after startup.


adjust new route to 1.3 and fix post route to use TO Perl

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/557/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/513/


e2e (chronium) fails during GitHub Action run

Link: https://github.com/chanzuckerberg/napari-hub/actions/runs/3315618790/jobs/5476465800

@codemonkey800 Do you have more context on this issue?


test_logger_configuration is failing

What is wrong?
After the fix of async runner in #1338, these tests turn out to be failing and were marked as xfail
https://github.com/ethereum/trinity/blob/0e3e67d2ceb52e124fcb866f11d3f9704cfe04bf/tests/integration/test_trinity_cli.py#L247-L270
How can it be fixed
Need to investigate further

I wanted to understand this better and looked into it today. Here's a quick brain dump.
It seems that this is a regression that came with #1277 when the new logging was introduced that doesn't rely on multiprocessing.Queue anymore. It seems that the regression made it's way in because the AsyncProcessRunner had a bug where a test would sometimes automatically pass when the timeout of the test was hit. That bug was fixed by #1338 which now exposes this regression caused by #1277.
I wanted to prove that so I went back to an older commit and just cherry-picked the fix from #1388 to see if all the logging tests would still pass which is the case.
I tried to come up with a fix but couldn't find the root cause yet so I wanted to at least did a quick brain dump.
/cc @pipermerriam

Nothing immediately comes to mind.  I believe someone will just need to dig in and see where things aren't being correctly done.
Also, I think that it would be better if we could streamline how these are tested since the current mechanism is very indirect...

ISTM that there's something wrong with the code responsible for capturing trinity's output and matching it against the expected/unexpected patterns. I say that because for test_logger_configuration[command1,...] we expect only to see {'Started main process', '>>> ping'} patterns, and in two subsequent runs, both containing the expected pattern, one failed and the other passed. http://sprunge.us/PS4BLy is the one that failed, and http://sprunge.us/uDdS1U is the one that passed

And another clue that might lead us to to the buggy code is the fact that all tests that fail intermittently are the ones that have empty sets for one or more of the expected/unexpected patterns


QtCreator Added

@nesl247 okay i added qtcreator page.

As much as I'd love to accept this, the icon would be duplicated which we don't want. We are in the process of powering the site via the READMEs, so if you can actually add the instructions there (See the Alfred theme), it would be appreciated.
Please change this PR to only include the screenshot. Thanks!

I want to apologize if things have been a bit confusing. We've been doing a lot of work to make it easier to contribute to dracula, and we might have gone back and forth a few times. Sorry about that.


'event' is undefined when clicking space around modal

I assume it should close the modal when clicking the space around it. When doing so I get TypeError: Cannot read property 'stopPropagation' of undefined in Chrome browser.
This is the case even in unmodified example app.

Same error as @gaikaz.  It seems that there is a problem with the onClose handler when you click outside the modal. The same handler works fine when you click on 'Cancel button.

Fixed with https://github.com/gregthebusker/react-confirm-bootstrap/commit/61d2969902d427c0cff8dd4933affa0c748a4b2a


Revert "Updated dependencies to fix missing version"

Reverts openshift/osd-metrics-exporter#53

@arjunrn is this update invalid?

No, I'm reverting it because it's not really needed.

/lgtm


Bitwise indexes, Half-precision, Docs and Build Hashes

:tada: This PR is included in version 0.15.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to process the data for Scannet200?

I should preprocess scannet200 in this scannet200-original Repostitory(LanguageGroundedSemseg,https://github.com/RozDavid/LanguageGroundedSemseg)  or the Repostitory（scannet :https://github.com/ScanNet/ScanNet/blob/master/BenchmarkScripts/ScanNet200/preprocess_scannet200.py）?

Hi,
please use the preprocessing script from our repository. See here: https://github.com/JonasSchult/Mask3D#scannet--scannet200
Best,
Jonas


debconf trigged on every puppet run

When I use debconf to preseed my mysql root password, this debconf is triggered on every puppet run:
Debug: Debconf: calling exists? for mysql-server/root_password
Debug: Debconf: caching data for mysql-server/root_password
Debug: Debconf: getting items for package mariadb-server-10.1
Debug: Executing: '/usr/bin/debconf-show mariadb-server-10.1'
Debug: Debconf: item mysql-server/root_password_again => (password omitted)
Debug: Debconf: item mysql-server/root_password => (password omitted)
Debug: Debconf: item mysql-server/password_mismatch =>
Debug: Debconf: item mysql-server/error_setting_password =>
Debug: Debconf: get mysql-server/root_password
Debug: Debconf: set mysql-server/root_password to secret
Debug: Debconf: updating mysql-server/root_password
Notice: /Stage[main]/Mysql::Package/Debconf[mysql-server/root_password]/value: value changed '(password omitted)' to 'secret'
Debug: /Stage[main]/Mysql::Package/Debconf[mysql-server/root_password]: The container Class[Mysql::Package] will propagate my refresh event

This is caused by the fact that the password is stored as "(password omitted)".
Maybe it is a better solution to check "password" types in the password database:
echo "get mysql-server/root_password" | debconf-communicate mariadb-server-10.1
0 secret

Yes, this is obviously a bug in the current implementation. The current implementation uses debconf-show and it seems to be unable to retrieve the value of a password typed item. Using debconf-communicate looks like the proper interface to these values.

I just commit an updated provider that fixes the issue for me. If you want to verify this in your environment you can download the updated file: debian.rb. Replace the file of the same name in the debconf module at .../modules/debconf/lib/puppet/provider/debconf/debian.rb in your Puppet environment.

Yes, it is working now. Thank you for the quick resolution!


[Episode 151] Cluster Diagnostics with Crashd

Monday or Tuesday:

[x] Pick a topic and create/use an issue in the repo
[ ] If applicable coordinate with an upstream project if they're interested in attending the show
[x] Create the graphic
[ ] Schedule on Youtube
[ ] Add scheduled episode to TGIK playlist on YT
[ ] Create short link in rebrandly
[ ] Tweet about it from your personal account

[ ] Post tweets on internal #tgik slack channel and cc  Vykky

Before episode:

[ ] Create HackMD page for live notes.  Start with README template in this repo
Show Coordination with Assistant (if you have one)

[ ] Paste HackMD URL on internal #tgik channel
[ ] Assistant adds ~5-10 news items from around the web as possible "News of the Week"
[ ] Confirm show time and AV check
[ ] Last minute retweets or topic updates
[ ] Sync with external guest one last time if necessary

[ ] Get everything tested well before episode so you can debug
[ ] Use internal slack channel to ensure that AV se tup is good

After show (ideally right away but can wait until Monday):

Create/submit PR for this repo

[ ] Episode directory and README from hackmd
[ ] Any other files from the episode that might be useful
[ ] Update playlist.md with episode
Feel free to self merge

[ ] Edit YT description to point to episode directory here
[ ] OBS/OS Upgrade if applicable (Never on the day of a show)

Can someone fix the typo in the YouTube title? https://www.youtube.com/watch?v=KfN4MK4B2f8

@LappleApple Can this be closed? Looks like the title is fixed.

@vladimirvivien I still see "Diagnotics" (missing the "s")

Got it. Fixed @LappleApple .


Typographical Error

Found typo in the documentation

Codecov Report

Merging #106 (1b5b498) into master (f0290e9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   92.86%   92.86%
=======================================
  Files          16       16
  Lines        5943     5943
=======================================
  Hits         5519     5519
  Misses        424      424

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


fix(behavior_path_planner): candidate path header

Signed-off-by: kosuke55<EMAIL_ADDRESS>Description

If the header of candidate_path_ is not set, it is not visualize, because the header of output is replace with input by util::toPath()
Related links

https://github.com/autowarefoundation/autoware.universe/pull/2486
Tests performed

psim

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

FYI: before https://github.com/autowarefoundation/autoware.universe/pull/2486, the header of candidate is current one.

@rej55
Could you please check this PR?:pray:


Upgrade to Apache Kafka 3.7.0 [DO NOT MERGE]

The RC is close to being voted, so let's get ready for the release
This PR pulls in Apache Kafka staging repo into the POM.  This must be removed before this is merged.

@ozangunalp could you review, especially the substitution changes.

@robobario FYI

@k-wall thanks for this initiative!


Right tapers broke

Probably after #156 - I'll bisect in a moment.
Reproduction: test_open_shape.zip

I should work on my testing skills :sweat_smile:

Well, I also always forget to test with open shapes.


ELM-1499 Add access credentials for G4S to their server

Adding access for G4S ahead of them testing the connection to the SFTP server for them to deposit data.
This will add a new (well renamed from just g4s) user account g4s_test to the existing G4S sftp server.

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform


Clarify where to set druid.monitoring.monitors.

It can cause problems to set monitors globally for all server types.

By the way, it would also be good to have a column in the table that says what process types a monitor is valid for. Some of them say it in the descriptions, but they don't all say it, and it'd help people to know for sure.
I didn't add this column in this PR because I don't actually know comprehensively which monitors work where. But it would be good to figure that out and add the information.


Update README.md

Hello! I added a game (buttons) i wrote in Ada. I hope it can be interesting.

It would be cool if you could add a screenshot of your game in the github repo readme.
Thx.

Il giorno ven 24 mag 2019 alle ore 21:41 ohenley<EMAIL_ADDRESS>ha scritto:

It would be cool if you could add a screenshot of your game in the github
repo readme.
Thx.
Thanks for your suggestion.
The problem is that, to be honest, I don't know how to do this on github.
I mean, I know how to add a link to a screenshot in readme, but I don't
know where to put the file without having to add it to the source files.
Do you have any tip?
Thx.

A.Cervetti

There is many ways... ask google. Here is one:
https://ardalis.com/add-images-easily-to-github
Take care,
Olivier
On Tue, May 28, 2019 at 9:33 AM Andrea Cervetti<EMAIL_ADDRESS>wrote:

Il giorno ven 24 mag 2019 alle ore 21:41 ohenley<EMAIL_ADDRESS>
ha scritto:

It would be cool if you could add a screenshot of your game in the github
repo readme.
Thx.
Thanks for your suggestion.
The problem is that, to be honest, I don't know how to do this on github.
I mean, I know how to add a link to a screenshot in readme, but I don't
know where to put the file without having to add it to the source files.
Do you have any tip?
Thx.

A.Cervetti
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ohenley/awesome-ada/pull/26?email_source=notifications&email_token=AAK4NVQOL3GZ3WKAQCU43NDPXUYCPA5CNFSM4HPLGNWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWMEIWI#issuecomment-496518233,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK4NVXWJIPUO75BTC6VSWLPXUYCPANCNFSM4HPLGNWA
.

Done.
Thanks.
A.