DOC: Add link to discussion of cluster effect size

I found this information useful, probably worth adding to the stats tutorial about clustering.

https://14602-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/discussions/plot_background_statistics.html
@drammock feel free to merge if you're happy

thx @larsoner


Support Dropwizard applications without logback

This is a prototype for splitting the tight integration between Dropwizard and Logback. This feature was requested several times (See #1367). The idea is too provide users the ability to exclude Logback from Dropwizard and replace it with an external logging solution, if they really want to. The main work of this PR is too initialize the Logback infrastructure lazily, so the user can swap it to an external logging mechanism in the config file.
With this change users can swap logback to other logging implementation like that:
 <dependencies>

        <dependency>
            <groupId>io.dropwizard</groupId>
            <artifactId>dropwizard-core</artifactId>
            <version>1.1.0-SNAPSHOT</version>
            <exclusions>
                <exclusion>
                    <groupId>ch.qos.logback</groupId>
                    <artifactId>logback-classic</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>ch.qos.logback</groupId>
                    <artifactId>logback-access</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>org.slf4j</groupId>
                    <artifactId>log4j-over-slf4j</artifactId>
                </exclusion>
            </exclusions>
        </dependency>

        <dependency>
            <groupId>log4j</groupId>
            <artifactId>log4j</artifactId>
            <version>1.2.17</version>
        </dependency>

        <dependency>
            <groupId>org.slf4j</groupId>
            <artifactId>slf4j-log4j12</artifactId>
            <version>1.7.21</version>
        </dependency>

    </dependencies>

server:
  type: simple
  applicationContextPath: /application
  adminContextPath: /admin
  connector:
    type: http
    port: 8080
  requestLog:
    type: external
logging:
  type: external

public class ExampleApplication extends Application<ExampleConfiguration> {

    @Override
    protected void bootstrapLogging() {
    }
}

Coverage decreased (-0.09%) to 81.756% when pulling 0b1d9e39ba3064b05e040c2d1c1741c5c7224d6f on remove_logback_dependency into ec084500d9c707d7f1dcaf20341f71f1580376f7 on master.

Coverage decreased (-0.1%) to 84.566% when pulling c4e5c3d59db138a56d77c6c7eddcf34cab585c47 on remove_logback_dependency into 9295413c7b52b74d2c783df08d4da5290e4ceada on master.

Coverage decreased (-0.05%) to 84.272% when pulling 7f5bccffa0f8f6a01af7d985647f262022aa0076 on remove_logback_dependency into 243874e59a78eddbcc227fba20e7c520cb8ebcaf on master.

@dropwizard/committers Could you please review? This should be ready to merge. I tested it locally and it works with custom logging solutions. Would be nice to polish it and adopt to the framework.

@arteam mind updating the release notes too?

Will do. It probably make sense to mention in the docs how to switch to an external logging.

@arteam this is just pending documentation, correct?

Correct, I need to get my hands on it. Will do today.

Coverage decreased (-0.05%) to 84.481% when pulling df3a6df4cd9bf9cd706993843c962c8cae40d9e8 on remove_logback_dependency into efb9e0e9eb42e388535620b7b94340c183f487b3 on master.

Coverage decreased (-0.05%) to 84.481% when pulling df3a6df4cd9bf9cd706993843c962c8cae40d9e8 on remove_logback_dependency into efb9e0e9eb42e388535620b7b94340c183f487b3 on master.

Coverage decreased (-0.05%) to 84.481% when pulling df3a6df4cd9bf9cd706993843c962c8cae40d9e8 on remove_logback_dependency into efb9e0e9eb42e388535620b7b94340c183f487b3 on master.

I've added docs about disabling Logback to the core guide.

Thanks!

@arteam  is there a slated release for this anytime soon, i,e 1.2.0 ?

We don't have an ETA for 1.2.0, unfortunately. You can use snapshots builds for the time being, if you want to test this feature.
We could cut a release candidate for 1.2.0, but we need to discuss this among the maintainers.

@arteam is there a repository holding the latest snapshot of Dropwizard, or do we have to build it ourselves?
(also, thank you greatly for this PR, works perfectly)

I think you can use the Nexus snapshot repository, our snapshots should be there:
<repository>
  <id>sonatype-nexus-snapshots</id>
  <name>Sonatype Nexus Snapshots</name>
  <url>http://oss.sonatype.org/content/repositories/snapshots</url>
</repository>

I just tried to use this feature with 1.2.0-SNAPSHOT.
io.dropwizard.jackson.LogbackModule imports the Level class from Logback. Could you please fix that?

Thanks for pointing this out. It seems the configuration of the Jackson's ObjectMapper in Dropwizard is still dependent on Logback. I will take a look on it.


London 10 | Andrius Isin | Full-Stack-Project-Assessment | Level 300

FRONTEND: https://video-recomendations.onrender.com/
BACKEND SERVER: https://video-recomendations-server.onrender.com/

Looking really good! 🎉

Thanks, Daniel, for your review :)  I have made changes


Fix lexing failure

When there is an invalid token consisting of one character followed by eof, long_error tries to print more than one character, which results in a failure.
# Yojson.Safe.from_string "x";;
Exception: Failure "lexing: empty token".

# Yojson.Safe.from_string "xx";;
Exception: Yojson.Json_error "Line 1, bytes 0-2:\nInvalid token 'xx'".

I don't know whether there was a valid reason to look for more characters. If not, the fix is very easy.

We're only using the junk pattern and rule for error reporting so this should work.

opam release in progress https://github.com/ocaml/opam-repository/pull/7691


AssertionError: Can not find executable file

python scripts/run.py --config-name=wanjinyou dataset_name=example case_name=ngp_fox mode=render_path is_continue=true
Error executing job with overrides: ['dataset_name=example', 'case_name=ngp_fox', 'mode=render_path', 'is_continue=true']
Traceback (most recent call last):
File "scripts/run.py", line 74, in main
assert False, 'Can not find executable file'
AssertionError: Can not find executable file
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

Hi, it seems in scripts/run.py, Hydra automatically changes the working directory. I have updated run.py to override the working directory. You can try the latest version and run command python scripts/run.py --config-name=wanjinyou dataset_name=example case_name=ngp_fox mode=train +work_dir=$(pwd).

did you update run.py ?

did someone manage to solve this? would really appreciate the help.
My environment:
Windows 10
Nvidia RTX 2080 GPU

