Fine-grained ChannelBuffer interfaces

We need to split the current ChannelBuffer interface into multiple smaller ones:

SequentialReadBuffer
SequentialWriteBuffer
SequentialIoBuffer extends the two above
RandomReadBuffer extends SequentialReadBuffer
RandomWriteBuffer extends SequentialWriteBuffer
RandomIoBuffer extrends the two above

Users will usually use RandomIoBuffer in most cases (i.e. ChannelBuffer is renamed to RandomIoBuffer)
Do you have an idea for better names? Let me know!

There's also the issue of ReadableChannelBuffer and WritableChannelBuffer...  Are these must siblings of RandomReadBuffer and SequentialReadBuffer and RandomWriteBuffer and SequentialWriteBuffer respectively?
Is this just a tag interface or is policy actually enforced?  I imagine the sequential buffers would have only relative methods and the random ones would have absolute methods?  Of course with a random buffer one could still do sequential IO ... just with everything being absolute.

Maybe it should be Relative vs Absolute instead of Random and Sequential?

@Trustin: I think we should rename ChannelBuffer to RandomIoBuffer and then "create" ChannelBuffer interface again which extends RandomIoBuffer. The new ChannelBuffer interface should then get marked as @deprecated and our implementation of RandomIoBuffer should also implement ChannelBuffer. So we would not break to much of users code.


Drag-and-drop of data into MM does not work

Loading data into MM by drag-and-drop into the gui window does not work. A progress bar is created, but data are not loaded.

