Google Cloud Shell Idea

Google Cloud Shell Idea - Codespace Alternative
Easily use any of the container images, using docker, on google cloud via their free cloud shell/IDE

https://cloud.google.com/shell

Tutorial for nmfs-opensci container images:

https://github.com/MichaelAkridge-NOAA/Open-Science-Codespaces/tree/main/google-cloud-shell

Quick Start Link:

Screenshot:

@MichaelAkridge-NOAA Thanks! I added your links to the README.md.


test(atomic): added cypress tests for result badge

https://coveord.atlassian.net/browse/KIT-965

Closing temporarily because I wish to use fixtures


receive related memory leak

the pgm
-module(nx).
-export([start/0]).
repeat(_,0) -> ok;
repeat(F,N) -> F(), repeat(F, N - 1).
start() ->
Nrep = 10000000,
repeat(
fun() -> receive after 1 -> ok end end,
Nrep)
.
results in
Failed to allocate memory: {Atom_HOME}/src/libAtomVM/scheduler.c:68.
./gen.sh: line 17: 13623 Aborted                 ${ATOM_BIN}/AtomVM nx.avm
when run (under linux) as
ulimit -Sv 10850
${ATOM_BIN}/AtomVM nx.avm
changing Nrep to 1000 leads to normal pgm termination (Return value: ok).

