[Explore]Change Dataset search can't get back to default list view after filtering

Once a user inputs copy to filter the list of datasources, the list successfully filters dynamically but they can't get back to the default state that shows all datasets.
Expected results
Once the user deletes the input, the list should automatically show all datasets again.
Actual results
The user is stuck with their first filter input and can't redo their search/filter.
Screenshots

How to reproduce the bug

Go to 'Explore'
Click on the more icon next to the dataset name
Filter the list by typing a dataset name
Stuck (try to see the full list again).

it's resolved already by chore: Show datasets when search input is empty
@Steejay thanks for filing this issue, still need to speed up a bit to catch up with the changes in the repo 😉

it's resolved already by chore: Show datasets when search input is empty
@Steejay thanks for filing this issue, still need to speed up a bit to catch up with the changes in the repo 😉


Documentation cleanup, post-gogradle build changes

This is some docs. cleanup of typos and other spec. changes/mixups that were found while fixing gogradle.
Also, sought to incorporate proposed change from PR https://github.com/apache/openwhisk-wskdeploy/pull/1094 which for some reason did not build at the time it was submitted

Closes https://github.com/apache/openwhisk-wskdeploy/pull/1094


Youtube-DLG

Checklist

[ ] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2021.12.17
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE

My Youtube-DLG cannot download video since like this


MultipartEncodedDataProcessor shouldn't read all bytes into memory

I quickly browsed source code of the MultipartEncodedDataProcessor class of the project. I found that it reads all bytes to memory then write them to request body once. This isn't the best way to use multipart/form-data. It should read one block of bytes  and send it to server one by one.
copied from open-feign

Yes, you are right, I will take a look on it ASAP

When I use ‘feign-form-spring’ and 'feign-form' in my springcloud project.I found that I could not use@Requestboby annotation for Map,User  like below:
public String postBobyAParamAPathVar(     		@RequestBody(required=true) Map<String, Object> map,     		@RequestParam(value="userName") String userName,     		@PathVariable(value="id") String id){ ....... }

So, this is a limitation of Feign library. Close issue


Optimize the slice_string function

对slice_string函数进行优化：
1.支持负数索引
2.可以指定步长，正数或负数
3.自动调整下标范围到边界

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


Add a comprehensive and deterministic data generator

The current data generator is simplistic and is just a proof of concept that the exporter works.
Will need to add the correct one in.
Data generation criteria:

3 node cluster (forces remote IActorRef serialization this way) all launched inside same process.
Should launch ~100 remembered entities
Each entity should store:

Primitive types - string, int - and these should be tagged using an IWriteEventAdapter
Custom types serialized using the default object serializer (i.e. Hyperion or Newtonsoft - doesn’t matter); these also need to be tagged, but inside the actor application code itself (i.e. literally wrapping the message in a Tagged object before calling Persist)
Custom types serialized using a custom serializer - these should also be tagged via either an adapter or inside the actor class.

Tags

Some message types must have only a single tag (edge case)
Some should have no tags
Some should have more than 1 tag
All of the above scenarios need to be exercised.

This should be complete now, need someone to look over the code and make sure that it satisfies all the requirements.


Make this work with Ubuntu 18

A few small changes to make this work with Ubuntu 18

Hi @catskillmarina,
I've make a quick review of your PR, did you still need this functionality ?
If yes, we should discuss about how rework your PR :)


user.rs:75-79: Create NodeId object that will give MD5...

The puzzle 137-e669be66 from #137 has to be resolved:
https://github.com/h1alexbel/fakehub/blob/5551f71edb028eafb18cbe2a5659099d5dfe666e/server/src/objects/user.rs#L75-L79
The puzzle was created by @rultor on 03-Sep-24.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd Main quality problem: Lack of clear description of the issue. Suggestion: Add a concise summar...

I see you're talking to me, but I can't reply since I'm not a chat bot.


Failing restore at "Getting materialized views to exclude"

Hello,
It is a few days I'm trying to troubleshoot this issue and if anybody could help that would be great!
I'm trying to restore a cassandra backup stored in S3 with the following command:
cain restore --src s3://xxxx/xxxx/database/cassandra -n my_namespace --keyspace my_keyspace -t<PHONE_NUMBER>0007 -l role=cassandra
2019/09/23 14:00:53 Restore started!
2019/09/23 14:00:53 Getting clients
2019/09/23 14:00:53 Getting pods
2019/09/23 14:00:53 Testing existence of data dir
2019/09/23 14:00:53 Getting current schema
2019/09/23 14:00:54 Found schema: 43fa34
2019/09/23 14:00:54 Calculating paths. This may take a while...
2019/09/23 14:01:20 Validating pods match restore
2019/09/23 14:01:20 Getting materialized views to exclude
2019/09/23 14:01:20 error in Stream: command terminated with exit code 2
Any idea what's wrong?

This is happening because I'm using an old version of cassandra which doesn't support materialized views


panic: runtime error: invalid memory address or nil pointer dereference

I tried to upgrade to 3.4 (windows_x64 exe) when trying to get around #42  and I hit another issue:
mark.exe -f .\things.md -u XXXX -p XXXX -b https://atom.atlassian.net/wiki/
(same command works with 3.3.1 and produces #42)
.md file (same as other issue)
<!-- Space: DD -->
<!-- Parent: On-boarding docs -->
<!-- Parent: Systems Development -->
<!-- Parent: Application -->
<!-- Parent: BuildTools -->
<!-- Title: Repositories using BuildTools -->

<!-- Macro: :autogen:
     Template: ac:status
     Title: AUTOMATED
     Color: Yellow -->

:autogen:
this document is generated, do not edit manually.

----

## Build Tools Type: Cake

blah blah blha cake is good
blah blah

Here's some useful links:
 - [How does Cake BuildTools work](www.google.com)

### Repositories:

#### [NotMyRepo](https://github.com/NotMyOrg/OrMyRepo)
```Some Repository Desc from github api```

(the google link is temporary, but i'm tempted to leave it as a troll, maybe i should use lmjgtfy.com)
trace log:
2020-12-02 18:22:14 [TRACE]: loaded macro "@\\{([^}]+)\\}"
                             ├─ template: ac:link:user
                             ├─ expr: @\{([^}]+)\}
                             └─ config:
                                     Name: ${1}
2020-12-02 18:22:14 [TRACE]: loaded macro ":autogen:\r"
                             ├─ template: ac:status
                             ├─ expr: :autogen:
                             └─ config:
                                     Title: AUTOMATED
                                     Color: Yellow
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x20 pc=0x7c91ad]

goroutine 1 [running]:
github.com/kovetskiy/mark/pkg/mark.ResolveRelativeLinks(0xc00021ff60, 0xc00017c400, 0xb78, 0xc00, 0x892214, 0x1, 0xc00, 0x0, 0x0, 0xc00, ...)
        /home/runner/work/mark/mark/pkg/mark/link.go:33 +0x9d
main.main()
        /home/runner/work/mark/mark/main.go:216 +0x9e2

any pointers (pun not intended) 😄

I am having the same issue

Hey, I'm looking into that

Hey, just fixed the issue, please check v3.5
The issue was introduced in #33

Yup that's workin 👍


Web: cannot open a folder (Windows)

Looks like the parsing of folder/workspace path on Windows from the URL is not working properly.

Credits to @dbaeumer who found this in stable already 👍


Enhancement: also highlight FIXME and XXX

FIXME and XXX are both highlighted in Vim's traditional syntax highlighting, along with TODO, can we add them to match what Vim does?

Hi, this is already supported out of the box with nvim-treesitter
https://github.com/nvim-treesitter/nvim-treesitter/blob/8ec9a50965ccd34d504db86f09c580edfcff0c0f/queries/comment/highlights.scm#L18-L19
https://github.com/nvim-treesitter/nvim-treesitter/blob/8ec9a50965ccd34d504db86f09c580edfcff0c0f/queries/comment/highlights.scm#L36-L37
You are probably not using the comment parser, make sure you have it installed.


Rendering widgets in the Dashboard

What's new
Rendering widgets in the Dashboard
This is just a test to see how to register the widgets into the widget manage without having a notebook model.
Fixed
#103
Todo/issues
To register the widgets we need an IContext<INotebookModel> and an IRenderMimeRegistry
Notes

Sorry, I don't have too much time to work on this anymore.


Add a new column ClientId to the database

This adds a new column with an id of a client via which a compose request was sent.
The logic is similar to the insertion of an image name. Frontend sends a request which contains a new value of ClientId set to "ui". Next the value is parsed to appear in a new column of the database.
Frontend part https://github.com/RedHatInsights/image-builder-frontend/pull/1426

/retest

This should hopefully do the trick. Got tangled in the pointers and types a bit.

The integration test needs a tweak, since the client_id has a default value now.

/retest

/retest

I fixed it in iqe plugin but we the MR/PR sync doesn't work without the env var I'm adding and I'm also trying to fix the api.sh test that's running in Schutzbot but I'm not sure if that's going to work, if not I'll do some manual testing to figure it out.

Ok so Schutzbot is fixed an I'm releasing a new iqe plugin, should be green in < 1h ... The sync is broken because of some cert issues in the Gitlab pipeline, I'll have to investigate later but the api update doesn't break testing on stage/prod so it was fine to merge.

/retest


Timber method get_widgets - FATAL ERROR

I have problem with new Timber with Twig 2.0. When i try get widgets with code:
$this->context['widget_footer'] = Timber::get_widgets('widget-footer');
$this->context['widget_phone'] = Timber::get_widgets('widget-phone');

i got error:
 Fatal error: Uncaught LogicException: Function "dynamic_sidebar" is already registered. in /home/dawidryba/projekty/praca/sowdent.dev/wp-content/themes/krframe/krFrame/vendor/twig/twig/lib/Twig/Extension/Staging.php:30 Stack trace: #0 /home/dawidryba/projekty/praca/sowdent.dev/wp-content/themes/krframe/krFrame/vendor/twig/twig/lib/Twig/ExtensionSet.php(162): Twig_Extension_Staging->addFunction(Object(Twig_SimpleFunction)) #1 /home/dawidryba/projekty/praca/sowdent.dev/wp-content/themes/krframe/krFrame/vendor/twig/twig/lib/Twig/Environment.php(807): Twig_ExtensionSet->addFunction(Object(Twig_SimpleFunction)) #2 /home/dawidryba/projekty/praca/sowdent.dev/wp-content/themes/krframe/krFrame/vendor/timber/timber/lib/FunctionWrapper.php(70): Twig_Environment->addFunction(Object(Twig_SimpleFunction)) #3 /home/dawidryba/projekty/praca/sowdent.dev/wp-includes/class-wp-hook.php(298): Timber\FunctionWrapper->add_to_twig(Object(Twig_Environment)) #4 /home/dawidryba/projekty/praca/sowdent.dev/wp-includes/plugin.php(203): WP_Hook->apply_ in /home/dawidryba/projekty/praca/sowdent.dev/wp-content/themes/krframe/krFrame/vendor/twig/twig/lib/Twig/Extension/Staging.php on line 30

If i comment one get_widgets then work fine.
I check method get_widgets before update and difference is:
OLD:
public static function get_widgets( $widget_id ) {
      return trim(Helper::function_wrapper('dynamic_sidebar', array($widget_id), true));
}