This would not be expected to work. Pycro-manager uses different libraries for data storage (https://github.com/micro-manager/NDTiffStorage) and image viewing (https://github.com/micro-manager/NDViewer) than the ones in micro-manager, in order to support additional features like arbitrary image dimensions, multi-resolution pyramids, and flexible modification/injection/deletion of image data on-the-fly. I do think it would be a very nice feature to support though, if you're interested in taking a stab at adding it (would be on the Java side).

Ah, I see. I think coding that is beyond my abilities, though... I did see somewhere that you support viewing with napari, I'll give that a try.


Model is predicting low numbers for nearly all lineages

Nearly all lineages are being coloured as having low numbers of unsampled infections.
We should switch from using this model as being the default colouration for trees, back to Divergence (residual from clock prediction)
Need to go back and check fit and outputs for the number of infections model.

Default display has been switched back to divergence by @GopiGugan

Run pipeline to capture intermediate outputs if necessary
See if regression model needs to be updated, or if there is something that has changed for the inputs that is causing the model to go awry

Downloaded recoded.json and lineage-specific Newick files from Paphlagon to my local machine for some testing

new branch iss497 56a0786d30e18aeda59d2ba4ba1d6da16e89b0a6

I'll try to wrap up some of this refactoring and then I may need to hand this off to someone else

@GopiGugan can you please retrieve by_lineages from the database and run it through the make_beadplots function, and send me a CSV of the summary stats and predicted number of infections for each lineage?

Summary stats seem reasonable:

Scatterplot of predicted number of infections (from HUNePi model) against sample size (number of sequences):

The issue seems to be that there is a small number of outlier lineages with very high predicted numbers of infections.  For the diagnostic summary_stats.csv data that @GopiGugan sent me, the maximum predicted number is about 4.4 million, but most of the predicted numbers fall within the range of 10 to 100,000.  On a linear scale, this causes most lineages to be coloured purple/blue:

A log-transform on mapping predicted numbers of infections to the colour scale should resolve this on the front end

Ok this is fixed, tree colours look much better:


APS 736 - Build list of requests for placements from new api endpoints

Context
Jira
A new endpoint has recently been introduced that allows us to fetch Requests For Placement. We should use this where we currently list Placement Applications (applications/{applicationId}?tab=placementRequests).
Changes in this PR

In order to create the new list of Requests for Placement we need to change the way that these are rendered. We now render out all of the RfP's document to show the user more information about the Request For Placement. This may come at the cost of the user not being able to find the information they want as quickly initially.

Screenshots of UI changes
Before

After

Great stuff! Do you think it would be easier to scan the list of RfPs if the "Status" row were at the top (rather than the bottom) of each element?

@edavey yes, good idea and easy change

@vubalasu good idea

@richpjames looks good not sure is it good to sort rfps latest first

good point Bala! thank you


Support for migrating multiple schemas at once

First and simple implementation for #1039

Closing, because if we chose to implement this, it will most certainly be implemented in the core and not in a specific plugin.


Will it improve performance if I apply other stable diffusion model which is finetuned on certain content images?

As title said.

I think Stable Diffusion v2 may not be suitable for some special domains, such as medical image. The reason is that stable diffusion is not able to generate those domain-specific images. So if you use other diffusion models finetuned on certain content images, I guess🤔 the performance will be improved.

Yes. The certain content images refer general images with specific content such as human face rather than domain specific images. Is replacing weight of UNet by existing model and not finetuning LAControNet  (plug-and-play) OKay?


Update github-impersonation.md

Corrected Acceptable Use Polices and misinformation or disinformation links and redirected to Github docs.
Acceptable Use Polices --> https://docs.github.com/en/site-policy/acceptable-use-policies/github-acceptable-use-policies
misinformation or disinformation --> https://docs.github.com/en/site-policy/acceptable-use-policies/github-misinformation-and-disinformation

These were also renamed with redirects in https://github.com/github/site-policy/pull/615/files


Connect dataset dialog

This ticket will be solved in the new create dialog, one for dominating everybody.
https://github.com/CartoDB/cartodb/issues/1914


[repo] Prepare release core-1.9.0-beta.1

Note: This PR was opened automatically by the prepare release workflow.
Changes

CHANGELOG files updated for projects being released.

I noticed this PR was merged.
Post a comment with "/CreateReleaseTag" in the body if you would like me to create the release tag core-1.9.0-beta.1 for the merge commit and then trigger the package workflow.

/CreateReleaseTag

I just pushed the core-1.9.0-beta.1 tag.
The package workflow should begin momentarily.

The packages for core-1.9.0-beta.1 are now available: https://github.com/CodeBlanchOrg/opentelemetry-dotnet/actions/runs/9306967458/artifacts/1553425783.

The release core-1.9.0-beta.1 has been published and packages should be available on NuGet momentarily.
Have a nice day!


Consider making colored markers for more colors

This was mentioned in e.g. https://github.com/pointhi/leaflet-color-markers/issues/7 ; ref that.


Allow updating the callback function on an existing button instance.

Works on #243.

Pull Request Test Coverage Report for Build 2042

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 91.263%

Totals

Change from base Build 2036:
0.005%

Covered Lines:
8367

Relevant Lines:
9168

💛  - Coveralls


Back Button/Back Stack Isn't Working Correctly

I think I found a fairly significant bug relating to the back button/back navigation stack.
Start the app. In TAB_1 add a fragment to the current tab. Switch to TAB_2 and add a fragment to the current tab. Switch to TAB_1 and hit the back button. The fragment in TAB_2 is popped instead of the fragment in TAB_1. What you see is that the back button in the action bar goes away but TAB_1 still shows "Second Level". If you switch to TAB_2 you see "Tab 2 Root".
UPDATE
Looks like it's because fragment tags aren't unique, so mBaseTabActivity.getSupportFragmentManager().findFragmentByTag(currentFragmentTag) isn't returning the fragment we were trying to get.
I attempted to fix this by building unique fragment tags. I put this line inside addFragmentInTab():
String fragmentId = tabId.getUniqueTabIdName() + ':' + getCountOfFragmentsInTab(tabId) + ':' + fragment.getFragmentId();

And then replaced fragment.getFragmentId() with fragmentId. I created the getCountOfFragmentsInTab() function based on your getCountOfFragmentsInCurrentTab() function.
So far this seems to be working.

Hi!
Thanks for reporting the issue and finding the solution.
So if it works I'm glad to hear that. Anyway I'll take a look at it ASAP.
Dmitry

Hi!
You are right, when you add the same fragment (instance of same Class) to several tabs then you can have a collision. Right now when you add Fragment to Tab it will use method getFragmentId() which returns ClassName by default.
I agree There is should be a fix for such a case. I'll think over the solution, probably your one is the best here.
Note:
If you have all fragments as unique classes, this problem is gone.
Dmitry

@primehalo Your solution is quite right. I've improved it a little bit and pushed. Currently tag for the fragment in tab counting as path to it, e.g. if We have such stack Tab1 -> Fragment11 -> Fragment12 then top fragment will be pushed to FragmentManager with tag "Tab1|Fragment11|Fragment12". I think It is more readable than in your case, but the result is the same.
Thanks for raising this issue!


Method to delete all data

Given a situation where an app uses the global Paper.book() as well as some named ones, is there a way built into paper that can delete all books in one call? My situation is I've been handed a code base where the previous developer has left but there does not seem to be anyway of tracking down the different versions and keys they used to for different versions of the app.

No, Paper doesn't have the function for that. The only way to remove the old data without knowing the names for the "books" is to remove all folders in app's internal storage which has files with *.pt extension. But as I said, Paper DB doesn't have the built-in function for that.

Thanks for the response, I have found a (somewhat hacky) way of doing it by iterating through the files directory and attempting to call Paper.book(file.name).destroy() while catching exceptions.


BUG: error reported by ete3 parser when the original c3 tree has no branch length

need a new method to handle empty branch length in c3 to satisfy the fixed format of ete3

to reproduce:
from cogent3 import make_tree

tree = make_tree("(a,b,c);")
tree.get_figure()

from myproject import trs_tree
trs = trs_tree.cogent3_to_ete3()

t = trs(tree)

Traceback...
You may want to check other newick loading flags like 'format' or 'quoted_node_names'


optee: switch to optee v4.1.0 from mainline

This patch drops use of Renesas OP-TEE in favor of using vanilla OP-TEE. It uses new meta-xt-common/meta-xt-security layer which provides generic recipes for OP-TEE.
To do so we need to mask recipes from meta-renesas. Also, new recipe deploys all optee-related binaries to deploy/images/spider/optee so we need to change path in "boot_artifact" target in YAML.

Acked-by: Ruslan Shymkevych<EMAIL_ADDRESS>


Fixed double spacing in mimes.php

Found two errant double spaces in mimes.php, fixed herein.

The error reported by travis-ci seems to be coming from the job with PHP 5.4 and using pgsql DB. The actual error is coming from composer:
sh -c "composer install --dev --no-progress"
You are using the deprecated option "dev". Dev packages are installed by default now.
Loading composer repositories with package information

[Composer\Downloader\TransportException]
Peer fingerprint did not match


lane_detection_publisher.py

(CRLKAS) ma@ma-ROG-Strix:~/Desktop/Carla-Ros-Lane-Keeping-System/src/LaneDetection$ python3 lane_detection_publisher.py
Traceback (most recent call last):
File "lane_detection_publisher.py", line 38, in
from LaneDetection.models import laneNet_class
ModuleNotFoundError: No module named 'LaneDetection'
Why ModuleNotFoundError: No module named 'LaneDetection' ?
Can you tell me how to slove this problem
Thanks.

same problem. How did you solve it??

https://github.com/Erendrgnl/Carla-Ros-Lane-Keeping-System/tree/master/src/LaneDetection
download weights and copy files to models/LaneNet/weights.

I put weight files but same error occurs..


RUM-7549 Handle trace with coroutines

What does this PR do?
The main focus is to avoid sharing the same list of scopes across tracers, and add some internal logs to better diagnose issues with tracing.

Codecov Report
Attention: Patch coverage is 34.54545% with 36 lines in your changes missing coverage. Please review.

Project coverage is 70.02%. Comparing base (a8ff9aa) to head (32afc41).
Report is 40 commits behind head on develop.

Files with missing lines
Patch %
Lines

...ain/java/com/datadog/opentracing/PendingTrace.java
14.29%
30 Missing :warning:

.../src/main/java/com/datadog/opentracing/DDSpan.java
50.00%
3 Missing :warning:

...in/java/com/datadog/opentracing/DDSpanContext.java
66.67%
2 Missing :warning:

...rc/main/java/com/datadog/opentracing/DDTracer.java
83.33%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2457      +/-   ##
===========================================
- Coverage    70.10%   70.02%   -0.08%
===========================================
  Files          770      770
  Lines        28535    28566      +31
  Branches      4781     4780       -1
===========================================
- Hits         20004    20003       -1
- Misses        7185     7223      +38
+ Partials      1346     1340       -6

Files with missing lines
Coverage Δ

...entracing/scopemanager/ContextualScopeManager.java
56.76% <100.00%> (ø)

.../kotlin/com/datadog/android/trace/AndroidTracer.kt
95.35% <100.00%> (-0.65%)
:arrow_down:

...rc/main/java/com/datadog/opentracing/DDTracer.java
59.09% <83.33%> (+0.27%)
:arrow_up:

...in/java/com/datadog/opentracing/DDSpanContext.java
78.20% <66.67%> (-0.71%)
:arrow_down:

.../src/main/java/com/datadog/opentracing/DDSpan.java
49.46% <50.00%> (-1.10%)
:arrow_down:

...ain/java/com/datadog/opentracing/PendingTrace.java
57.14% <14.29%> (-13.11%)
:arrow_down:

... and 31 files with indirect coverage changes


lwm2m: Long endpoint names are truncated due to short buffer

Describe the bug
When using long endpoint names for the lwm2m client, the ep= query in the bootstrap/register URL is truncated. Increasing  CONFIG_LWM2M_RD_CLIENT_ENDPOINT_NAME_MAX_LENGTH doesn't help.
To Reproduce
Steps to reproduce the behavior:

Set CONFIG_LWM2M_RD_CLIENT_ENDPOINT_NAME_MAX_LENGTH to a big number e.g. 100
Run the lwm2m_client sample with a long endpoint name (e.g. 80 characters)
Build and run the client and observe the packets in wireshark
In the bootstrap or registration URL, ep=abc123xxx gets truncated

Expected behavior
The endpoint name can be as long as configured with CONFIG_LWM2M_RD_CLIENT_ENDPOINT_NAME_MAX_LENGTH
Impact
Long endpoint names, recommended in the lwm2m 1.0 specification 6.3.1 Endpoint Client Name, are not possible.
Additional context
To serialize the bootstrap/register URL a static 64-byte buffer is used. This silently truncates long strings.
https://github.com/zephyrproject-rtos/zephyr/blob/d8283b63ec6fe9aa1948db163923afc0594a2bde/subsys/net/lib/lwm2m/lwm2m_rd_client.c#L120
Should this buffer be increased to at least CONFIG_LWM2M_RD_CLIENT_ENDPOINT_NAME_MAX_LENGTH or something like MIN(<longest configured query string>)?

Hi, thanks for reporting. I think your proposal is legit - endpoint query string will be the longest one in most cases so it'll make sense to correlate the query_buffer size with this config value. I'll send a PR.


Fix: Escape table name in SQLite columnInfo call

Fixes #2184. Not escaping the table name could lead to errors with some table names (group, for example)

Looks like tests are failing with Node 4.x version. Tried to reproduce this locally with 4.8.4 by running just the sqlite3 tests DB='sqlite3' npm test, but the tests seemed to run fine with the sqlite3 subset. Is this a random failure or am I missing something here?

Yeah, that was just a random fail. PR seems good, I'll merge this when I see that tests are passing  :) Thank you!


20200722 mallmen updates

Fixes for:
https://github.com/RedHatOfficial/ocp4-vsphere-upi-automation/issues/37

update downloads.govc in group_vars/all.yml

https://github.com/RedHatOfficial/ocp4-vsphere-upi-automation/issues/38

update group_vars/all.yml with variable to do NetworkPolicy by default, can change to Multitenant or Subnet
add tasks to roles/common to configure isolation mode
update roles/common/tasks/install.yml to check for isolationMode other than NetworkPolicy before calling extra tasks

https://github.com/RedHatOfficial/ocp4-vsphere-upi-automation/issues/39

add new roles/finish_install to wait for bootstrap node to boot, wait for bootstrap to finish, approve csrs, wait for install to finish
update main yml task files to add this role to end

https://github.com/RedHatOfficial/ocp4-vsphere-upi-automation/issues/40

update group_vars/all.yml with cpu/ram on bootstrap/master/worker nodes
update roles/dhcp_pxe/tasks/main.yml to use variables instead of hardcoded values

re-commited pr after merging lastest updates from master


Move Feature Requests from CFN resource coverage repo

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Tell us about your request
Someone should move all of the issues in the CFN resource coverage repo that are CFN language specific from that repo to this one.
Tell us about the problem you are trying to solve. What are you trying to do, and why is it hard?
What outcome are you trying to achieve, ultimately, and why is it hard/impossible to do right now?
What is the impact of not having this problem solved? The more details you can provide, the better we will be able to understand and solve the problem.
Those issues belong in this repo (some were probably prematurely closed since they weren’t resource coverage issues)
Are you currently working around this issue?
How are you currently solving this problem?
DMing jlhood on Slack
What is the expect behavior with this new feature
Please include some code examples showing how this feature can be used.
N/A
Additional context
Anything else we should know?
Attachments
If you think you might have additional information that you wouldd like to include via an attachment,
please do - we will take a look. (Remember to remove any personally-identifiable information.)
N/A

@richardhboyd Yep, we're working on it! 😊

Done


Deployment to Heroku failed

Hi there, I'm deploying my telegram bot to Heroku but it failed with the following errors: Error R10 (Boot timeout) -> Web process failed to bind to $PORT within 60 seconds of launch.
Looks like the port is not configured correctly. It was running fine on my local though. Any help guys?

you need to run worker on heroku, not web
by the way - why did you close issue?


pkg/vdso: find vdso file from absolute path

this will fix find vdso file when running in docker.
and may be this will fix https://github.com/parca-dev/parca-agent/issues/1543

@v-thakkar cc

This is deprecated after recent changes. Please feel free to report if it doesn't work as you expected. Thanks a lot for your contribution.


Worker deregister button is broken

Currently on master branch, the settings -> worker -> deregister worker button does not work.

Oh My 😮

I'll take care of this. Also, fix this vue warning while I'm there.
vue.runtime.esm.js?2b0e:619 [Vue warn]: You may have an infinite update loop in a component render function.

found in

---> <VueGoodTable>
       <ManageWorker>
         <TabPane> at node_modules/vue-bulma-tabs/src/TabPane.vue
           <Tabs> at node_modules/vue-bulma-tabs/src/Tabs.vue
             <Index> at src/views/settings/index.vue
               <AppMain> at src/components/layout/AppMain.vue
                 <Root>

Oh, just saw you assigned this one already. :) Okay, have fun. :)