python scripts/run.py --config-name=wanjinyou dataset_name=example case_name=ngp_fox mode=train +work_dir=$(pwd)
Working directory is /DATA/zbz/new-project/f2-nerf
Error executing job with overrides: ['dataset_name=example', 'case_name=ngp_fox', 'mode=train', '+work_dir=/DATA/zbz/new-project/f2-nerf']
Traceback (most recent call last):
File "scripts/run.py", line 78, in main
assert False, 'Can not find executable file'
AssertionError: Can not find executable file
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
I also meet this error，although use the upgraded file and command


Continuation of Issue#352

The issue is the division between multiclass and continuous.
The quick solution is to filter continuous by checking whether labels cover the integer of [0, C).
However, it causes the following issues:

numerical class labels of random integers are not allowed
1-indexed class labels are not allowed
rank regression will be mis-recognized as multiclass

To address those issues, we need to enhance input_validator or preprocessing BEFORE instantiating dataset class.
At least, we know that the given task is either classification or regression, so we should use this knowledge.

It has been resolved with #386


Actions

Revert the following section in the Github Actions file — main.yml  — once #139 sorted out.
From
- name: Release
        uses<EMAIL_ADDRESS>
To
- name: Release
        uses: softprops/action-gh-release@v1

or alternatively...
- name: Release
        uses: softprops/action-gh-release@v2

Closed by 560cab5


modify Ping-Pong-Game/style.css

Modify the position of the Ping-Pong-Game game panel to the middle of the vertical orientation of the screen

Fixes #1972


Added a script to deduplicate files

Fixes

Fixes #33
Description

This PR adds a script named deduplicate.py in the scripts folder to organize the duplicate files in the www/l folder and replaces them with a symbolic link.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

The flake8 error is coming from the genicons.py file which I never modified it.
./scripts/genicons.py:19:1: E402 module level import not at top of file
Error: Process completed with exit code 1.

Maybe I can modify it to fix the issue.

@ysfchn I can't merge it with that check failing due to lack of privileges.  We'll have to wait for @TimidRobot to return next week.

@mathemancer  Oh okay, no problem!


Reset command

It's working nice, I like it!
Would be nice to have an .au reset command that resets everything (Especially the mutes) because if I take down the processes mid game the voice channels stay muted until I restart everything and trigger some event like getting in a lobby, this way if the server crashed, I can restart it and reset.
Or better yet, the bot should make sure to reset everything on shutdown (unmuting channels) and on startup

Yep, great idea. We need a formal way to "end" the .au game for sure, and the bot should probably try and unmute all users before exiting as well

Just added a .au end to completely end the game (and unmute all players), but also added an .au force <stage> to help override if the capture portion doesn't report the right state :)
Ex: .au f discuss


Missing dependency FDSQueueNonEmpty

Is SmallKanren ready to use in Pharo 9?
I am interested in trying SmallKranen in gtoolkit (64 bit linux) and gtoolkit is based on pharo 9. After using Iceberg to load Cons and FunctionalDataStructures, there is a missing dependency (FDSQueueNonEmpty) when I try to load SmallKanren.

I'm going to close this on the assumption that it sounds like it's probably working now, but feel free to open another issue if there's still a problem. Also feel free to get in touch if you want to talk about your use case, since there are undocumented features that might be helpful if you run into an issue.


[Help Wanted] ModuleNotFoundError: No module named '_SimpleITK'

#help wanted
I have installed SimpleITK in the Anaconda Environment using conda install -c simpleitk simpleitk. I imported it in the Python Console or Spyder Env using "import SimpleITK as sitk". Everything was going fine, until a few days ago . Now I get the following error:

File "C:\Users\Raluca Sandu\Anaconda3\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_SimpleITK'

So far, I have uninstalled and installed the package. I have updated my Python, currently running : Python 3.6.5 |Anaconda custom (64-bit).
I can't tell what am I doing wrong especially since I haven't touched anything. Any direction on how to solve this issue and use SimpleITK again is highly appreciated.

Are you using the root environment for everything? Or using different environments?
I am wondering if additional packages were installed which removed or changed the version of dependencies of the "_SimpleITK" library preventing it from being loaded.
I'd recommend creating a new environment for your work with SimpleITK. You can find an example of doing that in the SimpleITK Notebooks repository.
It is possible that either the SimpleITK package is not specifying a dependency properly, another package is not correctly specifying the require dependencies, or the conda tools did not respect the requirements when updating packages. Either way creating a environment file with the packages you are using to reproduce the problem is an important test to tack the issue down.


Is it possible to ignore a cell before JSON format ?

Thanks for this cool project !
I try to do something but i can't.
I have a file and i would like ignore a cell who is before my table.
Because when i export to json, it just takes the column where there is the cell.
I don't want modify my file.
Do you think is possible ?
Thank's a lot.

The sheet_to_json function takes a second options argument.  IF you set the range key to a number, it will start from specified row (0-indexed).  More information in a js-xls issue which prompted the feature
There are actually a few ways to control what cells to use:

XLSX.utils.sheet_to_json(ws, {range:3}) will skip the first 3 rows and use the remaining rows and columns

XLSX.utils.sheet_to_json(ws, {range:"A4:D7"}) will use the specified A1-reference-style range

XLSX.utils.sheet_to_json(ws, {range:{s:{r:1,c:0},e:{r:3,c:3}}}) will use the range object, whose form is described in the README

To eliminate the headers, you can also set the header key to:

1: generate array of arrays
"A": generate array of objects whose keys are the A1-style column labels (A,B,C,...,Z,AA,AB,...)
array ["label1","label2",...] use the specified labels as keys.

The relevant test case (which shows more use patterns) is json output
If you know the structure of the file and you want a more custom output, you can write your own function.  https://github.com/SheetJS/js-xlsx/blob/master/xlsx.js#L5255 is the code for the json output, which is fairly straightforward.

You are the best !
I have used this XLSX.utils.sheet_to_json(ws, {range:3}) and that work like a charm.
Thank you very much.

@jeremydelacasa closing now, for future issues please raise in the XLSX tracker https://github.com/SheetJS/js-xlsx


color: add Equals() method to compare two colors

Hey fatih,
We talked about this a few weeks ago on slack and I finally got around to implementing it. Let me know if it needs any more work to get accepted.

Hi @andrewaustin. LGTM, this is cool! I've made a simple comment, after that I think this is safe to merge :)

Oops, removed that extra white space.

Thanks @andrewaustin :+1:


Change blue background on nav to underline

Currently the blue background and focus states cannot exist simultaneously. Copy the gov and NHS pattern on a line indicating the current page.
Also remove the padding by default at the top of heroes.

With the focus states being a background color and an underline, and the proposed active link styling also having an underline, how would the active and focus styles coexist?

The underline in the focus state is at the bottom of the box, the underline for the active link is underneath the text inside the box