NEW:
public static function get_widgets( $widget_id ) {
		$output = new FunctionWrapper( 'dynamic_sidebar', array( $widget_id ), true );
		return trim( $output );
}

I've got this problem as well. Not knowing much about this my impression is that  the dynamic_sidebar function is getting created each time the get_widgets call is run and it's getting created in some shared scope.
My work around is to catch the original dynamic_sidebar function with an output buffer:
public function getSidebar($id) {

    ob_start();

    dynamic_sidebar( $id );

    $html = ob_get_contents();

    ob_end_clean();

    return $html;

  }

  public function context($context) {

    $context['posts'] = Timber::get_posts();
    $context['pagination'] = Timber::get_pagination();

    $context['headerWidgets'] = $this->getSidebar(WidgetArea\HomeHeaderWidgetArea::$id);
    $context['bodyWidgets'] = $this->getSidebar(WidgetArea\HomeBodyWidgetArea::$id);

    return $context;

  }

I can confirm this error too.

Until this is fixed, you can try to use the following:
$this->context['widget_footer'] = Timber\Helper::ob_function( 'dynamic_sidebar', array( 'widget-footer' ) );
$this->context['widget_phone'] = Timber\Helper::ob_function( 'dynamic_sidebar', array( 'widget-phone' ) );

What @danimbrogno came up with as a solution is actually already built into Timber. Helper::ob_function() is thought for functions that normally echo output. It catches the output and returns it, so that you can add it to context.

Closed by #1422


Issue installing

npm WARN package.json<EMAIL_ADDRESS>scripts['server'] should probably be scripts['start'].
npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! Invalid name: "@angular2-material/card"
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/softvar/angular2-webpack-starter/npm-debug.log

I can npm install @angular2-material/card just fine. This is likely a problem with your environment.

I'm able to install other packages.
Have tried it with node versions: 0.12.7, 4.0.0 and 5.0.0 and corresponding npm versions.
I cloned https://github.com/AngularClass/angular2-webpack-starter and ran npm install and got this error. The project seems very promising in learning Angular2. I'm stuck right now. Would you be willing to help me in resolving this?


Add Subscription section

Add pages about subscriptions, plans, payment options. This section needs to cover details about user subscription and payment options. Also cancelling plan and upgrading plans.

Explain the pricing model to user:

ree trial with limited usage and slow machine
paid plans with more usage
special plan for GPU-powered containers
add-ons for even more usage or even more AI

@marko-baric-ca suggestion we have info about plans how to upgrade and pay, but not details about the plans or prices themselves

@ivan-burazin @marko-baric-ca
This is one of the options without displaying the prices.
Each plan would then further be described in its own section regarding the CPU, memory, usage,... and their available add-on packages.
Let me know what you think and whether this should be changed to include the prices of each subscription plan.

@stefanicjuraj i would remove the plans themself from the docs.

Yes, let's remove the plan details from the docs. We can just generally explain that plans define how moch power and usage time you get from your environments. But the exact plans and their prices can be left out.

@stefanicjuraj lmk when updated


Is vsdbg available for win-arm64?

Is vsdbg available for Windows Arm64? If so how do you install it?

To make sure I understand your question, are you asking because you are trying to remote debug to Windows ARM64?
If so, you can use the following command (make sure to replace <folder-to-install-to>) --
powershell -NoProfile -ExecutionPolicy RemoteSigned -Command "[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12; &([scriptblock]::Create((Invoke-WebRequest -useb 'https://aka.ms/getvsdbgps1')))  -Version latest -RuntimeID win10-arm64 -InstallPath <folder-to-install-to>"

@gregg-miskelly
That's perfect thank you!
I was attempting to use "win-arm64" which was failing and didn't think to try "win10-arm64".


Remove anonymous function calls. Remove default exports from.

backported from #4662

There's an acceptance test which failed; trying it again


feat(startup): remove unused variable

as the macos 'ssh-agent' works without issues,
there is no need for runnig ssh-agent before the command

On linux environments this variable was empty, therefore had no effect. Meaning, +1 to merge based on a Fedora linux execution environment.
@iamkirkbater can you confirm it still works as planned on macos? Can you still use your ssh-agent/keyring/whatever to connect outbound with this change?

/LGTM


Parse remote refs

Hi @skryukov.
I want to make Full support for external $refs for your skooma gem(see this point in the Feature plans).

Hey, @killondark, thanks for the PR!
The explicit requirement for JSONSkooma::Sources::Remote is intentional, as accessing external resources can be insecure.
For skooma, the bootstrap code might look something like this:
skooma = Skooma::RSpec[Rails.root.join("docs", "openapi.yml")]

skooma.schema.registry.add_source(
  "https://raw.githubusercontent.com/username/test_yml/",
  JSONSkooma::Sources::Remote.new("main"),
)

config.include skooma, type: :request

So, I think we have everything in place for this particular case. What do you think?


Improve formatter

Format only changed files if any
Use gofumpt as formatter

Codecov Report

Merging #1472 (2e1b103) into master (fa61371) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1472      +/-   ##
==========================================
- Coverage   39.52%   39.36%   -0.17%
==========================================
  Files         598      590       -8
  Lines       35068    34956     -112
==========================================
- Hits        13862    13760     -102
+ Misses      19678    19666      -12
- Partials     1528     1530       +2

Impacted Files
Coverage Δ

app/proxyman/outbound/errors.generated.go
0.00% <0.00%> (-100.00%)
:arrow_down:

common/task/task.go
88.88% <0.00%> (-7.41%)
:arrow_down:

testing/servers/tcp/tcp.go
81.81% <0.00%> (-5.46%)
:arrow_down:

testing/scenarios/common.go
73.07% <0.00%> (-4.62%)
:arrow_down:

app/proxyman/outbound/handler.go
38.05% <0.00%> (-4.48%)
:arrow_down:

transport/internet/udp/hub.go
73.61% <0.00%> (-4.17%)
:arrow_down:

transport/internet/kcp/dialer.go
80.39% <0.00%> (-3.93%)
:arrow_down:

transport/internet/system_listener.go
48.07% <0.00%> (-3.85%)
:arrow_down:

transport/internet/kcp/connection.go
69.61% <0.00%> (-2.77%)
:arrow_down:

transport/internet/quic/hub.go
72.00% <0.00%> (-2.67%)
:arrow_down:

... and 13 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 fa61371...2e1b103. Read the comment docs.

I remember that gofumpt will mess up file v2ray-core/main/distro/all/all.go


controlsd: alert on cruise mismatch for >1s

fixes #22780

This creates an annoying issue on Toyotas whereby if the cruise is cancelled by the brakes while the car is in standstill, it throws an alert.
https://youtube.com/shorts/A9rEyBFc_-g


Frontend: Buy Token

Updated the Token page to have options to buy token.
the UI looks at the moment looks like this:

closes issue: #226

Hey @0xChqrles,
Please let me know if any more change is required, I will be happy to do the changes and push it to the branch.


It should be mentioned that AppLayout as designed to be a root layout

AppLayout is designed to be a root element. While it's mentioned in client-side documentation https://cdn.vaadin.com/vaadin-app-layout/2.0.4/#/elements/vaadin-app-layout, it's completely omitted from a server-side.

For best results, the component should be added to the root level of your application (i.e., as a direct child of ).

Now adding it as a child makes NavBar rendered at the top of the page, not its parent element. (which causes frustration, at the very least)

Moving this to the docs repository. Let's take this into account when creating demos for AppLayout.


fix: Index out of range in examples/credit-card-form when ccn is empty

Fixes Index out of range error in examples/credit-card-form when credit card number(CCN) is not provided.
To reproduce:

Run examples/credit-card-form/main.go
Do not type anything in "Card number"
Press UP or DOWN keys on your keboard (or press TAB several times)

ccnValidator will trigger and will attempt to read a symbol which is located at len("")-1, that will cause Index out of range error (because CCN is empty).

Thanks so much!


Cleanup server helper

Add error handling
Reduce memory allocation
Reuse built-in buffer

@mururu Could you check this?


Supported go versions for installation (goenv 1.23.3)

I have installed goenv using brew on my mac.
goenv shows that I have got version 1.23.3 installed.
goenv install --list only shows till go version 1.12beta1.
Since there are have been a lot of go versions released after 1.12, is there a way that we can update the list to include all the newer versions of go?

I got the 2.0.0 beta installed using the following
brew uninstall goenv
brew install --HEAD goenv
I am able to see the latest go versions after this.


JsonField

Dear @volkanceylan
Please consider to add a new field type called JsonField
To work with JSON strings stored in the database.
Currently, I am using StringField and parsing the JSON to a certain type.
It will be nice to have JsonField which will handle these parse and stringify things.
FYI
https://docs.microsoft.com/en-us/sql/relational-databases/json/json-data-sql-server?view=sql-server-2017

@dfaruque which library (js/ts) are you using to display json content in form ?

@kilroyFR I have used Serenity's PropertyGrid and slightly customized GridEditor.ts

@dfaruque ,
it's probably not the full blown solution you are suggesting - but could this here help solving your issue? https://radekmaziarka.pl/2018/01/22/dapper-json-type-custom-mapper/
John

OK, nice idea, i'll consider this, just can't do it specific to SQL server, but agree it should handle serialize / deserialize

Thanks, I have mentioned the SQL server for example. Many database engines have such a feature nowadays.

BTW, handling serialize/deserialize will be enough.

There is a JsonField type now.

How I can use JsonField type to write data to postgres database to column with type 'json'?


RMSE Calculation problems

Hi,thanks for sharing the elegant code,The code is very easy to reproduce, and I also get results similar to those in your paper.
But I find a Calculation problem In line 290 of your code
you used test_ rmse +=rmse_ Loss. item() / len (test_loader) to calculate test RMSE.
I wonder maybe this calculation method  using     batch average RMSE instead of overall RMSE.     This calculation will result in a smaller RMSE result, and the true RMSE should be the root mean square error of all predicted results.
Below are my reproduction results and reproduction code charts（FFVE-RMSE used the corrected RMSE calculation method）

Thank you for your keen interest in our project and for bringing up the associated concerns.
The code segment in question originates from others earlier open-source endeavors. Following your discovery, we conducted a thorough review and determined that this computation method does not impact the training and validation of our method. However, we have indeed identified the issue of using batch average RMSE instead of overall RMSE, specifically in the C-MAPSS turbofan engine dataset's test set.
Furthermore, it's worth noting that the experiments with our method on the NASA lithium-ion battery dataset are based on a different open-source framework, ensuring that this issue does not affect the results of our work on the NASA lithium-ion battery dataset.
While we cannot pinpoint when this issue originated in others previous work, we are committed to actively addressing and rectifying it. Following this, we will promptly update and correct the relevant work on our GitHub repository.
Should you be willing, we would like to express our gratitude by acknowledging your name on our project's main page.
Once again, we sincerely thank you for your valuable contributions to our project!

Here is the code that we will update later:

Appreciation for your reply and looking forward to further test results
Thank you for your swift response and for addressing the issues I raised in my previous correspondence.
It is reassuring to know that, following your thorough review, the identified computation method does not impact the training and validation of your method, a conclusion with which I concur. However, I do have concerns about obtaining lower RMSE results when using the batch average RMSE as opposed to the overall RMSE on the test set of the C-MAPSS turbofan engine dataset.（As shown in my previous  image)