as already suggested (fix for bettio#88 #93), adding
free(listener);
after line 199 in scheduler.c has the potential to resolve this issue but i'm unaware of unwanted side effects.

has been fixed.


Telemetry Documentation

Added
Documentation covering the configuration and usage of tracers in the instrumentation package.
Changed
Fixed

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 56.48%.  Consider uploading reports for the commit 6364d0a to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #318      +/-   ##
============================================
- Coverage     58.72%   56.48%   -2.25%
+ Complexity      838      731     -107
============================================
  Files           111      106       -5
  Lines          4068     3842     -226
  Branches        428      395      -33
============================================
- Hits           2389     2170     -219
- Misses         1484     1489       +5
+ Partials        195      183      -12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Unsafe storage key in near_contract_standards

Storage key for non-fungible token is unsafe, this may be conflict with user's custom storage key if user define the same first storage key variant.
e.g.
#[derive(BorshSerialize, BorshStorageKey)]
enum StorageKey {
    MyKey {
        account_hash: Vec<u8>
    },
    // other variants
}

https://github.com/near/near-sdk-rs/blob/master/near-contract-standards/src/non_fungible_token/core/core_impl.rs#L54:L58

First and foremost, this is a valid concern. Thank you for the report.
Here is how near-sdk-contract-tools currently handles storage keys for most components (e.g. NEP-141):

A default storage key prefix is defined here. In the case of NEP-141, this default prefix is "~$141".
All storage entries managed by a component use that prefix to construct their keys.
All components allow the user to override the default storage prefix. Here is a test that demonstrates that functionality.

With that being said, there are not currently any safeguards that prevent (or even warn a user against) duplicating/overloading a storage key. That seems like a difficult problem to solve, though, without creating a complex compile-time key management system and prohibiting the use of env::storage_* methods.


Deployment without tenant Id gets empty string instead of null on camunda

Hi,
When deploying a diagram without informing a tenant Id in the "Deployment Configuration" modal, the process is deployed using an empty string as tenant Id.
This way, a process that should be visible to all tenants does not appear in REST API or camunda cockpit.
Thanks,
Thomas

@siffogh Could you have a look at this?


Sort and format bolt generated workspace members

Changes

Current dependencies on/for this PR:

main

PR #317   👈

This stack of pull requests is managed by Graphite.


Ability to cast media files to device

This can be achieved using the APIs in Windows.Media.Casting namespace. Check this out: https://docs.microsoft.com/en-us/windows/uwp/audio-video-camera/media-casting.

@Jaiganeshkumaran This issue should be resolved in v0.21.


Crops are not working once the CDN Media Url provider is configured at startup.

Hi,

If we configure the site to use CDN Media URL provider the crops no longer crop as expected, these instead just return all the same size. Once this has been configured the returned path for the media files then looks like https://{cdnUrl}/media/{mediafolder}/{imageName}?width=1280&height=540&rnd=132786959387800000 - which i'm not sure is correct as it does not appear to be using the cached file it creates in blob.
Also the cached file it creates in the "cache" blob container does not have a file extension therefore just has a key and is not actually {filename}.jpg.
If we remove the .AddCdnMediaUrlProvider() line from Startup then the cropping works as expected.
In terms of rendering the image i have tried multiple ways :
   var image = Url.GetCropUrl(Model.HeaderImage, 965, 700);
   var imageTest = Model.HeaderImage.GetCropUrl(1930, 1400);

   var image = Url.GetCropUrl(Model.HeaderImage, cropAlias: "headerImage");
   var imageTest = Model.HeaderImage.GetCropUrl(cropAlias: "headerImage");

Hi @rasmusjp Any chance you guys have any other suggestions to look at?
Cheers
Dale

Hi @dalemccutcheon
I've had a look at Azure CDN and found out that you'll need to change the query string caching behaviour rule to cache every unique url. This can be done on the endpoint caching rules. When you've made the change you'll need to purge the cache to ensure the non cropped images are removed

@rasmusjp  thanks for the update i am trying this right now - the cdn is requesting images still with the blob container name i.e. {{cdn}}/{{blobContainer}}/media/gakjdfkgah/gjakfg when it should just be {{cdn}}/media/gakjdfkgah/gjakfg Do you know how i can fix that?
Thanks
DAle

Hi @dalemccutcheon
I think this might be the problem https://github.com/umbraco/Umbraco.StorageProviders/blob/d422db55e59e3f1d4138967552dc5881533e71bd/src/Umbraco.StorageProviders.AzureBlob/DependencyInjection/CdnMediaUrlProviderExtensions.cs#L36
You should be able to workaround this for now by setting the url in code

When you say setting it in the code what do you mean? Where can i set that value?

Hi @dalemccutcheon
The AddCdnMediaUrlProvider takes a callback where you can configure the options, there's an example in the README

I think this might be the problem
https://github.com/umbraco/Umbraco.StorageProviders/blob/d422db55e59e3f1d4138967552dc5881533e71bd/src/Umbraco.StorageProviders.AzureBlob/DependencyInjection/CdnMediaUrlProviderExtensions.cs#L36

@rasmusjp Adding the container name to the configured URL here might indeed cause issues, as it would result in these examples having different CDN URLs:
{
  "Umbraco": {
    "Storage": {
      "AzureBlob": {
        "Media": {
          "ConnectionString": "...",
          "ContainerName": "test",
          "Cdn": {
            "Url": "https://my-cdn.example.com/"
          }
        }
      }
    }
  }
}

The above would result in https://my-cdn.example.com/test being used as base CDN URL, but doing it in code will result in using that exact URL and thereby also not exposing the container name 😄
.AddCdnMediaUrlProvider(options => {
    options.Url = new Uri("https://my-cdn.example.com/");
});

@dalemccutcheon Besides trying to configure the CDN URL from code using the above example, also ensure you've updated the query string caching behavior to Cache every unique URL: https://docs.microsoft.com/en-us/azure/cdn/cdn-query-string (and purged/clear the CDN cache).

Hey guys,
I think i have this working now there are a few different things i've had to change here such as removing app.UseStaticFiles() within the Startup.cs as well as app.UseResponseCaching() which appears to break the crops.
I did also have to add this : -
.AddCdnMediaUrlProvider(options => {
options.Url = new Uri("https://my-cdn.example.com/");
});
This seemed to fix the domain issue i was having.
Cheers
Dale

@rasmusjp Adding the container name to the configured URL here might indeed cause issues, as it would result in these examples having different CDN URLs:

Yeah, we should probably remove it from this package since it's Cloud specific

@rasmusjp PR for removing it is here: https://github.com/umbraco/Umbraco.StorageProviders/pull/24 👍🏻


Unable to move file error

I'm getting the following error when I attach a file to a non-archival project: Unable to move file: EXDEV: cross-device link not permitted, rename 'P:\DigitalProjects_BCDAMS\987443698_hetag\uhlib_987443698_002.pdf' -> 'C:\Item_002\uhlib_987443698_002_ac.pdf'
Thoughts: since this is non-archival, it didn't create any container directories when I first saved the project. When I add a file, is carpenters expecting to move the file into a directory that doesn't exist and that's what's causing the error?


Show xpubs for accounts

Is your feature request related to a problem? Please describe.
In my don't trust verify attitude, I wanted to understand what the flavors in JAM are and what they do and if the seed phrase and address derivation works as expected. I always do that for new wallets. It was more difficult here, because xpubs were not shown. I could pull it off in the end, but considerable effort was required.
Describe the solution you'd like
Would be good to have a power user mode where users can display xpubs/zpubs for each flavor (account)

tank you for the input, afaik we have plans for that: https://www.figma.com/file/kfejZJFlwBywvLEnPEmJo1/JoinMarket-UI?type=design&node-id=4139-77309&t=btiIGZLJOw9L8jKf-11


Do not change directory to run operation on cloned repository

DOn't change directories in order to operation on cloned repository (such as tox, build container, etc.),
Once we run more than one check run, we fail since the directory was changed.
Done for all the except SonarQube (Need to find out how to do it)

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To re-run tox comment /retest tox in the PR.
To re-run build-container command /retest build-container in the PR.
To re-run python-module-install command /retest python-module-install in the PR.
To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To add a label by comment use /<label name>, to remove, use /<label name> cancel

Supported labels

hold
verified
wip
lgtm

/hold

/hold cancel

/wip

/retest tox

/retest tox

/retest sonarqube

/retest build-container

/retest sonarqube

/retest sonarqube

/retest build-container

/retest build-container tox build-container

/retest python-module-install

/retest build-container tox build-container python-module-install

/retest sonarqube

/retest tox

/wip cancel

Container quay.io/myakove/github-webhook-server:latest pushed


Google Drive sync timeouts

I am often getting timeout when trying to sync with Google Drive. The three rectangles at the top keep animating for a while and then I get this in the Console:
Uncaught TypeError: Object [object Object] has no method 'execute' main.js:29817
Error: A timeout occurred.
at r (https://stackedit.io/res-min/main.js:29250:65)
at https://stackedit.io/res-min/main.js:23183:8
at Array.forEach (native)
at Function.E.each.E.forEach (https://stackedit.io/res-min/main.js:3625:46)
at Object.onPeriodicRun (https://stackedit.io/res-min/main.js:23181:6)
at https://stackedit.io/res-min/main.js:29017:79 main.js:23185
(anonymous function)
Tried resetting the StackEdit, it helped only for a while

Closing as too old.


Bullet list entry starting with a shortcode  is deleted when moving between Rich Text and Markdown

Describe the bug

When changing from markdown to rich text the bullet list item is being removed if it starts with shortcode , e.g :
* <ShortCode/>
To Reproduce

start the development environment
yarn develop
Go to posts admin page
http://localhost:8080/#/collections/posts
Click on New Post
Scroll to the Body element
Verify the Markdown mode is toggled on
Type the following sequence
* Backspace Space <ShortCode/>
Note: The Backspace is needed only to remove the automatically added ZWNBSP.  If there is no ZWNBSP added automatically you don't need the Backspace
Toggle Rich Text mode
The list item disappears

Expected behavior

The list item should remain in place
Screenshots

Applicable Versions:

Netlify CMS version: latest
Git provider: Github
OS: macOS Monterey 12.6.1
Browser version chrome 107.0.5304.110

Node.JS version:

CMS configuration

Additional context

The #6634 was closed due to missing info. The issue contains the steps to reproduce the issue. Please let me know if you can't reproduce the issue

@yuri-g-taboola is this still happening in decap 3?

Closing as stale


"Event triggers"-Tab for PostgreSQL

There is no tab in DBeaver to show event-triggers of a PostgreSQL-Database.
It would be nice if you could check your event-triggers of a PostgreSQL-Database inside the DBeaver-UI.

pg_catalog.pg_event_trigger can be helpful in this case

@ridicolos
Thanks for the idea!

Event triggers support implemented for PostgreSQL, Greenplum and EnterpriseDB (versions > 9.3)

it is impossible to rename event trigger and trigger in metadata viewer: new name is used as old name

DDL isn't correctly updated after rename in navigator - event trigger and usual trigger name remains old in trigger source until manual refresh

event type isn't taken into account in case default event type is used.

if more than one ticket is created at the moment - all triggers disappear if one of them was saved (except saved trigger)

The strange highlight of the trigger function field after choosing trigger function - gif below - happens without holding the mouse button

Changed comment refreshes in the source only after disable-enable show comments option

1,2,3,4,6 Fixed.

verified

Thank you! :heart:

Thanks for the feedback @sofiarosati


The sound coming out is corrupted

The sound is very very noisy and you can't almost hear the original sound. The problem is for sure not the sample, as it is clean and also if I change the file I get the same result.

Are you using the sound file from the example?

No
On Sun, Jul 14, 2024, 08:01 biner88 @.***> wrote:

Are you using the sound file from the example?
—
Reply to this email directly, view it on GitHub
https://github.com/biner88/metronome/issues/13#issuecomment-2227206623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXAPPLC2KHHZAK6FEUVPAILZMIHZ7AVCNFSM6AAAAABKW4L6XGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRXGIYDMNRSGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sound files are specially processed. You can refer to the file parameters in the example to re-edit your file.

Ahh ok I see, thanks 🙏
On Sun, Jul 14, 2024, 08:46 biner88 @.***> wrote:

Sound files are specially processed. You can refer to the file parameters
in the example to re-edit your file.
—
Reply to this email directly, view it on GitHub
https://github.com/biner88/metronome/issues/13#issuecomment-2227215922,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXAPPLDQGG7ZWB3F5OQELVLZMINDDAVCNFSM6AAAAABKW4L6XGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRXGIYTKOJSGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

@manudicri  Is the same true for examples?

Hi @biner88
I haven't tried yet. Anyway, I think you can reopen this issue since the problem is there on Android. Just like the video I attached.
My audio file is a .WAV, 44,1khz 32bit. I got to try with other audio files as soon as I can.
Thanks
Manu

You can try the audio file provided by the sample, I tested it fine on multiple Android devices. You can upload your audio file here and I'll test it.

Hi,
I noticed that the .wav file in the sample repository has its 'encoding' field set to 'Microsoft PCM', whereas my .wav file is encoded as 'Microsoft IEEE'. This seems to be causing issues when playing the file on Android. I converted the file to PCM encoding, which resolved the issue in the Android simulator.
I will test it on a real device and update you on the results.
Thank you
Manu

Working fine.
You can close this ticket.
Thank you


Update vendoring for 2.25 release

Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

Which it has by now.

Did we want to try and figure out how to get https://github.com/prometheus/prometheus/pull/8029 to pass CI tests?

That would be cool. (But I wouldn't delay the release for it.)

It still looks quite red. :crying_cat_face:

It is green

Largest delta ever!


If I use a persistent connection, the connection automatically keeps alive?

hi~nrk, I found in our system that the connection obtained by using the persistent configuration item does not seem to have a keep-alive function

Feel free to open a PR.


⚠ Fortnite has degraded performance

In df7f8d2, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1535 ms

Resolved: Fortnite performance has improved in bdeb207.


General improvements + many improvements for English version

Only affects English version

Better wording
Hide big Ai-chan
Screenshots in English

Affects all

More readable text
Slightly more clear screenshot positioning
Change (c)  to ©

Not sure about the changes you made in src/.vuepress/theme/layouts/Layout.vue, there are generally single quotes in the JavaScript parts so I think the reformatting was unnecessary.
I think you could probably also implement the style tags that hide things better by adding them to the preamble. But there again not entirely sure why you are doing that.
Also don't fully agree with some of the rewording: Misskey is software and the platform is the Fediverse. Why that'll make you ask why you weren't here already!? I don't think we should "guilt trip" people when they visit our landing page.

As for the code formatting/single quotes, I think that was VSCode doing that, I'll see if I can reverse it
For the style tags, that seemed to be the best way to have modifications only for the English version, but I'm not the best at VuePress so if you can think of a better implementation.
I'll admit that wording on the landing page was a bit much, I'll rework that today.

I think you could probably also implement the style tags that hide things better by adding them to the preamble.

@Johann150 Which file is the preamble again? ^^"

My mistake, what I meant is apparently called "frontmatter".

@Johann150 I can't seem to find a way to add the style to Frontmatter just for the English version. Any suggestions?

Fist of all, your many improvement commits should be seperated by each topics https://github.com/misskey-dev/misskey-hub/pull/76#issue-1222029827 and be seperately reopend these here. Your each commits are worth of discussions and lead to breaking changes for  Misskey Official website.  especially, "Better wording", "Hide big Ai-chan" and releted to src/.vuepress/theme/layouts/Layout.vue .
Your commits are too many to comment to each of them and have a discussion.

@4ioskd @syuilo see #79 #80 #81 #82


Using without npm

I tried to use the lib without npm,
by copying the moment-business.js file,
and source it with:
<script src="moment-business.js"></script>
then use it with
import business from 'moment-business';
but I got error below:
Uncaught SyntaxError: Cannot use import statement outside a module
Any idea how to solve this ?

This is most likely late, but for someone experiencing this issue, try:
<script type="module" src="moment-business.js"></script>


Transfer Learning using Save_and_Load.py

Hi @nreimers
I have tried to use Save_and_Load.py for pre-training on a bigger dataset and after reloading training on a smaller set. But it seems that Save_and_Load.py only works for the same dataset. Like you train for few epoch on a dataset, then save the model and then continue training the new loaded model. Because if i use new dataset after loading pre-existing model it throws a Key-Error. Is this how it is supposed to be working?

Are the labels / tags the identical between the two datasets?
If not, continuing training is much more challenging, as the CRF no longer fits. There, you would need to replace the CRF layer with a new layer for your dataset. This is not support right now in this framework.

Yes the labels are exactly the same (BIO tagged). I even have tried continuing training by making a copy of first dataset, it still produces the key-error.


Use getauxval on Android with API level > 18

We received analytics that devices of the device family Oppo A37x
are crashing with SIGILL when trying to load libcrypto.so.
These crashes were fixed by using the system-supplied getauxval function.
(This is the PR https://github.com/openssl/openssl/pull/11257 against OpenSSL_1_1_1)

This pull request is ready to merge

ping @openssl: ready to merge since 8 days

ping @openssl: ready to merge since 8 days

Cannot be merged without OTC decision.

OTC: This is ok for inclusion in 1.1.1

Merged to 1.1.1.  Thanks for the contribution.


OS-dependent symlink generation during installation

Fails on Windows with error:
yarn run v1.22.0
$ gulp build
[02:48:32] Requiring external module babel-register
[02:48:34] Using gulpfile D:\dev\bug\DHTest\tests\Application\gulpfile.babel.js
[02:48:34] Starting 'build'...
[02:48:34] Starting 'buildAdmin'...
[02:48:34] Starting 'buildShop'...
[gulp-chug] Gulpfile, gulpfile.babel.js, contents is empty. Reading directly from disk...
[02:48:34] 'buildAdmin' errored after 50 ms
[02:48:34] Error in plugin 'gulp-chug'
Message:
    Problem finding locally-installed `gulp` for gulpfile undefined. (Try running `npm install gulp` from <path>\DHTest\vendor\sylius\sylius\src\Sylius\Bundle\AdminBundle to install a local gulp for said gulpfile.)

Error: Cannot find module 'gulp' from '<path>\DHTest\vendor\sylius\sylius\src\Sylius\Bundle\AdminBundle'
Details:
    domainEmitter: [object Object]
    domain: [object Object]
    domainThrown: false
[02:48:34] 'build' errored after 60 ms
[02:48:34] The following tasks did not complete: buildShop
[02:48:34] Did you forget to signal async completion?
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Steps to reproduce:

composer create-project sylius/plugin-skeleton ProjectName
go to the installation folder
execute: (cd tests/Application && yarn install)
go to the installation folder, execute: (cd tests/Application && yarn build)

Issue suggested by the message:
Problem finding locally-installed `gulp` for gulpfile undefined. (Try running `npm install gulp`

is actually not the problem.
Symlink in a format not accepted by Windows is:
https://github.com/Sylius/PluginSkeleton/blob/master/node_modules
Solution:
use a way to generate the symlink for the local platform.

@ideaconnect did you try PHP's built-in, cross-platform functions: symlink and link
Also, maybe first check if the link doesn't exist.

I have rebased on master yet from what I can see there is an error with services when running in test environment and is related to the latest update in Sylius itself. Fixed by update of the bundles config:
ApiPlatform\Core\Bridge\Symfony\Bundle\ApiPlatformBundle::class => ['all' => true],

Later an error:
Symfony\Component\Routing\Exception\RouteNotFoundException: Unable to generate a URL for the named route "api_doc" as such route does not exist. in vendor/symfony/routing/Generator/CompiledUrlGenerator.php:50

started to occur and it seems that the reason was lack of configuration related to api platform from latest Sylius/Sylius.
I have added minimum of compatibility updates for 1.8-dev for the existing tests to pass, yet probably you have your own procedure.
@pamil @vvasiloi
On Windows:

link does not support folders, gives permission denied even with admin privileges
symlink to folders works only with admin privileges and requires target path to be already present on Windows - in: ./win32/ioutil.c of PHP's code:

        if ((attr = GetFileAttributesW(target)) == INVALID_FILE_ATTRIBUTES) {
                SET_ERRNO_FROM_WIN32_CODE(GetLastError());
                return -1;
        }

Technically the problem of path being present is not our case here as the empty node_modules folder already exists in Application yet changes made now are prepared for a scenario when such folder does not yet exist and is created after yarn execution.
Functions respectively rely on:
php_win32_ioutil_symlink and php_win32_ioutil_symlink calls (as defined in ./Zend/zend_virtual_cwd.h of the php src)
which rely on:
php_win32_ioutil_symlink_w and php_win32_ioutil_link_w
which translate into:

php_win32_ioutil_symlink_w => CreateSymbolicLinkW(link, target, (attr & FILE_ATTRIBUTE_DIRECTORY ? 1 : 0));
php_win32_ioutil_link_w => CreateHardLinkW(link, target, NULL);

As microsoft says in their docs:
https://docs.microsoft.com/en-us/windows/win32/api/winbase/nf-winbase-createhardlinkw
CreateHardLinkW works only with files.
and CreateSymbolicLinkW can be used only without admin privileges if another flag is set (not set in the code of php) and system is in the developer mode.
https://docs.microsoft.com/en-us/windows/win32/api/winbase/nf-winbase-createsymboliclinkw
Therefore I suggest to always run the function symlink yet on windows, if it fails fallback to a Junction.
Since Junctions support already relative paths i guess there is no need for my additional approach of creating Symbolic Links (that is a different term on Windows).
In case a file already exists and is a link or directory I do not attempt to re-create it.


Repair achievements

Github has a bug where occasionally, pull requests show 400% more commits than they're supposed to.
* bcc9d205c - (HEAD -> repair-achievements, origin/repair-achievements) Added tests to ensure achievements don't crash. (10 minutes ago) <Poobslag>
*   70363ac51 - (origin/steam, steam) Merge branch 'main' into steam (11 minutes ago) <Poobslag>
|\
| *   9f66ff542 - (origin/main, origin/HEAD, main) Merge pull request #2711 from Poobslag/steam-rank-algorithms (11 minutes ago) <Poobslag>
| |\
| | * e4902c31c - (origin/steam-rank-algorithms) Restored missing rank methods required by steam. (13 minutes ago) <Poobslag>

The steam branch is currently at 70363ac51, and the repair-achievements branch is currently at bcc9d205c. They are direct descendants, but Github is being Github. I will try opening a new pull request and hope Github is not Github.

This issue was in fact fixed by #2713. Github is thankfully, no longer Github.


renameIceURLs causes issue on firefox

Hi.
The renameIceURLs call causes issues on firefox in the following code:
config.iceServers.forEach(function (server) {
server.url = server.urls;
delete server.urls;
});
Basically, if the iceServers ALREADY only has server.url then this is wiped out and is set to undefined.
To put it another way, if the entry does not have 'urls' but is already defined as 'url' in the config then this buggers it up..
The correct way to do this should have been server.usr = server.url || server.urls where server.url take precedence
Regards

iceServers shouldn't have a no-longer-spec .url set.
Possibly the code could warn there but Chrome would still accept it without warning so I am not sure how useful that is.

Ping?

This was in an older version of adapter.js but can now be closed as it's no longer relevant
Regards

I see, thanks for your response!


Compilation errors on Windows 7 Professional with go1.5.2.windows-386

D:\Platform\Go\test-01-Burrow\Burrow>set GOPATH=D:\Platform\Go\test-01-Burrow\Burrow
D:\Platform\Go\test-01-Burrow\Burrow>set GOBIN=D:\Platform\Go\test-01-Burrow\Burrow\bin
D:\Platform\Go\test-01-Burrow\Burrow>go install
/D/Platform/Go/test-01-Burrow/Burrow
.\logger.go:54: undefined: syscall.Dup2
.\logger.go:55: undefined: syscall.Dup2
.\main.go:211: undefined: syscall.SIGSTOP
D:\Platform\Go\test-01-Burrow\Burrow>

Windows is not a target platform for me. The problem is obviously the way that Burrow is setting up the logs and registering signal handlers, but this is not something that I'm going to be able to fix (as I do not have windows systems to work on), nor something that I would work on.
If you have a patch that will resolve the issue, I encourage you to submit a PR for it. Otherwise, I am closing this issue as a "wont fix" as Windows is not a supported platform.


Add support for component aws-ddbstream

Found 0 matching posts for subject:aws-ddbstream in Camel - Users within 700 days


Tutorial module: Events, interactions, and inventory

Akin to #2153 and #2155 - a dedicated tutorial module for taking a step-by-step approach to learning how events work (beyond the super basics) and how to interact with something (inventory, interaction screen, etc)
Maybe #2153 will minimally touch on how to catch an event of interest, but just to show how you can get an entity and use a component attached to it. This one should go into details about you consume/cancel events, do more fancy things with them, and interact with some core systems like inventory.
We have a forum thread on interaction screens: http://forum.terasology.org/threads/interaction-screens.1106/ - it linked to a tutorial n the wiki which I removed in the great wiki purge intending to get something updated elsewhere, namely this dedicated tutorial!
The original tutorial is still available at https://github.com/MovingBlocks/TempDoc/blob/master/Tutorial:-How-to-make-blocks-with-custom-UI.md
Bonus: probably should wait and see what magic new inventory magic @Josharias is sprinkling on the engine before doing this :-)
We should also be sure to update / replace the forum thread when this is done.
I marked this with the multiplayer label thinking this might get advanced enough to touch on specific things to consider there. Unsure. Might be needed in the others too (like marking variables in Components for multiplayer)

Addition: With #2175 merged we now have some fun explosions to play with. Tutorials are always better if they contain explosions, right?
Probably some good examples in there of configuration, the "Action" setup, and interactions between things.

Another angle: Show how a layered set of related systems might flow through a series of events differently. Thinking something like damage being modified due to equipment (both armor and weaponry), magic (shielding etc), and special modes like ghost/god mode.
Got to thinking about it while reviewing https://github.com/MovingBlocks/Terasology/pull/2323 which makes ghost/god mode not hurt the player, but by explicitly checking for that in the health system. The really old issue #652 that the PR fixes gets a little into event cancellation, which should probably be used instead, but I don't think we have any great examples or best practices for how to do so.
I did find a water breathing setup in the AlterationEffects module that uses event cancellation to prevent drowning damage, but I'm not sure how completed all that was. I think it was one of the things @MarcinSc started near the end of sorts of the active period for Throughout The Ages development (alchemy/potions) - unsure if usable, pinging in case of knowledge :-)
Pinging @portokaliu and @xrtariq2594 for general interest.

Addendum from #3003 - on death the player's inventory now persists. Different gameplay modes may want to vary that (lose all, keep all, drop all, partial, etc). We should improve support for that if needed and cover it in the tutorial.

Closing as we have a bunch of tutorial modules and improved on the docs in the engine wiki about events and interaction between systems.


Fix: add guard for dev mode

Checklist

[x] Related issue: #105
[x] Changesets done
[x] Docs Updated

Body

This pull request adds a new check to ensure that developers who enable dev mode with dev.global have at least one guild ID provided in the dev.guilds array. If there are no guild IDs provided, the code will throw an error.
This check also takes into account any dev commands that may be present in the commands set. If there are any dev commands, but no guild IDs are provided, the code will also throw an error

Awesome ty!


[0.9.0] avaliable attribute list

Thank you for your brilliant work.
Now I am using 0.9.0, I am trying to set attribute of camera sensor. But I don't know what key word to use.
I think it would be great if you could make a list of attribute of all actors, blueprints.
Or, you can just tell me how to check, I can check it by myself.

Vehicle List, If you are interested.
INFO: [a2, audi, vehicle]
INFO: [chevrolet, impala, vehicle]
INFO: [police, dodgecharge, vehicle]
INFO: [wranglerrubicon, jeep, vehicle]
INFO: [isetta, bmw, vehicle]
INFO: [model3, tesla, vehicle]
INFO: [prius, toyota, vehicle]
INFO: [ford, mustang, vehicle]
INFO: [micra, nissan, vehicle]
INFO: [grandtourer, bmw, vehicle]
INFO: [tt, audi, vehicle]
INFO: [t2, volkswagen, vehicle]
INFO: [mercedesbenz, coupe, vehicle]
INFO: [nissan, patrol, vehicle]
INFO: [c3, citroen, vehicle]
INFO: [leon, seat, vehicle]
INFO: [carlacola, carlamotors, vehicle]
INFO: [mini, cooperst, vehicle]

Carmera Sensor attribute list, if you are interested:
 image_size_x
 image_size_y
 fov
 post_processing

Hi @hdy117,
Thanks for sharing the info! and sorry for the problem of getting your issue duplicated a million times :sweat_smile:
Yes indeed, we didn't expose any method to retrieve the available attributes in the Python API. I will add a method to retrieve them.


Could not find a declaration

Next.js 14
Typescript error
Could not find a declaration file for module @piotr-cz/swr-idb-cache.
<EMAIL_ADDRESS>implicitly has an  any  type.
There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json exports.
The @piotr-cz/swr-idb-cache library may need to update its package.json or typings.

Do you have any idea how to fix this?
If the issue occurs only in VScode, try restarting TS Server (ctrl + shift + p > TypeScript: Restart TS Server)

Seems that the package.json file should be updated like this: https://github.com/huozhi/bunchee?tab=readme-ov-file#typescript

this is the PR to fix this https://github.com/piotr-cz/swr-idb-cache/pull/16

@KoBa4 Can you confirm that #16 fixed this issue?

Fix released in v1.0.3


Replace useSDKChainId with useDashboardEVMChainId

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR updates chain ID usage in multiple components to useDashboardEVMChainId.
Detailed summary

Updated chain ID usage in various components to useDashboardEVMChainId
Added import for useDashboardEVMChainId in relevant files

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

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

#4137  👈
#4132 : 1 other dependent PR (#4133 )
#4130
#4129
#4128
main

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

Join @kien-ngo and the rest of your teammates on  Graphite


METRON-943: Create traffic connections report in zeppelin

Contributor Comments
Adds a Zeppelin dashboard that lets the user get connection counts filtered by a CIDR block.
The implementation is a little kludgy because Spark/Hive don't have easy operations to handle things like IPs and CIDRs. I'd have liked to keep it in one paragraph, but the clunkiness around handling the ips and cidr made that a pain that seemed significantly worse.  If someone knows an easy way to take care of it, I can quick try it.
Do we want/need similar paragraphs for the other sources?  Given that we're primarily looking for volume of connections, it seems unnecessary, but it might be nice to have (possibly as a follow-on?)  It's pretty to repeat the paragraphs as needed.
Also, let me know if there are any verbiage changes (since that came up on another dashboard), or other adjustments that should be made.
Test Plan
To test, spin up full-dev.   To get Yaf data, it'll be necessary to start the sensor-stub
service sensor-stubs start yaf

It'll also be necessary to add Yaf to the list of sensors run in Ambari. To do so, stop Metron, edit "Metron Parsers" to include "yaf" (or be yaf only). Start Metron and ensure a yaf topology is present.
Let data flow through.
Once some data has gone through, we'll need to have an instance of Zeppelin.  Because of the size of the Vagrant instance, we'll want to shut down unneeded services.  Shutdown Metron, Kibana, Storm, Kafka, and HBase.
Install Zeppelin from "Actions - Add Service".  It'll prompt you to install Spark and Hive, do so.  Configuration is pretty trivial, all that's necessary is to set an arbitrary Hive database password.  Let this run.  The Hive service check likes to fail on our Vagrant, but it's benign (some impersonation configuration issue unrelated to actually running our queries).  Ignore it and accept the installation.
From Metron's "Service Actions", run the "Zeppelin Notebook Import", to load our notebooks into Zeppelin.  Use the quick links to navigate to the Zeppelin UI.
Go into the "Metron - Connection Volume Report" notebook.  Queries can be made by CIDR, e.g. <IP_ADDRESS>/8 to get total amount of traffic by source or by destination IP range.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root incubating-metron folder via:
mvn -q clean integration-test install && build_utils/verify_licenses.sh

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

perfect. + 1 on the dashboard.  Looks like travis failed, though

Resolve conflicts in the spec file now that METRON-945 is in master


docs: update Box example to match the API

Hi ! Thanks for your jobs.
Example in the Readme.md don't match the current API of dessert-box.
There is a little fix in the README.md

@Melvynx closing as the API now returns just the component again!


Add client function for searching posts

Summary
Add client function for searching posts.

Feel free to clobber this one with yours when you rebase, I just needed it for the webapp. No plans to write the reducer anytime soon, I was going to wait for you to do that ;)


[ASK] Scan using button or continous mode

I was looking at Honeywell Demo Apps and they have some button, 'Scan' button and 'Continous' mode, is there any function that I could use for manually scan (not using hardware button) or maybe 'Continous' mode scan? thank you

Hi @felixcary.
For "manually scan" also known in documentation as softwareTrigger I just released a new version v3.2.0+13 which allows to startScanning() and stopScanning(), you can check the plugin readme for more info and the example project for better understanding.
Regarding the "Continuos" scanning mode, that is a property configuration which is already available by setting the corresponding property. For advanced use of this plugin it's mandatory to read the official Honeywell documentation, in the readme I point how to get the official documentation and to make devs lives easier I also included in the source code of this plugin a doc folder with the necessary info to know each property key and value.
From doc folder, file BarcodeReader.html, you can find:
PROPERTY_TRIGGER_SCAN_MODE
public static final java.lang.String PROPERTY_TRIGGER_SCAN_MODE
Trigger scan mode. This setting determines what the behavior will be when the trigger is pressed.

and
TRIGGER_SCAN_MODE_CONTINUOUS
public static final java.lang.String TRIGGER_SCAN_MODE_CONTINUOUS
Continuous. When this setting is applied, pressing the trigger button will allow to scan multiple barcodes, however duplicate barcodes are not scanned.

Then with the names of the properties you can go to doc folder, file BarcodeReaderProperties.java and find:
public static final String PROPERTY_TRIGGER_SCAN_MODE = "TRIG_SCAN_MODE";
public static final String TRIGGER_SCAN_MODE_CONTINUOUS = "continuous";

And finally you can set the key,value pair you need to set the property:
honeywellScanner.setProperties({
   'TRIG_SCAN_MODE': 'continuous',
});

Note: take into account that .setProperties({...}) it's a SET function, so you should set all properties you need at once.

Hi @luis901101, thank you so much for your help! I'll try the new version and also property configuration.


distanceLines3d: Additional ouput: points on the line

Hi
IMO it would be a nice feature if the function "distanceLines3d" could provide the points on the lines with the minimal distance as additional output.
Kind regards

Hi,
I agree, this would be nice!
I had a quick look, but this does not seem that straightforward...
I add the task on my TODO list!
regards

Here is some code. But it works only if the two line matrices have the same size in the first dimension
clearvars; close all

line1 = [0 -25 -50 0 0 1; 25 25 0 1 1 0.5];
line2 = [0 -50 0 1 0 0; -50 0 50 1 0 2];

a = dot(line1(:,4:6),line1(:,4:6),2);
b = dot(line1(:,4:6),line2(:,4:6),2);
e = dot(line2(:,4:6),line2(:,4:6),2);

den = a.*e-b.*b; % 0, if lines are parallel
r = line1(:,1:3)-line2(:,1:3);
c = dot(line1(:,4:6),r,2);
f = dot(line2(:,4:6),r,2);

s = (b.*f - c.*e)./den;
t = (a.*f - c.*b)./den;

pt1 = line1(:,1:3) + line1(:,4:6).*s;
pt2 = line2(:,1:3) + line2(:,4:6).*t;

figure
xlim([-100, 100]),ylim([-100, 100]),zlim([-100, 100])
hold on; grid on
xlabel x; ylabel y; zlabel z;
drawLine3d(line1,'b')
drawLine3d(line2,'r')
drawPoint3d(pt1);
drawPoint3d(pt2);
view(3)
axis equal

distanceLines3d(line1, line2)
distancePoints3d(pt1, pt2)

thanks,
I'll add it ASAP.

Hi,
I have added a fix. You can use the following syntax:
[d, pt1, pt2] = distanceLines3d(line1, line2);
I think the code can be improved , but it seems to work.
regards,
David


Requiring @babel/runtime

After installing paymongo on an empty express app and running the ff code, it errors out.
const paymongo = require("paymongo");

Something similar to this link.

Hello! Thank you for submitting an issue.

I don't think I need to make another issue for this one but there's another typo I found in the paymentMethods area
{
  data: {
    attributes: {
      type: 'card' // The only available type for now is 'card'.
      details: {
        card_number: '4343434343434345',
        exp_month: 02,
        exp_year: 23,
        cvc: '123',
      }
    }
  }
}

After the type: 'card' part, it's missing a comma.

the only way im able to get it to work is by requiring the runtime in my app explicitly
npm i @babel/runtime
is this the intended use-case for this?

Do we have corresponding pod dependencies for React Native? Thank you in advance :)

the only way im able to get it to work is by requiring the runtime in my app explicitly
npm i @babel/runtime
is this the intended use-case for this?

Hello, it's not the intended use-case, but it's one way to fix this. I will look into this. Thanks.

Do we have corresponding pod dependencies for React Native? Thank you in advance :)

Hello @kimberleyvarona none so far, however, I don't see the purpose of have one. This package is explicitly for node server use only. I'm not quite familiar with React Native, but is it possible to use this in React Native?

Ohh I see. I found some community contribution to React Native library providing support with Paymongo. But I believe it would be best if it would be coming from you guys. By the way, I manage to play around with the API and use Webview for the meantime.

+1 on having this problem, would love to see this dependency dropped and have a native implementation.

+1 on having this problem, would love to see this dependency dropped and have a native implementation.

I'll add this to the pipeline. Thanks.


Link http://go.microsoft.com/fwlink/?LinkID=71255 is broken

Not sure what is the correct location, same link is used in various others locations.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2bcb7cc8-20a5-5816-dfdf-e9c7c69fe30c
Version Independent ID: 28ef839c-641c-a6bf-b3e1-bcb650141763
Content: PackageRelationship Class (System.IO.Packaging)
Content Source: xml/System.IO.Packaging/PackageRelationship.xml
Product: .net
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

As far as I can tell, the Open Packaging Conventions specification is part of ECMA-376 and its latest version can be downloaded from https://www.ecma-international.org/publications/files/ECMA-ST/ECMA-376, Fourth Edition, Part 2 - Open Packaging Conventions.zip.
Unless there is a better link (ideally directly to the PDF file), I think that link could be used here.

I just updated the link to point to https://www.ecma-international.org/publications/standards/Ecma-376.htm while we decided what needs to happen here.

@mairaw, any updates on this? Thanks!


[video_player_android] add feature to control setAllowedCapturePolicy feature in ExoPlayer

Use case
I want to prevent internal sound capture on Android 10+
but I can't access the setAllowedCapturePolicy  feature in ExoPlayer to do that.
Proposal
Provide an API to set the allowedCapturePolicy in Exoplayer.

i got this error for m3u8 http "Video player had error com.google.android.exoplayer2.ExoPlaybackException: Source error"


Rearrange type parameters around Block trait

Thanks!
bors r+

I'll merge it. Hope we have bors here soon.


Modular TokenTactics

Token Tactics is now modular.
the original tokentactics.ps1 now lives within /modules/ as TokenHandler.ps1.
The original PS1 has been broken into 2 module files.

Added & Tested the -Device and -Browser spoof parameters to all the RefreshTo- commands.
Spoofs the User-Agent header to potentially bypass Conditional Access Policies.


Amongusdiscord.exe crashing

The amongusdiscord.exe part of the bot seems to crash. It crashes more frequently when muting/unmuting players at the beginning of the round, however has happened in the middle many times.
I tried about 8 times but was unsuccessful in completing a whole game without it crashing.
I trialed the bot several times with just myself and an alt account and it worked fine. however once playing with 8 people in the discord it crahes.
The final log messages before crashing was:
HTTP 404 Not Found, {"message": "Unknown Message", "code": 10008}
I was using version 2.0.3 but did encounter the same problem in the couple of times I tested 2.0.4
Thanks

Just a heads up, I ran into this error as well but not from running in it an 8 player game (I haven't done that quite yet). Instead, I've been working on a modified version for use in my server. I received this message by deleting this message before the last reaction was added:

Just a heads up, I ran into this error as well but not from running in it an 8 player game (I haven't done that quite yet). Instead, I've been working on a modified version for use in my server. I received this message by deleting this message before the last reaction was added:

To help troubleshoot, perhaps there is something deleting the message before the reacts are able to be applied. @stormlv368, what bots do you currently have installed? And is it possible one of them are deleting messages from the AmongUsDiscord bot?
To assist further, I was able to find the section of the code that is applying the reactions:
https://github.com/denverquane/amongusdiscord/blob/4b972971baf087b45f2e76f9192a14e52af8413e/discord/message_handlers.go#L32-L45
https://github.com/denverquane/amongusdiscord/blob/4b972971baf087b45f2e76f9192a14e52af8413e/discord/message_handlers.go#L101-L106
I'm not entirely sure how the MessageReactionAdd method is handled, but it seems to be using HTTP methods to achieve it.
It's also possible these two issues are not related. And it's just two different HTTP errors.
*Additionally, although I did see this error message for each react that wasn't able to find the message, it didn't crash for me. It just kept going.

The message is not being deleted. And no other bots should be deleting the messages.
However the bots I have are as followed:
Dyno Bot - Majority of the functions are turned off though (especially auto mod which is the main thing that would try and interfere with the among us bot.)
Ephemeral Roles - Adds a role to every user that joins a voice channel (can not manage messages)
Reaction Roles - Only used in one channel so shouldn't be interfering with the bot.
One thing that may be the problem is the fact that users are reacting to the message as soon as their colour pops up, rather than waiting until all the colours are there. I am unsure if this is the problem, and I am yet to test it properly - I will be testing it tonight with a few friends.
Thanks.

Just to check in:
I'm almost positive that this is a race condition of some sort. We're definitely not locking/unlocking the status message properly, and it is accessed by many asynchronous parts of the code. I went ahead and made a "threadsafeRefactor" branch; this was going to be necessary at some point, so it might as well be now, while the codebase is still growing but not out of control.
So far I have all the in-game user data, and channels being tracked wrapped up in thread-safe accessors/transformer functions. Next step is for me to refactor the discord-related userData (the largest portion), and as we've seen here, definitely the singular status message, to both be locking/threadsafe.
Just wanted to post and let you guys know it's being actively worked 👍

Just an update.
I have done some extra testing and have found that if you let the bot react with all the colours before linking accounts then everything works perfectly. Other than a few minor hiccups, however not once did the bot crash for me.
Thanks

have you tried the latest release (2.0.5)? let us know if it works for you. thanks!


Stop target _black attackts

See https://www.jitbit.com/alexblog/256-targetblank---the-most-underestimated-vulnerability-ever/

Not really an issue for this project as trusted user content.


Donuts nl patch 1

Hi @DonutsNL , Very sorry I have been away from this project for a bit.  I tried to Merge this branch and its showing a conflicting file debug/debug_dump-2023-06-21-21:43:30.php.  Are you able to resolve?

Strange that file isnt (or shouldnt) be present in the merge. Its part of .gitignore. Are you able to delete it?

I sais only those with write access are able to merge/correct the issue. Thats not me. Also i can use your help with testing and reviewing the latest version of which i am not sure is part of this pullrequest.

I created a new pull request into version 1.3.0 branch and updated the sourcefiles accordingly.


docker-up-vinyldns.sh fails to reliably start VinylDNS services

Describe the bug
bin/docker-up-vinyldns.sh frequently fails to start VinylDNS services.
VinylDNS Version
master
To Reproduce
Steps to reproduce the behavior:

Run bin/docker-up-vinyldns.sh from within the root of the vinyldns repository
Observe that the script frequently exits with messages such as Exceeded retries waiting for VinylDNS API to be ready, failing.

Expected behavior
I would expect bin/docker-up-vinyldns.sh to reliably start VinylDNS services.
Stack trace or error log output
bin/docker-up-vinyldns.sh
Starting portal server and all dependencies in the background...

Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Retrying Again
Exceeded retries waiting for VinylDNS API to be ready, failing

Additional context
I have observed this problem on my local Macbook Pro, as well as in TravisCI.

Hm, we set that retry with all the vinyl devs pretty much using the same machines, we could increase the retry another minute or two to accommodate for other machines.
Could also add an arg variable to override timeout?

Right, I think increasing the timeout should be fine.  We have to wait for mysql to start up, then run our migrations (which run on an empty environment), and finally wait for the app to start up which itself takes time (akka-http most of it).

The change here is to edit the bin/docker-up-vinyldns.sh script, and change the RETRY=40 to RETRY=60 in two places.

I am re-opening this issue, as it seems I'm not having difficulty reliably running VinyLDNS in TravisCI, even when using the --timeout=120 flag:
https://travis-ci.org/vinyldns/go-vinyldns/jobs/448542183
In conversation about this, @rebstar6 pointed out that it's arguably unnecessary to start both the API server and the portal for running my vinyldns/go-vinyldns integration tests; just the API should suffice. However, it appears there's no easy way to run a local Dockerized API server in absence of the portal, as docker-up-api-server.sh expects a local VinylDNS build & jar file. She suggested changing docker-up-api.sh to docker-up-api-local.sh and creating a docker-up-api.sh that functions similar to docker-up-vinyldns.sh but starts just the API.

The problem is we bind docker/portal/application.conf to the portal image in our docker compose file, and we have been changing that up since publishing the image in ways that are no longer compatible. Will just remove that conf because really the portal image can just use the default that is packaged in there

@nimaeskandary I see. I did also create PR #318 , though I can see you may have a preferable solution in mind.

@mdb Will try to kill two birds with one stone and fix this issue, and allow for you to just start up the api if you want

That'd be great -- thanks @nimaeskandary !

@mdb @NorseGaud a fix for this was just merged. Also allows you start up the docker-up-vinyldns.sh script without the portal using the --api-only flag


Electrical Lift

[ ] Create mod shaft power integration

@OmegaHaxors suggested this one...
Possibly with integration with Create's linear chassis, the electrical lift can move players up and down in a structure.
Potentially, either support a cable based elevator, or a hydraulic elevator. Logic circuits should probably be hand-made PLC's.

This is going to be made in the form of a generator/motor that integrates with Create Shafts.


Initialized gradle project. Added collections class and tests

Hey guys, what do you think? I initialized a gradle project ( Would you prefer using maven? ) and made a small section with some methods for the students to implement and tests to check them.
I tested it in the gitpod, and it looks like the default Java VSCode extension doesn't support Java 21 yet, but the fix is already in it's own pull request: https://github.com/redhat-developer/vscode-java/pull/3314 , so it should work by December.

Let's do this again, but with Maven instead of gradle :)


_arguments:comparguments:325: can only be called from completion function

The zsh completion scripts make zsh throw an error.
_arguments:comparguments:325: can only be called from completion function
This is due to the fact, that both zsh completion scripts call _arguments outside a function.
Fix
Put the completion scripts into a function, as shown below:
kubectx.zsh
 #compdef kubectx kctx=kubectx

__kubectx() {
  local KUBECTX="${HOME}/.kube/kubectx"
  PREV=""

  local all_contexts="$(kubectl config get-contexts --output='name')"
  if [ -f "$KUBECTX" ]; then
      # show '-' only if there's a saved previous context
      local PREV=$(cat "${KUBECTX}")

      _arguments \
        "-d:*: :(${all_contexts})" \
        "(- *): :(- ${all_contexts})"
  else
      _arguments \
        "-d:*: :(${all_contexts})" \
        "(- *): :(${all_contexts})"
  fi
}

kubens.zsh
#compdef kubens kns=kubens

__kubens() {
  _arguments "1: :(- $(kubectl get namespaces -o=jsonpath='{range .items[*].metadata.name}{@}{"\n"}{end}'))"
}

It doesn't a throw an error for me on my oh-my-zsh setup with zsh 5.8 (x86_64-apple-darwin20.0).
Can we first try to figure out what causes the error?

It doesn't a throw an error for me on my oh-my-zsh setup with zsh 5.8 (x86_64-apple-darwin20.0).
Can we first try to figure out what causes the error?

The error is thrown, when the autocompletions are loaded, e.g. when I start a new instance of my zsh (zsh 5.7.1 (x86_64-apple-darwin19.0)).
Despite the error thrown, the completion itself does work.
The error thrown due to the fact that the _arguments are called outside a function. This is why I wrapped the completion scripts with an the functions __kubens() and __kubectx() silencing the error.

The error is thrown, when the autocompletions are loaded, e.g. when I start a new instance of my zsh (zsh 5.7.1 (x86_64-apple-darwin19.0)).
Despite the error thrown, the completion itself does work.
The error thrown due to the fact that the _arguments are called outside a function. This is why I wrapped the completion scripts with an the functions __kubens() and __kubectx() silencing the error.

I was seeing this issue until I realised that rather than sourcing the completions they needed to be put into a directory on zsh's fpath
E.g. something like (untested)
# Create a folder ~/.zsh to put files zsh will load
mkdir "${HOME}/.zsh"
flamegraph --completions zsh > "${HOME}/.zsh/.flamegraph-completion.zsh"

# Add .zsh to your fpath when the zsh shell starts
echo 'fpath+=~/.zsh' >>"${HOME}/.zshrc"

ok, I found a solution!

Load the kubectx completion before ohmyzsh (as stated in the docs)
ignore the error log by changing the _arguments \ statement to _arguments >/dev/null 2>&1 \

sending a pr

I found this script somwhere and it helped, just copy and paste it into the zsh terminal and the commands will run just fine.
cd ~
mkdir ~/.zsh
swift package completion-tool generate-zsh-script > ~/.zsh/_swift
echo -e "fpath=(~/.zsh \$fpath)\n" >> ~/.zshrc
echo -e "autoload -Uz compinit && compinit" >> ~/.zshrc


MINOR: Improve release script

These are the fixes for the problems I encountered while creating the RC for the 3.8.0 version.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@soarez these are the changes we discussed already.

Feedback applied @soarez

No need to run the CI for this one as it won't have any impact on CI


Frontend support for creation of snapshots - Part 2

This is a followup PR for the closed #222 and frontend support for #221.
Changes:

Watch creation of VolumeSnapshot CR and trigger creation of backend snapshot. On successful snapshot creation, create a VolumeSnapshotContent and bind it with VolumeSnapshot.
Create snapshot CRD during installation and delete during uninstallation.
Add permissions to the cluster role.
Add necessary packages.

Test Setup

Kubernetes cluster on AWS
Trident deployed with AWS-CVS as backend
CSI hostpath driver along with csi-provisioner and csi-snapshotter

The following explains the flow of snapshot and restore process.
Snapshot Flow
User

Create a custom VolumeSnapshotClass with snapshotter as netapp.io/trident. This is not registered with any driver and can be considered as a dummy class.

apiVersion: snapshot.storage.k8s.io/v1alpha1
kind: VolumeSnapshotClass
metadata:
  name: trident-snapclass
  annotations:
    snapshot.storage.kubernetes.io/is-default-class: "true"
snapshotter: netapp.io/trident

Create a trident PVC.

kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: podpvc
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 1Gi
  storageClassName: basic

Create a VolumeSnapshot for the trident PVC with the dummy snapshot class.

apiVersion: snapshot.storage.k8s.io/v1alpha1
kind: VolumeSnapshot
metadata:
  name: snapshot-test
spec:
  snapshotClassName: trident-snapclass
  source:
    name: podpvc
    kind: PersistentVolumeClaim

This VolumeSnapshot is ignored by the csi-snapshotter since the class is not associated with the registered driver. (Currently, each external-snapshotter can be configured with only a single driver)
Trident K8s Frontend

Watch for VolumeSnapshot created for trident PVCs.
Trigger a backend storage snapshot.
On successful creation of a snapshot, create a VolumeSnapshotContent and bind it to the VolumeSnapshot. For example:

apiVersion: snapshot.storage.k8s.io/v1alpha1
kind: VolumeSnapshotContent
metadata:
  name: content-<snapshot-test-uid>
spec:
  csiVolumeSnapshotSource:
    driver: netapp.io/trident
    snapshotHandle: <backend-snapshot-id>
  volumeSnapshotRef:
    kind: VolumeSnapshot
    name: snapshot-test
    namespace: <snapshot-namespace>
    uid: <snapshot-test-UID>
  persistentVolumeRef:
    kind: "PersistentVolume"
    name: <pv-name>
    namespace: <pv-namespace>
    uid: <pv-UID>
  snapshotClassName: trident-snapclass

This again is ignored by the csi-snapshotter.

Update the VolumeSnapshotStatus with relevant information and VolumeSnapshotSpec with snapshotContentName.

Trident Orchestrator

Create a snapshot on the corresponding backend storage.
Store the state of the snapshot in the persistent store with SnapshotID as the key.

CreateVolumeFromSnapshot flow
User

Create a trident PVC with dataSource as the VolumeSnapshot

kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: podpvc-from-snapshot
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 1Gi
  storageClassName: basic
  dataSource:
    kind: VolumeSnapshot
    name: snapshot-test
    apiGroup: snapshot.storage.k8s.io

Trident K8s frontend

In processPendingClaim, similar to checking if a clone is required, check if a dataSource is present.
Extract the VolumeSnapshotContent information from the VolumeSnapshot and then get the snapshotHandle.
Pass on the snapshotHandle to create a volume from snapshot.

Trident Orchestrator

Fetch the snapshot object from the persistent store using the snapshotHandle (ID) as the key.
Create a volume using the snapshot as the source.
The function signature would look like CreateVolumeFromSnapshot(snapshotID string, volConfig *storage.VolumeConfig) (externalVol *storage.VolumeExternal, error)

Question on VolumeSnapshotClass

There are multiple ways to create the dummy VolumeSnapshotClass.

We can document it as a prerequisite for using the VolumeSnapshot feature and provide an example class for the user to create it manually.
We can also deploy a default class during the installation controlled by a flag.
What would you prefer?

Thanks for the PR!
Like we discussed before, this is a great interim solution on our path to one that would use CSI sidecars to manage snapshots. We want to make sure that will be a seamless transition. That snapshots created and managed by this code here would still be fully usable once we move to CSI.
As part of the transition to CSI, we'll be re-creating any PVs/PVCs under management, so that the PVs are using the csi driver instead of the iSCSI or NFS drivers we use today.
Can you describe what work, if any, would need to be done to make sure that a seamless transition occurs? What should we be doing now to minimize if not eliminate any conversion work?

Thanks Garrett. Sure, I will post an update on the transition approach soon.

@innergy
One simple approach would be similar to import section in https://kubernetes.io/blog/2018/10/09/introducing-volume-snapshot-alpha-for-kubernetes/

A prerequisite would be to have a VolumeSnapshotClass with snapshotter set to the Trident CSI driver (io.netapp.trident.csi)
Create new VolumeSnapshotContent objects for each of the existing ones. The new content objects will have the new driver and VolumeSnapshot information while retaining the snapshotHandle from the existing non-CSI object.

 apiVersion: snapshot.storage.k8s.io/v1alpha1
kind: VolumeSnapshotContent
metadata:
  name: csi-<old-name>
spec:
  csiVolumeSnapshotSource:
    driver: io.netapp.trident.csi
    snapshotHandle: <snapshot-id from existing content>
  volumeSnapshotRef:
    kind: VolumeSnapshot
    name: csi-snapshot-test
    namespace: <snapshot-namespace>

Create new VolumeSnapshot object with the above created content information.

apiVersion: snapshot.storage.k8s.io/v1alpha1
kind: VolumeSnapshot
metadata:
  name: snapshot-test
spec:
  snapshotClassName: <name of the new CSI class>
  snapshotContentName: <name of the above created content>

The CSI snapshot controller will bind these objects and update the VolumeSnapshot status to ready.


Add a new option to exclude unwanted k8s node labels from CiliumNode

Updates to CiliumNode object can trigger expensive operations like InjectLabels() on the ipcache subsystem. Since updates to cilium node objects are sent to all nodes in the cluster, these unwanted updates can end up wasting quite some CPU cycles.
exclude-node-label-patterns allows for dropping unwanted labels added by other controllers from being propagated to CiliumNode CRD and data stored in kvstore under cilium/state/nodes/v1/*

@christarazi do you mean retain only identity relevant labels on CiliumNode ? Are we sure there aren't features elsewhere replying on specific labels ? I remember seeing custom labels / annotations being used in BGP features. But I don't know the specifics.

cc @cilium/sig-bgp ^

Can I assume the change introduces a feature to move specific labels from the .metadata.labels of the CiliumNode CRD?
We rely on the labels of the CiliumNode to select the node to apply BGP policy but with a user-defined selector. So, users are responsible for not excluding the labels used for the selector. It applies to any resources selects node by labels, like CiliumNetworkPolicy (host firewall), CiliumNodeConfig and so on.

I haven't reviewed the PR yet, but at a high-level, could we use --labels instead? https://docs.cilium.io/en/stable/gettingstarted/terminology/#id4

I am doing something similar in my PR https://github.com/cilium/cilium/pull/26924 (using --node-labels). Seems to me that we could really just include only specific labels (security relevant), instead of excluding a lot of unnecessary ones. Thoughts?

This one slipped through cracks, opening it again since it is still relevant. Will rebase and address conflicts and comments shortly.

Seems to me that we could really just include only specific labels (security relevant), instead of excluding a lot of unnecessary ones

@oblazek we do have some subsystems relying on labels that aren't necessarily used for identity. See @YutaroHayakawa's comment above https://github.com/cilium/cilium/pull/28290#issuecomment-1738397992

Additionally I wonder if there are implications dropping labels that could be required downstream by other subsystems??

@derailed that's the reason I wanted to make this an opt-in flag so that operators can consciously make that call.

/test

/test


Bump org.apache.commons:commons-compress from 1.25.0 to 1.26.0 (#13183)

Bumps org.apache.commons:commons-compress from 1.25.0 to 1.26.0.

updated-dependencies:

dependency-name: org.apache.commons:commons-compress dependency-type: direct:production update-type: version-update:semver-minor ...

Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

CVE-2024-26308 and CVE-2024-25710

/component-test minio zip-deflater tarfile

Tests are failing for missing snapshot.


Were do i put my bots user and password

Hi how do i use my reddit account and use this tipbot? I want to put it on my sub reddit

As of right now, subreddits getting their own account with the bot is not supported. Ask @just-an-dev for more details about that. If you're talking about getting the bot to work on your subreddit, it already works everywhere the bot isn't banned.

this bot works on all sub reddit :) you just have to register.

@just-an-dev im trying to fork it :P but i have another problem
it is
File "/home/myname/sodogetip/models.py", line 281, in get_user_address return data[0].get('address')
IndexError: list index out of range
. i want to fork it so me and others could tip in our crypto but had that problem and cant find a fix. idk im 14 and i was reading aa forum and it said you get better by doing open source stuff and i couldn't this to work :( I hope your not mad idk lol

@mrmetech let me know if you need help for your fork (open a new issue, or come in PM on reddit)


WIP: Create a ServerError class.

NOTE: This is a preview, I'm looking for some feedback and the definition of ServerError...once we settle on the basic idea, then i'll go and change as many ParseErrors into ServerErrors that makes sense.
see: #661
The problem this pr is trying to solve:
When an error occurs on the server, a message should
be returned to the client, and a message should be logged.
Currently, noting is often logged.
Byt adding a ServerError class, we can embed a ParseError into
the ServerError.  The ServerError contains the message for the log
and the ParseError contains the error for the client.
In addition, this pr will stop calling the default express error handler
which causes two problems: 1. it writes to console instead of log file
2. the output is completely useless! :)

grrr.


[Signup Page] Bad design and usability

The current login page for our cloud storage website is poorly designed and difficult to use. The colors are jarring, the layout is cluttered, and the text is difficult to read.
Changes that could be made to the signup page:

Use a more modern and stylish design.
Use a simpler layout with less clutter.
Use larger fonts and easier-to-read text.
Make the login process faster and more responsive.

By making these changes to the signup page, we can make it more user-friendly and inviting.

| An Example for a better looking signup page is: |
Contributor - Sabhi Sharma (GSSOC'23)

i can make sign -in page and sign-up page more attractive by using tailwindcss please assign me for this issue @MonalikaPatnaik @i-am-SnehaChauhan


Support "elastic" throughput mode on AWS::EFS::FileSystem

Add the missing "elastic" type to EFS throughput mode validation.
See: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-efs-filesystem.html#cfn-efs-filesystem-throughputmode

Not sure how much visibility this issue has on it, but I could use this addition. Haha

Thank you for the PR. Merged manually to tweak the change slightly via aae0832d586322ad93856256f5e69ec7e2da5a34.


Feature request: Add support for persisted queries & caching

This is a feature request.
I'd like to propose to add support for persisted queries. This could be an additional proxy service layer that handles both storage and resolving of query ids to their respective query statements. Once persisted queries are supported, caching could be implemented for these persisted queries (now GET requests). A very basic solution could simply allow saving persisted queries (just like the example queries) in the graphql explorer and assigning a TTL for those (with optional stale-while-revalidate).

It could even be implemented as an APQ service layer where the query itself defines its desired TTL in the extensions section of the query. The query would then be collected by the proxy (alongside its respective query hash) and stored. For more information on how APQ would work: https://www.apollographql.com/docs/apollo-server/performance/apq/

I am not quite sure what this is asking for; if it is about ensuring that the same query returns the same result, block constraints in the query fulfill that requirement. If it is about speeding up queries, graph-node can be configured to do extensive caching internally.


Low Latency API

This only affects Android.
This is a new audioplayers backend based on SoundPool to reduce impacts on visuals or UI performance.
Note that some features of the AudioPlayer API won't work with this mode.
Comments and feedbacks are welcome :)

I absolutely love this. Added some minor reviews (details), mostly points to ponder than anything else.
A couple general points:

What happens for ios if this mode is specified? Nothing changes? like release mode? I expect that to be the case, but I want to discuss
Did you consider throwing exceptions for methods not implemented? So the user knows something is wrong, instead of failing silently? ofc with a proper message explaining etc

Did some changes.
I think it is good that someone test it using the example app.
Both Android/iOS.
Thanks

I will try to test tomorrow @feroult

@feroult tested and it seems to work! Got some of these messages though:
W/AudioTrack(24278): AUDIO_OUTPUT_FLAG_FAST denied by server; frameCount 0 -> 168192


Update dynamic inventory for TO API 2.0

What does this PR (Pull Request) do?

[ ] This PR updates API routes for dynamic inventory to API 2.0

Which Traffic Control components are affected by this PR?

CDN in a Box
Documentation
Grove
Traffic Control Client
Traffic Monitor
Traffic Ops
Traffic Ops ORT
Traffic Portal
Traffic Router
Traffic Stats
Traffic Vault

What is the best way to verify this PR?

If this is a bug fix, what versions of Traffic Control are affected?

The following criteria are ALL met by this PR

[ ] This PR includes tests OR I have explained why tests are unnecessary
[ ] This PR includes documentation OR I have explained why documentation is unnecessary
[ ] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[ ] This PR includes any and all required license headers
[ ] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[X] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Wrong branch


Improve Use Cases page

It would be cool if we have illustrations as cards for every use-case on the "Uses cases" page.
I really like how https://headlessui.com implemented this on their website:

i want to work on this issue @marcoroth ,but i want to know what i have to exactly done to resolve this issue

Hey @29deepanshutyagi, thanks for checking-in.
I guess we would need some illustrations in the same format for all the use-cases. If that's not your thing, we could start with adding cards in different colors and maybe just have the text on them instead of an image.


Seems /third_party/swagger-ui/ is missing

I've used it all the time for an API reference, and now it seems the only available option is local API server :(

Dup of https://github.com/kubernetes/kubernetes.github.io/issues/105.
fwiw, swagger-ui will be disabled by default on local api server: https://github.com/kubernetes/kubernetes/pull/23025. You will be able to turn it on by setting a flag.
Please reply to https://github.com/kubernetes/kubernetes.github.io/issues/112#issuecomment-196527345 if you prefer swagger UI.

Added 404.md forwarding


Feature Request: Option for change loglevel?

Is there anyway to change loglevel for the default warn level?

The current available config is the -debug flag, which enables more verbose logging.
# both are equivalent

imagor -debug -imagor-secret 1234

DEBUG=1 IMAGOR_SECRET=1234 imagor

Actually, I wanna less details in production mode.
For example: only error messages were shown.


Rotate ssh key for paulbowsher

Hi @boffbowsh, I notice that this is a 2048-bit key. That's sufficient, but you might consider upgrading this to a 4096-bit key, which may be more secure, e.g.:
ssh-keygen -t rsa -b 4096

Upgraded, thanks @mattbostock


Removed boto3 dependency and the s3 configuration from the API

Fixes

Fixes #2305  by @krysal
Description
Amazon buckets ain't getting used from the API side. Removed this from the Pipfiles and the code related to S3 configs, including the environment variable(s) from template files. Deleted s3.py and removed related code.

Testing Instructions

Checklist

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

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

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

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

Developer's Certificate of Origin 1.1

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

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

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

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

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

@YashGaur000 since you have already closed this, let's start from there.

Create a new branch in your local repository:
$ git checkout -b fix_2675

Fetch the current state of origin/main:
$ git fetch origin main

Hard reset fix_2675 to origin/main:
$ git reset --hard origin/main

Make and commit your changes.

Push to your fork.

Make a new PR.

In general, always checkout a new branch from origin/main for each issue you're aiming to fix, make your changes on that branch and make the PR. This prevents multiple channels of work from intersecting or overlapping in PRs.


BuildConfig.DEBUG is  Boolean.parseBoolean("true") with release build type

With Android Studio, build the project and found DEBUG is Boolean.parseBoolean("true"), which might be Boolean.parseBoolean("false").

Hm, maybe you've switched build type and didn't sync gradle in Android
Studio? BuildConfig should be regenerated from scratch, I don't remember
putting "debuggable true" for release build type.
On Wed, Jun 1, 2016 at 10:12 AM, Yang Feng<EMAIL_ADDRESS>wrote:

With Android Studio, build the project and found DEBUG is
Boolean.parseBoolean("true"), which might be Boolean.parseBoolean("false").
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/issues/180, or mute
the thread
https://github.com/notifications/unsubscribe/AA7B3H0QMyPOQGLw-AbK41p1v21dwuM0ks5qHTDCgaJpZM4IrQDT
.

You are right, BuildConfig is regenerated from scratch, and my Android Studio has sync after the build type switching. With some investigation, I found what make the magic change to make BuildConfig.DEBUG is always 'true' is bellowing line within the app/build.gradle in the 'release' section of buildTypes config:
testCoverageEnabled true
removing this line and the BuildConfig.java seems correct and DEBUG = false for release build.
then I add the line back, BuildConfig.DEBUG =  Boolean.parseBoolean("true");
Would like to double check and apply the patch if it is fine.

Huh, makes sense, lemme check on my side then!
On Fri, 3 Jun 2016, 12:13 Yang Feng<EMAIL_ADDRESS>wrote:

You are right, BuildConfig is regenerated from scratch, and my Android
Studio has sync after the build type switching. With some investigation, I
found what make the magic change to make BuildConfig.DEBUG is always 'true'
is bellowing line within the app/build.gradle in the 'release' section of
buildTypes config:
testCoverageEnabled true
removing this line and the BuildConfig.java seems correct and DEBUG =
false for release build.
then I add the line back, BuildConfig.DEBUG = Boolean.parseBoolean("true");
Would like to double check and apply the patch if it is fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/issues/180#issuecomment-223530203,
or mute the thread
https://github.com/notifications/unsubscribe/AA7B3Oh0M2xxNwR1QhstJ8GrzDTmEwUFks5qH_BFgaJpZM4IrQDT
.


added my squirrel

Ok || Please tell others to contribute here as well || Give a star to the repo if u like || 😊


Add Address Sanitizer to GitHub Actions

          I would like to suggest adding profile with asan build

                "CMAKE_C_COMPILER": "clang-17",
                "CMAKE_CXX_COMPILER": "clang++-17",
                "CMAKE_CXX_FLAGS" : "-ggdb -fvisibility=default -fno-omit-frame-pointer -fsanitize=address -fsanitize-address-use-after-scope -fsanitize=undefined",
                "CMAKE_EXE_LINKER_FLAGS" : "-fsanitize=address -fsanitize=undefined"

And running it always before creating release.
Originally posted by @arturbac in https://github.com/stephenberry/glaze/issues/773#issuecomment-1955290393

@arturbac, do you know how to get the address sanitizer running on a macOS GitHub action? I tried to add address sanitizer flags to the CMake in this merge request (#777), but it isn't linking to the address sanitizer. I'm not sure how to link against the address sanitizer library in Clang in my CMake.

I am working on linux, so I just added to Your profile( temporary didn't have time to define CMakeUserProfile from scratch) and used that profile
        {
            "name": "dev-release-arturbac",
            "displayName": "Developer Mode",
            "description": "Builds Glaze w/ tests and downloaded test dependencies",
            "generator": "Ninja",
            "cacheVariables": {
                "CMAKE_BUILD_TYPE": "Release",
                "glaze_DEVELOPER_MODE": true,
                "BUILD_TESTING": true,
                "CMAKE_EXPORT_COMPILE_COMMANDS": true,
                "CMAKE_C_COMPILER": "clang-17",
                "CMAKE_CXX_COMPILER": "clang++-17",
                "CMAKE_CXX_FLAGS" : "-ggdb -fvisibility=default -fno-omit-frame-pointer -fsanitize=address -fsanitize-address-use-after-scope -fsanitize=undefined",
                "CMAKE_EXE_LINKER_FLAGS" : "-fsanitize=address -fsanitize=undefined"
            }
        },

You can test directly adding -lasan and -lubsan to linker flags

And one thing to note for using ubsan at github action You probably want it to stop exection and fail, so set ENV
UBSAN_OPTIONS=print_stacktrace=1:halt_on_error=1

Thanks for these tips.

And if asan fail that was not first initialized
ASAN_OPTIONS=check_initialization_order=0
IMO just ignore unless You have plenty of globals that can fail before asan library constructor init or use LD_PRELOAD or add -lasan -lubsan to LINK_FLAGS or something
But they are writting that
Note that this option is not supported on macOS.

I think I got it added in #779


Update previous cells instead of re-initializing all cells

Currently, UpdateCellList! clears all cells and adds particles again. For use cases like SPH simulations where only small changes in position happen between updates, it will be much faster to only update cells that changed, using the existing cell lists from the previous step.

That is not trivial, because the construction of the lists must run over particles. One has to check the cell for each particle, and then adding it to a list is not more expensive than doing any kind of cell update. (I do not clear the particle lists memory - wise).
In MD what we do is to not update the lists at every simulation step. We compute the lists up to a cutoff greater than the actual simulation cutoff and update the lists only at every few steps. Possibly by following the displacement of the particles in an auxiliary array.
(Most times packages store the explicit neighbor lists between steps, but this may be prohibited for very large systems).
Maybe something or the sort can be done in SPH simulations as well.

By the way, from what I remember of the presentation of Trixi.jl, the scalability you are getting with it is of another order of what CellListMap can deliver. So I'm unsure if you will be able to use it for the kind of simulations your group is used. Nevertheless, I would be very grateful if you provide any kind of feedback on what could be missing, or what could be improved about the usability here, which may guide me towards future improvements. Also, if there is any kind of system for which CellListMap is useful to you, I would love to know. Thanks for the feedback and contributions.

Thanks! I am currently working on an SPH multiphysics code, which is separate from Trixi.jl, but we're trying to get a similar multithreading scalability as with Trixi.jl. So far, I've been using my own cell list implementation for the neighborhood search, but now I implemented one based on CellListMap.jl, and I am looking to compare the two.

Do you need periodic boundaries? Most of the complications here are to support triclinic PBCs.

We want to have periodic boundaries soon, that's also a reason why I implemented the NHS based on CellListMap.jl. Our implementation doesn't support that.

Maybe helpful:

The issue with the scaling here is on the construction of the cell lists. The mapping scales well for systems large enough.
However, with many cores, the cell list construction end up being the limiting step sooner or later.
In my tests, simply reducing lists that are computed in parallel is enough to break the scaling. Probably to improve on that we need to compute redundant lists that are not reduced.

I found similar results in my implementation. Reducing was actually worse than the serial implementation for me. I tried one list per thread, which made the update scale perfectly, but ended up ruining the map speed because I had to iterate over all lists.
I ended up looping (multithreaded) over all cells, marking all cells containing particles that are supposed to be in another cell. Then, in a serial loop, I update the marked cells. This doesn't scale for full updates (all particles changed cells), but it scales well when only a few particles changed cells.

Ah, I see, running over all cells to check which cells have to be updated can be multi-threaded, and then that can be useful. I think something like that can be implemented here as well. I'll take a shot at it when I have some time.

I recently implemented a cell list neighborhood search that works with GPUs. For that, I had to write a fully parallel update with atomic operations. This might be interesting to you.
https://github.com/trixi-framework/PointNeighbors.jl/pull/42
I now get much better scaling of the incremental update step. I haven't implemented the initialization yet, but I'm sure it will scale similarly.

"semi parallel" refers to my previous strategy that I explained earlier.

I ended up looping (multithreaded) over all cells, marking all cells containing particles that are supposed to be in another cell. Then, in a serial loop, I update the marked cells.

Do have already some instructions on how to use it? I'm certainly interested.
(does it support already PBCs?)

We extracted the neighborhood search from TrixiParticles.jl to a new package.
The main functions are initialize!, update! (self-explanatory) and foreach_point_neighbor, which is similar to your map_pairwise!.
Check out the docs for these functions here: https://trixi-framework.github.io/PointNeighbors.jl/dev/reference/
We also have a PR for a CellListMap.jl backend, which just wraps a CellListMap in our API, so that we can use CellListMap.jl as a backend for TrixiParticles.jl and to easily run benchmarks against your implementation.
https://github.com/trixi-framework/PointNeighbors.jl/pull/8