[BUG] Class MPS_Accelerator must implement all abstract methods

Class MPS_Accelerator must implement all abstract methods
lost this abstract methods
def supported_dtypes(self):
return False

Resolved by #4363, closing.


Fix JMX Karaf import

JMX in a Tomcat deployment was failing in Karaf with the exception java.lang.RuntimeException: java.io.IOException: java.lang.ClassNotFoundException: javax.management.openmbean.CompositeDataSupport. This was because of a missing dependency from the wrap of org.glassfish.external/opendmk_jmxremote_optional_jar.
Unfortunately adding an $Import-Package=javax.management.openmbean here would exclude the other files included from the jar so $DynamicImport-Package is used.

@drigodwin see https://ops4j1.jira.com/wiki/display/paxurl/Wrap+Protocol, section 20.4.2 "Indicating Optional Dependencies": I think we just need a ,* at the end of Import-Package=... for it to not exclude everything else?
See the example lower down in the same file:
<bundle dependency="true">wrap:mvn:com.squareup.okhttp/okhttp/2.2.0$Import-Package=org.apache.http.*;resolution:=optional,android.util.*;resolution:=optional,okio;version=%27[1.2.0,1.3.0)%27,*</bundle>

@drigodwin have you tried @aledsage's suggestion of appending ",*" to the Import-Package? Note that some characters could need to be escaped.
If that doesn't work then DynamicImport-Package is a fair fallback and LGTM.

I've tested the .* and that also works, thanks @aledsage. I've swapped back to using $Import-Package here.

thanks, merging.


Simple Vanilla JS Programs

@trishan9 kindly assign this issue to me

@trishan9 kindly assign this issue to me

Sure, I assign this issue to you!

Issue has been assigned to you under Hacktoberfest!

Happy contribution!


Test suite fails to run without mocha dependency

Running of tests fails when mocha is not installed globally.
Test suite depends that the system has it somewhere,
but it does not get installed automatically by the package.json file.
When running npm test command, test suite failed with the following error messages:
[02:29:35] Finished 'test:unit' after 2.07 s
[02:29:35] Starting 'test:angular'...
[02:29:35] Starting 'test:angular:unit'...
WARN `start` method is deprecated since 0.13. It will be removed in 0.14. Please use
  server = new Server(config, [done])
  server.start()
instead.
[02:29:35] 'test:angular:unit' errored after 144 ms
[02:29:35] Error: Cannot find module 'mocha'
    at Function.Module._resolveFilename (module.js:489:15)
    at Function.resolve (internal/module.js:18:19)
    …
[02:29:35] 'test' errored after 2.22 s
[02:29:35] Error in plugin 'run-sequence'
Message:
    An error occured in task 'test:angular:unit'.
[02:29:35] 'test:angular' errored after 149 ms
[02:29:35] Error in plugin 'run-sequence'
Message:
    An error occured in task 'test'.
[02:29:35] 'test:angular' errored after 150 ms
[02:29:35] Error in plugin 'run-sequence'
Message:
    An error occured in task 'test:angular:unit'.
npm ERR! Test failed.  See above for more details.

I understand that different applications might use different versions of mocha.
Still, it would be logical that the test suite would depend on a specified version of the required tool,
as it would make it more reliable to run tests on a new computer or virtual machine (without need to manually setup unlisted requirements).
Way to solve this issue would be to add mocha to development dependencies with:
npm install --save-dev mocha

Tried this, and then did re-run the test suite. Worked nicely after that.
My suggestion is to add the updated dependencies to the package.json for the future developers.

#1118 is now merged, so this should be closed.


Tx-builder: Refactor code and fixes #108

This PR closes #108.
The code for tx-builder was a little messy, it was a huge file with all the logic, so I decided to split the code up into several files/components.
I think the code still needs another refactor, to simplify how the txs are being validated and created but I didn't want to add more complexity to this PR.
The fixes for #108 consisted of two parts.

When to make eth input visible
https://github.com/gnosis/safe-react-apps/pull/115/files#diff-16e5843f84a6e1b9db9232230df37ba17898969b8f316153ba26767187885567R191

I added a useEffect that determines when eth value is visible. Basically when the address is not a contract or when the contract has a payable method.

Avoid creating tx data when the method is receive or fallback.
https://github.com/gnosis/safe-react-apps/pull/115/files#diff-16e5843f84a6e1b9db9232230df37ba17898969b8f316153ba26767187885567R129

*. Besides splitting up the code, I changed how the to value is set.
When the user enters a valid address in the main input, the same value is copied toInput state (the controlled value to To input).
https://github.com/gnosis/safe-react-apps/pull/115/files#diff-16e5843f84a6e1b9db9232230df37ba17898969b8f316153ba26767187885567R185

We have some logic in the component, that could we extracted:
Building descriptions
Building "raw" tx for Safe Apps SDK

Yes, I mentioned that in the PR description, Also I would add how we handle the errors, actually, we have one error for all the inputs, I would change that so each input has its own error, TextField input already supports that.

Different question:
Would it make sense to split this in multiple PRs?

the fix for #108 it's really tiny, just a few lines, perhaps we can create a different issue/PR to continue refactoring the points mentioned above.

We shouldn't leave this PR behind. I was trying to validate it and I see that the changes to solve #108 are really small, but I totally get that this goes against our statement of keeping PRs simple and focused on the task they should solve. It's difficult now to trust that the most of the changes are just to keep the code as it was but splitting it into different files, because it's difficult to find the lines of the actual fix.
I was trying to validate this fix as it is. One thing I detected is that if you add one transaction and then you delete it while reviewing the list, a magic 0 appears on the interface

@dasanra I can try to create a PR with only the fix, it should be pretty simple. I think it's only 2 lines of code.
But we also should also review/merge this PR.
wdyt is best?

The from the URL's only the Tx builder loads. the WC and Compound Apps keep loading forever
I used the dev environment to load my safe and load the apps there as "custom apps". Only the Compound app loads properly and can be used to create a transaction, the other 2 just load forever
https://safe-team.dev.gnosisdev.com/app/#/safes/0x9913B9180C20C6b0F21B6480c84422F6ebc4B808/transactions

@francovenica For me all three are working correctly. When you open the app links directly is normal that those 2 only show a loader, because they are waiting to be inside the Safe iframe.
I could add any of the three apps without issues in dev environment :thinking:

Safe and env: https://safe-team.dev.gnosisdev.com/app/#/safes/0x9913B9180C20C6b0F21B6480c84422F6ebc4B808/balances
My bad. I was copying the URL as text and the custom app seems to add a default "http://" instead of "https://" and it was causing an error (or at least is the only explanation I can find since I just copy/pasted the urls :shrugs:)
I was able to use now the WalletConnect app to create and execute a tx
I could access the Tx builder, but I could not use the tx builder:
I cannot write anything in the "Address" input. I cannot type or copypaste anything in there
I tried to use the Transfer method of the ERC20 ABI, to transfer an ERC20 from the safe to some other account
In the address field I need to enter the address of a token I have, but since I cannot enter anything I cannot use the Tx builder
ERC20 ABI:
https://ethereumdev.io/abi-for-erc20-contract-on-ethereum/
Gif:

@francovenica it should be fixed now

@nicosampler Ok, the issue with the Address field was solved. But I have another issue now
I complete the entire form with valid values, but I cannot add transactions. When I press the button the tx is not being added at all.
I tested this same values in the other Tx builder (The one that is there by default) and it works fine.
I don't see errors in the console or in the network tab when I click the button
To reproduce this actually there is no need to fill the form. You can compare it using both txBuilders and just hitting the Add transaction button. (you can use the safe address hash in the ABI field to show the full form)
The Tx builder by default will show you an error because the fields are empty
The Tx builder added as a custom app will not show the error
Is like if the Add Transaction button had no onClick action at all

@francovenica yes, sorry 🤦♂ , seems like I didn't test it yesterday.
It should be working now!

Is working fine now. Approved


Using move buttons during a line clear causes panic

Steps to reproduce:

Clear a line
While line clears press the right arrow
Observe panic

@Wryte Oops :) Thanks!

@Wryte Oops. Thanks!


[Bug] Can no longer use nested relationships in dot notation for a text column

Bug report
What I did
When I upgraded to 4.1, text columns that were previously set up with a nested relationship (e.g. 'model1.model2.name') now throw an exception.
What I expected to happen
[     'name' => 'model1.model2.name',     'label' => 'Name',     'type' => 'text', ],
I would expect the above configuration to output model2's name field in the column.
The documentation says standard dot notation is accepted here:
https://backpackforlaravel.com/docs/4.1/crud-columns#text-1
What happened
BadMethodCallException
Call to undefined method App\Models\Model2::model2()
What I've already tried to fix it
I upgraded my patch version to 4.1.47
Read through the 4.1 upgrade guide to see if it mentioned changes to this
https://backpackforlaravel.com/docs/4.1/upgrade-guide
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 7.4.4 (cli) (built: Mar 17 2020 13:49:19) ( ZTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
LARAVEL VERSION:
v8.43.0@3ec63771a07e251f4fb6c7ef2411a79a314bba65
BACKPACK VERSION:
4.1.47@b2325eedd661adb75c23683aa6ee1d291b3578e7
Thanks!

Hmm you're right @byjl - what you're saying should be working. And it seems to be for me...
Can you please copy-paste the PHP code that isn't working for you? The addField() or field() statement?
Cheers.

@tabacitu Absolutely. It's neither of those functions, it's when adding the columns to the table using:
$this->crud->addColumns([             ['name' => 'model1.model2.name', 'label' => 'Name', 'type' => 'text'],         ]);
This is within the setup() function of my CrudController.
For whatever reason, this tries invoking: App\Models\Model2::model2() which throws a BadMethodCallException
seemingly stemming from:
vendor\backpack\crud\src\app\Library\CrudPanel\Traits\Relationships.php:44
return $relation_model->{$related_method}();
Thank you again for the help.

Hmm @pxpm could you please take a look? Relationships are your specialty.

Hello @byjl
Sorry for late reply and thanks for the detailed report.
From my tests https://github.com/Laravel-Backpack/CRUD/pull/3606 is fixing this issue.
But is not related with the line you described, that applies to fields. Can you check if that PR fixes it for you ?
Thanks in advance,
Pedro

Thank you for getting back to me @pxpm.
I tried requiring "backpack/crud:dev-relationship-columns-definition" and I kept running into various dependency issues.
So I wasn't able to test the whole PR per se, but I did copy+paste these changes from app\Library\CrudPanel\Traits\ColumnsProtectedMethods and it did fix the issue.

Thanks again

Thanks for the feedback @byjl and sorry for the mess up.
I will make sure we review it properly and ping you when we merge it.
I will be closing this for now as it seems it's fixed by #3606
Thanks for the report.
Pedro


Remove service message

Can I remove the service message using the this
for eg
xyz joined the group
pqr left the group

I want to remove the telegram message whenever user join or leave the group

https://github.com/php-telegram-bot/core/issues/730 I am reffering to this , I am not sure about PHP though

Join/left messages are bundled in fields containing the users in Message.
Please try this, couldn't test it, but should work or be very close.
import info.mukel.telegrambot4s.api._
import info.mukel.telegrambot4s.methods._
import info.mukel.telegrambot4s.models._

/**
  * Remove join/left messages from groups.
  */
object GroupBot extends TelegramBot
  with Polling {
  lazy val token = "TOKEN"

  override def receiveMessage(msg: Message): Unit = {
    super.receiveMessage(msg)
    if (msg.newChatMembers.isDefined || msg.leftChatMember.isDefined)
      request(DeleteMessage(msg.source, msg.messageId))
  }
}

GroupBot.run()

Thanks It worked , but only for single group ,when I added the bot in two group it tarted throwing me this error .
ERROR info.mukel.telegrambot4s.clients.AkkaClient - Telegram API exception
info.mukel.telegrambot4s.api.TelegramApiException: Bad Request: message can't be deleted
	at info.mukel.telegrambot4s.clients.AkkaClient.$anonfun$apply$3(AkkaClient.scala:56)
	at scala.concurrent.Future.$anonfun$flatMap$1(Future.scala:302)
	at scala.concurrent.impl.Promise.$anonfun$transformWith$1(Promise.scala:37)
	at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:60)
	at scala.concurrent.impl.ExecutionContextImpl$AdaptedForkJoinTask.exec(ExecutionContextImpl.scala:140)
	at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
	at java.util.concurrent.ForkJoinPool$WorkQueue.pollAndExecAll(ForkJoinPool.java:1021)
	at java.util.concurrent.ForkJoinPool$WorkQueue.execLocalTasks(ForkJoinPool.java:1046)
	at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1058)
	at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
	at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:157)

works perfect after giving permission.

hi . my bot delete the join and left service messages in my group but speed of deleting is so slow . when i add 50 person in one action with telegraph 50 service message apear in room and about 30 seconds for delete this 50 service messages with bot . my bot writen in php . see my code please . can you help me for solve this problem ? thanks