I am keenly anticipating the enhancements you will make. I look forward to examining the revised code and observing the test results on the C-MAPSS dataset post-adjustment.
Separately, I would like to express once again my admiration for the elegance and readability of your source code. Your paper is comprehensive and well-articulated, making it one of the most impressive I have recently encountered. Thank you for contributing such a remarkable open-source project to the community.
Best regards

Thank you for your affirmation. It is the best praise for our work!
Currently, we have already updated the erroneous code. Unfortunately, our original computing resources are temporarily unavailable, preventing us from immediately recalculating the RMSE. Perhaps we will need to use new equipment for the recalculation in the future. Consequently, we plan to update the performance metrics on the C-MAPSS turbofan engine dataset at a later time.
Additionally, correcting errors in the paper is a challenging task. We are actively considering how to inform others of the correct calculation results as soon as possible to prevent the dissemination of inaccurate findings. Your assistance in sharing the correct results with others is highly welcome.

Okay, Got it

Hi, we have promptly updated the data affected by the mentioned issue, and you can find all the errata here!
It's worth noting that, due to the inability to use the equipment mentioned in the paper, we used a new device to compute the results. Therefore, there may be some variations in SCORE compared to the original results (SCORE will remain unchanged if the same equipment is used). If you need to make performance comparisons, you can refer to these new results.
Furthermore, from the experimental results, all the conclusions mentioned in the paper still hold true, so there is no need to worry about any textual errors in the paper.
Finally, we apologize for the error in our code and appreciate your valuable contributions!


Update dependencies

Update webrtc version to 0.5. This seems the quickest fix to problems which raises when cargo update is used.

Thanks for the PR!
Is there a difference between this one and #40 ?

Oh. I'm sorry, I didn't check it. They're the same, except that in my PR the patch version of webrtc is not set.

No, worries! The other PR kind of stagnated anyway. I took another look now, and I think my concerns over the other one was unfounded.
closing as duplicate of #40 (which has a slightly cleaner commit history)

Are you going to publish a new release in crates.io?

There is one thing making me reluctant to make a new release, and that is that I suspect Firefox support may have gotten worse since the 0.3 release.
There is definitely weird things going on in Firefox, but I'm not exactly sure where the issue comes from FF itself, Matchbox or the bevy example app.


mqtt_reconnect

Hi !
If I restart the mosquitto server, i have to restart the container to control my devices again .

Hi !
If I restart the mosquitto server, i have to restart the container to control my devices again .

Fixed by editing this part of main.py
def connect_mqtt(config, client_id) -> mqtt_client:
def on_connect(client, userdata, flags, return_code):
# pylint: disable=unused-argument
if return_code == 0:
client.connected_flag = True
logging.info('Connected to MQTT Broker as %s.', client_id)
client.subscribe('{}/+/+/set'.format(
config['home_assistant']['state_topic']))
else:
logging.critical('Failed to connect, return code %d', return_code)

@dst6se If you still get this, I would very much like to use your version in my system. But I'm not sure how to implement it, and there is no issue page on your fork. Can you please let me how I do that? I can build, I'm not sure what to build. It stops on this:
`git submodule update --init
Cloning into '/home/pi/telldus-core-mqtt/telldus'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:mliljedahl/telldus.git' into submodule path '/home/pi/telldus-core-mqtt/telldus' failed
Failed to clone 'telldus'. Retry scheduled
Cloning into '/home/pi/telldus-core-mqtt/telldus'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:mliljedahl/telldus.git' into submodule path '/home/pi/telldus-core-mqtt/telldus' failed
Failed to clone 'telldus' a second time, aborting
`

@dst6se If you still get this, I would very much like to use your version in my system. But I'm not sure how to implement it, and there is no issue page on your fork. Can you please let me how I do that? I can build, I'm not sure what to build. It stops on this:
`git submodule update --init Cloning into '/home/pi/telldus-core-mqtt/telldus'<EMAIL_ADDRESS>Permission denied (publickey). fatal: Could not read from remote repository.
Please make sure you have the correct access rights and the repository exists. fatal: clone of 'git@github.com:mliljedahl/telldus.git' into submodule path '/home/pi/telldus-core-mqtt/telldus' failed Failed to clone 'telldus'. Retry scheduled Cloning into '/home/pi/telldus-core-mqtt/telldus'<EMAIL_ADDRESS>Permission denied (publickey). fatal: Could not read from remote repository.
Please make sure you have the correct access rights and the repository exists. fatal: clone of 'git@github.com:mliljedahl/telldus.git' into submodule path '/home/pi/telldus-core-mqtt/telldus' failed Failed to clone 'telldus' a second time, aborting `

Hi , the easy way is to just copy my modified main.py to the docker
ex
docker cp ./main.py telldus:/usr/src/telldus-core-mqtt/

Fantastic, and such a quick answer! Thank you very much! Appreciate it!

Fantastic, and such a quick answer! Thank you very much! Appreciate it!

NP, hope it works well. For my it is rock solid :)
Thanks
Danne

Yep, it does work well, but I am having a bit of a problem copying to the container, it is not stored from run to run. I had to have the container running and use this command:
 docker cp ./main.py tellstick:/usr/src/telldus-core-mqtt/
I call the container tellstick, not telldus, but that shouldn't matter. When I do the copy, I can do this:
docker exec tellstick cat /usr/src/telldus-core-mqtt/main.p
That shows me what the file contains. I changed "humidity" to "fuktighet" in line 19 to have an easy way of seeing if the copy stuck:
         const.TELLSTICK_HUMIDITY: 'fuktighet',
But that's back to "humidity" on the next run.

Yep, it does work well, but I am having a bit of a problem copying to the container, it is not stored from run to run. I had to have the container running and use this command:
 docker cp ./main.py tellstick:/usr/src/telldus-core-mqtt/
I call the container tellstick, not telldus, but that shouldn't matter. When I do the copy, I can do this:
docker exec tellstick cat /usr/src/telldus-core-mqtt/main.py
That shows me what the file contains. I changed "humidity" to "fuktighet" in line 19 to have an easy way of seeing if the copy stuck:
 const.TELLSTICK_HUMIDITY: 'fuktighet',
But that's back to "humidity" on the next run.

You are right !
I forgot to mention this
In my script when i start the container i do like this
docker run -d
--name telldus
--restart=unless-stopped
-e TDM_MQTT_SERVER=<IP_ADDRESS>
-e TDM_MQTT_PORT=1884
-e TDM_MQTT_USER=telldus_mqtt
-e TDM_MQTT_PASS=E58FuJjw4zpwmOanSr3
-v /media/Media/docker_data/telldus/conf/tellstick.conf:/etc/tellstick.conf:ro
--device=/dev/bus/usb:/dev/bus/usb:rwm
mliljedahl/telldus-core-mqtt:latest
so
-v /media/Media/docker_data/telldus/conf/main.py:/usr/src/telldus-core-mqtt/main.py:ro \
Point out the file to your config dir , and should never go back to the orginal one !

Brilliant, thank you again! I added #Tors versjon to the end of that main.py so I didn't have to scroll up to find the telltale signs, and that works now!  Thanks again! This container is actually something I've been looking for a long time, so I can get all my Pi's from Stretch and up to Bullseye, and I can use it on a bunch of other Pi's that are already on Bullseye, where I had to drop it. I bought a bunch of Tellstick Duo's cheap a few years ago, when Stretch was still the version in use. :-D

@dst6se I have one problem with this thing, and since you seem to know your stuff much, much better than me: A certain type of Oregon sensors sometimes send a totally wrong value. I could correct that if I was able to send the "model" info on MQTT, or even everything that it shows in the debug window. I can see the sensors F824 type when I run docker compose up without the -d option, so I know it's in there somewhere. I just have no idea how to get it out! If you could help me that would be great, Danne!


Load balancing upstream API by IP location and geography

A cool feature to have would be to support routing to upstream API service by IP and location.
This will allow us to push update to specific regions while holding back release from other regions.
:100:

Related to #157


doc menu update

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #653 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #653   +/-   ##
=======================================
  Coverage   86.93%   86.93%
=======================================
  Files          71       71
  Lines        3345     3345
  Branches      634      634
=======================================
  Hits         2908     2908
  Misses        420      420
  Partials       17       17

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 01633e0...368f508. Read the comment docs.


What about BTC symbols?

Hey guys, would you mind adding BTC symbol mapping as well.

"BTC": "Ƀ" should solve it.


Variable scope not being seen properly in Dark/Light (Visual Studio) themes

From @rhires on February 11, 2016 3:2

I wanted to title this "It's not just a mouthwash!" but...
Anyway, this brought me back to making sure I knew about scope - when I have a variable outside of the if statement, and then reference it inside the if statement, the system shouldn't complain about that. It's perfectly valid syntax. Why does VSCode think my variable is unused?
This is a C# file (which the little info in the bottom info bar agrees with), running 0.10.8, on Mac OS X Yosemite. This appears to be a new issue.
Copied from original issue: Microsoft/vscode#2916

From @Tyriar on February 11, 2016 3:10
The variable is assigned but not used, a warning is expected because technically the variable is still doing nothing.
var should be colored though, I can reproduce it. It looks like it's a problem with the "Dark (Visual Studio)" and "Light (Visual Studio)" themes.

Moving to OmniSharp for commenting on the warning on the variable.
@Tyriar Let's have a separate issue for the 'var' coloring.

Thanks. Will take a look!

The issue with the warning is by design. We were tracking the colorization issue with #100, but haven't been to reproduce it in a long while.


Implement ModuleParams inheritance in the VtolLandDetector class.

Describe problem solved by the proposed pull request
This PR implements ModuleParams inheritance in the VtolLandDetector class following PR #12356.
Test data / coverage
NOTE: I do not have hardware to flight test this PR.
Additional context
See PR #12356, #12209 and #9756.  This PR requires #12356 prior to acceptance.
@dagar and @bkueng , please let me know if you have any questions on this PR.  Thanks!
-Mark

Thanks everyone!


Logs throwing errors on startup after migration

EOS 0.2.17 version of synapse was 1.36.0
EOS 0.3.0 version of synapse is 1.42.0