@michelvocks Are you also taking care of the hot reload problem?

@Skarlso You can overtake that. Currently busy with the docker pipeline execution.

Cool. Will do. Thanks.
On Fri, 20 Sep 2019, 15:44 Michel Vocks<EMAIL_ADDRESS>wrote:

@Skarlso https://github.com/Skarlso You can overtake that. Currently
busy with the docker pipeline execution.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gaia-pipeline/gaia/issues/212?email_source=notifications&email_token=AABMUQQ7T6NOUKVVAN3GBVDQKTHSLA5CNFSM4IU2JWE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7GXR4Y#issuecomment-533559539,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABMUQTLC6R724YKQMQLUOLQKTHSLANCNFSM4IU2JWEQ
.

 But I think I know what the problem is.
Funnily enough, it's related to the above vue error. Basically, the periodic check causes a race on the data, an endless update loop. Because there is no comparison before assigning the data just clobbering it over again and again, which causes an update which overwrites the data, which causes an update... etc.
And the deregister is not working because said loop is overwriting the deregister action. I think. I tried JSON.stringify compare, but it only solved the issue temporarily and in a very ugly way. :)
I'll investigate further.

Found the problem. We do this sort of thing in all of the front-end stuff but what's different here is the timestamp. The different timestamp causes an update feedback loop to the render because the data changes constantly.
I'll come up with something to that. Probably something like, removing that datapoint from the comparison. ¯_(ツ)_/¯

Uh, or not. I removed for now that completely, and it still seems to complain about that. WTH....


MGMT-12867 Add a signature to nsenter failures

Not all nsemter command will cause the agnet to fail command execution, This signature will help to catch those cases
an example:
level=info msg="failed executing nsenter [--target 1 --cgroup --mount --ipc --pid -- mdadm --stop /dev/md127]
the level of this command is info and it will not cause the agnet to fail the step execution but it did cause an issue with the installation later

/hold
not sure how to test it

not sure how to test it

tools/add_triage_signature.py can run this check in dry run mode against a particular triage issue (-i) or recent (30 days) (-r)

See CONTRIBUTING.md for more details on signature testing / development

Yes I agree, this should not be "nsenter" specific. We use nsenter when we want to escape the container into the host's namespaces, but that's just an implementation detail that could theoretically change

My reference was https://issues.redhat.com/browse/AITRIAGE-4689
Looking at the ticket again i can see that step signature does cover this case.
looking at the existing signature i couldn't see the failure this is why it looked like it wasn't covered.
Closing this PR


Add support for airco with integrated air purifier

The air quality sensor data and air purifier feature of the airco unit is not available.


Add e2e tests to 13-mongo-typeorm with flush from TypeOrm import instead of api call

[x] duplicate e2e tests
[x] replace truncate api call by direct drop call from on TypeOrm
[x] handle DI

https://github.com/typeorm/typeorm/issues/5732
https://stackoverflow.com/questions/16493902/truncate-a-collection

<EMAIL_ADDRESS>

Done


CLI: Update README

Let's create the first version of the README for the CLI (PR against cli branch)
Main ideas:

Change install methods (npx instead of cloning).
Add foundry instructions (where it says hardhat / hardhat.config, etc)

First version of CLI and Foundry Readme already merged into cli branch


Scaffolding updates: Inserts required package references in the MvcMovie.csproj project file. i cant find where the .csproj file is

[Enter feedback here]

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

ID: 1d975a83-1550-b284-d0d8-7fe2ea206ff5
Version Independent ID: 48e81b42-7c2b-afbf-71e6-d4f442a4bd1f
Content: Part 4, add a model to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/adding-model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

... but having said that, using NuGet Pacakge Manager, also a right-click menu item on the project, is the preferred approach for VS users to deal with packages.


Add Behat and Docker-based Development Environment

This PR adds the "simplest possible" Behat suite and configuration to get us started. For browser-based testing, we're using PhantomJS+Selenium, provided by a Docker container. Docker!? Yes, this PR also adds a .docker directory that includes a basic development environment for those that choose to use it.
I couple the two together because configuring Behat and any headless-browser through Selenium isn't always for the faint-of-heart. Having this provided via Docker means that if a contributor can run Docker, they can run our test suite. I think that's valuable.
This has been tested on Ubuntu 17.10, 16.04 and CentOS 7.4 using Docker CE, provided by https://docs.docker.com/engine/installation. I'm using Docker version 17.09.0-ce at the moment.
Left to do...

[ ] Additional documentation for Docker where there are gaps
[ ] Additional documentation for Behat
[ ] Run Behat on TravisCI
[ ] Verify Mac OS X / Windows support for Docker environment
[ ] node is version v9.3.0 in workspace. It should match what's in TravisCI. Either change TravisCI version or ours.
[ ] Install mysql-client on workspace container.
[ ] Resolve test run-time issue with db container.

There are some open questions I have, but will need to add them to this PR later. Need to run.
Related: #927

@mdwheele Why was this PR closed?
I am stumbled over this while looking for a docker based OpenCFP.

@andygrunwald
Mostly, it was due to lack of time and honestly, too much in one PR. There's currently an open PR to wrap up Behat tests separate from Docker. I took the work I did and created https://github.com/mdwheele/opencfp-docker. I've had quite a few folks pull that down and give it a whirl, but I've been holding off sending a PR back upstream until I feel like I have consensus that it is "good enough". If you want to take it for a spin and let me know how it goes for your deployment, I'll put together a PR to add it here.

Faire enough.
Iowas asking, because i was missing the context here.
But I agree with your arguments.
I will give it a try based on your docker work (docker only, not the behat
part). Not sure when, but I try to do it in the next days. Can’t promise
anything.
Thanks in advance for your work.
Dustin Wheeler<EMAIL_ADDRESS>schrieb am Fr. 30. März 2018 um
18:53:

@andygrunwald https://github.com/andygrunwald
Mostly, it was due to lack of time and honestly, too much in one PR.
There's currently an open PR to wrap up Behat tests separate from Docker. I
took the work I did and created https://github.com/mdwheele/opencfp-docker.
I've had quite a few folks pull that down and give it a whirl, but I've
been holding off sending a PR back upstream until I feel like I have
consensus that it is "good enough". If you want to take it for a spin and
let me know how it goes for your deployment, I'll put together a PR to add
it here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencfp/opencfp/pull/928#issuecomment-377569004, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATiQA3EMuJi_v3CEEmCsucbILlq2noDks5tjmMkgaJpZM4RDaYt
.


add month range calendar #161

Add upper and lower date limits.

I found a bug.
After correction, resend the request.


Add support for http caching and keep-alive

[ ] Client that the user can keep around

[ ] For native, this would wrap ureq::Agent, and use http-cache-semantics to handle caching
[ ] For web, it would delegate everything to fetch (which already handles caching), and hold on state of its own

I'm not 100% but walkers currently doesn't seem to have disk-based tile caching, so everything gets reloaded every launch of the app. It would be great to have that.
BIG CAVEAT: some tile providers (possibly including MapBox) may have restrictions in how a client is allowed to give access uses to the raw data, or the maximum amount of map data that may be cached, or other such things designed to (legally) prevent scraping.

This is very similar to:

https://github.com/emilk/ehttp/issues/4

I'm not 100% but walkers currently doesn't seem to have disk-based tile caching

It has, you can enable it by setting a path where you want to store it:
https://docs.rs/walkers/latest/walkers/struct.HttpOptions.html#structfield.cache

BIG CAVEAT

Yes, there are plenty, at least on OSM servers: https://github.com/podusowski/walkers/issues/104


Disabling ClockPicker when the field is readable

Did make a change on my side for this:
if (this.input.is('[readonly]')==true) {
return;
}
Now works better, in my case.


DataModel.load() failed - status: 500

I use kong, konga, docker and Cassandra database in my project. First, in keyspace kong, I insert 8 records into table kong.apis using INSERT queries. Next, I access http://localhost:1337/#!/apis, and see a list of apis successfully because the table kong.apis contain only 8 records.  After that, I insert extra 1 record into kong.apis also using insert query, and then I access http://localhost:1337/#!/apis again but I see no apis.

error_with_9_rows_inserted.txt
If I add a new api through clicking on "ADD NEW API" button on http://localhost:1337/#!/apis, it is successful.
In general, once there are more than 8 records added into kong.apis using INSERT query, and reload the page, error happens and no api display.

@scbsoccer , that is strange. What happens if you try to fetch the apis directly from Kongs admin API via postman or any other REST client?