@$newmember = $result->message->new_chat_member->id;
@$leftmember = $result->message->left_chat_member->id;
@$gpid = $result->message->chat->id;
@$messageid = $result->message->message_id;
$telegram = new telegram(TOKEN,HOST,USERNAME,PASSWORD,DBNAME);
if(!empty($newmember)|| !empty($leftmember)){
$telegram->deleteMessage($gpid,$messageid);
die();
}


Shouldn't all tracked be option selected instead of all users option?

Shouldn't all tracked be option selected instead of all users option?

Maybe. I worry that changing that might lead to situations where some new browser starts appearing that's not tracked yet and people will miss the fact that a feature may not be as well supported as it appears...

I'd love to see that, or some explanation of the relevance & meaning of untracked. I spent a little time trying to identify the 5% (AU) discrepancy with flexbox after noticing it wasn't in older versions of IE. If the default showed all tracked, I probably wouldn't have even noticed. I'm assuming the missing 5% isn't actually relevant anymore - correct me if I'm wrong.
Sidenote, If a new browser pops onto the scene, it's probably not going to get a rush of widespread usage if it doesn't have current CSS support. I suspect it would end up tracked well before it hit significant usage figures.
Thanks for all your great work on caniuse, @Fyrd.

I only recently noticed "all users" vs "all tracked" option because I was having a similar issue as @ShaneOfMacquarie. The numbers weren't adding up and I was confused. I don't think I mind what it defaults to as much as I most definitely mind that there's a help icon next to it or an about page I can easily find for an explanation of what it means. I didn't find the option self-explanatory at all and had to  search google to find this issue and get an idea of what it means. Eventually after clicking through and searching all the pages on the site I found a more detailed explanation at the very bottom of this page: https://caniuse.com/usage-table.

Could also be a setting allowing you to set what will be selected by default.
Sebastian


Add support for ParityDb configuration

Issue summary
ParityDb doesn't have a lot of knobs but there are some that we would like to expose to the user in our config file. Ie:
[parity_db]
stats = false
compression = "lz4" # other possible values are "snappy" and "none"

Task summary

[ ] Add similar parity_db section than above to forest Config struct
[ ] Change stats to false by default (to avoid any potential overhead)

Acceptance Criteria

[ ] We can enable statistics manually
[ ] We can dump the default config file with config dump, tweak parity_db section and load back the .toml
[ ] Changing compression works

Other information and links
https://docs.rs/parity-db/latest/parity_db/struct.Options.html

In general, I feel like the database configuration should be in a separate file. Otherwise we will bump into serialisation/deserialisation issues (I think that at the moment, Forest-paritydb won't be able to ingest a TOML config with a rocksdb section)

If we change the default db backend one day, we can handle this special case and invite the user to comment its rocksdb section and dump a new config file.
It's unlikely that users will jump dynamically between backends.

Not that unlikely; I met in Lisbon folks that were interested more in DB observability than overall performance if it was reasonable. Say, ParityDb is more performant, but its ecosystem is close to empty. On the other hand, you would have RocksDb, which would be okay and may have additional tooling for some DB inspection. That would be a reason to switch.
On the code side, we would have less random conditional compiling (that is, it would go away from the node config).
Just something to consider. I'm not pushing it.


work on eePlumB instructions and gather mission-date list

Some progress on the instructions! Feedback, and requests for clarity in the 'What are we labeling' section appreciated!

Okay, lots here!
This PR is big progress on the following issues: #10, #14, #17, #13, #19, #20. At least some of these can be added to close with the approval and merge of this branch.
#10:
I think this is done. Between your work and the edits, I feel like this is in a good spot.
#14:
Also, I think we're good here. I'm sure there will be tweaks after the undergrads use it.
#17:
Again, we've got a baseline. I added some 'tips' to the end to help folks who come across empty or no-data tiles.
#13:
Again, I think we've covered a lot of this, but this is probably another one I'll get feedback on Monday about.
#19:
This is in the drive and linked in the instructions! This PR definitely closes #19.
#20:
This seems to be a duplicate of 19. I'm going to make myself feel good and close #20, too.
Please read through the instructions and let me know if they look good enough for undergrads! If you think we can close any of the other issues, please say so. :)


Added setting to exclude more item types

Adds a single setting (default false so as not to disrupt anyone updating unknowingly) that when enabled prevents the Spells tab from appearing on classes, subclasses, backgrounds, and features.
Included 'race' as well preemptively.

don't do features, features is genuinely useful, esp for Feats, now that owned items' spells can be modified it's great for magic initiate and the like
also warlock invocations

Might simply make a single button in the module settings to pop a dialog with checkboxes for each item type.


Javascript error, keeps throwing every second.