2021-12-22T07:06:04.450346Z 2021-12-22 07:06:04,449 - root - 336 - WARNING - main - ***** STARTING SERVER *****
2021-12-22T07:06:04.450886Z 2021-12-22 07:06:04,450 - root - 337 - WARNING - main - Server /usr/local/lib/python3.8/site-packages/synapse/app/homeserver.py version 1.42.0
2021-12-22T07:06:04.451276Z 2021-12-22 07:06:04,450 - root - 338 - INFO - main - Server hostname: wr5bumrkyhtsh6roigttxtuahkol7gzq6n7ufdijc5lhkf3oy53dbtqd.onion
2021-12-22T07:06:04.451685Z 2021-12-22 07:06:04,450 - root - 339 - INFO - main - Instance name: master
2021-12-22T07:06:04.452059Z 2021-12-22 07:06:04,451 - synapse.app.homeserver - 357 - INFO - main - Setting up server
2021-12-22T07:06:04.452418Z 2021-12-22 07:06:04,451 - synapse.server - 308 - INFO - main - Setting up.
2021-12-22T07:06:04.460986Z 2021-12-22 07:06:04,460 - synapse.storage.databases - 49 - INFO - main - [database config 'master']: Checking database server
2021-12-22T07:06:04.461622Z 2021-12-22 07:06:04,460 - synapse.storage.databases - 52 - INFO - main - [database config 'master']: Preparing for databases ['main', 'state']
2021-12-22T07:06:04.462356Z 2021-12-22 07:06:04,461 - synapse.storage.prepare_database - 116 - INFO - main - ['main', 'state']: Checking existing schema version
2021-12-22T07:06:04.474862Z 2021-12-22 07:06:04,469 - synapse.app._base - 161 - ERROR - main - Exception during startup
2021-12-22T07:06:04.475371Z Traceback (most recent call last):
2021-12-22T07:06:04.475784Z   File "/usr/local/lib/python3.8/site-packages/synapse/app/homeserver.py", line 360, in setup
2021-12-22T07:06:04.476123Z     hs.setup()
2021-12-22T07:06:04.476432Z   File "/usr/local/lib/python3.8/site-packages/synapse/server.py", line 310, in setup
2021-12-22T07:06:04.476690Z     self.datastores = Databases(self.DATASTORE_CLASS, self)
2021-12-22T07:06:04.476902Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/databases/__init__.py", line 57, in __init__
2021-12-22T07:06:04.477259Z     prepare_database(
2021-12-22T07:06:04.477538Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/prepare_database.py", line 117, in prepare_database
2021-12-22T07:06:04.477821Z     version_info = _get_or_create_schema_state(cur, database_engine)
2021-12-22T07:06:04.478107Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/prepare_database.py", line 656, in _get_or_create_schema_state
2021-12-22T07:06:04.478405Z     executescript(txn, sql_path)
2021-12-22T07:06:04.478776Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/prepare_database.py", line 643, in executescript
2021-12-22T07:06:04.479170Z     execute_statements_from_stream(txn, f)
2021-12-22T07:06:04.479452Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/prepare_database.py", line 648, in execute_statements_from_stream
2021-12-22T07:06:04.479753Z     cur.execute(statement)
2021-12-22T07:06:04.480139Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/database.py", line 298, in execute
2021-12-22T07:06:04.480598Z     self._do_execute(self.txn.execute, sql, *args)
2021-12-22T07:06:04.481044Z   File "/usr/local/lib/python3.8/site-packages/synapse/storage/database.py", line 331, in _do_execute
2021-12-22T07:06:04.482508Z     return func(sql, *args)
2021-12-22T07:06:04.482918Z sqlite3.OperationalError: attempt to write a readonly database
2021-12-22T07:06:04.483296Z ********************************************
2021-12-22T07:06:04.483672Z  Error during initialisation:
2021-12-22T07:06:04.484070Z     attempt to write a readonly database
2021-12-22T07:06:04.484422Z  There may be more information in the logs.
2021-12-22T07:06:04.484709Z ********************************************
2021-12-22T07:06:13.425849Z This server is configured to use 'matrix.org' as its trusted key server via the
2021-12-22T07:06:13.427238Z 'trusted_key_servers' config option. 'matrix.org' is a good choice for a key
2021-12-22T07:06:13.427570Z server since it is long-lived, stable and trusted. However, some admins may
2021-12-22T07:06:13.427807Z wish to use another server for this purpose.
2021-12-22T07:06:13.428106Z
2021-12-22T07:06:13.428382Z To suppress this warning and continue using 'matrix.org', admins should set
2021-12-22T07:06:13.428640Z 'suppress_key_server_warning' to 'true' in homeserver.yaml.

This appears to be happening because synapse expects some of its files to be owned by user 991 rather than root. Here's the synapse permissions before migration:
root@start9-8d7bdd8b:~# ls -al volumes/
bitcoind/      btcpayserver/  btc-rpc-proxy/ c-lightning/   lnd/           spark-wallet/  synapse/       thunderhub/
root@start9-8d7bdd8b:~# ls -al volumes/synapse/
total 5076
drwxr-xr-x  4  991  991    4096 Dec 29 22:19 .
drwxr-xr-x 10 root root    4096 Dec 25 18:22 ..
-rw-r--r--  1 root root     985 Dec 25 18:27 cert.pem
-rw-r--r--  1<PHONE_NUMBER>960 Dec 29 22:19 homeserver.db
-rw-r--r--  1  991  991    1769 Dec 28 16:25 homeserver.yaml
-rw-------  1 root root    1704 Dec 25 18:27 key.pem
drwxr-xr-x  4  991  991    4096 Dec 25 18:35 media_store
-rw-r--r--  1 root root     468 Dec 25 18:27 mvfpwfbd6tpqi46gzueyasugkhxzs7773n72542raxwngfw7ypb2w2ad.onion.log.config
-rw-r--r--  1  991  991      59 Dec 25 18:27 mvfpwfbd6tpqi46gzueyasugkhxzs7773n72542raxwngfw7ypb2w2ad.onion.signing.key
drwxr-xr-x  2 root root    4096 Dec 25 18:27 start9

and here's after:
root@embassy:/embassy-data/package-data/volumes/synapse/data/main# ll
total 5076
drwxr-xr-x 4 root root    4096 Dec 29 22:35 ./
drwxr-xr-x 3 root root    4096 Dec 29 22:35 ../
-rw-r--r-- 1 root root     985 Dec 29 22:35 cert.pem
-rw-r--r-- 1 root root 5160960 Dec 29 22:35 homeserver.db
-rw-r--r-- 1 root root    1769 Dec 29 22:35 homeserver.yaml
-rw------- 1 root root    1704 Dec 29 22:35 key.pem
drwxr-xr-x 4 root root    4096 Dec 29 22:35 media_store/
-rw-r--r-- 1 root root     468 Dec 29 22:35 mvfpwfbd6tpqi46gzueyasugkhxzs7773n72542raxwngfw7ypb2w2ad.onion.log.config
-rw-r--r-- 1 root root      59 Dec 29 22:35 mvfpwfbd6tpqi46gzueyasugkhxzs7773n72542raxwngfw7ypb2w2ad.onion.signing.key
drwxr-xr-x 2 root root    4096 Dec 29 22:35 start9/

@dr-bonez can we add a feature to preserve file ownership attributes?

Fixed


Response construct use propery promotion

Also fix a problem with the last __construct, that permit array|null  in $headers and later this will break.
Before:
protected array $headers = [];

    /**
     * Response constructor.
     *
     * @param int $status
     * @param array|null $headers
     * @param string $body
     */
 public function __construct(
        int     $status = 200,
        ?array  $headers = [],
        string  $body = ''
    )
    {
        $this->status = $status;
        $this->headers = $headers;
        $this->body = $body;
    }

$this->headers = $headers; will break if we send a null.

Curiously was passing the tests, than use null.  :open_mouth:
@walkor you can decide how do you want it.
We can also check in the constructor if is a null to be converted to array().

I agree with your PR


GroupingChooser (DimensionChooser 2.0)

DimensionChooser showing its age, especially next to some of our recent work on FilterChooser.  Thinking about:

More of a omni-box based approach, like FilterChooser?
Drag and Drop,  like our recent client work on column choosing?
Use Favorites, rather than History, to be like FilterChooser?

Also thinking to make a new control rather than breaking the API of the existing control.  Not sure about this one.

Opened up draft PRs for review and feedback:
https://github.com/xh/hoist-react/pull/2200
https://github.com/xh/toolbox/pull/446

This image is enough for me.  Its love at first sight.
On Tue, Nov 24, 2020 at 8:29 AM Tom Tirapani<EMAIL_ADDRESS>wrote:

[image: Screenshot 2020-11-24 at 13 28 24]
https://user-images.githubusercontent.com/3017757/100100651-07a2aa00-2e59-11eb-9e28-9b9988357bbb.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xh/hoist-react/issues/2177#issuecomment-732973529,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARTL2PI6W7THXKWRHXHD4LSROYLLANCNFSM4TNC2N4Q
.


task/install: rest-bench has been removed

Removed this tool from upstream.  Don't install (or remove) it any more!
Signed-off-by: Sage Weil<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1734/
Test PASSed.


GUI does not appear on XRCamera3D

Tested versions
Reproductible with 4.3-dev5
System information
Windows 11 - Godot 4.3-dev5 Vulkan (Forward+)
Issue description
GUI (Control) does not appear on the PC screen when using XRCamera3D. I understand why it doesn't appear in the HMD but I see no reason it doesn't show up on the desktop mirror view...
Steps to reproduce
Add a GUI to a XR scene.
Minimal reproduction project (MRP)
Don't think it's necessay to provide a project but I can provide it if needed.

CC @BastiaanOlij
I am fairly certain this is intended as the XRCamera3D view is supposed to render as if it is an HMD. The rendering path for XR is different and totally skips over 2D/GUI

CC @BastiaanOlij
I am fairly certain this is intended as the XRCamera3D view is supposed to render as if it is an HMD. The rendering path for XR is different and totally skips over 2D/GUI

That was my theory... But I don't see think it's a good idea to remove the gui on the mirror view. If the engine does not handle this, how can I display a Gui on the mirror view on desktop then (and of course, not in the HMD) ?

Hey @ultrasuperpingu ,
We don't render 2D elements in XR mode because there is no good way to render them in place as there is no depth information and because there is no guarantee headsets have horizontal panels.
On the desktop we just show a copy of the left eye render result.
If you want to do what you want, you'll need a different setup. You can create a subviewport and add your XROrigin3D and XRCamera3D to that, and set that viewport up for XR (so setting use_xr to true and all that). This will handle output to the HMD.
You can now use a TextureRect with the texture set to a ViewportTexture pointing to the subviewport. This will now show the left eye. You do need to play around with sizing and positioning.
You can now also add your GUI elements as per normal.

@BastiaanOlij As I understand your advice correctly, the GUI will appear in the HMD no? I'm looking to display a gui on the desktop mirror but not in the HMD. Is it possible ?

@BastiaanOlij As I understand your advice correctly, the GUI will appear in the HMD no? I'm looking to display a gui on the desktop mirror but not in the HMD. Is it possible ?

No, the GUI will never appear in the HMD unless you render the GUI to a separate viewport and use that viewport as a texture on a Quadmesh or something similar.
The steps I described here will present the GUI on the desktop, not on the HMD.

Ok, I tried this but I guess I didn't understand it well, I couldn't make it work :(
I can't really understand the logic.
If I understand correctly, you suggest to render the entire scene in a subviewport (configured with use_xr=true), render it in a texture (with a TextureRect configured with a ViewportTexture pointing to the subviewport) and use this textureRect in a control on the main viewport (window) and then add the GUI on this viewport. Am I right ?
But even if I make it work, it seems a little complicated and more than that, it's very intrusive just to display a GUI.
I understand this is like this probably by design and for good reasons. If it's the case, this issue should be closed (and maybe a proposal on this should be open). But if the GUI pipeline is avoid on the desktop viewport because 2D GUI does not make sense in the HMD, maybe it should be added back ?

@ultrasuperpingu unfortunately that is a bug I recently ran into as well but haven't figured out why.
This worked fine as an approach in Godot 3, but it seems to not work specifically with OpenXR in Godot 4 (it does work with the native mobile interface), so there is something not correct with the way we do things here.
I am planning to investigate this soon.


Revert back esm support

Since styled-system issue was resolved could you revert this change?
https://github.com/rebassjs/rebass/commit/48996e575fdaa420d6724d0424a3428e7a19f41e

@jxnblk So what's the problem? Let me help to resolve it. Reverting esm after every release is not a solution.

I'll likely work on this some over the weekend. Might want to hold off on the build setup until more of the sub packages are in so that we can solve the issue for all packages – this is still on my radar, so wanna keep this issue open to track it


set-ios-info-plist-unified 1.1.5

New Pull Request Checklist
Please mark the points which you did / accept.

[/] I read the Step Development Guideline
[/] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[/] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[/] I read and accept the Abandoned Step policy

Hi @emeraldimp
Thanks for the step update, we'll review it asap!

So, the step & update is perfect, there's just one minor thing. Recently we started to normalize the type and project tags used in the steps/in this steplib, to make it easier for others to browser & discover steps e.g. in the workflow editor.
Could you please update your type_tags and project_type_tags following this guide: https://github.com/bitrise-io/bitrise/blob/master/_docs/step-development-guideline.md#step-grouping-convention
For the previous version of your step we did that automatically when we normalized the whole collection, so you can find our recommended tags for your step at: https://github.com/bitrise-io/bitrise-steplib/blob/master/steps/set-ios-info-plist-unified/1.1.4/step.yml#L13
Simply just: normalize the tags in your step.yml then create a new version of the step and share that, nothing else required :)
I hope it's OK, but of course if you have any questions or feedback please let us know!


ci(release): Publish to npm with semantic-release

Setup semantic-release & travis CI
Add .editorconfig

I can't approve it, but...at least I expressed my emotions :D


Update Versioning to match new standard

Goes along with DigitalGlobe/DeepCore-Dev#431

To make things a little more standard, we're going to slightly change the way version numbers are created. Currently we use MAJOR_VERSION.MINOR_VERSION.BUILD_NUMBER. A more standard way is to use MAJOR_VERSION.MINOR_VERSION.PATCH_VERSION+BUILD_NUMBER. That's what we're moving to.


Code from Simulating an epidemic

I am working on a project and the animation in the video "Simulating an epidemic" would be very helpful to help people visualize something if I can adapt them. Would it be able to provide the code for that video?
Thanks

See: https://github.com/3b1b/manim/blob/shaders/from_3b1b/old/sir.py

Thanks! Could you help me do an animation actually?

This link is invalid. Do you have the latest link？THANKS A LOT!!!

This link is invalid. Do you have the latest link？THANKS A LOT!!!

@MustForgiveHer All of his videos have been moved to his videos repository. This is what you're looking for:
https://github.com/3b1b/videos/blob/master/_2020/sir.py

@MustForgiveHer All of his videos have been moved to his videos repository. This is what you're looking for:
https://github.com/3b1b/videos/blob/master/_2020/sir.py


Optimise the count negatives in a sorted matrix

Is your feature request related to a problem? Please describe.
Problem
This problem has O(n^2) complexity which doesn't exploit the property of row and column sorted matrix. I will try to exploit this property and optimize it to O(n)
Describe the solution you'd like
A coded C++ solution with an optimized approach

Sure @kiruba-r11 , you can go ahead.


Update repository and homepage urls

Addresses
https://github.com/apache/arrow-rs/issues/12

We had to perform a small re-write of master. The commits may look a bit odd, but it should not cause conflicts. Could you kindly rebase this against the latest master to make it easier to review?

@jorgecarleitao done


Publish semantic versioning policy

Saw this thread in the Stylelint issue tracker and thought that cssnano also follows a similar approach with regard to patch/minor/major versions. It would be nice to have a document for this starting from version 4.

👍


Pépito C++ program

Hello!
I have created a C++ program that connects to the API and parses every message.
The program starts with testing the parsing method with local test data.
I think the project could be a good "base"  to extend upon.
https://github.com/Avicted/pepito_app
Best regards,
Victor

Hi,
thanks for sharing,
so it's more like an implementation example than a library ?
Maybe I could add a the link in the example section.

Hi!
Exactly! It's a showcase of how one could create a native application that reacts to the events.
I agree that it would fit into the example section.
Best regards,
Victor


Add generated API docs to make verify target

This commit updates the make verify step to incorporate API docs
generation verification by ensuring changes to API types or manual
edits to the generated file will cause presubmit build failures.

/lgtm


Count completed missions

When a mission is completed, the log message is like this one:
{
  "timestamp": "2022-03-04T10:59:24Z",
  "event": "MissionRedirected",
  "MissionID": <number>,
  "Name": "Mission_MassacreWing",
  "NewDestinationStation": "Braun Station",
  "NewDestinationSystem": "Gliese 868",
  "OldDestinationStation": "",
  "OldDestinationSystem": "LP 932-12"
}

Moreover, if it would be possible to know how many active mission the user has, it would also be possible to send a notification when all missions are completed

At the beginning of the log file, current missions are summarized:
{
  "timestamp": "2022-03-04T07:32:12Z",
  "event": "Missions",
  "Active": [
    {
      "MissionID": 851388534,
      "Name": "Mission_MassacreWing_name",
      "PassengerMission": false,
      "Expires": 0
    },
    {
      "MissionID": 851389346,
      "Name": "Mission_MassacreWing_name",
      "PassengerMission": false,
      "Expires": 0
    },
    {
      "MissionID": 851391153,
      "Name": "Mission_MassacreWing_name",
      "PassengerMission": false,
      "Expires": 0
    }
  ],
  "Failed": [],
  "Complete": []
}

Completed missions get logged:
{
  "timestamp": "2022-03-04T07:41:22Z",
  "event": "MissionCompleted",
  "Faction": "LP 932-12 Dynamic Inc",
  "Name": "Mission_Massacre_name",
  "MissionID": 851468600,
  "TargetType": "$MissionUtil_FactionTag_Pirate;",
  "TargetType_Localised": "Pirates",
  "TargetFaction": "LP 932-12 Society",
  "KillCount": 10,
  "NewDestinationSystem": "Gliese 868",
  "DestinationSystem": "LP 932-12",
  "NewDestinationStation": "Bacon Port",
  "DestinationStation": "Hurston Hub",
  "Reward": 1945952,
  "FactionEffects": [
    {
      "Faction": "LP 932-12 Dynamic Inc",
      "Effects": [
        {
          "Effect": "$MISSIONUTIL_Interaction_Summary_EP_up;",
          "Effect_Localised": "The economic status of $#MinorFaction; has improved in the $#System; system.",
          "Trend": "UpGood"
        }
      ],
      "Influence": [
        { "SystemAddress":<PHONE_NUMBER>122, "Trend": "UpGood", "Influence": "+" }
      ],
      "ReputationTrend": "UpGood",
      "Reputation": "+++++"
    },
    {
      "Faction": "LP 932-12 Society",
      "Effects": [],
      "Influence": [
        { "SystemAddress":<PHONE_NUMBER>481, "Trend": "DownBad", "Influence": "+" }
      ],
      "ReputationTrend": "DownBad",
      "Reputation": "+"
    }
  ]
}

And new missions generate this log:
{
  "timestamp": "2022-03-04T07:41:45Z",
  "event": "MissionAccepted",
  "Faction": "Gliese 868 Services",
  "Name": "Mission_Massacre",
  "LocalisedName": "Kill LP 932-12 Society faction Pirates",
  "TargetType": "$MissionUtil_FactionTag_Pirate;",
  "TargetType_Localised": "Pirates",
  "TargetFaction": "LP 932-12 Society",
  "KillCount": 12,
  "DestinationSystem": "LP 932-12",
  "DestinationStation": "Hurston Hub",
  "Expiry": "2022-03-05T22:38:43Z",
  "Wing": false,
  "Influence": "++",
  "Reputation": "++",
  "Reward": 14473384,
  "MissionID": 851580817
}


What the website should ACTUALLY be like

This issue wants to be a brief - to collect all my thoughts around what the debug-mind website should actually be (and not the placeholder joke that is now).
The Basics

use the font and logo and key colors already in use

and keep the overall UI consistent with those

keep the existing infra and pipeline (GitHub OSS repo -> Netlify)
a good footer that mentions the repo being OSS (the one already there is ok)
good responsiveness, should look good on desktop and mobile
should be a11y compliant

The Important Stuff

have a big CTA/preview of the latest episode from the YT playlist
have a smaller list with all the other episodes, and maybe some extra info (extracted from the description box)
have the link/button/CTA to the first aid kit/ mental health from Dr.K website -> https://explore.healthygamer.gg/mentalhealthresources
link/preview/showcase the shorts, that exist in multiple socials

youtube https://www.youtube.com/@kelset/shorts
Instagram https://www.instagram.com/debug_mind/
tiktok https://www.tiktok.com/@debug_mind

link to the twitter and threads accounts as well... maybe (the threads one is not used tbh)

twitter https://twitter.com/debug_mind
threads https://www.threads.net/@debug_mind

Good To Have

figure out a way to integrate this separate list of resources on the website https://github.com/kelset/mental-health-resources

that way, I can close off that other repo and keep my hand-picked resources as part of this repo

better copy
use as little JS as possible - it's a small website, really doesn't need anything like a Framework (Next, Remix, etc)

fixed by #4


[SUGGESTION] Add Microsoft Image creator

Hello, could be great to also integrate a button for use image creator, this uses DALL-E and it's free.

I attached a picture here where you can find

input 'create me an image of xxx'


Night Drone Supprt

Just had a new issues that I could not find specifically referenced and thought I'd let you know. The new night drones seem to have a new name prefix (Swat_) so to make them discoverable I added the following:
localName.indexOf('Swat_')=== 0) to the drone.js
Hope this helps someone else.
J

