Add Java to "Tools and Languages" section of the welcome page

Can we also add Java among other entries to improve awareness?
We have relatively complete Java support in VS Code with an extension portfolio that covers lots of things:

Java Language Support
Java Debugger
Java Test Runner - JUnit
Maven for Java
Spring Boot Extension Pack
Tomcat for Java

Java also deserves its place in that list.

In VS Code, the default recommendation for java files is the Java Extension Pack. The pack contains extensions to support everyday Java dev works.
When Java appears in welcome page, it can lead users to install the Java Extension Pack just like what the Python link does.


Inmvolbing issue 11: Do not count escape sequences for column width

The escape sequences of styles are removed from string, but others are not.
So, issue #11 remains.

No all escape sequences are removed from the strings.
Issue#!1 will be closed.


HTML escaping fixes

For consistency and correctness; probably not exploitable.

This is great, merged, thanks @andersk!  I'm a big fan of replacing these bits of messy manual HTML generation in JS code with templates.


Problems_ARGProfile

Hello.
My name is Cristopher Yerena. I am trying to use the ARGprofile pipeline. At the moment of "run" , it appears the following message:

Could you help me?
I installed fastq-dl from Anaconda.org and change the number id of paired -end.
Thanks

Hello.
Thanks for the answer. I have installed the kma tool in my system, it "run" in a different way compared to the past message.
Yes, I have installed the snakemake workflow management, but I have the next problem.

Could you help me?
Thanks

Hello! Sorry for the late response.
Can you walk me though the installation process? I assume you are cloning the repository and then installing the environment with the following command "mamba env create --name argprofiler --file rules/environment_argprofiler.yaml"?
Niki

Good afternoon, Nikki,
Thanks for the answer and sorry for the delay.
I am trying to install and use the pipeline in my EC2 instance (Ubuntu 20)
According the instructions of the Github:

Clone git clone https://github.com/genomicepidemiology/ARGprofiler.git

Install Snakemake
mamba create -c conda-forge -c bioconda -n snakemake snakemake

mamba activate snakemake

snakemake --help
    4. HPC
    snakemake --profile profile_argprofiler

After running....(image attached)
Could you help me?
Pd. I have another Virtual machine but the I can t install snakemake for cache unwriting stuff...
Thanks
Enviado do Outlookhttp://aka.ms/weboutlook