Any ideas how to solve this error?
TypeError: Cannot read property '0' of undefined
at Scope.$scope.setColor (http://appsforce.back/myboard/js/controllers/boardsItemView.js:49:58)
at Array.$parseFunctionCall (http://appsforce.back/myboard/lib/angular.js:12404:18)
at $parseObjectLiteral (http://appsforce.back/myboard/lib/angular.js:12468:38)
at Object.regularInterceptedExpression (http://appsforce.back/myboard/lib/angular.js:12910:21)
at Scope.$digest (http://appsforce.back/myboard/lib/angular.js:14300:40)
at Scope.$apply (http://appsforce.back/myboard/lib/angular.js:14571:24)
at done (http://appsforce.back/myboard/lib/angular.js:9698:47)
at completeRequest (http://appsforce.back/myboard/lib/angular.js:9888:7)
at XMLHttpRequest.requestLoaded (http://appsforce.back/myboard/lib/angular.js:9829:9) (11:21:42:094 | error, null)
at http://appsforce.back/myboard/lib/angular.js:11655

@kiswa What input do you need

That tag was to see if anyone else had ideas on this.
My only guess is that the build wasn't run, or didn't complete correctly.

I was having this issue as well after following the instructions at https://taskboard.matthewross.me/docs/intro.html
So since I had a libs.min.js and app.min.js file (like the demo board), I just copied and pasted the code from the demo board javascript files https://taskboard.matthewross.me/demo/lib/libs.min.js and https://taskboard.matthewross.me/demo/js/app.min.js into my local files and the problem was resolved.

Well, I do still see the error when trying to save and edit tasks.. but no longer every second. hmmm

These errors still appear in the latest version of TaskBoard


Crash on Launch

Issue description: Crash on launch
What happens: When I launch Minecraft (All The Mods Modpack), the game simply crashes after about 10 minutes of loading. I have given the game 6 gigs of RAM.
What you expected to happen: The game to launch like normal
Steps to reproduce:

Install Refined Storage + Forge
Launch Minecraft
Crash
...

Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>5
Refined Storage: 1.2.14

Does this issue occur on a server? No
If a (crash)log is relevant for this issue, link it here: http://pastebin.com/amFgFgbm

you ran out of memory. not an rs issue

So do I give the game more RAM? Sorry im new to this :/

no, you contact the modpack authors

Alright I see, thanks :)


ButtonType in XML-View within Semantic Page has no effect on the button

OpenUI5 version: 1.62.1
Browser/version (+device/version): Chrome Version 72.0.3626.109 (Official Build) (64-bit) Desktop Device
URL (minimal example if possible): https://jsbin.com/vovaniciga/edit?js,output
Steps to reproduce the problem (see the above url example):

Create an XML-View with a semantic page based on following module: sap/f/semantic/SemanticPage
Create a titleCustomTextActions aggregation with a Button in it and try to define any available ButtonType. Button module: sap/m/Button

What is the expected result?
Based on the defined ButtonType on the Button the styling should change accordingly. When you try to set the type of the button to "Emphasized" for example, it should have a blue background.
What happens instead?
The type of the button does not change and is always defined as Transparent for some reason.
Any other information? (attach screenshot if possible)
As far as I can tell the type will be correctly set in the XMLVTemplatProcessor, but somewhere overwritten with the type Transparent.

Hello @e-wag ,
Thank you for sharing this finding.
That is the expected behavior by design but I've created an internal incident<PHONE_NUMBER> to adjust the documentation, since it is not mentioned there.
The status of the issue will be updated here in GitHub.
Regards,
Niya


State is shown off even when on

Om home app State is shown off even when the camera is on. Switch will work fine on setting on and off but next time home app is opened the status reads again "OFF"

OK managed to have a workaround for this.
I used a stateful "fake" switch using the plugin below to associate to the camera switch and then the state persistence is working
homebridge-statefuldummy

Thanks a lot for the PR. Just merged it and published the updated package.

I confirm it has been stable for more than 4 days now. Thank you guys.

@DJay-X you should try the updates now.

@williamli Thanks a lot. I did and tried it a couple of times with success. It seems to work smooth now. Fingers crossed.
Thanks again to you and of course @josepramon for your work. It helps a lot. 👍🏻


chore: Fix eslint failures in CI

Before running eslint, you have to cd into src/webapp. As a result, the pre-commit eslint hook was failing when run. We didn't notice this for quite a long while, presumably because the hook was not being run often at all because we don't update our Javascript code often. In #6800 it caused a failure, which prompted me to open #6803.
Making pre-commit run from a subdirectory is somewhat tricky. The maintainer of pre-commit is adament about not supporthing this feature, and suggests that users hack their way around the limitation as I have done here.
I also took the liberty of updating our JS linting plugins to their latest versions. Many of them are duplicated 2 or 3 times across the repository, which is unfortunate, and will surely cause problems in the future (and may already be causing problems now).
I have eslint autofixing where possible. This PR also includes the autofixes that resulting from running pre-commit run --all-files eslint. A few manual fixes were applied to the files in docs/js/, as well as a few ignores. Some for an undefined variable that was clearly defined, and others for various Vue issues that I am not comfortable fixing at this time.
For the undefined variable there's probably some proper way of making it count as defined, but as-is my editor seems to recognize it as defined, and the code seems to actually work - I'd rather not break it.
Closes #6803

The main files of interest to reviews will be .pre-commit-config.yaml and src/webapp/.eslintrc.js. Most of the rest is auto-updates applied by eslint.

fwiw, thanks @WillDaSilva I ran into this twice and should've opened this myself.


Linux build supporting older versions of GLIBC

See here for some background:
https://kobzol.github.io/rust/ci/2021/05/07/building-rust-binaries-in-ci-that-work-with-older-glibc.html

I will merge this as a starting point for integrating the Linux builds.
I like the idea of working with musl to avoid these compatibility issues but it is predicated on making sure we have good CI test coverage across LabVIEW supported platforms that it doesn't introduce any issues. If so we can just backup this step to still use manylinux but instead link GLIBC.
The integration testing is part of #139


Build errors when building template example

I get the following build error when building the template example, after installing CADMesh:
[ 20%] Building CXX object CMakeFiles/cadmesh_template.dir/src/DetectorConstruction.cc.o
/data/et119/terasa/git/geant4/CADMesh/examples/template/src/DetectorConstruction.cc: In member function ‘virtual G4VPhysicalVolume* DetectorConstruction::Construct()’:
/data/et119/terasa/git/geant4/CADMesh/examples/template/src/DetectorConstruction.cc:49:54: error: incomplete type ‘G4VisAttributes’ used in nested name specifier
     world_logical->SetVisAttributes(G4VisAttributes::Invisible);
                                                      ^~~~~~~~~
/data/et119/terasa/git/geant4/CADMesh/examples/template/src/DetectorConstruction.cc:52:13: error: expected primary-expression before ‘*’ token
     CADMesh * mesh = new CADMesh((char*) "det.stl");
             ^
/data/et119/terasa/git/geant4/CADMesh/examples/template/src/DetectorConstruction.cc:52:15: error: ‘mesh’ was not declared in this scope
     CADMesh * mesh = new CADMesh((char*) "det.stl");
               ^~~~
/data/et119/terasa/git/geant4/CADMesh/examples/template/src/DetectorConstruction.cc:52:15: note: suggested alternative: ‘cosh’
     CADMesh * mesh = new CADMesh((char*) "det.stl");
               ^~~~
               cosh
/data/et119/terasa/git/geant4/CADMesh/examples/template/src/DetectorConstruction.cc:52:26: error: expected type-specifier before ‘CADMesh’
     CADMesh * mesh = new CADMesh((char*) "det.stl");

The first error is probably caused by restructuring of the GEANT4 version I link against (10.4), and can be fixed by #include "G4VisAttributes.hh". (though the developers warn that usage of ::Invisible is deprecated and should be replaced by a call to ::GetInvisible())
From my understanding of the codebase the other errors are caused by the absence of a CADMesh constructor in the code. Shouldn't this code call the TessellatedMesh constructor explicitely? I managed to fix it by calling the CADMesh::TessellatedMesh constructor instead, but I do not know if that is the way it is intended to work. If yes, I can prepare a PR.

Is this on the master branch? If so, I am making updates in preparation for version 2, and I havne't yet updated the template example I don't think.
In the mean time, v1.1 is the current release version, and can be found here:
https://github.com/christopherpoole/CADMesh/releases/tag/v1.1
https://github.com/christopherpoole/CADMesh/tree/v1.1

Just updated the template example to reflect the ongoing changes to the API. See here.

Great. I was looking at this example to implement the recent changes in my g4py interface to CADMesh, but I should probably wait until v2 is final.


feat: Migration to .NET 8

Closes #562

@nminaya please check this PR

@Jadhielv Why not migrating directly to .NET 9?

@nminaya Thanks for the suggestion. We'll work on migrating directly to .NET 9 in a future update.


Vietnamese Leader

I'm willing to be Vietnamese Dev_Voice leader<EMAIL_ADDRESS>I'm on Discord channel now.

Thanks @TaQuangTien! I'll add you to the list.

Empty Vietnamese templates are in the repository :+1:
Please start with _common.yaml and the homeassistant*.yaml files in sentences/vi

Hello @synesthesiam
I'm interested in making smart speaker for Vietnamese. With some experience on Vietnamese NLP since 2018, I'm very excited to contribute on Rhasspy project to make it well support on Vietnamese language.
I've just submited  the first PR for Vietnamese. I'm willing to be the language leader of Vietnamese.
Thank you.


Where there are examples or APIS for Spring Cloud Sleuth integrates with Spring Integration

I want to traced rabbitMQ by Spring Cloud Sleuth ,by there url I don't know how to do it，Do you have a more detailed description that can help me，Thanks！

You can read the code.
Here are the message headers we're using - https://github.com/spring-cloud/spring-cloud-sleuth/blob/v1.3.0.RELEASE/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/messaging/TraceMessageHeaders.java
This is how we create a span from a message - https://github.com/spring-cloud/spring-cloud-sleuth/blob/v1.3.0.RELEASE/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/messaging/HeaderBasedMessagingExtractor.java
This is how we fill out a message from a span - https://github.com/spring-cloud/spring-cloud-sleuth/blob/v1.3.0.RELEASE/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/messaging/HeaderBasedMessagingInjector.java


Set up data tests in PLUTO

Fixes #962. See the issue for test framework details.
This PR establishes dbt framework for running data tests post PLUTO build, keeping in mind future migration of the entire PLUTO pipeline to dbt.
I also implemented first data test that I've been working on with Amanda; this test helps us identify DOF PTS condo bbls that likely have incorrect residential unit count.
Data Test description

As mentioned above this test helps us identify DOF condo records that likely have incorrect residential unit number. The test consists of 2 parts:

First, we check DOF semi-raw data (pluto_rpad_geo table) to find prime bbls where multiple unit condos have res unit > 1. Why? Because each residential unit in a condo building should have exactly one condo.

For example, the following prime bbl would be flagged if it has the following records:

primebbl
bbl (unit bbl)
coop_units (res units)

75123
1
3

75123
2
3

75123
3
1

Second, we cross-check this prime bbl with the corrections file pluto_input_research.csv, the Developments database (DevdB), and a file of bbls that should be ignored during the check.

Why there is a file of bbls that should be ignored during the check? Because DevDB isn't perfect and it only contains building construction records going back to 2010ish. So for example, if a lot has 2 buildings built in 1980 and only 1 building was renovated post 2010ish, DevDB will should residential units for that 1 building.

test logic tree:

flowchart TD

check_name["condominium has multiple units with coop_units > 1?"]
corrected["Has res unit total been manually corrected?"]
in_ignore_file["Is in the 'ignore' csv file?"]
devdb["Is res unit total aligns with DevDB?"]
over_50["Is res unit count diff >= 50?"]
fix{"Manually research bbl. Needs correction?"}
add_to_corrections["Add to corrections & report to DOF"]
add_to_ignore_file["Add to 'ignore' csv file"]

ignore{"Ignore ✅"}

check_name --> |Yes| corrected
check_name --> |No| ignore

corrected --> |Yes| ignore
corrected --> |No| in_ignore_file

in_ignore_file --> |Yes| ignore
in_ignore_file --> |No| devdb

devdb --> |Yes| ignore
devdb --> |No| over_50

over_50 --> |Yes| fix
over_50 --> |No| ignore

fix --> |Yes| add_to_corrections
fix --> |No| add_to_ignore_file

dbt lineage graph:

Commits

Initiate dbt project inside of PLUTO directory

Add DevDB dataset to PLUTO recipes. We need DevDB for the data test

Create a custom dbt test for residential units in tests/assert_condo_bbl_unit_count_research_required.sql file. In dbt, you can create your own custom, "singular" test which is pretty much a sql query stored in a tests directory. This is slightly different from macros that we've created before in a way that it's not reusable for different tables.

Test description and our next steps in the case of failure are documented in the config object of the test file. At the moment, dbt doesn't have description property for tests, but it should be available with dbt next release (they recently implemented this feature here).
This test is refactored in commit 4, so it may not be necessary to check out this commit; I kept it just in case.

Refactor the test by creating intermediate tables. The reasons to do so are: 1) make the test more readable and 2) use these intermediate tables for creating reports to send to DOF.

I created models/qaqc/ directory that is intended for test/qaqc intermediate tables that are NOT a part of the build and for qaqc reports. As we migrate PLUTO build, we can use the reports subdirectory for QAQC app tables. I also added yaml files to describe various tables in the qaqc dir. This directory contains some of the refactored tables used in the test.
Simplify the test itself given the refactored tables.
Add a seed file that will contain condo bbls that should be ignored during the test. This seed file is manually filled out during manual research step.

Run dbt test as a separate step in PLUTO build GHA.

Test Documentation in dbt

As I previously mentioned, dbt is in process of implementing description property for tests: i.e. we will be able to document generic and singular tests in a yaml file. And then the test descriptions will be parsed to dbt docs pages here:

For now, we can document tests in a test's config.meta property which can be a dict with any key-value pairs. This property is available for both generic and singular tests.

Where test results and test properties can be accessed? In dbt-generated artifacts: manifest.json & run_results.json

This is very cool. Do you have any examples of runs, and the output json files from running tests?

This looks great! I'd leave the actual stamp for Damon and Finn, who know more about DBT norms and PLUTO than I do. Also curious about the outputs, e.g. how many offenders this finds.

@alexrichey, thank you! Will provide output files asap. When working on PLUTO 24v3, ~26 bbl records failed the check. Out of the 26, about a half of condo bbls needed a manual correction.

dbt output files:

manifest.json - this file has model & test attribute info.

snippet:

    "test.pluto.assert_condo_bbl_unit_count_research_required": {
        "database": "db-pluto",
        "schema": "sf_pluto_dbt_v2__tests",
        "name": "assert_condo_bbl_unit_count_research_required",
        "resource_type": "test",
        "package_name": "pluto",
        "path": "assert_condo_bbl_unit_count_research_required.sql",
        "original_file_path": "tests/assert_condo_bbl_unit_count_research_required.sql",
        "unique_id": "test.pluto.assert_condo_bbl_unit_count_research_required",
        "fqn": [
            "pluto",
            "assert_condo_bbl_unit_count_research_required"
        ],
        "alias": "assert_condo_bbl_unit_count_research_required",
        "checksum": {
            "name": "sha256",
            "checksum": "f57c898be98c0eefcecc517436fbd5e156e4e4574fc8f3541abae1c516f57e51"
        },
        "config": {
            "enabled": true,
            "alias": null,
            "schema": "_tests",
            "database": null,
            "tags": [
                "de_check"
            ],
            "meta": {
                "description": "\n\t\t\t\tThis test identifies condo records that likely have incorrect unit counts in DOF PTS data, affecting the final PLUTO table. \n\t\t\t\tThe first 2 CTEs determine prime BBLs for condos with multiple unit records where their residential unit count != 1 (expected value). \n\t\t\t\tThe remaining logic cross-checks these BBL \"offenders\" with the PLUTO corrections file and the Developments database (aka DevDB) to narrow down the records needing manual review.\n\t\t\t",
                "next_steps": "\n\t\t\t\t1. Manually research failing records. \n                2. If confirmed, update the corrections file with the correct total and res unit number; \n                not confirmed bbls need to be added to the `ignored_bbls_for_unit_count_test` seed file.\n                3. Re-run PLUTO build. If this check passes, share reports from `models/qaqc/reports` with Amanda to send them to DOF.\n\t\t\t"
            },
            "group": null,
            "materialized": "test",
            "severity": "ERROR",
            "store_failures": true,
            "store_failures_as": "table",
            "where": null,
            "limit": null,
            "fail_calc": "count(*)",
            "warn_if": "!= 0",
            "error_if": "!= 0"
        },
        "tags": [
            "de_check"
        ],

run_results.json - this one has test results.

snippet:

    "results": [
    {
        "status": "fail",
        "timing": [
            {
                "name": "compile",
                "started_at": "2024-10-07T22:16:38.607169Z",
                "completed_at": "2024-10-07T22:16:38.687421Z"
            },
            {
                "name": "execute",
                "started_at": "2024-10-07T22:16:38.690504Z",
                "completed_at": "2024-10-07T22:17:12.531862Z"
            }
        ],
        "thread_id": "Thread-1 (worker)",
        "execution_time": 33.95416831970215,
        "adapter_response": {
            "_message": "SELECT 1",
            "code": "SELECT",
            "rows_affected": 1
        },
        "message": "Got 12 results, configured to fail if != 0",
        "failures": 12,
        "unique_id": "test.pluto.assert_condo_bbl_unit_count_research_required",
        "compiled": true,
        "compiled_code": "\n\n-- Find records where multiple unit BBLs have coop_apts > 1\nWITH condo_prime_bbls AS (\n    SELECT primebbl\n    FROM \"db-pluto\".\"sf_pluto_dbt_v2\".\"pluto_rpad_geo\"\n    WHERE coop_apts > 1\n    GROUP BY primebbl\n    HAVING count(*) > 1\n),\n\nprimebbl_offenders AS (\n    SELECT\n        primebbl,\n        sum(coop_apts) AS coop_apts,\n        sum(units) AS units\n    FROM \"db-pluto\".\"sf_pluto_dbt_v2\".\"pluto_rpad_geo\"\n    WHERE\n        primebbl IN (SELECT primebbl FROM condo_prime_bbls)\n        AND tl NOT LIKE '75%'\n    GROUP BY primebbl\n),\n\n-- Identify bbl offenders that are not actively corrected\nuncorrected_primebbl_offenders AS (\n    SELECT *\n    FROM primebbl_offenders\n    WHERE primebbl NOT IN (\n        SELECT bbl\n        FROM \"db-pluto\".\"sf_pluto_dbt_v2\".\"qaqc_int__active_condo_bbl_unitsres_corrections\"\n    )\n),\n\nnot_ignored_primebbls AS (\n    SELECT *\n    FROM uncorrected_primebbl_offenders\n    WHERE primebbl NOT IN (\n        SELECT bbl\n        FROM \"db-pluto\".\"sf_pluto_dbt_v2\".\"ignored_bbls_for_unit_count_test\"\n    )\n),\n\n-- Join DevDB with uncorrected offenders\noffenders_joined_with_devdb AS (\n    SELECT\n        l.*,\n        r.units_co,\n        r.classa_prop,\n        r.count_bins,\n        l.coop_apts = r.classa_prop AS dof_matches_devdb_units,\n        r.classa_prop - l.coop_apts AS diff\n    FROM not_ignored_primebbls AS l\n    LEFT JOIN \"db-pluto\".\"sf_pluto_dbt_v2\".\"qaqc_int__devdb_bbl_units_summary\" AS r\n        ON l.primebbl = r.bbl\n)\n\n-- Final selection of records where the unit difference exceeds threshold\nSELECT *\nFROM offenders_joined_with_devdb\nWHERE diff >= 50 OR diff <= -50\t-- 50 is an arbitrary threshold\nORDER BY dof_matches_devdb_units",
        "relation_name": "\"db-pluto\".\"sf_pluto_dbt_v2__tests\".\"assert_condo_bbl_unit_count_research_required\""
    }
],

cc: @alexrichey @fvankrieken @damonmcc


Participant-joined callback

Is there a way to know when a participant joined and left the room?
My goal is to stop automatically stop streaming when all participant leave, and start streaming again when a participant joins again.

Webhooks are pretty cool! In this case however the go-server-app runs on many different machines, each app automatically connecting to their own room, each most likely behind some kind of firewall. Setting up webhooks to all these apps is quite hard to impossible (cries in NAT). I was hoping to get these events through the already-established signaling channel.
Just now I saw room.GetParticipants(), and it seems I can poll this to get an up-to-date list of participants so that works just as well!

With the Go SDK, you can register for a callback for participant changes: https://github.com/livekit/server-sdk-go/blob/main/callback.go#L96
example

oooh, thank you so much! Totally missed that!


permission concept for links

What feature do you want to see added?
When adding links to jenkins itself those links might not work when the target requires admin permissions. So some links could be restricted to admins only
Another idea would be to add a new permission that grants the right to see the link. Then we could have 3 levels of access:
admin only
link permission
anyone
Upstream changes
No response
Are you interested in contributing this feature?
No response

Hey @mawinter69 Sir, can I work on this issue?