@gamewagon Thanks for that, indeed it did help me.

Blaze_ prefix for Airborne Night Blaze


[Tutorial] Fix post IFU issues with FA

Last IFU mixed some upstream and ROCm changes, rendering testing and benchmarking impossible.
This PR fixes merge issues.

tutorials/06-fused-attention.py::test_op_fwd[False-4-48-1024-64] PASSED                                                [  6%]
tutorials/06-fused-attention.py::test_op_fwd[False-4-48-2048-64] PASSED                                                [ 12%]
tutorials/06-fused-attention.py::test_op_fwd[False-4-48-4096-64] PASSED                                                [ 18%]
tutorials/06-fused-attention.py::test_op_fwd[False-4-48-1024-128] PASSED                                               [ 25%]
tutorials/06-fused-attention.py::test_op_fwd[False-4-48-2048-128] PASSED                                               [ 31%]
tutorials/06-fused-attention.py::test_op_fwd[False-4-48-4096-128] PASSED                                               [ 37%]
tutorials/06-fused-attention.py::test_op_fwd[True-4-48-1024-64] PASSED                                                 [ 43%]
tutorials/06-fused-attention.py::test_op_fwd[True-4-48-2048-64] PASSED                                                 [ 50%]
tutorials/06-fused-attention.py::test_op_fwd[True-4-48-4096-64] PASSED                                                 [ 56%]
tutorials/06-fused-attention.py::test_op_fwd[True-4-48-1024-128] PASSED                                                [ 62%]
tutorials/06-fused-attention.py::test_op_fwd[True-4-48-2048-128] PASSED                                                [ 68%]
tutorials/06-fused-attention.py::test_op_fwd[True-4-48-4096-128] PASSED                                                [ 75%]
tutorials/06-fused-attention.py::test_op_bwd[4-48-1024-64] PASSED                                                      [ 81%]
tutorials/06-fused-attention.py::test_op_bwd[4-48-2048-64] FAILED                                                      [ 87%]
tutorials/06-fused-attention.py::test_op_bwd[4-48-4096-64] PASSED                                                      [ 93%]
tutorials/06-fused-attention.py::test_op_bwd[1-16-8192-64] PASSED                                                      [100%]