Closing as outdated. Feel free to reopen if needed.


Add PREVAIL (eBPF verifier) to the infrastructure page

Signed-off-by: Elazar Gershuni<EMAIL_ADDRESS>

Just one comment to make it consistent with the other entries. I'm not sure why the image looks the way it does in the deploy preview, it should just be blank

Can you squash it down to one commit?

Can you squash it down to one commit?

FYI, github allows the person doing the merge to do a squash without the PR owner having to do it.  (The merge button has a downdown arrow with multiple options which include "squash and merge").

@elazarg can you pull in https://github.com/ebpf-io/ebpf.io-website/pull/288 so I can make sure the preview looks ok?

Thanks for adding @elazarg !


#2925 add indexes to mongodb collections

Closes #2925
So far the results seem promising. I executed the same test as described in #2812, and the response time pretty much stays constant at a median of 100ms, and 95th percentile of 200ms:

Signed-off-by: Florian Bacher<EMAIL_ADDRESS>

Codecov Report

Merging #2955 (663df2b) into master (da5038a) will decrease coverage by 13.95%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #2955       +/-   ##
===========================================
- Coverage   36.25%   22.29%   -13.96%
===========================================
  Files         104        1      -103
  Lines        6728      471     -6257
  Branches      217        0      -217
===========================================
- Hits         2439      105     -2334
+ Misses       3963      357     -3606
+ Partials      326        9      -317

Flag
Coverage Δ

moduleA
?

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

Impacted Files
Coverage Δ

mongodb-datastore/handlers/events.go
22.29% <0.00%> (-1.10%)
:arrow_down:

bridge/client/app/_models/service.ts

bridge/client/app/_services/clipboard.service.ts

helm-service/pkg/helm/helmv3_executor.go

bridge/client/app/_models/event-labels.ts

bridge/client/app/_models/event-icons.ts

helm-service/controller/mock_handler.go

...ponents/ktb-event-item/ktb-event-item.component.ts

.../configurationchanger/canary_weight_manipulator.go

api/handlers/event.go

... and 94 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da5038a...c0258bb. Read the comment docs.

Codecov Report

Merging #2955 (663df2b) into master (da5038a) will decrease coverage by 13.95%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #2955       +/-   ##
===========================================
- Coverage   36.25%   22.29%   -13.96%
===========================================
  Files         104        1      -103
  Lines        6728      471     -6257
  Branches      217        0      -217
===========================================
- Hits         2439      105     -2334
+ Misses       3963      357     -3606
+ Partials      326        9      -317

Flag
Coverage Δ

moduleA
?

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

Impacted Files
Coverage Δ

mongodb-datastore/handlers/events.go
22.29% <0.00%> (-1.10%)
:arrow_down:

bridge/client/app/_models/service.ts

bridge/client/app/_services/clipboard.service.ts

helm-service/pkg/helm/helmv3_executor.go

bridge/client/app/_models/event-labels.ts

bridge/client/app/_models/event-icons.ts

helm-service/controller/mock_handler.go

...ponents/ktb-event-item/ktb-event-item.component.ts

.../configurationchanger/canary_weight_manipulator.go

api/handlers/event.go

... and 94 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da5038a...c0258bb. Read the comment docs.


small fix of optimizer

why duplicate it?

I use separate SGDParam and SGDMomParam to make the docs being generated properly.


clear away the glitch in password input

Fixes #
cleared the glitch in password input section.
Describe the changes you have made in this PR -
added a margin and solid colored border to make input section visible.
Screenshots of the changes (If any) -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.711% when pulling 744121bc313be751a6196793417fb144ae7c27e5 on prashant601:master into f0c04dd16522d0fe2deb678c37a3a1401d8e1471 on CircuitVerse:master.


Import icons directly.

Purpose
Found two stray icon index imports which increased three chunks by 80k each.

Coverage remained the same at 27.737% when pulling 8fc6d7eb5cd933f839c3e8846361a1e8ee55b838 on rjstires:icon-import-fixes into a323a164ae88598fa2be8605e8f902b2467f14a2 on linode:develop.


Bump version to <IP_ADDRESS>

https://github.com/apache/pulsar/pull/13066 introduced a breaking change to MockZooKeeper API and causes test failure. https://github.com/streamnative/kop/pull/951 fixed this bug and this PR is to verify it.
Since the change only affects tests, this PR won't cause the breaking change.

There is a problem with #951, I'll rebase to master once the fix has been merged to branch-2.8.1.


updated roadmap because of incorrect WinUI date

Docs Build status updates of commit 23bf43b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/roadmap.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[SPARK-GREENPLUM-3]Add transaction support for copy a partition's data to green plum #3

For that gp's copyIn is atomic, I will close this pr.


Made changes in Navbar and Footer

Related Issue
Improved CSS of Navbar and Footer
Closes: #39
Are you creating this PR under Hacktoberfest
Yes, I'm opening this PR under hacktoberfest. (Please remove this section if you are not a hacktoberfest participant)
Describe the changes you've made
Changes in Navbar and Footer.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.

Screenshots
Put any screenshot(s) of the project here.

@yashikabhargava, could you please reply to why it is showing conflicting files? I'm not able to get this.

@yashikabhargava, could you please reply to why it is showing conflicting files? I'm not able to get this.

I'm sorry for the late reply, the conflicts were there because your fork was not the original one.

@Rashmisingh-18 can you please move all the script that you have added in index.html to main.js

@Rashmisingh-18 This PR has a lot of conflicts.


JiraStopWatch cannot connect to a jira server.

JiraStopWatch v.1.9.0. cannot connect to a Jira-Server (JIRA V7.2.1). Version 1.8.0 has no problems to connect.
Following two lines are output in the jirastopwatch.log:

16.02.2017 07:54:01 Request: https://jira.******.net/rest/api/2/filter/favourite
16.02.2017 07:54:01 Response: OK - []

Version 1.8.0:

Version 1.9.0:

Could you verify the following:
Click on the gear-icon and check that your base-url is correct
Click on the padlock-icon and check that your username/password is correct
I will need the entire logfile, in order to figure out, what is wrong.

My Bad. I was clicking on the red "Not connected" link. It turns out this is different from the padlock button.


Small headsup

Hi,
whenever a new Source 1/2 replay parser comes out, I read their code to find out how they did stuff.
Especially some comments in the send table parsing code made me smile.
Tough stuff, I know.
I think my codebase (https://github.com/skadistats/clarity) has a quite nice implementation, so, if you missed it before, feel free to have a look.
Good work anyway! Keep it on!

Hey,
Yeah I've been checking your parser every now and then to see if you are done with CS2. Nice to see that you got it working!
I originally picked manta due to me being more familiar with GO, but looking back, I'm not so sure it was the correct choice.
I especially like this function 🤯
public Field getFieldForFieldPath(S2FieldPath fp) {
            switch (fp.last()) {
            case 0: return field.getChild(fp.get(0));
            case 1: return field.getChild(fp.get(0)).getChild(fp.get(1));
            case 2: return field.getChild(fp.get(0)).getChild(fp.get(1)).getChild(fp.get(2));
            case 3: return field.getChild(fp.get(0)).getChild(fp.get(1)).getChild(fp.get(2)).getChild(fp.get(3));
            case 4: return field.getChild(fp.get(0)).getChild(fp.get(1)).getChild(fp.get(2)).getChild(fp.get(3)).getChild(fp.get(4));
            case 5: return field.getChild(fp.get(0)).getChild(fp.get(1)).getChild(fp.get(2)).getChild(fp.get(3)).getChild(fp.get(4)).getChild(fp.get(5));
            case 6: return field.getChild(fp.get(0)).getChild(fp.get(1)).getChild(fp.get(2)).getChild(fp.get(3)).getChild(fp.get(4)).getChild(fp.get(5)).getChild(fp.get(6));
            default: throw new UnsupportedOperationException();
        }
    }

This is actually very interesting, because I thought this idea wasn't possible as I thought that I'm forced to to branching down the line but seems like its just due to how my fields/traversing is set up.
I kind of gave up understanding the sendtable/serializer/field logic at some point but this gave me new hope. I'll definitely take a deeper look at how you made it, as I cringe every time I have to deal with the current code.

The function you mention is my try to implement this the fastest way possible.
There is a class Field:
https://github.com/skadistats/clarity/blob/master/src/main/java/skadistats/clarity/io/s2/Field.java
Implementations:
https://github.com/skadistats/clarity/tree/master/src/main/java/skadistats/clarity/io/s2/field
The code that converts the sendtables into a nested structure of fields is here
https://github.com/skadistats/clarity/blob/master/src/main/java/skadistats/clarity/processor/sendtables/FieldGenerator.java
The code went through several revisions since Dota Source 2 first came out in 2017 (iirc), and it is hack free (apart from one single one I put in for CS2).

After some poking around, I would like to implement "fields" the way you did it if you don't mind (of course with a mention where the idea came from). The implementation is sooo much more understandable.

I don't mind at all!
Good luck, and if you have questions, don't hesitate to ask.

Thanks


Support PlaceholderAPI in command rewards

What does your feature do?
Add support for PAPI placeholders in command rewards
How would people use it?
Standard PAPI syntax. Lots of people in the Discord ask about placeholders for commands, it would be nice to just be able to tell them anything from PAPI works
https://github.com/PikaMug/Quests/blob/master/main/src/main/java/me/blackvein/quests/Quest.java#L452

Use https://ci.codemc.org/job/PikaMug/job/Quests/14/


Interpolation is computed twice

Due to #1464, interpolant coefficients for piecewise polynomial interpolators are computed twice if node sensitivity is requested.

Is this a problem, or just inefficient? There is no way to get node sensitivity later as far as I can see.

Changed the label. This is inefficient especially when doing calibration with e.g.,CubicSplineInterpolator where the node sensitivity computation is requested every step, then the linear system is solved twice.
For several piecewise polynomial interpolators, determining coefficients are not necessarily required to calculate coefficient sensitivities, while these two share some variables, e.g., intervals, gradients.


Your system does not support this UnifiedNlp package

I have a Chinese Android watch (Android 4.4.4 API 19 - not Android Wear) on which I'm trying to install the UnifiedNlp package.
I keep getting the error "Your system does not support this UnifiedNlp package. Either install a matching package or a compatibility Xposed module." when self-checking. Alongside 2 other errors:
"The system did not bind the UnifiedNlp service. If you just installed UnifiedNlp you should try to reboot this device." (device was rebooted) and
"You either disabled network-based location (in system settings) or the system is not supported." (location is enabled and on "High Accuracy")
I tried all different versions (NetworkLocation.apk, LegacyNetworkLocation.apk, and UnifiedNlp.apk), all in vain. No version worked.
Also, installing the standalone UnifiedNlp or the one that comes witch microG does make a difference.
Am I missing something?
If the system is not supported: why? What is the cause for a system not to be supported?

Chinese ROMs usually come with another location service that conflict.
If you want an easier way I have created an installer that automatically remove conflicting packages and works perfectly in 99% of cases: microG unofficial installer.

Thanks a lot.
I already came across your installer, but since I cannot flash anything on the device and there is no custom recovery for it, I did not try it.
Is there any other way to use your installer without flashing?

FlashFire allow to flash a zip inside Android itself.
Some people reported it works but I haven't tried it.

Thanks, I am still trying to root the device. As soon as I have tried flashing your installer I will report back.

Thanks a lot for you help. I managed to install GmsCore correctly!

Hi, ale5000!
I've used your installer, but have exactly same errors. Can you, please, advice me direction to dig.
It's android 5.1 (lenovo z90a40). Stock rom with gapps, which was replaced with opengapps, after was mostly removed, and after was your "microG-unofficial-installer-ale5000-v1.0.21-beta-signed.zip" flashfired.

XposedGmsCoreUnifiedNlp works :)

