Deadlock in 2.0.8 causes ANR

Before filing an issue please check that the issue is not already addressed by the following:

Github Issues
Changelog

Description
[Description of the issue]
Steps to Reproduce

User installs widget to app. Widgets needs application object also containing Twilio stack.
User observes ANR between app and widget running in background

Code (optional)
Expected Behavior
no lock
Actual Behavior
lock, ANR
Reproduces how Often
seems to happens once a day on that users device
Logs
I have this stack trace from Google Play
"main" prio=5 tid=1 Native
  | group="main" sCount=1 dsCount=0 obj=0x761afea0 self=0x7632c96a00
  | sysTid=10981 nice=0 cgrp=default sched=0/0 handle=0x7636adda98
  | state=S schedstat=(<PHONE_NUMBER>1<PHONE_NUMBER>5 52491 ) utm=3670 stm=544 core=1 HZ=100
  | stack=0x7fc238f000-0x7fc2391000 stackSize=8MB
  | held mutexes=

  #00  pc<PHONE_NUMBER>01ad44  /system/lib64/libc.so (syscall+28)

  #01  pc<PHONE_NUMBER>067e00  /system/lib64/libc.so (pthread_cond_wait+96)

  #02  pc<PHONE_NUMBER>70b0a0  /data/app/de.spieleck.app.badgers-1/lib/arm64/libtwilio-rtd-native.so (???)

  #03  pc<PHONE_NUMBER>3a8e54  /data/app/de.spieleck.app.badgers-1/lib/arm64/libtwilio-rtd-native.so (Java_com_twilio_chat_ChatClient_nativeUpdateToken+472)

  #04  pc<PHONE_NUMBER>0478b0  /data/app/de.spieleck.app.badgers-1/oat/arm64/base.odex (Java_com_twilio_chat_ChatClient_nativeUpdateToken__Ljava_lang_String_2JLcom_twilio_chat_StatusListener_2+188)

  at com.twilio.chat.ChatClient.nativeUpdateToken (ChatClient.java)

  at com.twilio.chat.ChatClient.updateToken (ChatClient.java:371)
- locked <0x00e0e959> (a com.twilio.chat.ChatClient)

  at de.spieleck.app.badgers.chat.twilio.BasicChatClient.onTokenUpdated (BasicChatClient.java:304)

  at com.twilio.accessmanager.AccessManager.updateToken (AccessManager.java:82)

  at de.spieleck.app.badgers.chat.twilio.BasicChatClient$GetAccessTokenAsyncTask.onPostExecute$552c4e01 (BasicChatClient.java:196)
  or                     .onPostExecute (BasicChatClient.java:196)

Or attach it as a file.
Chat Android SDK
2.0.8
Android API
7.1
Android Device
Motorola Moto Z

Please collect logs as described here. Full unedited logs help reproduce and fix issues faster.

As I cannot reproduce this on local devices or emulators, but it happens on a users one. Is there a best practise, how to collect Twilio logs on a users device?

Do you collect Crashlytics logs? I think they're able to save at least relevant part of the user's log for analysis.
This snippet is clearly not enough to identify the problem. If you could post your source code, please do so via private support.twilio.com ticket then I will be able to analyse and try to reproduce. Otherwise, only more detailed logs would help, preferably in DEBUG log level.

No luck in reproducing this.
I will close this for now, but if you get a more reproducible case or better logs from crashlytics, don't hesitate to reopen.


enhance: display network sandbox info for debug

Signed-off-by: Eric Li<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
enhance: display network sandbox and more other informations in NetworkSettings for pouch inspect
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
NONE
Ⅳ. Describe how to verify it
root@ubuntu-1:~/pouch# pouch inspect 1eff
[
    {
        "Args": null,
        "Created": "2018-06-24T06:45:10.18231873Z",
        "ExecIDs": null,
        "GraphDriver": {
            "Data": {
                "MergedDir": "/var/lib/pouch/containerd/state/io.containerd.runtime.v1.linux/default/1eff63a55384f8900c7a445813f4ab3f1925cd19fbca427eb0c1f48c4af99639/rootfs",
                "UpperDir": "/var/lib/pouch/containerd/root/io.containerd.snapshotter.v1.overlayfs/snapshots/835/fs"
            },
            "Name": "overlayfs"
        },
        "HostConfig": {
            "LogConfig": {
                "Type": "json-file"
            },
            "NetworkMode": "bridge",
            "OomScoreAdj": -500,
            "RestartPolicy": {
                "Name": "no"
            },
            "Runtime": "runc",
            "ShmSize": 0,
            "BlkioDeviceReadBps": null,
            "BlkioDeviceReadIOps": null,
            "BlkioDeviceWriteBps": null,
            "BlkioDeviceWriteIOps": null,
            "BlkioWeightDevice": null,
            "CgroupParent": "default",
            "DeviceCgroupRules": null,
            "Devices": [],
            "MemoryExtra": 0,
            "MemorySwappiness": 0,
            "MemoryWmarkRatio": 0,
            "OomKillDisable": false,
            "Ulimits": null
        },
        "Id": "1eff63a55384f8900c7a445813f4ab3f1925cd19fbca427eb0c1f48c4af99639",
        "Image": "registry.hub.docker.com/library/busybox:latest",
        "Mounts": [],
        "Name": "1eff63",
        "NetworkSettings": {
            "Networks": {
                "bridge": {
                    "Aliases": null,
                    "EndpointID": "3fb1ca7f73e0f4c2da7caded9b133d77c352278358959b34865cd96644d9d5f0",
                    "Gateway": "<IP_ADDRESS>",
                    "IPAddress": "<IP_ADDRESS>",
                    "IPPrefixLen": 24,
                    "Links": null,
                    "MacAddress": "02:42:c0:a8:05:02",
                    "NetworkID": "0f07a5852c5cb1ce5254108ed68134f06bbc3f60a0cae8999a403a6c07ea04c5"
                }
            },
            "SandboxID": "7a78335d06fca3ea79c1c129b4a8b5c8ea29e57db9db666a7fa71ddaa36be0c4",
            "SandboxKey": "/var/run/pouch/netns/7a78335d06fc",
            "SecondaryIPAddresses": null,
            "SecondaryIPv6Addresses": null
        },
        "Snapshotter": {
            "Data": {
                "MergedDir": "/var/lib/pouch/containerd/state/io.containerd.runtime.v1.linux/default/1eff63a55384f8900c7a445813f4ab3f1925cd19fbca427eb0c1f48c4af99639/rootfs",
                "UpperDir": "/var/lib/pouch/containerd/root/io.containerd.snapshotter.v1.overlayfs/snapshots/835/fs"
            },
            "Name": "overlayfs"
        },
        "State": {
            "FinishedAt": "2018-06-24T06:58:14.12474128Z",
            "Pid": 1757,
            "StartedAt": "2018-06-24T06:58:26.45954277Z",
            "Status": "running"
        },
        "Config": {
            "Cmd": [
                "sh"
            ],
            "Entrypoint": null,
            "Env": [
                "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
            ],
            "Hostname": "1eff63a55384",
            "Image": "registry.hub.docker.com/library/busybox:latest",
            "OnBuild": null,
            "Shell": null,
            "Tty": true
        }
    }
]

Ⅴ. Special notes for reviews

Codecov Report

Merging #1589 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1589      +/-   ##
==========================================
- Coverage   41.69%   41.66%   -0.04%
==========================================
  Files         268      268
  Lines       17408    17404       -4
==========================================
- Hits         7258     7251       -7
- Misses       9256     9258       +2
- Partials      894      895       +1

Impacted Files
Coverage Δ

apis/server/container_bridge.go
82.43% <100%> (-0.32%)
:arrow_down:

daemon/mgr/system.go
75% <0%> (-1.73%)
:arrow_down:

ctrd/image.go
77.71% <0%> (-1.72%)
:arrow_down:

daemon/logger/jsonfile/utils.go
73.17% <0%> (+1.62%)
:arrow_up:

Do we need to add some fields in result of pouch network inspect to lock it? @shaloulcy @rudyfly

LGTM


Support for emitting tuples in the native format

Straightforward port from NativeFormatWriter.cpp.
It's not used yet, but will be. I just don't want to sit on this or I'll
lose it before I get to using it.

👍


Debug style & milestone

Next step: get reviews + courses data from s3 after backend finishes processing the data


Show something in "My Rules" even when none exist yet

Not sure if something should be in that panel, but apart from a "My Rules" header there is nothing if no rules were created yet. Is that expected behaviour for the current version?
If there are no rules yet, perhaps a short text indicating so with a hint on how to create them might be helpful. Otherwise users might think something failed to render (as I did at first 😄).
EDIT: Now that I think about it, this is probably the wrong repo as the Almond GUI is most likely not specific to that HA add-on.

Fixed by https://github.com/stanford-oval/almond-server/pull/198


Pagination info returned has incorrect values

I’m using the new pagination API and everything works except that the NBPaginationInfo object I get back has incorrect properties. The numberOfTotalPages, numberOfTotalItems and numberOfTotalAvailableItems properties are all 0, making it impossible for me to know how many items to expect.
Is this a known issue, or could it be something wrong with my implementation?

You better get on this, @hlfcoding!

@tomlongo To use legacy pagination, you need to make sure in the NB Control Panel your app still is able to use legacy pagination. See the NB docs for more info. For most apps, especially ones created since the introduction of the new token-based pagination (late 2014), this is no longer possible. If you're sure you can use legacy pagination, just set shouldUseLegacyPagination to true (defaults to false), on your client instance. numberOfTotalPages, numberOfTotalItems, numberOfTotalAvailableItems (computed), are all 0 when legacy pagination isn't available one way or the other.
@DavidHuie Thanks for the reminder!

Hello @hlfcoding thanks for the help. I see so what you're saying is that the numberOfTotalPages etc. is only available in the legacy pagination, not with the new? I'd rather use the new one given the legacy is deprecated, but is there a way I can get that same information using the new method?