The bwd tests was broken by https://github.com/ROCmSoftwarePlatform/triton/pull/356, in which I only tested the fwd kernel.
After fixing the _attn_bwd_preprocess kernel and relax the rtol, all bwd tests pass on MI250X.


Move to configuring the fabric bot via JSON

PR Summary
Move to configuring the fabric bot via JSON
PR Context
This is how FabricBot supports configuration going forward.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Is this JSON file automatically generated by the old fabric-bot configuration site?

Yeah, this is auto-generated from the old config site.


composer initial

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             master   #1   +/-   ##
=====================================
  Coverage         0%   0%
  Complexity       13   13
=====================================
  Files             1    1
  Lines            36   36
=====================================
  Misses           36   36

Flag
Coverage Δ
Complexity Δ

#php
0% <ø> (ø)
13 <ø> (ø)
:arrow_down:

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 bcbdec1...df16313. Read the comment docs.


Why am I getting a different performance & result of LaplaceBEM running on phantom?

I am trying to use the code to reproduce the speedup for Laplace 1st-kind problem. So I ran the first non-relaxed case (corresponding to the non-relaxed case on the first row of the table below) using:
theta =  0.5, N = 2048 (recursions = 5) , ncrit = 400

In the terminal, I first ran (without preconditioning)
./LaplaceBEM -theta 0.5 -ncrit 400 -p 8 -recursions 5 -fixed_p -gmres -solver_tol 1e-5
and I got the result like this:
initialised 2048 triangles
...
Solver: GMRES
Preconditioner: Identity
it: 001, res: 2.311e-04, fmm_req_p: 8
it: 002, res: 9.630e-05, fmm_req_p: 8
it: 003, res: 4.506e-05, fmm_req_p: 8
it: 004, res: 1.935e-05, fmm_req_p: 8
Final residual: 9.2838e-06, after 5 iterations

TIMING:
	setup : 3.6045e+00s
	solve : 1.5399e+01s
external phi: 0.192, exact: 0.19245, error: 2.3505e-03
error: 5.344e-03

The answer seemed to be correct, so then I tried to used the diagonal matrix as the preconditioner (try to match the "t_solve = 0.27" in the table):
./LaplaceBEM -theta 0.5 -ncrit 400 -p 8 -recursions 5 -fixed_p -gmres -diagonal -solver_tol 1e-5
I got this:
initialised 2048 triangles
......
Creating plan: 0.00376582
Executing plan: 3.58662
1st-kind equation being solved

TIMING:
	setup : 3.5906e+00s
	solve : 4.9171e-03s
external phi: 2.1197e-10, exact: 0.19245, error: 1.0000e+00
error: 1.000e+00

it seems that the preconditioner is not working this time. I tried the "local" preconditioned with "fgmres" solver, and got the same error. I compiled the code using the Makefile in the "examples" folder, and gave me no error on LaplaceBEM executable. And the gcc version on phantom is 4.6.
Would you @slayton58  help me out on how to use the diagonal preconditioner correctly to get a similar performance of t_solve = 0.27 in this case?
Thank you.
Tingyu

Has the hostname changed at all for phantom (from phantom.seas.gwu.edu)? I can no longer see the machine..

No, but there is a power outage today at our building. The machine is down currently...

That'll do it.. Let me know when the machine is back up and I'll take a look

Sure, thank you.

On Jun 24, 2015, at 5:11 PM, slayton58<EMAIL_ADDRESS>wrote:
That'll do it.. Let me know when the machine is back up and I'll take a look
—
Reply to this email directly or view it on GitHub.

Hi Simon, the machine is back now.

Comment line 147 in include/executor/ExecutorSingleTree.hpp to remove excess output, then add '-lazy_eval' to your command line.
You can also specify a number of OMP threads to use which affects runtime for me at 8k panels and above

Comment line 147 in include/executor/ExecutorSingleTree.hpp to remove excess output, then add '-lazy_eval' to your command line.
You can also specify a number of OMP threads to use which affects runtime for me at 8k panels and above

Thanks for the reply, Simon. After I use -lazy_eval flag and tweaking OMP_NUM_THREADS, I got the performance of LaplaceBEM which is very close to the data presented in the paper. I think these cases are run with straight GMRES (without a preconditioner).
Then I tried to run with -diagonal preconditioner, so I uncommented the line 215-221 and line 261-265 in "LaplaceBEM.cpp" to enable the diagonal preconditioner. But I found the "diagonal preconditioned" case always needs more iterations and time to converge than the "unpreconditioned" case. So I wonder did you get a similar observation, or am I using the diagonal preconditioner correctly? (I used -lazy_eval and tested with different problem size and accuracy).
Thank you.

@slayton58 Note that in the thesis it says (p.54, Chapter 5 on results with Laplace BEM):
"All tests were performed using a canonical right-preconditioned GMRES (algorithm 1) …"
I transferred this assertion to the paper, and repeated it in the caption of each figure.
Therefore, if our suspicions are right, and the tests were done without a preconditioned, there is a mistake in the paper in how the results are reported. We must get to the bottom of this and correct the manuscript as needed.

The GMRES solver used requires a preconditioner -- it cannot be run without specifying one. You say "no preconditioner", I say "Identity preconditioner".

I guess what you're saying is that the statement in p.54 refers to the fact that the code implements an algorithm for pre-conditioned GMRES and the only way to not precondition is to use an identity preconditioner. But the statement is misleading to the reader because "all tests were performed with a preconditioned GMRES" will be interpreted as having run the experiments with some preconditioning applied on the linear system—the subject of the sentence is "tests."
The statement in p.54 is the only time that the string "precondition" appears in Ch. 5. There is no other mention of what preconditioner was used with the Laplace tests. But I note that the table of parameters for the tests with red blood cells do say "Preconditioner … None" (p.90).
What I want is enough information to reproduce the results.


Marksman LSP

There's an LSP server for Markdown called Marksman offering some additional Markdown features. There's no other server for Markdown defined in the default configuration. Is it worthy enough to add? :smile:

Done, thanks for letting me know about this language server. Is nice to finally have a table of content for quick navigation :D


pycharm: 2017.1 -> 2017.1.2 [release 17.03]

Motivation for this change
This is the same as https://github.com/NixOS/nixpkgs/pull/25673, but done against the release-17.03 branch instead of master, as things were moved around on master (idea was renamed to jetbrains)
Please merge, as otherwise pycharm-professional isn't available for 17.03 (version 2017.1 tarball was removed by vendor).
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

LGTM, I'd like for the maintainers to chime in on whether the commits that move stuff around can be picked to release as well to reduce the friction involved in updating stuff.

@joachifm this rename of idea -> jetbrains? I think that it could be cherry-picked, it's just bikeshedding.

Just a note: PyCharm was recently updated to 2017.1.3, so this PR is a little bit outdated ;) Even more so if s/idea/jetbrains/ is going to get cherry-picked into release-17.03 - if this is the case, this PR should be closed.
I'll make a new PR for 2017.1.3, against the master branch in some minutes. Currently testing it. If anyone's going to work on PyCharm, please consider checking it out as well.

Either way, what I'm worried about is that 2017.1 tarball is not available anymore so I had to comment out idea.pycharm-professional from my configuration.nix's environment.systemPackages. 17.03 needs an update.

In case you'll decide against cherry-picking idea->jetbrains rename, I've updated this PR with 2017.1.3 as well.

cp:ed with: f0c4be2873d40b3c116ce64660d0a609c782b1d5 958443e0147f24cfbfb020062bf6aec8bc7a44a5 d56d0ab956e70baf7e35514189befbf3e1785d7a b50009158b480d776a7395c3164b6540b985409e


Optimize Optitype resource usage

Currently for the optitype step we request 64GB of ram and say nothing about CPUs.
https://github.com/griffithlab/analysis-wdls/blob/main/definitions/tools/optitype_dna.wdl
But when Cromwell requests 64GB of RAM from Google we get this reported back:
"machineType": "custom-10-65536",
Jul 06 12:02:22 malachi-immuno-test java[14290]: 2022-07-06 12:02:22,783 cromwell-system-akka.dispatchers.backend-dispatcher-8451 INFO  - PipelinesApiAsyncBackendJobExecutionActor [UUID(78967613)immuno.optitype:NA:2]: To comply with GCE custom machine requirements, cpu was adjusted from 1 to 10

This is an expensive machine that we then massively underutilize in the optitype script here:
https://github.com/genome/docker-immuno_tools-cwl/blob/master/optitype_script.sh
The bottom line is that we wind up with a machine with 64GB of RAM and 10 CPUs.  Its not clear if we need that much ram, but we never use more than 4 CPUs.  Thread usage for sambamba and bwa mem steps in the script are hard code to use 4 CPUs.

Let's parameterize both memory and CPU usage in the WDL, update optitype_script.sh to accept these parameters as input.
Then update the sambamba and bwa mem steps to use these resources more reasonably. i.e. all available CPUs should be used for the bwa mem tasks and available RAM - 1GB (or 80% say) should be used for sambamba.
This will involve updating the tools/optitype_dna.wdl, optitype_script.sh, and then rebuild the docker image, and test the workflow.
In testing we should note the impact on total run time and cost.

In terms of the resources to actually use once we have the parameters to specify them, let's try using 8 CPUs and 32GB of RAM

I guess we will need to make any new parameters optional.  Otherwise this may break use of this tool in the CWL version of the pipeline:
https://github.com/genome/analysis-workflows/blob/master/definitions/tools/optitype_dna.cwl


Parsing error

I got another build fail when parsing a URL
Not sure if we can catch these as well, following up on #1
https://github.com/quicoto/mastodon-get-replies/actions/runs/4362831948/jobs/7628194627
Thank you

That error message is almost certainly a red herring: The very fact that it says there was an error, indicates the the error was caught.
As far as I can see this only happened once? I wonder if this was an OOM, or if GitHub killed the job for another reason 🤔

No idea, just wanted to raise this in case was something on our side we could improve.
Feel free to close if not relevant
Thank you for having a look

Another fail, different message I think
https://github.com/quicoto/mastodon-get-replies/actions/runs/4363887432/jobs/7630534273

@quicoto Please make sure you are at least on version 2.0.1. Fixed an important bug there.
(I've also re-written the action, so that future versions shouldn't need you to update your fork, as the action now downloads the latest version automatically for every run.)

@nanos Updated to the latest, Thank you !
https://github.com/quicoto/mastodon-get-replies/tree/main


Title: Basic to-do list using tkinter and java