@difr: Cuold you please post a screenshot of the microG self check and a copy of /system/framework/framework-res.apk?

@ale5000-git
Sorry, can't provide ss now, it's not my phone. But it was exactly the same
"Your system does not support this UnifiedNlp package. Either install a matching package or a compatibility Xposed module."
"The system did not bind the UnifiedNlp service. If you just installed UnifiedNlp you should try to reboot this device."
Which are disappeared after "XposedGmsCoreUnifiedNlp" start to work.
And here is "framework-res.apk"
framework-res.apk.zip

@difr
Apparently your framework-res.apk has some dependancies and I'm not able to decompile it (it is the first time that I'm seeing this).
Can you please post the entire framework folder so I can decompile it and see the problem?

here it is
https://github.com/difr/temp/blob/master/framework.zip

@ale5000-git
here it is
https://github.com/difr/temp/blob/master/framework.zip

My Acer B1-A71's 6577T has an xml for specifying a location service in /etc called location_service_conf.xml. Apparently you can put gms there pointing to the core but this still doesn't work for me though now it says it's registered. Original service provider is Mediatek, but deleting it doesn't make difference. Framework-res.apk also misses lines and putting there any doesn't make any difference also.
What should I do?


Convenience features

A couple things I noticed that would be great to have:

CSS tweak - the background is hard to read for some people. My dad can't read the text well (the presentation appears too dark to him). So the option to a traditional black/white theme would be nice.  I also found myself to often change the background style manually to be less hard on the eyes.
Disable auto-scrolling if the user scrolls above the last line and continue auto scrolling if the user is at the bottom again.

this allows to start reading the response while it writes. GPT4/Bing can really write a lot and for quite a while.

Enable the prompt field while the AI writes, just disable sending until ready. So we can already prepare the followup and just need to send it once ready.
The STOP button is glitchy (bing mode), a couple times it deleted my prompt alongside the half-finished AI answer.
Initial prompt modification support - In Jailbreak the system in normal mode the first prompt - it would be nice to modify that.
This would also allow multi-lingual support so a non english speaker could just update the initial prompt.

Replied language prompt maybe add to the options.

I didn't notice that, though it doesn't invalidate the usefulness of modification to the prompt.
From setting the tone to specializing it on a purpose (like a programming language). A lot that can be done or improved.


Fix Backpressure Task Notification In DiskManager

Ref #82

Looks like a race condition caused one of the tests to fail. This was because we are sending messages back from the executor to the manager, and only after we send do we decrement our completed counter.
If the manager checks if it can submit more work after it gets the message (via try_submit_work), but before we decrement our completed counter (via complete_work), then the task will never get notified.
The fix for this is to unconditionally notify the task, though in the future, we should only do this for certain messages (those that actually cause a counter decrement),  but for now its probably fine, since two atomic operations should be pretty fast for a single call to poll.

Closes #82


Kanban Board Setup

https://github.com/Ridwanullahi-code/royal-meals-website/projects/1
We are just only two students

Hi  TEam 👋🏾 ,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are still some which aren't been addressed yet.

currently, on your kanban board tasks are not yet assigned to team members as required 👍🏾   when a task has been assigned You would see a circle with the user's avatar on each card.

your kanban board

required kanban

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Status: Approved :tada: :heavy_check_mark:
Hi @team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

Kanban board set up properly. :heavy_check_mark:
Tasks assigned correctly. :heavy_check_mark:

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

[OPTIONAL] It is required to state in your PR summary whose student A and student B are.
Ex: Student A = @Ridwanullahi-code, Student B = @kennankole.
Kindly include that in your PR Summary.


Fix/macro audit 8 part 2

All test-related and some smaller nit change requests will be looked at in future codebase refactoring PRs.


Problem with fs.writeFile

This appears to be caused by the fact that fs module is mutated by promisify('fs'). Internally, fs.writeFile uses some other fs methods in a callback fashion, so it expects fs to be un-modified.
Repro:
const promisify = require('promisify-node');
const fs = promisify('fs');
fs.writeFile('./testfile.txt', 'Hello World')
  .then(() => {
    console.log('File saved!');
  })
  .catch((error) => {
    console.error('Error doing something:', error);
  });

Neither 'File saved!' or 'Error doing something' ever gets logged. The file gets created but no data is written to it. Tested on OS X 10.11 using Node v5.