De: blackadder901 @.>
Enviado: quinta-feira, 4 de abril de 2024 09:49
Para: genomicepidemiology/ARGprofiler @.>
Cc: cristoyerenahs @.>; Author @.>
Assunto: Re: [genomicepidemiology/ARGprofiler] Problems_ARGProfile (Issue #3)
Hello! Sorry for the late response.
Can you walk me though the installation process? I assume you are cloning the repository and then installing the environment with the following command "mamba env create --name argprofiler --file rules/environment_argprofiler.yaml"?
Niki
—
Reply to this email directly, view it on GitHubhttps://github.com/genomicepidemiology/ARGprofiler/issues/3#issuecomment-2037120738, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOQAS6IZRPXBFZ5YRMUZO73Y3VD47AVCNFSM6AAAAABERL5A4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZXGEZDANZTHA.
You are receiving this because you authored the thread.Message ID: @.***>

Good afternoon, Nikki,
Thanks for the answer. I also tried to install the pipeline under your instructions:

Clone git clone https://github.com/genomicepidemiology/ARGprofiler.git

mamba env create --name argprofiler --file rules/environment_argprofiler.yaml

The env began to run....but I attached the results.

Enviado do Outlookhttp://aka.ms/weboutlook

De: blackadder901 @.>
Enviado: quinta-feira, 4 de abril de 2024 09:49
Para: genomicepidemiology/ARGprofiler @.>
Cc: cristoyerenahs @.>; Author @.>
Assunto: Re: [genomicepidemiology/ARGprofiler] Problems_ARGProfile (Issue #3)
Hello! Sorry for the late response.
Can you walk me though the installation process? I assume you are cloning the repository and then installing the environment with the following command "mamba env create --name argprofiler --file rules/environment_argprofiler.yaml"?
Niki
—
Reply to this email directly, view it on GitHubhttps://github.com/genomicepidemiology/ARGprofiler/issues/3#issuecomment-2037120738, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOQAS6IZRPXBFZ5YRMUZO73Y3VD47AVCNFSM6AAAAABERL5A4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZXGEZDANZTHA.
You are receiving this because you authored the thread.

Hello there,
If I understand correctly you are trying to run the pipeline on a clean cloud instance.
You don't need to install Snakemake! What you need to do is to install either mamba or micromamba in order to create and activate the environment we provide (The environment provides the software to run the pipeline).
Use this guide to install micromamba (https://mamba.readthedocs.io/en/latest/installation/micromamba-installation.html).
If the installation is successful then you can do the following:
1.mamba env create --name argprofiler --file rules/environment_argprofiler.yaml
2.mamba activate argprofiler
3.snakemake --profile profile_argprofiler
From then on you should be able to execute the workflow with no issues!
I hope this helpes.
Best regards,
Niki


Support Diesel v2.1.x

This PR adds support for the new diesel v2.1.x version. I have not tested the new MultiConnection with this PR. But it at least fixes a simple upgrade from 2.0.x to 2.1.x.

Thanks!


Making cloudaux _version key an ephemeral field.

Coverage remained the same at 63.645% when pulling 4b960f674fc19932a74052cb0d2f8a00fb7872ca on make_cloudaux_version_ephemeral into c3a6c33cff60b405f04b971a04ca62be73a413b4 on develop.

Coverage remained the same at 63.645% when pulling 4b960f674fc19932a74052cb0d2f8a00fb7872ca on make_cloudaux_version_ephemeral into c3a6c33cff60b405f04b971a04ca62be73a413b4 on develop.


[BUG] AntiXray doesn't work first time config file is created

Anti-X-ray isn't enabled the first time you launch the server with this mod and the ./config/antixray.toml file is created

I am unable to reproduce this issue and looking through the code it shouldn't happen, because of this null check: WorldConfig.java#L23

Weird. I tried again and I can reproduce the bug. When the loadConfig() method is called, Config.java#L13 only gets a value assigned if ./config/antixray.toml exists, else only creates ./config/antixray.toml and assigns nothing to Config.java#L13 (so it's null). This doesn't happen for me with #2.

Ok, I did some more testing and was able to reproduce it.


ARROW-12856: [C++][Gandiva] Implement castBIT and castBOOLEAN functions

Implement castBIT and castBOOLEAN functions for UTF8 input values

@projjal can you please check the build failures.

@projjal can you please check the build failures.

Build failures look unrelated


Event not triggering: session:request:initiated

I am working on an integration and session:request:initiated  is not triggering when a new chat is started. I saw it was brought up in issue: https://github.com/crisp-im/node-crisp-api/issues/17 but no resolution. Other events such as message:send are working for me.
Steps done to troubleshoot:

Incognito window to start a new chat
Cleared out all crisp cookies and started a new chat
Looked at source and session:request:initiated  is included in DEFAULT_RTM_EVENTS

Code example:
const Crisp = require('node-crisp-api')

const TOKEN_KEY = <'TOKEN'>
const TOKEN_IDENTIFIER = <'TOKEN'>

const CrispClient = new Crisp()

console.info('Authenticating...')

CrispClient.authenticateTier("plugin", TOKEN_IDENTIFIER, TOKEN_KEY);

console.info('Listening for events...')

CrispClient.on('session:request:initiated', (message) => {
  console.log("session started", message);
})

Session:request:initiated is not a chat started event. This event is only used when you manually request to chat with a user from the MagicMap.

On 12 Jan 2022, at 21:49, David Groechel @.***> wrote:
I am working on an integration and session:request:initiated is not triggering when a new chat is started. I saw it was brought up in issue: #17 https://github.com/crisp-im/node-crisp-api/issues/17 but no resolution. Other events such as message:send are working for me.
Steps done to troubleshoot:
Incognito window to start a new chat
Cleared out all crisp cookies and started a new chat
Looked at source and session:request:initiated is included in DEFAULT_RTM_EVENTS
Code example:
const Crisp = require('node-crisp-api')
const TOKEN_KEY = <'TOKEN'>
const TOKEN_IDENTIFIER = <'TOKEN'>
const CrispClient = new Crisp()
console.info('Authenticating...')
CrispClient.authenticateTier("plugin", TOKEN_IDENTIFIER, TOKEN_KEY);
console.info('Listening for events...')
CrispClient.on('session:request:initiated', (message) => {
console.log("session started", message);
})
—
Reply to this email directly, view it on GitHub https://github.com/crisp-im/node-crisp-api/issues/39, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGTRQZYLOFBTUAZGPW4GLLUVXSM5ANCNFSM5LZ6WHJA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.

No, there’s no such event. You’d need to catch a message:send event from the user and distinguish which one is the first for the session.


Rebase with move-language:main

Test Plan
local testing

LGTM-- but we still can't seem to run CI due to problem on the github side. It would be good to hold off until that works again.
Also, it might be best to land this rebase before the other outstanding patches (like #208). It will be easier to adjust those afterwards than vice versa.


Difference between Tray Icons "CPU Usage" and "CPU History"

Regarding Tray Icons, what is the difference between "CPU Usage" and "CPU History"?

Ever tried to compare?
All  * hostory entries ahow values dor laat aeveral swconds as graph ,  and * usage show only cirrent value as graph with flat form

"CPU Usage" fills the entire icon with a single value of the current utilization (progress style).
"CPU History" shows the last ~ 15 seconds of utilization using multiple values (graph style).


Tests Failing

I was messing around with the tests some, it seems to be an upstream issue with pytest_runner that causes tests on python 3.3 to fail. The error:
travis@93b9068ce8dd:~/wiki-master$ python setup.py test
['tests', 'wiki', 'wiki.web']
running pytest
Traceback (most recent call last):
  File "setup.py", line 29, in <module>
    'wiki=wiki.cli:main'
  File "/home/travis/opt/python/3.3.6/lib/python3.3/distutils/core.py", line 147, in setup
    dist.run_commands()
  File "/home/travis/opt/python/3.3.6/lib/python3.3/distutils/dist.py", line 930, in run_commands
    self.run_command(cmd)
  File "/home/travis/opt/python/3.3.6/lib/python3.3/distutils/dist.py", line 949, in run_command
    cmd_obj.run()
  File "/home/travis/wiki-master/.eggs/pytest_runner-4.0-py3.3.egg/ptr.py", line 157, in run
  File "/home/travis/wiki-master/.eggs/pytest_runner-4.0-py3.3.egg/ptr.py", line 118, in install_dists
AttributeError: type object 'test' has no attribute 'install_dists'

I created a new branch which removed 3.3 testing, is there any reason we should keep testing 3.3? If so maybe an upstream ticket should be opened to resolve this? Interestingly python 3.3 testing behaves differently if ran directly or using a virtualenv (all tests pass using a virtualenv) I'll clean up the dockerfile I was using for testing, create a gist and post the link here so others can easily have a look.
Second issue is that tests are still failing on python 3.4 on windows. I haven't had time to dig into that as much as I don't have windows test machines setup, maybe I'll setup some virtual machines later. But in the appveyor logs it looks like there's a problem with jinja2 and then it also seems to fail with
File "c:\projects\wiki\.eggs\pytest_runner-4.0-py3.4.egg\ptr.py", line 157, in run
  File "c:\projects\wiki\.eggs\pytest_runner-4.0-py3.4.egg\ptr.py", line 118, in install_dists
AttributeError: type object 'test' has no attribute 'install_dists'

which is the same error python 3.3 was throwing on linux.

Opened pull request #77 to fix this, turns out that it was setuptools causing these issues. Just need to update it before running the tests.

I think this can be closed now, the tests seem to be working again.


Add AmazonS3FileManagerDriver

Changes:

Add AmazonS3FileManagerDriver
Edit LocalFileManagerDriver to have consistent error artifact output
Edit BaseFileManagerDriver with more exception to error artifact translations for common scenarios
Improve test coverage of LocalFileManagerDriver

Also manually tested using the driver against a real s3 bucket, in a similar style to those added in the unit tests.

The unit tests are currently failing due to an unrelated unit test -TestAwsIotCoreEventListenerDriver.test_try_publish_event. That particular unit test works locally, but it seems to be the only test that is using moto3's decorators and I'm wondering if they don't work in the github environment.
I would have expected this to have been caught in the original PR, not sure why I'm just seeing this behavior now.


detect bypass

Domain
https://exee.app/
Link
https://exee.app/rLYao7tH
Version
0.0.0
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
dont skip link back main link

Duplicate of #769.


⚠ Bofrost.com has degraded performance

In aaeda19, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 964 ms

Resolved: Bofrost.com performance has improved in f001240.


RT_ASSERT Error happened when inserting

Hi Jiacheng,
Thanks for sharing your excellent work. We want to evaluate several kinds of Learned Index. When we inserted 100,000,000 kv pairs into LIPP, RT_ASSERT Error happened. But when 80,000,000 kv pairs were inserted, it worked fine. So is there some limits of total amount of kv pairs?
int main(){
    LIPP<int , int> lipp{};
    for(int i=0;i<100000000;i++){
        lipp.insert(std::pair<int , int>(i,i));
    }
}

Output
RT_ASSERT Error at ./index/lipp/lipp.h:805,path_size < MAX_DEPTH

Hi Dear,
I guess you could just modify the value of MAX_DEPTH (in line 799) with a larger value (>128).
We just it to static create a node array to avoid the cost of dynamic allocations.
It seems with a larger number of kv pairs, the size of the path is larger than MAX_DEPTH we expected.
Sincerely,
Jiacheng

Hi Jiacheng,
It works fine. Thanks again for sharing your excellent work.


Create a recycle bin special location

Add option to recycle files instead of permanent delete
Move recycled items to a hidden location on the device
Contents can be viewed like in drive view
Folders cannot be browsed
Recycled items can be permanently deleted (add button to clear recycle bin)
Read contents of the hidden folder on connection to device
No need to confirm recycle

Done in 45b484c9b720e43d57d899e9d5aa784118725207


Spatial Reference support

Different spatial reference systems can be defined in different ArcGIS servers. The common one used by popular web-based maps engines such as Google, Bing, Apple maps is WGS84 with EPSG (European Petroleum Survey Group) code of 4326.

Spatial Reference is int type:
spatialReference = 4326

Use it when create 'AGSPoint' and 'AGSGeometry':
AGSPoint *point = [AGSPoint pointWithX: y: spatialReference:self.mapInfo.spatialReference];
AGSGeometry *geometry = [AGSGeometryEngine projectGeometry:point toSpatialReference:_agsMapView.spatialReference];

Add property spatialReference. Sample:
<ArcGISMapView
    spatialReference = {{ wkid: 4326 }}
    ref={element => agsView = element}
/>


MobArena integration not working

As per the MobArena integration page I added "example: arena" to my plugins/Magic/config.yml then I added the suggested mage class but I received 10.06 22:59:08 [Server] INFO Failed to parse armor for class mage: magic:wizardhelmet as an error. That also happened for each of the items when I added them to an already working class. Let me know what you think.

+1 to this. After updating my server to 1.16 and updating my Magic and MobArena plugins, my Mage class no longer works despite making no changes to the configs.
Magic version: 7.9.10
Mob Arena version: 0.104.2

Do you still see this message in your logs?

[Magic] Integrated with MobArena, use "magic:" in arena configs for Magic items, magic mobs can be used in monster configurations

I'm not sure what could be going on there since it's still working for me :(

Sorry, any updates here? It's still been working on my dev server fine.


hpx::partitioned_vector serializer

PR for issue #1645

@hkaiser @sithhell libhpx_partitioned_vector isn't correctly linked by cmake into the serialized partitioned_vector unit test. The unit test can be built manually using hpxcxx without error. I've reviewed the partitioned_vector unit tests and haven't been able to sort out what needs to be added into cmake to fix the linking issue - do ya'll have any tips or suggestions?

@ct-clmsn In theory it is sufficient to declare the partitioned_vector_component as a dependency, but that has been done. I have no idea right away, but I'll have a look.

@hkaiser thanks!

@ct-clmsn There was a problem in the segmented parallel::fill implementation which was causing your liker errors. I added a commit to your branch here: https://github.com/STEllAR-GROUP/hpx/commit/e1cc39c01227284cc4632c34bcfd1a7d3dad4e18. If this goers through now we can merge things from that branch. Thanks for you patience!

@ct-clmsn I have updated the commit mentioned above (see https://github.com/STEllAR-GROUP/hpx/commit/a5b25d0f5c9ed0b72a9f1235c96bc66991b1d282). I hope you don't mind me slightly simplifying your implementation... Things should be ready for merging now.

@hkaiser apologies for unnecessary complexity! i had concerns the most straight forward implementation might lack nuance.

@ct-clmsn No apologies needed. Sometimes the simplest solution turns out to be the best in the end... The main issue was that bug in parallel::fill we had - this probably has thrown you off. Thanks again for your contributions!


I'm getting INVALID_APP_ID when logging in

Hi, I realise this might not be entirely within the scope of this project and might be more of a nativescript/android/spotify thing, but after googling around a LOT, while you don't have the Android guide, I thought I'd give it a go asking if you could shed some light on this.
I have so far:

created an application on the spotify dashboard
ran "keytool -list -v -keystore ~/.android/debug.keystore -alias androiddebugkey -storepass android -keypass android" to generate an SHA1 fingerprint, which I've then copied over to the dashboard
I've copied the Client ID to my natiscript app and that seems to be recognised

But when I try to log in, I get err: INVALID_APP_ID
I'm guessing I'm missing something like adding the SHA1 fingerprint to my nativescript app, but I can't find any info on this for some reason.. any help?
Thank you in advance!

FIXED IT:
I was getting the App Id from the android manifest file, while the one used was in the settings.json file ha


Remove reliance on React's deprecated _rootNodeID

//index.module.js
...
686: var rootNodeID = this._reactInternalInstance && this._reactInternalInstance._rootNodeID;
...
732: var initialRender = function initialRender() {
            reaction = new Reaction(initialName + "#" + rootNodeID + ".render()", function () {
...

As you can see, mobx-react attempt to capture React's _rootNodeID property in order to provide a friendly name for a Reaction. I believe React has deprecated this property starting in v0.15 and removed it completely in v16. I noticed this issue when setting up mobx-react-devtools, as it prints this value to the console.

Addressed in 4.4.0


Create Account delete Option with endpoint

Creating the UI and connecting the Delete Account endpoint at profile settings!
The ENdpoint is created!
for any endpoint query? ask me!

@MidouWebDev are you working on this? If not I'll work on this

@mani-barathi i wanted to, by adding a danger zone section in the settings page where we will have the delete account option but if you wanna start now no problem go ahead cuz i planned to start working on it in June 1st

@MidouWebDev For the latest changes, should I pull from master or develop?

@mani-barathi develop

@mani-barathi actually, we are just going to revert back to master based development.  Getting everyone to make changes to the develop branch is too much extra work.  Make your PR to the master branch. TY!

@PraveenMalethia, @codyseibert
I'm trying to send a POST request to /api/users/delete-profile , but the response fails stating the method is not allowed. I've also checked the backend, the view states it requires a POST request.
I'm confused about whether something is wrong in the backend or in the request.

@mani-barathi it might not be deployed yet.  You could try running the api locally?

@codyseibert ok I'll try running it locally!

@mani-barathi did you finish the frontend part ?
DM me on Discord plz

@MidouWebDev yeah!

#439


[DDW-490] Use a small icon for the window and a large icon for the desktop

This PR fixes window icon issues on Linux which were a result of using high quality icon which was poorly downscaled in order to be used as a small window title bar icon.

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (yarn run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[ ] There are no flow errors or warnings (yarn run flow:test)
[ ] There are no lint errors or warnings (yarn run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running yarn run manage:translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn run storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

@amias-iohk what about the desktop icon? Is that one looking well too?

they seem a little smaller , mainnet is old style , staging is new style


[BUG]: [QA | Upload ID Proof field is not accepting the required format and the text should be changed]

Describe the bug

Upload ID Proof field is not accepting jpeg format
Upload ID Proof field is not accepting the .jpg/ .jpeg and png format.

To Reproduce
Steps to reproduce the behavior of point 1:

Go to new user -> register a new user with mobile no.
Click on upload your id
See error

Expected behavior

Upload ID Proof field should accept the jpeg format and the text should be changed
Upload ID Proof field should accept the .jpg/ .jpeg and png format and the text should be changed

Screenshots

https://jam.dev/c/a1a823f2-df6e-46bb-901b-f6a3a3a21c52

Hi @rajeshcherukumalli this issue has been fixed in dev env please test.
cc:- @Ramu-kandimalla

Hi @iknoorkaur
It seems the fix has been done for the upload in user registration not e-filing as per the details mentioned in steps to reproduce and jam that was attached . Will make the changes as per new video that is just attached .
cc:- @Ramu-kandimalla  @rajeshcherukumalli

Hi @shambhavigoel1002 , i was refering to the 2nd point for which the screenshot was attached.
cc : @Ramu-kandimalla @rajeshcherukumalli

Hi @iknoorkaur  made the changes in dev env please test
cc :- @rajeshcherukumalli @Ramu-kandimalla

Hi @shambhavigoel1002 Tested in dpg dev, but the changes are not reflecting. can you please recheck once.
Attaching the video for reference:
upload.webm
cc : @rajeshcherukumalli @Ramu-kandimalla

Tested in dpg-dev, working fine as expected:
https://jam.dev/c/ebcdf3a9-4e93-4dca-aa3e-089643b25bc6
@rajeshcherukumalli @Ramu-kandimalla


Survey list doesn't update on remote changes (not reactive)

@shobhitagarwal1612 FYI
Proposed logic: stream of List<Survey> would be derived from (pseudocode):
onlineStatus.flatMapLatest { if (online) remoteSurveysFlow else localSurveysFlow }

I was able to create a working prototype which updates the list on remote changes. However, I'm facing an issue when trying to stop listening to remote changes.

I was able to create a working prototype which updates the list on remote changes. However, I'm facing an issue when trying to stop listening to remote changes.

Can you share as a draft PR?

https://github.com/google/ground-android/pull/2027

This is a bit different than what I had in mind I'll send a PR to clarify.


KAFKA-9625: Broker throttles are incorrectly marked as sensitive configurations

Not needed since #8260 was merged


[Request] Build target 3.1?

The main reason is that I don't want to install 2.1 SDK.
If I don't, dotnet migrate-2019 analyze will show
It was not possible to find any compatible framework version
The framework 'Microsoft.NETCore.App', version '2.1.0' was not found.

I'm facing the same issue:
It was not possible to find any compatible framework version
The framework 'Microsoft.NETCore.App', version '2.1.0' was not found.
  - The following frameworks were found:
      3.1.8 at [/usr/share/dotnet/shared/Microsoft.NETCore.App]

You can resolve the problem by installing the specified framework and/or SDK.

Meanwhile setting environment variable DOTNET_ROLL_FORWARD_ON_NO_CANDIDATE_FX=2
Will make it work even on .NET 5


C++/C#: Fix invalid overlap

Fixes https://github.com/github/codeql-c-analysis-team/issues/34
Instruction.getDefinitionOverlap() depends on SSAConstruction::getMemoryOperandDefinition(), which in turn depends on SSAConstruction::hasMemoryOperandDefinition(). When the definition in question came from a Chi instruction, hasMemoryOperandDefinition() incorrectly bound overlap to the overlap relationship between the original (non-Chi) instruction and the use. The fix is to make use of the actualDefLocation parameter to getDefinitionOrChiInstruction(), which specifies the location for the result of the Chi in that case.
This PR is easier to review commit-by-commit, so that you can see the new sanity test introduced, along with the original failures it revealed.

The only remaining failures are the three SAMATE test failures from the internal repo. I've opened a separate PR there to fix those failures.


Make it easier to inherit exposures and assigns

Maybe using methods on the type https://crystal-lang.org/api/0.22.0/Crystal/Macros/TypeNode.html#ancestors%3ACrystal%3A%3AMacros%3A%3AArrayLiteral(TypeNode)-instance-method

This is really helpful https://gitlab.com/Zatherz/magicjson/blob/master/src/magicjson/infect.cr#L3-29 See https://github.com/luckyframework/record/issues/42


French translations: fixes and improvements

For now, only half translations are fixed and no missing translations are submitted. I'll try to do it asap ;)

BTW, I noticed several recurrent mistakes:

The translation of "Settings is "Paramètre" (with an accent)
The translation of "to keep" is "garder" (and not "guarder"), but in the context of personal information, it's better to use the word "conserver"

Hope this can help ;)


T1.5 History page design

Page design: http://iati.surge.sh/page-templates/01-5-history
Model would be something like:

title
intro_text - a text area
content_editor
history list - where you can add multiple items, each item would have:

history_item_date (text field) - best keep as a text field rather than datetime for flexibility
history_item_content (text and links)

From Trello card: https://trello.com/c/b39taEKy/89-t15-history-page-design

Now implemented. Closing issue.


⚠ NCAA has degraded performance

In 0a701a5, NCAA (https://www.ncaa.com) experienced degraded performance:

HTTP code: 200
Response time: 956 ms

Resolved: NCAA performance has improved in 32fe623.


feat(biome_css_analyze): implement noDescendingSpecificity

Summary

Implement noDescendingSpecificity and the specificity calculation
close #2810
Test Plan

Added tests and snapshots

Thank you for your review.
I have made the corrections you pointed out. Could you please check it again?
Also, I noticed that the performance test in the CI is failing with my PR. But I'm not sure where to make the necessary changes as I lack knowledge. Could you please help me with this?

Yeah, the regression seems acceptable to me too!


docs: fix access to env var using VITE_ prefix in README.md

Priority*

[x] High: This PR needs to be merged first, before other tasks.
[ ] Medium: This PR should be merged quickly to prevent conflicts due to common changes. (default)
[ ] Low: This PR does not affect other tasks, so it can be merged later.

Purpose of the PR*
In my previous large PR I forgot to add the VITE_ prefix to accessing the env var via import.meta.env.
Reference
https://vite.dev/guide/env-and-mode#env-files

I haven't tested this super thoroughly, but it seems that adding an env var to .env requires restarting pnpm dev for the variable to be accessible via import.meta.env.VITE.... If that is indeed the case, it should be mentioned in that section.


migrate some material files to nullsafety

Description
NNBD migration for part of material
Related Issues

dart-lang/sdk#40146

Tests
I added the following tests:
None
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Can you rebase this perform submitting it? I recently added a field to MaterialApp (restorationScopeId), which probably needs to be marked as nullable as part of this PR to avoid breakage, but it's not included in this PR, yet.
Sorry for the inconvenience...

Rebased.


error: language item required, but not found: eh_personality

cargo test
cargo test -- response
cargo test -- response --target thumbv7em-none-eabi
Compiling atat v0.11.1 (D:\GPS-rs\atat\atat)
error: language item required, but not found: eh_personality
error: language item required, but not found: eh_personality
error: aborting due to previous error
error: could not compile atat
To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: aborting due to previous error
error: build failed

Try with cargo test --lib

Forgive my ignorance

No worries! Ask away :)


create REPLACE_WRONG_WORD question type

Sentence appears with EN translation and a VN sentence with wrong word-- select wrong word in VN sentence and choose from multiple choice for a correct answer.

This one and FILL_IN_BLANK(#22) seem like they could share a phrase type that looks something like this:

This question type seems like it could be used pretty effectively for tricky tone-matching/spelling questions in addition to the vanilla translate stuff.


add rudimentary support for rendering components from controllers

This change introduces the ability to render components from
controllers, mimicking the behavior introduced in
https://github.com/rails/rails/pull/37919.
[Fixes: #38]

@jonspalmer right now, this just renders the component without a layout, matching the behavior of rendering partials from controllers.
Is that the behavior you'd expect? Or would you expect layouts to work?

I think without layouts is what I'd expect. Otherwise you'd get double rendered layouts or the like. I'd need to go review the examples in more detail.


Update reconstruction module and notebook

The code is out of date with current bindings:

https://github.com/opendifferentialprivacy/smartnoise-samples/tree/master/attacks/reconstruction

(This may also get wrapped into the auto-generation of notebooks)

Closing, dupe with #69


docs: Add docs for RunnableConfigurableFields

[x] docs: core: Add docs for RunnableConfigurableFields

Description: Added incode docs for RunnableConfigurableFields with example
Issue: #18803
Dependencies: NA
Twitter handle: NA

Looks great, thanks!


Fix Karma Safari test in mac OS

Follow up of #1426, This fix changes the karma plugin to make the tests in safari. The issue with the former plugin was because of the security policies safari asks to open the file redirect.html, if you don't make this action the build fails.

Using the karma-safari-applescript-launcher allows to comunicate with safari directly. The first time it will only ask in OSX if allows the terminal to run it. later builds it will not ask again.
Your pull request should…

[x] have a concise title

[x] be against the correct branch (features can only go into develop)

[x] pass automated testing

[x] have a clean commit history

[x] have proper commit messages (title and body) for all commits

[ ] have appropriate tags applied

This is working for me, but I mean, I don't have a Mac, so... I bugged some of my colleagues, so I'll wait what they have to say.

Works for me on my Debian box.  Still worried about the buildbot stuff, but it works here so I don't think this is a blocker.

There's also not a ton we can do about it unfortunately.


[WIP] Reviewer leaderboard

This adds the Reviewer Leaderboard, a table of reviewers with agregated stats of their reviews the ability to filter on Reviewer, Fund, & Round.
Fixes #1250

This doesn't currently have a link anywhere on the site, where should we link to it from?

@frjo is this live anywhere that we can look at? We're not able to get the branch running locally since it's so far behind master at this point. Happy to implement and advance the changes with @harumhelmy , but curious if there's somewhere we can look at it first before updating the code!

@rudietuesdays I have rebased this branch from master now so it's up to date.


MacroManager::LoadMacros file verify and fix

Inside of the function MacroManager::LoadMacros we need it to check whether or not the current Macro we're reading isn't missing any keys or values.
If the file is incorrect- correctly assign values to the macro and rewrite the file. Make sure to only mess with values that are unset or not existing.

This issue is no longer needed as we will be using a different library to handle json.


Todo List done

Like the PR to the upstream branch :). Thanks for that put a smile on my face. Will review the code shortly.

Temporarily closed with with #10


Reorganize

I propose these changes to the directory structure and file nameing:

Rename README to README.md: Use the markdown extension to allow github to render the readme file and to be consistent with contributing.md
Rename tests/ directory to examples/, because there are mostly demonstrational/example projects inside your current test directory. At least for me this caused confusion, because you do not provide documentation and the first thing I was looking for was an example.
Move the unittests directory now located in examples to "tests/".

Hi,
It would be nice indeed to split examples from tests. Some remarks on your changes:

first as I just merged to master some pending change requests, you''ll need to resolve some conflicts.
I would prefer to keep misc, TLV and unittests under tests and move only bootstrap_server, client, lightclient, secureclient, server and utils to the example folder.
be careful that utils/tinydtls is a git submodule. I never tried to mode one and there may be some subtleties.

Regards

I'll prepare a new pull request


Fixed tag with Boolean/Null value trigger parse error

'instance,bool_tag=,value_tag=value cpucount=10i,free=1i,test="string",bool=false,value=1.11':

@gianarb  any problem with merging this PR?

Just the red test, can you have a look? I can but next week

@gianarb fixed the test

Thanks a lot


how do i compile this with clang?

compiling it using clang with msys just errors for me

how what are the errors?

how what are the errors?

when i compile with regular makefile:
g++ -O3 -std=c++17 -Wall -Wextra -DNDEBUG -march=native -Isrc -Ithird_party -MMD -MP -c src/book/opening_book.cpp -o tmp/src/book/opening_book.o -static -static-libgcc -static-libstdc++ -Wl,--no-as-needed
g++ -O3 -std=c++17 -Wall -Wextra -DNDEBUG -march=native -Isrc -Ithird_party -MMD -MP -c src/cli/cli.cpp -o tmp/src/cli/cli.o -static -static-libgcc -static-libstdc++ -Wl,--no-as-needed
make: g++: No such file or directory
make: *** [Makefile:125: tmp/src/book/opening_book.o] Error 127
make: *** Waiting for unfinished jobs....
make: g++: No such file or directory
make: *** [Makefile:125: tmp/src/cli/cli.o] Error 127

when i compile by changing cxx to clang:
In file included from src/matchmaking/tournament/base/tournament.cpp:1:
In file included from src/matchmaking/tournament/base/tournament.hpp:7:
In file included from src/engine/uci_engine.hpp:9:
In file included from src/matchmaking/tournament/roundrobin/roundrobin.cpp:1:
In file included from src/matchmaking/tournament/roundrobin/roundrobin.hpp:4:
In file included from src/matchmaking/match/match.hpp:6:
In file included from src/matchmaking/player.hpp:5:
In file included from src/engine/uci_engine.hpp:9:
In file included from src/matchmaking/tournament/tournament_manager.cpp:1:
In file included from src/matchmaking/tournament/tournament_manager.hpp:3:
In file included from src/matchmaking/tournament/roundrobin/roundrobin.hpp:4:
In file included from src/matchmaking/match/match.hpp:6:
In file included from src/matchmaking/player.hpp:5:
In file included from src/engine/uci_engine.hpp:9:
src/engine/process/process_win.hpp:132:19: src/engine/process/process_win.hpp:132:19: src/engine/process/process_win.hpp:132:19: warning: warning: warning: variable 'bytesAvail' set but not used
      [-Wunused-but-set-variable]variable 'bytesAvail' set but not used
      [-Wunused-but-set-variable]variable 'bytesAvail' set but not used
      [-Wunused-but-set-variable]
  132 |             DWORD bytesAvail = 0;
  132 |             DWORD bytesAvail = 0;
      |
      |
  132 |             DWORD bytesAvail = 0;                  ^
                  ^

      |                   ^
1 warning generated.
1 warning generated.
1 warning generated.
1 warning generated.
1 warning generated.
1 warning generated.
1 warning generated.
clang -O3 -std=c++17 -Wall -Wextra -DNDEBUG -march=native -Isrc -Ithird_party -MMD -MP -o fast-chess tmp/src/book/opening_book.o tmp/src/cli/cli.o tmp/src/elo/elo_pentanomial.o tmp/src/elo/elo_wdl.o tmp/src/engine/uci_engine.o tmp/src/globals/globals.o tmp/src/main.o tmp/src/matchmaking/match/match.o tmp/src/matchmaking/sprt/sprt.o tmp/src/matchmaking/tournament/base/tournament.o tmp/src/matchmaking/tournament/roundrobin/roundrobin.o tmp/src/matchmaking/tournament/tournament_manager.o tmp/src/pgn/pgn_builder.o tmp/src/pgn/pgn_reader.o tmp/src/time/timecontrol.o tmp/src/util/logger/logger.o -static -static-libgcc -static-libstdc++ -Wl,--no-as-needed
clang: warning: argument unused during compilation: '-static-libstdc++' [-Wunused-command-line-argument]
lld: error: unknown argument: --no-as-needed
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Makefile:122: fast-chess] Error 1

@Disservin

fixed with https://github.com/Disservin/fast-chess/commit/81ea475d6f1f1392e86ec6bcae87a05b051b6d5d

btw no need to ping i get notifications anyway


Added 24 parser tests and a few scanner ones

Scanner tests now allow identifiers to start with underscores.
Added a few complicated tests for the scanner.
Added 24 tests for the parser.
Added instructions for creating and using the tests for the parser.
Fixed scanner's README.

By the way, thanks for changing the folder names, the repo is now perfect! Turns out my root folder problem was because I didn't specifiy a target directory for git clone. It's all fixed now. :)

Damn this looks good.
Thank you for contributing 😊


Style not correct with bootstrap

In vendor/ember-notify.css:
.ember-notify {
  /* the following is purely so that it doesn't look broken without Foundation installed */
  box-sizing: border-box;
  background: #ccc;
  padding: 10px;
}

However, that background causes problems when messageStyle='bootstrap' is used. The gray wins instead of the green, pink, etc.

Okay thanks, what about if you change that to border: #ccc;?

Based on my first test, I think that should work.  I cleared my browser cache before testing, but having someone else verify as well might be a good idea. :)
Thanks!

This has been done in a recent release. Sorry for not letting you know earlier!


Release artifact jobs build entire workspace when only a single package was released

In this job for the samply-v0.12.0 release, I can see that it's building benchmarks and wholesym-addr2line, which are two members of my workspace which have publish = false in their Cargo.toml. These compilations added quite a bit of extra time to this job. Can they be skipped?

This should be an ideal case for https://opensource.axo.dev/cargo-dist/book/reference/config.html#precise-builds

Perfect, thank you. I've added it to my config and will give it a try on the next release.


platforms: add zc706 + coraz7_07s

Requires https://github.com/m-labs/migen/pull/206

@astro thanks, that looks fine!


login multi user

how to login multi user in your platform

You can't have multiple users in the MetaTrader 5 desktop client.

dear @Ichinga-Samuel you can get pathway in Account and if was empty you can ignore but if it fill you can use path for login another user in same system


Use is_shipping_required utils method where it ce be done

Update places where the non-optimized checkout model method was used instead of is_shipping_required utils that use CheckoutLineInfo.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-1973 Optimize is_shipping_required in checkout flow


WEBAPP-86: Use extracted enpoints module

This PR just changes imports and fixes flow errors. Apparently moving it out of the project made flow more sensible.


Blog app - integration specs for views and fix n+1 problems

Project requirements
n+1

Make sure that the N+1 problem is solved when fetching all posts and their comments for a user. This is the corresponding wireframe view: blog all user posts
Integration specs

Use Capybara to write integration tests for each view in your project.

User index page:
I can see the username of all other users.
I can see the profile picture for each user.
I can see the number of posts each user has written.
When I click on a user, I am redirected to that user's show page.

user show page:
I can see the user's profile picture.
I can see the user's username.
I can see the number of posts the user has written.
I can see the user's bio.
I can see the user's first 3 posts.
I can see a button that lets me view all of a user's posts.
When I click a user's post, it redirects me to that post's show page.
When I click to see all posts, it redirects me to the user's post's index page.

User post index page:
I can see the user's profile picture.
I can see the user's username.
I can see the number of posts the user has written.
I can see a post's title.
I can see some of the post's body.
I can see the first comments on a post.
I can see how many comments a post has.
I can see how many likes a post has.
I can see a section for pagination if there are more posts than fit on the view.
When I click on a post, it redirects me to that post's show page.

Post show page:
I can see the post's title.
I can see who wrote the post.
I can see how many comments it has.
I can see how many likes it has.
I can see the post body.
I can see the username of each commentor.
I can see the comment each commentor left.

RSPEC TESTING APROVE

BEFORE FIXING

AFTER FIXING 1rst ERROR

AFTER FIXING 2nd ERROR

@Nemwel-Boniface this project was already approved. A few hours ago

@Nemwel-Boniface this project was already approved. 41 min ago

@AmaliaGM I missed some of the missing specs pointed out by @Nemwel-Boniface so I made a mistake 🙏 kindly fix the requested changes as you can see in the project requirements they are all required for this milestone

@Davidosky007 I will fix them. But already submitted the project.

All recommendations were addressed and applied. Thank you very much for his advice.


Prebaking Container Image with Runner

hi @imjohnbo
Got a chance to play around with this today - a couple of questions:
i) Is there a reason why we are not prebaking the runner into the Dockerfile? Runner will also auto update, so even if we bundle an older version, they can get auto updated. Current startup time for this image to become active per pod is 50-60s. I have a prebaked runner which is less than 5s.
ii) Do we have plans to support both an official "DIND" and "regular" version of the runner docker image?

👋 @bharathkkb

i) Is there a reason why we are not prebaking the runner into the Dockerfile?

No, I like your approach better! 😄I'd welcome a PR, or I can get to it soon.

ii) Do we have plans to support both an official "DIND" and "regular" version of the runner docker image?

I'd prefer to move away from DinD if possible and support only one runner docker image, but I'm open to your input?

I'd prefer to move away from DinD if possible...

We should keep an eye on whether any replacements to DinD support container features like services, container. We might be limited to DinD for the time being – in any case, I'd rather support just one runner docker image if possible.


Design Companion Asset and Initialise

Description
This task is to gather assets that can deeply immerse the user further in the game through a visually appealing companion. The assets of the companion need to match in style of the assets of the environment. This will change from each environment. Furthermore, the asset must be placed on the map.
Dependencies

[] Map (#4 )

Milestones
List of steps that need to be completed for this task.

[ ] Download any sprite/entity skin (Aug. 19)
[ ] Create companion entity with location on map (Aug. 20)
[ ] Give companion entity a visual skin/avatar that has been downloaded (Aug. 23)
[ ] Run game and show map with entity on it (Aug. 24)

Completion Deadline: Aug. 24
Documentation

TBD

Member

Name (GitHub Username) (Slack Handle)
Maxwell (MaxwellSchroder) (@MagWell)
Saakshi Gupta (saakshigupta2002) (@saakshigupta)
Atharva Gupta (atharvagupta2003) (@atharva0179)
Daksh Jain (Daksh2607) (@daksh_52443)
Raghavendra Singh Gulia (RagGul)(tantrik8119)
Gaurika Diwan(gaurikadiwan)(@gaurikadiwan_03168)

Lets reduce the number of team members working on this to 2, ideally


feat: add fork and catch signal

That doesn't compile in /real_test
src/function/00_launcher.c: In function ‘function_launcher’:
src/function/00_launcher.c:10:50: error: ‘crash_test_function’ undeclared (first use in this function); did you mean ‘a_test_function’?
   10 |     load_test(&testlist, "crash test function", &crash_test_function);
      |                                                  ^~~~~~~~~~~~~~~~~~~
      |                                                  a_test_function
src/function/00_launcher.c:10:50: note: each undeclared identifier is reported only once for each function it appears in
make: *** [Makefile:68: obj/function/00_launcher.o] Error 1


Fix ClassCastException in ValueProcessorDelegate#offerToStore() method

Fixes: https://github.com/SpongePowered/SpongeCommon/issues/2491

Er... is this the problem? Everything else in the Keys class has a mutable type parameter.
public static final Key<ImmutableBoundedValue<Integer>> EXPERIENCE_FROM_START_OF_LEVEL

https://github.com/SpongePowered/SpongeAPI/blob/4d86523fbb3b10dc60952465313f8ab9d2a1f64b/src/main/java/org/spongepowered/api/data/key/Keys.java#L770

Yes, this is a problem if you try to do this:
Set<ImmutableValue<?>> dataValues = player.getValues();

And then later:
Player player;
dataValues.forEach(player::offer);

The bottom line is to save all the player’s data, and then try to offer each of them to the player, even if some data cannot be changed. Player#offer() method should quietly return a failed transaction, but instead it throwing exception...


[Docs] Typo in CONTRIBUTING.md. ui-setup needs to change to ui-build

Current State
Go to this link to see the incorrect command mentioned.
https://github.com/meshery/meshery/blob/master/CONTRIBUTING.md#build-and-export-ui

Desired State
The command should be:
make ui-build

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Closing as the appropriate steps are correct.

The PR was wrong. The issue isn't.


Netty causes ClassLoader leak in container environments

I'm running Netty in a container-like environment where modules can be loaded and unloaded. Unloading a module that has used Netty causes the ClassLoader to be leaked.
I'm able to reproduce with this minimal example (running in the container environment, obviously):
public class GatewayHook extends AbstractGatewayModuleHook {

    private final Logger logger = LoggerFactory.getLogger(getClass());

    private final NioEventLoopGroup eventLoop = new NioEventLoopGroup(0);

    @Override
    public void setup(GatewayContext gatewayContext) {
        logger.info("setup()");
    }

    @Override
    public void startup(LicenseState licenseState) {
        logger.info("startup()");

        try {
            Bootstrap bootstrap = new Bootstrap();

            bootstrap.group(eventLoop)
                .channel(NioSocketChannel.class)
                .option(ChannelOption.CONNECT_TIMEOUT_MILLIS, 5000)
                .option(ChannelOption.TCP_NODELAY, true)
                .handler(new ChannelInitializer<SocketChannel>() {
                    @Override
                    protected void initChannel(SocketChannel socketChannel) throws Exception {
                        logger.info("initChannel");
                    }
                });

            bootstrap.connect("localhost", 1234).get(2, TimeUnit.SECONDS);
        } catch (Throwable t) {
            logger.error("failed getting un-gettable endpoints: {}", t.getMessage(), t);
        }
    }

    @Override
    public void shutdown() {
        logger.info("shutdown()");

        try {
            eventLoop.shutdownGracefully().get();
        } catch (Throwable e) {
            logger.error("Error waiting for event loop shutdown: {}", e.getMessage(), e);
        }
        try {
            GlobalEventExecutor.INSTANCE.shutdownGracefully().get();
        } catch (Throwable e) {
            logger.error("Error waiting for GlobalEventExecutor shutdown: {}", e.getMessage(), e);
        }
        try {
            DefaultAddressResolverGroup.INSTANCE.close();
        } catch (Throwable e) {
            logger.error("Error closing DefaultAddressResolverGroup: {}", e.getMessage(), e);
        }
        InternalThreadLocalMap.destroy();
        FastThreadLocal.removeAll();
    }

}

I've tried shutting down as many global looking Netty resources as I could find but it still leaks. I've uploaded a JProfiler heap dump here. If you group by ClassLoader and then select the only NonLockingURLClassLoader instance you'll see the Netty classes still hanging around.
I have of course tested this without invoking any Netty code and it does not leak.
Netty version
Netty 4.1.9.Final
JVM version (e.g. java -version)
java version "1.8.0_112"
OS version (e.g. uname -a)
macOS 10.12.3

@kevinherron can you try calling FastThreadLocal.removeAll() as last method before complete the shutdown method ?

@normanmaurer I added that and tested again, still leaks.

@kevinherron can you provide me with something that I can easily run to reproduce ?

@kevinherron also can you please share a standard dump that can be opened with an OSS tool ?

@normanmaurer here's a dump in hprof format
Can I email you instructions for setting up an environment to reproduce? It should take less than 5 minutes from downloading the software to reproducing.

@kevinherron norman dot maurer at googlemail dot com ;)

@normanmaurer did you get my email?

@kevinherron yes but had no time yet :(

@normanmaurer ok that's no problem; I just wanted to make sure it arrived.

It's not (or not only) a ClassLoader leak.
The io.netty.util.concurrent.GlobalEventExecutor singleton is still up (along with a FastThreadLocalThread instance).

@slandelle right... that's what leads to the ClassLoader leaking, is that there's a handful of Netty bits still running or referenced that I can't figure out how to shut down.

Could be Finalizers.
Could be some Netty resources being cached ThreadLocals while they are being loaded from a global thread that the unloaded module doesn't own.

ideally we can provide a reasonably centralized place to cleanup/unload static state.

@kevinherron is this issue still present ? I suspect I fixed it

@normanmaurer funny timing... I've got a backlog card for upgrading Netty to latest and testing using the proof of concept I developed. I'll let you know soon.

@normanmaurer is there a particular minimum Netty version you want me to use?

@kevinherron go for 4.1.22.

It doesn't appear to be fixed... still a bunch of Netty classes showing up in JProfiler after unloading the module.
HPROF before unloading the module
HPROF after unloading the module and a GC run
The only class provided by the PoC module is com.kevinherron.GatewayHook, which you can see is properly unloaded; it's just some Netty classes still hanging around. Everything else in the dump that isn't io.netty or com.kevinherron is part of the container.

@kevinherron could you create a small reproducer and explain how to run it ?

@normanmaurer do you need anything else?


Updating help text for each commands

I can help create PR on the typings help <command>. Do you have a specific format in mind?
The help in typings install is ok, but the options could use more love. :smile:
I think npm help <command> is way to verbose.

Ah, the help command is complete man pages. I don't think we need that. We currently have the --help flags, want to continue using those or implement a whole command? I'm happy to accept any PR improving docs :smile:

Ar yes, I meant typings <command> -h

What does typings i --production do?  Shorthand -P? how about shorthand for --save-peer?

Calling typings install [<source>!]<pkg>... or typings install [<registry>!]<pkg>...?

Shorthands are here: https://github.com/typings/typings/blob/master/src/bin.ts#L81-L88. So -P is --save-peer. typings install --production enables only installing production dependencies (E.g. skipping dev). It can be inverted with --dev. --production is default enabled when NODE_ENV=production.
As for the <source> vs <registry>, I've been using source mostly because registry is overloaded here with typings/registry and I call it the "registry API". Plus I just enabled registry: in typings.json. However, I'd love a better term :smile: If more people can agree on replacing source with registry and fix the other usages of mine, then :+1:

I think this is resolved? Feel free to open a PR or re-open this issue if there needs more corrections. Thanks for helping improve the documentation!


[Bug]: Audit Logs - Workspace details not available in description tag of user.invited event

Is there an existing issue for this?

[X] I have searched the existing issues

Description
In the Audit Log event user.invited the workspace details is not available in the description which is present in the API response. This is only applicable after the new UI is released from EE PR #811

Steps To Reproduce

Invite a user to any workspace
Check the Audit Log entry of that
The workspace details will be missing in the subtitle description

Public Sample App
No response
Issue video log
No response
Version
release

Closing this one.
If the issue is seen again, please reopen the ticket.


Investigate default connection picking behavior in NotebookContexts

When selecting the default connection in NotebookContexts.getActiveContexts, the only criteria used is whether the serverName matches the one in the provided connection profile. We should investigate whether things like database name and user name should also be considered.
Note that this doesn't cause any issues with picking a "wrong" connection. All the other active connections for the supported providers are still included in the dropdown, just that the default connection may not be the most accurate.

This has since been fixed by the connection changes in the Attach To dropdown.


fix typo error

Fix typo errors

There seems to be a build error:
https://github.com/microbiome/miaViz/actions/runs/4499982965/jobs/7918528335?pr=72
Could you check and see if it is possible to fix.
Run devtools::check() in R before pushing your edits.

I'm not sure but this change: assay.type   vssdv = assay_name, assay_name = "counts",' but it seems to be the cause of the failing check. I would suggest you restore the line to original.

This is a bit tricky: the former argument is assay_name but in the new Bioconductor releases the packages are all switching to assay.type so that is the new one. If that causes problems we could either 1) temporarily use assay_name or 2) update the .github/workflows/ so that Github Actions installs the latest Bioconductor devel versions of mia, miaViz, scater, and scuttle.

If support is needed just let us know.

Hi @Vivian-Ginika - are you still working on this one?

Hello,
Yes I am, though I am yet to figure it out and had to pause to work on
another project for the contribution on outreachy but I have resumed
working on the PR. Though I will appreciate any help where necessary.
Thank you.
On Tue, Apr 4, 2023, 9:23 AM Leo Lahti @.***> wrote:

Hi @Vivian-Ginika https://github.com/Vivian-Ginika - are you still
working on this one?
—
Reply to this email directly, view it on GitHub
https://github.com/microbiome/miaViz/pull/72#issuecomment-1495553466,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3JGY43KGYCITPOYWS7566DW7PK65ANCNFSM6AAAAAAWENJWWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great. Just let us know when something specific is unclear, and we can take it from there.

Hi @Vivian-Ginika - still failing. Can you actively report here the fail message and seek for help. Otherwise we may miss it.


Allow mafia to be used with the development version (1.23.*) of cabal

cabal >= 1.23 is required for building with GHC 8.0 RC1, and thanks to recent commits, it now works properly with mafia

We should probably add a regression test for supported version at some point. :+1:

Yeah that would be great


add cloudTrail Script

Create query and retrieve query result from cloudTrail lake per given principalId, and eventDataStore (additional filter could be timestamp and key)
Query S3 object size to see if egress bytes match with events
Print results in table as example below

Currently CloudTrail lake is created manually. Could do it in a cdk, but I would assume would be done in the elsa deployment setup?
We could identify if the data transferred is via cli or pre-signed url (via the userAgent), but I think if we create the accessKeyId unique per sharing, then I don't think it would be relevant to distinguish those events as most likely the sharing would either be pre-signed or the s3-cli

Excellent work - thoughts on whether we can get this being used in the short term for tracking AG data sharing? @ohofmann  is the output of this enough to be useful for you?
Do you have a nodejs runtime installed (or could install it?)
We can probably bundle it up into a util you can run..

Spotted this yesterday - yes, very helpful. What is the input to this currently? A unique tag/access key embedded in the URLs of a single data access request or something else?

Because we are using SSO and not static IAM - the access key changes each (AWS) session

I mean we could make a map of access keys and the corresponding sharing request ID in this repo - the access keys themselves aren't secrets.
Long term the idea is that Elsa retains this access key -> release map in its db

Just to note. IIRC. We normally use a dedicated IAM user for PreSigning purpose. So they won't be changed quite often as SSO would on every login.
However, there could be potentially that the IAM user's credential will be rotated/revoked, yes. So, there is still needing a release map store some where for historical record keeping.

Is the dedicated IAM user to that we can get long(er) lived pre-signed keys?

Is the dedicated IAM user to that we can get long(er) lived pre-signed keys?

 Ref: aws-blog
Other IAM may still have expires=... within 7 days in the url query parameters, but I believe the access is still limited to their maximum IAM session.

Also for the Audit trail. Same scenario here. So that we could track S3 object access pattern. It was a bit of ad-hoc activity in the past. But, I reckon, this will become more routine trail now.

Hmm.. I was hoping to use the access key to distinguish between different releases of data.. I mean if the rate of data release is minimal we can have a bunch of IAM users or something I guess.
I know that extra parameters can be signed into a pre signed url.. I wonder whether it is possible for us to add a custom param (Elsa=ABCD) and get that into the cloud trail log?

add a custom param (Elsa=ABCD)

Might be possible. Something custom Sig4 signing process on canonical path, I think. Not really sure though.

Yah. May be we can do like per IAM user per cohort/flagship release (or something). One high level common view.

use the (varying) access key to distinguish between different releases of data

Or, another thinking is -- since one IAM user can have multiple access keys; per release per access key still doable that way, under some "common" IAM user. As long as we do not revoke it (either remove or make it inactive), etc.

Or, another thinking is -- since one IAM user can have multiple access keys (not sure upper bound though); per release per access key still doable that way

Unfortunately the upper bound is 2 I think (from memory)

upper bound is 2

Ouch!

I know that extra parameters can be signed into a pre signed url.. I wonder whether it is possible for us to add a custom param (Elsa=ABCD) and get that into the cloud trail log?

Possible, one of the cloudtrail logs is requestParameters which might appear there. I'll explore that if that is the case.

Seems to be correct! CloudTrail does record all parameter including those custom one. Added hello=world param in the presignedUrl
{X-Amz-Date=20220708T063518Z, bucketName=examp-temp-dev, X-Amz-Algorithm=AWS4-HMAC-SHA256, X-Amz-SignedHeaders=host, Host=examp-temp-dev.s3.ap-southeast-2.amazonaws.com, X-Amz-Content-Sha256=UNSIGNED-PAYLOAD, X-Amz-Expires=900, hello=world, key=example/test-head-1.csv}

Just wondering if we decided to use copy command (aws s3 cp), no custom parameter will be present there.

Nice - does that mean that the hello=world actually participated in the signing algorithm? (if you change to hello=earth in the pre-signed url text does it fail?)

Yes, it will fail

Just wondering if we decided to use copy command (aws s3 cp), no custom parameter will be present there.

Yes I think we'd do AWS<->AWS direct copies slightly differently.. possibly I'd make a per-release S3 access point. And track access that way.

Same here - other than perhaps moving it out of the application folder to a top-level util/ folder - happy to just get this merged and we can refactor as we go


fix(nimble): Change 'move_count' to uint8_t type

Change 'move_count' to uint8_t type to prevent out-of-bounds errors during memmove with compiler optimization enabled.

please add proper prefix to commit title (see other commits as guideline) and include error message in commit message

I'm having trouble understanding this fix,    this basically limit move_count to 255
what is exactly fixed here?

okay, so I looked at this and it appears that GCC is not able to properly track relations between globals (eg ble_store_config_num_peer_secs) and memmove in ble_store_config_delete_obj

I'd suggest to use assert() or if() to hint GCC
BTW BLE_STORE_MAX_CCCDS() might have similar issues  (if also configured to 1)

Basically, the size has been increased since, when compiler optimization was enabled, it was previously creating out-of-bounds errors. So, now it will prevent these errors.

Hi,
please check if https://github.com/apache/mynewt-nimble/pull/1737 solves this issue for you


Add validation for single statement lexical declarations

This patch fixes #3275 and fixes #3276.
JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik<EMAIL_ADDRESS>

One more thing: this is true for function statements except if/else statements, you could also add that as well.


Refactor the process of create Omise charge by extract class

1. Objective
Reduce the dependencies and functionality of a controller class, OmisePaymentModuleFrontController, by using a refactoring technique, extract class.
The OmisePaymentModuleFrontController has been used to create Omise charge.
According to the pull request #17, it has emphasized the class, OmisePaymentModuleFrontController , has many dependencies and functionality. This situation may tend to the class has error prone, especially in part of software maintenance.
Including the next steps, this class will been modified to add more functions such as store the order, handle the errors and display the proper result.
So, it will be better to reduce the dependencies and functionality of a class, OmisePaymentModuleFrontController.
Related information:
Related issue: -
Related ticket: -
Required pull request: #17
2. Description of change

Add a new delegate class, Charge, to create Omise charge request and interface with Omise-PHP.
Remove all functions in controller class, OmisePaymentModuleFrontController, that have been moved to delegate class.

3. Quality assurance
Environments:

Platform: PrestaShop <IP_ADDRESS>
Omise plugin: Omise-PrestaShop <IP_ADDRESS>
PHP: 5.6.28
Mozilla Firefox: 50.1.0

Details:

Make sure that the Omise-PrestaShop has been installed and enabled.
At the front-end, add a product to cart.
Proceed to checkout.

Test case 1: Successfully checkout

At the step, 05. Payment, fill the valid card information and click button, Submit Payment. The system will process the payment and display the result page.

The screenshot below shows the result page display the successful payment.

The screenshot below shows charge detail at Omise dashboard. This charge was created from the above checkout.

Test case 2: Failed checkout

From the above success checkout, refresh the web browser page.

The screenshot below shows the result page display the error payment.

4. Impact of the change
-
5. Priority of change
Normal
6. Additional notes
The function to handle the create Omise charge error will be developed for the next pull request.
The screenshot below shows the passed unit tests at Travis CI.

FYI: Im gonna close this PR since this one already merged to #19 and we're going to review this changes in that PR.
Thanks for the changes 👍

Note, the reason we moved is, because PR #14 and #17 were made to introduce Omise Charge service to the plugin but this PR did refactoring code based on those changes (#14, #17).
To make the review process more easy to follow with and more clear the big picture of those changes (including this refactoring changes). We decided to merge all of them into 1 branch, which is #19.


[Making aegir smaller] Add support for pkg/pkg-bundle

This adds the missing pkg.scripts field to allow aegir to be bundled using pkg/pkg-bundle which is part of #215.
This is part of an effort to write https://github.com/mkg20001/aegir-bundle which depends on writing https://github.com/mkg20001/pkg-bundle
Note: Currently WIP, the CLI isn't working properly yet.
Commands that work:

[ ] aegir build https://github.com/mkg20001/aegir-bundle/issues/4
[ ] aegir coverage https://github.com/mkg20001/aegir-bundle/issues/5
[x] aegir docs
[x] aegir lint-commits
[ ] aegir lint https://github.com/mkg20001/aegir-bundle/issues/6
[ ] aegir release
[ ] aegir test https://github.com/mkg20001/aegir-bundle/issues/7

Same feedback from https://github.com/ipfs/aegir/pull/216#issuecomment-425967638 applies to this one, @mkg20001.

I don't think I'll be able to finish this alone. Most of the work is in https://github.com/mkg20001/pkg-bundle which is a fork of pkg that uses the existing bundling code to create single-file bundles, instead of node executables like pkg did and the other part of the effort goes into https://github.com/mkg20001/aegir-bundle which is handling the bundling of aegir and retrieving/executing of the bundle.
Maintaining, or rather writing it is hard as it's basically messing with node's internals. Sure, it's possible to write something like this cleanly but I simply don't have the resources right now to focus on so many things and also do each of those things great.
Giving this endeavor up doesn't seem like a wise choose considering it isn't far from completion, but I don't think I can and should be the only one putting work into this.
So to make it short: Would there be any members of the IPFS team/maintainers of aegir/other people that would like to help me finish this? Remember that this might make a huge impact on installation time and performance of aegir which could make all of IPFS's contributors life easier.

// cc @ipfs/dx

Maintaining, or rather writing it is hard as it's basically messing with node's internals

Yeah, that doesn't sound very fun nor maintainable in the future as it'll be breaking often.
I think an easier effort right now (with maybe not the same amount of impact but at least easier to do) is to go through the top-level dependencies and try to remove the biggest ones or find other ways of installing them to reduce the disk space.
The only clear decision I will make is that the DX WG won't take ownership over this specific change as it'll be too much work at this point. Maybe there are other JS/NodeJS devs more excited about this and want to implement it?

Maintaining, or rather writing it is hard as it's basically messing with node's internals

Yeah, that doesn't sound very fun nor maintainable in the future as it'll be breaking often.

It should be noted though that https://github.com/zeit/pkg has not needed many changes in a long time and most of the changes pkg-bundle would need would come from upstream, so rebasing and testing would be the only maintaince thing that would be necessarry to do.

considering it isn't far from completion

There is only a few bugs with the child_process transformer that prevent things such as webpack/mocha from running correctly. Once these are solved the CLI should just work. (Considering I didn't need to change a single line in aegir's code so far to get lint-commits and docs cmds to work, I don't expect this to break soon. The only changes I think will be necesarry would be to the test cmd to spawn nyc/mocha in a way that pkg-bundle could detect that the executable is inside the bundle and spawn that using the wrapper)

old pr gonna close this, feel free to reopen


Document Migration Impact on modifing openid scope claims

According to the OpenID protocol specifications, any user attributes must not send for the 'openID' scope.
The migration guide should track the following points relevant to this modification.

'openID' scope does not send any user attributes.
This change will affect only for new tenant creations.
The previous behaviour can be restored by modifying the oidc-scope-config.xml file.

<Scope id="openid" displayName="Open ID" description="Indicate the use of OpenID Connect protocol.">
     <Claim>
         sub,email,email_verified,name,family_name,given_name,middle_name,nickname,preferred_username,upn,groups,profile,picture,website,gender,birthdate,zoneinfo,locale,updated_at,phone_number,phone_number_verified,address,street_address,country,formatted,postal_code,locality,region,roles
    </Claim>
 </Scope>

Following attributes are no longer send by any default OpenID scopes.
upn, locality, region, postal_code, groups, roles, country, street_address, formatted

Added to docs with https://github.com/wso2/docs-is/pull/2991, https://github.com/wso2/docs-is/pull/2997


SessionManager.default Memory leak

Hi,
I am profiling sessionManager.default and I'm  curious why every time I call sessionManger.default it creates a memory leak.
update
I also profiled the sample project and has the same issue
@IBAction func didTapButton(_ sender: Any) {
      let _ = SessionManager.default
}

I've seen leaks like this before, they're usually a false positive or something from the underlying Swift standard library, so there's a not a lot we can do about it. It will likely be fixed in a future version of the tools.


Catch ENOTEMPTY in cleanup_keg

Fixes #11904

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

The output of brew cleanup:
% brew cleanup
Warning: Skipping (old) /usr/local/Cellar/maven/3.8.2 due to it being linked
Removing: /usr/local/Cellar/unbound/1.13.2_1... (52 files, 5.6MB)
Warning: Directory not empty @ dir_s_rmdir - /usr/local/Cellar/unbound/1.13.2_1
Error: Could not cleanup old kegs! Fix your permissions on:
  /usr/local/Cellar/unbound/1.13.2_1

How can I reproduce this situation:

brew install unbound
sudo brew services start unbound
Bump revision of unbound.rb
brew install --build-bottle unbound
brew cleanup

Do you need to fake being root here? Would just doing chmod work for the test you had in mind?

Thanks again @umireon!


Authentication

Had an attempt, but was unhappy with the result. Saved the 'failed' attempt and started from scratch.

18/03/2020 - When trying to run rails server got an error saying "Your yarn packages are out of date". Ran "yarn install --check-files" and fixed the issue.

24/03/2020 - My first attempt at creating the authentication system was not looking that great, so I saved the files into a folder and planning to start from scratch.

25/03/2020 - Started the project from scratch:

cloned
ran "yarn install --check-files"
started building on the app and created a welcome dashboard with a navbar
encountered issues with the navbar dropdown menus and searched on about 10 forums until I (hopefully) have found a fix.

27/03/2020 - Issue should now be almost done, although there are two problems encountered:

after a lot of struggle with the dropdown menus and some workarounds, I have managed to make them work, but they work only from the homepage.
CanCanCan gem, which restricts what a user can see/manage seems to be denying access to any role (user or admin). Need to investigate the issue. Have disabled this option for now.

Following this tutorial : https://www.nopio.com/blog/authentication-authorization-rails/  for my attempt at the authentication system I have introduced three new gems into the application:
Devise, CanCanCan and Rolify:

Devise is used as a flexible authentication solution which comes with multiple modules.

Rolify is used to create and assign roles to users, which helps to differentiate between each separate user that have access to their own playlists only and users with administrative role that have access to all the playlists on the database. This gem is only used for creating and assigning roles.

CanCanCan works with Rolify to restrict access to resources based on user's role.

The development started with cloning the application then creating a homepage and by adding bootstrap and styles( navbar ).
After that I have then created a database with a table called playlists that stores the playlists and their rating, for now. This will be further developed at a later point in the project. The table contains, for now, two columns: Name (string) and rating ( float).
Links to the playlists table have been created with a few buttons and menus.
During this process I have encountered multiple issues with the dropdown menus as they would work only from the homepage. The pages could be accessed and used by navigating to the URL of that page.
After a temporarily work-around has been found for the drop down menus issue I proceeded to integrate the authentication system and the gems specified earlier.
Firstly I have installed Devise and added bootstrap to it, created a Users table (email and password only ) and linked it to the navbar. At first all the buttons would be visible (log in, sign up, log out, edit), but soon after implementing them I have set it up so that it only shows relevant buttons (sign in and sign up ) when not in a session and (edit and log out) when in an active session.
After these changes I have added username and user id to the User table so that playlists could be assigned to specific users only.
After making sure that a logged-in user can successfully create a playlist and that the playlist is successfully assigned to them, I have started working with the Rolify gem to add a User and an Admin role.
Modified seeds.rb to automatically seed the database with two users with different roles and with some playlists assigned to each of them.
On my attempt to setup CanCanCan and introduce abilities to the application I have stumbled over some issues with granting and restricting access to resources, which is still to be investigated.

The default database authentication has now been replaced with a Spotify OAuth flow. To accomplish this, I have used the OmniAuth Spotify provider.
Additionally, I have reviewed the Devise configuration, making changes where appropriate and cleaning up some minor things.


scan in docker failed - permission denied on home folder

When I run ./runscan.sh it fails with the following error:
Running scan in Docker...
Ctrl-C to break.
Traceback (most recent call last):
  File "./crawler.py", line 620, in <module>
    crawler = Crawler(**vars(ap.parse_args()))
  File "./crawler.py", line 186, in __init__
    fh = logging.FileHandler(os.path.join(out_path, 'log.txt'))
  File "/usr/lib/python3.5/logging/__init__.py", line 1008, in __init__
    StreamHandler.__init__(self, self._open())
  File "/usr/lib/python3.5/logging/__init__.py", line 1037, in _open
    return open(self.baseFilename, self.mode, encoding=self.encoding)
PermissionError: [Errno 13] Permission denied: '/home/<local user>/out/log.txt'
Scan failed. See log.txt for details.

Is this fixed by #48?

Fixed in #48


[release-0.3] fix: Add missing requirements to windows.11 and windows.2k16 preferences

What this PR does / why we need it:
Add missing requirements to windows.11 and windows.2k16 preferences and move functests to u1.large instancetype.
Manualy cherry-pick of 16096fb1e1507142ebd74d2801e3bc24884e563 and f39e6c5993f1565334c2046f95019875b035b584.
CONFLICTS:

common-clusterpreferences-bundle.yaml
common-instancetypes-all-bundle.yaml
common-preferences-bundle.yaml

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

fix: Add missing requirements to windows.11 preferences
fix: Add missing requirements to windows.2k16 preferences

/hold

/hold cancel

/lgtm

/retest


Confusing task completion message

When I run a workflow with remote task, I get this:
INFO: Running module small_data:
INFO: small_data (index=0) is completed.
INFO: small_data (index=3) is completed.
INFO: small_data (index=7) is completed.
INFO: small_data (index=6) is completed.
INFO: small_data (index=2) is completed.
INFO: small_data (index=4) is completed.
INFO: small_data (index=5) is completed.
INFO: small_data (index=1) is completed.
INFO: small_data (index=8) is completed.
INFO: small_data (index=9) is completed.
INFO: small_data (index=12) is completed.
INFO: small_data (index=10) is completed.
INFO: small_data (index=14) is completed.
INFO: small_data (index=11) is completed.
INFO: small_data (index=13) is completed.
INFO: small_data (index=15) is completed.
INFO: small_data (index=16) is completed.
INFO: small_data (index=17) is completed.
INFO: small_data (index=19) is completed.
INFO: small_data (index=18) is completed.
INFO: small_data (index=20) is completed.
INFO: small_data (index=21) is completed.
INFO: small_data (index=22) is completed.
INFO: small_data (index=23) is completed.
INFO: small_data (index=24) is completed.
INFO: small_data (index=27) is completed.
INFO: small_data (index=26) is completed.
INFO: small_data (index=25) is completed.
INFO: small_data (index=29) is completed.
INFO: small_data (index=28) is completed.
INFO: small_data (index=31) is completed.
INFO: small_data (index=32) is completed.
INFO: small_data (index=34) is completed.
INFO: small_data (index=33) is completed.
INFO: small_data (index=30) is completed.
INFO: small_data (index=35) is completed.
INFO: small_data (index=36) is completed.
INFO: small_data (index=37) is completed.
INFO: small_data (index=38) is completed.
INFO: small_data (index=39) is completed.
INFO: small_data (index=40) is completed.
INFO: small_data (index=41) is completed.
INFO: small_data (index=43) is completed.
INFO: small_data (index=44) is completed.
INFO: small_data (index=42) is completed.
INFO: small_data (index=45) is completed.
INFO: small_data (index=46) is completed.
INFO: small_data (index=47) is completed.
INFO: small_data (index=48) is completed.
INFO: small_data (index=50) is completed.
INFO: small_data (index=52) is completed.
INFO: small_data (index=54) is completed.
INFO: small_data (index=49) is completed.
INFO: small_data (index=55) is completed.
INFO: small_data (index=53) is completed.
INFO: small_data (index=51) is completed.
INFO: small_data (index=56) is completed.
INFO: small_data (index=57) is completed.
INFO: small_data (index=59) is completed.
INFO: small_data (index=58) is completed.
INFO: small_data (index=60) is completed.
INFO: small_data (index=61) is completed.
INFO: small_data (index=63) is completed.
INFO: small_data (index=64) is completed.
INFO: small_data (index=66) is completed.
INFO: small_data (index=65) is completed.
INFO: small_data (index=62) is completed.
INFO: small_data (index=69) is completed.
INFO: small_data (index=67) is completed.
INFO: small_data (index=68) is completed.
INFO: small_data (index=70) is completed.
INFO: small_data (index=71) is completed.
INFO: small_data (index=73) is completed.
INFO: small_data (index=72) is completed.
INFO: small_data (index=74) is completed.
INFO: small_data (index=76) is completed.
INFO: small_data (index=75) is completed.
INFO: small_data (index=77) is completed.
INFO: small_data (index=78) is completed.
INFO: small_data (index=79) is completed.
INFO: small_data (index=80) is completed.
INFO: small_data (index=81) is completed.
INFO: small_data (index=82) is completed.
INFO: small_data (index=83) is completed.
INFO: small_data (index=84) is completed.
INFO: small_data (index=85) is completed.
INFO: small_data (index=87) is completed.
INFO: small_data (index=86) is completed.
INFO: small_data (index=88) is completed.
INFO: small_data (index=89) is completed.
INFO: small_data (index=91) is completed.
INFO: small_data (index=92) is completed.
INFO: small_data (index=94) is completed.
INFO: small_data (index=93) is completed.
INFO: small_data (index=95) is completed.
INFO: small_data (index=90) is completed.
INFO: small_data (index=96) is completed.
INFO: small_data (index=97) is completed.
INFO: small_data (index=100) is completed.
INFO: small_data (index=101) is completed.
INFO: small_data (index=98) is completed.
INFO: small_data (index=99) is completed.
INFO: small_data (index=103) is completed.
INFO: small_data (index=104) is completed.
INFO: small_data (index=102) is completed.
INFO: small_data (index=105) is completed.
INFO: small_data (index=106) is completed.
INFO: small_data (index=108) is completed.
INFO: small_data (index=107) is completed.
INFO: small_data (index=113) is completed.
INFO: small_data (index=112) is completed.
INFO: small_data (index=114) is completed.
INFO: small_data (index=115) is completed.
INFO: small_data (index=109) is completed.
INFO: small_data (index=110) is completed.
INFO: small_data (index=111) is completed.
INFO: small_data (index=118) is completed.
INFO: small_data (index=116) is completed.
INFO: small_data (index=119) is completed.
INFO: small_data (index=117) is completed.
INFO: small_data (index=123) is completed.
INFO: small_data (index=122) is completed.
INFO: small_data (index=121) is completed.
INFO: small_data (index=120) is completed.
INFO: small_data (index=125) is completed.
INFO: small_data (index=124) is completed.
INFO: small_data (index=126) is completed.
INFO: small_data (index=127) is completed.
INFO: small_data (index=130) is completed.
INFO: small_data (index=133) is completed.
INFO: small_data (index=132) is completed.
INFO: small_data (index=128) is completed.
INFO: small_data (index=129) is completed.
INFO: small_data (index=131) is completed.
INFO: small_data (index=134) is completed.
INFO: small_data (index=135) is completed.
INFO: small_data (index=136) is completed.
INFO: small_data (index=137) is completed.
INFO: small_data (index=140) is completed.
INFO: small_data (index=141) is completed.
INFO: small_data (index=139) is completed.
INFO: small_data (index=138) is completed.
INFO: small_data (index=143) is completed.
INFO: small_data (index=144) is completed.
INFO: small_data (index=142) is completed.
INFO: small_data (index=146) is completed.
INFO: small_data (index=147) is completed.
INFO: small_data (index=148) is completed.
INFO: small_data (index=145) is completed.
INFO: small_data (index=149) is completed.
 They are submitted by batches as 8 tasks to the cluster. However I first see messages ... is completed, which makes me think the actual computation was done in no time, and confused me for a bit. I then recall this is simply saying the substeps are submitted to the job scheduler, not exactly complete with output.
Can we improve the logic here to avoid the confusion?

I think this was a new message added recently because we see all the "skipped" messages but no "completed" ones, but the messages was added without considering the tasks.


Fix error 404 with worker-json.js

On the BO we have the error
GET http://localhost:3000/worker-json.js 404 (Not Found)

Not happening on prod build with npm run build


StreamSubscription should have getters/setters for the handlers

StreamSubscription currently has onData, onError and onDone functions that take the new handler.
That feels odd.
-> Make them setters and add corresponding getters.
Either keep the name, or change to dataHandler, doneHandler, errorHandler.
This would correspond to dart:isolate names.

It would have been nice if we had setters as well. With the current Dart class and type restrictions, we cannot have methods and setters with the same base name, and we cannot change the onDone method into a getter with type void Function(void Function()) and add a setter with argument type void Function().
So, it would be a breaking change to replace the onX methods, and adding a setter with a different name, like doneHandler, and then slowly migrating to that, is probably not worth it.
We can reconsider this for Dart 3.0 (but probably not even there).
I'd consider defining extension setters in package:async for people who want it.


Add moving carousel in the about us' gallery section

I want to improve the about us page ui and also add an automatically moving carousel there. You can assign me level later. Just assign the issue now.
ISSUES HIGHLIGHTED

@swatified resolved this issue!


Apple Watch MVVM Infra

Story:
we need to build the well-designed structure.
Reference:
https://github.com/ttt246/Brain/pull/210

It takes around 4 days.


Other models - Syntax, IFS

Hi, is there a way of generating images based on different algorithms such as syntax or IFS as in PyFractal?
Thanks!

For now, this project just generates fractals from the Julia and Mandelbrot sets using the escape time algorithm. You can send a pull request implementing other fractal algorithms, if you wish.


Prompt Enclosure config resets to none when refresh or switch level

Bug report
Description
In sandbox, when you select a prompt enclosure defence option, then refresh or switch to a different level and back to sandbox you'll see that the none option is now selected and the message is not being transformed
Reproduction steps
Steps to reproduce the behaviour:

open the app and go to sandbox
open Defence Configuration panel, then the Prompt Enclosure panel, and select an option other than none
configure  the pre-prompt if you like.
refresh the page OR navigate to a different level and back to sandbox
open Defence Configuration panel, then the Prompt Enclosure panel
Observe that the none option is now selected
Send a message
Observe that the message is not transformed
Re-select the option you selected before
Observe that you are not allowed to configure the prompt any more (until you select a different option and select the first one again). Also there is an info message in the chat that you have disabled the option you just selected

Expected behaviour
When you refresh or switch to a different level and back to sandbox, the selected Prompt Enclosure option should remain selected, and the user's message should still get transformed
Screenshots
configure the prompt enclosure defence

after refresh

After reselecting the original option

Software (please complete the following information):

Browser: chrome

Additional context
Probably related to #741 and #744
Acceptance criteria
GIVEN In sandbox and a Prompt Enclosure option is selected
WHEN refresh page or switch to a different level then back to sandbox
THEN That same Prompt Enclosure option remains selected.
AND Further messages get transformed
AND the prompt config is editable

should be fixed in #753

It was fixed! I will close the issue


How to answer on replyMarkup using Telegram java TdApi?

Help please to make reply on replyMarkup.
https://stackoverflow.com/questions/56923326/how-to-answer-on-replymarkup-using-telegram-java-tdapi

You need to use method GetCallbackQueryAnswer to send a callback query to a bot passing CallbackQueryPayloadData with data from the button as a payload.
For example, button
          InlineKeyboardButton {
            text = "3"
            type = InlineKeyboardButtonTypeCallback {
              data = bytes [15] { 73 65 6E 64 5F 72 65 61 63 74 69 6F 6E 5F 32 }
            }
          }

has data = bytes [15] { 73 65 6E 64 5F 72 65 61 63 74 69 6F 6E 5F 32 }.

Thank you so much. It exactly what i looked for.

Hi. Help please to make reply on replyMarkupShowKeyboard.

@PavelZubkov replyMarkupShowKeyboard contains options of text responses for the user. Application needs only to send a text message with the corresponding text.

Thank you very much

error! code: %!d(float64=400) msg: Failed to parse JSON object as TDLib request: Unknown class ""

that I use GetCallbackQueryAnswer for use.


Stop a notification

Hello,
Your extension is very handy, thanks for it. However, I'm on a situation where it's been annoying: I have added a single remind of 1 minute for test it, and it is appearing every minute. There is no way to stop it appearing, it's like the only option is to dismiss it.
I don't see any way of erasing or cancelling it. Could you please help me ?
Thanks in advance.

Hey @danielo515, thanks so much for taking the time to raise the issue. I really appreciate the effort 🙇 I fixed it and now it should work fine. If am not asking for too much would you mind testing it out locally before I push it to the marketplace ? Cheers 😄

Sure! Just provide me with the non-official version and some basic instructions and I'll check it out.
Regards

Clone this repo and do an npm install and place it inside...
Windows %USERPROFILE%.vscode\extensions
Mac ~/.vscode/extensions
Linux ~/.vscode/extensions
based on your platform...
Make sure you remove the extension installed from marketplace before doing this step 🙂

Hello @cg-cnu
I followed your steps

Uninstall the marketplace version
Clone your repository into Mac ~/.vscode/extensions
Npm install
Reload VSCode

When I try to use your extension I get the following error:
command 'remindMe.remind' not found
Regards

Ok, you forgot to mention the execution of the compile script

Hey @danielo515 Sorry for the trouble. Yep, forgot to mention the compile script.
Run this after install npm run compile and restart vscode instance. Thanks 👍

I did it already. The extension works fine ! Many thanks

Once again thanks so much for bringing the issue to my notice and testing it out. I will release the extension with the update 😄


unregistering core system does not change associated private member

Describe the bug
member spatialAwarenessSystem in MixedRealityToolkit (and other core services) are not set to null when the associated service is unregistered.
This force to reset the whole configuration with ResetConfiguration to have the cache properties cleared.
To reproduce
i'm trying to change spatial awareness profile by code without changing whole mrtk profil configuration.
so i first disable the current spatailawareness
MixedRealityToolkit.Instance.UnregisterService<IMixedRealitySpatialAwarenessSystem>()
then i register my new profile this way:
//MyNewSpatialAwarenessProfile is a SerializeField in my component
object[] args = { MixedRealityToolkit.Instance, MyNewSpatialAwarenessProfile };
MixedRealityToolkit.Instance.RegisterService<IMixedRealitySpatialAwarenessSystem>(MixedRealityToolkit.Instance.ActiveProfile.SpatialAwarenessSystemSystemType, args: args)

the problem is that MixedRealityToolkit.SpatialAwarenessSystem is still pointing to the previous system, not the new registered one.
Expected behavior
when calling UnregisterService on a CoreSystem, i expect to have the corresponding property set to null
public bool UnregisterService<T>(T serviceInstance)
{
    ...
    if (IsCoreSystem(interfaceType))
    {
        ClearCoreSystemCache(); //a bit too much of cache cleaning but avoid testing which system should be cleared :p, could make a bit more code to clean only the corresponding system cache
        ...
    }
    ...
}

Your Setup (please complete the following information)

Unity Version2018.3.11f1
MRTK Version 2.0 RC1 Refresh

Additional context

@davidkline-ms