Is your proposal related to a problem? Please describe.
Tkinter: Create a simple to-do list GUI where users can add, delete, and view tasks.
Java:Manage the task list. Python communicates with Java to add, remove, and retrieve tasks from the list.
Add any other context or screenshots about the proposal request here.
No response

@Kushal997-das


Update Presto website links and version

I've signed the CLA.

We have Starbust Presto init action for Presto SQL distribution.


Can this image be used for multiple mailboxes and mailaccounts?

Hi,
Can this Docker image be used with multiple mailboxes and accounts?
I have several gmail accounts which I want to use getmail for to collect my mail.
If it is possible how do I configure this to do that?
TIA

Yes, I use this container as part of a mail aggregator for public email and internal notification messages (because I like keep my messages local, when possible).
Make sure your secrets provider makes the credentials available within the container at /run/secrets as follows:
$ ls /run/secrets
email1_at_hotmail_password  email2_at_gmail_password

Here is a redacted docker-compose.yml for the stack:
#!/usr/bin/env docker-compose
---
version: "3.6"

services:
  dovecot:
    container_name: dovecot
    environment:
      DOVECOT_AUTH_PORT: "1313"
      DOVECOT_POSTMASTER_ADDRESS<EMAIL_ADDRESS>      DOVECOT_USE_MAILDIR: "1"
    hostname: imaps.the.domain.name
    image: crashvb/dovecot:202103220020
    ports:
    - "993:993/tcp"
    restart: always
    tty: true
    volumes:
    - dovecot-config:/etc/dovecot:rw
    - provider-secrets-dovecot:/run/secrets:ro
    - /media/mail:/var/mail:rw

  getmail:
    container_name: getmail
    environment:
      GETMAIL_RCEMAIL1_AT_HOTMAIL: |
        [retriever]
        type            = SimplePOP3SSLRetriever
        server          = pop3.live.com
        port            = 995
        username        =<EMAIL_ADDRESS>        password        = $${EMAIL1_AT_HOTMAIL_PASSWORD}

        [destination]
        type            = Maildir
        path            = /var/mail/crashvb/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
      GETMAIL_RCEMAIL2_AT_GMAIL: |
        [retriever]
        type            = SimplePOP3SSLRetriever
        server          = pop.gmail.com
        port            = 995
        username        = email2
        password        = $${EMAIL2_AT_GMAIL_PASSWORD}

        [destination]
        type            = Maildir
        path            = /var/mail/crashvb/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
    image: crashvb/getmail:202103220433
    restart: always
    tty: true
    volumes:
    - getmail-data:/etc/getmail:rw
    - provider-secrets-getmail:/run/secrets:ro
    - /media/mail:/var/mail:rw

  postfix:
    container_name: postfix
    environment:
      POSTFIX_AUTH_PATH: "dovecot:1313"
      POSTFIX_MAILNAME: the.domain.name
      POSTFIX_MYDESTINATION: |
        /^localhost$$/                  ACCEPT
        /^localhost.localdomain$$/      ACCEPT
      POSTFIX_MYDESTINATION_TYPE: hash
      POSTFIX_VIRTUAL_ALIAS_MAPS: |
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>        /@localhost$$/  <EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>        /@(.+)\.domain1\.info$$/<EMAIL_ADDRESS>        /@(.+)\.domain2\.info$$/<EMAIL_ADDRESS>        /@(.+)\.domain3\.info$$/<EMAIL_ADDRESS>        /@(.+)\.localhost$$/<EMAIL_ADDRESS>        /@(.+)\.localhost\.localdomain$$/<EMAIL_ADDRESS>      POSTFIX_VIRTUAL_ALIAS_MAPS_TYPE: pcre
      POSTFIX_VIRTUAL_MAILBOX_DOMAINS: |
        /^domain1\.info$$/                  OK
        /^domain2\.info$$/                  OK
        /^domain3\.info$$/                  OK
        /^localhost$$/                      OK
        /^localhost\.localdomain$$/         OK
        /^(.+)\.domain1\.info$$/            OK
        /^(.+)\.domain2\.info$$/            OK
        /^(.+)\.domain3\.info$$/            OK
        /^(.+)\.localhost$$/                OK
        /^(.+)\.localhost\.localdomain$$/   OK
      POSTFIX_VIRTUAL_MAILBOX_DOMAINS_TYPE: pcre
      POSTFIX_VIRTUAL_MAILBOX_MAPS: |
<EMAIL_ADDRESS>the.domain.name/crashvb/
      POSTFIX_VIRTUAL_MAILBOX_MAPS_TYPE: hash
    hostname: smtps.the.domain.name
    image: crashvb/postfix:202103221757
    links:
    - "dovecot"
    ports:
    - "465:465/tcp"
    restart: always
    tty: true
    volumes:
    - postfix-config:/etc/postfix:rw
    - provider-secrets-postfix:/run/secrets:ro
    - /media/mail:/var/mail:rw

volumes:
  dovecot-config:
    name: "dovecot-config"
  getmail-data:
    name: "getmail-data"
  postfix-config:
    name: "postfix-config"

Thanks for your reply and explanation, I will give it a try.
I only need the getmail part as I'm already running my own Mailserver on a Synology NAS. The getmail part is to retrieve mail for gmail and move them into the users mailboxes.
I will try to implement the getmail part...
Thanks again!

Hi,
I have tested your container but I am running into some problems.
These are the following problems I am running into:

the container is stuck on unhealty (no idea how to fix this)
I specified the volume for mail to mount, but it ignores that and creates its own volume in docker. The other volumes do work.
Once the container is running it does not fetch any mail, I cannot see anywhere in the log that it does so or that it has connected to e.g. Gmail.

See my docker-compose file which I used.
My goal is to use Getmail for 5 users to retrieve mails from Gmail. These are then delivered to the Mail folder and from there they are moved via a script to the users mailboxes on my Mailserver.
I am currently using this Getmail christophkoenig/getmail container and it does work, but has limited capabilities, such as cronjob. Yours does have cronjob which is why I want to switch to yours.
If I can get it working of course.
Could you take a look and point out what I am doing wrong?
version: '3.6'
services:
  getmail:
    container_name: getmail
    image: crashvb/getmail:202103220433
    restart: "no"
    network_mode: "bridge"
    hostname: getmail
    mem_limit: 25m
    mem_reservation: 25m
    security_opt:
      - no-new-privileges:true
    volumes:
      - /volume1/docker/getmailrc/data:/etc/getmail:rw
      - /volume1/docker/getmailrc/secrets:/run/secrets:ro
      - /volume1/docker/getmailrc/mail:/var/mail:rw
      - /etc/localtime:/etc/localtime:ro
      - /etc/TZ:/etc/timezone:ro
    environment:
      GETMAIL_RCEMAIL1_AT_GMAIL_1026: |
        [retriever]
        type            = SimpleIMAPSSLRetriever
        server          = imap.gmail.com
        port            = 993
        username        = user1
        password        = $${1026_AT_GMAIL_PASSWORD}

        [destination]
        type            = Maildir
        path            = /var/mail/1026/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
        message_log_syslog = true
      GETMAIL_RCEMAIL1_AT_GMAIL_1033: |
        [retriever]
        type            = SimpleIMAPSSLRetriever
        server          = imap.gmail.com
        port            = 993
        username        = user2
        password        = $${1033_AT_GMAIL_PASSWORD}

        [destination]
        type            = Maildir
        path            = /var/mail/1033/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
        message_log_syslog = true

The healthcheck script only ensures that cron is running. supervisord shields the process for signal handling and provides basic recovery in the even of a cron failure; however, it's cron that does the heavy-lifting to invoke getmail on interval. e.g. of what pgrep should see:
$ ps -ef
UID          PID    PPID  C STIME TTY          TIME CMD
root           1       0  0 Dec09 pts/0    00:01:33 /usr/bin/python3 /usr/bin/supervisord --configuration=/etc/supervisor/supervisord.conf
root          16       1  0 Dec09 pts/0    00:00:00 /usr/sbin/cron -f -L 15
root          17       1  0 Dec09 pts/0    00:00:19 /usr/bin/tail --follow /var/log/cron.log
root      158928       0  1 12:40 pts/1    00:00:00 bash
root      158936  158928  0 12:40 pts/1    00:00:00 ps -ef

The image has an explicit volume defined as part of the Dockerfile. Depending on the method used to instantiate the image, including docker-compose, it should be able to correlate the between the declared volume and the location being provided; are you getting any errors?
The name of the secret specified in both of the password sections should be the aligned to the following pattern:  GETMAIL_RCfoobar -> foobar_PASSWORD. Try exec'ing into the container once running and invoke getmail manually; as a sanity-check that the configuration is well-formed after templating.

Okay So the Health check is fixed, Portainer shows a healty status. Thank you for that!
But I can't seems to get it working with secrets.
If I run docker-compose with secrets it is not working, my password are not imported into the rcfile. So no mail is fetched.
But is I manually put in the passwords into the env: it is working!
Any idea why secrets isn't working for me?
This is my docker-compose file with secrets "NOT WORKING!"
version: '3.6'

services:
  getmail:
    container_name: getmail
    image: crashvb/getmail:latest
    restart: "always"
    network_mode: "bridge"
    hostname: getmail
    mem_limit: 25m
    mem_reservation: 25m
    security_opt:
      - no-new-privileges:true
    volumes:
      - /volume1/docker/getmailrc/config:/etc/getmail:rw
      - /volume1/docker/getmailrc/secrets:/run/secrets:ro
      - /volume1/docker/getmailrc/mail:/var/mail:rw
      - /etc/localtime:/etc/localtime:ro
      - /etc/TZ:/etc/timezone:ro
    environment:
      TZ=Europe/Amsterdam:
      GETMAIL_SCHEDULE=0 * * * *: #every hour
      GETMAIL_RCEMAIL_AT_GMAIL_1026: |
        [retriever]
        type            = SimpleIMAPSSLRetriever
        server          = imap.gmail.com
        port            = 993
        username        = user1
        password        = $${GETMAIL_RCEMAIL_AT_GMAIL_1026_PASSWORD}

        [destination]
        type            = Maildir
        path            = /var/mail/1026/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
        message_log_syslog = true
      GETMAIL_RCEMAIL_AT_GMAIL_1033: |
        [retriever]
        type            = SimpleIMAPSSLRetriever
        server          = imap.gmail.com
        port            = 993
        username        = user2
        password        = $${GETMAIL_RCEMAIL_AT_GMAIL_1033_PASSWORD}

        [destination]
        type            = Maildir
        path            = /var/mail/1033/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
        message_log_syslog = true

The log file:
2021-12-14_09:35:56 | entrypoint | Invoking: 00ca-certificates ...
2021-12-14_09:35:56 | entrypoint | Configuring entrypoint for first run ...
2021-12-14_09:35:56 | entrypoint | Importing SSL certificates ...
2021-12-14_09:35:56 | entrypoint | Updating CA certificates ...
Updating certificates in /etc/ssl/certs...
0 added, 0 removed; done.

Running hooks in /etc/ca-certificates/update.d...

done.

2021-12-14_09:35:58 | entrypoint | Invoking: getmail ...
2021-12-14_09:35:58 | entrypoint | Configuring entrypoint for first run ...
2021-12-14_09:35:58 | entrypoint | Generating: getmail ...
2021-12-14_09:35:58 | entrypoint | 	GETMAIL_SCHEDULE=0 * * * *
2021-12-14_09:35:58 | entrypoint | 	CRON_TRUNCATION_SCHEDULE=0 0 * * 0
2021-12-14_09:35:58 | entrypoint | Adapting configuration files ...