Just hit this too. Tested on Ubuntu 14.04 using Node v5. Pretty unfortunate :(

Actually, to add to this, because it mutates the module then all other files that require this module are affected. This cause cause some serious havoc if not rectified. For example, by promisifying fs it essentially breaks express since it internally uses fs to serve static content.

I get this too:
ohnobinki@gibby ~/require-semver $ node -e 'require("fs").writeFile("a", "b\n", () => console.log("done"))'; cat a
done
b
ohnobinki@gibby ~/require-semver $ node -e 'require("promisify-node")(require("fs")).writeFile("a", "b\n").then(() => console.log("done"))'; cat a
ohnobinki@gibby ~/require-semver $

Interestingly, if I don’t invoke require('fs') myself but let promisify-node do that for me, it works as expected:
ohnobinki@gibby ~/require-semver $ node -e 'require("promisify-node")("fs").writeFile("a", "b\n").then(() => console.log("done"))'; cat a
done
b

So there’s at least a workaround. I have no idea why it’s behaving this way. I tried adding to promisify-node’s testsuite and couldn’t reproduce the behavior inside the testsuite, even when supposedly deleting 'fs' from require.cache and using promisify(require('fs')) rather than promisify('fs').
Oh, but it looks like when you pass a string and have promisify() call require() for you, it automatically behaves as if you specified noMutate as true. And somehow specifying noMutate works around this bug. So this also works:
ohnobinki@gibby ~/require-semver $ node -e 'require("promisify-node")(require("fs"), undefined, true).writeFile("a", "b\n").then(() => console.log("done"))'; cat a
done
b


File input in Android Crosswalk Webview

When using a standard html file input the change event is not fired. This worked fine on the desktop app and also in the Chrome WebView on Android 4.4.4, but not the crosswalk WebView. My code:
HTML: https://github.com/whiteout-io/mail-html5/blob/master/src/tpl/write.html#L35
JS: https://github.com/whiteout-io/mail-html5/blob/master/src/js/controller/write.js#L631
Thanks

Still got the same problem: @mmocny
"cordova-plugin-crosswalk-webview": {
"XWALK_VERSION": "23+",
"XWALK_LITEVERSION": "xwalk_core_library_canary:17+",
"XWALK_COMMANDLINE": "--disable-pull-to-refresh-effect",
"XWALK_MODE": "embedded",
"XWALK_MULTIPLEAPK": "true"
},


feat(chaos): Bump duration of experiments

This PR addresses a timeout in the chaos step of the CI/CD pipeline where the discourse charm does not become healthy in the specified duration (default of 60 seconds). A local test confirms the charm becomes stable, so with this change, the total duration of the chaos experiments is increased, so that the test can (hopefully) pass.
With fixing the pipeline run, some additional steps were needed:

there is now a cleanup of S3 related variables in the integration test for S3
I had to add dependencies for the linting to pass
Functions in the unit and integration tests are refactored in order for pylint to pass
Some pylint issues are disabled, please have a look and supply feedback
I added a trivy.yaml config fail to ensure trivy doesn't break the pipeline and created an issue to address trivy findings
I added a workflow action to sync Github issues to our Jira (so that the issue above lands in Jira as well)

Unittests are failing due to the absence of pylint in the tox env

It keeps getting longer...

and unfortunately still wouldn't pass - on a local setup it passes in 9min, however in CI I'm not sure how much it would take :/

It keeps getting longer...

and unfortunately still wouldn't pass - on a local setup it passes in 9min, however in CI I'm not sure how much it would take :/

May I suggest to use the debugging shell and run from the debugging shell the experiment, it'll allow you to check what's hapenning, and what's taking that long

It keeps getting longer...

and unfortunately still wouldn't pass - on a local setup it passes in 9min, however in CI I'm not sure how much it would take :/

May I suggest to use the debugging shell and run from the debugging shell the experiment, it'll allow you to check what's hapenning, and what's taking that long

That's a great suggestion, I did that previously when looking at an issue (wrong parsing of time) and it doesn't provide much insight as to why Litmus is failing. The thing I will do is simulate the test in pytest with a test that kills the discourse pod and waits for it to be alive, capturing the state and logs from the pod, that is something I'm missing in Litmus. This is getting a real 'private eye' vibe.

well at least using the shell you can see if it's really the pod taking really long to reboot using kubectl / juju commands

well at least using the shell you can see if it's really the pod taking really long to reboot using kubectl / juju commands

TIL about that - thanks a bunch!

I'm converting this to draft due to the CI/CD issue that needs more attention - I can't seem to reproduce it locally with or without pytest, although I think I'm following all steps from the pipeline to the extent possible. Stay tuned.

Few linting issues needs to be solved and I think we'll be good with this one, isn't it ?

Few linting issues needs to be solved and I think we'll be good with this one, isn't it ?

Well - I can address the linting issues, however these are not my main concern. I'll see what I can do here in terms of ensuring the integration tests pass and then the chaos tests as well. What I found out is that enabling S3 on the S3 integration test causes a subsequent migration to break - I think that is expected as we're using localstack and are just interested in object creation, however recompiling assets in the rake job would like to access actual objects (that's my understanding). I'm trying to do a cleanup and setting S3 to false in the app config settings during the S3 test and hope the chaos test will pass. I'm not sure at this point whether I should do something else to have Discourse know S3 should not be used (might need to unset an ENV variable, don't know yet whether set_config does that for me, might need to do a migrate_from_s3 rake task as described here https://meta.discourse.org/t/how-to-turn-off-s3-storage/160338/6). With the latter we need to think about testing actual migrations, how to enable that and how to test it - migrate to S3, migrate from S3, what to do when access to S3 is lost etc.

I'm not a big fan of leaving so many disabled line-too-long, wasn't there a better approach for these ?

I'm not a big fan of leaving so many disabled line-too-long, wasn't there a better approach for these ?

I did try finding a middle ground between black and pylint, did not succeed. To me it is trial and error - bend a line somewhere, see whether black bends it elsewhere, watch pylint complain again. There are 8 ignores, I'd leave this as an issue in the backlog with a good-first-issue label for someone to tackle. I'm too old for this.

This looks weirdly similar to another PR: #33

Yes, they are indeed similar - this PR here is the base one and I split it in 4 different PRs with cherry picks - one about Jira sync, one about Trivy config, one about linter issues and that one here related to the Chaos tests. This would be the last PR to merge and with syncing from main, only the chaos related changes will be here.

I'm not a big fan of leaving so many disabled line-too-long, wasn't there a better approach for these ?

Well @jdkandersson is also not a big fan of line-too-long and left a bunch of great feedback on the feat/linter PR, so if you have a look at that, you'll see there are no more line-too-long disables :) Thanks for commenting!

58 comments and 46 commits later 😉


Fix assert in dartdoc_checker and enable asserts for our helper scripts

Description
See title, dartdoc itself still runs with asserts disabled.
Related Issues
Fixes https://github.com/flutter/flutter/issues/64103.
Tests
I added the following tests:
n/a
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

/cc @srawlins


Add ASCII parser

I'd like to work on a parser for ASCII formatted (human readable) Go diagrams.
I propose we call this format ASCII, rather than Sensei's format, forum format or something else, because its use predates Sensei's Library (e.g. in newsgroups, email, old Go websites, command line Go programs etc). There doesn't seem to be any reference to ASCII in the existing codebase for Glift, so I don't envision any conflicts at the moment.
 2 .  4 .  7 .  6 .  3 .  2 .  4 . . . . |
$$ | . . . , . . . . . , . . . . . , 1 .  7 .  6 .  3 .  The dollar signs have limited meaning in our case, so we can easily strip them out.
One of the existing issues with these diagrams on sites like Sensei's Library is that move numbers in diagrams can only go up to 10 (represented by 0) after which you need to continue with a new diagram, where the previous moves have been flattened.
My experience is that this can be very frustrating and limiting, so I suggest that our implementation should support move numbers up to three digits in length (or maybe of arbitrary length) with (optional) zero padding to maintain a nice, human readable alignment.

so I suggest that our implementation should support move numbers up to three digits in length (or maybe of arbitrary length) with (optional) zero padding to maintain a nice, human readable alignment.

I don't think our goal should be to reinvent an ASCII format, just to parse what currently exists.
There is one big trickiness with ascii formats:  How big is the board? it's a little tricky, especially if the user has chosen to crop the board in some non-trivial way.

Ok, how about I start simple for now and see whether it's easy to add the nice to haves later?
The move number limitation 'bug' has been annoying me for about a decade now, so it would bring me great personal satisfaction if there was an opportunity to squish it without much extra work.

Anyway, I take your

The move number limitation 'bug' has been annoying me for about a decade now, so it would bring me great personal satisfaction if there was an opportunity to squish it without much extra work.

It's a separation of concerns.  This task is for parsing.  If we wish to invent a new ASCII format, that should be a separate bug (although I don't recommend it).

Fair enough, thanks.
On 14/02/2015 5:27 PM, "Josh Hoak"<EMAIL_ADDRESS>wrote:

The move number limitation 'bug' has been annoying me for about a decade
now, so it would bring me great personal satisfaction if there was an
opportunity to squish it without much extra work.
It's a separation of concerns. This task is for parsing. If we wish to
invent a new ASCII format, that should be a separate bug (although I don't
recommend it).
—
Reply to this email directly or view it on GitHub
https://github.com/Kashomon/glift/issues/114#issuecomment-74364054.

Realistically, unless there's a compelling reason to have this, I won't be working on this. If a compelling reason does come up, feel free to reopen.


Support static linking of simulator

This doesn't seem to resolve the issue, will reopen when I have a working solution.


make buttoninfo an embed

This should be a ClosableEmbed menu / ViewState, so it just needs its own View

I'll do this if fluff doesn't want it


after parameter with getOverview/fetchAll doesn't work.

overview = await reddit.getUser(user).getOverview({after: last_id}).fetchAll();
fetches the whole history even though last_id is the fullname  of the latest post by the user.

I just tried this out and I found it works as intended. This fetches overview of a user, then fetches more from the user (in this case only 1 instead of infinite).
await r.getUser('SpyTec13').getOverview({after: 't1_f3c11ld'}).fetchMore({amount: 1})


WSL2 distro start failed with 0x80070057 on build 21390

Windows Build Number
Microsoft Windows [Version 10.0.21390.1]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
Ubuntu-20.04
Other Software
No response
Repro Steps

Enable WSL2 from msft, everything seems OK.
https://docs.microsoft.com/en-us/windows/wsl/install-win10

Update kernel to the lastest version: wsl --update

Set WSL default version to 2

Restart Windows and try to install ubuntu-20.04:

Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070057
Error: 0x80070057 The parameter is incorrect.

Press any key to continue...

Additional information:

WSL 1 app start successfully.
Disable GUI(guiapplications=false in .wslconfig) dosn't work for me.  See #6839 for more details.

Expected Behavior
The Ubuntu-20.04 should work.
Actual Behavior
WSL2 report  0x80070057 The parameter is incorrect.
Diagnostic Logs
network_diag_info.zip
wsl_logs.zip

Thank you for providing the logs! We took a look and saw that this was networking related and are looking into this further.

Thank you for reporting this @wangdan1600.
Can you share the output of:
Get-WindowsOptionalFeature -Online | ? State -eq Enabled | select FeatureName

In an elevated powershell prompt ?
Also, does the issue still reproduce after a reboot ?

Thank you for reporting this @wangdan1600.
Can you share the output of:
Get-WindowsOptionalFeature -Online | ? State -eq Enabled | select FeatureName

In an elevated powershell prompt ?
Also, does the issue still reproduce after a reboot ?

I'm sorry, I reinstall win10 OS last week, and the WSL2 run normally.
Before I open this issue, I tried enable/disable/reenable VirtualMachinePlatform feature many times, and reboot many times, and none of these methods worked.
If you couldn't find any usable information from the logs,  I think you can close this issue.
Thanks, BR.

Not starting WSL2 on Windows 11 22000.51 DEV Build

@adarsha2003 can you please share the output of running this command in an elevated powershell prompt?
Get-WindowsOptionalFeature -Online | ? State -eq Enabled | select FeatureName
As well does this still happen after a reboot? And can you please include networking logs?

@craigloewen-msft Sorry now i cant share logs. Now i have reinstalled windows 11, now WSL2 is working fine. I think the problem is due to because i have upgraded many times using dev channel. There could be problem in one of the windows 10 upgrades.
Thanks for you support

I'm seeing a similar issue, the etl trace shows a networking issue while creating the network for WSL2. The output for the Get-FeatureName command above is
Printing-PrintToPDFServices-Features
Printing-XPSServices-Features
SearchEngine-Client-Package
WorkFolders-Client
Printing-Foundation-Features
Printing-Foundation-InternetPrinting-Client
MSRDC-Infrastructure
VirtualMachinePlatform
Microsoft-Windows-Subsystem-Linux
MicrosoftWindowsPowerShellV2Root
MicrosoftWindowsPowerShellV2
NetFx3
WCF-Services45
WCF-TCP-PortSharing45
NetFx4-AdvSrvs
SmbDirect
MediaPlayback
WindowsMediaPlayer

I'm seeing a similar issue on Windows 10 21H2 19044.1415. The output of above command is:
PS C:\WINDOWS\system32> Get-WindowsOptionalFeature -Online | ? State -eq Enabled | select FeatureName

FeatureName
-----------
Printing-PrintToPDFServices-Features
Printing-XPSServices-Features
WCF-Services45
WCF-TCP-PortSharing45
MediaPlayback
WindowsMediaPlayer
SmbDirect
MSRDC-Infrastructure
NetFx4-AdvSrvs
SearchEngine-Client-Package
MicrosoftWindowsPowerShellV2Root
MicrosoftWindowsPowerShellV2
WorkFolders-Client
Internet-Explorer-Optional-amd64
Printing-Foundation-Features
Printing-Foundation-InternetPrinting-Client
Microsoft-Windows-Subsystem-Linux
HypervisorPlatform
VirtualMachinePlatform
Containers-DisposableClientVM
Microsoft-Hyper-V-All
Microsoft-Hyper-V
Microsoft-Hyper-V-Tools-All
Microsoft-Hyper-V-Management-PowerShell
Microsoft-Hyper-V-Hypervisor
Microsoft-Hyper-V-Services
Microsoft-Hyper-V-Management-Clients

I sent logs through Feedback Hub (they reference this issue in the title, I did not find the share link). A strange thing is happening with Hyper-V Virtual Switches:

in practice, every time I try to run/install a Linux image a new WSL Switch is created.
I have only one Ethernet adapter and is already associated with the Bridge Switch.
I also removed the Bridge Switch and retried to run Ubuntu, but the error is still the same.
Hope this helps.

Still happen on Windows 11 when I'm trying to install Ubuntu 22.04 LTS from Microsoft store.
I ran the command above and got this output:
FeatureName
-----------
Printing-PrintToPDFServices-Features
Printing-XPSServices-Features
WCF-Services45
WCF-TCP-PortSharing45
MediaPlayback
WindowsMediaPlayer
MicrosoftWindowsPowerShellV2Root
MicrosoftWindowsPowerShellV2
WorkFolders-Client
SearchEngine-Client-Package
Printing-Foundation-Features
Printing-Foundation-InternetPrinting-Client
MSRDC-Infrastructure
Microsoft-Windows-Subsystem-Linux
NetFx4-AdvSrvs

/logs

WslLogs-2022-05-25_20-44-47.zip
Same issue, wsl 2 with ubuntu 22.04, here is the log file.

https://docs.microsoft.com/en-us/windows/wsl/install-manual#step-4---download-the-linux-kernel-update-package
Linux kernel update package is needed.Seems like it works


chore(harness): crystal 1.1.1 instead of 1.0.0

Update default driver test harness crystal version to 1.1.1
Tested briefly OK (ubuntu 20.04 in WSL2, Win10) with ./harness up
and compiled a driver that used Time .step (new in 1.1.0), which was not compiling in 1.0.0

One thing to double check is that core is on 1.1.1 as well


Update ecflow version to 5.11.4 (latest)

Summary
This PR contains an update to the ecflow version from 5.8.4 to 5.11.4. This is needed to enable building ecflow on the macOS using the new<EMAIL_ADDRESS>compilers.
Testing
I have tested this manually on my Mac and verified that ecflow builds successfully. I also verified that using ecflow 5.84 doesn't work, which is why the version bump to 5.11.4 is necessary.
Applications affected
This is only a change to the ecflow version, and my understanding is that JEDI is the only app using ecflow. Please correct me if this is wrong.
Systems affected
macOS and HPC systems
Dependencies

[ ] waiting on jcsda/spack/pull/370

Issue(s) addressed
Resolves #877
Checklist

[x] This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
[ ] These changes have been tested on the affected systems and applications.
[ ] All dependency PRs/issues have been resolved and this PR can be merged.

@srherbener I merged the spack companion PR, please update this PR. Thanks!

@srherbener I merged the spack companion PR, please update this PR. Thanks!

Thanks! This PR is updated now.


empty page strenching up and down

Info:

OS: Win11 22H2 (22621.521)
Logseq ver: 0.7.1
Theme ver: v2.5.3
[x] ⚠ Do i have issue on SolExt theme with DISABLED my custom.css? No never edited that.
[x] ⚠ Do i have the same on default theme? Not when disabled this plugin, but plugin + default theme will not fix this.

Description:
Opening today's journal page, and it will blink like this

No other page exhibits such behavior. Seems related to #today-queries.

it's Logseq lazy load conflicting somehow...
issue was fixed partially, but still exists  ((
related to:  https://github.com/yoyurec/logseq-solarized-extended-theme/issues/46

@Tantalus13A98B5F does it fixed in the latest version AwesomeStyler?

Thank you for asking! With AwesomeStyler, it's fixed. However, with AwesomeUI, it's back.

hmm.... strange. AwesomeUI uses layout positioning which do issue, but it was redone.
will try to fix, but first i need to reproduce...
so i need to open Journals, then click today item. it should no have templated text and backlinks, right?


⚠ Msite has degraded performance

In d37842c, Msite ($MSITE_URL) experienced degraded performance:

HTTP code: 200
Response time: 5199 ms

Resolved: Msite performance has improved in e0984a7.


Adding a .vsconfig to make pre-reqs easier to discover [-ci skip]

When opening this repo I had some build errors on missing components.  This .vsconfig file makes those opening in VS discover those missing components in advance with a link to install them quickly within VS...making a smoother clone and load experience for the repo.
Affects VS only, no source code changes, no tests needed, etc. Just a helper for those using VS.

Thanks @Pomianowski -- FYI you don't need the "if" conditions in your Actions workflows anymore as [skip ci] and others work natively now on GitHub Actions: https://github.blog/changelog/2021-02-08-github-actions-skip-pull-request-and-push-workflows-with-skip-ci/


Error when running this local

I get the following error when trying to execute this locally:
RangeError: Maximum call stack size exceeded

Update the packages:
"svelte": "^3.46.4",
"svelte-loader": "3.1.2",
"svelte-preprocess": "^4.10.2",


Output does not match TF implementation

First, I want to thank you for doing this! There are a lot of little pieces to get right and the results so far are pretty amazing.
That said, images are quite different from the Tensorflow implementation. I'm interested in using this with points found with Ganbreeder, which means that I want to match the TF version as closely as possible. I think it's all using cuDNN underneath right? If so, I imagine we could get this closer aligned.
My main question is how to modify the Tensorflow graph to output the intermediate calculations so we can compare activations and figure out which layer(s) are causing the biggest error. I would be happy you with this.
 Will redo matching to see if it corresponds better. Sorry for the noise. **


Feature/native js support

Description

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test

Checklist

[ ] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 20 (40.0%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.1%) to 30.744%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/nativeJS/NativeJSUtils.tsx
3
4
75.0%

src/pages/playground/Playground.tsx
2
13
15.38%

Files with Coverage Reduction
New Missed Lines
%

src/commons/utils/DialogHelper.tsx
1
23.68%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
3523

Relevant Lines:
10652

💛  - Coveralls

I think it's OK now


Reply with notice on some commands

Hi
instead of reply in query with msg.sendPrivate
a command for sending a notice would be nice to have!
its already in node irc:
Client.notice(target, message)
Sends a notice to the specified target.
Arguments:

    target (string) – is either a nickname, or a channel.
    message (string) – the message to send as a notice to the target.

same with:
HUBOT_IRC_SEND_NOTICE_MODE set it to true
but the bot sends now all output to channel instead to the user who triggered the event or command
[17:40:21] [+Jaglag]: !bitcoin in eur
[17:40:23] -Butlerz@#xREL- EUR: 215.38 (H: 215.67 | L: 215.11)
Is there any workaround or something i can do?

It sounds like you need a way to send a notice to a user that does not go to a channel. I am curious why sendPrivate doesn't allow your use-case – perhaps you mean sending a private message to a user without being in the context of a message handler?

Hi parkr,
/notice message
Is a different kind of sending a message in irc.
Private message open a private chat and the user have to klick on that private message window to read it.
Normal channel message is a bit annoying when large response.
Notice is the only way to don't annoying ppl in chat with large response and the comfort to don't switch to a different window.

Hi @Jaglag, this comment that uses robot.adapter.command should work for you if you replace MODE with notice.
If it doesn't, feel free to comment back, and I'll re-open.


css network training data

Hi, could you provide the whole dataset(8k patches image_rgb and image_uvw) for css network training? There are only 5 examples in the 'data/db/crops'. Thanks a lot.

Hi there!
Unfortunately we are not able to release the training dataset since it was generated from a proprietary dataset provided by Parallel Domain. A possible alternative would be to use CARLA to extract patches and poses from there. Then using the respective car models and poses, one car render NOCS (e.g. using nocs-renderer).

Hi there!
Unfortunately we are not able to release the training dataset since it was generated from a proprietary dataset provided by Parallel Domain. A possible alternative would be to use CARLA to extract patches and poses from there. Then using the respective car models and poses, one car render NOCS (e.g. using nocs-renderer).

Hi Sergey;
How do you generate a json file similar to crops.json that you have in the repository for training.
More specifically, starting from CARLA and your nocs-renderer, how to populate:
latent
intrinsics
scale
in the json file?
The nocs images generated by nocs-renderer look like grayscale images, while sdflabel takes colored nocs images, how to generate those?


Fonts appear lighter on Windows browsers than on Mac.

I loaded the site on Windows browsers for the first time, and noticed that fonts appear much lighter than they do on Mac. Screenshots below:
Safari 12 on Mac

Chrome 75 on Windows

Firefox 68 on Windows

Not a bug. That’s all due to the differences in font rendering between platforms. On Practical Typography, I serve slightly lighter versions of my fonts to Mac users to adjust for the heavier rendering.

^ in depends on the note from @mbutterick we should close this issue, it is not a bug, the web-browser, the OS and the results are different.


clean up ElementUtils and TypeUtils interfaces

ElementUtils class should expose the same methods as Java 7 Elements interface and delegate to it. This would avoid having to use processingEnv.getProcessingEnvironment().getElementUtils() and keep to processingEnv.getElementUtils()
Note : ElementUtils can not extends the Elements directly since new methods can be added in later versions of Java (isFunctionalInterface(TypeElement) added in Java 8)

done


Parsing URIs in optional fields causes unnecessary failures

It would be nice if we could tell the parser to ignore invalid data in optional properties, particularly things like member:image where a value might be there that isn't a parseable URI. Totally fine if those simply get treated like they aren't present.

How would this help you? Did you often receive invalid data from a third parties in production?

Actually it came up for us trying to use a test/sample service stood up by an LTI Consumer we're working on an integration. with. In this case we were able to work around it by moving to a different version of their service, but the reality is the spec does not say those have to be valid URLs; we're fine with just ignoring them (possibly even in production) if a consumer sends invalid data in those fields.
In general, since we have no control over what a consumer sends, also having no control over how it's parsed (made more difficult since the parser wants to read the request stream directly meaning we can't modify it) puts us in a rather helpless position as users of both a service we can't control and a client we can't control.

I think you're talking about conditional property (de)serialization, it's not so trivial especially coz it introduces more complexity and you should take control over all properties and show proper message in case of error (required is missing, parsing error, etc.).
Here you can find a simple example There are several other places where you can inject this logic, here is my experiments in the same area (note that I implemented opposite logic, I skipped property serialization depending on some conditions).
So far so good, IMHO the easies way will be a create a custom build of this lib with the functionality you need OR re-throw the exception here passing the response body, so the caller will be able to handle its as needed.
Hope it helps 🤞

PR https://github.com/LtiLibrary/LtiLibrary/pull/104 was merged in 2018 so closing this now. Thanks @andyfmiller 🥇


Custom directories specified in Setup Wizard silently ignored when given a folder on a different drive.

Running Windows 10 Pro.
First time install. Setup Wizard appears. Clicking "Next". Suggested to use directories in C:\Users\Will\AppData\Roaming\.... Because my C:\ drive is getting pretty full, I opted to create an equivalent folder on a larger E:\ drive at E:\Programs\AppData\Roaming\ and changed the corresponding paths. After finishing the setup process, I installed the latest Godot version from GitHub, but the various files were all installed to the C:\ drive rather than the E:\ drive.
If installing to an alternate drive is not supported by Godot for some reason, then we should open a bug ticket (if there isn't one already) and add an error message to the setup wizard that disallows paths of that nature. Otherwise, Godot-Manager should save to the directory specified by the user.
Furthermore, I would recommend that validation of whether or not it can save to the given directories is performed when the user attempts to move to the next step of the setup wizard. That way the app does not just silently fallback to do it the default way without informing the user.
Edit: Hmmm. I wonder if this happened because the folder I added was marked as "Hidden"? Since it doesn't show up in the file dialog if you try to navigate there manually one folder at a time. Seems like if I enter "AppData" into the path in the FileDialog, it'll navigate me there regardless, so that works fine.
Edit 2: It appears that if you modify the path after the setup wizard completes, the icon to "save changes" or "undo changes" to the settings only appears if you've clicked on the "Browse" button to open a file dialog and selected a path that way. If you just manually change the path text in the LineEdit, there is no GUI update that allows you to save those changes, nor any indication that direct path changes for other LineEdits will be saved if you click "Browse" only for one file path. Like, I can't tell, from the GUI alone, whether it is going to persist whatever changes I make.

Ah, just checked the console I ran Godot-Manager from. Lots of errors.
Mono: Log file is: 'C:/Users/Will/AppData/Roaming/Godot-Manager/mono/mono_logs/2024-03-23_15.15.17_3384.log'
ERROR: Attempted to convert an unmarshallable managed type to Variant. Name: 'Task`1' Encoding: 21.
   at: (modules/mono/mono_gd/gd_mono_marshal.cpp:1124)
ERROR: Index p_current = 4 is out of bounds (get_tab_count() = 4).
   at: set_current_tab (scene/gui/tab_container.cpp:593)
Downloaded 7782 bytes
Downloaded 8047 bytes
Downloaded 7968 bytes
Downloaded 7891 bytes
Downloaded 1 bytes
Downloaded 7782 bytes
Downloaded 23907 bytes

Then a bunch of repetitions of the following error message:
ERROR: Error calling method from signal 'text_changed': 'Panel(SettingsPanel.cs)::OnGodotInstallLocation': Method not found..
at: emit_signal (core/object.cpp:1242)

Sorry, was working on catching up the Bug List after getting some more free time from my focus on IRL work.  This has been an issue, and I couldn't track it down, till now.  Thanks to the Error message, I now know why it isn't properly updating.  I will get this fixed shortly.


Haralick feature extraction

Refactoring ComputeHaralickFeatures.py

Calculates 26 Haralick features from an intensity image with the labels.

Uses graycomatrixext to compute gray-level co-occurence matrix.

Refactored version of haralick feature extraction

_ init is updated

@slee172 Fixed the bug in graycomatrixext and merged it. You can use it in your function now.

@cdeepakroy  Do we still use checking parameters on graycomatrixext.py? For example, ComputeHaralickFeatures.py check the offsets and graycomatrixext.py also check it again.

You don't have to check it again in ComputeHaralickFeatures.py. Let graycomatrixext.py do it.
Also, copy the docstrings for the parameters you want to expose from graycomatrixext

@cdeepakroy  I think graycomatrixext.py can be used for other files, so how about capitalizing the first name?

@slee172 Do you mean graycomatrixext ----> Graycomatrixext?
Next, we are going to rename all functions to snake case (lower_case) to follow python guidelines. graycomatrixext already is in that format. So let us maybe leave it that way?

@cdeepakroy Updated the haralick feature extraction.

Empty lines are added and parameters of grapycomatrixext are set for haralick feature extraction. In addition, order-related numbers will be modified soon.

@cdeepakroy Updated the ComputeHaralickFeatures.py modifying order-numbers to strings. Local test passed.

@slee172 Minor comments. Rest LGTM!
Now think of how to write a unit test for this.