Unfortunately, and mostly for performance reasons I cannot get into here, the NationBuilder API currently does not have a separate endpoint that provides item or page counts, and only the legacy pagination mode provides that information. The new pagination response is geared towards infinite scroll and simple prev-next button use cases.

hmm, that's a shame. The way I'm implementing this is to use a local syncing model, so it goes off and fetches all people at once, stores it in CoreData, them from then on it only pings the API for records changed since the last sync every few minutes.
This means viewing/searching people is instantaneous for the user, and it takes the pressure off your API because the app never needs to query it unless updating data.
However, that initial sync takes some time for users with a lot of people, so I need a way of determining the total pages so i can show them a progress bar. How safe is it to use the legacy pagination? I'm weary using something that is deprecated is all.

This means viewing/searching people is instantaneous for the user, and it takes the pressure off your API because the app never needs to query it unless updating data.

I'm not sure that's actually better, because it sounds like your app could be fetching a lot of data all at once. I may be wrong about your app, but if your users ever all update at around the same time (ie. for some event), it could actually increase load on our API. We've had issues with large nations that perform large-scale people-sync via API.

However, that initial sync takes some time for nations with a lot of people, so I need a way of determining the total pages so i can show them a progress bar.

An alternate approach you can do is sort-fetch the newest person and the oldest (first added) person, then guesstimate based on the last synced id how many people are left.

How safe is it to use the legacy pagination? I'm weary using something that is deprecated is all.

If you can see a 'Use legacy pagination' checkbox in your nation's admin/oauth/test_tokens path, go ahead and use it. This is my unofficial (but probably accurate) forecast: we will not foreseeably remove legacy pagination support altogether, and will notify ahead of time if that happens. You should reach out to our services team if you need additional help, or if you want to ask for an exception to be made for your nation to allow legacy pagination to be enabled.
I feel like we've reached the end of discussion. Closing...


Updated README

For now, we may want to keep the repo README short and concise and instead update the files located in docs/. It won't be scalable to update the documentation in two places. Once we move to sauce-docs we can update the links here after removing the docs/ directory.

Thanks for contributing to the saucectl project!
A PR well described will help maintainers to quickly review and merge it
Before submitting your PR, please check our contributing
guidelines, applied for this repository. Avoid large PRs, help reviewers by making them as simple
and short as possible.

Description

Motivation and Context

Types of changes

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

Checklist

[x] I have read the contributing document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

After the discussion today in the Retrium, I'm going to close this PR.


Atomic Constants