2021-12-14_09:35:58 | entrypoint | 	rc-email_at_gmail_1026
2021-12-14_09:35:58 | entrypoint | 	rc-email_at_gmail_1027
2021-12-14_09:35:58 | entrypoint | 	rc-email_at_gmail_1028
2021-12-14_09:35:58 | entrypoint | 	rc-email_at_gmail_1029
2021-12-14_09:35:58 | entrypoint | 	rc-email_at_gmail_1033
2021-12-14_09:35:58 | entrypoint | Applying ownership and permissions ...

2021-12-14_09:35:58 | entrypoint | Launching: /usr/bin/supervisord --configuration=/etc/supervisor/supervisord.conf ...

When checking secrets via terminal I get this:
Run a command in a running container
root@nas:~# docker exec getmail ls /run/secrets
GETMAIL_RCEMAIL_AT_GMAIL_1026_PASSWORD
GETMAIL_RCEMAIL_AT_GMAIL_1033_PASSWORD

WITHOUT SECRETS IN DOCKER-COMPOSE
This is the docker-compose file without secrets, "WORKING"!
version: '3.6'

services:
  getmail:
    container_name: getmail
    image: crashvb/getmail:latest
    restart: "always"
    network_mode: "bridge"
    hostname: getmail
    mem_limit: 25m
    mem_reservation: 25m
    security_opt:
      - no-new-privileges:true
    volumes:
      - /volume1/docker/getmailrc/config:/etc/getmail:rw
      - /volume1/docker/getmailrc/secrets:/run/secrets:ro
      - /volume1/docker/getmailrc/mail:/var/mail:rw
      - /etc/localtime:/etc/localtime:ro
      - /etc/TZ:/etc/timezone:ro
    environment:
      TZ=Europe/Amsterdam:
      GETMAIL_SCHEDULE=0 * * * *: #every hour
      GETMAIL_RCEMAIL_AT_GMAIL_1026: |
        [retriever]
        type            = SimpleIMAPSSLRetriever
        server          = imap.gmail.com
        port            = 993
        username        = user1
        password        = 123456789

        [destination]
        type            = Maildir
        path            = /var/mail/1026/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
        message_log_syslog = true
      GETMAIL_RCEMAIL_AT_GMAIL_1033: |
        [retriever]
        type            = SimpleIMAPSSLRetriever
        server          = imap.gmail.com
        port            = 993
        username        = user2
        password        = 123456788

        [destination]
        type            = Maildir
        path            = /var/mail/1033/

        [options]
        verbose         = 1
        delete          = true
        read_all        = false
        message_log_syslog = true

Short answer:
Try changing this:
      GETMAIL_RCEMAIL_AT_GMAIL_1026: |
        [retriever]
        ...
        password        = $${GETMAIL_RCEMAIL_AT_GMAIL_1026_PASSWORD}

... to this:
      GETMAIL_RCEMAIL_AT_GMAIL_1026: |
        [retriever]
        ...
        password        = $${EMAIL_AT_GMAIL_1026_PASSWORD}

Long Answer:
Template generation of the of the rc files is done by:

Looping over all environment variables prefixed with GETMAIL_RC
Then extracting the "name" of the both the rc file to be created, and the secret to be loaded from the pattern (e.g. GETMAIL_RCFOO will set name=FOO)
The secret is then imported into an environment variable of pattern <name>_PASSWORD (e.g. FOO_PASSWORD) from a file in secrets called <name>_password (e.g. foo_password)
One sourced into the environment, the variable is available for expansion using envsubst.

I'm sorry but it is still not working even after changing as you suggested.
This is the result in environment after the container is build by docker-compose:
as you can see no password is imported, in the generated rcfile there is also no password.
[retriever] type = SimpleIMAPSSLRetriever server = imap.gmail.com port = 993 username =<EMAIL_ADDRESS>password = ${EMAIL_AT_GMAIL_1026_PASSWORD} [destination] type = Maildir path = /var/mail/1026/ [options] verbose = 1 delete = true read_all = false message_log_syslog = true

I have also tried without the {...} but didn't make a difference.
So I guess maybe it just isn't possible with docker on a Synology?

It looks like there's a literal ${...} included in the generated rc file; this should never happen as environment variable that do not exist will be substituted with empty strings (e.g. it should be ... password = , with no value). It might be that the double dollar sign escaped in yaml is not being applied correctly?!?
Here is a control test using only docker-cli (as part of the makefile):
$ cat makefile.config.local
#!/usr/bin/make -f

runargs=--env=GETMAIL_RCGETMAIL_TEST="THIS IS A TEST >>>\$${GETMAIL_TEST_PASSWORD}<<<."  --env=GETMAIL_SCHEDULE="* * * * *" --volume=/var/lib/docker/secrets:/run/secrets:ro \
        crashvb/getmail:SNAPSHOT \

49c5563a1b46af6f03d5b5110fc5869a1aba1e208d2136d74439a10aba135dc7
docker ps  --all=true --filter=name=getmail-SNAPSHOT
CONTAINER ID   IMAGE                      COMMAND                  CREATED        STATUS                                     PORTS     NAMES
49c5563a1b46   crashvb/getmail:SNAPSHOT   "/sbin/entrypoint /u…"   1 second ago   Up Less than a second (health: starting)             getmail-SNAPSHOT
docker logs --follow=true  getmail-SNAPSHOT
2021-12-15_17:03:58 | entrypoint | Invoking: 00ca-certificates ...
2021-12-15_17:03:58 | entrypoint | Configuring entrypoint for first run ...
2021-12-15_17:03:58 | entrypoint | Importing SSL certificates ...
2021-12-15_17:03:58 | entrypoint | Updating CA certificates ...
Updating certificates in /etc/ssl/certs...
0 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
done.
2021-12-15_17:03:58 | entrypoint | Invoking: getmail ...
2021-12-15_17:03:58 | entrypoint | Configuring entrypoint for first run ...
2021-12-15_17:03:58 | entrypoint | Generating: getmail ...
2021-12-15_17:03:58 | entrypoint |      GETMAIL_SCHEDULE=* * * * *
2021-12-15_17:03:58 | entrypoint |      CRON_TRUNCATION_SCHEDULE=0 0 * * 0
2021-12-15_17:03:58 | entrypoint | Adapting configuration files ...
2021-12-15_17:03:58 | entrypoint | Importing GETMAIL_TEST_PASSWORD from secrets ...
2021-12-15_17:03:58 | entrypoint |      rc-getmail_test
2021-12-15_17:03:58 | entrypoint | Applying ownership and permissions ...
2021-12-15_17:03:58 | entrypoint | Launching: /usr/bin/supervisord --configuration=/etc/supervisor/supervisord.conf ...
Configuration error: configuration file /etc/getmail/rc-getmail_test incorrect (File contains no section headers.
file: /etc/getmail/rc-getmail_test, line: 1
'THIS IS A TEST >>>JunkPassword123<<<.\n')

$ make shell
docker exec --interactive=true --tty=true getmail-SNAPSHOT /bin/login -f root -p
17:04:01 root@getmail-SNAPSHOT [~]
$ cat /etc/getmail/rc-getmail_test
THIS IS A TEST >>>JunkPassword123<<<.
17:04:10 root@getmail-SNAPSHOT [~]
$ cat /run/secrets/getmail_test_password
JunkPassword123

I remove one $ sign even that is not working.
This is the result:

[retriever] type = SimpleIMAPSSLRetriever server = imap.gmail.com port = 993 username =<EMAIL_ADDRESS>password = [destination] type = Maildir path = /var/mail/1026/ [options] verbose = 1 delete = true read_all = false message_log_syslog = true

do you think it could be related to the use of Portainer? I’m use portainer stack to create the container.

Removing the one dollar sign looks like it's working better because now the password is blank; implying that the template is correct, but the environment variable referenced is missing (looking at the earlier logs this appears to be the case).
From the output above you will want to see lines similar to this:
2021-12-15_17:03:58 | entrypoint | Importing GETMAIL_TEST_PASSWORD from secrets ...

From what I can tell above, you should have a file within the container at /run/secrets/email_at_gmail_1026_password (lowercase) containing the corresponding password for the template. Please confirm that portainer is making that available.
In a pinch you can just mount the file there from outside the container as a bind mount, or mount a normal volume over /run/secrets containing the file (assuming nothing else is already doing that). Long term using a secrets provider would be recommended.

No worries; I'm glad you got it working :)


Function LevelUp is not a GameINstance Member

Running Game.LevelUp() returns
sol: runtime error: [string "Game.LevelUp()"]:1: Function LevelUp is not a GameInstance member and is not a global.
stack traceback:
[C]: in function '__index'
[string "Game.LevelUp()"]:1: in main chunk

Developer functions were removed by CDPR this is one of them.


Assert that mark_dependence operands are distinct.

Various downstream passes assume that the dependent value is not the same as the base value.

@swift-ci test


Add sqrt for proper noise sigma calculation

Just a small fix to add back a missing square root to match the original paper (and the documentation).

Looks good to me.


Mark wildcard imports as unused when all references are resolved

Summary
If a file contains, e.g., from foo import *, but all references are resolved, we should mark the import as unused (similar to Flake8).
This change is gated behind preview for now.
Closes https://github.com/astral-sh/ruff/issues/9028.

It's the latter -- just that there are no unknown references in the file. So it's relatively conservative.

It would definitely get the case you posted right, yes. Given our semantic model (where we basically assume all branches are traversed), I think it's more likely to have false negatives than false positives.

By right you mean it leaves the import because it can't be proven to be unused? I read through the code and my understanding is that it would remove the import .Would you mind adding it as a test case (and other test cases where it isn't safe to remove the imports)

Hmm, the boke warning migth be incorrect
https://github.com/bokeh/bokeh/blob/929ee8ab38207eca08d1601a0164952fb21c9c6a/src/bokeh/models/__init__.py#L66
I also don't understand why it only flags the first star import here
https://github.com/DisnakeDev/disnake/blob/594c12ad5230522a12008445217e71726cc60611/disnake/ext/commands/__init__.py#L12

Maybe you were thinking of something like this?
from bar import x, y
from foo import *

print(x, y)

Where foo contains a definition of x? In this case, we would incorrectly remove from foo import *, which matches Flake8. I still think it's fine though should likely be an unsafe fix.

Maybe you were thinking of something like this?
from bar import x, y
from foo import *

print(x, y)

Where foo contains a definition of x? In this case, we would incorrectly remove from foo import *, which matches Flake8. I still think it's fine though should likely be an unsafe fix.

Lol yes, that's what I meant. I had to rewrite the import statements a couple of time until I got the python syntax right. I must then have switched the names

I don't think we can mark them as unused when they might be used to override previous imports, and think we shouldn't because when i use star imports i use them as reexports (mainly in __init__.py[i]). Looking at the ecosystem checks all of the cases already have an noqa: F403 annotation, being effectively reexports we wouldn't want to additionally lint with F401 too.

For what it's worth, this PR omits wildcard imports in init.py files. But I'm not convinced this is a good change and there are clear concerns.