1.7.9 release

It appears that the build on travis is failing (https://travis-ci.org/angular/angular.js), so the 1.7.9 security release that was cut 3 days ago (https://github.com/angular/angular.js/releases/tag/v1.7.9) hasn't been deployed, so isn't available on any CDNs yet.
The build failures seem to be a timeout issue (or lack of timeout I suppose):

ngRoute promises should time out if the promise takes long enough
[chrome 48 OS X 10.11 #01]   Message:
[chrome 48 OS X 10.11 #01]     Failed: waitForAngular() should have timed out, but didn't
[chrome 48 OS X 10.11 #01]   Stack:
[chrome 48 OS X 10.11 #01]     Error: Failed: waitForAngular() should have timed out, but didn't

Do these jobs just need to be re-tried?

The primary release is on npm. The release is not gated on the Travis build. We need a Googler to push the release to the Google CDN.

@petebacondarwin thanks for the reply, know how long that typically takes?

Can be up to a week depending upon the availability of an appropriate
Googler to run the scripts.
On Sat, 23 Nov 2019 at 00:53, Jay Klehr<EMAIL_ADDRESS>wrote:

@petebacondarwin https://github.com/petebacondarwin thanks for the
reply, know how long that typically takes?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular.js/issues/16919?email_source=notifications&email_token=AAAD2J6WJOIFXVY6EY2WV5TQVB5JHA5CNFSM4JQTVXGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE7IVUI#issuecomment-557746897,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAD2J66ZZDVIJYVYEAYDYLQVB5JHANCNFSM4JQTVXGA
.

Of course, you can still get it from another CDN, e.g.:

<EMAIL_ADDRESS>https://www.jsdelivr.com/package/npm/angular

The new version is already available from Google's CDN (see https://ajax.googleapis.com/ajax/libs/angularjs/1.7.9/angular.min.js).


Navbar is hidden in pages even if  hideNavBar is false

I am developing for Android.
When I am using the following router code
<Router>
    <Route name="launch" component={HomePage} initial={true} title="Launch"/>
    <Route name="settings" component={SettingsPage} title="Settings" />
</Router>

When I launch the app and is in the HomePage, everything is well and fine.

But when I navigate to the settings page the navbar disappears even though it is there in the code of the component.

Here is the layout for the settings page
<DrawerLayoutAndroid
          drawerWidth={300}
          drawerPosition={DrawerLayoutAndroid.positions.Left}
          renderNavigationView={() => navigationView}
          ref={'DRAWER'}>
            <View style={{flex: 1}}>
                <ToolbarAndroid
                  title="Test"
                  actions={[{title: 'Settings'}]}
                  onActionSelected={this.onActionSelected}
                  style={{backgroundColor: '#FF0000',height: 56}}
                  navIcon={require('../../assets/images/hamburger.png')}
                  titleColor='#FFF'
                  onIconClicked={this.onIconClicked}>
                </ToolbarAndroid>

                <ScrollView
                    style={styles.settingsScroller}>
                    <Text>Settings</Text>
                </ScrollView>
            </View>
        </DrawerLayoutAndroid>

Am I doing it wrong or is it a bug ?

+1 I have same issue too

Current version was not tested on android devices at all. I'm preparing now 1.0 release which will work on both devices, check #44

Please check now


How to read nested json files using wr.sw.read_json()

P.S. Don't attach files. Please, prefer add code snippets directly in the message body.
Hi, So im trying to read a nested json file and converting it to parquet. with this code.
df = wr.s3.read_json(path=PATH,path_suffix='.json', orient="record", lines=True)
wr.s3.to_parquet(df=df, path=PATH2, dataset=True, mode="append")
However, when I run it it shows this error.
errorMessage": "Expected object or value",
"errorType": "ValueError",
I tried with another json file that is not nested and it works, so Im kind of stuck here.

Do you mind sharing the JSON or at least something resembling the similar structure you are trying to read?
Also please note:
orient=records means you expect a JSON array like [{column -> value}, ... , {column -> value}]
lines=True reads the file as a json object per line
So according to these flags your file must contain something like (note the nested JSON as well):
[{"id": {"0": 4, "1": 5, "2": 6}, "value": {"0": "foo", "1": "boo", "2": {"0": "buzz"}}}]
[{"id": {"0": 4, "1": 5, "2": 6}, "value": {"0": "foo", "1": "boo", "2": {"0": "buzz"}}}]
[{"id": {"0": 4, "1": 5, "2": 6}, "value": {"0": "foo", "1": "boo", "2": {"0": "buzz"}}}]

Which is read fine by:
df1 = wr.s3.read_json(path="s3://<dummy>/df.json", orient="record", lines=True)

Ok, let me try it. But I think I found the problem. My json doesnt have the square brackets between the registries.
{"id": {"0": 4, "1": 5, "2": 6}, "value": {"0": "foo", "1": "boo", "2": {"0": "buzz"}}}, {"id": {"0": 4, "1": 5, "2": 6}, "value": {"0": "foo", "1": "boo", "2": {"0": "buzz"}}}, {"id": {"0": 4, "1": 5, "2": 6}, "value": {"0": "foo", "1": "boo", "2": {"0": "buzz"}}}
This is sort of what I have

Yes, missing square brackets is definitely going to cause problems. Adding them works:
[{"id":{"0":4,"1":5,"2":6},"value":{"0":"foo","1":"boo","2":{"0":"buzz"}}},{"id":{"0":4,"1":5,"2":6},"value":{"0":"foo","1":"boo","2":{"0":"buzz"}}},{"id":{"0":4,"1":5,"2":6},"value":{"0":"foo","1":"boo","2":{"0":"buzz"}}}]

In[4]: df = wr.s3.read_json(path="s3://antonkuk/json/df.json", orient="record", lines=True)
In[5]: df
                                                   0  ...                                                  2
0  {'id': {'0': 4, '1': 5, '2': 6}, 'value': {'0'...  ...  {'id': {'0': 4, '1': 5, '2': 6}, 'value': {'0'...
[1 rows x 3 columns]


Feature/help widget

@trixeenya Please sync this with master -- the diff contains code from the previous PR (#2)


Obx doesn't trigger createState from it's stateful child widget when redraw

Describe the bug
Whenever an Obx have a child Stateful widget, and the Obx redraw, it doesn't recreate the child widget, it's only gonna rebuild it's state.
This means that there is a difference between the value in the state of the widget, and the value of the widget itself.
**Reproduction code
class ObxErrorDemoWidget extends StatelessWidget {
  const ObxErrorDemoWidget({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    RxInt baseValue = 0.obs;
    TextEditingController textController = TextEditingController();

    return Scaffold(
      body: SafeArea(
        child: Center(
          child: Column(
            children: [
              Obx(() {
                return CounterDemoStfullWidget(valueCounter: baseValue.value);
              }),
              Padding(
                padding: const EdgeInsets.all(16.0),
                child: Container(
                  height: 30,
                  child: TextField(
                    controller: textController,
                    keyboardType: TextInputType.number,
                  ),
                ),
              ),
              ElevatedButton(
                onPressed: () {
                  int newValue = int.parse(textController.text);
                  baseValue.value = newValue;
                },
                child: Text('Update Obx with new value'),
              )
            ],
          ),
        ),
      ),
    );
  }
}

class CounterDemoStfullWidget extends StatefulWidget {
  final int valueCounter;

  const CounterDemoStfullWidget({Key? key, required this.valueCounter}) : super(key: key);

  @override
  State<CounterDemoStfullWidget> createState() => _CounterDemoStfullWidgetState(valueCounter: valueCounter);
}

class _CounterDemoStfullWidgetState extends State<CounterDemoStfullWidget> {
  int valueCounter;

  _CounterDemoStfullWidgetState({required this.valueCounter});

  @override
  Widget build(BuildContext context) {
    return Container(
      color: Colors.yellow.withOpacity(0.5),
      child: Padding(
        padding: const EdgeInsets.all(16.0),
        child: Column(
          children: [
            Text(
                "Counter STATE Value is $valueCounter"), // <== whenever the widget is redraw with Obx, it won't recreate the state, it's just gonna rebuild the state
            Text("Counter WIDGET Value is ${widget.valueCounter}"), // <== Whenever the widget Is redraw the Obx, it won't recreate the state, the valueCounter from the widget itself is the new one, but won't propagate to the state since it's not recreating the state

            ElevatedButton(
              onPressed: () {
                setState(() {
                  valueCounter++;
                });
              },
              child: Text('Increment Counter'),
            )
          ],
        ),
      ),
    );
  }
}

To Reproduce
Steps to reproduce the behavior:

Click on "Increment counter". The value in the state of the CounterDemoStfullWidget is updated correctly.
Fill the textfield with any number different than the state
Click on "Update Obx with new value"
The Obx is gonna redraw the CounterDemoStfullWidget, the widgetValue is correct (your new number) but the CounterDemoStfullWidget won't be recreated, only redraw with an outdated state.

Screenshots

Flutter Version:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.4, on macOS 12.5.1 21G83 darwin-x64, locale en-FR)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.73.1)

Getx Version:
I'm using version 4.6.3
Minimal reproduce code
Same code than above

Sorry if I missed something, but it looks like your complaint is about the Obs not completely recreating the widget.
In Flutter a widget is only "reset" when its key changes, or when an ancestor's key changes.
The state update is a different process from that, it calls the build method again, so that the new rendering displays the value corresponding to a variable.
That said, it seems to me that this is exactly what is happening.
If you want a widget to be reset at each state update, you need to assign a Unikey to it, in your example:
CounterDemoStfullWidget(key: Unikey(), valueCounter: baseValue.value)


how to make FixedChannelPool Multiplex

I hava a problem using FixedChannelPool, I read the code, and I know if where is 4 channels in channelpool, named A B C D, that I acquire one, if it is A, then B C D channels in deque.
What I want is that I get A and the other can also get A before I call release. I can find out which thread use this channel to send tcp msg by a seq num (I think is more efficient to reuse channel and create less fd), so how can I do? Please help me.

@wsrspirit you will need to implement your own ChannelPool, sorry.

I think maybe change SimpleChannelPool so it can easy to extend to do so, give me some advice?

@wsrspirit sorry but I am currently too busy to look into it myself. If you will find out that we need some changed feel free to submit a PR tho


[CS2103T-W09-3] LinkedOn

LinkedOn is a desktop app for managing teams optimized for use via a Command line interface (CLI). As an employee, you will be able to easily view which members are in the teams you are on. As an employer, you can edit the team information and team members in the team. With our app, team management would be easier than ever.

Codecov Report

Merging #66 (2ed37d9) into master (77a32bf) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2ed37d9 differs from pull request most recent head 7a8b6c2. Consider uploading reports for the commit 7a8b6c2 to get more accurate results

@@            Coverage Diff            @@
##             master      #66   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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


Add unit tests for clustering

I will mark this as draft until I push some unit tests (already have some created locally
Currently I added the jest setup and types for clusters (took me a while to figure them out)

Not the most ground breaking tests but it is a start
also adds proper types for clusters (which is more progress towards strict types)
I will be on vacation from tomorrow, but somewhere next week I might continue work on stricter types

Sorry for delay. I got stuck in work.)
About PR - good start for tests. This should be a nice stability improvement for our development.)
And I have good news - soon I may be spending some of my working time on this project, and it should be on a permanent basis.)


Show all dashboards across multi tenancy

Is it possible to list in the dashboard tab all dashboards the user can access or a minima tenant + global dashboards ?
Actually, i need to swicth to the global, custom or private to select dashboard.
Should be great if we can access dashboard from an single location without having to switch tenant.
Dashboard could be grouped by tenant if desired.

Great idea. Closing and linking to https://github.com/opensearch-project/security/issues/1869 as that RFC addresses the re-vamp of sharing with Dashboards.


Solved issue (#5)

First raw layout
Some change to previous client
First upload prototype

Thanks for working on this functionality! I've taken a first stab at reading the diff and asked some initial clarifying questions!

Also, it seems this PR has a broader scope than #5, can you update the title to reflect that?

Also, it seems this PR has a broader scope than #5, can you update the title to reflect that?

Done!


Please provide CookCLI for Linux arm64

... as I would like to use it on this platform.
Thanks a lot.

Can I read your "thumbs up" as a consent? If so, do you have plans when to implement it?

Yep, I added an experimental build to the latest release https://github.com/cooklang/CookCLI/releases/tag/v0.1.3, could you please check if it's working? If yes, I update the release flow.

Ok, lets check:

[ ] cook recipe read
[ ] cook recipe read --output-format
[ ] cook recipe read --only-ingredients
[ ] cook shopping-list: Speicherzugriffsfehler (segmentation fault?)
[ ] cook server: Speicherzugriffsfehler (segmentation fault?)
[ ] cook seed
[ ] cook version

Thanks for checking @Lonerider2010. It might be related to glibc bug mentioned in https://github.com/cooklang/CookCLI/issues/39 and https://github.com/cooklang/CookCLI/issues/29. I don't have a solution yet.

@Lonerider2010 I think I found a workaround and this issue is fixed in v0.1.4.

@dubadub, thank you for the new version. As you can see in my checklist above, the shopping-list is ok now and ready for the next shopping ;-)
With the server, things are not as well. When I start cook server <dir-path>, I can start the web page on my browser.
There I can see the home page, but neither the sub-directories of <dir-path>, nor the .cook files within this structure.
Even more, if I try to click around, cook crashes again with the Speicherzugriffsfehler (segmentation fault?).
My impression is that cook is able to start the server and show the static parts of the web page, but is not able to fill in the dynamic parts like the sub-directories or a list of the .cook files.

I don't have disposable Raspberry Pi at hand and use a Docker image with Ubuntu 20.04.4 aarch64 (kernel 5.10.47-linuxkit). I'm not able to reproduce the issue you have. What I did:
cd /tmp
mkdir server-test
cd server-test
cook seed
cook server

After that I can open http://<IP_ADDRESS>:9080 and navigate without any issues. Could you please try this as well? So we can eliminate any potential issues with recipe files?

@dubadub, now I followed your instructions, but with the same result as with my environment. So obviously it does not depend on my recipe files. Just to mention: these files are ok for

the other functions of cookCLI (see my checklist above)
CookVSCode
Obsidian Cooklang Editor
cooklang-parser

My question: is there a kind of debug mode available for CookCLI? Can I produce a debug log that would help you to find the problem and hopefully solve it?

With migration CLI to Rust and now we have an ARM version as well. One can download binary from the releases.


Add Terratest for compute-group

I added terratest to automatically test this module. It creates a compute group based on load balancer and check whether it is provisioned successfully. Also, golang is introduced, kitchen terraform is removed and docker image is updated to the newest version.
In addition, HTTP request is used to execute end to end test. Terratest makes an HTTP request to the public IP address and see whether it actually contains nginx and is available from public.

Why it's deleted here?

Refers to: test/integration/fixtures/outputs.tf:4 in 83e38d4. [](commit_id = 83e38d46a13e9efde131fb6c5a033a56184d3d14, deletion_comment = True)

LGTM for merge, but we need to create issue to do the refactoring.


some questions

Your code is pretty good!
But some troubles I have when I run the code:

I read the paper where used 12 features to analyze the dataset and why you only group by user_id and skill_id?
I want to carry out a CART (classify tree) base on the data.How to achieve？
how to use the model I saved where the file suffixed with .hdf5 to test data?

Hi streamsteel,

I read the paper where used 12 features to analyze the dataset and why you only group by user_id and skill_id?

I've grouped by user_id to form sequences of answers. The skill_id is encoded together with the answer because we want to find the probability of a student answering correctly a question B given its previous answers to other questions.

I want to carry out a CART (classify tree) base on the data.How to achieve？

I cannot help on this. I believe it's better to start a fresh project if you are not using Deep Learning and wont use keras to build your model.

how to use the model I saved where the file suffixed with .hdf5 to test data?

There is a jupyter notebook in this repository that you can follow to achieve this. Just omit the Part 4 as you are not going to train your model. The rest of the code should remain the same. Remember to put the path to your .hdf5 file in the "best_model_file" in Part 1.

Thank you very much for your answer.
My current project needs to be implemented BKT(Bayesian knowledge tracking).Would you mind if I use part of your code to complete the paper?

Not at all. Feel free to use it in your project, just not forget to give it the proper credits.


Asking training code.

Hi Pengpeng Liu, I'm interested in your paper and code.
Now, I'm studying optical flow and your performance is very well.
So, I want to run training code and when are you possible to upload the training code?
Thanks.

See readme.


Add support for stm32f401.

Add support for the stm32f401 MCU.
Tested with my BSP repo for the nucleo-f401re board.
https://github.com/jkristell/nucleo-f401re

@jkristell Recapitulating my earlier look reminded me that something is missing: I would expect the STM32F401 clock limits and setup also to be part of the first commit. Can you please make sure that the frequency limit of 84MHz is obeyed and that everything (well, serial for now) works nicely up to that speed?

@therealprof I will do that. As everything just worked when I tested with my board running from the different clocks, I didn't even look at the setup code. But I will take a look and add code and documentation for that. Thanks!

Uhm, why did you close this and delete the branch?

I will send you a new pull request soon. Messed this up a bit. Still not used to github :).


add ci rules

[fatal] run test [type_time] err: sql:SELECT TIME('1000009:10:10.1999999999999');: failed to run query

LGTM

LGTM


emacs-24.3 + flycheck problem

While assisting @awodey, I had this problem.
Aquamacs-24.3 has a problem in installing flycheck. It still gives "Package let-alist-1.0.1' is unavailable" error. I was puzzled because I added [GNU ELPA](http://elpa.gnu.org/) in the package-archives` list. For now, I resolved his issue by upgrading Aquamacs to 24.4. However, it is a problem.

This is similar to the problem I had yesterday on my Windows machine. I got the same error, and including http://elpa.gnu.org did not help.
For me, the solution was: delete .emacs.d directory, and install Emacs 24.3.

I see. I'll dig it deeper.

I found some lessons:

Due to the recent changes in flycheck, it explicitly requires that "All packages depending on Flycheck must be recompiled!". However, it seems that package.el does not do this for users. As @leodemoura mentioned, we need to delete .emacs.d/elpa and re-install the packages manually. This is very painful, of course.
Moreover, there is a problem in GNU ELPA repository and Emacs. The newly introduced flycheck dependency let-alist is located in GNU ELPA repository. However, due to the mentioned bug, emacs cannot find this package and complains about it. That's why we had the problem.

I don't think this is a problem limited to emacs-24.3. I did have the same problem in emacs-24.4. Actually, the homebrew version of emacs (24.4.1) does have the same problem (as of Feb 2015). So I had to install emacs from http://emacsformacosx.com on my machine.

Should we close this issue?
The command (setq package-check-signature nil), that you suggested in a different thread, solved the problem. I used in two different machines. Here the recipe that should be used

Execute (setq package-check-signature nil). I executed it on the scratch buffer.
M-x package-list-packages
Select the latest flycheck and press x

I agree.


Controller namespaces in the names of UI sections?

Suppose I have these controllers
MyApp.Controllers.Web.ThingiesController
MyApp.Controllers.Web.OtherController
MyApp.Controllers.Admin.SomethingController
MyApp.Controllers.AccountController
...

I would like to see the "Web" and "Admin" in swagger UI, currently it doesn't appear there at all, instead I see at the top level:
Thingies
Other
Something
Account

Which becomes very complicated when I have more controllers in few namespaces.
What I would like to see is
Admin.Something
Web.Thingies
Web.Other
Account

Thanks.

Well, you can use ApiExplorerSettingsAttribute class to change the name of the controller displayed on the SwaggerUI.
[ApiExplorerSettings(GroupName = @"Admin/Something")]
public class Something : Controller
{
}

But there is a little catch: The sorting happens on the beginning letter and overriding the sorting order in SwaggerGen won't fix it as the UI sorts it client sided as far as I get it.
So using the above you'd get
Account
Admin.Something
Web.Thingies
Web.Other
The only way to solve that would be to prefix every controller (i.e. putting Account in either Web or Admin "group").

@TsengSR I think best would be able to implement this: IApiDescriptionProvider, and re-use as much code from this: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs as possible.
I think I need a way to override this https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs#L75
Then it would work automatically.

There's easier ways. The config option - GoupActionsBy (available through the AddSwaggerGen call in Startup.cd) should be exactly what you need.

Closing due to inactivity. I believe GroupActionsBy is exactly what you need

@domaindrivendev what is GroupActionsBy called now?


Access a cell formular string in Sylvan Data Excel

Hi Mark,
I am looking for a way to read the cell formula as plain string. It doesn't seem to be supported out of the box but I am wondering if you know a way to access to formula as string representation?

Love the loading times from your library,
renderhjs

No, unfortunately this is not supported. I try to maintain parity between the supported formats, and while it is easy enough to get the formula text for an .xlsx file, it is quite involved to get it for .xls and .xlsb files. If this is a feature you need, I'd suggest you use a more feature-rich library.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter fronius-wattpilot for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.fronius-wattpilot
   -

ERRORS:

[x] :heavy_exclamation_mark: [E033] @iobroker/adapter-core 2.6.8 specified. 3.1.4 is required as minimum, 3.1.6 is recommended. Please update dependencies at package.json
[x] :heavy_exclamation_mark: [E036] @iobroker/testing 3.0.2 specified. 4.1.3 is required as minimum,  4.1.3 is recommended. Please update devDependencies at package.json
[x] :heavy_exclamation_mark: [E039] "keywords" within package.json must contain some keywords besides "template, home automation" related to adapter
[x] :heavy_exclamation_mark: [E046] devDependency must not require a specific version. Use "~1.2.3" or "^1.2.3" syntax. Please update "@alcalzone/release-script-plugin-iobroker:3.0.0"
[x] :heavy_exclamation_mark: [E162] js-controller 3 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.
[x] :heavy_exclamation_mark: [E519] "@alcalzone/release-script" requires plugin "@alcalzone/release-script-plugin-license". Please add.

WARNINGS:

[x] :eyes: [W028] Minimum node.js version 18 recommended. Please adapt "{'engines' : { 'node' >= '16.0.0' } }" at package.json.
[x] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[x] :eyes: [W046] devDependency should not require a specific version. Use "~1.2.3" or "^1.2.3" syntax. Please update "mocha:10.1.0"
[ ] :eyes: [W115] "common.tier" is required in io-package.json. Please check https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/objectsschema.md#adapter.
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W432] Dependency 'admin':'>4.0.9' should specify '>='
[ ] :eyes: [W434] Dependency 'admin':'>4.0.9' should specify '>='
[ ] :eyes: [W519] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".
[ ] :eyes: [W522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

Thanks for processing and closing this issue.
I will recheck for new issues later. So it's no problem if this issue has been closed premature.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.  Let's work together for the best user experience.
your
ioBroker Check and Service Bot


Add Alistair to Design System team

I believe the list should be alphabetical (see line 1). Also, @aliuk2012 noticed I'm not in the list either (since #83) – would you mind adding me back?


Add support for headers in the settings

I need to supply a header called PRIVATE-TOKEN with my download request from some API. The curl command would look like the following:
curl --header "PRIVATE-TOKEN: kejf9343fdf33424" "https://api.example.com/download-me.zip"
Support for headers would be greatly appreciated.

Resolved in #5.


[410] doc changes change password prompt terminator to colon

doc changes to manual.md, manual.txt, docs.md, rclone.1
indications are i should change only manual.md and docs.md but since the *.txt and *.1 files are committed to github i offer them here.

I've merged that thanks.
I dropped the changes to MANUAL.txt/md and rclone.1 as these will get auto updated at the next release when I build the docs.
Thanks
Nick


Change default flatpak branch name

By default, the flatpak branch name is "master", it would probably be a good thing to default to something else like "main" for v4

Not until flatpak-builder does that change


Create Vagrant env for this tool

For simplifying our life, it will be great if we have vagrant

so the result is, I am using centos/7 in vagrant and could install autoxtrabackup while provisioning:
[vagrant@localhost ~]$ autoxtrabackup --version
Developed by Shahriyar Rzayev from Azerbaijan MUG(http://mysql.az)
Link : https://github.com/ShahriyarR/MySQL-AutoXtraBackup
Email<EMAIL_ADDRESS>Based on Percona XtraBackup: https://github.com/percona/percona-xtrabackup/
MySQL-AutoXtraBackup Version: 1.4.8

But I need to test it further, to publish.

This thing is implemented and will be available on 1.4.9 release.


Process killed while trying to access Zarr dataset on S3

NetCDF version: v4.9.0
HDF5 version: 1.10.8
OS: Linux
I am trying to access the zarr datasets here (https://power-analysis-ready-datastore.s3.us-west-2.amazonaws.com/index.html) and here (https://hrrrzarr.s3.amazonaws.com/index.html) using netcdf.open. I am seeing two issues here:

If I only append '#mode=zarr' or '#mode=nczarr,zarr' without specifying the driver to be used, the process stalls for a bit and then gets killed. This happens even if I build netCDF without NCZarr support.

I also see a process abortion if I try and open the dataset here ("https://hrrrzarr.s3.us-west-1.amazonaws.com/prs/20180712/20180712_12z_anl.zarr#mode=zarr,s3")
The error message is as follows:
"netcdf/libnczarr/zclose.c:228: zclose_type: Assertion `type && type->format_type_info != NULL' failed.
Abort
"
Please find my repro code below:

#include <stdio.h>
#include <string.h>
#include <netcdf.h>

// ISSUE 1
#define FILENAME "https://power-analysis-ready-datastore.s3.us-west-2.amazonaws.com/power_901_annual_meteorology_utc.zarr/#mode=zarr"
// ISSUE 2
//#define FILENAME "https://hrrrzarr.s3.us-west-1.amazonaws.com/grid/HRRR_chunk_index.zarr#mode=zarr,s3"

int main()
{
    int ncid1;
    int status;

    // Open the file
    status = nc_open(FILENAME, NC_NOWRITE, &ncid1);
	printf("status code after open = %d\n", status);

    status = nc_close(ncid1);
    printf("status code after close = %d\n", status);

    printf("End of test.\n\n");

    return 0;
}

Pinging @DennisHeimbigner

I have a branch that attempts to infer the storage type for existing files.
I have not yet had the time to test it and publish the PR.
But in any case, a missing format should provide a better error report.
Is either my error or a malformed zarr file. I will investigate.

Ok, I have some information.
Problem #1 was fixed for 4.9.1 at some point in the sense that an error is reported instead of aborting.
Problem #2 is a bit more complex. The chunk_id array, for example, is using the dtype "|O",
where O means that the variable values are pickel'd  (in the python sense) serialized values.
We currently do not support such values and they are not part of the Zarr specification, so technically
they are illegal.
As with #1, this now reports an error rather than aborting. Perhaps better would be to mark such
variables as unreadable so that at least the readable parts of the file can be accessed.

Thank you for taking a look.
Is there any way to identify the fix for (1) so that we can patch the library on our end?
When is the v4.9.1 of netCDF-C scheduled to be released?
For (2), I tried reading the same file using Zarr-Python, and it seemed to work fine.
import s3fs
import zarr
url = "s3://hrrrzarr/prs/20180712/20180712_12z_anl.zarr"
fs = s3fs.S3FileSystem(anon=True)
store = zarr.open(s3fs.S3Map(url, s3=fs))
print(store.info_items())

"Perhaps better would be to mark such
variables as unreadable so that at least the readable parts of the file can be accessed"
Was this comment in reference to (2)?
Is this something that would be implemented for v4.9.1 of netCDF-C?

4.9.1 should be out inside a week [Ward?].
My comment should have been marked as "note to self", but it does refer to #2.
The file is readable by at least some Python zarr implementations because
the can interpret the non-standard "O" dtype. But again, this is not legal vis-a-vis the Zarr V2 specification.

I'm working on v4.9.1 as we speak; as is always the case, I wanted it out weeks ago, I would like it to be out today or tomorrow, and I'm hopeful it will be by the end of the week.  We'll see what other little things crop up in the meantime.


Swap out darts for point branding (all 3)

[x] replace darts logo with point
[x] use dark teal as primary background
[x] include pink & light teal (maybe in place of some of the orange?

@Nikomus I've left the colouring app changes for you. Don't want to tinker with it. Especially since you might not agree with the changes I made.
I'll add the cream background for help. It is one of their primary colours for both brands & used for text on dark bg


Cleaned up visuals

The css needed tweaking after translating from react native to reactjs

C'est très bien. J'ai compris ce que tu a fait.
@jclaerhout doit vérifier s'il est d'accord avec les changements.

Good first iteration of translation, but some things are missing :

[ ] The style should be placed in separate files. Create a "styles" folder with a .css file for each .js file.

[ ] The buttons in the navbar need to be spaced a little further apart.

[ ] The login page lacks a navigation link to the registration page.

I have added a margin to the nav buttons.
The inline style is necessary for some aspects of the material ui library. If we are going to have inline style anyways I don't see the issue with keeping it for the rest.
The link to the registration page is implemented both in the router and the login page. It has been commented out due to issues which stop the Registration page from rendering properly. These issues are not within the scope of this branch so I will  leave them as is.

After considering Euan's comment, everything seems correct


Plans for psr/log v.3?

Enhancement:
Hi,
Are there any plans to migrate from psr/log from v2 to v3 ?
I did a quick look into the code, and i think it is not so easy to change it.
actual:
#public function log($level, $toLog, array $context = array())
psr/log v.3
public function log($level, string|\Stringable $message, array $context = []): void
Problem 1 the return instead of void.
Problem 2 $toLog has a different type.
Maybe a solution could be to remove psr/log in the long term?
So usere are not tied to v1|v2
Greetz

This is a good question. You can find more details about the this topic here https://github.com/rollbar/rollbar-php-laravel/issues/128#issuecomment-1052377338.

Until we have a clear roadmap to v4 this is not really an option. I am going to close this issue for now, and we will revisit it once we start working on a v4.

This has been resolved and will be part of our v4.0 release.

@danielmorell Sounds good. Can you provide an UPGRADE.md in the 4.x Branch with infos what changed, and how to migrate v3 => v4?
Maybe the 5.x branch of https://github.com/rollbar/rollbar-php-symfony-bundle needs some changes to support v4.

@Chris53897 I added an issue to create upgrade documentation (see #599).

This was fixed in #577.


Migrate pipeline from VMWare to Community Concourse

Description
Once the community Concourse has been setup, let's move the necessary pipelines over. This includes retrieving the necessary credentials form VMWare.
Questions

So far there are AWS S3 blobstores used for several artifacts. Would it make sense to use instead GCS blobstores instead where possible?
Would we need a new repository for this?

Context

a rough overview of the assets has been collected in https://docs.google.com/document/d/1846mRZ1X-Z5mv0ayHykp2N7B7aT1fujEGV-ntuVxO98

ToDo

[ ] ...

PRs
Acceptance Criteria

the pipelines are capable to release candidates for:

AliCloud, AWS, Azure, GCP, OpenStack, vSphere, vCloud heavy stemcells
AWS, GCP light stemcells

remove nimbus and replace with??


libjpeg: 2.0.4 -> 2.0.5

Motivation for this change
https://github.com/libjpeg-turbo/libjpeg-turbo/releases/tag/2.0.5
Things done

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

[x] NixOS

[x] Fits CONTRIBUTING.md.

This should probably be backported after merging as well, since it fixes CVE-2020-13790.

unstable, 20.09, and 20.03 already have patches which is why I did not mention the CVE.


Support autocompletion on higher order expectations

As per our discussion the other day, here are the basic requirements for autocompletion on higher order expectations.
Currently, you do get autocompletion for the first property:
// This works with full autocomplete
expect($object)
  ->foo->toBeString();

However, once we move past the first expectation methods and on to a second (or third, fourth etc) property, we lose autocomplete.
// I don't get autocomplete for anything past the very first property
expect($object)
  ->foo->toBeString()
  ->bar()->baz->toBeInt() // No autocomplete
  ->pest->is->best; // No autocomplete

The basic rule is that anything after expect($value) should return auto completions for both the expectation api methods AND the object passed in to expect. Taking it one step further, it is possible to do nested higher order expectations, ->foo->bar->toBeString(), so in this case it should autocomplete based on the return type of foo as well as providing autocompletion for the expectation api methods in case we want to simple perform expectation methods on foo (in this case).
Here's a real life example from Larastreamers:
expect($channel)
         ->youTubeCustomUrl->toBe('christophrumpel')
         ->name->toBe('Christoph Rumpel')
         ->description->toStartWith('Hi, I\'m Christoph Rumpel')
         ->onPlatformSince->toIso8601String()->toBe('2010-01-12T19:15:29+00:00')
         ->country->toBe('AT')

I hope that makes some form of sense 😄
Kind Regards,
Luke

Edge cases:
expect($object)
         ->something->toBe('firstObject')
         ->and($anotherObject)
         ->something->toBe('otherObject')

They could also chain another expect instead of and

@adelf this is pretty similar to HigherOrderCollectionProxy, do you maybe have some great insights or ideas on how to solve this?
I did try once to solve it for HigherOrderCollectionProxy in the collector plugin, but only solved 1/3 of the issue...
https://github.com/olivernybroe/collector-intellij/blob/main/src/main/kotlin/dev/nybroe/collector/types/HigherOrderTypeProvider.kt

It probably could be covered by completion contributor, inspection suppressor and type provider.
Do you have tests for that?

@adelf Hmm interesting 🤔
Yes, I have a branch with failing tests for this feature. Just created a PR #233 with it 👍

This is now in the current EAP 💯


WIP: Added the option to output contaminant reads to file

That's a feature I needed, it would be nice to be able to output contaminant reads in a separate file.
This PR does exactly that, but there are a few points I'd like to discuss with the maintainers:

First, I made the tests pass even when mistakingly writing reads with missing line feeds. So I'd like to improve the tests to make sure I didn't break anything.
Second, if I understand correctly, the inverse flag of the CLI makes chopper output the 'failing' reads. Maybe we could replace this flag with dedicated file outputs.
And last but not least, I am a pretty bad Rust developer, so I didn't find another way of protecting concurrent writes other than that terrible Arc<Mutex<Box<dyn Write + Send>>>. There must be another simpler, more elegant technique ^^

Anyway, thanks for reading this through, have a good day :)

Hi,
Thanks for contributing! I will look at the code you wrote. Did you perhaps benchmark the time your version takes compared to the latest release?
Best,
Wouter

Did you perhaps benchmark the time your version takes compared to the latest release?

I'm sorry, I didn't. There might indeed be a performance cost, but overall it seems to be working just fine


implemented PriorityQueue

But what you think about the user setting priority as a number instead of enum? Do you think it would be any useful in most scenarios?

Yes, I think code must be obvious as possible. 1,2,3,4 - is not.


chore: remove HeaderSyncMode::Continuous & debug.continuous

Closes #8651.

Thank you, for all the help.


Add support for --unprivileged mode on Linux

It will be nice when trippy does not need to invoke sudo powers to work on Linux, in so its usability will be on par with traceroute and mtr.

That's actually already the case, the same way traceroute needs to be given permission to do what it does you can give that permission to trippy and not full admin access.
sudo setcap CAP_NET_RAW+p $(which trip)

Taken from option 3 here https://github.com/fujiapple852/trippy/#unix
So you don't need to use sudo to run it. Just to give it permission to access raw sockets.

Thanks, yes, it didn't occur to me. Then what would be left to do in the issue at hands?

@almereyda Linux provides (at least) three mechanisms for an unprivileged (non-root) user to run a ping/traceroute.
The first way is what @c-git is referring to; where the CAP_NET_RAW capability can be set on the executable which allows that executable to open raw sockets, even if the process itself is started by a non-root user.  This is the approach taken by both mtr (note that mtr invokes the mtr-packet executable to do the actual tracing and so needs that capability).  This is the preferred approach for running Trippy on Linux as @c-git points out.  Note that Trippy is what is called a "capabilities aware" program, meaning it can use the provided capabilities to do what it needs (open the raw sockets) and then drop the capabilities at runtime so after startup it runs completely unprivileged.
The second approach is similar and involves setting the setuid bit via chmod +s on the executable which makes the executable process run with root permissions even if it is started by a non-root user.  Whilst more powerful, this is also more dangerous as the process runs with more permissions that it strictly needs and retains that permissions for the entire time the program is active.
The third approach, which is what this issues addresses, involves using some platform specific Linux APIs in order to be able to send and receive probes for ping/traceroute without requiring any privileges.  This is what the Linux native traceroute program does.  Whilst this is arguably the best option, it does require lots of Linux specific code which complicates Trippy (much as the Windows platform code does, unavoidably) and so whilst I would like to add it, it needs some fairly significant changes to support cleanly.  Note that MacOS also support this approach but in a way that is almost zero impact to the existing design of the code.
I hope that clears it up!

Thank you for the thorough explanation. I do now understand better why this imposes an effort on the maintenance, because the major computation platforms all handle selectively increased privilege of process permissions differently.


Upgrade version of sitetree

from Brandon:
Both Ewa and I have encountered the problem of temporarily hiding a sitetree item through the Admin interface, only to have it disappear and become impossible to bring back.
This appears to be a bug in sitetree that was recently fixed:
"https://github.com/idlesign/django-sitetree/pull/164
Could you upgrade the version of sitetree being used with the production site to the most recent trunk?"

The issue with disappearing seems to have been resolved in idlesign's django-sitetree repo (https://github.com/idlesign/django-sitetree) by pull request 169. Since our project uses our caktus/django-sitetree repo we should update our repo accordingly.

our repo has been updated

We should probably tweak our requirements to install a specific tag/commit/branch from our repo.

Fixed in our fork of django-sitetree.


Update user-access-token-js.md

fix refresh tokens code snippet

@0rland0Wats0n : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Fix bash syntax highlighting

Fixes #273

Thank you @bourdakos1
I've reverted the version bump and changelog here, and opened #275. Let me know if you need me to change anything else!


Feature: Service or button to clear data

Configuration changes are not effective immediately if expiration time has not been passed.
This makes experimenting and initial configuration a bit difficult as configuration is refreshed only when nord pool or entso-e data is provided (or after trigger time if used).
Implement a service or button to clear cache for a single sensor or alternatively all data.

I second this! My use-case:

I come home with the car.
HA reads SOC of the EV battery, and calculates how many hours charging it would require to go to 80% or 100% again
Only then I know the start time, and the hours required.
Nicest feature would be to just monitor all the attributes, if one or more changes, recalculate. If that becomes too complicated indeed just a service or a button to recalculate would do.

Automatic recalculation from attribute changes could cause some side effects, so for the first phase it's a bit too risky. :-)
I'll try to create a service for this functionality for the 0.3.2.
About your use-case you could use trigger_time already, it's kind of implemented for similar situations already.
Anyhow this lead me thinking, maybe start_time and end_time should be able to set from an input_number (or other) entity as well. As if you want to calculate cheapest hours from e.g. from 18:00 - 09:00 and plug your car in at 20:00. The cheapest hours could have been already from 18:00 to 20:00 and that would have ended up in two missed hours already.

Hi @kotope, thanks for your quick reply. I had overlooked trigger_time, good one, thanks for pointing that out! Is this one dynamic? Can I change it from an automation or something? Because that is what I assumed would be possible with start time and end time, but what I understand now this is probably not the case (yet?)
For the coming home scenario actually first_hour, last_hour and number_of_hours should be settable as a minimum, and there should be a manual recalculate function. Or does it work when first_hours is in the past already? Then I can just set it at the earliest possible.
(if trigger_time is settable that could do the trick, but then you would loose potential valuable time waiting for up to an hour for a re-calculate?)
Anyway, thanks for your thoughts and any further help/comments are appreciated! If you want someone to test or to bounce some ideas of I am available.

Hi @vanrace, sorry I've missed your latest comment. Just noticed this as I'm implementing the service and checkout the issue id :-)
With 0.3.1 trigger_hour can be set to be dynamic; start and end times not, but maybe later..
If you have a good idea or enchancement, you could create an enhancement issue.
When passing ideas or asking questions, please use the discussion board so I can keep the issue list more organised and it would be easier to follow (at least for me) :-)

Service to clear data is now merged into dev. Will be released with 0.3.2.

Thanks!!
Verzonden vanuit Outlook voor iOShttps://aka.ms/o0ukef

Van: Toni Korhonen @.>
Verzonden: Saturday, November 9, 2024 3:30:03 PM
Aan: kotope/aio_energy_management @.>
CC: vanrace @.>; Mention @.>
Onderwerp: Re: [kotope/aio_energy_management] Feature: Service or button to clear data (Issue #31)
Hi @vanracehttps://github.com/vanrace, sorry I've missed your latest comment. Just noticed this as I'm implementing the service and checkout the issue id :-)
With 0.3.1 trigger_hour can be set to be dynamic; start and end times not, but maybe later..
If you have a good idea or enchancement, you could create an enhancement issue.
When passing ideas or asking questions, please use the discussion board so I can keep the issue list more organised and it would be easier to follow (at least for me) :-)
—
Reply to this email directly, view it on GitHubhttps://github.com/kotope/aio_energy_management/issues/31#issuecomment-2466239572, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABVEMOJPAFFMPDUQX2ZIIUDZ7YL6XAVCNFSM6AAAAABQIJPAM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWGIZTSNJXGI.
You are receiving this because you were mentioned.Message ID: @.***>

New service to clear data released with 0.3.2


Parcel Support

Describe the feature
Support for Parcel Bundler. Would be great to use a library like @linaria in our parcel Setup.
So far no ways to make @linaria work in parcel Bundler was found.
Motivation
Migrating from Styled-components to @linaria easy fairly easy since both have comparable API. But the issue for us is that @linaria doesnt have first-class support for Parcel.
Possible implementations
There is a very old linaria parcel plugin out in the wild, but its never updated in the last 5 years and doesnt seems to work with Parcel 2+
Related Issues
https://github.com/callstack/linaria/issues/519

Hi? Has any news about plans to implement?

@kaifaty Nice to have, but it's not a high priority currently.


Add --etherscan-api-key flag to forge test to make the etherscan trace feature more apparent.

Component
Forge
Describe the feature you would like
forge create allows one to pass --etherscan-api-key in to use the cmd.
I propose adding the --etherscan-api-key to forge test to keep the API consistent and make it more obvious as to how to get access to better traces with etherscan (i.e. I can just type forge test -h and figure it out vs. having to go through docs etc)
Additional context
No response

we definitely can add an arg for this but I believe it should already be picked up if the env var is set, @onbjerg ?

It does pick it up. I am only proposing this because it took me forever to figure it out (but maybe that's just me).

Closed in #1349


[EAK-422] Add a separate logger for Authoring Kit core bundle

Add a separate logger file for the logs related to the Authoring Kit core bundle. Log file name and rotation frequency are discussable.

Scheduled for release in v.2.5.1


switch to github.com/astreter/amqpwrapper

Using github.com/astreter/amqpwrapper instead of own RabbitMQ connector implementation

!deploy

!deploy

!deploy


Feature: Allow script-query variables from runtimeConfig

Currently it is not possible to define query variables at runtime - e.g. in case the same build is used in different environments (gtm_preview).
To solve this, publicRuntimeConfig.gtm.variables should override gtm.variables, if set.
P.S.: I've got it working since the "id" param is not url-escaped 🤫, but that seems kind of hacky 😉

+1

Yes lets fix this asap! i feel like this is the primary use case for the runtime config?

@JohannesLamberts how exactly did you bake it into the ID field?

This one is not solving the issue ? If you define it there directly >> https://github.com/nuxt-community/gtm-module/issues/69

@kissu that feature does not work properly

this is an issue for us too, let's fix it asap please 🙏

@sergiobellon this is a workaround:
publicRuntimeConfig: {
    gtm: {
      // hack until https://github.com/nuxt-community/gtm-module/issues/94 is fixed
      id: `GTM-xxxx&gtm_auth=${process.env.GTM_AUTH}&gtm_preview=${process.env.GTM_PREVIEW}&gtm_cookies_win=x`
    }
  }

@Martin7mind this is working with runtime variables, right ?

correct @kissu

+1 This is a must for config driven single builds. @Martin7mind solution works in the meantime.


reduce number of * in help

Too much **** in the help - I think it can be reduced and it will look neater :)

Fixed in https://github.com/mapbox/osm-tag-stats/blob/master/util/help.js.


fix(dal): pick correct ordering node in ReplaceSubgraph

If an ordered node's children are replaced, and the RemoveEdge operations come before the ReplaceSubgraph update for the ordering node, the ordering node's vector clock write will have been incremented during the update, which leads us to pick the wrong node, with the result that the ordering node now that the ordering node now has incorrect children in its order property. This fixes that by not incrementing the vector clock write for the ordering node when processing a RemoveEdge update (and only when processing a RemoveEdge update).
Discovery of this bug came from the VectorClock tuple work, but this bug exists on main.

bors try


[codelab] Telink TLSR9518 codelab first draft to publish for openthread.io

Add TLSR9518 codelab to OpenThread.io.
Please let me know if there are any issues with the content.

Also @pulin1103 can you add an entry to the TOC for this codelab?  After the Silabs one: https://github.com/openthread/ot-docs/blob/main/site/en/guides/_toc.yaml#L27

@Vyrastas Thank you for providing feedback! I have made the necessary changes to the document and images based on your suggestions. Please review it again, and if there are any issues, please let me know.

Thank you for reviewing it carefully again. I have made the modifications as per your request. If there are any issues, please let me know.

Thanks for your contribution @pulin1103 !


Update Composer dependencies (2021-07-07-00-07)

Visual regression test passed!
View the visual regression test report


docs: fix spelling and improve grammar

I read the rest of the docs and found a few ways I believe will improve the readability.

I believe that satisfies all the points in the code review. We should be good to go.

Appreciated!


Publish 8.1.1

Publish 8.1.1

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


Allow ctx.result(future)?

Akka-http has this, it's really cool and Javalin should have it.

@ShikaSD has made a very elegant and almost-working implementation in #186, but I'm starting to get a little worried. The reason the PR doesn't fully work is because the contextPath is nulled when startAsync() is called, and the ResourceHandler relies on it to not be null. Upon googling I found a thread (https://bugs.eclipse.org/bugs/show_bug.cgi?id=433321) about unexpected behavior.
What worries me the most is the following post by @gregw:

It is indeed a problem that it is unclear what methods on the request/response are safe to access asynchronously and which are unsafe.   During the formation of the specification I advocated that there should be no access to the request object for this very reason.
It get's even worse in servlet 3.1, where async IO can be used, but a simple call to request.getParameter(String) will invoke a blocking read.
This is not a bug in Jetty and certainly cannot be resolved by simply adding a volatile.   It is essentially a bug in the specification.
We could change jetty so that the last non null values of context path and servlet path are left in the request while it is not dispatched.   This would probably make your simple case work.     But that is essentially encouraging fragile code that will break if reused in non simple cases.
The solution is to use techniques that are multithreaded safe - ie not accessing fields that may change.  You can do this by using the request attributes provided as part of the spec, or by taking your own copy of the path fields, or by a design that is not dependent on path fields after initial handling.
I don't know what else I can say.   It is certain that the path fields MAY change asynchronously so that any access from an async thread is in a race condition.    You think that Jetty SHOULD NOT null the path values between dispatches, and while that is an argument that can be made, it does not change the fact that those values MAY change in other circumstances.

I really wanted to have everything async, but I'm worried how this might affect current apps. I think it's safer to call startAsync() only if a user sets a Future as the result. This might mean we will only support futures in endpoint-handlers. Not because it's impossible to make it work otherwise, but because it will become a hot mess. This is a little sad, but endpoint-handlers should cover 99% of cases (?).

Added in #186


Fixed Test Code link in README

Current link is broken; fixed to link to the correct page on the documentation.

Thanks a lot @carlosmiranda!! Much appreciated.
It'd be great if you could also sign the Gauge's Contributor's License Agreement.

@mahendrakariya Very much welcome, I've just signed the CLA as well.

Thanks a lot!


Refactor go-utils for eventing

Update CLI to use go-utils changes introduced in https://github.com/keptn/go-utils/pull/174

Codecov Report

Merging #1913 into master will increase coverage by 0.01%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master    #1913      +/-   ##
==========================================
+ Coverage   32.66%   32.68%   +0.01%
==========================================
  Files         139      139
  Lines        8432     8433       +1
  Branches      116      116
==========================================
+ Hits         2754     2756       +2
+ Misses       5329     5328       -1
  Partials      349      349

Flag
Coverage Δ

#moduleA
36.85% <ø> (ø)

Impacted Files
Coverage Δ

cli/cmd/configure_monitoring.go
61.66% <50.00%> (ø)

cli/cmd/send_event.go
54.83% <50.00%> (ø)

cli/cmd/send_event_evaluationStart.go
76.52% <50.00%> (ø)

cli/cmd/send_event_newArtifact.go
78.12% <50.00%> (ø)

cli/cmd/send_event_approvalFinished.go
40.94% <100.00%> (+0.25%)
:arrow_up:

distributor/cmd/main.go
58.88% <0.00%> (+1.11%)
:arrow_up:

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 3473595...d74a6ff. Read the comment docs.


Optimize the installation experience

Book keeping issue to optimize the installation experience,
so that new users can install ingress controller as easy as  kubectl apply -f 'pathToOurYamlOnGitHub'.

[ ] Omit Ingress controller required params
For the ingress controller to work correctly, three variable are required:

[ ] ClusterName(can be introspected from kubernetes.io/cluster/cluster-name:owned tag)
[x] VPC_ID(can be introspected from ec2Metadata)
[x] AWS REGION(can be introspected from ec2Metadata)
These variable can still be manually specified for advanced use case like ec2Metadata unavailable or running ingress controller on EKS control plane.

[ ]  Migrate the helm into helm charts repo.(omit the quay.io registry plugin)
[ ] Create an managed AWS policy to capture the needed IAM permission.

Please: omit the quay.io registry plugin. Helm is a tool that usually requires creating wrappers because it lacks some functionalities, this makes it hard to work with those wrappers. Examples of wrappers are "helmfile" and "helmsman". Those don't support something as specific as the Helm Registry Plugin.


Creation of a Mock REST API endpoint

Child of #1
In order to test the application functionality we will need a mock rest api endpoint. This solution will not have to do anything except respond with expected and/or unexpected REST error or success codes.

@cathyhashmap this item has been assigned to you. Please review and advise the ETA.

The point of this task is to create a fake endpoint for testing that exposes the same api's so that we can code against it. Not to test the rest api's themselves...the title is a bit misleading there

This is now done as part of hashmap/drilltest


how to enable trace: 'retain-on-failure' feature on java

how to enable trace: 'retain-on-failure' feature on java

if you're using junit 5 you can use TestWatcher
or if you're using testng use a testTestListener
Start the trace when you create the browsercontext like you normally would but only save it if the test fails in the appropriate test failure methods.
if you're using junit 5 you can also use junit-playwright which. has a enableTracingOnlyOnFailure() which will handle it for you.

'retain-on-failure' is a feature of playwright test runner which is only available in Node.js. Internally it runs the tests with tracing enabled but only writes the result to a file if the test fails. You can implement such logic as described in the previous comment.


Pr/v1.5 backports 2019 04 11

Backport of PR #7657 NOT done, as it broke the cherry-pick script for the later backports.
Following backports applied against v1.5:
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick ce889ce39084e8be9a199d155ce41ac3e8169003
Applying: ce889ce390 allocator: Block Allocate() and Release() until key list is initialized
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 8091d2441f38f1fe763783ea4b23ed19c45f048b
Applying: 8091d2441f allocator: Provide info and warning messages around key allocation
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 7bc3528336f4496763fffe4d23b69b50b49ec933
Applying: 7bc3528336 kvstore: Protect Unlock() from timeout overwrite
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 341274337e5dc928b6ed4326776651fa26e23fc0
Applying: 341274337e kvstore: Return from LockPath() when local locking is cancelled
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 6202595c7230b8f957ba4214a21e50d64e907e4a
Applying: 6202595c72 cilium: ipsec_linux, remote DeleteIPSecEndpint and use SPI version
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 9fea830dcc8db7d293f3cede4fae0a40ab714aaa
Applying: 9fea830dcc cilium: ipsec_linux only set spi bit in xfrm mark on egress
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 74c656bb9e53d6228abfe203cac55719102fdc28
Applying: 74c656bb9e cilium: Policy rules are no longer unique for key
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 913a2cdb4f03e16a98af3b182ffa86f7a681f27d
Applying: 913a2cdb4f cilium: ipsec tests should use decodeIPSecKey for strings to hex
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 074cb88ad91605c7b7d0c09905b4d627f0036a23
Applying: 074cb88ad9 cilium: support aead state keys
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 7972c7f00c7f379ed35cf3f84295430454d18703
Applying: 7972c7f00c cilium: docs update encryption algo example to use GCM
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 7e389f35911db69615226b9429e5502091eafffa
Applying: 7e389f3591 vendor: update k8s dependencies to 1.14.1
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick da45c47b11f97c84ed996589477d27fc48bfe19e
Applying: da45c47b11 test: update k8s test versions to v1.14.1
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 9195a107b1d3a54b4b0fd28b23f53f937f899692
Applying: 9195a107b1 policy: Fix metrics for policy revision
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick a078bb313112a801a39234f9fafa610475d77359
Applying: a078bb3131 identity: Don't serialize reference counts
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 127800d273e41d4ab52792109ba01ef794d3631b
Applying: 127800d273 Documentation: clean up upgrade instructions
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 25cb8f0d501eed1832059e1b31ebf8bdeb3bfcaf
Applying: 25cb8f0d50 Change suiteName to not match test folders names.
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 1a370790e4f9341e4fafde5140d712b1b5d90149
Applying: 1a370790e4 agent: Delete endpoints which failed to restore synchronously
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 21b156fbb55c12fbf734be3b892d27d95c85a287
Applying: 21b156fbb5 exec: return for any error from context
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 7c40dedd4624b03973b17b8bf7aa910f0b26174b
Applying: 7c40dedd46 endpoint: add Context field to regenerationContext
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 7c588a70a3eed245067bca595305edc9a2be30da
Applying: 7c588a70a3 endpoint: use parent context with prepareForProxyUpdates
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick bb1f2e1dc561b99a0b83a15be08c0fb15b8e9b10
Applying: bb1f2e1dc5 daemon: pass down context on endpoint creation into regeneration functionality
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 1f602bb2f0992560c2fb577acaa0cd6ed92ab31b
Applying: 1f602bb2f0 loader: check whether context is cancelled
vagrant@runtime1:~/go/src/github.com/cilium/cilium$ contrib/backporting/cherry-pick 7a352e08365b9c5bdcb099600d42f666541f8809
Applying: 7a352e0836 daemon: pass context down into QueueEndpointBuild

This change is

test-me-please

test-missed-k8s

test-docs-please


Get element client width in directive

I want to get element client width in directive , I find that  bind() and first time update() couldn't get element client width, Maybe the DOM still not ok，what should I do to get the client width

Hi!
Please make sure to ask the question on the forums or the gitter chat.
You can take a look at requestAnimationFrame or Vue.nextTick


User-defined launch images in project template

The user should be able to set a custom launch image in their NativeScript application.

We've decided that the way to implement this is to teach the CLI to respect asset catalogs (as described in https://github.com/NativeScript/nativescript-cli/issues/683) found in App_Resources/iOS and insert them in the project when preparing it for build. Asset catalogs can then be used to describe app icons and launch images.


add setDumpPath for Windows.

close #10
Added a setDumpPath method by method_channel for Windows.
Maybe that setDumpPath might too late for  c++ world, since there still some logic (like plugin register) execute before the dart world. (But anyway, it seems to be enough for my usuage.

Thanks for your PR. I'll test it later


Docker in subfolder

Can the app exist in a subfolder using your dockerfile?
something like www.domain.com/chat/
Clicking join takes you to www.domain.com/invite and not www.domain.com/chat/invite
Thank You,
-Jake

Any news here? This is a showstopper for using this software.


[NEW]: Binance denies owning WazirX, India’s largest crypto exchange 2 years after announcing the acquisition

Date
2022-08-05
Description
Binance CEO Changpeng Zhao said the firm did "not own any equity in Zanmai Labs, the entity operating WazirX". He added that "Binance only provides wallet services for WazirX as a tech solution," while WazirX was responsible for KYC and other operations on the exchange.
This comes 2 year after Binance announced they’d acquired WazirX in a blog post titled “ Binance Acquires India’s Leading Digital Asset Platform WazirX”.  The blog post was edited after the tweets to clarify that “no equity was purchased in the transaction” and that the deal was limited to “certain assets and intellectual property”
The clarification follows India’s Enforcement Directorate freezing WazirX’s assets worth over $8 million, citing suspected violation of foreign exchange rule.
Nischal Shetty, co-founder of WazirX, has disputed Zhao’s claims, asserting Binance indeed acquired WazirX. He confirmed that Binance owned the WazirX domain name and controlled root access to their AWS account.
Links
https://techcrunch.com/2022/08/06/binance-says-it-doesnt-own-indian-exchange-wazirx-years-after-acquisition-announcement/
https://www.cnbctv18.com/cryptocurrency/wazirx-binance-engage-in-war-of-tweets-investors-crypto-14409062.htm
https://www.binance.com/en/blog/all/binance-acquires-indias-leading-digital-asset-platform-wazirx-to-launch-multiple-fiattocrypto-gateways-404105749895733248
https://cointelegraph.com/news/indian-authorities-freeze-8-1m-in-wazirx-funds-as-part-of-aml-investigation
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
Nemo
Personal link
https://twitter.com/captn3m0

Wrote this on the phone, will put inline links in a bit.

Thanks! https://web3isgoinggreat.com/?id=no-one-wants-to-admit-to-owning-the-wazirx-crypto-exchange


Support disabling animations in the CLI

This is just the first step; once this lands we need to also look at reducing animation in subinvocations of pub and using a less verbose and more static reporter in flutter test. The latter would help with CI logs, too.


[FEATURE REQ] Persisting AuthenticationRecord automatically

Library name
Azure.Identity
Please describe the feature.
Many customers are asking for a way to cache the full User's credentials and persist them on disk to enable "auto-login" scenarios on subsequent runs of the app.
The only way to achieve this today is manually saving and retrieving the authRecord on disk:
private const string _tokenPath = "authToken.bin";

public GraphServiceClient GetGraphServiceClient(string clientId, string[] scopes)
{
    var credentialOptions = new InteractiveBrowserCredentialOptions
    {
        ClientId = clientId,
        TokenCachePersistenceOptions = new TokenCachePersistenceOptions() { UnsafeAllowUnencryptedStorage = true }
    };

    InteractiveBrowserCredential browserCredential;

    if (!File.Exists(_tokenPath))
    {
        // this code path happens on first execution
        browserCredential = new InteractiveBrowserCredential(credentialOptions);
        var tokenRequestContext = new TokenRequestContext(scopes);
        var authRecord = browserCredential.Authenticate(tokenRequestContext);
        using var authRecordStream = File.OpenWrite(_tokenPath);
        authRecord.Serialize(authRecordStream);
    }
    else
    {
        // this code path happens on following executions
        using var authRecordStream = File.OpenRead(_tokenPath);
        var authRecord = AuthenticationRecord.Deserialize(authRecordStream);
        credentialOptions.AuthenticationRecord = authRecord;
        browserCredential = new InteractiveBrowserCredential(credentialOptions);
    }

    return new GraphServiceClient(browserCredential);
}

This should be an out-of-the-box functionality, and we could simplify the above code to something similar to this (adding a new property on the TokenCachePersistenceOptions class to automatically handle the persistence on disk):
public GraphServiceClient GetGraphServiceClient(string clientId, string[] scopes)
{
    var credentialOptions = new InteractiveBrowserCredentialOptions
    {
        ClientId = clientId,
        TokenCachePersistenceOptions = new TokenCachePersistenceOptions() { StoreOnDisk = true }
    };

    var browserCredential = new InteractiveBrowserCredential(credentialOptions);
    return new GraphServiceClient(browserCredential, scopes);
}

Hi @Licantrop0
Thanks for your suggestion. However, I don't anticipate that we'd implement a feature like this considering that the basic implementation is straight forward (as in your example or in our sample), and custom serialization of credentials should be handled intentionally by the app.


Cluster test

-- draft

:umbrella: The latest upstream changes (presumably 3f1804d1944185409cf818013336baf66e1e1afd) made this pull request unmergeable. Please resolve the merge conflicts.


Check that upcoming librustzcash changes to primitives and proofs still compile with Zebra

Motivation
The zcashd developers have asked us to report any breaking API changes in this librustzcash PR:
https://github.com/zcash/librustzcash/pull/755
Testing
The compiler should identify any breaking API changes for us, and our CI should identify any bugs introduced by the refactor.

Hey team! Please add your planning poker estimate with Zenhub @arya2 @conradoplg @dconnolly @oxarbitrage @teor2345 @upbqdn

This is done in PR #5966, we just need to turn that PR into a ticket I guess.
(It could be a while before the changes get released by ECC.)

The tests passed, the new upgrade ticket is #5978.


Cover smaller types in ForeignKeyTypeChecker

bigint is now default type used for all association columns in Rails. However, table that they refer to may be created long time ago and thus having integer primary key.
Since bigint is bigger type, it covers smaller integer. This PR disables warning about such fields

@djezzzl I made it work with PostgreSQL. Seems that different DBs (and even versions, locally SQLite specs aren't passing for me) have different type names.
WDYT about supporting this feature for PostgreSQL only right now? Users of MySQL/SQLite can add PRs based on this one. If you agree, I'll squish commits


build: fix lcov file path

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@744663c). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage        ?   28.47%
=======================================
  Files           ?       10
  Lines           ?      144
  Branches        ?        0
=======================================
  Hits            ?       41
  Misses          ?      103
  Partials        ?        0

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 744663c...a7ab079. Read the comment docs.


Can't seem to get the reading IO for named channels and time-steps

Results in a cannot be found, even through the writing seems to be putting it in the correct place. Will see if this is an issue on the C++ side.

Do you have a reproducible example of this? It seems to be working for me. Although the current API is mainly developed with a single channel and time step in mind. It could be expanded to simplify handling of multi channel/time data in a single file.

This was an issue on the C++ side to do with the write tree flag as mentioned in LibAPR, def needs to be removed and automated.


Add support for image-indexes

Is your feature request related to a problem?
Currently, we cannot sign image indexes, and we use image indexes in some scenarios.
Without singing them, we can't fully run notation with ratify as we cannot validate this kind of artifact.
What solution do you propose?
To be able to sign image-indexes too, same as regular images.
What alternatives have you considered?
Excluding specific pods/namespaces that use image indexes or forcing the use of images only in our environment which is not good enough.
Any additional context?
No response

Hi @roeishuster , do you expect to sign both the image index and its child image manifest or sign the image index only? Is it a scenario of signing and verifying multi-arch image? It would be good to elaborate more on your feature request. Thanks

@FeynmanZhou What we see in our pods verification is the image index only, so we think it would be sufficient to be able to sign the index only.

Hi @roeishuster , notation should be able to sign the image index only. Have you ever signed it before verifying with Ratify? Is there any specific errors?

@FeynmanZhou it looks like you are right, we can close this :)


Refactor configuration to be able to pass a different SSH key per cloud provider

What does this PR do?
Refactor configuration to allow using a different SSH key per cloud provider.
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge

Wonder if we can recommend running inv setup from test-infra if keys are not found in the config, to update it

/merge -c

Wonder if we can recommend running inv setup from test-infra if keys are not found in the config, to update it

Indeed, I'll do a PR on test-infra to update this. I think we just miss the local key configuration, I think the others are already created when running inv setup
I will also check the behavior when no keys are configured and print a message, indicating to run inv setup if it fails

/merge


should we show legs that are less than one minute? (per transit)

See screenshot Bob.

The root cause of this (snipping to particular stoppoints) has been removed in #185 Could still occur if the fastest station lays right outside of the max_distance circle but.


feat: support authorization header in project config

Allow the options page to define an authorization header for project-specific top-level browser requests. Required for sites with authentication enabled in helix 5.
Fixes #737

@rofe Since I made a change to en/messages.json is there something I need to do to get the rest translated? What's the process there?

New or updated strings will be detected automatically after merging to main, and a new PR will be created for the translations a few days later. If you want to release them simultaneously, you can DM the localization team.

Oops, I just noticed that when opening a new tab with the same URL (e.g. by clicking a link), you get a 401 again. Likely root causes:

you don't call updateProjectAuthHeaderRules() in the background script to (re)define rules based on existing tokens (we do it for updateAdminAuthHeaderRules())
updateSessionRules may not be the right choice for such long-lived tokens. Maybe updateDynamicRules is the better choice here.

@rofe Thanks for catching that. Once I added it to the background script things worked as expected. After looking into updateDynamicRules I agree that seems like maybe the better options since the the persistence across tabs and windows is greater. I think since we are executing updateProjectAuthHeaderRules anyway in the background script it's not an issue but just in case there is some edge case I went with updateDynamicRules.

:tada: This PR is included in version 6.45.0 :tada:
The release is available on:

GitHub release
v6.45.0

Your semantic-release bot :package::rocket:


Can not connect to remote cassandra host.

My cassandra database is hosted on another machine but on the same local network. Is there any reason you can't access it this way?

I am also facing same issue. My cassandra is hosted locally.

Hello! Which version of plugin you are using?

@futuarmo Its hadesarchitect-cassandra-datasource version: 1.0.0

@smtkhaitan our 1.x versions supports only Grafana versions < 8. To use plugin with Grafana 8 and higher please use v2.0. It doesn't published in Grafana Plugins List yet, but now you can download it here and install manually: here is the link:
https://github.com/HadesArchitect/GrafanaCassandraDatasource/releases/tag/2.0.0
Uncompress a file into the Grafana plugins directory (grafana/plugins).
Please, give some feedback when you'll try it

@joaocps please, tell which version of plugin and Grafana you are using

@futuarmo I'm using the version 8.4.4. I installed the plugin directly from the grafana plugins list, so that must be the problem. I will install manually and then give feedback right back to you.
Regards!

I downloaded the last release from , unzip and move the folder to /var/lib/grafana/plugins. Restart the grafana.server service but find out another error:

So far I can't reproduce it.
The path you are having looks wrong to me. You have /var/lib/grafana/plugins/cassandra-plugin/hadesarchitect-cassandra-datasource/cassandra-plugin_linux_amd64, the cassandra-plugin part seems weird.  Could you please put the folder hadesarchitect-cassandra-datasource directly in the /var/lib/grafana/plugins ?

Hmmm it's weird. Could you please execute ls -la /var/lib/grafana/plugins/hadesarchitect-cassandra-datasource command and paste the results here? Also, if that's possible, I'd schedule a zoom call to debug it further. I can't reproduce it so we may need to debug it together.

@HadesArchitect
Let me see the best time to schedule this, this week will not be easy for me. Anyway, here's the requested result:

May relate to our issue: https://forum.golangbridge.org/t/fork-exec-no-such-file-or-directory-ld-preload/23824

@HadesArchitect when do you plan to release v2.0.0 also in the Grafana Plugins Catalog?
Currently is 1.1.4 and it does not support Grafana v8.
Thanks :)

@bygui86 It's already released but currently not published, due to the Grafana's review process. I suggest using 2.0.0(requires manual installation)

@andrew-landsverk-win @joaocps Could you guys please try to replace the original file with a this one?
cassandra-plugin_linux_amd64.zip

Mark for myself: https://stackoverflow.com/a/58185179/2214896

Any progress on this? I'm running into the same issue, Grafana 8.3.3, Plugin version 2.0.0, CentOS 7.

Same Error on our side :
msg="Could not start plugin" pluginId=hadesarchitect-cassandra-datasource err="fork/exec /var/lib/grafana/plugins/hadesarchitect-cassandra-datasource/cassandra-plugin_linux_amd64: no such file or directory"
Ubuntu 18.04 => Grafana 8.5.0

Dear  @HadesArchitect ,
You solution with the new files work.
I have to unsigned the plugin in order to test.
Thank you

Dear @HadesArchitect ,
You solution with the new files work. I have to unsigned the plugin in order to test.
Thank you

Out of curiosity, how do you unsign the plugin first?

Dear @HadesArchitect ,
You solution with the new files work. I have to unsigned the plugin in order to test.
Thank you

Out of curiosity, how do you unsign the plugin first?
Stop grafana server

export GF_PLUGINS_ALLOW_LOADING_UNSIGNED_PLUGINS=hadesarchitect-cassandra-datasource
allow_loading_unsigned_plugins=hadesarchitect-cassandra-datasource
in /etc/grafana/grafana.ini
Remove MANIFEST.txt from folder plugin
mv /var/lib/grafana/plugins/hadesarchitect-cassandra-datasource/MANIFEST.txt .
And relaunch grafana server
Regards

You solution with the new files work. I have to unsigned the plugin in order to test.

You are awesome, thank you for your contribution!
I'll release the updated version as 2.0.1 in a couple of hours.

@PaulB2Code It's not working for me, I've added allow_loading_unsigned_plugins=hadesarchitect-cassandra-datasource manually in the grafana ini, but I get a modified signature error, If i remove the manifest.txt from var/lib/grafana/plugins/hadesarchitect-cassandra-datasource/ I get a plugin not found error. Any help? Am I missing something obvious? P.S. I'm trying with the version 2.1


ERA001: autofix removes valid comments that do not contain code or removes not everything

Example:
# Example of headers:
# X-token: <token_id>
# X-scope: no-scope
# End of comment about headers

# {
#    "some-key": {
#         '1': 1,
#         '2': 2',
#     }
# }

# some = 1
# awesome = 2
#
# foobar = 3
# barfoo = 4

# example code from IPython:
# In [20]: parse.parse_qsl('&l=55.123;123.1&count=3', keep_blank_values=1)
# Out[20]: [('l', '55.123'), ('123.1', ''), ('count', '3')]
# End of comment

# (пример, пример2) = (example1, example2)

# if not success:  # TODO: comment
#    errors.append('Ошибка: error')

# comment
# 1. [{пример первый}]
# 2. [[пример]]
# 3. [] - пусто

# Пример: {"description": "weight: "}

# comment about calculation
# 100 / 2 = 50
# end of comment

# comment:
# - tuple: ()
# - list: []
# - set: set()
# - dict: {}

# example of output:
# 'OK: message sent. Message-ID: 123456'

# MySQL: enum('positive', 'negative')

# query: lock + user.id

Ruff output:
$ ruff --fix --select ERA001 ruff_errors/ERA001.py
Found 24 errors (24 fixed, 0 remaining).

Contents of the file after fix:
# Example of headers:
# X-token: <token_id>
# End of comment about headers

#    "some-key": {

#

# example code from IPython:
# End of comment

# if not success:  # TODO: comment

# comment
# 1. [{пример первый}]
# 3. [] - пусто

# comment about calculation
# end of comment

# comment:

# example of output:

I don't think we're in a position to improve this much, the logic itself is derived from eradicate, and I'd bet eradicate has the same or very similar behavior. Perhaps we should remove the autofix from this rule though. It seems correct to flag that there's some commented-out code here, but rely on users to remove it.
Alternatively, we could only autofix if all lines in a comment block are identified as code.

We could make this as a Suggested (Applicability::Suggested) fix.
    /// The fix may be what the user intended, but it is uncertain.
    /// The fix should result in valid code if it is applied.
    /// The fix can be applied with user opt-in.

I like

Alternatively, we could only autofix if all lines in a comment block are identified as code.

Perhaps we could use Suggested in that case and Manual otherwise?

Another example: it thinks the comment is code even though params isn't a valid keyword
# params: abc
query = """
SELECT :abc;
"""

Another example: it thinks the comment is code even though params isn't a valid keyword
# params: abc
query = """
SELECT :abc;
"""

I believe this is considered as a valid code (variable: annotation):
params: int

I am absolutely in favor of not automatically removing these comments in any situation. Yes, commented-out code is usually a mistake, but you really don't want to loose actual comments accidentally.
Our example:
# foo and bar (JIRA-9999)

@bodograumann what is the benefit of enabling the ERA001 rule for you then?

Was about to open a new issue, but I think this fits here:
While testing out ruff, I ran into some false positives for ERA001 that confused me a bit.
Some examples:
# Keep this list sorted alphabetically
# (it is used in example)

# Check if the ip address in either:
# local (loopback) or internal (private)

##
# Client (consumers)
##

...

##
# Server (protocol)
##

All trigger ERA001 even though they are not really code.
I had also hoped that code fragments that are part of a larger comment wouldn't trigger ERA001, yet all of these do:
# XXX: Cannot use:
# self.example(foo, 'bar')
# because of issue #1337

# XXX: Example category has a duplicate 'code' and is omitted
# EXAMPLE_CATEGORY = '1.0', 'Example Category'

# Note: Admin site instance should be passed when calling
# ExampleAdminView.as_view(site=site)

# main categories themselves are not part of the return value of
# get_subcategories()

This is all using ruff 0.0.286.
I've already marked ERA001 as unfixable in my configuration file. I'm hoping it is possible to tweak the heuristics of ERA001 a bit so it takes the surrounding context into account or something?
For now I'll just be working around this by rewording/reformatting these comments until they don't trigger anymore.

Yeah the ERA001 heuristics are very prone to false positives, unfortunately. I think there's room for improvement, but it's not entirely clear how.
def foo():
    x = 1
    # add to x
    # x += 1
   return x

I think this should raise as an ERA001 violation but if we ignore comments that are part of a larger non-code context then we won't raise here. Maybe that's worth the false negative to reduce the false positives.

This seems like a bug:
A single parenthesis triggers ERA001:
echo "# )" | ruff --select ERA001,RUF100 --stdin-filename test.py
test.py:1:1: ERA001 [*] Found commented-out code

Adding a noqa to that line triggers RUF100:
echo "# )  # noqa: ERA001" | ruff --select ERA001,RUF100 --stdin-filename test.py
test.py:1:6: RUF100 [*] Unused `noqa` directive (unused: `ERA001`)

This seems like a bug:
A single parenthesis triggers ERA001:
echo "# )" | ruff --select ERA001,RUF100 --stdin-filename test.py
test.py:1:1: ERA001 [*] Found commented-out code

Adding a noqa to that line triggers RUF100:
echo "# )  # noqa: ERA001" | ruff --select ERA001,RUF100 --stdin-filename test.py
test.py:1:6: RUF100 [*] Unused `noqa` directive (unused: `ERA001`)

Yeah this is a bug. Would you mind creating a new issue for it as it's separate from the rest (solution: We should use Indexer.comment_ranges to find comments and not rely on a regex)

@jaap3 - Funnily enough that specific case was reported recently and is fixed on main.

Is there scope for a PR for better algorithms, and a better set of false positive rules here?
It will deviate from the eradicate plugin, would that be okay or should it be a different rule entirely then?

Self-documenting here for later.
Examples of false negatives:
# if foo:

# elif bar:

# if (
#  x
# ):

# else:

Examples of false positives:
# continue
# pragma: cover
# (something)
# O(1)
# this is O(1)

#  Checking if the first default_path is there in api_paths if true then, return the action i.e create, read, update, delete

this is the comment I wrote, it contains return in this sentence only because of that, it is rising a warning

That should be fixable, let me do just that much for now.


Felix fails to program rules for ICMP port

Looks like a mismatch between plugin and felix: the plugin is specifying both icmp_type and ports.  Also looks like felix needs to handle multiple ICMP ports and port ranges, which we may not at present (just spotted that multiport doesn't support icmp, d'oh).
Apr 18 19:58:33.553531 <IP_ADDRESS> err calico-felix: Failed to convert rules to fragments: [{'ip_version': 4, 'src_tag': u'ed1c1daf-8cb8-44c8-a2cc-6ca518f31301', 'icmp_type': 255, 'protocol': u'icmp', 'dst_ports': [255]}].  Will DROP!
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/calico/felix/frules.py", line 153, in rules_to_chain_rewrite_lines
  File "/usr/lib64/python2.7/site-packages/calico/felix/frules.py", line 209, in rule_to_iptables_fragments
  File "/usr/lib64/python2.7/site-packages/calico/felix/frules.py", line 295, in _rule_to_iptables_fragment
AssertionError: Protocol icmp not supported with dst_ports ({'protocol': u'icmp', 'dst_ports': ['255'], 'src_tag': u'ed1c1daf-8cb8-44c8-a2cc-6ca518f31301', 'ip_version': 4, 'icmp_type': 255, 'src_ports': []})

Looks like there's also a missing field in the data model.  OpenStack lets you match on ICMP code as well but we don't have a field for that.  I'll write up a suggestion in the datamodel design doc.


make error originating from ARA SDK on MacOS (x86)

The following error occurs when attempting to run make on MacOS (x86):
In file included from /Users/frank/Desktop/HARP/ARA_SDK/ARA_Library/Dispatch/ARAPlugInDispatch.h:22:
/Users/frank/Desktop/HARP/ARA_SDK/ARA_Library/Dispatch/ARADispatchBase.h:251:54: error: a non-type template parameter cannot have type 'auto'
        return this->_ptr->structSize >= SizedStruct<member>::getImplementedSize ();

The error appears to originate from this conditional branch in the ARA SDK. Running clang -dM -E -x c /dev/null from the command line, the following values are displayed for the macros:
#define __clang__ 1
#define __clang_major__ 10

Interestingly, the code compiles and make runs successfully when the second branch and conditional statements are commented out, i.e.:
//#if defined (__clang__) && (__clang_major__ < 10)
// see clang bug: https://bugs.llvm.org/show_bug.cgi?id=35655
// as workaround, we're copying the code from SizedStruct<member>::getImplementedSize () here
static_assert (std::is_class<StructType>::value && std::is_standard_layout<StructType>::value, "C compatible standard layout struct required");
return this->_ptr->structSize >= reinterpret_cast<intptr_t> (&(static_cast<StructType*> (nullptr)->*member)) + sizeof (static_cast<StructType*> (nullptr)->*member);
//#else
//    return this->_ptr->structSize >= SizedStruct<member>::getImplementedSize ();
//#endif

The resulting build seems to work fine.

The ARA_SDK has been updated recently, and this no longer seems to be occurring. We can close for now...


"Record" button to export a clip of the current output

It'd be cool to be able to hit record and export a GIF or something of the current rendering stack.

@ervanalb did most of this last night, you can write out a stream of images using a checkbox on the GUI.

This is hella slow on my machine though, probably we want to find a better way to do this

it also causes weird graphical glitches

Once outputs_are_nodes_too is merged in, I'll work on FFMPEG integration. Shouldn't be too hard. Hopefully the huge amount of refactoring fixed those weird graphical glitches but if not we'll look into it.

Throwing this to @zbanks since he took care of the dedicated FFmpeg output node. Shouldn't be hard to tack something similar onto ScreenOutputNode.