Description
Need easy access to reasonably standard atomic constants (mass, etc...)
Need access to atomic constants that are specific to elastic scattering (scattering factors, Cromer-Mann coefficients, etc...), inelastic scattering (has some constants), X-ray fluorescence (also has some contants, etc...
Existing Options
Existing options may not solve the entire problem, but they serve as a starting point, either to beg/borrow/incorporate or simply as education on how someone else did this

[ ] pyspec
[ ] xraylib

Installation Procedures
To get xraylib set up 'the easy way'
from @tacaswell:

checkout my fork of conda-recipes and the xraylib branch (https://github.com/tacaswell/conda-recipes/tree/xraylib)

conda build ~/other_source/conda-recipes/xraylib/

(modify this to point at the actual location your system built)
conda install -c file:///home/tcaswell/miniconda/conda-bld/ xraylib
At that point you should be able to import xraylib in python.
You will need autoreconf installed for this to work

Does anyone know what the Atomic Scattering function in xraylib units are?  Also, it returns a 3 element list but what are those values, and with which units?

https://github.com/tschoonj/xraylib/wiki/The-xraylib-API-list-of-all-functions#scattering-factors

Thank you!

I'm pretty sure this was done when we added xraylib to scikit-xray


remove docs

addresses #157

@kgiusti  I wonder whether I should also remove the man page docs, given that we don't intend users to ever start the router from the cli?

@kgiusti I wonder whether I should also remove the man page docs, given that we don't intend users to ever start the router from the cli?

I'd keep them, certainly the "qdmanage/qdstat" stuff still applies - but that's just my $0.02 so I'm happy to go along with the general consensus

@kgiusti  resolved conflict, ok to merge?

@kgiusti resolved conflict, ok to merge?

@pwright merge man, merge!


Update: Uauth - walletconnect v1 deprecation handling

Description
This PR will be merged after this PR for final WalletConnect V1 deprecation.
This update includes bringing UAuth to handle WalletConnect V2 usage and removal of V1 including:

Removing the V1 bridge url defaulting to the WC bridge as WC has removed support for this bridge
Adding optionalChains as init prop if wanting to override the W3O init chains as the optional chains
Adding additionalOptionalMethods to add further methods outside of the default methods defined
Update Docs around this change

PLEASE NOTE- Checklist must be complete prior to review.
Checklist

[x] Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
[x] Check the box that allows repo maintainers to update this PR
[x] Test locally to make sure this feature/fix works
[x] Run yarn check-all to confirm there are not any associated errors
[x] Confirm this PR passes Circle CI checks
[x] Add or update relevant information in the documentation

Docs Checklist

[ ] Include a screenshot of any changes (see docs README on running locally)
[ ] Add/update the appropriate package README (if applicable)
[ ] Add/update the related module in the docs demo (if applicable)
[ ] Add/update the related package in the docs/package.json file (if applicable)

If this PR includes changes to add an injected wallet or SDK wallet module:
Please complete the following using the internal demo package.
To run this demo use the command yarn && yarn dev to get the project running at http://localhost:8080/
Tests with demo app (injected)

[ ] send transaction
[ ] switch chains
[ ] sign message
[ ] sign typed message
[ ] disconnect

Tests with demo app (SDK)

[ ] send transaction
[ ] switch chains
[ ] sign message
[ ] sign typed message
[ ] disconnect

@V-Shadbolt can you have a look when you have a moment? Would like to get out quick as WC V1 deprecation officially today.

Login looks to be working as expected on my dev environment with the new WC v2 implementation!


Bug Fix: Add g_ to name of g vector name in ps_point

Just a quick bug fix introduced in #2828 to add std::string("g_") to the names of the ps_point method.
Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Intended Effect
How to Verify
Side Effects
Documentation
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Steve Bronder
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

In order to prevent the fail on Travis, this PR needs a merge with fresh develop. Sorry for the inconvenience.

Ahhh, oh my b. Aight cool I'll close this


Fix keywords for 🍡

🍡  is Japanese sweet, not 🍖  🍗 !

LOL how did this happen. I gotta try :dango: next time in :jp:!
Thanks Risa! :sparkles:


Cannot add step to native iOS build

Hi,
I have some Xamarin tests in the Xamarin Test cloud, i want to be able to add this step to my workflow so that I can upload the ipa.
I can locally do:
test-cloud submit  ipaFile cloudToken --devices deviceToken --series "master" --locale "en_GB" --user userEmail
And this works so i would like to do the same for a normal built ipa file. Is there some check to make sure the project is a Xamarin project? if so, i don't think its necessary.

actually i might just fork and make a pull request for this

So, the problem is that this step is only available for Xamarin projects right now and you want to use it for non Xamarin apps too? Or is there something else preventing you from using this step for non Xamarin apps?

Yes that's it. I'm using it for a native ios project not a Xamarin project. I have the command working, but in bitrise its a non supported step for my workflow.
Sent from my iPhone

On 25 Jun 2016, at 07:53, Viktor Benei<EMAIL_ADDRESS>wrote:
So, the problem is that this step is only available for Xamarin projects right now and you want to use it for non Xamarin apps too? Or is there something else preventing you from using this step for non Xamarin apps?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

So the step works as-it-is for non Xamarin apps, right? In that case we'll only have to remove the related project type tag (actually add the ios one too).
In the meantime you can turn off the "Filter by platform" option on bitrise.io when you search for a step. If the toggle is turned off you'll see every step available in the StepLib.

Yes that's how I am doing it at the moment
Sent from my iPhone

On 25 Jun 2016, at 07:58, Viktor Benei<EMAIL_ADDRESS>wrote:
So the step works as-it-is for non Xamarin apps, right? In that case we'll only have to remove the related project type tag (actually add the ios one too).
In the meantime you can turn off the "Filter by platform" option on bitrise.io when you search for a step. If the toggle is turned off you'll see every step available in the StepLib.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Awesome, we'll release a new version then with the added ios project type tag!
Thanks again for reporting!

Fixed in v0.9.3, available in the steplib now. Thanks for reporting @Reedyuk !


Simplify E-Cash Mint Server Module Consensus Settings

The consensus settings for this module contains a field mint_amounts which allows for arbitrary denomination amounts to be issued:
https://github.com/fedimint/fedimint/blob/9d871a719056415503d92e7d73e05c53d4b145f0/modules/fedimint-mint-common/src/config.rs#L20-L22
Currently we only ever use base-2 amounts, and we do this by generating the amounts and setting mint_amounts to the generated values:
https://github.com/fedimint/fedimint/blob/9d871a719056415503d92e7d73e05c53d4b145f0/fedimintd/src/lib.rs#L55-L60
https://github.com/fedimint/fedimint/blob/9d871a719056415503d92e7d73e05c53d4b145f0/modules/fedimint-mint-common/src/config.rs#L29-L34
https://github.com/fedimint/fedimint/blob/9d871a719056415503d92e7d73e05c53d4b145f0/modules/fedimint-mint-server/src/lib.rs#L789-L791
In the first two examples above, we can also see that the values are being generated using a specified maximum denomination value.
We want to retain the ability to use other bases for denominations, but we don't need to be able to issue arbitrary note amounts with no shared denomination base. Let's replace the mint_amounts field with a field denomination_base which would be used to generate the actual denomination amounts. Let's also put a cap on denomination size at 1 million coins. We can hardcode this value and remove the existing parameter since there isn't much to be gained in tweaking it. Here is the existing parameter:
https://github.com/fedimint/fedimint/blob/662799742c6a13c3342b0b3655bf1d82f6ca1e02/fedimintd/src/fedimintd.rs#L66-L69
Let's add a method to MintGenParamsConsensus to generate the denomination amounts (i.e. consolidate the calls to Tiered::gen_denominations() in the links above).
Originally discussed in https://github.com/fedimint/fedimint/discussions/3327

Consider this self-assigned. I'm already most of the way to having a PR ready.


Added rebuy option to the shop

You can now buy items multiple times using the buy again option.
Checking if the product has installation is now a function

This looks good, but what about instead of the rebuy menu, the menu would ask you how many of that item you would like to buy at same time?  The menu would have 1,2,3,4,5,6,7,8 options.

Hi.
Sure!


return parsed object instead of string ?

Hi,
Thanks for the library, it works great.
I was just wondering if there's any way to receive an object (non stringified) rather than a string, i didn't find anything in the doc about that.
Of course i can call parse myself, but it's a waste because i imagine you must have it as an object and then call stringify on it ? (unless i'm wrong on that point and you directly construct it as a string, in which case never mind)

From docs:
...
result = convert.xml2js(xml, options);    // to convert xml text to javascript object
result = convert.xml2json(xml, options);  // to convert xml text to json text


4-Order

Tasks

[ ] DB table.
[ ] Create.
[ ] List.
[ ] Show.
[ ] Tests.

Involved data

User
Created
Status draft / done
Product cost
Domestic
Shipping speed
Shipping cost
Total cost

Useful

https://github.com/janis-rullis/pr1/wiki

Due date

February 29, 2020

#18 Evaluate @lauris-printify code against requirements in the MVP
DB table

Has all necessary.

Create

Checks ownership.
src/Order/OrderCreator.php.
src/Controller/OrderController.php::createOrder().
src/CostCalculator/DomesticCostCalculator.php
src/Order/OrderCreator.php

List

@Route("/users/{id_user}/orders", name="viewOrders", methods={"GET"})
OrderController::getOrders()
src/Repository/Prod/OrderRepo.php::getAll()

Show

@Route("/users/{id_user}/orders/{id}", name="viewOrder", methods={"GET"})
OrderController::getOrderById()
src/Repository/Prod/OrderRepo.php::getById()x`

Tests

tests/Order/OrderCreatorTest.php
tests/Order/OrderValidatorTest.php


join us

You can transfer it to the AtomLinter org if you'd like. You'll still be the maintainer.
The sooner you do it, the less the statistics of downloads you will lose)
flow:

run apm unpublish linter-roodi command to unpublish package
move repo to AtomLInter org (you will be an administrator)
fix repo url in package.json
publish again

This allows us to more easily keep track of linters that need to be fixed after major updates to the core linter. Additionally, by pulling you into the organization it allows us to keep linter creators up to date with changes and plans people have to advance the project. Lastly it has been a good tool when people have issues with the linters where users can rely on a larger stable of developers to help them with their issues on any given linter.
welcome to the biggest atom package community ::)

Sounds good, when can I do this? What permissions do I need to move the repo to the org?

Done


"This functionality is not implemented in the portable version of this assembly" with Xamarin.droid

It was throwing the following exception only if its built with 3.0.3 .. I had to downgrade the plugin to 2.3.0 in Droid to make it work.
This functionality is not implemented in the portable version of this assembly. You should reference the NuGet package from your main application project in order to reference the platform-specific implementation.
Bug Information
Version Number of Plugin:3.0.3
Device Tested On:Android Galaxy 5.0.1
Simulator Tested On:
Version of VS: 7.2
Version of Xamarin:<IP_ADDRESS>
Versions of other things you are using:
MVVMlight <IP_ADDRESS>
Steps to reproduce the Behavior
Just add the 3.0.3 plugin in both droid and PCL
Run the project on device and it'll crash upon launch
Expected Behavior
It should not cause the crash
Actual Behavior
It crashes the app
Screenshots

Upgrade all of your nugets


License confusion

In the readme it states that the license for BoxBilling is Apache 2.0 but then it goes on to state things that are not in line with that license:
"Those that wish to distribute a modified version of BoxBilling must gain permission from BoxBilling before releasing the software. All authorised modified versions of BoxBilling must retain this copyright notice. All modified releases of BoxBilling must release the software under the same license as the BoxBilling software (Apache License 2.0)"
This is explicitly against the terms of the Apache license which not only says that anyone can copy, modify and distribute without seeking permission but also that they can then apply any license to the derivative work.
"The Apache License is a permissive open source software license — so you can release your modified version of the Apache licensed product under any license of your choice."
I just think that it's important to be clear and consistent about licensing if this project is going to move forward again, and at the moment this is contradictory.
References:
http://www.apache.org/licenses/LICENSE-2.0.html
https://resources.whitesourcesoftware.com/blog-whitesource/top-10-apache-license-questions-answered

Yeah, indeed you’re fully right!
Those licenses agreements are fully wrong.
Check, search for CircleBilling. A fork of BoxBilling and you will see how this is interpreted by other activ dev.
Envoyé de mon iPhone

Le 7 août 2020 à 23:29, S4 Hosting<EMAIL_ADDRESS>a écrit :

In the readme it states that the license for BoxBilling is Apache 2.0 but then it goes on to state things that are not in line with that license:
"Those that wish to distribute a modified version of BoxBilling must gain permission from BoxBilling before releasing the software. All authorised modified versions of BoxBilling must retain this copyright notice. All modified releases of BoxBilling must release the software under the same license as the BoxBilling software (Apache License 2.0)"
This is explicitly against the terms of the Apache license which not only says that anyone can copy, modify and distribute without seeking permission but also that they can do applying any license to the derivative work.
"The Apache License is a permissive open source software license — so you can release your modified version of the Apache licensed product under any license of your choice."
I just think that it's important to be clear and consistent about licensing if this project is going to move forward again, and at the moment this is contradictory.
References:
http://www.apache.org/licenses/LICENSE-2.0.html
https://resources.whitesourcesoftware.com/blog-whitesource/top-10-apache-license-questions-answered
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@fordnox This could do with being cleared up. It will definaitly be putting off some developers from contributing. I have a couple of extentions im working on and would like to choose a licence that works with boxbilling licence but as it stands i get the feeling you may end up changing the licence of box billing.

It will definaitly be putting off some developers from contributing.

Yeah, I'm fairly sure it is. It's super confusing, and a lot of people won't want to contribute code (or documentation or anything else) to a project that then sounds like it is going to grab copyright of those submissions even if that is explicitly against the license.

@fordnox any thoughts on this?

@fordnox @S4-Hosting https://docs.github.com/en/free-pro-team@latest/github/building-a-strong-community/adding-a-license-to-a-repository

Removed confusion from Readme file


feat: add package for asset image

Description
Add the possibility to specify a package name for asset images.
Requirements

[ ] Be able to specify a package name for asset images

Additional Context
Sometimes the assets images are located in a package (if we have a specific package to build a design system for example). If we want to display an image in the app, it can't be loaded because the path can't be found.
The changes are simple, I created a commit for it but I don't have the rights to push to this repository.

hello @Zigotote , thanks for creating this issue and for this nice idea. it will be great to have this feature in this package. please create a PR from your fork repo.

Detailed instructions on how to contribute to this package can be found in the CONTRIBUTING.md file.

Hi @ishafiul, I'm not used to open source contribution and didn't thought about the Contributing file. Thanks for your explanation, I just opened the PR.


Multi Guard

Package
filament/filament
Package Version
3.2
Laravel Version
11
Livewire Version
3.5
PHP Version
8.3.0
Problem description
Hello,
I am encountering an issue in my Laravel project where I have two separate authentication systems — one for regular users and one for administrators via the Filament admin panel. Both use the same database, and it seems that the sessions are overlapping.
Problem description:
Expected behavior
Example 1: When the administrator logs out of the admin panel, the current user session on the main site is also terminated.
The administrator is authenticated via the admin guard, and the user is authenticated via the web guard.
After the admin logs out, the regular user is also logged out from their session on the main site.
Example 2: When the user logs out from the main site, the administrator is also logged out of the admin panel.
Steps to reproduce
The user logs in through the web guard, and the admin logs in through the admin guard.
When the user logs out, the admin session is also terminated, although they should function independently.
Suspected cause: It seems that both systems are sharing the same session or cookie, causing a conflict. I have attempted to configure separate guards for users and admins, but the sessions still overlap.
I would appreciate assistance in properly configuring the system so that logging out from one session (admin or user) does not affect the other.
Thank you in advance for your help!
Reproduction repository (issue will be closed if this is not valid)
https://github.com/DanielCodeWeb/FOTPROGLARAVEL3
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

I think this is the default Laravel behaviour when invalidating the session and I am not aware of any fixes, sorry.


Massmailer bugfixes & improvements

This PR fixes and improves the massmailer module further.
New feature
I've modified the API and front-end so that you can now see the list of recipients for a message. This will automatically be populated after clicking the preview button.

Fixes

Fixed the filters in the UI. Previously, they did not save
Prevented the possibility of a single client being duplicated in the recipient list, (11 times in this example)
Minor optimizations

Before
With the extra info being fetched from the DB & nothing to prevent a client from being added multiple times
Array
(
   [0] => Array
       (
           [id] => 4
           [first_name] => Johnie
           [last_name] => Doe
       )
   [1] => Array
       (
           [id] => 3
           [first_name] => Jannet
           [last_name] => Doe
       )
   [2] => Array
       (
           [id] => 2
           [first_name] => John
           [last_name] => Doe
       )
   [3] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [4] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [5] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [6] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [7] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [8] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [9] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [10] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [11] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [12] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [13] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
   [14] => Array
       (
           [id] => 1
           [first_name] => Jane
           [last_name] => Doe
       )
)

After
Without the extra info being fetched from the DB & prevention from a client from being added multiple times
Array
(
    [0] => Array
        (
            [id] => 4
        )
    [1] => Array
        (
            [id] => 3
        )
    [2] => Array
        (
            [id] => 2
        )
    [3] => Array
        (
            [id] => 1
        )
)

Looks like DISTINCT may have a measurable hit to performance. https://stackoverflow.com/questions/5341276/sql-distinct-keyword-bogs-down-performance/5344601#5344601
I'll mark this as a draft and run through a script to create like 10k test clients and see what kind of impact it has

Results
Time to query the client list was almost identical for both options, however using DISTINCT did end up being a bit faster.
This is pulling 26.5k clients from the DB. I've left it using DISTINCT based on these results.
Using PHP:
Execution time: 0.044584609 seconds
Using DISTINCT:
Execution time: 0.038741386 seconds
Other changes
My most recent commit also fixes the filters in the UI (they previously wouldn't save) and did a micro-optimization in the send workflow to prevent the system from needing to pull the message from the DB for each recipient.

Nice work 🙂.
One observation - is there any reason the array is still a multidimensional array and not a simple list of IDs now the extra data has been dropped?

That's just how it's returned when querying the DB. I'm unsure if we can actually change that behavior


[backport fortress] Calculate camera intrinsics : Refactor

🎉 New feature
Summary
[backport fortress] Calculate camera intrinsics : Refactor
Required in this PR https://github.com/gazebosim/gz-sensors/pull/383
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@osrf-jenkins run tests please!

@iche033 I can't see where is the error on Windows, any thoughts? or we can merge this PR ?

yes merging


How long & Single Object

I am trying to use diffusion SDF for modeling a single object (where each instance is relatively similar to the other instances). I think the model is experiencing some sort of mode collapse and mostly producing an average shape (with some small variations of very easy-to-decipher features). Have you had any experience with this issue? Do you have any suggestions?
Related, what is the timeline for training? The readme says a few days to get to a certain loss. Is a few days usually required? Im currently doing tests with ~1 day (up to 10k epochs - only currently using around 2-300 meshes, which has worked successfully for DeepSDF before), am I maybe not waiting long enough?
I've also tried playing with:

LR
batch size
KLD weight
briefly/currently trying higher:

plane resolution
point cloud size

Upto 1 day of training these all make small differences, but they mostly are converging on the same/similar solution.

Also, I get weird artefacts in all of my reconstructions - either if I used the zero vector, or I try to encode a mesh.
The artefacts are such that I get a smooth surface in the quadrant [-1, -1, -1] and all of the other quadrants have jagged edges. Not sure what could be causing this - maybe something about re-ordering the planes, or sampling?
In the below image, the bottom left is the -1, -1, -1 quadrant. It looks like the other quadrants have somewhat distinct slicing issues.

I have re-created the code for diffusionsdf in my own project, but I have checked everything and think that it should all be the same.

For the first question, can I take a look at samples of the training data, stage 1 reconstructions, and stage 2/3 generations? Our VAE has substantially more parameters than DeepSDF so a longer training time would be expected. For the second question, I don't think I understand. Is the image a generated sample or a reconstruction from stage 1? By artifacts, are you referring to the unsmooth surfaces?

Thanks for the quick reply.
I am only trying to train stage 1 right now. I have generated samples by either: 1) reconstruction from a point cloud, or 2) provide a latent vector to the VAE bottleneck and reconstruct that (I often use a vector of zeros to see what the "average" shape is). The reconstructed and average shapes look similar - they have the same smooth and unsmooth regions with the same patterns (even on different training runs with different training parameters). The attached image was a reconstruction of a mesh from an inputted point cloud.
Yes, by artefacts I mean the unsmooth surfaces - it seems peculiar that it is smooth in one quadrant (<0, <0, <0) and unsmooth in specific ways in the other quadrants. This "artefact" seems to show up in all of my training runs and generations.
And just to say this - that object does look "similar" to what I expect the objects to look like. It is a femur bone. It just doesn't seem to be doing great at reconstruction - e.g, the unsmooth areas, and it is very close to the average shape regardless of the example.

We didn't encounter this issue when training on single categories of the Acronym / ShapeNet datasets. Maybe your data is just too similar. We'd recommend starting with training on just 1 sample and see if the model is able to overfit such that there are no artifacts. If this works, then maybe some tuning will eventually make this work. Feel free to email me if you want to discuss further.


Add OFN UK catalogue source

@simonLouvet as Emilie has left, are you taking this one over now?

check OFN UK et FR


Investigate why Windows PowerShell CI tests are taking so long

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.

Steps to reproduce
All tests running in the Windows PowerShell CI job are taking 4-5x longer than PowerShell Core tests.
For example, a recent run completed with the following times:

PowerShell Core on Windows:   20m
Windows PowerShell at:   1h 33m
PowerShell Core on Linux:   5m
PowerShell Core on MacOS:   10m

Expected behavior
The tests should be running in 30-40 minutes max.

Actual behavior
Currently running in over double that time.

Error details
No response
Environment data
This is not occurring when tests run locally, this is only occurring in CI server.

Visuals
No response

PR #1175 adds caching to Uninstall to improve performance.


adding Quantin 2007 / Hartmann 2007 Chronology functions?

While doing a lit review I cam across Quantin et al 2007 10.1016/j.icarus.2006.07.008 and Hartmann et al 2007 10.1016/j.icarus.2006.09.009
for a chronology function with a decreasing production rate past 3 Gyr. I haven't looked at the full citation graph to work it out, but as far as I can tell Hartmann 2007 is not cited by the @ggmichael 2013 paper. Perhaps there is some more recent thinking on this that makes the 2010 iteration more correct?
In the paper as far as I can tell the full neukum style numerical expression coefficients are not provided in either paper. An expression is given in Hartmann 2007 but I think it just determines the linear trend post 3 Gya rather than the typical expression

For the 2013 paper, I was in contact with W. Hartmann to make sure the results the software was producing were correct. It sounds like I would need to do the same for Quantin et al 2007. Hartmann 2007 was a lunar, not Mars, paper.
There is the possibility for the user to add new chronology systems or, if needed, I'm glad to collaborate on such topics in the context of an actual application.


Lost Origin on death

hi , i am using Origin on my server with Mohist and when i died my origin was lost , and this spaming my console,
Player Droopi29 doesn't have an attached origin capability. This should be possible.
i'm on mohist, with the Origin mod version <IP_ADDRESS> with origin added mod  and when i reconnect i have my origins like before the death !

If a bug cannot be replicated on a regular server, just don't. Mohist is broken, check with them.


PackagesNotFoundError: The following packages are not available from current channels:    - cudatoolkit=11.7

(base) C:\openvoice>conda install pytorch==1.13.1 torchvision==0.14.1 torchaudio==0.13.1 cudatoolkit=11.7 -c pytorch -c nvidia
Collecting package metadata (current_repodata.json): done
Solving environment: unsuccessful initial attempt using frozen solve. Retrying with flexible solve.
Collecting package metadata (repodata.json): done
Solving environment: unsuccessful initial attempt using frozen solve. Retrying with flexible solve.
PackagesNotFoundError: The following packages are not available from current channels:

cudatoolkit=11.7

Current channels:

https://conda.anaconda.org/pytorch/win-64
https://conda.anaconda.org/pytorch/noarch
https://conda.anaconda.org/nvidia/win-64
https://conda.anaconda.org/nvidia/noarch
https://repo.anaconda.com/pkgs/main/win-64
https://repo.anaconda.com/pkgs/main/noarch
https://repo.anaconda.com/pkgs/r/win-64
https://repo.anaconda.com/pkgs/r/noarch
https://repo.anaconda.com/pkgs/msys2/win-64
https://repo.anaconda.com/pkgs/msys2/noarch

To search for alternate channels that may provide the conda package you're
looking for, navigate to
https://anaconda.org

and use the search bar at the top of the page.

solution:
pip install torch==2.1.1 torchvision==0.16.1 torchaudio==2.1.1 --index-url https://download.pytorch.org/whl/cu118
conda install nvidia::cudatoolkit
conda install nvidia/label/testing::cudatoolkit


Infra structure scan entries

This is to inline with Infra structure scan entries.

@nameisaravind please update your branch


BigcommerceNetworkError: Fetch to Bigcommerce failed: The "name" argument must be of type string. Received null

When using
import useAddItem from '@bigcommerce/storefront-data-hooks/cart/use-add-item'

const addItem = useAddItem();
const addToCart = async () => {
  await addItem({
    productId: product.id,
  });
};

I get this error:
BigcommerceNetworkError: Fetch to Bigcommerce failed: The "name" argument must be of type string. Received null
at new BigcommerceNetworkError (\node_modules\@bigcommerce\storefront-data-hooks\api\utils\errors.js:47:28)
at Object.<anonymous> (\node_modules\@bigcommerce\storefront-data-hooks\api\utils\fetch-store-api.js:72:27)
at step (\node_modules\@bigcommerce\storefront-data-hooks\api\utils\fetch-store-api.js:44:23)
at Object.throw (\node_modules\@bigcommerce\storefront-data-hooks\api\utils\fetch-store-api.js:25:53)
at rejected (\node_modules\@bigcommerce\storefront-data-hooks\api\utils\fetch-store-api.js:17:65)

My pages\api\bigcommerce\cart.ts looks like this:
import cartApi from '@bigcommerce/storefront-data-hooks/api/cart';

export default cartApi();

Example of my .env variables:
BIGCOMMERCE_STOREFRONT_API_URL=https://{domain}.mybigcommerce.com/graphql
BIGCOMMERCE_STOREFRONT_API_TOKEN=''
BIGCOMMERCE_STORE_API_URL=''
BIGCOMMERCE_STORE_API_TOKEN=''
BIGCOMMERCE_STORE_API_CLIENT_ID=''
BIGCOMMERCE_CHANNEL_ID=

Hi @Tormand, thanks for opening the issue.
Can you send the Request Payload? (Dev Tools -> Network -> Cart Request -> Request Payload)
Looks like you're sending the product name but that's strange 🤔

Hi @jorgemasta
My payload looks like this:
{
  "item": {
    "productId":281
  },
  "locale":"en-US"
}

In \node_modules@bigcommerce\storefront-data-hooks\api\utils\fetch-store-api (__awaiter function), it dosen't contain any arguments. Shouldn't it contain the payload?

thisArg contains the correct values(as far as I can see):
{
   "commerceUrl":"BIGCOMMERCE_STOREFRONT_API_URL from .env",
   "apiToken":"BIGCOMMERCE_STOREFRONT_API_TOKEN from .env",
   "cartCookie":"bc_cartId",
   "cartCookieMaxAge":2592000,
   "applyLocale":true,
   "storeApiUrl":"BIGCOMMERCE_STORE_API_URL from .env",
   "storeApiToken":"BIGCOMMERCE_STORE_API_TOKEN from .env",
   "storeApiClientId":"BIGCOMMERCE_STORE_API_CLIENT_ID from .env",
   "storeChannelId":"BIGCOMMERCE_CHANNEL_ID from .env",
   "customerCookie":"SHOP_TOKEN"
}

I can see in fetchStoreApi it sends the correct body in ``options:
body: "{\"line_items\":[{\"quantity\":1,\"product_id\":281}],\"channel_id\":\"10\",\"locale\":\"en-US\"}"

endpoint has the value "/v3/carts"

I tried setting up https://github.com/vercel/commerce with the same .env variabels, and I get same error. Now I have a feeling my BIGCOMMERCE_STOREFRONT_API_TOKEN might not be correctly created.
I created my token from https://developer.bigcommerce.com/api-reference/store-management/tokens/api-token/createtoken with
{
  "channel_id": 10,
  "expires_at":<PHONE_NUMBER>,
  "allowed_cors_origins": [
    "http://localhost:3001"
  ]
}

Yeah... it's strange.
We have this gist to add BIGCOMMERCE_STOREFRONT_API_TOKEN to .env.
I hope you find it useful.

Hey @jorgemasta
I found the issue was in my .env.
The BIGCOMMERCE_STORE_API_URL was missing the whole url(https://api.bigcommerce.com/stores/{{store-hash}}/), I just inserted the store hash.


feat: add sorting to recent TX in webgui

Description
Add sorting to recent transaction in web gui. Added indicator (I hope this indicator works in all browser normally).

utACK


3d model will not upload

Do you want to request a feature or report a bug?

What is the current behavior?
Hi there! So I am trying to use Ar.js, but whenever I add a 3d model to the scene my project stops working. Can anyone help?

    <a-assets>
        <a-asset-item id="NSLogo73" src="/models/gltf/NSLogo73.gltf"></a-asset-item>
    </a-assets>

    <a-entity gltf-model="#NSLogo73"></a-entity>

<a-box position= ' 0 0.9 0' material opacity 0.5;'<a-box>
 <a-marker-camera preset='hiro'></a-marker-camera>    </a-scene>

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
I want the model to appear when the hiro marker is used. Thanks! Also, none of the demos in the ar.js folder work for me?
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Operating System and Device Name

hi @lajunex0 , try using a remote url for your src attribute. for example:  src="https://raw.githubusercontent.com/nicolocarpignoli/nicolocarpignoli.github.io/master/ar-playground/models/CesiumMan.gltf">

Thanks!! It works now.
Best,
LaJune


General: Move LTI custom parameters to OnlineCourseConfiguration

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added @PreAuthorize and checked the course groups for all new REST Calls (security).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context
The custom parameters for LTI launches are complicated to configure, and most importantly put the responsibility of configuring it correctly on the Platform (e.g. Moodle), instead of on the Artemis course being used as an online course. We want the configuration to be done on Artemis, instead of making/allowing instructors of the platforms to do the configuration.
There were two custom parameters, requireExistingUser and lookupUserByEmail. requireExistingUser defines whether we allow users to be created as part of an LTI launch, or if we only want already existing Artemis users to be able to launch those LTI exercises. lookupUserByEmail defines whether we check if there's already an Artemis user with the email provided in the launch. Both parameters could be configured on the platform and be different for each exercise. If no parameters were provided, both values would default to false.
Description
In this PR we change these custom parameters so that:

requireExistingUser - we add this as a new OnlineCourseConfiguration field. For each online course, the instructor can configure whether new users are allowed, or if only existing users can launch LTI exercises of that course. The default is false (meaning new users are allowed). The behavior of the parameter stays the same.
lookupUserByEmail - we remove the parameter, and default the behavior to true. We always want to check if there's already a user in Artemis with the same email, otherwise we will create a user with a duplicate email (or get an error when we try to do so), which is undesired behavior

Steps for Testing
Prerequisites:

1 Instructor in Artemis
1 Instructor in https://moodle.ase.in.tum.de/

TestrequireExistingUser:

Log in to Artemis as instructor
Create/edit a course to be an onlineCourse
Verify setting/unsetting the requireExistingUser checkbox works and the state is saved with the course
Login to Moodle as instructor
Setup LTI1.0 or LTI1.3 exercise (you don't need to set custom parameters anymore)
Launch LTI exercise with a new user with requireExistingUser = true and verify that you get an error
Launch LTI exercise with a new user with requireExistingUser = false and verify that you don't get an error
Launch LTI exercise with an existing user with requireExistingUser = true and verify that you don't get an error

Test lookupUserByEmail:

Log in to Artemis as instructor
Create/edit a course to be an onlineCourse
Login to Moodle as instructor
Setup LTI1.0 or LTI1.3 exercise (you don't need to set custom parameters anymore)
Create a user in Moodle with an email matching the email of an existing user in Artemis
Login in Moodle as the user created in the step before
Launch the LTI exercise and verify you were matched to the user with the same email in Artemis

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

LtiService.java
%
%

Lti10Service.java
%
%

Lti13Service.java
%
%

Screenshots
There is just one added checkbox when creating/editing a course for setting the "requireExistingUser" field in the OnlineCourseConfiguration. This will be improved/reworked in the PR to improve the LTI configuration in the UI: https://github.com/ls1intum/Artemis/pull/5812

Codecov Report
Base: 75.47% // Head: 75.52% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (715921a) compared to base (8352654).
 Consider uploading reports for the commit a95574f to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5836      +/-   ##
===========================================
+ Coverage    75.47%   75.52%   +0.05%
===========================================
  Files         1076     1105      +29
  Lines        42671    43706    +1035
  Branches      8075     8260     +185
===========================================
+ Hits         32204    33010     +806
- Misses        5450     5576     +126
- Partials      5017     5120     +103

Flag
Coverage Δ

client
75.52% <ø> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

...ebapp/app/course/manage/course-update.component.ts
77.50% <ø> (-0.47%)
:arrow_down:

...d/exercise/exercise-lti-configuration.component.ts
0.00% <ø> (ø)

...al-groups-table/tutorial-groups-table.component.ts
78.26% <0.00%> (-21.74%)
:arrow_down:

...utorial-groups/services/tutorial-groups.service.ts
70.00% <0.00%> (-16.67%)
:arrow_down:

...it-tutorial-group/edit-tutorial-group.component.ts
81.35% <0.00%> (-9.56%)
:arrow_down:

...gistered-students/registered-students.component.ts
68.96% <0.00%> (-9.22%)
:arrow_down:

...utorial-groups/course-tutorial-groups.component.ts
77.77% <0.00%> (-6.71%)
:arrow_down:

...-button/tutorial-groups-import-button.component.ts
93.33% <0.00%> (-6.67%)
:arrow_down:

...-tutorial-group/create-tutorial-group.component.ts
89.74% <0.00%> (-6.42%)
:arrow_down:

...torial-group-form/tutorial-group-form.component.ts
74.56% <0.00%> (-5.88%)
:arrow_down:

... and 46 more

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

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


Support branch based version requirements

Currently, Carthage supports the following for version requirements:
Carthage supports several kinds of version requirements:

>= 1.0 for “at least version 1.0”
~> 1.0 for “compatible with version 1.0”
== 1.0 for “exactly version 1.0”
"some-branch-or-tag-or-commit" for a specific Git object (anything allowed by git rev-parse)

And it appears that punic only supports tags. It'd be nice if punic could support any Git object as a version requirement, much like Carthage does

Punic supports all four version requirements. Will look at updating the docs if that's not clear.

Perfect, thanks for the clarification!


Some features need to be upgraded: Orchard.Framework, Settings

Hi, after updating Orchard 1.9.2 to 1.10.1, I'm having a hard time fixing this warning on admin page Some features need to be upgraded: Orchard.Framework, Settings. But when updating the Settings it brings error Transaction not connected, or was disconnected . Is there anyone here encounter the error and warning?

Try to have a look at the logs (src\Orchard.Web\App_Data\Logs). What's in there?

Please have a look at the error logs as per ViRuSTriNiTy suggestion.

I've found what causing the error. There is a duplicate entry on this table Settings_ShellFeatureStateRecord Orchard.Packaging and Orchard.packaging. I deleted Orchard.packaging (small letter) and successfully updated the settings.
But still I can't enable/disable Package Module properly. Sometimes it adds Orchard.packaging and sometimes Orchard.Packaging


snapshot weights and optionally init with pre-trained weights

yes, but I just wanted to note it since the logger doesn't necessarily need
to know about torch.
On Thu, Mar 28, 2019 at 3:26 PM Aurick Zhou<EMAIL_ADDRESS>wrote:

@azhou42 approved this pull request.
In rlkit/core/logger.py
https://github.com/katerakelly/rlkit/pull/4#discussion_r270215416:

@@ -248,26 +249,36 @@ def pop_prefix():
global _prefix_str
_prefix_str = ''.join(_prefixes)

-def save_itr_params(itr, params):
+def save_weights(weights, names):

''' save network weights to given paths '''

NOTE: breaking abstraction by adding torch dependence here

aren't we going to get rid of this abstraction in the great refactor
anyways? seems fine to me
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/katerakelly/rlkit/pull/4#pullrequestreview-220295125,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADplVpK_aFLnkP7dCTb8KqpDdTgcnpJbks5vbUGKgaJpZM4cROB3
.

--
Kate Rakelly
UC Berkeley EECS PhD Student
<EMAIL_ADDRESS>


smp: handle security request command

I think it might be better to emit a security_request event, and let pandora or other application handle it.

I'm OOO, feel free to take the ownership on this 👍

Supersede by #291


Usage of deprecated methods in results_controller.rb

There are a lot of *.find methods being called (like Result.find) which are deprecated, and we should be using .first/last/all/where (and their variants).

find is not deprecated.


Add addtl scte214 and low latency dash structs

I think this is the last of the PRs I'll need to use the oss lib proper versus fork.  They are pretty minor and I think backward compat with what you have in place currently.  Added the scope and scte214 structures we are using for low latency CMAF dash processing.  I noticed that you had recently added about 90% of what I had teed up.  Appreciate that!   Are there plans prior to 1.0.0 to refactoring the property names to more idiomatic rust (snake case).  Some properties are and some aren't and know you wouldn't want to do that short of a major revision change.  If there are such plans, I'd be happy to help.

Thanks!
I suppose a cleanup of attribute naming should be planned; the current inconsistency is undesirable.


Update chart with dropdown menu

The current chart implementation has a dropdown menu that is hidden. the @material/mwc-select component being used does not currently allow for passing sizing information to the mwc-menu used internally. This is causing the menu to not display when the height of the container extends past the view height of the screen.
An alternative dropdown component needs to be implemented or the current needs to be fixed.
In the mean time, the chart is set to show all counties in addition to the total count in SC.

@ncraley this one was a challenge


Bug: Websocket is not getting new messages

There is some problem that when the user enters in a room, the client subscribes to a topic OK but don't receive the messages

Fixed


deployment: Remove extra /host path from yaml file

The /host path was extra and unnecessary in the deployment file.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

cc: @klihub @fmuyassarov


Support multi-part pages

This adds support for storing individual parts of multi-part pages (guides and travel advice).
 There are specs in spec/integration/streams/process_sub_pages_spec.rb and I recommend reviewing those first, as there may be edge cases I've missed.
Trello: https://trello.com/c/oDKoKiim/415-5-update-etl-to-handle-sub-pages-for-guides-basepath

This ready for another code review.

@pmanrubia is this what you had in mind?

I've found a bug with the logic for deprecating old items. It is not taking into account locale, so if you have two items like /government/world-location-news/ambassadors-speech-at-international-anticorruption-day and /government/world-location-news/ambassadors-speech-at-international-anticorruption-day.fr, then it is marking one of them as outdated because they share the same content ID.
I'm going to add a test for this and fix this now.


Fixed divisor issue

For #3116
This is another attempt to fix the divisor problems. The specs pass and the Sandcastle demos work. It basically reverts what I did in #3113 and adds lines 98 and 99 to set the divisor cache when a VAO is created.
I'm not completely sure what the Chrome bug is that I'm working against. The title for the Chromium commit is "Fixed issue with vertex attrib divisors of 0 not tracking correctly". So the VAO actually does store the divisor state, except has a problem when the divisors are 0. I was able to test a simpler version of setVertexAttribDivisor for this, but it won't work on Firefox because they still seem to store the divisors as global state. Basically two implementations are needed to fix two different browser bugs. The solution here works all around, but on Chrome there will be some excess glVertexAttribDivisor calls when the divisor is > 1. At least Canary now works as expected.

It may not be related to this PR, but the billboards disable if you select "Instanced Disabled" in the Sandcastle example:
http://localhost:8080/Apps/Sandcastle/index.html?src=development/Billboards Instancing.html&label=Development
Windows/Chrome

Windows/Chrome

It is not related to platform/browser.  This is an issue for all platforms and browsers I tested.

This fixes the tests on Chrome/Firefox/IE on Windows.  Mac is also OK.
Merging this now.  @lilleyse please open a separate pull request for the Sandcastle example fix.


WordPress.org and plugin releases

I see that you made a release 1.0.1:

But WordPress.org is not aware of it and still allows everyone to download 1.0.0 by default:

That's because in your trunk/readme.txt you still point to the 1.0 which automatically instructs WP.org to zip and distribute the tags/1.0.0.
To make the 1.0.1 release go live and instruct WP.org to use tags/1.0.1 as the source of the zip you need to update in trunk/readme.txt the Stable tag: 1.0 to Stable tag: 1.0.1. Just creating a new directory inside SVN tags/ is not enough
And you will need to do this step for each and every release.
Basically, your trunk/* can always be the same as the latest tag you want to release. And for each release, you need to manually increment the Stable tag: X.X.X value inside the readme.txt.
You can leave in trunk/ only one readme.txt file and remove all the rest (and still update it and increment the Stable tag for each release), but I find it easier to just copy the release-prepared code to both tags/X.X.X and trunk/.
Here is the interface I used to identify this issue: https://plugins.trac.wordpress.org/browser/spatie-ray/

Thanks, I hope I've done everything right in 1.0.2 🙂

Thanks, I hope I've done everything right in 1.0.2 🙂


Update..

The puzzle 21-20ae1e1a from #21 has to be resolved:
https://github.com/ammaratef45/Attendance/blob/9f6a8fd62dbe74ab63635d65df334098944ad00c/attendance/lib/ProfilePage/profile_page_view.dart#L167-L167
The puzzle was created by @ammaratef45 on 06-Apr-19.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@ammaratef45/z please, pay attention to this issue

Job #24 is now in scope, role is DEV

The job #24 assigned to @ammaratef45/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

The puzzle 21-20ae1e1a has disappeared from the source code, that's why I closed this issue.

Job was finished in 35 hours, bonus for fast delivery is possible (see §36)

The job #24 is now out of scope

Order was finished: +35 point(s) just awarded to @ammaratef45/z


possible cfg++ implementation bug

there's a possible bug here:
https://github.com/lucidrains/denoising-diffusion-pytorch/blob/main/denoising_diffusion_pytorch/classifier_free_guidance.py#L647
according to line 3 in cfg++ algorithm

instead of:
pred_noise = model_output if not self.use_cfg_plus_plus else model_output_null
x_start = self.predict_start_from_noise(x, t, pred_noise)

it should be:
pred_noise = model_output if not self.use_cfg_plus_plus else model_output_null
x_start = self.predict_start_from_noise(x, t, model_output)  # <=== don't use null output here

@adam-ca yes you are right, thank you

@adam-ca are you seeing the improvements the paper claims? i haven't tried this yet

i haven't tried that yet either


Add flag for opting into worker build id versioning

What was changed
Added flag for opting into the new versioning feature
Keeping this draft since it would be confusing if we cut a release with this in it before server is released with it, and docs exist, etc etc. @cretz If we have a good way we can check this in but keep it hidden, that would be awesome.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

If we have a good way we can check this in but keep it hidden, that would be awesome.

Meh, SDK is beta. If you just document that it's experimental/unsupported, I think that's good enough. Alternatively you could add a bool capability for this in the GetSystemInfoResponse.Capabilities and error from the Rust side if that isn't set but this bool is.

You'll need to run poe gen-protos since it looks like some changed

Superseded by https://github.com/temporalio/sdk-python/pull/340


Have any due date for a new release?

Context
We have some computer with only has installed VS 2019 and our projects using a psake to build the projects
Problem
If we try to build a project with a machine only have VS2019 have problems because search a msbuild targets doesn't exists.
Question
I see, current code in GitHub in master branch solve our problem. Has a due date for a new release with current code?
Environment
Module version used: 4.7.4
Operating System and PowerShell version: Windows 10, PowerShel 5.1 and VS2019

Sorry for the long delay!
psake v4.9.0 has been released to the PSGallery, Chocolatey, and NuGet!


Fixed weight decay bug

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


Sprint PR PI26-Sprint-5

PI 26 Sprint 5

Issue with ET1 Vetting when cases have a claiment type of company the name displays as Null Null as the first and last name have not been set.


[build] Update Swift version in Info.plist for locally built toolchains

Currently for toolchains built locally via build-toolchain, the version in Info.plist is 5.0.YYYYMMDD.
Since the main branch is currently targeting Swift 5.8, let's bump the version in Info.plist.

@swift-ci please smoke test

@swift-ci please smoke test


Save temporary file in case of unexpected exit.

Not sure if it is possible to efficiently keep a saved state of the current modified file in case of an unexpected exit (shutdown, restart, terminal closed). Maybe it would be possible to detect the signal that the terminal sends this program and gracefully save the state. Will have to look more into this later.
When the user opens the file again, we can ask them if they want to keep the changes or not.

Maybe (brainstorming) we can use the saved redo. We could write the redo to disk more often. And make the hash/sha match the saved version of the file. So that if there is an unexpected exit, the hash will match and the redo will still be there.
We could further write a flag with the saved redo (or somewhere in .ci_edit/...) while the file is being edited -- saying that the file did not close properly. When the file does save properly, we'd clear the flag.
Later when a file is opened again and the flag is set (and there is redo available) we can prompt the user about restoring the edits they were working on.

Maybe (brainstorming) we can use the saved redo. We could write the redo to disk more often. And make the hash/sha match the saved version of the file. So that if there is an unexpected exit, the hash will match and the redo will still be there.
I think we'll run into a problem with this approach if the user rewrites the redo chain history (undo, truncate, and makes new changes). If this rewriting goes to before the current redo index of the old saved file, then we would miss all previous redos. This might be fine if we were to implement the redo tree, so that we could then just keep a pointer at some previous state and trace back if we want to revert to an old state.

Also, if a person were to completely write a large file without saving (living dangerously), then a long redo chain would cause a lot of processing to occur. Maybe it would be worth it to occasionally save the entire file instead. We could speed this up by pickling or compressing the file if speed/size is an issue.


Lunr search not returning results in production, but works on local build

Environment

Basically Basic version: 1.4.1
Ruby gem or remote theme version:
Jekyll version: jekyll-3.8.4
Git repository URL: https://github.com/RobertsLab/sams-notebook
GitHub Pages hosted (if yes provide URL to site): https://robertslab.github.io/sams-notebook/
Operating system: Ubuntu 16.04 and 18.04

Expected behavior
Lunr search should return search results.
Steps to reproduce the behavior
Lunr search works fine on local build, but fails to return any results on production build using remote theme with GitHub Pages.

It's not working because the JavaScript isn't loading for the search index. If you look at the browser's inspector console you'll see errors.

The path is wrong, and looking at your _config.yml I noticed you're improperly setting url and baseurl.
You'll want to use the following instead:
baseurl: "/sams-notebook"
url: "https://robertslab.github.io"

Doh! So sorry to bother you with this.
There's even a comment in the _config.yml file.  :no_mouth:
Thanks for the quick response.
Everything's great now!


Update UG

Update UG in the following ways

Updated command information for general and tag commands
Updated command summary table
Re-ordered certain parts of the UG
Updated Glossary
Updated QuickStart
Update on How to Use User Guide

Refer to https://docs.google.com/document/d/1xQvtcPoOtZJPF-_VDjUxj7taC_0nH-pHcMNJyuiYR1o/edit for full discussion on updates.
Closes #190
Closes #191
Closes #192

Codecov Report

Merging #261 (a2bf4cc) into master (db93c03) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #261   +/-   ##
=========================================
  Coverage     68.82%   68.82%
  Complexity      571      571
=========================================
  Files           112      112
  Lines          1957     1957
  Branches        207      207
=========================================
  Hits           1347     1347
  Misses          535      535
  Partials         75       75

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Terrain objects are not shown in minimap

The minimap does not show terrain items, making it difficult to know at a glance the "lay of the land".
Terrain will differ from Units in that once revealed they should remain displayed on the minimap even when shrouded by the Fog of War.
Alternately, they could be shown in the minimap by default and not need to be revealed (You assume that players already know the battlefield they are fighting on)
Here is an example from Age of Empires 2 to illustrate what I mean:

Edit: I am tempted to have a stab at this myself, but wanted to submit an issue first as mentioned in the README!

That feature is definitely required. I was actually planning to implement it at some point, but for that, some uniform input like e.g. texture would be required.
At some point, the map will be generated from heightmap and some map areas will be painted in certain ways - with that, we will have 2 textures - one for heightmap and the other for colors - with that, the above feature will be easy to implement.


Long subtraction with different fractional bits

    a = APyFixedArray.from_float([1.2345, -5.4321], bits=252, int_bits=128)
    b = APyFixedArray.from_float([9.8765, 5.6789], bits=252, int_bits=130)
    # a - b is not correct
    assert (a - b).is_identical(
        APyFixedArray(
            [
<PHONE_NUMBER>33171890260459372109042286592,
<PHONE_NUMBER>78659199234528198038053191680,
            ],
            bits=255,
            int_bits=131,
        )
    )
    assert (b - a).is_identical(
        APyFixedArray(
            [
<PHONE_NUMBER>33171890260459372109042286592,
<PHONE_NUMBER>78659199234528198038053191680,
            ],
            bits=255,
            int_bits=131,
        )
    )

As seen both a - b  and b - a yields the same result. b - a is correct.

Good catch, thank you. The fix was an easy two-line swap in the subtraction method, completed in #327. I assume you spotted this when you were trying to achieve higher coverage for the subtraction method. I should have scored full coverage for the subtraction method when I rewrote parts of it earlier this month, but I simply forgot to at the time.


[Removal]: Brisk Tawny Bat

Hotspot Name
Brisk Tawny Bat
Hotspot b58 Address
13ZGw7dFXnnnzk7qW1PCX3DFWcx3j61vE9jeVGWh2C83k3DJUDF
Discord Handle
Stace#6672
Hotspot Manufacturer
Bobcat
Removal Reason
I don’t know  how the hotspot is in the deny list. I haven’t done something wrong
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


feat(code-block): add rh-code-block

What I did

code-block, toolbar, and panel
tabs pattern
copy button patterns
light/dark

Testing Instructions

Usability testing on the DP

Notes to Reviewers
Please review the text in the code tab. I wrote this myself, since the design docs don't cover these kinds of details.

What if we added a prefix slot to tabs?
Then you could style tabs.html with:
.rh-tabs {
  --rh-tab-panel-padding: 0;

  max-width: 1000px;
}

rh-tabs::part(wrapper) {
  display: grid;
  grid-template-columns: 1fr max-content;
  grid-template-areas: "tabs actions" "panel panel";
}

rh-tabs::part(prefix) {
  grid-area: actions;
  display: flex;
  flex-direction: row;
  gap: var(--rh-space-sm, 8px);
  border-block-end: var(--rh-border-width-sm, 1px) solid var(--rh-color-border-subtle-on-light, #c7c7c7);
}

rh-tabs::part(container) {
  display: contents;
}

rh-tabs::part(tabs-container) {
  grid-area: tabs;
}

rh-tabs::part(panels) {
  grid-area: panel;
}

I'm not sure like the idea of sticking extra stuff in tabs slot that aren't tabs this would help to avoid that.

This is maybe a case for disconnected tabs (<rh-tab for="idref">)
a rough sketch of the idea:
<rh-toolbar>
  <rh-tab for="html">HTML</rh-tab>
  <rh-tab for="react">React</rh-tab>
  <zoopy-button floop="copify">Copy</zoopy-button>
  <rh-toolbar-panel>
    <rh-tab-panel id="html">
      <rh-code-block>Standards!</rh-code-block>
    </rh-tab-panel>
    <rh-tab-panel id="react">
      <rh-code-block>Who needs 'em?</rh-code-block>
    </rh-tab-panel>
  </rh-toolbar-panel>
</rh-toolbar>

I really like how composable it is.  I do wonder about the expandable feature.  How would you feel about putting that into as something that you could toggle on with a boolean?  I think it might be frustrating for a user to have to add the action buttons markup depending on the length of the code block content.
Example:

Code block is going to be used almost exclusively within WYSIWYG content blocks so it's also going to be tough to make changes if we make it too un-opinionated.

I'd rather leave that to the pattern level, tbh

question about names:
if we go with the pr as it is right now, which implements a widget that contains a toolbar and some content related to that toolbar, then what do we call those things?
as of f5b9e74 I called the toolbar element rh-toolbar, and the containing elemen rh-toolbar-panel (currently prefixed)
i guess this is ok, BUT! compare to rh-tabs, where the containing element is called rh-tabs, the toolbar is a shadow part, and the content area is called rh-tab-panel
so maybe we should have something like
<rh-thingy><!-- insert better name here -->
  <rh-button slot="toolbar">I'm a tool!</rh-button><!-- don't quote me on that -->
  <rh-tab slot="toolbar" for="a">I'm a tab button!</rh-tab>
  <rh-tab-panel id="a">A</rh-tab-panel>
</rh-thingy>

Blocked by https://github.com/patternfly/patternfly-elements/pull/2523

no longer blocked on https://github.com/patternfly/patternfly-elements/pull/2523, we'll save that for #1091

@heyMP, @zeroedin we're kicking the can down the road, so sleep on your above objections for a minute ;)

thanks @heyMP that's clear.
Try putting max-width: 100vw on the code block element. I think that's a page issue

@coreyvickery at your leisure


Pin PyTorch-nightly version

Hi there 👋
This project relies on the latest version of PyTorch-nightly which is fine, but what's not is that any issues with the nightly version affects this project too.
It actually already happened to this user: https://github.com/Lightning-AI/lit-gpt/issues/327
Proposal: pin version of PyTorch-nightly and update it if necessary.

The version that I propose is already more than a month old, but it's the one that I've been using on a server with a GPU and it seems to work without any problems (though I haven't tested all the code).

Yeah actually, why don't we just take the latest (from yesterday)?

So I've just checked the latest version of PyTorch-nightly which is 20230810: looks like everything works except for chat/base.py: after sending a prompt it fails with exception:
File ".../lit_gpt/model.py", line 273, in scaled_dot_product_attention
    return torch.nn.functional.scaled_dot_product_attention(
RuntimeError: cutlassF: no kernel found to launch!

And this problem has been around for at least 2 weeks. So maybe it's not a bug - it's a feature 😄.
What should we do:

stick to the one month old version, maybe later fix this issue and update pinned PyTorch version
if the rest of the code relies on the newer version - pin the newest one and fix the issue in another PR

I'd go with (2), working around the issue with https://github.com/Lightning-AI/lit-gpt/issues/327#issuecomment-1664674460.

The original reason for this PR was issue #327: after updating PyTorch chat/base.py stopped working and on the earlier version of torch it worked fine.
The problem was in the wrongly provided precision: by default bf16 was used for all cards, even for T4, which doesn't support it.
On previous versions of torch it didn't cause any issues for some reason.
So, since the issue that let to this PR is fixed, then maybe we can drop this PR and continue using the latest and greatest version of torch? At least until next issue ;)

We can close this issue. But we should soon pin to the PyTorch 2.1 release. Which had a first release candidate published this week


Atomic-GUI Error

Everything else works perfectly. When I run Start-Atomic GUI I receive the following error. I have tried specifying a different port but I continue to receive this error and am unable to access the GUI. I've also tried re-installing the UniversalDashboard module with no luck.
Start-Process: /home/vagrant/AtomicRedTeam/invoke-atomicredteam/Public/Start-AtomicGUI.ps1:263
Line |
263 |      start-process http://localhost:$port
|      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| This command cannot be run due to the error: No such file or directory.

I'm running on a CentOS 7 box with the latest version of Powershell Core installed

After running start-atomicgui (and ignoring that error), can you visit http://localhost:8487 in your browser and see if the GUI is working in your browser?  Also, are you on the Slack Atomic Red Team workspace where we can chat interactively perhaps? I'm @OrOneEqualsOne over there.

I think I heard that you got this figured out, so closing for now.


Support label "istio.io/dataplane-mode=none"

What would you like to be added:
In both istio and kmesh, we all use istio.io/dataplane-mode: {ambient/kmesh} to indicate whether a specific pod or a group of pod redide in a namespace should be managed.
Recently in istio 1.22, "istio.io/dataplane-mode=none" is suported to exclude a specific pod from being managed.
We should respect this label too, so that i think we donot need to label bypass kmesh.net/bypass=enabled to exclude a pod from being managed by the kmesh.
In workload/ads mode,  "istio.io/dataplane-mode=none" and  kmesh.net/bypass=enabled looks redudant, this is the second issue i want to discuss
Why is this needed:

IMO, istio.io/dataplane-mode=none only take effect when the pod is created and will not change during the entire lifecycle of pod.
However we could add or remove kmesh.net/bypass=enabled anytime which is mainly used for dubug.

IMO, istio.io/dataplane-mode=none only take effect when the pod is created and will not change during the entire lifecycle of pod.

How do you know this? I can arbitarrily label this and confirmed it take effects by looking at the port ztunnel listened.
Kmsh is different, it do manage with stream.


Auto-sub not showing in DaVinci menu at all

My background is non existent with coding, so the instruction for installing it, or even worse, for debugging it is a nightmare.
What this even mean? Is there a shortage for letters on keyboard to put everything in context in a nice well written guide, instead of 3 words thrown out of nowhere for guessing what i should do?
For exemple because i cant find the auto-sub in davinci menu, i went to FAQ, and got even more confused:
Why would i want to find fusion folder? And why only on windows?
2. Can't find Fusion folder
Use Everything to quickly search your computer for it (Windows only).
Or the other FAQ like:

Auto-Subs not opening
Verify that Resolve detects your Python installation by opening the Console from the top menu/toolbar in Resolve and clicking py3 at the top of the console. Ensure that Path in your system environment variables contains the following:

C:\Users<your-user-name>\AppData\Local\Programs\Python\Python312
C:\Users<your-user-name>\AppData\Local\Programs\Python\Python312\Scripts\
Where do i find "Path", so i can check the path? And WHAT do i check there? Is there a shortage of SS so i can see what to expect to see and what it should look like??? ALSO, IT SHOULD BE STATED THIS IS FOR WINDOWS ONLY, so i would not lose 30 min of my life to discover that this is not related to mac users
Why would i want to find fusion folder? And why only on windows?
2. Can't find Fusion folder
Use Everything to quickly search your computer for it (Windows only).

MacOS Error
WHERE SHOULD THIS APPEAR?
<urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self-signed certificate in certificate chain (_ssl.c:1006)>
Solution: Run this command in the terminal (replace the Python directory with wherever Python is installed on your computer).

 How do i even know where is python installed if i've installed it via brew, following the instructions? There is no Python folder under Application folder. What does "replace PY dir with wherever py is install" even mean?? WHEEERE should i write the path in this line commnad???? BEFORE, AFTER, in the middle? Where do i copy the path from? Why is this so undocumented for 99%of the users who would actually need it, and all the instructions are written like code lines? We are not programmers and we have never heard of terminal or other installed apps like PY and whatever other apps i installed via brew / instruction.
This is borderline offensive for me at least. Spending 3 hours to decipher hieroglyphs that one would assume i was born with the knowledge to know what they mean and that "terminal" is an app on macOS. Is it to much to write 2 more words  and explain like this: "For the installation process you will need to open the macos app called "terminal" via shortcut "cmd+space" and type in the search field "terminal". Once found, open the app and a black widow will appear and you should paste the following code/command there: -> SS with the terminal opened with the text pasted". Is this to much?
So the question is, what now? I have installed all the "stuff", with no errors, but i cant see the auto-sub in the menu. I've attached SS with the only error i could find related to all i was able to understand from the installing description and FAQ.

@Razvand21 This is a free plugin. There is no need for such aggressive communication.
That being said, if you are not technical enough to get the v1 up and running, you can pay for Davinci Resolve Studio, which also has auto subtitles option.
Finally, if you had taken the time to read through some prior issues you would have found that v2 is currently being developed and v1 has been deprecated. I suggest waiting until v2 is released and then try the mac installer:
https://github.com/tmoroney/auto-subs/issues/101#issuecomment-2439903780

Auto Subs V1 Manual Install Tutorial
Here's a video that should help you @Razvand21

@AdyTech99  I appreciate your input, although i need support for the IOS version. I'm not on Windows.
@Iojoe92 I have the payed version. The problem is with language support. I need other languages as well.
Also this is not aggressive communication. This is the frustration of 99% of people who are in the creative business and have to learn how to be a programmer for a product intended to be used by anyone but programmers. How would you feel if you would like to use a bus for transport, instead of walking, but in order to do that, you should know how to repair the engine to start?
Or if there was a anticancer pill, but you need to know how to make it yourself, instead of just buying it from a drug store. How would you feel if you need that pill and of course the instruction would sound like:
**"Using stoichiometric relationships.
The coefficients in the balanced equation are used to derive stoichiometric factors
The balanced chemical equation, the stoichiometric factor relating the two substances of interest is  3molI2/2molAl
FAQ:
If the barbutation doesn't work, try: N2(g)+3H2(g)→2NH3(g)
Solution
Again, we need to use the mole ratio from the balanced equation to convert between NaN3 and N2. Then we can use the molar mass of N2 and the density of the gas to convert to volume in L.
Solving for moles of N2 :
100gNaN3×1molNaN365.0gNaN3×3molN22molNaN3=2.31molN2(5.1.11)"**
You get my point. **Why do i have to learn chemistry in order to cure myself, is this a product only for chemists? Do i need fire, is this liquid, is it gaz or it's a rock? There are 100000 questions just to begin to understand what is that. **
And all of this is my time being wasted, multiplied by the time spent by other 10k potential users, that have to learn everything from scratch, instead of just one guy making it simple to use, and sparing countless hours of frustrations for countless people. AND THE MOST IMPORTANT it could spare countless hours spent on support pages waisting the time of the devs/people responding to questions plus the time of the actual user.
This is a communication targeted at the ignorance of the people doing their hobby/job, and assuming that anyone should know everything they are doing because for them is obvious/simple.
It's the same for doctors, who speak to you like you graduated medicine school.
It's like making the back end of a CRM software without the UI. It's simply wrong.
The targeted public for the CRM is not other Devs. And for the better or worst, if you don't say anything to solve a problem, you actually take part of the ignorance cultivated among us. It's like passing besides a fight on the street like nothing is happening, or watching a theft and pretend you don't see it.
I'm not trying to offend anyone, but i'm trying to open the eyes of the Devs doing "products for the masses" accessible only to a very very very narrow audience because of the assumption that if you don't know their area of expertise/knowledge, you are worthless, and you do not deserve their time to put 3 words extra in a sentence to actually make sense.
And yes, i'm waiting for a v2 with a mac installer, and if it works i'll definitely "buy a coffee" for a product that i can actually use, to show my appreciation for dev's time spent to make it usable for the intended target, and the great idea.

Yeah, I get what you're saying. It took me a full week to figure out how to install it, and I am a programmer.
I'd love to help you out with a step by step instruction.

The brew python installation causes more trouble than it is worth, install python from here instead: https://www.python.org/ftp/python/3.12.7/python-3.12.7-macos11.pkg

Run the rest of the commands in Terminal:

brew install ffmpeg
pip install -U stable-ts OR pip3 install -U stable-ts
pip install -U openai-whisper OR pip3 install -U stable-ts

Download this file: https://github.com/tmoroney/auto-subs/blob/main/auto-subs.py and place it in /Library/Application Support/Blackmagic Design/DaVinci Resolve/Fusion/Scripts/Utility folder

NOTE: To find the Utility folder quickly, type in Terminal: open "/Library/Application Support/Blackmagic Design/DaVinci Resolve/Fusion/Scripts/Utility"

There you go, it should show up in DaVinci Resolve!
Feel free to ask for clarification on any step, I'm happy to help.

is this for windows user?

is this for windows user?

@yujingoh09 the steps should be same for both Mac and Windows. The only difference is that the Python install links will vary.

YEEEES!! Thank you! I can see it now! Just a small quirk: "auto-subs.py" movet to the main root and is called "AutoSubs V1.py". The problem is it doesn't work for now. Though i can see the file was updated 2 days ago, so maybe there are some glitches. Also V2.lua was uploaded one hour ago, so maybe there is work in progress. I'll check tomorrow, but for now it doesn't work. It shouws in the menu, but it's not responsive. (doesn't do anything)

AutoSubs v2 is out.

Yes and it works! 10 sec install AND working, vs one week not working! This is what i call progress! Also i've bought Moroney a well deserved coffee!


⚠ GitLab Server has degraded performance

In 03afe90, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6981 ms

Resolved: GitLab Server performance has improved in d6fce9e.


Can the width and height be different?

It seems the width and height of the feature map must be same.
Can they be differenct?
https://github.com/leaderj1001/Attention-Augmented-Conv2d/blob/1ce94a3072c2d9aabe258313b3a17c974d987411/AA-Wide-ResNet/attention_augmented_conv.py#L35-L36

They can be different. In Tensorflow code, it depends on W, H of input tensor.

having a question and need some help... why the size should be 2*self.shape - 1?


Update codeql-analysis.yml

This pull request updates the GitHub Actions workflow configuration for CodeQL analysis to use the latest versions of the actions.
Updates to GitHub Actions workflow:

.github/workflows/codeql-analysis.yml: Updated actions/checkout from version 3 to version 4.
.github/workflows/codeql-analysis.yml: Updated actions/setup-dotnet from version 1 to version 4.
.github/workflows/codeql-analysis.yml: Updated github/codeql-action/init from version 2 to version 3.
.github/workflows/codeql-analysis.yml: Updated github/codeql-action/autobuild from version 2 to version 3.
.github/workflows/codeql-analysis.yml: Updated github/codeql-action/analyze from version 2 to version 3.

Thank you @fthorntonai


Fix Revoking Permissions subsection

It seems the text of the subsection was copied from the "Granting
Permissions to a User" and was still referring to granting
permissions, instead of revoking them.

@iarenaza Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@pivotal-issuemaster This is an Obvious Fix

@iarenaza This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary.

Thank you!